Rakefile shall launch unit tests for only changed modules

Looks like not a way to go. "If we change something like galera that is used by openstack but we don't check openstack how will we know if there are regressions"?

There is another opinion though, that the galera module changes shall be only covered by the unit tests of the *galera* module but not openstack or osnailyfactor. Those shall be covered by integration tests, like Noop framework. So, this point makes the bug valid back :) Let's discuss it in mail list perhaps

"If we change something like galera that is used by openstack but we don't check openstack how will we know if there are regressions"

I suppose that such problems are not a subject for unit tests. If I change something in galera module, then the only goal of unit tests is to check that functionality of puppet classes/function/resources of galera module is not broken, that's it. Any composition/integration layer issues (like the one described above) will be caught by noop tests.

I personally would like to keep some sort of integration rake tests that isn't NOOP for fuel-library because NOOP 1) takes forever and 2) does not actually run cleanly on my OSX machine (too many linux assumptions, not enough mocks). I think it would be acceptable to make the unit tests only check the changed modules (as done via fuellib_unit_tests) but I think it's also a good idea to run all of the modules in at least 1 job to check integration via rake rather than noop.

Related fix proposed to branch: master
Review: https://review.openstack.org/272346

We should consider skipping 8.0 for this and fix just in master.

Fix proposed to branch: master
Review: https://review.openstack.org/274286

This is actually not a bug, but CI enhancements. The changes don't included in the release and only intended to speed up the development process.

Related fix proposed to branch: master
Change author: Maksim Malchuk <email address hidden>
Review: https://review.fuel-infra.org/16576

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

commit f60d5ed950baefd2223388d66e71b7f037affffd
Author: Maksim Malchuk <email address hidden>
Date: Sat Jan 30 00:26:08 2016 +0300

    Rakefile support for Git changes

    This commit enables integration with Git. Now before run
    RSpec or Lint tasks Rakefile will check for Git changes
    and run these tasks on changed modules only otherwise
    they ran against all modules.

    Change-Id: I79ffbb8e2ed0c10cf03a1b2e176b75b314a3e79d
    Closes-Bug: #1537063
    Closes-Bug: #1513900

Fix proposed to branch: stable/8.0
Review: https://review.openstack.org/274769

Change abandoned by Maksim Malchuk (<email address hidden>) on branch: stable/8.0
Review: https://review.openstack.org/274769
Reason: https://review.openstack.org/#/c/274712
should be merged first

Fix proposed to branch: master
Review: https://review.openstack.org/274878

Reviewed: https://review.openstack.org/274769
Committed: https://git.openstack.org/cgit/openstack/fuel-library/commit/?id=4734ab2aec82fa3ccb1aa7a4143f9c3fe11fddaa
Submitter: Jenkins
Branch: stable/8.0

commit 4734ab2aec82fa3ccb1aa7a4143f9c3fe11fddaa
Author: Maksim Malchuk <email address hidden>
Date: Sat Jan 30 00:26:08 2016 +0300

    Rakefile support for Git changes

    This commit enables integration with Git. Now before run
    RSpec or Lint tasks Rakefile will check for Git changes
    and run these tasks on changed modules only otherwise
    they ran against all modules.

    Change-Id: I79ffbb8e2ed0c10cf03a1b2e176b75b314a3e79d
    (cherry picked from commit f60d5ed950baefd2223388d66e71b7f037affffd)
    Closes-Bug: #1537063
    Related-Bug: #1513900

Reviewed: https://review.openstack.org/274878
Committed: https://git.openstack.org/cgit/openstack/fuel-library/commit/?id=7de0ee245c03736e22c5b6943e65a7d20deeb151
Submitter: Jenkins
Branch: master

commit 7de0ee245c03736e22c5b6943e65a7d20deeb151
Author: Maksim Malchuk <email address hidden>
Date: Mon Feb 1 21:38:35 2016 +0300

    Parameter skip_file didn't work with git changes

    This commit fixes issue when skip_file didn't used at all

    Change-Id: Iaa783fb37d8000f0578019c8e257ab8e4c992e9b
    Partial-Bug: #1537063

Fix proposed to branch: stable/8.0
Review: https://review.openstack.org/274944

Reviewed: https://review.openstack.org/274944
Committed: https://git.openstack.org/cgit/openstack/fuel-library/commit/?id=c4c4c9943ead8ea573ee9ba6e051be57576316cc
Submitter: Jenkins
Branch: stable/8.0

commit c4c4c9943ead8ea573ee9ba6e051be57576316cc
Author: Maksim Malchuk <email address hidden>
Date: Mon Feb 1 21:38:35 2016 +0300

    Parameter skip_file didn't work with git changes

    This commit fixes issue when skip_file didn't used at all

    Change-Id: Iaa783fb37d8000f0578019c8e257ab8e4c992e9b
    (cherry picked from commit 7de0ee245c03736e22c5b6943e65a7d20deeb151)
    Partial-Bug: #1537063

Reviewed: https://review.fuel-infra.org/16576
Submitter: Aleksandra Fedorova <email address hidden>
Branch: master

Commit: 513c7a12a0481b4155a8442f7abe17681be3538e
Author: Maksim Malchuk <email address hidden>
Date: Sat Jan 30 21:32:55 2016

Remove fuellib review_syntax_check and unit_tests jobs

Jobs fuellib_review_syntax_check and fuellib_unit_tests
should be removed from stable/8.0 and master because we
already have the same functionality in the upstream
OpenStack jobs.

Change-Id: Ic7ce913a410d1e6379abf02a4eeb9cb197f85ba3
Closes-Bug: #1513900
Related-Bug: #1537063