Skip to content

AWS DHCP option sets#636

Merged
Bobi-Wan merged 7 commits intogardener:mainfrom
Bobi-Wan:feat/aws-dhcp-option-sets
Sep 16, 2025
Merged

AWS DHCP option sets#636
Bobi-Wan merged 7 commits intogardener:mainfrom
Bobi-Wan:feat/aws-dhcp-option-sets

Conversation

@Bobi-Wan
Copy link
Copy Markdown
Contributor

What this PR does / why we need it:
Adds the following changes to AWS models:

  • DHCPOptionSet model and collection task
  • add column DHCPOptionSetID to VPC model ( to be able to link to the previous resource)
  • migration file additions for above
  • aws_orphan_dhcp_option_set view to filter on DHCP option sets with missing VPCs

Which issue(s) this PR fixes:
Fixes #

Special notes for your reviewer:
Not all DHCP option set information has been persisted. For example, tags and specific configurations, which are also dynamic (key-value strings) aren't collected, as they aren't needed for determining orphan resources. They can be added on a later iteration if needed.

Release note:

AWS DHCP option set

@Bobi-Wan Bobi-Wan requested a review from a team as a code owner September 11, 2025 13:22
@gardener-robot gardener-robot added the needs/review Needs review label Sep 11, 2025
…ion sets.

add metric description for DHCP option sets.
Register the task handler
@gardener-robot gardener-robot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. needs/second-opinion Needs second review by someone else labels Sep 11, 2025
@Bobi-Wan Bobi-Wan force-pushed the feat/aws-dhcp-option-sets branch from 6e2ce31 to ffafa88 Compare September 11, 2025 13:22
aws dhcp optionsets
aws vpcs
openstack pools
Comment thread pkg/aws/models/models.go
@gardener-robot gardener-robot added the needs/changes Needs (more) changes label Sep 16, 2025
Copy link
Copy Markdown
Member

@dnaeon dnaeon left a comment

Choose a reason for hiding this comment

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

/lgtm

@gardener-robot gardener-robot added reviewed/lgtm Has approval for merging and removed needs/changes Needs (more) changes needs/review Needs review needs/second-opinion Needs second review by someone else labels Sep 16, 2025
@Bobi-Wan Bobi-Wan merged commit 337af12 into gardener:main Sep 16, 2025
4 checks passed
@gardener-robot gardener-robot added the status/closed Issue is closed (either delivered or triaged) label Sep 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

reviewed/lgtm Has approval for merging size/L Denotes a PR that changes 100-499 lines, ignoring generated files. status/closed Issue is closed (either delivered or triaged)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants