by

Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Activity
David Caro
@david-caro
that can be very useful for us too :{
Alex Ioannidis
@slint
Not yet unfortunately...
David Caro
@david-caro
okok, just wondering :), we can try to join efforts at some point ;)
Lars Holm Nielsen
@lnielsen
yes, HA for RabbitMQ and Redis woudl be nice :-) then we wouldn’t have to juggle machines around when they have to reboot the entire data centre :-)
Alex Ioannidis
@slint
@david-caro At the moment the mininmal set of MRs that seems to be needed to get the broker fixed are 38 and 39, which seem though to be already included in 21
Alex Ioannidis
@slint
I've cherry-picked/rebased on qa the two MRs and created a separate MR. If @/all can take a look and are :ok_hand: with merging these two on qa, I guess we can do so (tomorrow morning would be best I guess).
Merging qa to master is going to be a separate beast though, there's been a lot of commits adding types and general improvements...
David Caro
@david-caro
@slint I commented on !38, can you check? and on !39, it does not affect us (inspire or hepdata) but I think it's not a good idea to put flower and the broker on the same node
(otherwise, I think it looks ok)
Alex Ioannidis
@slint
@david-caro I believe !38 is basically running already on CDS videos production, so it shouldn't be an issue
If you're ok, I can merge !54 and close !38 and !39
Esteban J. G. Gabancho
@egabancho
all the PRs Harris made are running on CDS since then
Alex Ioannidis
@slint
@david-caro I've made the changes discussed in !38 and !38 (I changed the class to ::rabbitmq::server as well).
All of these are were merged in qa. Ideally it would be nice to merge qa to master as well, since the latest changes (excluding the 2 aforementioned MRs) have been running for over a week
Alex Ioannidis
@slint
For Zenodo our deadline is until the 24th (our RabbitMQ instance is in the Critical set)
So, if @/all are ok, I'd propose that on Mon 22nd we merge qa to master
Esteban J. G. Gabancho
@egabancho
:+1:
Alex Ioannidis
@slint
Hey @/all, is everybody cool for merging qa to master?
Esteban J. G. Gabancho
@egabancho
:+1:
David Caro
@david-caro
:+1:
Alex Ioannidis
@slint
There's also a fix for new Elasticsearch nodes failing to install the ElasticHQ plugin
But for it's only necessary for qa
David Caro
@david-caro
okok
Alex Ioannidis
@slint
Whenever somebody has time (and merge rights ;)), please have a look at https://gitlab.cern.ch/ai/it-puppet-module-invenio/merge_requests/56
Nicola
@ntarocco
I have created a PR (https://gitlab.cern.ch/ai/it-puppet-module-invenio/merge_requests/59) to comply to CERN Puppet recommendation. Email we received:
Default TLS/SSL configurations have been added to puppet to ensure safe and
unified configurations. For more information see the recent ASDF:
https://indico.cern.ch/event/714180/

Unfortunately, it seems that your hostgroup does not use the defaults but
has hardcoded TLS/SSL settings. Please refer to the following URL to adapt
your configuration: https://hackmd.web.cern.ch/s/HJhPMOhiz
and mentioned latest committers
Lars Holm Nielsen
@lnielsen
:+1:
Thanks a lot
looks good to me
Nicola
@ntarocco
it is actually tested already in CDS and Videos
so there should be no problem
Lars Holm Nielsen
@lnielsen
FYI: I’m merging PRs to it-puppet-module-invenio ….there’s a lot of pending PRs…..all changes are being merged to QA
after the reboot campaign we can merge to production
Lars Holm Nielsen
@lnielsen

Any ideas why I'm seeing this:

Notice: /Stage[main]/Collectd::Plugin::Python/File[/opt/zenodo/lib/python2.7/site-packages]/owner: owner changed 'zenodo' to 'root'
Notice: /Stage[main]/Collectd::Plugin::Python/File[/opt/zenodo/lib/python2.7/site-packages]/group: group changed 'zenodo' to 'root’

Somehow the collectd puppet module is picking up my apps Python virtual environment and changing users on it (which it should not be doing!)?

Nicola
@ntarocco
happens on videos too
Lars Holm Nielsen
@lnielsen
:-)
Nicola
@ntarocco
and it looks random
it already happened in the past
Lars Holm Nielsen
@lnielsen
Hmm…..super annoying…..block deployment
Nicola
@ntarocco
what I was doing is to run puppet, disable it, fix permissions, deploy
:'(
Lars Holm Nielsen
@lnielsen
:(
Esteban J. G. Gabancho
@egabancho
I remember it happening only if you run puppet manually, maybe I am wrong
Lars Holm Nielsen
@lnielsen
@/all I’m merging three commits from qa to master on it-puppet-module-invenio:
commit 0e7deed1a7ccb7968d7282500d57f62d8ef350d2 (HEAD -> qa, origin/qa, origin/HEAD)
Author: Lars Holm Nielsen <lars.holm.nielsen@cern.ch>
Date:   Mon Jun 3 10:53:46 2019 +0200

    cache: allow vm.overcommit_memory for redis

    * Fixes issue with Redis not being able to persist changes to disk when
      using more than half of the memory. See detailed description in code.

commit 61dfa49357aa7e316dd6e48531282436a40356be
Author: Karolina Zdzislawa Przerwa <karolina.zdzislawa.przerwa@cern.ch>
Date:   Tue Mar 26 07:50:46 2019 +0000

    lb backup: add /ping response

commit 5e186fa65f820d802629efd1b0c59e2e7a9e469f
Author: Karolina Zdzislawa Przerwa <karolina.zdzislawa.przerwa@cern.ch>
Date:   Mon Mar 25 14:48:54 2019 +0000

    nginx template: backup urls aliases configuration

    * fixes all the subpages redirect to static page
Lars Holm Nielsen
@lnielsen
FYI: the load balancer backup pages where not probably deployed (missing merge to production branch and deployment to all three load balancers)
i’ve fixed it now
Alex Ioannidis
@slint
Probably this commit (from Zenodo) has to be included as well for the backup page to give a 503 response code (for e.g. Uptime Robot to work): https://gitlab.cern.ch/ai/it-puppet-hostgroup-zenodo/commit/f4e68237161c05923e6631f07b2ed00175443595
Lars Holm Nielsen
@lnielsen
:+1: Will you add a commit…..it’s already merged
Alex Ioannidis
@slint
:+1:
Zacharias Zacharodimos
@zzacharo
Hey @/all here there is a pr about checking on the load balancer level if host of the request is valid and then redirect to a parametrised default host...
Zacharias Zacharodimos
@zzacharo
@/all reminder for review :)