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.

Sign up for GitHub

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

Jump to bottom

Cosmetic: Enhance error message for "profile rename" command #975

Closed
tregubovav-dev opened this issue Jul 9, 2024 · 0 comments · Fixed by #980
Closed

Cosmetic: Enhance error message for "profile rename" command #975

tregubovav-dev opened this issue Jul 9, 2024 · 0 comments · Fixed by #980
Labels
Bug Confirmed to be a bug Easy Good for new contributors
Milestone
incus-6.3

Comments

@tregubovav-dev
Copy link

tregubovav-dev commented Jul 9, 2024 •
edited
Loading

Required information

  • Distribution: Ubuntu
  • Distribution version: 22.04 LTS
  • The output of "incus info" or if that fails:
    • Kernel version: 6.8.0-1006-raspi #6-Ubuntu SMP PREEMPT_DYNAMIC Mon Jun 17 11:30:51 UTC 2024 aarch64 aarch64 aarch64 GNU/Linux
    • Incus version: 6.0.0
    • Storage backend in use: ceph (microceph)

Issue description

incus profile rename <profile_name> <new_profile_name> returns error Error: Query affected 0 rows instead of 1 when profile does not exist instead of Error: Profile not found as other profile commands do in such cases.

Steps to reproduce

  1. Execute incus profile rename a b (ensure that profile a in current project does not exist)
  2. Check error meassge
@stgraber stgraber added Bug Confirmed to be a bug Easy Good for new contributors labels Jul 10, 2024
@stgraber stgraber added this to the incus-6.3 milestone Jul 10, 2024
stgraber added a commit to stgraber/incus that referenced this issue Jul 10, 2024
@stgraber
incusd: Improve profile rename errors
7de1849 
Closes lxc#975

Signed-off-by: Stéphane Graber <stgraber@stgraber.org>
@stgraber stgraber mentioned this issue Jul 10, 2024
Merged
stgraber added a commit that referenced this issue Jul 18, 2024
@stgraber
incusd: Improve profile rename errors
db39317 
Closes #975

Signed-off-by: Stéphane Graber <stgraber@stgraber.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Bug Confirmed to be a bug Easy Good for new contributors
Milestone
incus-6.3
Development

Successfully merging a pull request may close this issue.

incusd: Improve profile rename errors stgraber/incus
2 participants
@stgraber @tregubovav-dev