Bug 1974501 - [master] Assisted Service Operator should be Infrastructure Operator for Red Hat OpenShift
Summary: [master] Assisted Service Operator should be Infrastructure Operator for Red ...
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: OpenShift Container Platform
Classification: Red Hat
Component: assisted-installer
Version: 4.8
Hardware: Unspecified
OS: Unspecified
high
medium
Target Milestone: ---
: 4.9.0
Assignee: David Zager
QA Contact: bjacot
URL:
Whiteboard: AI-Team-Platform
Depends On:
Blocks: 1974931
TreeView+ depends on / blocked
 
Reported: 2021-06-21 20:49 UTC by David Zager
Modified: 2021-10-18 17:36 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: No Doc Update
Doc Text:
Clone Of:
: 1974931 (view as bug list)
Environment:
Last Closed: 2021-10-18 17:35:54 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Github openshift assisted-service pull 2070 0 None open Bug 1974501: infrastructure operator rename 2021-06-22 20:13:06 UTC
Red Hat Product Errata RHSA-2021:3759 0 None None None 2021-10-18 17:36:15 UTC

Internal Links: 1974931

Description David Zager 2021-06-21 20:49:06 UTC
Description of problem:
Assisted Service Operator (long form) and assisted-service-operator (short form) need to be updated to Infrastructure Operator for Red Hat OpenShift (long form) and infrastructure-operator (short form) respectively.

How reproducible: Always

Comment 1 David Zager 2021-06-21 21:14:00 UTC
In the assisted-service project we need to update the operator in the following ways:

1. Change the CSV name in {config/manifests/bases/ and deploy/olm-catalog/manifests/}assisted-service-operator.clusterserviceversion.yaml to infrastructure-operator.clusterserviceversion.yaml
2. sed 's/assisted-service-operator/infrastructure-operator/g'
    - in Makefile: update the name given to the deployment controller binary as well as the bundle images we publish upstream
    - to operator CSV: this should impact the operator name, deployment name, labels, operator container command, and spec.skips.

Updates to community-operators (this is how we communicate changes to ACM):
1. Move community-operators/assisted-service-operator -> community-operators/infrastructure-operator
2. Change package name from assisted-service.package.yaml -> infrastructure-operator.package.yaml

Changes to https://github.com/open-cluster-management/assisted-service-chart:
1. Update display names https://github.com/open-cluster-management/assisted-service-chart/blob/2486b0a815e97978b262e9011c5618c114be0dea/stable/assisted-service/Chart.yaml#L3-L4
2. Update automation to point to community-operator changes https://github.com/open-cluster-management/assisted-service-chart/blob/2486b0a815e97978b262e9011c5618c114be0dea/autogen-chart/config.yaml

Changes to https://github.com/open-cluster-management/hub-crds:
1. Update automation for https://github.com/open-cluster-management/hub-crds/tree/main/crds/assisted-service-operator -> https://github.com/open-cluster-management/hub-crds/tree/main/crds/infrastructure-operator

Can I get a thumbs up on this plan @mgoldboi @asegurap

Comment 2 David Zager 2021-06-22 18:54:05 UTC
The changes covered by this bug include updating the following fields in the ClusterServiceVersion:

1. metadata.annotations.description
2. spec.descpription
3. spec.install.deployments

All other references (ie the operator's metadata.name and the list of operator versions skipped will continue as assisted-service-operator).

Comment 4 David Zager 2021-06-24 20:13:58 UTC
There isn't really anything to verify.

Comment 8 errata-xmlrpc 2021-10-18 17:35:54 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory (Moderate: OpenShift Container Platform 4.9.0 bug fix and security update), and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHSA-2021:3759


Note You need to log in before you can comment on or make changes to this bug.