Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

workaround sporadically missing apt-src on Debian #387

Merged
merged 1 commit into from
Mar 6, 2017

Conversation

dirk-thomas
Copy link
Member

Workaround for #369.

@dirk-thomas dirk-thomas self-assigned this Feb 27, 2017
@dirk-thomas dirk-thomas force-pushed the workaround_apt-src_stretch branch from 9ef52f3 to 2d14665 Compare February 27, 2017 20:37
@dirk-thomas dirk-thomas changed the title workaround sporadically missing apt-src on Debian Stretch workaround sporadically missing apt-src on Debian Feb 27, 2017
Copy link
Member

@tfoote tfoote left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A bit of a hack, but there doesn't seem to be a cleaner solution. I'm surprised that this is continuously coming up and that the debian mirrors haven't synchronized.

The overhead of this is minimal so +1

@dirk-thomas
Copy link
Member Author

Currently ros-infrastructure/buildfarm_deployment#145 is blocking this to be tested.

@mikaelarguedas
Copy link
Contributor

@dirk-thomas this can be tested now that ros-infrastructure/buildfarm_deployment#145 is resolved

@dirk-thomas
Copy link
Member Author

All recently built Lunar Debian Stretch jobs (for amd64 as well as armv8) finished without any of the builds failing with this problem. Therefore I am leaning to not merge this PR and close it (until we see the problem again).

@mikaelarguedas
Copy link
Contributor

Sounds good to me, almost done rebuilding all the Stretch packages and all is green so far

@dirk-thomas dirk-thomas closed this Mar 1, 2017
@dirk-thomas dirk-thomas deleted the workaround_apt-src_stretch branch March 1, 2017 23:11
@dirk-thomas dirk-thomas restored the workaround_apt-src_stretch branch March 5, 2017 06:31
@dirk-thomas
Copy link
Member Author

This has failed again for Debian Jessie armv8 (http://build.ros.org/view/Kbin_djv8_dJv8/job/Kbin_djv8_dJv8__depth_image_proc__debian_jessie_arm64__binary/35/console). Since it hasn't happened for Stretch in the past rebuild but does happen for Jessie maybe this should be updated to only apply the workaround on Jessie?

@dirk-thomas dirk-thomas reopened this Mar 5, 2017
@dirk-thomas dirk-thomas force-pushed the workaround_apt-src_stretch branch from 2d14665 to b114d69 Compare March 5, 2017 19:32
@dirk-thomas dirk-thomas force-pushed the workaround_apt-src_stretch branch from b114d69 to f96614f Compare March 5, 2017 19:41
@tfoote
Copy link
Member

tfoote commented Mar 6, 2017

I see you reopened this. For the record I'm seeing about 10-20 instances of this per day over the weekend.

@dirk-thomas
Copy link
Member Author

Indeed, I took care of manually retriggering all of them for the sync.

Did you see them only on Jessie (that's what the patch targets now) or also any on Stretch or other platforms?

@dirk-thomas dirk-thomas merged commit 4cb54b9 into master Mar 6, 2017
@dirk-thomas dirk-thomas deleted the workaround_apt-src_stretch branch March 6, 2017 18:11
@tfoote
Copy link
Member

tfoote commented Mar 6, 2017

Searching the notifications I only see them on Jessie armv8 since Feb 16th.
Before that I see both Jessie amd64 and armv8.

These errors appear to have started Jan 24th armv8.
I'm guessing the Jessie amd64 mirror got fixed. And now the armv8 ports are slightly behind.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants