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

bodhi updates edit UPDATE disables autokarma and autotime #4563

Closed
juhp opened this issue Jun 12, 2022 · 7 comments · Fixed by #4564
Closed

bodhi updates edit UPDATE disables autokarma and autotime #4563

juhp opened this issue Jun 12, 2022 · 7 comments · Fixed by #4564
Milestone

Comments

@juhp
Copy link

juhp commented Jun 12, 2022

I have been using bodhi updates edit UPDATE with no options as workaround for unlocking submitted sidetag updates.
(Yes that is probably a hack but apart from some timing issues, it seems to work.)

However I noticed anyway that this command seems to disable autokarma and autotime?
which does not seem to desireable?

Admittedly running edit with no options is probably not that common (except for this workaround perhaps),
but I would not expect it to change the state of the update.

@juhp juhp changed the title bodhi updates edit UPDATE diables autokarma and autotime bodhi updates edit UPDATE disables autokarma and autotime Jun 12, 2022
@juhp
Copy link
Author

juhp commented Jun 18, 2022

This seems kind of worse now I am afraid, with bodhi-client-6.0.0-2 I get:

ERROR: This update was created from a side-tag. Please add --from_tag and try again.

(That should be --from-tag.) But if I try that I get:

{"status": "error", "errors": [{"location": "body", "name": "from_tag", "description": "The supplied from_tag doesn't exist."}, {"location": "body", "name": "builds", "description": "ACL validation mechanism was unable to determine ACLs."}]}

I guess I should not delete the sidetag until finalizing the update details.
(Also doesn't bodhi already know that the update was from a sidetag?)

@mattiaverga
Copy link
Contributor

Well, there has been no bodhi update since you opened this ticket, and #4564 has not yet been deployed.

However, I've screwed up things in #4482 and I'm going to add a fix in #4564 which will be available in the next release (or you can build bodhi-client yourself from the PR and try it out). Basically, I'm going to not requiring the --from-tag flag if the update being edited is from a side-tag. In these cases, the build list is always refreshed from the side-tag in the backend (that's how it used to work before my faulty PR). It would be nice in future to add a way to let the user choose to refresh the buildlist or not, but it's not so easy and I already caused too much problems...

@juhp
Copy link
Author

juhp commented Jun 20, 2022

Well, there has been no bodhi update since you opened this ticket, and #4564 has not yet been deployed.

(I updated to bodhi-client-6.0.0-2.fc36 on Friday:)

@mattiaverga
Copy link
Contributor

Well, there has been no bodhi update since you opened this ticket, and #4564 has not yet been deployed.

(I updated to bodhi-client-6.0.0-2.fc36 on Friday:)

It was only a rebuild, no code changes (same 6.0.0)

@juhp
Copy link
Author

juhp commented Jun 21, 2022

Alright I am not here to argue, but I don't see any earlier stable build of bodhi-client-6.0.0 for F36 in Bodhi... happy to be shown wrong though

@mattiaverga mattiaverga added this to the 6.0.1 milestone Jun 21, 2022
@mattiaverga
Copy link
Contributor

I think I misunderstood. Indeed, the problem reported was introduced with 6.0.0. The fix will be probably released with the next bugfix release (6.0.1?).
If you need a fix ASAP and you're willing to test the fix, you can manually download and install the RPMs built as part of the PR tests at https://github.com/fedora-infra/bodhi/actions/runs/2524168750 (look at the bottom of the page, you'll find F35, F36 and Rawhide RPMs).

@juhp
Copy link
Author

juhp commented Jun 22, 2022

Okay thanks - I have installed bodhi-client-6.0.0^202206191348git0fe1ded-1.fc36 - will try it out next time I have a chance to push a sidetag update.

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 a pull request may close this issue.

2 participants