Skip to content

chore: add bundle yaml for ambient integration#1414

Open
NohaIhab wants to merge 8 commits intomainfrom
kf-8459-chore-update-latest-edge-ambient
Open

chore: add bundle yaml for ambient integration#1414
NohaIhab wants to merge 8 commits intomainfrom
kf-8459-chore-update-latest-edge-ambient

Conversation

@NohaIhab
Copy link
Contributor

@NohaIhab NohaIhab commented Mar 2, 2026

Closes #1402

Should be aligned with https://github.com/canonical/charmed-kubeflow-solutions/tree/track/1.11/modules/kubeflow-ambient

Summary

  • Replaced Istio Pilot and Gateway with new Istio charms (istio-beacon-k8s, istio-ingress-k8s, and istio-k8s), updated their channels and repo references, and removed Knative-related charms.
  • Added new relations to support the updated Istio service mesh and ingress configuration, replacing old ingress and gateway relations with new service mesh and gateway metadata relations.
  • Updated kubeflow-profiles options to specify the new Istio gateway service account and enable ambient mesh mode.
  • Increased the scale of katib-db and kfp-db from 1 to 3, this matches the recommendation by mysql team
  • Added the kubeflow-trainer application
  • Changed kserve-controller deployment mode from serverless to rawdeployment.
  • Added trust: true to the minio application, now required from the service mesh lib

Testing

  • Install and setup juju and microk8s, tutorial steps from this commit can be followed
  • Clone this repo and checkout the branch
git clone https://github.com/canonical/bundle-kubeflow.git
cd bundle-kubeflow
git checkout kf-8459-chore-update-latest-edge-ambient
  • Add kubeflow model
juju add-model kubeflow
  • Deploy the bundle from the branch
cd releases/latest/edge
juju deploy ./bundle.yaml --trust
  • Wait until all charms are active
  • Run the UATs from main branch
  git clone https://github.com/canonical/charmed-kubeflow-uats.git ~/charmed-kubeflow-uats
  cd ~/charmed-kubeflow-uats
  git checkout main
  tox -e kubeflow-local -- --include-kubeflow-trainer-tests

@NohaIhab NohaIhab requested a review from MattiaSarti March 13, 2026 18:25
@NohaIhab NohaIhab marked this pull request as ready for review March 13, 2026 18:25
@NohaIhab NohaIhab requested a review from dariofaccin March 13, 2026 18:33
Copy link

@MattiaSarti MattiaSarti left a comment

Choose a reason for hiding this comment

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

question: was this tested, even if just manually? If so, can we add the commands to reproduce the testing to the pull request's description?

@NohaIhab
Copy link
Contributor Author

Hi @MattiaSarti, yes, I tested it on testfilnger and added the instructions to the description. Good suggestion!

@NohaIhab NohaIhab requested a review from MattiaSarti March 17, 2026 13:24
Copy link

@MattiaSarti MattiaSarti left a comment

Choose a reason for hiding this comment

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

Hi @MattiaSarti, yes, I tested it on testfilnger and added the instructions to the description. Good suggestion!

Thanks, @NohaIhab, that's perfect, then!

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.

Update the latest/edge bundle.yaml to include ambient integration

2 participants