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 #915/8ce9c198 backport][stable-2] Fix NoneType errors with elb_classic_lb #918

Conversation

patchback[bot]
Copy link

@patchback patchback bot commented Jul 6, 2022

This is a backport of PR #915 as merged into main (8ce9c19).

SUMMARY

fixes: #589
fixes: #914

Fixes two NoneType related bugs when creating new ELBs.

(includes extra tests this time to trigger the bugs)

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

plugins/modules/elb_classic_lb.py

ADDITIONAL INFORMATION

Fix NoneType errors with elb_classic_lb

SUMMARY
fixes: #589
fixes: #914
Fixes two NoneType related bugs when creating new ELBs.
(includes extra tests this time to trigger the bugs)
ISSUE TYPE

Bugfix Pull Request

COMPONENT NAME
plugins/modules/elb_classic_lb.py
ADDITIONAL INFORMATION

Reviewed-by: Alina Buzachis <None>
(cherry picked from commit 8ce9c19)
@ansibullbot
Copy link

@ansibullbot ansibullbot added bug This issue/PR relates to a bug community_review integration tests/integration module module needs_triage new_contributor Help guide this first time contributor plugins plugin (any type) tests tests labels Jul 6, 2022
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.

✔️ ansible-galaxy-importer SUCCESS in 4m 52s
✔️ build-ansible-collection SUCCESS in 5m 29s
✔️ ansible-test-sanity-aws-ansible-python38 SUCCESS in 10m 39s
✔️ ansible-test-sanity-aws-ansible-2.11-python38 SUCCESS in 10m 52s
✔️ ansible-test-sanity-aws-ansible-2.12-python38 SUCCESS in 10m 28s
✔️ ansible-test-sanity-aws-ansible-2.13-python38 SUCCESS in 9m 40s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 6m 03s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 6m 07s
✔️ ansible-test-splitter SUCCESS in 2m 54s
✔️ integration-amazon.aws-1 SUCCESS in 22m 07s
⚠️ 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-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

@tremble tremble added the mergeit Merge the PR (SoftwareFactory) label Jul 6, 2022
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded (gate pipeline).

✔️ ansible-galaxy-importer SUCCESS in 4m 06s
✔️ build-ansible-collection SUCCESS in 5m 53s
✔️ ansible-test-sanity-aws-ansible-python38 SUCCESS in 8m 41s
✔️ ansible-test-sanity-aws-ansible-2.11-python38 SUCCESS in 10m 29s
✔️ ansible-test-sanity-aws-ansible-2.12-python38 SUCCESS in 9m 46s
✔️ ansible-test-sanity-aws-ansible-2.13-python38 SUCCESS in 9m 51s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 6m 24s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 6m 55s
✔️ ansible-test-splitter SUCCESS in 2m 44s
✔️ integration-amazon.aws-1 SUCCESS in 32m 07s
⚠️ 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-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

@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit 8c1eb89 into stable-2 Jul 6, 2022
@softwarefactory-project-zuul softwarefactory-project-zuul bot deleted the patchback/backports/stable-2/8ce9c198da3cc15424b97046fe340c2a20b646c9/pr-915 branch July 6, 2022 17:41
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Sep 18, 2023
)

Update the name attribute value in the examples

SUMMARY

Problem-

All the examples had same name key value irrespective of different purposes of the tasks

Action taken -

Made changes in the name of the tasks under examples
Corrected it with relevant  name key value to the comments of the task


ISSUE TYPE


Docs Pull Request

COMPONENT NAME




ADDITIONAL INFORMATION

Reviewed-by: Joseph Torcasso <None>
Reviewed-by: Markus Bergholz <[email protected]>
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Sep 18, 2023
)

Update the name attribute value in the examples

SUMMARY

Problem-

All the examples had same name key value irrespective of different purposes of the tasks

Action taken -

Made changes in the name of the tasks under examples
Corrected it with relevant  name key value to the comments of the task


ISSUE TYPE


Docs Pull Request

COMPONENT NAME




ADDITIONAL INFORMATION

Reviewed-by: Joseph Torcasso <None>
Reviewed-by: Markus Bergholz <[email protected]>
mandar242 pushed a commit to mandar242/amazon.aws that referenced this pull request Sep 22, 2023
)

Update the name attribute value in the examples

SUMMARY

Problem-

All the examples had same name key value irrespective of different purposes of the tasks

Action taken -

Made changes in the name of the tasks under examples
Corrected it with relevant  name key value to the comments of the task

ISSUE TYPE

Docs Pull Request

COMPONENT NAME

ADDITIONAL INFORMATION

Reviewed-by: Joseph Torcasso <None>
Reviewed-by: Markus Bergholz <[email protected]>

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections/community.aws@0e08a40
alinabuzachis pushed a commit to alinabuzachis/amazon.aws that referenced this pull request Oct 6, 2023
)

Update the name attribute value in the examples

SUMMARY

Problem-

All the examples had same name key value irrespective of different purposes of the tasks

Action taken -

Made changes in the name of the tasks under examples
Corrected it with relevant  name key value to the comments of the task

ISSUE TYPE

Docs Pull Request

COMPONENT NAME

ADDITIONAL INFORMATION

Reviewed-by: Joseph Torcasso <None>
Reviewed-by: Markus Bergholz <[email protected]>

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections/community.aws@0e08a40
alinabuzachis pushed a commit to alinabuzachis/amazon.aws that referenced this pull request Oct 6, 2023
)

Update the name attribute value in the examples

SUMMARY

Problem-

All the examples had same name key value irrespective of different purposes of the tasks

Action taken -

Made changes in the name of the tasks under examples
Corrected it with relevant  name key value to the comments of the task

ISSUE TYPE

Docs Pull Request

COMPONENT NAME

ADDITIONAL INFORMATION

Reviewed-by: Joseph Torcasso <None>
Reviewed-by: Markus Bergholz <[email protected]>

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections/community.aws@0e08a40
alinabuzachis pushed a commit to alinabuzachis/amazon.aws that referenced this pull request Oct 6, 2023
)

Update the name attribute value in the examples

SUMMARY

Problem-

All the examples had same name key value irrespective of different purposes of the tasks

Action taken -

Made changes in the name of the tasks under examples
Corrected it with relevant  name key value to the comments of the task

ISSUE TYPE

Docs Pull Request

COMPONENT NAME

ADDITIONAL INFORMATION

Reviewed-by: Joseph Torcasso <None>
Reviewed-by: Markus Bergholz <[email protected]>

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections/community.aws@0e08a40
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Oct 24, 2023
)

Update the name attribute value in the examples

SUMMARY

Problem-

All the examples had same name key value irrespective of different purposes of the tasks

Action taken -

Made changes in the name of the tasks under examples
Corrected it with relevant  name key value to the comments of the task


ISSUE TYPE


Docs Pull Request

COMPONENT NAME




ADDITIONAL INFORMATION

Reviewed-by: Joseph Torcasso <None>
Reviewed-by: Markus Bergholz <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This issue/PR relates to a bug community_review integration tests/integration mergeit Merge the PR (SoftwareFactory) module module new_contributor Help guide this first time contributor plugins plugin (any type) tests tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants