Monday, 2017-01-09

*** timrc has quit IRC00:32
*** jlk has quit IRC00:35
*** jamielennox is now known as jamielennox|away00:41
*** jamielennox|away is now known as jamielennox00:58
*** rattboi has quit IRC01:12
*** jesusaur has quit IRC01:12
*** tobiash_ has quit IRC01:12
*** herlo has quit IRC01:12
*** patrickeast has quit IRC01:12
*** mgagne has quit IRC01:12
*** saneax-_-|AFK has quit IRC01:12
*** mordred has quit IRC01:12
*** jhesketh has quit IRC01:12
*** olaph has quit IRC01:12
*** herlo has joined #zuul01:12
*** herlo has quit IRC01:12
*** herlo has joined #zuul01:12
*** olaph has joined #zuul01:13
*** mordred has joined #zuul01:13
*** jhesketh has joined #zuul01:13
*** rattboi has joined #zuul01:13
*** Guest96631 has joined #zuul01:13
*** saneax-_-|AFK has joined #zuul01:13
*** tobiash has joined #zuul01:13
*** jesusaur has joined #zuul01:14
*** patrickeast has joined #zuul01:17
*** timrc has joined #zuul01:19
*** timrc_ has quit IRC01:23
*** morgan has quit IRC01:23
*** auggy has quit IRC01:23
*** mattclay has quit IRC01:23
*** timrchavez_ has joined #zuul01:24
*** jlk has joined #zuul01:26
*** jlk has quit IRC01:26
*** jlk has joined #zuul01:26
*** auggy has joined #zuul01:28
*** mattclay has joined #zuul01:29
*** morgan has joined #zuul01:36
*** adam_g has quit IRC02:44
*** adam_g has joined #zuul02:45
*** bhavik1 has joined #zuul05:01
*** bhavik1 has quit IRC05:03
*** saneax-_-|AFK is now known as saneax05:06
*** Cibo_ has quit IRC06:14
*** bhavik1 has joined #zuul07:04
*** hashar has joined #zuul07:43
*** bhavik1 has quit IRC08:03
*** openstackgerrit has quit IRC08:18
*** mptacekx has joined #zuul08:25
mptacekxHi, I have one general zuul question. Can someone please reply on following ? Should projects section of layout.yaml contains all openstack projects (3,5k+) or just the projects we would like to listen to and potential dependencies of some commits should be merged by zuul-merger automatically?08:30
*** mattclay has quit IRC09:37
*** mattclay has joined #zuul09:38
*** saneax is now known as saneax-_-|AFK10:18
*** saneax-_-|AFK is now known as saneax11:12
mptacekxHi, can someone please give me some estimation when zuul v3 is ready to be taken by 3rd party CI's ?11:52
*** wznoinsk has joined #zuul12:00
*** hashar is now known as hasharLunch12:12
*** saneax is now known as saneax-_-|AFK12:41
*** saneax-_-|AFK is now known as saneax12:48
*** saneax is now known as saneax-_-|AFK13:11
*** saneax-_-|AFK is now known as saneax13:15
*** jamielennox is now known as jamielennox|away13:39
*** saneax is now known as saneax-_-|AFK14:00
*** mptacekx has quit IRC15:14
*** hasharLunch is now known as hashar15:48
*** hashar is now known as hasharAway17:32
jeblairmordred: what's the status of 410801and 410802?17:41
jeblairmordred: see comment on 29795017:42
* SpamapS knocks off the zuul dust17:42
*** toabctl has quit IRC17:42
SpamapSjeblair: the automated board looks positively achievable btw.. seems we've either been chipping away nicely, or untagging things. ;)17:42
jeblairSpamapS: \o/17:43
jeblairclarkb: istr some discussion about 410939; are you okay with that?17:44
jeblairi'm going to self-approve 41257117:45
clarkbre 410939 I do still think we should avoid change for changes sake. Yes, we are making a lot of changes but the fewer that are user visible the easier ti will be for them to transition to the new stuff17:46
clarkbbut I am claerly outnumbered on that so...17:46
jeblairclarkb: i agree with your principle there.  however, i think this is a good change to make: we *almost never* want to stop both of those daemons at the same time, so i feel bad telling users that's an okay configuration when it might back them into a corner.17:49
*** openstackgerrit has joined #zuul17:49
openstackgerritMerged openstack-infra/nodepool: Fix some doc typos  https://review.openstack.org/41257117:49
mordredjeblair: I believe they are still good - I totally thought they'd landed already17:49
jeblair(i feel the same way about zuul and geard)17:49
clarkbjeblair: if that is the argument I am not sure it was ever expressed. Just that it is legacy code and we want to chagne it17:49
jeblairmordred: oh, hrm, were they duplicated on a branch by any chance?  (though if so, i'd expect them to be in merge conflict now)17:50
clarkbjeblair: and it will require everyone to update their daemon invocations (either to remove the invalid --no-builder flag, or to add a new daemon invocation for the builder separately)17:50
openstackgerritDavid Shrewsbury proposed openstack-infra/nodepool: Create per-provider ZK work threads  https://review.openstack.org/41799417:50
jeblairclarkb: you know my sysadmin preferences -- i hate change :)17:50
Shrewsjeblair: If you get a chance, would like to see if you think 994 is on the right path for what you mentioned last week17:51
*** Guest96631 is now known as mgagne17:51
*** mgagne has quit IRC17:51
*** mgagne has joined #zuul17:51
clarkbjeblair: pabelanger maybe we can write down what the intent is there so that if we get angry users they can see what the justifcation was instead of just changing what was perceived to be bad legacy code?17:51
jeblairclarkb: maybe we should deprecate --no-builder to make it just print a warning for a bit?17:52
clarkbjeblair: that is probably a good idea too (on the master side ya?)17:52
jeblairclarkb: yep17:52
mordredjeblair: I submitted them to both branches17:53
mordredjeblair: they have landed on neither17:53
jeblairmordred: ah, i see the master ones are better reviewed17:54
pabelangerjeblair: clarkb: +1 for deprecate --no-builder, removal at a later date17:54
jeblairpabelanger: that wasn't exactly what i was suggesting :)17:54
jeblairpabelanger: i say remove it, but leave the option working so it doesn't blow up.17:54
clarkbI think jeblair is saying put the warning in now on the master side and remove it on the v3 side now17:54
clarkbthen when v3 merges into master later people will have had warning of the change17:55
mordredno-op command line args are cheap17:55
clarkboh gotcha ya that works too17:55
jeblairyep.  i don't think we need to go through a full deprecation cycle for this17:55
pabelangerokay, no-op is fine too17:55
pabelangerI'll update my patch17:55
clarkbbut also would be good to add the justification to the commit message17:55
jeblairmordred: should we +3 those now then?17:55
mordredjeblair: sure - unless you have concern about the nodepool-dsvm job being broken at the moment17:56
clarkb"Nodepool is now a properly distributed system and we do not want users to couple the starting/stopping of individual daemons when this is not required. As a result we have explicitly uncoupled the daemons"17:56
clarkbor similar17:56
pabelangerclarkb: Oh, BTW: mind looking at https://review.openstack.org/#/c/414662/. Was the only way I could get glean to actually work, I needed to use a tarball17:56
pabelangerclarkb: ++17:56
clarkbpabelanger: re glean, I had it working just fine with git locally17:57
clarkbpabelanger: let me boot my VM and check what flags I needed17:57
jeblairmordred: i do have concerns with those words, but i don't know what they mean :(   are you saying if we land that, the job will be broken, or...?17:57
jeblairmordred: (cause it looks green to me?)17:57
mordredjeblair: the job is currently broken17:57
mordredit broke sometime after jan 417:57
jeblairmordred: oh, not related to this change17:57
mordredyah17:57
pabelangerclarkb: issues I had with that, I could never get the depend-on version of glean to actually checkout inside DIB. I couldn't get the right ref17:58
jeblairmordred: so put another way: "hey, maybe we should make that job work before we land anything to nodepool" ?17:58
clarkbpabelanger: its teh same ref as in the calling environment17:58
mordredjeblair: well - yes. sc86cal is currently logged in to a held node investigating17:58
clarkbpabelanger: however it is possible we need to make it a proper branch/tag due to some dib thing17:59
mordredjeblair: the problem is that in the nodepool-dsvm and the shade-functional-neutron devstacks, nova is not getting ips back from neutron for servers17:59
pabelangerclarkb: Ya, that was actually the fix. I had to create a local branch, then use that branch for DIB17:59
pabelangerclarkb: so, I can do that, or the tarball17:59
mordredjeblair: it is exceptionally unclear as to why it isn't - but it's not related to shade (the same break happens with python-openstackclient too)17:59
jeblairmordred: okay.  yeah, i would be annoyed with myself if i +3d these and we ended up with 2 problems somehow, even though these should be fine.  so i'll hold off for now.18:00
mordred++18:00
pabelangerclarkb: https://review.openstack.org/#/c/414662/7/devstack/plugin.sh worked I believe18:01
pabelangerclarkb: using branch ttt18:01
clarkbpabelanger: source-repository-simple-init:glean git /tmp/glean.git https://git.openstack.org/openstack-infra/glean is how the repo gets installed18:02
clarkbpabelanger: iirc source-repository element only preserves branches and tags so that would likely explain it18:02
jeblairShrews: +2 with note18:03
Shrewsjeblair: cool. i thought that was a very weird option anyway18:03
clarkbpabelanger: I think my only concern with the tarball is that its not really how I would expect someone locally to do things (so we might run across corner case issues), but if it works its not that different than using a branch18:04
clarkbpabelanger: and with a tarball the benefit is that you don't add any new state to the repository (which itself could be side effecty)18:05
jeblairShrews: yeah, totally insane.  but we were backed up against the edge of a snake pit with a bunch of angry wet cats chasing us.  or at least that's how i remember it.18:05
pabelangerclarkb: I can switch patch to git, are you find with a hardcode branch? or do you want something more dynamic?18:05
clarkbpabelanger: I think we can just use tarball until we have problems with it18:05
pabelangerclarkb: okay18:05
pabelangerclarkb: well, we already have an issue with it. There is a bug in diskimage-builder, so we actually need a new release for it to work18:06
pabelangerso...18:06
clarkboh in that case lets use git :) because my theoretical tar isn't common and may be prone to issues is maybe less theoretical now18:07
clarkbpabelanger: I would make a branch called nodepool-devstack-plugin or similar and set that as the ref18:08
pabelanger++18:09
openstackgerritDavid Shrewsbury proposed openstack-infra/nodepool: Remove --no-launches nodepoold option  https://review.openstack.org/41800118:09
Shrewspabelanger: re 410939, you want to submit for features/zuulv3 branch too? or want me to go ahead and add it to my stack?18:22
Shrewsi'm in that code now, so i can do it if you want18:22
pabelangerShrews: sure, I haven't started yet18:22
Shrewscool. it simplifies my testing a bit by not having to type that extra option  :)18:26
openstackgerritDavid Shrewsbury proposed openstack-infra/nodepool: Remove --no-builder nodepoold option  https://review.openstack.org/41801318:28
*** harlowja_ has joined #zuul18:31
*** harlowja has quit IRC18:31
ShrewsI think the -s option is next on the chopping block18:32
jeblairShrews: I hope so, though i'm worried we may end up needing something like that for authenticated zk access?18:34
Shrewsjeblair: hrm, good point18:35
openstackgerritPaul Belanger proposed openstack-infra/nodepool: Remove the ability for nodepoold to launch a builder  https://review.openstack.org/41093918:39
openstackgerritMerged openstack-infra/zuul: Fix test_rerun_on_abort  https://review.openstack.org/41142618:40
harlowja_mordred any knowledge on https://www.ansible.com/open-tower yet :-P18:53
harlowja_is that happenening yet, lol18:53
mordredrbergeron: ^^ I think that question was really for you ;)18:54
jlk"soon"18:54
mordredharlowja_: so - _yes_ it is definitely happening. but no I don't have any info on timeframe18:55
harlowja_:-P18:55
harlowja_be real nice18:55
harlowja_i think18:55
harlowja_especially with stuff like kolla-ansible18:56
mordredyah19:00
mordredI'm looking forward to it19:00
rbergeroni know nothing about kolla-ansible!19:03
rbergeronoh, you'er asking about open-tower.19:03
rbergeronWE ARE WORKING ON IT and i yell about it on a weekly basis at the bare minimum.19:03
harlowja_rbergeron ya19:03
harlowja_is that like maybe this month?19:03
harlowja_:-P19:03
rbergeronharlowja_: i cannot confirm or deny such things but i would not recommend you hold your breath on such things until january 31st. :(19:05
rbergeronsince we like you being alive.19:05
harlowja_lol19:05
* mordred enjoys live harlowja_s more than dead harlowja_s19:05
clarkbhowever this may be good science for testing the hypothesis that if harlowja_ holds hsi breaks for 2 weeks we all get what we want19:06
rbergeronharlowja_: all that said, if you have specific use cases or "we're not waiting so we're going to xyz instead" stories you would like to share with me via email, I am all for additional ammunition to cut through the bureaucracy (there really isn't much of this) or the politics (there is a bit more here, but not much, but enough to move things along slowly)19:06
harlowja_rbergeron i can propose something via email, sure19:07
rbergeronbut the important thing to me is to do it right, and make sure it's resourced correctly (since handling inevitable flood of contributors and governance and documentation takes effort, and it's Muy UnBueno for happiness without)19:07
harlowja_right right19:08
harlowja_makes sense19:08
rbergeronblah blah, y'all know this stuff :)19:08
harlowja_ya19:08
rbergeronrobyn@redhat if you want to drop me a line, i am happy to add it to my boiling pot of19:08
rbergeronof .. I guess my boiling pot of Feeling of Hurry Up Already19:08
rbergeron:)19:09
harlowja_or i'll suffocate; i can't hold my breath much longer19:09
harlowja_lol19:09
rbergeron(and i was joking about kolla-ansible, also ;D)19:09
harlowja_:-P19:13
openstackgerritDavid Shrewsbury proposed openstack-infra/nodepool: Handle provider removal from config  https://review.openstack.org/41811221:00
*** jamielennox|away is now known as jamielennox21:00
*** timrc has quit IRC21:01
Shrewspabelanger: i thought https://review.openstack.org/410939 was supposed to go against master, and features/zuulv3 branch would totally remove it (as in https://review.openstack.org/418013). Did I misread scrollback earlier?21:05
*** jlk has quit IRC21:06
pabelangerShrews: Ah, yes. I need to move that to master21:08
*** hasharAway is now known as hashar21:26
*** timrc has joined #zuul21:49
*** jlk has joined #zuul21:51
*** jlk has quit IRC21:51
*** jlk has joined #zuul21:51
clarkbyup thats it, note you can't have that change number on master iirc because new change will be made due to branch move21:54
jheskethMorning21:55
jeblairit's zuul meeting time in #openstack-meeting-alt22:01
openstackgerritPaul Belanger proposed openstack-infra/nodepool: Source glean installs in simple-init  https://review.openstack.org/41466222:02
openstackgerritPaul Belanger proposed openstack-infra/nodepool: Remove the ability for nodepoold to launch a builder  https://review.openstack.org/41813722:18
openstackgerritPaul Belanger proposed openstack-infra/nodepool: Remove the ability for nodepoold to launch a builder  https://review.openstack.org/41813722:19
*** saneax-_-|AFK is now known as saneax23:25
*** harlowja_ has quit IRC23:40
*** harlowja has joined #zuul23:41

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