Skip to content

Conversation

noalimoy
Copy link

@noalimoy noalimoy commented Sep 29, 2025

Summary

This PR migrates the GitHub Actions workflow responsible for building and publishing the PVCViewer Controller Docker image from the kubeflow/kubeflow repository to the kubeflow/notebooks repository.

Related: #619

Verification

Successfully built and published a test image:
GitHub Actions run

workflow-success

Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign thesuperzapper for approval. For more information see the Kubernetes Code Review Process.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@noalimoy noalimoy force-pushed the feature/migrate_pvcviewer_controller_docker_publish_to_notebooks branch 2 times, most recently from 425e4da to 53689ac Compare September 29, 2025 12:31
@andyatmiami
Copy link
Contributor

/hold

I want to make sure:

  • we finalize the decision on container registry
  • we appropriately "seed" a package to appropriate container registry so desired permissions are in place

@noalimoy noalimoy force-pushed the feature/migrate_pvcviewer_controller_docker_publish_to_notebooks branch from 53689ac to 0636c9a Compare September 30, 2025 09:10
@google-oss-prow google-oss-prow bot added area/controller area - related to controller components area/v1 area - version - kubeflow notebooks v1 labels Sep 30, 2025
@noalimoy noalimoy force-pushed the feature/migrate_pvcviewer_controller_docker_publish_to_notebooks branch 3 times, most recently from 88afc72 to 104bfe1 Compare September 30, 2025 10:28
@noalimoy noalimoy force-pushed the feature/migrate_pvcviewer_controller_docker_publish_to_notebooks branch from 104bfe1 to 832a156 Compare September 30, 2025 10:32
@noalimoy noalimoy marked this pull request as ready for review September 30, 2025 11:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/ci area - related to ci area/controller area - related to controller components area/v1 area - version - kubeflow notebooks v1 do-not-merge/hold size/M

Projects

Status: Needs Triage

Development

Successfully merging this pull request may close these issues.

2 participants