Bug 1918690 - Should update the KCM resource-graph timely with the latest configure [NEEDINFO]
Summary: Should update the KCM resource-graph timely with the latest configure
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: OpenShift Container Platform
Classification: Red Hat
Component: kube-controller-manager
Version: 4.7
Hardware: Unspecified
OS: Unspecified
low
medium
Target Milestone: ---
: 4.11.0
Assignee: Filip Krepinsky
QA Contact: zhou ying
URL:
Whiteboard: LifecycleStale
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2021-01-21 11:47 UTC by zhou ying
Modified: 2022-08-10 10:36 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: No Doc Update
Doc Text:
Clone Of:
Environment:
Last Closed: 2022-08-10 10:35:38 UTC
Target Upstream Version:
mfojtik: needinfo?


Attachments (Terms of Use)
Resource Graph (388.35 KB, image/png)
2021-01-21 11:48 UTC, zhou ying
no flags Details


Links
System ID Private Priority Status Summary Last Updated
Github openshift cluster-kube-controller-manager-operator pull 613 0 None open Bug 1918690: update resource-graph to include current resources 2022-04-14 13:52:14 UTC
Red Hat Product Errata RHSA-2022:5069 0 None None None 2022-08-10 10:36:00 UTC

Description zhou ying 2021-01-21 11:47:50 UTC
Description of problem:
Should update the KCM resource-graph timely with the latest configure 

Version-Release number of selected component (if applicable):
payload: 4.7.0-0.nightly-2021-01-19-095812

How reproducible:
always

Steps to Reproduce:
1) Check the KCM resource-graph with command :
`oc exec po/kube-controller-manager-operator-86ccb6fb8c-xrc4s -- cluster-kube-controller-manager-operator resource-graph |dot -Tpng >output0121.png` 

Actual results:
1) From the result picture show  the comfigmap serviceaccount-ca in openshift-kube-controller-manager  is unioned from comfigmap initial-serviceaccount-ca in openshift-config  and configmap default-ingress-cert in openshift-config-managed . But actruly can’t find the configmap initial-serviceaccount-ca in project openshift-config.  

Expected results:
1) update the KCM resource-graph timely with the latest configure.


Additional info:

Comment 1 zhou ying 2021-01-21 11:48:39 UTC
Created attachment 1749368 [details]
Resource Graph

Comment 2 Maciej Szulik 2021-01-21 12:03:36 UTC
This is internal utility, so I'm lowering the priority.

Comment 3 Jan Chaloupka 2021-01-27 14:51:46 UTC
initial-serviceaccount-ca CM in openshift-config NS is hardcoded in the code [1] and marked as static.

[1] https://github.com/openshift/cluster-kube-controller-manager-operator/blob/master/pkg/cmd/resourcegraph/resourcegraph.go#L115

Comment 5 Jan Chaloupka 2021-02-04 12:13:55 UTC
Need more time to understand what the graph is supposed to do and how to improve the logic.

Comment 6 Michal Fojtik 2021-03-06 12:26:53 UTC
This bug hasn't had any activity in the last 30 days. Maybe the problem got resolved, was a duplicate of something else, or became less pressing for some reason - or maybe it's still relevant but just hasn't been looked at yet. As such, we're marking this bug as "LifecycleStale" and decreasing the severity/priority. If you have further information on the current state of the bug, please update it, otherwise this bug can be closed in about 7 days. The information can be, for example, that the problem still occurs, that you still want the feature, that more information is needed, or that the bug is (for whatever reason) no longer relevant. Additionally, you can add LifecycleFrozen into Keywords if you think this bug should never be marked as stale. Please consult with bug assignee before you do that.

Comment 7 zhou ying 2021-03-08 00:43:09 UTC
still could be reproduced.

Comment 8 Michal Fojtik 2021-03-08 01:27:01 UTC
The LifecycleStale keyword was removed because the bug got commented on recently.
The bug assignee was notified.

