Without this, some OpenStack services will return invalid URLs if
you are using SSL termination at a proxy in front of the service.
Change-Id: I3ee6b5838f4703e3b8b1b6632dd45c94057b2202
Closes-Bug: 1597935
This file is to be included by all puppet modules.
First feature is to make sure that allow_virtual is set to true for all
openstack modules. It's set to false for puppet 3.6.1 to
4.0.0 [1]
[1] https://docs.puppet.com/puppet/3.6/reference/release_notes.html#puppet-361
Change-Id: I9d0a10d48124ee71ea80134ceef96436c8ee196e
Closes-Bug: 1599113
Related-Bug: 1597753
Depends-On: I2699eaa3a10589c9a0c680bb1de489994fe01b67
This is useful when the user (a puppet provider) is expecting a error.
With this it doesn't have to wait for the long retry cycle to have its
error back. This replace the awkward mechanism in
self.request_without_retry which was dynamically mangling
request_timeout and disabled even legit retry.
Examples of such function are self.fetch_user, self.fetch_project in the
keystone provider.
Change-Id: I589da0100ad3ccf3abf17ac8b26c827793ace484
Related-Bug: 1597357
We had a prelude note when releasing Mitaka. This note
keep showing-up at every release while we're on Newton now.
We don't need a prelude release note, we already have nice sections
for every note we push.
This patch aims to remove this prelude starting from Newton.
Change-Id: I50918c49bb7ce20d241159d9ecbbff6a1a2c1999
Change Ic54b9707b6ef36032d71fe9dc04a7e310293f127 missed release note,
so this patch adds it.
Alos improve unit test for policy-rcd provider.
Change-Id: Ie6d4dd1e5406acd097969a9ff7fccd5dfa7f8632
Use provider for managing policy-rc.d file to be sure that
services stopped right after package installed.
Also added tests.
Change-Id: Ic54b9707b6ef36032d71fe9dc04a7e310293f127
Mysql puppet providers apply downcase method for hostnames, that
lead us to errors when we use hostnames with upper-case letters.
Change-Id: I0da9e9aac3504d3defdfbb8b916ae7e9ae2339db
Closes-Bug: 1584247
Bindep is a binary automation tool used in openstack-infra. With it,
projects can better control the default packages installed into the
OS. Because we are using puppet, we actually don't want any packages
installed by default as to better test our manifests.
Change-Id: If45b2a41aa6fa8d187a9a341d547b16d0b378cbf
Co-Authored-By: Paul Belanger <pabelanger@redhat.com>
This change updates the require statements in the providers
to resolve the issues with the autoloader in puppet. This
issue technically always existed but was masked prior to the
split with puppet server and puppet agent because both used to
share the same loadpath. Documentation on this issue can be
read at https://tickets.puppetlabs.com/browse/PUP-4450.
Change-Id: I72a27fe92a41a5eb845adba6b82252b8aac1c164
On debian os family systems the methodology is to
start services when installed. This causes a problem
with keystone sometimes at random in CI jobs.
Change-Id: Id0b38743a9bf536f69d155e1d6e664a5585e5e1d
It may be useful to be able to execute a command without a retry.
A good use case is provider in [1], where the keystone_user resource try
to get an user by fetching it. It is expected to fail when the user is
absent. With the current implementation, it will takes 60 seconds for
the provider to give up on the user.
[1] https://review.openstack.org/299301
Closes-Bug: #1563898
Change-Id: I5b334e3ffd26df4ba8584d77a5e41b56e73536c8
Give non-retry calls the full timeout seconds to finish and also
don't log that you're going to retry them when you are not.
Change-Id: I51cbe3905b55f035718245b45bfc34536b8ce602
EPEL is causing CI failures randomly, because Fedora servers are quite
often down and do not offer EPEL repos in a stable way.
This patch just disables the usage of EPEL which is not needed to
download RabbitMQ.
Change-Id: Ifc04092b924b867063fc5c50960b1600b3fe379d
* Update CHANGELOG with release notes URL.
* Update README with new release number & name.
* Update metadata.json with new tag & dependencies.
* Add release note prelude for Mitaka.
* Update Reno release note to drop prelude.
* Improve release note landing page.
Change-Id: Id57b311e22d813f01d2ae8bc5003610a0544808b
ReNo [1] is the release management tool in OpenStack.
This patch adds the basic structure to start using it.
* Update .gitignore
* Add a basic note "use-reno"
* Add releasenotes/ dir and basic files
* Add python files: setup.cfg, setup.py, test-requirements.txt and
tox.ini.
[1] http://docs.openstack.org/developer/reno
Change-Id: I77f0375f185053425906ee52f6ad3bc83c7dd24d
This add the function normalize_ip_for_uri to the parser. It encloses
into brackets any valid IPv6 address thrown at it.
Change-Id: I093dd5a4e6294e20761cb3d33373652eeadeac36
Closes-bug: 1531960
mock is not used anywhere in the module, let's drop it.
This patch will help to hit this bug:
https://launchpad.net/bugs/1492636
Change-Id: I9a131fb79ba112a5dc1b774e31c5b5381d46d2ee
This change adds the rspec-puppet-facts gem to the spec helper so that
we can centralize the management of the base operating systems that we
support. rspec-puppet-facts allows us to simplify our unit tests and
provides a more complete list of Operating Systems and their associated
facts for the unit tests. With this change we can now loop over and
test CentOS, Debian, Fedora, RedHat, and Ubuntu by simply providing a
list of supported os to rspec-puppet-facts.
Additionally this change includes a central object for managing our
supported os list for rspec-puppet-facts and providing our default facts
like os_service_default. This central object should replace the usage of
@default_facts within the unit tests for each module.
Change-Id: Id2b35992bfe360a12412652270c738423f857906
- fixed idempotency issues with single string values
- fixed deletion of multiple values
This patch is required for I95cf32c3211bc4498eaa68e6e748a27dfd9af0fa to pass
acceptance tests
Change-Id: I619de3038cd2690bebe47cd601c085692506ac3d
When :array_matching => :all is used for value, the values comes
in form of array even when they are passed as single string and
so <SERVICE DEFAULT> mechanism does not currently work without hack
on each inherited resource.
This is required for I95cf32c3211bc4498eaa68e6e748a27dfd9af0fa to pass
acceptance tests.
Change-Id: Ibbc8a376496b8120d7c1d87f7a9005024ae14984
Add support for providing additional parameters to be used when
constructing the database uri. This change adds an 'extra' parameter
which is a hash that will be joined together when constructing the
database uri. It should be noted that the charset option, which can be
provided as a standalone option, will override charset when passed in
the 'extra' parameter. This is to maintain backwards compatibility.
Usage Example:
os_database_connection({
...
extra => { 'charset' => 'utf-8', 'read_timeout' => 60 }
})
Change-Id: I7ef078b76ac6cd7bea42a0ac92b383cefd47167a
Co-Authored-By: Alex Schultz <aschultz@mirantis.com>
