The final check during the prep stage, where `puppet module list` is
run so that its output can be used for debugging, should run on the
current host in the loop, not the first host in the array. It should
only exit 0.
Change-Id: I5a29908f5c63209dd7416c5e281f60198a4b552c
This patch updates the spec_helper_acceptance.rb file to use the common
module install script and Puppetfile from the new
openstack/puppet-openstack-integration repo. This will allow us to take
advantage of Zuul's Depends-On feature so that we can test changes
across multiple related modules. This is based on idea 1 from this
brainstorming session[1].
This will probably be abstracted out into a separate function or
eventually a separate gem so that it can be more easily shared across
the OpenStack modules and the Infra modules.
[1] https://etherpad.openstack.org/p/puppet-git-dependencies
Change-Id: I9117cfd40a90059af9e3c9d2763ec05f6cdb39ed
Co-Authored-By: James E. Blair <jeblair@hp.com>
Co-Authored-By: Spencer Krum <spencer.krum@hp.com>
When running with BEAKER_provision=no, the following happens:
In Trusty: beaker is running dpkg with '--force' option so even
if Puppet is already installed, the return code will be 0.
In CentOS: beaker is running 'rpm -ivh' the second time and since
the package is already here, the return code will be 1.
Using the new puppet install helper will abort installing puppet
if BEAKER_provision is no, so tests will continue as normal.
Change-Id: I410ebcc23bdc27ae0e632d1f330cab374cbc8912
Co-Authored-By: Colleen Murphy <colleen@gazlene.net>
Since SELinux is enforced by default, we need to make sure
openstack-selinux is installed from the latest version to have OpenStack
correctly working.
Change-Id: I0b904b0f325a10ee952f07c5b3556765b53bf267
This patch splits out release notes into a separate dedicated file for
consistency with the puppetlabs puppet modules. Additionally, this
patch improves the release notes by:
- Fixing the tense to be consistent with commit message standards ("Add
new feature" instead of "Added new feature" or "Adds new feature")
- Breaking up changes into categories to make it easier for operators
to know what happened in the change: backwards-incompatible changes,
features, bugfixes, and maintenance commits
- Linewrapping to 80 chars
- Adding release dates, formatted according to ISO-8601
Change-Id: I015761a2f063b08f393e69c9ef6ba33a9a32003e
Install packages from RDO properly, by using regular packages and not
ones from testing repo that are too unstable.
Change-Id: I3ffb21aa6eeddac60e2d009e233ad330207293f5
This reverts commit b76fbcbea34b510191964cc3ed6a4c8e6f2acbfe.
With 268370457c118e62b93990cea31066c477ab0dbc now merged, we need to go
back to looking for os_region_name in [glance_store] instead of
[DEFAULT].
Change-Id: I197d012387d630ca6fb3a38c48f0603add6c6de1
This change deprecates the following parameters:
- port (replaced by public/internal/admin_url)
- public_protocol (replaced by public_url)
- public_address (replaced by public_url)
- internal_protocol (replaced by internal_url)
- internal_address (replaced by internal_url)
- admin_protocol (replaced by admin_url)
- admin_address (replaced by admin_url)
Add deprecation warnings if any of those values are provided
while maintaining full backward compatibility.
Closes-bug: #1274979
Change-Id: Iac152347534874f8763d8df4f81d1568d3c5e222
The testing keystone password was set to 'a_big_secret' throughout the
test manifest except for in the glance-api class, so the
glance-api.conf would have the wrong password and tests for
glance_image would fail. This patch sets the password to be consistent
across the manifest.
Change-Id: I8b781d3080b2dd5e894c84a23c892e4f153d8aa6
OpenStack Infra has jobs to run this on both Ubuntu Trusty and CentOS7.
* Add minitest to Gemfile (dependency to run beaker on centos - see 11
http://projects.theforeman.org/issues/2650 for details)
* separate nodepool files to have trusty & centos7 support in OS infra
* class_spec: add case for repo configuration and support
RH systems.
* don't install rabbitmq module that is not used (yet)
* install the apt module explicitly since it's no longer installed
implicitly by the rabbitmq module
Change-Id: I17a571c8d56d55337ee0c8491fb4aad272074cf6
Closes-bug: #1444736
Without this patch, the nodesets incorrectly define a vagrantcloud vm
to install, resulting in the following error:
> The box you're adding has a name different from the name you
> requested.
This patch uses the trusty vagrant nodeset from the puppetlabs-apache
module and updates the default nodeset as well. Fill do the same for
the centos nodeset in a separate patch after RedHat tests are
implemented.
Change-Id: Iffac76b48d496d93b9cb63323ac103619319de47
We want db_sync to run even though manage_service or enabled
can be set to False.
Change-Id: Ic3472ce6a6e704a2fa035df6e1c8866b8ea6bcbc
Closes-Bug: 1452278
The previous commit that changed the service description
from Openstack to OpenStack actually introduced a bug that means
that users cannot upgrade between those two versions of the code.
An attempt to use this manifest to upgrade from earlier versions
of the code before this change was made is currently not supported
because keystone_service does not support updating it's description.
This commit adds the service description as a class parameter so that
folks using older version of the module can update it to the incorrectly
spelled version.
Change-Id: Id8b01c7456b067c19fcf6cc1503c22a11b7d3fd9
In ::glance::api, os_region_name is set in DEFAULT which work for both
Juno & Kilo.
'glance_store' section has been introduced in Kilo which breaks Juno.
Until we decide our policy to manage master, let's use DEFAULT so both
releases work well.
Change-Id: I1c519146d3fe62f8c8fc8c2d85805db11af21e4c
Depends on https://review.openstack.org/#/c/174969/
This reverts commit e2301958e9dd4086fb6f5689499e5bb39e87e208.
Change-Id: I86db306b90cfdba53cf40bd9e3b2cce453e821d0
When we revert the change that moved os_region_name, we must also
revert this fix that resulted from that change.
Depends on https://review.openstack.org/#/c/174968/
This reverts commit 354ae692a502a6089f4f82b3b58ee2aaa42443a2.
Change-Id: I65249b0734ae1a1bfbf0e5e153d19787f6aded5f
This reverts commit d302170d260f4e6585fedc1b4709c953d0491c56.
We can unrevert this change once we are ready for kilo specific
changes.
Change-Id: I3079727c8333de3e9b4382a68fca3ef293ce2d48
This reverts commit e6e8f6a56231409028ff1e7cdcd44781ca37d5b0.
Kilo deprecates these settings, but they still work. We should wait to
merge this until upstream distro packages are ready so that master is
not broken.
Change-Id: Ib05943e89969eb810527943426c566cf4a77e7ad
