summaryrefslogtreecommitdiff
AgeCommit message (Collapse)Author
2019-03-24Replace openstack.org git:// URLs with https://HEADmasterIan Wienand
This is a mechanically generated change to replace openstack.org git:// URLs with https:// equivalents. This is in aid of a planned future move of the git hosting infrastructure to a self-hosted instance of gitea (https://gitea.io), which does not support the git wire protocol at this stage. This update should result in no functional change. For more information see the thread at http://lists.openstack.org/pipermail/openstack-discuss/2019-March/003825.html Change-Id: I2b03578180d2afc582438d1dc1b8cfbf8547df3f Notes (review): Code-Review+2: Jeremy Stanley <fungi@yuggoth.org> Workflow+1: Jeremy Stanley <fungi@yuggoth.org> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Tue, 26 Mar 2019 14:09:03 +0000 Reviewed-on: https://review.openstack.org/646931 Project: openstack-infra/puppet-ethercalc Branch: refs/heads/master
2018-08-13Use versioncmp functionColleen Murphy
Without this patch, puppet uses lexicographic ordering to compare the version string in the operatingsystemrelease fact to the string '14.04', which works fine on puppet 4 since it is comparing two strings but it is more correct to use the versioncmp function. Change-Id: I56d989d22229db3d606e4ac7f788925c216721a0 Notes (review): Code-Review+2: Clark Boylan <cboylan@sapwetik.org> Code-Review+2: Ian Wienand <iwienand@redhat.com> Workflow+1: Ian Wienand <iwienand@redhat.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Mon, 13 Aug 2018 22:57:00 +0000 Reviewed-on: https://review.openstack.org/590489 Project: openstack-infra/puppet-ethercalc Branch: refs/heads/master
2018-08-13Add test for ethercalc::apacheColleen Murphy
Add the ethercalc::apache class to the test fixture and ensure the service is accessible over port 80. This also corrects a minor idempotency issue in this class due to the way the ssl-cert package and the /etc/ssl/private file resources were ordered. Change-Id: I4cdba5b1881c3e9891ca9f8c2c4dd50ca5662d4e Notes (review): Code-Review+2: Clark Boylan <cboylan@sapwetik.org> Code-Review+2: Ian Wienand <iwienand@redhat.com> Workflow+1: Ian Wienand <iwienand@redhat.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Mon, 13 Aug 2018 22:56:20 +0000 Reviewed-on: https://review.openstack.org/585294 Project: openstack-infra/puppet-ethercalc Branch: refs/heads/master
2018-08-13Add idempotency testColleen Murphy
Add a test to ensure the module is idempotent. By doing this extra work, the ethercalc service has enough time to figure things out and get started, so we can also enable the checks to ensure the service is responding. Change-Id: Iede6db9dda0814bee9072c6dcc554ccc1aa89af1 Notes (review): Code-Review+2: Ian Wienand <iwienand@redhat.com> Code-Review+2: Monty Taylor <mordred@inaugust.com> Code-Review+2: Clark Boylan <cboylan@sapwetik.org> Workflow+1: Ian Wienand <iwienand@redhat.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Mon, 13 Aug 2018 22:56:14 +0000 Reviewed-on: https://review.openstack.org/585293 Project: openstack-infra/puppet-ethercalc Branch: refs/heads/master
2018-08-13Add workaround for ethercalc bugColleen Murphy
There is a bug with global npm installs[1] which causes node to be unable to find the SocialCalc.js module. We can work around it by symlinking the socialcalc module into the location that node expects to find it[2]. [1] https://github.com/audreyt/ethercalc/issues/542 [2] https://github.com/audreyt/ethercalc/issues/542#issuecomment-330039410 Change-Id: I6f751c19c772cda74f5a7b601238b4519039d257 Notes (review): Code-Review+2: Clark Boylan <cboylan@sapwetik.org> Code-Review+2: Ian Wienand <iwienand@redhat.com> Workflow+1: Ian Wienand <iwienand@redhat.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Mon, 13 Aug 2018 22:56:14 +0000 Reviewed-on: https://review.openstack.org/585292 Project: openstack-infra/puppet-ethercalc Branch: refs/heads/master
2018-08-09Turn off binary symlink in nodejs moduleColleen Murphy
On Xenial, the nodejs puppet module tries to create a symlink from /usr/bin/node to /usr/bin/nodejs. Depending on the resource ordering, this might happen before or after the nodejs package creates its own /usr/bin/node. If puppet has the chance to make the symlink itself, it breaks node by creating a broken link cycle. This patch uses the legacy_debian_symlinks parameter to prevent the puppet module from doing that. This problem was fixed and the option was deprecated and removed in later versions of the nodejs puppet module, so we will have to remove the parameter if we update the nodejs module. Change-Id: I61e591dbeabe4c043f87b04a7c56c0539120f8a5 Notes (review): Code-Review+2: Ian Wienand <iwienand@redhat.com> Code-Review+2: Monty Taylor <mordred@inaugust.com> Code-Review+2: Clark Boylan <cboylan@sapwetik.org> Workflow+1: Clark Boylan <cboylan@sapwetik.org> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Fri, 10 Aug 2018 20:16:27 +0000 Reviewed-on: https://review.openstack.org/585291 Project: openstack-infra/puppet-ethercalc Branch: refs/heads/master
2018-08-09Add basic rspec testIan Wienand
Add a basic apply test Change-Id: Iff4d74806c67c9480e3679fcca56d8470b9fd29e Notes (review): Code-Review+2: Monty Taylor <mordred@inaugust.com> Code-Review+2: Colleen Murphy <colleen@gazlene.net> Code-Review+2: Clark Boylan <cboylan@sapwetik.org> Workflow+1: Clark Boylan <cboylan@sapwetik.org> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Fri, 10 Aug 2018 20:16:26 +0000 Reviewed-on: https://review.openstack.org/527822 Project: openstack-infra/puppet-ethercalc Branch: refs/heads/master
2018-08-09Update ethercalc to work with Ubuntu XenialJens Harbott
- Set up service via systemd instead of upstart. - defupalt to using nodejs v6.x because of library issues. - upstream nodejs puppet knows how to do legacy symlinks on required platforms. Depends-On: I2355cf58c899bf5f78173b2ed9da26548a9592d0 Depends-On: Ia7966fb9578d0d79f3a7f9480e3a956555737dc8 Change-Id: Ia3f4d3bbacbbe1a42a33a4f934173fb54a582a8e Notes (review): Code-Review+2: Colleen Murphy <colleen@gazlene.net> Code-Review+2: Ian Wienand <iwienand@redhat.com> Code-Review+2: Clark Boylan <cboylan@sapwetik.org> Workflow+1: Clark Boylan <cboylan@sapwetik.org> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Fri, 10 Aug 2018 20:11:35 +0000 Reviewed-on: https://review.openstack.org/527144 Project: openstack-infra/puppet-ethercalc Branch: refs/heads/master
2018-07-12Update Gemfile for Zuulv3Colleen Murphy
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: I186e6963cd028354676c4e1240d0971adcd9b476 Notes (review): Code-Review+2: Clark Boylan <cboylan@sapwetik.org> Workflow+1: Clark Boylan <cboylan@sapwetik.org> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Mon, 16 Jul 2018 19:08:46 +0000 Reviewed-on: https://review.openstack.org/582013 Project: openstack-infra/puppet-ethercalc Branch: refs/heads/master
2018-06-19Merge "Fix scope of vhost template variables"Zuul
2018-05-09Fix scope of vhost template variablesColleen Murphy
Dynamic scoping for variables in ERB templates was removed in puppet 4[1] which means that the variables defined in the manifest cannot be found when it is referenced in the httpd::vhost defined type and will be evaluated as nil when puppet runs. Use scope.lookupvar instead to be explicit about the variable's source. [1] https://puppet.com/docs/puppet/4.10/lang_updating_manifests.html#dynamic-scoping-in-erb Change-Id: I8d2b351537ad8fc0a06c58e1dd23d8423f38a328 Notes (review): Code-Review+2: Clark Boylan <cboylan@sapwetik.org> Code-Review+2: Ian Wienand <iwienand@redhat.com> Workflow+1: Ian Wienand <iwienand@redhat.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Tue, 19 Jun 2018 22:45:03 +0000 Reviewed-on: https://review.openstack.org/564882 Project: openstack-infra/puppet-ethercalc Branch: refs/heads/master
2018-04-27Fix defined type namespacingColleen Murphy
Without this patch, puppet 4 apply tests can't find the mod defined type and fail with: Evaluation Error: Resource type not found: Mod at /etc/puppetlabs/code/modules/ethercalc/manifests/apache.pp:43:15 This patch ensures the resource reference is namespaced the same way the resource declaration is already. Change-Id: I891572c245f878a198f972e33e7ddbff74a5a731 Notes (review): Code-Review+2: Clark Boylan <cboylan@sapwetik.org> Code-Review+2: Joshua Hesketh <josh@nitrotech.org> Workflow+1: Joshua Hesketh <josh@nitrotech.org> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Tue, 01 May 2018 22:48:10 +0000 Reviewed-on: https://review.openstack.org/564535 Project: openstack-infra/puppet-ethercalc Branch: refs/heads/master
2017-08-24Allow multiple places to install curlClark Boylan
Curl is an important package that we end up needing in a few places. To avoid puppet unique name conflicts with the package "curl" only define the curl package resource if it isn't already defined. This allows us to define it in multiple places that may or may not overlap depending on how hosts are constructed. Change-Id: Icd0c652ff2f522ce2ea9086965f53e1a03f0d4fd Notes (review): Code-Review+2: Paul Belanger <pabelanger@redhat.com> Code-Review+2: James E. Blair <corvus@inaugust.com> Workflow+1: James E. Blair <corvus@inaugust.com> Verified+2: Jenkins Submitted-by: Jenkins Submitted-at: Thu, 24 Aug 2017 21:17:12 +0000 Reviewed-on: https://review.openstack.org/497611 Project: openstack-infra/puppet-ethercalc Branch: refs/heads/master
2017-08-18Depend on helper gem for spec_helper_acceptanceColleen Murphy
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: I2d36cc46c473598a8a66259554b454b73ae98b83 Notes (review): Code-Review+2: Clark Boylan <cboylan@sapwetik.org> Workflow+1: Clark Boylan <cboylan@sapwetik.org> Verified+2: Jenkins Submitted-by: Jenkins Submitted-at: Wed, 23 Aug 2017 07:50:51 +0000 Reviewed-on: https://review.openstack.org/495589 Project: openstack-infra/puppet-ethercalc Branch: refs/heads/master
2017-06-24Fix beaker on xenialColleen Murphy
Add a xenial nodeset and update the spec helper to install puppet 3 from the Ubuntu repos instead of from puppetlabs. Change-Id: Ie8a5537579c40ce443489967100134eb9ee60e94 Notes (review): Code-Review+2: Monty Taylor <mordred@inaugust.com> Code-Review+2: Ian Wienand <iwienand@redhat.com> Workflow+1: Ian Wienand <iwienand@redhat.com> Verified+2: Jenkins Submitted-by: Jenkins Submitted-at: Tue, 27 Jun 2017 19:58:54 +0000 Reviewed-on: https://review.openstack.org/477113 Project: openstack-infra/puppet-ethercalc Branch: refs/heads/master
2017-05-01Add bindep.txt filePaul Belanger
Bindep is a tool for checking the presence of binary packages needed to use an application / library. It started life as a way to make it easier to set up a development environment for OpenStack projects. Change-Id: I541c13ea91f007a63ef8f96e40d2047d5f34616a Signed-off-by: Paul Belanger <pabelanger@redhat.com> Notes (review): Code-Review+2: yolanda.robla <yroblamo@redhat.com> Workflow+1: yolanda.robla <yroblamo@redhat.com> Verified+2: Jenkins Submitted-by: Jenkins Submitted-at: Tue, 02 May 2017 14:24:27 +0000 Reviewed-on: https://review.openstack.org/461557 Project: openstack-infra/puppet-ethercalc Branch: refs/heads/master
2017-02-13Don't use npm list --parseableClark Boylan
Turns out we lose version info with --parseable (so not sure in what world this is parseable output) so stop using the --parseable flag. This corrects behavior where we reinstall ethercalc every puppet run which can upgrade unpinned dep libs in ways that break them on older node (we are using the ethercalc prescribed node but its a little old). Change-Id: Ib9aebecf3a0c36c50e5069b1a871886e4ddb1a42 Notes (review): Code-Review+2: Joshua Hesketh <joshua.hesketh@rackspace.com> Workflow+1: Clark Boylan <cboylan@sapwetik.org> Verified+2: Jenkins Submitted-by: Jenkins Submitted-at: Tue, 14 Feb 2017 02:07:18 +0000 Reviewed-on: https://review.openstack.org/433386 Project: openstack-infra/puppet-ethercalc Branch: refs/heads/master
2017-01-20Convert from etherpad to ethercalcClark Boylan
This begins the conversion process. We keep the bulk of the nodejs and apache configuration. Remove mysql and replace with redis and so on. This is not a complete conversion, future work should include authenticated redis configuration and systemd support. However, this should be able to get the service running with a basic set of features and functionality. Change-Id: Id10247211d9643e81bb1b6e8fb67377ba6de873a Notes (review): Code-Review+2: Jeremy Stanley <fungi@yuggoth.org> Code-Review+2: James E. Blair <corvus@inaugust.com> Workflow+1: Clark Boylan <cboylan@sapwetik.org> Verified+2: Jenkins Submitted-by: Jenkins Submitted-at: Tue, 24 Jan 2017 23:07:27 +0000 Reviewed-on: https://review.openstack.org/422921 Project: openstack-infra/puppet-ethercalc Branch: refs/heads/master
2017-01-06Fix a couple openid parametersClark Boylan
This needs to be a proper url, also our attempt at looking up the var was not working, just use the same lookup of vhost_name that is used elsewhere and known to work. Change-Id: If906f69688bf4412f50fcfafc05861b957ce42a8 Notes (review): Code-Review+2: Paul Belanger <pabelanger@redhat.com> Code-Review+2: Jeremy Stanley <fungi@yuggoth.org> Workflow+1: Jeremy Stanley <fungi@yuggoth.org> Verified+2: Jenkins Submitted-by: Jenkins Submitted-at: Fri, 06 Jan 2017 20:09:19 +0000 Reviewed-on: https://review.openstack.org/417473 Project: openstack-infra/puppet-ethercalc Branch: refs/heads/master
2016-11-15Support openid authenticationClark Boylan
Whether to thwart spam or to make more private pads add support for very simple auth mechanism using mod_auth_openid. Change-Id: Ife0daf670a20afde46516c60f877e1da8026758a Notes (review): Code-Review+2: Jeremy Stanley <fungi@yuggoth.org> Code-Review+2: Paul Belanger <pabelanger@redhat.com> Workflow+1: Clark Boylan <cboylan@sapwetik.org> Verified+2: Jenkins Submitted-by: Jenkins Submitted-at: Fri, 06 Jan 2017 17:23:43 +0000 Reviewed-on: https://review.openstack.org/397479 Project: openstack-infra/puppet-ethercalc Branch: refs/heads/master
2016-11-15Use nodejs class to install nodejsClark Boylan
This is a backward incompatible change. Sorry. But now we can rely on a much better portable system for installing nodejs using the nodesource packages instaed of building it ourselves. This is important because the system packages on some distros (like Ubuntu Trusty) can no longer talk to npm resulting in a failed etherpad install. You can continue to use the system packages if you desire using the same old config. But you now need to provide the nodejs class' repo_url_suffix as your nodejs_version parameter should you not want to use the system packages. Note that transitioning from system to the nodesource packaged nodejs is not currently supported by this change. Basically this means that if you have an existing system setup which uses old npm you may want to just redeploy your server from scratch using the nodesource packages to move forward. Change-Id: Ia686f9eba15e0a3dfa81d08aecc212853b3f7189 Notes (review): Code-Review+2: Jeremy Stanley <fungi@yuggoth.org> Code-Review+2: Paul Belanger <pabelanger@redhat.com> Workflow+1: Clark Boylan <cboylan@sapwetik.org> Verified+2: Jenkins Submitted-by: Jenkins Submitted-at: Fri, 06 Jan 2017 17:19:38 +0000 Reviewed-on: https://review.openstack.org/397488 Project: openstack-infra/puppet-ethercalc Branch: refs/heads/master
2016-11-14Put /usr/local/bin at front of pathClark Boylan
If things get installed by something other than the system package manager they often end up in /usr/local/bin and they often shadow things that the system package manager may have installed. By putting this at the end of the path we don't get the benefit of that shadowing and instead get whatever the system package manager installed. This is particularly unhelpful when attempting to use an npm updated by npm itself which ends up in the local dir. Change-Id: Ifb58c98162f0064a2ae70868921eea507306ad7e Notes (review): Code-Review+2: Jeremy Stanley <fungi@yuggoth.org> Workflow+1: Jeremy Stanley <fungi@yuggoth.org> Verified+2: Jenkins Submitted-by: Jenkins Submitted-at: Thu, 05 Jan 2017 22:14:47 +0000 Reviewed-on: https://review.openstack.org/397478 Project: openstack-infra/puppet-ethercalc Branch: refs/heads/master
2016-11-14Install curl as dep for etherpad dep installClark Boylan
Etherpads dependency install process requires curl, ensure that it is installed before we need it. Change-Id: I711e43f0dd9c8287c26e2861498eb3ebb191e878 Notes (review): Code-Review+2: Jeremy Stanley <fungi@yuggoth.org> Workflow+1: Jeremy Stanley <fungi@yuggoth.org> Verified+2: Jenkins Submitted-by: Jenkins Submitted-at: Thu, 05 Jan 2017 22:14:42 +0000 Reviewed-on: https://review.openstack.org/397477 Project: openstack-infra/puppet-ethercalc Branch: refs/heads/master
2016-11-14Fix utf8 4 byte collation methodClark Boylan
The default collate method used by the mysql puppet module is not compatbile with our 4 byte encoding. Update the collate method to match the 4 byte encoding. Change-Id: I8f65316ba609ecf30c9797d631bdeb4a083b16ce Notes (review): Code-Review+2: Jeremy Stanley <fungi@yuggoth.org> Workflow+1: Jeremy Stanley <fungi@yuggoth.org> Verified+2: Jenkins Submitted-by: Jenkins Submitted-at: Thu, 05 Jan 2017 22:14:36 +0000 Reviewed-on: https://review.openstack.org/397476 Project: openstack-infra/puppet-ethercalc Branch: refs/heads/master
2016-06-21Use new infra_spec_helper for gem dependenciesSpencer Krum
Change-Id: I1a5743fa780b55f15dae0d2867d1eb65cfd666a7 Notes (review): Code-Review+2: Joshua Hesketh <joshua.hesketh@rackspace.com> Workflow+1: Joshua Hesketh <joshua.hesketh@rackspace.com> Verified+2: Jenkins Submitted-by: Jenkins Submitted-at: Wed, 22 Jun 2016 07:17:03 +0000 Reviewed-on: https://review.openstack.org/332524 Project: openstack-infra/puppet-ethercalc Branch: refs/heads/master
2016-04-14Pin google-api-client; sanitize GemfileSpencer Krum
Change-Id: I79a14dbd21cb54e7d064d87f46a75f60bd991708 Notes (review): Code-Review+2: Monty Taylor <mordred@inaugust.com> Code-Review+2: James E. Blair <corvus@inaugust.com> Workflow+1: James E. Blair <corvus@inaugust.com> Verified+2: Jenkins Submitted-by: Jenkins Submitted-at: Thu, 14 Apr 2016 22:54:37 +0000 Reviewed-on: https://review.openstack.org/306113 Project: openstack-infra/puppet-ethercalc Branch: refs/heads/master
2016-03-21Order of the classes parameters is refactoredAndrey Nikitin
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: If012896a95088ae836d5dfa35aa3cad553aee516 Notes (review): Code-Review+1: Arie Bregman <abregman@redhat.com> Code-Review+2: yolanda.robla <yroblamo@redhat.com> Code-Review+1: Ricardo Carrillo Cruz <ricardo.carrillo.cruz@gmail.com> Code-Review+2: Paul Belanger <pabelanger@redhat.com> Workflow+1: Paul Belanger <pabelanger@redhat.com> Verified+2: Jenkins Submitted-by: Jenkins Submitted-at: Tue, 22 Mar 2016 15:42:55 +0000 Reviewed-on: https://review.openstack.org/295138 Project: openstack-infra/puppet-ethercalc Branch: refs/heads/master
2016-03-18Merge "Add missing Gemfile to the project"Jenkins
2016-03-11Merge "Fix races in httpd install and config"Jenkins
2016-03-11Add missing Gemfile to the projectYolanda Robla
All our puppet projects rely on a Gemfile for tests to run. This was missing, so add here as well. Also fix the linter errors that this new Gemfile addition is causing. Change-Id: I24143df44c313ea76472649bb9a6fa0ce4536307 Notes (review): Code-Review+1: Ricardo Carrillo Cruz <ricardo.carrillo.cruz@gmail.com> Code-Review+2: Paul Belanger <pabelanger@redhat.com> Code-Review+2: Colleen Murphy <colleen@gazlene.net> Workflow+1: Colleen Murphy <colleen@gazlene.net> Verified+2: Jenkins Submitted-by: Jenkins Submitted-at: Fri, 18 Mar 2016 00:04:26 +0000 Reviewed-on: https://review.openstack.org/291678 Project: openstack-infra/puppet-ethercalc Branch: refs/heads/master
2016-02-02fix module to work with latest mysql versionYolanda Robla
Change-Id: I14328be9f00a94a6eb18670f85ee0b61b434833c Notes (review): Code-Review+2: Colleen Murphy <colleen@gazlene.net> Code-Review+2: Paul Belanger <pabelanger@redhat.com> Workflow+1: Paul Belanger <pabelanger@redhat.com> Code-Review+1: Ricardo Carrillo Cruz <ricardo.carrillo.cruz@gmail.com> Verified+2: Jenkins Submitted-by: Jenkins Submitted-at: Fri, 11 Mar 2016 17:25:34 +0000 Reviewed-on: https://review.openstack.org/275247 Project: openstack-infra/puppet-ethercalc Branch: refs/heads/master
2015-11-12Etherpad requires 4byte utf8 charsetSpencer Krum
Change-Id: I284f7dca1a71ebe2065253822aeaea268e316712 Notes (review): Code-Review+2: Clark Boylan <cboylan@sapwetik.org> Code-Review+2: yolanda.robla <yroblamo@redhat.com> Workflow+1: yolanda.robla <yroblamo@redhat.com> Verified+2: Jenkins Submitted-by: Jenkins Submitted-at: Thu, 12 Nov 2015 19:11:03 +0000 Reviewed-on: https://review.openstack.org/244760 Project: openstack-infra/puppet-ethercalc Branch: refs/heads/master
2015-11-12Merge "Add mysql class to etherpad to optionally configure db"Jenkins
2015-10-29Bump connection limits to handle websocketsClark Boylan
Websockets mean we make more connections to apache. Bump the connection limits to 4096 connections total as this appears to handle summit load. Change-Id: I87b6d06b5b730bde48d60fcbd6c025de5581b672 Notes (review): Code-Review+2: Spencer Krum <nibz@spencerkrum.com> Code-Review+1: Craige McWhirter <craige@mcwhirter.io> Code-Review+2: yolanda.robla <yroblamo@redhat.com> Workflow+1: yolanda.robla <yroblamo@redhat.com> Verified+2: Jenkins Submitted-by: Jenkins Submitted-at: Fri, 30 Oct 2015 02:42:47 +0000 Reviewed-on: https://review.openstack.org/240352 Project: openstack-infra/puppet-ethercalc Branch: refs/heads/master
2015-10-27Set higher nofile limit for node in init scriptClark Boylan
You must build additional pylons ... I mean we need more file descriptors. Set that in upstart directly since it does not respect limits.conf. This rewrites the upstart job to be more upstarty so that the limits can be applied without su undoing the work. Change-Id: Ibf670f07d75c4ffcf333c971d03c620f839b04d3 Notes (review): Code-Review+1: James E. Blair <corvus@inaugust.com> Code-Review+2: Jeremy Stanley <fungi@yuggoth.org> Workflow+1: Jeremy Stanley <fungi@yuggoth.org> Verified+2: Jenkins Submitted-by: Jenkins Submitted-at: Tue, 27 Oct 2015 22:15:58 +0000 Reviewed-on: https://review.openstack.org/239599 Project: openstack-infra/puppet-ethercalc Branch: refs/heads/master
2015-10-13Add mysql class to etherpad to optionally configure dbYolanda Robla
Change-Id: Ib97188f3597fccf3d0a7b6f6e394ae6bacb12f65 Notes (review): Code-Review+1: Danilo Ramalho <dramalho@thoughtworks.com> Code-Review+2: Spencer Krum <nibz@spencerkrum.com> Code-Review+2: Monty Taylor <mordred@inaugust.com> Workflow+1: Monty Taylor <mordred@inaugust.com> Verified+2: Jenkins Submitted-by: Jenkins Submitted-at: Thu, 12 Nov 2015 16:37:06 +0000 Reviewed-on: https://review.openstack.org/234184 Project: openstack-infra/puppet-ethercalc Branch: refs/heads/master
2015-10-09Fix races in httpd install and configJeremy Stanley
Add necessary dependency/ordering hints to the etherpad_lite::apache class so that configuration parent directories are created before they're needed rather than later through package installation, and so that vhost configuration won't be loaded until Apache module dependencies are in place. Change-Id: Iaeafd30b19276ff438bd9e2708031c1064a2314d Notes (review): Code-Review+2: Jeremy Stanley <fungi@yuggoth.org> Code-Review+2: yolanda.robla <yroblamo@redhat.com> Code-Review+1: Clint Adams <clint@gcfm.net> Workflow+1: yolanda.robla <yroblamo@redhat.com> Verified+2: Jenkins Submitted-by: Jenkins Submitted-at: Fri, 11 Mar 2016 19:12:04 +0000 Reviewed-on: https://review.openstack.org/215169 Project: openstack-infra/puppet-ethercalc Branch: refs/heads/master
2015-09-16Ensure abiword package is always presentSamuel de Medeiros Queiroz
Change I9831526581a5fc1f2014c9985637664c79b937bf moved the required packages list statement into a conditional block, making it to be ensured only in the case nodejs_version was set to 'system'. This was causing the abiword library to not be installed and then etherpad.o.o was appending the following error message to new pads: Error: Abiword does not exist at this path, check your settings file -- To suppress these warning messages change suppressErrorsInPadText to true in your settings.json This patch makes the abiword library to always be installed by moving it out of that conditional statement. Change-Id: I50f55e4408d850b2bf449b121b9014d511518481 Notes (review): Code-Review+1: Clint Adams <clint@gcfm.net> Code-Review+2: Jeremy Stanley <fungi@yuggoth.org> Code-Review+1: Ronald Bradford <ronald.bradford@gmail.com> Code-Review+2: Elizabeth K. Joseph <lyz@princessleia.com> Workflow+1: Elizabeth K. Joseph <lyz@princessleia.com> Verified+2: Jenkins Submitted-by: Jenkins Submitted-at: Thu, 17 Sep 2015 20:52:12 +0000 Reviewed-on: https://review.openstack.org/224103 Project: openstack-infra/puppet-ethercalc Branch: refs/heads/master
2015-09-10Use the correct event module nameJeremy Stanley
In the connection tuning configuration, use the correct name for mpm_event_module so that the parameters will be applied when that module is in use. Change-Id: Icc8c84cec5a78d96805ed7a991e2ef4b1b53d9d8 Notes (review): Code-Review+2: Clark Boylan <cboylan@sapwetik.org> Code-Review+2: James E. Blair <corvus@inaugust.com> Workflow+1: James E. Blair <corvus@inaugust.com> Verified+2: Jenkins Submitted-by: Jenkins Submitted-at: Thu, 10 Sep 2015 19:16:21 +0000 Reviewed-on: https://review.openstack.org/222261 Project: openstack-infra/puppet-ethercalc Branch: refs/heads/master
2015-09-08Support apache /server-statusJames E. Blair
Change-Id: I554cd3a219c20f80b45bc9a45c793a0642ad17e6 Notes (review): Code-Review+1: Anita Kuno <anteaya@anteaya.info> Code-Review+2: Jeremy Stanley <fungi@yuggoth.org> Workflow+1: Jeremy Stanley <fungi@yuggoth.org> Verified+2: Jenkins Submitted-by: Jenkins Submitted-at: Tue, 08 Sep 2015 20:05:22 +0000 Reviewed-on: https://review.openstack.org/221424 Project: openstack-infra/puppet-ethercalc Branch: refs/heads/master
2015-09-08Fix connection tuning on >= trustyJames E. Blair
The tuning config file needs a .conf extension. Also, trusty uses mpm_worker_event by default, so support that as well. Change-Id: Ica3a7ceab29f120a24ca93cc5491f5fe8ce9e54a Notes (review): Code-Review+1: Clint Adams <clint@gcfm.net> Code-Review+1: Anita Kuno <anteaya@anteaya.info> Code-Review+2: Jeremy Stanley <fungi@yuggoth.org> Code-Review+1: venkatamahesh <Venkata.Kotha@infinite.com> Code-Review+2: Clark Boylan <cboylan@sapwetik.org> Workflow+1: Clark Boylan <cboylan@sapwetik.org> Verified+2: Jenkins Submitted-by: Jenkins Submitted-at: Tue, 08 Sep 2015 18:16:18 +0000 Reviewed-on: https://review.openstack.org/221423 Project: openstack-infra/puppet-ethercalc Branch: refs/heads/master
2015-09-02Fix websocket configurationJames E. Blair
Enable the proxy_websocket module. Correct the ProxyPass directives in use with websockets. The current versions result in 404 on socket.io.js because they strip the socket.io component of the path. Change-Id: I3505b4e9693602ec04baec2487871ce57a6dc7a6 Notes (review): Code-Review+2: Clark Boylan <cboylan@sapwetik.org> Code-Review+2: Jeremy Stanley <fungi@yuggoth.org> Workflow+1: Jeremy Stanley <fungi@yuggoth.org> Verified+2: Jenkins Submitted-by: Jenkins Submitted-at: Wed, 02 Sep 2015 18:42:31 +0000 Reviewed-on: https://review.openstack.org/219790 Project: openstack-infra/puppet-ethercalc Branch: refs/heads/master
2015-08-21Merge "Allow system nodejs install"Jenkins
2015-08-20Allow system nodejs installClark Boylan
Allow setting the nodejs_version to 'system' which will tell the module to install the nodejs and npm packages from their system package manager rather than building the specified version from source, and symlink the expected interpreter name. Change-Id: I9831526581a5fc1f2014c9985637664c79b937bf Notes (review): Code-Review+2: Jeremy Stanley <fungi@yuggoth.org> Code-Review+1: Craige McWhirter <craige@mcwhirter.io> Code-Review+2: Spencer Krum <nibz@spencerkrum.com> Code-Review+2: yolanda.robla <yroblamo@redhat.com> Workflow+1: Jeremy Stanley <fungi@yuggoth.org> Verified+2: Jenkins Submitted-by: Jenkins Submitted-at: Fri, 21 Aug 2015 13:58:33 +0000 Reviewed-on: https://review.openstack.org/214821 Project: openstack-infra/puppet-ethercalc Branch: refs/heads/master
2015-08-19Fix target path for regular git clone during testsSpencer Krum
Use same target directory for zuul-cloner and the regular git command. Change-Id: I57e34c58760f1571f79649f85d739b192476f834 Co-Authored-By: Fabien Boucher <fabien.boucher@enovance.com> Notes (review): Code-Review+1: Fabien Boucher <fboucher@redhat.com> Code-Review+2: yolanda.robla <yroblamo@redhat.com> Workflow+1: yolanda.robla <yroblamo@redhat.com> Verified+2: Jenkins Submitted-by: Jenkins Submitted-at: Fri, 21 Aug 2015 07:45:46 +0000 Reviewed-on: https://review.openstack.org/214841 Project: openstack-infra/puppet-ethercalc Branch: refs/heads/master
2015-08-19Switch to proxypass with retry=0 for etherpadJames E. Blair
We think Apache may be caching 503 responses (possibly themselves a result of [1]). Set retry=0 so that we do not cache any failures and switch to proxypass so that we can set that option. [1] https://bz.apache.org/bugzilla/show_bug.cgi?id=37770 Change-Id: Ibd1087bc8feb288bfb8f66f4e6de61ebe8414233 Notes (review): Code-Review+2: Jeremy Stanley <fungi@yuggoth.org> Workflow+1: Jeremy Stanley <fungi@yuggoth.org> Verified+2: Jenkins Submitted-by: Jenkins Submitted-at: Wed, 19 Aug 2015 18:35:37 +0000 Reviewed-on: https://review.openstack.org/214741 Project: openstack-infra/puppet-ethercalc Branch: refs/heads/master
2015-08-06Merge "Fix warning about httpd module"Jenkins
2015-07-28Boilerplate beaker-rspec filesSpencer Krum
Change-Id: I3ae18482462b90826452059f51223c5c5000e87e Notes (review): Code-Review+1: Clint Adams <clint@gcfm.net> Code-Review+2: James E. Blair <corvus@inaugust.com> Workflow+1: James E. Blair <corvus@inaugust.com> Verified+2: Jenkins Submitted-by: Jenkins Submitted-at: Thu, 30 Jul 2015 00:25:18 +0000 Reviewed-on: https://review.openstack.org/206377 Project: openstack-infra/puppet-ethercalc Branch: refs/heads/master
2015-07-16Fix warning about httpd modulePaul Belanger
Change-Id: I5a75b1488b587ff6fd564afabf9f707199d2a184 Signed-off-by: Paul Belanger <pabelanger@redhat.com> Notes (review): Code-Review+2: yolanda.robla <yroblamo@redhat.com> Code-Review+1: K Jonathan Harker <code@gentlydownthe.net> Code-Review+2: Spencer Krum <nibz@spencerkrum.com> Workflow+1: Spencer Krum <nibz@spencerkrum.com> Verified+2: Jenkins Submitted-by: Jenkins Submitted-at: Thu, 06 Aug 2015 03:34:44 +0000 Reviewed-on: https://review.openstack.org/202761 Project: openstack-infra/puppet-ethercalc Branch: refs/heads/master
2015-07-14Merge "Fix variable access warnings"Jenkins