Bug 1276460

Summary: [perf] oo-admin-repair --orphaned-envs does not need to populate data hashes
Product: OpenShift Online Reporter: Andy Grimm <agrimm>
Component: PodAssignee: Vu Dinh <vdinh>
Status: CLOSED WONTFIX QA Contact: Jianwei Hou <jhou>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 2.xCC: abhgupta, aos-bugs, jgoulding, jokerman, mmccomas, vdinh
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2017-05-31 18:22:11 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:
Bug Depends On:    
Bug Blocks: 1277547    

Description Andy Grimm 2015-10-29 18:52:22 UTC
oo-admin-repair is brutally slow (i.e., takes tens of minutes) in OpenShift Online production due to the initial population of hashes for all users, domains, and apps.  For the --orphaned-envs subcommand, there is no reason to populate these hashes, so this code should be skipped.

Comment 1 Vu Dinh 2015-11-02 18:52:37 UTC
PR <https://github.com/openshift/origin-server/pull/6303> is submitted to fix this issue.

Comment 2 Abhishek Gupta 2015-11-12 20:44:14 UTC
Note to QE: The fix for this bug involved changes to the portion of the script that fixes stale ssh keys as well. A regression test for that option would be helpful.

Comment 3 openshift-github-bot 2015-11-12 21:32:56 UTC
Commit pushed to master at https://github.com/openshift/origin-server

https://github.com/openshift/origin-server/commit/f3e646ae24bfa71b482b2936d055e0fb6122091c
Bug 1276460: oo-admin-repair --orphaned-envs no need to populate hashes

The 'oo-admin-repair' commands populate hashes for user, domain, app and district
regardless if they are necessary or not. After this change only needed hashes are
for specific commands by using several flags to determine which commands are used.
This change will improve the performance by avoid populating unnecessary hashes.

The current "--ssh-keys" option removes orphaned environment variables which should
only be done by using "--orphaned-envs" option. This change removes that task from
"--ssh-keys" option. Furthermore, with this change, the orphaned variables are now
only removed from the domains but not from the gears as they are updated when the
applications are updated. Even if the orphaned variables remain in the gears, there
are no harms. Also, the orphaned variables will be grouped by domain and then removed
all at once for each domain to reduce database access. All changes are intended to
further improve the overall performance of the oo-admin-repair command.

Bug 1276460
Link <https://bugzilla.redhat.com/show_bug.cgi?id=1276460>

Signed-off-by: Vu Dinh <vdinh>

Comment 4 Jianwei Hou 2015-11-26 03:31:46 UTC
Verified on devenv_5723, oo-admin-chk is able to detect the stale ssh keys and oo-admin-repair could fix that.

Comment 6 Eric Paris 2017-05-31 18:22:11 UTC
We apologize, however, we do not plan to address this report at this time. The majority of our active development is for the v3 version of OpenShift. If you would like for Red Hat to reconsider this decision, please reach out to your support representative. We are very sorry for any inconvenience this may cause.