Support full BVT for Astute in case of change /magent folder

Bug #1588769 reported by Vladimir Sharshov
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Confirmed
High
Alexander Kurenyshev

Bug Description

Detailed bug description:

Changes in /magent https://github.com/openstack/fuel-astute/tree/master/mcagents folder can be very dangerous. It can not be catch by standard jobs because it build as separate package and build in bootstrap and install separately from local master repository on installed nodes. Also it should be install on master node.

Steps to reproduce:

Make logical mistake in any mcagents folder and push review.
Expected results:

It should catch such updates and mark review with -1.

Actual result:

It will pass all tests without any errors.

Reproducibility:

Always

Workaround:

Core-review should require BVT or QA +1 test in case of magents updates.

Impact:

It can raise critical errors.

Additional information:

Check history of this bug: https://bugs.launchpad.net/fuel/+bug/1569325

Tags: area-ci
Revision history for this message
Vladimir Sharshov (vsharshov) wrote :

Fast solution: run full BVT test on every Astute request. Special care for depends on parameters, because mostly times such change in magent connected with some change in Fuel-web and without it will fail

tags: removed: area-infra-apps
Revision history for this message
Dmitry Kaigarodеsev (dkaiharodsev) wrote :

we've already got a deployment tests for astute package:

https://ci.fuel-infra.org/job/master.fuel-astute.pkgs.ubuntu.review_astute_patched/
https://ci.fuel-infra.org/job/mitaka.fuel-astute.pkgs.ubuntu.review_astute_patched/

please clarify if existing tests are not cover this bug request

Changed in fuel:
status: Confirmed → Incomplete
assignee: Fuel Infra Apps (fuel-infra-apps) → Vladimir Sharshov (vsharshov)
description: updated
Revision history for this message
Vladimir Sharshov (vsharshov) wrote :

Hi, my option about currents tests: "It can not be catch by standard jobs because it build as separate package and build in bootstrap and install separately from local master repository on installed nodes. Also it should be install on master node."

So i ask to modify this job to support scenario with updates in case of magents.

Changed in fuel:
status: Incomplete → Confirmed
Changed in fuel:
assignee: Vladimir Sharshov (vsharshov) → Fuel build team (fuel-build)
Roman Vyalov (r0mikiam)
Changed in fuel:
assignee: Fuel build team (fuel-build) → Alexander Kurenyshev (akurenyshev)
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.