[8.0-pkg-install-ubuntu] Too many tests in one job

Bug #1521352 reported by Ivan Udovichenko
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Released
Medium
Fuel CI

Bug Description

Hello,

https://packaging-ci.infra.mirantis.net/job/8.0-pkg-install-ubuntu/1490/consoleFull

We used to have one Jenkins job which checked just package installation, removal and execution of tests/runtests.sh script.

Now we have another test in this job, which from my understanding checks dependency conflicts.

Could you please separate it from 8.0-pkg-install-ubuntu job and move it to another one ?
For example: 8.0-pkg-check-dependencies-ubuntu

Ideally, I would like to see a separate job for tests/runtests.sh script.

Seperation will make much easier for human beings to read logs and debug issues if they come up.

So, the main point here is to separate jobs and if one of the jobs fails on early stages just skip all other jobs and don't waste time and hardware resources on other tests execution.

Currently Jenkins job 8.0-pkg-check-dependencies-ubuntu runs all tests even if execution of tests/runtests.sh script exited with 1+ value.
It worth mentioning that script which checks dependency conflicts could take 20+ minutes. Correct me if I'm wrong.

Thank you very much.

description: updated
Changed in fuel:
milestone: none → 8.0
status: New → Confirmed
tags: added: packaging-ci
Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Related fix proposed to fuel-infra/jenkins-jobs (master)

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

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

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

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

Related fix proposed to branch: master
Change author: Ivan Remizov <email address hidden>
Review: https://review.fuel-infra.org/15610

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

Fix proposed to branch: master
Change author: Ivan Remizov <email address hidden>
Review: https://review.fuel-infra.org/15611

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote :

Fix proposed to branch: master
Change author: Ivan Remizov <email address hidden>
Review: https://review.fuel-infra.org/15616

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

Related fix proposed to branch: master
Change author: Ivan Remizov <email address hidden>
Review: https://review.fuel-infra.org/15617

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

Reviewed: https://review.fuel-infra.org/15610
Submitter: Ivan Remizov <email address hidden>
Branch: master

Commit: be04a5bf708ac428eee0df029525062e23fdec6a
Author: Ivan Remizov <email address hidden>
Date: Tue Dec 29 14:53:08 2015

Separate fuel repo test to own jenkins job

Split fuel install test to two jobs. First one run package version check and
install test, and second one run repository consistency check.

Change-Id: I74d88084499214ee1c570ab68bcab2da89d209ea
Related-Bug: #1521352

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Related fix merged to fuel-infra/zuul-layouts (master)

Reviewed: https://review.fuel-infra.org/15611
Submitter: Ivan Remizov <email address hidden>
Branch: master

Commit: 5f209c6198d73b3752872cb5c4a3ab0303db5aa5
Author: Ivan Remizov <email address hidden>
Date: Wed Dec 30 10:02:45 2015

Trigger fuel repository check after install test

Change-Id: Id6c0ea0e044793f678f2eb5fea3a65cc0087ca5c
Related-Bug: #1521352
Depends-On: I74d88084499214ee1c570ab68bcab2da89d209ea

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Related fix proposed to fuel-infra/zuul-layouts (master)

Related fix proposed to branch: master
Change author: Ivan Remizov <email address hidden>
Review: https://review.fuel-infra.org/15930

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Related fix merged to fuel-infra/zuul-layouts (master)

Reviewed: https://review.fuel-infra.org/15930
Submitter: Ivan Remizov <email address hidden>
Branch: master

Commit: 27e866686213a328aafa818dd4d3d1ff5e6c99e0
Author: Ivan Remizov <email address hidden>
Date: Wed Dec 30 16:19:51 2015

Skip fuel-repotest for MOS-0.9

Related-Bug: #1521352
Change-Id: I3337d4ffdfbcaac012399128227eada4e06e00d2

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

Fix proposed to branch: master
Change author: Ivan Remizov <email address hidden>
Review: https://review.fuel-infra.org/16010

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote :

Fix proposed to branch: master
Change author: Ivan Remizov <email address hidden>
Review: https://review.fuel-infra.org/16011

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

Reviewed: https://review.fuel-infra.org/15617
Submitter: Ivan Remizov <email address hidden>
Branch: master

Commit: 6a8f85032a3d63a032335ad5fa2d07cd59157fef
Author: Ivan Remizov <email address hidden>
Date: Mon Jan 11 08:59:01 2016

Separate repo test to own jenkins job

Split install test to two jobs. First one run package version check and
install test, and second one run repository consistency check.

Change-Id: Ia91fce83e2721832ae960f3577b91cf310654d01
Related-Bug: #1521352

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Related fix merged to fuel-infra/zuul-layouts (master)

Reviewed: https://review.fuel-infra.org/15616
Submitter: Ivan Remizov <email address hidden>
Branch: master

Commit: 1a911ae41432c1333bf145a113296affd3b3d013
Author: Ivan Remizov <email address hidden>
Date: Mon Jan 11 09:04:27 2016

Trigger repository check after install test

Change-Id: I0e3ad19cab75cba833730bfe5b7c7c9806e64104
Related-Bug: #1521352
Depends-On: Ia91fce83e2721832ae960f3577b91cf310654d01

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

Reviewed: https://review.fuel-infra.org/16011
Submitter: Ivan Remizov <email address hidden>
Branch: master

Commit: 311fc24c60ddd92ae668e27e3891576f39557405
Author: Ivan Remizov <email address hidden>
Date: Mon Jan 11 14:51:59 2016

Disable 9.0-pkg-repotest-centos

Change-Id: I8856c6d9d31973137fc9e2be5170dbdff8793ed5
Closes-Bug: #1521352
Depends-On: I130342d5b22832cfedb1f8fc8013c377d38b86a1

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote :

Reviewed: https://review.fuel-infra.org/16010
Submitter: Ivan Remizov <email address hidden>
Branch: master

Commit: dab1100da9f3c01158991f4349837e2b1eada2a8
Author: Ivan Remizov <email address hidden>
Date: Mon Jan 11 14:41:02 2016

Disable master-pkg-repotest-centos

Change-Id: I130342d5b22832cfedb1f8fc8013c377d38b86a1
Closes-Bug: #1521352
Depends-On: I0e3ad19cab75cba833730bfe5b7c7c9806e64104

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

Related fix proposed to branch: master
Change author: Ivan Remizov <email address hidden>
Review: https://review.fuel-infra.org/16040

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

Fix proposed to branch: master
Change author: Ivan Remizov <email address hidden>
Review: https://review.fuel-infra.org/16041

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

Reviewed: https://review.fuel-infra.org/16041
Submitter: Ivan Remizov <email address hidden>
Branch: master

Commit: f3163092b107f2e66157d03bc00e4aa4ce9e935d
Author: Ivan Remizov <email address hidden>
Date: Tue Jan 12 11:51:26 2016

Disable master-pkg-repotest-ubuntu

Change-Id: I5967004468efbd13fffbe4f51efae54d9efaa36e
Closes-Bug: #1521352

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

Reviewed: https://review.fuel-infra.org/16040
Submitter: Ivan Remizov <email address hidden>
Branch: master

Commit: 7e93fd7b360ae3df17e4231a42ddc773b2dfb0b8
Author: Ivan Remizov <email address hidden>
Date: Tue Jan 12 11:48:07 2016

Remove docker test from pkg-install job

`repo-test-{pkg_type}-docker` exists both in pkg-install and in pkg-repotest jobs.
This patch removes duplication.

Change-Id: I8e8b9c76090d31dd805e2409e53a1d4cf5f0e6ff
Related-Bug: #1521352

Ivan (iremizov)
Changed in fuel:
status: Fix Committed → Fix Released
Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Change abandoned on fuel-infra/jenkins-jobs (master)

Change abandoned by Alexander Evseev <email address hidden> on branch: master
Review: https://review.fuel-infra.org/14317

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Change abandoned on fuel-infra/zuul-layouts (master)

Change abandoned by Alexander Evseev <email address hidden> on branch: master
Review: https://review.fuel-infra.org/14318

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.