Skip to content

chore: integrate rock image admission-webhook:1.10.0-c537efa-20260901050257 - #273

Open
ckfbot wants to merge 1 commit into
track/1.10from
integrate-admission-webhook-1.10.0-c537efa-20260901050257
Open

ckfbot wants to merge 1 commit into
track/1.10from
integrate-admission-webhook-1.10.0-c537efa-20260901050257

Conversation

@ckfbot

@ckfbot ckfbot commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

This PR was opened automatically by the charmed-analytics-ci library as part of the Rock CI system after the rock image was built and published.

🔗 Triggered by upstream PR: canonical/kubeflow-notebook-rocks#271

🔧 Updated Rock References

The following image paths were updated:

  • File: metadata.yaml
    • Path: resources.oci-image.upstream-source

@ckfbot
ckfbot requested a review from a team as a code owner September 1, 2026 05:13
@NohaIhab
NohaIhab requested a review from a team September 1, 2026 11:51

@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.

We may have a problem. The last commit on track 1.10 of the rock was 84088ea and for that commit the latest rock rebuild tag seems to be 1.10.0-84088ea-20260901050926. What's more, the pull request descriptions says this originated from https://github.com/canonical/kubeflow-notebook-rocks, not https://github.com/canonical/kubeflow-dashboard-rocks, and I believe it's because the latter still has the rock definition for track/1.10, while we formally moved the rock definition to the former, so now both repositories have a competing branch for the same track of the same rock!

@dariofaccin

Copy link
Copy Markdown
Contributor

@MattiaSarti @mvlassis Mattia is right here. We need to remove the dashboard-related rocks from the track/1.10 as well.

@mvlassis

mvlassis commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

Very good catch @MattiaSarti, raised canonical/kubeflow-notebook-rocks#337 for it

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.

5 participants