The logic in the Gemfile was relying on Zuulv2 variables to find out
whether the spec helper gem was already available on disk, and since
Zuulv3 has changed things it was failing to find it and downloading the
master version instead. This patch ensures the Gemfile looks for the gem
in the right place when running in CI.
Change-Id: I74fd209d1b8257ebd146e50fa36fda8ceecc4e03
Instead of keeping a local copy of spec_helper_acceptance.rb and
requiring updates to all modules for any change, we can move it into the
common helper gem and require it from there. This will make it easier to
create and review changes that affect all puppet modules. Also change
the Gemfile to look for the gem in the local workspace if running in a
zuul environment.
Change-Id: Id8e6d1901d6f17a393d235682a8a864c3397698e
Add a xenial nodeset and update the spec helper to install puppet 3 from
the Ubuntu repos instead of from puppetlabs. Also fix linter errors.
Change-Id: Iae66db33213aadcaad76879e0fe9b40db4e65f86
Check the output of the `storage status` command for appearance of
"Not Applied" migrations, and run an upgrade if so. This works
around an earlier assumption that the command would exit nonzero
when there are pending migrations, but it seems to exit zero even
under those conditions.
Change-Id: I5ff8026bfaaf31c8abcd55efc217256605be9e73
Make SSL/TLS certificate file handling and snakeoil fallback logic
more flexible, no longer expecting you to unconditionally set file
paths so you can just provide contents and allow the module to store
them in a location it considers sane.
Also adjust the vhost template to match the pattern we've been
standardizing on across existing modules (such as our recently
improved mediawiki module), with clearer variable scope lookups and
stronger crypto configuration.
While in the vicinity, also remove a hard-coded default ServerAdmin
E-mail address default to make the module less OpenStack-community
specific.
Change-Id: I238682ac05bdd20ec293b7a16370563763e35249
This change provides the puppet files required to configure Phabricator
/ Pholio for HTTPD and requisite HTTPS certificates.
Change-Id: I2b04fac576417a032b1b4f961a4cc19378eeea7c
Implements: Spec Pholio
This set of patches delivers the changes required to install and
configure an instance of Pholio for the UX team.
Change-Id: I394da0070214af5ceadb3e236eca8a1b78562a72
Implements: Spec Pholio
This changes adds the following configuration options to Phabricator
/ Pholio:
* Configures MySQL options
* Timezone set to UTC
* Disables the requirement for email verification
* Disables the requirement to have an administrator approve new
accounts.
* Sets the default viewing policy to "public".
* Enables requiring HTTPS for all connections.
Change-Id: Iab189f099efaa5354d944a15cf15776f2b39788a
Implements: Spec Pholio
Order and intendation of those parameters are changed
to follow Puppet Style Guide recommendation [0].
Moreover, it will allow to an user to find much faster
a variable in a list of variables.
[0]. https://docs.puppetlabs.com/guides/style_guide.html
Change-Id: Ifa545d915d25f587e965dbb2852dbfe190f763a2
This has nothing to puppet-phabricator so it has
been moved to system-config.
Depends On: https://review.openstack.org/#/c/240682/
Change-Id: If7d6c0580de6ee57fde882ca1d2b097dff760485
This patch addresses the following:
- Corrects the default setting for isMailingList
(turns out people are *NOT* mailing lists. Someone tell marketing!)
- Corrects the default setting for isApproved
Changed the SQL syntax for the user migration for clarity.
Change-Id: I56cdc52706a026fea0c0c28883dc01da6fa1018f
Implements: spec maniphest migration
This change updates the migration script to work with the current
version of Storyboard and stable Phabricator.
Change-Id: Ie4b4a3a218221ed38a49e97ba78f9976668459c8
Implements: spec maniphest migration
Use same target directory for zuul-cloner and
the regular git command.
Change-Id: I4fb3d2535cffb3b4c2be74a230b827861ffb31dc
Co-Authored-By: Fabien Boucher <fabien.boucher@enovance.com>
In anticipation of puppet 4, start trying to deal with puppet 4 things
that can be helpfully predicted by puppet lint plugins. Also fix lint
errors caught by the puppet-lint-empty_string-check gem as well as
arrow alignment errors raised by the newer version of puppet-lint.
Change-Id: I541aff1e4848e46918f7d6e1750c2afa26fb2f3a
The mysql server needs to have a password set, so allowing the default
to be the empty string is not appropriate and will result in unexpected
failures.
Change-Id: I13c804509cbf9c922f367800d704e20ce5b818f9
puppet-httpd is the openstack-infra version of puppetlabs-apache
(0.0.4) release.
This patchset will remove the puppetlabs-apache namespace from -infra
allowing for possible future patchsets to use newer puppetlabs-apache
modules.
Change-Id: I816668b67f461bf1e6769a50a730716efcb2cf8e
Signed-off-by: Paul Belanger <pabelanger@redhat.com>
Phabricator complains about some of its MySQL configuration
on startup. Resolve this issue by providing an optional class
to handle MySQL configuration with the appropriate parameters
set.
Monty Taylor