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

elbv2 module_util - support alb attributes and add compare method for check_mode #696

Conversation

jatorcasso
Copy link
Contributor

@jatorcasso jatorcasso commented Mar 4, 2022

SUMMARY
ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME
  • community.aws.elb_application_lb
  • amazon/aws/plugins/module_utils/elbv2.py

@softwarefactory-project-zuul
Copy link
Contributor

Build failed.

@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.

@softwarefactory-project-zuul
Copy link
Contributor

Build failed.

@jatorcasso
Copy link
Contributor Author

recheck

@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.

Copy link
Collaborator

@jillr jillr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

changelogs/fragments/696-elbv2-support-alb-attributes.yml Outdated Show resolved Hide resolved
@ansibullbot ansibullbot added the tests tests label Mar 9, 2022
@jatorcasso jatorcasso changed the title elbv2 module_util - support alb attributes elbv2 module_util - support alb attributes and add compare method for check_mode Mar 9, 2022
@jatorcasso
Copy link
Contributor Author

updated PR to include adding a compare_elb_attributes() to support check_mode when modifying ALB attributes

@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.

@softwarefactory-project-zuul
Copy link
Contributor

Build failed.

Copy link
Contributor

@mandar242 mandar242 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@jillr jillr added the mergeit Merge the PR (SoftwareFactory) label Mar 9, 2022
@softwarefactory-project-zuul
Copy link
Contributor

Build failed (gate pipeline). For information on how to proceed, see
http://docs.openstack.org/infra/manual/developers.html#automated-testing

@jillr
Copy link
Collaborator

jillr commented Mar 10, 2022

regate

@softwarefactory-project-zuul
Copy link
Contributor

Build failed (gate pipeline). For information on how to proceed, see
http://docs.openstack.org/infra/manual/developers.html#automated-testing

@alinabuzachis alinabuzachis added mergeit Merge the PR (SoftwareFactory) backport-3 PR should be backported to the stable-3 branch and removed mergeit Merge the PR (SoftwareFactory) labels Mar 11, 2022
@softwarefactory-project-zuul
Copy link
Contributor

Build failed (gate pipeline). For information on how to proceed, see
http://docs.openstack.org/infra/manual/developers.html#automated-testing

@jatorcasso
Copy link
Contributor Author

regate

@softwarefactory-project-zuul
Copy link
Contributor

Build failed (gate pipeline). For information on how to proceed, see
http://docs.openstack.org/infra/manual/developers.html#automated-testing

@jillr
Copy link
Collaborator

jillr commented Mar 11, 2022

regate

@softwarefactory-project-zuul
Copy link
Contributor

Build failed (gate pipeline). For information on how to proceed, see
http://docs.openstack.org/infra/manual/developers.html#automated-testing

@alinabuzachis
Copy link
Contributor

regate

@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded (gate pipeline).

@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit b14e392 into ansible-collections:main Mar 12, 2022
@patchback
Copy link

patchback bot commented Mar 12, 2022

Backport to stable-3: 💚 backport PR created

✅ Backport PR branch: patchback/backports/stable-3/b14e392a65945aead32acb78c7e5b6b852048346/pr-696

Backported as #708

