Skip to content

ipam: Remove obsolete fields from cloud provider IPAM types#45138

Open
HadrienPatte wants to merge 1 commit intomainfrom
pr/HadrienPatte/ipam-deprecated-fields
Open

ipam: Remove obsolete fields from cloud provider IPAM types#45138
HadrienPatte wants to merge 1 commit intomainfrom
pr/HadrienPatte/ipam-deprecated-fields

Conversation

@HadrienPatte
Copy link
Copy Markdown
Member

Remove deprecated fields that have been unused for 5+ years:

Remove long deprecated cloud provider IPAM fields `spec.eni.instance-id`, `spec.eni.min-allocate`, `spec.eni.pre-allocate`, `spec.eni.max-above-watermark` and `status.azure.interfaces[].GatewayIP`.

@maintainer-s-little-helper maintainer-s-little-helper bot added the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Apr 2, 2026
@HadrienPatte HadrienPatte added area/ipam IP address management, including cloud IPAM release-note/minor This PR changes functionality that users may find relevant to operating Cilium. labels Apr 2, 2026
@maintainer-s-little-helper maintainer-s-little-helper bot removed the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Apr 2, 2026
Remove deprecated fields that have been unused for 5+ years:
* Azure `AzureInterface.GatewayIP` (obsolete since v1.10, see #15182)
* AWS `ENISpec.InstanceID` (obsolete since v1.8, see #10569)
* AWS `ENISpec.{MinAllocate,PreAllocate,MaxAboveWatermark}` (obsolete since v1.8, see #10089)

Signed-off-by: Hadrien Patte <hadrien.patte@datadoghq.com>
@HadrienPatte HadrienPatte force-pushed the pr/HadrienPatte/ipam-deprecated-fields branch from c3c61e4 to e45b778 Compare April 2, 2026 12:19
@HadrienPatte
Copy link
Copy Markdown
Member Author

/test

@HadrienPatte HadrienPatte marked this pull request as ready for review April 2, 2026 14:50
@HadrienPatte HadrienPatte requested review from a team as code owners April 2, 2026 14:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/ipam IP address management, including cloud IPAM release-note/minor This PR changes functionality that users may find relevant to operating Cilium.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant