Skip to content

ci: added openshift certified operators repo to release job#1635

Merged
vrutkovs merged 2 commits into
masterfrom
add-certified-openshift-repo
Dec 5, 2025
Merged

ci: added openshift certified operators repo to release job#1635
vrutkovs merged 2 commits into
masterfrom
add-certified-openshift-repo

Conversation

@AndrewChubatiuk
Copy link
Copy Markdown
Contributor

added openshift certified operators repo to release job

@vrutkovs
Copy link
Copy Markdown
Collaborator

iiuc our operator is not yet conformant with Red Hat Certified Operators collection, but once we update it this PR would be very useful. Lets keep it open for some time

@AndrewChubatiuk AndrewChubatiuk force-pushed the add-certified-openshift-repo branch 2 times, most recently from 8ddba66 to e4ce2ce Compare December 4, 2025 19:03
@AndrewChubatiuk AndrewChubatiuk force-pushed the add-certified-openshift-repo branch from e4ce2ce to 545fc41 Compare December 4, 2025 21:23
Comment thread Makefile
$(KUSTOMIZE) build config/crd/overlay > dist/crd.yaml

olm: operator-sdk opm yq docs
$(eval DIGEST = $(shell $(CONTAINER_TOOL) buildx imagetools inspect $(REGISTRY)/$(ORG)/$(REPO):$(TAG)-ubi --format "{{print .Manifest.Digest}}"))
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

We should make sure it returns the manifest list digest instead of arch-specific manifest

Copy link
Copy Markdown
Contributor Author

@AndrewChubatiuk AndrewChubatiuk Dec 4, 2025

Choose a reason for hiding this comment

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

it does return list digest, you can recheck if you want

@vrutkovs vrutkovs merged commit 86226d5 into master Dec 5, 2025
6 checks passed
@vrutkovs vrutkovs deleted the add-certified-openshift-repo branch December 5, 2025 10:20
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.

2 participants