Bug 1804598

Summary:	Installed Operators should show 'Copied' if CSV.status.reason is Copied
Product:	OpenShift Container Platform	Reporter:	Yadan Pei <yapei>
Component:	Management Console	Assignee:	Robb Hamilton <rhamilto>
Status:	CLOSED NOTABUG	QA Contact:	Yadan Pei <yapei>
Severity:	medium	Docs Contact:
Priority:	unspecified
Version:	4.4	CC:	aos-bugs, jokerman, rhamilto, yapei
Target Milestone:	---	Keywords:	Regression
Target Release:	4.4.0
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2020-02-19 14:15:54 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:

Description Yadan Pei 2020-02-19 09:06:12 UTC

Description of problem:
We show 'Copied' status on Installed Operators list page if CSV.status.reason in current namespace in earlier version, now it always shows 'Succeed' for all namespaces, seems like a regression brought in bug 1798858, not sure if this change is intended 

Version-Release number of selected component (if applicable):
4.4.0-0.nightly-2020-02-18-164016
 
How reproducible:
Always

Steps to Reproduce:
1. Subscribe OpenShift Pipeline Operator into all namespaces on the cluster
2. Operators -> Installed Operators -> Choose openshift-operators namespace 
3. Operators -> Installed Operators -> Choose other namespace, such as default

Actual results:
2. Status is shown 'Succeed'
3. Status is shown 'Succeed'

Expected results:
3. Status should show 'Copied'
Check csv.status.phase and csv.status.reason in 'openshift-operators' namespace
# oc get csv openshift-pipelines-operator.v0.8.2 -n openshift-operators -o json | jq '.status.phase'
"Succeeded"
# oc get csv openshift-pipelines-operator.v0.8.2 -n openshift-operators -o json | jq '.status.reason'
"InstallSucceeded"

Check csv.status.phase and csv.status.reason in 'default' namespace
# oc get csv openshift-pipelines-operator.v0.8.2 -n default -o json | jq '.status.phase'
"Succeeded"
# oc get csv openshift-pipelines-operator.v0.8.2 -n default -o json | jq '.status.reason'
"Copied"

Additional info:

Comment 1 Robb Hamilton 2020-02-19 14:15:54 UTC

Hi, Ya Dan.  This is not a regression as it is an intentional change.  See https://github.com/openshift/console/pull/4270#issuecomment-585990907

Comment 2 Yadan Pei 2020-02-20 09:12:10 UTC

Robb, Thanks for pointing me the link, it's very helpful