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

Locked VM by Admin may unlock by User #6022

Closed
3 tasks
kvakanet opened this issue Nov 18, 2022 · 1 comment
Closed
3 tasks

Locked VM by Admin may unlock by User #6022

kvakanet opened this issue Nov 18, 2022 · 1 comment

Comments

@kvakanet
Copy link

kvakanet commented Nov 18, 2022

Description
Locked VM by Admin may unlock by User, but if simple unlock VM return error:"[one.vm.unlock] Cannot unlock: Lock is owned by another user"

To Reproduce

  1. Lock VM by oneadmin onevm lock <VM_ID>
  2. Unlock VM by user onevm unlock <VM_ID> --user oneuser --password onepassword
  3. Get output: [one.vm.unlock] Cannot unlock: Lock is owned by another user
  4. Lock same VM by user onevm lock <VM_ID> --user oneuser --password onepassword
  5. Unlock VM by user onevm unlock <VM_ID> --user oneuser --password onepassword
  6. And VM unlocked by user.

Expected behavior
A lock set by an administrator should not be overwritten by a user lock and removed later by user.

Details

  • Affected Component: Oned
  • Hypervisor: KVM
  • Version: 6.4

Additional context
Add any other context about the problem here.

Progress Status

  • Code committed
  • Testing - QA
  • Documentation (Release notes - resolved issues, compatibility, known issues)
@paczerny
Copy link
Member

paczerny commented Jan 2, 2023

Only lock owner can change the lock level. Fix --all flag, it now equals --use
PRs:

rsmontero pushed a commit to OpenNebula/docs that referenced this issue Jan 4, 2023
rsmontero pushed a commit to OpenNebula/docs that referenced this issue Jan 4, 2023
rsmontero pushed a commit to OpenNebula/docs that referenced this issue Jan 4, 2023
rsmontero pushed a commit to OpenNebula/docs that referenced this issue Jan 4, 2023
rsmontero pushed a commit to OpenNebula/docs that referenced this issue Jan 4, 2023
(cherry picked from commit f75eb63)
(cherry picked from commit ff286c2)
rsmontero pushed a commit that referenced this issue Jan 6, 2023
* This commit syncs oned and API specification for the ALL flag. The internal defines were not consistent with the API specification.
rsmontero pushed a commit that referenced this issue Mar 1, 2023
* This commit syncs oned and API specification for the ALL flag. The internal defines were not consistent with the API specification.

(cherry picked from commit 82d2191)
feldsam pushed a commit to FELDSAM-INC/one that referenced this issue Nov 4, 2024
* This commit syncs oned and API specification for the ALL flag. The internal defines were not consistent with the API specification.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants