Bug 1863446 - [Assisted-4.5-M2] clean all does not remove ConfigMaps and PVC
Summary: [Assisted-4.5-M2] clean all does not remove ConfigMaps and PVC
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: OpenShift Container Platform
Classification: Red Hat
Component: assisted-installer
Version: 4.5
Hardware: Unspecified
OS: Unspecified
unspecified
high
Target Milestone: ---
: 4.7.0
Assignee: Michael Filanov
QA Contact: Yuri Obshansky
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2020-08-03 16:22 UTC by Yuri Obshansky
Modified: 2021-09-01 18:24 UTC (History)
1 user (show)

Fixed In Version:
Doc Type: No Doc Update
Doc Text:
Clone Of:
Environment:
Last Closed: 2021-09-01 18:23:54 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)
Config Maps (49.70 KB, image/png)
2020-08-03 16:22 UTC, Yuri Obshansky
no flags Details
PVC (49.88 KB, image/png)
2020-08-03 16:23 UTC, Yuri Obshansky
no flags Details
Overall status of project after clean-all (16.33 KB, image/png)
2020-08-03 16:23 UTC, Yuri Obshansky
no flags Details


Links
System ID Private Priority Status Summary Last Updated
Red Hat Product Errata RHSA-2021:3262 0 None None None 2021-09-01 18:24:19 UTC

Description Yuri Obshansky 2020-08-03 16:22:37 UTC
Created attachment 1703961 [details]
Config Maps

Description of problem:
clean-all doesn't remove part of resources in project 

[root@seal55 bm-inventory]# make clear-all TARGET=oc-ingress APPLY_NAMESPACE=False NAMESPACE=assisted-installer-qe INGRESS_DOMAIN=apps.ocp.prod.psi.redhat.com
rm -rf /home/bm-inventory/build
kubectl -n assisted-installer-qe get pod -o name | grep create-image | xargs kubectl -n assisted-installer-qe delete 1> /dev/null || true
error: You must provide one or more resources by argument or filename.
Example resource specifications include:
   '-f rsrc.yaml'
   '--filename=rsrc.json'
   '<resource> <name>'
   '<resource>'
kubectl -n assisted-installer-qe get pod -o name | grep generate-kubeconfig | xargs kubectl -n assisted-installer-qe delete 1> /dev/null || true
python3 ./tools/clear_deployment.py --delete-namespace False --namespace "assisted-installer-qe" || true

See attached screenshots


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


How reproducible:


Steps to Reproduce:
1.
2.
3.

Actual results:


Expected results:


Additional info:

Comment 1 Yuri Obshansky 2020-08-03 16:23:11 UTC
Created attachment 1703962 [details]
PVC

Comment 2 Yuri Obshansky 2020-08-03 16:23:37 UTC
Created attachment 1703972 [details]
Overall status of project after clean-all

Comment 3 Michael Filanov 2020-08-04 06:11:14 UTC
APPLY_NAMESPACE=False define that we don't remove the namespace - so not all the resources are deleted.
PVC is defined not to be deleted in order to support upgrades. 

I'll check regarding the configmap.

Comment 4 Michael Filanov 2020-08-18 08:34:08 UTC
https://github.com/openshift/assisted-service/pull/153

Comment 5 Michael Filanov 2020-08-18 09:43:41 UTC
Fixed the script to delete configmaps, but as i said the PVC will still exist and it's part of the design

Comment 6 Yuri Obshansky 2020-08-19 20:58:38 UTC
Verified on master August 19, 2020

Comment 8 ximhan 2021-08-20 08:00:10 UTC
OpenShift engineering has decided to NOT ship 4.8.6 on 8/23 due to the following issue.
https://bugzilla.redhat.com/show_bug.cgi?id=1995785
All the fixes part will be now included in 4.8.7 on 8/30.

Comment 10 ximhan 2021-08-21 06:45:09 UTC
OpenShift engineering has decided to NOT ship 4.7.25 on 8/24 due to the following issue.
https://bugzilla.redhat.com/show_bug.cgi?id=1995785
All the fixes part will be now included in 4.7.26 on 8/31.

Comment 13 errata-xmlrpc 2021-09-01 18:23: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 (Important: OpenShift Container Platform 4.7.28 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:3262


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