Skip to content

fix(charts): correct webhook configuration scope and metadata#258

Merged
unmarshall merged 1 commit into
ai-dynamo:mainfrom
shmuel-runai:grove-33813/main-fix-webhook-charts
Nov 17, 2025
Merged

fix(charts): correct webhook configuration scope and metadata#258
unmarshall merged 1 commit into
ai-dynamo:mainfrom
shmuel-runai:grove-33813/main-fix-webhook-charts

Conversation

@shmuel-runai

@shmuel-runai shmuel-runai commented Nov 12, 2025

Copy link
Copy Markdown
Contributor

What type of PR is this?

/kind bug

What this PR does / why we need it:

  • Remove invalid namespace field from webhook config metadata
  • Change scope from '*' to 'Namespaced' for target resources
  • Affects PCS validating/defaulting webhooks and authorizer webhook

Which issue(s) this PR fixes:

Special notes for your reviewer:

Does this PR introduce a API change?

NONE

Additional documentation e.g., enhancement proposals, usage docs, etc.:

- Remove invalid namespace field from webhook config metadata
- Change scope from '*' to 'Namespaced' for target resources
- Affects PCS validating/defaulting webhooks and authorizer webhook
@unmarshall unmarshall merged commit 5bb02dc into ai-dynamo:main Nov 17, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants