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

feat(inventory): improve api options #397

Merged
merged 9 commits into from
Nov 23, 2023

Conversation

jooola
Copy link
Collaborator

@jooola jooola commented Nov 21, 2023

SUMMARY
  • Rename the inventory token option to api_token, use aliases for backward compatibility.
  • Rename the inventory token_env option to api_token_env, use aliases for backward compatibility.
  • Deprecate the inventory api_token_env option, suggest using a lookup plugin ({{ lookup('ansible.builtin.env', 'YOUR_ENV_VAR') }}) or use the well-known HCLOUD_TOKEN environment variable name.
  • Let ansible parse the options, remove homemade options parsing.
  • Improve and document the existing api_endpoint option.
ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

hcloud inventory

@jooola jooola force-pushed the inventory-cleanup branch 2 times, most recently from 26b7a89 to 3961f10 Compare November 21, 2023 20:04
@jooola jooola marked this pull request as ready for review November 22, 2023 18:09
apricote
apricote previously approved these changes Nov 23, 2023
- Rename `token` option to `api_token`, use aliases for backward compatibility.
- Rename `token_env` option to `api_token_env`, use aliases for backward compatibility.
- Deprecate the `api_token_env` option, suggest using a lookup plugin or use the well-known `HCLOUD_TOKEN` environment variable name.
@jooola jooola merged commit 9905bd0 into ansible-collections:main Nov 23, 2023
28 checks passed
@jooola jooola deleted the inventory-cleanup branch November 23, 2023 13:53
jooola pushed a commit that referenced this pull request Nov 24, 2023
🤖 I have created a release *beep* *boop*
---


##
[2.4.0](2.3.0...2.4.0)
(2023-11-24)


### Features

* add `hetzner.hcloud.all` action group
([#396](#396))
([6581ed5](6581ed5))
* **inventory:** improve api options
([#397](#397))
([9905bd0](9905bd0))
* remove `hcloud_` prefix from all modules names
([#390](#390))
([933a162](933a162))
* rename api_endpoint module argument
([#395](#395))
([7c9fbf8](7c9fbf8))


### Bug Fixes

* **inventory:** improve performance
([#402](#402))
([f85d8f4](f85d8f4))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).
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.

2 participants