Bug 2053118 - nncp Available condition reason should be exposed in `oc get`
Summary: nncp Available condition reason should be exposed in `oc get`
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: OpenShift Container Platform
Classification: Red Hat
Component: Networking
Version: 4.11
Hardware: Unspecified
OS: Unspecified
low
medium
Target Milestone: ---
: 4.11.0
Assignee: Christoph Stäbler
QA Contact: Aleksandra Malykhin
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2022-02-10 14:52 UTC by Ruth Netser
Modified: 2022-08-10 10:49 UTC (History)
5 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2022-08-10 10:49:18 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Github nmstate kubernetes-nmstate pull 1023 0 None Merged display Reason column when listing NNCPs 2022-03-24 06:34:16 UTC
Github openshift kubernetes-nmstate pull 272 0 None open Rebase to kubernetes-nmstate v0.71.0 2022-05-10 10:01:09 UTC
Red Hat Product Errata RHSA-2022:5069 0 None None None 2022-08-10 10:49:37 UTC

Description Ruth Netser 2022-02-10 14:52:43 UTC
Description of problem:
nncp Available condition reason should be exposed in `oc get`
When the status is Unknown, there's no visible status of the resource.
For easier debugging it will be useful to add a column with the current reason for "'status': 'Unknown', 'type': 'Available'"

Version-Release number of selected component (if applicable):


How reproducible:


Steps to Reproduce:
1.
2.
3.

Actual results:


Expected results:


Additional info:
When the conditions are:
{'lastHearbeatTime': '2022-02-10T14:08:39Z',
 'lastTransitionTime': '2022-02-10T14:08:39Z',
 'message': 'Policy is progressing 0/1 nodes finished',
 'reason': 'ConfigurationProgressing',
 'status': 'Unknown',
 'type': 'Available'}

This is what is displayed:
$ oc get nncp
NAME   STATUS
stam   

When the conditions are:

{'lastHearbeatTime': '2022-02-10T14:11:44Z',
 'lastTransitionTime': '2022-02-10T14:11:44Z',
 'message': '1/1 nodes successfully configured',
 'reason': 'SuccessfullyConfigured',
 'status': 'True',
 'type': 'Available'}

Status is Available

Comment 1 Radim Hrazdil 2022-03-15 14:38:50 UTC
Hello Ruth,

We have added a Progressing condition [0], so now when a policy is in progress, you should get
$ oc get nncp
NAME   STATUS
stam   Progressing

Do you find this good enough or do you want to add the Rason column?


[0] https://github.com/nmstate/kubernetes-nmstate/pull/982

Comment 2 Ruth Netser 2022-03-22 19:28:27 UTC
@rhrazdil thanks for the fix; if this is not a big issue, adding the Reason column would be very useful as well.

Comment 3 Radim Hrazdil 2022-03-24 06:34:16 UTC
PR adding Reason column to NNCP `oc/kubectl get nncp` list: https://github.com/nmstate/kubernetes-nmstate/pull/1023

Comment 4 Petr Horáček 2022-04-14 12:41:46 UTC
Waiting for the U/S PR to get to OpenShift's D/S

Comment 5 Petr Horáček 2022-04-22 13:06:46 UTC
This fix was merged U/S.

Comment 9 Aleksandra Malykhin 2022-05-17 14:19:31 UTC
Verified on kubernetes-nmstate-operator.4.11.0-202205171127

The reason column is displayed correctly and showing corresponding values.

[kni@provisionhost-0-0 ocp-edge-auto_ocp-edge-cluster-0]$ oc get nncp -w
NAME                STATUS        REASON
upgrade-br-marker   Progressing   ConfigurationProgressing
upgrade-br-marker   Progressing   ConfigurationProgressing
upgrade-br-marker   Available     SuccessfullyConfigured

Comment 11 errata-xmlrpc 2022-08-10 10:49:18 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 (Important: OpenShift Container Platform 4.11.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-2022:5069


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