Closed jiridanek closed 3 months ago
Hi @jiridanek. Thanks for your PR.
I'm waiting for a opendatahub-io member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test
on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test
label.
I understand the commands that are listed here.
Here's how it runs, https://github.com/jiridanek/notebooks/actions/runs/9206160094
/ok-to-test
/label tide/merge-method-squash
@jiridanek: The label(s) tide/merge-method-squash
cannot be applied, because the repository doesn't have them.
/override ci/prow/images /override ci/prow/notebooks-e2e-tests
@jiridanek: jiridanek unauthorized: /override is restricted to Repo administrators, approvers in top level OWNERS file, and the following github teams:.
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: atheo89
The full list of commands accepted by this bot can be found here.
The pull request process is described here
/test notebook-jupyter-intel-pyt-ubi9-python-3-9-pr-image-mirror /test notebook-jupyter-intel-tf-ubi9-python-3-9-pr-image-mirror /test images
/override ci/prow/notebooks-e2e-tests
as it is failing on issue unrelated to GitHub Actions
@jiridanek: jiridanek unauthorized: /override is restricted to Repo administrators, approvers in top level OWNERS file, and the following github teams:.
/override ci/prow/notebooks-e2e-tests
@atheo89: Overrode contexts on behalf of atheo89: ci/prow/notebooks-e2e-tests
/retest
New changes are detected. LGTM label has been removed.
/override ci/prow/notebooks-e2e-tests
@atheo89: Overrode contexts on behalf of atheo89: ci/prow/notebooks-e2e-tests
https://issues.redhat.com/browse/RHOAIENG-7525
Description
build only images that changed compared to the PR target branch and images that depend on theseHow Has This Been Tested?
See GHA run
on this PR, which is slow because opendatahub-io/notebooks does not yet have container repository cache populated
on a PR inside my own repo, which is the same as above except fast https://github.com/jiridanek/notebooks/actions/runs/9322564678
and on a branch inside my own repo, which is able to push as well as pull into the cache https://github.com/jiridanek/notebooks/actions/runs/9322194970
Merge criteria:
[ ] The commits are squashed in a cohesive manner and have meaningful messages.
[x] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[x] The developer has manually tested the changes and verified that the changes work