Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ubi9: split based on the branding #2133

Merged
merged 2 commits into from
Jul 19, 2023
Merged

ubi9: split based on the branding #2133

merged 2 commits into from
Jul 19, 2023

Conversation

guits
Copy link
Collaborator

@guits guits commented Jul 17, 2023

this splits ubi9 by creating ceph-releases/<ceph-release>/ubi9-<branding>
given that quincy and reef are both based on ubi9 and require ceph release
specific changes.

Signed-off-by: Guillaume Abrioux gabrioux@ibm.com

@guits guits requested a review from ktdreyer July 17, 2023 14:52
@ktdreyer
Copy link
Member

This reverts some of the change in #2122 where I removed the individual tags for each image. As one example, sed now searches for the v4.10 tag (and it should not. cephadm uses v4.12 now):

-  -e "s|registry.redhat.io/openshift4/ose-prometheus:|cp.icr.io/cp/ibm-ceph/prometheus:|" \
+  -e "s|registry.redhat.io/openshift4/ose-prometheus:v4.10|cp.icr.io/cp/ibm-ceph/prometheus:v4.10|" \

We already line up image tags between IBM and Red Hat's products. It's simpler and more future-proof to only replace the image URLs without touching the tags.

@guits
Copy link
Collaborator Author

guits commented Jul 18, 2023

This reverts some of the change in #2122 where I removed the individual tags for each image. As one example, sed now searches for the v4.10 tag (and it should not. cephadm uses v4.12 now):

-  -e "s|registry.redhat.io/openshift4/ose-prometheus:|cp.icr.io/cp/ibm-ceph/prometheus:|" \
+  -e "s|registry.redhat.io/openshift4/ose-prometheus:v4.10|cp.icr.io/cp/ibm-ceph/prometheus:v4.10|" \

We already line up image tags between IBM and Red Hat's products. It's simpler and more future-proof to only replace the image URLs without touching the tags.

@ktdreyer is that for quincy / ubi9-ibm ?

@guits
Copy link
Collaborator Author

guits commented Jul 18, 2023

@ktdreyer I've updated the branch, your comment should be addressed.

Copy link
Member

@ktdreyer ktdreyer left a comment

Choose a reason for hiding this comment

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

I've composed the main branch (with compose-rhcs.sh), and then diff'd that against this PR's output.

Here are the changes this PR makes:

-  -e "s|registry.redhat.io/rhceph/rhceph-6-rhel9:|cp.icr.io/cp/ibm-ceph/ceph-6-rhel9:|" \
+  -e "s|registry.redhat.io/rhceph/rhceph-7-rhel9:latest|cp.icr.io/cp/ibm-ceph/ceph-7-rhel9:latest|" \
   -e "s|registry.redhat.io/openshift4/ose-prometheus:|cp.icr.io/cp/ibm-ceph/prometheus:|" \
-  -e "s|registry.redhat.io/openshift-logging/logging-loki-rhel8:|cp.icr.io/cp/ibm-ceph/logging-loki-rhel8:|" \
-  -e "s|registry.redhat.io/rhceph/rhceph-promtail-rhel9:|cp.icr.io/cp/ibm-ceph/promtail-rhel9:|" \
   -e "s|registry.redhat.io/openshift4/ose-prometheus-node-exporter:|cp.icr.io/cp/ibm-ceph/prometheus-node-exporter:|" \
-  -e "s|registry.redhat.io/rhceph/rhceph-6-dashboard-rhel9:|cp.icr.io/cp/ibm-ceph/ceph-6-dashboard-rhel9:|" \
+  -e "s|registry.redhat.io/rhceph/rhceph-7-dashboard-rhel9:latest|cp.icr.io/cp/ibm-ceph/ceph-7-dashboard-rhel9:latest|" \
   -e "s|registry.redhat.io/openshift4/ose-prometheus-alertmanager:|cp.icr.io/cp/ibm-ceph/prometheus-alertmanager:|" \
-  -e "s|registry.redhat.io/rhceph/rhceph-haproxy-rhel9:|cp.icr.io/cp/ibm-ceph/haproxy-rhel9:|" \
-  -e "s|registry.redhat.io/rhceph/keepalived-rhel9:|cp.icr.io/cp/ibm-ceph/keepalived-rhel9:|" \
-  -e "s|registry.redhat.io/rhceph/snmp-notifier-rhel9:|cp.icr.io/cp/ibm-ceph/snmp-notifier-rhel9:|" \
+  -e "s|registry.redhat.io/rhceph/rhceph-haproxy-rhel9:latest|cp.icr.io/cp/ibm-ceph/haproxy-rhel9:latest|" \
+  -e "s|registry.redhat.io/rhceph/keepalived-rhel9:latest|cp.icr.io/cp/ibm-ceph/keepalived-rhel9:latest|" \
+  -e "s|registry.redhat.io/rhceph/snmp-notifier-rhel9:latest|cp.icr.io/cp/ibm-ceph/snmp-notifier-rhel9:latest|" \

Here are the remaining changes we need:

  • Please remove all references to the latest tag from this sed command. It should be the way we set it up in ubi9-ibm: update container image list #2122
  • Please add loki and promtail back - not sure why these are gone in this latest version?

We should only see 6 changing to 7, not these other changes.

We do need to rename the grafana container for 7, but I can submit a separate PR for that after we finish this one.

this splits ubi9 by creating `ceph-releases/<ceph-release>/ubi9-<branding>`
given that quincy and reef are both based on ubi9 and require ceph release
specific changes.

Signed-off-by: Guillaume Abrioux <gabrioux@ibm.com>
This moves the file at `ceph-releases/ALL/ubi9-ibm/__DOCKERFILE_BRANDING__`
to `ceph-releases/quincy/ubi9-ibm/__DOCKERFILE_BRANDING__` because it is specific
to a particular release.
Also, this removes some duplicate files.

Signed-off-by: Guillaume Abrioux <gabrioux@ibm.com>
@ktdreyer
Copy link
Member

The steps I used to test this:

  1. Run BRANDING=ibm VERSION=7 ./contrib/compose-rhcs.sh on the main branch
  2. Set that composed output aside somewhere (like with cp)
  3. Switch from main to this PR's branch (quick-clean-up)
  4. Run BRANDING=ibm VERSION=7 ./contrib/compose-rhcs.sh again, this time on the quick-clean-up branch
  5. Compare that output to the output from main

With the latest updates, this PR now shows the correct diff (just changing 6 to 7).

@guits guits merged commit 2ba0710 into main Jul 19, 2023
6 checks passed
@guits guits deleted the quick-clean-up branch July 19, 2023 13:45
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