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

build: use 20.04 based image #21388

Merged
merged 8 commits into from
Aug 2, 2022
Merged

Conversation

lizan
Copy link
Member

@lizan lizan commented May 20, 2022

Fixes #21171

Additional Description:
Risk Level: Med
Testing: CI
Docs Changes: Done
Release Notes: Done
Platform Specific Features:

@repokitteh-read-only repokitteh-read-only bot added the deps Approval required for changes to Envoy's external dependencies label May 20, 2022
@repokitteh-read-only
Copy link

CC @envoyproxy/dependency-shepherds: Your approval is needed for changes made to (bazel/.*repos.*\.bzl)|(bazel/dependency_imports\.bzl)|(api/bazel/.*\.bzl)|(.*/requirements\.txt)|(.*\.patch).
envoyproxy/dependency-shepherds assignee is @mattklein123

🐱

Caused by: #21388 was opened by lizan.

see: more, trace.

keith
keith previously approved these changes May 20, 2022
Copy link
Contributor

@moderation moderation left a comment

Choose a reason for hiding this comment

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

Fix deps commit date

version = "9cdf86f3fe383c7f894d1814da3f0bc5bec8c8c4",
sha256 = "88e76fdd6b2ee52e66b6b5933eb25e4263285c8b6133b330699c8796c28da606",
version = "08c1f7edd91d5c891e8b94e87f0eafa6b4224fad",
sha256 = "2b12185033fb9178dfaba24412004f89f36263dec1f06c039d9e718e5533d500",
strip_prefix = "envoy-build-tools-{version}",
urls = ["https://github.com/envoyproxy/envoy-build-tools/archive/{version}.tar.gz"],
release_date = "2022-05-12",
Copy link
Contributor

Choose a reason for hiding this comment

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

release_date = "2022-05-18",

Signed-off-by: Lizan Zhou <[email protected]>
mattklein123
mattklein123 previously approved these changes May 20, 2022
@repokitteh-read-only repokitteh-read-only bot removed the deps Approval required for changes to Envoy's external dependencies label May 20, 2022
Signed-off-by: Lizan Zhou <[email protected]>
@repokitteh-read-only repokitteh-read-only bot added the deps Approval required for changes to Envoy's external dependencies label May 20, 2022
mattklein123
mattklein123 previously approved these changes May 20, 2022
@repokitteh-read-only repokitteh-read-only bot removed the deps Approval required for changes to Envoy's external dependencies label May 20, 2022
@htuch
Copy link
Member

htuch commented May 25, 2022

@lizan can you fix CI to get this merged? Thanks.

@phlax
Copy link
Member

phlax commented May 25, 2022

iiuc some of the CI fails are kinda genuine - the binaries that are being built are not finding the host glibc i think

not sure exactly the implications - but it may mean we just need to shift which OS flavours we support

@lizan lizan added the waiting label May 26, 2022
@lizan
Copy link
Member Author

lizan commented May 26, 2022

iiuc some of the CI fails are kinda genuine - the binaries that are being built are not finding the host glibc i think

not sure exactly the implications - but it may mean we just need to shift which OS flavours we support

Yeah this is the problem and I'm trying to figure out what's the best way. Are we comfortable drop 18.04 support from deb distribution?

@phlax
Copy link
Member

phlax commented May 26, 2022

Are we comfortable drop 18.04 support from deb distribution?

at this point i think its a case of support what we support out of the box, and figure out the peripherals once we have that sorted

@phlax phlax mentioned this pull request May 27, 2022
@phlax
Copy link
Member

phlax commented May 27, 2022

(as an aside - im sooooo pleased you triggered this error. Setting up the distro tests took a lot of effort and i was ambivalent about how useful - but this was one of the main things they were intended to catch, so it feels like it was worth it after all)

@github-actions
Copy link

This pull request has been automatically marked as stale because it has not had activity in the last 30 days. It will be closed in 7 days if no further activity occurs. Please feel free to give a status update now, ping for review, or re-open when it's ready. Thank you for your contributions!

@github-actions github-actions bot added the stale stalebot believes this issue/PR has not been touched recently label Jun 26, 2022
@phlax
Copy link
Member

phlax commented Jun 27, 2022

@lizan not sure how you want to proceed with this - lmk if you want me to raise a PR to drop ubuntu 18.04 support from the distro/tests

@github-actions github-actions bot removed the stale stalebot believes this issue/PR has not been touched recently label Jun 27, 2022
@lizan lizan added the no stalebot Disables stalebot from closing an issue label Jun 28, 2022
@lizan
Copy link
Member Author

lizan commented Jun 28, 2022

@phlax Yeah my plan is to drop 18.04 support, though I think we can wait a bit until 1.23.0 release.

ci/README.md Outdated Show resolved Hide resolved
@repokitteh-read-only repokitteh-read-only bot added deps Approval required for changes to Envoy's external dependencies and removed waiting labels Jul 28, 2022
Signed-off-by: Lizan Zhou <[email protected]>
Signed-off-by: Lizan Zhou <[email protected]>
Signed-off-by: Lizan Zhou <[email protected]>
@lizan
Copy link
Member Author

lizan commented Aug 1, 2022

@phlax @mattklein123 this is good to go

Copy link
Member

@phlax phlax left a comment

Choose a reason for hiding this comment

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

when i saw this change i was kinda hesitant as it would be good to support 2 debian versions (i use debian as a ref implementation, on the assumption that this will give a reasonable indication of other deb-based distros)

after thinking about it - i guess we are a year into bullseye (altho next is still a year out)

so lgtm - with the caveat that this may well break some existing users

@repokitteh-read-only repokitteh-read-only bot removed the deps Approval required for changes to Envoy's external dependencies label Aug 1, 2022
Copy link
Member

@mattklein123 mattklein123 left a comment

Choose a reason for hiding this comment

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

Into the future! (Well a bit further into the future :))

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no stalebot Disables stalebot from closing an issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update build base image to ubuntu 20.04
7 participants