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
KEP-1731: Extend the KEP with the OBS implementation #3750
Conversation
1405ba3
to
68cc229
Compare
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
@saschagrunert @Conan-Kudo I updated based on the feedback from the OBS team. Please take a look again and let me know if this is okay. |
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.
Modulo the current open discussion that has @shaunix pinged, looks great.
- Additional efforts would be needed to get reproducible builds working | ||
- We would also lose cosign signatures for binaries built by OBS | ||
|
||
`kubepkg` will be extended with a subcommand to create a tarball with all required binaries and files (e.g. systemd |
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.
there has to be some kind of integrity check, no?
Signed-off-by: Marko Mudrinić <mudrinic.mare@gmail.com>
f51dd9a
to
5e007ac
Compare
Signed-off-by: Marko Mudrinić <mudrinic.mare@gmail.com>
5e007ac
to
327dbf7
Compare
All the comments should be addressed now as part of 327dbf7 @kubernetes/sig-release-leads @kubernetes/release-engineering @dirkmueller @darix @adrianschroeter Please take a look at the recent update. |
/approve |
/lgtm |
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.
Let's lift the hold next Tuesday to give everyone a bit room for another round of review.
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: Conan-Kudo, johnbelamaric, saschagrunert, xmudrii The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
|
||
#### Ownership of OBS infrastructure and commitments | ||
|
||
Packages will be hosted, published, and served from the [OpenBuildService instance hosted and maintained by SUSE][obs-build]. The instance is under the full ownership and control of SUSE. SUSE provide us: |
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.
OpenBuildService
-> Open Build Service
I have open comments but don't consider them blocking, feel free to resolve @xmudrii! Thanks! /lgtm |
/unhold We met the Tuesday deadline for feedback. |
One-line PR description: Extend the KEP-1731 with the Open Build Service (OBS) implementation
Issue link: Publishing Kubernetes packages on community infrastructure #1731
Other comments: On hold for discussion and in-depth review by SIG Release Leads and Release Managers
/hold
/assign @saschagrunert @cpanato @puerco @Verolop @jeremyrickard @justaugustus
cc @kubernetes/release-engineering