Skip to content

[MLRun] Add ingressClassName to Ingress manifests#1059

Merged
alexpirogovski merged 1 commit intov3io:developmentfrom
alexpirogovski:add_ingressClassName
Feb 12, 2025
Merged

[MLRun] Add ingressClassName to Ingress manifests#1059
alexpirogovski merged 1 commit intov3io:developmentfrom
alexpirogovski:add_ingressClassName

Conversation

@alexpirogovski
Copy link
Collaborator

Checklist

[Place an '[x]' (no spaces) in all applicable fields. Please remove irrelevant fields.]

  • Chart Version bumped
  • Title of the PR starts with chart name (e.g. [mychartname])

Description

Since the ingressClass steering through annotations is deprecated, the ingressClassName field is now added to Ingress templates and exposed via Values to set the desired ingressClass. Default is an empty string for compatibility.

@alexpirogovski alexpirogovski changed the title [mlrun] Add ingressClassName to Ingress manifests [MLRun] Add ingressClassName to Ingress manifests Feb 12, 2025
Copy link
Collaborator

@liranbg liranbg left a comment

Choose a reason for hiding this comment

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

bump chart version as well

@alexpirogovski alexpirogovski merged commit bd127e2 into v3io:development Feb 12, 2025
22 checks passed
@alexpirogovski alexpirogovski deleted the add_ingressClassName branch February 12, 2025 11:38
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.

2 participants