[CI] a new job for running bash unit tests

Bug #1471710 reported by Bartłomiej Piotrowski
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Released
Medium
Mateusz Matuszkowiak

Bug Description

One review[1] is introducing unit tests for a bash script using bats[2]. We currently do not have any way to run such tests against linked scripts in Jenkins.

[1] https://review.openstack.org/#/c/198355/2
[2] https://github.com/sstephenson/bats

tags: added: devops
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: Mateusz Matuszkowiak <email address hidden>
Review: https://review.fuel-infra.org/9130

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

Related fix proposed to branch: master
Change author: Mateusz Matuszkowiak <email address hidden>
Review: https://review.fuel-infra.org/9132

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

Reviewed: https://review.fuel-infra.org/9132
Submitter: Andrey Nikitin <email address hidden>
Branch: master

Commit: 2030289b7e888073cdf9e2ef2ee44f6bc3937665
Author: Mateusz Matuszkowiak <email address hidden>
Date: Wed Jul 8 10:32:18 2015

Added new fuel-infra-packages/bats project

Change-Id: I64ee9146e7f4113874a5e0d496d45918789fafd1
Related-Bug: #1471710

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

Related fix proposed to branch: master
Change author: Mateusz Matuszkowiak <email address hidden>
Review: https://review.fuel-infra.org/9134

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

Reviewed: https://review.fuel-infra.org/9134
Submitter: Mateusz Matuszkowiak <email address hidden>
Branch: master

Commit: fafd07c6c08f07457b2f2c687b25204856731253
Author: Mateusz Matuszkowiak <email address hidden>
Date: Thu Jul 9 13:28:51 2015

Added bats package for later usage by infra-ci

Change-Id: I37a072b42df970a26b577315f023896c5a10e68a
Related-Bug: #1471710

Igor Shishkin (teran)
Changed in fuel:
milestone: 7.0 → 8.0
Revision history for this message
Bartłomiej Piotrowski (bpiotrowski) wrote :

To make things clear, said job should be executed on every patchset, like "project-pattern: 'fuel-infra/jenkins-jobs'".

Dmitry Pyzhov (dpyzhov)
tags: added: area-devops
Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Related fix proposed to fuel-infra/puppet-manifests (master)

Related fix proposed to branch: master
Change author: Mateusz Matuszkowiak <email address hidden>
Review: https://review.fuel-infra.org/13882

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

Reviewed: https://review.fuel-infra.org/13882
Submitter: Mateusz Matuszkowiak <email address hidden>
Branch: master

Commit: 2ed5e43b7c70c16e9b891f800bd3472d0ad515bd
Author: Mateusz Matuszkowiak <email address hidden>
Date: Tue Nov 17 08:35:16 2015

Move bats package from infra-ci to fuel-ci

Change-Id: I208c0501c3767af08526c24d54c15097f49a4a4f
Related-Bug: #1471710

Revision history for this message
Igor Shishkin (teran) wrote :

@Mateusz, the commit linked above is merged what else is left on our side?

Revision history for this message
Bartłomiej Piotrowski (bpiotrowski) wrote :

The main patchset, adding actual job, isn't merged yet.

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/9130
Submitter: Igor Belikov <email address hidden>
Branch: master

Commit: 053eff20026c858fe167783d450b5fa9cb8e3d11
Author: Mateusz Matuszkowiak <email address hidden>
Date: Wed Nov 25 13:19:13 2015

New jenkins job to run bash unit tests

This job will run against every patchset created in a fuel-library repository.
Also, explicitly its shell script is not using the '-e' bash flag to
avoid premature builds job failing.

Change-Id: I36ebebfb1211ff890584e12196056cdf43183f0e
Partial-Bug: #1471710

Revision history for this message
Mateusz Matuszkowiak (mmatuszkowiak) wrote :

Job has been merged, now its up to ci-team to push that job into the fuel-ci jenkins.

Changed in fuel:
status: In Progress → Fix Committed
Revision history for this message
Mateusz Matuszkowiak (mmatuszkowiak) 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.