🤖 @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 12, 2022
… check_mode (#696)

elbv2 module_util - support alb attributes and add compare method for check_mode

SUMMARY

Add support for modifying alb specific attributes for ansible-collections/community.aws#963
Add compare_elb_attributes method to support check_mode

ISSUE TYPE

Feature Pull Request

COMPONENT NAME

community.aws.elb_application_lb
amazon/aws/plugins/module_utils/elbv2.py

Reviewed-by: Mandar Kulkarni <mandar242@gmail.com>
Reviewed-by: Jill R <None>
Reviewed-by: Joseph Torcasso <None>
(cherry picked from commit b14e392)
softwarefactory-project-zuul bot pushed a commit that referenced this pull request Mar 18, 2022
… check_mode (#696) (#708)

[PR #696/b14e392a backport][stable-3] elbv2 module_util - support alb attributes and add compare method for check_mode

This is a backport of PR #696 as merged into main (b14e392).
SUMMARY

Add support for modifying alb specific attributes for ansible-collections/community.aws#963
Add compare_elb_attributes method to support check_mode

ISSUE TYPE

Feature Pull Request

COMPONENT NAME

community.aws.elb_application_lb
amazon/aws/plugins/module_utils/elbv2.py
softwarefactory-project-zuul bot pushed a commit to ansible-collections/community.aws that referenced this pull request Mar 26, 2022
elb_application_lb - support alb attributes

SUMMARY

Support modifying different alb specific attributes
Fixes #571
Depends-On ansible-collections/amazon.aws#696

ISSUE TYPE

Feature Pull Request

COMPONENT NAME
elb_application_lb

Reviewed-by: Jill R <None>
Reviewed-by: Markus Bergholz <git@osuv.de>
Reviewed-by: Alina Buzachis <None>
Reviewed-by: Joseph Torcasso <None>
jatorcasso added a commit to jatorcasso/community.aws that referenced this pull request Mar 28, 2022
elb_application_lb - support alb attributes

SUMMARY

Support modifying different alb specific attributes
Fixes ansible-collections#571
Depends-On ansible-collections/amazon.aws#696

ISSUE TYPE

Feature Pull Request

COMPONENT NAME
elb_application_lb

Reviewed-by: Jill R <None>
Reviewed-by: Markus Bergholz <git@osuv.de>
Reviewed-by: Alina Buzachis <None>
Reviewed-by: Joseph Torcasso <None>
(cherry picked from commit 85bfce4)
goneri pushed a commit that referenced this pull request Sep 20, 2022
elb_application_lb - support alb attributes

SUMMARY

Support modifying different alb specific attributes
Fixes #571
Depends-On #696

ISSUE TYPE

Feature Pull Request

COMPONENT NAME
elb_application_lb

Reviewed-by: Jill R <None>
Reviewed-by: Markus Bergholz <git@osuv.de>
Reviewed-by: Alina Buzachis <None>
Reviewed-by: Joseph Torcasso <None>

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections/community.aws@85bfce4
GomathiselviS pushed a commit to GomathiselviS/amazon.aws that referenced this pull request Sep 20, 2022
elb_application_lb - support alb attributes

SUMMARY

Support modifying different alb specific attributes
Fixes ansible-collections#571
Depends-On ansible-collections#696

ISSUE TYPE

Feature Pull Request

COMPONENT NAME
elb_application_lb

Reviewed-by: Jill R <None>
Reviewed-by: Markus Bergholz <git@osuv.de>
Reviewed-by: Alina Buzachis <None>
Reviewed-by: Joseph Torcasso <None>

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections/community.aws@85bfce4
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Sep 18, 2023
elb_application_lb - support alb attributes

SUMMARY

Support modifying different alb specific attributes
Fixes ansible-collections#571
Depends-On ansible-collections#696

ISSUE TYPE

Feature Pull Request

COMPONENT NAME
elb_application_lb

Reviewed-by: Jill R <None>
Reviewed-by: Markus Bergholz <git@osuv.de>
Reviewed-by: Alina Buzachis <None>
Reviewed-by: Joseph Torcasso <None>
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Sep 18, 2023
elb_application_lb - support alb attributes

SUMMARY

Support modifying different alb specific attributes
Fixes ansible-collections#571
Depends-On ansible-collections#696

ISSUE TYPE

Feature Pull Request

COMPONENT NAME
elb_application_lb

Reviewed-by: Jill R <None>
Reviewed-by: Markus Bergholz <git@osuv.de>
Reviewed-by: Alina Buzachis <None>
Reviewed-by: Joseph Torcasso <None>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-3 PR should be backported to the stable-3 branch community_review feature This issue/PR relates to a feature request mergeit Merge the PR (SoftwareFactory) module_utils module_utils needs_triage plugins plugin (any type) tests tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants