+++ This bug was initially created as a clone of Bug #1302045 +++ Description of problem: In basic_suite_3.6/engine-answer-file.conf , the value OSETUP_RPMDISTRO/enableUpgrade=none:None Should be: OSETUP_RPMDISTRO/enableUpgrade=bool:False Version-Release number of selected component (if applicable): 5a2c9e79d8bf15581242075c5eb53b9b34f5ae96 How reproducible: Always - though I'm not sure how it worked so far! --- Additional comment from Red Hat Bugzilla Rules Engine on 2016-01-26 18:12:16 IST --- Bug tickets must have version flags set prior to targeting them to a release. Please ask maintainer to set the correct version flags and only then set the target milestone. --- Additional comment from Red Hat Bugzilla Rules Engine on 2016-01-26 18:12:16 IST --- Target release should be placed once a package build is known to fix a issue. Since this bug is not modified, the target version has been reset. Please use target milestone to plan a fix for a oVirt release. --- Additional comment from Red Hat Bugzilla Rules Engine on 2016-01-26 18:13:04 IST --- Bug tickets must have version flags set prior to targeting them to a release. Please ask maintainer to set the correct version flags and only then set the target milestone. --- Additional comment from Red Hat Bugzilla Rules Engine on 2016-01-26 18:13:04 IST --- Target release should be placed once a package build is known to fix a issue. Since this bug is not modified, the target version has been reset. Please use target milestone to plan a fix for a oVirt release. --- Additional comment from Red Hat Bugzilla Rules Engine on 2016-01-26 18:13:26 IST --- Bug tickets must have version flags set prior to targeting them to a release. Please ask maintainer to set the correct version flags and only then set the target milestone.
The real issue is that we did not test the success of the installation. Proposed patch to the engine installation test: diff --git a/basic_suite_3.6/test-scenarios/001_initialize_engine.py b/basic_suite_3.6/test-scenarios/001_initialize_engine.py index 3347a1e..bc950ce 100644 --- a/basic_suite_3.6/test-scenarios/001_initialize_engine.py +++ b/basic_suite_3.6/test-scenarios/001_initialize_engine.py @@ -35,9 +35,16 @@ def test_initialize_engine(prefix): '/tmp/answer-file', ) - engine.ssh( + result = engine.ssh( [ 'engine-setup', '--config-append=/tmp/answer-file', ], ) + + testlib.assert_true_within_short( + lambda: False if result != 0 else True + ) + testlib.assert_true_within_long( + lambda: engine.service('ovirt-engine').alive() + ) David - where should I send this patch to?
(In reply to Yaniv Kaul from comment #0) > How reproducible: > Always - though I'm not sure how it worked so far! A wild guess: Because of bug 1301902
The above code in comment 1 is a bit flawed - should check for result.code value.
Replied by email