Skip to content

AGENT-500: Add configurable step to oc-mirror to generate tarfile - #1497

Merged
openshift-merge-robot merged 5 commits into
openshift-metal3:masterfrom
bfournie:oc-mirror-to-file
Jan 18, 2023
Merged

AGENT-500: Add configurable step to oc-mirror to generate tarfile#1497
openshift-merge-robot merged 5 commits into
openshift-metal3:masterfrom
bfournie:oc-mirror-to-file

Conversation

@bfournie

@bfournie bfournie commented Dec 21, 2022

Copy link
Copy Markdown
Contributor

Add the ability to use oc-mirror to create a mirror registry by generating a tarfile first and then publishing the tarfile to the registry. The can be used to test a common use case for disconnected installs.

Add the ability to use oc-mirror to create a mirror registry by
generating a tarfile first and then publishing the tarfile to the
registry. The can be used to test a common use case for disconnected
installs.

In addition, a function to create a tarfile with only images needed
to bootstrap a host is included. This will be used for testing future
functionality to remove the external registry.
@openshift-ci
openshift-ci Bot requested review from flaper87 and sadasu December 21, 2022 16:55
@bfournie bfournie changed the title Add configurable step to oc-mirror to generate tarfile AGENT-500: Add configurable step to oc-mirror to generate tarfile Jan 4, 2023
Comment thread oc_mirror.sh
Comment thread oc_mirror.sh Outdated
Comment thread oc_mirror.sh
Comment thread oc_mirror.sh Outdated
Comment thread oc_mirror.sh
@andfasano

Copy link
Copy Markdown
Member

/approve

@openshift-ci

openshift-ci Bot commented Jan 13, 2023

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: andfasano

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

The pull request process is described here

Details 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

@openshift-ci openshift-ci Bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jan 13, 2023

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

/lgtm

@openshift-ci openshift-ci Bot added the lgtm Indicates that a PR is ready to be merged. label Jan 16, 2023
@bfournie

Copy link
Copy Markdown
Contributor Author

/retest

@bfournie

Copy link
Copy Markdown
Contributor Author

/test e2e-metal-ipi-serial-ipv4

@openshift-ci

openshift-ci Bot commented Jan 18, 2023

Copy link
Copy Markdown

@bfournie: The specified target(s) for /test were not found.
The following commands are available to trigger required jobs:

  • /test e2e-agent-compact
  • /test e2e-metal-ipi-bm
  • /test e2e-metal-ipi-ovn-ipv6
  • /test e2e-metal-ipi-serial-ipv4
  • /test images

The following commands are available to trigger optional jobs:

  • /test e2e-agent-sno-ipv6
  • /test e2e-metal-ipi-bm-bond
  • /test e2e-metal-ipi-ovn-dualstack
  • /test e2e-metal-ipi-serial-ovn-ipv6
  • /test e2e-metal-ipi-upgrade
  • /test e2e-metal-ipi-upgrade-ovn-ipv6
  • /test e2e-metal-ipi-virtualmedia

Use /test all to run the following jobs that were automatically triggered:

  • pull-ci-openshift-metal3-dev-scripts-master-e2e-agent-compact
  • pull-ci-openshift-metal3-dev-scripts-master-e2e-agent-sno-ipv6
  • pull-ci-openshift-metal3-dev-scripts-master-e2e-metal-ipi-bm
  • pull-ci-openshift-metal3-dev-scripts-master-e2e-metal-ipi-bm-bond
  • pull-ci-openshift-metal3-dev-scripts-master-e2e-metal-ipi-ovn-dualstack
  • pull-ci-openshift-metal3-dev-scripts-master-e2e-metal-ipi-ovn-ipv6
  • pull-ci-openshift-metal3-dev-scripts-master-e2e-metal-ipi-serial-ipv4
  • pull-ci-openshift-metal3-dev-scripts-master-e2e-metal-ipi-serial-ovn-ipv6
  • pull-ci-openshift-metal3-dev-scripts-master-e2e-metal-ipi-virtualmedia
  • pull-ci-openshift-metal3-dev-scripts-master-images
Details

In response to this:

/test ci/prow/e2e-metal-ipi-ovn-ipv6

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@bfournie

Copy link
Copy Markdown
Contributor Author

/test e2e-metal-ipi-ovn-ipv6

@bfournie

Copy link
Copy Markdown
Contributor Author

/retest-required

@bfournie

Copy link
Copy Markdown
Contributor Author

Both e2e-metal-ipi-serial-ipv4 and e2e-metal-ipi-bm have passed multiple times on this PR after the last commit as can be seen in the PR history, however each time a retest is run invariably one of them fails due a random issue. The failure is completely unrelated to this PR which has added a code which is not even exercised. Attempting to override after the last failure
/override e2e-metal-ipi-serial-ipv4

@openshift-ci

openshift-ci Bot commented Jan 18, 2023

Copy link
Copy Markdown

@bfournie: /override requires failed status contexts, check run or a prowjob name to operate on.
The following unknown contexts/checkruns were given:

  • e2e-metal-ipi-serial-ipv4

Only the following failed contexts/checkruns were expected:

  • ci/prow/e2e-agent-compact
  • ci/prow/e2e-agent-sno-ipv6
  • ci/prow/e2e-metal-ipi-bm
  • ci/prow/e2e-metal-ipi-bm-bond
  • ci/prow/e2e-metal-ipi-ovn-dualstack
  • ci/prow/e2e-metal-ipi-ovn-ipv6
  • ci/prow/e2e-metal-ipi-serial-ipv4
  • ci/prow/e2e-metal-ipi-serial-ovn-ipv6
  • ci/prow/e2e-metal-ipi-virtualmedia
  • ci/prow/images
  • pull-ci-openshift-metal3-dev-scripts-master-e2e-agent-compact
  • pull-ci-openshift-metal3-dev-scripts-master-e2e-agent-sno-ipv6
  • pull-ci-openshift-metal3-dev-scripts-master-e2e-metal-ipi-bm
  • pull-ci-openshift-metal3-dev-scripts-master-e2e-metal-ipi-bm-bond
  • pull-ci-openshift-metal3-dev-scripts-master-e2e-metal-ipi-ovn-dualstack
  • pull-ci-openshift-metal3-dev-scripts-master-e2e-metal-ipi-ovn-ipv6
  • pull-ci-openshift-metal3-dev-scripts-master-e2e-metal-ipi-serial-ipv4
  • pull-ci-openshift-metal3-dev-scripts-master-e2e-metal-ipi-serial-ovn-ipv6
  • pull-ci-openshift-metal3-dev-scripts-master-e2e-metal-ipi-virtualmedia
  • pull-ci-openshift-metal3-dev-scripts-master-images
  • tide

If you are trying to override a checkrun that has a space in it, you must put a double quote on the context.

Details

In response to this:

Both e2e-metal-ipi-serial-ipv4 and e2e-metal-ipi-bm have passed multiple times on this PR after the last commit as can be seen in the PR history, however each time a retest is run invariably one of them fails due a random issue. The failure is completely unrelated to this PR which has added a code which is not even exercised. Attempting to override after the last failure
/override e2e-metal-ipi-serial-ipv4

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@bfournie

Copy link
Copy Markdown
Contributor Author

/override ci/prow/e2e-metal-ipi-serial-ipv4

@openshift-ci

openshift-ci Bot commented Jan 18, 2023

Copy link
Copy Markdown

@bfournie: Overrode contexts on behalf of bfournie: ci/prow/e2e-metal-ipi-serial-ipv4

Details

In response to this:

/override ci/prow/e2e-metal-ipi-serial-ipv4

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@openshift-merge-robot
openshift-merge-robot merged commit 271c66f into openshift-metal3:master Jan 18, 2023
@bfournie
bfournie deleted the oc-mirror-to-file branch January 18, 2023 20:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants