Skip to content

OPENSHIFTP-688: Download sigfile#333

Open
mpkredhat wants to merge 1 commit intoredhat-cop:mainfrom
mpkredhat:br-sigstore
Open

OPENSHIFTP-688: Download sigfile#333
mpkredhat wants to merge 1 commit intoredhat-cop:mainfrom
mpkredhat:br-sigstore

Conversation

@mpkredhat
Copy link

@mpkredhat mpkredhat commented Feb 12, 2026

This PR mirrors the sig file required in disconnected installations.
This step is part of disconnected changes to get inline with current mirroring best practices.
Corresponding Redhat issue : https://issues.redhat.com/browse/OCPBUGS-70297


- name: Mirror the registry
when: setup_registry.autosync_registry
when: setup_registry.autosync_registry and sig_output
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why and sig_output this would fail on 4.20 and earlier versions.

Comment on lines +134 to +135
--force=false
register: sig_output
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

--continue-on-error should be added so this works across release.

Comment on lines +124 to +126
shell: |
oc adm release info | grep Digest: | awk '{print $2}' | sed 's/:/-/g'
register: RELEASE_DIGEST
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
shell: |
oc adm release info | grep Digest: | awk '{print $2}' | sed 's/:/-/g'
register: RELEASE_DIGEST
shell: |
oc adm release info | grep Digest: | awk '{print $2}' | sed 's/:/-/g'
register: RELEASE_DIGEST

Needless space in command

You need to use L126 https://github.com/redhat-cop/ocp4-helpernode/blob/main/tasks/setup_registry.yaml#L126 rather than the current approach in the PR

~hum... how does this work with nightly build? You can download openshift-installer at https://openshift-release-artifacts-ppc64le.apps.ci.l2s4.p1.openshiftapps.com/4.22.0-0.nightly-ppc64le-2026-02-12-194353
And see what openshift-installer outputs.

We might need to check ~

@mpkredhat mpkredhat force-pushed the br-sigstore branch 2 times, most recently from 23e4273 to 8db2eda Compare February 13, 2026 10:51
@prb112
Copy link
Contributor

prb112 commented Feb 14, 2026

Left offline comments

tasks/main.yml Outdated
Comment on lines +46 to +52
- name: Copy mirror-isc.yaml
copy:
src: ../templates/mirror-isc.yaml.j2
dest: /usr/local/src/mirror-isc.yaml
mode: "0655"
owner: root
group: root
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's save this for a different PR...

@mpkredhat mpkredhat changed the title OPENSHIFTP688: Download oc-mirror and sigfile OpenShiftP-688: Download oc-mirror and sigfile Mar 4, 2026
@mpkredhat mpkredhat changed the title OpenShiftP-688: Download oc-mirror and sigfile OPENSHIFTP-688: Download oc-mirror and sigfile Mar 4, 2026
@mpkredhat mpkredhat changed the title OPENSHIFTP-688: Download oc-mirror and sigfile OPENSHIFTP-688: Download sigfile Mar 4, 2026
Signed-off-by: Mandar Kulkarni <mandar.kulkarni6@ibm.com>
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.

3 participants