Skip to content

feat: Update manifests to match upstream kfp 2.16.0 release - #203

Merged
mvlassis merged 6 commits into
mainfrom
kf-8571-upgrade-manifests
Apr 14, 2026
Merged

mvlassis merged 6 commits into
mainfrom
kf-8571-upgrade-manifests

Conversation

@mvlassis

@mvlassis mvlassis commented Apr 2, 2026

Copy link
Copy Markdown
Contributor

Ref: canonical/kfp-operators#869

This PR updates the charms according to the update to the manifests files corresponding to the 2.16.0 release of Kubeflow pipelines. This task is part of the team's upgrade to Kubeflow 26.03.

The diff between the 2 version was generated by following the instructions on the upstream README. It can be found here.

@github-actions github-actions Bot added the Libraries: Out of sync The charm libs used are out-of-sync label Apr 2, 2026
@mvlassis mvlassis changed the title feat: Update manifests feat: Update manifests to match upstream kfp 2.16.0 release Apr 2, 2026
@mvlassis
mvlassis requested review from NohaIhab and misohu April 2, 2026 10:29
@mvlassis
mvlassis requested review from a team and removed request for a team April 3, 2026 17:08
@mvlassis
mvlassis requested a review from MattiaSarti April 6, 2026 09:12

@MattiaSarti MattiaSarti left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

suggestion (blocking): while I understand the necessity of changing user ID and group ID with the upstream image, I would not go for root and I would try some other ID, instead, if it was possible to avoid the temporary changes to run it as root even on latest/edge. For example, have we tried with 1000, like they do upstream? They have securityContext.runAsNonRoot set to true, so I don't expect root privileges to be necessary.

@mvlassis

mvlassis commented Apr 7, 2026

Copy link
Copy Markdown
Contributor Author

It seems that running as user 1000 creates a permission error on the Pebble side: https://github.com/canonical/envoy-operator/actions/runs/24070484106/job/70206016603?pr=203

@MattiaSarti

MattiaSarti commented Apr 7, 2026

Copy link
Copy Markdown
Contributor

juju.errors.JujuError: {"error":"cannot upload charm: invalid charm archive: parsing containers: container "envoy" has invalid uid 1000: uid cannot be in reserved range 1000-9999","error-code":"bad request"}

@mvlassis, then feel free to use root, it's an intrinsic limitation of Juju, as confirmed from the error's logs:

juju.errors.JujuError: {"error":"cannot upload charm: invalid charm archive: parsing containers: container "envoy" has invalid uid 1000: uid cannot be in reserved range 1000-9999","error-code":"bad request"}

This reverts commit cfd1f24.
@mvlassis
mvlassis merged commit 08fa585 into main Apr 14, 2026
31 of 33 checks passed
@mvlassis
mvlassis deleted the kf-8571-upgrade-manifests branch April 14, 2026 08:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Libraries: Out of sync The charm libs used are out-of-sync

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants