Skip to content

Update jessie-backports URL#18530

Merged
murgatroid99 merged 5 commits intogrpc:masterfrom
murgatroid99:jessie-backports_url_fix
Mar 29, 2019
Merged

Update jessie-backports URL#18530
murgatroid99 merged 5 commits intogrpc:masterfrom
murgatroid99:jessie-backports_url_fix

Conversation

@murgatroid99
Copy link
Copy Markdown
Member

Update jessie-backports URL as described in this post.

@murgatroid99 murgatroid99 added the release notes: no Indicates if PR should not be in release notes label Mar 26, 2019
@murgatroid99 murgatroid99 requested a review from apolcyn as a code owner March 26, 2019 22:46
@nicolasnoble
Copy link
Copy Markdown
Contributor

Hopefully this will fix the issues we've seen in #18477. CC @jtattermusch and @jiangtaoli2016.

Copy link
Copy Markdown
Contributor

@jtattermusch jtattermusch left a comment

Choose a reason for hiding this comment

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

LGTM, but updated docker images need to be pushed for tests to pass.

@murgatroid99
Copy link
Copy Markdown
Member Author

I had to change a lot more files to get all of the updated docker files to actually build, so it would be good to check them again.

@jtattermusch
Copy link
Copy Markdown
Contributor

LGTM again. I checked the test failures and they seem to be mostly:

#17794

@murgatroid99 murgatroid99 merged commit e6cd312 into grpc:master Mar 29, 2019
ejona86 added a commit to ejona86/grpc that referenced this pull request Apr 5, 2019
Jessie has a line that looks like this, now-a-days, so the regex is no
longer matching:
deb http://security.debian.org/debian-security jessie/updates main

But because that line was changed, downloads are also working correctly
out-of-the-box. The sed was originally added in grpc#18530.
@srini100
Copy link
Copy Markdown
Contributor

Does this need to be backported to v1.19.x to fix #18646?

@lock lock bot locked as resolved and limited conversation to collaborators Jul 9, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

priority/P0/RELEASE BLOCKER release notes: no Indicates if PR should not be in release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants