Openvswitch 2.3.2 RPM failed to compile in PRSTRK.

Bug #1504094 reported by Artem Silenkov
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Released
Critical
Artem Silenkov

Bug Description

Packaging is failed because of failed %check section.

https://review.fuel-infra.org/#/c/12328/

DEBUG: Check PROC
DEBUG: + cd openvswitch-2.3.2
DEBUG: + echo 'Check PROC'
DEBUG: + '[' -d /proc/sys/net/core/ ']'
DEBUG: + ls /proc/sys/net/core/
DEBUG: somaxconn
DEBUG: xfrm_acq_expires
DEBUG: xfrm_aevent_etime
DEBUG: xfrm_aevent_rseqth
DEBUG: xfrm_larval_drop
DEBUG: + make check TESTSUITEFLAGS=-j8

/proc/sys/net lacks important keys inside

--net=host to docker works locally for me = package is compiling normally.

Tags: area-build
Changed in fuel:
importance: Undecided → Critical
assignee: nobody → Roman Vyalov (r0mikiam)
milestone: none → 8.0
Roman Vyalov (r0mikiam)
Changed in fuel:
assignee: Roman Vyalov (r0mikiam) → Fuel build team (fuel-build)
status: New → Triaged
assignee: Fuel build team (fuel-build) → Dmitry Burmistrov (dburmistrov)
Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Fix proposed to infra/obs (feature/new-build-system)

Fix proposed to branch: feature/new-build-system
Change author: Dmitry Burmistrov <email address hidden>
Review: https://review.fuel-infra.org/12791

Changed in fuel:
status: Triaged → In Progress
Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Fix merged to infra/obs (feature/new-build-system)

Reviewed: https://review.fuel-infra.org/12791
Submitter: Igor Shishkin <email address hidden>
Branch: feature/new-build-system

Commit: df60c909f71d0a5671c2b421bb0928de53e05ddb
Author: Dmitry Burmistrov <email address hidden>
Date: Tue Oct 13 14:10:46 2015

[8.0][builder][rpm] Host network at docker containers

  Use host network at rpm docker containers

  Closes-Bug: #1504094

Change-Id: Iaa82b28b541c45b962c4baf87cb9f57acaa6af3c

Changed in fuel:
status: In Progress → Fix Committed
Changed in fuel:
status: Fix Committed → In Progress
Revision history for this message
Dmitry Burmistrov (dburmistrov) wrote :

We can't use `--net=host` at our docker containers for some reasons.

Openvswitch specfile violates rpm packaging guidelines:
>Packages must not depend or use any network resources that they don't themselves create (ie, for tests)

So fix openvswitch unittests or remove %check section

Changed in fuel:
assignee: Dmitry Burmistrov (dburmistrov) → Artem Silenkov (asilenkov)
status: In Progress → Triaged
Revision history for this message
Artem Silenkov (asilenkov) wrote :

We could create nothing because container is strictly limited and has no network stuff inside at all.
/proc/sys/net is empty. So we can't "fix tests". They are fine.

There is no reason to drop tests just because we can't turn on container capabilities for some reason.
Please give us an ability to deal with network. It is working locally and in OBS before.

Please describe reasons so we could find appropriate solution.

Revision history for this message
Artem Silenkov (asilenkov) wrote :

I can't neither turn %check off nor merge with minus on my own.

Please propose reasonable solution.

We are terribly blocked with this issue - neutron can't be merged until this issue is resolved somehow.

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Related fix merged to packages/centos7/openvswitch (master)

Reviewed: https://review.fuel-infra.org/12328
Submitter: Artem Silenkov <email address hidden>
Branch: master

Commit: c7d98111260a10a0329ee31f4347d1043d4f1bdd
Author: Aleksandr Mogylchenko <email address hidden>
Date: Wed Oct 14 12:18:26 2015

Add OVS 2.3.2 to MOS 8.0 scope

Add OpenVSwtich 2.3.2 (latest in LTS series) to MOS

Partial-Bug: #1500374
Related-Bug: #1504094
Change-Id: I13786f39cf1fc242e42d8c289ee09557069d57a3

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Related fix proposed to packages/centos7/openvswitch (master)

Related fix proposed to branch: master
Change author: Artem Silenkov <email address hidden>
Review: https://review.fuel-infra.org/12828

Changed in fuel:
status: Triaged → In Progress
Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Related fix merged to packages/centos7/openvswitch (master)

Reviewed: https://review.fuel-infra.org/12828
Submitter: Artem Silenkov <email address hidden>
Branch: master

Commit: 1bfa1f14855ac0adb64485ea562ba799df733c5e
Author: Artem Silenkov <email address hidden>
Date: Wed Oct 14 13:38:58 2015

Add OVS 2.3.2 to MOS 8.0 scope

Add OpenVSwtich 2.3.2 (latest in LTS series) to MOS

Change-Id: I6726bfd71fb6b58f052d150cf96ac3428e6e4baf
Partial-Bug: #1500374
Related-Bug: #1504094

Dmitry Pyzhov (dpyzhov)
tags: added: area-build
Changed in fuel:
status: In Progress → Fix Committed
Changed in fuel:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.