Skip to content

Add Red Hat approved OpenStack operator icon#452

Open
posikoya wants to merge 1 commit into
openstack-k8s-operators:mainfrom
posikoya:posikoya/fix-test-operator-icon
Open

Add Red Hat approved OpenStack operator icon#452
posikoya wants to merge 1 commit into
openstack-k8s-operators:mainfrom
posikoya:posikoya/fix-test-operator-icon

Conversation

@posikoya

@posikoya posikoya commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

Added the official, Red Hat-approved OpenStack operator icon to the ClusterServiceVersion manifest. The asset was sourced directly from an approved Red Hat site and encoded to Base64 to ensure the operator displays the logo correctly in the OperatorHub UI.

used the link from: openstack-k8s-operators/openstack-operator#914 (comment) to render the image

image

Jira: OSPRH-12506

@posikoya posikoya self-assigned this Jun 9, 2026
@openshift-ci openshift-ci Bot requested review from evallesp and lpiwowar June 9, 2026 15:15
@posikoya posikoya force-pushed the posikoya/fix-test-operator-icon branch from 11b86aa to e99781c Compare June 9, 2026 15:43
@centosinfra-prod-github-app

Copy link
Copy Markdown

Build failed (check pipeline). Post recheck (without leading slash)
to rerun all jobs. Make sure the failure cause has been resolved before
you rerun jobs.

https://gateway-cloud-softwarefactory.apps.ocp.cloud.ci.centos.org/zuul/t/rdoproject.org/buildset/432c4b63a7c940069c57f02841286009

✔️ openstack-k8s-operators-content-provider SUCCESS in 3h 00m 30s
test-operator-kuttl FAILURE in 49m 21s
✔️ podified-multinode-edpm-deployment-crc-test-operator SUCCESS in 2h 42m 33s

@posikoya posikoya force-pushed the posikoya/fix-test-operator-icon branch from e99781c to 5d7a7d0 Compare June 10, 2026 09:26
@centosinfra-prod-github-app

Copy link
Copy Markdown

Build failed (check pipeline). Post recheck (without leading slash)
to rerun all jobs. Make sure the failure cause has been resolved before
you rerun jobs.

https://gateway-cloud-softwarefactory.apps.ocp.cloud.ci.centos.org/zuul/t/rdoproject.org/buildset/5a919e5af7764ecfa122ca9fe283b6f7

✔️ openstack-k8s-operators-content-provider SUCCESS in 3h 04m 42s
test-operator-kuttl FAILURE in 49m 13s
✔️ podified-multinode-edpm-deployment-crc-test-operator SUCCESS in 2h 36m 44s

@posikoya

Copy link
Copy Markdown
Contributor Author

recheck

@centosinfra-prod-github-app

Copy link
Copy Markdown

Build failed (check pipeline). Post recheck (without leading slash)
to rerun all jobs. Make sure the failure cause has been resolved before
you rerun jobs.

https://gateway-cloud-softwarefactory.apps.ocp.cloud.ci.centos.org/zuul/t/rdoproject.org/buildset/014e0e1c2afd4680ae32e4e422322eb7

✔️ openstack-k8s-operators-content-provider SUCCESS in 2h 46m 26s
test-operator-kuttl FAILURE in 50m 18s
podified-multinode-edpm-deployment-crc-test-operator FAILURE in 2h 30m 35s

@posikoya posikoya force-pushed the posikoya/fix-test-operator-icon branch 2 times, most recently from 23ef8ae to 53e38d1 Compare June 15, 2026 11:45
@centosinfra-prod-github-app

Copy link
Copy Markdown

Build failed (check pipeline). Post recheck (without leading slash)
to rerun all jobs. Make sure the failure cause has been resolved before
you rerun jobs.

https://gateway-cloud-softwarefactory.apps.ocp.cloud.ci.centos.org/zuul/t/rdoproject.org/buildset/f0f5c3b11e34494eb687b442cab423b3

✔️ openstack-k8s-operators-content-provider SUCCESS in 2h 48m 03s
test-operator-kuttl FAILURE in 47m 16s
✔️ podified-multinode-edpm-deployment-crc-test-operator SUCCESS in 2h 32m 33s

@Valkyrie00

Copy link
Copy Markdown
Contributor

LGTM but I'm waiting first for this review #454

@stuggi

stuggi commented Jun 16, 2026

Copy link
Copy Markdown
Contributor

like discussed with @posikoya yesterday on slack and commented on https://redhat.atlassian.net/browse/OSPRH-12506?focusedCommentId=17278628 , isn’t this obsolet since the test-operator was integrated in the openstack-operator with Add test operator into the openstack-operator-index by lpiwowar · Pull Request #752 · openstack-k8s-operators/openstack-operator and there is no dedicated/standalone install method for the test-operator? from what I see. we do not ship a bundle downstream for test-operator - https://catalog.redhat.com/en/search?q=test-operator .

@posikoya

Copy link
Copy Markdown
Contributor Author

recheck

Added the official, Red Hat-approved OpenStack operator icon to the
ClusterServiceVersion manifest. The asset was sourced directly from an
approved Red Hat site and encoded to Base64 to ensure the operator displays
the logo correctly in the OperatorHub UI.
@posikoya posikoya force-pushed the posikoya/fix-test-operator-icon branch from 53e38d1 to 90c2c39 Compare June 16, 2026 11:03
@stuggi

stuggi commented Jun 16, 2026

Copy link
Copy Markdown
Contributor

like discussed with @posikoya yesterday on slack and commented on https://redhat.atlassian.net/browse/OSPRH-12506?focusedCommentId=17278628 , isn’t this obsolet since the test-operator was integrated in the openstack-operator with Add test operator into the openstack-operator-index by lpiwowar · Pull Request #752 · openstack-k8s-operators/openstack-operator and there is no dedicated/standalone install method for the test-operator? from what I see. we do not ship a bundle downstream for test-operator - https://catalog.redhat.com/en/search?q=test-operator .

added the hold label as I think we should not need/land it, if I am wrong let me know.

@imatza-rh

Copy link
Copy Markdown

/lgtm
/approve

@openshift-ci

openshift-ci Bot commented Jun 16, 2026

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is NOT APPROVED

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

The full list of commands accepted by this bot can be found 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

@stuggi

stuggi commented Jun 16, 2026

Copy link
Copy Markdown
Contributor

/lgtm /approve

@imatza-rh please check my previous comment

@imatza-rh

Copy link
Copy Markdown

/lgtm /approve

@imatza-rh please check my previous comment

Hey, I have no strong opinion from my side. The code change itself is lgtm (valid SVG, correct YAML placement), but I'll defer to you and @posikoya on whether it's needed given the downstream bundling situation.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants