🌱 Change ExtensionConfig to Cluster scoped

[killianmuldoon]

Signed-off-by: killianmuldoon [email protected]

Update types and runtimeSDK proposal to make the ExtensionConfig Cluster scoped. This was the intention from the design stage, but was never explicit in the proposal, and was incorrectly implemented in the initial version of the type.

/area runtime-sdk

[sbueringer]

/lgtm

Nice catch, good that we talked about it :)

/assign @fabriziopandini

[fabriziopandini]

/lgtm
/approve

[k8s-ci-robot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: fabriziopandini

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

• OWNERS [fabriziopandini]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[enxebre]

does this mean extensions can't be scoped to clusters in a given namespace?

[sbueringer]

does this mean extensions can't be scoped to clusters in a given namespace?

They can be scoped via the namespaceSelector in the ExtensionConfig. It's the same as with MutatingWebhookConfiguration / ValidatingWebhookConfiguration, the CRD itself is cluster-wide but then you can filter via namespaceSelector.