-
Notifications
You must be signed in to change notification settings - Fork 55
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
release: Merge brew job with release job #886
Conversation
ravanelli
commented
Jun 30, 2023
- We decided to add brew into the release job, since make sense to upload the metadata to brew everytime we release something.
/hold We need it first https://github.com/coreos/coreos-assembler/pull/3519/files as the backports for it.
|
5d51d3a
to
da989d6
Compare
should we also be deleting upload-brew.Jenkinsfile ? |
23c0927
to
7281b48
Compare
01c4d91
to
abfc89b
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
All the backport PRs are merged. We are ok to merge this PR now too. |
- We decided to add brew into the release job, since make sense to upload the metadata to brew everytime we release something; - Delete jobs/upload-brew.Jenkinsfile.
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.
When these jobs were merged, the RHCOS secrets are being passed in when creating the pod in all cases. This means that the secrets are being passed in when creating the pod in FCOS and causing the pods to fail. We'll need to determine how to differentiate between FCOS and RHCOS here.
https://github.com/coreos/fedora-coreos-pipeline/pull/886/files#diff-4768730b889afbd1361511a1e552ee9a9f29b11a7e578aa3ede5e57db08762acR91-R93