Change in ovirt-engine[master]: packaging: setup: Check health

Code Review gerrit at ovirt.org
Tue May 30 08:31:11 UTC 2017


>From Sandro Bonazzola <sbonazzo at redhat.com>:

Sandro Bonazzola has submitted this change and it was merged.

Change subject: packaging: setup: Check health
......................................................................


packaging: setup: Check health

This patch introduces a new utility that can create a standalone script
that can be copied to an older engine machine and ran to get a report.

Currently this report does not do much, but it's expected that we'll
move most of the checks done during engine-setup to be ran also by this
script.

Details:

packaging/setup/bin/bundle-ovirt-engine-health

- creates this "bundle" standalone script

packaging/setup/bin/ovirt-engine-health

- The health checker itself. Can also be ran directly on the engine
  machine, but is mainly intended to be wrapped in the bundle.

packaging/setup/plugins/ovirt-engine-checks:

- A new plugins directory, that is shared by both engine-setup and
ovirt-engine-health.

packaging/setup/plugins/ovirt-engine-health:

- A new plugins directory, specific to the health checker

Also patched the spec and make files to allow building.

Also added packaging/setup/plugins/README to document this directory a
bit.

Also copied the code that checks cluster/DC versions compatibility to a
new plugin in plugins/ovirt-engine-checks, as a POC.

To try this, on the engine machine, after setup, run:

sh /usr/share/ovirt-engine/setup/bin/bundle-ovirt-engine-health

It will create (by default) ovirt-engine-health-check.run in current
directory. Copy this one to the target machine and run it there.

Change-Id: Iafb49d107c3b2b5623d11d0c4bbf907d6398e987
Signed-off-by: Yedidyah Bar David <didi at redhat.com>
---
M Makefile
M ovirt-engine.spec.in
A packaging/setup/bin/bundle-ovirt-engine-health
A packaging/setup/bin/ovirt-engine-health
M packaging/setup/bin/ovirt-engine-setup
M packaging/setup/ovirt_engine_setup/constants.py
A packaging/setup/plugins/README
A packaging/setup/plugins/ovirt-engine-checks/ovirt-engine/db/__init__.py
A packaging/setup/plugins/ovirt-engine-checks/ovirt-engine/db/engine.py
A packaging/setup/plugins/ovirt-engine-checks/ovirt-engine/db/versions.py
A packaging/setup/plugins/ovirt-engine-health/base/core/__init__.py
A packaging/setup/plugins/ovirt-engine-health/base/core/misc.py
12 files changed, 564 insertions(+), 3 deletions(-)

Approvals:
  Sandro Bonazzola: Looks good to me, approved
  Jenkins CI: Passed CI tests
  Yedidyah Bar David: Verified



-- 
To view, visit https://gerrit.ovirt.org/75372
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Iafb49d107c3b2b5623d11d0c4bbf907d6398e987
Gerrit-PatchSet: 22
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Yedidyah Bar David <didi at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland at redhat.com>
Gerrit-Reviewer: Ido Rosenzwig <irosenzw at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Lev Veyde <lveyde at redhat.com>
Gerrit-Reviewer: Martin Mucha <mmucha at redhat.com>
Gerrit-Reviewer: Rafael Martins <rafael at rafaelmartins.eng.br>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo at redhat.com>
Gerrit-Reviewer: Simone Tiraboschi <stirabos at redhat.com>
Gerrit-Reviewer: Yedidyah Bar David <didi at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>


More information about the Engine-commits mailing list