Comment 9 Michal Fojtik 2021-04-07 02:22:29 UTC
This bug hasn't had any activity in the last 30 days. Maybe the problem got resolved, was a duplicate of something else, or became less pressing for some reason - or maybe it's still relevant but just hasn't been looked at yet. As such, we're marking this bug as "LifecycleStale" and decreasing the severity/priority. If you have further information on the current state of the bug, please update it, otherwise this bug can be closed in about 7 days. The information can be, for example, that the problem still occurs, that you still want the feature, that more information is needed, or that the bug is (for whatever reason) no longer relevant. Additionally, you can add LifecycleFrozen into Keywords if you think this bug should never be marked as stale. Please consult with bug assignee before you do that.

Comment 10 Jan Chaloupka 2021-04-30 12:03:41 UTC
Need more time to understand what the graph is supposed to do and how to improve the logic.

Comment 11 Jan Chaloupka 2021-05-20 14:10:02 UTC
Need more time to understand what the graph is supposed to do and how to improve the logic.

Comment 12 Michal Fojtik 2021-06-19 14:29:43 UTC
This bug hasn't had any activity in the last 30 days. Maybe the problem got resolved, was a duplicate of something else, or became less pressing for some reason - or maybe it's still relevant but just hasn't been looked at yet. As such, we're marking this bug as "LifecycleStale" and decreasing the severity/priority. If you have further information on the current state of the bug, please update it, otherwise this bug can be closed in about 7 days. The information can be, for example, that the problem still occurs, that you still want the feature, that more information is needed, or that the bug is (for whatever reason) no longer relevant. Additionally, you can add LifecycleFrozen into Keywords if you think this bug should never be marked as stale. Please consult with bug assignee before you do that.

Comment 13 Michal Fojtik 2021-07-21 13:23:11 UTC
This bug hasn't had any activity in the last 30 days. Maybe the problem got resolved, was a duplicate of something else, or became less pressing for some reason - or maybe it's still relevant but just hasn't been looked at yet. As such, we're marking this bug as "LifecycleStale" and decreasing the severity/priority. If you have further information on the current state of the bug, please update it, otherwise this bug can be closed in about 7 days. The information can be, for example, that the problem still occurs, that you still want the feature, that more information is needed, or that the bug is (for whatever reason) no longer relevant. Additionally, you can add LifecycleFrozen into Keywords if you think this bug should never be marked as stale. Please consult with bug assignee before you do that.

Comment 14 Filip Krepinsky 2021-07-23 11:37:02 UTC
I’m adding UpcomingSprint, because I was occupied by fixing bugs with higher priority/severity, developing new features with higher priority, or developing new features to improve stability at a macro level. I will revisit this bug next sprint.

Comment 15 Michal Fojtik 2021-08-20 15:04:41 UTC
This bug hasn't had any activity in the last 30 days. Maybe the problem got resolved, was a duplicate of something else, or became less pressing for some reason - or maybe it's still relevant but just hasn't been looked at yet. As such, we're marking this bug as "LifecycleStale" and decreasing the severity/priority. If you have further information on the current state of the bug, please update it, otherwise this bug can be closed in about 7 days. The information can be, for example, that the problem still occurs, that you still want the feature, that more information is needed, or that the bug is (for whatever reason) no longer relevant. Additionally, you can add LifecycleFrozen into Keywords if you think this bug should never be marked as stale. Please consult with bug assignee before you do that.

Comment 16 Filip Krepinsky 2021-08-20 17:20:15 UTC
I’m adding UpcomingSprint, because I was occupied by fixing bugs with higher priority/severity, developing new features with higher priority, or developing new features to improve stability at a macro level. I will revisit this bug next sprint.

