David Caro has posted comments on this change.

Change subject: Added scripts for 3.4 to latest upgrade job
......................................................................


Patch Set 20:

(1 comment)

http://gerrit.ovirt.org/#/c/25989/20/jobs/ovirt-engine_upgrade_stable_to_3.4/cleanup.sh
File jobs/ovirt-engine_upgrade_stable_to_3.4/cleanup.sh:

Line 55:     if which engine-setup &>/dev/null; then
Line 56:         engine-cleanup --config-append="${cleanup_file}" \
Line 57:         || :
Line 58:     fi
Line 59:     yum -y remove ovirt-engine\* vdsm\* httpd mod_ssl || :
> why removing httpd?
It's a service that we don't want running on the slaves. It should only be 
installed/running for the tests.
Line 60:     rm -f /root/.pgpass
Line 61:     enable_engine_repos "$workspace/disabled_repos.list"
Line 62:     if [[ -d /var/log/ovirt-engine ]]; then
Line 63:         export -f store


-- 
To view, visit http://gerrit.ovirt.org/25989
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I869ec85c9990677c52ca61549fce4f50f2650f33
Gerrit-PatchSet: 20
Gerrit-Project: jenkins
Gerrit-Branch: master
Gerrit-Owner: David Caro <dcaro...@redhat.com>
Gerrit-Reviewer: David Caro <dcaro...@redhat.com>
Gerrit-Reviewer: Eyal Edri <ee...@redhat.com>
Gerrit-Reviewer: Kiril Nesenko <knese...@redhat.com>
Gerrit-Reviewer: Ohad Basan <oba...@redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbona...@redhat.com>
Gerrit-Reviewer: Yedidyah Bar David <d...@redhat.com>
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to