*** gregoryo has joined #puppet-openstack | 01:19 | |
*** vnogin has joined #puppet-openstack | 02:52 | |
*** vnogin has quit IRC | 02:56 | |
*** skramaja has joined #puppet-openstack | 04:17 | |
*** jaosorior has joined #puppet-openstack | 04:28 | |
*** rcernin has quit IRC | 04:54 | |
*** rcernin has joined #puppet-openstack | 04:54 | |
*** agurenko has joined #puppet-openstack | 04:55 | |
*** openstackgerrit has quit IRC | 05:18 | |
*** ykarel has joined #puppet-openstack | 05:54 | |
*** jaosorior has quit IRC | 05:59 | |
*** dpawlik has joined #puppet-openstack | 06:00 | |
*** dpawlik has quit IRC | 06:05 | |
*** dpawlik has joined #puppet-openstack | 06:12 | |
*** ykarel_ has joined #puppet-openstack | 06:25 | |
*** ykarel has quit IRC | 06:27 | |
*** ykarel__ has joined #puppet-openstack | 06:33 | |
*** ykarel_ has quit IRC | 06:35 | |
*** ykarel__ is now known as ykarel | 06:41 | |
*** vnogin has joined #puppet-openstack | 06:53 | |
*** vnogin has quit IRC | 06:57 | |
*** rcernin has quit IRC | 07:03 | |
*** openstackgerrit has joined #puppet-openstack | 07:08 | |
openstackgerrit | Tobias Urdin proposed openstack/puppet-openstack-integration master: Test Sahara API with WSGI https://review.openstack.org/590357 | 07:08 |
---|---|---|
*** amoralej|off is now known as amoralej | 07:16 | |
*** rpittau has joined #puppet-openstack | 07:43 | |
*** jaosorior has joined #puppet-openstack | 07:46 | |
*** jpena|off is now known as jpena | 07:55 | |
*** ykarel is now known as ykarel|lunch | 07:59 | |
*** piepmatz has joined #puppet-openstack | 08:09 | |
openstackgerrit | wutao proposed openstack/puppet-nova master: Configure access_key and secret_key as secret https://review.openstack.org/591294 | 08:18 |
*** derekh has joined #puppet-openstack | 08:36 | |
*** jaosorior_ has joined #puppet-openstack | 08:36 | |
*** jaosorior has quit IRC | 08:40 | |
*** dtantsur|afk is now known as dtantsur | 08:49 | |
*** ykarel|lunch is now known as ykarel | 08:56 | |
*** salmankhan has joined #puppet-openstack | 09:00 | |
*** gregoryo has quit IRC | 09:06 | |
*** gfidente has joined #puppet-openstack | 09:11 | |
zigo | When applying config for my node, I get: Error: Could not retrieve catalog from remote server: Error 500 on SERVER: Server Error: Evaluation Error: Error while evaluating a Resource Statement, Evaluation Error: Error while evaluating a Function Call, pick(): must receive at least one non empty value at /usr/share/puppet/modules/keystone/manifests/db.pp:62:33 on node zigo-controller-node-1.infomaniak.ch | 09:14 |
zigo | Weird... | 09:14 |
zigo | Hum ... Got it ! :P | 09:15 |
*** gfidente has quit IRC | 09:26 | |
*** rpittau has quit IRC | 09:36 | |
*** rpittau has joined #puppet-openstack | 09:36 | |
*** jaosorior_ is now known as jaosorior | 09:39 | |
*** tosky has joined #puppet-openstack | 09:59 | |
*** hjensas has joined #puppet-openstack | 10:02 | |
*** jaosorior has quit IRC | 10:09 | |
*** jaosorior has joined #puppet-openstack | 10:11 | |
*** tosky has quit IRC | 10:19 | |
*** gfidente has joined #puppet-openstack | 10:37 | |
openstackgerrit | Merged openstack/puppet-trove stable/queens: Make providers use auth_url for authentication https://review.openstack.org/589096 | 10:38 |
*** ykarel is now known as ykarel|afk | 10:38 | |
*** vnogin has joined #puppet-openstack | 11:10 | |
*** jpena is now known as jpena|lunch | 11:14 | |
*** amoralej is now known as amoralej|lunch | 11:32 | |
*** morazi has joined #puppet-openstack | 11:32 | |
tobias-urdin | mwhahaha: these patches depend on each other, I forgot how I resolved it a while ago.. remember you helped me then | 11:34 |
tobias-urdin | https://review.openstack.org/#/c/590354/ https://review.openstack.org/#/c/590357/ | 11:35 |
*** tosky has joined #puppet-openstack | 11:41 | |
*** ykarel|afk is now known as ykarel | 11:45 | |
*** rpittau is now known as rpittau|afk | 12:10 | |
*** rpittau|afk is now known as rpittau | 12:10 | |
*** jpena|lunch is now known as jpena | 12:13 | |
*** jaosorior has quit IRC | 12:18 | |
*** jaosorior has joined #puppet-openstack | 12:19 | |
*** amoralej|lunch is now known as amoralej | 12:25 | |
*** jistr is now known as jistr|call | 12:32 | |
*** trown|outtypewww is now known as trown | 12:32 | |
*** tosky has quit IRC | 12:35 | |
*** jistr|call is now known as jistr | 13:04 | |
mwhahaha | tobias-urdin: drop the depends-on for https://review.openstack.org/#/c/590357/ | 13:19 |
mwhahaha | tobias-urdin: i think https://review.openstack.org/#/c/590357/ could be a needed-by instead but it's not necessary | 13:19 |
*** skramaja has quit IRC | 13:30 | |
tobias-urdin | mwhahaha: if i drop the depends-on it will fail CI | 13:35 |
tobias-urdin | not that it works with it but :p | 13:35 |
mwhahaha | tobias-urdin: then you're doing it wrong :D | 13:35 |
mwhahaha | i'll look in a biot | 13:35 |
mwhahaha | tobias-urdin: ok so it's in reverse then, https://review.openstack.org/#/c/590354/ shouldn't have the depends-on | 13:36 |
tobias-urdin | douh so i change to the wrong order, probably wanted to test changes easier, silly me | 13:37 |
openstackgerrit | Tobias Urdin proposed openstack/puppet-sahara master: Add Sahara API WSGI support https://review.openstack.org/590354 | 13:37 |
*** ade_lee has joined #puppet-openstack | 13:38 | |
*** bnemec-pto is now known as bnemec | 14:10 | |
*** _ix has quit IRC | 14:11 | |
tobias-urdin | mwhahaha: regarding those horizon patches that I used a depends-on instead, iirc it's a real pain to rebase ordering for patches and make them "relates to" for gerrit again | 14:29 |
tobias-urdin | last time I tried it just uploaded new patches | 14:29 |
mwhahaha | tobias-urdin: you can do it via the ui | 14:30 |
mwhahaha | tobias-urdin: click rebase on the ui and check the box to reparent | 14:30 |
mwhahaha | tobias-urdin: then put the patch # in | 14:30 |
*** ykarel is now known as ykarel|away | 14:30 | |
openstackgerrit | Tobias Urdin proposed openstack/puppet-horizon master: Add wsgi_processes and wsgi_threads to horizon init https://review.openstack.org/589871 | 14:31 |
tobias-urdin | mwhahaha: ah! nice | 14:31 |
openstackgerrit | Tobias Urdin proposed openstack/puppet-horizon master: Add wsgi_processes and wsgi_threads to horizon init https://review.openstack.org/589871 | 14:31 |
*** ykarel|away has quit IRC | 14:36 | |
*** _ix has joined #puppet-openstack | 14:48 | |
*** vnogin has quit IRC | 15:01 | |
tobias-urdin | mwhahaha: https://review.openstack.org/#/q/topic:bug/1765453+(status:open+OR+status:merged) | 15:21 |
tobias-urdin | that puppet-sahara change depends on the p-o-i change for beaker test to work | 15:23 |
*** vnogin has joined #puppet-openstack | 15:28 | |
tobias-urdin | mwhahaha: oh nvm I know how I can fix it | 15:30 |
tobias-urdin | or not really, have to set it an NV before merging :( what u think? | 15:31 |
*** agurenko has quit IRC | 15:35 | |
mwhahaha | tobias-urdin: hmmm that means you did it in a non-backwards compatible fashion | 16:00 |
mwhahaha | tobias-urdin: so that's fix is to address the backwards compatibility and not the tests | 16:00 |
tobias-urdin | how do you mean? the tests are broken and my stuff will fix it, but can't merge because it's broken | 16:01 |
mwhahaha | tobias-urdin: let me take a look | 16:01 |
mwhahaha | tobias-urdin: ok so it's currently broken right? | 16:02 |
mwhahaha | tobias-urdin: then yea switch it to non-voting and we can merge the bits | 16:02 |
zigo | Hi there! | 16:05 |
zigo | What's the thing in puppet-openstack-integration that cleans-up the old apache config from the keystone package? | 16:05 |
zigo | It looks like it's conflicting in what I'm implementing. | 16:05 |
mwhahaha | zigo: hmm? | 16:06 |
tobias-urdin | you mean this? https://github.com/openstack/puppet-openstack-integration/blob/master/manifests/keystone.pp#L106 | 16:06 |
zigo | mwhahaha: I still get /etc/apache2/sites-available/wsgi-keystone.conf when apache starts ... | 16:06 |
zigo | Hum... | 16:07 |
zigo | mwhahaha: In the CI, the file gets deleted completely ... | 16:07 |
mwhahaha | yea | 16:07 |
mwhahaha | er we don't delete | 16:07 |
mwhahaha | oh apache deletes it | 16:07 |
zigo | I can do what you pointed at though ... | 16:07 |
mwhahaha | yea you'd need to define it as a file if you don't want it removed | 16:08 |
mwhahaha | by default puppet-apache purges the existing stuff unless defined | 16:09 |
mwhahaha | https://github.com/puppetlabs/puppetlabs-apache/blob/df1d942000323419bfe9bae73328a0271755e77b/manifests/init.pp#L37 | 16:09 |
zigo | mwhahaha: Though in keystone.pp of poi, there's only "include ::apache", is this enough for it to purge files? | 16:12 |
mwhahaha | zigo: yes because it's the default | 16:12 |
zigo | Oh, what I'm looking for seems to be in the scenario files ! :) | 16:13 |
zigo | Hum... | 16:13 |
zigo | No... | 16:13 |
zigo | mwhahaha: FYI, that's my code: https://salsa.debian.org/openstack-team/debian/openstack-cluster-installer/blob/debian/queens/puppet/manifests/controller.pp | 16:14 |
zigo | :) | 16:14 |
*** vnogin has quit IRC | 16:15 | |
zigo | All the params of that controller class are sent by my custom ENC. | 16:15 |
zigo | Anyway, got to go. | 16:16 |
zigo | Bye. | 16:16 |
mwhahaha | zigo: yea not sure i'll look later and try and figure something out | 16:20 |
*** jpena is now known as jpena|off | 16:34 | |
*** piepmatz has quit IRC | 16:56 | |
*** dtantsur is now known as dtantsur|afk | 16:59 | |
*** derekh has quit IRC | 17:01 | |
*** trown is now known as trown|lunch | 17:07 | |
openstackgerrit | Merged openstack/puppet-murano master: Deprecate auth_uri option https://review.openstack.org/589079 | 17:14 |
*** openstackgerrit has quit IRC | 17:19 | |
*** openstackgerrit has joined #puppet-openstack | 17:25 | |
openstackgerrit | Merged openstack/puppet-nova master: Configure access_key and secret_key as secret https://review.openstack.org/591294 | 17:25 |
openstackgerrit | Tobias Urdin proposed openstack/puppet-openstack-integration master: Revert "Revert "Set bionic beaker as not voting"" https://review.openstack.org/591481 | 17:53 |
*** morazi has quit IRC | 18:12 | |
*** amoralej is now known as amoralej|lunch | 18:14 | |
*** amoralej|lunch is now known as amoralej|off | 18:14 | |
*** salmankhan has quit IRC | 18:15 | |
*** trown|lunch is now known as trown | 18:22 | |
*** jpena|off is now known as jpena | 19:04 | |
*** jpena is now known as jpena|off | 19:06 | |
*** morazi has joined #puppet-openstack | 19:18 | |
*** mgagne_ has joined #puppet-openstack | 19:24 | |
*** rajinir has quit IRC | 19:25 | |
*** jgrassler has quit IRC | 19:25 | |
*** mgagne has quit IRC | 19:25 | |
*** salmankhan has joined #puppet-openstack | 19:39 | |
openstackgerrit | Julia Kreger proposed openstack/puppet-ironic master: Correct default number of threads https://review.openstack.org/591493 | 19:39 |
*** salmankhan has quit IRC | 19:44 | |
*** rajinir has joined #puppet-openstack | 20:05 | |
*** salmankhan has joined #puppet-openstack | 20:32 | |
openstackgerrit | Merged openstack/puppet-openstack-integration master: Revert "Revert "Set bionic beaker as not voting"" https://review.openstack.org/591481 | 20:41 |
*** bno1 has left #puppet-openstack | 20:45 | |
*** bno1 has joined #puppet-openstack | 20:46 | |
*** trown is now known as trown|outtypewww | 20:55 | |
*** gfidente is now known as gfidente|fk | 21:49 | |
*** rcernin has joined #puppet-openstack | 22:01 | |
*** _ix has quit IRC | 22:11 | |
*** salmankhan has quit IRC | 23:25 | |
*** gfidente|fk is now known as gfidente | 23:42 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!