Wednesday, 2015-05-06

*** dkranz has joined #openstack-infra00:00
pleia2clarkb: hm, I'll refresh things on my end and see how it goes00:00
*** gokrokve has quit IRC00:01
*** Krinkle|detached is now known as Krinkle00:03
*** ayoung has joined #openstack-infra00:03
pleia2clarkb: thanks, I'll have another go at it tomorrow (evening plans now)00:04
clarkbpleia2: thank you for double checking my documentation00:05
clarkbfungi: how does jjb updating go?00:06
clarkbwe seem to be runing not many jobs00:07
fungiclarkb: it's still running00:07
*** hemna is now known as hemnafk00:07
fungihowever, zuul is also basically caught up on its backlog now, per the job queue graph00:07
*** claudiub has quit IRC00:08
*** gokrokve has joined #openstack-infra00:08
anteayayay00:11
clarkbfungi: maybe? looks like it needs to blast through the results queue then it will have a bunch more load00:11
jeblairfungi, clarkb: i think geard is straining a bit00:11
clarkbjeblair: ya00:11
fungii can stop the jjb updates again00:11
anteayajeblair: what are you looking at that tells you that, is it in graphite?00:12
jeblairanteaya: no, top.00:12
anteayaah00:13
jeblairfungi: yeah, might be a good idea00:13
anteaya:(00:13
clarkbmaybe leave one running?00:13
jeblairclarkb, fungi: yeah, maybe one at a time00:14
jeblairstrace is all about CAN_DO00:14
*** sputnik13 has quit IRC00:14
*** haypo has left #openstack-infra00:14
ianwhttps://review.openstack.org/#/c/179689/ <- patchset 9 not picked up, been about 30 minutes, so i guess not caught up00:15
*** mtanino has quit IRC00:15
ianwalthough the test still won't work due to cffi , one of those days :)00:15
clarkbjeblair: anything you can do jenkins can do better?00:16
clarkbthat should be a jenkins motto00:16
*** achanda has quit IRC00:18
*** otter768 has joined #openstack-infra00:18
jeblair(zuulv3 will not need each worker to register 7000 gearman functions)00:18
*** dkranz has quit IRC00:18
*** achanda has joined #openstack-infra00:19
clarkbyou would basically say CAN_DO run_ansible right?00:19
clarkbthen ansible can figure out what to do00:20
jeblairyeah, more or less00:21
jeblairfungi: did you stop jjb?00:22
*** otter768 has quit IRC00:23
*** achanda has quit IRC00:23
clarkbhrm we can probably update gearman-plugin to do additive CAN_DOs and only fallback on the unperformant RESET_ABILITIES then can do everything if we need to remove things00:24
* clarkb checks if there is a CAN'T DO00:24
jeblairclarkb: there is00:24
clarkbthere is so we don't even need to fallback, just take a diff and do minimal updates00:24
clarkbI think the problem here is for every job that is updated we CAN_DO all the jobs00:24
clarkbso its an exponential update00:24
sdagueso... gerrit event stream stopped again?00:24
jeblairclarkb: yep00:25
clarkbsdague: no, gearman is unable to keep up with job updates00:25
sdaguebecause the fix changes pushed 15 minutes ago for the gate have not appeared in zuul yet00:25
sdagueok00:25
*** rloo has quit IRC00:25
jeblairer, that might also have happened00:25
clarkbgah00:25
jeblairhttp://paste.openstack.org/show/215173/00:26
jeblairi think we learned something from my debug patch00:26
clarkbthat one account has all the connections?00:27
jeblairyeah, all of the stuck threads are servicing that account00:27
jeblairand those times are the start times for when they switched to handling that connection00:28
sdagueis that blockable?00:28
jeblairyeah, we can disable the account00:28
sdagueuntil such time as they can get interactive to help figure out what they are doing that's causing this00:29
jeblairi believe that will prohibit stream-events00:29
clarkbjeblair: how do we know they are stuck?00:29
clarkbjeblair: we can also make a group without stream-events perms and put users in that group00:29
clarkbjeblair: either way should work00:29
sdagueclarkb / fungi: you guys have the changes that are needed to unblock things, right?00:29
clarkbsdague: I am sure I can find them00:30
sdagueso I can sign off and those can be shepharded00:30
*** freerunner has quit IRC00:30
sdaguehttps://review.openstack.org/#/c/180341/ PS 300:30
sdagueand it's depends on00:30
clarkbjeblair: looks like 02 and 07 are running a pair of jenkins-jobs each00:30
jeblairclarkb: because since april 30 is a long time to try to send a packet?00:30
sdagueok, night folks00:31
jeblairclarkb: i think fungi has afk'd; can you kill 3 of those00:31
jeblair?00:31
clarkbjeblair: ya00:31
jeblairclarkb: you should run 'ssh review gerrit show-queue'00:32
clarkboh actually its just one per, but two processes because sudo, killing on 0700:32
*** dims_ has joined #openstack-infra00:32
*** freerunner has joined #openstack-infra00:32
*** ildikov has quit IRC00:32
clarkbah and those are the active ones00:33
clarkband have taken up all our threads?00:33
*** tnovacik has quit IRC00:33
*** dims has quit IRC00:33
clarkbjeblair: were you going to disable the account? I can do it if it helps00:34
*** mriedem has quit IRC00:34
jeblairclarkb: i'll do it00:35
*** Sukhdev has quit IRC00:36
jeblairclarkb: ready for me to restart gerrit?00:37
clarkbI think so, you want ti disable first otherwise the account stuff is cached and you ahve to wait for it to clear out or flush all caches00:38
jeblairi also have a thread dump00:38
jeblairyeah, should be disable now00:38
clarkbthen I am ready00:39
jeblairclarkb: can you kill that last jjb run?00:39
clarkbI can00:39
clarkbdone00:40
fungii'm back, sorry. was otp00:41
jeblair#status notice Restarted gerrit due to stuck stream-events connections.  Events since 23:49 were missed and changes uploaded since then will need to be rechecked.00:41
openstackstatusjeblair: sending notice00:41
fungilooks like you got the three jjb updates i had running killed off00:42
clarkbfungi: there was a third?00:42
clarkbfungi: I got 02 and 0700:42
-openstackstatus- NOTICE: Restarted gerrit due to stuck stream-events connections. Events since 23:49 were missed and changes uploaded since then will need to be rechecked.00:42
clarkband tcpdump on zuul seems to confirm that was it00:42
clarkbmaybe the third one completed?00:42
jeblairgeard is happy now00:43
fungi04 was the other00:43
fungiseems it may have completed on its own00:43
jeblairfungi: can you start up one of those again and we'll see how it goes?00:43
fungirestarting now, yeah00:43
clarkbnothing running on 04 so it completed I think00:43
fungigoing again on 0700:43
openstackstatusjeblair: finished sending notice00:43
jeblairfungi: and we made progress on stream-events; see http://paste.openstack.org/show/215173/00:44
*** tiswanso has joined #openstack-infra00:45
fungijeblair: hrm... i saw that last time too. they have something like half a dozen different ci systems which use the same account (compare against show-connections and you'll see they're various ip addresses)00:45
fungii asked around about it at the time00:45
*** ildikov has joined #openstack-infra00:46
*** samueldmq has quit IRC00:47
*** tiswanso has quit IRC00:47
*** tiswanso has joined #openstack-infra00:48
*** Somay has quit IRC00:49
*** samueldmq has joined #openstack-infra00:50
jeblairfungi: oh neat, this happened already?00:50
*** Krinkle is now known as Krinkle|detached00:51
jeblairfungi: so at least that's confirmation :)00:51
fungiyeah, you were travelling. i collected several fresh jstack traces with thread names properly resolved, and corresponding connections and queues outputs from gerrit00:51
jeblairfungi, clarkb: looks like geard can handle one full jjb run00:52
clarkbjeblair: thats good news00:52
clarkbI am playing with the start of a patch to basically do the CAN_DO CANT_DO diff00:52
fungii still have them. unfortunately i didn't get much out of it other than confirming that there wasn't any immediately obvious correlation, and the method the stream events workers were blocking in matched the upstream bug00:52
jeblairso i think we can proceed with the jjb runs one at a time00:52
clarkbfigure while the code is familiar to give it a go00:52
fungijeblair: wfm, i can keep tabs on them for a while still and kick off the remaining ones in sequence after this one completes00:53
anteayado you want me to get the gerrit id for netapp-ci?00:53
jeblairanteaya: no thanks, i already disabled it and sent an email00:54
anteayajeblair: ah thank00:55
anteayas00:55
fungithey apparently have different ci systems for a bunch of cinder drivers and share the same account between them rather than asking for separate accounts for them (from back when they still needed to request accounts)00:55
anteayaah00:56
anteayathey have at least two different gerrit accounts00:56
anteayaNetApp FC CI and NetApp NAS CI00:57
anteayawhich was the one that was disabled?00:57
jeblairanteaya: "NetApp CI" <xdl-openstack-jenkins@netapp.com>00:58
*** freerunner has quit IRC00:58
anteayaoh00:59
jeblairanteaya: "at lesat two" :)00:59
anteayawhich my search in gerrit didnt' even turn up00:59
anteayaso I couldn't have found them anyway01:00
anteayamy experience with this group from the cinder threads is they drag their feet01:00
anteayaand aren't the most proactive01:00
*** cody-somerville has quit IRC01:00
anteayacinder mailing list threads01:01
dougwigdo you folks know the root cause of why they broke things, so the rest of us can avoid it?01:01
anteayanope01:01
*** patrickeast has quit IRC01:02
anteayajust that all stuck threads were servicing them01:02
*** gokrokve has quit IRC01:02
anteayahence the invitation for a chat01:02
anteayahoping we can learn01:02
anteayawill share if we find anything01:02
anteayathanks for asking01:02
fungicould be something as innocuous as they're all running some particular release of jenkins01:02
dougwigok, i'll try to avoid launching 10 zuuls in the meantime.  :)01:02
anteayadougwig: thanks01:03
fungier, of the jenkins gerrit-trigger plugin i mean01:03
anteayayeah it might be some custom script they wrote01:04
*** SumitNaiksatam has joined #openstack-infra01:04
*** Somay has joined #openstack-infra01:04
openstackgerritDarragh Bailey proposed openstack-infra/jenkins-job-builder: Separate out custom yaml tags into separate classes  https://review.openstack.org/18037001:05
openstackgerritDarragh Bailey proposed openstack-infra/jenkins-job-builder: Support lazy resolving of include yaml tags  https://review.openstack.org/6358001:05
*** ddieterly has joined #openstack-infra01:06
openstackgerritClark Boylan proposed openstack-infra/gearman-plugin: Register the diff of functions  https://review.openstack.org/18037101:06
clarkbjeblair: zaro ^ my initial testing says that works01:06
clarkbfungi: anteaya it could also be related to having one account hold many open sessions01:07
clarkbfungi: anteaya if perhaps there is some state that updates racily01:07
clarkbI need to pop out now, I think I have a handful of gearman-plugin changes now between mine and jeblairs to prep for inclusion on say 07 in the morning01:07
*** weshay has quit IRC01:08
*** alexpilotti has quit IRC01:08
anteayaclarkb: it is possible, yes01:09
*** jtriley has quit IRC01:11
anteayaI stand corrected01:12
anteayaakerr actually shows up to meetings: https://wiki.openstack.org/wiki/ThirdPartySystems/NetApp_CI01:12
anteayaand they are using zuul01:12
anteayaameade also attends meetings01:12
anteayahopefully they are responsive01:12
ameadeim here!01:14
ameadeanteaya:  reading scrollback01:14
*** shashankhegde has quit IRC01:16
ameadeweird there should only be one thing listening for that account, ill take a look01:17
anteayaameade: thank you01:19
anteayaameade: the other netapp account, netapp fc doesn't seem to be doing the same thing01:19
*** zhiwei has joined #openstack-infra01:19
anteayaameade: what might be the difference between the two accounts?01:19
ameadekk interesting01:21
anteayafinding anything yet?01:21
ameadethe two accounts represent two completely seperate systems01:21
ameadefibre channel has a unique set of challenges that our other system couldnt do, but we want to get it all back under one system soon01:21
ameadejust grabbed my laptop01:22
anteayathanks01:22
anteayaso the ci that was shut down is the non fibre channel one, yes?01:22
*** alexsyip has quit IRC01:22
anteayaI'm going with FC is fibre channel01:22
anteayaits a stretch I know01:22
ameadeanteaya: lol correct01:23
anteayaso yay for figuring out fibre channel in a way that didn't bring down our gerrit01:23
anteayanow for the blocked system01:23
anteayalet's start with what tools you are running and versions, shall we?01:24
anteayaand any customizations you might have running01:24
ameadei image its zuul doing this command and that should be out of the box01:25
anteayadid you git clone zuul?01:25
anteayado you recall the last time it was updated?01:25
ameadeZuul version: 2.0.0.225.g6d2665801:26
ameadeand darn we did clone it01:26
ameadeit's been awhile then01:26
anteayaokay so we have that01:27
anteayaso zuul and what else do you have running?01:27
ameadeNodepool, Jenkins01:28
*** Swami has quit IRC01:28
anteayahow did you get nodepool, git again?01:28
anteayaand what version?01:28
anteayaor sha01:28
ameadewe have a big fork of nodepool with custom logic01:29
anteayalet's dig into that a bit shall we?01:29
*** alexpilotti has joined #openstack-infra01:29
anteayais this big fork of nodepool unique to this account? so the fc one is not running the same fork?01:29
ameadethat's correct01:30
anteayawe have a big lead here then01:30
anteayahelp me understand why you forked and what you needed the customization to do01:30
anteayaas long as I'm not crossing a proprietary boundary by asking01:30
ameadethe customization just does bootstrapping of the netapp controllers and configures the nodes accordingly01:31
Kennanhi :clarkb01:31
anteayahave you any idea when this system with the customized nodepool came online?01:31
anteayaKennan: he is out for a bit01:32
Kennancould I ask you one question ?01:32
Kennan:anteaya01:32
anteayaKennan: ask away, he or someone else will do their best when they can01:32
ameadeanteaya: oh it's been there from the start which has been at least 6 months01:32
clarkbKennan: usually its best to just ask01:32
Kennando you know how to make IRC show review link for kilo branch review?01:32
KennanI sent review for kilo in our project01:33
ameadeis the issue that ssh connections aren't going away?01:33
anteayafungi clarkb jeblair do we have any idea when the stuck gerrit thread issue came to our attention?01:33
KennanI did not find it have show in IRC chat01:33
anteayaameade: correct01:33
clarkbanteaya: ya, jeblair's status announcement included the time it went under01:33
Kennanbut seems master can work01:33
anteayaameade: so threads get stuck, gerrit stops streaming events, zuul doesn't do anything because it can't hear events that aren't broadcast01:33
*** gokrokve has joined #openstack-infra01:34
Kennan:anteaya do you know how to make openstack show other branch code review link in IRC ?01:34
anteayaclarkb: sorry, like how many weeks/months this has been an issue01:34
clarkbKennan: you need to make sure that https://git.openstack.org/cgit/openstack-infra/project-config/tree/gerritbot/channels.yaml specifies stable/kilo under your channel01:34
clarkbanteaya: oh that, I think it started when we upgraded to trusty01:34
ameadeanteaya: kk, that makes sense at least...no idea why we would have zombie connections though..doing some digging01:34
clarkbanteaya: so that was march ya?01:34
anteayaameade: thanks01:34
Kennan:clarkb OK thanks. Let me check that01:34
anteayaclarkb: sounds about right, I can go back to the announcements and look01:35
anteayaSaturday, March 21, 201501:35
*** ddieterly has quit IRC01:41
*** spzala has quit IRC01:41
*** ddieterly has joined #openstack-infra01:43
*** ddieterly has quit IRC01:43
*** derekh_ has joined #openstack-infra01:46
*** signed8bit_ZZZzz is now known as signed8b_01:47
*** signed8b_ is now known as signed8bit_ZZZzz01:47
ameadeanteaya: pretty sure i just figured it out01:48
*** alexpilotti has quit IRC01:48
ameadeanteaya: do you want the long or the short? and how do I prove it to you? lol01:48
anteayaameade: give it to me anyway you like01:49
* anteaya listens01:49
*** derekh has quit IRC01:50
ameadehehe, i had a python script for when I was setting up the FC system a month or so ago that listened to the event stream which i had running with the other accounts creds that has a code path that doesnt close the connection01:50
anteayaameade: can we ensure that script doens't run anymore?01:51
*** r-daneel has quit IRC01:52
ameadeanteaya: yeah definitely01:53
anteayathank you01:53
anteayaafter that can you reply to http://lists.openstack.org/pipermail/third-party-announce/2015-May/000177.html01:53
ameadeanteaya: really sorry for the firedrill01:53
anteayasaying what you did, so others don't do the same01:53
anteayaand saying you won't do it again01:54
anteayathanks for showing up so quickly and helping to track it down01:54
anteayathank you01:54
ameadeanteaya: sure thing, I appreciate the help01:54
anteayait will be a relief for us to know the source01:54
anteayaameade: great, glad we got to the bottom of it01:54
anteayawhen you post in reply to the -announce thread you will get a reply saying an admin has to let it through01:57
anteayathat's me and I'm waiting01:57
openstackgerritIan Wienand proposed openstack-infra/devstack-gate: [WIP] run centos7 with neutron  https://review.openstack.org/17968901:57
ameadeanteaya: kk01:58
*** salv-orlando has joined #openstack-infra01:58
* ameade makes haste01:58
anteayathanks01:58
*** ddieterly has joined #openstack-infra01:58
anteayaI didn't mean it like that01:58
*** ddieterly has quit IRC01:58
anteayajust saying that I'm ready to let the post through01:58
anteayanot that you have to hurry01:58
*** jtriley has joined #openstack-infra01:59
openstackgerritHuang Rui proposed openstack-infra/project-config: Move zvm-driver to stackforge-attic  https://review.openstack.org/17973802:00
*** ajmiller has joined #openstack-infra02:01
*** ivar-lazzaro has quit IRC02:03
ameadeanteaya: sent :)02:03
anteayathank you02:03
*** salv-orlando has quit IRC02:04
anteayayou sent to the third-party-announce mailing list?02:04
*** ivar-lazzaro has joined #openstack-infra02:04
ameadeanteaya: oh crap i only replied, one sec02:05
anteayayou sent to jim02:05
anteayathanks02:05
ameadeanteaya: oh i did a reply all so it should have went to that ML02:06
anteayainteresting02:06
anteayamaybe the mail server takes a minute or so02:06
anteayaI don't yet see it in my admin page02:07
anteayaso in the mean time02:07
anteayathe script is stopped?02:07
ameadeanteaya: yeah02:09
anteayagreat thank you02:10
anteayajeblair: do we need anything more from ameade before the account can be re-enabled?02:10
*** fedexo has joined #openstack-infra02:10
anteayaameade: he may be offline for the night02:10
*** signed8bit_ZZZzz has quit IRC02:11
anteayaameade: check back tomorrow and we will see where we are, thanks again for your quick response02:11
anteayareally appreciate it02:11
ameadeanteaya: that sounds good, thanks02:11
anteayadougwig: don't write a script that keeps connections open :)02:11
anteayaameade: thank you02:12
anteayadougwig: and if you write it, don't run it02:12
dougwigok, writing it right now.02:12
ameadedougwig: at least not for any stretch of time lol02:12
dougwig:)02:12
anteayatoo much fun for the day02:13
anteayaI'm off for the night02:13
anteayaenjoy your local time of day and see you tomorrow02:13
dougwignight02:13
anteayanight02:13
*** ddieterly has joined #openstack-infra02:13
*** unicell1 has quit IRC02:16
*** ddieterl_ has joined #openstack-infra02:16
*** ddieterly has quit IRC02:17
*** shashankhegde has joined #openstack-infra02:17
fungijenkins07 jjb update has finished. 04 running now02:18
*** sigmavirus24 is now known as sigmavirus24_awa02:19
*** ivar-laz_ has joined #openstack-infra02:21
*** yamahata has quit IRC02:22
fricklerso the gate pipeline is still wedged? lots of reviews with all checks finished, but no progress02:23
*** mestery has joined #openstack-infra02:23
*** dims_ has quit IRC02:24
ameadefungi: were you talking about the netapp-ci earlier when you mentioned 'half a dozen systems'? we only have two and i wanna make sure that's clear02:24
*** ivar-lazzaro has quit IRC02:24
*** otter768 has joined #openstack-infra02:25
*** gokrokve_ has joined #openstack-infra02:29
*** gokrokve_ has quit IRC02:31
*** gokrokve_ has joined #openstack-infra02:31
*** mwagner_lap has joined #openstack-infra02:32
*** gokrokve has quit IRC02:33
*** jtriley has quit IRC02:34
*** changbl has joined #openstack-infra02:35
fungiameade: yeah, one of the other cinder devs had suggested that, but was apparently also mistaken. thanks for the clarification02:39
ameadefungi: also the issue should be fixed so sorry if yall chased your tails02:40
*** tkelsey has joined #openstack-infra02:40
*** achanda has joined #openstack-infra02:40
*** baoli has joined #openstack-infra02:42
*** tkelsey has quit IRC02:45
*** otter768 has quit IRC02:46
*** Somay has quit IRC02:47
*** otter768 has joined #openstack-infra02:47
*** nelsnelson has joined #openstack-infra02:47
fungithanks again ameade!02:49
openstackgerritMerged openstack/requirements: Add python-tuskarclient to requirements  https://review.openstack.org/17423302:49
*** zz_dimtruck is now known as dimtruck02:49
fungijenkins04 wrapped, so 02 (last one) is going now02:50
*** nelsnelson has quit IRC02:52
*** gokrokve_ has quit IRC02:52
openstackgerritSamuel de Medeiros Queiroz proposed openstack-infra/devstack-gate: Remove arbitrary devstack config options  https://review.openstack.org/17998202:52
*** ivar-laz_ has quit IRC02:53
*** gokrokve has joined #openstack-infra02:59
*** sputnik13 has joined #openstack-infra03:04
*** sarob has joined #openstack-infra03:06
*** markvoelker has quit IRC03:10
*** fifieldt has joined #openstack-infra03:11
*** yamahata has joined #openstack-infra03:12
openstackgerritMerged openstack/requirements: Add python-manilaclient to global requirements.  https://review.openstack.org/13500703:15
openstackgerritMerged openstack/requirements: Add new angular deps to global-requirements.txt for Horizon  https://review.openstack.org/15118603:15
openstackgerritMerged openstack/requirements: add pycparser tracking to what-broke  https://review.openstack.org/17612003:15
*** otter768 has quit IRC03:16
openstackgerritMerged openstack-infra/project-config: Add sahara tempest job to sahara and tempest  https://review.openstack.org/16778003:18
*** ivar-lazzaro has joined #openstack-infra03:19
*** dimtruck is now known as zz_dimtruck03:20
*** fifieldt has quit IRC03:20
fungii'm passing out, but i'll reenable puppet agent on the jenkins masters tomorrow when i wake up03:22
*** dims has joined #openstack-infra03:24
*** dims has quit IRC03:30
*** jtriley has joined #openstack-infra03:31
*** fedexo has quit IRC03:34
*** jtriley has quit IRC03:36
*** tiswanso has quit IRC03:37
*** krtaylor has quit IRC03:37
openstackgerritIan Wienand proposed openstack-infra/devstack-gate: Remove arbitrary devstack config options  https://review.openstack.org/17998203:38
openstackgerritIan Wienand proposed openstack-infra/devstack-gate: [WIP] run centos7 with neutron  https://review.openstack.org/17968903:40
*** gokrokve has quit IRC03:40
*** gokrokve has joined #openstack-infra03:41
*** links has joined #openstack-infra03:41
*** Mmike has quit IRC03:42
*** frickler has quit IRC03:42
openstackgerritMerged openstack-infra/project-config: Puppet: don't run beaker jobs on previous stable branches  https://review.openstack.org/18023303:43
*** ddieterl_ has quit IRC03:44
*** Kennan2 has joined #openstack-infra03:45
*** Kennan has quit IRC03:46
*** shashankhegde has quit IRC03:50
*** baoli has quit IRC03:50
*** fedexo has joined #openstack-infra03:50
*** sdake has joined #openstack-infra03:51
*** camunoz has quit IRC03:53
*** coolsvap|afk is now known as coolsvap03:53
*** patrickeast has joined #openstack-infra03:54
*** camunoz has joined #openstack-infra03:54
*** krtaylor has joined #openstack-infra03:54
openstackgerritMerged openstack/requirements: Update python-cinderclient to 1.2.0  https://review.openstack.org/17727003:56
openstackgerritMerged openstack/requirements: Add contextlib2 lib to aid in py2/py3 compatability  https://review.openstack.org/16422203:56
*** sdake_ has joined #openstack-infra03:56
*** Mmike has joined #openstack-infra03:57
*** Mmike has quit IRC03:57
*** Mmike has joined #openstack-infra03:57
*** sdake has quit IRC04:00
sdake_jeblair thanks :)04:00
*** sdake has joined #openstack-infra04:01
*** adrian_otto has joined #openstack-infra04:03
*** sdake__ has joined #openstack-infra04:03
adrian_ottoI created a branch in gerrit that I want to remove, but I'm not sure how. Any ideas?04:03
*** sdake_ has quit IRC04:04
*** pal has joined #openstack-infra04:06
*** sdake has quit IRC04:07
*** yfried|afk is now known as yfried04:08
*** changbl has quit IRC04:10
*** markvoelker has joined #openstack-infra04:11
*** sdake__ is now known as sdake04:13
*** gokrokve has quit IRC04:15
*** markvoelker has quit IRC04:16
*** cfriesen has joined #openstack-infra04:17
*** cfriesen has left #openstack-infra04:18
*** gokrokve has joined #openstack-infra04:18
openstackgerritMerged openstack-dev/pbr: Issue #1451976: handle commits with non-utf8 text  https://review.openstack.org/18026504:20
openstackgerritAdrian Otto proposed openstack-infra/project-config: Add delete to ACL rights for magnum  https://review.openstack.org/18040104:21
*** bhunter71 has quit IRC04:25
*** sks has joined #openstack-infra04:27
*** mgedmin has quit IRC04:30
*** mgedmin has joined #openstack-infra04:31
*** adrian_otto has quit IRC04:32
*** salv-orlando has joined #openstack-infra04:35
*** salv-orlando has quit IRC04:38
*** ddieterly has joined #openstack-infra04:45
*** abregman has joined #openstack-infra04:48
*** ddieterly has quit IRC04:50
*** btully has joined #openstack-infra04:50
*** mrmartin has joined #openstack-infra04:50
*** patrickeast has quit IRC04:56
*** sarob has quit IRC04:56
*** gokrokve_ has joined #openstack-infra05:01
*** adrian_otto has joined #openstack-infra05:03
*** camunoz has quit IRC05:03
*** harlowja_at_home has joined #openstack-infra05:03
*** gokrokve has quit IRC05:03
*** yfried has quit IRC05:04
*** samueldmq has quit IRC05:05
*** gokrokve_ has quit IRC05:05
*** btully has quit IRC05:07
*** ivar-lazzaro has quit IRC05:08
*** ivar-lazzaro has joined #openstack-infra05:08
*** Swami has joined #openstack-infra05:11
*** hdd has joined #openstack-infra05:11
*** ivar-lazzaro has quit IRC05:11
*** markvoelker has joined #openstack-infra05:12
*** ivar-lazzaro has joined #openstack-infra05:12
*** deepakcs has joined #openstack-infra05:15
*** camunoz has joined #openstack-infra05:15
*** markvoelker has quit IRC05:16
*** crc32 has quit IRC05:17
*** mrmartin has quit IRC05:17
*** otter768 has joined #openstack-infra05:17
*** SumitNaiksatam has quit IRC05:19
*** jtriley has joined #openstack-infra05:20
*** SumitNaiksatam has joined #openstack-infra05:20
*** otter768 has quit IRC05:22
*** sdake_ has joined #openstack-infra05:23
*** ildikov has quit IRC05:25
*** jtriley has quit IRC05:26
*** sdake has quit IRC05:27
*** BharatK has joined #openstack-infra05:27
*** shashankhegde has joined #openstack-infra05:32
*** subscope_ has joined #openstack-infra05:35
*** ildikov has joined #openstack-infra05:38
AJaegergood morning05:39
openstackgerritSergey Kolekonov proposed openstack-infra/project-config: Add apps-catalog-ci group for apps-catalog project  https://review.openstack.org/18020105:45
*** Somay has joined #openstack-infra05:45
*** adrian_otto has quit IRC05:45
*** gokrokve has joined #openstack-infra05:49
*** niska has quit IRC05:49
*** markvoelker has joined #openstack-infra05:51
*** frickler has joined #openstack-infra05:52
*** markvoelker has quit IRC05:55
*** rlandy has joined #openstack-infra05:57
deepakcsAJaeger, morning, saw ur comment in https://review.openstack.org/#/c/173408/ - looks like using the new parameterised job template to skip tests is the reccomended apporach05:57
*** SumitNaiksatam has quit IRC05:57
*** Somay has quit IRC05:57
deepakcsAJaeger, thanks for helping out!05:58
AJaegerdeepakcs, hope the others agree - that'S what we discsused last night here at least ;)05:58
AJaegerdeepakcs, you're welcome05:58
deepakcsAJaeger, lets hope :) this issue has seen many approaches/iterations till now05:58
AJaeger;)05:58
*** Somay has joined #openstack-infra05:58
AJaegerclarkb, fungi, jeblair : loquacities released last night (around 3:00 UTC) version 0.26 of openstack-doc-tools but this one does not show up on pypi. What needs to be done to get it up there?06:00
AJaegercinerama, StevenK, pleia2 Hi!06:00
*** SumitNaiksatam has joined #openstack-infra06:01
cineramagreetings06:01
StevenKAJaeger: Hai!06:01
AJaegerpleia2, are you around?06:02
*** subscope_ has quit IRC06:02
*** niska has joined #openstack-infra06:03
*** Swami has quit IRC06:03
AJaegeris there any way to ping pleia2 or shall we start without her?06:03
cineramahmm06:03
*** fedexo has quit IRC06:04
*** masayukig_ has quit IRC06:04
cineramamaybe let's go ahead and get started06:04
cineramabit late here06:04
cineramaso i should probably go over our status06:05
AJaegercinerama, I have no idea how far are you and where my input is needed, so please go ahead ;)06:05
*** Longgeek has joined #openstack-infra06:05
*** isviridov_away is now known as isviridov06:05
*** camunoz has quit IRC06:05
cineramaso right now we have translate-dev.openstack.org up and running for us to play around with a bit. so all the puppetizing, etc on the server side is done modulo any hiccups we discover later06:06
cineramaand we have puppetizing for installing the client on the proposal slaves06:06
*** jcoufal has joined #openstack-infra06:06
cineramaso the next big piece is the proposal scripts06:06
cineramawhich i have a draft of almost ready to land06:06
StevenK\o/06:06
AJaegergreat!06:07
cineramaoh by the way, you can have a look at notes and stuff here: https://etherpad.openstack.org/p/zanata-install06:07
cineramathe proposal scripts are going to need some love from someone who has a good grasp of our process on this...i'm coming mostly from the zanata side06:07
AJaegercinerama, that looks involved ;(06:08
cineramain the true spirit of software development i've thrown away more than i've actually written06:08
*** masayukig_ has joined #openstack-infra06:08
cineramas/written/intend to actually ship/06:08
StevenKI have been digging into matching up locales that we already have in OpenStack projects to the list of what languages are up on translate-dev: http://paste.openstack.org/show/215326/06:08
cineramaoh cool06:08
AJaegerI should have a good grasp on the infra side but not sure what I would be signing up for ;)06:08
StevenKI've read through the scripts, I think I get the push side06:08
*** mrunge has joined #openstack-infra06:08
cineramaStevenK: looks like i should add some manual mappings as per your findings06:08
cineramaso there06:09
StevenKBut I think we also need a script that pre-populates based on what we have06:09
cineramas a list of bugs in that etherpad, which i will add to tomorrow06:09
AJaegercinerama, did you look at all langauges in *transifix*? There are 70+ languages in use06:09
StevenKYeah. translate-dev has 3506:09
AJaegersorry, StevenK06:09
StevenK*3406:09
cineramaStevenK: prepopulates the zanata server?06:09
StevenKcinerama: So this whole mapping thing is just silly, IMO06:10
*** salv-orlando has joined #openstack-infra06:10
AJaegerStevenK, so, we need to get all 70+ languages out from transifex and into zanata06:10
cineramaStevenK: oh the mapping from zanata locale to locale we're using?06:10
StevenKOther way round, but yes06:10
AJaegercinerama, we can rename languages also in the projects, let's make this a *one time* mapping06:10
StevenKI think we should fix zanata06:10
StevenKI can generate a list of languages to add and ones to remove06:11
cineramaso one of the things my stuff does is to generate the zanata.xml for the project on the fly. part of that file can store the mappings between their locales and ours so we don't need to change06:11
StevenKAnd then feed that to cinerama or pleia206:11
AJaegerStevenK, whatever works but I don't want to have a mapping as part of pushing/pulling every time.06:11
cineramait seems like the way they want the zanata.xml to be is to have it just live in the project but it is okay to generate it since we can pull down most of it from the server06:11
StevenKzanata can do the mapping as part of the push/pull, but I'd prefer the locale directories match what is in zanata06:11
AJaegercinerama, if that mapping is easier, it's fine - but IMO we shouldn't need it06:11
cineramai don't have an opinion on whether we should make the projects change or whether we handle it06:12
pleia2apologies, had a meeting elsewhere that ran late06:12
StevenKWe have some projects that don't match -- as it were. Some use tr and others use tr_TR, for example06:12
AJaegerThe use case for a generated zanata.xml is that a nwe language gets added on the server, it needs to be added to the project without manual adding.06:12
cineramathe current issue with handling those mappings in zanata is that the pull will also create the directory with the zanata locale name; imo this is a bug, which i have filed with them06:13
AJaegerpleia2, glad to have you with us!06:13
*** scheuran has joined #openstack-infra06:13
cineramaAJaeger: that case you mention *should* work - if the language is in the project, it will get pulled down06:13
StevenKcinerama: Which is my point about fixing the language list on zanata06:13
AJaegercinerama, great.06:13
*** camunoz has joined #openstack-infra06:13
*** HeOS has quit IRC06:14
cineramai haven't gotten a response yet about my bug w/the pull but i just filed it this afternoon06:14
AJaegercinerama, StevenK, if we have any mismatches between locales, let's clean those up when there's no reason for it.06:14
*** tnovacik has joined #openstack-infra06:14
StevenKAJaeger: When is there no reason for them? :-)06:14
AJaegerI'm fine with pushing a change to 20 projects renaming language files from tr to tr_TR if needed to make our tooling easier ;)06:14
*** ivar-laz_ has joined #openstack-infra06:15
cineramaso other than that - one of the big issues was with the minimum percentages. AJaeger alerted me to the fact that we can just use the method for cleaning up files with unacceptable percentages using the method we have now to clean up po files06:15
*** ddieterly has joined #openstack-infra06:15
cineramaugh that's a bad sentence, but hopefully you know what i mean06:15
*** ivar-lazzaro has quit IRC06:15
AJaegerStevenK, we have en_US and en_GB since those are different, and if we have tr and tr_TR because they are different, it's fine.06:15
AJaegerBut if tr is actually tr_TR, let's rename06:15
AJaegerStevenK, like we shouldn't rename en_AU to en_US ;)06:16
StevenKAJaeger: Absolutely06:16
cineramaanyway the zanata folks have marked the percentages bug as high priority. i'll grab the link for a sec because i would like someone who knows transifex better to respond to them06:16
StevenKFor example, zanata has es-mx, and we use es_MX. I'd prefer that we just add es_MX to zanata06:17
cineramahttps://bugzilla.redhat.com/show_bug.cgi?id=121527406:17
openstackbugzilla.redhat.com bug 1215274 in Component-zanata-client "Should be able to specify minimum percentage completion on pull" [High,New] - Assigned to djansen06:17
cineramathis bug06:17
cineramaStevenK: one of the biggest mapping issues is zh_* iirc06:17
StevenKzh_* isn't in zanata at all06:17
pleia2cinerama: ah thanks, I was just looking for it in my bug mail06:18
AJaegercinerama, StevenK: We can ask Daisy from the translation team for help, especially with zh_*06:18
StevenKThose two look fine, it's just zh_TW and zh_CN, there is no bare zh06:18
cineramaoh that's right, they're all zh-Hans-*06:19
*** pal has quit IRC06:19
cineramathat is the weirdness06:19
cineramaor at least that is what i get when i search06:19
AJaegerplease ask Daisy for these06:19
cineramaso more mapping06:19
cineramaum anyway06:19
*** pal has joined #openstack-infra06:19
cineramaso we should be able to basically do what we need to but there is going to be some weirdness06:20
pleia2added the mapping bug to the etherpad06:20
*** Ala has joined #openstack-infra06:20
*** pal has joined #openstack-infra06:20
*** ddieterly has quit IRC06:20
cineramai kind of got on a long side quest about calculating percentages per file and then pulling down each individual file06:20
cineramabut i don't think we're going to use that06:20
StevenKcinerama: Searching https://translate-dev.openstack.org/language/list for zh gives me nothing06:21
*** Longgeek has quit IRC06:21
AJaegerRegarding the bug: WE can either do the following:06:21
*** Longgeek has joined #openstack-infra06:21
AJaeger1) Do everything on our side as post-processing and close that bug as not-needed-anymore06:21
cineramaStevenK: weird. when i go to the "add new language" thingy i get zh-Hans-CN zh-Hans-HK etc06:21
StevenKI have no add new language, but I'm no admin.06:22
AJaeger2) Do it on project-level and do post-processing for openstack-manuals since that's the only odd one06:22
AJaeger3) Do it on document level like transifex does today06:22
cineramaAJaeger: so i think it is in their interests as well as ours to have feature parity w/transifex06:22
cineramaAJaeger: oh so with transifex it is per document?06:22
AJaegercinerama, with transifex it'S per document06:22
*** Longgeek has quit IRC06:23
cineramaAJaeger: that was what i was asking about earlier :)06:23
*** Longgeek has joined #openstack-infra06:23
AJaegerLet me write up "feature parity with transifex" in this part and send you as email.06:23
AJaegerThat will give us the most flexibility.06:23
AJaegerCould one of you then add to redhat bugzilla, please? I don't have an account there currently...06:23
AJaegerBut if more discussion is needed, I can create one...06:24
*** camunoz has quit IRC06:24
*** yfried has joined #openstack-infra06:24
StevenKHmmm, I think my RedHat bugzilla account is dead because it was @canonical.com, but that will require archeology06:25
cineramaso to do that ourselves, we have to use the rest api to grab the translations and then assemble the po files. the python client they have, while not currently maintained (i think?) has an example of how to do this06:25
pleia2AJaeger: please do, you need an account to comment on bugs too, which we've had to do some as they seek more information06:25
pleia2and zanata upstream is very committed to prioritizing our bugs, so make sure we add them to the bug section of the etherpad so we can keep track and bring them to their attention06:26
AJaegerpleia2, let me still send around my answer for review.06:26
*** Somay has quit IRC06:26
AJaegerpleia2, ok06:26
*** gokrokve_ has joined #openstack-infra06:27
*** ildikov has quit IRC06:27
pleia2it's several emails a day, but I also subscribed to https://www.redhat.com/mailman/listinfo/zanata-bugs to keep an eye on things (both our bugs and other things that come along which may impact us)06:28
*** gokrokve has quit IRC06:28
cineramaso aside from the zanata.xml generation, a lot of the stuff for the proposal scripts could be in flux depending on timelines for the bugs getting resolved vs when it's desirable to deploy For Real(tm)06:28
AJaegerwe can start anytime with *pushing* to zanata like we push to transifex06:29
StevenKYup06:29
AJaegerThat will show us whether that part works fine06:29
StevenKAnd we should do that first06:29
cineramaagreed, the push changes very little06:29
StevenKExcept for the whole missing languages thing06:29
cineramaStevenK: feel free to try your shiny new admin powers06:29
StevenKGASP06:30
StevenKWhen did I get those?06:30
AJaegerAnd then I suggest to pick a single project and move that over to zanata06:30
cineramaStevenK: heh heh06:30
AJaegerONce that's done, do the rest...06:30
cineramaAJaeger: that sounds like a pretty sensible way of going about it06:30
AJaegerAll of that needs to be properly scheduled, so we need some "project plan" for it.06:31
cineramawell, we should be able to push as soon as we get the push script adapted, at least to the dev instance06:31
*** Somay has joined #openstack-infra06:31
StevenK"Underscores should be replaced with dashes."06:31
AJaegerHorizon and openstack-manuals are the projects with most translations. So, we should ask Daisy and translation team whether we start with Horizon/openstack-manuals, or with a project with far fewer strings like nova06:31
StevenKHo hum06:31
*** camunoz has joined #openstack-infra06:32
AJaegerStevenK, that would be a simple mapping, replace dashes with underscores when pushing/pulling06:32
StevenKRight, so Zanata doesn't like underscores06:32
cineramaAJaeger: so i had been using nova for my personal tests, but in a way a complicated project will turn up more bugs06:32
StevenKAJaeger: We can't do that currently, it isn't quite that simple.06:32
AJaegeropenstack-manuals is the most complicated ;)06:33
*** harlowja_at_home has quit IRC06:33
cineramai am generating simple mappings like that for the zanata.xml now06:33
AJaegerStevenK, that's sad06:33
StevenKzh-cn would work, since we can transform zh_CN by .lower() and then replacing _ with -06:33
*** dhp has quit IRC06:33
StevenKBut there isn't a consistency in the current list either06:34
StevenKYou have en-gb and en-AU06:34
StevenKMy CDO is twitching06:34
*** dhp has joined #openstack-infra06:34
pleia2heh06:34
*** notnownikki has joined #openstack-infra06:34
StevenKCan I declare lower case with dashes and bend zanata to my will?06:35
openstackgerritKai Qiang Wu(Kennan) proposed openstack-infra/project-config: Add new branch for channels work  https://review.openstack.org/18041606:35
*** ivar-lazzaro has joined #openstack-infra06:35
cineramathey look like this06:35
cinerama<locale map-from="en_AU">en-AU</locale>06:35
*** mgedmin has quit IRC06:36
cineramaso the mapping can be pretty arbitrary06:36
StevenKcinerama: Yup, but we need to store that information somewhere.06:36
*** wayneeseguin has quit IRC06:36
AJaegercinerama, can't we change this to en-au in zanata? WE should be able to enforce some conventions that might make life easier06:36
StevenKExactly my point06:36
*** jyuso has joined #openstack-infra06:36
*** ivar-laz_ has quit IRC06:36
cineramaStevenK: the arbitrary mappings? i agree. i had been generating simple ones06:36
cineramaStevenK: but some, like the zh ones, are less obvious06:37
cineramaStevenK: we could maintain a list of the arbitrary mappings for the xml generation to rely on06:37
cineramaor of course we can change the projects06:37
*** wayneeseguin has joined #openstack-infra06:38
cineramaoh WOW06:38
cineramai can add en-gb *and* en-GB06:38
*** alex_xu has quit IRC06:38
cineramanot sure that is a good idea06:39
StevenKcinerama: http://i.stack.imgur.com/jiFfM.jpg06:39
cineramaStevenK: actual LOL06:39
*** jyuso1 has quit IRC06:39
AJaeger:)06:40
*** wayneeseguin has quit IRC06:40
*** sc` has quit IRC06:40
cineramaso...that's worth filing a bug about i think06:40
StevenKAgreed06:40
*** sc` has joined #openstack-infra06:40
StevenKHuh, you can't remove languages?06:41
cineramaso it sounds like we need to 0) sort out the locales mess enough that we're happy 1) get push going 2) start with 1 project ...06:41
cineramaStevenK: aaaaand there's another bug to file06:41
*** tteggel has quit IRC06:41
*** bkero has quit IRC06:41
StevenKBecause Slovenian is 'sl-'06:41
pleia2cinerama: yep, that's how I see it06:41
cineramaStevenK: yeah i saw that, that's what it suggested06:41
*** mgedmin has joined #openstack-infra06:41
*** yamahata has quit IRC06:41
*** tteggel has joined #openstack-infra06:41
StevenKSad face06:41
cineramathey are going to need to sort out some of this stuff on the locales end06:42
cineramaStevenK: you could do it in the database trololo06:42
*** alex_xu has joined #openstack-infra06:42
StevenKMapping it back to GNU, sl_ makes NO SENSE06:42
cineramaso i added that but i didn't do it, i swear06:42
cineramathat is what it suggested06:42
cineramain the little drop down thing06:42
*** bkero has joined #openstack-infra06:42
cineramabug #306:43
openstackbug 3 in Launchpad itself "Custom information for each translation team" [Low,Fix released] https://launchpad.net/bugs/306:43
cineramassh no06:43
StevenKsl or sl_SL (normalized to sl-sl)06:43
StevenKHaha06:43
*** tkelsey has joined #openstack-infra06:43
*** masayukig_ has quit IRC06:44
cineramaso do we each have some "action items"?06:44
* AJaeger has the AI to answer the Red Hat bug06:45
cineramame: put up xml generation & pull stuff06:45
pleia2once we have the new bugs added to the etherpad, I'll email carlos to make sure his team is in the loop with our latest collection ;)06:45
StevenKcinerama: I think I will continue to look at locales06:46
AJaegercould somebody write up the next steps in the etherpad?06:46
StevenKCome up with a list to add and a list to remove when we can06:46
cineramaright and StevenK will look at locales06:46
cineramapleia2: would you like to update/file bugs?06:47
*** tkelsey has quit IRC06:47
pleia2cinerama: I won't get to it until tomorrow, but I can give it a go tomorrow06:48
cineramathey are updating our bugs as we speak which is very cool :)06:48
*** masayukig_ has joined #openstack-infra06:48
pleia2I'll let you all know if I need more info for filing them06:48
AJaegergreat!06:48
*** wayneeseguin has joined #openstack-infra06:48
*** yamahata has joined #openstack-infra06:48
cineramapleia2: cool. i just want someone less sleepy to write it :)06:48
*** viktors|afk is now known as viktors06:48
cineramadoes anyone have anything else?06:48
cineramapleia2: that is probably going to wind up being one of Future Us06:49
*** jcoufal_ has joined #openstack-infra06:49
pleia2cinerama: hehe, indeed06:50
cineramathis reminds me of the seinfeld thing about Day Guy vs Night Guy06:50
cineramawhere Night Guy thinks it's a good idea to stay up late and watch a movie or whatever, resulting in Day Guy cursing him yet again06:51
*** soren has joined #openstack-infra06:51
pleia2sounds about right06:51
AJaeger;)06:52
AJaegercinerama, pleia2: Thanks for staying up late for this discussion!06:52
*** markvoelker has joined #openstack-infra06:52
cineramaanyway meeting adjourned, thanks everyone for their time06:52
* flaper87 drops this for AJaeger and pleia2 https://review.openstack.org/#/c/180333/06:52
*** jcoufal has quit IRC06:52
*** wayneeseguin has quit IRC06:53
pleia2thanks all06:53
AJaegerThanks everybody, have a great night or day!06:53
deepakcsAJaeger, good night06:53
AJaegerfungi, clarkb, jeblair: None of the daily proposal jobs run so far, they should have started 45 mins ago and systems seem idle06:53
AJaegerdeepakcs, thanks - but it's 8am for me ;)06:53
pleia2flaper87: alas, too sleepy, if no one else gets to it I'll try to have a look in my morning :)06:53
*** asselin__ has quit IRC06:53
deepakcsAJaeger, ah , i thought its night06:54
*** shardy_z is now known as shardy06:54
*** asselin__ has joined #openstack-infra06:54
AJaegerflaper87, I'll check - had my morning green tea already ;)06:54
flaper87pleia2: ah no worries, have a great night :)06:54
flaper87they are easy patches06:54
flaper87at least the first one06:54
flaper87AJaeger: thank YOU!!!!!06:54
*** hdd has quit IRC06:56
*** markvoelker has quit IRC06:56
StevenKAJaeger, cinerama, pleia2: Thanks!06:57
AJaegerYeah, the first one is easy, the second one needs more time ;)06:57
*** GheRivero_ has joined #openstack-infra07:02
AJaegerThanks, StevenK !07:02
flaper87AJaeger: replied :)07:02
AJaeger;)07:03
AJaegerflaper87, +2 ed ;)07:03
flaper87AJaeger: thank you!07:04
*** hdd has joined #openstack-infra07:06
*** achanda has quit IRC07:07
*** mpavone has joined #openstack-infra07:08
waynri have successfully generated jenkins xml using pure python classes to produce the necessary data structure for the XmlBuilder in JJB07:08
* waynr dances07:08
* AJaeger applaudes waynr 07:08
*** achanda has joined #openstack-infra07:11
waynri've also made tons of progress in making various JJB modules and classes usefully useable as a library07:11
*** sabeen has quit IRC07:11
*** achanda has quit IRC07:11
waynrand now time for bed07:11
*** shashankhegde has quit IRC07:13
*** GheRivero_ has quit IRC07:13
*** frickler_ has joined #openstack-infra07:14
*** frickler_ has quit IRC07:14
openstackgerritAndreas Jaeger proposed openstack-infra/project-config: Optionally pass regex to skip to tempest-dsvm-full-plugin jobs  https://review.openstack.org/18030907:15
AJaegersleep well, waynr !07:16
*** ddieterly has joined #openstack-infra07:16
*** ildikov has joined #openstack-infra07:16
*** ddieterly has quit IRC07:20
*** woodster_ has quit IRC07:20
*** pal has quit IRC07:22
*** dizquierdo has joined #openstack-infra07:24
*** markus_z has joined #openstack-infra07:27
*** gsagie has joined #openstack-infra07:28
openstackgerritJulien Danjou proposed openstack/requirements: Bump pbr to 0.11  https://review.openstack.org/18042307:30
*** sergsh_ has joined #openstack-infra07:31
*** sergsh has joined #openstack-infra07:31
*** samuelBartel has joined #openstack-infra07:34
*** panda|Zz is now known as panda|bfast07:38
*** _nadya_ has joined #openstack-infra07:41
*** hdd_ has joined #openstack-infra07:45
*** jlanoux has joined #openstack-infra07:46
*** hdd has quit IRC07:46
*** panda|bfast is now known as panda07:47
*** panda is now known as Guest5601807:47
*** Guest56018 is now known as panda07:48
*** HeOS has joined #openstack-infra07:49
*** gsagie has quit IRC07:51
AJaegerflaper87, could you import translations again, please? https://review.openstack.org/#/c/176633/ and https://review.openstack.org/#/c/179630/ are open07:51
*** hdd_ has quit IRC07:57
*** jistr has joined #openstack-infra07:59
*** hdd has joined #openstack-infra07:59
*** hasharMeeting is now known as hashar08:01
hasharjeblair: thank you for the nodepool and gear tags!08:01
*** Somay has joined #openstack-infra08:03
ajohi :)08:05
*** hdd has quit IRC08:05
ajoI have two accounts in gerrit, one with an old email (which I used for 10 days..) and one with my current email (2 years)08:05
*** hdd has joined #openstack-infra08:05
ajois it possible to delete the old one?08:06
ajoI have people sending me review requests on the old one08:06
*** dizquierdo has quit IRC08:06
*** ociuhandu has joined #openstack-infra08:09
*** _nadya_ has quit IRC08:12
*** hdd has quit IRC08:13
*** hdd has joined #openstack-infra08:13
*** yfried is now known as yfried|afk08:15
openstackgerritJerome Hourquebie proposed stackforge/python-jenkins: Adds Cloudbees folder plugin support  https://review.openstack.org/18018508:16
*** ddieterly has joined #openstack-infra08:17
*** Hal has joined #openstack-infra08:17
*** Hal is now known as Guest5505608:17
*** salv-orlando has quit IRC08:18
*** salv-orlando has joined #openstack-infra08:18
*** yfried|afk is now known as yfried08:20
*** jtomasek has joined #openstack-infra08:20
*** ddieterly has quit IRC08:21
jtomasekhey, can aynone please help me with openstack/tuskar-ui gate? the tests won't get run at all https://review.openstack.org/#/c/180211/08:22
*** e0ne has joined #openstack-infra08:22
*** ivar-lazzaro has quit IRC08:23
*** sputnik13 has quit IRC08:26
*** sputnik13 has joined #openstack-infra08:29
*** ssam2 has joined #openstack-infra08:31
*** dguitarbite has joined #openstack-infra08:32
*** MaxV has joined #openstack-infra08:34
openstackgerritAndrew Smith proposed openstack-infra/jenkins-job-builder: Add support for docker builder plugin  https://review.openstack.org/17898008:35
*** _nadya_ has joined #openstack-infra08:36
*** ildikov has quit IRC08:41
*** e0ne has quit IRC08:43
*** Somay has quit IRC08:44
*** dbite has joined #openstack-infra08:44
*** tkelsey has joined #openstack-infra08:44
*** sputnik13 has quit IRC08:46
*** sputnik13 has joined #openstack-infra08:48
openstackgerritDavide Guerri proposed openstack-infra/shade: Add floating IP pool resource methods  https://review.openstack.org/18025508:51
openstackgerritDavide Guerri proposed openstack-infra/shade: Add port resource methods  https://review.openstack.org/17996808:51
openstackgerritDavide Guerri proposed openstack-infra/shade: Add keystone service resource methods  https://review.openstack.org/17762108:51
*** MaxV has quit IRC08:52
openstackgerritDavide Guerri proposed openstack-infra/shade: Add keystone endpoint resource methods  https://review.openstack.org/18032808:52
*** sputnik13 has quit IRC08:53
*** ildikov has joined #openstack-infra08:56
*** jtriley has joined #openstack-infra08:57
*** jtomasek has quit IRC08:58
*** smccully has quit IRC08:58
*** russellb has quit IRC08:58
*** trey has quit IRC08:58
*** gema has quit IRC08:58
*** adreznec has quit IRC08:58
*** mfink- has quit IRC08:58
*** SotK has quit IRC08:58
*** tsufiev has quit IRC08:58
*** mrda has quit IRC08:58
*** ianw has quit IRC08:58
*** shardy_ has joined #openstack-infra08:59
*** shardy has quit IRC09:00
*** fhubik has joined #openstack-infra09:00
*** jtomasek has joined #openstack-infra09:00
*** smccully has joined #openstack-infra09:00
*** russellb has joined #openstack-infra09:00
*** trey has joined #openstack-infra09:00
*** gema has joined #openstack-infra09:00
*** adreznec has joined #openstack-infra09:00
*** mfink- has joined #openstack-infra09:00
*** SotK has joined #openstack-infra09:00
*** tsufiev has joined #openstack-infra09:00
*** mrda has joined #openstack-infra09:00
*** ianw has joined #openstack-infra09:00
*** fhubik has quit IRC09:00
*** fhubik has joined #openstack-infra09:00
*** jtriley has quit IRC09:02
*** shardy_ has quit IRC09:04
*** shardy has joined #openstack-infra09:05
*** freerunner has joined #openstack-infra09:06
*** _nadya_ has quit IRC09:10
*** redixin has joined #openstack-infra09:13
*** cdent has joined #openstack-infra09:16
*** ddieterly has joined #openstack-infra09:17
*** otter768 has joined #openstack-infra09:19
*** freerunner has quit IRC09:20
openstackgerritDavide Guerri proposed openstack-infra/shade: Add floating IP pool resource methods  https://review.openstack.org/18025509:20
*** ddieterly has quit IRC09:22
*** otter768 has quit IRC09:23
*** e0ne has joined #openstack-infra09:25
*** pblaho has joined #openstack-infra09:25
*** hdd_ has joined #openstack-infra09:27
*** freerunner has joined #openstack-infra09:28
*** hdd has quit IRC09:28
ams__Hi, I just wanted to check if there's anything else I need to do on this review - https://review.openstack.org/#/c/178980/ I see the +1 from Yolanda. Not entirely clear if I have to do anything else at this point.09:34
yolandaams_, well, i'm not core there, so it's just an opinion09:35
yolandayou need to wait for core reviews09:35
*** redixin has left #openstack-infra09:36
*** fhubik is now known as fhubik_afk09:41
*** _nadya_ has joined #openstack-infra09:41
*** fhubik_afk is now known as fhubik09:41
*** fhubik is now known as fhubik_afk09:42
ams__yolanda: ah ok, so I just have to wait. Cool, thanks!09:43
yolandayes, it needs more feedback from other people09:43
*** zhiwei has quit IRC09:45
*** tnovacik has quit IRC09:49
*** macjack has joined #openstack-infra09:51
*** Longgeek has quit IRC09:53
openstackgerritDavid Caro proposed openstack-infra/jenkins-job-builder: Added parallelization options  https://review.openstack.org/18046309:54
*** fhubik_afk is now known as fhubik09:56
*** e0ne is now known as e0ne_09:59
openstackgerritMerged openstack-infra/project-config: Make zaqarclient functional gate non-voting  https://review.openstack.org/18033210:03
*** freerunner has quit IRC10:03
openstackgerritMerged openstack-infra/project-config: Extend Zaqar's gate jobs  https://review.openstack.org/18033310:04
*** freerunner has joined #openstack-infra10:04
*** e0ne_ is now known as e0ne10:05
*** freerunner has quit IRC10:05
sdagueI pushed a bunch of rechecks into the queue to hopefully lessen the pain on the fires yesterday10:07
*** dims has joined #openstack-infra10:07
sdaguethere were a remarkable number of changes in gerrit where there was no jenkins vote on the current patch10:08
*** hdd_ has quit IRC10:11
*** jtriley has joined #openstack-infra10:11
*** hdd_ has joined #openstack-infra10:12
*** hdd__ has joined #openstack-infra10:15
*** hdd_ has quit IRC10:16
*** jtriley has quit IRC10:16
*** fhubik is now known as fhubik_afk10:17
*** ddieterly has joined #openstack-infra10:18
*** ddieterly has quit IRC10:23
rakhmerovhi, we need help with stackforge/python-mistralclient tags. We now have several tags starting with 2015.1 (done by mistake) and now we can't user normal versions like 0.2.0 because looks like pbr checks that versions are not rolling back (it looks at git history)10:26
rakhmerovsetting version = 0.2 in setup.cfg doesn't help either10:26
rakhmerovany recommendations on this?10:27
*** lennyb has joined #openstack-infra10:28
rakhmerovthe only solution I see is to remove tags 2015.1.x at all but I heard it's not so easy to do10:28
*** Longgeek has joined #openstack-infra10:35
*** coolsvap is now known as coolsvap|afk10:38
*** claudiub has joined #openstack-infra10:38
*** pc_m has joined #openstack-infra10:42
openstackgerritFabien Boucher proposed openstack-infra/puppet-zuul: Attempt to add an acceptance test with beaker for puppet-zuul  https://review.openstack.org/18021410:45
AJaegerfungi, clarkb, jeblair: The post jobs like heat-upstream-translation-update get a NOT_REGISTERED - looks related to the daily proposals10:46
*** yfried has quit IRC10:50
rakhmerovfungi, jeblair: can you comment on tagging problem?10:51
*** yfried has joined #openstack-infra10:55
*** freerunner has joined #openstack-infra10:56
*** ildikov has quit IRC10:57
*** dizquierdo has joined #openstack-infra11:02
*** freerunner has quit IRC11:03
fungiAJaeger: if we added those jobs in the past day, puppet is not yet reenabled on the jenkins masters so they don't have it yet. about to check them over and reenable11:06
AJaegerfungi, these are ancient jobs - on jenkins.openstack.org11:07
fungisdague: yeah, there were several unrelated incidents yesterday (between zuul emergency restarts and the gerrit stream-events problem) where we lost change events11:08
*** fhubik_afk is now known as fhubik11:08
AJaegerfungi, the job I mentioned is now on jenkins.openstack.org11:09
fungiAJaeger: odd. i'll look into it as soon as i can11:09
AJaegerfungi, thanks.11:09
sdaguefungi: yeh, and all the gerrit event stream dies over the last couple of months11:09
sdaguesome of these went back a ways11:10
AJaegerfungi, https://jenkins.openstack.org/view/All/job/sahara-upstream-translation-update/ - and status.openstack.org shows NON_REGISTERED for this job11:10
AJaeger(see post jobs for sahara with the red light)11:10
sdaguethere were about 100 reviews without a jenkins vote11:10
*** freerunner has joined #openstack-infra11:10
fungirakhmerov: tags can't really be deleted. they are forever. your options are to continue releasing versions later than your highest tags or stop using pbr or start a new project11:10
fungisdague: not at all surprising11:11
AJaegerfungi, post and proposal job shouldn't keep you from your coffee - just wanted to point it out while I see it11:12
rakhmerovfungi: that s..cks ) Ok, thanks. I got it11:12
fungirakhmerov: or another option is to get involved with pbr development... submit a patch which allows you to configure a list of tags to ignore or something11:13
rakhmerovfungi: yeah, ok11:14
rakhmerovfungi: one more question11:14
rakhmerovgiven that mistral is still in stackforge can we do something while moving it to openstack/ ? Like ignoring python-mistralclient at all and uploading it into openstack/python-mistralclient from scratch?11:15
rakhmerovor removing the tags again11:15
*** ildikov has joined #openstack-infra11:16
fungirakhmerov: that would make it a little easier to prevent from impacting our infrastructure, but doesn't solve the problem of local clones developers/end users might be maintaining. the ultimate problem is that tag deletion doesn't register on a remote update, so everyone would need to know to reclone or to manually delete local copies of those tags11:17
fungirather than just changing their git remote after the rename (which would not get rid of those stray tags)11:18
rakhmerovfungi: yeah, I see, you're right11:18
*** ddieterly has joined #openstack-infra11:19
*** otter768 has joined #openstack-infra11:20
*** isviridov is now known as isviridov_away11:20
openstackgerritFabien Boucher proposed openstack-infra/puppet-zuul: Attempt to add an acceptance test with beaker for puppet-zuul  https://review.openstack.org/18021411:22
*** ddieterly has quit IRC11:23
*** freerunner has quit IRC11:24
*** otter768 has quit IRC11:24
sdaguefungi: it's also nice to see what the gate looks like now that we're working well. recheck of 100 patches burst us just about to quota, but are all flushed out now11:26
*** yfried is now known as yfried|afk11:27
*** boris-42 has joined #openstack-infra11:27
openstackgerritRenat Akhmerov proposed openstack-infra/project-config: Rename stackforge/mistral to openstack/mistral  https://review.openstack.org/17532811:27
fungisdague: yeah, it's too bad every day can't be a dance party like that11:28
*** BobBall has quit IRC11:29
*** yamahata has quit IRC11:30
fungiAJaeger: looks like none of the upstream-translation-update jobs reregistered in zuul after it was restarted. probably a problem with jenkins.o.o or at least the proposal worker. i'll see what i need to do to get that reengaged momentarily11:30
sdaguefungi: https://review.openstack.org/#/c/165240 - another review seems blocked on that11:30
sdagueabout getting us a wheel mirror going11:30
*** rfolco has joined #openstack-infra11:31
*** yfried|afk is now known as yfried11:31
*** samueldmq has joined #openstack-infra11:33
samueldmqmorning11:33
*** BobBall has joined #openstack-infra11:34
*** BobBall is now known as Guest2176611:34
sdaguefungi / AJaeger : https://review.openstack.org/#/c/179409 ... I think the sdk effort is blocked until that is landed11:35
*** _nadya_ has quit IRC11:35
fungiAJaeger: okay, the proposal jobs are properly registered in zuul following a graceful restart of jenkins.o.o11:36
openstackgerritBharat Kumar Kobagana proposed openstack-infra/project-config: Skip test_volume_boot_pattern for GlusterFS until bug 1444806 is fixed  https://review.openstack.org/17340811:36
openstackbug 1444806 in Cinder "test_volume_boot_pattern tempest test failure for glusterfs backend - Part 2" [Undecided,In progress] https://launchpad.net/bugs/1444806 - Assigned to Bharat Kumar Kobagana (bharat-kobagana)11:36
*** woodster_ has joined #openstack-infra11:37
*** freerunner has joined #openstack-infra11:38
sdaguefungi: hey, can you look at the gerrit event stream again?11:39
fungisdague: sure, checking11:39
sdagueoh... zuul is processing a bunch of events, might just be normal load11:40
*** hashar has quit IRC11:40
fungilooks like it. newest change in check was 4 minutes ago and zuul is churning through events/results11:40
sdagueyeh, I just pushed a bunch in11:40
*** hashar has joined #openstack-infra11:40
sdagueI was surprised they didn't show up yet11:40
fungialso i have a jjb update going on one of the masters which is putting geard under some load, slowing it down11:41
sdaguegotcha11:41
*** yfried is now known as yfried|afk11:41
sdagueQueue lengths: 472 events, 108 results.11:42
*** freerunner has quit IRC11:42
sdagueyeh, the events and results seem to be climbing, I guess that's the geard load?11:42
*** zz_dimtruck is now known as dimtruck11:43
sdagueah, there it popped11:43
sdagueI guess it was computing all the merge checks11:43
*** yfried|afk is now known as yfried11:44
*** hashar has quit IRC11:44
*** hashar has joined #openstack-infra11:44
*** shardy_ has joined #openstack-infra11:45
*** shardy has quit IRC11:46
*** hashar_ has joined #openstack-infra11:47
mgedminfungi, lifeless told me to ask you to review https://review.openstack.org/#/c/180060/11:49
mgedmin(it's not urgent)11:49
*** deepakcs has quit IRC11:49
*** hashar has quit IRC11:49
*** weshay has joined #openstack-infra11:50
*** shardy_ has quit IRC11:50
*** shardy has joined #openstack-infra11:51
*** dprince has joined #openstack-infra11:51
*** gulic has joined #openstack-infra11:52
*** gulic has left #openstack-infra11:52
*** Guest21766 has quit IRC11:53
*** ibiris is now known as ibiris_away11:54
*** dkranz has joined #openstack-infra11:54
*** alexpilotti has joined #openstack-infra11:55
*** _nadya_ has joined #openstack-infra11:56
*** ldnunes has joined #openstack-infra11:56
*** BobBall_ has joined #openstack-infra11:57
samueldmqhow do I follow the merge process (in zuul ?) of a review which received workflow+111:58
samueldmqfor example, https://review.openstack.org/#/c/179983/11:58
*** markvoelker has joined #openstack-infra11:59
*** freerunner has joined #openstack-infra11:59
*** kgiusti has joined #openstack-infra11:59
openstackgerritMerged openstack-infra/project-config: Adds refs/heads/stable/* to compute-hyperv  https://review.openstack.org/18018711:59
*** ibiris_away is now known as ibiris12:00
*** _nadya_ has quit IRC12:00
*** jtriley has joined #openstack-infra12:00
samueldmqsdague, ^ see me question just above, I think you may have the answer :)12:01
*** jyuso has quit IRC12:01
*** jyuso1 has joined #openstack-infra12:01
sdaguesamueldmq: can you reframe the question12:02
sdaguesamueldmq: I'm not sure I know what you are asking exactly12:03
samueldmqsdague, I want to see where my change exactly is12:03
sdaguesamueldmq: ok, so http://status.openstack.org12:03
sdaguesamueldmq: ok, so http://status.openstack.org/zuul12:03
samueldmqsdague, if it is being reverified, in merge process, whatever12:03
openstackgerritMerged openstack-infra/project-config: Adds refs/heads/stable/* to networking-hyperv  https://review.openstack.org/18016012:04
*** bknudson has joined #openstack-infra12:04
openstackgerritMerged openstack-infra/project-config: Adds pypi-jobs to compute-hyperv  https://review.openstack.org/17978212:04
samueldmqsdague, yes that' what I want, thanks12:04
*** Mithrandir has left #openstack-infra12:04
openstackgerritMerged openstack-infra/project-config: Adds publish-to-pypi to networking-hyperv  https://review.openstack.org/17984412:04
*** btully has joined #openstack-infra12:05
*** jtriley has quit IRC12:05
*** sdake has joined #openstack-infra12:07
*** hashar_ has quit IRC12:08
*** mherdad has joined #openstack-infra12:08
*** aysyd has joined #openstack-infra12:10
openstackgerritQiming Teng proposed openstack/requirements: Add senlin and its client to project list  https://review.openstack.org/18050112:10
openstackgerritMerged openstack-infra/project-config: Add missing ceilometer services to heat job  https://review.openstack.org/17847212:11
*** sdake_ has quit IRC12:12
*** ddieterly has joined #openstack-infra12:14
*** erlon has joined #openstack-infra12:15
*** eharney has quit IRC12:17
*** btully has quit IRC12:20
*** dimtruck is now known as zz_dimtruck12:21
*** gyee has joined #openstack-infra12:22
*** ryanpetrello_ has quit IRC12:22
*** e0ne is now known as e0ne_12:22
*** ryanpetrello_ has joined #openstack-infra12:22
*** ryanpetrello_ has quit IRC12:25
*** hashar has joined #openstack-infra12:25
*** _nadya_ has joined #openstack-infra12:25
*** ryanpetrello_ has joined #openstack-infra12:25
*** mherdad has quit IRC12:26
*** mherdad has joined #openstack-infra12:27
*** sabeen1 has joined #openstack-infra12:29
*** e0ne_ is now known as e0ne12:29
*** abregman has quit IRC12:31
openstackgerritDavid Shrewsbury proposed openstack-infra/shade: Update server API for get/list/search interface  https://review.openstack.org/18029312:31
*** BharatK has quit IRC12:32
*** dkranz has quit IRC12:34
openstackgerritMerged openstack-infra/project-config: Use surveil gerrit acl for python-surveilclient  https://review.openstack.org/18025712:35
openstackgerritMerged openstack-infra/project-config: adding akanda horizon plugin to stackforge  https://review.openstack.org/18029412:36
openstackgerritMerged openstack-infra/project-config: Add apps-catalog-ci group for apps-catalog project  https://review.openstack.org/18020112:36
*** freerunner has quit IRC12:36
*** ajmiller has quit IRC12:38
*** rbradfor has joined #openstack-infra12:39
*** bswartz has quit IRC12:43
*** freerunner has joined #openstack-infra12:44
*** BharatK has joined #openstack-infra12:47
*** abregman has joined #openstack-infra12:47
*** baoli has joined #openstack-infra12:48
AJaegerthanks, fungi !12:49
AJaegersdague, approved it.12:49
*** dkranz has joined #openstack-infra12:50
sdagueAJaeger: thanks12:50
fungistill struggling with jjb on jenkins02. getting occasional proxy errors trying to update there. may need to put it in shutdown, but before i do i'll make sure to bring all the other masters out of shutdown12:50
*** hashar has quit IRC12:52
*** dims has quit IRC12:52
*** openstackgerrit has quit IRC12:53
*** openstackgerrit has joined #openstack-infra12:53
*** dims has joined #openstack-infra12:53
*** miqui has joined #openstack-infra12:53
mordredmorning fungi12:54
*** hashar has joined #openstack-infra12:54
*** mattt has left #openstack-infra12:54
*** links has quit IRC12:54
fungimorning mordred12:56
*** baoli_ has joined #openstack-infra12:57
AJaegersdague, could you review this patch and the followon ones, please? https://review.openstack.org/#/c/180308/12:57
*** gordc has joined #openstack-infra12:58
*** BobBall_ has quit IRC12:58
sdagueAJaeger: so... why exactly?12:59
mordredfungi: I'm online for a little bit before heading back into meeting land ... anything I can help with?12:59
sdaguebecause that seems to expose internal details that we don't really care about12:59
sdagueand when ceph support moves into a plugin we now need to change all the job names?13:00
*** baoli has quit IRC13:00
fungimordred: do you like java? do you like jenkins? do you like reviewing java patches to jenkins plugins?13:00
fungiif so, i have just the thing for you...13:00
mordredfungi: oh god13:01
AJaegersdague, you're talking about the first one? We had some fun debugging why https://review.openstack.org/173408 failed and that came out as name clash.13:01
mordredfungi: I mean13:01
*** spzala has joined #openstack-infra13:01
AJaegersdague, I'd like to avoid such name clashes to make it clear which job is used.13:01
dstufftmeeting land sounds like a terrible place to visit13:01
*** sks has quit IRC13:01
fungidstufft: you just have to hope your car doesn't break down while you're passing through13:01
AJaegerYEs, might be some more renaming if you change something - but if you look at a job in zuul/layout.yaml, you know where it comes from13:02
sdagueAJaeger: ... I'm still confused13:02
dstufftfungi: gotta lock them car doors and pray13:02
AJaegersdague, confusion is bad ;) What are you confused about?13:02
sdagueAJaeger: so the intent in devstack is to push a lot of things into external plugins13:02
*** Somay has joined #openstack-infra13:02
mordreddstufft: it's very dangerous13:03
*** BobBall has joined #openstack-infra13:03
sdagueso we're going to add -plugin- to all trove jobs?13:03
sdagueand zaqar? and everything on stackforge... ?13:03
sdaguethis just doesn't seem useful13:03
*** BobBall is now known as Guest424013:03
sdaguedstufft: or find the monorail out13:03
Clintmmm.. monorail13:03
dstufftsdague: piublic transportation? what kind of socialist utopia is this meeting land!13:04
fungiClint: what's that word?13:04
sdague:)13:04
*** jswarren has joined #openstack-infra13:04
fungithat's right! monorail!13:05
AJaegersdague, it would indeed make such a change a few more lines. But if you look at the debugging we needed to do for the other patch, it could help us avoid that time.13:05
sdagueAJaeger: mostly it seems like a massive leak of an abstraction out of the system13:05
*** ddieterly has quit IRC13:05
*** wayneeseguin has joined #openstack-infra13:05
AJaegersdague, I'm not getting that point13:06
fungi(well, sir, there's nothing on earth like a genuine, bona fide, electrified, six-car monorail)13:07
* fungi stops13:07
sdagueAJaeger: we do a lot of configuration of jobs in the system13:07
sdaguewhether or not a devstack plugin is being used doesn't seem like top level information that's useful most of the time13:07
*** antonym has quit IRC13:08
fungisdague: i mainly suggested namespacing the plugin jobs because AJaeger expressed a concern about collisions between parameterized job name extensions and existing jobs of the same name13:08
fungiit has come up exactly once, as far as i know, so may not be worth actually spending time to fix long-term13:09
openstackgerritBharat Kumar Kobagana proposed openstack-infra/project-config: Skip test_volume_boot_pattern for GlusterFS until bug 1444806 is fixed  https://review.openstack.org/17340813:09
openstackbug 1444806 in Cinder "test_volume_boot_pattern tempest test failure for glusterfs backend - Part 2" [Undecided,In progress] https://launchpad.net/bugs/1444806 - Assigned to Bharat Kumar Kobagana (bharat-kobagana)13:09
*** eharney has joined #openstack-infra13:09
*** Guest4240 has quit IRC13:09
AJaegersdague, fungi: If you all think, it's not worth it, I'll stop arguing. We had the same problem with name clashes when mordred moved the docs infra jobs.13:10
openstackgerritMerged openstack-infra/gearman-plugin: Stop sending status updates  https://review.openstack.org/18024913:11
sdagueAJaeger: so is the real concern that a parameterized job can stomp a non parameterized job (or visa versa)?13:11
AJaegersdague, yes. Or that two different parameterized jobs can clash13:11
sdagueso can't we test for that in unit tests?13:11
sdague2 things evaled to the same name?13:11
*** radez_g0n3 is now known as radez13:12
AJaegersdague, unit tests show us if we really have two clashed instantions.13:12
fungiif you have a template foo{bar}{baz} and another template foo-quux{baz} and you instantiate them both and pass foo:-quux to the first when doing so, you can end up with two jobs colliding on the same name13:12
sdaguefungi: sure13:12
fungier, bar:-quux13:12
AJaegerBut then jjb will throw an error and you have to figure out why it suddenly fails13:12
sdagueAJaeger: right, so isn't that the place to put the check, and make it explicit about what the failure is13:13
fungianyway, yes i think if the jjb error says you have two jobs with the same name and tells you the name, hunting that down should be easy once you know this is possible13:13
AJaegerOr by accident you instantiate the wrong one.13:13
AJaegerLet me check again the failure...13:13
*** tnovacik has joined #openstack-infra13:13
sdagueit seems like we are catching it already, so it's just a matter of being more helpful on "oh, these 2 things collided"13:13
fungiif the jjb error doesn't make that clear, we should definitely do something to fix that13:13
*** sks has joined #openstack-infra13:13
AJaegersorry, meeting time now, will catch up later...13:14
sdagueand ideally even backtrack to the templates that were the basis for both of those13:14
*** BobBall has joined #openstack-infra13:14
*** BobBall is now known as Guest2055813:14
fungii've forgotten too much of jjb's internals to remember if it keeps track of the files where it got the various pieces it loaded13:14
sdagueI mostly just loath adding 7 new characters to basically all job names, which are already so long they break zuul formatting half the time13:15
openstackgerritMerged openstack-infra/project-config: added chef-rake-jobs for openstack-chef-repo  https://review.openstack.org/17972113:15
sdaguebecause a thing happened once13:15
fungiso doing that may entail some additional improvement to its job datastructure, but seems like a good thing for it to have if it doesn't yet13:15
openstackgerritMerged openstack-infra/project-config: New stackforge project: Bansho  https://review.openstack.org/18016613:16
*** mherdad has quit IRC13:17
fungias a separate point, improving the zuul status display to not break when presented with long job names (perhaps by truncating them and then adding an alt-name/hover tooltip with the full name) should also be on the table13:17
openstackgerritSergey Kraynev proposed openstack-infra/project-config: Add functional test job for python-heatclient  https://review.openstack.org/18053613:19
*** e0ne is now known as e0ne_13:19
mordredclarkb: I reviewed your fix race patch (shudder) and left a +2 and also 2 questions13:20
*** otter768 has joined #openstack-infra13:21
*** e0ne_ is now known as e0ne13:21
*** bswartz has joined #openstack-infra13:21
*** peristeri has joined #openstack-infra13:21
*** tcammann has joined #openstack-infra13:22
*** changbl has joined #openstack-infra13:22
*** fhubik has quit IRC13:22
*** tiswanso has joined #openstack-infra13:23
sdaguefungi: yeh, css only truncation is interesting, and otherwise you are playing font metrics games. I've actually spent some time looking at that.13:24
*** Somay has quit IRC13:25
*** otter768 has quit IRC13:25
*** gokrokve has joined #openstack-infra13:25
*** Guest20558 has quit IRC13:26
sdaguefungi: unrelated - I do believe I've got a reasonably patch series to bring back the inline zuul status with all the optimizations asked for, make it easy to test, and turned off by default until a final patch flips a flag - https://review.openstack.org/#/q/status:open+project:openstack-infra/system-config+branch:master+topic:hideci_zuul_optimize,n,z13:26
sdagueand because that effort started a long time ago, I'd love to get some reviews in there13:26
*** esker has joined #openstack-infra13:26
*** pabelanger has joined #openstack-infra13:26
*** pabelanger has joined #openstack-infra13:26
sc68caldtroyer: I +1'd this but it's one of those things that shows how complicated and organic the neutron devstack lib has become - https://review.openstack.org/#/c/179787/1/lib/neutron_plugins/ml213:28
*** freerunner has quit IRC13:28
*** gokrokve_ has quit IRC13:29
mordredsdague: "We need to fetch zuul targeted status urls over http as well because13:30
mordreddefault python 2.7 apparently blows it's stack on zuul's ssl cert."13:30
mordredthat seems like an unhappy thing13:30
*** BobBall_ has joined #openstack-infra13:30
pabelangermorning13:31
sdaguemordred: yes, fungi pushed a fix for a cert chain issue13:31
sdagueI do not know if that landed13:31
mordredas in - default python 2.7 not being happy with zuul's cert...13:31
mordredoh, good13:31
sdagueit was this code that discovered that issue13:31
*** freerunner has joined #openstack-infra13:32
sdagueonce that's landed, I'll fix this code to use https by default13:32
fungisdague: mordred: yep, still needs reviews... https://review.openstack.org/17993513:33
*** ddieterly has joined #openstack-infra13:33
*** mriedem has joined #openstack-infra13:34
*** antonym has joined #openstack-infra13:35
*** sigmavirus24_awa is now known as sigmavirus2413:35
mordredfungi: hahaha. nice13:37
anteayamorning pabelanger13:37
sdaguefungi: hmmm... https://jenkins01.openstack.org/job/gate-infra-puppet-apply-centos6/27/consoleFull13:37
*** freerunner has quit IRC13:37
sdaguecentos6 timeouts in puppet apply13:37
sdaguegoing to fail out a bunch of project-config changes13:37
pabelangeranteaya, You still in Ontario? Or have you relocated some where for openstack?13:37
sdague2015-05-06 13:15:28.046 | Error: /Stage[main]/Openstack_project::Thick_slave/Package[requests]/ensure: change from 2.7.0 to latest failed: Could not get latest version: HTTP-Error: 503 No healthy backends13:38
sdague2015-05-06 13:36:49.624 | Build timed out (after 30 minutes). Marking the build as failed.13:38
*** openstackgerrit has quit IRC13:38
fungisdague: yep, known issue. seems there's some issue with the git on centos 6.x which surfaced right after the "ghost" vulnerability update to glibc13:38
sdagueoh, fun...13:38
*** openstackgerrit has joined #openstack-infra13:38
fungisdague: unfortunately i've never been able to reproduce it on demand to get a full strace of the process13:38
fungibasically it seems to end up blocked on a socket read at some point during remote operations13:39
*** packet has joined #openstack-infra13:39
sdagueoh....13:39
sdagueso the git_timed that was added to devstack might have been for centos6 work around13:39
fungiwhich, given the bits of the system libs eth ghost fix backports touched, is not entirely unlikely it's caused some new bug13:39
sdaguebecause that seems to be the kind of thing it would recover from13:39
sdagueand would make sense coming from ianw13:40
fungisdague: yep, hence why i suggested we might want similar timeout/retry built into zuul-cloner (what the apply jobs are using)13:40
*** akshai has joined #openstack-infra13:40
sdaguefungi: gotcha13:40
sdagueok13:40
sdaguemordred: thanks for kicking those first 2 patches in13:41
*** sdake_ has joined #openstack-infra13:42
*** lifeless has quit IRC13:42
mordredsdague: sure - I'm looking at the next one - I think the feature flag is a great idea there13:42
*** mpavone has quit IRC13:43
openstackgerritJames E. Blair proposed stackforge/gertty: Fix diff display of deleted empty files  https://review.openstack.org/18054613:43
*** sdake__ has joined #openstack-infra13:43
mordredit looks like jeblair is up!13:44
*** sdake has quit IRC13:44
*** gyee has quit IRC13:45
*** craigbr has joined #openstack-infra13:46
*** freerunner has joined #openstack-infra13:46
*** mriedem has quit IRC13:46
*** gyee has joined #openstack-infra13:46
*** mriedem has joined #openstack-infra13:47
*** sdake_ has quit IRC13:48
*** freerunner has quit IRC13:48
fungiokay, i've brought jenkins05 and 06 back online from when they were in prepare for shutdown yesterday (left 03 alone for now in case jeblair is still poking that one) and put 02 into prepare for shutdown so i can try to get its job updates to complete successfully13:48
jeblairfungi: no, i'll put 03 back in13:49
fungi02 was pretty clearly starting to suffer from the thread leak issue we see regularly, which explains why jjb kept getting proxy errors on update13:49
fungithanks jeblair13:49
fungionce i confirm all the masters are back online and have jjb updated properly, then i'll manually apply puppet updates and make sure jjb still runs to completion in a timely manner, then leave puppet agent reenabled on all of them13:51
sdaguemordred: yeh, given that we've played revert games with this in the past, it seems like a feature flag lets us have more reviewable code for fixes, instead of it being a giant chunk of revert / revert13:51
mordredsdague: yes. also - it means you can probably be fancy and enable it for yourself for a while13:51
fungi05 already has new slaves and is running jobs on them, so that's a good sign13:51
sdaguemordred: correct13:51
mordredsdague: kindof like how I run tip of pbr and git-review locally13:51
fungisame for 0613:51
sdaguemordred: also it means a subset can test more easily13:52
dtroyersc68cal: since that is not a new variable, just a new check it 'feels' ok to me.  I don't know enough to verify that the assertion in the commit message is true though…13:52
sdagueespecially given that the proxy doesn't support logging in to gerrit, because I don't know how to intercept the openid call13:52
mordredjeblair, clarkb: when you get a chance, could I get a rereview on 175891? there are currently a couple of brokens in the upstream ansible modules that landed that we need to cut a shade release for, and I'd like to get that patch in the release13:54
openstackgerritMerged openstack-infra/devstack-gate: Archive glusterfs logs and config  https://review.openstack.org/17150013:54
*** JayJ_ has joined #openstack-infra13:54
openstackgerritMerged openstack-infra/devstack-gate: Remove heat from default configuration  https://review.openstack.org/17060813:54
openstackgerritMerged openstack-infra/devstack-gate: Remove pip cache move  https://review.openstack.org/17890213:54
openstackgerritMerged openstack-infra/devstack-gate: Make sysctl vm dump errexit safe  https://review.openstack.org/17890313:54
openstackgerritMerged openstack-infra/devstack-gate: Check for mount before creating swap  https://review.openstack.org/17890413:54
openstackgerritMerged openstack-infra/project-config: enable python-jobs for networking-zvm  https://review.openstack.org/17973013:54
jeblairsdague: so when that merges, we will have three zuul status pages.  do you or jhesketh have a plan to consolidate the code base onto the work that jhesketh has done in zuul?13:55
*** dboik has joined #openstack-infra13:55
sdaguejeblair: so I think we discussed this the last time, this was about having a minimal indicator inside of gerrit13:56
jheskethjeblair: I haven't read all the backlog, but I assume you're talking about the live zuul stuff13:56
jheskethjeblair: are the 3 places you're counting: upstream zuul, status.o.o/zuul and now gerrit?13:57
jeblairsdague: yeah, and i think the results of last time, where we discovered it wasn't using any of the stuff like jquery-visibility, etc, suggested that maybe the idea of code reuse really is a good idea13:57
jeblairjhesketh: yep.  "upstream" of course is running on zuul.o.o13:57
openstackgerritJoshua Hesketh proposed openstack/requirements: Add python-magic to global requirements  https://review.openstack.org/18055213:57
*** freerunner has joined #openstack-infra13:57
sdaguejeblair: ok, I took your comments as "this should also do that"13:58
jeblairsdague: yeah, i'm not going to -2 your patches or anything, but i would like to still have a conversation about how to move forward and not have 3 javascript status pages mantained by a bunch of people who don't know javascript :)13:58
jheskethjeblair: okay.. so we should definately update status.o.o to use what's in the zuul repo. The stuff in gerrit though is actually quite different at this point13:58
anteayaI didn't even know about zuul.o.o13:59
sdaguejeblair: ok, seems like a good conversation to have over a meal at summit.13:59
jeblairone of the reasons i like jhesketh's in-zuul-tree stuff is it actually gets reviewed and patched by javascript folks :)13:59
openstackgerritJoshua Hesketh proposed openstack-infra/os-loganalyze: Create devstack plugin for testing os-loganalyze  https://review.openstack.org/18055313:59
jeblairsdague: i better pack a bunch of energy bars if i want to eat anything ;)14:00
sdagueheh14:00
jheskethjeblair: because what is in gerrit is quite different to the other two places, I think the first step is to bring some of that code into the zuul repo. Then all we have to do is consume that within gerrit rather than having it out-of-tree14:01
jheskethsdague: 180553 might interest you btw14:01
jheskethsdague: I don't think I'll tackle the tests tonight, but that's the next step14:01
sdaguejhesketh: \o/14:01
sdaguethank you14:01
mordredjhesketh: oh - you're still awake - 175891?14:02
jheskethno trouble, sorry it took so long14:02
* jhesketh isn't sure when he's awake anymore14:02
jeblairjhesketh: okay cool, sounds like a plan (or a plan for a plan) :)14:02
sdagueI will play with that today and have review feedback by the time you get up tomorrow14:02
jheskethsdague: thanks buddy14:02
jheskethjeblair: yeah, I might look at doing the easier of those tasks this week (updating status.o.o to upstream javascript)14:03
jhesketh(assuming personal life doesn't get in the way which it might for $reasons)14:03
sdagueso... upstream js being zuul.o.o?14:03
*** gokrokve has quit IRC14:03
sdaguebecause, zuul.o.o is way heavier on my browser than status.o.o/zuul14:04
jheskethsdague: well what's in zuul's repo, yes. Not sure if we'd hot-link across domains or host a second copy14:04
sdagueto the point that I found it unusable14:04
*** macjack has left #openstack-infra14:04
*** tonytan4ever has joined #openstack-infra14:04
sdaguelike chrome goes into odd lockups on it14:04
jhesketh:-(14:04
jheskethokay so it might need some optimising14:04
*** signed8bit has joined #openstack-infra14:05
*** isviridov_away is now known as isviridov14:05
sdagueyeh, like the whole UI thread seems to hang on every update, so you'll be scrolling then it will lose all the UI events and just stop14:05
jheskethsdague: any clues or how to reproduce?14:05
jheskethyuck14:06
sdagueyeh, just try to keep scrolling up and down14:06
sdagueit will hang14:06
sdagueit comes back14:06
sdaguebut that kind of responsiveness issue is... pretty suboptimal14:06
*** abregman has quit IRC14:06
AJaegerfungi, sdague: here'S the error message for the duplicate definition http://logs.openstack.org/08/173408/8/check/gate-project-config-layout/0931ead/console.html14:06
openstackgerritMerged openstack-infra/project-config: Move DIB test dep installation into DIB  https://review.openstack.org/17920914:06
*** craigbr has quit IRC14:06
sdagueI believe also when I was on my laptop on battery the page was noticably more power draw as well14:07
openstackgerritMerged openstack-infra/project-config: Pylint job for Murano project  https://review.openstack.org/17288714:07
openstackgerritMerged openstack-infra/project-config: Look for yum instead of rpm when caching devstack  https://review.openstack.org/17968714:07
sdagueI remember looking at the QA sprint14:07
AJaegerit basically is only "Duplicate definitions for job 'check-tempest-dsvm-full-glusterfs' specified"14:07
AJaegerAnd then you're lost14:07
openstackgerritMerged openstack-infra/project-config: Do a better job of identifying CentOS 6  https://review.openstack.org/17969414:07
openstackgerritMerged openstack-infra/project-config: Add Python SDK to PROJECTS for SDK gate  https://review.openstack.org/17940914:07
*** dustins has joined #openstack-infra14:07
openstackgerritMerged openstack-infra/system-config: create proxy for testing hideci  https://review.openstack.org/17936014:07
jheskethsdague: I'd hardly call that responsive14:07
fungiAJaeger: git grep glusterfs?14:07
*** dustins has quit IRC14:08
AJaegerfungi, this is followup to the name clash discussion where I stepped out an hour ago.14:08
*** dustins has joined #openstack-infra14:08
fungiAJaeger: yep, just suggesting the next step i'd have done after seeing that error14:08
*** craigbr has joined #openstack-infra14:08
fungiAJaeger: though i agree that improving jjb to be able to report the files in which the offending jobs or job-templates originated14:08
fungiis a good idea14:08
sdaguejhesketh: yeh that UI thread halt also means that if it happens to be in that refresh loop and you hit pgup or pgdown nothing happens, and then you are confused and start hitting lots of keys :)14:08
mordredfungi, AJaeger: ++14:09
AJaegerfungi, ah! Yes, seems that improving jjb to throw better errors might be the right approach - just one that I won't sign up for ;)14:09
*** lifeless has joined #openstack-infra14:10
fungiAJaeger: and i can see sdague's point that making more verbose job names just to avoid the risk of future name collisions is sort of a hackish workaround14:10
AJaegerfungi, stepping out and rereading, I'm agreeing with him now.14:11
fungiAJaeger: but improving jjb's error handling there is probably a fairly low-hanging-fruit patch we can sick a new python-savvy contributor on14:11
sdaguejhesketh: if you run the heap profile tool in chrome web dev you can actually get a nice visual as well14:11
sdagueon zuul.o.o you see basically a 1 second freeze of the tools every 5 seconds during the refresh14:12
fungiAJaeger: especially since it doesn't need any special infrastructure to implement and test14:12
sdagueon s.o.o/zuul it's about < 100ms14:12
*** isviridov is now known as isviridov_away14:12
AJaegerfungi, yes agreed.14:12
AJaegerSo, I'll abandon my patches and let the glusterfs guys take what they like or go back to the previous version.14:13
AJaegerfungi, sdague thanks for the discussion!14:13
sdagueAJaeger: no prob14:13
*** amitgandhinz has joined #openstack-infra14:19
*** abregman has joined #openstack-infra14:19
openstackgerritMerged openstack-infra/system-config: add dbg function to hideci  https://review.openstack.org/17957314:22
openstackgerritMerged openstack-infra/project-config: Pass through arbitrary devstack config options  https://review.openstack.org/17998314:22
openstackgerritMerged openstack-infra/system-config: Revert "Revert zuul status in gerrit"  https://review.openstack.org/17936114:22
AJaegersdague, I give a procedural -2 to https://review.openstack.org/#/c/179714  so that you don't approve it again ;)14:23
sdagueAJaeger: oh, yeh14:24
sdagueI saw the deletes14:24
sdagueI guess I messed that one up14:24
jheskethsdague: okay, I'll have a poke at some profiling stuff.. I haven't done anything like that before so it'll be a good learning experience14:24
sdaguejhesketh: yeh, that 1 second hang also causes clicks to expand, collapse to go missing14:25
sdagueand this is on my 4 core haswell, on 50 mbs fiber14:25
sdaguewhich means I expect that might be worse on more modest machines / network14:26
mtreinishsdague: you obviously need to pay for more bandwidth and hardware14:26
sdaguemtreinish: :)14:26
jheskethheh, there's your problem! ;-)14:26
*** e0ne is now known as e0ne_14:27
*** jlvillal has quit IRC14:28
*** jlvillal has joined #openstack-infra14:29
*** gokrokve has joined #openstack-infra14:32
AJaegerfungi, do you know what happened with openstack-doc-tools 0.26 release? It never showed up on pypi.14:33
*** nelsnelson has joined #openstack-infra14:34
*** gargs has joined #openstack-infra14:34
*** gargs has left #openstack-infra14:34
AJaegerhttps://pypi.python.org/pypi/openstack-doc-tools/ has 0.25 only14:34
kashyapfungi: Hi, I think OpenStack Infra also uses Supybot, doesn't it?14:37
*** stevemar has joined #openstack-infra14:37
*** e0ne_ is now known as e0ne14:37
kashyapfungi: If so, does OpenStack Infra mind borrowing this little useful Feature from Fedora? :-) https://github.com/fedora-infra/supybot-fedora/commit/1ba62ced08487fe4dcc8b5040c8fc64ae3b8ce0f14:37
*** jtriley has joined #openstack-infra14:38
jheskethmordred: okay, reviewed 17589114:39
jheskethnow it's probably time for sleep14:39
AJaegergood night, jhesketh !14:39
anteayakashyap: we haven't adopted teh behaviour of using bots to drive social change14:39
jhesketho/14:40
fungikashyap: yep, the "openstack" bot is basically supybot running the meetbot plugin along with another plugin now to generate links to bug reports14:40
anteayakashyap: feel free to share your thoughts about how you would like to be addressed with those with whom you interact14:40
fungikashyap: http://docs.openstack.org/infra/system-config/irc.html#meetbot14:40
kashyapanteaya: Sure. I can see what you're saying. In this case, it _is_ useful because - getting 20 empty pings can be stressful over the course of a day.14:40
anteayakashyap: sure, so I am learning to ignore them14:41
kashyapanteaya: As the blog indicates in that commit message.14:41
anteayakashyap: as I ignore questions addressed to "guys"14:41
kashyapanteaya: Also this -- http://openstack.redhat.com/IRC_etiquette14:41
anteayakashyap: as fungi indicates you are welcome to offer a patch14:41
anteayakashyap: if you don't like how you are being treated tell people14:42
anteayadifferent people like to be treated different ways14:42
kashyapanteaya: People treat me just fine, I don't have an issue :-)14:42
anteayagreat14:42
*** craigbr has quit IRC14:42
fungiAJaeger: it likely ran afoul of the upload job not being registered in zuul, like the translation jobs you pointed out14:43
AJaegerfungi, so, what can we do? Tag a 0.26.1 release?14:43
fungiAJaeger: i can manually trigger the jobs which never ran for it. will take me a few minutes to get to14:43
AJaegerfungi, no urgency for that one... Thanks!14:44
*** jtriley has quit IRC14:44
*** JayJ_ has quit IRC14:44
fungikashyap: so, yes, a supybot plug-in might make more sense as i'm not sure we want to maintain a supybot fork14:45
*** jtriley has joined #openstack-infra14:45
kashyapfungi: Yep, agreed.14:45
fungikashyap: aside from that, i have no opinion on the naked ping issue. like anteaya, i basically ignore those and _if_ someone persists with them i ask them to stop, and _if_ they don't i /ignore their nick altogether14:46
*** claudiub has quit IRC14:46
*** craigbr has joined #openstack-infra14:47
fungii've also started to intentionally ignore questions directed at me specifically even though they could be answered by many other people in-channel14:47
fungifor example, the one you just asked ;)14:48
kashyap:-)14:48
kashyapThanks for the response, anyhow.14:48
Davieyfungi: ping.14:48
fungiyou're welcome!14:48
* fungi adds Daviey to his /ignore list (okay, not really)14:49
Daviey /trolling :)14:49
fungi"trollping"14:49
anteayacan we get a bot to just ignore Daviey?14:50
anteaya:)14:50
fungii thought Daviey _was_ a bot?14:50
*** bhunter71 has joined #openstack-infra14:52
*** Sukhdev has joined #openstack-infra14:53
*** bknudson has quit IRC14:55
Davieyfungi: I might be as consistent, high out, multi-threaded and deterministic as a bot.. but I am indeed a humanoid.15:01
openstackgerritDavid Caro proposed openstack-infra/jenkins-job-builder: Removed unnecessary parameter from delete_old_jobs  https://review.openstack.org/18057515:01
openstackgerritMerged openstack-infra/project-config: Add new branch for channels work  https://review.openstack.org/18041615:02
*** pc_m has quit IRC15:02
*** gyee has quit IRC15:03
*** yfried has quit IRC15:04
*** marun has joined #openstack-infra15:04
*** BobBall_ has quit IRC15:05
*** sks has quit IRC15:05
*** gyee has joined #openstack-infra15:06
openstackgerritDavid Caro proposed openstack-infra/jenkins-job-builder: Added parallelization options  https://review.openstack.org/18046315:06
*** marcusvrn_ has joined #openstack-infra15:07
jeblairfungi: ping # noqa15:09
*** BobBall has joined #openstack-infra15:09
*** Sukhdev has quit IRC15:10
*** freerunner has quit IRC15:10
fungiheh15:12
*** sputnik13 has joined #openstack-infra15:12
hasharjeblair: thanks for the nodepool/gear tagging :]15:12
jeblairhashar: you're welcome; i might try my hand at zuul soon! :)15:13
*** freerunner has joined #openstack-infra15:13
*** armax has joined #openstack-infra15:13
hasharjeblair: zuul has a bunch of patches pending a final approval. Would be nice to have them land in15:13
jeblairyeah, i'll try to do that first15:13
*** sdake has joined #openstack-infra15:14
hasharsome I have been running them for a few months already :}15:14
hasharthere is one that alter the zuul-cloner  git cache behavior though and that has an impact on your setup https://review.openstack.org/#/c/117626/15:15
BobBallAnyway to throttle how often nodepool tries to recreate nodes if (for example) quota is exhausted?15:17
*** freerunner has quit IRC15:17
*** sdake__ has quit IRC15:17
*** ayoung is now known as eliza15:18
BobBallI've got an API quota for /servers - once that's exceeded nodepool will just spin trying to create new servers AND keep the quota as exhausted15:18
*** lennyb has quit IRC15:18
*** sks has joined #openstack-infra15:18
openstackgerritMerged openstack/requirements: Updated oslo_middleware to 1.2.0  https://review.openstack.org/17992615:18
*** eliza is now known as ayoung15:18
jeblairBobBall: yeah, you can set the 'rate' parameter to match your api rate limit15:18
BobBallBingo.  Thanks.15:18
*** rlandy has quit IRC15:19
fungiBobBall: also set your max-servers to something less than your known quotas (taking instance count, ram, cores, et cetera into account)15:19
BobBallyeah - max-servers is set15:19
BobBallbut that didn't stop me hitting the API limit15:20
jeblairBobBall: ('rate' is a poorly named option and is actually the inverse of the rate -- so if you can do 10/sec rate should be 0.1)15:20
fungiand give it a little breathing room if you're using snapshot image builds15:20
fungiBobBall: also check for leaks using nodepool alien-list15:20
BobBall*nod* Do that regularly.  Surprisingly I don't see any leaked for ages :)15:21
*** otter768 has joined #openstack-infra15:21
fungiBobBall: nodepool should _not_ try to exceed the configured maximum, so sounds like something is wrong there15:21
BobBallIt didn't exceed the maximum number of servers15:22
BobBallbut IAD couldn't build servers for a while (networking issues that I'm still trying to track down)15:22
*** ayoung is now known as ayoung-eliza15:22
BobBalland that ended up with very quick churn15:22
BobBallwhich meant that eventually I exceeded 1000 calls to /servers in 24 hours15:22
fungiBobBall: oh! so you were running afoul of api throttles15:22
BobBallYup15:22
fungi"limits" as nova calls them15:23
*** ayoung-eliza is now known as ayoung15:23
fungiBobBall: then yes, the rate option is definitely what you want. we use it for exactly the same reasons15:23
BobBall*nod*15:23
*** sputnik13 has quit IRC15:24
*** r-daneel has joined #openstack-infra15:24
jeblairBobBall: and if you (perhaps later) find you need higher limits, i believe you can ask rax for them.  i think they are set at a default 'try to protect the users from themselves' level :)15:24
BobBallDoes each server create get allocated a Task?  i.e. if I set the rate to 60.0 (which I believe is seconds) it'll only create one server per minute at most, right?15:25
BobBallI've got a support ticket to increase the rate too, but I want to throttle it so I'm confident I won't abuse it.15:25
jeblairBobBall: yes; there are other tasks that factor in as well, such as list server calls, etc....15:25
mordredShrews, SpamapS: jhesketh made an excellent point about duplicate server names in the inventory code - I think it's a good and hard enough point that we need to solve it properly - so I'd like to go ahead and release what we've got now and go back and attack inventory better15:26
jeblairBobBall: nodepool doesn't break those out though, it's just one value for all api calls.  ideally, this could be automated.  however, i think it's completely different on different clouds.15:26
jeblairyay openstack standardization15:26
mordredShrews, SpamapS: is there anything in that patch we should pull out to get into this release?15:26
*** otter768 has quit IRC15:26
jeblairmaybe we can add that to shade ;)15:26
* mordred reads jeblair and BobBall scrollback15:26
BobBall*nod* that's fine.  I'll just go for "rate: 60.0" and tune it if it's too slow15:27
BobBallIt's not that exciting mordred :)15:27
* mordred saw "completely different on different clouds" and figured it might be ... but yeah, you're on it15:27
jeblairmordred: oh, yeah supporting dupes sounds important15:28
*** xyang1 has joined #openstack-infra15:28
mordredjeblair: uhm, yeah15:28
Shrewsmordred: yeah, that's a good catch. we should fix that15:29
mordredShrews: we should consider what to do about the "cloud appends random suffix" problem too15:29
mordredShrews: I betcha the two will wind up being related15:29
Shrewsmordred: nuke the provider that does that???15:29
mordredShrews: well, yes. we will do that too15:29
Shrewsthat's just multi-colored crazy pants15:30
*** adrian_otto has joined #openstack-infra15:30
openstackgerritBharat Kumar Kobagana proposed openstack-infra/project-config: Skip test_volume_boot_pattern for GlusterFS until bug 1444806 is fixed  https://review.openstack.org/17340815:31
openstackbug 1444806 in Cinder "test_volume_boot_pattern tempest test failure for glusterfs backend - Part 2" [Undecided,In progress] https://launchpad.net/bugs/1444806 - Assigned to Bharat Kumar Kobagana (bharat-kobagana)15:31
*** krtaylor has quit IRC15:31
*** hashar is now known as hasharMeeting15:32
jeblairShrews: will you be wearing yours to the summit?15:33
clarkbmordred: I don't understand your first comment on https://review.openstack.org/#/c/179988/5/src/main/java/hudson/plugins/gearman/MyGearmanWorkerImpl.java LinkedList is not thread safe15:33
Shrewsjeblair: i have decided not to wear pants at this summit15:33
mordredclarkb: your comment said it was15:33
anteayaShrews: thanks for the advanced warning15:33
clarkbmordred: which comment?15:33
Shrewsanteaya: :)15:34
anteayaShrews: I hope your kilts are in fine form15:34
anteayautilikilts are the best15:34
mordredclarkb: "handleSessionEvent() method puts all events on a thread15:34
mordredsafe queue for synchronous processing"15:34
mordredclarkb: I may have read that wrong though15:34
anteayamulti-coloured crazy kilts15:35
anteayabit of a ring to that15:35
clarkbmordred: ya thats the new code15:35
clarkbmordred: which uses a thread safe list15:35
mordredclarkb: kk15:36
fungiShrews: we can be known by our crazyshorts instead15:37
*** openstackgerrit has quit IRC15:37
anteayaand crazyshirts15:37
*** openstackgerrit has joined #openstack-infra15:37
clarkbmordred: note the type is changing and we initialize the variable in the constructor15:37
* fungi already wears crazyshirts15:37
*** sks has quit IRC15:37
anteayayou do15:37
anteayaand I can spot you across a crowded room because of it15:38
anteayanot sure if that is a bug or feature15:38
*** cody-somerville has joined #openstack-infra15:39
*** hdd__ has quit IRC15:40
pblahohi all.... since when http://logs.openstack.org returns html logs as application/octet-stream ?15:41
*** Sukhdev has joined #openstack-infra15:43
*** dizquierdo has quit IRC15:44
*** yamahata has joined #openstack-infra15:46
*** bknudson has joined #openstack-infra15:46
*** dkranz has quit IRC15:48
*** akshai has quit IRC15:48
*** dannywilson has joined #openstack-infra15:49
*** akshai has joined #openstack-infra15:49
*** ashleighfarnham has joined #openstack-infra15:51
*** patrickeast has joined #openstack-infra15:51
*** Swanson has joined #openstack-infra15:53
*** signed8bit has quit IRC15:53
timrcclarkb, jeblair, fungi, mordred and anyone else that may know: Where do upstream zuul-mergers get their refs?  Do you push out to git mirrors? I was under the impression that refs were fetched from gerrit repos directly, but is that wrong?15:53
*** patrickeast_ has joined #openstack-infra15:54
*** viktors is now known as viktors|afk15:54
jeblairtimrc: they are fetched from gerrit directly; i think clarkb started a patch to make that an option, but it hasn't landed yet?15:54
fungitimrc: the upstream zuul mergers pull from gerrit's git service and then create the zuul refs15:54
clarkbjeblair: correct, gozer uses that patch for reasons though15:55
mordredyah15:55
timrccody-somerville, There you have it ^^15:55
clarkbI think jesusaurus has been trying to curate the patch /me has continued to review it in spurts15:55
clarkbmordred: responded to your comments, let me know if that helps15:56
*** patrickeast has quit IRC15:56
*** patrickeast_ is now known as patrickeast15:56
timrcclarkb, It's failing tests for some mysterious reason.15:56
timrcThat's the patch I was telling jeblair about that needs to do have a retry for timeouts connecting to the git mirror (arguably there should not be timeouts to the git mirror, but yay resilience)15:56
clarkbwell you have to retry because until gerrit 2.10 or something there is no gerrit event for replication completed15:57
*** scheuran has quit IRC15:57
clarkbso there is no way to know if your data has arrived until you check, and those checks may fail15:57
mordredclarkb: ah - I grok the disconnect15:57
mordredclarkb: my brain wasnt catching that that was the constructor because I'm stupid15:57
mordredclarkb: yeah - I think the patch looks great15:58
sdagueclarkb / fungi / dstufft - so given you all were knee deep in the cffi bit last night - https://review.openstack.org/#/c/180549/ is a different way to get around the install vs. build issue in devstack than what we landed, which is a little cleaner. Your thoughts are appreciated.15:59
clarkbmordred: cool, thanks for reviewing16:00
clarkbtimrc: I will take a look at that change again today as soon as I am awak16:00
*** sarob has joined #openstack-infra16:00
*** dkranz has joined #openstack-infra16:00
*** hdd__ has joined #openstack-infra16:01
timrcclarkb, Well we are timing out on the clone.... it just sit there until the timeout limit is reached and then fails.  Developers then recheck and eventually their change passes tests.  So I think we need to fix the actual timeout issue but also add a retry so things are tried again N number of times after a timeout occurs.16:01
*** claudiub has joined #openstack-infra16:01
timrcThat way developer rechecks don't effectively become that retry.16:01
clarkbtimrc: honestly, if that is becoming a problem you should upgrade to latest gerrit and update the feature to use the replication events16:02
clarkbtimrc: because you can't fix it that way, you can only add more duct tape16:02
*** hdd__ has quit IRC16:03
*** sergsh has quit IRC16:03
*** sergsh_ has quit IRC16:03
timrcclarkb, I think upgrading gerrit is on our road map, but it's a two step process.  First upgrading to the last version that can run on Precise and then again to the Trusty version some time in the future.16:03
clarkbalso I rmember looking at these fails once, and going "I have no idea what is happening" maybe second time around will be more enlightening16:03
jeblairclarkb: i agree that would be a better solution -- but if the cloner knows that a git operation that should have succeeded failed, then why not retry?16:04
timrcIt's good to know there's a gerrit event we can listen for.16:04
clarkbjeblair: it already does retry is what I am saying16:04
jeblairoh16:04
clarkbjeblair: so timrc is suggesting adding more retries which IMO is pointless16:04
jeblairclarkb: we have 1 retry and the suggestion is N retries?16:04
*** jlanoux has quit IRC16:04
jeblairwhere N>116:04
clarkbjeblair: its a configurable retry that is arbitrary iirc16:04
timrcclarkb, I didn't think it retried... it only attempts the clone once and keeps checking the clock to see if it's timed out and then fails.16:04
*** tkelsey has left #openstack-infra16:04
jeblairoh, er.... so what's the question? :)16:04
clarkbya https://review.openstack.org/#/c/76057/28/doc/source/zuul.rst16:05
clarkbpoll_interval and poll_timeout are completely configurable so can be set to an arbitrary retry and timeout for what you want16:05
jeblairclarkb: that translates to a number of retries?16:05
clarkbjeblair: yes it should do a fetch every interval seconds until the timeout elapses16:06
clarkbif fetch time is > timeout then you will only do one16:06
openstackgerritVictor Sergeyev proposed openstack-infra/project-config: Move Ironic python3.4 job from experimental to gate  https://review.openstack.org/18059916:06
jeblairclarkb: got it16:08
openstackgerritDavid Shrewsbury proposed openstack-infra/shade: Update images API for get/list/search interface  https://review.openstack.org/18060016:08
timrcclarkb, Hm.16:08
*** ajmiller has joined #openstack-infra16:08
timrcSo I would have to set the timeout to some multiple of the timeout git uses to clone to get multiple tries?16:08
*** bknudson has quit IRC16:09
clarkbtimrc: yes I think so16:09
*** shashankhegde has joined #openstack-infra16:09
clarkbtimrc: but its not a clone fwiw16:09
clarkbtimrc: its just a fetch of the ref16:09
timrcclarkb, Doh, fetch.16:11
clarkbthe initial clone of the repo by the merger will only be attempted once but usually thats fine because you can have a bit more contorl over iniial project configuration than you do for every ref replication16:13
*** gyee has quit IRC16:13
*** bknudson has joined #openstack-infra16:14
openstackgerritMerged openstack-infra/devstack-gate: Force openvswitch as the Neutron agent for Ironic  https://review.openstack.org/17211816:15
openstackgerritMerged openstack-infra/devstack-gate: Add dstat to subnode in multinode mode  https://review.openstack.org/17797016:16
clarkbok so it looks like the tests are timing out, I am going to bump the timeout but I am going to guess that we end up retrying fetches for the refs in the test framework indefinitely for some reason16:16
openstackgerritMerged openstack-infra/devstack-gate: Follow up with some comments to clarify OVS commands  https://review.openstack.org/17362116:16
*** markus_z has quit IRC16:16
openstackgerritMerged openstack-infra/devstack-gate: Add host counter to ansible inventory  https://review.openstack.org/17792016:16
jlvillalIs it already a known issue that the "Toggle CI" button is missing?  It is for me, at least.16:21
clarkbjlvillal: it is there for me let me try without caches16:21
clarkbjlvillal: also you might try refreshing those16:21
*** gokrokve has quit IRC16:21
fungijlvillal: i bet someone approved a change to some of the gerrit javascript16:21
*** gokrokve has joined #openstack-infra16:21
clarkbah yup a hard refresh for me has removed it16:22
fungii'll touch the proper file16:22
*** shashankhegde has quit IRC16:22
fungiclarkb: jlvillal: it's fixed for me now... you?16:22
clarkbback for me16:22
jlvillalclarkb: It is working now :)16:23
jlvillalfungi: ^^16:23
fungithere's something about the 'touch ~gerrit2/review_site/etc/GerritSiteHeader.html' exec in our puppetry which isn't taking care of this automagically. i wonder if adding a sleep to it would help16:23
jlvillalfungi: clarkb:  Thanks!16:23
*** soren has quit IRC16:25
*** Krinkle|detached is now known as Krinkle16:25
clarkbtimrc: jesusaurus if I bump the test timeout that change passes, so I think either the test timeout was reduced or things changed such that the polling is slower now and can't fit in under the timeout16:26
clarkbwill leave a comment16:26
*** shashankhegde has joined #openstack-infra16:26
clarkblooks like the longest test takes ~35 seconds and timeout is 30 seconds16:26
timrcclarkb, Ah.  Thanks.16:27
*** jcoufal_ has quit IRC16:27
*** amitgandhinz has quit IRC16:28
openstackgerritFabien Boucher proposed openstack-infra/puppet-zuul: Manage logging configuration files (feature flag)  https://review.openstack.org/18061416:28
*** jistr has quit IRC16:29
openstackgerritAlec Hothan proposed openstack-infra/project-config: Need to allow merge commits in vmtp.git  https://review.openstack.org/18061616:31
openstackgerritJeremy Stanley proposed openstack-infra/system-config: Avoid race on reload_gerrit_header  https://review.openstack.org/18061716:32
fungijlvillal: clarkb: ^16:32
jlvillalfungi: Thanks16:32
*** claudiub has quit IRC16:33
*** bhunter71 has quit IRC16:34
*** freerunner has joined #openstack-infra16:34
*** krtaylor has joined #openstack-infra16:35
*** dannywilson has quit IRC16:35
SpamapSmordred: can you restate or point me to the point about duplicate hostnames?16:37
*** sputnik13 has joined #openstack-infra16:37
*** Guest55056 has quit IRC16:38
*** bhunter71 has joined #openstack-infra16:39
*** marcusvrn1 has joined #openstack-infra16:39
*** ssam2 has quit IRC16:39
*** freerunner has quit IRC16:40
mordredSpamapS: restating will be easier16:40
*** freerunner has joined #openstack-infra16:41
openstackgerritDerek Higgins proposed openstack-infra/system-config: Add a Fedora 21 node to tripleo cloud  https://review.openstack.org/16977816:41
mordredSpamapS: one can have duplicated hostnames both inside of a single cloud, and also potentially across different clouds16:41
*** whoops has joined #openstack-infra16:41
*** freerunner has quit IRC16:41
mordredSpamapS: so when inventory reads your hosts and combines them into the inventory, at the moment, you'd be losing data if you had such a situation16:41
mordredI think I've got a solution though16:41
*** marcusvrn1 has quit IRC16:41
*** marcusvrn1 has joined #openstack-infra16:41
SpamapSmordred: indeed. a) Don't do that! b) scream and yell when dupes are detected c) Add cloud name when dupes are detected.16:42
*** marcusvrn has quit IRC16:42
*** marcusvrn_ is now known as marcusvrn16:42
mordredSpamapS: hang on - it's a really important usecase16:42
*** e0ne has quit IRC16:43
SpamapSI like c, but I've been accused of favoring fail-open in the past.. which woudl we bad if I ever took over designing nuclear plants.16:43
ameadejeblair: hey did you see my response about the netapp-ci? please let me know if you need any more info16:44
*** freerunner has joined #openstack-infra16:44
mordredSpamapS: which is to reverse the hostanme/instance-$uuid group creation we're doing right now - and to return the 'hostname' as 'instance-$uuid' and then make a group for each hostname16:45
mordredSpamapS: that way, referring to a hostname will work like you expect - and if you need to reference a specific instance that shares a name, you have that ability16:45
SpamapSmordred: That is the more accurate solution for sure.16:45
mordredSpamapS: important use case is that we have tons of pets aroudn here16:45
mordredand that's not likely to ever change16:46
SpamapSIts ok, I'm not coming for your pets. I just don't want to adopt any more. :)16:46
mordred:)16:46
*** patrickeast has quit IRC16:46
SpamapSI specifically wrote language last night in my infra-cloud doc that points out it's not a requirement of infra-cloud to run pets. :)16:47
*** wolsen_ is now known as wolsen16:47
* SpamapS should finish that and get it up for review. :_P16:47
mordredSpamapS: well, it may need to run a some not-very-loved pets16:47
mordredSpamapS: for instance, we run pypi mirrors in each nodepool cloud region16:47
SpamapSthats not really a pet16:48
mordredSpamapS: they're not 100% ephemeral - but we don't create/delete them frequently16:48
mordredyah16:48
SpamapSIt's expensive cattle.16:48
SpamapSThere's nothing special about each mirror other than you only need one.16:48
SpamapSSome day somebody should figure out how to mirror pypi onto swift really.16:48
SpamapSand git16:49
SpamapSand all the things16:49
mordredSpamapS: well, that's actually the opposite direction from what we've been looking at16:49
SpamapSrunning devpi?16:49
*** marun has quit IRC16:49
mordredgod no16:49
mordredbecause it would increase rather than decrease the exposure to non-atomic updates16:49
mordreddevpi not so much with being scalable16:50
*** shashankhegde has quit IRC16:50
SpamapSYeah swift can't do rsync/btrfs-cow-snap+symlink magic can it?16:50
mordredSpamapS: although, I DO agree that mirroring in to swift is a general thing that people shold figure out for this16:50
*** marcusvrn2 has joined #openstack-infra16:51
*** shashankhegde has joined #openstack-infra16:51
fungiSpamapS: keep in mind that the design of our pypi mirrors is mostly intended to mitigate the issue that networking in and at the edge of clouds sucks16:51
jeblairameade: we think that the problem has been going on for some time (longer than apr 29), but we didn't have the info in place to determine what accounts were involved.16:51
*** abregman has quit IRC16:51
SpamapSfungi: typically object storage has a fairly fat pipe to compute .. unless you're doing it wrong.16:51
mordredSpamapS: well, our current plan for atomic updates is based on afs volume publishing, rather than btrfs - but yes, that is the nature of the problem with using something like swift for this specific usecase16:51
*** marun has joined #openstack-infra16:51
mordredSpamapS: not to object storage16:51
mordredSpamapS: the problem that we're solving is that network access to pypi things outside of the cloud is bad16:52
jeblairameade: and in general, it should be fine to leave the stream event ssh connections open, but i could see the possibility that if your local script stopped reading from them, gerrit might hang on a write call if it isn't select()ing properly for writes16:52
jeblairameade: so i guess that sounds like a good starting point -- maybe we should re-enable the account then and see if it happens again?16:52
SpamapSmordred: right, I got discombobulated in that context.16:52
*** Ala has quit IRC16:53
*** marcusvrn1 has quit IRC16:53
jeblairSpamapS, mordred: (maybe swift with two containers and some dns switching or something)16:54
jeblairbut yeah, honestly it'd be better if that were _in_ swift16:54
jeblaircontainer symlink?16:54
jeblairor cow/release16:55
*** crc32 has joined #openstack-infra16:55
openstackgerritFabien Boucher proposed openstack-infra/system-config: Let puppet-zuul manage log configuration files  https://review.openstack.org/18062416:55
ameadejeblair: yeah i think that is the only way to be sure, the script was indeed running before april 29th, it just got restarted then16:56
*** claudiub has joined #openstack-infra16:56
*** sdake_ has joined #openstack-infra16:56
ameadejeblair: it's at least easier for yall to detect now right?16:56
*** gordc has quit IRC16:57
*** peristeri has quit IRC16:57
openstackgerritRenat Akhmerov proposed openstack-infra/project-config: Rename stackforge/mistral to openstack/mistral  https://review.openstack.org/17532816:58
SpamapSjeblair: I wonder if CEPH allows CoW via the swift frontend.16:58
mordredjeblair, SpamapS: I could see something like that potentially working - it's really atomic update of the index file that you care about - especially if you handle purges in a lazy manner16:58
jeblairameade: well, unfortunately we tend to only notice when it goes belly-up; but yeah, we can associate the problem connections with an account now16:58
SpamapSlike, "snapshot this container, now expose it as a new container, now atomically replace the old one with the new one"16:58
jeblairameade: okay, it's re-enabled; let's give it a shot16:59
ameadejeblair: awesome, thanks and sorry for the pain17:00
*** sdake has quit IRC17:00
*** hasharMeeting has quit IRC17:00
fungijeblair: if you want to compare the jstack trace i have from a couple weeks ago with a contemporary connections and queue list (from shortly after we got your patch onto review.o.o successfully), let me know. i still have it all17:01
*** derekh_ has quit IRC17:01
openstackgerritAlec Hothan proposed openstack-infra/project-config: Need to allow merge commits in vmtp.git  https://review.openstack.org/18061617:01
*** shashankhegde has quit IRC17:02
*** pc_m has joined #openstack-infra17:02
jeblairfungi: thanks, i'm pretty sure i recognized the stack traces as the usual one we've been seeing17:02
*** xyang1 has quit IRC17:02
fungijeblair: i meant as far as correlating to who the stuck account was from them as well17:03
*** xyang1 has joined #openstack-infra17:03
fungior i'll see if i can map it up myself from what's there17:03
*** shashankhegde has joined #openstack-infra17:04
*** ayoung has quit IRC17:04
jeblairfungi: oh, i thought you said it was the same?17:04
fungijeblair: no, from a prior similar incident while you were travelling17:04
*** tonytan4ever has quit IRC17:04
fungijeblair: april 2917:04
fungiso i guess ~week ago17:05
*** SumitNaiksatam has quit IRC17:05
*** xyang1 has quit IRC17:05
pc_mAnyone know of a way to copy some files with commit history from one repo to an existing stackforge repo?17:06
openstackgerritSirushti Murugesan proposed openstack-infra/project-config: Add grenade jobs for Heat  https://review.openstack.org/17835217:07
fungijeblair: on review.o.o in my homedir there are three jstack traces (jstack.1 through .3) taken ~10-20 seconds apart while stream-events was hung, and i just copied files named show-connections and show-queue up there which were redirects from stdout taken in the same timeframe17:07
*** gyee has joined #openstack-infra17:08
jeblairfungi: yeah, so all the stream events workers were servicing netapp-ci17:08
*** martingc has left #openstack-infra17:09
*** patrickeast has joined #openstack-infra17:10
anteayapc_m: not easily, what has happened that you are in this situation17:10
fungithat seems like a good correlation with yesterday in that case17:10
*** sdake has joined #openstack-infra17:10
jeblairyolanda, clarkb: so when i look at 161663, i actually wonder how much jeepyb stuff should be in the gerrit module?17:10
pc_manteaya: There are some vendor plugin files in VPN repo that I'd like to move out to stackforge, but would like to preserve history if possible.17:11
clarkbjeblair: you mean collapse the jeepyb module into gerrit?17:11
jeblairyolanda, clarkb: does that belong in openstack_modules/gerrit or puppet-openstackci, etc...?  i suppose it depends on how much we think jeepyb is actually required to use gerrit17:11
jeblairclarkb: no, the opposite17:11
mordredjeblair: move things rfom gerrit into jeepyb module17:11
jeblairno let me start over17:11
anteayapc_m: to a new stackforge repo, easy to do, to an existing stackforge repo tricky to impossible17:12
jeblair161663 is a change to puppet-gerrit that invokes code in puppet-jeepyb.  that means puppet-gerrit knows about jeepyb and i'm not sure it should17:12
cinerama pleia: hi there17:12
*** yamahata has quit IRC17:12
clarkbjeblair: I see,17:12
anteayapc_m: the impossible part being the amount of infra support you would need to get there17:12
cineramapleia2: so i added some bugs, but i feel like i'm missing one17:12
pc_manteaya: existing repo, where there are already other drivers/plugins.17:12
clarkbjeblair: basically puppet-gerrit and puppet-jeepyb should be invoked at the same level, but not nested like that17:12
clarkbjeblair: I think that makes sense to me17:12
pleia2cinerama: added bugs to buzilla?17:13
jeblairya, that's my gut reaction.  i haven't actually traced things out to make sure it works :)17:13
mordredjeblair: ++17:13
pc_manteaya: can you elaborate on what you mean by "infra support"?17:13
cineramapleia2: this thing is kind of weird. so i thought we couldn't add e.g. zh-tw but it seems to detect it; it just doesn't appear in the little dropdown thing17:13
cineramapleia2: yep. i added them to the etherpad17:13
anteayapc_m: well you would have to convince fungi or one of the others to walk you through the process17:13
cineramapleia2: i feel like there is one more bug we discussed last night that needs to be added17:13
anteayapc_m: but after we did this in december with the *aas repos we said we wouldn't do it again17:13
pleia2cinerama: huh, that's weird17:13
anteayapc_m: it isn't a supported workflow17:13
*** sdake__ has joined #openstack-infra17:14
pleia2cinerama: I'll re-browse logs to see if I can dig it up17:14
*** sdake_ has quit IRC17:14
cineramapleia2: you can try it for yourself if you go to add language. also you can add totally random locales that make no sense17:14
pleia2cinerama: oh goodie, testing...17:14
pc_manteaya: I'm looking at this purely as a move of modules from VPN repo to stackforge. I know I can just copy the files and then remove from vpn repo, and I guess the history will still be there in neutron-vpnaas, but would like to have history in stackforge.17:14
fungipc_m: you won't be able to put the additional files through code review and still retain their original history. there are ways to directly manipulate the files in the repository "behind gerrit's back" so to speak, but they involve having permission to push commits past gerrit17:15
openstackgerritSamuel de Medeiros Queiroz proposed openstack-infra/devstack-gate: Remove arbitrary devstack config options  https://review.openstack.org/17998217:15
pc_manteaya: Not looking for infra support, other than ideas on what my options are and how I could do it.17:15
sdake__hey quick q relating to creating a branch17:15
sdake__I submitted a change request to add https://github.com/openstack-infra/project-config/blob/master/gerrit/acls/stackforge/kolla.config#L317:15
sdake__yet rgerrit doesn't have a create branch button17:15
anteayapc_m: split the code from the existing repo and create a new stackforge repo with teh new split out code17:15
anteayapc_m: that is the supported workflow17:16
*** dizquierdo has joined #openstack-infra17:16
pleia2cinerama: I don't understand this add language UI at all17:16
*** achanda has joined #openstack-infra17:17
fungipc_m: basically our gerrit is not set up in such a way as to allow projects to use the full flexibility of git when it comes to repository management. it's implemented with the intent of preventing anyone from bypassing code review for changes to a repository it manages. those two goals tend to be in conflict with one another at times17:17
* pleia2 seeks docs17:17
pc_manteaya: How is the commit history preserved?17:17
mordredfungi: I disagree17:17
mordredthere is no reason a stackforge project coudl not grant itself permissions to do direct push to support something like this17:17
cineramapleia2: file a bug17:17
cineramapleia2: :)17:18
*** sdake has quit IRC17:18
mordredwe don't enforce policy on stackforge folks17:18
clarkbjeblair: I am going to take your gearman-plugin change, stack mine on it locally, and test the pair with my test setup. If that works I move for merging both changes and restarting one master with them17:18
fungimordred: you disagree that our current gerrit implementation is designed to support projects recombining bits and pieces from multiple existing git repositories?17:18
pc_mfungi: forgive my ignorance on this... if the destination is a vendor repo, is there some way to bring in the full history?17:18
cineramapleia2: but seriously, yes, it is kind of strange. i see the possible intent (namely, let the user add the language with the name they want)17:18
jeblairmordred: we don't what?17:18
pleia2cinerama: nods, going to see if I can figure out how they think it works, and tell them that my brain explodes on it17:18
clarkbjeblair: that seem sane? and while that is going on I can make my use CANT_DO patch better17:18
cineramapleia2: but if we can't delete stuff, that is a recipe for having lots of dupe entries17:18
pc_mfungi: Or some way to replay the commits?17:19
fungimordred: i suppose we could just grant push --force permissions in stackforge repo acls and tell the devs using those that they're on their own if they shoot themselves in the foot with it17:19
clarkbso I wrote the initial "this is stackforge" doc on ci.openstack.org17:19
jeblairfungi, mordred: please no17:19
clarkbit explicitly says "it is a way to use our tools and process"17:19
jeblairclarkb: yes17:19
clarkbif people want force push use github17:19
sdake__hey folks, I'm in the kolla release group here: https://review.openstack.org/#/admin/groups/461,members but I can't create a branch in gerrit17:19
sdake__any suggestions?17:19
mordredwow. ok. that's not what I'm suggesting17:19
mordredI'm suggesting that we have two conflicting things here17:19
mordredone is that for a project to be able to slurp in some additional history usually requires one of us to do a push to a ref location as admins17:20
mordredand we don't like doing admin things like that for stackforge projects if we can avoid it17:20
mordredso - one of the solutions to that woudl be for the stackforge project to self-service the sequence of actions needed to accomplish such a thing17:20
fungipc_m: you could repropose each non-merge commit in a branch created by using git filter-branch to split out the history for the files you want, as a way to retain their original authorship information17:20
*** HeOS has quit IRC17:21
mordredincluding temporaryily granting permisions to push to a ref and then revoking it after. or we could do a push for someone.17:21
fungipc_m: but it wouldn't preserve other related parts of the git history such as timestamps, review details, et cetera17:21
mordredit's not that it's not possible17:21
pc_manteaya: fungi: Let me try to rephrase the question... If I have vendor service and driver modules in the neutron-vpnaas repo, and I want to move those files to a stackforge repo (networking-cisco), what options do I have and what are the consequences of the options?17:21
cineramapleia2: i'm kind of wondering how we will test the proposal scripts17:21
cineramapleia2: suggestions are welcome17:22
*** gyee has quit IRC17:22
*** shardy has quit IRC17:22
jeblairpc_m: do the two git repos share a common git history (perhaps far in the past?)17:22
*** otter768 has joined #openstack-infra17:22
*** dizquierdo has quit IRC17:23
fungipc_m: you could copy those files from one repo to the other and propose it as a new change to the destination repo, followed by a change to delete them from the original repo (and include some provenance details in the commit message, such as co-authored-by lines and original commit shas). you lose all actual git metadata that way, but it requires no modification to current acls at least17:23
*** amitgandhinz has joined #openstack-infra17:23
pleia2cinerama: the actual commit-to-gerrit bit?17:23
sdake__for kolla shouldn't I be able to create a branch here: https://review.openstack.org/#/admin/projects/stackforge/kolla17:23
sdake__I am in the create group in project-config17:23
clarkbsdake__: https://review.openstack.org/#/admin/projects/stackforge/kolla,branches there17:24
pc_mjeblair: Well... the neutron-vpnaas repo came from neutron repo. The networking-cisco repo came from neutron and has ML2 stuff in it I think.17:24
sdake__clarkb thanks!17:24
cineramapleia2: yeah. i guess i can kind of comment the gerrit bits out and have at it17:24
pleia2cinerama: we can use the sandbox repo https://git.openstack.org/cgit/openstack-dev/sandbox17:24
clarkbsdake__: the link is in the top panel tab thing fwiw17:24
cineramapleia2: that might be a good idea17:25
sdake__clarkb I guess I missed that :)17:25
*** sabeen1 has quit IRC17:25
*** ivar-lazzaro has joined #openstack-infra17:26
pc_mfungi: Do you mean, in the new commit, refer to the shas from the old repo's modules?17:26
fungiyeah, if you wanted to retain some paper trail at least17:27
*** SumitNaiksatam has joined #openstack-infra17:27
*** tonytan4ever has joined #openstack-infra17:27
*** otter768 has quit IRC17:27
fungipc_m: you could also git filter-branch the files you want to move into a separate branch, pull it into a new branch in a local clone of the target repository, then rebase those changes onto the intended branch and repropose the result to code review (which would have the effect of preserving the authorship info, though not really much else)17:27
asselin__lifeless, so I submitted these 2 patches, but I thought you said you audited pbr settings for upstream? https://review.openstack.org/#/c/180626/  https://review.openstack.org/#/c/180622/17:27
*** ivar-lazzaro has quit IRC17:27
pleia2cinerama: so it seems like the add translations interface is built around adding a language that already exists in their database, right? how did you add different cased en-au?17:27
pc_mfungi: The copying was what I was originally thinking. I was just wondering if the new repo could maintain the git history too.17:28
*** boris-42 has quit IRC17:28
*** ivar-lazzaro has joined #openstack-infra17:28
cineramapleia2: you just hit save on whatever you type rather than selecting from the dropdown17:29
clarkboh sweet the stop sending status updates change has laredy merged, makes my testing a little easier17:29
jeblairwoo!17:29
*** marcusvrn has quit IRC17:29
jeblairclarkb: let me know if that doesn't work ;)17:29
fungipc_m: to retain the git history, you'd need to do something similar but merge the new branch into the existing one and push --force it into gerrit (and cross your fingers that it works, because you're bypassing code review testing)17:30
clarkbjeblair: will do17:30
pleia2cinerama: I see, and if you create one that's not in their database, it autofills info below... but gives no way to edit it17:30
pc_mfungi: 99% of the files were authored by me, so I'm not too worried about that. More of just an interest to have the commit history, in case people wanted to know why changes were made.17:30
cineramapleia2: yup. so you can have lots of spam17:30
pleia2cinerama: could you find a way to delete them?17:30
cineramapleia2: also i did something weird to my project i was testing on and i can't fix it17:30
cineramapleia2: no, that's a different bug i filed :)17:31
pleia2aha :)17:31
cineramapleia2: see if you can reproduce this: go on your project and find the thing where you can manage languages17:31
cineramapleia2: there is a choice (i forget what now) where you can limit the languages17:31
cineramapleia2: but if you pick it IIRC the state looks the same the next time you visit the dialog but getting the xml from rest won't grab all the languages17:32
jeblairmordred: why did you not remove python-yaml in 167610?17:33
pc_mfungi: on the new branch option... would an empty directory structure be added to the target repo and committed as the first commit, and then branch to do the rest? Trying to understand how to stage that in the new repo.17:33
pleia2cinerama: so there's a "Inherit languages from project settings." checkbox17:34
*** AJaeger_ has joined #openstack-infra17:35
clarkbjeblair: I am still seeing a WORK_STATUS between GRAB_JOB and SUBMIT_JOB, I assume that is expected since the statuses that were removed happen after SUBMIT_JOB17:35
fungipc_m: filter-branch creates a new branch in the starting repository, then you'd want to pull that branch into the target repository, probably as a new branch so you can have a little more flexibility on how you combine it with the intended existing branch there17:35
*** ivar-laz_ has joined #openstack-infra17:35
cineramapleia2: yes, i think once ticked, there is no going back17:35
asselin__jeblair, not sure we can +w this yet...https://review.openstack.org/16742617:36
jeblairclarkb: er, i expect: >GRAB_JOB_UNIQ <JOB_ASSIGN_UNIQ >WORK_STATUS >WORK_COMPLETE17:36
*** mattfarina has joined #openstack-infra17:36
jeblairasselin__: depends on should dtrt17:36
*** neoice_ is now known as neoice17:36
mordredjeblair: hrm. I feel like there was a reason I did not17:36
fungipc_m: pull may be a poor choice of wording there. you'd want to add the starting repository as a remote and then checkout that remote as a new local branch in the target repo17:36
asselin__jeblair, dtrt?17:36
pleia2cinerama: wow, yeah, that's a mess17:36
asselin__ok...do the right thing17:36
jeblairasselin__: do the right thing.  sorry -- the depends-on header in the change i +Ad will prevent it from actually landing until the one it depends on lands.  and zuul should actually correctly enqueue it automatically now.17:37
fungipc_m: where in this case the "remote" repo is likely a direct file path or file:// url to your local surgery in the originating repo17:37
clarkbjeblair: geard doesn't seem to log the JOB_ASSIGN_UNIQ let me find a good chunk of logs to paste17:37
jeblairasselin__: so basically i said "i'm comfortable landing this as soon as the dependency lands, especially sinc zuul will still test it one last time right before it lands"17:38
*** pelix has quit IRC17:38
pc_mfungi: I think I got it. thanks.17:38
mordredjeblair: nope. I think you're righ t- I think it should come out17:38
pleia2cinerama: want me to submit this bug?17:38
*** ivar-lazzaro has quit IRC17:38
jeblairasselin__: that way i can knock it off my list and we don't have to worry about remembering it later.17:38
cineramapleia2: please do. it's a stumper17:38
pleia2k17:38
openstackgerritMonty Taylor proposed openstack-infra/puppet-nodepool: Consume puppet-diskimage_builder  https://review.openstack.org/16761017:38
mordredjeblair: thanks17:38
*** btully has joined #openstack-infra17:38
asselin__jeblair, ok thanks. I will see what's left on the depends on so we can get that pushed through too17:38
pc_mfungi: Any suggestions on which of those approaches would make the most sense to attempt (given the conditions: existing repos, me as author on files)?17:39
*** emagana has joined #openstack-infra17:39
clarkbjeblair: oh I may have mixed some of the streams, noticing the ports are different now, http://paste.openstack.org/show/215482/17:40
jeblairmordred: i see a lot of dib nodepool patches in merge conflict; what's the skinny?17:40
clarkbjeblair: I think that looks correct to me if I account for the ports being different workers17:40
mordredjeblair: I need to make another pass through the17:40
fungipc_m: part of that determination depends on how many non-merge commits you end up with in the filter-branch result, and whether you really need to preserve the original timestamps17:40
clarkbjeblair and submits coming from the clients17:40
jeblairclarkb: ah yeah, crossing streams bad17:40
mordredjeblair: I'll get that cleaned up today17:40
jeblairmordred: ok, wasn't sure if we were waiting on something17:40
fungipc_m: and also if you have multiple branches (for example stable branches) you need to do similar things to17:41
AJaeger_cinerama: I tested the proposal scripts locally17:41
pc_mfungi: Is there a way I can introspect that on the files in the neutron-vpnaas repo?17:41
AJaeger_cinerama: I basically comment out the function for sending the patch to gerrit and also some setup - and then run locally in my environment17:41
cineramaAJaeger_: ok sounds good17:41
*** alexpilotti has quit IRC17:41
fungipc_m: i believe git log <some filename> will give you commit logs relevant to a specific file, without having to dig into how to actually split those commits out first17:41
AJaeger_Once that works fine, I send the patch and monitor that everything really works as it should17:42
jeblairclarkb: ah, yeah, forgot initial work_data.  looks good though.17:42
asselin__clarkb, perhaps you can readd your +2 to this and we can merge? https://review.openstack.org/#/c/167425/17:42
AJaeger_cinerama: So, checking https://review.openstack.org/#/q/status:open++branch:master+topic:transifex/translations,n,z and the latest builds here https://jenkins.openstack.org/17:42
AJaeger_So, I review all failures for the proposal jobs and the content of the send patches...17:43
AJaeger_cinerama: The proposal job runs every morning at 6:00 UTC17:43
jeblairnibalizer: did you ever have that conversation about your patch?17:43
*** Sukhdev has quit IRC17:43
clarkbasselin__: I would need to rereview it17:43
*** eharney has quit IRC17:43
fungiAJaeger_: what was the name of the repo for the tag jobs you wanted triggered?17:44
AJaeger_fungi: openstack-doc-tools17:44
AJaeger_fungi: version 0.2617:44
asselin__clarkb, yes, ofcourse, rereview, and then readd :)17:44
zaromorning17:44
nibalizerjeblair: no, clarkb and I haven't circled around on that yet17:44
fungiAJaeger_: thanks, starting on that now17:44
AJaeger_thanks a lot, fungi!17:44
anteayamorning zaro17:44
jeblairnibalizer: i was just about to review changes that i think might conflict with it and i thought it might be nice not to merge those.  :)17:45
nibalizerclarkb: got time today to lay out your plan?17:45
nibalizerjeblair: i appreciate that17:45
clarkbnibalizer: I thought I wrote it down somewhere, basic idea was move each logical chunk of the file at a time17:45
clarkbnibalizer: to reduce the number of variables that can break17:46
clarkband since that file is already organized into chunks it should be simple to find those boundaries17:46
*** vhoward has left #openstack-infra17:46
jeblairclarkb: do you think that will reduce the overall risk?  or does that just mean we have 10 chances to break everything? :)17:46
*** alexpilotti has joined #openstack-infra17:46
pc_mfungi: Overall, we're talking about 5 files and 3 test files.17:46
*** gyee has joined #openstack-infra17:46
*** Sukhdev has joined #openstack-infra17:47
clarkbjeblair: I think it confines the risk, last time we did this every single host was affected, but in this case we can make the centos changes and make sure the git mirror is happy before moving on and making the afs changes and so on17:47
*** freerunner has quit IRC17:47
nibalizerhttps://review.openstack.org/#/c/171359/ is the review17:47
jeblairclarkb, nibalizer: (this will be so much nicer when we have high-level functional testing, but we have what we have now :|)17:47
clarkbjeblair: also I can trigger jobs and zaro's test case of offline node, submit job, online node is working. So I think I am about as ready on my change as I will be17:47
jeblairclarkb: re gearman: cool17:48
*** Swami has joined #openstack-infra17:48
*** craigbr has quit IRC17:48
nibalizerso one thing to note is that from the perspective of doing it, and I know gerrit doesn't show it that way, is that its a 10 line change :)17:48
clarkbnibalizer: sort of,17:48
clarkbnibalizer: you have to keep in mind that if you have a stray include somewhere then its a 300 line change17:48
nibalizerwhat has happened is 10 lines from one file has moved into another file, then the file was renamed to the first file17:49
*** freerunner has joined #openstack-infra17:49
clarkbnibalizer: yes but names matter in puppet17:49
nibalizerso this file is not very big http://git.openstack.org/cgit/openstack-infra/system-config/tree/modules/openstack_project/manifests/server.pp17:50
nibalizerthe english version of the change is to put that exim resource (behind a feature flag) into template17:50
*** alexpilotti_ has joined #openstack-infra17:50
clarkbbut do so with the server name17:50
*** e0ne has joined #openstack-infra17:50
nibalizerso one idea would be to do moving exim behind a feature flag in one change, then just switch servers one at a time to use template directly instead of server17:51
nibalizerthen server gets deleted ?17:51
*** alexpilotti has quit IRC17:51
*** alexpilotti_ is now known as alexpilotti17:51
zaroclarkb: i have tried with jjb and looks good17:52
jeblairnibalizer: that sounds good, and i think would reduce overall risk17:52
jeblair(at the expense of probably a bit more infra-root time)17:53
clarkband considering we flattened base into template that makes sense to me17:53
*** ZZelle has quit IRC17:53
clarkbzaro: thanks17:53
*** kgiusti has quit IRC17:53
*** ZZelle has joined #openstack-infra17:53
jeblair(i'd probably still rename template->server or something when all is completed; template still means "nodepool template" to me)17:54
clarkbzaro: https://review.openstack.org/#/c/180371/ is a followup change to make function registration quicker, but it doesn't work quite right yet. I left a paste with the geard logs showing it unregistering some functions that should remain registered (I think)17:54
fungimordred: lifeless: we have a new pbr challenge i think... AJaeger_ pushed a "0.26" tag to openstack/openstack-doc-tools which caused pbr to generate wheel and sdist files with 0.26.0 in their names, but our pypi upload script expects that to match the tag17:55
clarkbfungi: I think we need to enforce semver's three tuple in zuul layout now17:55
fungicurious as to what you think is the best workaround for that17:55
jeblairnibalizer, clarkb: do we have a plan?17:55
clarkbjeblair: nibalizer ya that proposal sounds good to me17:56
*** cody-somerville has quit IRC17:57
*** tim_o has quit IRC17:57
AJaeger_fungi: It wasn't me that actually pushed the tag ;) but I asked for it, so I'm still guilty ;) And that's how we did it for some time.17:57
AJaeger_fungi, should I ask for a 0.26.0 tag to get pushed instead?17:58
AJaeger_Or do we need 0.26.1 now?17:58
pc_mfungi: So it looks like all the files have an original committer of dougwig, as he created the *aaS repos, the second version are all owned by me.17:58
*** signed8bit has joined #openstack-infra17:58
fungiAJaeger_: yep, not at all anyone's fault, this is an unanticipated interaction between our release automation tooling and the latest pbr release17:58
mordredfungi, clarkb: I think enforcing the 3 tuple is a good idea17:59
fungiAJaeger_: i _think_ a 0.26.0 tag will work and replace the existing tarballs on tarballs.o.o and upload normally17:59
AJaeger_fungi: ok, will ask loquacities and annegentle for it.17:59
clarkbshould check it before pushing it though17:59
clarkbdo a `python setup.py sdist` and check the output17:59
jeblairnibalizer: if that sounds good to you and you think we can get those merged by end of next week, i'll procedural-2 changes that touch server.pp and template.pp;  need your ack first though.18:00
fungiclarkb: AJaeger_: right. and even pip install the resulting tarball into a fresh virtualenv for a sanity check18:00
nibalizerjeblair: works for me18:01
nibalizerill spin those patches up today18:01
jeblair(ooh -- new gertty feature idea: apply the same review to a set of changes)18:01
nibalizerjeblair: call it 'the hammer'18:01
fungiclarkb: mordred: i'll spin up a straw man change to the (pre-)release regex for layout.yaml now18:01
nibalizeronly when minus reviews tho18:01
crinklejeblair: the stackforge modules would use that :)18:01
mordredfungi: woot18:02
AJaeger_fungi, clarkb: So, installing pbr 0.11 first?18:03
clarkbAJaeger_: if you use a fresh venv ou should get that version18:04
fungiAJaeger_: tox -e venv python setup.py sdist18:04
clarkblocally it works if I tag 0.26.0 and sdist and pip install .18:04
fungiAJaeger_: that's how the job is building the tarball18:04
AJaeger_clarkb: yeah, that's what I did as well...18:05
AJaeger_fungi, clarkb: Thanks18:05
fungiAJaeger_: git clean -dfx first or add -r to the tox invocation to make sure you get latest pbr18:05
fungiAJaeger_: you can also double-check after by running .tox/venv/bin/pip list18:05
*** ZZelle_ has joined #openstack-infra18:06
jeblairnibalizer: amusingly i -2d another of your changes :)18:06
pc_mfungi: I did git log of all the files and there is a total of 17 unique commits for all the files.18:06
AJaeger_thanks, fungi. Looks fine: "openstack-doc-tools (0.26.0, /home/aj/Software/vcs/OpenStack/openstack/openstack-doc-tools)"18:07
jeblaircrinkle: what's the use case for the stackforge modules?18:07
pleia2cinerama: ok, so I figured out what it's doing, it does actually go back to including all 40 languages when you check "Inherit languages from project settings." again (see the top of the screen after nagivating away from that tab in the UI), but when you uncheck it again, it preserves the shorter list, I wouldn't have expected this behavior (I'd expect it to go back to listing all 40)18:07
clarkbasselin__: done, the delta was small so it was easy :)18:07
anteayapc_m: would you consider adding the git log as a readme file and copying the code as new files to the stackforge repo?18:08
asselin__clarkb, \0/18:08
AJaeger_clarkb, fungi : So that looks sane - thanks!18:08
crinklejeblair: sometimes we have to emergency pin a bunch of gems across 20 modules, so getting to +2 on a bunch of patches would be neat18:08
openstackgerritClint 'SpamapS' Byrum proposed openstack-infra/shade: Fix delete_server when wait=True  https://review.openstack.org/18065918:08
anteayapc_m: it seems like a lot of work to preserve the history of 17 commits18:08
pc_mfungi: I'm wondering if it is best to just copy as new files and reference the old repo shas18:08
pc_manteaya: Yeah, seems like it.18:08
*** freerunner has quit IRC18:08
anteayapc_m: will save you a ton of work and pain18:09
fungipc_m: pragmatism ftw!18:09
jeblaircrinkle: ah cool.  i think it should be doable; i'll put it on the backlog18:09
cineramapleia2: oh weird18:09
pc_manteaya: Would it be better to put those commits into a README, or include them in the commit message?18:09
clarkbjeblair: do you want me to approve my gearman-plugin change before or after running it on say jenkins07?18:09
*** markmcclain has quit IRC18:09
clarkbjeblair: also would you like to rereview prior?18:09
*** doug-fish has left #openstack-infra18:09
*** signed8bit has quit IRC18:09
SpamapSShrews: https://review.openstack.org/180659 is the fix for delete_server18:10
SpamapSmordred: ^18:10
jeblairclarkb: was just about to go re-review; i think it's fine if you want to test it on 07 now.18:10
pleia2cinerama: indeed, I'll still submit the bug, they may say it's expected behavior (having to add back list of default languages is easier than remembering+culling the list again if you unchecked accidentally?) but it's not what I expected18:10
cineramapleia2: ok. so not a "bug" per se, but Shenanigans(tm)18:11
*** signed8bit has joined #openstack-infra18:11
pleia2cinerama: yeah, at the very least they should document it18:11
*** freerunner has joined #openstack-infra18:11
*** reed has quit IRC18:11
*** eharney has joined #openstack-infra18:12
*** ryanpetrello_ has quit IRC18:12
clarkbok putting jenkins07 in shutdown mode now, unless. pleia2 you look busy, but any interest in doing a jenkins plugin upgrade from a source build?18:12
clarkbpleia2: I can wait a bit too if you are but need to finish other things first18:12
*** f1ller is now known as filler18:12
*** _nadya_ has quit IRC18:13
openstackgerritJeremy Stanley proposed openstack-infra/project-config: Only support PBR>=0.11.0 semver release tags  https://review.openstack.org/18066018:13
fungiclarkb: mordred: lifeless: ^18:13
cineramapleia2: i'm uploading another thing now, going to see how the proposal goes18:13
cineramapleia2: oh CRAP. can you fix that tickbox or whatever? it's not pulling all the locales now18:14
*** Sukhdev has quit IRC18:14
cineramapleia2: that is the magic of the xml creation18:14
*** reed has joined #openstack-infra18:14
fungiclarkb: mordred: lifeless: while the regular expressions there check out (based on manual testing expected and unexpected strings), i'm unconvinced that assuming all projects must support version tags as if they're using pbr (even when they're not) should be necessary to take advantage of our pre-release and release pipelines18:14
*** e0ne_ has joined #openstack-infra18:15
*** e0ne has quit IRC18:15
cineramapleia2: if we can't pull down all the locales reliably in the xml we have to explicitly list them all on the command line for push/pull and this is a thing that i Do Not Want18:15
pc_manteaya: fungi: Thanks for all the useful advice. I think I have a handle on how to do this. To get full history of these files, we'd have to go way back to the original neutron repo, so it is probably not worth it.18:15
mordredfungi: that's a good point18:15
jeblairfungi: especially for non-python projects?18:15
clarkbfungi: mordred except this is more "use semver" than "use pbr"18:15
fungijeblair: especially for any projects18:15
clarkband it seems reasonable to say use semver18:15
clarkbfungi: I do have a comment or two as soon as I can finish verifying one thing18:16
fungiclarkb: well, "use pbr's pep-440+semver mutant hybrid concept of versioning)18:16
clarkbsure, but that only affects the pre release pipeline18:17
jeblairclarkb: lgtm will let you aprv on your schedule18:17
clarkbrelease is pure semver three tuple18:17
fungitrue enough18:17
*** e0ne has joined #openstack-infra18:17
pleia2cinerama: in my project? "Inherit languages from project settings." is selected18:17
pleia2cinerama: it thinks it has 40 languages to translate18:18
cineramapleia2: i just created a new project and now the file is stuffed for some reason18:18
cineramathis makes me sad18:18
pleia2ugh18:18
clarkbfungi: see comment18:18
*** ryanpetrello_ has joined #openstack-infra18:18
pleia2clarkb: I may be tied up for a bit, have this and then a couple other things on my plate today18:19
*** freerunner has quit IRC18:19
AJaeger_pleia2, cinerama: We better speed up - transifex has several "bad gateway" errors today ;/ hope they fix it...18:19
cineramapleia2: actually i might have made a subtle mistake18:19
clarkbpleia2: ok, I will likely move forward on this, but ping if you end up with free time and are interested18:19
pleia2cinerama: this is why we need to restrict language+project stuff to admins :)18:19
*** e0ne_ has quit IRC18:20
clarkb07 is in shutdown mode18:20
*** sdake has joined #openstack-infra18:20
fungiclarkb: good point. i need to double-check the actual pre-release normalization pbr's doing18:20
*** mrmartin has joined #openstack-infra18:21
cineramapleia2: yep, was my bug. carry on with the...other bugs :)18:21
openstackgerritMerged openstack-infra/puppet-openstackci: Create a log server like http://logs.openstack.org  https://review.openstack.org/16742518:21
fungiclarkb: based on http://git.openstack.org/cgit/openstack-dev/pbr/tree/pbr/tests/test_version.py#n30 i _think_ you're wrong, though i do need to remove the [0-9]* prefix since pbr is going to strip it18:22
pleia2cinerama: phew18:22
*** freerunner has joined #openstack-infra18:22
AJaeger_fungi, clarkb : We have 0.26.0 now - thanks! https://pypi.python.org/pypi/openstack-doc-tools/18:23
zaroclarkb: commented on change 179988 will take a look at the follow on change.18:23
clarkbfungi: hrm time to read the pep440 normalization rules I think18:23
*** sdake_ has joined #openstack-infra18:23
*** sdake__ has quit IRC18:24
cinerama[WARN] Skipping file glance/locale/pt_BR/LC_MESSAGES/glance-log-info.po; no locale entry found in zanata.xml18:24
cinerama[WARN] Skipping file glance/locale/en_GB/LC_MESSAGES/glance-log-info.po; no locale entry found in zanata.xml18:24
cineramaAARGH18:24
fungiclarkb: yeah, it looks like it will only generate pre-release versions matching ^([0-9]+\.){3}(a|b|rc)[0-9]+$18:24
cineramathis is why we put the mapping18:24
clarkbzaro: ya the node label thing should be unrelated to my change since it will just do registerFunctions if they have been updated by some external event18:24
clarkbfungi: it being pbr?18:24
fungiclarkb: yep, though i'm fine reworking that regex to be a visual superset of the release regex rather than a logical one18:24
clarkbfungi: see https://www.python.org/dev/peps/pep-0440/#pre-release-separators18:25
clarkbfungi: I think pbr is wrong if it adds the . :/18:26
cineramapleia2: ok i really don't get this one: [WARN] Skipping file glance/locale/fr/LC_MESSAGES/glance-log-error.po; no locale entry found in zanata.xml18:26
openstackgerritMerged openstack-infra/system-config: Use common log server implementation  https://review.openstack.org/16742618:26
cineramait's like, right there18:26
anteayapc_m: welcome glad you have a way forward, having the commits in a commit message would make for a very large commit message18:26
*** sdake has quit IRC18:26
fungiclarkb: i'm in agreement based on "The normal form for this is without a separator."18:27
clarkbfungi: I am running an sdist build of nova (should've picked a smaller repo)18:27
*** baoli_ has quit IRC18:27
cineramacrap and now it's doing the thing where it decides it doesn't like the .ini file18:27
*** SumitNaiksatam_ has joined #openstack-infra18:28
asselin__clarkb, jeblair so is logs.openstack.org now using puppet-openstack ci?18:28
jeblairasselin__: did the changes merge?18:29
asselin__yes18:29
clarkbwill have to wait for the next puppet run18:29
clarkbbut it should at that point18:29
adrian_ottois there a Gerrit admin around who can delete some duplicate branches for me?18:30
*** SumitNaiksatam has quit IRC18:30
*** SumitNaiksatam_ is now known as SumitNaiksatam18:30
clarkbwe only update puppet modules at the beginning of each run for both system-config and its deps18:30
clarkbadrian_otto: sure18:30
clarkbadrian_otto: what branches on which projects?18:30
adrian_ottomagnum and python-magnumclient both have a kilo-stable18:30
pleia2cinerama: hm, weird18:30
adrian_ottowe want to keep stable/kilo and delete kilo-stable18:31
adrian_ottoclarkb: make sense?18:31
clarkbadrian_otto: yup18:31
clarkbadrian_otto: all done18:32
*** mtanino has joined #openstack-infra18:32
cinerama[DEBUG] Loading configuration from the path /home/jenkins/.config/zanata.ini18:32
cinerama[ERROR] Execution failed:18:33
cineramaorg.zanata.client.exceptions.ConfigException: Username must be specified18:33
cineramathe username is in the @*$#% zanata.ini18:33
*** yamahata has joined #openstack-infra18:33
*** dannywilson has joined #openstack-infra18:34
* mordred hands cinerama a fluffy alapaca of happiness18:34
*** gyee has quit IRC18:37
pleia2software is horrible18:39
cineramamordred, you know i've never been real clear on the difference between alpacas and llamas. are alpacas better?18:39
nibalizerpleia2: not exactly 100% of it...18:39
nibalizerjust almost all of it18:39
pleia2alpacas are smaller18:39
mordredcinerama: I think alpacas also have nicer hair18:40
*** hashar has joined #openstack-infra18:40
jeblairllamas hold more water in their fur if you wanted to throw a wet one18:41
clarkbfungi: ok more info pbr made a 2015.1.0.0b1 so its normalized but != the tag18:41
fungimordred: emus are cool too. and they don't have any hair at all. though not in the way that worstcat has no hair18:41
mordredemus make strange noises18:41
anteayawould I want to throw a wet llama though18:41
anteayamy back says no18:42
jeblairmordred makes strange noises18:42
fungiclarkb: okay, so needs to be ^([0-9]+\.){3}0(a|b|rc)[0-9]+$ to match pbr 0.11.0 behavior18:42
*** doug-fish has joined #openstack-infra18:42
mordredjeblair: fungi says that ^^ and you think I make weird noises?18:42
jeblairmordred: i've seen your perl18:42
*** mattfarina has quit IRC18:42
*** markmcclain has joined #openstack-infra18:42
clarkbfungi: I think instead of {3}0 you need {3}[0-9]+18:42
fungiheh... your perl is showing18:42
fungiclarkb: pbr lets the leader on the alphabetic component be nonzero?18:43
clarkbfungi: because you *could* decide to have a .2b1? maybe?18:43
clarkbfungi: so many questions :) clearly we need simpler versioning18:43
clarkbI propose that all versions exist on a natural number line18:44
fungiclarkb: not a multidimensional array?18:44
clarkbnope, a single dimension of infinitely countable (the coutable is important here I think) values is what I want18:45
lifelessfungi: whats https://review.openstack.org/#/c/180660/ about ?18:46
fungiclarkb: i vote for a real number line instead. so i can begin versioning my releases with transcendentals18:46
clarkbfungi: anyways commented with what I have discovered so others can chime in there18:46
fungilifeless: a 0.26 tag was pushed to a pbr-using project18:46
fungilifeless: our tarball and wheel jobs of course ended up with 0.26.0 in the filenames18:46
lifelessasselin__: well, we haven't spoken to ttx yet18:47
fungilifeless: then our pypi upload job went to download the 0.26 tarball and wheel, which dne18:47
lifelessasselin__: There is a potential reason to keep the version clamps in servers, and until we've had that discussion I wouldn't advocate wholesale removal18:47
clarkbits a simplicity thing, our tags should match the versions we want because scripts and humans all have to understand them18:47
*** gokrokve has quit IRC18:47
*** openstackgerrit has quit IRC18:47
*** changbl has quit IRC18:47
*** markvoelker has quit IRC18:47
*** Longgeek has quit IRC18:47
*** jtomasek has quit IRC18:47
*** smccully has quit IRC18:47
*** russellb has quit IRC18:48
*** trey has quit IRC18:48
*** gema has quit IRC18:48
*** adreznec has quit IRC18:48
*** mfink- has quit IRC18:48
*** SotK has quit IRC18:48
*** tsufiev has quit IRC18:48
*** mrda has quit IRC18:48
*** ianw has quit IRC18:48
*** yamahata has quit IRC18:48
*** mrmartin has quit IRC18:48
*** e0ne has quit IRC18:48
*** ZZelle has quit IRC18:48
*** whoops has quit IRC18:48
*** ajmiller has quit IRC18:48
*** adrian_otto has quit IRC18:48
*** sc` has quit IRC18:48
*** mmedvede has quit IRC18:48
*** jogo has quit IRC18:48
*** rkukura has quit IRC18:48
*** zaneb has quit IRC18:48
*** mkoderer has quit IRC18:48
*** morganfainberg has quit IRC18:48
*** gibi has quit IRC18:48
*** asilenkov has quit IRC18:48
*** jesusaurus has quit IRC18:48
*** plol has quit IRC18:48
*** pothole has quit IRC18:48
*** vipul has quit IRC18:48
*** devananda has quit IRC18:48
*** mriedem has quit IRC18:48
*** niska has quit IRC18:48
*** cloudnull has quit IRC18:48
*** hogepodge has quit IRC18:48
*** raginbajin has quit IRC18:48
*** briancline has quit IRC18:48
clarkbas soon as I have to think about what the mapping is, trouble results (note that I am doing that now)18:48
SpamapSdohhhh18:48
asselin__lifeless, ok I see...please -1 then if you haven't already.....18:48
lifelessasselin__: the audit I did was for conflicting versions, not the presence of versions18:48
*** niska has joined #openstack-infra18:48
SpamapSShrews: good feedback on my delete_server fixes, posted an updated patch18:48
*** yamahata has joined #openstack-infra18:49
*** mrmartin has joined #openstack-infra18:49
*** e0ne has joined #openstack-infra18:49
*** ZZelle has joined #openstack-infra18:49
*** whoops has joined #openstack-infra18:49
*** ajmiller has joined #openstack-infra18:49
*** adrian_otto has joined #openstack-infra18:49
*** mriedem has joined #openstack-infra18:49
*** sc` has joined #openstack-infra18:49
*** mmedvede has joined #openstack-infra18:49
*** cloudnull has joined #openstack-infra18:49
*** zaneb has joined #openstack-infra18:49
*** jogo has joined #openstack-infra18:49
*** rkukura has joined #openstack-infra18:49
*** hogepodge has joined #openstack-infra18:49
*** devananda has joined #openstack-infra18:49
*** vipul has joined #openstack-infra18:49
*** pothole has joined #openstack-infra18:49
*** plol has joined #openstack-infra18:49
*** jesusaurus has joined #openstack-infra18:49
*** asilenkov has joined #openstack-infra18:49
*** gibi has joined #openstack-infra18:49
*** morganfainberg has joined #openstack-infra18:49
*** mkoderer has joined #openstack-infra18:49
*** raginbajin has joined #openstack-infra18:49
*** briancline has joined #openstack-infra18:49
*** gokrokve has joined #openstack-infra18:49
*** openstackgerrit has joined #openstack-infra18:49
*** changbl has joined #openstack-infra18:49
*** markvoelker has joined #openstack-infra18:49
*** Longgeek has joined #openstack-infra18:49
*** jtomasek has joined #openstack-infra18:49
*** smccully has joined #openstack-infra18:49
*** russellb has joined #openstack-infra18:49
*** trey has joined #openstack-infra18:49
*** gema has joined #openstack-infra18:49
*** adreznec has joined #openstack-infra18:49
*** mfink- has joined #openstack-infra18:49
*** SotK has joined #openstack-infra18:49
*** tsufiev has joined #openstack-infra18:49
*** mrda has joined #openstack-infra18:49
*** ianw has joined #openstack-infra18:49
lifelessclarkb: fungi: ok, ack.18:50
*** mattfarina has joined #openstack-infra18:50
fungilifeless: there are of course several potential places we can tackle that impedance mismatch18:50
fungibut all the ones i can think of result in some distasteful assumption or another18:51
lifelessone of the things we've wanted for ages18:51
*** mattfarina has quit IRC18:51
lifelessis 'run tests on releasing'18:51
lifelessif there was such a thing, we could make the tag based on the pbr output after running the tests18:52
fungiyep. getting some shim between pushing the tag and having the tag end up in the official repo and providing a feedback loop to tests run in that shim has been the challenge18:52
clarkblifeless: how would you trigger a release in that situation? I think we had some assumptions that test on releasing would be triggered by reviewable tags in gerrit18:52
clarkblifeless: we could make gerrit -1/-2 in that situation if the tag was wrong, but we would have to propose a new tag18:53
lifelessso I don't know all the bits limitations - I hesitate to propose implementation18:53
lifelessbut in broad strokes we want a human to say 'please release version <loose string>', a robot to verify 'this release won't explode the world', and then to make a canonicalised tag.18:54
*** ajmiller has quit IRC18:54
lifelessjenkins jobs via zuul are the obvious middle step18:55
*** ajmiller has joined #openstack-infra18:55
fungii can think of ways to work around the lack of actual tag review in gerrit, such as having a separate tags repo where tags are uploaded as files/blobs in new commits, then tested by applying into the actual target repo and returning test results to the review on that change18:55
fungibut that's pretty ugly and hard for anyone besides us to comprehend18:55
clarkbya I mostly wanted to not overengineer a distant future today, when the problem we have is evident as of yesterday18:56
reedwhich version of puppet do you guys use?18:56
lifelessk18:56
clarkbreed: latest 3.X, I think 3.7.something18:56
reedACK18:57
*** mattfarina has joined #openstack-infra19:00
lifelessthe spec for pbr had a CLI that would report the version19:00
lifelesswe have the CLI now19:00
lifelessbut nothing much behind it19:01
*** craigbr has joined #openstack-infra19:01
*** soren has joined #openstack-infra19:01
*** hashar has quit IRC19:02
*** freerunner has quit IRC19:03
*** Sukhdev has joined #openstack-infra19:03
fungilifeless: related to that discussion, clarkb and i read pep-440 to indicate that 1.2.3a4 should be already normalized, rather than normalize to 1.2.3.0a419:03
fungihow did pbr arrive at the latter?19:04
*** jtriley_ has joined #openstack-infra19:04
fungi(if you recall)19:04
*** annegentle has joined #openstack-infra19:06
*** freerunner has joined #openstack-infra19:06
*** wayneeseguin has quit IRC19:06
*** gordc has joined #openstack-infra19:06
*** jtriley has quit IRC19:06
clarkbwaiting on one more job to finish on jenkins07 before I can apply the gearman plugin update there. Going to grab lunch while I wait19:07
*** gyee has joined #openstack-infra19:07
*** achanda has quit IRC19:07
*** ivar-laz_ has quit IRC19:07
clarkbasselin__: http://puppetboard.openstack.org/report/static.openstack.org/dba07e303ea584fee3a4bfb34d90966e5dc04c6b19:08
clarkbasselin__: other than that I think it looks good19:08
*** ivar-lazzaro has joined #openstack-infra19:08
asselin__clarkb, ok cool! will look into that...19:08
*** freerunner has quit IRC19:09
fungiclarkb: keep in mind that puppet is still disabled on all the jenkins masters19:10
lifelessfungi: we wanted to keep the semver component strictly semver19:10
lifelessfungi: we're in a half-breed situation basically19:11
fungiclarkb: i'm about done confirming that jjb updates are complete and no-op on rerunning so i can reenable19:11
*** hashar has joined #openstack-infra19:11
fungilifeless: okay, makes sense. to me the details of fine-tuning that regex are interesting, but academic until we determine whether we even want to limit all projects to pbr-centric normalized versions in tags19:12
pleia2clarkb: ok, have some time now if we're still playing with jenkins plugins19:12
lifelesswe can of course change pbr more19:12
lifelessas long as it accepts anything it ever output19:12
clarkbpleia2 waiting for jobs to finish so grabbing lunch19:12
clarkbfungi noted, shouldnt be an issue19:12
clarkbfungi (jenkins that is)19:13
pleia2clarkb: lunch is a good idea, sync up after :)19:13
openstackgerritmelanie witt proposed openstack-infra/elastic-recheck: Update query for libvirt socket hangup bug 1451506  https://review.openstack.org/18068419:13
openstackbug 1451506 in OpenStack Compute (nova) "spawn failed with "libvirtError: internal error: received hangup / error event on socket" in the gate" [Undecided,Confirmed] https://launchpad.net/bugs/145150619:13
fungilifeless: clarkb: mordred: jeblair: what do you think of having pbr-pre-release and pbr-release pipelines with these strict regexes, and then relax the pre-release and release pipeline regexes further?19:13
*** hashar has quit IRC19:13
*** amitgandhinz has quit IRC19:14
*** kgiusti has joined #openstack-infra19:14
*** hashar has joined #openstack-infra19:14
mordredfungi: how about call it "semver-release"?19:14
*** boris-42 has joined #openstack-infra19:14
fungipbr-pep-440-semver-pre-release19:14
*** craigbr has quit IRC19:14
mordredbecause non-pbr could still desire to have the same rules enforced19:14
*** zul has quit IRC19:14
* fungi grins19:14
mordred++19:14
*** tonytan4ever has quit IRC19:15
*** zul has joined #openstack-infra19:16
*** salv-orl_ has joined #openstack-infra19:16
*** isviridov_away is now known as isviridov19:18
*** yamahata has quit IRC19:18
*** mrmartin has quit IRC19:18
*** e0ne has quit IRC19:18
*** ZZelle has quit IRC19:18
*** whoops has quit IRC19:18
*** adrian_otto has quit IRC19:18
*** sc` has quit IRC19:18
*** mmedvede has quit IRC19:18
*** jogo has quit IRC19:18
*** rkukura has quit IRC19:18
*** zaneb has quit IRC19:18
*** mkoderer has quit IRC19:18
*** morganfainberg has quit IRC19:18
*** gibi has quit IRC19:18
*** asilenkov has quit IRC19:18
*** jesusaurus has quit IRC19:18
*** plol has quit IRC19:18
*** pothole has quit IRC19:18
*** vipul has quit IRC19:18
*** devananda has quit IRC19:18
*** jtriley_ has quit IRC19:18
*** annegentle has quit IRC19:19
*** yamahata has joined #openstack-infra19:19
*** mrmartin has joined #openstack-infra19:19
*** e0ne has joined #openstack-infra19:19
*** ZZelle has joined #openstack-infra19:19
*** whoops has joined #openstack-infra19:19
*** adrian_otto has joined #openstack-infra19:19
*** sc` has joined #openstack-infra19:19
*** mmedvede has joined #openstack-infra19:19
*** zaneb has joined #openstack-infra19:19
*** jogo has joined #openstack-infra19:19
*** rkukura has joined #openstack-infra19:19
*** devananda has joined #openstack-infra19:19
*** vipul has joined #openstack-infra19:19
*** pothole has joined #openstack-infra19:19
*** plol has joined #openstack-infra19:19
*** jesusaurus has joined #openstack-infra19:19
*** asilenkov has joined #openstack-infra19:19
*** gibi has joined #openstack-infra19:19
*** morganfainberg has joined #openstack-infra19:19
*** mkoderer has joined #openstack-infra19:19
*** zaneb has quit IRC19:19
*** zbitter has joined #openstack-infra19:19
*** mrmartin has quit IRC19:19
*** annegentle has joined #openstack-infra19:19
*** salv-orlando has quit IRC19:19
*** mrmartin has joined #openstack-infra19:19
*** morganfainberg has quit IRC19:20
*** jtriley has joined #openstack-infra19:21
*** morganfainberg has joined #openstack-infra19:21
*** zbitter is now known as zaneb19:21
*** Somay has joined #openstack-infra19:23
*** otter768 has joined #openstack-infra19:23
*** tonytan4ever has joined #openstack-infra19:24
*** ociuhandu has quit IRC19:25
mordredShrews, SpamapS, clarkb, fungi, jeblair: I'm about to push the shade tag ... any last second concerns?19:27
funginone from me19:27
*** otter768 has quit IRC19:28
*** coolsvap|afk has quit IRC19:28
*** cody-somerville has joined #openstack-infra19:31
*** coolsvap|afk has joined #openstack-infra19:31
jeblairmordred: nak19:32
*** uberjay has joined #openstack-infra19:32
*** jtriley has quit IRC19:34
uberjayi just asked this in #openstack-cinder, but it really belongs over here: I appear to have some unfortunate timing. i'm in the middle of setting up the log server for our os-ci, and ~1hr ago the log server puppet configuration was cleaned up/refactored in infra/system-config (https://github.com/openstack-infra/system-config/commit/518c54f841d6db15398a09d1251841f03015b6cf), which causes the os-ext-testing logserver setup to fail because of missing19:35
uberjay templates: http://paste.openstack.org/show/215509/19:35
*** jtriley has joined #openstack-infra19:35
*** sdake has joined #openstack-infra19:36
jeblairuberjay: we didn't write the os-ext-testing module or design any of our systems for it, but we understand the need and utility of it.  so we're in the process of refactoring our modules into a puppet-openstackci module that can be reliably used to run a 3rd party ci system19:36
jeblairuberjay: http://specs.openstack.org/openstack-infra/infra-specs/specs/openstackci.html19:37
jeblairuberjay: we'd love your help on that; i'm not sure if it is in a state where it's ready for you to use yet (we now use it to run our log server, but i'm not certain how easy it would be for you to do so at this point).  it's early days.19:38
jeblairuberjay: if you're not up for all of that right now, you might want to try pinning modules versions to specific commits.  lots of stuff is going to continue to change over the next few months19:39
uberjayjeblair: yeah, I assumed that's what happened. hm. thanks for the spec link, this is helpful.19:39
*** sdake_ has quit IRC19:39
*** akshai has quit IRC19:39
jeblairuberjay: asselin__'s repo might be helpful here too; i'm not sure19:40
uberjayjeblair: ok, yeah, unfortunately I don't have the time to help with early-days testing for this -- I wish I did. :)19:40
uberjayjeblair: ah, I suspect that's what I'm using!19:40
jeblairuberjay: no worries, you're welcome to join whenever it gets usable enough for you :)19:41
fungifwiw asselin__ is also helping drive the openstack-ci puppet module work, so may have some additional suggestions19:41
*** Somay has quit IRC19:42
uberjayfungi: that's a great point. i just realized the same person was responsible for the openstackci-puppet repo.19:43
fungisdague: were you planning to backport 180549 to stable/kilo as well?19:43
*** craigbr has joined #openstack-infra19:43
fungijust curious mostly19:43
uberjayasselin__: if you're around, do you have any suggestions on how to approach the divergence with os-ext-testing & the infrastructure modules?19:43
*** sdake_ has joined #openstack-infra19:44
*** ajo has quit IRC19:44
uberjayasselin__: in the meantime, I could hold the system-config/project-config modules to specific commits, or just manually configure the logserver.19:45
*** sdake has quit IRC19:47
*** shashankhegde has quit IRC19:51
*** _nadya_ has joined #openstack-infra19:51
*** baoli has joined #openstack-infra19:52
*** prometheanfire has joined #openstack-infra19:53
*** craigbr has quit IRC19:53
prometheanfirewhy does review.openstack have such bad ssh config settings (macs and ciphers at the least, some better kex options would be nice too)19:53
mordredprometheanfire: it's not a real ssh server19:54
prometheanfire:(19:55
*** Rockyg has joined #openstack-infra19:55
jeblairprometheanfire: actually, it might be able to be improved upon.  you might take a look at https://review.openstack.org/Documentation/config-gerrit.html and propose a patch to http://git.openstack.org/cgit/openstack-infra/puppet-gerrit/ if you see an opportunity for improvement19:57
jeblairprometheanfire: also, we're planning a gerrit upgrade on saturday which will include a new version of the built-in sshd19:58
prometheanfirethat's nice :D19:59
*** armax has quit IRC20:00
fungiwe'll hopefully be on mina-sshd 0.14 once the upgrade window completes20:00
clarkbpleia2: I am back and ready when you are20:00
pleia2clarkb: yep, I'm ready20:00
clarkbpleia2: first thing is to double check https://jenkins07.openstack.org/ is not running any jobs (it seems done)20:01
*** prometheanfire has left #openstack-infra20:01
pleia2yep20:01
jeblairnibalizer: do you want me to merge 171369 or should it wait for the other stuff from earlier?20:01
clarkbpleia2: then we have two options for installing the plugin, we can either use the Jenkins admin gui to install it while jenkins is still up, or we can stop jenkins then manually install the plugin.20:01
fungihuh. kde's getting interesting. how did i not know about krita?20:02
clarkbpleia2: I tend to use the second option because it gives me control over the process and we don't ahve the weirdness of an installed but not fully running plugin going on20:02
pleia2clarkb: to confirm, I know no jobs are running because the build systems are Idle, right?20:02
clarkbpleia2: correct20:02
pleia2perfect20:02
pleia2clarkb: +1 to second way, installing plugins via web guis is meh20:03
jeblairfungi: oh yeah, i've heard good things about krita but have not used it20:03
clarkbpleia2: in either case we are going to install the build artifact that resulted from https://review.openstack.org/#/c/179988/ 's check test20:03
pleia2clarkb: ok20:03
clarkbhrm do we not upload a test hpi? I thought we did20:03
pleia2the gearman-plugin.jar ?20:04
fungikrita's drawing tablet support looks better-implemented than the gimp20:04
*** rkukura has quit IRC20:04
clarkbpleia2: the .hpi or .jpi20:04
pleia2ah, ok20:04
clarkbhttp://tarballs.openstack.org/ci/gearman-plugin/?C=M;O=A actually I think it is there?20:04
clarkbor is that jeblairs?20:04
*** hashar has quit IRC20:04
clarkbI think thats jeblairs hrm maybe I should go ahead and approve this so we can get a nice hpi built for us20:05
clarkbpleia2: the other option is to build the hpi youself with `mvn package`20:05
* pleia2 nods20:05
clarkbpleia2: but I prefer getting jenkins to do it so we can all operate off of the same package20:05
clarkblet me approve20:05
pleia2yeah, I don't want to get into the business of building java packages20:06
clarkbnow we wait a few minutes to get the package built and it should show up at the link above20:06
*** ildikov has quit IRC20:06
pleia2go go gadget gate job20:06
fungiit's "fun" (in the same way that inserting your extremities into a wood chipper might also be considered fun)20:06
*** dizquierdo has joined #openstack-infra20:07
pleia2(best to make sure you have the right version wood chipper)20:07
clarkbthere was a time I would randomly try kde instead of gnome2, but then their indexer thing happened and killed my machine and I switched to lightweight desktops with xmonad as the window manager20:09
openstackgerritMerged openstack-infra/gearman-plugin: Fix race between adding job and registering  https://review.openstack.org/17998820:09
*** freerunner has joined #openstack-infra20:09
fungiclarkb: we're not cd'ing that ^ right?20:09
clarkbfungi: we are not20:09
clarkbfungi: we don't really have a good way to control jenkins plugin versions automagically20:09
jeblairbest to just get rid of jenkins20:10
fungijust making sure turning puppet back on for the jenkins masters here in a few moments won't cause a problem20:10
clarkbfungi: it shouldn't20:10
clarkbfungi: the jenkins puppet plugin management is write once20:10
fungiahh, that's right, we have to do the plugin upgrade dance in jenkins anyway20:10
clarkbfungi: it doesn't upgrade anything20:10
fungiyep. remembering now20:10
*** notnownikki has quit IRC20:10
*** jcoufal has joined #openstack-infra20:11
*** jcoufal has quit IRC20:11
fungijeblair: perhaps soon, all 0 of our jenkins plugins will automatically be at their intended versions20:11
jeblairclarkb, fungi, zaro: i was just looking at 172534 and thought to myself, "I wonder if anyone understands the BC jar situation on gerrit."20:11
pleia2heh20:11
jeblairbecause we had that issue with 2.8 and i think we learned something about symlinks but i am not sure what it was and then i flew around on planes20:12
clarkbjeblair: puppet was not properly cleaning things up, I wrote a change that *should* fix that20:12
asselin__uberjay, yes, you can pin your system config20:12
fungijeblair: oh, right, so... gerrit seems to provide its own when we unpack the war. 172534 could be overcomplicating this?20:13
asselin__uberjay, I'l update my repo to pull in the latest20:13
clarkbjeblair: https://review.openstack.org/#/c/178251/20:13
openstackgerritsebastian marcet proposed openstack-infra/openstackid: Migration of resource server  https://review.openstack.org/17885420:13
openstackgerritsebastian marcet proposed openstack-infra/openstackid-resources: OpenstackId resource server  https://review.openstack.org/17885320:13
*** gyee has quit IRC20:13
jeblairfungi: ah, yeah, maybe 534 is unecessary?  maybe we just stop tidying?20:13
uberjayasselin__: what's the correct way to pin a repo?20:13
jeblairclarkb: ?20:13
*** lennyb has joined #openstack-infra20:14
clarkbjeblair: basically we have to run the tidy after installing gerrit and initializing it but before starting the service20:14
clarkbjeblair: 178251 should do that properly now20:14
*** yamahata has quit IRC20:14
jeblair(in other words: for 2.8 we need to tidy correctly, for 2.10 perhaps we don't install libs and don't tidy; just let gerrit do its thing)20:14
fungijeblair: clarkb well, for the libs that get unpacked from the gerrit war though, we could just drop the symlinks and use those instead?20:14
fungiyeah, possibly20:15
clarkboh right because 2.10 wants the newer versions20:15
clarkbya I think so20:15
*** asselin__ is now known as asselin20:15
*** AJaeger_ has quit IRC20:15
fungisince 2.8 _would_ work with distro-provided bc we blew away teh one from the war and symlinked. 2.10 won't work with the one from the system, so we're working around our workaround20:15
clarkbpleia2: ok http://tarballs.openstack.org/ci/gearman-plugin/gearman-plugin-0.1.1.6.0314ab1.hpi is available now20:15
pleia2cool20:15
jeblairclarkb: i think 178251 has errors20:16
pleia2wget to jenkins07?20:16
clarkbpleia2: so next step is hop on jenkins07 and stash that somewhere (in your homedir)20:16
clarkbpleia2: yup20:16
pleia2k20:16
pleia2done20:16
clarkbjeblair: ah yup, fixing20:16
openstackgerritMatthew Treinish proposed openstack-infra/subunit2sql: WIP: Add CLI tool to graph aggregate failure counts for tests  https://review.openstack.org/17803920:16
jogoI ran recheck on a patch that had a +W but wasn't in the gate20:17
*** mattfarina has quit IRC20:17
jogoand it went through check and just stopped and didn't go into gate20:17
clarkbpleia2: now if you ls -l /var/lib/jenkins/plugins | grep gearman you will see the existing plugin20:17
mtreinishjogo: dependency?20:17
fungijogo: did it already have a +1 when you left a recheck comment?20:17
jogomtreinish: no20:17
clarkbpleia2: should be one dir, then >=1 [hj]pi files20:17
jogofungi: see https://review.openstack.org/#/c/177222/720:17
fungijogo: if so, gerrit didn't add a new +120:17
openstackgerritAlexander Yip proposed openstack-infra/project-config: Enable congress and tempest in devstack in congress-dsvm-api test  https://review.openstack.org/18072620:17
pleia2clarkb: yep -rw-r--r--  1 jenkins nogroup  762K Oct 22  2014 gearman-plugin.hpi20:18
*** shashankhegde has joined #openstack-infra20:18
clarkbpleia2: what we are going to do is, stop jenkins, rm the plugin dir, move the existing .hpi/jpi aside, copy new one in, set perms to jenkins:nogroup, start jenkins20:18
asselinuberjay, just comment out theses lines:  https://github.com/rasselin/os-ext-testing/blob/master/puppet/install_master.sh#L36-L4420:18
fungijogo: note the lack of "Verified+1" in the new gerrit comment20:18
asselinuberjay, and manually pick what you want20:18
pleia2clarkb: easy enough, shall I?20:18
fungijogo: you need to get that change reapproved20:18
clarkbpleia2: oh and don't forget to double check jenkins actually stopped after stopping it20:18
*** HeOS has joined #openstack-infra20:18
pleia2clarkb: right, thanks20:18
clarkbpleia2: yup go for it, let me know if anything pop up20:18
*** Krinkle is now known as Krinkle|detached20:19
jogofungi: ahh thanks wasn't sure20:19
nibalizerjeblair: 171369 can go20:19
uberjayasselin: ah, ok. i'd done that, but thought there may be a more clever way. ;)20:19
fungijogo: either approve it yourself, or get sdague to remove and readd his approval, or whatever20:19
*** mdenny has joined #openstack-infra20:19
jogofungi: just put a +W of my own there20:19
jogofungi: a +W is sufficient I don't need to +2 it as well right?20:19
fungiright20:19
jeblairnibalizer: done20:20
nibalizer\o/20:20
pleia2clarkb: there is a gearman-plugin and gearman-plugin.hpi - anything to do with the non .hpi one?20:20
jeblairzaro: see comments in https://review.openstack.org/17253420:20
fungijogo: as long as there's at least one code review +2 already present20:20
clarkbpleia2: thats a dir right? you can delete it20:20
*** dkranz has quit IRC20:20
jogofungi: perfect20:20
clarkbpleia2: aiui the hpi file gets unpacked into that dir somehow and I don't trust it to delete files if they go away across versions20:20
clarkbpleia2: once you start jenkins you should see a new dir with that ame20:21
jeblairclarkb, pleia2: process to stop jenkins: /etc/init.d/jenkins stop; killall jenkins; killall -9 jenkins; ps aux|grep jenkins; kill kill kill; reboot20:21
pleia2clarkb: yeah, it is, thanks20:21
clarkbjeblair: maybe even nova delete; nova boot :)20:21
openstackgerritClark Boylan proposed openstack-infra/puppet-gerrit: Run lib tidy after plugin install, before start  https://review.openstack.org/17825120:22
clarkbjeblair: ^ that may not be useful for very long but for completeness that should make puppet happy20:22
fungihah20:22
fungii usually stop, then kill -1, then -7, then -9, then sometimes have to start down the same path for the parent pid20:22
fungi_most_ of the time it dies on the -7 sigsegv20:23
clarkbfungi: I usually stop then go straight to -920:23
fungiand the parent then terminates20:23
jeblairclarkb: yeah, i think it's worth getting in now20:23
fungisometimes i even get lucky and stop works as intended20:23
jeblairclarkb: +3; it's in zuul's hands now20:23
openstackgerritsebastian marcet proposed openstack-infra/openstackid-resources: OpenstackId resource server  https://review.openstack.org/17885320:23
openstackgerritsebastian marcet proposed openstack-infra/openstackid: Migration of resource server  https://review.openstack.org/17885420:24
lennybHi All, is there a way to trigger zuul with disabled Gerrit account ?20:24
*** zz_ja has quit IRC20:25
pleia2clarkb: ok, jenkins07 is coming back up20:25
*** hemnafk is now known as hemna20:26
clarkbpleia2: looks like jenkins is coming back up, once its up we will want ot make sure its running jobs and we can double check the plugin version in the manage plugins window20:26
pleia2and already building some things20:26
jeblairlennyb: i would say intentionally no20:26
* pleia2 nods20:26
*** e0ne has quit IRC20:26
*** mattfarina has joined #openstack-infra20:26
*** mattfarina has quit IRC20:26
lennybjeblair: thanks.20:26
clarkbpleia2: https://jenkins07.openstack.org/pluginManager/installed is the shortcut to the plugin page20:27
clarkbpleia2: or you can do manage jenkins -> manage plugins -> installed20:27
pleia2clarkb: ah cool, I was clicking around20:27
clarkbpleia2: version lgtm20:27
*** zz_ja has joined #openstack-infra20:27
pleia2lyz is missing the Overall/Administer permission20:28
clarkbpleia2: now its just a matter of keeping track of jobs and making sure the server doesn't go haywire, I can keep an eye on it if you want to move onto other stuff20:28
clarkbpleia2: oh, hrm let me check that20:28
pleia2so maybe we did find something I still need to be added to!20:28
fungipleia2: did you make sure to check the admins group checkbox when you did the openid authentication, or did you just click through?20:28
clarkbpleia2: did you check the box with openid to send the group info?20:29
*** freerunner has quit IRC20:29
clarkbya that20:29
openstackgerritMerged openstack-infra/system-config: Pull o_p::server invocation out of afs roles  https://review.openstack.org/17136920:29
*** smarcet has joined #openstack-infra20:29
jeblairnibalizer: you see fbo is doing some work on your base refactor spec too, right?20:29
pleia2fungi: doooh20:29
pleia2fungi: I don't read login screens! :)20:29
*** emagana has quit IRC20:29
pleia2much better, that was it20:29
fungipleia2: don't feel bad. i once spent half an hour confused because i did the same thing20:29
*** freerunner has joined #openstack-infra20:30
funginecessary learning pain20:30
*** ivar-lazzaro has quit IRC20:30
pleia20.1.1.6.0314ab1 yay20:30
smarcetfungi: hello20:30
*** baoli has quit IRC20:30
*** ivar-lazzaro has joined #openstack-infra20:31
fungismarcet: hi there20:31
smarcetfungi: was wondering how should approve these changes https://review.openstack.org/#/c/178853 and https://review.openstack.org/#/c/17885420:31
*** baoli has joined #openstack-infra20:31
*** rbradfor has quit IRC20:31
smarcetfungi: sorry who should20:31
clarkbhrm https://jenkins07.openstack.org/job/gate-ironic-python27/1587/console not sure if that is related or not20:31
*** lennyb has left #openstack-infra20:32
*** ildikov has joined #openstack-infra20:32
smarcetfungi: should i just give them +2? or should i ask to someone else to review them?20:32
fungismarcet: since you're probably the only developer on that project, i guess you should for now20:32
smarcetfungi: roger doger thx :)20:32
pleia2clarkb: hm :\20:32
fungismarcet: once those projects gain regular reviewers, that's time to revisit that workflow20:32
clarkbpleia2: actually that is a requested job abort I think zuul did that20:33
clarkbpleia2: so we may be fine, still watching though20:33
pleia2clarkb: phew20:33
* pleia2 nods20:33
fungismarcet: if there are other devs at tipit familiar with that codebase, then we should probably add them to the core review groups for them so that they can look over changes like that20:33
clarkbpleia2: the RequestAbortedException is what jenkins does when zuul says it no longer needs a job (maybe an earlier test failed or somethign)20:34
*** e0ne has joined #openstack-infra20:34
pleia2clarkb: I see20:34
clarkbhttps://jenkins07.openstack.org/job/gate-nova-pep8/11582/console and https://jenkins07.openstack.org/job/gate-nova-docs/11507/console were both successful20:34
pleia2yep, just looked at the latter, great20:34
fungismarcet: looks like mrmartin can also +2/approve changes to those repos right now20:35
smarcetfungi: yeah marton has +2 rites20:35
*** radez is now known as radez_g0n320:35
smarcetshould i ask to him?20:35
mrmartinI can20:36
mrmartinbut not now if you don't mind, it is a bit late here20:36
clarkbpleia2: and https://jenkins07.openstack.org/job/gate-poppy-python34/91/console ran to completion and appears to have properly failed20:36
mrmartinsmarcet: I've added this to my backlog, and will check tomorrow20:36
clarkbpleia2: so I think we are good so far20:36
*** Swami has quit IRC20:37
pleia2clarkb: great20:37
smarcetmmartin: sure np20:37
smarceti will add u are reviewer20:37
nibalizerjeblair: no i don't see that.. link?20:37
mrmartinsmarcet: thanks!20:38
*** achanda has joined #openstack-infra20:38
pleia2clarkb: so once this is all good, then we do it on all the jenkinses?20:39
clarkbpleia2: yup20:39
clarkbpleia2: we can do two or three at a time too to speed things up20:39
* pleia2 nods20:40
clarkbpleia2: probably going to let this sit in place until tomorrow though so we can check for any leaked nodes (that shouldn't happen anymore)20:40
pleia2clarkb: wfm20:40
jeblairnibalizer: https://review.openstack.org/#/q/owner:%22Fabien+Boucher%22+status:open+topic:downstream-puppet,n,z20:40
pleia2I'm around all morning tomorrow, after noon gets dicey because lunch meeting + getting to the valley by 5pm for an event20:40
openstackgerritsebastian marcet proposed openstack-infra/openstackid: Migration of resource server  https://review.openstack.org/17885420:40
clarkbpleia2: noted, we can get an early start20:40
clarkbjeblair: anything you want us to check as far as work statuses go?20:41
nibalizerjeblair: ah i thought you meant an update to the spec itself20:41
clarkbjeblair: tcpdump maybe?20:41
nibalizerfbo: thanks!20:41
openstackgerritEmilien Macchi proposed openstack-infra/project-config: puppet: run beaker on both trusty & centos7  https://review.openstack.org/17543420:41
jeblairclarkb: nah, i believe it.  :)20:41
clarkbpleia2: the rollback process is basically the same too, stop jenkins, really stop it, put old hpi back in place, start jenkins20:42
yolandajeblair, i answered to some of your  reviews20:42
yolandai need more feedback on that20:42
jeblairyolanda: which one(s)?20:42
jeblairyolanda: (i still haven't gotten to the bottom of the list :)20:42
pleia2clarkb: easy enough, and timestamps make figuring out which .bak it is simple enough20:43
*** unicell has joined #openstack-infra20:43
yolandahttps://review.openstack.org/#/c/161663/20:43
yolandahttps://review.openstack.org/#/c/171526/20:43
yolandathese 220:44
yolandajeblair, thx for the approval on some, really appreciated20:44
*** unicell has quit IRC20:45
*** unicell has joined #openstack-infra20:45
clarkbasselin: any progress on that error? I am willing to helpout now that the jenkins thing is done20:45
*** achanda has quit IRC20:45
*** e0ne has quit IRC20:45
jeblairyolanda: regarding 161663, i don't think my suggestion contradicts any of the advice you have previously received.  my thought is that the puppet-gerrit module should know nothing about jeepyb because it is possible to run gerrit without jeepyb.20:46
pabelangerGood news everybody, grafyaml appears to be working! So far, you can use a simple yaml to create a dashboard, with some rows.20:46
pabelangerwith the help of python-voluptuous its actually very simple to setup the grafana schema20:47
jeblairyolanda: so i'm asking is it possible to remove manifests/remotes from puppet-gerrit entirely and move it to openstack_project/gerrit for now?  (and later into openstackci)20:47
yolandajeblair, ok , your suggestion makes sense20:47
*** moshele has joined #openstack-infra20:48
pabelangerI have to run now, but wanted to give a simple update.20:48
pabelangerchat in the morning20:48
*** pabelanger has quit IRC20:48
jeblairpabelanger: w00t, thanks!20:48
jogomoshele: o/ I think you need to ask here20:48
yolandai'll add the change to remove from there, and move to openstack_project using that jeepyb manage_projects class20:49
jeblairyolanda: (and also when you do that, i think you could do the inline suggestion about the log file path too)20:49
moshelejogo: do you know who?20:49
jogomoshele: just ask in general20:49
openstackgerritMerged openstack-infra/puppet-gerrit: Run lib tidy after plugin install, before start  https://review.openstack.org/17825120:49
jeblairwe're a collective20:49
jeblairyolanda: re 171526, answer is "yes".  :)20:50
*** ayoung has joined #openstack-infra20:50
openstackgerritMerged openstack-infra/system-config: Bump stankevich-python version  https://review.openstack.org/17482020:50
yolandajeblair, about logfile, do you want to send output of cronjob and from notify into same logfile?20:50
openstackgerritMerged openstack-infra/system-config: Bump stankevich-python version  https://review.openstack.org/17482020:50
yolandaor i'm not following you?20:51
openstackgerritMatthew Treinish proposed openstack-infra/subunit2sql: Add db api method to get uuid from test_id  https://review.openstack.org/18074120:51
openstackgerritMatthew Treinish proposed openstack-infra/subunit2sql: Add db api method to get per status counts for a test  https://review.openstack.org/18074220:51
moshelecan you enable the Mellanox CI account? all the jenkins jobs are stoped and we need it to debug the issue in Our CI20:51
jeblairyolanda: no; i'm just saying the cron entry can hardcode the logfile and the notify can hardcode its logfile, and the consumer of the module doesn't need to specify that20:51
yolandai'm ok with that20:51
jeblairmoshele: have you cahnged it to a configuration where it will not leave comments in gerrit while you debug the issue?20:52
yolandaso it sounds like a plan, thx20:52
jeblairmoshele: s/cahnged/changed/20:52
yolandareally, i'm going to start working at nights, it's much easier :)20:52
*** unicell has quit IRC20:52
jeblairyolanda: heh, i usually review earlier, i'm just not caught up after traveling yet20:52
yolandajeblair, for the other holded changes, i agree with you, i hope that this lock is released soon20:53
moshelejeblair: all the nodes are down and yes it configured not to leave comments20:53
*** _nadya_ has quit IRC20:53
yolandajeblair, not also for the reviews, but for the opportunities to talk in chat, is so helpful20:53
jeblairmoshele: what's the username?20:53
*** _nadya_ has joined #openstack-infra20:53
*** unicell has joined #openstack-infra20:54
clarkbpleia2: I am also working on a second change to the gearman plugin that we may want to bundle into the large scale plugin upgrades20:54
*** unicell has quit IRC20:55
clarkbI should start working on figuring out why it doesn't do what I expect20:55
*** samueldmq has quit IRC20:55
moshelejelair: Gerrit Account: Mellanox CI <mlnx-openstack-ci@dev.mellanox.co.il>20:55
openstackgerritEmilien Macchi proposed openstack-infra/project-config: puppet: run beaker on both trusty & centos7  https://review.openstack.org/17543420:55
EmilienMnibalizer: ^^^^^^^^ I think this patch is ready for review, I've added Centos7 support for beaker20:56
*** aysyd has quit IRC20:56
pleia2clarkb: cool20:56
clarkbzaro: jeblair do you know if there was a specific reason to always reset abilities and reregister everything? was it just simpler or is there something I should be afraid of there20:57
openstackgerritMerged openstack-infra/puppet-logstash: Add missing vhost_name  https://review.openstack.org/17663120:57
*** _nadya_ has quit IRC20:58
jeblairmoshele: done20:58
moshelejeblar: thanks20:58
jeblairclarkb: i think programming was hard and we only had like 500 jobs at the time so why bother? :)20:59
jeblairclarkb: i just had one thought you should watch out for -- there might be cases where a node wants to remove all jobs; you should detect that and use RESET instead of CANT_DO there21:00
clarkbjeblair: good point21:00
clarkbI think that is easy to chek for, just look for an empty new list21:00
jeblairya21:00
*** unicell has joined #openstack-infra21:01
*** emagana has joined #openstack-infra21:01
fungiNumber of jobs generated:  555621:01
fungiorder of magnitude in just a couple years21:02
jeblairi like that there's space for another digit there21:02
fungiindeed21:02
fungiconverse moore's law. 10x increase in 2 years21:02
fungii hereby name thee "cruft's law of accumulation"21:03
jeblair(i hope we have zuul3 before we have the fleet of jenkins-job-builder machines whose only purpose is to continually update the 50000 jenkins jobs across our 30 jenkins masters)21:03
*** Sukhdev has quit IRC21:03
*** gokrokve has quit IRC21:03
*** gokrokve has joined #openstack-infra21:04
openstackgerritStephen Gordon proposed openstack-infra/project-config: Add openstack-nfv to gerritbot  https://review.openstack.org/18074421:04
*** gokrokve has quit IRC21:05
*** gokrokve has joined #openstack-infra21:05
fungiso even with the jenkins masters caught up on jjb updates from less than 24 hours prior, a full update with today's puppet applied doesn't complete within the exec timeout21:05
clarkbso it looks like we do setFunctions multiple times, the first time we include build:foobar:master then the second time we exclude it hence the CANT_DOs for the :master job21:05
*** moshele has left #openstack-infra21:05
*** shashankhegde has quit IRC21:05
fungii think our template expansions are just getting unwieldy21:05
clarkbI want to say that this behavior must exist in the old code too, will need to also test with a real label21:05
pleia2oof21:05
jeblairfungi: hrm, that's weird; i would expect it to take < 5 mins21:05
*** emagana has quit IRC21:06
fungiInfo: /etc/jenkins_jobs/config: Scheduling refresh of Exec[jenkins_jobs_update]21:06
fungiError: /Stage[main]/Jenkins::Job_builder/Exec[jenkins_jobs_update]: Failed to call refresh: Command exceeded timeout21:06
fungithat was doing puppet agent --test on jenkins0221:06
jeblairclarkb: oh, seems like :master is not something we ever want21:06
clarkbjeblair: ya for our use case I think its fine21:06
clarkbbut this may be a general case bug21:07
fungiit looks like this template expansion is courtesy of a change switching the nova api version in a bunch of devstack-gate job defs21:07
jeblairclarkb: okay, as long as the general case isn't too hard :)21:07
clarkbjeblair: well if its preexisting I am not going to fix it here :)21:07
*** ajmiller has quit IRC21:07
clarkbtesting that next21:08
jeblairfungi: oh.  so maybe time to just bump the timeout?21:08
*** gokrokve has quit IRC21:08
*** gokrokve_ has joined #openstack-infra21:08
*** bswartz has quit IRC21:08
fungijeblair: maybe. i'm seeing how much longer the catch-up takes after that21:09
jeblairfungi: it only happens on subscription, so we only catchup on the next change :/21:09
fungito get some idea of what a reasonable timeout bump might look like21:09
jeblairfungi: ah, yeah. makes sense.21:09
fungirighth, catching up by manually running jjb again, not reapplying from puppey21:10
fungis/y$/t/21:10
jeblairyou're talking weird again21:10
clarkbjeblair: but pretty sure my change makes function registrations N * slaves instead of N * N * slaves where N is number of jobs updated so should make a huge difference21:11
*** rbradfor has joined #openstack-infra21:11
fungithat would also explain why job updates take so much longer when we're busy. i assumed it was load on the jenkins masters, but sounds like it's a multiplication due to number of slaves21:12
*** cody-somerville has quit IRC21:12
fungijeblair: additional wall clock time was 7m20s21:12
jeblairclarkb: and that's just initial ones; should make updates M*slaves where M is number of added and removed jobs rather than N*slaves.21:12
fungicurrent timeout is 10m? if so, should probably bump to 2021:12
jeblairfungi: sgtm21:12
fungion the way21:13
openstackgerritMerged openstack-infra/system-config: Don't hardcode names on certs  https://review.openstack.org/17906721:13
*** tiswanso has quit IRC21:13
*** Swami has joined #openstack-infra21:13
*** sdake has joined #openstack-infra21:14
*** ldnunes has quit IRC21:14
*** smarcet has quit IRC21:15
*** zz_ja has quit IRC21:15
clarkbjeblair: confirmed for some reason we were already removing build:foo:master after first registering it21:16
clarkbjeblair: so I will just update to optimize for the remove all jobs case21:16
jeblairclarkb: neat!21:16
*** pc_m has quit IRC21:16
*** ociuhandu has joined #openstack-infra21:16
*** dboik has quit IRC21:16
*** zz_ja has joined #openstack-infra21:17
*** sdake_ has quit IRC21:18
openstackgerritMerged openstack-infra/puppet-cgit: Manage cgitrc config file  https://review.openstack.org/17913221:19
*** adrian_otto has quit IRC21:19
*** freerunner has quit IRC21:19
openstackgerritJeremy Stanley proposed openstack-infra/system-config: Increase JJB timeout to 20min on Jenkins masters  https://review.openstack.org/18075321:20
*** maurosr has quit IRC21:21
funginot sure whether that ^ is where we want it, but increasing the default timeout in puppet-jenkins seemed less downstream-friendly21:21
*** jgrimm_ has quit IRC21:22
*** zz_ja has quit IRC21:22
openstackgerritgreghaynes proposed openstack-infra/nodepool: WIP: Use shade to get the client objects  https://review.openstack.org/16860321:22
jeblairfungi: i feel like we're in no position to make promises about puppet-jenkins at this point, it's still a huge tangle of openstack specific code.  i reckon wherever is fine for now :)21:22
mordredI agree with that21:22
jeblairfungi: (i mean, the real answer is in our openstack-specific hiera yaml file, but we don't have that yet)21:22
*** maurosr has joined #openstack-infra21:22
openstackgerritgreghaynes proposed openstack-infra/nodepool: WIP: Use shade to get the client objects  https://review.openstack.org/16860321:23
*** jgrimm has joined #openstack-infra21:23
*** sabeen has joined #openstack-infra21:23
*** Longgeek has quit IRC21:23
*** zz_ja has joined #openstack-infra21:23
*** otter768 has joined #openstack-infra21:24
*** ddieterly has quit IRC21:26
*** esker has quit IRC21:28
*** david-lyle has quit IRC21:29
*** otter768 has quit IRC21:29
*** kgiusti has left #openstack-infra21:31
*** shashankhegde has joined #openstack-infra21:31
*** cdent has quit IRC21:33
openstackgerritClint 'SpamapS' Byrum proposed openstack-infra/infra-manual: Initial infra manual for infra-cloud  https://review.openstack.org/18076221:34
SpamapSjeblair: ^ an initial distilling of the notes we took into something that resembles a framework for building a cloud.21:35
SpamapSVERY high level.21:35
openstackgerritMatthew Treinish proposed openstack-infra/subunit2sql: Add graph command to show aggregate counts for tests  https://review.openstack.org/17803921:35
openstackgerritMatthew Treinish proposed openstack-infra/subunit2sql: Add db api method to get per status counts for a test  https://review.openstack.org/18074221:35
SpamapSEvery time I descended into the lower levels I realized we didn't actually answer many of those questions.21:35
pleia2SpamapS: I probably missed the discussion, but why is this doc vs spec?21:36
SpamapSpleia2: I'm fuzzy on the details as it happend a while ago, but mostly because there's nothing to "change".21:37
*** craigbr has joined #openstack-infra21:38
SpamapSpleia2: and also because we'd just copy the content of the spec directly to the manual anyway.21:38
SpamapSSo it's sort of like.. lets just document this as it should be, and then build on top of it from there.21:38
pleia2I see, thanks21:38
SpamapSpleia2: as you can see, I'm not 100% married to the "just start in the manual" idea. :)21:39
SpamapSmore of a 2nd-date-this-looks-interesting stage right now :)21:39
pleia2SpamapS: so when you say "nothing to change" - will we be using the tripleo framework for managing a cloud, so that work is already pretty much done?21:40
SpamapSpleia2: as it says, no TripleO21:40
*** bhunter71 has quit IRC21:40
SpamapSpleia2: or rather, it doesn't say TripleO, it says Puppet. :)21:41
pleia2 SpamapS yeah, but I mean... we don't have puppet configs ourselves in -infra to manage an openstack cloud21:41
*** asahlin is now known as asahlin_afk21:41
pleia2so it seems like a big change21:41
SpamapSpleia2: we have some in stackforge. :)21:41
pleia2yeah, but you still need to glue them together, and it's not trivial21:41
pleia2puppet-openstack is not a thing anymore, so you've got all your puppet-nova puppet-neutron etc bits there21:42
SpamapSpleia2: the idea being to make it work _as close_ to all the other infra things we have... just happens to be on top of an Ironic-all-in-one box.21:42
SpamapSpleia2: yes, the glueing things together is where infra comes in. :)21:42
pleia2SpamapS: and that's why I'd vote for a spec21:42
pleia2plus specs make it much easier for newcomers to look at one place for our big projects (I think running a cloud is a big project)21:43
uberjayis it possible to have nodepool not automatically delete an instance after it's no longer needed? I need to go dig a little deeper and see why some tests are failing, and the logs aren't illuminating enough.21:43
SpamapSpleia2: it's so big, it would be like, 10 specs. :)21:43
pleia2SpamapS: indeed!21:43
*** baoli has quit IRC21:43
SpamapSpleia2: which is another reason it may be good to start with a not-spec, and then have specs for all the things.21:43
SpamapSor not, we could also have one big spec that spawns "babies"21:44
pleia2SpamapS: I guess I have no objection to manual docs, but since I haven't yet seen a spec, I was confused about the order of operations (and a lot of data is very similar)21:44
*** baoli has joined #openstack-infra21:44
greghaynesSpamapS: oh no, heat convergence again21:44
*** ajmiller has joined #openstack-infra21:44
SpamapSgreghaynes: I think Heat convergence is a cool idea... because lifeless and I came up with it. ;)21:44
greghaynesI just mean all the specs21:44
SpamapSOh sorry21:45
greghaynesthe specs that refer to sub specs21:45
SpamapSI thought maybe something was happening. :)21:45
lifelessSPECS21:45
SpamapSyes, heat convergence was at 5 specs when last I poked at it.21:45
*** btully has quit IRC21:45
SpamapSBecause really, it was refactoring 90% of Heat....21:45
pleia2SpamapS: anyway, it's fine, just wondering :)21:45
SpamapSpleia2: thank you for asking. I'm wondering myself. :)21:45
*** achanda has joined #openstack-infra21:46
greghayneswell be talking about this at summit, maybe add that question to the etherpad?21:46
SpamapSis there already a slot for htis?21:46
SpamapSthis21:46
*** alexpilotti has quit IRC21:46
pleia2mordred added a placeholder description for it during our meeting (under fishbowl)21:47
greghaynesI think the coalescing into slots hasnt happened, but its on the summit etherpad21:47
openstackgerritClark Boylan proposed openstack-infra/gearman-plugin: Register the diff of functions  https://review.openstack.org/18037121:47
*** packet has quit IRC21:47
clarkbjeblair: zaro ^ that appears to work21:47
clarkbthough I haven't tested with any labels yet, I need to setup a slave in my testbed21:47
clarkbuberjay: you can "hold" the node21:48
clarkbuberjay: do a nodepool hold $nodeid21:49
*** jtomasek has quit IRC21:49
*** packet has joined #openstack-infra21:49
uberjayclarkb: ah, so hold the node, then manually run my job, and it will stick around after failure?21:49
clarkbuberjay: ya, a held node should not be deleted21:50
clarkbuberjay: so when you are done with it you will want to nodepool delete $nodeid otherwise it won'y get cleaned up21:50
uberjayclarkb: excellent, thanks! i'll give it a shot21:50
ianwAJaeger: why abandon https://review.openstack.org/#/c/180309/ ?  i think the template of the regex is a good idea21:50
ianw(maybe you just didn't want to rebase?)21:50
*** achanda has quit IRC21:51
*** craigbr has quit IRC21:52
*** sdake has quit IRC21:53
*** Krinkle|detached is now known as Krinkle21:53
*** craigbr has joined #openstack-infra21:53
*** sdake has joined #openstack-infra21:53
*** sabeen has quit IRC21:54
*** baoli has quit IRC21:54
*** baoli has joined #openstack-infra21:55
*** mrmartin has quit IRC21:56
*** stevemar has quit IRC21:57
*** ajmiller_ has joined #openstack-infra21:58
openstackgerritMichael Krotscheck proposed openstack-infra/project-config: Initial integration tests for storyboard  https://review.openstack.org/15604421:58
*** zz_ja has quit IRC21:58
*** Krinkle is now known as Krinkle|detached21:58
clarkbSpamapS: commented on that fwiw21:59
*** maurosr has quit IRC21:59
openstackgerritMatthew Treinish proposed openstack-infra/subunit2sql: Add db api method to get per status counts for a test  https://review.openstack.org/18074222:01
openstackgerritMatthew Treinish proposed openstack-infra/subunit2sql: Add graph command to show aggregate counts for tests  https://review.openstack.org/17803922:01
*** achanda has joined #openstack-infra22:01
*** ajmiller has quit IRC22:02
openstackgerritMerged openstack-infra/elastic-recheck: Update query for libvirt socket hangup bug 1451506  https://review.openstack.org/18068422:02
openstackbug 1451506 in OpenStack Compute (nova) "spawn failed with "libvirtError: internal error: received hangup / error event on socket" in the gate" [Undecided,Confirmed] https://launchpad.net/bugs/145150622:02
*** achanda has quit IRC22:02
*** Kennan2 has quit IRC22:02
*** achanda has joined #openstack-infra22:03
*** Kennan has joined #openstack-infra22:03
*** maurosr has joined #openstack-infra22:03
*** zz_ja has joined #openstack-infra22:06
*** jgrimm has quit IRC22:07
*** gordc has quit IRC22:07
openstackgerritMatthew Treinish proposed openstack-infra/subunit2sql: Add graph command to show aggregate counts for tests  https://review.openstack.org/17803922:07
clarkbpleia2: 07 is still looking good, so we should probably expect to do the others tomorrow22:07
pleia2clarkb: great22:07
*** jtriley has quit IRC22:07
*** zz_ja has quit IRC22:07
*** zz_ja has joined #openstack-infra22:07
clarkbok time to make a second container and see if I can make jenkins talk to you22:08
clarkber s/you/it/22:08
*** dkranz has joined #openstack-infra22:09
*** jgrimm has joined #openstack-infra22:09
*** zz_ja has quit IRC22:10
*** maurosr has quit IRC22:12
*** ociuhandu has quit IRC22:14
*** maurosr has joined #openstack-infra22:14
*** zz_ja has joined #openstack-infra22:15
*** sdake has quit IRC22:17
krotscheckclarkb,sdague,AJaeger: https://review.openstack.org/#/c/156044/ is a rebase with +2A that was blocked on a rebaseable merge conflict. Could one of you three drop the +A on it again?22:17
*** sdake has joined #openstack-infra22:17
*** sdake has quit IRC22:17
clarkbfrom my local jenkins trying to connect to a slave http://paste.openstack.org/show/215538/ (I can fix it, just wow)22:17
*** sdake has joined #openstack-infra22:17
openstackgerritSteve Leon proposed openstack-infra/project-config: Add devstack gate job for Cue project  https://review.openstack.org/18077422:18
*** adrian_otto has joined #openstack-infra22:18
fungii've currently got jenkins05 updating now. should be done with the last couple shortly22:18
clarkbkrotscheck: done22:18
fungijjb updating that is22:18
krotscheckclarkb: Thanks :)22:19
fungipuppet is reenabled on jenkins.o.o and 01-05 so far22:19
*** packet has quit IRC22:19
*** dprince has quit IRC22:20
*** isviridov is now known as isviridov_away22:21
*** gokrokve_ has quit IRC22:23
clarkbjeblair: zaro ok I have a slave attached to my master and its running jobs with label expressions so that looks good22:23
jeblairSpamapS, pleia2: so the idea was that most of what we eventually needed was detailed documentation for the system we're building; it seemed like we could just create the documentation, and treat that as the spec.22:24
pleia2jeblair: ok, fair enough22:25
jeblairSpamapS: however, i think i may have miscommunicated that a bit, as i had intended that it go into system-config/docs22:25
jeblairSpamapS: which where http://ci.openstack.org/ is built from, and has our operational documentation, as opposed to infra-manual, which is "the user manual for the system".  infra-cloud isn't user facing in the usual sense for us :)22:26
jeblairpleia2: anyway, in general i'm a fan of the specs process, and we might end up with one or more here; i thought this might end up being a special case though.  we'll see?22:27
pleia2jeblair: yep, we'll see :)22:27
jogohaha https://www.openstack.org/brand/interop/ uses github mirror not git.o.o22:28
pleia2google rank of github (better than ours) makes it continue to be a tough sell22:28
jeblairwow, if only this were an open source project so we could change that22:28
*** dizquierdo has quit IRC22:29
jeblairhogepodge: since i can't submit a patch to do so, would you please change that?  ^22:29
*** tonytan4ever has quit IRC22:30
*** DinaBelova has quit IRC22:30
*** tonytan4ever has joined #openstack-infra22:31
*** DinaBelova has joined #openstack-infra22:31
*** tonytan4ever has quit IRC22:31
*** tonytan4ever has joined #openstack-infra22:31
jogojeblair: even better they say " The list of required capabilities (with must-pass tests) and designated code sections are published on github and summarized below."22:31
openstackgerritMerged openstack-infra/project-config: Initial integration tests for storyboard  https://review.openstack.org/15604422:31
clarkbjeblair: zaro and i have confirmed taht we RESET_ABILITIES instead of sending CANT_DO if we are told to unregister all functions22:32
hogepodgejogo yeah. thanks22:32
pleia2\o/22:32
mordredSpamapS: I added some comments to the doc patch - the patch itself shoudl be super easy to migrate to the system-config repo fwiw22:32
jeblairhogepodge: equivalent link: https://git.openstack.org/cgit/openstack/defcore/tree/2015.04.json22:32
jogohogepodge: thanks22:33
*** whoops has quit IRC22:33
jeblair(i actually really like the openstack logo and repo destription up there)22:33
hogepodgejeblair: curious, why is one link worse than the other?22:34
hogepodgejogo ^22:34
jeblairhogepodge: well, we put a lot of work into making a nice git server for openstack.  it has openstack branding, it runs open source software, it's maintined by the community, and it's more reliable than github22:34
hogepodgejeblair: +122:35
hogepodgejeblair: thanks22:35
jeblairnp22:35
jeblairhogepodge: pleia2 did the lion's share of the work, but we're all very proud of it :)22:35
hogepodgejeblair: I'll be sure to update links wherever else I find them. TIL22:36
anteayayay pleia222:36
pleia2hogepodge: that's great, thanks22:36
jogojeblair: yeah our link looks good22:37
*** annegentle has quit IRC22:38
*** sarob has quit IRC22:38
*** salv-orl_ has quit IRC22:38
openstackgerritSpencer Krum proposed openstack-infra/system-config: Add the ability for template to manage exim  https://review.openstack.org/18078222:39
fungihogepodge: also, those third-party hosting sites have a tendency to close up and disappear over time (codehaus, gitorious, sourceforge, google code, et cetera)22:39
jogojeblair: I am reading  https://www.openstack.org/brand/ and having trouble seeing where the use of https://git.openstack.org/static/openstack.png in https://git.openstack.org/cgit/ is allowed22:39
fungihogepodge: pretty sure git.openstack.org will be around as long as openstack itself22:39
*** salv-orlando has joined #openstack-infra22:40
jeblairjogo: note the "openstack.org" at the end of "git.openstack.org" :)22:40
clarkbjeblair: even if it wasn't the trademark policy explicitly allows the logo for non commercial community use22:40
jogoclarkb: its not that simple anymore22:41
clarkbpretty sure openstackgitmirror.foo.com could do the same22:41
clarkbas long as they don't host ads22:41
*** baoli has quit IRC22:41
zarojeblair, clarkb, fungi, pleia2 : this just merged upstream and i think we need it for gerrit upgrade: https://review.openstack.org/#/c/18078122:41
*** baoli has joined #openstack-infra22:42
zaroclarkb: have been roaming, will take a look at your gearman-plugin change now22:42
fungizaro: i read through the bug report that's supposedly addressing and it had something to do with problems which only manifested when anonymous access is disallowed22:42
jogoYou must keep sufficient space around our Brand Assets so they appear clean and uncluttered.22:43
jogohttps://www.openstack.org/brand/openstack-logo/logo-download/22:43
uberjayasselin: totally worked, thanks a lot -- I really appreciate it!22:43
jogoI don't think we can have the blue background :)22:43
zarofungi: i must have missed that, re-reading again.22:43
jeblairjogo: the blue background was from the openstack website :)22:43
*** dkranz has quit IRC22:44
hogepodgejogo: jeblair: pleia2: fungi: all git links updated.22:44
jogojeblair: that appears to contradict the latest usage guidelines22:44
openstackgerritSpencer Krum proposed openstack-infra/system-config: Move afs servers to using o_p::template  https://review.openstack.org/18078322:44
jogohogepodge: another question  on https://www.openstack.org/brand/ try clicking the primary openstack logo22:44
jogoit goes to an email addr22:45
*** annegentle has joined #openstack-infra22:45
fungizaro: https://code.google.com/p/gerrit/issues/detail?id=2296 supposedly we have that bug in our version?22:45
nibalizerjeblair: clarkb can you look at 180783 and 180782 to see if that looks like what we discussed in channel?22:45
*** emagana has joined #openstack-infra22:45
jogo'Use of the primary logo is limited to educational and community building activities, for example educational presentations about OpenStack that are not attached to a commercial product or brand, or a blog post to talk about your involvement in the OpenStack community, not tied to a commercial product or brand'22:46
zarofungi: yeah, our version on jgit 3.7.0, fix for that was to update to 3.7.122:46
jogoif I say I work for HP in a slide deck can I have the OpenStack logo as well?22:46
jogomordred: ^22:47
zarofungi: and the one i just pushed up is a fix to that update22:47
fungizaro: i'm not opposed to adding more bugfixes to that branch and test again before we upgrade, but there are likely to be more that come up after we upgrade too so we'll need to plan for some future point release updates anyway22:47
hogepodgejogo: I'll ask. I'm sure the e-mail creates a ticket, but it is weird to have it as a header.22:48
jogohogepodge: yeah22:48
*** adrian_otto has quit IRC22:48
jogohogepodge: I am find it amusing how confusing the trademark policy is for community use22:48
zarofungi: are you proposing to wait and see?22:48
hogepodgejogo I think it's being reviewed right now, and hopefully will improve22:50
mtreinishjogo: heh, are you pointing out the issues with the openstack.org site? I remember when we went through that and submitted feedback forms on everything22:50
*** mrunge has quit IRC22:50
mtreinishI guess nothing changed22:50
jogo hogepodge: for example see my quote and question above.22:50
jogomtreinish: oh I forgot about that, I found typos in the bylaws22:51
jogohogepodge: glad to see it being cleaned up22:51
fungizaro: only if you don't think there's time to test it and confirm that we're still in good shape to upgrade after that patch which just merged today is added22:51
hogepodgejogo typos?22:51
fungijogo: hogepodge: the restricted use logos url mentioned at https://www.openstack.org/brand/openstack-logo/ is also 40422:52
mordredjogo: well, I'm not sure that's an infraqusetion really  ... but I use openstack logos on my slides all the time22:52
hogepodgeI'd totally be game for posting issues here https://etherpad.openstack.org/p/openstack-org-bug-reports22:52
hogepodgeand can follow up on them22:52
fungihogepodge: or just convince people to open lp bugs for openstack-org22:53
hogepodgeI know it's not the greatest bug tracker in the world. Or we could just create something in launchpad or watever the preferred bug tool is22:53
Rockyghogepodge, jogo:  and all the freakin' typefaces.....UGGGGGGLY22:53
mtreinishfungi: is it tracked on lp?22:53
hogepodgefungi that22:53
fungimtreinish: yes22:53
*** craigbr has quit IRC22:53
jogohogepodge: the bylaws at one point had bad links in them etc.22:53
fungiall things our community would just fix... once the www site is being automatically deployed from public source in gerrit22:54
*** Krinkle|detached is now known as Krinkle22:54
Rockygfungi:  ++22:54
jogomordred: no its not infra, asking you as a BoD member22:55
mordredjogo: ah! gotcha22:55
mordredjogo: so, I believe the intent is taht openstack developers should be able to talk about openstack without fear of reprisal from the openstack foundation22:55
RockygUhoh, time for some political doublespeak!22:55
mtreinishfungi: maybe that should be somewhere on the site then. It's not at all obvious, and at one point I didn't think that was the case22:55
jogomordred: I hope so, but that isn't what it says. If I say I work for HP that is partially commercial by HP22:55
jogoHP gets commercial value out of that ... I think22:56
mordredI do not believe that I agree with that22:56
mordredI believe if you were talking about HP and HP products that would be correct22:56
zarofungi: reading again it seems like the upgrade to jgit 3.7.1 was a mistake but instead of going back to 3.7.0 they went to 4.0.022:56
*** bswartz has joined #openstack-infra22:56
mordredI believe mentioning that HP employs you is a statement of fact that does not obviate the fact that you are an openstack developer22:56
jogoyou getting in front of a room of people and saying impressive things and saying I work for HP is good PR for HP no?22:56
zarofungi: so yeah, maybe just better to stay with 3.7.0 for now.22:56
fungimtreinish: https://wiki.openstack.org/wiki/Website22:56
jogomordred: hopefully, just pointing out the phrasing is open to interpretation22:57
mordredjogo: sure. but you are not standing up on stage talking about how awesome hp is and using openstack logos to do it22:57
fungimtreinish: i agree, the www site lacks the usual "how to report a problem with this site" links22:57
*** armax has joined #openstack-infra22:57
jogomordred: I bet someone somewhere has, but this is just a fun tangent22:58
jogofungi: ++22:58
*** SumitNaiksatam has quit IRC22:58
mordredjogo: :)22:58
mtreinishfungi: heh, yeah :)22:58
fungimtreinish: might be another good thing to put in a bug report ;)22:58
jogo. The framework for the www.openstack.org site is maintained at https://github.com/toddmorey/openstack-org, and patches are welcome22:58
*** bknudson has quit IRC22:58
jogolol openstack.org uses github22:58
fungijogo: also last updated 4 years ago22:59
jogooh that is 4 years old22:59
jogoyeah ...22:59
reedjogo, I think that information is old22:59
fungii think that wiki article is a little moldy around the edges22:59
reedfungi, which one?22:59
reedWebsite?22:59
fungireed: https://wiki.openstack.org/wiki/Website22:59
fungiyeah22:59
reedlets tag it as old23:00
* reed tags23:00
jeblairhttps://git.openstack.org/cgit/openstack-infra/openstackweb/23:00
fungihowever i can confirm that the e-mail address for todd and the bug tracker in lp are still correct and active23:00
jeblairhasn't been touched in 4 months23:00
reedhttps://github.com/OpenStackweb/ is newer23:01
fungiand yes, openstack-infra/openstackweb once we get the go-ahead from tipit that they're ready to start using that, at which point i'll refresh it from https://github.com/OpenStackweb/openstack-org23:01
reedor maybe not23:01
pleia2I think our git.o.o was last updated when clarkb was working on https://review.openstack.org/#/c/141571/23:01
*** packet has joined #openstack-infra23:01
jeblairreed: wait... so further activity is happening on the github repo?23:02
reedhttps://github.com/OpenStackweb/openstack-org it is newer23:02
fungipleia2: right, i don't want to keep pushing into gerrit for them until they confirm they're going to start using it23:02
pleia2fungi: yep23:02
jeblairreed: why not gerrit?23:02
openstackgerritmelanie witt proposed openstack-infra/elastic-recheck: Add query for libvirt cannot recv data bug 1452485  https://review.openstack.org/18078723:02
openstackbug 1452485 in OpenStack Compute (nova) "Error from libvirt "Cannot recv data: Connection reset by peer"" [Undecided,New] https://launchpad.net/bugs/145248523:02
reedjeblair, i'm as surprised as you are23:02
* fungi is less surprised, more just frustrated23:02
*** annegentle has quit IRC23:02
jeblairreed: okay, so we just need to decide on a day, we'll push the latest into gerrit, and then they can use that.23:03
pleia2I hold out hope we're still inching to the finish line23:03
reedthe positive thing is that the repo is public23:03
jeblairreed: can you help coordinate that?23:03
hogepodgeone should always assume a wiki article is moldy. Wiki is moldy by design23:03
reedjeblair, I can ask, that's all I can do at the moment23:03
*** dustins has quit IRC23:03
*** samueldmq has joined #openstack-infra23:04
jeblairreed: okay, i need someone at the foundation to take responsibility for this23:04
*** armax has quit IRC23:04
fungii think ttx plans to walk that gauntlet23:04
reedjeblair, it's something we need to discuss with Thierry23:04
jogojeblair: btw I like the favicon from infra way better then openstack.org23:05
jogoreed:  ^23:05
jeblairreed: ok23:05
jogothe .org has a ugly white box around it23:05
*** maurosr has quit IRC23:05
fungijogo: i think that's logo wonks getting pedantic about allowed background colors23:05
jogofungi: yup!23:06
fungitransparent background on a logo means you have no control over the background color! onoes!23:06
pleia2heh23:06
jogofungi: logo wonks would disapprove of https://review.openstack.org/23:06
jeblairttx: let's work on setting a date to switch the openstack-org workflow to gerrit.  we don't need to do anything else, just get them to submit new changes through gerrit.  they can self-approve with no testing and continue deploying the way they are now.  just a switch to gerrit is all.23:06
jheskethMorning23:07
*** zz_ja has quit IRC23:07
jeblairttx: we'll set a date to make that change, we will push the latest code into gerrit, and then they'll stop using the github repo23:07
fungi(amd in fact some of the tipit staff already do this for openstackid now)23:07
*** maurosr has joined #openstack-infra23:08
*** zz_ja has joined #openstack-infra23:09
jogogreat bug https://bugs.launchpad.net/openstack-org/+bug/131524023:09
openstackLaunchpad bug 1315240 in openstack-org "http://www.openstack.org/community/ congratulates jenkins frequently" [Low,Confirmed]23:09
jeblairhehe :)23:10
fungithat's no bug23:10
fungiwell, okay, it _should_ be congratulating zuul23:10
fungior the infra team23:10
*** baoli has quit IRC23:10
jeblairi hear the openstack proposal bot is a top contributor.  a veritable _machine_.23:10
anteayaha ha ha23:11
fungiindeed. i've had to disqualify her from several elections already23:11
fungion mere technicalities, mind you23:11
jogofungi: even better since it uses some launchpad feed things like https://launchpad.net/skylable get picked up23:12
mtreinishfungi: that feels like discrimination against non-human people23:12
jogosince openstack org is an open group23:12
fungimtreinish: brushing up on your asimov i see23:12
*** ondergetekende has quit IRC23:13
fungi("the bicentennial man" is one of my favorite explorations of personal rights)23:13
openstackgerritmelanie witt proposed openstack-infra/elastic-recheck: Add query for libvirt cannot recv data bug 1252947  https://review.openstack.org/18078723:13
openstackbug 1252947 in OpenStack Compute (nova) "libvirtError: Cannot recv data: Connection reset by peer" [Undecided,Confirmed] https://launchpad.net/bugs/125294723:13
*** adrian_otto has joined #openstack-infra23:13
mtreinishfungi: heh, man it's been a while since I read any asimov. Probably too long23:14
zarojeblair: about https://review.openstack.org/#/c/172534/23:15
mtreinishyeah, "The Bicentennial Man" is great23:15
zarojeblair: the bouncy castle jars are not packaged into the war23:15
fungizaro: what was unpacking it then?23:15
*** ondergetekende has joined #openstack-infra23:15
zarojeblair: it's like the mysql lib, it's downloaded23:16
fungizaro: oh--the gerrit init exec downloads it?23:16
zaroi believe that's due to licencing23:16
*** hemna is now known as hemnafk23:16
zarofungi: yep, so it's problematic23:16
jeblairfungi, zaro: ah, yeah, the gerrit init must have done it then...23:16
fungiwow. double-ick23:16
jeblairfungi, zaro: so we should probably either rely on that, or find a way to disable that and do what zaro was doing in his patch23:17
jogohaha the community page is pointing to bzr23:17
*** panda has quit IRC23:17
zarorelying on that is problematic i think23:17
zarobecause it may download different versions on updates to gerrit23:17
anteayajogo: while your are at it can you do something about SEO for things23:17
jogohttps://code.launchpad.net/nova23:18
anteayabecause I would like folks to find our docs easier23:18
fungijogo: maybe you should stop looking at that website. it's going to just keep disappointing you23:18
zarothen we would be left with different versions the file system23:18
*** panda has joined #openstack-infra23:18
jogofungi: I am having fun laughing at how bad it is23:18
fungizaro: great point. the filename is variable, so we'd end up having to clean any out prior to running init again23:18
anteayajogo: as long as you are enjoying yourself23:19
jeblairjogo: i think it's a good website i'd like to be able to help maintain :)23:19
zarojeblair: not sure if it's diableable, but deleting the jars is basically acomplishing the same thing23:19
jogojeblair: ++, the rough edges  just need some polish23:20
*** weshay has quit IRC23:22
zarojeblair: don't think it's possible to disable downloading of those libs, at least there's no option to disable for 'init' command23:23
jeblairzaro: i agree23:24
jeblairzaro: okay, so you're proposing to keep "tidying" the gerrit-downloaded jars and use our downloaded jars as in your change23:24
*** packet has quit IRC23:24
*** zz_dimtruck is now known as dimtruck23:24
zarojeblair: yes, unless you think there's a better way.23:24
openstackgerritmelanie witt proposed openstack-infra/elastic-recheck: Add query for libvirt cannot recv data bug 1252947  https://review.openstack.org/18078723:25
openstackbug 1252947 in OpenStack Compute (nova) "libvirtError: Cannot recv data: Connection reset by peer" [Undecided,Confirmed] https://launchpad.net/bugs/125294723:25
*** otter768 has joined #openstack-infra23:25
jeblairzaro: no, that makes sense.  and it looks like your change is still compatible with the tidy change because you use different names for the jars you download23:26
jeblairzaro: so i think you're change is what we need then23:26
jeblairfungi: sound right to you too?23:26
fungijeblair: yeah, i'm good with it23:26
jeblairfungi, zaro: oh, one more thing -- do we want to put the new stuff in a conditional based on gerrit version?23:27
jeblairso the puppet-gerrit module can continue (at least for a while as long as someone cares) to be used for 2.8?23:27
anteayait would be clearer what is going on23:27
anteayaand gerrit downloaded jars aren't triggered in any automated way, are they?23:28
fungii guess we would do that with an additional class parameter? or a comparison on a variable which is expanded as part of the war filename?23:28
openstackgerritmelanie witt proposed openstack-infra/elastic-recheck: Add context "libvirt" back into query for bug 1451506  https://review.openstack.org/18079323:29
openstackbug 1451506 in OpenStack Compute (nova) "spawn failed with "libvirtError: internal error: received hangup / error event on socket" in the gate" [Undecided,Confirmed] https://launchpad.net/bugs/145150623:29
zarothat's doable, you can get version info from war with --version23:29
fungizaro: well, i'm wondering how to feed that back into a puppet variable to switch on in the manifest though23:30
zarojeblair: i'll give it a try.23:30
zarofungi: that's probably how i would imagine doing it23:30
anteayait executes if contactstore == true23:30
zarofungi: is that puppet possible?23:30
*** otter768 has quit IRC23:31
anteayawhen does contactstore == true?23:31
*** nelsnelson has quit IRC23:31
fungizaro: it's not something i've ever tried, which is why i point out that it's an interesting mechanism to be considering for that purpose23:31
*** craigbr has joined #openstack-infra23:31
fungianteaya: it's passed in as a class parameter23:32
zaroanteaya: it's always true for our gerrit23:32
anteayathanks23:32
*** Rockyg has quit IRC23:32
*** rfolco has quit IRC23:34
zaroalright, i'll look into making it work for both version,  seems like it would be useful for others.23:34
SpamapSjeblair: you did not miscommunicate. I'm just unfamiliar with the layout of infra still. :) Moving the patch over.23:34
*** dims_ has joined #openstack-infra23:36
*** soren has quit IRC23:37
*** dims has quit IRC23:38
*** dimtruck is now known as zz_dimtruck23:39
openstackgerritClint 'SpamapS' Byrum proposed openstack-infra/system-config: Initial manual for infra-cloud  https://review.openstack.org/18079623:39
SpamapShm thats weird I'd swear I kept the change-id23:42
*** sarob has joined #openstack-infra23:42
SpamapSBefore I thrash a bit... thoughts? Better to just make it a new review, or keep the change-id?23:42
clarkbits new either way23:43
anteayait is the same change-id, it is a new repo23:45
anteayaso gerrit makes a new review number since the repo is different23:46
SpamapSgot it23:46
SpamapSwon't worry about it then23:46
anteayayou linked in comments so the history can be tracked23:46
*** david-lyle has joined #openstack-infra23:47
*** mmedvede_ has joined #openstack-infra23:47
jesusaurushas infra had any problems with gerrit replication causing the web ui and ssh api to be unresponsive?23:47
fungiSpamapS: yeah, you can see it's the same change id by going to https://review.openstack.org/#/q/Icb35adf70fa98e64dbbe0464b95af3b5de51a980,n,z23:48
fungijesusaurus: what gerrit release?23:48
fungijesusaurus: if 2.10.3 then sounds familiar to https://groups.google.com/forum/#!msg/repo-discuss/Kv4bWLESbQ4/-oSNbuTQwkUJ23:49
jesusaurusfungi: no, I'm stil on an ancient 2.423:50
fungijesusaurus: nope, doesn't sound familiar then23:50
jesusaurushrm, okay23:51
*** _nadya_ has joined #openstack-infra23:54
*** ayoung has quit IRC23:55
*** signed8bit is now known as signed8bit_ZZZzz23:56
*** _nadya_ has quit IRC23:59
*** ZZelle_ has quit IRC23:59

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