Monday, 2018-12-10

*** bhavikdbavishi has joined #zuul02:47
openstackgerritIan Wienand proposed openstack-infra/zuul master: Rework zuul nodepool stats reporting  https://review.openstack.org/62028503:57
openstackgerritIan Wienand proposed openstack-infra/zuul master: Add a statsd check for clashing keys  https://review.openstack.org/62043603:57
openstackgerritMerged openstack-infra/zuul master: Rework zuul nodepool stats reporting  https://review.openstack.org/62028504:42
openstackgerritMerged openstack-infra/zuul master: Add a statsd check for clashing keys  https://review.openstack.org/62043604:42
*** chkumar|off is now known as chandan_kumar04:54
*** bjackman has joined #zuul06:14
*** swest has quit IRC06:24
*** swest has joined #zuul06:44
openstackgerritneilsun proposed openstack-infra/zuul master: Add type check for zuul conf  https://review.openstack.org/59191707:04
*** neilsun has joined #zuul07:27
*** evrardjp_ is now known as evrardjp08:11
*** themroc has joined #zuul08:15
*** quiquell has joined #zuul08:22
quiquellGood morning08:22
quiquelltobias-urdin, corvus: Can we change ansible_python_interpreter at jobs ?08:23
quiquellmordred: ^ ? I found you hacoded it here to python2 https://github.com/openstack-infra/zuul/blob/7aefaf8a06f16f329ef1679c3c6f8386d6b6958b/zuul/executor/server.py#L155108:24
*** bhavikdbavishi has quit IRC08:34
*** swest has quit IRC08:43
*** bhavikdbavishi has joined #zuul08:51
*** jpena|off is now known as jpena09:01
*** swest has joined #zuul09:08
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: Implement zookeeper-auth  https://review.openstack.org/61915609:23
*** bhavikdbavishi has quit IRC09:37
*** AJaeger has quit IRC09:45
*** electrofelix has joined #zuul09:49
*** AJaeger has joined #zuul09:53
*** gtema has joined #zuul09:54
quiquelltristanC: Hello, do you know if we can change ansible_python_interpreter at zuul jobs ?10:42
quiquelltristanC: I think it's harcoded to python210:42
tristanCquiquell: i don't think you can change it at zuul jobs10:48
quiquelltristanC: Do you know if we can do it withing a running ansible ?10:50
tristanCquiquell: i don't know... what are you trying to do?10:52
quiquelltristanC: We need to exercise some jobs with python3 to discover failures10:53
quiquelltristanC: We can hack around pip execuble and virtualenv executable10:54
quiquelltristanC: But looks like setting ansible_python_interpreter is like the way to go10:54
quiquelltristanC: But I found this https://github.com/openstack-infra/zuul/blob/7aefaf8a06f16f329ef1679c3c6f8386d6b6958b/zuul/executor/server.py#L155110:54
tristanCquiquell: perhaps try https://github.com/ansible/ansible/issues/6345#issuecomment-37129460 ?10:56
quiquelltristanC: Hummm will try thanks! do you know if there is any plan on supporting this from zuul I see there is a TODO there10:57
tristanCquiquell: actually i had a patch to set it from the nodepool definition: https://review.openstack.org/#/c/570668/12/zuul/executor/server.py10:59
tristanCbecause fedora container image doesn't have python2 anymore11:00
tristanCnot sure what is the actual plan from zuul11:00
quiquelltristanC: Yep we are also having some fedora28 jobs and all at tripleo, but we don't face the problem because we have bash glue code that calls ansible-playbook in the middle11:00
*** sshnaidm has quit IRC11:15
*** sshnaidm has joined #zuul11:16
openstackgerritneilsun proposed openstack-infra/zuul master: Add type check for zuul conf  https://review.openstack.org/59191711:24
*** quiquell is now known as quiquell|brb11:55
*** bhavikdbavishi has joined #zuul11:58
*** quiquell|brb is now known as quiquell12:10
*** bhavikdbavishi has quit IRC12:26
*** gtema has quit IRC12:27
*** bhavikdbavishi has joined #zuul12:34
*** jpena is now known as jpena|lunch12:37
*** bjackman has quit IRC12:38
*** bjackman has joined #zuul12:54
*** bjackman has quit IRC12:58
*** bjackman has joined #zuul12:58
*** rlandy has joined #zuul12:58
*** nilashishc has joined #zuul13:02
ShrewsHi all. Thanks to 2 days of winter weather, I find myself stuck at home with very deteriorated/spotty internet. Don't expect me around today.13:28
*** jpena|lunch is now known as jpena13:35
*** bhavikdbavishi has quit IRC13:42
*** bjackman has quit IRC13:44
*** openstackstatus has joined #zuul14:19
*** ChanServ sets mode: +v openstackstatus14:19
*** chandan_kumar is now known as chkumar|off14:25
*** quiquell is now known as quiquell|off14:30
fungiShrews: hope your power stays on!14:36
fungiall we got was rain and tons of wind14:36
openstackgerritMerged openstack-infra/zuul master: Fixed the necesssary to necessary  https://review.openstack.org/62364614:37
fungiactually still going on14:37
openstackgerritMerged openstack-infra/zuul master: Fixed the word from congfiguration to configuration  https://review.openstack.org/62363414:40
ssbarnea|roverdoes anyone knows the reasons behind the unsafe ugo+rw permissions from https://github.com/openstack-infra/zuul-jobs/blob/master/roles/fetch-zuul-cloner/tasks/main.yaml#L24 ?14:46
ssbarnea|roverthis breaks ansible because it refuses to load ansible.cfg when is world readable (no to mention security implications of world writable files)14:46
ssbarnea|rovercorrection: world writable14:47
pabelangerhistorical IIRC, that is what jobs did in openstack.14:48
pabelangeralso, shouldn't really need to use zuul-cloner14:48
pabelangersince zuul setups up repo state at beginning14:48
pabelangerI'd personally like to see a large deprecated warning message for that role14:49
pabelangerand eventually removal14:49
ssbarnea|roverpabelanger: but on rdo we have some code that is using it, now the question is what would happen if I remove the "o" from there? would i break it?14:49
mordred++14:49
pabelangerssbarnea|rover: rdo is using a forked version14:50
pabelangerbecause that role doesn't work for rdo14:50
ssbarnea|roveryep, i know. i can patch it but I wanted to know the implications14:50
pabelangerhttps://review.openstack.org/576933/ would need to be landed for rdo14:51
pabelangerbut that breaks openstack14:51
pabelangerssbarnea|rover: I don't think we should be patching zuul-cloner for more people to use, it really was just a stepping stone to migrate jobs to zuulv314:51
ssbarnea|roverpabelanger: i will try to patch the rdo fork to see what it breaks.14:52
openstackgerritJonathan Herlin proposed openstack-infra/zuul master: Add spacing to Queue lengths line  https://review.openstack.org/62396015:28
*** bjackman has joined #zuul15:34
*** bjackman has quit IRC15:46
*** neilsun has quit IRC15:47
*** pwhalen has joined #zuul15:49
*** bhavikdbavishi has joined #zuul15:58
*** bhavikdbavishi has quit IRC16:02
*** bhavikdbavishi has joined #zuul16:03
*** bhavikdbavishi has quit IRC16:20
*** bhavikdbavishi has joined #zuul16:26
openstackgerritBenoît Bayszczak proposed openstack-infra/zuul master: add fetch_vault_secrets Ansible module  https://review.openstack.org/62031116:27
*** nilashishc has quit IRC16:30
*** chkumar|off has quit IRC16:32
*** themroc has quit IRC16:34
*** openstackgerrit has quit IRC16:35
*** openstackgerrit has joined #zuul16:57
openstackgerritBenoît Bayszczak proposed openstack-infra/zuul master: add fetch_vault_secrets Ansible module  https://review.openstack.org/62031116:57
*** bhavikdbavishi has quit IRC17:37
*** bhavikdbavishi has joined #zuul17:37
*** smyers has quit IRC17:40
*** smyers has joined #zuul17:42
*** jpena is now known as jpena|off18:14
*** adam_g has quit IRC18:15
*** adam_g has joined #zuul18:15
*** smyers has quit IRC18:19
*** smyers has joined #zuul18:20
*** bhavikdbavishi has quit IRC18:43
*** bhavikdbavishi has joined #zuul18:43
*** smyers_ has joined #zuul19:10
*** smyers has quit IRC19:11
*** smyers_ is now known as smyers19:11
*** electrofelix has quit IRC19:11
*** bhavikdbavishi has quit IRC19:20
*** hashar has joined #zuul20:15
*** hashar has quit IRC21:19
*** panda is now known as panda|off23:06

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!