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

Editing an update using CLI sets type to 'bugfix' if not provided #2528

Closed
ghost opened this issue Aug 6, 2018 · 6 comments · Fixed by #2541
Closed

Editing an update using CLI sets type to 'bugfix' if not provided #2528

ghost opened this issue Aug 6, 2018 · 6 comments · Fixed by #2541
Labels
Client Issues with the bodhi command line interface tool High priority These issues are higher priority than normal

Comments

@ghost
Copy link

ghost commented Aug 6, 2018

While editing an update using CLI, if we don't provide the update type, it will modify it to bugfix

So if we edit a security update,

bodhi updates edit --severity low moin-1.9.9-1.el6

update's type will be changed to bugfix. I think this might be the reason, as the default is shared between create and edit update.

@bowlofeggs
Copy link
Contributor

Was this not fixed by #2464 ? If not, was it caused by it? Is this a regression in 3.9, or was it present before?

@bowlofeggs bowlofeggs added Critical We can't go on living in this sqalor, drop everything and fix it! Client Issues with the bodhi command line interface tool labels Aug 16, 2018
@ghost
Copy link
Author

ghost commented Aug 16, 2018

@bowlofeggs: Neither. This bug is logically different from #2464. It was present before as there is a common default option in CLI for creating/editing update. I think a simple fix will be moving --type out of the common options here. But it will move it to either top or bottom in the --help section.

@bowlofeggs
Copy link
Contributor

I have confirmed that this issue is present in Bodhi 3.8, so I will degrade its priority to high and will not consider it a 3.9 release blocker.

@bowlofeggs bowlofeggs added High priority These issues are higher priority than normal and removed Critical We can't go on living in this sqalor, drop everything and fix it! labels Aug 20, 2018
@mergify mergify bot closed this as completed in #2541 Dec 12, 2018
mergify bot pushed a commit that referenced this issue Dec 12, 2018
fixes #2528

Signed-off-by: Vismay Golwala <vgolwala@redhat.com>
@bowlofeggs
Copy link
Contributor

This fix is planned for inclusion in Bodhi 3.13: #2917.

@bowlofeggs
Copy link
Contributor

@bowlofeggs
Copy link
Contributor

Bodhi 3.13.0 has been released:

https://github.com/fedora-infra/bodhi/releases/tag/3.13.0

Fedora Infrastructure plans to deploy it to production on Monday.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Client Issues with the bodhi command line interface tool High priority These issues are higher priority than normal
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant