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

rds_instance: fix 'typeError'when tagging gp3 db #1437

Conversation

mandar242
Copy link
Contributor

SUMMARY

This PR fixes bug causing error when managing tags on a gp3 rds database instance, caused by a conditional statement comparing allocated_storage even when it is not provided.
currently code returns TypeError: '>=' not supported between instances of 'NoneType' and 'int'.

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

rds_instance

ADDITIONAL INFORMATION

@mandar242 mandar242 changed the title [WIP] rds_instance: fix 'typeError'when tagging gp3 db rds_instance: fix 'typeError'when tagging gp3 db Mar 23, 2023
@softwarefactory-project-zuul

This comment was marked as resolved.

@tremble tremble added mergeit Merge the PR (SoftwareFactory) backport-5 PR should be backported to the stable-5 branch labels Mar 24, 2023
@tremble tremble removed the mergeit Merge the PR (SoftwareFactory) label Mar 24, 2023
@tremble
Copy link
Contributor

tremble commented Mar 24, 2023

2023-03-24 11:47:59.216798 | controller | TASK [rds_instance_tagging : assert] *******************************************
2023-03-24 11:47:59.216802 | controller | task path: /home/zuul/.ansible/collections/ansible_collections/amazon/aws/tests/integration/targets/rds_instance_tagging/tasks/test_tagging_gp3.yml:48
2023-03-24 11:47:59.271907 | controller | fatal: [testhost]: FAILED! => {
2023-03-24 11:47:59.271945 | controller |     "msg": "The conditional check 'result.db_instance_id_gp3entifier == 'ansible-test-7e22c197e475-gp3'' failed. The error was: error while evaluating conditional (result.db_instance_id_gp3entifier == 'ansible-test-7e22c197e475-gp3'): 'dict object' has no attribute 'db_instance_id_gp3entifier'. 'dict object' has no attribute 'db_instance_id_gp3entifier'"
2023-03-24 11:47:59.271952 | controller | }

@softwarefactory-project-zuul
Copy link
Contributor

Build failed.
https://ansible.softwarefactory-project.io/zuul/buildset/943d8d92147a4a8d85d6ce523165fc58

✔️ ansible-galaxy-importer SUCCESS in 4m 23s
✔️ build-ansible-collection SUCCESS in 12m 27s
✔️ ansible-test-sanity-aws-ansible-python38 SUCCESS in 9m 40s (non-voting)
✔️ ansible-test-sanity-aws-ansible-2.12-python38 SUCCESS in 10m 22s
✔️ ansible-test-sanity-aws-ansible-2.13-python38 SUCCESS in 11m 44s
✔️ ansible-test-sanity-aws-ansible-2.14 SUCCESS in 10m 33s
✔️ ansible-test-units-amazon-aws-python36 SUCCESS in 9m 11s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 9m 32s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 8m 26s
✔️ ansible-test-units-amazon-aws-python310 SUCCESS in 9m 25s
✔️ cloud-tox-py3 SUCCESS in 6m 44s
✔️ ansible-test-changelog SUCCESS in 4m 15s
✔️ ansible-test-splitter SUCCESS in 5m 01s
✔️ integration-amazon.aws-1 SUCCESS in 15m 07s
✔️ integration-amazon.aws-2 SUCCESS in 32m 16s
✔️ integration-amazon.aws-3 SUCCESS in 43m 45s
✔️ integration-amazon.aws-4 SUCCESS in 24m 45s
✔️ integration-amazon.aws-5 SUCCESS in 26m 08s
✔️ integration-amazon.aws-6 SUCCESS in 25m 46s
✔️ integration-amazon.aws-7 SUCCESS in 16m 50s
✔️ integration-amazon.aws-8 SUCCESS in 18m 55s
✔️ integration-amazon.aws-9 SUCCESS in 16m 37s
integration-amazon.aws-10 FAILURE in 23m 24s
✔️ integration-amazon.aws-11 SUCCESS in 14m 26s
Skipped 33 jobs

@tremble tremble mentioned this pull request Mar 24, 2023
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.
https://ansible.softwarefactory-project.io/zuul/buildset/1735371045d54b9b9729a4b6501e3037

✔️ ansible-galaxy-importer SUCCESS in 3m 45s
✔️ build-ansible-collection SUCCESS in 12m 35s
✔️ ansible-test-sanity-aws-ansible-python38 SUCCESS in 10m 59s (non-voting)
✔️ ansible-test-sanity-aws-ansible-2.12-python38 SUCCESS in 11m 10s
✔️ ansible-test-sanity-aws-ansible-2.13-python38 SUCCESS in 8m 29s
✔️ ansible-test-sanity-aws-ansible-2.14 SUCCESS in 10m 03s
✔️ ansible-test-units-amazon-aws-python36 SUCCESS in 12m 21s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 9m 30s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 9m 15s
✔️ ansible-test-units-amazon-aws-python310 SUCCESS in 10m 14s
✔️ cloud-tox-py3 SUCCESS in 8m 44s
✔️ ansible-test-changelog SUCCESS in 4m 13s
✔️ ansible-test-splitter SUCCESS in 4m 38s
✔️ integration-amazon.aws-1 SUCCESS in 16m 15s
✔️ integration-amazon.aws-2 SUCCESS in 32m 31s
✔️ integration-amazon.aws-3 SUCCESS in 34m 35s
✔️ integration-amazon.aws-4 SUCCESS in 28m 29s
✔️ integration-amazon.aws-5 SUCCESS in 22m 07s
✔️ integration-amazon.aws-6 SUCCESS in 17m 06s
✔️ integration-amazon.aws-7 SUCCESS in 15m 56s
✔️ integration-amazon.aws-8 SUCCESS in 19m 22s
✔️ integration-amazon.aws-9 SUCCESS in 15m 59s
✔️ integration-amazon.aws-10 SUCCESS in 23m 36s
✔️ integration-amazon.aws-11 SUCCESS in 12m 44s
Skipped 33 jobs

@tremble tremble added the mergeit Merge the PR (SoftwareFactory) label Mar 24, 2023
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded (gate pipeline).
https://ansible.softwarefactory-project.io/zuul/buildset/c809a1414ab743d996c79adc88d6fc88

✔️ ansible-galaxy-importer SUCCESS in 4m 17s
✔️ build-ansible-collection SUCCESS in 12m 38s
✔️ ansible-test-splitter SUCCESS in 4m 42s
✔️ integration-amazon.aws-1 SUCCESS in 17m 32s
✔️ integration-amazon.aws-2 SUCCESS in 33m 20s
✔️ integration-amazon.aws-3 SUCCESS in 33m 43s
✔️ integration-amazon.aws-4 SUCCESS in 27m 33s
✔️ integration-amazon.aws-5 SUCCESS in 29m 57s
✔️ integration-amazon.aws-6 SUCCESS in 14m 22s
✔️ integration-amazon.aws-7 SUCCESS in 17m 17s
✔️ integration-amazon.aws-8 SUCCESS in 21m 19s
✔️ integration-amazon.aws-9 SUCCESS in 16m 52s
✔️ integration-amazon.aws-10 SUCCESS in 22m 40s
✔️ integration-amazon.aws-11 SUCCESS in 15m 32s
Skipped 33 jobs

@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit f63d191 into ansible-collections:main Mar 24, 2023
@patchback
Copy link

patchback bot commented Mar 24, 2023

Backport to stable-5: 💚 backport PR created

✅ Backport PR branch: patchback/backports/stable-5/f63d1919bc9df032c864c2ec3e085900cdf3f9bf/pr-1437

Backported as #1444

🤖 @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 24, 2023
rds_instance: fix 'typeError'when tagging gp3 db

SUMMARY

This PR fixes bug causing error when managing tags on a gp3 rds database instance, caused by a conditional statement comparing allocated_storage even when it is not provided.
currently code returns TypeError: '>=' not supported between instances of 'NoneType' and 'int'.

ISSUE TYPE

Bugfix Pull Request

COMPONENT NAME

rds_instance
ADDITIONAL INFORMATION

Reviewed-by: Mark Chappell
(cherry picked from commit f63d191)
softwarefactory-project-zuul bot pushed a commit that referenced this pull request Mar 24, 2023
[PR #1437/f63d1919 backport][stable-5] rds_instance: fix 'typeError'when tagging gp3 db

This is a backport of PR #1437 as merged into main (f63d191).
SUMMARY

This PR fixes bug causing error when managing tags on a gp3 rds database instance, caused by a conditional statement comparing allocated_storage even when it is not provided.
currently code returns TypeError: '>=' not supported between instances of 'NoneType' and 'int'.

ISSUE TYPE


Bugfix Pull Request

COMPONENT NAME

rds_instance
ADDITIONAL INFORMATION

Reviewed-by: Mark Chappell
@mandar242 mandar242 deleted the rds_instance_allocated_storage_fix branch May 9, 2023 02:45
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Oct 24, 2023
Changelog Rename ec2_url to endpoint_url

SUMMARY
In ansible-collections#992 ec2_url was renamed to endpoint_url, with ec2_url remaining as an alias.  This fragment is to improve visibility for a cross-collection change.
ISSUE TYPE

Docs Pull Request

COMPONENT NAME
changelogs/fragments/
ADDITIONAL INFORMATION

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 mergeit Merge the PR (SoftwareFactory)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants