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

Update hostname_variable tag override example. #618

Merged

Conversation

deekayen
Copy link

SUMMARY

Between Ansible Tower 3.6 and 3.7, the tag formatting changed for overriding the hostname variable. Users doing the 3.7 upgrade may be surprised when their inventories update with the ip##-##-##-##.example.com hostname format when they were previously able to import from AWS using the Name tag as the inventory identifier.

ISSUE TYPE
  • Docs Pull Request
COMPONENT NAME

ec2\

Between Ansible Tower 3.6 and 3.7, the tag formatting changed for overriding the hostname variable.
@ansibullbot ansibullbot added community_review docs inventory inventory plugin needs_triage new_contributor Help guide this first time contributor small_patch Hopefully easy to review labels Jun 30, 2021
@tremble
Copy link
Contributor

tremble commented Aug 25, 2021

recheck

@markuman
Copy link
Member

markuman commented Feb 7, 2022

@deekayen Sorry for the late delay.
Do you have time to rebase your MR?

@markuman markuman added backport-2 PR should be backported to the stable-2 branch backport-3 PR should be backported to the stable-3 branch labels Feb 7, 2022
@ansibullbot ansibullbot added needs_rebase https://docs.ansible.com/ansible/devel/dev_guide/developing_rebasing.html needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR and removed community_review labels Feb 7, 2022
@tremble tremble changed the base branch from main to stable-3 April 14, 2022 09:20
@tremble tremble removed the backport-3 PR should be backported to the stable-3 branch label Apr 14, 2022
@tremble
Copy link
Contributor

tremble commented Apr 14, 2022

The script was removed in preparation for 4.0.0, I've moved this over to stable-3 where the script still exists

@tremble tremble added the mergeit Merge the PR (SoftwareFactory) label Apr 14, 2022
@ansibullbot ansibullbot added community_review and removed needs_rebase https://docs.ansible.com/ansible/devel/dev_guide/developing_rebasing.html needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR labels Apr 14, 2022
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded (gate pipeline).

@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit cb2b186 into ansible-collections:stable-3 Apr 14, 2022
@patchback
Copy link

patchback bot commented Apr 14, 2022

Backport to stable-2: 💚 backport PR created

✅ Backport PR branch: patchback/backports/stable-2/cb2b186fafe1385c6c2533f64a868a312839554b/pr-618

Backported as #1066

🤖 @patchback
I'm built with octomachinery and
my source is open — https://github.com/sanitizers/patchback-github-app.

patchback bot pushed a commit that referenced this pull request Apr 14, 2022
Update hostname_variable tag override example.

SUMMARY
Between Ansible Tower 3.6 and 3.7, the tag formatting changed for overriding the hostname variable. Users doing the 3.7 upgrade may be surprised when their inventories update with the ip##-##-##-##.example.com hostname format when they were previously able to import from AWS using the Name tag as the inventory identifier.
ISSUE TYPE

Docs Pull Request

COMPONENT NAME
ec2\

Reviewed-by: Mark Chappell <None>
(cherry picked from commit cb2b186)
softwarefactory-project-zuul bot pushed a commit that referenced this pull request Apr 14, 2022
[PR #618/cb2b186f backport][stable-2] Update hostname_variable tag override example.

This is a backport of PR #618 as merged into stable-3 (cb2b186).
SUMMARY
Between Ansible Tower 3.6 and 3.7, the tag formatting changed for overriding the hostname variable. Users doing the 3.7 upgrade may be surprised when their inventories update with the ip##-##-##-##.example.com hostname format when they were previously able to import from AWS using the Name tag as the inventory identifier.
ISSUE TYPE

Docs Pull Request

COMPONENT NAME
ec2\

Reviewed-by: Mark Chappell <None>
alinabuzachis pushed a commit to alinabuzachis/community.aws that referenced this pull request May 25, 2022
Fix on_denied and on_missing bugs

SUMMARY
This pull request:

Changes the default value of on_denied to be error, so that it agrees with what is stated in the documentation.
Changes the default value of on_missing to be error, and updates the documentation to explain this.

Fixes ansible-collections#617.
ISSUE TYPE

Bugfix Pull Request

COMPONENT NAME
aws_ssm lookup

Reviewed-by: Alina Buzachis <None>
Reviewed-by: Shane Frasier <maverick@maverickdolphin.com>
Reviewed-by: Markus Bergholz <git@osuv.de>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-2 PR should be backported to the stable-2 branch community_review docs inventory inventory plugin mergeit Merge the PR (SoftwareFactory) needs_triage new_contributor Help guide this first time contributor small_patch Hopefully easy to review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants