As per OpenStack licensing guide lines [1]:
[H104] Files with no code shouldn't contain any license header nor
comments, and must be left completely empty.
[H102 H103] Newly contributed Source Code should be licensed under
the Apache 2.0 license.
[1] http://docs.openstack.org/developer/hacking/#openstack-licensing
Change-Id: I2b4749b688468a042111e8e21fa8d12c16b161ee
This patch set modifies lines which are importing objects instead
of modules. As per openstack import guide lines, user should import
modules in a file not objects.
http://docs.openstack.org/developer/hacking/#imports
Change-Id: I863d4433a69712cfda5f8df89e55a6ca2f820f64
Prevent false positive results in cases when something
is wrong with the environment and tests have not started at all.
Change-Id: Icb8cd6ebe523ef51505ab2c7daadb10c62aa220f
Related-Bug: 1586480
* Disable STRICT_VARIABLE checks.
* Switch to the Puppet version 4.5.0
* Disable Noop tests for Puppet4
they should be moved to a separate job
Related-Bug: 1586480
Change-Id: Ib540dc243647fffeb4b2d101f9f902b9e4c1c104
As we already have yaql expression almost in each task, implement
basic syntax tests for these expressions.
Change-Id: Idb7768eca9be3def4783a70b2ef3b1dafc0d0510
Closes-Bug: #1588910
DocImpact
* Update cinder-vmware conf for cinder multi backend
* Fix the spec framework for this module so the
spec can be actually used
* Disable non-working specs in the vmware modules
somebody should fix them later.
* Add spec for vmware::cinder::vmdk
It tests only this templates and
should be finished later too.
* Enable spec for this module in the Fuel CI
Change-Id: Id3b8cdb10acd0d7a63a085c03795169c5f304c13
Closes-Bug: #1569874
Closes-Bug: #1496009
Using Include and Require functions for class
declaration allows the class to me declared many
times within a single manifest without any
duplicate decalration error. But the cost of this
is that the file and line where the class is declared
is not saved in the catalog object.
The Coverage parser cannot determine if this class was
declared in the particular task and skips these classes
making the coverage report in the Noop tests much less
relevant.
Change-Id: Ie5d5ced4a9225bda5b0bf6c2a733c0d44c48118c
- Mellanox functionality was moved to plugin. Tasks were deprecated in
8.0. This patch removes orphaned manifest
Change-Id: Iec4ca60310e6df352401ed7465c1784acc4a2ad9
Closes-Bug: 1567482
Signed-off-by: Sergii Golovatiuk <sgolovatiuk@mirantis.com>
We should switch to stable/mitaka branch of puppet modules
for stable/mitaka branch of Fuel. Once the development of
Newton Fuel will be started the refs will be returned to master.
Change-Id: If9ce7009acab8cf83a9b82e1347447adefb18e5b
All puppet stuff related to dynamic dnsmasq feature
was moved to 'fuel' module after Docker removal.
Also it's necessary to setup Hiera DB on master node
in order to get manifests for update DHCP ranges work.
Change-Id: Ic2b90ff3464a6cb5a83c660386e6da47f90e0f88
Closes-bug: #1549409
Use pcmk_* resources from the new pacemaker module
instead of the old cs_* resources from the upstream
corosync modules and custom cs_rsc_* resources from the
duct tape patched corosync module.
The new pacemaker module is rewritten from the scratch
and provides improved performance, transparency and
debuggability. It uses it's own XML engine and depends
neither on "pcs" nor on "crm" applications.
"corosync" modules will still be used to install the
corosync cluster, and the "pacemaker" modules will
be used to work with the cluster's configuration.
Refactor pacemaker services to use pacemaker::service
wrapper. It controlls primitive creation and run in the
uniform way and creates ocf_handler helpers for
manual primitive management.
Add run_failed_log feature to fuel_noop_tests to
replay the failed tests after they were fixed.
Change-Id: Ifd5813c657031cdd7b86368cd971742483a5b8a1
Related-Bug: 1414583
Related-Bug: 1453773
Related-Bug: 1482248
Do not put debug messages.
Show all test results instead of failed only.
Align concurency to cpucount.
Also fix bashisms in rm commands.
Related blueprint: deployment-data-dryrun
Change-Id: I05438471873efb44e7432c6544ea44919e6e0a8c
Signed-off-by: Bogdan Dobrelya <bdobrelia@mirantis.com>
It's incompatible with MOS Liberty change.
Key items:
* switch puppet-horizon module to master
* always run collectstatic&compress for MOS/UCA packages
* remove workaround for openstack-dashboard-apache package:
Liberty MOS packages didn't contain 'openstack-dashboard-apache' package,
but Mitaka MOS contains and it will be installed by default because
os_package_type for MOS is 'debian':
https://github.com/openstack/puppet-horizon/blob/master/manifests/params.pp#L32-L34
For UCA packages we are using os_package_type set to 'ubuntu', so UCA isn't affected.
* remove workaround for neutron packages:
Naming scheme for Liberty MOS packages was aligned with UCA,
for Mitaka MOS it's aligned with debian:
https://github.com/openstack/puppet-neutron/blob/master/manifests/params.pp#L104-L107
UCA packages are still not affected because of os_package_type set to 'ubuntu'.
Closes-Bug: #1517508
Partial-blueprint: puppet-mitaka-modules
Change-Id: I45274034b158171cd390d82dd78d707c98510e23
* Use external noop tests library and fixtures
* Corrections and fixes for the spec library to make it work
Partial blueprint: deployment-data-dryrun
Fuel-CI: disable
Change-Id: Ie8d1f80bbee04b9bacb43ba4fa39a423624a8301
Since removing this module, we no longer need this exception.
Change-Id: Ie651a6484b254d9bd09d389eaa4dc23a8280ca33
Related-Blueprint: granular-task-idempotency
This change updates the heat task to be idempotent. In order to do this,
we have changed the wait_for_heat_config exec to be refresh only and is
now only triggered if configuration changes are done. Additionally this
change cleans up the pacemaker integration for heat and moves it into
the cluster module. With the addition to the cluster module, unit tests
have been added to the new cluster::heat_engine class.
The resources that were always being executed prior to this change:
- Cs_resource[p_heat-engine]
- Exec[wait_for_heat_config]
Change-Id: Ie0f7137206f2733259bff7b0c2c86fb1634add76
Related-Blueprint: granular-task-idempotency
Update version of 'networkx' package from requirements.txt,
'mock' package from test-requirements.txt.
Remove 'argparse' from requirements.txt as it's included in Python 2.7.
Change-Id: I042c223feddd6218beab6904c34d222ec3f1bd67
Closes-bug: #1542814
This commit adds support to automatically exclude modules
defined in the fuel-library Puppetfile from rspec and
lint tests.
Change-Id: I398c7824d3cfb76002eaed962b9161252a14a529
Closes-Bug: #1540573
We want no more downstream efforts for the module.
All patches shall be sent upstream only now.
Closes-bug: #1534560
Change-Id: I82753a625672635de2ddd340d60d7fbe344ad50a
Signed-off-by: Bogdan Dobrelya <bdobrelia@mirantis.com>
This change also updates the puppetfile validation script to allow for
github.com/fuel-infra
Change-Id: I274ec4c2f12b9effb76d59499d58ee713c78664d
Related-Bug: #1535832
Co-Authored-By: Alex Schultz <aschultz@mirantis.com>
This change updates the fixtures files for the modules within
fuel-library to use the relative path to the upstream modules used by
fuel-library. With this change we are also adding an initial prep task
that is used by CI to pull down all the modules first and only one time
to reduce the amount of repeated calls to the upstream git repositories.
Included in this change is also an update to exclude all the upstream
modules from the spec and lint tasks as those are maintained in the
upstream modules and should not be done when testing fuel-library.
This change also adds documentation to the README.md on how to run the
tests from the root of fuel-library.
Change-Id: I9cde21b94a8367fd45cf046af48d7618076a385c
Related-Bug: #1535832
Backwards compatible change to prepare
the noop tests to consume external fixtures
from the .fixtures.yml file
Partial blueprint deployment-data-dryrun
Change-Id: I4ddb580d1179caa81d0320520bccbe9f6e11c5c1
Signed-off-by: Bogdan Dobrelya <bdobrelia@mirantis.com>
This change updates the scripts used by Fuel CI to pull in puppet 3.8
rather than 3.4.
In order to support testing under Puppet 3.8, we have to setup the
puppet settings as part of our overrides setup. This is due to a change
after 3.4 where these settings are no longer setup as part of the
TestHelper inititalize call. This was done as part of PUP-11118.
Without setting up up the settings, rspec may error when we attempt
to access the Puppet scope or functions.
Change-Id: I26c20036cc9a829fc5d7d351530ac358b1a5ace8
Related-Bug: 1500414
Closes-Bug: 1513969
This change fixes the regression introduced by
I814667283ccfc6ebd7c16d961270f688e9fb9c4c in which the rspec coverage
information was no longer being written out. Additionally this change
enables coverage report information to be generaged by the Jenkins jobs
as part of the default run via fuel_noop_tests.sh. This change will
allow us to update the jobs to capture the information with Jenkins and
report on trends.
Change-Id: I6cc729d7ff2928bdf78ee17487f2158b4ef12490
Closes-Bug: #1524967
Related-Bug: #1506557
* Add basic doc build system with sphinx and tox
* Update README.md for how-to build docs
* Update .gitignore to exclude docs builds
* Add index and move README for Noop tests framework
* TODO split and add more details for how-to noop tests
setup/use/create
Inspired by https://github.com/openstack/openstack-manuals
Partial-bug: #1483288
Change-Id: I982e0baaff5de95c7d0c4b3f098754a80676f0d0
Signed-off-by: Bogdan Dobrelya <bdobrelia@mirantis.com>