Comment 17 Michal Fojtik 2021-09-19 18:00:54 UTC
This bug hasn't had any activity in the last 30 days. Maybe the problem got resolved, was a duplicate of something else, or became less pressing for some reason - or maybe it's still relevant but just hasn't been looked at yet. As such, we're marking this bug as "LifecycleStale" and decreasing the severity/priority. If you have further information on the current state of the bug, please update it, otherwise this bug can be closed in about 7 days. The information can be, for example, that the problem still occurs, that you still want the feature, that more information is needed, or that the bug is (for whatever reason) no longer relevant. Additionally, you can add LifecycleFrozen into Whiteboard if you think this bug should never be marked as stale. Please consult with bug assignee before you do that.

Comment 18 Michal Fojtik 2021-10-20 10:38:34 UTC
This bug hasn't had any activity in the last 30 days. Maybe the problem got resolved, was a duplicate of something else, or became less pressing for some reason - or maybe it's still relevant but just hasn't been looked at yet. As such, we're marking this bug as "LifecycleStale" and decreasing the severity/priority. If you have further information on the current state of the bug, please update it, otherwise this bug can be closed in about 7 days. The information can be, for example, that the problem still occurs, that you still want the feature, that more information is needed, or that the bug is (for whatever reason) no longer relevant. Additionally, you can add LifecycleFrozen into Whiteboard if you think this bug should never be marked as stale. Please consult with bug assignee before you do that.

Comment 19 Michal Fojtik 2021-11-19 11:35:16 UTC
This bug hasn't had any activity in the last 30 days. Maybe the problem got resolved, was a duplicate of something else, or became less pressing for some reason - or maybe it's still relevant but just hasn't been looked at yet. As such, we're marking this bug as "LifecycleStale" and decreasing the severity/priority. If you have further information on the current state of the bug, please update it, otherwise this bug can be closed in about 7 days. The information can be, for example, that the problem still occurs, that you still want the feature, that more information is needed, or that the bug is (for whatever reason) no longer relevant. Additionally, you can add LifecycleFrozen into Whiteboard if you think this bug should never be marked as stale. Please consult with bug assignee before you do that.

Comment 20 Michal Fojtik 2021-12-19 13:34:03 UTC
This bug hasn't had any activity in the last 30 days. Maybe the problem got resolved, was a duplicate of something else, or became less pressing for some reason - or maybe it's still relevant but just hasn't been looked at yet. As such, we're marking this bug as "LifecycleStale" and decreasing the severity/priority. If you have further information on the current state of the bug, please update it, otherwise this bug can be closed in about 7 days. The information can be, for example, that the problem still occurs, that you still want the feature, that more information is needed, or that the bug is (for whatever reason) no longer relevant. Additionally, you can add LifecycleFrozen into Whiteboard if you think this bug should never be marked as stale. Please consult with bug assignee before you do that.

Comment 21 Filip Krepinsky 2022-04-04 20:54:54 UTC
I’m adding UpcomingSprint, because I was occupied by fixing bugs with higher priority/severity, developing new features with higher priority, or developing new features to improve stability at a macro level. I will revisit this bug next sprint.

Comment 22 Filip Krepinsky 2022-04-14 14:06:58 UTC
took me a long time to tackle this, posted a PR with updated resources (there is a lot of new resources)

Comment 25 zhou ying 2022-04-24 03:53:43 UTC
Checked with latest version , the issue has fixed :

 oc get clusterversion 
NAME      VERSION                              AVAILABLE   PROGRESSING   SINCE   STATUS
version   4.11.0-0.nightly-2022-04-23-153426   True        False         135m    Cluster version is 4.11.0-0.nightly-2022-04-23-153426

steps:
1) extract the resource-graph from kcm-operator pods:
oc exec po/kube-controller-manager-operator-55d887cf7b-qmgpt  -- cluster-kube-controller-manager-operator resource-graph |dot -Tpng >output0424.png
2) check the comfigmap serviceaccount-ca in openshift-kube-controller-manager  is unioned from comfigmap kube-apiserver-server-ca in openshift-config-managed  and configmap default-ingress-cert in openshift-config-managed

Comment 27 errata-xmlrpc 2022-08-10 10:35:38 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.