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

route53_health_check: add support for enabling latency graphs #1201

Conversation

mandar242
Copy link
Contributor

SUMMARY

Fixes #1189
Add support for enabling Latency graphs (MeasureLatency) during creation of a Route53 Health Check.

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

route53_health_check

ADDITIONAL INFORMATION

You can't change the value of MeasureLatency after you create a health check.

@ansibullbot
Copy link

@ansibullbot ansibullbot added community_review feature This issue/PR relates to a feature request integration tests/integration module module needs_triage plugins plugin (any type) tests tests labels Oct 26, 2022
@github-actions
Copy link

github-actions bot commented Oct 26, 2022

Docs Build 📝

Thank you for contribution!✨

This PR has been merged and your docs changes will be incorporated when they are next published.

@softwarefactory-project-zuul
Copy link
Contributor

Build failed.

✔️ ansible-galaxy-importer SUCCESS in 4m 14s
✔️ build-ansible-collection SUCCESS in 5m 38s
✔️ ansible-test-sanity-aws-ansible-python38 SUCCESS in 10m 42s
✔️ ansible-test-sanity-aws-ansible-2.12-python38 SUCCESS in 10m 31s
✔️ ansible-test-sanity-aws-ansible-2.13-python38 SUCCESS in 10m 29s
ansible-test-units-amazon-aws-python36 FAILURE in 6m 24s
ansible-test-units-amazon-aws-python38 FAILURE in 6m 22s
ansible-test-units-amazon-aws-python39 FAILURE in 5m 35s
✔️ cloud-tox-py3 SUCCESS in 3m 49s
✔️ ansible-test-splitter SUCCESS in 3m 16s
✔️ integration-amazon.aws-1 SUCCESS in 7m 12s
⚠️ integration-amazon.aws-2 SKIPPED
⚠️ integration-amazon.aws-3 SKIPPED
⚠️ integration-amazon.aws-4 SKIPPED
⚠️ integration-amazon.aws-5 SKIPPED
⚠️ integration-amazon.aws-6 SKIPPED
⚠️ integration-amazon.aws-7 SKIPPED
⚠️ integration-amazon.aws-8 SKIPPED
⚠️ integration-amazon.aws-9 SKIPPED
⚠️ integration-amazon.aws-10 SKIPPED
⚠️ integration-amazon.aws-11 SKIPPED
⚠️ integration-amazon.aws-12 SKIPPED
⚠️ integration-amazon.aws-13 SKIPPED
⚠️ integration-amazon.aws-14 SKIPPED
⚠️ integration-amazon.aws-15 SKIPPED
⚠️ integration-amazon.aws-16 SKIPPED
⚠️ integration-amazon.aws-17 SKIPPED
⚠️ integration-amazon.aws-18 SKIPPED
⚠️ integration-community.aws-1 SKIPPED
⚠️ integration-community.aws-2 SKIPPED
⚠️ integration-community.aws-3 SKIPPED
⚠️ integration-community.aws-4 SKIPPED
⚠️ integration-community.aws-5 SKIPPED
⚠️ integration-community.aws-6 SKIPPED
⚠️ integration-community.aws-7 SKIPPED
⚠️ integration-community.aws-8 SKIPPED
⚠️ integration-community.aws-9 SKIPPED
⚠️ integration-community.aws-10 SKIPPED
⚠️ integration-community.aws-11 SKIPPED
⚠️ integration-community.aws-12 SKIPPED
⚠️ integration-community.aws-13 SKIPPED
⚠️ integration-community.aws-14 SKIPPED
⚠️ integration-community.aws-15 SKIPPED
⚠️ integration-community.aws-16 SKIPPED
⚠️ integration-community.aws-17 SKIPPED
⚠️ integration-community.aws-18 SKIPPED
✔️ ansible-test-changelog SUCCESS in 2m 40s

@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.

