-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
[proxmox] Use proxmoxer_version instead of server API version #6980
[proxmox] Use proxmoxer_version instead of server API version #6980
Conversation
Backport to stable-6: 💔 cherry-picking failed — conflicts found❌ Failed to cleanly apply 796ad35 on top of patchback/backports/stable-6/796ad3565eab36a0146c3f3d084306969a5e51ee/pr-6980 Backporting merged PR #6980 into main
🤖 @patchback |
@UnderGreen thanks for fixing this! |
Backport to stable-7: 💚 backport PR created✅ Backport PR branch: Backported as #6982 🤖 @patchback |
* Use proxmoxer_version instead of server API version * Add changelog fragment (cherry picked from commit 796ad35)
Fixing this in stable-6 would require some adjustments. (The original PR was backported there as well: #6845) |
I will create PR to stable-6 today, no problem 👍 |
…e-collections#6980) * Use proxmoxer_version instead of server API version * Add changelog fragment (cherry picked from commit 796ad35)
…n instead of server API version (#6984) * [proxmox] Use proxmoxer_version instead of server API version (#6980) * Use proxmoxer_version instead of server API version * Add changelog fragment (cherry picked from commit 796ad35) * Update unit tests requirements file * Add python 2.6 constraint for tests
SUMMARY
Fixes authorization via tokens. Check for the version is done against
proxmoxer
, not the API itself.Fixes #6975, #6974.
ISSUE TYPE
COMPONENT NAME
module_utils/proxmox
ADDITIONAL INFORMATION