-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
OpenStack packaging updates & Jammy support #7622
OpenStack packaging updates & Jammy support #7622
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice work; I've mentioned some minor tweaks and queries.
hack/release/packaging/docker-build-images/ubuntu-jammy-build.Dockerfile.amd64
Outdated
Show resolved
Hide resolved
hack/release/packaging/utils/lib.sh
Outdated
@@ -1,3 +1,5 @@ | |||
#!/bin/bash | |||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This file is designed only to be included (with .
or source
), not to be executed in its own right. That's why I didn't have that hashbang line before, and I think it's better to stay as it was.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The purpose for adding the shebang line is to be able to parse the file correctly with shellcheck
. I haven't done a full linting pass on these files, but I intend to do so in the future, so that's why this line was added. If you think it's a better idea to leave it unchanged though, I'll revert this.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't want to impose my view here, but I do think it's wrong for tools to drive an incorrect agenda. Perhaps there's another way to tell shellcheck that this is a file that it should check?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hmm. I'll back this out for now, and then we can figure out the best solution for this when an actual linting pass happens.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. I'm happy to leave the shellcheck point to your judgement.
Update to our Openstack packaging system: * Add support for Ubuntu Jammy amd64 and ppc64le * Fix build-deps for Ubuntu Focal/Jammy * Clean up CentOS 7 ppc64le builds * Convert build image creation to use `docker buildx bake` (see https://docs.docker.com/build/bake/) * Clean up build output
[Release 3.26] OpenStack packaging updates & Jammy support (cherry-pick #7622)
Description
A revamp of the existing packaging support for OpenStack
docker buildx bake
which can build multiple images in parallelRelease Note