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
Rename hcloud_*_facts to hcloud_*_info #60926
Rename hcloud_*_facts to hcloud_*_info #60926
Conversation
The test
The test
The test
The test
The test
The test
The test
The test
The test
The test
The test
The test
|
The test
|
The test
|
@cschmitt-hcloud @LKaemmerling As a maintainer of a module in the same namespace this new module has been submitted to, your vote counts for shipits. Please review this module and add |
@cschmitt-hcloud @LKaemmerling As a maintainer of a module in the same namespace this new module has been submitted to, your vote counts for shipits. Please review this module and add |
@LKaemmerling this PR contains more than one new module. Please submit only one new module per pull request. For a detailed explanation, please read the grouped modules documentation |
@LKaemmerling this PR contains more than one new module. Please submit only one new module per pull request. For a detailed explanation, please read the grouped modules documentation |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. I've found one minor detail; if you want to change it, please do so.
What's missing are porting guide entries and a changelog. See 2d98734 for how this should look like.
hcloud_datacenter_info: | ||
description: | ||
- The datacenter info as list | ||
- This module was called C(hcloud_datacenter_facts) before Ansible 2.9, returning C(ansible_facts) and C(hcloud_datacenter_facts). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- This module was called C(hcloud_datacenter_facts) before Ansible 2.9, returning C(ansible_facts) and C(hcloud_datacenter_facts). | |
- This module was called C(hcloud_datacenter_facts) before Ansible 2.9, returning C(hcloud_datacenter_facts) as C(ansible_facts). |
You have to decide whether you want to change this in all places (or in none :) ).
@LKaemmerling This PR was evaluated as a potentially problematic PR for the following reasons:
Such PR can only be merged by human. Contact a Core team member to review this PR on IRC: |
@LKaemmerling this PR contains the following merge commits: Please rebase your branch to remove these commits. |
@LKaemmerling This PR was evaluated as a potentially problematic PR for the following reasons:
Such PR can only be merged by human. Contact a Core team member to review this PR on IRC: |
@LKaemmerling this PR contains the following merge commits: Please rebase your branch to remove these commits. |
@LKaemmerling @cschmitt-hcloud thanks for working on this! |
* Rename hcloud_datacenter_facts to hcloud_datacenter_info * Rename hcloud_location_facts to hcloud_location_info * Rename hcloud_image_facts to hcloud_image_info * Rename hcloud_floating_ip_facts to hcloud_floating_ip_info * Rename hcloud_server_type_facts to hcloud_server_type_info * Rename hcloud_server_facts to hcloud_server_info * Rename hcloud_ssh_key_facts to hcloud_ssh_key_info * Rename hcloud_volume_facts to hcloud_volume_info * Fix typo in hcloud_image_info * Add to porting guide and add changelog fragment * Reword porting guide
SUMMARY
This will rename all
hcloud_*_facts
modules to*_info
.Closes #60516
ISSUE TYPE
COMPONENT NAME
ADDITIONAL INFORMATION