Skip to content
This repository has been archived by the owner on Oct 15, 2024. It is now read-only.

Add ELB Listener Rules object #1193

Merged
merged 2 commits into from
Feb 29, 2024
Merged

Conversation

vboulineau
Copy link
Contributor

Adding ELB Listener Rule resource.
It cannot be cleaned up just by using Cloud Control as it requires to list ELB -> Listener -> ListenerRule

@vboulineau vboulineau requested a review from a team as a code owner February 20, 2024 20:41
resources/elbv2-listenerrule.go Outdated Show resolved Hide resolved
Comment on lines 16 to 18
elb *elbv2.LoadBalancer
listener *elbv2.Listener
rule *elbv2.Rule
Copy link
Member

Choose a reason for hiding this comment

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

Do we really need to store whole objects here? Looks like we're only using ARNs and names from the resources.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It's indeed not required, I've changed to only store what's necessary.

@der-eismann der-eismann merged commit c3ae179 into rebuy-de:main Feb 29, 2024
2 checks passed
bjfish25 added a commit to Optum/aws-nuke that referenced this pull request Jun 21, 2024
* Bump the golang group with 1 update (rebuy-de#1188)

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump the golang group with 1 update (rebuy-de#1191)

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Add release note config (rebuy-de#1192)

* Add support for Redshift Serverless namespaces, snapshots and workgroups. (rebuy-de#1194)

* feat: Adding Cloudfront response headers policies (rebuy-de#1140)

* Bump the golang group with 1 update (rebuy-de#1195)

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Add ELB Listener Rules object (rebuy-de#1193)

Co-authored-by: Philipp Trulson <der-eismann@users.noreply.github.com>

* Bump the golang group with 2 updates (rebuy-de#1201)

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump the golang group with 1 update (rebuy-de#1205)

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Add manual trigger to workflow (rebuy-de#1208)

* Bump the golang group across 1 directory with 2 updates (rebuy-de#1217)

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump github.com/aws/aws-sdk-go from 1.51.31 to 1.52.3 in the golang group (rebuy-de#1220)

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump the golang group across 1 directory with 2 updates (rebuy-de#1227)

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump github.com/aws/aws-sdk-go from 1.53.10 to 1.53.15 in the golang group (rebuy-de#1229)

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Add cloudformationiface mock (#19)

---------

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Philipp Trulson <der-eismann@users.noreply.github.com>
Co-authored-by: Maarten Dirkse <maarten.dirkse@gmail.com>
Co-authored-by: Oliver Fletcher <oliver.fletcher@rackspace.com>
Co-authored-by: Vincent Boulineau <58430298+vboulineau@users.noreply.github.com>
Co-authored-by: Bradley Fisher <brad.fisher25@gmail.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants