[fuel-ci] Set RELEASE_VERSION to 9.0 for some jobs

Bug #1647666 reported by Dmitry Belyaninov
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Released
High
Alexander Evseev

Bug Description

There is job used clear Fuel 9.0 (without updates)- [0].

FORCE_DISABLE_UPDATES True
UPDATE_MASTER False

But new check [1] fails without RELEASE_VERSION setted to 9.0

    assert_equal(fuel_release, settings.FUEL_RELEASE_VERSION, err_msg)
AssertionError: Fuel version should be `9.2` but it is `9.0`

So we should add appropreate variable setting for JJB.

[0] https://product-ci.infra.mirantis.net/job/9.x.system_test.ubuntu.add_updated_node_to_environment/60/injectedEnvVars/
[1] https://github.com/openstack/fuel-qa/commit/96de67b04506d12047f29cad3d883753b49a425c

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Fix proposed to fuel-infra/jenkins-jobs (master)

Fix proposed to branch: master
Change author: Alexander Kurenyshev <email address hidden>
Review: https://review.fuel-infra.org/29410

Changed in fuel:
status: New → In Progress
Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Fix merged to fuel-infra/jenkins-jobs (master)

Reviewed: https://review.fuel-infra.org/29410
Submitter: Dmitry Kaigarodsev <email address hidden>
Branch: master

Commit: bd9ea550c61230e217973c8da6962a9500cebb9d
Author: Alexander Kurenyshev <email address hidden>
Date: Fri Dec 16 13:43:48 2016

Add FUEL_RELEASE_VERSION for test 9.1 to 9.0

Fix sets a variable FUEL_RELEASE_VERSION=9.0
for test with adding new 9.1 node to the 9.0
environment. For now check in tests failed
with `AssertionError: Fuel version should be
`9.2` but it is `9.0``

Change-Id: Iec68f11afe76cabb738dcd3883e5de752c7ccc2a
Closes-Bug:#1647666

Changed in fuel:
status: In Progress → Fix Committed
Revision history for this message
Nastya Urlapova (aurlapova) wrote :
Changed in fuel:
status: Fix Committed → Confirmed
Revision history for this message
Nastya Urlapova (aurlapova) wrote :
Revision history for this message
Alexander Kurenyshev (akurenyshev) wrote :

Here we have another behavior:
for all these jobs there is a parameter FUEL_RELEASE_VERSION set to 9.0.

And for all these jobs there are:
ENABLE_MOS_CENTOS_HOLDBACK=false
ENABLE_MOS_CENTOS_HOTFIX=false
ENABLE_MOS_CENTOS_OS=false
ENABLE_MOS_CENTOS_PROPOSED=false
ENABLE_MOS_CENTOS_SECURITY=false
ENABLE_MOS_CENTOS_UPDATES=false
ENABLE_MOS_UBUNTU_HOLDBACK=false
ENABLE_MOS_UBUNTU_HOTFIX=false
ENABLE_MOS_UBUNTU_OS=false
ENABLE_MOS_UBUNTU_PROPOSED=false
ENABLE_MOS_UBUNTU_SECURITY=false
ENABLE_MOS_UBUNTU_UPDATES=false

this means that we shouldn't update the master node to 9.2.
But if we take a look on this [1] we could find some of that variables set to `true`
Thats why we have an updated master and get the error in assert:
AssertionError: Fuel version should be `9.0` but it is `9.2`

So, actually this is another bug with the same error as in the description.
Please, contact Ilya Bumarskov or Ruslan Khozinov for additional info about failed tests.

[1] https://product-ci.infra.mirantis.net/job/9.x.acceptance.ubuntu.install_mu_no_ha_base_negative/16/injectedEnvVars/

Changed in fuel:
assignee: Alexander Kurenyshev (akurenyshev) → Fuel CI (fuel-ci)
Revision history for this message
Alexander Evseev (aevseev) wrote :

Job 9.x.acceptance.ubuntu.install_mu_no_ha_base_negative inherits parameters from runner job (9.x.acceptance.runner) having some of listed parameters set to true.

And Jenkins has a bug: https://issues.jenkins-ci.org/browse/JENKINS-14437

Due to the bug, Jenkins is unable to override job parameters by EnvInject plugin, so it's need to trigger job from runner without current parameters.

Changed in fuel:
status: Confirmed → In Progress
Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Fix proposed to fuel-infra/jenkins-jobs (master)

Fix proposed to branch: master
Change author: Alexander Evseev <email address hidden>
Review: https://review.fuel-infra.org/29432

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Fix merged to fuel-infra/jenkins-jobs (master)

Reviewed: https://review.fuel-infra.org/29432
Submitter: Alexander Evseev <email address hidden>
Branch: master

Commit: d6e37ad71ee0117c19b82bc7f2fba69f28b80e29
Author: Alexander Evseev <email address hidden>
Date: Mon Dec 19 13:05:58 2016

[product-ci] Don't inherit parameters by MU acceptance

Do not use current runner job parameters for MU acceptance jobs due to
Jenkins bug preventing overriding build parameters by EnvInject plugin.

See: https://issues.jenkins-ci.org/browse/JENKINS-14437

Change-Id: Id07bddf8508dde6598e2129c1bc8e37ce105fa96
Partial-Bug: #1647666

Roman Vyalov (r0mikiam)
Changed in fuel:
assignee: Fuel CI (fuel-ci) → Alexander Evseev (aevseev-h)
Changed in fuel:
status: In Progress → Fix Committed
Revision history for this message
Alexander Evseev (aevseev) wrote :

My previous change broke acceptance tests:

+ '[' '' = latest ']'
+ UBUNTU_MIRROR_URL=http://mirror.seed-cz1.fuel-infra.org/pkgs/snapshots//

Need to add more parameters.

Changed in fuel:
status: Fix Committed → In Progress
Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Fix proposed to fuel-infra/jenkins-jobs (master)

Fix proposed to branch: master
Change author: Alexander Evseev <email address hidden>
Review: https://review.fuel-infra.org/29751

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Fix merged to fuel-infra/jenkins-jobs (master)

Reviewed: https://review.fuel-infra.org/29751
Submitter: Alexander Evseev <email address hidden>
Branch: master

Commit: cf38de7a3354c62f35e270729877bff9398cf53f
Author: Alexander Evseev <email address hidden>
Date: Wed Jan 11 06:48:36 2017

[product-ci] Add more parameters to 9.x accepance jobs

Need to inherit parameters about repo snapshots.

Change-Id: If91e6d10b283c87130d8977753b2525075a7648b
See: https://review.fuel-infra.org/29432
Partial-Bug: #1647666

Changed in fuel:
status: In Progress → Fix Committed
Revision history for this message
Alexander Evseev (aevseev) wrote :

MU acceptance jobs still failing, due to incorrect parameter set by fuel-qa:

2017-01-15 18:54:30 - ERROR - ssh_client.py:580 -- Command 'cd mos_playbooks/mos_mu && ansible-playbook playbooks/mos9_prepare_env.yml -e '{"env_id":1, "snapshot_repo":"9.0-2017-01-13-184421"}'' returned exit code 2 while expected [0]

changed: [nailgun.test.domain.local] => (item={u'name': u'mos9.2-updates', u'url': u'http://mirror.fuel-infra.org/mos-repos/ubuntu/9.0-2017-01-13-184421', u'section': u'main restricted', u'priority': 1050, u'suite': u'mos9.0-updates', u'type': u'deb'})

URL contained snapshot ID must be like:

http://mirror.fuel-infra.org/mos-repos/ubuntu/snapshots/9.0-2017-01-13-184421

Parameter to be used for this URL (snapshot_repo) is set by fuel-qa:

https://github.com/openstack/fuel-qa/blob/stable/mitaka/fuelweb_test/tests/tests_upgrade/tests_install_mu/test_install_mu_base.py#L83

Revision history for this message
Nastya Urlapova (aurlapova) wrote :
Changed in fuel:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.