Bug 1713378

Summary:	Empty PHASE field in `oc get hco`
Product:	Container Native Virtualization (CNV)	Reporter:	Irina Gulina <igulina>
Component:	Installation	Assignee:	Oren Cohen <ocohen>
Status:	CLOSED ERRATA	QA Contact:	Irina Gulina <igulina>
Severity:	medium	Docs Contact:
Priority:	medium
Version:	2.0	CC:	cnv-qe-bugs, fdeutsch, ncredi, ocohen, rhallise, stirabos
Target Milestone:	---	Flags:	ocohen: needinfo-
Target Release:	2.3.0
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:	hyperconverged-cluster-operator:v2.3.0-49	Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2020-05-04 19:10:36 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 Irina Gulina 2019-05-23 13:57:47 UTC

Description of problem:
oc get hco --all-namespaces constantly displays nothing for a PHASE field 

Version-Release number of selected component (if applicable):
hco-bundle-registry:v2.0.0-17

How reproducible:
always

Steps to Reproduce:
1. watch oc get hco --all-namespaces
2. install hco
3.

Actual results:
oc get hco --all-namespaces
NAMESPACE                 NAME                     AGE     PHASE
kubevirt-hyperconverged   hyperconverged-cluster   5m24s   


Expected results:
oc get hco --all-namespaces
NAMESPACE                 NAME                     AGE     PHASE
kubevirt-hyperconverged   hyperconverged-cluster   5m24s   Succeeded

Otherwise what is that field for?

Comment 1 Fabian Deutsch 2019-05-24 08:44:41 UTC

Not nice, but not critical. moving out.

Comment 2 Ryan Hallisey 2019-07-31 13:16:22 UTC

There was some work upstream around this, but it was deprioritized for upgrades.  Pushing out to 2.2.

Comment 4 Simone Tiraboschi 2020-01-28 15:31:49 UTC

Phase field is not relevant in our case because we are using the conditions as recommended;
The simplest option is to remove this:
https://github.com/kubevirt/hyperconverged-cluster-operator/blob/master/deploy/crds/hco.crd.yaml#L11

Comment 5 Nelly Credi 2020-02-17 08:19:29 UTC

Please add fixed in version

Comment 6 Irina Gulina 2020-02-19 12:00:47 UTC

I still see the Phase field. Simone, please let me know when it will be cherry-picked.

Comment 7 Simone Tiraboschi 2020-02-19 12:44:46 UTC

We need to merge also https://github.com/kubevirt/hyperconverged-cluster-operator/pull/459 to be able to regenerate fresh CRDs on each build also for HCO operator downstream.

Comment 9 Irina Gulina 2020-02-26 15:36:09 UTC

Phase column has been removed. Thanks

>> oc get hco -n openshift-cnv
NAME                      AGE
kubevirt-hyperconverged   6m53s

Comment 12 errata-xmlrpc 2020-05-04 19:10:36 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, 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/RHEA-2020:2011