Skip to content

Fix broken ProviderConfig schema in extension-provider-aws#687

Merged
Bobi-Wan merged 2 commits intogardener:mainfrom
Bobi-Wan:fix/provider-aws-broken-providerconfig
Nov 5, 2025
Merged

Fix broken ProviderConfig schema in extension-provider-aws#687
Bobi-Wan merged 2 commits intogardener:mainfrom
Bobi-Wan:fix/provider-aws-broken-providerconfig

Conversation

@Bobi-Wan
Copy link
Copy Markdown
Contributor

@Bobi-Wan Bobi-Wan commented Nov 4, 2025

What this PR does / why we need it:
Fixes the following bug:
Bug:
While strict decoding a providerConfig (defined in aws extension),
we encounter a field name change (capabilitySets -> capabilityFlavours).
Even though we don't use it, the strict decoder fails and reports an error.

PR breaking the current functionality:
gardener/gardener-extension-provider-aws#1480
Fix:
Bump to provider extension aws, so the expencted
cloudprofile.spec.providerconfig has the same schema as the
ones in the dev landscape.

Dependency changes all come from

go get github.com/gardener/gardener-extension-provider-aws@latest
go mod tidy

Special notes for your reviewer:
I am expecting something similar to pop up in other extensions as well,
as the change is triggered by a public API change in gardener/gardener:
gardener/gardener-extension-provider-aws#1480

Release note:

Match changed public API in gardener-extension-provider-aws

@Bobi-Wan Bobi-Wan requested a review from a team as a code owner November 4, 2025 12:25
@gardener-robot gardener-robot added needs/review Needs review size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Nov 4, 2025
…et fix for breaking spec

Bug:
While strict decoding a providerConfig (defined in aws extension),
we encounter a field name change (capabilitySets -> capabilityFlavours).
Even though we don't use it, the strict decoder fails and reports an error.

PR breaking the current functionality:
gardener/gardener-extension-provider-aws#1480

Fix: bump to provider extension aws, so the expencted
cloudprofile.spec.providerconfig has the same schema as the
ones in the dev landscape.
@Bobi-Wan Bobi-Wan force-pushed the fix/provider-aws-broken-providerconfig branch from 337ac9e to 65075d2 Compare November 5, 2025 08:19
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/review Needs review labels Nov 5, 2025
@Bobi-Wan Bobi-Wan merged commit 9361b95 into gardener:main Nov 5, 2025
4 checks passed
@gardener-robot gardener-robot added the status/closed Issue is closed (either delivered or triaged) label Nov 5, 2025
@Bobi-Wan Bobi-Wan deleted the fix/provider-aws-broken-providerconfig branch January 7, 2026 09:07
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/M Denotes a PR that changes 30-99 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