New jenkins job needed for fuel-library unit tests

Bug #1396510 reported by Aleksandr Didenko
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Released
High
Igor Belikov

Bug Description

As a part of https://blueprints.launchpad.net/fuel/+spec/fuel-library-rspec-testing we need a new jenkins job with gerrit trigger for "fuel-library" project. This job will be used as fuel-library unit tests for new commits to fuel-library.
While we're testing and improving it, it should not put +1/-1 in the gerrit reviews, if it's possible.

Could you please create it and provide me with the access to configure this job?

Changed in fuel:
importance: Undecided → Medium
Revision history for this message
Aleksandra Fedorova (bookwar) wrote :

Access to Fuel Jenkins is limited and we shouldn't use it for tests and temporary configurations. We can create Jenkins job at internal Jenkins for you to play with, but I don't see the purpose of it.

To create Jenkins job we need:

1) requirements - which packages should be installed on the server to run the script,
2) bash script - what we need to run from the checked out fuel-library code.

Both items can be initially tested without Jenkins or Gerrit Trigger involved. Internally Jenkins just produces the source code which you need to test, nothing more.

Once requirements are clear and script is tested, we update our puppet manifest for Jenkins slaves and update our slaves to fit the requirements. After that we can create Jenkins job via Jenkins Job Builder.

Changed in fuel:
status: New → Incomplete
assignee: Fuel DevOps (fuel-devops) → Aleksandr Didenko (adidenko)
Changed in fuel:
importance: Medium → High
Changed in fuel:
status: Incomplete → In Progress
Revision history for this message
Aleksandr Didenko (adidenko) wrote :

1) requirements - which packages should be installed on the server to run the script,

"bundle" ruby gem with appropriate "bundle" binary

2) bash script - what we need to run from the checked out fuel-library code.

./utils/jenkins/fuel_unit_tests.sh

But the script will work only after https://review.openstack.org/#/c/137334/ is merged. So we're waiting for the merge for now.

Revision history for this message
Aleksandr Didenko (adidenko) wrote :

Please install requirement on on https://fuel-jenkins.mirantis.com jenkins slaves:

1) requirements - which packages should be installed on the server to run the script,
"bundle" ruby gem with appropriate "bundle" binary

Changed in fuel:
assignee: Aleksandr Didenko (adidenko) → Fuel DevOps (fuel-devops)
importance: High → Critical
Igor Belikov (ibelikov)
Changed in fuel:
assignee: Fuel DevOps (fuel-devops) → Igor Belikov (ibelikov)
Revision history for this message
Igor Belikov (ibelikov) wrote :

"bundle" ruby gem and binary are installed on fuel jenkins slaves.

Changed in fuel:
assignee: Igor Belikov (ibelikov) → Aleksandr Didenko (adidenko)
Igor Shishkin (teran)
tags: added: jenkins
Revision history for this message
Aleksandra Fedorova (bookwar) wrote :

I've checked that https://fuel-jenkins.mirantis.com/job/fuellib_review_syntax_check/ works with both old and new version of the script, so we can safely use the same job for master and stable branches.

Revision history for this message
Aleksandr Didenko (adidenko) wrote :

That's great, thank you. We'll use a copy of this syntax job for unit-tests and noop tests, since they use the same approach. I'll also prepare unit test script for stable branches and let you know when done.

Changed in fuel:
milestone: none → 6.1
importance: Critical → High
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-library (master)

Reviewed: https://review.openstack.org/137334
Committed: https://git.openstack.org/cgit/stackforge/fuel-library/commit/?id=ba4b1c92372dd42a0f68fa5d567694948ea13a22
Submitter: Jenkins
Branch: master

commit ba4b1c92372dd42a0f68fa5d567694948ea13a22
Author: Aleksandr Didenko <email address hidden>
Date: Wed Nov 26 11:24:44 2014 +0200

    Unit tests script for fuel-library

    Script that will run unit tests (rake spec) for needed modules as
    a part of our CI for fuel-library project. Currently it has a list
    of modules with disabled unit tests, due to large amount of broken
    or missing rspec tests. We'll fix those eventualy and reduce the
    list of modules without unit tests.

    Also includes:
      - fixes for swift unit tests broken by our modifications
      - fixes for murano syntax errors

    Implements blueprint: fuel-library-rspec-testing
    Partial-bug: #1396510

    Change-Id: Id53d8d6f36fdcdf7766e143bbb9b788af299882b
    Signed-off-by: Sergii Golovatiuk <email address hidden>

Revision history for this message
Aleksandr Didenko (adidenko) wrote :

OK, the script is merged into master branch.

Is it possible to create a non-voting CI job for 'master' branch only? Because we don't have this script in other branches yet.

1) (should be installed already for syntax check job) requirements - which packages should be installed on the server to run the script,

"bundle" ruby gem with appropriate "bundle" binary

Actually it would be better to copy https://fuel-jenkins.mirantis.com/job/fuellib_review_syntax_check/ job because unit tests job is going to use the same approach with 'bundle' gem.

2) bash script - what we need to run from the checked out fuel-library code.

./utils/jenkins/fuel_unit_tests.sh

Changed in fuel:
assignee: Aleksandr Didenko (adidenko) → Fuel DevOps (fuel-devops)
Igor Belikov (ibelikov)
Changed in fuel:
assignee: Fuel DevOps (fuel-devops) → Igor Belikov (ibelikov)
Revision history for this message
Igor Belikov (ibelikov) wrote :
Changed in fuel:
status: In Progress → Fix Committed
Igor Belikov (ibelikov)
Changed in fuel:
status: Fix Committed → Won't Fix
status: Won't Fix → Fix Released
tags: added: system-test-not-required
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.