8000 Added missing examples of some parameters. by RRohi · Pull Request #117 · ansible-collections/community.proxmox · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Added missing examples of some parameters. #117

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

Merged
merged 5 commits into from
Jun 27, 2025

Conversation

RRohi
Copy link
Contributor
@RRohi RRohi commented Jun 24, 2025
SUMMARY

Troubleshooted my playbook for way too long through trial and error because there weren't any examples of pubkey, api_token_id, api_token_secret parameters.
Decided to add examples for next people who might struggle with these.

ISSUE TYPE
  • Docs Pull Request
COMPONENT NAME

community.proxmox.proxmox

ADDITIONAL INFORMATION

N/A

N/A

RRohi added 2 commits June 25, 2025 00:36
Added missing examples explaining the correct use of the following parameters:
pubkey, api_token_id, api_token_secret
Added what kind of token is being mentioned.
@RRohi
Copy link
Contributor Author
RRohi commented Jun 24, 2025

My struggle is documented here.
#113

By the time I created the issue, I had already spent several hours getting token authentication to work through trial and error.

Copy link
Collaborator
@felixfontein felixfontein left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for this improvement!

RRohi and others added 3 commits June 25, 2025 06:32
Co-authored-by: Felix Fontein <felix@fontein.de>
Co-authored-by: Felix Fontein <felix@fontein.de>
Co-authored-by: Felix Fontein <felix@fontein.de>
Copy link
codecov bot commented Jun 25, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 59.39%. Comparing base (af96bed) to head (1e29a50).
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #117   +/-   ##
=======================================
  Coverage   59.39%   59.39%           
=======================================
  Files          42       42           
  Lines        4576     4576           
  Branches      936      936           
=======================================
  Hits         2718     2718           
  Misses       1729     1729           
  Partials      129      129           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@Thulium-Drake Thulium-Drake merged commit acc51a4 into ansible-collections:main Jun 27, 2025
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants
0