*** dcramer__ has joined #openstack-infra | 00:01 | |
*** colinmcn_ has joined #openstack-infra | 00:01 | |
asselin | Hi, I've been stuck trying to get my new jenkins slave talking to my jenkins master. logs is here which shows the jenkins stack trace. http://paste.openstack.org/show/75377/ | 00:02 |
---|---|---|
*** UtahDave has joined #openstack-infra | 00:03 | |
asselin | I'm not use "credentials" since documentation says it optional and I couldn't figure out what it is, but it seems to be needed? | 00:03 |
clarkb | asselin: which docs? | 00:03 |
clarkb | asselin: is this with nodepool? | 00:03 |
asselin | clarkb, nodepool | 00:03 |
asselin | yes | 00:03 |
clarkb | asselin: it is optional if your jenkins is configured with a default set of credentials. Otherwise it is required iirc | 00:04 |
clarkb | and you ahve to give nodepool a credential ID | 00:04 |
asselin | clarkb, do you have references for credentials docs? I've searched but still missing it.... | 00:05 |
asselin | clarkb, also, what does it even look like? integer? random hex string? | 00:05 |
asselin | clarkb, I have the user and apikey set.... | 00:06 |
clarkb | it looks like a sha1 | 00:06 |
clarkb | or uuid | 00:07 |
clarkb | basically hex | 00:07 |
*** mriedem has joined #openstack-infra | 00:07 | |
asselin | clarkb, and it's not the same as the api key? (that's also hex) | 00:07 |
clarkb | asselin: no it isn't the same. the api key is like a password | 00:07 |
asselin | clarkb, so credentials id is something setup with jenkins, right? or in the nodepool.yaml file? | 00:08 |
clarkb | the credentials id can be find in credentials.xml in the root of your jenkins home dir | 00:08 |
clarkb | it is something jenkins has that you need to tell nodepool about | 00:08 |
*** markmcclain1 has joined #openstack-infra | 00:09 | |
*** markmcclain1 has quit IRC | 00:09 | |
clarkb | you manage it through the credentials config in the jenkins UI | 00:09 |
*** markmcclain1 has joined #openstack-infra | 00:09 | |
asselin | clarkb, ok, I see the 'id' in the credentials.xml file....not in the ui. I will try that. | 00:11 |
*** markmcclain has quit IRC | 00:12 | |
*** msabramo has joined #openstack-infra | 00:13 | |
*** prad has joined #openstack-infra | 00:13 | |
*** ihrachyshka_ has quit IRC | 00:14 | |
asselin | clarkb, that was it! INFO:nodepool.NodeLauncher:Node id: 42 added to jenkins | 00:15 |
asselin | clarkb, of course, there's the next issue....:) | 00:16 |
asselin | I'll save that for tomorrow. thank you clarkb! | 00:17 |
*** MarkAtwood has quit IRC | 00:18 | |
fungi | i didn't know about credentials.xml and just fished it out of /var/lib/jenkins/config.xml | 00:19 |
*** Sukhdev has quit IRC | 00:20 | |
*** markmcclain1 has quit IRC | 00:23 | |
*** thuc_ has quit IRC | 00:28 | |
*** thuc has joined #openstack-infra | 00:29 | |
openstackgerrit | Joe Gordon proposed a change to openstack-dev/hacking: Add posargs to tox -epep8 https://review.openstack.org/86196 | 00:29 |
openstackgerrit | Joe Gordon proposed a change to openstack-dev/hacking: Move hacking to pep8 1.5.4 https://review.openstack.org/86197 | 00:29 |
*** matsuhashi has joined #openstack-infra | 00:31 | |
sdague | fungi: I thought the fedora image was cached - http://logs.openstack.org/77/86177/1/check/check-tempest-dsvm-neutron/7f4b91b/logs/devstacklog.txt.gz ? | 00:31 |
sdague | http://logs.openstack.org/77/86177/1/check/check-tempest-dsvm-neutron/7f4b91b/logs/devstacklog.txt.gz#_2014-04-08_22_15_13_389 | 00:31 |
*** esker has joined #openstack-infra | 00:31 | |
sdague | better link | 00:31 |
*** dims has joined #openstack-infra | 00:32 | |
*** thuc has quit IRC | 00:33 | |
openstackgerrit | Joe Gordon proposed a change to openstack-dev/hacking: Use pep8's noqa argument https://review.openstack.org/86199 | 00:35 |
*** weshay has joined #openstack-infra | 00:36 | |
*** andreaf has quit IRC | 00:37 | |
fungi | sdague: just checked a ready node of that type in the same provider, and it has /home/jenkins/cache/files/Fedora-x86_64-20-20131211.1-sda.qcow2 | 00:37 |
fungi | md5sum is d41d8cd98f00b204e9800998ecf8427e | 00:37 |
sdague | hmmm | 00:38 |
*** sabari has quit IRC | 00:38 | |
fungi | so presumably cached on the same image | 00:38 |
sdague | so are we missing a piece to copy it to the devstack tree? | 00:39 |
sdague | http://logs.openstack.org/77/86177/1/check/check-tempest-dsvm-neutron/7f4b91b/logs/devstacklog.txt.gz#_2014-04-08_22_15_09_187 | 00:40 |
fungi | i think d-g copies the contents of /home/jenkins/cache/files to /opt/stack/new/files | 00:43 |
fungi | er, /opt/stack/new/devstack/files | 00:43 |
*** prad has quit IRC | 00:43 | |
sdague | so it's on a different image http://logs.openstack.org/77/86177/1/check/check-tempest-dsvm-full/fef1c61/logs/devstacklog.txt.gz#_2014-04-08_22_15_57_861 | 00:45 |
sdague | the failure is in devstack-precise-hpcloud-region-b-3512265 | 00:46 |
sdague | yeh, all the fails are hp-region-b | 00:46 |
*** dansmith has quit IRC | 00:48 | |
fungi | i wonder whether those nodes were from stale images? | 00:49 |
fungi | the node i checked in a ready state had it | 00:49 |
*** dansmith has joined #openstack-infra | 00:50 | |
*** wchrisj has quit IRC | 00:50 | |
sdague | I don't know | 00:50 |
sdague | but I had 4 hp region b fails on - https://review.openstack.org/#/c/86177/ | 00:50 |
sdague | and everything else seemed to work | 00:50 |
*** colinmcn_ has quit IRC | 00:52 | |
*** SumitNaiksatam has quit IRC | 00:53 | |
*** derekh has quit IRC | 00:53 | |
*** yamahata has quit IRC | 00:54 | |
*** e0ne has joined #openstack-infra | 00:55 | |
*** matsuhashi has quit IRC | 00:56 | |
*** Ryan_Lane has quit IRC | 00:56 | |
*** SumitNaiksatam has joined #openstack-infra | 00:57 | |
*** e0ne has quit IRC | 01:00 | |
*** alff__ has quit IRC | 01:03 | |
*** alff_ has quit IRC | 01:03 | |
*** alff has joined #openstack-infra | 01:03 | |
*** alff has quit IRC | 01:05 | |
*** matsuhashi has joined #openstack-infra | 01:05 | |
*** mriedem has quit IRC | 01:07 | |
*** colinmcnamara has joined #openstack-infra | 01:07 | |
*** wchrisj has joined #openstack-infra | 01:09 | |
*** colinmcnamara has quit IRC | 01:10 | |
*** signed8b_ has joined #openstack-infra | 01:11 | |
*** signed8bit has quit IRC | 01:14 | |
*** SumitNaiksatam has quit IRC | 01:15 | |
*** mriedem has joined #openstack-infra | 01:19 | |
*** wchrisj has quit IRC | 01:20 | |
*** ilives has joined #openstack-infra | 01:24 | |
*** nosnos has joined #openstack-infra | 01:29 | |
jeblair | clarkb, fungi: should i start reiussuing all our certs? | 01:31 |
clarkb | jeblair: yes, I think we should revoke then reissue | 01:31 |
clarkb | jeblair: we were waiting on you for that because you had done them before so presumably have account creds to do such things | 01:31 |
*** mbacchi has quit IRC | 01:34 | |
*** changbl has joined #openstack-infra | 01:35 | |
*** thuc has joined #openstack-infra | 01:39 | |
*** gokrokve has joined #openstack-infra | 01:42 | |
*** thuc has quit IRC | 01:44 | |
*** Ryan_Lane has joined #openstack-infra | 01:48 | |
*** Ryan_Lane has joined #openstack-infra | 01:48 | |
harlowja | qq, how often is http://pypi.openstack.org/ refreshed against http://pypi.python.org/ ? | 01:51 |
*** yaguang has joined #openstack-infra | 01:51 | |
*** Ryan_Lane has quit IRC | 01:52 | |
*** e0ne has joined #openstack-infra | 01:55 | |
*** MarkAtwood has joined #openstack-infra | 01:59 | |
*** MarkAtwood has quit IRC | 01:59 | |
clarkb | once a day at least, but possibly more often if we release code that triggers an update | 01:59 |
*** e0ne has quit IRC | 02:00 | |
*** jgrimm has quit IRC | 02:06 | |
*** UtahDave has quit IRC | 02:06 | |
*** esker has quit IRC | 02:08 | |
*** esker has joined #openstack-infra | 02:09 | |
*** mwagner_lap has quit IRC | 02:10 | |
*** zhiyan is now known as zhiyan_ | 02:11 | |
harlowja | clarkb thx | 02:12 |
*** zhiyan_ is now known as zhiyan | 02:12 | |
*** pcrews has quit IRC | 02:15 | |
*** jgrimm has joined #openstack-infra | 02:17 | |
*** pcrews has joined #openstack-infra | 02:21 | |
*** yamahata has joined #openstack-infra | 02:22 | |
*** weshay has quit IRC | 02:23 | |
*** julim has quit IRC | 02:26 | |
*** mriedem has quit IRC | 02:31 | |
*** shardy has quit IRC | 02:42 | |
*** AaronGr has quit IRC | 02:46 | |
*** AaronGr has joined #openstack-infra | 02:47 | |
*** gyee has quit IRC | 02:47 | |
*** AaronGr has quit IRC | 02:48 | |
*** aconrad has joined #openstack-infra | 02:48 | |
*** AaronGr has joined #openstack-infra | 02:48 | |
*** Ryan_Lane has joined #openstack-infra | 02:49 | |
*** Ryan_Lane has quit IRC | 02:53 | |
*** e0ne has joined #openstack-infra | 02:55 | |
*** marun has quit IRC | 02:55 | |
*** fifieldt has joined #openstack-infra | 02:56 | |
*** e0ne has quit IRC | 02:58 | |
*** malini_afk is now known as malini | 03:05 | |
*** ianw has quit IRC | 03:09 | |
*** ianw has joined #openstack-infra | 03:10 | |
*** matsuhashi has quit IRC | 03:12 | |
*** harlowja is now known as harlowja_away | 03:12 | |
*** msabramo has quit IRC | 03:16 | |
*** dcramer__ has quit IRC | 03:16 | |
*** msabramo has joined #openstack-infra | 03:17 | |
*** david-lyle has joined #openstack-infra | 03:18 | |
*** cody-somerville has joined #openstack-infra | 03:19 | |
*** jeckersb is now known as jeckersb_gone | 03:24 | |
*** nosnos has quit IRC | 03:26 | |
*** amcrn has quit IRC | 03:30 | |
*** yamahata has quit IRC | 03:35 | |
*** malini is now known as malini_afk | 03:47 | |
*** dcramer__ has joined #openstack-infra | 03:49 | |
*** wenlock has joined #openstack-infra | 03:50 | |
*** e0ne has joined #openstack-infra | 03:55 | |
*** e0ne has quit IRC | 04:00 | |
*** dcramer__ has quit IRC | 04:00 | |
*** chandan_kumar has joined #openstack-infra | 04:04 | |
*** zhiyan is now known as zhiyan_ | 04:06 | |
*** thuc has joined #openstack-infra | 04:10 | |
*** thuc_ has joined #openstack-infra | 04:12 | |
*** thuc has quit IRC | 04:14 | |
*** dcramer__ has joined #openstack-infra | 04:14 | |
*** matsuhashi has joined #openstack-infra | 04:24 | |
*** nosnos has joined #openstack-infra | 04:25 | |
*** SumitNaiksatam has joined #openstack-infra | 04:30 | |
*** jepoy has joined #openstack-infra | 04:30 | |
*** wchrisj has joined #openstack-infra | 04:37 | |
*** mwagner_lap has joined #openstack-infra | 04:39 | |
*** dcramer__ has quit IRC | 04:39 | |
*** asettle has quit IRC | 04:42 | |
*** matsuhashi has quit IRC | 04:42 | |
*** dcramer_ has joined #openstack-infra | 04:43 | |
*** cody-somerville has quit IRC | 04:44 | |
*** alexandra_ has joined #openstack-infra | 04:45 | |
*** alexandra_ is now known as asettle | 04:46 | |
*** jepoy_ has joined #openstack-infra | 04:46 | |
*** jepoy_ has quit IRC | 04:47 | |
*** jepoy has joined #openstack-infra | 04:47 | |
*** jepoy has quit IRC | 04:48 | |
*** matsuhashi has joined #openstack-infra | 04:50 | |
*** Ryan_Lane has joined #openstack-infra | 04:50 | |
*** wchrisj has quit IRC | 04:52 | |
*** Ryan_Lane has quit IRC | 04:54 | |
*** e0ne has joined #openstack-infra | 04:55 | |
*** mrmartin has joined #openstack-infra | 04:56 | |
*** HenryG has quit IRC | 04:59 | |
*** e0ne has quit IRC | 05:00 | |
*** chandan_kumar has quit IRC | 05:00 | |
*** thuc_ has quit IRC | 05:01 | |
*** thuc has joined #openstack-infra | 05:02 | |
*** ildikov has quit IRC | 05:02 | |
*** ildikov_ has quit IRC | 05:03 | |
*** dcramer_ has quit IRC | 05:03 | |
*** thuc has quit IRC | 05:06 | |
*** chandan_kumar has joined #openstack-infra | 05:14 | |
*** imcsk8_lap has quit IRC | 05:17 | |
*** dcramer_ has joined #openstack-infra | 05:17 | |
*** imcsk8_lap has joined #openstack-infra | 05:17 | |
*** sabari has joined #openstack-infra | 05:23 | |
*** andreykurilin_ has joined #openstack-infra | 05:23 | |
*** wenlock has quit IRC | 05:24 | |
*** sabari has quit IRC | 05:28 | |
*** andreykurilin_ has quit IRC | 05:28 | |
*** skraynev_afk is now known as skraynev | 05:29 | |
*** ildikov_ has joined #openstack-infra | 05:30 | |
*** zhiyan_ is now known as zhiyan | 05:31 | |
*** gokrokve has quit IRC | 05:32 | |
*** mrmartin has quit IRC | 05:38 | |
*** jhesketh has quit IRC | 05:38 | |
*** _nadya_ has joined #openstack-infra | 05:44 | |
*** gokrokve_ has joined #openstack-infra | 05:44 | |
*** _nadya_ has quit IRC | 05:46 | |
*** e0ne has joined #openstack-infra | 05:55 | |
tchaypo | dstufft: have you had a chance to look at my pull request since I rebased? | 05:56 |
tchaypo | (and fixed the embarassing line-too-long issue) | 05:56 |
*** e0ne has quit IRC | 05:58 | |
*** yolanda has joined #openstack-infra | 06:02 | |
*** sandywalsh_ has quit IRC | 06:03 | |
*** _nadya_ has joined #openstack-infra | 06:04 | |
*** sandywalsh has joined #openstack-infra | 06:05 | |
*** yolanda has quit IRC | 06:06 | |
*** jhesketh has joined #openstack-infra | 06:07 | |
*** sirushti has quit IRC | 06:08 | |
*** _sirushti has joined #openstack-infra | 06:09 | |
*** _sirushti is now known as sirushti | 06:09 | |
*** thuc has joined #openstack-infra | 06:12 | |
*** sandywalsh has quit IRC | 06:14 | |
*** sandywalsh has joined #openstack-infra | 06:16 | |
*** afazekas has quit IRC | 06:16 | |
*** thuc has quit IRC | 06:17 | |
*** yamahata has joined #openstack-infra | 06:17 | |
*** yamahata__ has joined #openstack-infra | 06:18 | |
*** yamahata has quit IRC | 06:20 | |
*** markmc has joined #openstack-infra | 06:33 | |
*** jamielennox is now known as jamielennox|away | 06:35 | |
*** _nadya_ has quit IRC | 06:35 | |
*** e0ne has joined #openstack-infra | 06:35 | |
*** gokrokve_ has quit IRC | 06:36 | |
*** andreykurilin_ has joined #openstack-infra | 06:38 | |
*** _nadya_ has joined #openstack-infra | 06:39 | |
*** e0ne has quit IRC | 06:40 | |
*** adam_g has quit IRC | 06:41 | |
*** _nadya_ has quit IRC | 06:41 | |
*** adam_g has joined #openstack-infra | 06:42 | |
*** adam_g has quit IRC | 06:42 | |
*** adam_g has joined #openstack-infra | 06:42 | |
*** aconrad has quit IRC | 06:46 | |
*** dims has quit IRC | 06:48 | |
*** e0ne has joined #openstack-infra | 06:48 | |
*** afazekas has joined #openstack-infra | 06:50 | |
*** e0ne has quit IRC | 06:51 | |
*** e0ne has joined #openstack-infra | 06:52 | |
*** e0ne has quit IRC | 06:53 | |
*** yfried has quit IRC | 06:57 | |
*** adam_g has quit IRC | 06:58 | |
*** adam_g has joined #openstack-infra | 06:58 | |
*** adam_g has quit IRC | 06:58 | |
*** adam_g has joined #openstack-infra | 06:58 | |
*** dims has joined #openstack-infra | 07:02 | |
*** yamahata__ has quit IRC | 07:03 | |
*** jhesketh has quit IRC | 07:04 | |
*** yolanda has joined #openstack-infra | 07:06 | |
*** flaper87|afk is now known as flaper87 | 07:06 | |
*** jcoufal has joined #openstack-infra | 07:08 | |
*** mrda is now known as mrda_away | 07:18 | |
*** shardy has joined #openstack-infra | 07:19 | |
*** ildikov has joined #openstack-infra | 07:20 | |
*** andreykurilin_ has quit IRC | 07:22 | |
*** jgallard has joined #openstack-infra | 07:28 | |
*** matsuhashi has quit IRC | 07:29 | |
mattoliverau | Night all | 07:29 |
*** matsuhashi has joined #openstack-infra | 07:31 | |
*** mrmartin has joined #openstack-infra | 07:31 | |
*** matsuhashi has quit IRC | 07:35 | |
*** gokrokve has joined #openstack-infra | 07:36 | |
*** sdake has quit IRC | 07:36 | |
*** sdake has joined #openstack-infra | 07:37 | |
*** gokrokve has quit IRC | 07:41 | |
*** BobBallAway is now known as BobBall | 07:41 | |
*** skolekonov has joined #openstack-infra | 07:42 | |
*** skolekonov has quit IRC | 07:43 | |
*** skolekonov has joined #openstack-infra | 07:43 | |
*** che-arne has joined #openstack-infra | 07:49 | |
*** matsuhashi has joined #openstack-infra | 07:49 | |
*** dizquierdo has joined #openstack-infra | 07:57 | |
*** Ildi has joined #openstack-infra | 07:58 | |
*** ildikov has quit IRC | 08:01 | |
*** jpich has joined #openstack-infra | 08:01 | |
*** hashar has joined #openstack-infra | 08:01 | |
*** yaguang has quit IRC | 08:04 | |
*** yaguang has joined #openstack-infra | 08:06 | |
*** ihrachyshka has joined #openstack-infra | 08:16 | |
*** locke105 has quit IRC | 08:18 | |
openstackgerrit | Sergey Skripnick proposed a change to openstack-infra/config: Make rally-scenarios job non-voting https://review.openstack.org/86254 | 08:21 |
*** ildikov has joined #openstack-infra | 08:22 | |
*** yaguang has quit IRC | 08:23 | |
*** yaguang has joined #openstack-infra | 08:24 | |
*** Ildi has quit IRC | 08:24 | |
*** markmc has quit IRC | 08:26 | |
*** jooools has joined #openstack-infra | 08:33 | |
*** roeyc has joined #openstack-infra | 08:34 | |
*** markwash has quit IRC | 08:35 | |
openstackgerrit | Vladan Popovic proposed a change to openstack-dev/pbr: Add tox -e genconfic for sample conf generation https://review.openstack.org/85207 | 08:35 |
*** gokrokve has joined #openstack-infra | 08:37 | |
*** e0ne has joined #openstack-infra | 08:38 | |
*** dizquierdo has quit IRC | 08:39 | |
*** e0ne_ has joined #openstack-infra | 08:39 | |
*** markmc has joined #openstack-infra | 08:40 | |
*** eyerediskin has joined #openstack-infra | 08:41 | |
eyerediskin | hi all. have a question about DEVSTACK_GATE_EXERCISES | 08:41 |
eyerediskin | can i set this variable inside pre_test_hook? | 08:41 |
eyerediskin | export DEVSTACK_GATE_EXERCISES=1 # in pre_test_hook | 08:41 |
eyerediskin | seems like this doesnt work | 08:42 |
*** gokrokve has quit IRC | 08:42 | |
eyerediskin | is this variable should be set only in job - builders: shell: - section? | 08:42 |
*** e0ne has quit IRC | 08:42 | |
eyerediskin | just like here: https://github.com/openstack-infra/config/blob/master/modules/openstack_project/files/jenkins_job_builder/config/devstack-gate.yaml#L444 | 08:44 |
*** ilyashakhat has quit IRC | 08:46 | |
*** yamahata has joined #openstack-infra | 08:47 | |
*** _nadya_ has joined #openstack-infra | 08:49 | |
*** chandan_kumar has quit IRC | 08:49 | |
*** sdake has quit IRC | 08:52 | |
*** ilyashakhat has joined #openstack-infra | 08:53 | |
*** e0ne_ has quit IRC | 09:02 | |
*** e0ne has joined #openstack-infra | 09:02 | |
*** chandan_kumar has joined #openstack-infra | 09:03 | |
*** e0ne_ has joined #openstack-infra | 09:03 | |
*** dizquierdo has joined #openstack-infra | 09:04 | |
*** jhesketh_ has joined #openstack-infra | 09:06 | |
*** e0ne has quit IRC | 09:07 | |
openstackgerrit | David Pursehouse proposed a change to openstack-infra/jeepyb: Improve message posted when inactive changes are abandoned https://review.openstack.org/86261 | 09:11 |
*** jhesketh_ has quit IRC | 09:13 | |
*** jhesketh_ has joined #openstack-infra | 09:14 | |
*** morganfainberg is now known as morganfainberg_Z | 09:16 | |
*** e0ne has joined #openstack-infra | 09:17 | |
*** david-lyle has quit IRC | 09:20 | |
*** e0ne_ has quit IRC | 09:20 | |
openstackgerrit | Sergey Skripnick proposed a change to openstack-infra/config: Fix pre_test_hook in rally-scenarios-job https://review.openstack.org/86254 | 09:24 |
eyerediskin | shame on me. i understand why exported envs are not available >_< | 09:25 |
chmouel | mordred: do you know if https://github.com/openstack-infra/gerritbot have sleekxmpp support (or if i can send a patch for this) | 09:33 |
*** gokrokve has joined #openstack-infra | 09:38 | |
*** yaguang has quit IRC | 09:40 | |
*** dizquierdo has quit IRC | 09:41 | |
*** gokrokve has quit IRC | 09:42 | |
*** zhiyan is now known as zhiyan_ | 09:43 | |
*** ominakov has joined #openstack-infra | 09:44 | |
*** matsuhas_ has joined #openstack-infra | 09:45 | |
*** matsuhashi has quit IRC | 09:45 | |
*** matsuhas_ has quit IRC | 09:45 | |
*** matsuhashi has joined #openstack-infra | 09:46 | |
openstackgerrit | Nikita Konovalov proposed a change to openstack-infra/storyboard: Hard delete for auth stuff https://review.openstack.org/86271 | 09:50 |
*** _nadya_ has quit IRC | 09:54 | |
*** Ryan_Lane has joined #openstack-infra | 09:54 | |
reed | http://summit.openstack.org/cfp/details/257 | 09:58 |
*** Ryan_Lane has quit IRC | 09:58 | |
reed | chmouel, where do we use xmpp? | 09:59 |
*** ominakov has quit IRC | 10:00 | |
chmouel | reed: in openstack in general? i don't think we do, I just wanted to run it for notifying internal xmpp server i have | 10:01 |
reed | chmouel, oh, ok | 10:01 |
reed | i wish we had more xmpp and less irc :) | 10:01 |
reed | maybe with plans to add some webrtc to the mix | 10:01 |
chmouel | reed: I am not so sure, groupchat are not that great in xmpp compared to irc chat room | 10:02 |
chmouel | (or the bitlbee interaction with it) | 10:02 |
rcarrillocruz | guys, i'm trying to pull info from gerrit DB. If I do a 'select subject from changes', it will only show the first line of the subject of the change | 10:02 |
reed | large chats maybe, small group chats are good for me | 10:02 |
rcarrillocruz | e.g. the patchset shows in the gerrit UI "Add README' and then a new line, and then 'Closes-Bug: foo' | 10:03 |
*** roeyc has quit IRC | 10:03 | |
rcarrillocruz | why the sql query will only show 'Add README' | 10:03 |
rcarrillocruz | anyone knows? | 10:03 |
*** chandan_kumar_ has joined #openstack-infra | 10:03 | |
*** jgallard has quit IRC | 10:06 | |
*** chandan_kumar has quit IRC | 10:06 | |
*** tianst20 has quit IRC | 10:06 | |
*** matsuhashi has quit IRC | 10:12 | |
*** dizquierdo has joined #openstack-infra | 10:14 | |
*** matsuhashi has joined #openstack-infra | 10:16 | |
*** yfried has joined #openstack-infra | 10:16 | |
*** jerryz has quit IRC | 10:19 | |
openstackgerrit | A change was merged to openstack-infra/release-tools: Remove tox locale overrides https://review.openstack.org/72219 | 10:20 |
*** mdenny has quit IRC | 10:22 | |
*** jp_at_hp has joined #openstack-infra | 10:24 | |
*** branen has quit IRC | 10:27 | |
*** alexpilotti has joined #openstack-infra | 10:29 | |
*** dims has quit IRC | 10:29 | |
sdague | ttx: https://review.openstack.org/#/c/85167/ - thoughts on this, it's currently blocking barbican IIRC | 10:29 |
*** asselin has quit IRC | 10:29 | |
sdague | chmouel: I think xmpp would be welcome addition to bots, there was some conversation about that the other day. Lots of internal orgs have hipchat which would let the bots talk to it | 10:30 |
ttx | sdague: I think it makes sense | 10:31 |
*** paul-- has joined #openstack-infra | 10:31 | |
ttx | sdague: that said i wasn't really involved in the original design so there may be a consequence I'm not seeing | 10:31 |
ttx | would be good to get mordred's +2 on that one | 10:32 |
*** chandan_kumar_ has quit IRC | 10:32 | |
*** derekh has joined #openstack-infra | 10:32 | |
sdague | sure | 10:33 |
sdague | jogo was, but he's only +1 on the repo | 10:33 |
sdague | SergeyLukjanov: you about? | 10:33 |
SergeyLukjanov | sdague, hey | 10:34 |
* SergeyLukjanov reading scrollback | 10:34 | |
sdague | so I'd *really* like to get this landed - https://review.openstack.org/#/c/84829/ because we've got about 6 days before we'd set the tempest branch | 10:34 |
sdague | and I'd like to see it working before we decide to not set the branch | 10:34 |
SergeyLukjanov | sdague, looking on it | 10:37 |
sdague | cool, thank you | 10:37 |
SergeyLukjanov | I'm a bit behind the reviews due to the "release is coming" :( | 10:37 |
sdague | SergeyLukjanov: yeh, no problem | 10:37 |
*** gokrokve has joined #openstack-infra | 10:39 | |
*** eloib_ has quit IRC | 10:39 | |
SergeyLukjanov | sdague, looks nice | 10:40 |
SergeyLukjanov | sdague, am I right that it's needed for testing clients on stable branches for example? | 10:40 |
sdague | no | 10:40 |
sdague | clients will test fine because they don't set stable branches | 10:41 |
sdague | oh... wait, yes, it could be used to test master clients on stable servers | 10:41 |
SergeyLukjanov | sdague, yup, that's what I mean :) | 10:41 |
sdague | I hadn't thought about that, but yes, it could be used for that | 10:41 |
sdague | I had one use case in mind, but I wrote it generically, and I think we'll find it generally useful | 10:42 |
SergeyLukjanov | sdague, your use case is to test stable server on master tempest? | 10:42 |
sdague | yep | 10:42 |
SergeyLukjanov | nice | 10:42 |
SergeyLukjanov | I really like that it's a general approach | 10:42 |
SergeyLukjanov | we could reuse it for mostly all branch override manipulations | 10:43 |
sdague | it's actually only needed for testing master tempest on stable servers | 10:43 |
*** gokrokve has quit IRC | 10:43 | |
sdague | yep | 10:43 |
sdague | yeh, the matrix will get interesting | 10:43 |
sdague | http://ci.openstack.org/devstack-gate.html doesn't seem to match the README.rst in devstack-gate, any idea where that's located? | 10:44 |
sdague | or what we need to do to fix the doc connect | 10:44 |
SergeyLukjanov | sdague, do we need for someone else or I can approve it? | 10:45 |
SergeyLukjanov | sdague, openstack-infra/config doc/source/devstack-gate.rst | 10:45 |
sdague | I feel like I addressed all the feedback. jeblair would be nice to look at it again, but there is also a time issue here. | 10:46 |
sdague | because we're going to get stable branches in the servers on tuesday | 10:46 |
sdague | and we really need the infra working by then | 10:46 |
SergeyLukjanov | sdague, it couldn't breaking anything I think | 10:46 |
*** dosaboy has quit IRC | 10:49 | |
*** dosaboy has joined #openstack-infra | 10:50 | |
SergeyLukjanov | sdague, first jeblair's comment is unanswered - https://review.openstack.org/#/c/84829/7..8/functions.sh | 10:50 |
SergeyLukjanov | sorry, need to go, will return back in ~40 mins, last chance to have lunch before the meetings | 10:50 |
*** Ryan_Lane has joined #openstack-infra | 10:55 | |
*** chandan_kumar_ has joined #openstack-infra | 10:58 | |
*** Ryan_Lane has quit IRC | 10:59 | |
*** matsuhashi has quit IRC | 11:05 | |
*** matsuhashi has joined #openstack-infra | 11:05 | |
*** HenryG has joined #openstack-infra | 11:07 | |
*** jlibosva has joined #openstack-infra | 11:08 | |
*** ominakov has joined #openstack-infra | 11:10 | |
*** ihrachyshka has quit IRC | 11:10 | |
sdague | SergeyLukjanov: is Sergey Skripnick on irc? | 11:14 |
SergeyLukjanov | sdague, hm, let me check | 11:15 |
sdague | There are days I wish there was meta reviewing in gerrit, so you could -1 someone's -1 :) | 11:15 |
sdague | because I think he missed that - https://review.openstack.org/#/c/85293/ is going away from bash parsing | 11:16 |
*** rlandy has joined #openstack-infra | 11:16 | |
SergeyLukjanov | sdague, redixin == Sergey Skripnick | 11:17 |
SergeyLukjanov | sdague, there is already a +1 from him :) | 11:18 |
*** matsuhashi has quit IRC | 11:19 | |
eyerediskin | sdague: xD | 11:19 |
SergeyLukjanov | sdague, +2 from me for https://review.openstack.org/#/c/84829/, but I prefer to wait for today someone else to review/approve it | 11:20 |
sdague | SergeyLukjanov: ok, cool | 11:22 |
sdague | as long as we merge today, I'm fine. I'm just getting nervous on not having this run enough before release to find bugs in it | 11:22 |
jeblair | chmouel: talk to kiall about xmpp support; he was looking into a bot framework that supports both | 11:28 |
chmouel | jeblair: like err? ^ | 11:28 |
*** julienvey has joined #openstack-infra | 11:28 | |
*** julim has joined #openstack-infra | 11:28 | |
jeblair | chmouel: that's the one; i also wanted to know if we could use it to replace all of our different bots (including meetbot) with just one | 11:29 |
chmouel | jeblair: yeah it probably could | 11:30 |
*** jamielennox|away is now known as jamielennox | 11:30 | |
jeblair | chmouel: http://eavesdrop.openstack.org/irclogs/%23openstack-infra/%23openstack-infra.2014-03-12.log | 11:30 |
chmouel | i started that https://github.com/chmouel/python-nonobot before I saw err | 11:30 |
jeblair | chmouel: search for 'errbot' there ^ | 11:31 |
jeblair | chmouel: there's lots of good conversation about it | 11:31 |
*** e0ne has quit IRC | 11:32 | |
*** e0ne has joined #openstack-infra | 11:32 | |
*** jamielennox is now known as jamielennox|away | 11:33 | |
chmouel | jeblair: cool, so it seems that you guys on it :) | 11:33 |
*** jaypipes has quit IRC | 11:33 | |
jeblair | chmouel: i think we'd like to do it, i don't know if kiall has had time; this could still be your thing. :) | 11:34 |
chmouel | jeblair: hah well i need it for some internal thing, not sure if moving everything over errbot is going to be straightforward as add porting gerritbot to use xmpp | 11:35 |
*** matsuhashi has joined #openstack-infra | 11:36 | |
*** e0ne has quit IRC | 11:36 | |
jeblair | chmouel: porting gerritbot to errbot might be about as easy and it means we can port the others over later | 11:36 |
chmouel | jeblair: cool you convinced me :) let's give it a try | 11:38 |
*** dims has joined #openstack-infra | 11:39 | |
*** matsuhashi has quit IRC | 11:39 | |
jeblair | sdague: couple more things on https://review.openstack.org/#/c/84829/ | 11:40 |
jeblair | (comments inline) | 11:40 |
jhesketh_ | jeblair: you're up early! | 11:41 |
*** ihrachyshka has joined #openstack-infra | 11:41 | |
openstackgerrit | Sean Dague proposed a change to openstack-infra/devstack-gate: allow for generic branch overrides https://review.openstack.org/84829 | 11:43 |
sdague | jeblair: done | 11:43 |
*** dkranz has quit IRC | 11:45 | |
openstackgerrit | Sean Dague proposed a change to openstack-infra/devstack-gate: discover all errors https://review.openstack.org/86292 | 11:46 |
sdague | based on the patches I'm getting bounced, I think we're basically nearly wedged in the gate over the heat / keystone issue, we've just got so little patch flow it's not been noticed | 11:48 |
*** ihrachyshka has quit IRC | 11:50 | |
*** ihrachyshka has joined #openstack-infra | 11:51 | |
*** ihrachyshka has quit IRC | 11:51 | |
*** yolanda has quit IRC | 11:54 | |
chmouel | jeblair: errbot code is weird, it's using py3to2 and using it with python3 print some obv python3 error :( | 12:00 |
*** markmc has quit IRC | 12:00 | |
*** dcramer_ has quit IRC | 12:01 | |
*** pcargnel has joined #openstack-infra | 12:02 | |
*** dims has quit IRC | 12:03 | |
*** e0ne has joined #openstack-infra | 12:03 | |
openstackgerrit | Jason Kincl proposed a change to openstack-infra/jenkins-job-builder: Add support for Copy Artifact project copy permissions https://review.openstack.org/82551 | 12:06 |
*** zhiyan_ is now known as zhiyan | 12:10 | |
openstackgerrit | Matthias Runge proposed a change to openstack-dev/hacking: Avoid IndexError https://review.openstack.org/86303 | 12:12 |
*** dprince has joined #openstack-infra | 12:13 | |
*** maxbit has joined #openstack-infra | 12:15 | |
*** markmc has joined #openstack-infra | 12:16 | |
*** ihrachyshka has joined #openstack-infra | 12:18 | |
*** eharney has quit IRC | 12:19 | |
*** pdmars has joined #openstack-infra | 12:20 | |
*** dkranz has joined #openstack-infra | 12:31 | |
*** aysyd has joined #openstack-infra | 12:35 | |
*** yolanda has joined #openstack-infra | 12:39 | |
*** jgallard has joined #openstack-infra | 12:39 | |
*** gokrokve has joined #openstack-infra | 12:40 | |
*** adalbas has quit IRC | 12:43 | |
*** wchrisj has joined #openstack-infra | 12:44 | |
*** maxbit has quit IRC | 12:44 | |
*** gokrokve has quit IRC | 12:45 | |
*** cody-somerville has joined #openstack-infra | 12:45 | |
*** mbacchi has joined #openstack-infra | 12:46 | |
*** jeckersb_gone is now known as jeckersb | 12:49 | |
openstackgerrit | A change was merged to openstack-infra/storyboard: Missing docstrings added https://review.openstack.org/85718 | 12:49 |
*** cody-somerville has quit IRC | 12:52 | |
*** thuc has joined #openstack-infra | 12:52 | |
*** YorikSar has quit IRC | 12:52 | |
*** thuc_ has joined #openstack-infra | 12:53 | |
*** e0ne has quit IRC | 12:55 | |
*** e0ne has joined #openstack-infra | 12:55 | |
*** dims has joined #openstack-infra | 12:56 | |
*** Ryan_Lane has joined #openstack-infra | 12:56 | |
*** thuc has quit IRC | 12:57 | |
*** thomasem has joined #openstack-infra | 12:57 | |
*** eharney has joined #openstack-infra | 12:58 | |
*** adalbas has joined #openstack-infra | 13:00 | |
*** Ryan_Lane has quit IRC | 13:00 | |
*** jgriffit1 is now known as jgriffith | 13:01 | |
kiall | jeblair / chmouel .. I've not had time yet .. Other things keep popping up making me work weekends ;) | 13:03 |
*** signed8bit has joined #openstack-infra | 13:05 | |
*** mwagner_lap has quit IRC | 13:06 | |
*** zehicle_at_dell has quit IRC | 13:07 | |
*** mriedem has joined #openstack-infra | 13:08 | |
*** dkranz has quit IRC | 13:10 | |
*** weshay has joined #openstack-infra | 13:11 | |
*** dkliban has joined #openstack-infra | 13:12 | |
*** sc68cal has quit IRC | 13:14 | |
*** nosnos has quit IRC | 13:14 | |
*** skraynev is now known as skraynev_afk | 13:16 | |
*** ociuhandu has quit IRC | 13:18 | |
*** malini_afk is now known as malini | 13:19 | |
*** homeless has joined #openstack-infra | 13:19 | |
*** sc68cal has joined #openstack-infra | 13:21 | |
*** dkranz has joined #openstack-infra | 13:22 | |
*** ameade has joined #openstack-infra | 13:24 | |
*** jeckersb is now known as jeckersb_gone | 13:31 | |
*** dcramer_ has joined #openstack-infra | 13:32 | |
*** zz_gondoi is now known as gondoi | 13:34 | |
*** e0ne_ has joined #openstack-infra | 13:36 | |
*** e0ne has quit IRC | 13:39 | |
*** freyes has joined #openstack-infra | 13:40 | |
*** thuc_ has quit IRC | 13:40 | |
*** thuc has joined #openstack-infra | 13:40 | |
*** gokrokve has joined #openstack-infra | 13:41 | |
*** julim is now known as julim_WFH | 13:43 | |
*** gokrokve has quit IRC | 13:45 | |
*** thuc has quit IRC | 13:45 | |
*** chandan_kumar_ has quit IRC | 13:46 | |
*** jeckersb_gone is now known as jeckersb | 13:46 | |
*** changbl has quit IRC | 13:47 | |
*** prad has joined #openstack-infra | 13:49 | |
*** andreaf has joined #openstack-infra | 13:50 | |
*** aconrad has joined #openstack-infra | 13:51 | |
*** zns has joined #openstack-infra | 13:53 | |
*** yamahata has quit IRC | 13:53 | |
*** chandan_kumar_ has joined #openstack-infra | 13:54 | |
*** maurosr has quit IRC | 13:54 | |
*** maurosr has joined #openstack-infra | 13:55 | |
*** ociuhandu has joined #openstack-infra | 13:56 | |
*** Ryan_Lane has joined #openstack-infra | 13:57 | |
openstackgerrit | Sergey Skripnick proposed a change to openstack-infra/config: Add rally-cli job https://review.openstack.org/86331 | 13:57 |
mriedem | logstash still down? | 13:58 |
*** jcoufal has quit IRC | 13:59 | |
*** jcoufal has joined #openstack-infra | 14:00 | |
*** yfried has quit IRC | 14:01 | |
*** yfried has joined #openstack-infra | 14:01 | |
*** Ryan_Lane has quit IRC | 14:01 | |
*** gokrokve has joined #openstack-infra | 14:06 | |
*** locke105 has joined #openstack-infra | 14:08 | |
*** mrodden has quit IRC | 14:12 | |
*** gilliard has quit IRC | 14:15 | |
*** msabramo has quit IRC | 14:17 | |
fungi | mriedem: the elasticsearch cluster is still replicating since the restart... 4 shards left but they're all from the last few days so it's impacting indexing performance until it completes | 14:20 |
*** thuc has joined #openstack-infra | 14:20 | |
*** thedodd has joined #openstack-infra | 14:22 | |
*** mrodden has joined #openstack-infra | 14:22 | |
mriedem | fungi: about how long do you expect that to take? | 14:23 |
*** thuc has quit IRC | 14:23 | |
*** thuc has joined #openstack-infra | 14:23 | |
mriedem | and i'm getting 'oops something went wrong' in kibana - i'm assuming a result of that? | 14:23 |
*** saju_m has joined #openstack-infra | 14:26 | |
*** gokrokve has quit IRC | 14:26 | |
*** mwagner_lap has joined #openstack-infra | 14:27 | |
fungi | mriedem: perhaps, but not sure. clarkb was going to dig deeper once we're a little further along with heartbleed catch-up | 14:28 |
openstackgerrit | Sergey Lukjanov proposed a change to openstack-infra/config: Extract experimental-tripleo jobs to template https://review.openstack.org/84026 | 14:29 |
*** mdenny has joined #openstack-infra | 14:30 | |
*** gokrokve has joined #openstack-infra | 14:32 | |
*** dkranz has quit IRC | 14:36 | |
*** katyafervent is now known as katyafervent_awa | 14:36 | |
*** yamahata has joined #openstack-infra | 14:37 | |
anteaya | stabbings in penn, stabbings in TO, weird | 14:38 |
openstackgerrit | A change was merged to openstack-infra/devstack-gate: allow for generic branch overrides https://review.openstack.org/84829 | 14:39 |
*** dripton_ is now known as dripton | 14:39 | |
*** katyafervent_awa is now known as katyafervent | 14:40 | |
*** malini has left #openstack-infra | 14:40 | |
jeblair | clarkb: i'm in a room with holger, btw | 14:41 |
*** wenlock has joined #openstack-infra | 14:42 | |
*** thuc has quit IRC | 14:42 | |
*** thuc has joined #openstack-infra | 14:43 | |
*** msabramo has joined #openstack-infra | 14:45 | |
*** jgrimm has quit IRC | 14:46 | |
*** thuc has quit IRC | 14:47 | |
*** dkranz has joined #openstack-infra | 14:51 | |
*** cody-somerville has joined #openstack-infra | 14:53 | |
*** cody-somerville has quit IRC | 14:53 | |
*** cody-somerville has joined #openstack-infra | 14:53 | |
*** thuc has joined #openstack-infra | 14:54 | |
*** david-lyle has joined #openstack-infra | 14:55 | |
*** saju_m has quit IRC | 14:58 | |
*** resker has joined #openstack-infra | 14:58 | |
*** mrmartin has quit IRC | 14:59 | |
*** dcramer_ has quit IRC | 15:01 | |
*** esker has quit IRC | 15:02 | |
*** lcostantino has joined #openstack-infra | 15:03 | |
*** mfer has joined #openstack-infra | 15:03 | |
lcostantino | sdague, regarding https://review.openstack.org/#/c/84429/ tempest tests. It's feasible to do multi-node tests right now? | 15:04 |
*** gilliard has joined #openstack-infra | 15:06 | |
*** thuc_ has joined #openstack-infra | 15:09 | |
*** dizquierdo has quit IRC | 15:10 | |
*** thuc has quit IRC | 15:12 | |
*** ilives has quit IRC | 15:13 | |
*** thuc_ has quit IRC | 15:14 | |
*** dcramer_ has joined #openstack-infra | 15:15 | |
*** atiwari has joined #openstack-infra | 15:18 | |
*** dstanek is now known as dstanekafk | 15:19 | |
*** dstanekafk is now known as dstanek | 15:20 | |
*** alexpilotti has quit IRC | 15:20 | |
*** alexpilotti has joined #openstack-infra | 15:20 | |
*** jgrimm has joined #openstack-infra | 15:24 | |
clarkb | good morning infra | 15:24 |
clarkb | jeblair: oh fun maybe you can talk about that bug | 15:24 |
*** skolekonov has quit IRC | 15:24 | |
*** flaper87 is now known as flaper87|afk | 15:25 | |
rcarrillocruz | morning guys, i have a question. I'm wrapping up a bugfix for gerrit-lp integration, and that implies changes to two different projects/git repos. What's the convenntion in terms of commits? I assume i must put the same 'Closes-Bug: foo' in both, but should I also reference in the commit message that there's another patchset in other project addressing this bug? | 15:27 |
*** dolphm has quit IRC | 15:29 | |
phschwartz | clarkb: morning | 15:29 |
fungi | rcarrillocruz: a couple of options depending on the situation | 15:29 |
*** dolphm has joined #openstack-infra | 15:30 | |
*** julienvey has quit IRC | 15:30 | |
fungi | rcarrillocruz: if the two repositories correspond to separate projects in launchpad then you can just add bugtasks on that bug for each project and use closes-bug in both commits | 15:30 |
rcarrillocruz | the two repos are jeepyb and openstack-infra/config | 15:31 |
fungi | rcarrillocruz: if the two repositories share a single project in launchpad, then whichever change needs to merge first should have a partial-bug header and the change merging second can use closes-bub | 15:31 |
fungi | bug | 15:31 |
rcarrillocruz | i guess my case fall in the first case you described | 15:31 |
fungi | and yes, config+jeepyb would fall into the latter category (both use openstack-ci as the project on launchpad currently) | 15:31 |
rcarrillocruz | lol | 15:31 |
* rcarrillocruz recalls last comment | 15:31 | |
*** dcramer_ has quit IRC | 15:32 | |
rcarrillocruz | thx fungi | 15:32 |
fungi | yw | 15:32 |
*** wchrisj has left #openstack-infra | 15:32 | |
openstackgerrit | Marc Abramowitz proposed a change to openstack-infra/jenkins-job-builder: `test` command read/write stdin/stdout by default https://review.openstack.org/84778 | 15:32 |
openstackgerrit | Marc Abramowitz proposed a change to openstack-infra/jenkins-job-builder: add ability to parse from a file object https://review.openstack.org/85177 | 15:32 |
*** resker has quit IRC | 15:32 | |
*** esker has joined #openstack-infra | 15:33 | |
notmyname | we've got a proposed patch to python-swiftclient that adds a dependency to requirements.txt (it's already in global requirements). however, jenkins never passes because grenade fails with an ImportError error | 15:36 |
notmyname | any ideas on why that happens or how to fix it? | 15:36 |
clarkb | notmyname: have you asked #openstack-qa? | 15:37 |
clarkb | and do you have a link to the change? | 15:37 |
notmyname | no, I haven't asked in -qa | 15:38 |
notmyname | https://review.openstack.org/#/c/85453/ | 15:38 |
sdague | lcostantino: it is not | 15:39 |
lcostantino | sdague, ok thanks. will add that as a comment regarding the test. so when it's available i can incorporate them | 15:40 |
jeblair | http://packaging.python.org/en/latest/ <-- where to learn about what's going on in python packaging | 15:40 |
*** mfink has joined #openstack-infra | 15:41 | |
*** thuc has joined #openstack-infra | 15:41 | |
*** thuc has quit IRC | 15:42 | |
*** thuc has joined #openstack-infra | 15:43 | |
*** msabramo has quit IRC | 15:43 | |
*** dkehn_ is now known as dkehnx | 15:43 | |
*** thuc has quit IRC | 15:43 | |
clarkb | sdague: does grenade sync in requirements from older branches? | 15:43 |
*** thuc has joined #openstack-infra | 15:44 | |
clarkb | sdague: if so what does that mean for clients that want to add dependencies? | 15:44 |
clarkb | notmyname: if you look at the grenade log it doesn't appear that the futures lib is synced into the reqs | 15:44 |
*** thuc has quit IRC | 15:44 | |
zaro | morning | 15:44 |
notmyname | clarkb: ya, sdague is helping in -qa :-) | 15:45 |
*** zhiyan has quit IRC | 15:45 | |
zaro | a jjb test job just said it could not connect to git.o.o. anybody know anything about this? | 15:45 |
*** dcramer_ has joined #openstack-infra | 15:45 | |
*** thuc has joined #openstack-infra | 15:45 | |
zaro | http://logs.openstack.org/51/82551/7/gate/gate-jenkins-job-builder-docs/e350f6b/console.html | 15:46 |
clarkb | zaro: temporary failure in name resolution. DNS continues to plague us | 15:47 |
*** branen has joined #openstack-infra | 15:47 | |
*** rloo has joined #openstack-infra | 15:49 | |
*** markmcclain has joined #openstack-infra | 15:49 | |
*** mdenny has quit IRC | 15:49 | |
*** mdenny has joined #openstack-infra | 15:49 | |
*** sabari has joined #openstack-infra | 15:51 | |
*** ihrachyshka has quit IRC | 15:51 | |
*** msabramo has joined #openstack-infra | 15:51 | |
*** dkliban has quit IRC | 15:51 | |
lifeless | coudl someone promote 86367,1 in the trpleo check queue please? we believe it will fix a systematic failure of all jobs | 15:54 |
*** dcramer_ has quit IRC | 15:56 | |
sdague | clarkb: right now it means they need to backport those requirements to stable/* g-r | 15:56 |
sdague | which in one way seems weird, but in another way, we don't cut stable/* for clients, and they have to be able to coexist with stable/* servers on the same env | 15:57 |
*** dkranz has quit IRC | 15:57 | |
fungi | lifeless: on it | 15:59 |
*** Sukhdev has joined #openstack-infra | 15:59 | |
fungi | lifeless: done | 15:59 |
lifeless | fungi: thanks | 15:59 |
openstackgerrit | Matthew Treinish proposed a change to openstack-infra/config: Only run jobs with neutron on neutron changes https://review.openstack.org/84840 | 16:00 |
openstackgerrit | Matthew Treinish proposed a change to openstack-infra/config: Change names for duplicate neutron jobs https://review.openstack.org/83172 | 16:00 |
*** dcramer_ has joined #openstack-infra | 16:03 | |
*** zhiyan has joined #openstack-infra | 16:03 | |
*** changbl has joined #openstack-infra | 16:05 | |
sdague | fungi / clarkb: any idea what's up with ES recovery? | 16:06 |
clarkb | sdague: yeah, 3 shards left to recover | 16:06 |
sdague | we've been at a really light patch load, so I'm concerned that it's taking this long | 16:06 |
clarkb | one is almost done which will get us to 2 | 16:06 |
clarkb | sdague: I think there were disk pressure issues | 16:06 |
sdague | clarkb: ok, what happened to set it off? | 16:06 |
*** freyes has quit IRC | 16:07 | |
clarkb | 05 ended up recovering a ton of shards compared to the others ending up with almost no disk | 16:07 |
clarkb | that means indexing has been slow | 16:07 |
clarkb | but recovery and indexing compete with each other | 16:07 |
*** asselin has joined #openstack-infra | 16:07 | |
clarkb | next time we do this I think Iwill completely disable indexing until the cluster is green | 16:07 |
sdague | ok | 16:07 |
sdague | is it something we can get ahead of? I wasn't really paying attention to the even that pushed things over the edge | 16:08 |
openstackgerrit | A change was merged to openstack-infra/jenkins-job-builder: Add support for Copy Artifact project copy permissions https://review.openstack.org/82551 | 16:08 |
*** dcramer_ has quit IRC | 16:08 | |
clarkb | sdague: I think there is a good chance. We seem to be keeping up now | 16:08 |
*** chandan_kumar_ has quit IRC | 16:08 | |
clarkb | and it is almost done recovering | 16:08 |
*** jpich has quit IRC | 16:08 | |
clarkb | which should allow it to go back to normal relatively quickly | 16:08 |
clarkb | we will have much better numbers once it recovers | 16:09 |
sdague | clarkb: ok, I was more thinking about in future. We seem to lose ES for a couple of days pretty regularly | 16:09 |
clarkb | sdague: this was induced | 16:09 |
sdague | oh | 16:09 |
*** hashar has quit IRC | 16:09 | |
sdague | for cert reissue? | 16:09 |
clarkb | I did for X in elasticsearchnodes sudo reboot | 16:09 |
clarkb | to patch openssl yes | 16:09 |
sdague | ok | 16:09 |
sdague | gotcha | 16:09 |
sdague | so I'll consider this one an extradinary event | 16:10 |
clarkb | monday evening we made the decision to reboot the world rathe rthan try to keep track of patch levels | 16:10 |
clarkb | it has caused some bumps but means we are secure | 16:10 |
fungi | security is an illusion ;) | 16:10 |
clarkb | fungi: after monday I completely agree | 16:11 |
*** dkranz has joined #openstack-infra | 16:11 | |
clarkb | you mean to tell me the library encrypting half the worlds encrypted traffic will let you read memory from either side? | 16:11 |
clarkb | also, the proper news has picked up on it and the amount of wrongness they are reporting makes me sad | 16:11 |
clarkb | PFS isn't a magic bullet people | 16:11 |
clarkb | I am glad I got an early start this morning | 16:12 |
sdague | clarkb: especially when it's a mem leak | 16:12 |
clarkb | still haven't been able to dive into #heartbleed today | 16:12 |
clarkb | sdague: right, PFS fixes one particular aspect of the issue. But not the really scary bits | 16:13 |
sdague | yeh | 16:13 |
clarkb | but the people on my TV are saying facebook et al are safe because PFS | 16:13 |
dstufft | Depends on what the scary bits are :] | 16:13 |
sdague | last night at the makerspace one of the guys demonstrated a little bit of python that connected to sites and scanned for session cookies in known formats | 16:13 |
dstufft | If you've been bulk collectiong SSL data waiting for a chance to get the private key, well not having PFS is kind of scary | 16:14 |
clarkb | dstufft: I am less worried about people reading traffic in the future via decrypting captured streams and more worried about leaked credentials and private certs making mitm and direct exploit possible | 16:14 |
sdague | he pointed it at openssl.org and in the first few packets got their ssl priv key | 16:14 |
fungi | pfs really only helps you if the nsa didn't observe you while you did session set-up (or hasn't backdoored your system to leak keys to them via one of a number of basically undetectable side channels) | 16:14 |
clarkb | dstufft: granted I am not a super secret agent person and am more concerned for my parents banking info | 16:14 |
openstackgerrit | Ricardo Carrillo Cruz proposed a change to openstack-infra/jeepyb: Add change abandoned logic to jeepyb https://review.openstack.org/86382 | 16:15 |
*** dkliban has joined #openstack-infra | 16:15 | |
dstufft | fungi: hrm? "didn't observe you" ? | 16:15 |
clarkb | dstufft: I think PFS is a good thing, but really don't think faces on TV should tell the world they are safe beacuse PFS | 16:15 |
clarkb | which is what they were doing last night | 16:16 |
dstufft | yea well that's stupid | 16:16 |
sdague | the whole read out the whole webserver memory over the network is just epicly bad | 16:16 |
clarkb | sdague: indeed | 16:16 |
*** denis_makogon has joined #openstack-infra | 16:17 | |
*** gyee has joined #openstack-infra | 16:17 | |
dstufft | let's run the entire world on some netcode written by like 2-4 guys which is a giant unmaintinable mess, what could possibly go wrong | 16:17 |
dstufft | watching people discover that revocation is a pipedream is fun tho! | 16:18 |
clarkb | :P | 16:18 |
clarkb | also, it is funny that I told people I know about this monday and it took a full 24 hours for news cycle to pick it up | 16:18 |
clarkb | I think my family thought I was crazy in that 24 hours | 16:18 |
dstufft | heh | 16:19 |
clarkb | "You want me to do what? I hvae to change all of my passwords? more than once? and shouldn't use online banking? how am I supposed to pay bills?" | 16:19 |
fungi | dstufft: in theory pfs helps you against an adversary who started recording your traffic some time after your initial peer relationships were established, and also increases the effort of playing the entire session forward for someone who got the entire thing, but in the latter case it is basically possible to work out the diffie-hellman exchanges in series as i understand it (you just need to know | 16:19 |
dstufft | my family just assumes I'm right :V | 16:19 |
fungi | all of the ones which came before) | 16:19 |
sdague | dstufft: yeh, the fact that all the browsers gave up revocation for speed.... so the mitm won't actually be solved until the old certs expire | 16:20 |
fungi | so yes, pfs is useful, but it's no magic bullet | 16:20 |
*** aconrad has quit IRC | 16:20 | |
clarkb | we are down to one shard recovering, it is a shard in an active index | 16:21 |
*** dcramer_ has joined #openstack-infra | 16:22 | |
dstufft | fungi: hm, i'm pretty sure that's not true, but i'd have to go look at the relevant specs again. the various DH exchange in PFS should mean that even if you have the private key that you cant derrive the session key, course then you have things like session tickets thrown in the muck which are stored in memory most the time :( | 16:22 |
dstufft | sdague: It wasn't just speed, even when they bothered to check the revocations it was always a soft fail | 16:23 |
dstufft | because SPOF | 16:23 |
fungi | dstufft: right, but it's not entropic, it's deterministic, just needs a ton of brute-forcing | 16:23 |
fungi | dstufft: it saves you from the adversary who simply relies on getting a copy of your keys, but doesn't completely eliminate the threat of an adversary engaging in offline brute-forcing of your traffic | 16:24 |
*** melwitt has joined #openstack-infra | 16:24 | |
fungi | but also, it's been theorized that some black-box ipsec vendors, for example, implemented code at the request of the nsa so as to leak rekey updates in the stream so they can be figured out without needing to brute force much or possibly at all | 16:25 |
*** markwash has joined #openstack-infra | 16:25 | |
dstufft | well for DHE you're brute forcing the discrete log problem, so unless the DHE keys are tiny a brute force should be infeasable | 16:25 |
mordred | sdague: re: patch above on requirements ... shouldn't that project just not list themselves in the requirements repo? | 16:25 |
*** andreaf has quit IRC | 16:26 | |
mordred | sdague: I mean, the list in taht repo controls whether we enforce reqs on them by way of the mirror | 16:26 |
sdague | mordred: possibly | 16:26 |
*** thomasem has quit IRC | 16:26 | |
mordred | sdague: it seems that we should have one and only one place to keep the enforced/not-enforced list | 16:26 |
*** thomasem has joined #openstack-infra | 16:26 | |
mordred | that said - it's possible we still need the flag you have there | 16:26 |
fungi | dstufft: agreed. today's infeasable doesn't protect you against tomorrow's new mathematical breakthrough when all your encrypted communication was logged, which is what some of the pfs hype seems to imply | 16:26 |
sdague | I guess the question was how to track something like barbican that is starting with more and moving towards compliance | 16:26 |
mordred | and a way for $something to read the reqs list and set the flag | 16:26 |
sdague | and to not self wedge themselves on devstack jobs | 16:27 |
sdague | because if you don't req sync your git trees, good luck | 16:27 |
sdague | I agree this one might have other options | 16:27 |
sdague | maybe we should poke the barbican folks on it | 16:28 |
openstackgerrit | Ruby Loo proposed a change to openstack-infra/config: Add documentation jobs for python-ironicclient https://review.openstack.org/86390 | 16:32 |
*** markmc has quit IRC | 16:32 | |
*** sabari has quit IRC | 16:32 | |
mordred | sdague: I'm in support of your goals - I just want to make sure we don't make two different lists that have to be maintained | 16:32 |
*** gmoro has quit IRC | 16:33 | |
clarkb | sdague: ok we are green now. some shards are relocating to free up more space on 05, but we should see the backlog drop. I am using the zuul status graph shifted to the last two hours to watch it | 16:33 |
lifeless | textcons | 16:33 |
lifeless | bah | 16:33 |
*** yamahata has quit IRC | 16:36 | |
clarkb | sdague: I think we may have found the scaling limits of geard :( | 16:40 |
sdague | clarkb: in queue depth? | 16:40 |
clarkb | sdague: yeah, looking at the workers they are sitting in the GRAB_WAIT state for longer than I expect | 16:41 |
clarkb | which I think means that the geard side isn't giving them work to do quickly enough | 16:41 |
clarkb | jeblair: ^ is that something you can peak at? | 16:41 |
*** ildikov has quit IRC | 16:42 | |
*** dprince has quit IRC | 16:42 | |
*** ildikov_ has quit IRC | 16:42 | |
ruhe | * | 16:44 |
ruhe | sorry | 16:44 |
*** e0ne_ has quit IRC | 16:45 | |
*** gmoro has joined #openstack-infra | 16:45 | |
*** e0ne has joined #openstack-infra | 16:45 | |
*** MIDENN_ has joined #openstack-infra | 16:47 | |
*** mdenny has quit IRC | 16:47 | |
*** yamahata has joined #openstack-infra | 16:48 | |
*** jhesketh_ has quit IRC | 16:49 | |
*** harlowja_away is now known as harlowja | 16:49 | |
*** e0ne has quit IRC | 16:50 | |
*** afazekas has quit IRC | 16:51 | |
*** freyes has joined #openstack-infra | 16:57 | |
*** CaptTofu has joined #openstack-infra | 16:58 | |
*** andreykurilin_ has joined #openstack-infra | 16:58 | |
mordred | clarkb: uhoh | 16:59 |
mordred | clarkb: does that mean it's time for us to start working on patches to c-gearmand? | 16:59 |
*** gokrokve has quit IRC | 17:01 | |
clarkb | mordred: maybe, I think logstash gearman can get away with c gearmand without trouble | 17:01 |
clarkb | zuul has more specific use cases | 17:01 |
*** dprince has joined #openstack-infra | 17:03 | |
*** amcrn has joined #openstack-infra | 17:03 | |
mordred | clarkb: oh - this is logstash gearman scaling? | 17:04 |
*** SumitNaiksatam has quit IRC | 17:06 | |
clarkb | ya | 17:06 |
anteaya | clarkb: reading backscroll - I got my mom to check her version of openssl using the terminal | 17:06 |
anteaya | the pain but she listened and she did it | 17:07 |
*** nati_ueno has joined #openstack-infra | 17:09 | |
*** derekh has quit IRC | 17:10 | |
hogepodge | This has been in the queue for a while. Not really critical, but any thoughts on it? https://review.openstack.org/#/c/82859/ | 17:11 |
openstackgerrit | Marc Abramowitz proposed a change to openstack-infra/jenkins-job-builder: Add `browse` command https://review.openstack.org/85580 | 17:12 |
*** vishy has quit IRC | 17:12 | |
*** vishy has joined #openstack-infra | 17:13 | |
*** mfink has quit IRC | 17:15 | |
*** jooools has quit IRC | 17:17 | |
*** nati_uen_ has joined #openstack-infra | 17:17 | |
jeblair | clarkb: you figure the geard queue size is impacting the perf? like grabbing a job is O(n)? | 17:17 |
*** aconrad has joined #openstack-infra | 17:18 | |
clarkb | jeblair: ya | 17:18 |
jeblair | clarkb: it's quite possible (it's not designed for performance and is not at all tuned; to be fair, the docs say not to use it, but of course there's no reason we can't use it and make it better) | 17:18 |
mordred | jeblair: :) | 17:19 |
clarkb | ya :) | 17:19 |
clarkb | however I am happy to switch to the C server for this since it should just work | 17:19 |
clarkb | I think one line of code needs changing in the workers | 17:19 |
*** krotscheck has quit IRC | 17:19 | |
jeblair | clarkb: yep, grabbing a job is O(n) | 17:20 |
*** nati_ueno has quit IRC | 17:20 | |
jeblair | hrm, though that should be worst case | 17:20 |
jeblair | clarkb: for logstash i think it should be O(1) | 17:21 |
*** SumitNaiksatam has joined #openstack-infra | 17:21 | |
clarkb | ok, it could also be swap causing problems. We are apparently dipping into that | 17:22 |
*** jlibosva has quit IRC | 17:22 | |
*** krotscheck has joined #openstack-infra | 17:22 | |
*** ominakov has quit IRC | 17:23 | |
jeblair | clarkb: yeah, geard itself should be O(1) in this case, but it's possible that queue.remove or other queue methods could be slow; i haven't examined their performance | 17:23 |
*** chandan_kumar_ has joined #openstack-infra | 17:24 | |
*** zns has quit IRC | 17:25 | |
*** eyerediskin has left #openstack-infra | 17:25 | |
openstackgerrit | Marc Abramowitz proposed a change to openstack-infra/jenkins-job-builder: Add `browse` command https://review.openstack.org/85580 | 17:26 |
*** jgallard has quit IRC | 17:26 | |
*** markmcclain has quit IRC | 17:29 | |
*** dstanek has quit IRC | 17:31 | |
*** dstanek has joined #openstack-infra | 17:32 | |
rloo | hi, we want to generate docs for python-ironicclient. I thought I'd add Jenkins job for it: https://review.openstack.org/86390 | 17:32 |
rloo | but how do I test it? | 17:33 |
mordred | rloo: you probably also want to add entries in teh zuul layout.yaml | 17:33 |
clarkb | hogepodge: we are still fighting the heartbleed fires. I wouldn't expect the review queues for infra to move much this week | 17:33 |
*** chandan_kumar_ has quit IRC | 17:34 | |
mordred | rloo: tox -evenv python setup.py build_sphinx | 17:34 |
rloo | mordred: yes, I thought I would test the jenkins job first to make sure it works. but maybe I just add to zuul too, and hope it all works after it is merged? | 17:34 |
mordred | rloo: is what you should run lcoally | 17:34 |
mordred | if that works and puts docs in doc/build | 17:34 |
*** wchrisj has joined #openstack-infra | 17:34 | |
mordred | then the jenkins jobs should work | 17:34 |
rloo | mordred: yes, that works in python-ironicclient. | 17:35 |
mordred | great! then the job templates should Just Work™ | 17:35 |
rloo | mordred: so I will add zuul changes too. But will the changes need to be merged, before it can all be verified to work? | 17:35 |
mordred | yes | 17:35 |
rloo | mordred. ha ha, ok, I'll hope that it just works then ;) Thanks! | 17:36 |
*** thedodd has quit IRC | 17:36 | |
*** dcramer_ has quit IRC | 17:36 | |
*** Ryan_Lane has joined #openstack-infra | 17:36 | |
hogepodge | clarkb of course. my bad | 17:37 |
*** gokrokve has joined #openstack-infra | 17:37 | |
*** zhiyan is now known as zhiyan_ | 17:37 | |
sdague | clarkb: is there anything that folks from the outside of infra-root can help with on heartbleed mitigation? | 17:38 |
clarkb | sdague: not really. We are completely patched now (and were Monday evening PDT) and are now going through the laborous process of swapping out crednetials and secrets | 17:38 |
clarkb | turns out you can't just do these things in random order which has made it fun. Also, lots of things depend on other people patching and redoing certs and things from their side first (eg lp, github,pypi) | 17:39 |
*** Ryan_Lane1 has joined #openstack-infra | 17:40 | |
*** Ryan_Lane has quit IRC | 17:41 | |
sdague | nice | 17:42 |
fungi | not to mention it's a list of ~200 credentials we're replacing/reissuing/whatever | 17:43 |
fungi | for all manner of arbitrary stuff | 17:43 |
*** skraynev_afk is now known as skraynev | 17:44 | |
sdague | mordred: so did you have an opinion on the pbr requirements change - https://review.openstack.org/#/c/86189/ ? (the -1s are because of keystoneclient race with heat, unrelated) | 17:44 |
anteaya | I've mostly seen my role as staying out of the way | 17:45 |
*** wchrisj has quit IRC | 17:45 | |
clarkb | and we are doing our best to do it in a non impactful way | 17:46 |
*** chandan_kumar_ has joined #openstack-infra | 17:47 | |
*** mwagner_lap has quit IRC | 17:48 | |
openstackgerrit | Ricardo Carrillo Cruz proposed a change to openstack-infra/jeepyb: Add change abandoned logic to jeepyb https://review.openstack.org/86382 | 17:48 |
*** dcramer_ has joined #openstack-infra | 17:50 | |
*** thomasem has quit IRC | 17:51 | |
*** morganfainberg_Z is now known as morganfainberg | 17:51 | |
*** _nadya_ has joined #openstack-infra | 17:51 | |
*** thomasem has joined #openstack-infra | 17:55 | |
*** Ryan_Lane1 has quit IRC | 17:56 | |
openstackgerrit | Marc Abramowitz proposed a change to openstack-infra/jenkins-job-builder: PEP8ify https://review.openstack.org/86407 | 17:56 |
mordred | sdague: looking | 17:57 |
openstackgerrit | Ruby Loo proposed a change to openstack-infra/config: Add documentation jobs for python-ironicclient https://review.openstack.org/86390 | 17:58 |
*** Ryan_Lane has joined #openstack-infra | 17:58 | |
*** Ryan_Lane has joined #openstack-infra | 17:58 | |
*** Sukhdev has quit IRC | 17:59 | |
mordred | sdague: works for me | 17:59 |
therve | clarkb, Do you have 5 minutes to talk about https://review.openstack.org/#/c/83761/ ? | 17:59 |
clarkb | therve: sure | 18:00 |
therve | clarkb, I put a comment after your review. I think devstack is the most straight forward way (and what I had been suggested), but we can try other ways | 18:00 |
clarkb | therve: if it needs all of those things running then devstack is the way to do it | 18:01 |
clarkb | therve: but now my question because why do you need all of that stuff to validate a template? | 18:01 |
clarkb | s/because/becomes/ | 18:01 |
*** _nadya_ has quit IRC | 18:01 | |
therve | clarkb, I'm using the public API to do it. | 18:01 |
clarkb | therve: oh are you actually deploying the template? | 18:01 |
clarkb | not just doing static analysis and compilation checks? | 18:02 |
therve | clarkb, No heat has a validating API | 18:02 |
therve | The heat server is doing that, mostly. | 18:02 |
openstackgerrit | Michael Krotscheck proposed a change to openstack-infra/storyboard: Changed task states https://review.openstack.org/85868 | 18:02 |
mordred | therve: -1 - bug in that template | 18:02 |
*** hashar has joined #openstack-infra | 18:02 | |
therve | mordred, Yes? | 18:03 |
openstackgerrit | Marc Abramowitz proposed a change to openstack-infra/jenkins-job-builder: PEP8ify https://review.openstack.org/86407 | 18:03 |
therve | Good catch thanks | 18:03 |
* mordred has to get his -1 stats up | 18:03 | |
openstackgerrit | Michael Krotscheck proposed a change to openstack-infra/storyboard-webclient: Updated UI for tasks status https://review.openstack.org/85870 | 18:03 |
*** dcramer_ has quit IRC | 18:04 | |
therve | clarkb, The validation could be done using heat-as-a library, but it wouldn't be the same as using the client and public API. | 18:04 |
openstackgerrit | Thomas Herve proposed a change to openstack-infra/config: Add a new job for heat-templates https://review.openstack.org/83761 | 18:05 |
clarkb | therve: right, but you aren't testing heat, you are testing your template | 18:05 |
clarkb | therve: note I am actually fine with the test as is after your explanation | 18:06 |
therve | clarkb, I'm testing both. I want to have the same gate on heat itself later on | 18:06 |
clarkb | therve: it just seems ridiculous that this is the way heat wants users to test things | 18:06 |
clarkb | seems like it should be a function of heatclient not heatserver | 18:06 |
jeblair | therve: maybe the test of the heat api could be in tempest? | 18:06 |
therve | clarkb, How would the cloud users know the cloud deployed codebase though? | 18:07 |
clarkb | therve: they shouldn't need to, the templates should be backward and forward compatible | 18:07 |
clarkb | its basically another api | 18:07 |
therve | OK. I think you said you're fine, so I'll just shut up for now :) | 18:09 |
*** sabari has joined #openstack-infra | 18:10 | |
openstackgerrit | Marc Abramowitz proposed a change to openstack-infra/jenkins-job-builder: Make all help text start with lowercase letter https://review.openstack.org/86410 | 18:11 |
*** dstufft has quit IRC | 18:12 | |
*** dstufft has joined #openstack-infra | 18:13 | |
*** freyes has quit IRC | 18:15 | |
*** glenng has joined #openstack-infra | 18:16 | |
openstackgerrit | Ruby Loo proposed a change to openstack-infra/config: Add documentation jobs for python-ironicclient https://review.openstack.org/86390 | 18:16 |
*** dcramer_ has joined #openstack-infra | 18:18 | |
mgagne | zaro: ping | 18:18 |
mgagne | zaro: what prevents us from migrating JJB to hacking? | 18:19 |
sdague | clarkb/fungi: a client exploit demonstration has been released - https://github.com/Lekensteyn/pacemaker | 18:19 |
glenng | Looking for some infrastructure help: I had my Launchpad username changed and I need to get my Gerrit review username updated to match. Anyone know how? | 18:20 |
*** thedodd has joined #openstack-infra | 18:20 | |
*** zns has joined #openstack-infra | 18:20 | |
fungi | sdague: neat--thanks! | 18:20 |
fungi | glenng: you can adjust your git global config to use the old ssh username when connecting to gerrit with git-review | 18:21 |
fungi | glenng: gerrit doesn't provide a way to change ssh usernames on accounts | 18:21 |
glenng | fungi: Yes, but others will have to use old username when working with me and that is what I wish to avoid. | 18:21 |
*** hashar has quit IRC | 18:24 | |
*** ominakov has joined #openstack-infra | 18:24 | |
fungi | glenng: why would the ssh username on your gerrit account be something anyone besides you would use? | 18:25 |
glenng | fungi: It isn't the ssh username, it is the profile username. | 18:26 |
openstackgerrit | A change was merged to openstack-infra/storyboard-webclient: Autoresize-width directive https://review.openstack.org/83945 | 18:26 |
openstackgerrit | Marc Abramowitz proposed a change to openstack-infra/jenkins-job-builder: Add `browse` command https://review.openstack.org/85580 | 18:26 |
fungi | glenng: gerrit uses numeric user ids for profile urls | 18:27 |
glenng | fungi: When someone wants to add me to a review request for example, then see the old username. | 18:27 |
glenng | fungi: Understood, but somewhere there is a database with numeric ID to username. | 18:27 |
fungi | glenng: there is, yes. gerrit chose not to support changes to ssh usernames around the 2.3 release because it could lead to conflicts with some personal repository features they implemented in that timeframe | 18:28 |
*** tomhe has joined #openstack-infra | 18:30 | |
*** wchrisj_ has joined #openstack-infra | 18:30 | |
glenng | fungi: So it cannot be changed… perhaps my only options is to create a new account. | 18:30 |
msabramo | mgagne: What is "hacking"? Why would you migrate JJB to it? | 18:30 |
clarkb | glenng: people can add you by name and email address | 18:31 |
mgagne | msabramo: it is the tool OpenStack projects use and created to run pep8 tests. | 18:31 |
clarkb | glenng: it is rare to add people to reviews by ssh username because most people don't tell others what their ssh username is | 18:31 |
*** morganfainberg has quit IRC | 18:31 | |
msabramo | mgagne: Does it do stuff on top of pep8? Like set up different defaults or something? | 18:32 |
mgagne | msabramo: http://docs.openstack.org/developer/hacking/ | 18:32 |
mgagne | msabramo: it enforces the Openstack style guides | 18:32 |
fungi | glenng: and also, while gerrit does allow accounts to be added to reviews and groups by their ssh username, it doesn't provide anywhere in the webui to actually look up the ssh username of an account unless you are the account holder (in which case it's in your preferences page) | 18:33 |
msabramo | mgagne: Thanks! | 18:34 |
mgagne | msabramo: the end goal is to make it uniform across all projects and so far, hacking has been the tool used to run pep8. JJB hasn't been converted to hacking due to lack of time. My wish is to not diverge from the established standard to ease maintainability. | 18:34 |
tomhe | jeblair: clarkb: Would the submit type "rebase if necessary" in Gerrit be a bad idea to use in a Zuul/Gerrit environment? My users are asking for it and I've been hesitant to switch from "merge if necessary" as Zuul/Jenkins would be verifying other commits than will be actually submitted. | 18:35 |
clarkb | tomhe: we found rebasing worflows to be bad | 18:35 |
mgagne | msabramo: My suggestion would be to fix errors reported by hacking and then migrate to it. Not sure how we can make a smooth transition without breaking the gate. | 18:35 |
clarkb | tomhe: actually wait, we used cherry pick | 18:35 |
clarkb | and cherry pick was bad because it edited every commit even if it was the equivalent to a fast forward | 18:36 |
clarkb | rebase if necessary may not do that | 18:36 |
fungi | tomhe: clarkb: yes, cherry-pick caused changes to be modified, which altered the committer and the gitsha, and broke git branch -d behavior | 18:36 |
mgagne | msabramo: if you can help with it, that would be great. Otherwise I make it happen this week with the help of zaro. | 18:36 |
fungi | tomhe: clarkb: i expect rebase if necessary to do something similar when it does decide to rebase | 18:37 |
tomhe | fungi: clarkb: Rebase would effectively rewrite the commits | 18:37 |
fungi | tomhe: clarkb: since that would still change the gitsha of the commit and probably the committer | 18:37 |
*** ociuhandu has quit IRC | 18:37 | |
fungi | merge if necessary has the benefit that it does not alter the original commit, merely adds a merge commit with the original commit and the branch tip as parents | 18:38 |
tomhe | fungi: clarkb: How do you deal with the complex log caused by all merges? I get complaints on the log being hard to grasp. | 18:38 |
clarkb | tomhe: it isn't that bad | 18:38 |
clarkb | tomhe: especially when you consider that you are just shifting complexity | 18:39 |
jeblair | tomhe: zuul does its own merging for tests, and i don't think rebase-if-necessary has been added to zuul, which means that its conflict detection and what's actually tested might be a little different... | 18:39 |
fungi | git log --no-merges or git log --max-parents=1 | 18:39 |
clarkb | with merge commits your local repo can resolve things against upstream | 18:39 |
clarkb | without them your brain must do that for you | 18:39 |
*** _nadya_ has joined #openstack-infra | 18:39 | |
*** thuc has quit IRC | 18:39 | |
jeblair | tomhe: (so if clarkb and fungi don't convince you that merge-if-necessary is better, you might want to add support for that to zuul to bring it in line with gerrit) | 18:39 |
*** thuc has joined #openstack-infra | 18:40 | |
tomhe | jeblair: :) | 18:40 |
jeblair | we have seen gerrit's rebase button do the wrong thing before too; so i think some of us are a little scared of automatic rebasing | 18:41 |
fungi | also, you learn quickly when looking at merge commits in the log that the first parent is the branch tip at the time of the merge and the second parent is the commit being added | 18:41 |
jeblair | (it's rare, but can happen especially in repetitive code) | 18:41 |
tomhe | jeblair: Good to know | 18:42 |
tomhe | jeblair: fungi: clarkb: Thanks all | 18:42 |
jeblair | np | 18:42 |
*** thuc has quit IRC | 18:44 | |
tomhe | jeblair: Oh, one more thing. I have a job filter using files matching which triggered correctly for the check and gate pipelines but nothing got triggered for post. Does the files matching not work in the post pipeline? | 18:48 |
clarkb | tomhe: it doesn't because the event for post does not include a file listing | 18:51 |
openstackgerrit | Marc Abramowitz proposed a change to openstack-infra/jenkins-job-builder: PEP8ify https://review.openstack.org/86407 | 18:51 |
clarkb | tomhe: if you look at the gerrit docs, there are docs on what is included in each event. or you can run the stream events command and see the events directly | 18:52 |
*** adalbas has quit IRC | 18:53 | |
tomhe | clarkb: ok, thanks | 18:54 |
*** CaptTofu has quit IRC | 18:56 | |
*** hashar has joined #openstack-infra | 18:58 | |
hashar | hey | 18:59 |
hashar | not sure who is in charge of upgrading Gerrit to 2.8, it apparently supports the TrivialRebase notifier out of the box now . Can be done in the All-Projects project.config https://gerrit.wikimedia.org/r/Documentation/config-labels.html#label_copyAllScoresOnTrivialRebase :D | 19:00 |
*** glenng has quit IRC | 19:00 | |
clarkb | hashar: zaro is the person to talk to and I think he plans to implement that | 19:00 |
*** morganfainberg_Z has joined #openstack-infra | 19:01 | |
hashar | copy pasting to his email :-] thanks clarkb | 19:01 |
*** morganfainberg_Z is now known as morganfainberg | 19:01 | |
hashar | {{done}} | 19:03 |
fungi | yep, that got brought up as one more thing we can stop maintaining a patched version of | 19:06 |
*** pdmars has quit IRC | 19:06 | |
fungi | and excess tooling around | 19:06 |
*** chandan_kumar_ has quit IRC | 19:07 | |
*** arosen1 has joined #openstack-infra | 19:07 | |
arosen1 | Hi, I had a quick question i figured you guys would know the answer right away. I ran: python /opt/stack/requirements/update.py /opt/stack/novadocker | 19:08 |
arosen1 | Then uploaded the patch that resulted in here: https://review.openstack.org/#/c/86414/ | 19:08 |
arosen1 | but the console log is now failing with: error: invalid command 'testr' | 19:08 |
arosen1 | http://logs.openstack.org/14/86414/1/check/gate-nova-docker-python26/709819b/console.html | 19:08 |
arosen1 | I needed to run update.py and update nova-docker otherwise the devstack patch for it keeps complaining that the requirements are out of sync. | 19:09 |
*** e0ne has joined #openstack-infra | 19:09 | |
openstackgerrit | Andreas Jaeger proposed a change to openstack-infra/config: Reduce number of files send for propose_translation https://review.openstack.org/82573 | 19:09 |
*** armax has joined #openstack-infra | 19:10 | |
*** adalbas has joined #openstack-infra | 19:10 | |
*** _nadya_ has quit IRC | 19:12 | |
*** e0ne has quit IRC | 19:13 | |
openstackgerrit | Marc Abramowitz proposed a change to openstack-infra/jenkins-job-builder: PEP8ify https://review.openstack.org/86407 | 19:15 |
*** ociuhandu has joined #openstack-infra | 19:19 | |
*** CaptTofu has joined #openstack-infra | 19:19 | |
*** e0ne has joined #openstack-infra | 19:19 | |
*** chandan_kumar_ has joined #openstack-infra | 19:20 | |
*** krotscheck has quit IRC | 19:20 | |
openstackgerrit | Thomas Herve proposed a change to openstack-infra/config: Add a new job for heat-templates https://review.openstack.org/83761 | 19:28 |
*** derekh has joined #openstack-infra | 19:29 | |
*** openstackgerrit has quit IRC | 19:34 | |
*** _nadya_ has joined #openstack-infra | 19:37 | |
*** thuc has joined #openstack-infra | 19:40 | |
* ttx has been waiting for the top gate test to complete for the last 40 min | 19:42 | |
*** openstackgerrit has joined #openstack-infra | 19:42 | |
ttx | still running, very slowly | 19:43 |
*** krotscheck has joined #openstack-infra | 19:43 | |
*** ihrachyshka has joined #openstack-infra | 19:44 | |
ttx | ..aaand fail. | 19:45 |
sdague | clarkb: do you think that gearman is going to recover? or should we just drop the log queue and call it a loss | 19:47 |
sdague | I'd rather have working current results than stay blind for the rest of the week | 19:48 |
*** ihrachyshka has quit IRC | 19:48 | |
*** thuc has quit IRC | 19:49 | |
*** chandan_kumar_ has quit IRC | 19:49 | |
derekh | anybody know why we don't seem to be getting any f20 nodes in the tripleo-check queue? nodepool seems to be only spinning up precise nodes again? | 19:51 |
fungi | derekh: looked to me like that cloud was probably strangled again | 19:52 |
fungi | i'll take a peek real fast | 19:52 |
derekh | fungi: thanks, I don't see any nodes in ERROR state, | 19:53 |
fungi | i show 33 in use running tests right now | 19:53 |
*** msabramo has quit IRC | 19:53 | |
*** msabramo1 has joined #openstack-infra | 19:53 | |
msabramo1 | mgagne: I'm playing around with hacking now; there are TODOs with no name; what do I do with those? TODO(unknown)? | 19:54 |
mgagne | msabramo1: that would be my suggestion | 19:54 |
mgagne | msabramo1: which errors are you fixing? | 19:54 |
mgagne | msabramo1: I'm working on some of them atm | 19:54 |
msabramo1 | mostly import sorting | 19:55 |
mgagne | msabramo1: damn, I fixed those already lol | 19:55 |
mgagne | msabramo1: import order, multiple imports | 19:55 |
msabramo1 | there are a ton in jenkins_jobs/modules that are easily fixed by inserting a blank line between std imports and 3rd party | 19:55 |
msabramo1 | oh, haha | 19:55 |
mgagne | msabramo1: yep | 19:55 |
msabramo1 | alright, maybe i'll hold off then | 19:56 |
mgagne | msabramo1: JenkinsJobsException is kind of a pita to fix | 19:56 |
mgagne | msabramo1: well, I should have noticed you ^^' | 19:56 |
fungi | derekh: looks like the demand for precise nodes is high enough that the proportional demand for f20 nodes within the available capacity ends up only being a couple nodes... probably want to increase the min-ready on them above 2 | 19:56 |
msabramo1 | mgagne: in one file, I changed it to errors.JenkinsJobException everywhere and reindented. In another I got lazy and did JenkinsJobException = errors.JenkinsJobException :-) | 19:57 |
mgagne | msabramo1: haha, true, not sure what's the "correct" style, some are using jenkins_jobs.errors.JenkinsJobException | 19:57 |
msabramo1 | mgagne: I have a branch where I added hacking to tox.ini; do you have that yet? I suppose you do | 19:57 |
*** mbacchi has quit IRC | 19:57 | |
mgagne | msabramo1: mostly fixing locally in a branch | 19:57 |
fungi | derekh: i think we got into the same situation after the jenkins master reboots for heartbleed as we were in when the node label was first added (went for a while with the label not registered because the existing demand for precise nodes won out and no f20 nodes got built until the backlog dropped far enough) | 19:58 |
mgagne | msabramo1: and last commit will be switch to hacking | 19:58 |
*** _nadya_ has quit IRC | 19:58 | |
mgagne | msabramo1: want me to ship branch so you can continue working on it? | 19:58 |
fungi | derekh: zuul spent a while not running any f20 jobs, which due to a chicken-and-egg issue with worker registration, caused nodepool to think there was no actual demand for them | 19:59 |
msabramo1 | mgagne: well I have to do some real work and you're on it now so might as well continue as long as it interests you, but you can ship it when you're done working on it | 19:59 |
msabramo1 | mgagne: and then I can pick up, but I will be busy soon getting ready and traveling to PyCOn | 19:59 |
mgagne | msabramo1: oh, nice | 20:00 |
mgagne | msabramo1: right in Montreal =) | 20:00 |
msabramo1 | mgagne: You are in Montreal? | 20:00 |
mgagne | msabramo1: yep | 20:00 |
openstackgerrit | A change was merged to openstack-infra/storyboard-webclient: Updated UI for tasks status https://review.openstack.org/85870 | 20:00 |
msabramo1 | mgagne: Cool, spending a lot of time at PyCon? | 20:00 |
mgagne | msabramo1: yep going, trying to find presentations fitting my interests | 20:01 |
msabramo1 | mgagne: Last two years, PyCon was in my neighborhood and it was easy. Now it's your turn. :-) | 20:01 |
*** atiwari has quit IRC | 20:01 | |
mgagne | msabramo1: haha | 20:01 |
mgagne | msabramo1: my employer is sponsoring the event =) | 20:01 |
*** david_lyle_ has joined #openstack-infra | 20:01 | |
msabramo1 | mgagne: Cool, mine too - SurveyMonkey | 20:01 |
mgagne | msabramo1: oh nice | 20:02 |
*** zns has quit IRC | 20:02 | |
msabramo1 | could add to tox.ini but keep optional: | 20:02 |
msabramo1 | +[testenv:flake8] | 20:02 |
msabramo1 | +deps = flake8 | 20:02 |
msabramo1 | +commands = flake8 | 20:02 |
msabramo1 | + | 20:02 |
msabramo1 | +[testenv:hacking] | 20:02 |
msabramo1 | +deps = hacking | 20:02 |
msabramo1 | +commands = flake8 | 20:02 |
derekh | fungi: ok, so will I increase min-ready f20 nodes to say 5? | 20:02 |
msabramo1 | + | 20:02 |
fungi | derekh: that may help | 20:03 |
msabramo1 | mgagne: ^ | 20:03 |
*** dklyle has joined #openstack-infra | 20:03 | |
mgagne | msabramo1: ok | 20:03 |
msabramo1 | mgagne: Do you work on python-jenkins? | 20:03 |
mgagne | msabramo1: the project itself? | 20:03 |
*** sdake has joined #openstack-infra | 20:04 | |
*** sdake has quit IRC | 20:04 | |
*** sdake has joined #openstack-infra | 20:04 | |
*** david-lyle has quit IRC | 20:04 | |
msabramo1 | mgagne: Sent some PRs for that for Python 3 compat and such - https://code.launchpad.net/~msabramo/python-jenkins/python3/+merge/214664 | 20:04 |
mgagne | msabramo1: no, I'm just an OpenStack dev/operator contributing whenever possible =) | 20:04 |
mgagne | msabramo1: zaro should be able to help you I guess | 20:05 |
msabramo1 | zaro: https://code.launchpad.net/~msabramo/python-jenkins/python3/+merge/214664 :-) | 20:05 |
*** UtahDave has joined #openstack-infra | 20:06 | |
openstackgerrit | Derek Higgins proposed a change to openstack-infra/config: Increase min-ready for f20 nodes to 5 https://review.openstack.org/86432 | 20:06 |
*** david_lyle_ has quit IRC | 20:06 | |
derekh | fungi: would be great if you have a chance ^^, thanks | 20:06 |
*** thuc has joined #openstack-infra | 20:09 | |
*** atiwari has joined #openstack-infra | 20:10 | |
*** thuc has quit IRC | 20:10 | |
*** mwagner_lap has joined #openstack-infra | 20:10 | |
*** thuc has joined #openstack-infra | 20:11 | |
jogo | sdague: so sorting out the deps issue for hacking rev | 20:12 |
sdague | jogo: yep | 20:13 |
jogo | sdague: what is the latest? | 20:13 |
*** jcoufal has quit IRC | 20:13 | |
*** zns has joined #openstack-infra | 20:13 | |
sdague | I believe we need - https://review.openstack.org/#/c/86189/ | 20:13 |
sdague | then we can unwind the mirror bit here - https://review.openstack.org/#/c/86171/ | 20:14 |
sdague | then we can let hacking have full control | 20:14 |
sdague | but we need the pbr change to land first | 20:14 |
jogo | I like that first patch, the old code is just strange | 20:15 |
openstackgerrit | A change was merged to openstack-infra/storyboard: Changed task states https://review.openstack.org/85868 | 20:15 |
jogo | mordred: https://review.openstack.org/#/c/86189/ | 20:15 |
*** openstackstatus has quit IRC | 20:15 | |
sdague | this is one of those situations where Depends-On: would be awesome | 20:15 |
sdague | because then we could figure out if all this worked together instead of lock stepping it | 20:15 |
*** openstackstatus has joined #openstack-infra | 20:16 | |
openstackgerrit | Darragh Bailey proposed a change to openstack-infra/jenkins-job-builder: Support recursive processing of paths https://review.openstack.org/68895 | 20:16 |
jogo | sdague: yup | 20:16 |
*** ociuhandu has quit IRC | 20:16 | |
sdague | so while waiting for that | 20:16 |
sdague | jogo: are you still working on the ephemeral buffer for debug message testing? | 20:17 |
mordred | sdague: you want me to land that? like, it's safe? | 20:17 |
jogo | nope, I finished working on it | 20:17 |
openstackgerrit | Darragh Bailey proposed a change to openstack-infra/jenkins-job-builder: Use yaml local tags to support including files https://review.openstack.org/48783 | 20:17 |
openstackgerrit | Darragh Bailey proposed a change to openstack-infra/jenkins-job-builder: Support lazy resolving of include yaml tags https://review.openstack.org/63580 | 20:17 |
jogo | sdague: err that isn't true | 20:18 |
jogo | sdague: https://review.openstack.org/#/c/85867/ | 20:18 |
openstackgerrit | Marc Abramowitz proposed a change to openstack-infra/jenkins-job-builder: PEP8ify https://review.openstack.org/86407 | 20:18 |
sdague | mordred: well, to the best of my understanding it's safe. It passes all the pbr tests. Though I'm not sure if it violates some other black magic | 20:18 |
*** ildikov has joined #openstack-infra | 20:19 | |
jogo | sdague: my patch worked when tested locally | 20:19 |
sdague | jogo: I thought the NullHandler skipped all of that | 20:19 |
mordred | sdague: ok- here goes | 20:19 |
jogo | have to sdague must messed something up there | 20:19 |
jogo | sdague: it doesn't try to format the log though | 20:20 |
jogo | sdague: see https://review.openstack.org/#/c/85867/1/nova/test.py | 20:20 |
sdague | jogo: yeh, I was looking | 20:20 |
*** markmcclain has joined #openstack-infra | 20:21 | |
*** hashar has quit IRC | 20:21 | |
sdague | jogo: you want me to take a spin on this? | 20:21 |
jogo | sdague: for https://review.openstack.org/#/c/86171/1 I am a little confused at where the new mirror-reqs file gets pulled in | 20:22 |
sdague | jogo: from the best of my understanding, the mirror script basically reads *requirements.txt | 20:22 |
jogo | sdague: nah, I will push another rev now | 20:22 |
sdague | jogo: ok, cool | 20:22 |
jogo | sdague: looks like a rebase issue | 20:22 |
sdague | jogo: it would be nice to self test this | 20:22 |
jogo | sdague: ahh thats what I assumed but wasn't sure | 20:22 |
jogo | sdague: yeah not sure how to do that though | 20:23 |
*** jp_at_hp has quit IRC | 20:23 | |
sdague | create a test which does a try loop around a crappy formatted debug message | 20:23 |
sdague | and make sure you get the exception | 20:24 |
jogo | sdague: even better mock out handle | 20:24 |
sdague | or that | 20:24 |
sdague | this is definitely self testable thought | 20:24 |
*** asselin_ has joined #openstack-infra | 20:24 | |
openstackgerrit | Marc Abramowitz proposed a change to openstack-infra/jenkins-job-builder: Make overrideDefaultExcludes line into 3 lines https://review.openstack.org/86434 | 20:27 |
openstackgerrit | Marc Abramowitz proposed a change to openstack-infra/jenkins-job-builder: Support pep8 & hacking (flake8) in or out of tox https://review.openstack.org/86407 | 20:31 |
*** jcoufal has joined #openstack-infra | 20:32 | |
*** zns has quit IRC | 20:32 | |
*** ociuhandu has joined #openstack-infra | 20:33 | |
*** ociuhandu has quit IRC | 20:33 | |
*** zns has joined #openstack-infra | 20:34 | |
*** harlowja is now known as harlowja_away | 20:36 | |
*** zns has quit IRC | 20:36 | |
*** zns has joined #openstack-infra | 20:37 | |
jogo | sdague: that should do it https://review.openstack.org/85867 | 20:42 |
jogo | sdague: now with a simple test | 20:42 |
clarkb | sdague: I am not sure if it will recover, looking at how it did while I was at lunch now | 20:43 |
*** jamielennox|away is now known as jamielennox | 20:43 | |
*** e0ne has quit IRC | 20:44 | |
*** e0ne has joined #openstack-infra | 20:44 | |
*** mbacchi has joined #openstack-infra | 20:45 | |
*** CaptTofu has quit IRC | 20:45 | |
jogo | clarkb: anything I can do to help get logstash back online? | 20:45 |
clarkb | it looks like it is still a sad panda | 20:45 |
clarkb | jogo: make geard better | 20:45 |
*** UtahDave has quit IRC | 20:46 | |
jogo | o_O what happend anyway? | 20:46 |
clarkb | sdague: jogo: I can restart the geard service to start queuing from scratch | 20:46 |
clarkb | jogo: we rebooted everything, elasticsearch took its time recovering, and geard has >400k events queued | 20:46 |
clarkb | and having 400k events queued appears to make it slow | 20:46 |
jogo | ahh | 20:47 |
jogo | so if you restart geard we loose some data? | 20:47 |
jogo | although not ideal I am fine with it | 20:47 |
clarkb | jogo: yes everything between when I do it and sometime yesterday afternoonish | 20:47 |
*** e0ne has quit IRC | 20:47 | |
openstackgerrit | Sean Dague proposed a change to openstack-infra/config: add grenade-dsvm-neutron to integrated gate https://review.openstack.org/86439 | 20:47 |
jogo | clarkb: its unfortunite to loose data but sure | 20:47 |
*** aconrad has quit IRC | 20:47 | |
jogo | clarkb: if that gets us out of this mess | 20:48 |
*** sweston has joined #openstack-infra | 20:48 | |
*** UtahDave has joined #openstack-infra | 20:48 | |
jeblair | dhellmann: what could be added to either python2 or python3 to make supporting both versions in the same source easier? | 20:48 |
*** e0ne has joined #openstack-infra | 20:50 | |
clarkb | sdague: jeblair you still think restarting geard is a good idea? | 20:52 |
sdague | clarkb: the recovery rate does not currently look promissing | 20:53 |
clarkb | ya I agree | 20:53 |
sdague | so unless we think that's going to change for some reason, I'd rather declare bankruptsy | 20:53 |
clarkb | wfm | 20:53 |
clarkb | jeblair: anything you know about geard that may indicate it would change for some reason? | 20:53 |
openstackgerrit | Joe Gordon proposed a change to openstack-dev/hacking: Use pep8's noqa argument https://review.openstack.org/86199 | 20:53 |
openstackgerrit | Joe Gordon proposed a change to openstack-dev/hacking: Move hacking to pep8 1.5.4 https://review.openstack.org/86197 | 20:53 |
*** e0ne has quit IRC | 20:54 | |
mtreinish | sdague: https://www.youtube.com/watch?v=HuGIgf-ICHM | 20:54 |
*** e0ne has joined #openstack-infra | 20:55 | |
*** harlowja_away is now known as harlowja | 20:55 | |
*** e0ne has quit IRC | 20:55 | |
*** mrda_away is now known as mrda | 20:55 | |
sdague | mtreinish: hah | 20:55 |
*** e0ne has joined #openstack-infra | 20:56 | |
*** pdmars has joined #openstack-infra | 20:56 | |
openstackgerrit | Jeremy Stanley proposed a change to openstack-infra/config: Finish puppeting accessbot https://review.openstack.org/86440 | 20:57 |
jogo | mtreinish: I think https://review.openstack.org/#/c/85867/ is better then your solution https://review.openstack.org/#/c/85759/4 | 20:57 |
*** dkliban has quit IRC | 20:57 | |
jogo | plus I have a test on mine too | 20:57 |
*** sweston has quit IRC | 20:58 | |
*** BadCub01 has joined #openstack-infra | 20:58 | |
mtreinish | jogo: considering mine doesn't even work I tend to agree :) | 20:58 |
*** nati_uen_ has quit IRC | 20:59 | |
sdague | jogo: one comment | 20:59 |
sdague | then I'm +2 | 20:59 |
*** e0ne has quit IRC | 20:59 | |
*** adalbas has quit IRC | 20:59 | |
zaro | mgagne: yo! | 20:59 |
mgagne | zaro: hi! | 21:00 |
sdague | ok, going for a walk while it's still nice out | 21:00 |
*** e0ne has joined #openstack-infra | 21:00 | |
zaro | msabramo1: i'll review your changes when i get a chance however i'm not approver on that. you'll need to talk to jeblair if you want anything approved there. | 21:01 |
msabramo1 | zaro: ok, thanks! | 21:01 |
jogo | sdague: patch fixed. enjoy the good weather | 21:01 |
zaro | msabramo1: i also have a patch waiting for approval. | 21:01 |
zaro | mgagne: hacking? | 21:01 |
*** MIDENN_ has quit IRC | 21:01 | |
mgagne | zaro: flake8/hacking | 21:01 |
*** mdenny has joined #openstack-infra | 21:02 | |
* jogo wonders why git review is so slow | 21:02 | |
sdague | it's slowest on nova | 21:03 |
sdague | it does depend on the project | 21:03 |
zaro | mgagne: hmm, i just recently saw a patch for flake8. | 21:03 |
*** e0ne has quit IRC | 21:03 | |
*** dklyle is now known as david-lyle | 21:03 | |
zaro | mgagne: https://review.openstack.org/#/c/86407 | 21:03 |
mgagne | zaro: yes, that's what I suggested msabramo1 to use instead of developing around pep8 | 21:04 |
*** zns has quit IRC | 21:04 | |
msabramo1 | mgagne, zaro: I can remove the .pep8 file; personally it makes more sense to me than putting the options in tox.ini | 21:05 |
zaro | mgagne: your 'add me' button is coming.. ;) | 21:05 |
mgagne | msabramo1: yes, that's what I'm doing =) | 21:05 |
mgagne | msabramo1: if other projects use flake8/hacking and tox.ini, I don't see why we couldn't | 21:05 |
*** derekh has quit IRC | 21:05 | |
mgagne | msabramo1: hardest part is composing a commit message =) | 21:06 |
msabramo1 | mgagne: cool! | 21:06 |
clarkb | sdague: jogo it does rebases and stuff | 21:06 |
msabramo1 | mgagne: hacking and flake8 seem very nice and with the .pep8 file I was able to see all of their feedback in my vim | 21:06 |
zaro | mgagne: jjb has been busy lately. need you back! | 21:06 |
clarkb | it tries to reduce the amount of thinking you have to do but that means a few network roundtrips and not cpu cheap operations | 21:06 |
mgagne | zaro: =) | 21:07 |
jogo | clarkb: ? | 21:07 |
jogo | msabramo1: you don't need .pep8 to get hacking working in vim | 21:07 |
zaro | mgagne: actually msabramo1 is why we need you. | 21:07 |
clarkb | jogo: re git review | 21:08 |
mgagne | zaro: haha, that's what I figured out =) | 21:08 |
msabramo1 | mgagne, zaro: Sorry to put a load on you guys | 21:08 |
jogo | clarkb: ohh yeah I know, I am on a train and bad connection | 21:08 |
mgagne | msabramo1: it's alright, I'll have to figure out a way to find more time and scale my review process =) | 21:08 |
zaro | msabramo1: np, can redeem yourself by reviewing jjb. | 21:09 |
*** pcargnel has quit IRC | 21:09 | |
msabramo1 | mgagne, zaro: I like JJB though and we're starting to use it as the basis for an internal tool | 21:09 |
msabramo1 | zaro: yeah, was thinking that already | 21:09 |
openstackgerrit | Mathieu Gagné proposed a change to openstack-infra/jenkins-job-builder: Fix H306 errors reported by flake8/hacking https://review.openstack.org/86445 | 21:10 |
openstackgerrit | Mathieu Gagné proposed a change to openstack-infra/jenkins-job-builder: Fix H301 errors reported by flake8/hacking https://review.openstack.org/86446 | 21:10 |
openstackgerrit | Mathieu Gagné proposed a change to openstack-infra/jenkins-job-builder: Fix H302 errors reported by flake8/hacking https://review.openstack.org/86447 | 21:10 |
openstackgerrit | Mathieu Gagné proposed a change to openstack-infra/jenkins-job-builder: Fix H101 errors reported by flake8/hacking https://review.openstack.org/86448 | 21:10 |
openstackgerrit | Mathieu Gagné proposed a change to openstack-infra/jenkins-job-builder: Fix H403 and H404 errors reported by flake8/hacking https://review.openstack.org/86449 | 21:10 |
openstackgerrit | Mathieu Gagné proposed a change to openstack-infra/jenkins-job-builder: Fix H902, H233, H231 and H703 errors reported by flake8/hacking https://review.openstack.org/86450 | 21:10 |
openstackgerrit | Mathieu Gagné proposed a change to openstack-infra/jenkins-job-builder: Migrate to flake8/hacking https://review.openstack.org/86451 | 21:10 |
mgagne | zaro, msabramo1: here we go, we should have hacking now | 21:10 |
zaro | wow! | 21:10 |
*** ociuhandu has joined #openstack-infra | 21:11 | |
mgagne | zaro: I started working in my free time on other python projects written by our devs. So I found time to explore tools written and used by OpenStack which I didn't use/learn yet and port them to it. =) | 21:11 |
*** skraynev is now known as skraynev_afk | 21:11 | |
zaro | mgagne: your way ahead of me, i have no idea what hacking is :) | 21:12 |
mgagne | zaro: looks to be a plugin for flake8 to enforce OpenStack style guidelines | 21:12 |
mgagne | zaro: flake8 wraps around pep8 and pyflakes | 21:13 |
*** jeckersb is now known as jeckersb_gone | 21:13 | |
openstackgerrit | Michael Krotscheck proposed a change to openstack-infra/storyboard: [WIP] Collate story metadata for status https://review.openstack.org/86452 | 21:13 |
*** lcostantino has quit IRC | 21:13 | |
zaro | mgagne: ahh, seems convoluted. maybe i'll let the brain trust handle that one. | 21:13 |
*** adalbas has joined #openstack-infra | 21:13 | |
mgagne | zaro: http://lists.openstack.org/pipermail/openstack-dev/2013-April/007201.html | 21:14 |
*** openstackstatus has quit IRC | 21:14 | |
*** gokrokve has quit IRC | 21:14 | |
SergeyLukjanov | mgagne, zaro, there was a discussion about using hacking in infra and agreement was not use it because of small projects | 21:15 |
SergeyLukjanov | jeblair, ^^ | 21:15 |
mgagne | SergeyLukjanov: of small projects? | 21:15 |
*** jcoufal has quit IRC | 21:16 | |
* zaro ?? as well | 21:16 | |
SergeyLukjanov | mgagne, the argument of jeblair was that our projects in infra are not so large as OpenStack ones, so, there is no need to use hacking | 21:16 |
SergeyLukjanov | it was discussed when I proposed hacking enable for zuul | 21:17 |
zaro | mgagne, msabramo1 : know anything about voluptuous? | 21:17 |
SergeyLukjanov | I'll try to find some links | 21:17 |
*** rfolco has quit IRC | 21:17 | |
mgagne | SergeyLukjanov: that one will need more explanation as the hard work (complying with hacking style) has been done =) | 21:18 |
openstackgerrit | Clark Boylan proposed a change to openstack-infra/config: Use snakeoil certs on numbered jenkins masters. https://review.openstack.org/86453 | 21:18 |
mgagne | SergeyLukjanov: and I guess people did not believe it was good invested time due to the small size of the projects | 21:18 |
mgagne | SergeyLukjanov: https://review.openstack.org/#/c/38013/ ? | 21:19 |
zaro | mgagne, msabramo1 : no, not the woman figure, the app to validate parameter types. | 21:19 |
jeblair | SergeyLukjanov, mgagne: we can use hacking in order to get the deps, etc; but we ignore the H rules | 21:19 |
msabramo1 | zaro: voluptuous - I think I saw this recently on PyPI but don't recall the details | 21:19 |
SergeyLukjanov | jeblair, yup, that's I mean | 21:19 |
msabramo1 | zaro: similar perhaps to schematics? which I have played with a tiny bit | 21:19 |
*** zns has joined #openstack-infra | 21:20 | |
jeblair | zaro: voluptuous is a yaml syntax checker, zuul uses it | 21:20 |
SergeyLukjanov | jeblair, "ignore the H rules" is a precise wording | 21:20 |
*** andreykurilin_ has quit IRC | 21:21 | |
*** openstackstatus has joined #openstack-infra | 21:22 | |
zaro | jeblair: thanks, was seeing if msabramo1 or mgagne had interest to make work with jjb as well. | 21:22 |
*** dkranz has quit IRC | 21:23 | |
*** SumitNaiksatam has quit IRC | 21:23 | |
*** andreykurilin_ has joined #openstack-infra | 21:26 | |
*** gokrokve has joined #openstack-infra | 21:27 | |
openstackgerrit | A change was merged to openstack-dev/pbr: make pbr use hacking directly https://review.openstack.org/86189 | 21:28 |
msabramo1 | jeblair: Got a python-jenkins change for you to look at: https://code.launchpad.net/~msabramo/python-jenkins/python3/+merge/214664 | 21:28 |
*** esker has quit IRC | 21:28 | |
*** thedodd has quit IRC | 21:30 | |
*** ameade has quit IRC | 21:30 | |
*** dcramer_ has quit IRC | 21:30 | |
*** mfer has quit IRC | 21:31 | |
pabelanger | Hmm, looks like I'm having some delete issue with rax and nodepool. http://goo.gl/eF6rLY I _think_ infra has the same issues, but don't see the starvation because of the multiple providers you use | 21:31 |
*** thedodd has joined #openstack-infra | 21:32 | |
*** nati_ueno has joined #openstack-infra | 21:32 | |
dhellmann | jeblair: to the language, or the stdlib? | 21:34 |
*** moted has quit IRC | 21:35 | |
fungi | pabelanger: our providers sometimes ignore nova delete calls completely, which is why nodepool retries them | 21:35 |
*** andreykurilin_ has quit IRC | 21:37 | |
dhellmann | jeblair: I think one of the things that prevents us from just using tulip/asyncio is the use of "yield from" | 21:37 |
*** weshay has quit IRC | 21:38 | |
*** markmcclain has quit IRC | 21:39 | |
clarkb | dhellmann: doesn't trolius fix that? | 21:39 |
clarkb | so you ifdef based on python version but from openstacks perspective the api is the same | 21:40 |
dhellmann | clarkb: well, yes, but the reason that fork exists is largely because of the syntax issues preventing us using the original code | 21:40 |
*** markmcclain has joined #openstack-infra | 21:41 | |
dhellmann | clarkb, jeblair : haypo (victor) is probably at pycon, too, and he'd be the best person to ask about that specific issue | 21:41 |
dhellmann | I'm not sure there's anything else we're "blocked" on in a similar way | 21:41 |
*** mriedem has quit IRC | 21:43 | |
*** dizquierdo has joined #openstack-infra | 21:43 | |
*** reed has quit IRC | 21:44 | |
*** aysyd has quit IRC | 21:49 | |
openstackgerrit | Clark Boylan proposed a change to openstack-infra/config: Use snakeoil certs on numbered jenkins masters. https://review.openstack.org/86453 | 21:50 |
*** nati_ueno has quit IRC | 21:50 | |
*** dizquierdo has quit IRC | 21:53 | |
*** nati_ueno has joined #openstack-infra | 21:53 | |
*** jeckersb_gone is now known as jeckersb | 21:55 | |
*** thuc has quit IRC | 21:55 | |
*** thuc has joined #openstack-infra | 21:56 | |
*** gokrokve has quit IRC | 21:58 | |
*** e0ne has joined #openstack-infra | 21:59 | |
*** thuc has quit IRC | 22:00 | |
*** eharney has quit IRC | 22:01 | |
*** david-lyle has quit IRC | 22:02 | |
*** e0ne has quit IRC | 22:02 | |
*** markmcclain has quit IRC | 22:02 | |
msabramo1 | mgagne: your hacking/flake8 changes look great. Made some tiny suggestions for the last one: https://review.openstack.org/86451 which you can take or leave | 22:03 |
mgagne | msabramo1: do you have tox installed? tox -epep8 should take care of everything for you | 22:05 |
msabramo1 | mgagne: Yep, I have tox | 22:05 |
mgagne | msabramo1: the pep8 target is used by Jenkins (and all other projects) to run the pep8 job, we can't rename it | 22:06 |
msabramo1 | mgagne: ah ok, didn't realize the "pep8" target is special | 22:06 |
mgagne | msabramo1: well it is "special" because it is actually actively used by someone (Jenkins) | 22:07 |
*** SumitNaiksatam has joined #openstack-infra | 22:07 | |
openstackgerrit | Mathieu Gagné proposed a change to openstack-infra/jenkins-job-builder: Migrate to flake8/hacking https://review.openstack.org/86451 | 22:08 |
msabramo1 | mgagne: "The [pep8]" addition was so that folks can run pep8 outside of tox, in case they don't have or use tox for some reason or maybe their editor's linter is going to do pep8 checks (mine I think uses flake8 rather than pep8 but who knows what other people have) | 22:08 |
msabramo1 | mgagne: Just trying to make as many things work as possible | 22:08 |
clarkb | mgagne: msabramo1 fwiw we want to rename it to something list pep8 specific. Something like staticcheck or whatever | 22:08 |
mgagne | msabramo1: I understand | 22:08 |
clarkb | but that requires adding a target in all of our projects, changing the jenkins scriptage then removing the old target from all projets | 22:08 |
clarkb | and it is a relatively low priority item with lots of effort ot make it happen | 22:09 |
msabramo1 | clarkb: yeah, sounds like a pain | 22:09 |
mgagne | clarkb: rename it? you mean the pep8 testenv? | 22:09 |
msabramo1 | clarkb: I guess projects could have both targets for a while until everything is in place | 22:09 |
*** prad has quit IRC | 22:10 | |
openstackgerrit | A change was merged to openstack-infra/config: Use snakeoil certs on numbered jenkins masters. https://review.openstack.org/86453 | 22:11 |
clarkb | mgagne: ya | 22:11 |
*** gokrokve has joined #openstack-infra | 22:12 | |
*** denis_makogon has quit IRC | 22:12 | |
mgagne | clarkb: is there a way to answer the need of msabramo1 (adding pep8 section) without duplicating the configs from flake8? | 22:13 |
*** kevinbenton has quit IRC | 22:14 | |
clarkb | mgagne: I think you have to duplicate them | 22:14 |
sdague | jogo: https://review.openstack.org/#/c/85867/ - py26 doesn't have NullHandler | 22:15 |
sdague | apparently | 22:15 |
*** thedodd has quit IRC | 22:15 | |
msabramo1 | mgagne, clarkb: too bad that this doesn't work | 22:16 |
msabramo1 | [pep8] | 22:16 |
msabramo1 | ignore = {[flake8]ignore} | 22:16 |
msabramo1 | exclude = {[flake8]exclude} | 22:16 |
mgagne | msabramo1: requires tox 1.4 | 22:16 |
mordred | tox doesn't process those sections - pep8 does | 22:16 |
msabramo1 | mordred: ah right | 22:17 |
mordred | but yeah | 22:17 |
mgagne | msabramo1: right =( | 22:17 |
mattoliverau | Morning all | 22:19 |
*** dizquierdo has joined #openstack-infra | 22:20 | |
openstackgerrit | Mathieu Gagné proposed a change to openstack-infra/jenkins-job-builder: Migrate to flake8/hacking https://review.openstack.org/86451 | 22:20 |
*** SumitNaiksatam has quit IRC | 22:22 | |
*** thomasem_ has joined #openstack-infra | 22:24 | |
*** thomasem has quit IRC | 22:24 | |
*** wchrisj_ has quit IRC | 22:28 | |
*** openstack has joined #openstack-infra | 22:34 | |
openstackgerrit | Marc Abramowitz proposed a change to openstack-dev/hacking: Minor README.rst tweaks https://review.openstack.org/86464 | 22:35 |
*** moted has joined #openstack-infra | 22:35 | |
*** jgrimm has quit IRC | 22:38 | |
openstackgerrit | Darragh Bailey proposed a change to openstack-infra/jenkins-job-builder: Use yaml local tags to support including files https://review.openstack.org/48783 | 22:39 |
openstackgerrit | Darragh Bailey proposed a change to openstack-infra/jenkins-job-builder: Support lazy resolving of include yaml tags https://review.openstack.org/63580 | 22:39 |
*** rlandy has quit IRC | 22:39 | |
*** alexpilotti has quit IRC | 22:40 | |
*** jhesketh_ has joined #openstack-infra | 22:47 | |
jhesketh_ | Morning | 22:49 |
openstackgerrit | Darragh Bailey proposed a change to openstack-infra/jenkins-job-builder: Support recursive processing of paths https://review.openstack.org/68895 | 22:49 |
mattoliverau | morning jhesketh_ | 22:49 |
*** khyati_ has joined #openstack-infra | 22:52 | |
*** msabramo1 has quit IRC | 22:53 | |
*** thuc has joined #openstack-infra | 22:54 | |
*** thuc has quit IRC | 22:54 | |
*** mrodden has quit IRC | 22:55 | |
*** thuc has joined #openstack-infra | 22:55 | |
*** armax has left #openstack-infra | 22:57 | |
*** thuc has quit IRC | 22:59 | |
*** Sukhdev has joined #openstack-infra | 22:59 | |
*** thuc has joined #openstack-infra | 23:00 | |
*** nati_ueno has quit IRC | 23:01 | |
*** dims has quit IRC | 23:02 | |
*** thuc has quit IRC | 23:04 | |
*** msabramo has joined #openstack-infra | 23:05 | |
*** thuc has joined #openstack-infra | 23:09 | |
*** thuc has quit IRC | 23:09 | |
sdague | SlickNik: where's the trove change to dump pyflakes at? | 23:09 |
*** thuc has joined #openstack-infra | 23:10 | |
SlickNik | sdague: https://review.openstack.org/#/c/67138/ | 23:10 |
*** thuc has quit IRC | 23:10 | |
sdague | SlickNik: can you approve it? | 23:10 |
sdague | that looks like my next fail point in - https://review.openstack.org/#/c/86171/ | 23:11 |
openstackgerrit | A change was merged to openstack-infra/jenkins-job-builder: Make all help text start with lowercase letter https://review.openstack.org/86410 | 23:11 |
*** thuc has joined #openstack-infra | 23:11 | |
clarkb | sdague: did you see logstash bankrupcy happened? we are "caught up" now | 23:12 |
sdague | nope, but awesome | 23:13 |
SlickNik | sdague: Done. It's in the merge pipeline now. | 23:15 |
sdague | SlickNik: thank! | 23:15 |
*** Clabbe has quit IRC | 23:16 | |
*** changbl has quit IRC | 23:21 | |
*** nati_ueno has joined #openstack-infra | 23:22 | |
*** gokrokve has quit IRC | 23:22 | |
*** glenng has joined #openstack-infra | 23:26 | |
openstackgerrit | K Jonathan Harker proposed a change to openstack-infra/jeepyb: Read the has-github value from the defaults file https://review.openstack.org/86471 | 23:26 |
jogo | sdague: damn, its is really small I can just make one here | 23:27 |
*** mrodden has joined #openstack-infra | 23:27 | |
*** wchrisj has joined #openstack-infra | 23:28 | |
*** Clabbe has joined #openstack-infra | 23:28 | |
*** glenng1 has joined #openstack-infra | 23:29 | |
*** ominakov has quit IRC | 23:30 | |
*** glenng has quit IRC | 23:30 | |
*** lcostantino has joined #openstack-infra | 23:34 | |
*** msabramo1 has joined #openstack-infra | 23:34 | |
*** zehicle_at_dell has joined #openstack-infra | 23:36 | |
*** msabramo has quit IRC | 23:37 | |
*** sabari has quit IRC | 23:38 | |
*** dprince has quit IRC | 23:41 | |
*** thuc has quit IRC | 23:45 | |
StevenK | clarkb, fungi: Could I have one of you take a quick look at https://review.openstack.org/#/c/85932/ ? | 23:45 |
*** thuc has joined #openstack-infra | 23:46 | |
*** wchrisj has quit IRC | 23:47 | |
*** thuc has quit IRC | 23:50 | |
*** atiwari has quit IRC | 23:53 | |
openstackgerrit | A change was merged to openstack-infra/config: Also clone os-cloud-config in prepare_tripleo.sh https://review.openstack.org/85932 | 23:55 |
*** dcramer_ has joined #openstack-infra | 23:59 | |
StevenK | fungi: Thanks! | 23:59 |
*** msabramo1 has quit IRC | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!