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 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
Due to a patch in RDO packaging, puppet-glance is broken for RHEL platforms.
This reverts commit 5f15e05704b2e549b8fab1c643a6ee743a0e92f4.
Change-Id: I7ba146714d1b0e8d461c556eff5a9f8c4ad9627f
Commit e6e8f6a56231409028ff1e7cdcd44781ca37d5b0 changed the section
where the os_region_name setting is placed from 'DEFAULT' to
'glance_store'. However, the provider was not updated to reflect
this change, so it broke.
Change-Id: I9e91faac2a45ac4eba4cc27c328e8dc67e619ab3
Puppet Labs is in the process of releasing a 2.0.0 version of
puppetlabs-concat, which will have many breaking changes that affect
many dependent modules. Changes have been merged to master, but not yet
released. While we figure out what changes we need to make in our own
modules and wait for updates in dependent modules, we should just pin
to a stable version.
Change-Id: Ie53c9c0c3f7ad84d310372e24eacc02a710cacf5
The location property of the glance_image type has nearly identical
functionality as the source param, yet one is a param and one is a
property. It is possible to change either of these for a glance image,
but only if it is as of yet unset, and therefore unsaved. It is not
possible to change the location of a glance image once it has been
saved, nor is it possible to query it afterward. Since having location
be a property when it is usually unsettable and is always unqueryable
does not make much sense, we change it to be a param to be consistent
with the source param.
Change-Id: I686c3fec9bf571d18e282888c626d795c9958a6b
Starting in Kilo, Red Hat-based platforms are using separate
glance-api and glance-registry packages (for an example, see
http://trunk.rdoproject.org/f21/current/).
Change-Id: I3ad476846d7dec3ca612e5b0d20b21b8577f4af0
Other components offer the option to decide whether or not to run the db sync
command. Glance was missing this feature. This commit add this feature for
Glance.
Change-Id: Iec31d724370a40aee56ead5839ef47ceacf7223d
Kilo oslo-messaging deprecates rabbit/kombu settings in the
DEFAULT section.
Move these settings to proper oslo_messaging_rabbit section.
Change-Id: Ica8cd70ece41f55b827f74d3f14cbc35f37ae13d
Setting up cache requires ops to be aware of how cache works in Glance
and the fact they'll need more space on disk. It can be dangerous when
using shared storage (Ceph for example).
This patch aims to default to 'keystone' without cachemanagement
anymore.
This is not backward compatible, so if the deployer is not overriding
this parameter, Glance API won't manage the cache anymore.
Change-Id: Ibac35501658b2ba79081cbc7a53245d4a4eb7839
It makes possible to control the value of notification_driver setting and defaults
to 'messaging' to maintain backwards compatibility.
Change-Id: I3f2c29adbfb605ef288962f72ba81c11811cb357
Previously when you set $configure_endpoint to false and did not have a
keystone_endpoint resource with the same name as is defined by
glance::keystone::auth, the catalog would not be compiled. This was
because a relationship was being established where one resource in the
relationship didn't exist.
This changes this so that the relationship involving the Keystone
endpoint is only defined if $configure_endpoint is set to true.
Also, the test for when $configure_endpoint is false was set to check
for an endpoint that doesn't get created even if $configure_endpoint is
true. Fixed so that it checks for an endpoint that has the correct name
(the same as when using the default values to call the class).
Change-Id: I6631a656a888d0df7ab173705bea0598bd2b2f58
Closes-Bug: 1368686
After https://review.openstack.org/152321 was merged, it was no
longer guaranteed that some now deprecated settings like auth_host
and auth_protocol would be present in glance-api.conf. This breaks
the Glance provider, which expects them to be present.
This patch adds support for identity_uri in provider, while
keeping backwards compatibility.
Change-Id: Ie8a2e45a4afea4828e02af36e64d65e7d4b7557e
This patch aim to update our specs test in order to work with the rspec-puppet release
2.0.0, in the mean time, we update rspec syntax in order to be prepared
for rspec 3.x move.
In details:
* Use shared_examples "a Puppet::Error" for puppet::error tests
* Convert 'should' keyword to 'is_expected.to' (prepare rspec 3.x)
* Fix spec tests for rspec-puppet 2.0.0
* Upgrade and pin rspec-puppet from 1.0.1 to 2.0.0
* Clean Gemfile (remove over-specificication of runtime deps of puppetlabs_spec_helper)
Change-Id: Ibd0333dc3c0959e846edfbad51fb48e8dfb8fbe9
Card: https://trello.com/c/eHXc1Ryd/4-investigate-the-necessary-change-to-be-rspec-puppet-2-0-0-compliant
