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

[PR #6839/e5dc6978 backport][stable-6] proxmox module utils: better error msg when token fails with old proxmoxer #6845

Merged
merged 1 commit into from
Jul 3, 2023
Merged

[PR #6839/e5dc6978 backport][stable-6] proxmox module utils: better error msg when token fails with old proxmoxer #6845

merged 1 commit into from
Jul 3, 2023

Conversation

patchback[bot]
Copy link

@patchback patchback bot commented Jul 3, 2023

This is a backport of PR #6839 as merged into main (e5dc697).

SUMMARY

Add logic to detect whether an old Promoxer complains about the token_name and token_value parameters and provide a better error message when that happens.

Fixes #5371

ISSUE TYPE
  • Bugfix Pull Request
  • Docs Pull Request
COMPONENT NAME

plugins/module_utils/proxmox.py

@russoz @patchback
proxmox module utils: better error msg when token fails with old prox…
dd29f78 
…moxer (#6839)

* proxmox module utils: clear msg when token fails with old proxmoxer

* add changelog frag

* better way to determine version

(cherry picked from commit e5dc697)
@patchback patchback bot mentioned this pull request Jul 3, 2023
Merged
@ansibullbot ansibullbot added backport bug This issue/PR relates to a bug docs_fragments docs_fragments plugin (shared docs) module_utils module_utils needs_ci This PR requires CI testing to be performed. Please close and re-open this PR to trigger CI needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR new_contributor Help guide this first time contributor plugins plugin (any type) and removed needs_ci This PR requires CI testing to be performed. Please close and re-open this PR to trigger CI needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR labels Jul 3, 2023
@felixfontein felixfontein merged commit d633f6c into stable-6 Jul 3, 2023
@felixfontein felixfontein deleted the patchback/backports/stable-6/e5dc69788799168c0f53c1da636262c81d8fdc5b/pr-6839 branch July 3, 2023 20:57
@felixfontein felixfontein mentioned this pull request Jul 20, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
bug This issue/PR relates to a bug docs_fragments docs_fragments plugin (shared docs) module_utils module_utils new_contributor Help guide this first time contributor plugins plugin (any type)
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@felixfontein @ansibullbot @russoz