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

cmd-koji-upload: tag brew builds #2289

Merged
merged 1 commit into from
Jul 30, 2021
Merged

Conversation

mike-nguyen
Copy link
Member

The tag passed to cmd-koji-upload was only verified as a valid tag but
never used to tag builds. Lets start tagging the builds if the principle
has permissions to do so.

I couldn't actually test the tagging because the principle I use does not
have permissions to tag builds.

Copy link
Member

@dustymabe dustymabe left a comment

Choose a reason for hiding this comment

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

Trivial LGTM

@darkmuggle
Copy link
Contributor

/test sanity

1 similar comment
@mike-nguyen
Copy link
Member Author

/test sanity

@miabbott
Copy link
Member

/retest

3 similar comments
@mike-nguyen
Copy link
Member Author

/retest

@mike-nguyen
Copy link
Member Author

/retest

@mike-nguyen
Copy link
Member Author

/retest

@miabbott
Copy link
Member

CI is failing on [2021-07-29T16:09:09.300Z] error: Hardlinking a3/aeb8c094807bf6d275fb9619d2449dba92a54e42cd253e9005cbced8fa23f2.file to statoverride: File exists

Which I think can be solved if the PR is rebased to pick up - #2293

@miabbott
Copy link
Member

CI is failing on [2021-07-29T16:09:09.300Z] error: Hardlinking a3/aeb8c094807bf6d275fb9619d2449dba92a54e42cd253e9005cbced8fa23f2.file to statoverride: File exists

Which I think can be solved if the PR is rebased to pick up - #2293

Yeah, I rebased locally and was able to build cosa and RHCOS successfully.

The tag passed to cmd-koji-upload was only verified as a valid tag but
never used to tag builds. Lets start tagging the builds if the principle
has permissions to do so.
@mike-nguyen
Copy link
Member Author

rebased 🤞

@miabbott miabbott merged commit 75d144e into coreos:main Jul 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants