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

[PR #2140/87e21522 backport][stable-8] ec2_vpc_vgw - fix call to parent static method in class VGWRetry #2141

Conversation

patchback[bot]
Copy link

@patchback patchback bot commented Sep 9, 2024

This is a backport of PR #2140 as merged into main (87e2152).

SUMMARY

When trying to create a VPN virtual gateway with a non-existent VPC id, module fails incorrectly with the following error

\\"/tmp/ansible_community.aws.ec2_vpc_vgw_payload_t4kolii1/ansible_community.aws.ec2_vpc_vgw_payload.zip/ansible_collections/community/aws/plugins/modules/ec2_vpc_vgw.py\\", line 173, in found\\nTypeError: super(type, obj): obj must be an instance or subtype of type

The fix consists in adding argument to super()

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

ec2_vpc_vgw

SUMMARY

When trying to create a VPN virtual gateway with a non-existent VPC id, module fails incorrectly with the following error
\\"/tmp/ansible_community.aws.ec2_vpc_vgw_payload_t4kolii1/ansible_community.aws.ec2_vpc_vgw_payload.zip/ansible_collections/community/aws/plugins/modules/ec2_vpc_vgw.py\\", line 173, in found\\nTypeError: super(type, obj): obj must be an instance or subtype of type

The fix consists in adding argument to super()

ISSUE TYPE

Bugfix Pull Request

COMPONENT NAME

ec2_vpc_vgw

Reviewed-by: Mark Chappell
(cherry picked from commit 87e2152)
Copy link
Contributor

Build succeeded.
https://ansible.softwarefactory-project.io/zuul/buildset/61102de1d0364c8197145d66fdc6291a

✔️ ansible-galaxy-importer SUCCESS in 3m 59s (non-voting)
✔️ build-ansible-collection SUCCESS in 10m 34s
✔️ ansible-test-splitter SUCCESS in 4m 22s
✔️ integration-community.aws-1 SUCCESS in 6m 52s
Skipped 21 jobs

@tremble tremble added the mergeit Merge the PR (SoftwareFactory) label Sep 9, 2024
Copy link
Contributor

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

ansible-galaxy-importer FAILURE in 4m 46s (non-voting)
✔️ build-ansible-collection SUCCESS in 10m 35s
✔️ ansible-test-splitter SUCCESS in 4m 16s
✔️ integration-community.aws-1 SUCCESS in 7m 38s
Skipped 21 jobs

@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit 4ccc0bd into stable-8 Sep 9, 2024
63 of 67 checks passed
@softwarefactory-project-zuul softwarefactory-project-zuul bot deleted the patchback/backports/stable-8/87e2152254b35949343df70c8fc68af3825573a6/pr-2140 branch September 9, 2024 15:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
mergeit Merge the PR (SoftwareFactory)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants