search

canonical / kubeflow-dashboard-operator

Operator for Kubeflow Dashboard
Apache License 2.0
1 stars 1 forks source link

Charm's resource is not up to date in `ch:1.8/edge` and `ch:latest/edge` #194

Closed DnPlas closed 1 month ago

DnPlas commented 1 month ago

Bug Description

The latest/edge and 1.8/edge Charmhub tracks have oci-image r683 attached, but this image corresponds to the charmedkubeflow/kubeflow-central-dashboard:1.8-0719441 container image and not to docker.io/kubeflownotebookswg/centraldashboard:v1.8.0, which is the expected one given the changes in #191 and #192.

To Reproduce

Deploying

  1. Deploy the charm from one of the tracks mentioned in the description
  2. Exec into the workload container and check for the __daemon__ user (this is added by the rock, but not by the upstream image)
  3. Confirm the image comes from the rock, despite the HEAD of gh:track/1.8 and gh:main point to the upstream image

Checking the publish job logs

  1. Observe the latest publish job is attaching resource r683
  2. Observe a previous publish job is also attaching the same resource, which could mean it is not updating it

Environment

CI environment.

Relevant Log Output

N/A

Additional Context

No response

syncronize-issues-to-jira[bot] commented 1 month ago

Thank you for reporting us your feedback!

The internal ticket has been created: https://warthogs.atlassian.net/browse/KF-5770.

This message was autogenerated

mvlassis commented 1 month ago

@DnPlas and I tried to manually upload the latest revision of oci-image using the docker.io/kubeflownotebookswg/centraldashboard:v1.8.0 image, but we get the following error trying to upload the charm to Charmhub:

Store operation failed:
- None: No compatible resource and package revisions in release request for charm 'ScuThnuIqjZRpljgOzuhZxluTu9aDspc' to channel edge
mvlassis commented 1 month ago

We uploaded a new revision of the oci-image (r686), and the Public Charm action works correctly. The compatibility errors seem to stem from the previous latest revision's architecture: it was arm64 instead of all. No code changes are needed.