✔️ ansible-galaxy-importer SUCCESS in 4m 32s
✔️ build-ansible-collection SUCCESS in 5m 23s
✔️ ansible-test-sanity-aws-ansible-python38 SUCCESS in 8m 58s
✔️ ansible-test-sanity-aws-ansible-2.12-python38 SUCCESS in 9m 13s
✔️ ansible-test-sanity-aws-ansible-2.13-python38 SUCCESS in 9m 01s
✔️ ansible-test-units-amazon-aws-python36 SUCCESS in 6m 18s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 5m 55s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 7m 06s
✔️ cloud-tox-py3 SUCCESS in 4m 09s
✔️ ansible-test-splitter SUCCESS in 3m 02s
✔️ integration-amazon.aws-1 SUCCESS in 9m 20s
⚠️ integration-amazon.aws-2 SKIPPED
⚠️ integration-amazon.aws-3 SKIPPED
⚠️ integration-amazon.aws-4 SKIPPED
⚠️ integration-amazon.aws-5 SKIPPED
⚠️ integration-amazon.aws-6 SKIPPED
⚠️ integration-amazon.aws-7 SKIPPED
⚠️ integration-amazon.aws-8 SKIPPED
⚠️ integration-amazon.aws-9 SKIPPED
⚠️ integration-amazon.aws-10 SKIPPED
⚠️ integration-amazon.aws-11 SKIPPED
⚠️ integration-amazon.aws-12 SKIPPED
⚠️ integration-amazon.aws-13 SKIPPED
⚠️ integration-amazon.aws-14 SKIPPED
⚠️ integration-amazon.aws-15 SKIPPED
⚠️ integration-amazon.aws-16 SKIPPED
⚠️ integration-amazon.aws-17 SKIPPED
⚠️ integration-amazon.aws-18 SKIPPED
⚠️ integration-community.aws-1 SKIPPED
⚠️ integration-community.aws-2 SKIPPED
⚠️ integration-community.aws-3 SKIPPED
⚠️ integration-community.aws-4 SKIPPED
⚠️ integration-community.aws-5 SKIPPED
⚠️ integration-community.aws-6 SKIPPED
⚠️ integration-community.aws-7 SKIPPED
⚠️ integration-community.aws-8 SKIPPED
⚠️ integration-community.aws-9 SKIPPED
⚠️ integration-community.aws-10 SKIPPED
⚠️ integration-community.aws-11 SKIPPED
⚠️ integration-community.aws-12 SKIPPED
⚠️ integration-community.aws-13 SKIPPED
⚠️ integration-community.aws-14 SKIPPED
⚠️ integration-community.aws-15 SKIPPED
⚠️ integration-community.aws-16 SKIPPED
⚠️ integration-community.aws-17 SKIPPED
⚠️ integration-community.aws-18 SKIPPED
✔️ ansible-test-changelog SUCCESS in 2m 14s

@mandar242 mandar242 added the mergeit Merge the PR (SoftwareFactory) label Oct 28, 2022
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded (gate pipeline).

✔️ ansible-galaxy-importer SUCCESS in 4m 15s
✔️ build-ansible-collection SUCCESS in 4m 53s
✔️ ansible-test-sanity-aws-ansible-python38 SUCCESS in 9m 34s
✔️ ansible-test-sanity-aws-ansible-2.12-python38 SUCCESS in 9m 41s
✔️ ansible-test-sanity-aws-ansible-2.13-python38 SUCCESS in 9m 43s
✔️ ansible-test-sanity-aws-ansible-2.14 SUCCESS in 9m 53s
✔️ ansible-test-units-amazon-aws-python36 SUCCESS in 6m 50s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 6m 53s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 7m 35s
✔️ cloud-tox-py3 SUCCESS in 4m 31s
✔️ ansible-test-splitter SUCCESS in 2m 46s
✔️ integration-amazon.aws-1 SUCCESS in 7m 10s
⚠️ integration-amazon.aws-2 SKIPPED
⚠️ integration-amazon.aws-3 SKIPPED
⚠️ integration-amazon.aws-4 SKIPPED
⚠️ integration-amazon.aws-5 SKIPPED
⚠️ integration-amazon.aws-6 SKIPPED
⚠️ integration-amazon.aws-7 SKIPPED
⚠️ integration-amazon.aws-8 SKIPPED
⚠️ integration-amazon.aws-9 SKIPPED
⚠️ integration-amazon.aws-10 SKIPPED
⚠️ integration-amazon.aws-11 SKIPPED
⚠️ integration-amazon.aws-12 SKIPPED
⚠️ integration-amazon.aws-13 SKIPPED
⚠️ integration-amazon.aws-14 SKIPPED
⚠️ integration-amazon.aws-15 SKIPPED
⚠️ integration-amazon.aws-16 SKIPPED
⚠️ integration-amazon.aws-17 SKIPPED
⚠️ integration-amazon.aws-18 SKIPPED
⚠️ integration-community.aws-1 SKIPPED
⚠️ integration-community.aws-2 SKIPPED
⚠️ integration-community.aws-3 SKIPPED
⚠️ integration-community.aws-4 SKIPPED
⚠️ integration-community.aws-5 SKIPPED
⚠️ integration-community.aws-6 SKIPPED
⚠️ integration-community.aws-7 SKIPPED
⚠️ integration-community.aws-8 SKIPPED
⚠️ integration-community.aws-9 SKIPPED
⚠️ integration-community.aws-10 SKIPPED
⚠️ integration-community.aws-11 SKIPPED
⚠️ integration-community.aws-12 SKIPPED
⚠️ integration-community.aws-13 SKIPPED
⚠️ integration-community.aws-14 SKIPPED
⚠️ integration-community.aws-15 SKIPPED
⚠️ integration-community.aws-16 SKIPPED
⚠️ integration-community.aws-17 SKIPPED
⚠️ integration-community.aws-18 SKIPPED
✔️ ansible-test-changelog SUCCESS in 2m 26s

@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit 63cca2c into ansible-collections:main Oct 28, 2022
softwarefactory-project-zuul bot pushed a commit that referenced this pull request Nov 1, 2022
route53_health_check: bump up version_added for measure_latency

SUMMARY

With amazon.aws 5.1.0 release before merging #1201 , changing version_added for measure_latency to 5.2.0

ISSUE TYPE


Bugfix Pull Request

COMPONENT NAME

route53_health_check

Reviewed-by: Alina Buzachis <None>
@tremble tremble added the backport-5 PR should be backported to the stable-5 branch label Mar 17, 2023
@patchback
Copy link

patchback bot commented Mar 17, 2023

Backport to stable-5: 💚 backport PR created

✅ Backport PR branch: patchback/backports/stable-5/63cca2cebc3506d6afa17d66c4f42cafd767fb35/pr-1201

Backported as #1432

🤖 @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 Mar 17, 2023
route53_health_check: add support for enabling latency graphs

SUMMARY

Fixes #1189
Add support for enabling Latency graphs (MeasureLatency) during creation of a Route53 Health Check.

ISSUE TYPE

Feature Pull Request

COMPONENT NAME

route53_health_check
ADDITIONAL INFORMATION

You can't change the value of MeasureLatency after you create a health check.

Reviewed-by: Alina Buzachis <None>
Reviewed-by: Bikouo Aubin <None>
(cherry picked from commit 63cca2c)
softwarefactory-project-zuul bot pushed a commit that referenced this pull request Mar 17, 2023
…ort for enabling latency graphs (#1432)

[PR #1201/63cca2ce backport][stable-5] route53_health_check: add support for enabling latency graphs

This is a backport of PR #1201 as merged into main (63cca2c).
SUMMARY

Fixes #1189
Add support for enabling Latency graphs (MeasureLatency) during creation of a Route53 Health Check.

ISSUE TYPE


Feature Pull Request

COMPONENT NAME

route53_health_check
ADDITIONAL INFORMATION


You can't change the value of MeasureLatency after you create a health check.

Reviewed-by: Mark Chappell
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Sep 18, 2023
Tagging - ec2_snapshot_copy

SUMMARY

Add the "resource_tags" alias, for consistency with other modules
minor docs clean-up
Use TagSpecification on creation rather than making a separate API call to tag the resource after creation.

Does not add purge_tags, since the module performs a one-shot action rather than managing the resources.
ISSUE TYPE

Docs Pull Request
Feature Pull Request

COMPONENT NAME
ec2_snapshot_copy
ADDITIONAL INFORMATION
Since we don't have a tags-only fragment, I've not switched this over to using a fragment.  If I find more modules with a similar use-case I'll try to find a standard fragment we can use.

Reviewed-by: Alina Buzachis <None>
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Sep 18, 2023
Tagging - ec2_snapshot_copy

SUMMARY

Add the "resource_tags" alias, for consistency with other modules
minor docs clean-up
Use TagSpecification on creation rather than making a separate API call to tag the resource after creation.

Does not add purge_tags, since the module performs a one-shot action rather than managing the resources.
ISSUE TYPE

Docs Pull Request
Feature Pull Request

COMPONENT NAME
ec2_snapshot_copy
ADDITIONAL INFORMATION
Since we don't have a tags-only fragment, I've not switched this over to using a fragment.  If I find more modules with a similar use-case I'll try to find a standard fragment we can use.

Reviewed-by: Alina Buzachis <None>
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Oct 24, 2023
Tagging - ec2_snapshot_copy

SUMMARY

Add the "resource_tags" alias, for consistency with other modules
minor docs clean-up
Use TagSpecification on creation rather than making a separate API call to tag the resource after creation.

Does not add purge_tags, since the module performs a one-shot action rather than managing the resources.
ISSUE TYPE

Docs Pull Request
Feature Pull Request

COMPONENT NAME
ec2_snapshot_copy
ADDITIONAL INFORMATION
Since we don't have a tags-only fragment, I've not switched this over to using a fragment.  If I find more modules with a similar use-case I'll try to find a standard fragment we can use.

Reviewed-by: Alina Buzachis <None>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-5 PR should be backported to the stable-5 branch community_review feature This issue/PR relates to a feature request integration tests/integration mergeit Merge the PR (SoftwareFactory) module module needs_triage plugins plugin (any type) tests tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

route53_health_check - Add support for Latency graphs (MeasureLatency) during creation
5 participants