Makefile race condition - repos.mk

Bug #1684426 reported by Maxim V. Yefimov
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mirantis OpenStack
Fix Released
Medium
Aleksey Zvyagintsev

Bug Description

The error occurs occasionally:

Step to reproduce:

Start the normal custom ISO build from jenkins with the job provided by Mirantis. Makeflags = -j --output-sync

Expected results: Normal build with a new ISO

Actual results:

Error:
09:53:19 ln: failed to create symbolic link '/home/jenkins/fem002-eiffel018/workspace/R6.9.0-qa-build-common@3/R6.9.0-qa-build-common/tmp/build/repos/fuel-main': No such file or directory
09:53:19 /mnt/jenkins/fem002-eiffel018/workspace/R6.9.0-qa-build-common@3/ecs-mos/fuel/main/repos.mk:47: recipe for target '/home/jenkins/fem002-eiffel018/workspace/R6.9.0-qa-build-common@3/R6.9.0-qa-build-common/tmp/build/repos/fuel-main.done' failed
09:53:19
make[1]: *** [/home/jenkins/fem002-eiffel018/workspace/R6.9.0-qa-build-common@3/R6.9.0-qa-build-common/tmp/build/repos/fuel-main.done] Error 1
09:53:19 make[1]: *** Waiting for unfinished jobs....

Changed in mos:
milestone: none → 9.x-updates
Changed in mos:
assignee: nobody → Anton Chevychalov (achevychalov)
Changed in mos:
milestone: 9.x-updates → 9.2-mu-2
Changed in mos:
status: New → In Progress
importance: Undecided → Medium
Revision history for this message
Anton Chevychalov (achevychalov) wrote :

Unable to reproduce on our side. Perhaps there is interference with side patches. But fix is obvious.

https://review.openstack.org/#/c/459686/ has been merged.

Changed in mos:
status: In Progress → Fix Committed
Revision history for this message
Alexander Rubtsov (arubtsov) wrote :

Anton,

The customer, who hit the issue, believes that the committed fix will make the build system more robust, but will not eliminate the race condition itself. He has proposed another patch: http://paste.openstack.org/show/ZWBOg9Ai0DAWiE1QT18E/

Would you please review the proposed fix above?

Changed in mos:
status: Fix Committed → Confirmed
Revision history for this message
Alexander Rubtsov (arubtsov) wrote :

Here is the relevant commit:
https://review.openstack.org/#/c/462517/

Changed in mos:
assignee: Anton Chevychalov (achevychalov) → Denis Meltsaykin (dmeltsaykin)
Revision history for this message
Denis Meltsaykin (dmeltsaykin) wrote :

I cannot do anything with the commit to the master branch, please cherry-pick the commit to stable/mitaka when it is merged in master.

Changed in mos:
assignee: Denis Meltsaykin (dmeltsaykin) → Aleksey Zvyagintsev (azvyagintsev)
Changed in mos:
status: Confirmed → Fix Committed
Revision history for this message
Alexander Rubtsov (arubtsov) wrote :

The link to the relevant commit for Mitaka series:
https://review.openstack.org/#/c/463955/

Revision history for this message
TatyanaGladysheva (tgladysheva) wrote :

Move to Fix Released on 9.2 mu2 since we are unable to check this fix.

Changed in mos:
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.