| jeblair | jhesketh, mordred: ^ | 00:00 |
|---|---|---|
| dims | jeblair, where are the logs you were looking at? (just curious) | 00:00 |
| jeblair | dims: https://etherpad.openstack.org/p/XrzCW0EARb | 00:00 |
| dims | thanks! | 00:00 |
| mordred | jeblair: +2 +A | 00:01 |
| *** matsuhashi has joined #openstack-infra | 00:01 | |
| *** msabramo has quit IRC | 00:01 | |
| openstackgerrit | A change was merged to openstack-infra/config: Reduce gearman logging level https://review.openstack.org/95972 | 00:02 |
| jeblair | i pushed it through ^ | 00:02 |
| fungi | lgtm | 00:03 |
| *** gokrokve has quit IRC | 00:03 | |
| *** lcheng_ has joined #openstack-infra | 00:03 | |
| fungi | pizza made and eaten, scrollback consumed as desert | 00:03 |
| fungi | dessert | 00:03 |
| *** gokrokve has joined #openstack-infra | 00:04 | |
| mattoliverau | fungi: sounds like a delicious dessert :P | 00:04 |
| jeblair | fungi: short version: logs here: https://etherpad.openstack.org/p/XrzCW0EARb nodepool status command sent to gearman taking > 30 seconds seems to be the problem | 00:04 |
| jeblair | fungi: cause unknown | 00:04 |
| openstackgerrit | Mark Sturdevant proposed a change to openstack/requirements: Remove hplefthandclient from global-requirements https://review.openstack.org/95974 | 00:04 |
| fungi | and additional logging seems to have exacerbated it | 00:04 |
| *** msabramo has joined #openstack-infra | 00:05 | |
| jeblair | fungi: i don't really have any reason to suspect that other than that's the main thing we've changed and it seems to have been getting worse | 00:05 |
| jeblair | fungi: so that one's just a shot in the dark | 00:05 |
| jeblair | fungi: i'm hoping that plus changing the timeout to 300 seconds will at least make things go | 00:06 |
| jeblair | fungi, mordred, clarkb, SergeyLukjanov: zuul is using a manually installed "gear" library with the timeout and logging changes | 00:06 |
| *** W00dy__ has joined #openstack-infra | 00:07 | |
| jeblair | fungi, mattoliverau, dims: i added in the extra logging entries between the status command start/stop to the etherpad | 00:07 |
| mattoliverau | Thanks jeblair | 00:07 |
| mordred | jeblair: there's a lot of us now | 00:08 |
| jeblair | fungi, mattoliverau, dims: the main context missing from that is that normally the log is awash in send/receive packet log entries, but there aren't any during that time because it's busy | 00:08 |
| mordred | it's hard to just ping infra-core | 00:08 |
| *** liyuezho has joined #openstack-infra | 00:08 | |
| *** gokrokve has quit IRC | 00:08 | |
| jeblair | #status log zuul is using a manually installed "gear" library with the timeout and logging changes | 00:08 |
| openstackstatus | jeblair: finished logging | 00:08 |
| jeblair | mordred: thanks for reminding me | 00:08 |
| jeblair | mordred: well i was really pinging infra-root there | 00:09 |
| *** msabramo has quit IRC | 00:09 | |
| mordred | jeblair: nod | 00:09 |
| jeblair | restarting zuul | 00:09 |
| *** mrodden has quit IRC | 00:09 | |
| jogo | so random idea / question: | 00:10 |
| mordred | uhoh | 00:10 |
| jogo | for unit tests, we have a problem where we skip some tests for optional dependencies | 00:10 |
| jogo | and those aren't listed in the local requirements file. | 00:10 |
| *** otherwiseguy has quit IRC | 00:10 | |
| mordred | yup | 00:10 |
| *** otherwiseguy has joined #openstack-infra | 00:11 | |
| jogo | long term it would be nice to do a requirments directory | 00:11 |
| jogo | not just a single list | 00:11 |
| jogo | but until then, can we just run the unit tests with all of global-requirements installed? | 00:11 |
| fungi | jogo: in the venv? | 00:11 |
| jogo | maybe as an additional job | 00:12 |
| jogo | fungi: presumably in a venv, but I don't think that is required | 00:12 |
| jeblair | zuul is running | 00:12 |
| anteaya | yay | 00:12 |
| fungi | jogo: worth noting, that won't solve version-dependent tests | 00:13 |
| anteaya | let's hope it stays that way, running that is | 00:13 |
| jogo | fungi: right. but if we have those (I am sure we do) that is pretty ugly | 00:13 |
| fungi | jogo: you could test it by replacing requirements.txt with the current global reqs and emptying out test-requires.txt, just to see what blows up | 00:14 |
| *** lcheng_ has quit IRC | 00:15 | |
| jogo | fungi: good idea | 00:15 |
| *** melwitt has quit IRC | 00:17 | |
| *** otherwiseguy has quit IRC | 00:17 | |
| fungi | jogo: as far as an additional job, that would be pretty easy (and i think necessarily separate, since otherwise we won't test that the listed test requirements are sufficient to run tests) | 00:18 |
| mordred | jogo: we've also discussed running unittests in a devstack | 00:19 |
| devananda | hemna: hi! around? i'm curious why you're removing hplefthandclient from global req's | 00:21 |
| jogo | mordred: that seems like overkill or at least as as second phase | 00:22 |
| *** SumitNaiksatam_ has joined #openstack-infra | 00:22 | |
| fungi | on the other hand, it may be nice to test that devstack provides a suitable environment for running unit tests (a long-standing request) | 00:23 |
| *** SumitNaiksatam has quit IRC | 00:23 | |
| *** SumitNaiksatam_ is now known as SumitNaiksatam | 00:23 | |
| *** zzelle has quit IRC | 00:24 | |
| *** chuckC has quit IRC | 00:24 | |
| *** bhuvan_ has quit IRC | 00:27 | |
| *** gabriel-bezerra has left #openstack-infra | 00:27 | |
| *** SumitNaiksatam has quit IRC | 00:28 | |
| jogo | fungi: yeah, although the overhead of spinning up another devstack sounds concerning | 00:29 |
| *** praneshp has quit IRC | 00:29 | |
| jogo | but not unreadable | 00:29 |
| jogo | unreasonable | 00:29 |
| mordred | jogo: well, we alraedy spin up throwaway nodes | 00:30 |
| mordred | jogo: thing to be aware of also - pip dependency lists are order dependent, and global-requirements is not intended to be installed, so it's alphabetized | 00:31 |
| mordred | so you'll get some potential weirdness | 00:31 |
| mordred | jogo: I actually think doing the first thing you mentioned is actually less hard than you might think - since it's already a supported construct in setuptools | 00:32 |
| *** gokrokve has joined #openstack-infra | 00:32 | |
| mordred | jogo: https://pythonhosted.org/setuptools/setuptools.html#declaring-extras-optional-features-with-their-own-dependencies | 00:33 |
| mordred | jogo: so we'd need to add some support to pbr to deal with inserting stuff into extra_requires | 00:33 |
| mgagne | is gerrit replication plugin a system wide option or can it be enabled per project? | 00:34 |
| *** dstanek_zzz is now known as dstanek | 00:34 | |
| *** zhiyan_ is now known as zhiyan | 00:34 | |
| mordred | mgagne: the replication plugin is system wide - but it has config to control what gets replicated | 00:34 |
| mordred | mgagne: one of the controls is that you can tell it to only replicate things that a particular user or group can access | 00:34 |
| mgagne | mordred: ok so it's only a matter of creating a group with limited access? | 00:35 |
| mordred | mgagne: so you could, if you needed really specific controls, "enable" replication for a repo by having a group that by default could see nothign and than explicitly granting that group view permissions on each repo you wanted | 00:35 |
| mordred | yah | 00:35 |
| mgagne | mordred: cool thanks | 00:35 |
| *** zhiyan is now known as zhiyan_ | 00:36 | |
| *** Sukhdev has quit IRC | 00:38 | |
| *** zhiyan_ is now known as zhiyan | 00:38 | |
| * jogo is still waiting for nova tests to finish | 00:38 | |
| *** yamahata has joined #openstack-infra | 00:39 | |
| *** zhiyan is now known as zhiyan_ | 00:41 | |
| openstackgerrit | Morgan Fainberg proposed a change to openstack-infra/gitdm: Update company affiliation for morganfainberg https://review.openstack.org/95980 | 00:42 |
| *** dstanek is now known as dstanek_zzz | 00:44 | |
| *** UtahDave has quit IRC | 00:45 | |
| *** SumitNaiksatam has joined #openstack-infra | 00:46 | |
| *** gokrokve_ has joined #openstack-infra | 00:56 | |
| devananda | jogo: i vaguely recall vim encoding lines in nova files at one point. do you know if/why they were removed? | 00:57 |
| *** BadCub01_ has quit IRC | 00:57 | |
| *** wenlock has joined #openstack-infra | 00:57 | |
| * jogo can't tell if devananda is trolling or not | 00:58 | |
| *** liyuezho has quit IRC | 00:58 | |
| devananda | jogo: sort of -- but i am actually looking for the reference | 00:58 |
| mordred | devananda: we removed them because they were both incorrect and also because we didn't want to support all the editors in teh world | 00:59 |
| devananda | that's what i thought | 00:59 |
| jogo | devananda: there was a big ML thread but https://wiki.python.org/moin/Vim | 00:59 |
| jogo | and what mordred said | 00:59 |
| jogo | turns out they aren't very secure, and are disabled by default etc | 00:59 |
| *** gokrokve has quit IRC | 01:00 | |
| devananda | remember what the ML thread month / title was? | 01:00 |
| devananda | i just realized there's a bunch still in ironic/client, and someone proposed converting them all to emacs lines | 01:00 |
| *** dstanek_zzz is now known as dstanek | 01:01 | |
| jogo | http://lists.openstack.org/pipermail/openstack-dev/2013-October/017353.html | 01:01 |
| devananda | jogo: perfect, thanks | 01:01 |
| devananda | i was searching 2014 :( | 01:01 |
| jogo | devananda: another reason they are a termi-ism | 01:01 |
| *** ArxCruz has joined #openstack-infra | 01:01 | |
| jogo | amazingly enough I just clicked the first link when I googled 'vim modelines openstack' | 01:02 |
| devananda | hah | 01:02 |
| * devananda fails | 01:02 | |
| jogo | I didn't say that to shame you, I said it because I am impressed with google | 01:02 |
| jogo | this time | 01:02 |
| jogo | impressed with google this time* | 01:03 |
| * devananda doesn't feel shame for not using google | 01:03 | |
| mordred | devananda: you're a google | 01:03 |
| * devananda googles mordred | 01:04 | |
| * mordred returns a wet cat | 01:04 | |
| jogo | "known as a notorious traitor" | 01:06 |
| openstackgerrit | A change was merged to openstack-infra/storyboard-webclient: Fix StoryBoard spelling https://review.openstack.org/93406 | 01:06 |
| *** wenlock has quit IRC | 01:07 | |
| *** chuckC has joined #openstack-infra | 01:10 | |
| jogo | mordred: thats what wikipedia says about you | 01:10 |
| fungi | wikipedia is never wrong | 01:14 |
| fungi | NEVER | 01:14 |
| * mordred throws wet cats at everyone | 01:15 | |
| * jesusaurus "corrects" whichever wiki page claims that wikipedia is never wrong | 01:16 | |
| morganfainberg | mordred, i .. great, do you know how hard it is to get wet cat smell out of carpet? | 01:16 |
| *** wenlock has joined #openstack-infra | 01:18 | |
| *** camunoz has quit IRC | 01:20 | |
| *** saschpe has quit IRC | 01:23 | |
| *** dstanek is now known as dstanek_zzz | 01:24 | |
| *** dstanek_zzz is now known as dstanek | 01:24 | |
| *** saschpe has joined #openstack-infra | 01:24 | |
| *** zhiyan_ is now known as zhiyan | 01:26 | |
| *** gokrokve_ has quit IRC | 01:27 | |
| *** gokrokve has joined #openstack-infra | 01:28 | |
| jogo | fungi: running nova unit tests with all of global deps installed worked | 01:28 |
| jogo | I am very surprised | 01:28 |
| *** wenlock has quit IRC | 01:29 | |
| *** weshay has quit IRC | 01:29 | |
| jogo | so assuming folks want to run the unit tests with all of global reqs installed, are there any infra side concerns? | 01:29 |
| jogo | when do we EOL havana? | 01:31 |
| *** gokrokve has quit IRC | 01:32 | |
| jeblair | fungi, mordred, mattoliverau, jhesketh: i updated https://etherpad.openstack.org/p/XrzCW0EARb with an strace | 01:34 |
| fungi | a while--i think we extended it past k assuming interested parties step up to keep it working | 01:35 |
| fungi | jogo: ^ | 01:35 |
| *** melwitt has joined #openstack-infra | 01:35 | |
| jeblair | fungi, mordred, mattoliverau, jhesketh: i ran the 'status' command repeatedly until it looked like it was very slow, and then started an strace | 01:35 |
| jeblair | fungi, mordred, mattoliverau, jhesketh: it's looking like the thread handling the status command (15898) was swapped out in favor of the connection handling thread (15899) which stayed running for a VERY long time | 01:36 |
| fungi | does the connection handling thread need to yield more often? | 01:37 |
| jogo | fungi: thanks | 01:37 |
| *** weshay has joined #openstack-infra | 01:37 | |
| fungi | just curious what the usual approach to that is | 01:37 |
| jeblair | fungi: it should spend most of its time in poll(), so i wouldn't expect it to be a problem | 01:39 |
| fungi | maybe there are connection setup/teardown operations which are blocking for longer periods? | 01:39 |
| devananda | jeblair: regarding making ironic vote on nova as a third-party driver, when'd be a good time to chat about how to make that happen? | 01:40 |
| devananda | jeblair: or is there a magical wiki that will answer all my questions on that? | 01:40 |
| jeblair | devananda: not today :) | 01:40 |
| devananda | jeblair: tomorrow? | 01:41 |
| jeblair | devananda: i hope so | 01:42 |
| devananda | great | 01:42 |
| jeblair | fungi: take a look at the timestamps in the strace; i don't fully understand it yet... | 01:42 |
| jeblair | but it's well past dinner time here, so i have to run | 01:42 |
| fungi | goes silent after poll for 4-5 seconds a couple times, yeah | 01:43 |
| fungi | looks like it accepts a new connection after each of those? | 01:45 |
| fungi | different initiating ip addresses | 01:46 |
| jeblair | fungi: yeah, i think that's the case; if you recall from the earlier logs, the only thing geard was duing during the slow time was accepting a handful of new connections | 01:46 |
| fungi | ahh, right. still not sure what the implication of that is, but maybe it'll come to me | 01:47 |
| *** nosnos has joined #openstack-infra | 01:48 | |
| openstackgerrit | A change was merged to openstack-infra/jenkins-job-builder: Fail if specified set of defaults is not found. https://review.openstack.org/90399 | 01:49 |
| mattoliverau | jeblair, fungi yeah the silence is strange, it that the time for the system to accepts the previous conncetion, and then what polls and waits for a second connection. Or is it working with the connection (without issuing system calls, which sounds doubtful) during the silent times. Unless there is a context switch to a different process (that the connection thread needs to wait for), which wont be | 01:52 |
| mattoliverau | logged as it isn't actually run by zuul. | 01:52 |
| clarkb | o/ | 01:53 |
| clarkb | reading the sb now | 01:53 |
| mattoliverau | Maybe I'll look at zuul io load, or number on context switches. Though I would think on a multi core/threaded system if it was in wait, it could continue running the status thread. | 01:53 |
| *** ArxCruz has quit IRC | 01:54 | |
| jeblair | mattoliverau, fungi: nothing substantial happens between the poll and accept calls, so i suspect the time really is spent in poll | 01:56 |
| jeblair | mattoliverau, fungi: here's the loop in question: http://git.openstack.org/cgit/openstack-infra/gear/tree/gear/__init__.py#n2321 | 01:56 |
| *** changbl has joined #openstack-infra | 01:57 | |
| jeblair | (and you can tell when it gets to the bottom with the write('1') | 01:57 |
| clarkb | jeblair are things "working" with the longer timeout? | 01:57 |
| clarkb | and it isnt possible to use C gearmand at this point right? | 01:58 |
| jeblair | clarkb: yeah, hasn't hit again yet | 01:58 |
| *** Ivo has quit IRC | 01:58 | |
| jeblair | clarkb: no one has added the needed bits to it | 01:58 |
| mattoliverau | lots of the memeory is used up in caches, so although most of it is free, maybe the gap is time in kernl space, as there seems to be many many gigs of caches being used. But yeah, or it's just idle, by why not run the other thread again then.. hmm. I'll look at the loop now. | 01:58 |
| fungi | seems to be working so far, and the queue backlogs are once again being whittled away | 01:58 |
| jeblair | clarkb: i'm not at all convinced we've hit some kind of scalability limit here... | 02:00 |
| fungi | it does feel like we're exposing some sort of infrequent race/deadlock | 02:01 |
| jeblair | clarkb: we have a thread that is spending _seconds_ inside of a system call while another thread is starving. i'm really interested in understanding why that's the case. | 02:01 |
| *** wenlock has joined #openstack-infra | 02:02 | |
| jeblair | fungi: i suspect "rate of client connects/disconnects" is related, the nodepool config changes that made hpcloud 1.1 work better may have been the trigger | 02:02 |
| *** Ivo has joined #openstack-infra | 02:02 | |
| *** alugovoi has quit IRC | 02:03 | |
| fungi | timing would be about right, given that today was the first heavy use day we've seen for a while | 02:03 |
| *** yaguang has joined #openstack-infra | 02:06 | |
| clarkb | jeblair wow fun (re seconds in syscall) | 02:06 |
| clarkb | poll is where that happens? | 02:07 |
| jeblair | yeah, there's an instance where it's 4 seconds from poll to the next call | 02:07 |
| fungi | according to the systrace anyway | 02:07 |
| clarkb | huh | 02:07 |
| jeblair | poll realeases the GIL, so if another thread gets scheduled, it should be able to run | 02:08 |
| clarkb | can you s/poll/epoll/? curious if that changes anything | 02:08 |
| *** zz_gondoi is now known as gondoi | 02:09 | |
| jeblair | clarkb: we're not actually dealing with reading/writing here, we're just accepting connections | 02:09 |
| jeblair | clarkb: (the poll is to determine if there's a connection to accept) | 02:10 |
| *** gondoi is now known as zz_gondoi | 02:10 | |
| jeblair | clarkb: and we're not blocking on the accept, it's the poll that's sitting there | 02:11 |
| clarkb | right but isnt the syscall different? | 02:11 |
| *** homeless has quit IRC | 02:11 | |
| *** zz_gondoi is now known as gondoi | 02:11 | |
| jeblair | clarkb: yeah, but why would this make a difference? | 02:11 |
| clarkb | bug in libc or kernel maybe | 02:12 |
| mattoliverau | The first poll was interuppted, and continued about 6 seconds later, the second poll wasn't, but still didn't continue for about 5 secionds.. So I don't know why it didn't get interupted but looks like it normally takes 5 or so seconds to poll, and that is the lost time. | 02:12 |
| jeblair | clarkb: the only way we have to test this is in production; i've been unable to produce it locally, even in a setup that's twice the size we're running | 02:12 |
| jeblair | clarkb: so i don't want to try _every_ idea :) | 02:12 |
| clarkb | ah ok | 02:12 |
| *** camunoz has joined #openstack-infra | 02:13 | |
| jeblair | okay, really dinner now. good luck, anyone who wants to work on this. ;) | 02:13 |
| *** nati_uen_ has quit IRC | 02:13 | |
| clarkb | the other thing we can do is put a timeout on the poll | 02:23 |
| clarkb | but without a reproduceable case this is hard to test | 02:23 |
| *** weshay has quit IRC | 02:24 | |
| mattoliverau | clarkb: yeah, lookingingat more of the polls in the strace, the first one only takes a second, and then it started to take longer. jeblair said we ran status a few times before running the strace, so maybe it has something to do with the number of connections. Maybe a red herring? or number of connections is the trigger? | 02:27 |
| mattoliverau | *he ran | 02:28 |
| *** rfolco has quit IRC | 02:29 | |
| *** wenlock_ has joined #openstack-infra | 02:30 | |
| *** gondoi is now known as zz_gondoi | 02:30 | |
| clarkb | mattoliverau: these poll() calls don't specify a timeout so should block | 02:31 |
| clarkb | but as I understand it they shouldn't block because the FDs polled on have data? | 02:31 |
| *** asselin_ has joined #openstack-infra | 02:32 | |
| *** mkerrin1 has joined #openstack-infra | 02:32 | |
| jeblair | so the problem could be related to the thread that's doing the connection handling (poll); or it could equally be related to the other thread, the one handling the status command | 02:33 |
| jeblair | perhaps it's not being scheduled because it isn't ready; maybe the sendto is blocking or something | 02:33 |
| *** mkerrin has quit IRC | 02:33 | |
| *** asselin has quit IRC | 02:33 | |
| *** Ivo has quit IRC | 02:33 | |
| *** YorikSar has quit IRC | 02:33 | |
| jogo | mordred: https://review.openstack.org/#/c/95724/ | 02:34 |
| jeblair | clarkb: no, they should block. | 02:34 |
| *** YorikSar has joined #openstack-infra | 02:34 | |
| jeblair | clarkb: which is fine. the problem is that the other _thread_ isn't running. | 02:34 |
| jeblair | clarkb: (which it should be able to do since the poll is blocking) | 02:34 |
| clarkb | jeblair: the handleStatus() thread is not running | 02:35 |
| clarkb | ? | 02:35 |
| mattoliverau | jeblair: yeah, interestingly, the sendto (status) is swapped out for a poll, but on line 88, a poll is swapped out for the sendto.. so yeah, the poll could be a red herring and sendto could be being blocked on something else, or isn't ready. | 02:35 |
| jeblair | clarkb: yeah it's the one doing all the sendto calls ^ | 02:36 |
| *** zhiyan is now known as zhiyan_ | 02:36 | |
| clarkb | jeblair: I think they are the same thread | 02:37 |
| clarkb | handleAdminRequest() is called in the main _pollLoop() | 02:38 |
| clarkb | er wait maybe that is a different class | 02:38 |
| jeblair | clarkb: no, that thread is 15898; the connection handling thread is 15899. | 02:39 |
| clarkb | nope just overridden | 02:39 |
| jeblair | clarkb: the packet handling loop and the connection handling loop are constructed the same way, so it's easy to confuse them | 02:39 |
| clarkb | ah I see, the poll are part of connection handling not packet handling | 02:41 |
| *** alugovoi has joined #openstack-infra | 02:43 | |
| *** unicell has joined #openstack-infra | 02:44 | |
| *** olaph has quit IRC | 02:47 | |
| jeblair | clarkb: yup | 02:48 |
| *** gyee has quit IRC | 02:49 | |
| *** zz_gondoi is now known as gondoi | 02:49 | |
| *** otherwiseguy has joined #openstack-infra | 02:49 | |
| *** zhiyan_ is now known as zhiyan | 02:49 | |
| *** gondoi is now known as zz_gondoi | 02:49 | |
| *** zz_gondoi is now known as gondoi | 02:50 | |
| *** alugovoi has quit IRC | 02:52 | |
| *** gondoi is now known as zz_gondoi | 02:54 | |
| *** otherwiseguy has quit IRC | 02:54 | |
| *** msabramo has joined #openstack-infra | 02:54 | |
| mattoliverau | I'm off to grab some lunch, bbs | 02:54 |
| *** rcarrill` has joined #openstack-infra | 02:55 | |
| *** dims has quit IRC | 02:57 | |
| *** rcarrillocruz has quit IRC | 02:58 | |
| *** Ivo has joined #openstack-infra | 03:00 | |
| lifeless | mordred: any thoughts on whats up in http://logs.openstack.org/07/94107/2/check/gate-pbr-python26/00eaef3/console.html ? | 03:04 |
| *** mmaglana has joined #openstack-infra | 03:08 | |
| jeblair | fungi, clarkb, mattoliverau: i got a full packet capture and i see duplicate acks, retransmissions, window updates and zero window packets | 03:11 |
| mgagne | mordred: although more cumbersome to manage, it's possible to list projects you want to replicate (with the gerrit plugin) using the remote.NAME.projects config https://review.typo3.org/plugins/replication/Documentation/config.html | 03:12 |
| jeblair | fungi, clarkb, mattoliverau: i'm leaning toward the idea that we're hitting a condition where nodepool can't accept any more data, and the send is blocking | 03:13 |
| mgagne | mordred: might be useful for people with inconsistent naming scheme where the gerrit project name does not match the one on the destination | 03:14 |
| *** harlowja_ is now known as harlowja_away | 03:16 | |
| *** zhiyan is now known as zhiyan_ | 03:16 | |
| *** reed has joined #openstack-infra | 03:20 | |
| *** dstanek is now known as dstanek_zzz | 03:21 | |
| *** zhiyan_ is now known as zhiyan | 03:22 | |
| openstackgerrit | A change was merged to openstack-infra/elastic-recheck: Add query for bug 1308715 https://review.openstack.org/95918 | 03:24 |
| uvirtbot | Launchpad bug 1308715 in nova/icehouse "Deadlock on quota_usages" [High,Fix committed] https://launchpad.net/bugs/1308715 | 03:24 |
| *** plars has quit IRC | 03:30 | |
| *** plars has joined #openstack-infra | 03:31 | |
| *** wenlock_ has quit IRC | 03:35 | |
| openstackgerrit | A change was merged to openstack-infra/gear: Log admin requests https://review.openstack.org/95964 | 03:37 |
| *** zhiyan is now known as zhiyan_ | 03:41 | |
| mmaglana | hi, i'm trying to setup the jenkins job for aviator (https://review.openstack.org/gitweb?p=stackforge%2Faviator.git) and was wondering if i can define somewhere in the config a specific ruby version (1.9.3 and 2.1.0) to be used for running tests? | 03:42 |
| *** arnaud has joined #openstack-infra | 03:44 | |
| *** nosnos has quit IRC | 03:47 | |
| *** yfried has joined #openstack-infra | 03:50 | |
| *** pcrews has quit IRC | 03:55 | |
| mattoliverau | jeblair: nice work. That sounds very plausible. It seems to fit what we are seeing in the strace and the symtoms. | 03:56 |
| *** jamespage has quit IRC | 04:00 | |
| *** gnuoy has quit IRC | 04:00 | |
| *** gnuoy has joined #openstack-infra | 04:00 | |
| *** jamespage has joined #openstack-infra | 04:00 | |
| *** david-lyle has joined #openstack-infra | 04:01 | |
| mgagne | mmaglana: afaik, you can't precisely select the ruby version. rvm/rbenv aren't installed on slaves. You can however (at least) use ruby1.9 to run your tests. | 04:08 |
| mgagne | mmaglana: example of a job/macro using ruby1.9: https://github.com/openstack-infra/config/blob/master/modules/openstack_project/files/jenkins_job_builder/config/macros.yaml#L152 | 04:08 |
| mmaglana | thanks mgagne. unfortunately, activesupport 3.2.8 is incompatible with ruby 1.9.1. it needs 1.9.3 or higher. i will try to downgrade activesupport though to see if it works. | 04:11 |
| mgagne | mmaglana: although the binary is named "ruby1.9.1", it is in fact a weird choice made by Debian. 1.9.1 represents the ABI version, no ruby itself. 1.9.3 could very well be installed on the slaves. | 04:11 |
| mmaglana | mgagne: oh! | 04:11 |
| mmaglana | interesting. | 04:11 |
| mgagne | mmaglana: indeed | 04:11 |
| *** afazekas has quit IRC | 04:12 | |
| mgagne | mmaglana: https://launchpad.net/ubuntu/+source/ruby1.9.1 | 04:12 |
| mmaglana | mgagne: nice. thanks for that! | 04:12 |
| mmaglana | slaves are running precise, right? | 04:13 |
| *** zhiyan_ is now known as zhiyan | 04:13 | |
| mgagne | mmaglana: by default, yes, not sure if they setup trusty slaves yet | 04:13 |
| mmaglana | mgagne: as long as it's not Lucid, I'm a happy camper! :) | 04:13 |
| mgagne | mmaglana: they usually aim for LTS releases | 04:14 |
| mmaglana | mgagne: cool. thanks a lot for your help! | 04:14 |
| mgagne | mmaglana: yw, can't wait for aviator to be properly setup at stackforge =) | 04:14 |
| *** matsuhashi has quit IRC | 04:15 | |
| mgagne | mmaglana: no trusty in view: https://github.com/openstack-infra/config/blob/master/modules/openstack_project/templates/nodepool/nodepool.yaml.erb | 04:15 |
| mmaglana | mgagne: same here! :) | 04:15 |
| *** yfried has quit IRC | 04:16 | |
| *** reed has quit IRC | 04:23 | |
| *** marcoemorais has joined #openstack-infra | 04:25 | |
| *** matsuhashi has joined #openstack-infra | 04:26 | |
| *** nosnos has joined #openstack-infra | 04:27 | |
| *** praneshp has joined #openstack-infra | 04:27 | |
| *** praneshp_ has joined #openstack-infra | 04:29 | |
| *** praneshp has quit IRC | 04:32 | |
| *** praneshp_ is now known as praneshp | 04:32 | |
| *** marcoemorais1 has joined #openstack-infra | 04:32 | |
| *** marcoemorais has quit IRC | 04:34 | |
| *** otherwiseguy has joined #openstack-infra | 04:35 | |
| *** lcheng_ has joined #openstack-infra | 04:36 | |
| *** afazekas has joined #openstack-infra | 04:40 | |
| *** wenlock_ has joined #openstack-infra | 04:44 | |
| openstackgerrit | Valeriy Ponomaryov proposed a change to openstack-infra/config: Enabled pylint job for manila project https://review.openstack.org/95765 | 04:49 |
| *** thomasbiege has joined #openstack-infra | 04:51 | |
| *** yfried has joined #openstack-infra | 04:51 | |
| *** yfried has quit IRC | 04:53 | |
| *** zhiyan is now known as zhiyan_ | 04:53 | |
| *** wenlock_ has quit IRC | 04:56 | |
| *** david-lyle has quit IRC | 04:57 | |
| *** david-lyle has joined #openstack-infra | 04:57 | |
| *** david-lyle has quit IRC | 05:02 | |
| *** wenlock_ has joined #openstack-infra | 05:06 | |
| openstackgerrit | A change was merged to openstack-infra/pypi-mirror: Add a pointer to the DIB pypi element's README https://review.openstack.org/86746 | 05:06 |
| *** wenlock has quit IRC | 05:06 | |
| openstackgerrit | Mark Maglana proposed a change to openstack-infra/config: Define Jenkins Job for Aviator https://review.openstack.org/96056 | 05:06 |
| *** thomasbiege has quit IRC | 05:09 | |
| *** thomasbiege has joined #openstack-infra | 05:12 | |
| *** zhiyan_ is now known as zhiyan | 05:14 | |
| openstackgerrit | Mark Maglana proposed a change to openstack-infra/config: Define Jenkins Job for Aviator https://review.openstack.org/96056 | 05:15 |
| *** krtaylor has quit IRC | 05:15 | |
| *** thomasbiege has quit IRC | 05:17 | |
| *** W00dy__ has quit IRC | 05:18 | |
| *** otherwiseguy has quit IRC | 05:21 | |
| *** Longgeek has joined #openstack-infra | 05:23 | |
| *** zhiyan is now known as zhiyan_ | 05:25 | |
| *** alexpilotti has quit IRC | 05:26 | |
| *** arnaud has quit IRC | 05:32 | |
| *** pcrews has joined #openstack-infra | 05:34 | |
| *** wenlock has joined #openstack-infra | 05:35 | |
| *** ildikov has quit IRC | 05:43 | |
| *** renlt has joined #openstack-infra | 05:43 | |
| *** starmer has joined #openstack-infra | 05:46 | |
| *** rlandy has joined #openstack-infra | 05:47 | |
| *** rcarrillocruz has joined #openstack-infra | 05:49 | |
| *** rcarrill` has quit IRC | 05:51 | |
| *** markmcclain has joined #openstack-infra | 05:52 | |
| *** resker has joined #openstack-infra | 05:54 | |
| *** talluri has joined #openstack-infra | 05:54 | |
| *** markmcclain1 has joined #openstack-infra | 05:54 | |
| *** alexpilotti has joined #openstack-infra | 05:54 | |
| *** wenlock has quit IRC | 05:55 | |
| *** markmcclain has quit IRC | 05:57 | |
| renlt | Hi, I'm now maintaining a project under stackforge, I want to create a new stable branch, but it seems that it does not work as a normal github repository does, which just to type a branch name to create..Would anyone shed some light on this, please? Thanks in advance! | 05:57 |
| *** esker has quit IRC | 05:57 | |
| *** alexpilotti has quit IRC | 05:59 | |
| *** markmcclain1 has quit IRC | 06:01 | |
| *** andreykurilin has joined #openstack-infra | 06:05 | |
| *** _nadya_ has joined #openstack-infra | 06:06 | |
| *** alexpilotti has joined #openstack-infra | 06:08 | |
| *** ildikov has joined #openstack-infra | 06:13 | |
| *** dkehn has quit IRC | 06:20 | |
| *** dkehn has joined #openstack-infra | 06:22 | |
| *** _nadya_ has quit IRC | 06:28 | |
| *** _nadya_ has joined #openstack-infra | 06:30 | |
| *** _nadya_ has quit IRC | 06:38 | |
| openstackgerrit | A change was merged to openstack-infra/config: Fix rally-cli tests https://review.openstack.org/95597 | 06:39 |
| rcarrillocruz | rcarrillocruz | 06:42 |
| rcarrillocruz | oops, wrong window | 06:42 |
| *** e0ne has joined #openstack-infra | 06:44 | |
| *** boris-42 has quit IRC | 06:45 | |
| *** wenlock_ has quit IRC | 06:46 | |
| *** doude has joined #openstack-infra | 06:48 | |
| *** e0ne has quit IRC | 06:48 | |
| *** boris-42 has joined #openstack-infra | 06:48 | |
| *** yfried has joined #openstack-infra | 06:48 | |
| *** e0ne has joined #openstack-infra | 06:49 | |
| *** e0ne has quit IRC | 06:49 | |
| *** yfried has quit IRC | 06:49 | |
| *** yfried has joined #openstack-infra | 06:49 | |
| *** e0ne has joined #openstack-infra | 06:50 | |
| *** rdopiera has joined #openstack-infra | 06:50 | |
| *** yfried_ has joined #openstack-infra | 06:50 | |
| *** renlt has quit IRC | 06:51 | |
| *** e0ne has quit IRC | 06:51 | |
| *** e0ne has joined #openstack-infra | 06:51 | |
| *** lcheng_ has quit IRC | 06:52 | |
| *** yfried has quit IRC | 06:54 | |
| *** e0ne has quit IRC | 06:56 | |
| *** yfried_ is now known as yfried | 06:57 | |
| *** rcarrill` has joined #openstack-infra | 06:59 | |
| *** praneshp has quit IRC | 06:59 | |
| *** rcarrillocruz has quit IRC | 07:01 | |
| *** skolekonov has joined #openstack-infra | 07:02 | |
| *** e0ne has joined #openstack-infra | 07:05 | |
| *** _nadya_ has joined #openstack-infra | 07:07 | |
| *** derekh_ has joined #openstack-infra | 07:08 | |
| *** jhesketh has quit IRC | 07:10 | |
| *** e0ne has quit IRC | 07:10 | |
| *** flaper87|afk is now known as flaper87 | 07:10 | |
| *** e0ne has joined #openstack-infra | 07:11 | |
| *** e0ne has quit IRC | 07:13 | |
| *** pblaho has joined #openstack-infra | 07:13 | |
| *** rcarrill` is now known as rcarrillocruz | 07:15 | |
| openstackgerrit | Sergey Skripnick proposed a change to openstack-infra/config: Make rally-install jobs voting https://review.openstack.org/96087 | 07:16 |
| *** zhiyan_ is now known as zhiyan | 07:18 | |
| *** e0ne has joined #openstack-infra | 07:20 | |
| *** jlibosva has joined #openstack-infra | 07:25 | |
| *** _nadya_ has quit IRC | 07:26 | |
| *** zhiyan is now known as zhiyan_ | 07:27 | |
| *** jcoufal has joined #openstack-infra | 07:28 | |
| *** e0ne has quit IRC | 07:36 | |
| *** e0ne has joined #openstack-infra | 07:36 | |
| *** e0ne has quit IRC | 07:39 | |
| mattoliverau | Night all, have a great day/night | 07:39 |
| *** mrda is now known as mrda-away | 07:43 | |
| openstackgerrit | Spencer Krum proposed a change to openstack-infra/config: Logstash: Modifying rewrite rules to allow kibana 3 https://review.openstack.org/93167 | 07:45 |
| nibalizer | jesusaurus: clarkb at long last you have your one character change -_- (sorry, ive been busy doing stuff) | 07:46 |
| openstackgerrit | Spencer Krum proposed a change to openstack-infra/config: Logstash: Modifying rewrite rules to allow kibana 3 https://review.openstack.org/93167 | 07:47 |
| *** ihrachyshka has joined #openstack-infra | 07:51 | |
| *** Hal_ has joined #openstack-infra | 07:52 | |
| *** ihrachyshka has quit IRC | 07:53 | |
| *** yfried has quit IRC | 07:53 | |
| *** markmcclain has joined #openstack-infra | 07:55 | |
| *** jgallard has joined #openstack-infra | 07:55 | |
| *** ihrachyshka has joined #openstack-infra | 07:56 | |
| *** andreykurilin has quit IRC | 07:56 | |
| openstackgerrit | afazekas proposed a change to openstack-infra/elastic-recheck: Add query for 1320617 https://review.openstack.org/94080 | 07:58 |
| *** jpich has joined #openstack-infra | 07:58 | |
| openstackgerrit | Stephan Renatus proposed a change to openstack-infra/config: Use strainer for chef checks when possible https://review.openstack.org/95471 | 07:58 |
| *** markmcclain has quit IRC | 08:00 | |
| openstackgerrit | Christian Berendt proposed a change to openstack-infra/config: cleaning up index.rst file https://review.openstack.org/96102 | 08:04 |
| *** yfried has joined #openstack-infra | 08:04 | |
| *** Hal_ has quit IRC | 08:07 | |
| *** Hal_ has joined #openstack-infra | 08:08 | |
| *** derekh_ has quit IRC | 08:09 | |
| *** yfried has quit IRC | 08:09 | |
| *** derekh_ has joined #openstack-infra | 08:11 | |
| *** habib has joined #openstack-infra | 08:12 | |
| *** matsuhashi has quit IRC | 08:14 | |
| *** matsuhashi has joined #openstack-infra | 08:15 | |
| *** matsuhashi has quit IRC | 08:17 | |
| *** matsuhashi has joined #openstack-infra | 08:17 | |
| *** andreaf has joined #openstack-infra | 08:18 | |
| *** doude has quit IRC | 08:20 | |
| *** yfried has joined #openstack-infra | 08:22 | |
| *** doude has joined #openstack-infra | 08:26 | |
| *** IvanBerezovskiy has joined #openstack-infra | 08:27 | |
| *** mmaglana has quit IRC | 08:27 | |
| openstackgerrit | Nikita Konovalov proposed a change to openstack-infra/storyboard: Project Groups API https://review.openstack.org/90736 | 08:30 |
| *** amotoki has joined #openstack-infra | 08:30 | |
| *** habib has quit IRC | 08:35 | |
| *** habib has joined #openstack-infra | 08:39 | |
| *** markmcclain has joined #openstack-infra | 08:50 | |
| *** isviridov|away is now known as isviridov | 08:53 | |
| *** doude has quit IRC | 08:56 | |
| *** e0ne has joined #openstack-infra | 08:58 | |
| *** ominakov has joined #openstack-infra | 08:59 | |
| *** e0ne_ has joined #openstack-infra | 09:03 | |
| *** pelix has joined #openstack-infra | 09:03 | |
| *** ildikov_ has joined #openstack-infra | 09:03 | |
| *** ildikov has quit IRC | 09:04 | |
| *** e0ne has quit IRC | 09:06 | |
| *** marcoemorais1 has quit IRC | 09:07 | |
| *** jp_at_hp has joined #openstack-infra | 09:12 | |
| *** melwitt has quit IRC | 09:18 | |
| *** zhiyan_ is now known as zhiyan | 09:19 | |
| *** rlandy has quit IRC | 09:21 | |
| *** yamahata has quit IRC | 09:22 | |
| *** thomasbiege has joined #openstack-infra | 09:24 | |
| *** markmcclain has quit IRC | 09:24 | |
| *** zhiyan is now known as zhiyan_ | 09:28 | |
| *** _nadya_ has joined #openstack-infra | 09:32 | |
| *** Hal_ has quit IRC | 09:33 | |
| *** zhiyan_ is now known as zhiyan | 09:37 | |
| *** zzelle has joined #openstack-infra | 09:37 | |
| *** thomasbiege has quit IRC | 09:37 | |
| *** Alexei_987 has joined #openstack-infra | 09:39 | |
| *** Hal_ has joined #openstack-infra | 09:41 | |
| *** Ajaeger has joined #openstack-infra | 09:43 | |
| *** unicell has quit IRC | 09:45 | |
| *** zhiyan is now known as zhiyan_ | 09:46 | |
| *** unicell has joined #openstack-infra | 09:47 | |
| *** Hal_ has quit IRC | 09:47 | |
| *** habib has joined #openstack-infra | 09:47 | |
| *** jooools has joined #openstack-infra | 09:48 | |
| *** boris-42 has quit IRC | 09:52 | |
| *** habib has quit IRC | 09:53 | |
| *** habib has joined #openstack-infra | 09:53 | |
| *** markmcclain has joined #openstack-infra | 09:54 | |
| *** thomasbiege has joined #openstack-infra | 09:55 | |
| *** habib has quit IRC | 09:56 | |
| *** habib has joined #openstack-infra | 09:56 | |
| *** thomasbiege has quit IRC | 09:57 | |
| *** maxbit has quit IRC | 10:02 | |
| *** unicell has quit IRC | 10:04 | |
| openstackgerrit | Daniel Berrange proposed a change to openstack-infra/git-review: Fix typo in manpage s/gireview/gitreview/ https://review.openstack.org/96125 | 10:05 |
| openstackgerrit | A change was merged to openstack-infra/git-review: Correct git review -l over http(s) https://review.openstack.org/95483 | 10:10 |
| *** boris-42 has joined #openstack-infra | 10:15 | |
| *** ihrachyshka has quit IRC | 10:18 | |
| openstackgerrit | yolanda.robla proposed a change to openstack-infra/storyboard-webclient: Fixes misaligned comments on very long lines Using <span> container also for main text, and switching orders of it Closes-Bug: #93 https://review.openstack.org/96128 | 10:19 |
| *** e0ne_ has quit IRC | 10:21 | |
| *** e0ne has joined #openstack-infra | 10:22 | |
| openstackgerrit | yolanda.robla proposed a change to openstack-infra/storyboard-webclient: Fixes misaligned comments on very long lines Using <span> container also for main text, and switching orders of it Closes-Bug: #93 https://review.openstack.org/96128 | 10:24 |
| *** e0ne has quit IRC | 10:26 | |
| *** wenlock has joined #openstack-infra | 10:31 | |
| *** jgallard has quit IRC | 10:33 | |
| openstackgerrit | Andreas Jaeger proposed a change to openstack-infra/config: Fix django_openstack_auth translation setup https://review.openstack.org/96132 | 10:38 |
| *** wenlock has quit IRC | 10:40 | |
| *** ihrachyshka has joined #openstack-infra | 10:41 | |
| *** thomasbiege has joined #openstack-infra | 10:41 | |
| openstackgerrit | mark mcclain proposed a change to openstack-infra/reviewstats: Update list to reflect recent changes to core team https://review.openstack.org/96133 | 10:43 |
| *** matsuhashi has quit IRC | 10:48 | |
| *** thomasbiege has quit IRC | 10:49 | |
| *** markmcclain has quit IRC | 10:50 | |
| openstackgerrit | yolanda.robla proposed a change to openstack-infra/config: Checks if requirements overlap between head and branch reqs https://review.openstack.org/84100 | 10:51 |
| *** markmc has joined #openstack-infra | 10:56 | |
| *** e0ne has joined #openstack-infra | 10:59 | |
| *** ildikov has joined #openstack-infra | 11:00 | |
| *** ildikov_ has quit IRC | 11:01 | |
| openstackgerrit | Andreas Jaeger proposed a change to openstack-infra/config: propose_translation_update: Handle .tx/config in git https://review.openstack.org/82384 | 11:02 |
| *** dizquierdo has joined #openstack-infra | 11:02 | |
| *** e0ne has quit IRC | 11:04 | |
| *** _nadya_ has quit IRC | 11:07 | |
| *** lcostantino has joined #openstack-infra | 11:08 | |
| *** _nadya_ has joined #openstack-infra | 11:12 | |
| *** saschpe has left #openstack-infra | 11:20 | |
| *** thomasbiege has joined #openstack-infra | 11:22 | |
| *** andreykurilin has joined #openstack-infra | 11:28 | |
| openstackgerrit | Andreas Jaeger proposed a change to openstack-infra/config: Remove duplication in translation scripts https://review.openstack.org/96142 | 11:32 |
| *** e0ne has joined #openstack-infra | 11:32 | |
| *** habib has quit IRC | 11:32 | |
| YorikSar | zzelle: Hello. Around? | 11:36 |
| zzelle | YorikSar, yes | 11:36 |
| YorikSar | zzelle: I've found out that the only external dependency (except argparse) in git-review is requests added by you. | 11:37 |
| YorikSar | zzelle: Is it really needed? | 11:37 |
| Ajaeger | The requirements proposal job currently fails, fixed via https://review.openstack.org/#/c/94722/ - any requirements core around to approve this, please? | 11:37 |
| zzelle | YorikSar, requests is used for http queries, hooks download | 11:38 |
| YorikSar | zzelle: It's just that the argument "git-review should have minimal set dependencies" is kind of wrong already... | 11:38 |
| zzelle | YorikSar, because urllib (or urllib2) does not check https certificates | 11:38 |
| *** andreykurilin_ has joined #openstack-infra | 11:39 | |
| YorikSar | zzelle: Ah that's why it's there. | 11:39 |
| *** andreykurilin has quit IRC | 11:39 | |
| zzelle | YorikSar, i wasn't sure, i remember it was a trouble for fungi or clarkb to introduce requests | 11:39 |
| anteaya | Ajaeger: https://review.openstack.org/#/admin/groups/131,members | 11:40 |
| zzelle | the security reason was more important than the miminal dependencies set | 11:40 |
| Ajaeger | anteaya: ok, will add to it ;) | 11:41 |
| anteaya | GheRivero might be up | 11:41 |
| Ajaeger | just fixing another fallout... | 11:41 |
| anteaya | just showing you who the cores are | 11:41 |
| Ajaeger | anteaya: thanks! | 11:41 |
| anteaya | markmc: might be around as well | 11:41 |
| YorikSar | zzelle: Yeah... I wonder if switching to using six is not that bad then. | 11:42 |
| zzelle | YorikSar, if we switch to six, we should target to switch everything | 11:43 |
| GheRivero | hi anteaya | 11:43 |
| openstackgerrit | Andreas Jaeger proposed a change to openstack/requirements: Rename climate -> blazar https://review.openstack.org/96144 | 11:43 |
| zzelle | YorikSar, that's why i am not really "fan" of the proposed change | 11:43 |
| YorikSar | zzelle: Sure. I wonder if Christian is up to it. | 11:43 |
| Ajaeger | Alex_Gaynor, GheRivero, dhellmann, markmc, sdague: https://review.openstack.org/96144 and https://review.openstack.org/#/c/94722/ are needed to unbreak the requirements proposal job, could you review, please? | 11:44 |
| GheRivero | Ajaeger: on my way | 11:44 |
| Ajaeger | Thanks, GheRivero ! | 11:44 |
| anteaya | hey GheRivero, thanks | 11:45 |
| YorikSar | zzelle: By the way. I wonder why there was no release of git-review for a long time. | 11:45 |
| sdague | Ajaeger: +2 | 11:45 |
| *** e0ne_ has joined #openstack-infra | 11:45 | |
| *** dims has joined #openstack-infra | 11:45 | |
| zzelle | YorikSar, if the change solves a bug, it's fine, i am ok with the change base. if it is cosmetic i prefer we avoid to add a new requirement | 11:46 |
| YorikSar | zzelle: It looks like there's a lot of sweet changes waiting in master but never used by broad audience. | 11:46 |
| Ajaeger | Thanks a lot everybody! | 11:46 |
| zzelle | YorikSar, fungi is assigned a bug on releasing git-review | 11:46 |
| YorikSar | zzelle: My guess is that it's just a cosmetic change. | 11:47 |
| *** _nadya_ has quit IRC | 11:47 | |
| zzelle | YorikSar, which is a good idea with the upgrade of gerrit | 11:47 |
| *** dstanek_zzz is now known as dstanek | 11:47 | |
| Ajaeger | wow, seldom had a patch that quickly in - thanks, GheRivero, sdague, markmc! | 11:47 |
| anteaya | SergeyLukjanov: so this patch merged: https://review.openstack.org/#/c/94896/ | 11:49 |
| YorikSar | zzelle: Oh, I didn't know there's a bug on this already. | 11:49 |
| anteaya | any objection to me submitting a patch to sync openstack-infra with the governance/programs.yaml? | 11:49 |
| anteaya | or did you want to do that? | 11:49 |
| zzelle | YorikSar, https://bugs.launchpad.net/git-review/+bug/1322365 | 11:49 |
| *** e0ne has quit IRC | 11:49 | |
| uvirtbot | Launchpad bug 1322365 in git-review "git-review 1.24 (or newer) needs to be released" [High,In progress] | 11:49 |
| zzelle | YorikSar, perhaps we should wait for the release before switching to six ? | 11:50 |
| *** miqui has quit IRC | 11:51 | |
| YorikSar | zzelle: As long as it doesn't fix anything, yes. | 11:51 |
| *** thomasbiege has quit IRC | 11:51 | |
| YorikSar | zzelle: Otherwise I think a new dependency should be introduced as early as possible. | 11:52 |
| *** _nadya_ has joined #openstack-infra | 11:52 | |
| zzelle | YorikSar, exactly | 11:52 |
| zzelle | YorikSar, i will ask Christian if the change is cosmetic and after discuss with fungi the release target | 11:53 |
| YorikSar | zzelle: We have another change waiting for new patchset: https://review.openstack.org/#/c/76197 | 11:54 |
| YorikSar | zzelle: btw, Christian's change somehow didn't make it to Zuul. | 11:55 |
| YorikSar | zzelle: There's no Jenkins comments at all. | 11:55 |
| zzelle | YorikSar, it seems to be linked to zuul troubles (yesterday) | 11:56 |
| *** _nadya_ has quit IRC | 11:56 | |
| *** rlandy has joined #openstack-infra | 11:57 | |
| zzelle | YorikSar, https://review.openstack.org/76197 could be interesting to add to the release ? | 11:57 |
| zzelle | YorikSar, i am also asking myself if we should provide support for WIP before release ? | 11:58 |
| YorikSar | zzelle: I guess we can either wait for both these changes to land or just release more often :) | 11:59 |
| YorikSar | zzelle: I still don't understand why there was no releases for 9 months. | 12:00 |
| zzelle | YorikSar, don't know ... perhaps for normal git-review usage over ssh git-review 1.23 is enough | 12:01 |
| *** markmcclain has joined #openstack-infra | 12:02 | |
| YorikSar | zzelle: Well... There were some bugs like '(detached' topic. | 12:02 |
| YorikSar | zzelle: But it looks like they were fixed only recently. | 12:03 |
| zzelle | YorikSar, indeed | 12:03 |
| *** markmcclain1 has joined #openstack-infra | 12:04 | |
| *** _nadya_ has joined #openstack-infra | 12:04 | |
| vponomaryov | Guys from infra core, could you please review commit: https://review.openstack.org/#/c/95207/ | 12:04 |
| *** markmcclain1 has quit IRC | 12:04 | |
| zzelle | YorikSar, i should leave, we can discuss with fungi tomorrow afternoon ? | 12:04 |
| YorikSar | zzelle: Sure | 12:04 |
| *** markmcclain1 has joined #openstack-infra | 12:04 | |
| YorikSar | zzelle: I think I'll be still around. | 12:04 |
| zzelle | YorikSar, bye | 12:04 |
| *** habib has joined #openstack-infra | 12:06 | |
| *** markmcclain has quit IRC | 12:06 | |
| *** yaguang has quit IRC | 12:09 | |
| *** dprince has joined #openstack-infra | 12:10 | |
| ildikov | anteaya: clarkb: are you around? | 12:11 |
| *** habib has quit IRC | 12:12 | |
| *** habib has joined #openstack-infra | 12:13 | |
| anteaya | ildikov: I am | 12:13 |
| anteaya | too early for clarkb yet | 12:13 |
| anteaya | how are you making out? | 12:13 |
| ildikov | anteaya: hi | 12:13 |
| anteaya | hi | 12:13 |
| ildikov | anteaya: ah, ok, I'm still not 100% aware of the timezones :) | 12:13 |
| anteaya | it is okay | 12:14 |
| anteaya | once you learn everyone's they travel somewhere | 12:14 |
| anteaya | you never win on the timezone question | 12:14 |
| ildikov | anteaya: so it seems that there is no problem with tox, I found the log with the Error message yesterday for the patch I showed | 12:14 |
| anteaya | great | 12:14 |
| ildikov | anteaya: yeap, that's true :) | 12:14 |
| anteaya | have you identified where the issue lies? | 12:14 |
| ildikov | anteaya: in this log file: http://logs.openstack.org/65/92365/6/check/gate-ceilometer-docs/cc989fb/console.html.gz | 12:15 |
| ildikov | anteaya: sorry: it's better: http://logs.openstack.org/65/92365/6/check/gate-ceilometer-docs/cc989fb/console.html.gz#_2014-05-16_15_08_02_834 | 12:15 |
| ildikov | anteaya: so in the linked lines you can see some error messages | 12:15 |
| ildikov | anteaya: any time I saw these errors, the build was always finished in the way that Sphinx generates all the doc it can | 12:16 |
| anteaya | yes I can see some error messages | 12:16 |
| ildikov | anteaya: and leaves out those, that have formatting issues | 12:16 |
| anteaya | interesting | 12:16 |
| anteaya | mostly for me that the log has error messages but the build finishes success | 12:16 |
| ildikov | anteaya: so the doc is generated, but some parts are missing, for instance here the Cinder table, as I showed yesterday | 12:17 |
| anteaya | right | 12:17 |
| ildikov | anteaya: yes, so I do not know if I can analyze the logs in that gate job or what could be the best solution for this issue | 12:17 |
| anteaya | okay so at a certain point there was action taken about logs | 12:18 |
| anteaya | since every project has a different interpretation about what WARNING means and so on | 12:18 |
| ildikov | anteaya: I didn't find a stop in case of errors config option, maybe it would worth some more time, but if there is no option like this, is there any other way to identify a partially failed build? | 12:18 |
| *** aysyd has joined #openstack-infra | 12:18 | |
| *** pdmars has joined #openstack-infra | 12:18 | |
| anteaya | some projects were setting everything to a certain level and the logs were getting way to big | 12:18 |
| *** pblaho has quit IRC | 12:18 | |
| anteaya | I wonder if the setting for the log has been adjusted and needs to be evaluated | 12:19 |
| *** krtaylor has joined #openstack-infra | 12:19 | |
| *** pdmars has quit IRC | 12:19 | |
| anteaya | ildikov I would suggest asking sdague to help you understand the issue with logging levels in the hopes that will help you understand how logging is handled for ceilometer | 12:19 |
| *** mbacchi has joined #openstack-infra | 12:20 | |
| ildikov | anteaya: but it's Sphinx log and not Ceilometer | 12:20 |
| *** thomasbiege has joined #openstack-infra | 12:20 | |
| *** julim has joined #openstack-infra | 12:21 | |
| ildikov | anteaya: I mean I cannot modify the log levels easily and also, in the run-docs.sh script, the doc build script is ran and then the result of it is success in most of the cases and then the run-docs.sh deals with the generated docs, even if it is generated only partially | 12:21 |
| anteaya | well I would have to dig into sphinx logging then since I know nothing about it | 12:22 |
| anteaya | I need to eat something first | 12:22 |
| ildikov | anteaya: I guess the made it work this way because of a reason, I mean that the build does not fail in case of a formatting error | 12:23 |
| *** weshay has joined #openstack-infra | 12:23 | |
| ildikov | anteaya: *they* made | 12:23 |
| *** smarcet has joined #openstack-infra | 12:24 | |
| ildikov | anteaya: sure, but if sdague knows more about this maybe, then I will annoy him a bit :) | 12:25 |
| ildikov | anteaya: do you know when he will be available? | 12:25 |
| openstackgerrit | A change was merged to openstack-dev/hacking: Updated from global requirements https://review.openstack.org/95608 | 12:25 |
| *** jgallard has joined #openstack-infra | 12:29 | |
| openstackgerrit | yolanda.robla proposed a change to openstack-infra/storyboard: Added new priority changed event for tasks Defined new constants and created new methods to generate event https://review.openstack.org/96151 | 12:31 |
| *** pdmars has joined #openstack-infra | 12:32 | |
| *** ArxCruz has joined #openstack-infra | 12:32 | |
| anteaya | ildikov: he is online now, he is probably busy with something else atm | 12:35 |
| anteaya | yeah, so my question is why does the build succeed when error messages are logged | 12:35 |
| anteaya | I am thinking that is a quality of one of our jobs, not necessarily sphinx, but I don't know | 12:36 |
| fungi | zzelle: YorikSar: i'm going to try and get git-review polished up for a 1.24 release this week | 12:36 |
| *** markmcclain1 has quit IRC | 12:36 | |
| anteaya | ildikov: for instance I am going to start in here: http://git.openstack.org/cgit/openstack-infra/jenkins-job-builder/tree/jenkins_jobs/sphinx/yaml.py | 12:37 |
| ildikov | anteaya: if you run the sphinx-build command it succeeds in case of this formatting error | 12:38 |
| *** dims has quit IRC | 12:39 | |
| ildikov | anteaya: so it is nothing to do with the gate job, I mean it receives the 'build succeeded' verdict from the sphinx command | 12:39 |
| anteaya | okay so then the issue may reside with sphinx-build logging | 12:39 |
| *** eharney has joined #openstack-infra | 12:39 | |
| anteaya | have you chatted with any of the sphinx devs at all? | 12:40 |
| SergeyLukjanov | anteaya, hey, mornging | 12:40 |
| *** andreaf has quit IRC | 12:41 | |
| anteaya | SergeyLukjanov: morning/afternoon | 12:41 |
| anteaya | SergeyLukjanov: what do you want to do about sync'ing infra with governance/programs.yaml? | 12:41 |
| ildikov | anteaya: not yet, that's the next step on the list, if analyzing of the logs in the gate job is not an option | 12:41 |
| anteaya | ildikov: I think you have demonstrated to me that this issue resides with sphinx | 12:41 |
| anteaya | the logs in the gate job are accessible to you with every test run | 12:42 |
| anteaya | you get all we have | 12:42 |
| fungi | it could reside within pbr. the build_sphinx target is provided by pbr, which then in turn calls out to the sphinx tools | 12:42 |
| ildikov | fungi: hi | 12:42 |
| anteaya | I didn't know pbr might have an effect on how build_sphinx interprets errors in the build log | 12:43 |
| openstackgerrit | yolanda.robla proposed a change to openstack-infra/storyboard: Added new priority changed event for tasks https://review.openstack.org/96151 | 12:43 |
| anteaya | would pbr still be playing a role if ildikov gets the same result by running build-sphinx? | 12:43 |
| fungi | ildikov: hi. i'm looking at pbr right now to see whether maybe it's had some recent changes to its sphinx hooks... though i'm not particularly familiar with that part of the code so it's unlikely i'll find an obvious cause | 12:44 |
| ildikov | fungi: you mean that this issue could be solved in pbr maybe? | 12:44 |
| *** thomasbiege has quit IRC | 12:44 | |
| fungi | anteaya: well, the result is that whatever's invoking the sphinx build is existing with a o return code afterward, which tox interprets as a success and so the job passes | 12:44 |
| fungi | er, a 0 return code | 12:45 |
| *** talluri has quit IRC | 12:45 | |
| *** W00dy__ has joined #openstack-infra | 12:45 | |
| jd__ | fungi: does an import problem with lockfile.linklockfile on the py33 nodes rings a bell to you? | 12:46 |
| *** lcostantino has quit IRC | 12:46 | |
| jd__ | something like that https://jenkins04.openstack.org/job/gate-gnocchi-python33/3/console | 12:46 |
| fungi | jd__: i asked a search engine friend of mine, and he responded with http://code.google.com/p/pylockfile/issues/detail?id=23 | 12:49 |
| jd__ | fungi: uh, I'm lost as the developer says it's fixed in 0.9.1 which is what is installed | 12:50 |
| *** nosnos has quit IRC | 12:50 | |
| SergeyLukjanov | anteaya, I'll sync infra projects today to gov | 12:50 |
| ildikov | fungi: hmm, I do not know how can Sphinx configured in that way to return something else in case of an error, if even it exists | 12:50 |
| anteaya | SergeyLukjanov: okay | 12:50 |
| anteaya | anyone have anotherjesse's email address | 12:51 |
| SergeyLukjanov | anteaya, /me was thinking about automating it, but it's not always could be determind | 12:51 |
| fungi | jd__: yeah, i too find that confusing... could be we're looking at a regression somewhere | 12:51 |
| anteaya | SergeyLukjanov: yeah, going to be tough to get the regex right | 12:51 |
| ildikov | fungi: is there a chance maybe to analyze the logs and searching for error and mark the gate job as 'Failed' in this case | 12:51 |
| ildikov | fungi: ? | 12:52 |
| *** prad has joined #openstack-infra | 12:52 | |
| jd__ | fungi: oh it's something more pervert I think | 12:52 |
| fungi | ildikov: it's supposed to fail build_sphinx on errors, i believe (and we also do something to make sure it catches warnings too) | 12:52 |
| StevenK | I'm guessing that docs.openstack.org/developer/ is populated by openstack-manuals, but I can't work out which bit of openstack-manuals does it. | 12:52 |
| *** miqui has joined #openstack-infra | 12:53 | |
| fungi | StevenK: it's api jobs for the individual projects, not the manuals | 12:53 |
| StevenK | fungi: Ah ha, I shall paw through -infra/config then | 12:54 |
| *** zehicle_at_dell has quit IRC | 12:54 | |
| fungi | StevenK: yeah, you should see plenty of api jobs | 12:54 |
| ildikov | fungi: I use the pure sphinx-build command locally, when I update some docs and it succeedes, even if there are some formatting errors in the logs | 12:54 |
| ildikov | fungi: or did you mean pbr or some other parts of the gate job, that should fail? | 12:55 |
| fungi | ildikov: it's also possible this is due to the sphinx configuration in ceilometer | 12:55 |
| fungi | ildikov: the process should return nonzero on error | 12:55 |
| jd__ | fungi: it's simpler to explain on a paste so here's my current thoughts http://paste.openstack.org/show/81871/ | 12:55 |
| *** jcoufal has quit IRC | 12:56 | |
| fungi | ildikov: i would start by comparing http://git.openstack.org/cgit/openstack/ceilometer/tree/doc/source/conf.py to the same file in other projects to see if maybe there's something missing for that | 12:57 |
| *** jcoufal has joined #openstack-infra | 12:57 | |
| StevenK | fungi: Strangely, not one for ironic | 12:58 |
| fungi | StevenK: "baremetal" maybe? | 12:58 |
| anteaya | anotherjesse doesn't have an email in launchpad, hmmmm | 12:58 |
| StevenK | fungi: No dice for that either | 12:58 |
| ildikov | fungi: ok, I will give it a try, tnx :) | 12:59 |
| anteaya | StevenK: maybe ask in the ironic channel? | 12:59 |
| anteaya | StevenK: they might know where their docs are | 12:59 |
| *** gibi has joined #openstack-infra | 12:59 | |
| StevenK | I know where they are, I'm trying to work out how they get to docs.openstack.org/developer/ | 13:00 |
| *** skolekonov has quit IRC | 13:00 | |
| anteaya | look at the post jobs in the zuul/layout.yaml file? | 13:01 |
| *** pafuent has joined #openstack-infra | 13:01 | |
| *** fifieldt has quit IRC | 13:02 | |
| *** _nadya_ has quit IRC | 13:02 | |
| fungi | StevenK: aha, there are separate docs projects for those... http://git.openstack.org/cgit/openstack/compute-api/tree/ | 13:02 |
| fungi | et cetera | 13:02 |
| *** rfolco has joined #openstack-infra | 13:03 | |
| StevenK | But not for ironic, but it seems to be done via the openstack-client-publish-jobs template | 13:03 |
| anteaya | StevenK: ironic doesn't have any post docs jobs: http://git.openstack.org/cgit/openstack-infra/config/tree/modules/openstack_project/files/zuul/layout.yaml#n878 | 13:03 |
| StevenK | anteaya: But python-ironicclient does | 13:04 |
| *** dkliban_afk is now known as dkliban | 13:04 | |
| *** pafuent has quit IRC | 13:05 | |
| *** andreykurilin_ has quit IRC | 13:05 | |
| fungi | StevenK: okay, so the project entry for openstack/ironic in zuul's layout includes the openstack-server-publish-jobs project template which has {name}-docs in the post pipeline, which ends up expanding to ironic-docs as a job name | 13:06 |
| *** dstanek is now known as dstanek_zzz | 13:07 | |
| fungi | StevenK: in the jjb config tree there's a openstack-publish-jobs.yaml which has the job template for {name}-docs | 13:09 |
| *** yfried has quit IRC | 13:10 | |
| fungi | StevenK: it in turn calls out to the "docs" builder in macros.yaml | 13:10 |
| *** mriedem has joined #openstack-infra | 13:10 | |
| fungi | StevenK: which runs '/usr/local/jenkins/slave_scripts/run-docs.sh {github-org} {project}' | 13:10 |
| StevenK | fungi: So where I'm really going with this is trying to get things like os-apply-config and friends up there, which doesn't fit into either -server or -client. However, it's >2300 and I need beauty sleep, so I shall dig more into all the crumbs you've fed me tomorrow. | 13:11 |
| fungi | StevenK: that should be enough to get you going after you sleep on it | 13:11 |
| openstackgerrit | yolanda.robla proposed a change to openstack-infra/storyboard-webclient: Added priority on task row https://review.openstack.org/96156 | 13:11 |
| *** bknudson has quit IRC | 13:11 | |
| fungi | jd__: i'm wondering if we have a system site packages negative interaction going on... looking to see if the tox.ini for gnocchi has any obvious clues | 13:12 |
| jd__ | fungi: I set sitepackages=False, but yeah, I'm surprised. I've sent a patch with a requirement specifying lockfile>=0.9.1 instead of just lockfile, to see | 13:12 |
| fungi | jd__: yeah, agreed. so it shouldn't be touching the system installed lockfile anyway | 13:13 |
| *** _nadya_ has joined #openstack-infra | 13:13 | |
| *** _nadya_ has quit IRC | 13:13 | |
| jd__ | yes, there's something weird or so big I miss it :( | 13:13 |
| fungi | jd__: i think i see the problem | 13:15 |
| *** mburned is now known as mburned_out | 13:16 | |
| fungi | -rw-rw-r-- 1 jenkins jenkins 16949 Mar 25 2013 /srv/static/pypi/openstack/lockfile/lockfile-0.9.1.tar.gz | 13:16 |
| fungi | 4e4c7ea4c4301500da5e7f3b51e01cfe /srv/static/pypi/openstack/lockfile/lockfile-0.9.1.tar.gz | 13:16 |
| jd__ | HAN HAN | 13:16 |
| fungi | jd__: then look at https://pypi.python.org/pypi/lockfile | 13:16 |
| jd__ | released on 23 Aug yeah | 13:17 |
| jd__ | do we have a time traveling machine? | 13:17 |
| fungi | seems the lockfile authors think it's perfectly okay to fix bugs and reuse the same version numbers when they rerelease | 13:17 |
| jd__ | sounds like that indeed :( | 13:17 |
| jd__ | sounds like we solved a mystery /nick scoobidoo | 13:18 |
| annegentle | ruh roh raggy | 13:18 |
| fungi | it was old man weathers underneath that monster mask all along | 13:18 |
| annegentle | fungi: lol | 13:18 |
| fungi | and he would have gotten away with it too if it weren't for us nosy developers | 13:18 |
| *** signed8bit has joined #openstack-infra | 13:19 | |
| *** jaypipes has quit IRC | 13:20 | |
| fungi | jd__: so anyway, the proper thing to do here is to "encourage" the lockfile authors to bump their release number, then adjust your requirements to insist on >= that new release | 13:22 |
| *** dstanek_zzz is now known as dstanek | 13:23 | |
| *** pafuent has joined #openstack-infra | 13:23 | |
| jd__ | fungi: I can do that, but can we reset the mirror in the meantime? | 13:23 |
| fungi | jd__: since there's really no telling who has already cached/shipped/baked in the previous 0.9.1, so just papering over it in our mirror is probably a bad idea | 13:23 |
| openstackgerrit | Radomir Dopieralski proposed a change to openstack-infra/config: Add gerritbot to the #tuskar channel https://review.openstack.org/96160 | 13:23 |
| *** jaypipes has joined #openstack-infra | 13:23 | |
| jd__ | fungi: fair enough | 13:23 |
| openstackgerrit | Anita Kuno proposed a change to openstack/requirements: Add barbican and python-barbicanclient to global-requirements https://review.openstack.org/96161 | 13:24 |
| fungi | jd__: better, i think, if we know that certain things are broken for people with the "old" 0.9.1 | 13:24 |
| fungi | which we won't know any longer if we replace it with the "new" 0.9.1 | 13:24 |
| *** signed8bit has quit IRC | 13:25 | |
| *** signed8bit has joined #openstack-infra | 13:25 | |
| *** rgerganov has joined #openstack-infra | 13:25 | |
| jd__ | fungi: FYI https://github.com/smontanaro/pylockfile/issues/11 | 13:25 |
| *** lcostantino has joined #openstack-infra | 13:26 | |
| fungi | jd__: aha! i was looking in their google code issue tracker linked from pypi, which is apparently old | 13:27 |
| jd__ | yeah they forgot to change the page from pypi… | 13:27 |
| jd__ | suckerz :) | 13:27 |
| *** W00die has quit IRC | 13:32 | |
| *** mfer has joined #openstack-infra | 13:32 | |
| *** bknudson has joined #openstack-infra | 13:34 | |
| *** prad_ has joined #openstack-infra | 13:34 | |
| anteaya | Ryan_Lane seems to have adjusted the timeout for wiki signin | 13:35 |
| anteaya | I have more than 5 minutes at a time now | 13:35 |
| *** prad has quit IRC | 13:37 | |
| *** prad_ is now known as prad | 13:37 | |
| *** alexpilotti has quit IRC | 13:39 | |
| *** homeless has joined #openstack-infra | 13:39 | |
| *** zz_gondoi is now known as gondoi | 13:40 | |
| *** gokrokve has joined #openstack-infra | 13:40 | |
| *** pafuent has quit IRC | 13:40 | |
| *** gokrokve has quit IRC | 13:40 | |
| *** gokrokve has joined #openstack-infra | 13:41 | |
| *** W00die has joined #openstack-infra | 13:42 | |
| fungi | ildikov: i've confirmed that the setup.py build_sphinx target provided by pbr does pass warningiserror=True into application.Sphinx() and then tries to build() and reraises any exceptions from that except for the docutils.utils.SystemMessage class | 13:44 |
| *** gabriel-bezerra has joined #openstack-infra | 13:44 | |
| gabriel-bezerra | Guys, doesn't check-grenade-dsvm update the entire devstack repository? | 13:45 |
| fungi | ildikov: but if it was from the ignored class there should have been a message starting with "reST markup error:" in the log (dumped to stdout by pbr) | 13:45 |
| ildikov | fungi: wow :) | 13:45 |
| fungi | er, to stderr | 13:45 |
| fungi | i'm starting to wonder if this job isn't using pbr for sphinx doc generation | 13:46 |
| fungi | except it must be, because ceilometer's setup.py doesn't have a build_sphinx target of its own | 13:47 |
| gabriel-bezerra | My patch changes the line 'enable_apache_site horizon.conf' to 'enable_apache_site horizon' but the tests fail with the old line in the logs | 13:47 |
| ildikov | fungi: yes, I've checked setup.py in Ceilo | 13:47 |
| fungi | gabriel-bezerra: it updates code but keeps old configuration | 13:47 |
| *** rgerganov has quit IRC | 13:48 | |
| fungi | gabriel-bezerra: so i think any attempt to modify the configuration may get ignored, if that's what you were trying to accomplish there (hard to know without additional context) | 13:48 |
| gabriel-bezerra | It also introduces a new function to devstack/lib/apache, and there is another breakage with "apache_site_config_for not found" | 13:48 |
| gabriel-bezerra | https://review.openstack.org/95977 | 13:48 |
| gabriel-bezerra | http://logs.openstack.org/77/95977/6/check/check-grenade-dsvm/670e5dc/console.html | 13:49 |
| ildikov | fungi: but I haven't seen "reST markup error" | 13:49 |
| gabriel-bezerra | search apache_site in that log | 13:49 |
| *** mburned_out is now known as mburned | 13:49 | |
| gabriel-bezerra | fungi: ^ | 13:49 |
| fungi | ildikov: agreed, so any exceptions from sphinx should have been reraised and that call should have returned a nonzero exit code | 13:49 |
| fungi | gabriel-bezerra: oh, a change *to* devstack. i think grenade should be starting with your proposed change in place in that case | 13:50 |
| gabriel-bezerra | exactly | 13:51 |
| ildikov | fungi: where is it set that what is reraised and what not? | 13:52 |
| gabriel-bezerra | Is there anything I can do to fix that? | 13:52 |
| *** otherwiseguy has joined #openstack-infra | 13:52 | |
| *** andreaf has joined #openstack-infra | 13:52 | |
| *** pafuent has joined #openstack-infra | 13:53 | |
| *** habib has quit IRC | 13:53 | |
| fungi | ildikov: pbr.packaging.LocalSDist.LocalBuildDoc.run() | 13:54 |
| *** jcoufal has quit IRC | 13:54 | |
| *** jcoufal has joined #openstack-infra | 13:55 | |
| *** jcoufal has quit IRC | 13:56 | |
| *** jcoufal has joined #openstack-infra | 13:56 | |
| *** habib has joined #openstack-infra | 13:56 | |
| *** habib has quit IRC | 13:57 | |
| *** habib has joined #openstack-infra | 13:57 | |
| *** gokrokve has quit IRC | 13:58 | |
| *** gokrokve has joined #openstack-infra | 13:58 | |
| fungi | gabriel-bezerra: is this the issue? http://logs.openstack.org/77/95977/6/check/check-grenade-dsvm/670e5dc/logs/old/devstacklog.txt.gz#_2014-05-28_11_56_19_380 | 13:59 |
| fungi | gabriel-bezerra: /usr/sbin/apachectl: 87: ulimit: error setting limit (Operation not permitted) | 13:59 |
| ildikov | fungi: tnx, looking | 14:00 |
| *** yfried has joined #openstack-infra | 14:00 | |
| *** amotoki has quit IRC | 14:00 | |
| *** BadCub has joined #openstack-infra | 14:01 | |
| fungi | ildikov: also, dhellmann and/or mordred may have some insight into sphinx+pbr failure modes and where we should be looking | 14:01 |
| *** gokrokve has quit IRC | 14:02 | |
| ildikov | fungi: I checked this run() function | 14:03 |
| fungi | ildikov: oops, sorry, not run() but _sphinx_run() | 14:04 |
| ildikov | fungi: so as a short summary, did you mean that for the utils.SystemMessage class the exceptions are not reraised and they should appear as reST markup error? | 14:04 |
| dhellmann | ildikov: I've lost some backtrace, do you have a link to the error log? | 14:04 |
| *** andreykurilin has joined #openstack-infra | 14:04 | |
| ildikov | fungi: yes, sorry I was lazy to write here the correct one :) | 14:04 |
| ildikov | dhellmann: http://logs.openstack.org/65/92365/6/check/gate-ceilometer-docs/cc989fb/console.html.gz#_2014-05-16_15_08_02_834 | 14:04 |
| fungi | dhellmann: quick summary, trying to figure out why that ^ job didn't fail | 14:04 |
| ildikov | dhellmann: in this case the docs gate job is green as the sphinx build is marked as succeeded, but the malformed table is of course missing from the generated doc | 14:05 |
| dhellmann | fungi: ah, got it | 14:05 |
| gabriel-bezerra | fungi: /opt/stack/new/devstack/lib/keystone: line 91: apache_site_config_for: command not found | 14:05 |
| fungi | (which has devolved into a dissecting of pbr's sphinx hooks) | 14:05 |
| *** kaisero has joined #openstack-infra | 14:05 | |
| ildikov | fungi: so if the not reraised exceptions would be the problem then they should be marked with the "reST markup error" I guess, or I misunderstood what you wrote above | 14:06 |
| fungi | gabriel-bezerra: ahh, yep that would also be a problem. looks like lib/apache may not be sourced yet at that point? | 14:06 |
| gabriel-bezerra | fungi: I think that what really breaks is this line http://logs.openstack.org/77/95977/6/check/check-grenade-dsvm/670e5dc/console.html#_2014-05-28_12_11_11_256 | 14:06 |
| gabriel-bezerra | fungi: It works on 2 vms I used to test | 14:06 |
| gabriel-bezerra | stack.sh worked there | 14:07 |
| gabriel-bezerra | that line with the ulimit error disappeared when I used sudo in a later patch | 14:07 |
| gabriel-bezerra | the line with enable_apache_site is also wrong, my patch removed the .conf suffix and it shows up in the log | 14:08 |
| gabriel-bezerra | it seems that my patch was not applied before those tests run | 14:09 |
| gabriel-bezerra | or partially applied? | 14:09 |
| ildikov | fungi: so it seems that 'warnerrors' is not in 'option_dict' in run(), but maybe it's a wrong direction | 14:10 |
| fungi | ildikov: yeah, the warnerrors point was somewhat tangential... you also have actual errors in that log, so i would have expected it to fail regardless | 14:11 |
| gabriel-bezerra | or it may have been applied on the wrong moment | 14:11 |
| W00die | Hi guys, I'm working with gerrit provided by openstack on a server and I'm kind of new, Is there a way to run the "manage-projects" command from jeepyb manually? | 14:12 |
| *** mengxd has joined #openstack-infra | 14:12 | |
| *** gokrokve has joined #openstack-infra | 14:12 | |
| dhellmann | ildikov, fungi : the clean_exit function in setup-test-env.sh looks suspicious | 14:13 |
| *** markmcclain has joined #openstack-infra | 14:13 | |
| dhellmann | also the fact that the venv environment is spinning up mongo to build docs is likely a problem | 14:13 |
| fungi | gabriel-bezerra: oh! somehow i forgot that devstack does actually have stable branches... so the initial devstack detup does not have your patch applied. you need to be able to upgrade from the tip of stable/icehouse to your patch on master (which means the initial devstack setup lacks your patch, but then it's applied by grenade during the upgrade) | 14:14 |
| ildikov | dhellmann: it was a next step on my list to fix it | 14:14 |
| pafuent | Hi. I would like to change the topics of Blazar IRC channels, but I get "You're not a channel operator". How I should change the topics? I need to became a channel operator? The change could be achieved through a patch in infra? | 14:14 |
| ildikov | dhellmann: I asked about my options here yesterday and clarkb mentioned this issue, that it shouldn't be needed there | 14:15 |
| dhellmann | ildikov: changing tox.ini to not run that script doesn't fix the issue locally; I still see warnings that are not converted to errors | 14:15 |
| fungi | W00die: we do sometimes run manage-projects manually. have you tried, and what isn't working for you? | 14:15 |
| ildikov | dhellmann: I do not know if we use this venv to anything else, I did not have the time to check it | 14:15 |
| *** maxbit has joined #openstack-infra | 14:16 | |
| dhellmann | ildikov: that env is meant to be used by infra jobs, so if the ceilometer team needs a generic env for running commands with the test environment configured we should add one with a different name | 14:16 |
| ildikov | dhellmann: sphinx-build itself reports that the build succeeded even if there were some formatting errors during the build process | 14:16 |
| dhellmann | ildikov: yeah, I'm looking at how that works now | 14:16 |
| anteaya | pafuent: SergeyLukjanov is the person you need to talk to | 14:17 |
| gabriel-bezerra | fungi: what should I do, then? | 14:17 |
| ildikov | dhellmann: so the simple sphinx-build behaves this way | 14:17 |
| pafuent | anteaya: Thanks, I'll contact him | 14:17 |
| anteaya | k | 14:17 |
| gabriel-bezerra | fungi: how am I going to possibly get that merged, if my patch doesn't come on the right moment? | 14:18 |
| *** resker has quit IRC | 14:18 | |
| *** esker has joined #openstack-infra | 14:19 | |
| ildikov | dhellmann: as for the venv, you mean that venv should remain for doc and another generic one should be created for any other case? | 14:19 |
| *** dstanek is now known as dstanek_zzz | 14:19 | |
| dhellmann | ildikov: that venv is used for several different CI jobs (docs, packaging, etc.), so yes, we should add a separate one for running commands that needs mongo and the other test environment stuff set up | 14:19 |
| *** yamahata has joined #openstack-infra | 14:20 | |
| *** msabramo has quit IRC | 14:20 | |
| ildikov | dhellmann: ok, great, I just wanted to be sure that we think about the same thing before modifying anything in tox.ini | 14:20 |
| ildikov | dhellmann: as for sphinx, fungi pointed me to pbr: http://git.openstack.org/cgit/openstack-dev/pbr/tree/pbr/packaging.py#n693 | 14:21 |
| *** e0ne_ has quit IRC | 14:21 | |
| dhellmann | ildikov: yes, and I noticed that ceilometer is using an old version of sphinx so I'm updating now to see if that fixes the problem | 14:21 |
| *** e0ne has joined #openstack-infra | 14:21 | |
| dhellmann | the warnings I am seeing are not going through the sphinx function that raises the exception when warningiserror is true | 14:22 |
| ildikov | dhellmann: well, it worth giving it a try, I mean the Sphinx update | 14:22 |
| fungi | gabriel-bezerra: what is it that you want to do? there's a good chance that you'll need to fix this in all supported versions, or provide some means to upgrade the configuration between devstack versions | 14:23 |
| ildikov | dhellmann: you mean the warnings of logs of the sphinx job? | 14:23 |
| fungi | gabriel-bezerra: you might want to bring it up with the devstack developers in #openstack-qa for better guidance on what they want | 14:23 |
| gabriel-bezerra | fungi: I want to fix the way apache sites are configured, because it doesn't work on Ubuntu 14.04 the way it is now. | 14:24 |
| *** david-lyle has joined #openstack-infra | 14:24 | |
| anteaya | hmmmm, I offered a patch to add barbican and python-barbicanclient to requirements/global-requirements but the tests failed since it can't find a barbican | 14:24 |
| dhellmann | ildikov: when I run "tox -e venv -- python setup.py build_sphinx" I see warnings for values missing from the glossary and for some formatting errors in measurements.rst but none of those cause the doc build to exit with an error code | 14:24 |
| anteaya | barbican isn't on pypi, what needs to happen to get it there? | 14:25 |
| anteaya | do we know? | 14:25 |
| ildikov | dhellmann: a-ha, ok, I see those also | 14:25 |
| gabriel-bezerra | fungi: So I put all the distribution dependent code in a single place: lib/apache, and created a new function that returns the full path of the site configuration file. | 14:25 |
| dhellmann | ildikov: in fact, I see an "ERROR: Unexpected indentation" which should trigger the failure, too | 14:26 |
| rdopiera | anteaya: I saw a pypi job defined in some projects, I would guess you need a correct setup.py and that job | 14:26 |
| rdopiera | anteaya: or you can do 'python setup.py register' maually | 14:26 |
| anteaya | rdopiera: cool thanks | 14:26 |
| rdopiera | anteaya: and add openstackci to the owners | 14:26 |
| fungi | gabriel-bezerra: right, i mostly understand the intent of the patch, just not how it fits with the life cycle of the devstack stable branches and what the upgrade story is for it | 14:26 |
| dhellmann | ildikov: if I run ".tox/py27/bin/sphinx-build doc/source doc/build" I get an error that it can't import oslo.messaging | 14:27 |
| anteaya | rdopiera: to the owners of the pypi package? | 14:27 |
| rdopiera | anteaya: yeah | 14:27 |
| ildikov | dhellmann: yes, that is what I noticed earlier and why I reported a bug about it, because usually no one checks the output of the docs gate job, if it succeeds | 14:27 |
| rdopiera | anteaya: in the 'roles' tab at pypi | 14:27 |
| ildikov | dhellmann: do you run it inside the virtual env? | 14:28 |
| anteaya | rdopiera: cool, thanks not sure if that is infra's job or barbican's | 14:28 |
| mriedem | sdague: figured out why we weren't hitting that cinder db_sync issue with rhel 6.5 and python-six 1.5.2 | 14:28 |
| gabriel-bezerra | fungi: Neither do I. What do you mean by "life cycle of the devstack stable branches" and the " upgrade story"? | 14:28 |
| rdopiera | anteaya: I suppose that pypi jenkins/gerrit job actually does setup.py register and upload | 14:28 |
| fungi | anteaya: does barbican need to be on pypi? is it a server (we don't release servers to pypi currently) or a client/library? | 14:28 |
| mriedem | we're shipping python-oslo-messaging 1.3.0a9 which didn't have this change: https://github.com/openstack/oslo.messaging/commit/13608437bb8d491badbc6c906f8d60b51e52cd79 | 14:28 |
| dhellmann | ildikov: yes | 14:29 |
| *** markmcclain has quit IRC | 14:29 | |
| fungi | gabriel-bezerra: saying i'm not sure how the devstack developers usually handle upgrades between branches for changes to the way service configuration is handled, so you should ask them | 14:29 |
| dhellmann | ildikov: oslo.messaging wasn't installed in the virtualenv for some reason | 14:29 |
| anteaya | fungi: the server | 14:29 |
| dhellmann | ildikov: let's move to #openstack-ceilometer to continue this | 14:29 |
| gabriel-bezerra | fungi: ok, I'll check that | 14:29 |
| ildikov | dhellmann: ok | 14:30 |
| fungi | anteaya: you'll be shocked to discover that we don't even release nova to pypi ;) | 14:30 |
| anteaya | the test failed because it couldn't find a barbican to download | 14:30 |
| anteaya | I'm shocked | 14:30 |
| fungi | anteaya: which test? | 14:30 |
| anteaya | only cause you told be to be though | 14:30 |
| anteaya | http://logs.openstack.org/61/96161/1/check/check-requirements-integration-dsvm/a29d5ca/ | 14:30 |
| anteaya | No distributions at all found for barbican (from -r /tmp/tmpddm9LL (line 7)) | 14:31 |
| anteaya | where does it have to be to be found? | 14:31 |
| fungi | anteaya: out of curiosity, why? your commit message just says "the projects should be included in the requirements repo" without any explanation for what other package on pypi is depending on them | 14:32 |
| fungi | anteaya: or did you mean to add them to projects.txt rather than global-requirements.txt? | 14:32 |
| W00die | thanks fungi, I was waiting for "manage-proyects" to run automatically but it never created my project, now that i ran it manually it did created it :D | 14:32 |
| anteaya | the fact they are incubated is insufficient for them to be included in requrirements? | 14:32 |
| *** jcoufal has quit IRC | 14:33 | |
| fungi | W00die: right, we trigger runs of manage-projects from our puppet master when configuration is updated | 14:33 |
| *** markmcclain has joined #openstack-infra | 14:33 | |
| anteaya | I looked at other patches, projects were added to global-requirements | 14:33 |
| anteaya | so that is what I did | 14:33 |
| anteaya | please do explain the difference? | 14:33 |
| *** dstanek_zzz is now known as dstanek | 14:34 | |
| mriedem | afazekas: added more information here: https://bugs.launchpad.net/devstack/+bug/1316328 | 14:34 |
| uvirtbot | Launchpad bug 1316328 in devstack "Stack trace when stack.sh hits "cinder-manage db sync"" [Undecided,Fix released] | 14:34 |
| mriedem | afazekas: is RDO building olso.messaging packages from trunk? | 14:34 |
| fungi | anteaya: projects.txt is the file which defines which projects should be forced to use requirements which match our global-requirements.txt and which will get requirements updates proposed when the global requirements list changes | 14:35 |
| fungi | anteaya: global-requirements.txt is an aggregate list of the projects used by other projects in openstack as python dependencies | 14:35 |
| anteaya | in that light then, I had intended them to be in global-requirements | 14:35 |
| W00die | fungui, is there a sugested documentation for connecting gerrit with github?, I see on the openstack doc that it is possible but i dont see much info on how it should be, I'm trying this: | 14:36 |
| W00die | http://www.packtpub.com/article/using-gerrit-with-github | 14:36 |
| W00die | but I'm not sure if its outdated | 14:36 |
| anteaya | if they are incubated, they should be available for use by other projects, should they not? | 14:36 |
| fungi | anteaya: which brings me back to the question, what other openstack projects are using those as python dependencies and listing them in their individual requirements.txt or test-requirements.txt files? | 14:36 |
| anteaya | none that I know of | 14:37 |
| anteaya | isn't the fact they are incubated sufficient? | 14:37 |
| fungi | sufficient for what? | 14:37 |
| anteaya | for inclusion in global-requirements? | 14:37 |
| anteaya | I can abandon my patch, if you prefer | 14:37 |
| *** rbowen has joined #openstack-infra | 14:37 | |
| *** wenlock has joined #openstack-infra | 14:38 | |
| fungi | to drive home the point, please take note of which of the following projects are listed in global-requirements.txt: nova, swift, glance, cinder, keystone... | 14:38 |
| *** wenlock has quit IRC | 14:38 | |
| rbowen | I wonder if someone can point me to a contact for the OpenStack youtube channel. There's a video - https://www.youtube.com/watch?v=T-e0f8PHa5s - that has the audio track wrong, and I was hoping we could get that fixed. | 14:38 |
| anteaya | I did | 14:39 |
| *** rgerganov has joined #openstack-infra | 14:39 | |
| anteaya | it sounds like you want me to abandon the patch, I will | 14:39 |
| *** wenlock has joined #openstack-infra | 14:39 | |
| anteaya | done | 14:39 |
| fungi | anteaya: yes, please abandon the patch. packages get added to global-requirements.txt when they are needed to install other openstack projects. it is not intended to act as a registry of official projects | 14:40 |
| *** andreykurilin has quit IRC | 14:41 | |
| fungi | rbowen: try contacting todd@openstack.org and he hopefully can get it corrected | 14:41 |
| rbowen | Thank you. | 14:41 |
| *** mrodden has joined #openstack-infra | 14:41 | |
| gabriel-bezerra | fungi: how does it work? It starts with the stable branch, then run stack.sh, unstack.sh, then applies the branch, then stack.sh, unstack.sh? | 14:42 |
| *** doude has joined #openstack-infra | 14:43 | |
| *** msabramo has joined #openstack-infra | 14:45 | |
| fungi | gabriel-bezerra: it seems to be described at http://git.openstack.org/cgit/openstack-dev/grenade/tree/README.rst | 14:45 |
| fungi | gabriel-bezerra: so it doesn't unstack... looks like it stops services individually and runs upgrade scripts for each of them, then starts them again | 14:46 |
| fungi | based on the description there | 14:46 |
| fungi | gabriel-bezerra: though again, it may be more fruitful to talk to the grenade developers in #openstack-qa if you have specific questions about it. #openstack-infra is more about the servers all this stuff runs on, not so much what gets run within a job | 14:47 |
| *** thedodd has joined #openstack-infra | 14:48 | |
| *** rgerganov has quit IRC | 14:49 | |
| *** alexpilotti has joined #openstack-infra | 14:50 | |
| gabriel-bezerra | oh, ok. thanks fungi. | 14:50 |
| *** jgrimm has joined #openstack-infra | 14:52 | |
| *** thedodd has quit IRC | 14:53 | |
| *** thedodd has joined #openstack-infra | 14:53 | |
| *** markmcclain has quit IRC | 14:55 | |
| *** habib has quit IRC | 14:55 | |
| *** gokrokve has quit IRC | 14:56 | |
| *** lcheng_ has joined #openstack-infra | 14:57 | |
| *** gokrokve has joined #openstack-infra | 14:57 | |
| *** radez_g0n3 is now known as radez | 14:58 | |
| phschwartz | morning infra | 15:02 |
| *** gokrokve has quit IRC | 15:02 | |
| *** mengxd has quit IRC | 15:02 | |
| *** rdopiera has quit IRC | 15:03 | |
| afazekas | mriedem: I do not know, what is packaged. I need to check the spec files. But probably one of the release tar balls is packaged. | 15:03 |
| * afazekas I need to hurry, I got on urgent SMS.. | 15:03 | |
| *** otter768 has joined #openstack-infra | 15:04 | |
| * afazekas will be back in ~1h | 15:05 | |
| *** otter768_ has quit IRC | 15:05 | |
| krtaylor | anteaya, from yesterday's meeting - did we decide on a third party string? Or did we just decide to agree on one | 15:07 |
| anteaya | we decided to agree on one | 15:08 |
| anteaya | I am working on that over here: https://etherpad.openstack.org/p/automated-gerrit-account-naming-format | 15:08 |
| anteaya | there are some issues | 15:09 |
| anteaya | for starters you will see starting on line 23 that gerrit confirms the following account names | 15:09 |
| *** andreykurilin has joined #openstack-infra | 15:09 | |
| anteaya | on line 3 from an email by HenryG there are some account names that don't match up with what gerrit has in the third party grouping | 15:10 |
| *** afazekas has quit IRC | 15:10 | |
| anteaya | also on line 8 you will see from something called gerrymander a list of accounts which I can match to gerrit names | 15:10 |
| anteaya | for instance rocktown | 15:10 |
| anteaya | where is that coming from? | 15:10 |
| *** alexpilotti has quit IRC | 15:11 | |
| anteaya | I was just about to find out what Daniel P. Berrange's irc nick was if I could | 15:11 |
| anteaya | so I could ask him | 15:11 |
| anteaya | hi phschwartz | 15:11 |
| krtaylor | anteaya, berrange maybe? | 15:12 |
| phschwartz | anteaya: I will be trying to get the wip patch I have done today for you to review. I got held up due to a back injury :( | 15:13 |
| anteaya | I don't see that nick in here | 15:13 |
| *** signed8bit has quit IRC | 15:13 | |
| anteaya | phschwartz: oh no, I hope you are okay | 15:13 |
| anteaya | phschwartz: how are you feeling? | 15:13 |
| *** signed8bit has joined #openstack-infra | 15:13 | |
| krtaylor | anteaya, danpb | 15:13 |
| anteaya | cool | 15:13 |
| anteaya | thanks | 15:13 |
| krtaylor | anteaya, so, this is account names prefix? | 15:14 |
| phschwartz | anteaya: In pain, need to make a dr appointment. I hurt it jumping shoulder first into rocks in the waves at the beach saving my friends 2 year old that got rolled up and pulled under. | 15:14 |
| krtaylor | wow | 15:14 |
| anteaya | krtaylor: what do you mean account names prefix? | 15:14 |
| *** matrohon has quit IRC | 15:14 | |
| anteaya | this is what I get when I go to gerrit groups to look up account names | 15:15 |
| fungi | krtaylor: i think the idea is that we have some particular pattern we'll use to rename them and for future names (whether that includes a common prefix, suffix or both) | 15:15 |
| anteaya | https://review.openstack.org/#/admin/groups/270,members | 15:15 |
| anteaya | krtaylor: note there is a subgroup in that group | 15:15 |
| anteaya | I am stumped on rocktown, bsn, pattabi-ayyasami-ci | 15:16 |
| krtaylor | anteaya, account names organization and standardization is good, but we were talking ML filtering, I was just looking for a tag | 15:16 |
| anteaya | right | 15:16 |
| fungi | krtaylor: since there will also be other criteria we'll potentially want to place on the names for these (for example, length?) the actual visible names will be chosen by the infra team when the accounts are created based on input from the operators requesting them | 15:16 |
| anteaya | Ci for one and Bot for another | 15:16 |
| anteaya | CI | 15:16 |
| mika | hm, https://launchpad.net/python-jenkins is pointing to openstack with a 404 - anyone of you aware where the most recent version is supposed to be? | 15:17 |
| krtaylor | fungi, yes, that sounds great | 15:17 |
| fungi | anteaya: in previous conversations (this has been coming up for about a year, initially suggested by HenryG) we may want things like "OpenStack Zuul Bot" for official systems and "IBM PowerVC Bot" for unofficial/third-party systems | 15:18 |
| krtaylor | well, if all third party testing is CI then CI works | 15:18 |
| *** ihrachyshka has quit IRC | 15:18 | |
| * krtaylor is looking at cinder | 15:18 | |
| anteaya | mika: looks like 0.2.1 was released last August: http://git.openstack.org/cgit/stackforge/python-jenkins/log/ | 15:19 |
| mika | anteaya: i see, thanks! | 15:20 |
| fungi | so that people can filter on a from header where the name starts with "OpenStack" for official/upstream messages and anything ending with "Bot" for all automated systems | 15:20 |
| *** UtahDave has joined #openstack-infra | 15:20 | |
| fungi | something along those lines anyway | 15:20 |
| anteaya | I agree that our systems should be using OpenStack in the account name | 15:20 |
| krtaylor | +1 | 15:20 |
| anteaya | got the sense that accounts would either use CI or Bot depended on how the definition split, | 15:21 |
| mika | anteaya: hm, would be nice to have a new release of python-jenkins, do you know whom we could ping about that? :) | 15:21 |
| anteaya | not having all third party ci named CI Bot | 15:21 |
| anteaya | mika: lets look at the log to see who contributes to it | 15:22 |
| mika | anteaya: Marc Abramowitz mainly | 15:22 |
| anteaya | james page did the last release | 15:22 |
| fungi | anteaya: yeah, i think we should factor out all but one common term just to keep things short, but probably keep a common prefix which will allow a simple filter to still catch all (official and unofficial) messages from automated systems in one go | 15:22 |
| anteaya | yes either one of those two | 15:22 |
| fungi | or suffix, or something | 15:22 |
| anteaya | msabramo: you are marc abramowitz if i remember correctly? | 15:23 |
| fungi | and this will work great until we get a contributor whose last name is actually "Bot" ;) | 15:23 |
| anteaya | we will convince that person to change their name | 15:23 |
| *** Ajaeger has quit IRC | 15:23 | |
| anteaya | it will be easier | 15:24 |
| krtaylor | thats a good start, I'd also like to see [Third Party xyz] in the subject like [Nova] | 15:24 |
| msabramo | anteaya: yes, that is correct | 15:24 |
| *** wenlock_ has joined #openstack-infra | 15:24 | |
| anteaya | msabramo: great, mika has some python-jenkins questions for you | 15:24 |
| fungi | krtaylor: subject mangling will be harder, i think, since gerrit decides that for us | 15:24 |
| msabramo | anteaya: OK, thanks | 15:25 |
| msabramo | mika: you have some questions about python-jenkins? | 15:25 |
| mika | msabramo: no questions, just a new (tagged) release would be awesome to see it show up in distributions :) | 15:25 |
| fungi | krtaylor: chances are that would require a new gerrit feature upstream in an upcoming release, or some sort of mail proxy which transforms messages emanating from the gerrit server | 15:25 |
| krtaylor | fungi, true for gerrit, but for general discussion, just getting everyone to agree to use Third Party would help | 15:26 |
| msabramo | mika: Ah yes, I agree. I don't have the power to do that. I'm not sure if that would be zaro or James E. Blair or who | 15:26 |
| fungi | krtaylor: sure | 15:26 |
| mika | msabramo: buddy of mine noticed that 0.2.1-0.1 as available in debian has problems e.g. with spaces in job names and this seems to be fixed in git already | 15:26 |
| mika | msabramo: i see, any chance you could talk to them? | 15:26 |
| msabramo | mika: I have proposed a lot of changes that have been merged, but I don't have the ability to approve changes, create release tags, etc. | 15:26 |
| mika | ah :) | 15:26 |
| fungi | mika: msabramo: anteaya: right now (since it was recently imported) the python-jenkins-ptl group in gerrit with permission to push tags to the project is empty. probably should add jeblair at a minimum | 15:27 |
| *** signed8bit has quit IRC | 15:27 | |
| anteaya | +1 | 15:27 |
| msabramo | fungi: Yeah python-jenkins was recently brought into stackforge by James E. Blair. Before that it lived for a while on Launchpad with some different authors and maintainers | 15:27 |
| *** signed8bit has joined #openstack-infra | 15:28 | |
| fungi | msabramo: i'm unsure what the project structure is for stackforge/python-jenkins, but i'm fairly sure we can work out here who needs to push tags and add them. input from jeblair would be helpful at any rate | 15:28 |
| *** wenlock_ has quit IRC | 15:28 | |
| msabramo | +1 to jeblair being able to push python-jenkins, or if he doesn't want it, he can suggest someone else | 15:29 |
| mika | jey, thanks folks :) | 15:29 |
| msabramo | python-jenkins git master could probably use a bit more testing before we push it | 15:29 |
| derekh_ | nodepool seems to be creating instances on tripleo-test-cloud over and above quota, any idea if there are loads of instance running it doesn't know about? | 15:30 |
| msabramo | mika: have you been testing the version from git master? | 15:30 |
| mika | msabramo: not yet | 15:30 |
| anteaya | krtaylor: I don't have any hope these folks will use the [Third Party][3rd party][third-party] email tag even if there is any consenseous from any group on what the tag should be | 15:30 |
| anteaya | krtaylor: these people genuinely believe that they are unique, not part of a group that already exists | 15:31 |
| fungi | derekh_: it's entirely possible that nodepool thinks things got deleted when they actually stuck around (during the ups and downs you've had). i'll see what the alien list looks like for those regions | 15:31 |
| *** gokrokve has joined #openstack-infra | 15:31 | |
| anteaya | it doesn't occur to them that there may already be a structure to address their concerns | 15:31 |
| msabramo | jenkins-job-builder uses python-jenkins; is it possible in OpenStack CI infrastructure to make JJB use the python-jenkins from git so that it can be tested? Or do we have to push a new release of python-jenkins in order to pick it up and test it with JJB? | 15:31 |
| anteaya | in the row of walls to bang my head on, I have given up on that one | 15:32 |
| anteaya | just as an fyi | 15:32 |
| krtaylor | anteaya, sigh, yeah, but we can start with a cross project wiki page, requirements and make baby steps | 15:32 |
| anteaya | if you want to carry any battle standard, I support you | 15:32 |
| anteaya | oh yes, by all means | 15:32 |
| anteaya | I'm working on the wiki pages | 15:32 |
| fungi | msabramo: what you probably need is a new gate job for python-jenkins which exercises it with jenkins-job-builder, to confirm changes to the former work with the latter | 15:33 |
| krtaylor | anteaya, I'll volunteer to be the first CI account page to work out any problems | 15:33 |
| anteaya | so I have listed all the accounts I have heard anything about on this etherpad: https://etherpad.openstack.org/p/automated-gerrit-account-naming-format | 15:33 |
| anteaya | awesome | 15:33 |
| derekh_ | fungi: cool, thanks, R2 seems to be running fins at about the rate I'd expect, R1 isnt being used at all, if we can get it back running for a couple of hours we'll now if the work lifeless did helped things | 15:33 |
| anteaya | we will need a seed account for the wikipages | 15:33 |
| anteaya | so thank you | 15:33 |
| krtaylor | anteaya, it was so hard to participate on my mobile client yesterday, I would have volunteered then | 15:34 |
| msabramo | zaro: fungi said "msabramo: what you probably need is a new gate job for python-jenkins which exercises it with jenkins-job-builder, to confirm changes to the former work with the latter". Is that something you'd like to do? | 15:34 |
| anteaya | np | 15:34 |
| anteaya | thanks for tuning in | 15:34 |
| *** pblaho has joined #openstack-infra | 15:35 | |
| *** pblaho has quit IRC | 15:35 | |
| krtaylor | anteaya, are you ready for prime time on the wiki template? | 15:35 |
| *** msabramo has quit IRC | 15:35 | |
| fungi | anteaya: it's worth noting that the lists of accounts people coughed up on the ml were based on historical data, but there are definitely accounts which have been renamed upon request over time | 15:35 |
| *** pblaho has joined #openstack-infra | 15:35 | |
| *** andreykurilin has quit IRC | 15:36 | |
| fungi | anteaya: so not being able to find some of them is an expected outcome | 15:37 |
| anteaya | krtaylor: no, not yet, I got some good feedback from yesterday's meeting that I need to incorporate, plus I have the template for openstack programs I need to put together | 15:37 |
| krtaylor | can't we just use the current gerrit account list? | 15:37 |
| *** jgallard has quit IRC | 15:37 | |
| anteaya | fungi: hmmm, okay, is rocktown ringing any bells for you? | 15:37 |
| anteaya | krtaylor: use the current gerrit account list for the name format discussion? | 15:38 |
| krtaylor | anteaya, ok, I'll wait, let me know if there is anything I can help with | 15:38 |
| fungi | anteaya: yeah, i believe it was renamed. you may need to beat the infra ml and openstack-ci bug lists with a search engine to figure out why/when though | 15:38 |
| krtaylor | anteaya, enumeration | 15:38 |
| anteaya | yes but if you look at line 5 there isn't even agreement between the gerrit group and the name that shows up in comments | 15:38 |
| fungi | krtaylor: yes, i think the list of third-party ci and voting third-party ci group members, plus any official accounts we happen to recognize which we didn't think of originally ought to be sufficient | 15:38 |
| anteaya | ah I was hoping to talk to danpb | 15:39 |
| anteaya | I don't think I care that much to go chasing the infra ml and openstack-ci bug lists | 15:39 |
| anteaya | I also think part of the exercise is account cleanup and removing stale accounts if they are stale | 15:40 |
| *** Sukhdev has joined #openstack-infra | 15:40 | |
| anteaya | but that is a sub goal not a prime goal | 15:40 |
| anteaya | if they are still active and are overlooked hopefully they will make themselves known | 15:40 |
| fungi | anteaya: right. suffice to say we can use the lists people provided on the ml as reminders of things to check into, but we already mostly have what we need to build a correct list ourselves anyway | 15:40 |
| anteaya | okay | 15:41 |
| anteaya | so right now I would go with line 19 and down on https://etherpad.openstack.org/p/automated-gerrit-account-naming-format | 15:41 |
| *** msabramo has joined #openstack-infra | 15:42 | |
| zaro | morning | 15:43 |
| anteaya | morning zaro | 15:45 |
| SergeyLukjanov | zaro, morning | 15:45 |
| *** gyee has joined #openstack-infra | 15:46 | |
| *** e0ne has quit IRC | 15:47 | |
| *** e0ne has joined #openstack-infra | 15:48 | |
| *** e0ne_ has joined #openstack-infra | 15:49 | |
| *** HenryG has quit IRC | 15:49 | |
| fungi | derekh_: found 57 alien nodes in that region aside from the te-broker, and the names of all of them suggest they were at some point created by nodepool. i'm removing them directly via novaclient now | 15:49 |
| derekh_ | fungi: ok, let me know if you need any reset, or want me to delete any, thanks | 15:50 |
| fungi | derekh_: well, novaclient is being very slow to respond to me on the deletes, so it's possible it's going to time out getting a response for them | 15:51 |
| *** e0ne has quit IRC | 15:52 | |
| *** HenryG has joined #openstack-infra | 15:52 | |
| zaro | msabramo: setting up a job to test both directions python-jenkins <-> jjb is probably a good idea. i'll see if i can propose a change for that. | 15:53 |
| derekh_ | fungi: ok, if it does, will see if I can find out why | 15:54 |
| anteaya | jeblair: when you are about: https://wiki.openstack.org/wiki/ThirdPartySystems/Foo | 15:54 |
| *** afazekas has joined #openstack-infra | 15:54 | |
| anteaya | let me know if you want additional expansion on any of the items in the table | 15:55 |
| fungi | derekh_: the list of instances i'm trying to delete is http://paste.openstack.org/show/81900/ | 15:55 |
| *** HenryG has quit IRC | 15:55 | |
| *** HenryG has joined #openstack-infra | 15:55 | |
| fungi | derekh_: all my nova delete calls went through without error, so now i'm just waiting for them to actually disappear from the nodepool alien-list output (11 remaining) | 15:56 |
| derekh_ | fungi: k, thanks, will check in a few minutes to see whats left | 15:56 |
| msabramo | zaro: cool, thanks! | 15:57 |
| msabramo | zaro: also note that mika was asking about a new release of python-jenkins | 15:58 |
| *** gokrokve has quit IRC | 15:58 | |
| *** gokrokve has joined #openstack-infra | 15:58 | |
| *** derekh_ has quit IRC | 16:00 | |
| *** marcoemorais has joined #openstack-infra | 16:00 | |
| *** otherwiseguy has quit IRC | 16:00 | |
| *** derekh_ has joined #openstack-infra | 16:01 | |
| openstackgerrit | craigtracey proposed a change to openstack-infra/config: Add nova-docker to the gerritbot config https://review.openstack.org/96192 | 16:01 |
| *** doude has quit IRC | 16:02 | |
| notmyname | jeblair: ttx: FYI storage polices patches have been proposed to master in swift. soft freeze in effect | 16:03 |
| *** gokrokve has quit IRC | 16:03 | |
| * anteaya is afk for a bit | 16:03 | |
| *** marcoemorais has quit IRC | 16:04 | |
| *** marcoemorais has joined #openstack-infra | 16:04 | |
| pleia2 | good morning | 16:05 |
| *** marcoemorais has quit IRC | 16:05 | |
| *** marcoemorais has joined #openstack-infra | 16:05 | |
| *** atiwari has joined #openstack-infra | 16:06 | |
| derekh_ | fungi: thanks, the region is back in use now | 16:06 |
| *** arunkant has joined #openstack-infra | 16:07 | |
| *** marcoemorais has quit IRC | 16:07 | |
| *** marcoemorais has joined #openstack-infra | 16:07 | |
| *** hogepodge has quit IRC | 16:08 | |
| *** hogepodge has joined #openstack-infra | 16:09 | |
| *** jooools has quit IRC | 16:10 | |
| *** crc32 has joined #openstack-infra | 16:11 | |
| fungi | derekh_: excellent | 16:11 |
| ttx | notmyname: ack | 16:12 |
| mordred | morning all | 16:12 |
| *** crc32 has quit IRC | 16:12 | |
| arunkant | Hi..I am trying to add new package dependency for addressing a keystone issue. This is my first time for requesting change in global requirements so want to see if following the process correctly. llink: https://review.openstack.org/#/c/95842/ | 16:13 |
| zaro | msabramo, mika : i'm not privileged to make a new release. best one to ask for that would be jeblair | 16:14 |
| *** IvanBerezovskiy has left #openstack-infra | 16:15 | |
| pleia2 | the jboss licensing/version world is definitely not my forte, anyone have thoughts on JBossAS vs JBossERB re: http://lists.openstack.org/pipermail/openstack-infra/2014-May/001274.html for our needs? | 16:16 |
| fungi | arunkant: at a quick look, seems like a reasonable change request though you've placed your additional requirement in the section which is commented as being for test-time-only dependencies (while your keystone review implies it's a run-time dependency) and you don't have it inserted in alphabetical order (which most of the rest of the contents seem to be in) | 16:16 |
| pleia2 | naturally I'm inclined toward the one we don't seem to need a license for, but if it's going to make zanata unmanagable... | 16:16 |
| fungi | pleia2: i think we ought to push for non-commercial infrastructure. if zanata doesn't work without the proprietary jboss distribution, then we should help or encourage them to fix that problem | 16:17 |
| clarkb | pleia2: whatever is available in ubuntu | 16:17 |
| arunkant | fungi: I will address those concerns in next patch..did not knew about the order and section..thanks for looking into this. | 16:18 |
| pleia2 | fungi: ok, open source high road it is :) | 16:18 |
| fungi | arunkant: sure, glad to help (keep in mind that i don't really exercise my requirements core rights for anything which isn't infrastructure related or a gate emergency, so you'll still need to solicit actual reviews from the real reqs core reviewers) | 16:19 |
| *** e0ne_ has quit IRC | 16:19 | |
| pleia2 | clarkb: yeah, JBossAS is the one in the Ubuntu repos (JBossERB is installable though, but it's a tarball kind of deal) | 16:19 |
| bodepd_ | jesusaurus: I'm getting started on the splits today. Can you create the empty projects that I need? | 16:19 |
| *** e0ne has joined #openstack-infra | 16:19 | |
| *** Hal_ has joined #openstack-infra | 16:20 | |
| mordred | look, it's a bodepd_ | 16:20 |
| *** otherwiseguy has joined #openstack-infra | 16:20 | |
| bodepd_ | I know. I exist. It's crazy. | 16:20 |
| mordred | I mean, I'm not sure I believe you EXIST | 16:21 |
| fungi | pleia2: a prime tenet of our infrastructure work is that we build our systems on top of components which can be reused by anyone, and basing some of it on a proprietary platform gets in the way of that | 16:21 |
| * mordred agrees with fungi | 16:22 | |
| pleia2 | fungi: so I don't think their enterprise version is actually *proprietary* it's just subscription-based and you need a license (like redhat) | 16:22 |
| bodepd_ | mordred: I don't even think I saw you at the summit, which is evidence that you might not exist as well | 16:22 |
| fungi | pleia2: so "high road" is perhaps not entirely accurate, but i do think we should help other projects be more free ;) | 16:22 |
| pleia2 | http://en.wikipedia.org/wiki/JBoss_Enterprise_Application_Platform says GPL | 16:22 |
| *** dprince has quit IRC | 16:22 | |
| mordred | bodepd_: that's fair. I think I heard that you were there though... | 16:23 |
| pleia2 | and we did look into getting redhat licenses before going with centos, so I wasn't sure where our line was | 16:23 |
| *** starmer has quit IRC | 16:23 | |
| fungi | pleia2: weird. how can they enforce subscription fees to obtain it? or is it just that it's freely obtainable from other sources but to get it directly from them you have to pay? | 16:23 |
| *** derekh_ has quit IRC | 16:23 | |
| mordred | pleia2: well, we were rather uncomfortable with that, but it was really just to test that openstack worked _on_ rhel | 16:24 |
| *** e0ne has quit IRC | 16:24 | |
| afazekas | mriedem: pong | 16:24 |
| pleia2 | fungi: I don't really know how that works | 16:24 |
| mordred | I don't think we should run any pieces of the actual infrastructure on things that requires licenses or subscriptions | 16:24 |
| mrodden | mordred: not sure if it still matters but i put out a 0.1.1 bash8 with that ignore (-i) fix | 16:24 |
| mordred | mrodden: woot! thanks! | 16:24 |
| mrodden | np | 16:24 |
| fungi | pleia2: because if it's really all released under the gpl, then someone else ought to be able to publish copies of it for free if they want (perhaps we could offer to mirror it for them for free?) | 16:25 |
| mordred | mrodden: I was starting to run it on some of our things to check it out - and we have a _bunch_ of bad indentation :) | 16:25 |
| openstackgerrit | A change was merged to openstack/requirements: Remove hp3parclient from global-requirements https://review.openstack.org/95971 | 16:25 |
| mrodden | mordred: i totally understand :) | 16:25 |
| openstackgerrit | A change was merged to openstack/requirements: Rename climate -> blazar https://review.openstack.org/96144 | 16:25 |
| mrodden | i haven't had time to put it to work against our internal stuff yet | 16:25 |
| mrodden | i'm not prepared for that yet | 16:26 |
| mrodden | also, bash8 still needs some work apparently | 16:26 |
| mordred | http://jbossas.jboss.org/downloads/ | 16:26 |
| *** Ivo has quit IRC | 16:26 | |
| pleia2 | mordred: so that's AS - the community version they said zanata may not do so great on | 16:27 |
| mordred | that's EAP | 16:27 |
| clarkb | as is packaged for ubuntu (also maybe we make this a trusty server to see what breaks? and agent master will break) | 16:27 |
| pleia2 | heh, EAP is at the top | 16:27 |
| pleia2 | oh, that's a confusing page | 16:27 |
| mordred | yeah it is | 16:27 |
| fungi | pleia2: and asp is listed as lgpl! | 16:28 |
| fungi | pleia2: perhaps zanata was suggesting that they could get us a technical support license for it? | 16:28 |
| mordred | I _think_ maybe we can mvn install it from there? | 16:28 |
| *** arnaud has joined #openstack-infra | 16:28 | |
| clarkb | mordred speaking of trusty, I think I am ready to start pushing your fixes through | 16:28 |
| mordred | woot! | 16:29 |
| pleia2 | fungi: well their email said it "doesn't have all the required libraries and versions" so I think it was more than support | 16:29 |
| clarkb | well review and so on | 16:29 |
| clarkb | and babysit | 16:29 |
| mordred | clarkb: oh. wait - you're goign to review them? that means I'll have to fix things ... | 16:29 |
| clarkb | mordred: yes :) | 16:29 |
| clarkb | actually iirc they have had a round of fixes and really just need babysitting now | 16:29 |
| *** arnaud has quit IRC | 16:30 | |
| mordred | this is not helpful: | 16:30 |
| mordred | http://www.redhat.com/products/jbossenterprisemiddleware/community-enterprise/ | 16:30 |
| fungi | pleia2: mordred: ewww... you have to create an account at jboss.org before you can download the release archives or view source code | 16:30 |
| openstackgerrit | Arun Kant proposed a change to openstack/requirements: Adding ldappool module dependency as needed by keystone bug #1320997. https://review.openstack.org/95842 | 16:30 |
| uvirtbot | Launchpad bug 1320997 in keystone "Common Ldap handler connection pooling" [Medium,In progress] https://launchpad.net/bugs/1320997 | 16:30 |
| mordred | markmc: get your people to make better pages ^^ | 16:30 |
| clarkb | ok afk to finish morning things. back soon. also I have dentist appointment at 2 I will afk for | 16:30 |
| mordred | fungi: is it a captive thing or a fake thing like mysql's? | 16:30 |
| *** hashar has joined #openstack-infra | 16:31 | |
| *** arnaud has joined #openstack-infra | 16:31 | |
| mordred | fungi: I supposed I can find that out... | 16:31 |
| pleia2 | yeah, so I went the `apt-get install jboss` route when I got started, then went to the zanata install docs and it was all kinds of scary (written for redhat, assumed enterprise jboss) | 16:31 |
| pleia2 | so right now is just my sanity check before I dive in there | 16:31 |
| *** bearhands is now known as comstud | 16:32 | |
| *** otherwiseguy has quit IRC | 16:32 | |
| fungi | weird that the jbossenterprisemiddleware/community-enterprise page is being served to me half in chinese | 16:32 |
| mordred | it's captive | 16:32 |
| afazekas | mriedem: I haven't seen six related issue with rdo icehouse, I had issue stable/icehouse devstack. The code part which referencing to six>=1.6 is exists in almost all openstack stable/icehouse component, but it is usually not called anywhere. The global requirements and all project requirements should have been changed when the added the call for six>=1.6 feature. | 16:33 |
| mordred | I just logged in, got to the download link, and it includes an auth token | 16:33 |
| mordred | pleia2: I think we should soldier on with apt-get install jboss | 16:33 |
| mordred | and if there are 12 other java libraries we have to install - well then, that's what puppet is for :) | 16:33 |
| mordred | and we can make a nice self-contained zanata puppet module for folks | 16:34 |
| pleia2 | ok, will do :) | 16:34 |
| *** jpich has quit IRC | 16:34 | |
| fungi | agreed. anything we can do to make zanata more consumable for others without needing to get enterprise subscriptions for things is a benefit to the community | 16:34 |
| openstackgerrit | Khai Do proposed a change to openstack-infra/config: add python coverage job to check pipeline. https://review.openstack.org/96198 | 16:34 |
| fungi | zaro: the commit message for that ^ could stand to mention what project it's for | 16:36 |
| mordred | btw - I read the article W00die linked in above: http://www.packtpub.com/article/using-gerrit-with-github | 16:36 |
| mordred | it's rather strange - but I went ahead and poked at gerrithub so that I could play with the things | 16:37 |
| mordred | and I'm 100% confused as to what benefits any of it has | 16:37 |
| jeblair | fungi, pleia2, mordred: last time i looked, jboss is basically not installable on rhel without extra (extremely expensive) licenses. afaict that's intentional due to business models. | 16:37 |
| jeblair | like, there just plain aren't any packages that do it | 16:38 |
| mordred | there is a plugin which will connect your gerrit to github - and there is a page you can go to to "import pull requests" - but it's manual - and it keeps the pull request open, and comments don't sync | 16:38 |
| openstackgerrit | Khai Do proposed a change to openstack-infra/config: add python coverage job to check pipeline for JJB. https://review.openstack.org/96198 | 16:38 |
| fungi | mordred: well, they seem to provide gerrit-as-a-service at least, so there's presumably some benefit to that part (the github integration is less interesting as far as i'm concerned) | 16:39 |
| mordred | fungi: oh - sure - that part is neat | 16:39 |
| mordred | I was more commenting on the point of the github plugin | 16:39 |
| jeblair | so yeah, i support the 'use ubuntu' route. and failing that, asking them 'how do we do this on centos without proprietary licenses?', and failing that, working on pootle. | 16:39 |
| mordred | jeblair: ++ | 16:39 |
| mordred | jeblair: given that apt-get install jboss is a thing, everything past that really should just be java, right? | 16:40 |
| morganfainberg | there is a github plugin for gerrit that does... more than the replication stuff? | 16:40 |
| morganfainberg | ... why? | 16:40 |
| fungi | morganfainberg: because of stockholm syndrome | 16:40 |
| jeblair | mordred: seems reasonable! can't wait to find out why we're wrong about that. ;) | 16:40 |
| morganfainberg | fungi, good answer. | 16:40 |
| jeblair | anteaya: the wiki page looks great, thanks | 16:41 |
| mordred | jeblair: well, we'll be wrong because of java | 16:41 |
| fungi | mordred: it's enterprise-grade *and* web-scale! | 16:41 |
| mordred | fungi: it can only be web-scale if it uses mongodb. does it use mongodb? | 16:42 |
| fungi | mordred: it might! it might... | 16:42 |
| W00die | hehe, doesn't openstack has it's repositories linked to github? I heard that what openstack does is close pull request and redirect them to gerrit | 16:42 |
| mordred | wow. that would be the best of both worlds - an "enterprise" grade java app that stores its data in mongo | 16:42 |
| mordred | W00die: we replicate to github using normal gerrit replication for dev convenience | 16:43 |
| mordred | W00die: and then we have a bot which closes the pull requests - that's right | 16:43 |
| *** bhuvan has joined #openstack-infra | 16:43 | |
| fungi | W00die: yep. there are some people who like to be able to "fork us over on github" so we replicate there. the pull-request-closer is more of a reaction to deal with people who think that because we mirror on github means anyone is actually watching their pull requests (one of the only bits you can't disable directly) | 16:44 |
| *** pblaho has quit IRC | 16:44 | |
| fungi | W00die: we disable issues, wiki, et cetera but there's no feature to disable pull requests or we would just do that instead | 16:45 |
| *** jlibosva has quit IRC | 16:46 | |
| mordred | wow. I'm learning all sorts of things today | 16:47 |
| mordred | http://docs.pythonpackages.com/en/latest/github-service.html | 16:47 |
| mordred | now - the real question is - why would that operate on the contents of the commit message and not on a tag? | 16:48 |
| *** whoops has joined #openstack-infra | 16:48 | |
| fungi | scary | 16:48 |
| mordred | I mean, otherwise that's a great little service to run for folks | 16:48 |
| openstackgerrit | Dan Bode proposed a change to openstack-infra/config: Remove pip module from config https://review.openstack.org/96203 | 16:48 |
| *** bodepd_ has quit IRC | 16:50 | |
| clarkb | mordred gerrithub scares me. iirc the great jenkins github fubar was a result of gerrithub | 16:50 |
| mordred | clarkb: lovely | 16:50 |
| *** gokrokve has joined #openstack-infra | 16:51 | |
| *** bodepd has joined #openstack-infra | 16:51 | |
| fungi | looks like the reqs update proposer is broken since our project renames... https://jenkins.openstack.org/job/propose-requirements-updates/38/console | 16:51 |
| clarkb | mordred I think your trusty stuff needs a rebase now that jesusaurus' refactor merged | 16:51 |
| *** ociuhandu has joined #openstack-infra | 16:51 | |
| fungi | fatal: Project not found: openstack/database-api | 16:51 |
| clarkb | thar was what prevented merging earlier | 16:52 |
| openstackgerrit | Andreas Jaeger proposed a change to openstack-infra/config: Remove duplication in translation scripts https://review.openstack.org/96142 | 16:52 |
| clarkb | fungi needs a projects.txt update? | 16:52 |
| fungi | clarkb: looks like it--about to propose a patch now | 16:53 |
| *** talluri has joined #openstack-infra | 16:53 | |
| *** Ajaeger has joined #openstack-infra | 16:54 | |
| *** otherwiseguy has joined #openstack-infra | 16:55 | |
| Ajaeger | infra team, I would appreciate a review of https://review.openstack.org/95377 - the jobs site for translation of log messages. Thanks! | 16:55 |
| *** _nadya_ has joined #openstack-infra | 16:56 | |
| openstackgerrit | A change was merged to openstack/requirements: Paramiko: bump to 1.13.0 https://review.openstack.org/81132 | 16:57 |
| *** talluri has quit IRC | 16:57 | |
| *** nati_ueno has joined #openstack-infra | 16:57 | |
| *** talluri has joined #openstack-infra | 16:57 | |
| openstackgerrit | Jeremy Stanley proposed a change to openstack/requirements: Update projects list for May 23 project renaming https://review.openstack.org/96205 | 16:58 |
| fungi | backports of that ^ to stable branches on the way too | 16:58 |
| mordred | clarkb: okie | 16:58 |
| mordred | clarkb: the nodepool or the install_puppet or both? | 16:59 |
| clarkb | mordred: the change that touches slave.pp | 16:59 |
| clarkb | because it was moved | 16:59 |
| *** rgerganov has joined #openstack-infra | 16:59 | |
| *** BadCub has quit IRC | 17:00 | |
| *** rdopiera has joined #openstack-infra | 17:02 | |
| *** talluri has quit IRC | 17:02 | |
| *** yamahata has quit IRC | 17:03 | |
| *** praneshp has joined #openstack-infra | 17:06 | |
| *** andreaf has quit IRC | 17:06 | |
| *** ominakov has quit IRC | 17:06 | |
| jesusaurus | bodepd: which splits are you starting with? | 17:06 |
| SergeyLukjanov | fungi, https://review.openstack.org/#/c/94722/ | 17:09 |
| *** ildikov has quit IRC | 17:09 | |
| *** andreaf has joined #openstack-infra | 17:10 | |
| *** andreaf has quit IRC | 17:10 | |
| openstackgerrit | Monty Taylor proposed a change to openstack-infra/config: Changes to get nodepool working with trusty https://review.openstack.org/88477 | 17:10 |
| *** otherwiseguy has quit IRC | 17:11 | |
| mordred | clarkb: ^^ | 17:11 |
| fungi | SergeyLukjanov: if that lands, we still need to deal with the oslotest rename there and also get that backported to the stable/icehouse branch | 17:11 |
| mordred | bodepd: btw - the pip provider in puppet does not honor proxy settings | 17:11 |
| SergeyLukjanov | fungi, yup, I mean it's already on the go :) | 17:12 |
| *** hashar has quit IRC | 17:12 | |
| mordred | bodepd: there is a place inside where it makes a direct ruby xmlrpc query to check if something needs to be upgraded, and that does not pass through configured proxy settings | 17:12 |
| *** andreaf has joined #openstack-infra | 17:12 | |
| mordred | bodepd: I've been meaning to write a patch for about a year now and simply haven't | 17:12 |
| fungi | SergeyLukjanov: SergeyLukjanov yep. SlickNik: are you backporting 94722 to stable/icehouse? | 17:13 |
| krtaylor | anteaya, we will rename all the public names for third party accounts, do we want to do that before requiring everyone to create their ThirdPartySystems contact page? | 17:13 |
| *** Hal_ has quit IRC | 17:13 | |
| krtaylor | anteaya, or is the rename only for new accounts, grandfathering in all existing | 17:13 |
| Ajaeger | fungi: do you want a backport of https://review.openstack.org/#/c/94722/ and https://review.openstack.org/#/c/96144/ - or will you make one patch that combines your patch with these two? | 17:14 |
| Ajaeger | fungi: missed your last message ;) | 17:14 |
| Ajaeger | Let me backport 96144 now - ok? | 17:14 |
| fungi | Ajaeger: oh, good point. i missed 96144 wasn't applied on icehouse yet | 17:15 |
| fungi | Ajaeger: separate backports are better if they landed separately in master | 17:15 |
| *** habib has joined #openstack-infra | 17:15 | |
| fungi | Ajaeger: i'll fix up 96205 and my backport to just take care of oslotest | 17:16 |
| *** harlowja_away is now known as harlowja_ | 17:16 | |
| *** marcoemorais has quit IRC | 17:16 | |
| *** dizquierdo has quit IRC | 17:16 | |
| openstackgerrit | Monty Taylor proposed a change to openstack-infra/config: Update get-pip URL to the new location https://review.openstack.org/92225 | 17:16 |
| *** marcoemorais has joined #openstack-infra | 17:16 | |
| Ajaeger | https://review.openstack.org/96209 is the backport | 17:16 |
| Ajaeger | fungi: ok | 17:16 |
| *** dstanek is now known as dstanek_zzz | 17:17 | |
| openstackgerrit | Jeremy Stanley proposed a change to openstack/requirements: Update projects list for oslo.test renaming https://review.openstack.org/96205 | 17:18 |
| *** habib has quit IRC | 17:18 | |
| *** habib has joined #openstack-infra | 17:19 | |
| zaro | fungi: i want to run jjb tests when there are changes to python-jenkins to verify that the upstream dependency doesn't break jjb. could you please help? | 17:20 |
| openstackgerrit | Monty Taylor proposed a change to openstack-infra/config: Add an entry for a new puppetmaster for puppet3 https://review.openstack.org/89931 | 17:21 |
| *** talluri has joined #openstack-infra | 17:21 | |
| *** Longgeek has quit IRC | 17:22 | |
| openstackgerrit | Michael Krotscheck proposed a change to openstack-infra/storyboard-webclient: Configure Timeline from Story details https://review.openstack.org/93563 | 17:22 |
| *** otherwiseguy has joined #openstack-infra | 17:22 | |
| fungi | zaro: sure, do we already have jjb jobs which use python-jenkins in some way (presumably with a mock jenkins of some kind)? | 17:22 |
| zaro | fungi: python-jobs runs the jjb tests that uses python-jenkins. is that what you mean? | 17:23 |
| fungi | zaro: i suppose that's a start. how much of python-jenkins is actually exercised by that? | 17:23 |
| fungi | zaro: enough to be able to at least tell whether or not changes to it will break jjb? | 17:24 |
| clarkb | none of it | 17:24 |
| *** Ryan_Lane has joined #openstack-infra | 17:24 | |
| *** vponomaryov has quit IRC | 17:24 | |
| clarkb | the tests only test yaml to xml compilation | 17:24 |
| zaro | fungi: ohh yeah, clarkb is correct. | 17:25 |
| fungi | zaro: so given clarkb's assertion there, you probably need something akin to the integration test we have for git-review changes | 17:25 |
| *** vponomaryov has joined #openstack-infra | 17:25 | |
| fungi | zaro: basically, spin up a jenkins master and then try to inject/delete some job definitions | 17:26 |
| zaro | zaro: ok. i gotha. | 17:26 |
| fungi | zaro: once that's a thing, you can test jjb and python-jenkins that way if you want | 17:26 |
| zaro | thanks. | 17:26 |
| fungi | zaro: alternatively, if python-jenkins grew some mechanism to mock jenkins interactions, you could use that instead of an actual jenkins backend for the job | 17:27 |
| fungi | zaro: the latter is preferable, but i have no idea how much work that would be to implement | 17:27 |
| openstackgerrit | Michael Krotscheck proposed a change to openstack-infra/storyboard: Remove unnecessary files https://review.openstack.org/95741 | 17:28 |
| *** mika has quit IRC | 17:28 | |
| *** mika has joined #openstack-infra | 17:30 | |
| fungi | i think nodepool may have grown very unhappy with us. the node count graph has plateaued | 17:30 |
| SlickNik | SergeyLukjanov / clarkb: Regarding https://review.openstack.org/#/c/95332/ I'm torn, and would like some input. | 17:30 |
| fungi | and we're running almost no jobs. looking into it | 17:30 |
| fungi | nodepoold is not running | 17:31 |
| zaro | msabramo: above ^ about JJB testing | 17:31 |
| SlickNik | SergeyLukjanov/clarkb: I'm using DEST=/opt/stack, because that's where the image-build scripts pulls down the OpenStack repos (trove / diskimage-builder). | 17:31 |
| fungi | i'm starting it back up and then looking into the logs to see why it died | 17:32 |
| clarkb | SlickNik: wfm | 17:32 |
| *** praneshp has quit IRC | 17:32 | |
| *** esker has quit IRC | 17:33 | |
| SlickNik | SergeyLukjanov: I could use a different dir as DEST to avoid confusion with a devstack-gate job, but that means that we won't be consistent as to where we're pulling OpenStack bits across jobs. | 17:33 |
| *** Ivo has joined #openstack-infra | 17:34 | |
| *** esker has joined #openstack-infra | 17:34 | |
| *** lcostantino has quit IRC | 17:34 | |
| fungi | [Wed May 28 16:40:43 2014] Out of memory: Kill process 7105 (nodepoold) score 896 or sacrifice child | 17:34 |
| fungi | [Wed May 28 16:40:43 2014] Killed process 7105 (nodepoold) total-vm:20821676kB, anon-rss:7164608kB, file-rss:0kB | 17:34 |
| Ajaeger | SlickNik: should I backport https://review.openstack.org/#/c/94722/ for you? | 17:34 |
| *** praneshp has joined #openstack-infra | 17:35 | |
| *** Ryan_Lane has quit IRC | 17:35 | |
| bodepd | mordred: do you guys carry a patch for the pip provider? That isn't related specifically to the stuff I was working on, but it's not an unreasonable thing for me to patch upstream. | 17:35 |
| mordred | bodepd: we don't - we don't have an proxy problems | 17:35 |
| *** Ryan_Lane has joined #openstack-infra | 17:35 | |
| bodepd | mordred: oh, I see that you guys have a pip3 provider | 17:35 |
| mordred | bodepd: we carry an alternate pip provider internally at HP - which I have a not-quite-working patch for in flight here | 17:36 |
| bodepd | jesusaurus: I was starting with pip today | 17:36 |
| mordred | bodepd: https://review.openstack.org/#/c/51425/ | 17:36 |
| mordred | bodepd: but that will probably just make you cry | 17:36 |
| fungi | bodepd: we were working around the challenges of having pip install python 2.7 and 3.3 versions of things on the same system, since the backported 3.3 debs to precise came with a separate python3-pip | 17:36 |
| *** UtahDave has quit IRC | 17:37 | |
| * mordred really wants to patch the pip provider to be 2/3 aware in general - and also to install pip using get-pip instead of distro packages | 17:38 | |
| mordred | but it's really above my current level of ruby zen | 17:38 |
| SlickNik | Ajaeger: Sure, go for it. | 17:38 |
| SlickNik | Ajaeger: looks like it merged this morning. | 17:38 |
| *** esker has quit IRC | 17:38 | |
| *** UtahDave has joined #openstack-infra | 17:39 | |
| bodepd | mordred: and being 2/3 aware is not sufficient just to select a provider? | 17:39 |
| Ajaeger | SlickNik: will do - yeah, it merged nicely | 17:39 |
| bodepd | mordred: you need to be able to specify both 2 and 3 packages? | 17:39 |
| clarkb | mordred: zen? | 17:39 |
| mordred | bodepd: yeah - we need to be able to tell it to install a package into the python3 install using pip, or into the python2 install using pip | 17:40 |
| mordred | bodepd: selecting a pip3 provider is fine, actually | 17:40 |
| *** arnaud has quit IRC | 17:40 | |
| *** Sukhdev has quit IRC | 17:40 | |
| *** lcostantino has joined #openstack-infra | 17:40 | |
| bodepd | mordred: is that built into pip, or is that something else (in ruby-land, it would be called rvm) | 17:40 |
| bodepd | mordred: people have this same issue with rvm, I would start there and see if someone has an elegant solution | 17:40 |
| Ajaeger | SlickNik, fungi: https://review.openstack.org/96212 is the backport | 17:41 |
| mordred | bodepd: totally different - you can have different parallel installs of python | 17:41 |
| bodepd | mordred: this class of problem was intended to be solved with composite namevars, but last I checked, they were a giant POS | 17:41 |
| bodepd | mordred: what is managing that process? Is it pip? | 17:41 |
| mordred | so, the pythons are installed via distro packages | 17:41 |
| nibalizer | rvm does enable sane multiple installs of ruby/rubygems | 17:41 |
| nibalizer | but the virtualenv way of doing it (python) is totally different | 17:42 |
| mordred | pip itself simply installs things into the python env it was executed from | 17:42 |
| bodepd | yeah, that is what I was thinking of is virtualenv. | 17:42 |
| mordred | so if you had a /usr/bin/python3 and that was the python that invoked pip | 17:42 |
| mordred | it will cause the packages to be installed into /usr/local/lib/python3.3/site-packages | 17:42 |
| bodepd | the easiest way to do it might be to create two types | 17:42 |
| Ajaeger | SlickNik:No, it did not merge yet - was approved but not through the gate yet | 17:42 |
| bodepd | pip2 and pip3 | 17:42 |
| mordred | bodepd: virtualenv is the way that a lot of people deal with this issue - but in our case, we're installing some system level things that need to be server-wide | 17:43 |
| mgagne | bodepd: and what if you want to test against 2 versions of python3... | 17:43 |
| * mgagne runs away | 17:43 | |
| bodepd | mgagne: each resource declaration would only be for one version | 17:43 |
| nibalizer | bodepd: so this is where its at now: http://git.openstack.org/cgit/openstack-infra/config/tree/modules/pip/lib/puppet/provider/package/pip3.rb | 17:43 |
| mordred | mgagne: yeah... | 17:43 |
| nibalizer | mordred: did your massive pip change land yet? | 17:44 |
| clarkb | lolno | 17:44 |
| *** ildikov has joined #openstack-infra | 17:44 | |
| mordred | bodepd: actually, (mgagne thanks) that's the reason in my brain I was thinking adding a version to the provider would be nice | 17:44 |
| bodepd | mgagne: if you need to test against both: https://gist.github.com/bodepd/780383747930235750d0 | 17:44 |
| mgagne | nibalizer: still a WIP according to the link above | 17:44 |
| clarkb | I am beginning to think that needs to come in bite size changes | 17:44 |
| *** rgerganov has quit IRC | 17:44 | |
| mordred | because you might want to install a package into python 2.6, python 2.7, python 3.2 or python 3.3 | 17:44 |
| bodepd | mordred: adding a version to the provider is a bit complicated, b/c of uniqueness | 17:44 |
| mordred | clarkb: I agree | 17:44 |
| mordred | bodepd: yah | 17:45 |
| mgagne | mordred: what about adding venv support to the package resource :D | 17:45 |
| bodepd | mordred: it puts you down the composite namevar path | 17:45 |
| mordred | ultimately, I think we could live without that - it's jsut a completeness thing | 17:45 |
| bodepd | unless... it accepted a list of versions | 17:45 |
| nibalizer | oh yes bodepd if you would comment on mordreds patch that would be much appreciated | 17:45 |
| mordred | nibalizer: I think it needs work too | 17:45 |
| mordred | nibalizer: like, last time we tried it, it broke | 17:46 |
| nibalizer | for anyone who doesn't know, bodepd literally wrote the book on types and providers | 17:46 |
| mgagne | nibalizer: and I'm still struggling to write code for them =) | 17:46 |
| nibalizer | mordred: :< i guess the soft-freeze on infra/config didn't hold ? | 17:46 |
| *** praneshp has quit IRC | 17:46 | |
| mika | i assume it's also jeblair who releases new version of jenkins-job-builder? (nice stuff in git which isn't yet in the 0.7.0 release) | 17:47 |
| bodepd | mordred: nibalizer still looking, but I would say that is the 'easy' way to do it | 17:47 |
| fungi | bodepd: yeah that was the other challenge i faced. if we wanted, for example, to have puppet install a package named "tox" for both python 2.7 and python 3.3, we'd have to find some way to work around the fact that puppet doesn't let you install two packages of the same name with different providers, due to nonuniqueness | 17:47 |
| mgagne | mika: zaro can too | 17:48 |
| clarkb | nibalizer: iirc heartbleed happened | 17:48 |
| jesusaurus | lifeless: SpamapS: im trying to find a slide from a presentation that has a large diagram of an ideal CI/CD system that has testing of code, creation of packages from the code, creation of images from the packages, testing of images and promotion of images to golden images. is this ringing a bell? do you know where I can find this slide? | 17:48 |
| bodepd | fungi: yeah, two ways: 1. create different types 2. make version a param that accepts an array | 17:48 |
| mika | zaro: any chance we could see a new jenkins-job-builder release? :) | 17:48 |
| mika | mgagne: thx! | 17:48 |
| mordred | jesusaurus: I think that's in the tripleo deck? | 17:48 |
| bodepd | fungi: there is something called composite namevars that definitely did not work after it was created | 17:48 |
| jesusaurus | mordred: i think you gave that presentation in seattle like a year ago | 17:48 |
| nibalizer | but yea plzcanhaz compostite namevars | 17:48 |
| bodepd | I doubt its been fixed b/c the issues were *complicated* | 17:49 |
| jesusaurus | mordred: maybe? where can i find the tripleo deck? where are the various decks kept? | 17:49 |
| mordred | jesusaurus: yeah - lemme send you a copy | 17:49 |
| fungi | bodepd: aha. so instead of using type "package" we'd have pip2package and pip3package types? | 17:49 |
| nibalizer | fungi: you get it in other places to, such as needing to install the system package sqlite3 and the rubygem sqlite3 -> kerboom! | 17:49 |
| mordred | jesusaurus: the tripleo deck just gets emailed around | 17:49 |
| jesusaurus | ah, okay | 17:49 |
| mgagne | bodepd: like most bugs opened for years at puppet: too complicated and bug stays open in limbo state. =) | 17:49 |
| clarkb | mordred: 88477 lgtm | 17:50 |
| *** _nadya_ has quit IRC | 17:50 | |
| anteaya | jeblair: wonderful thank you, I'll work on the template for openstack programs now | 17:50 |
| clarkb | jeblair: fungi SergeyLukjanov ^ I am happy to babysit that change if someone else wants to review and approve it | 17:50 |
| mgagne | fungi: you could still use the package resource but implement your own provider which would make clever use of the resource name | 17:50 |
| anteaya | krtaylor: we haven't exactly discussed timing and scope for third party naming | 17:51 |
| anteaya | krtaylor: I think agreeing on a format is our first step | 17:51 |
| Ajaeger | clarkb: could I get some comments from you on https://review.openstack.org/95377, please? | 17:51 |
| *** habib has quit IRC | 17:51 | |
| mgagne | fungi: package { 'tox@py27': ensure => present, provider => 'pyninja' } | 17:51 |
| bodepd | fungi: yeah, that woudl work. mgagne 's suggestion would work as well, but you would have to embed the parsing logic in the current provider | 17:52 |
| fungi | ahh | 17:52 |
| mgagne | bodepd: isn't it what we do with our ini provider? =) | 17:52 |
| jeblair | anteaya: template for openstack programs? | 17:52 |
| mgagne | bodepd: what about: venv { 'py27': ensure => present } and package { 'tox@py27': ensure => present, provider => 'venv' } :D | 17:53 |
| mordred | jeblair: sent | 17:53 |
| mordred | gah | 17:53 |
| mordred | jesusaurus: sent | 17:53 |
| anteaya | jeblair: yes a template for openstack programs | 17:54 |
| jeblair | anteaya: i don't understand what you're saying | 17:54 |
| anteaya | https://etherpad.openstack.org/p/third-party-wiki-templating | 17:54 |
| anteaya | line 20 | 17:54 |
| *** otherwiseguy has quit IRC | 17:54 | |
| fungi | mgagne: well, we're talking about system-wide installs rather than virtualenvs (parallel installations within the same system context) | 17:54 |
| jeblair | anteaya: i understand now, thanks! | 17:54 |
| anteaya | jeblair: np sorry it was opaque | 17:55 |
| mgagne | fungi: alright, lets not go down the path of venv hell in puppet then :P | 17:55 |
| anteaya | Ajaeger: how do I add a link to https://wiki.openstack.org/wiki/Category:ThirdPartySystems | 17:55 |
| nibalizer | mgagne: as an aside i find this module to be really good at dealing with python and virtualenv https://github.com/stankevich/puppet-python | 17:56 |
| Ajaeger | anteaya: [[Category:ThirdPartySystems]] | 17:56 |
| anteaya | Ajaeger: when I click the edit button I can only edit: This page links to all pages tagged as relevant for 3rd party systems. | 17:56 |
| fungi | mgagne: in fact, we're mainly using it to get the things we need to be able to create virtualenvs (that is, newer versions of pip, virtualenv, tox, et et cetera than are distributed by the operating system) | 17:56 |
| anteaya | Ajaeger: I don't understand the context for how to use that bit of code | 17:56 |
| clarkb | mordred: I note that the thread on 1.1 problems seems to have gone quiet. We do still see lots of error'd nodes though | 17:56 |
| mgagne | nibalizer: now that someone did it already, I lost interest :D | 17:56 |
| anteaya | Ajaeger: I need to create a new template | 17:57 |
| anteaya | Ajaeger: I don't know how to create a new template | 17:57 |
| Ajaeger | anteaya: sorry, was confused.... | 17:57 |
| Ajaeger | anteaya: What's the name of the new template? | 17:57 |
| Ajaeger | ProgramsThirdParty ? | 17:57 |
| jesusaurus | mordred: awesome thanks | 17:57 |
| Ajaeger | anteaya: go to https:/wiki.openstack.org/wiki/Template:ProgramsThirdParty and create the page | 17:58 |
| Ajaeger | And fill it with content ;) | 17:58 |
| anteaya | Ajaeger: ahh, okay that could work | 17:58 |
| Ajaeger | Copy from the other template what you like ;) | 17:58 |
| anteaya | just navigating to the url creates the page? | 17:58 |
| fungi | clarkb: 88477 lgtm (slightly sketchy about adding things to profile.d but i suppose since this is just envvar manipulation that's okay). you can approve when you're ready to watch it | 17:59 |
| clarkb | fungi: weren't those additions always there? | 17:59 |
| fungi | clarkb: oh, indeed! that was a rename | 18:00 |
| *** talluri has quit IRC | 18:00 | |
| Ajaeger | anteaya: Navigate to it and then there's a button "Create the page" | 18:00 |
| msabramo | zaro et al: I would lean towards mocking to test python-jenkins with JJB | 18:00 |
| lifeless | jesusaurus: we have one like that without the packages bit :) | 18:00 |
| lifeless | jesusaurus: in most of the tripleo presentations | 18:01 |
| msabramo | much faster and less error-prone than spinning up a bunch of Jenkins stuff | 18:01 |
| msabramo | and I already mock Jenkins in the python-jenkins tests, which have 100% coverage | 18:01 |
| Ajaeger | anteaya: if you go to the page, a box opens, if you save that the page is created... | 18:01 |
| Ajaeger | No need t osay Create (just checked) | 18:01 |
| msabramo | I'm not sure I'm a fan of having python-jenkins run tests with an upstream component like JJB - that seems backwards to me | 18:02 |
| clarkb | msabramo: I think its the other way around | 18:03 |
| fungi | msabramo: zaro: agreed that's the best possible solution. so really just need to figure out how to exercise jjb against python-jenkins using the mocks you already have to provide proper responses | 18:03 |
| clarkb | msabramo: its gating just like we do with other related components | 18:03 |
| *** melwitt has joined #openstack-infra | 18:03 | |
| mgagne | msabramo: for important projects such as pbr and JJB, we test against consumers of the projects | 18:03 |
| clarkb | but I agree. There hasn't been a need for that so far | 18:03 |
| fungi | yeah, the oslotest solution is more of an extreme example of that | 18:04 |
| msabramo | clarkb: yeah, I've seen this "gating" idea before. It seems weird to me, but I am willing to entertain that it may make total sense for the way OpenStack development works | 18:04 |
| clarkb | mordred: fungi: approved the trusty slave fixes change | 18:05 |
| msabramo | clarkb: but it is weird when I submit simple changes for something like python-jenkins and then it fails because of some weird problem in heat or neutron, which are things I don't know anything about | 18:05 |
| fungi | msabramo: well, the idea being that you can know when you propose a change to python-jenkins whether it will cause jenkins-job-builder to break. you might simply run that as an advisory job rather than actually having it prevent you from being able to approve and merge that change | 18:05 |
| mordred | clarkb: ossum | 18:06 |
| mgagne | fungi, msabramo: just like how we compare the XML configs of our CI jobs before and after a change in JJB. | 18:06 |
| msabramo | fungi: yeah, I see the value. It seems like maybe something that could be solved by inverting things. LIke when there is a new python-jenkins, it triggers a new build of JJB and that JJB test is where the problems are caught; not in python-jenkins | 18:06 |
| fungi | msabramo: but at least then you'll know that before you release a new python-jenkins version, jenkins-job-builder will likely need to receive a new version which will either conflict with that or support it | 18:06 |
| msabramo | but this is a very well-established paradigm in OpenStack so I don't think I should argue it | 18:07 |
| msabramo | I'm sure there is a lot of history behind it and you guys did this for good reasons | 18:07 |
| SergeyLukjanov | SlickNik, that's a point too | 18:07 |
| clarkb | msabramo: https://github.com/openstack/openstack/graphs/commit-activity illustrates it pretty well | 18:08 |
| *** dstanek_zzz is now known as dstanek | 18:08 | |
| fungi | msabramo: well, it's more than "these are tests you can run if there are particular downstream consumers of your library which you want to make sure can continue to use it" | 18:08 |
| SergeyLukjanov | clarkb, which CR I should review? :) | 18:08 |
| *** lcheng_ has quit IRC | 18:08 | |
| fungi | msabramo: we just happen to have a mechanism to run that for you on every proposec commit | 18:08 |
| fungi | proposed | 18:08 |
| clarkb | SergeyLukjanov: 88477 is the one I just approved | 18:08 |
| openstackgerrit | Arun Kant proposed a change to openstack/requirements: Adding ldappool module dependency as needed by keystone bug #1320997. https://review.openstack.org/95842 | 18:08 |
| uvirtbot | Launchpad bug 1320997 in keystone "Common Ldap handler connection pooling" [Medium,In progress] https://launchpad.net/bugs/1320997 | 18:08 |
| clarkb | SergeyLukjanov: feel free to look at it though maybe you will see something | 18:08 |
| SergeyLukjanov | clarkb, okay:) | 18:09 |
| mgagne | msabramo: JJB was written by openstack-infra to manage their own jobs. It wasn't widely used outside openstack before and made sense to do such tests so they don't break their own stuff. | 18:09 |
| *** lcheng_ has joined #openstack-infra | 18:09 | |
| fungi | msabramo: it's entirely optional. but it does help high-profile projects using your library be able to provide you real-time feedback on whether what you're doing will work for them | 18:09 |
| *** pafuent has left #openstack-infra | 18:09 | |
| msabramo | mgagne, fungi: yeah I can see that | 18:10 |
| openstackgerrit | A change was merged to openstack-infra/config: Add a doc8 stackforge project https://review.openstack.org/94236 | 18:10 |
| msabramo | perhaps with python-jenkins, it feel stranger, because it is a much more general module and it was recently imported to stackforge | 18:10 |
| fungi | msabramo: we already have plenty of libraries we use in various places which just instantly break us when they release new versions, so we've gotten pretty good at putting out those fires, but anywhere we can avoid more fire drills is nice | 18:11 |
| SergeyLukjanov | clarkb, it's lgtm, I've already reviewed it earlier today, but decided to postpone sending a mark to check ruby package names ;) | 18:11 |
| openstackgerrit | Verónica Musso proposed a change to openstack/requirements: Add django-angular as requirement https://review.openstack.org/92690 | 18:11 |
| bodepd | mgagne: I would say those things are just not supported :) | 18:12 |
| pleia2 | the jbossas4 package on Ubuntu is useless to us (and known "cannot be used as an application server yet" according to the README.Debian) | 18:13 |
| openstackgerrit | A change was merged to openstack-infra/config: Changes to get nodepool working with trusty https://review.openstack.org/88477 | 18:13 |
| clarkb | pleia2: wow | 18:13 |
| msabramo | fungi: yeah, alright I'll buy in. What was the original question, I forgot? | 18:13 |
| mordred | wow | 18:13 |
| mordred | so | 18:13 |
| pleia2 | at least one person has been working on it in a PPA, but that's kind of scary http://askubuntu.com/questions/17747/why-isnt-there-a-useful-jboss-package | 18:13 |
| mordred | it exists in ubuntu, but doesn't actually work? | 18:13 |
| pleia2 | elizabeth@zanata:/usr/share/doc/jbossas4$ cat README.Debian | 18:14 |
| pleia2 | jbossas4 is currently in a very alpha stage of packaging. I can be used | 18:14 |
| pleia2 | to build other libraries depending on JBoss like libhibernate3-java but | 18:14 |
| pleia2 | it is not complete and cannot be used as an application server yet. | 18:14 |
| fungi | msabramo: mostly whether there's a way we can devise an integration test to exercise python-jenkins from jenkins-job-builder calls | 18:14 |
| pleia2 | sad story | 18:14 |
| mordred | pleia2: that just makes me sadder than I can possibly describe | 18:14 |
| fungi | msabramo: leveraging your existing mocks so that we can emulate submitting job configs and deleting them again | 18:14 |
| fungi | msabramo: so that we test as many of the methods in the python-jenkins api as jenkins-job-builder currently relies on, ideally | 18:15 |
| morganfainberg | pleia2, wow | 18:15 |
| pleia2 | it's pretty much just synced from debian, where not a whole lot of work is happening (last changelog entry in 2012) | 18:16 |
| *** _nadya_ has joined #openstack-infra | 18:16 | |
| *** jamielennox is now known as jamielennox|away | 18:16 | |
| msabramo | fungi: I think basically we need more test coverage for JJB | 18:17 |
| clarkb | krtaylor: was it you who mentioned that your jenkins server is capable of copying console logs to swift? | 18:17 |
| clarkb | krtaylor: I am curious to know what your solution to that is | 18:17 |
| msabramo | fungi: it just tests the YAML to XML conversion currently. We would test it more thoroughly, by mocking similarly to how we do that in python-jenkins | 18:17 |
| *** dstanek is now known as dstanek_zzz | 18:17 | |
| msabramo | fungi: I have always thought that was something I might tackle at some point | 18:18 |
| fungi | msabramo: i don't doubt that. the original question was when someone (was it you?) said "will our next python-jenkins release break jenkins-job-builder" so i proposed a way to always know whether that's going to be the case. if we mock the python-jenkins api calls in jjb, then that won't really tell us whether python-jenkins itself works (integration testing) | 18:18 |
| msabramo | fungi, zaro: Although I think probably be want https://review.openstack.org/87382 to land first so that we know what our baseline coverage is and then can see it go up as we add tests | 18:18 |
| bodepd | nibalizer: mordred I reviewed and commented on the patch, I definitely did not process every single line of ruby code, but at least went through the code in general. | 18:19 |
| fungi | msabramo: yep. i was just waiting for test results to come back on that one | 18:19 |
| *** esker has joined #openstack-infra | 18:19 | |
| msabramo | fungi: looks like it was verified last night by Jenkins at 7:10pm | 18:20 |
| fungi | oh, that adds the tox coverage target. i was thinking about the one adding the job to the check pipeline | 18:20 |
| fungi | reviewing that too | 18:20 |
| msabramo | fungi: ah ok, different change sets | 18:20 |
| msabramo | fungi = Jeremy Stanley? | 18:20 |
| fungi | yeah | 18:20 |
| msabramo | ah ok, I forget all the names | 18:20 |
| fungi | names are unimportant | 18:21 |
| bodepd | the most important part for me is that I need to wait for tons of patches before I can split the bits out that I want | 18:21 |
| *** praneshp has joined #openstack-infra | 18:21 | |
| msabramo | fungi: so yeah, I guess what I was thinking before but didn't articulate well is that we wouldn't mock out the entire python-jenkins API (because then yes, we're not testing python-jenkins at all). We would mock out the underlying HTTP stuff | 18:21 |
| bodepd | mordred: actually, when is the best time to try to get patches in to split things out | 18:21 |
| *** gabriel-bezerra has left #openstack-infra | 18:21 | |
| fungi | msabramo: that could work | 18:22 |
| bodepd | mordred: more specifically, once your pip provider patches get in, is that a good time to split pip out? | 18:22 |
| msabramo | fungi: I haven't thought a ton about it but I might go for using a tool like CaptureMock or vcrpy to do said HTTP mocking | 18:22 |
| bodepd | mordred: could it happen before? Are there more design things that need to be discussed before splitting out modules? /cc jesusaurus | 18:22 |
| msabramo | fungi: Those tools automatically record the requests and responses from a real server into a file and then can replay from that file when it exists and is deemed to be up to date | 18:22 |
| fungi | msabramo: yep, that's at least a good way to get a starting point, and then you can edit it from there as you need to extend it | 18:22 |
| bodepd | mordred jesusaurus for example: do we still want to install all external deps from the forge, or could we pin revisions and install deps from source | 18:23 |
| msabramo | cool | 18:23 |
| nibalizer | bodepd: awesome thanks | 18:23 |
| mordred | bodepd: I think we can split before that patch - we can re-parent that patch at the new repo if needed | 18:23 |
| fungi | msabramo: though openstack itself is full of rest apis, so it's entirely likely we already have a ton of mindshare behind ways to test and mock http interactions | 18:23 |
| nibalizer | i'm planning on adding a ruby-lint test of some kind to pass over our .rb | 18:23 |
| nibalizer | do you have a prefered tool for linting? the two i found are both ruby2 only | 18:23 |
| *** nati_ueno has quit IRC | 18:24 | |
| fungi | msabramo: in which case we may have bits we can just hoist straight in to do that | 18:24 |
| msabramo | fungi: yeah, if there is some other recommended approach I am all ears. Personally I have used vcrpy a little bit in my own projects and I have played with CaptureMock but not gotten quite as far | 18:24 |
| mordred | bodepd: and _I_ don't think we need to do forge if we can do a from-source method that works well - but things like librarian have been a non-starter for us so far | 18:24 |
| clarkb | notmyname: are you around? any chance we can talk about the swift review sponsors thing again? | 18:24 |
| clarkb | notmyname: want to make sure that we don't forget about you on that and figure out something that will work | 18:24 |
| mordred | clarkb, fungi, jeblair: ^^ thoughts on non-forge split-out modules? | 18:24 |
| clarkb | mordred: r10k | 18:24 |
| msabramo | mgagne: Want to review https://review.openstack.org/87382? Once we get coverage for JJB, then it will be more rewarding to add tests to JJB | 18:24 |
| bodepd | mordred: have you seen librarian-puppet-simple :) | 18:24 |
| clarkb | librarian is poo | 18:25 |
| notmyname | clarkb: I'm here. | 18:25 |
| bodepd | (shameless plug :) ) | 18:25 |
| clarkb | but finch swaers r10k is good | 18:25 |
| bodepd | yeah, that is the real answer. I find it to be a little opinionated and complicated. | 18:25 |
| jesusaurus | clarkb: what dont you like about librarian? (ive never used it) | 18:25 |
| mordred | bodepd: I have not - but I think each time we try to use things that aren't our shell script, things go crazy | 18:25 |
| mordred | bodepd: however, our module install is run from a shell script - so conceivably one could do anythign in there | 18:25 |
| clarkb | notmyname: thinking back to that paste that jeblair made. Is that something that won't work because individuals will need to sign up themselves? | 18:25 |
| bodepd | jesusaurus: he's referring to librarian-puppet | 18:25 |
| fungi | mordred: seems fine, to me the bigger issue is separating the modules out so they can have their own life cycle separate from the config project. where they get published is more of an implementation detail | 18:26 |
| bodepd | jesusaurus: it's dep resolution code sucks and pretty much makes it impossible to use | 18:26 |
| clarkb | also it depends on puppet3 | 18:26 |
| clarkb | which means you lose | 18:26 |
| bodepd | clarkb: are you guys not using Puppet 3? | 18:26 |
| mordred | nope | 18:26 |
| clarkb | bodepd: not yet | 18:26 |
| mordred | we're slowly starting to get there | 18:26 |
| mordred | it's a big task | 18:26 |
| clarkb | slowly moving towards that but there isn't much puppet 3 does that is compelling neough to move other than new distro has it | 18:26 |
| clarkb | see python | 18:27 |
| jesusaurus | speaking of dep resolution, how will we deal with nibalizer's nondep puppetboard stuff with something other than our shell script? | 18:27 |
| notmyname | clarkb: I don't know. I think people signing up could be very good, and perhaps the existing patch would work well. I was hoping for something slightly visually different, but the self-sign-up isn't bad | 18:27 |
| bodepd | integrated hiera is killer. Also, the plugin system refactor stuff is pretty awesome | 18:27 |
| fungi | well, and new puppet plays poorly with old puppet, which makes piecemeal upgrades with a puppet master a bit of a pain | 18:27 |
| clarkb | notmyname: ok. as far as visuals go (or UX) what were you thinking? | 18:27 |
| notmyname | clarkb: suffice to say, it's just an idea right now, and one I'll be working on over the next couple of weeks. so I don't know what will work best :-) | 18:27 |
| mordred | jesusaurus: I think we should just keep using our script :) | 18:27 |
| bodepd | jesusaurus: I don;t have enough context. What is the issue? | 18:28 |
| notmyname | clarkb: I was hoping for another column on the main review page eg next to the author that would be the name of the core sponsor | 18:28 |
| bodepd | mordred: I'm ok with that. Perhaps have an install from github variable? | 18:28 |
| notmyname | clarkb: and also therefore able to sort and search on that | 18:28 |
| mordred | bodepd: yeah. I think that would be fine tbh | 18:28 |
| clarkb | notmyname: I see. That is technically hard beacuse gerrit | 18:28 |
| clarkb | notmyname: you can search using the proposed method though | 18:28 |
| nibalizer | https://review.openstack.org/#/c/83678/ is my patch to add a p3 master | 18:28 |
| notmyname | clarkb: right | 18:28 |
| *** crc32 has joined #openstack-infra | 18:28 | |
| *** rbowen has quit IRC | 18:29 | |
| clarkb | fungi: why doesn't docker work on precise nodes? | 18:29 |
| jesusaurus | bodepd: i dont recall why it was needed ( nibalizer can hopefully fill in that bit ) but we install nibalizer-puppetboard without dependency resolution because of varying versions of vcsrepo in different places | 18:29 |
| nibalizer | yea so i wrote a module that uses vcsrepo | 18:29 |
| jesusaurus | nibalizer: what was the deal there? | 18:29 |
| notmyname | clarkb: also, FYI the core sponsor thing will be getting secondary priority (as will everything else) since the storage polices patches have just been proposed and we're all focusing on those until they land | 18:29 |
| nibalizer | but it uses puppetlabs-vcserpo and openstack-infra/config uses openstack-vcsrepo | 18:29 |
| nibalizer | which is just a fork of vcsrepo | 18:30 |
| clarkb | notmyname: roger good to know. I will probably let this sit and ferment a bit then | 18:30 |
| nibalizer | but the 'puppet module install <modname>' does try to do recursive dependency resolution | 18:30 |
| fungi | clarkb: i believe docker has some specific kernel version and cgroups support requirements not met on our current precise nodes? | 18:30 |
| nibalizer | so we have one pass where we install modules with dependencies | 18:30 |
| bodepd | I find that I generally don't want to use the built in dep resolution b/c I wind up changing things | 18:30 |
| *** nati_ueno has joined #openstack-infra | 18:30 | |
| nibalizer | and one pass where we install modules without dependencies | 18:30 |
| fungi | clarkb: could be that one of the newer kernels available for precise would solve it though | 18:30 |
| bodepd | librarian-puppet-simple is librarian-puppet without dependencies | 18:30 |
| mordred | heh. nice | 18:31 |
| clarkb | fungi: ah ok, I was pretty sure docker works fine on precise but it may require those newer kernels | 18:31 |
| nibalizer | its almost like we need to go by to the 'provides' or 'features' stuff thats already built into the type/provider system | 18:31 |
| clarkb | bodepd: without puppet3 too? | 18:31 |
| mordred | nibalizer: your patch need fixing, btw | 18:31 |
| bodepd | clarkb: it doesn't support forge installs (source only) so Puppet is not a dep | 18:31 |
| nibalizer | fungi: yea the party line on that is precise with a backported raring kernel | 18:32 |
| nibalizer | (for docker) | 18:32 |
| nibalizer | at least until trusty came out, not sure what the party line is now | 18:32 |
| *** e0ne has joined #openstack-infra | 18:32 | |
| fungi | nibalizer: right, when i talked to ewindisch about it (can't remember now if it was at the summit or before) consensus seemed to be that we'd be better off targeting trusty slaves for docker jobs | 18:33 |
| nibalizer | mordred: you mean it has rebase changes? that doesn't suprise me, its like weeks and weeks old | 18:33 |
| mordred | nibalizer: well, also it failed jenkins | 18:33 |
| nibalizer | mordred: hrm | 18:33 |
| *** marcoemorais has quit IRC | 18:33 | |
| mordred | oh - and merge conflicts | 18:33 |
| nibalizer | the failure on the yaml looks like dns to me | 18:33 |
| clarkb | fungi: ya that is partially what prompted my question because there was a lot of interest in trusty for reasons I didn't quite grasp | 18:33 |
| mordred | nibalizer: actually, there is something I had in a different patch that needs to go in here... | 18:34 |
| nibalizer | 2014-05-28 08:04:54.027 | fatal: unable to connect to git.openstack.org: | 18:34 |
| *** marcoemorais has joined #openstack-infra | 18:34 | |
| nibalizer | 2014-05-28 08:04:54.027 | git.openstack.org: Temporary failure in name resolution | 18:34 |
| mordred | nibalizer: mind if I edit it real quick? | 18:34 |
| nibalizer | mordred: yea we need to sync up on the trusty stuff | 18:34 |
| nibalizer | mordred: please do! | 18:34 |
| nibalizer | but of course since it needs to rebase it might geet ugly | 18:34 |
| anteaya | jeblair fungi clarkb mordred SergeyLukjanov pleia2 zaro Ajaeger here is the first draft of the template for the programs template: https://wiki.openstack.org/wiki/Template:ProgramsThirdPartySubst | 18:35 |
| anteaya | let me know what I am missing and what I need to clarify | 18:35 |
| clarkb | Ajaeger: looking at the log levels tx extraction now | 18:36 |
| * Ajaeger hands on the mastery degree of wiki templates to anteaya ! | 18:36 | |
| *** dstanek_zzz is now known as dstanek | 18:37 | |
| anteaya | ha ha ha | 18:37 |
| Ajaeger | clarkb: great, thanks! | 18:37 |
| clarkb | Ajaeger: I am a bit worried that this appears to require people writing code to use different functions for each level | 18:37 |
| anteaya | thanks Ajaeger :D | 18:37 |
| clarkb | Ajaeger: do we expect them to do the right thing there? I suppose this is just going to be educational | 18:37 |
| Ajaeger | clarkb, ask dhellmann - he did that kind of work already | 18:37 |
| clarkb | Ajaeger: ok | 18:37 |
| krotscheck | nibalizer: You’re here! Im going to start working on puppet-storyboard, mind if I ping you for a review later? | 18:37 |
| Ajaeger | clarkb: yeah educational | 18:37 |
| nibalizer | krotscheck: yea! | 18:38 |
| Ajaeger | clarkb: here's what's done already https://blueprints.launchpad.net/oslo/+spec/log-messages-translation-domain | 18:38 |
| krotscheck | nibalizer: Thanks! | 18:38 |
| Ajaeger | already in icehouse | 18:38 |
| hogepodge | Can zuul target different gates for different branches? | 18:38 |
| Ajaeger | clarkb: The only missing part is actual setting up the jobs | 18:38 |
| krtaylor | anteaya, looks good | 18:39 |
| Ajaeger | clarkb: you discussed this in December with dhellmann ;) | 18:39 |
| hogepodge | i.e. have stackforge/project-name target master and stackforge/project-name/branchname target a branch? | 18:39 |
| Ajaeger | and proposed some of this | 18:39 |
| Ajaeger | clarkb: see also http://lists.openstack.org/pipermail/openstack-i18n/2014-May/000631.html | 18:40 |
| clarkb | Ajaeger: ya I remember discussing it I just don't think we got into specifics | 18:40 |
| fungi | anteaya: is the "program mission statement" supposed to be the mission statement for the program cut and pasted, or should it just be a link to where the mission statement for the program is actually published already? (or transcluded from another wiki page?) if it's duplicated, it may fall out of date over time and diverge | 18:40 |
| Ajaeger | clarkb: might be ;) | 18:41 |
| Ajaeger | anteaya: Quantity - do you want an exact number or something like "> 50" ? | 18:41 |
| clarkb | Ajaeger: in any case +2 we can see how it turns out | 18:42 |
| krtaylor | Ajaeger, my question too, and entities are? | 18:42 |
| krtaylor | anteaya, ^^^ | 18:42 |
| krtaylor | clarkb, yes, we have a script that takes care of swift log upload for PowerKVM | 18:42 |
| fungi | hogepodge: i'm not sure what you mean by "different gates" but you can definitely give each job name a pattern of branches where it should be run or not run | 18:42 |
| Ajaeger | clarkb: thanks! We can change the msgmerge as followup if you like... | 18:43 |
| anteaya | krtaylor: thanks | 18:43 |
| anteaya | fungi: good point | 18:43 |
| Ajaeger | clarkb: there're some more cleanup patches in the queue for those scripts ;) | 18:43 |
| mgagne | hogepodge: pipelines look to apply to all branches but as fungi pointed out, you can restrict branches against which the jobs will be run | 18:43 |
| anteaya | fungi: yeah, so a link would probably be best, I just hope they click it | 18:43 |
| hogepodge | fungi: We’re talking about how to drop support for Puppet 2.7 in the coming year, so at some point we want to drop that gate from master, but leave it in place for the stable branches. | 18:44 |
| fungi | hogepodge: for example http://git.openstack.org/cgit/openstack-infra/config/tree/modules/openstack_project/files/zuul/layout.yaml#n606 shows the "gate-swift-tox-func" job running on any branch which is not stable/havana or stable/icehouse | 18:44 |
| anteaya | Ajaeger: a guesstimate is fine, more than 5 would work for me | 18:44 |
| hogepodge | Ok, I see. Thanks. | 18:44 |
| anteaya | the point is that every third party person thinks they are the only one | 18:44 |
| Ajaeger | anteaya: then say it ;) | 18:44 |
| anteaya | say what? | 18:44 |
| fungi | hogepodge: or http://git.openstack.org/cgit/openstack-infra/config/tree/modules/openstack_project/files/zuul/layout.yaml#n465 runs "openstack-misc-guides-ja" only on branches named master | 18:44 |
| bodepd | jesusaurus: yeah, I found the code that related to it. I'm going to add a SOURCE_MODULES variable | 18:45 |
| fungi | hogepodge: tons of examples in there | 18:45 |
| bodepd | nibalizer: if I add a variable for installing from source and without deps, can I remove the NODEP variable and just have it install from source (based on a revision?) | 18:45 |
| anteaya | I am using entites to mean collection of folks representing a third party interest | 18:45 |
| *** arnaud has joined #openstack-infra | 18:45 | |
| anteaya | I can't say account, because they might not have an account yet | 18:45 |
| anteaya | I'm open to a better word than entities | 18:46 |
| Ajaeger | anteaya: change "Quantity: How many Third Party entities interact with your Program?" to "Quantity: How many Third Party entities interact with your Program (order of magnitude number is fine)? | 18:46 |
| jesusaurus | bodepd: do you plan on installing from a sha, or from a git tag, or what? | 18:46 |
| nibalizer | bodepd: whats the context? the install_modules.sh ? | 18:46 |
| Ajaeger | anteaya: or something like that | 18:46 |
| jesusaurus | nibalizer: yeah, install_modules.sh | 18:46 |
| anteaya | Ajaeger: go for it | 18:46 |
| anteaya | also while you are at it, can you show me how to get multi line in a value? | 18:46 |
| anteaya | for instance the meeting links aren't all going to have space on the same line | 18:47 |
| fungi | anteaya: we've used the term "organizations" to this purpose in the past as well | 18:47 |
| anteaya | they each should have a line of their own | 18:47 |
| anteaya | organizations, hmmmm | 18:47 |
| nibalizer | bodepd: i think thats fine | 18:47 |
| bodepd | jesusaurus: I was going to support all. They just work with git checkout | 18:47 |
| anteaya | well that gives them more credit than I have observed, but okay organizations might be a better fit | 18:48 |
| nibalizer | i'm assuming you want to install a module from source and don't want three loops so just change how puppetboard get installs? | 18:48 |
| fungi | anteaya: it's an optimistic term ;) | 18:48 |
| bodepd | nibalizer: yep, that is exactly the context I did not provide :) | 18:48 |
| krtaylor | anteaya, but I am still wondering what interacting means in this context of quantity - what is this trying to document? | 18:48 |
| anteaya | something to hope for | 18:48 |
| nibalizer | bodepd: do et, post review, i will +1 | 18:48 |
| nibalizer | bodepd: also drinks soon is a moral imperative | 18:48 |
| anteaya | we expect you to test your code | 18:48 |
| anteaya | and when we interact with you we will tell you that | 18:49 |
| Ajaeger | krtaylor: It's a different story if you think you're the first 3rd party driver to work with the project - or if you one of 60 - and those others work just fine ;) | 18:49 |
| anteaya | over and over an over | 18:49 |
| anteaya | so heads up | 18:49 |
| *** _nadya_ has quit IRC | 18:49 | |
| *** _nadya_ has joined #openstack-infra | 18:49 | |
| anteaya | for some reason these organizations feel they can do an end run around the requirements | 18:50 |
| anteaya | if they can just find the right person | 18:50 |
| Ajaeger | I'm leaving now. If somebody wants to give https://review.openstack.org/#/c/95377/ a review today, I would be gratefull. | 18:50 |
| openstackgerrit | A change was merged to openstack-infra/jenkins-job-builder: Add tox "coverage" target https://review.openstack.org/87382 | 18:50 |
| *** Ajaeger has quit IRC | 18:50 | |
| anteaya | so every program needs to tell them, we expect you to test your code | 18:50 |
| openstackgerrit | Monty Taylor proposed a change to openstack-infra/config: Add node def for puppet3 master https://review.openstack.org/83678 | 18:50 |
| mordred | nibalizer: ^^ | 18:51 |
| krtaylor | anteaya, are you addressing my question about quantity? | 18:51 |
| *** _nadya_ has quit IRC | 18:51 | |
| anteaya | yes interacting means in this context of quantity | 18:51 |
| anteaya | that was the question I was addressing | 18:52 |
| anteaya | have I missed the mark? | 18:52 |
| *** hashar has joined #openstack-infra | 18:52 | |
| clarkb | fungi: change 95935, its actually a bit harder to setup in log_processor because the log processors using logging.basicConfig() which does not appear to have a way to change the handler to WatchedFileHandler | 18:52 |
| krtaylor | anteaya, hehheh, I still don't understand what interacting means | 18:52 |
| anteaya | because you are doing it | 18:52 |
| fungi | clarkb: oh, ugh. +2 then | 18:52 |
| anteaya | it is like explaining swimming to a fish | 18:53 |
| krtaylor | lol | 18:53 |
| anteaya | think of the folks in the cinder meeting | 18:53 |
| clarkb | fungi: I think we should make the change, its just not a simple config change though. Will require MOARCODE | 18:53 |
| anteaya | the guy who when told that nightly builds was insufficient, changed his mind and talked about weekly builds | 18:53 |
| anteaya | remember that guy? | 18:54 |
| anteaya | that guy is going to read this page that cinder fills out | 18:54 |
| krtaylor | anteaya, so you mean number of drivers in your project that are doing 3rd party testing? | 18:54 |
| Alex_Gaynor | clarkb: FYI, I've pinged our racskpace internal folks about the service catalog issue we discussed last week | 18:54 |
| clarkb | Alex_Gaynor: great! | 18:54 |
| anteaya | yes | 18:54 |
| anteaya | number of accounts doing thrid party testing | 18:55 |
| anteaya | number of organizations that are | 18:55 |
| anteaya | or should be | 18:55 |
| anteaya | it doesn't have to be exact | 18:55 |
| anteaya | it just has to empireically state | 18:55 |
| anteaya | you are not the only person dealing with this project | 18:55 |
| fungi | Alex_Gaynor: any indication it might get worked on? | 18:55 |
| krtaylor | anteaya, so this template is meant to be used at a project level - nova, cinder, neutron, etc | 18:55 |
| anteaya | they still won't get the message | 18:55 |
| anteaya | yes | 18:55 |
| anteaya | this template is | 18:55 |
| anteaya | for organizations to consume | 18:56 |
| * krtaylor has a lightbulb moment | 18:56 | |
| anteaya | they don't read anything else anyway | 18:56 |
| anteaya | I can only hope they will read this | 18:56 |
| krtaylor | hehheh, so pessimistic | 18:56 |
| anteaya | realistic | 18:56 |
| krtaylor | sad, but true | 18:57 |
| anteaya | spent the last release in -neutron | 18:57 |
| anteaya | I have no illusions | 18:57 |
| nibalizer | mordred: whats #NB mean when its in a file with a little note? | 18:57 |
| *** shardy is now known as shardy_afk | 18:57 | |
| mgagne | nibalizer: Nota bene? :P | 18:59 |
| openstackgerrit | Andreas Jaeger proposed a change to openstack-infra/config: Remove usage of docbook-properties.sh https://review.openstack.org/94227 | 18:59 |
| nibalizer | oh | 19:00 |
| nibalizer | mgagne: thanks | 19:00 |
| *** jp_at_hp has quit IRC | 19:00 | |
| openstackgerrit | K Jonathan Harker proposed a change to openstack-infra/gitdm: Add Jonathan Harker https://review.openstack.org/93593 | 19:01 |
| *** atiwari has quit IRC | 19:02 | |
| *** esker has quit IRC | 19:03 | |
| openstackgerrit | A change was merged to openstack-infra/config: Remove duplication in translation scripts https://review.openstack.org/96142 | 19:03 |
| *** Sukhdev has joined #openstack-infra | 19:04 | |
| openstackgerrit | A change was merged to openstack-infra/config: cleaning up index.rst file https://review.openstack.org/96102 | 19:05 |
| *** dims has joined #openstack-infra | 19:07 | |
| openstackgerrit | A change was merged to openstack-infra/config: Enabled pylint job for manila project https://review.openstack.org/95765 | 19:07 |
| * mordred is baffled about people continuing to use pylint | 19:08 | |
| fungi | mordred: apparently there are still things it's able to catch which pyflakes does not due to the kind of analysis it performs | 19:09 |
| *** e0ne has quit IRC | 19:10 | |
| openstackgerrit | A change was merged to openstack-infra/config: add python coverage job to check pipeline for JJB. https://review.openstack.org/96198 | 19:10 |
| *** e0ne has joined #openstack-infra | 19:10 | |
| openstackgerrit | A change was merged to openstack/requirements: Remove openstack/database-api from list of projects https://review.openstack.org/94722 | 19:11 |
| *** e0ne has quit IRC | 19:14 | |
| *** e0ne_ has joined #openstack-infra | 19:14 | |
| *** andreykurilin has joined #openstack-infra | 19:15 | |
| *** _nadya_ has joined #openstack-infra | 19:15 | |
| mordred | clarkb, fungi: dhellmann fixed the upload wheels patch: https://review.openstack.org/#/c/56760/ | 19:17 |
| mordred | SergeyLukjanov and I both +2'd it - but given the nature of the patch, a couple of more eyes might be nice | 19:17 |
| clarkb | mordred: ok | 19:17 |
| SergeyLukjanov | mordred, ack | 19:17 |
| clarkb | note I will not approve as many things nowish because dentist at 2 | 19:18 |
| clarkb | but I will review | 19:18 |
| mordred | yah | 19:18 |
| openstackgerrit | A change was merged to openstack-infra/config: Update get-pip URL to the new location https://review.openstack.org/92225 | 19:18 |
| mordred | I think this is a "make sure someone can babysit" | 19:18 |
| clarkb | k | 19:18 |
| *** chuckC has quit IRC | 19:19 | |
| *** e0ne_ has quit IRC | 19:21 | |
| *** e0ne has joined #openstack-infra | 19:22 | |
| clarkb | mordred: wheels are zip files too? | 19:24 |
| clarkb | mordred: `file -b foo.whl` will say zip in there? | 19:24 |
| dhellmann | mordred: That patch has to +2; I think SergeyLukjanov didn't realize I reworked the latest patch it so your +2 was ok? | 19:24 |
| *** arnaud has quit IRC | 19:24 | |
| mordred | clarkb: yup | 19:25 |
| dhellmann | sigh: s/to/2/ | 19:25 |
| mordred | mordred@camelot:~/src/stackforge/gertty$ file -b /home/mordred/src/openstack-dev/pbr/dist/pbr-0.5.23.36.g00fbbd1-py2.py3-none-any.whl | 19:25 |
| mordred | Zip archive data, at least v2.0 to extract | 19:25 |
| SergeyLukjanov | dhellmann /me trying to realize, it's re https://review.openstack.org/#/c/56760/ ? | 19:26 |
| clarkb | mordred: well it does file -b | grep zip | 19:27 |
| dhellmann | SergeyLukjanov: yes, mordred's name is still on the patch but the latest patchset was my rebase -- I don't know how you treat that here, so maybe your +2 without the A was normal :-) | 19:27 |
| clarkb | which may not match that output | 19:27 |
| dhellmann | SergeyLukjanov: and clarkb seems to have found a potential issue | 19:27 |
| openstackgerrit | Jeremy Stanley proposed a change to openstack-infra/jeepyb: Wait for ACL creation in manage-projects https://review.openstack.org/94684 | 19:27 |
| SergeyLukjanov | dhellmann, I've not approved it to see more reviews for it | 19:27 |
| SergeyLukjanov | dhellmann :) | 19:28 |
| *** pelix has quit IRC | 19:28 | |
| dhellmann | SergeyLukjanov: yep, that's ok | 19:28 |
| *** W00die has quit IRC | 19:29 | |
| *** dangers_away is now known as dangers | 19:30 | |
| clarkb | dhellmann: doing grep -i there is fine | 19:30 |
| clarkb | dhellmann: mordred: why are we adding pypi-jobs to glance, horizon, et al? | 19:31 |
| clarkb | that seems like a noop beacuse they don't end up in the layout file | 19:31 |
| mordred | clarkb: because - part of the "start uploading wheels" discussion also was about starting to upload releases of the server projects too | 19:31 |
| clarkb | mordred: right but this change doesn't do that | 19:31 |
| mordred | clarkb: hrm. good point | 19:32 |
| openstackgerrit | Jeremy Stanley proposed a change to openstack-infra/jeepyb: Wait for ACL creation in manage-projects https://review.openstack.org/94684 | 19:32 |
| mordred | probably should split that into a separate patch | 19:32 |
| clarkb | ++ | 19:32 |
| fungi | mordred: also, what's the plan for server projects where the name is actually something else entirely on pypi? | 19:32 |
| *** nati_ueno has quit IRC | 19:32 | |
| dhellmann | do any of the validation tools in this repository tell me that I have jobs defined that do not appear in the layout? | 19:32 |
| clarkb | dhellmann: no, it only checks the inverse thing that jobs in layout are defined in jenkins | 19:33 |
| fungi | mordred: for example, https://pypi.python.org/pypi/Keystone | 19:33 |
| clarkb | the diff could probably be made strict when we check | 19:33 |
| dhellmann | clarkb: ok, I just thought I'd check (see some of the changes in the patches after this one) | 19:33 |
| dhellmann | so I should remove the pypi jobs from the server projects? | 19:34 |
| mordred | dhellmann: yeah. | 19:34 |
| mordred | fungi: oh for the love of pete | 19:34 |
| clarkb | they won't hurt anything but it is confusing | 19:34 |
| clarkb | mordred: hahaha | 19:34 |
| fungi | mordred: noticed this the other day with https://pypi.python.org/pypi/Tempest as well | 19:35 |
| clarkb | use an openstack prefix for everything? | 19:35 |
| clarkb | openstack-keystone, openstack-tempest | 19:35 |
| mordred | yeah - but we'd need to change that in the setup.cfg too | 19:35 |
| mordred | this may be more trouble than it's worth | 19:35 |
| *** wenlock_ has joined #openstack-infra | 19:38 | |
| *** W00die has joined #openstack-infra | 19:40 | |
| openstackgerrit | Doug Hellmann proposed a change to openstack-infra/config: Create and upload wheels https://review.openstack.org/56760 | 19:42 |
| openstackgerrit | Doug Hellmann proposed a change to openstack-infra/config: Add zuul template for rtfd jobs https://review.openstack.org/95825 | 19:42 |
| openstackgerrit | Doug Hellmann proposed a change to openstack-infra/config: Make it possible to run zuul layout test locally https://review.openstack.org/95783 | 19:42 |
| dhellmann | oops, that's wrong, hang on | 19:43 |
| clarkb | dhellmann: looks like you removed pypi-jobs from things that have them? in any case hanging on | 19:44 |
| openstackgerrit | A change was merged to openstack-infra/config: Remove usage of docbook-properties.sh https://review.openstack.org/94227 | 19:44 |
| dhellmann | clarkb: I removed the pypi jobs from anything that looked like a server project, as I thought had been requested? | 19:45 |
| clarkb | dhellmann: no it had been requested that you remove it from the servers that you added them too | 19:45 |
| clarkb | dhellmann: anything with existing jobs should keep them. We just don't want to be adding pypi-jobs to servers in this change | 19:45 |
| dhellmann | clarkb: ok, there are a bunch of jobs for publishing things that aren't on pypi | 19:46 |
| clarkb | dhellmann: but if those things don't use pypi-jobs then I guess it doesn't hurt to clean it up | 19:47 |
| mordred | hacking needs to stay at least | 19:48 |
| dhellmann | clarkb: ok, I'm going to restore them and just remove the ones that were being added by this patch originally | 19:48 |
| mordred | dhellmann: ++ | 19:48 |
| dhellmann | mordred: yeah, that was my original oops | 19:48 |
| mordred | then I think you're right - we shoudl go back through and clean house | 19:48 |
| *** wenlock has quit IRC | 19:49 | |
| *** otherwiseguy has joined #openstack-infra | 19:49 | |
| *** wenlock_ has quit IRC | 19:50 | |
| *** wenlock has joined #openstack-infra | 19:52 | |
| *** markmc has quit IRC | 19:52 | |
| openstackgerrit | Doug Hellmann proposed a change to openstack-infra/config: Create and upload wheels https://review.openstack.org/56760 | 19:54 |
| openstackgerrit | Doug Hellmann proposed a change to openstack-infra/config: Add zuul template for rtfd jobs https://review.openstack.org/95825 | 19:54 |
| openstackgerrit | Doug Hellmann proposed a change to openstack-infra/config: Make it possible to run zuul layout test locally https://review.openstack.org/95783 | 19:54 |
| clarkb | ok I am +2 on 56760 now, but can't babysit until later today (maybe tomorrow is a good time to babysit?) | 19:57 |
| msabramo | fungi: thanks for reviewing at merging https://review.openstack.org/87382 — that lets us see our coverage for JJB | 19:58 |
| *** shardy_afk is now known as shardy | 19:58 | |
| clarkb | jeblair: does diffing the infra-specs repo initial commit in gertty crash gertty for you? | 19:59 |
| openstackgerrit | Anita Kuno proposed a change to openstack-infra/config: The process for making a Third-Party CI voting needs docs https://review.openstack.org/93562 | 20:01 |
| *** chuckC has joined #openstack-infra | 20:03 | |
| *** Ryan_Lane1 has joined #openstack-infra | 20:03 | |
| clarkb | jeblair: also, anything new that would help debug the gear stuff? maybe debugging on the nodepool side? | 20:04 |
| *** Ryan_Lane has quit IRC | 20:05 | |
| *** nati_ueno has joined #openstack-infra | 20:06 | |
| *** jaypipes has quit IRC | 20:08 | |
| *** rlandy has quit IRC | 20:09 | |
| *** arnaud has joined #openstack-infra | 20:10 | |
| dhellmann | clarkb, mordred : here's a list of projects that are in projects.yaml but not on pypi using the same name: http://paste.openstack.org/show/81929/ | 20:10 |
| *** nati_ueno has quit IRC | 20:11 | |
| clarkb | dhellmann: thanks. Feel free to push a change to clean that up or I can get around to it post dentist | 20:11 |
| dhellmann | clarkb, mordred : some of those look like things we want to be publishing | 20:11 |
| jeblair | clarkb: not sure about gertty; i know there's at least a bug regarding changes with comments on the commit message; i think there's one other i haven't had time to track down | 20:12 |
| *** radez is now known as radez_g0n3 | 20:12 | |
| jeblair | clarkb: i'm pretty confident i understand what's going on with zuul/gear/nodepool. did you see the last? | 20:12 |
| openstackgerrit | Ryan Petrello proposed a change to openstack-infra/config: Gate a few new projects that are using pecan. https://review.openstack.org/96247 | 20:12 |
| clarkb | jeblair: I don't think I saw the last unless it was nodepool being unable to read fast enough | 20:12 |
| clarkb | I skimmed sb looking but didn't see muhc | 20:13 |
| jeblair | clarkb: that's it. :) | 20:13 |
| clarkb | ok | 20:13 |
| jogo | this file search support in gerrit is awesome | 20:13 |
| jeblair | clarkb: that blocks geard | 20:13 |
| jeblair | clarkb: which blocks all its clients | 20:13 |
| clarkb | jeblair: should each geard Connection maintain a write buffer that is pushed into the tcp stack asynchronously? | 20:13 |
| jeblair | clarkb: i've been heads down working on a solution | 20:13 |
| clarkb | in that case I will leave you to it | 20:13 |
| *** _nadya_ has quit IRC | 20:14 | |
| openstackgerrit | A change was merged to openstack-infra/git-review: Fix typo in manpage s/gireview/gitreview/ https://review.openstack.org/96125 | 20:16 |
| openstackgerrit | Mark Sturdevant proposed a change to openstack/requirements: Remove hplefthandclient from global-requirements https://review.openstack.org/95974 | 20:16 |
| *** mriedem has quit IRC | 20:17 | |
| clarkb | dhellmann: comment for you on your gertty change | 20:18 |
| *** mriedem has joined #openstack-infra | 20:18 | |
| openstackgerrit | Doug Hellmann proposed a change to openstack-infra/config: Ignore .egg directories https://review.openstack.org/96250 | 20:19 |
| openstackgerrit | Doug Hellmann proposed a change to openstack-infra/config: Remove unused pypi jobs from projects https://review.openstack.org/96251 | 20:19 |
| dhellmann | clarkb: ^^ cleanup change for you :-) | 20:19 |
| mordred | dhellmann: blast. when we did the renames the other day, we should have put oslo.version in the attic | 20:20 |
| Alex_Gaynor | dstufft: ping | 20:21 |
| dhellmann | clarkb: replied to your gertty comment | 20:22 |
| dhellmann | mordred: I keep meaning to ask you about oslo.version | 20:22 |
| dhellmann | we're going to need a place for versionutils, would it make sense to put it there? | 20:22 |
| *** r-daneel has joined #openstack-infra | 20:22 | |
| dhellmann | mordred: if nothing in oslo.version is useful right now, I could erase the contents and reuse the lib name | 20:23 |
| clarkb | dhellmann: ooohhhh so all of my projects are unreviewed and I didn't realize it was three states there | 20:24 |
| clarkb | dhellmann: what do you think about default for unreviewed, dark gray for subscribed, and can we do a different background + default for unsubscribed? | 20:25 |
| clarkb | or something like that | 20:25 |
| dhellmann | clarkb: I'm not 100% sure. I really went in and poked at things until there were no blank places on my terminal, so some of these states might not even be used. | 20:25 |
| clarkb | dhellmann: maybe I remove my -1 and let jeblair chime in | 20:25 |
| clarkb | I will do that | 20:25 |
| *** maxbit has quit IRC | 20:25 | |
| dhellmann | clarkb: I'm OK with a different background, but I have a pretty hard time with reading light on dark | 20:26 |
| dhellmann | clarkb: in a perfect world we would just read this from a configuration file | 20:26 |
| clarkb | dhellmann: ya, it is configurable now iirc | 20:26 |
| dstufft | Alex_Gaynor: pong | 20:26 |
| dhellmann | clarkb: oh, I wasn't able to figure that out | 20:26 |
| clarkb | but I think by default it should support light and dark terminals | 20:26 |
| clarkb | dhellmann: it just merged | 20:26 |
| Alex_Gaynor | dstufft: So, do you remember how when Django upladed pre-release wheels on pypi it broke stuff? | 20:27 |
| dhellmann | clarkb: ah! if that's the case, we can probably just abandon my change, unless I'm actually going in something like the right direction | 20:27 |
| dstufft | Alex_Gaynor: I remember that it broke stuff, I don't recall how | 20:27 |
| Alex_Gaynor | dstufft: Do you remember what the symptom was? | 20:27 |
| clarkb | dhellmann: I think what you are doing is good. light terminals exist and we should support them by default :) | 20:27 |
| Alex_Gaynor | dstufft: I seem to recall it was "pip install Django" broke, but I want to make sure | 20:27 |
| dstufft | Alex_Gaynor: no idea | 20:27 |
| Alex_Gaynor | dstufft: Ok, thanks. I've suggested openstack put off doing that for now. | 20:28 |
| dstufft | Alex_Gaynor: I wasn't around when it broke, and all the info I got from it was that it broke | 20:28 |
| dhellmann | clarkb: ok, I'll wait to hear what jeblair says | 20:28 |
| Alex_Gaynor | dstufft: and whatever broke definitely hasn't been fixed, I assume :-) | 20:28 |
| dstufft | Alex_Gaynor: not unless it was fixed inadvertently | 20:28 |
| dstufft | I have no idea what that was, or if Django did something wrong or what | 20:29 |
| Alex_Gaynor | dstufft: this is packaging, who knows! :-) | 20:29 |
| dstufft | more or less | 20:29 |
| *** e0ne has quit IRC | 20:29 | |
| Alex_Gaynor | thanks | 20:29 |
| *** e0ne has joined #openstack-infra | 20:30 | |
| mordred | dhellmann: what's versionutils? | 20:30 |
| *** terryw has joined #openstack-infra | 20:30 | |
| dhellmann | mordred: deprecation logging stuff: http://git.openstack.org/cgit/openstack/oslo-incubator/tree/openstack/common/versionutils.py | 20:30 |
| *** salv-orlando has joined #openstack-infra | 20:30 | |
| *** otherwiseguy has quit IRC | 20:31 | |
| dhellmann | mordred: I think we've been planning to call the lib oslo.versionutils because I am bad at names | 20:32 |
| *** terryw is now known as otherwiseguy | 20:32 | |
| *** andreaf has quit IRC | 20:32 | |
| *** e0ne has quit IRC | 20:33 | |
| *** ociuhandu has quit IRC | 20:33 | |
| clarkb | fungi: I just approved your pypi-mirror change to stop whitelisting external links | 20:33 |
| fungi | clarkb: yay! | 20:33 |
| fungi | openstack is free of pypi-external dependencies! | 20:34 |
| fungi | well, it already was, but this is clean-up | 20:34 |
| *** maxbit has joined #openstack-infra | 20:36 | |
| lifeless | mordred: ping (pbr python2.6 fail insight needed) | 20:36 |
| mordred | lifeless: ola | 20:36 |
| *** aconrad has joined #openstack-infra | 20:37 | |
| * clarkb & back later | 20:38 | |
| lifeless | mordred: https://review.openstack.org/#/c/94107/2 help | 20:38 |
| mordred | lifeless: yah. so - I don' tknow why it's only failing on 2.6 - but looking at the log output - that pip command sure does have the effect of writing files to the tree and not to the temp dir | 20:39 |
| *** mestery has quit IRC | 20:39 | |
| zaro | mika: did want a jjb release? | 20:39 |
| *** mestery has joined #openstack-infra | 20:40 | |
| *** radez_g0n3 is now known as radez | 20:40 | |
| mordred | lifeless: oh! no, that's not what that's doing -that's installing something into the virtualenv inside of which the test is running | 20:40 |
| lifeless | right, to avoid the easy_install | 20:41 |
| mordred | yeah. but it's installing into the tox venv | 20:41 |
| mordred | Copying pbr_testpackage.egg-info to /home/jenkins/workspace/gate-pbr-python26/.tox/py26/lib64/python2.6/site-packages/pbr_testpackage-0.0-py2.6.egg-info | 20:41 |
| mordred | I believe that is not what we want to do | 20:41 |
| lifeless | doing that prevents easy_install downloading pbr from pypi | 20:42 |
| lifeless | but thats not why we need to do it | 20:42 |
| lifeless | we need to do it because the tests that run setup.py have a setup_requires of pbr | 20:42 |
| mordred | right. | 20:43 |
| lifeless | and if we *don't* do it, we're not using the pbr that is being tested | 20:43 |
| mordred | but this isn't installing pbr | 20:43 |
| mordred | this is installing the testpackage | 20:43 |
| openstackgerrit | A change was merged to openstack-infra/nodepool: Add cloud-user to non-root user list https://review.openstack.org/94643 | 20:43 |
| lifeless | oh f* | 20:43 |
| *** smarcet has quit IRC | 20:44 | |
| lifeless | mordred: it shold be installing the root package, not the test package | 20:45 |
| mordred | yah | 20:45 |
| fungi | clarkb: !!! gerrit ls-members --recursive | 20:47 |
| fungi | clarkb: https://review.openstack.org/Documentation/cmd-ls-members.html | 20:47 |
| fungi | clarkb: tested and it works | 20:47 |
| *** lakshmiS has joined #openstack-infra | 20:47 | |
| *** esker has joined #openstack-infra | 20:48 | |
| clarkb | nice. I didnt realize it had that option | 20:49 |
| lifeless | mordred: http://paste.openstack.org/show/81940/ on 2.7 | 20:49 |
| *** nati_ueno has joined #openstack-infra | 20:50 | |
| openstackgerrit | A change was merged to openstack/requirements: Update projects list for oslo.test renaming https://review.openstack.org/96205 | 20:50 |
| mgagne | I just figured out the real inner working on project ACLs with refs/meta/config in Gerrit. Could the API be used instead to sync the ACL? Is it even possible? | 20:54 |
| fungi | mgagne: depends on what you mean by "the api" and "sync" | 20:55 |
| mgagne | fungi: instead of creating an acl config per project and using jeppyb to update it through git, couldn't the API be used to set those up? | 20:56 |
| *** miqui has quit IRC | 20:56 | |
| fungi | mgagne: perhaps, though that seems like it would make it harder to keep the actual state of the acls collected in a single git repository | 20:57 |
| mgagne | fungi: true, would probably make it harder by having to convert from the Gerrit ACL config format to API calls/format | 20:58 |
| fungi | mgagne: unless there's a bulk-replace option for a project acl, i worry they'd quickly get out of sync | 20:59 |
| mgagne | fungi: right, I can see the diff hell from here :-/ | 20:59 |
| lifeless | incoming | 20:59 |
| openstackgerrit | lifeless proposed a change to openstack-dev/pbr: Allow examining parsing exceptions. https://review.openstack.org/80856 | 20:59 |
| fungi | mgagne: and the current push via git into gerrit is effectively a bulk replace plus sanity checking | 20:59 |
| openstackgerrit | lifeless proposed a change to openstack-dev/pbr: Teach pbr VersionInfo about debian versions. https://review.openstack.org/81074 | 20:59 |
| openstackgerrit | lifeless proposed a change to openstack-dev/pbr: Teach pbr about post versioned dev versions. https://review.openstack.org/80449 | 20:59 |
| openstackgerrit | lifeless proposed a change to openstack-dev/pbr: Use the current pbr for testpackage tests. https://review.openstack.org/94107 | 20:59 |
| openstackgerrit | lifeless proposed a change to openstack-dev/pbr: Add a converter to version_tuples. https://review.openstack.org/80457 | 20:59 |
| openstackgerrit | lifeless proposed a change to openstack-dev/pbr: Break out a common version object from VersionInfo https://review.openstack.org/94108 | 20:59 |
| openstackgerrit | lifeless proposed a change to openstack-dev/pbr: Allow _run_cmd to run commands in any directory. https://review.openstack.org/96264 | 21:00 |
| fungi | mgagne: gerrit upstream has tried to improve acl management by making it more self-service now (there is a feature to allow users to propose acl changes for review and then project owners to approve/merge them) but it doesn't really match our intended management workflow | 21:01 |
| dhellmann | lifeless, mordred : at some point we should talk about using oslo-specs for pbr like we're starting to do for the other oslo libs | 21:01 |
| mordred | dhellmann: ++ | 21:01 |
| mgagne | fungi: ok, I'll check out jeepyb and see what I can do with it | 21:01 |
| mordred | dhellmann: how about now ... | 21:01 |
| mordred | dhellmann: I think we shoudl use oslo-specs for pbr | 21:01 |
| dhellmann | cool. I don't want to derail what lifeless is already doing, but maybe for future changes? | 21:02 |
| mordred | yah. because the barrier to entry for pbr changes should be fairly high at this point | 21:02 |
| dhellmann | right | 21:02 |
| lifeless | dhellmann: is there an oslo-specs now? I'm very happy to write up a spec for this | 21:02 |
| lifeless | getting broad buy-in is one of the action items on this anyway | 21:03 |
| dhellmann | lifeless: yes, we created it just before the summit (you may literally have been on a plane at the time) | 21:03 |
| dhellmann | lifeless: I would be very happy to have one or more spec on this to review :-) | 21:03 |
| dhellmann | like I said, I didn't want to derail work that was well under way | 21:04 |
| *** Sukhdev has quit IRC | 21:04 | |
| signed8bit | Curious as a first timer (https://review.openstack.org/#/c/95160/), what is needed to get to a merged state? clarkb gave a +2 code review but I assume it needs more to continue through? | 21:04 |
| mordred | signed8bit: it needs 2 +2s - lemme look | 21:05 |
| lifeless | so up to and including https://review.openstack.org/#/c/80856/ are nonconentious bugfixes basically | 21:05 |
| lifeless | I will write a spec for the top of the stack | 21:05 |
| signed8bit | Ah thanks mordred | 21:06 |
| *** aysyd has quit IRC | 21:06 | |
| *** wenlock_ has joined #openstack-infra | 21:06 | |
| mordred | signed8bit: and there you go ... | 21:06 |
| *** marcoemorais has quit IRC | 21:07 | |
| openstackgerrit | A change was merged to openstack-infra/gerritbot: Force SSL Configuration Parameter https://review.openstack.org/95160 | 21:07 |
| *** marcoemorais has joined #openstack-infra | 21:07 | |
| fungi | signed8bit: +2 from me as well. i'm not find of the thought of us authenticating over a plaintext fallback | 21:07 |
| fungi | s/find/fond/ | 21:08 |
| signed8bit | mordred: Thanks! | 21:08 |
| mordred | lifeless: +2'd all of those | 21:08 |
| openstackgerrit | Khai Do proposed a change to openstack-infra/jenkins-job-builder: reference tests as examples https://review.openstack.org/96279 | 21:09 |
| *** mfer has quit IRC | 21:09 | |
| openstackgerrit | A change was merged to openstack/requirements: update pycadf req to 0.5.1 https://review.openstack.org/95585 | 21:10 |
| *** ociuhandu has joined #openstack-infra | 21:11 | |
| signed8bit | fungi: Yeah personally I'd have preferred it to not assume SSL for only one port but I didn't want to require a config change for existing deployments | 21:12 |
| anteaya | fungi: line 85: https://etherpad.openstack.org/p/automated-gerrit-account-naming-format | 21:12 |
| fungi | signed8bit: pragmatic | 21:12 |
| anteaya | --recursive is handy | 21:12 |
| fungi | anteaya: indeed it is | 21:12 |
| openstackgerrit | Khai Do proposed a change to openstack-infra/jenkins-job-builder: reference tests as examples https://review.openstack.org/96279 | 21:12 |
| fungi | anteaya: that was, in fact, one of the first groups i tested it on ;) | 21:13 |
| anteaya | cool | 21:13 |
| anteaya | took me a minute to get the syntax correct | 21:13 |
| anteaya | it looks like part of the problem is that username and full name for some accounts are two entirely different things | 21:14 |
| anteaya | not very helpful | 21:14 |
| *** gyee has quit IRC | 21:14 | |
| *** Sukhdev has joined #openstack-infra | 21:15 | |
| anteaya | I still don't know which of the 3 brocade accounts are "Brocade CI" though: https://review.openstack.org/#/c/95138/ | 21:15 |
| anteaya | s/are/is | 21:16 |
| openstackgerrit | Andreas Jaeger proposed a change to openstack-infra/config: Extract translations for log messages https://review.openstack.org/95377 | 21:16 |
| *** BadCub has joined #openstack-infra | 21:18 | |
| *** marcoemorais1 has joined #openstack-infra | 21:19 | |
| *** marcoemorais has quit IRC | 21:20 | |
| *** marcoemorais has joined #openstack-infra | 21:21 | |
| *** esker has quit IRC | 21:23 | |
| *** esker has joined #openstack-infra | 21:24 | |
| *** marcoemorais1 has quit IRC | 21:25 | |
| *** gokrokve has quit IRC | 21:26 | |
| *** gokrokve has joined #openstack-infra | 21:27 | |
| *** gokrokve has quit IRC | 21:30 | |
| davidlenwell | So in the last week I've been getting a lot of these fials.. when it wasn't an issue before.. this commit was just a spec.. so I don't understand how it can fail. https://review.openstack.org/#/c/96249/ | 21:34 |
| anteaya | Zero tests were run. At least one test should have been run. | 21:35 |
| anteaya | add a teset | 21:35 |
| anteaya | test | 21:35 |
| *** david-lyle has quit IRC | 21:35 | |
| davidlenwell | So there is a test.. that does nothing | 21:35 |
| *** signed8b_ has joined #openstack-infra | 21:36 | |
| davidlenwell | nevermind .. I guess there isn't | 21:36 |
| davidlenwell | but what changed in the last week.. that didn't used to block things | 21:37 |
| *** signed8bit has quit IRC | 21:37 | |
| anteaya | what do you want to get from running the python27 tests if there are no tests in the repo? | 21:37 |
| anteaya | and I don't know what changed in the last week in the python27 tests | 21:38 |
| jogo | so I am looking into bumping up the large-ops number, 100 appears to work but only did a few runs. what do you recomend as the next step? | 21:38 |
| jogo | turn large-ops-testing on as silent? | 21:38 |
| jogo | for a week? | 21:38 |
| mgagne | Would there be a way to tell Gerrit to add the -x option when cherry-picking? | 21:39 |
| *** hashar has quit IRC | 21:39 | |
| openstackgerrit | Michael Krotscheck proposed a change to openstack-infra/puppet-storyboard: [WIP] Added initial puppet module for StoryBoard https://review.openstack.org/96290 | 21:39 |
| krotscheck | nibalizer: ^ First pass, still a lot of stuff from puppet-dashboard in the README, but it’s a start. | 21:41 |
| davidlenwell | anteaya: there will be tests in the repo.. but we haven't bothered yet because the requirments have been in flux until now.. right now we're just commiting all the specs .. which will lead to tests. | 21:42 |
| anteaya | davidlenwell: I wasn't going to say anything about all the whitespace errors in that file, since I figured you were planning on correcting them | 21:42 |
| anteaya | but I see that patch is approved for merging | 21:42 |
| anteaya | don't you want to clean up the whitespace errors? | 21:42 |
| davidlenwell | anteaya: I don't consider white space in an rst file an error. | 21:43 |
| anteaya | oh | 21:43 |
| mgagne | forgot what I said, it does already include -x, got misleading info on my end ^^' | 21:43 |
| davidlenwell | in a python file .. absolutely and that will fail pep8.. | 21:43 |
| anteaya | well until you add tests, why don't you stop testing against python27 | 21:43 |
| *** dangers is now known as dangers_away | 21:43 | |
| jogo | fungi: ^ | 21:43 |
| openstackgerrit | James E. Blair proposed a change to openstack-infra/gear: Use non-blocking IO https://review.openstack.org/96294 | 21:44 |
| clarkb | jogo: wasn't it higher than 100 | 21:44 |
| clarkb | jogo: so 100 is effectively lowering it? | 21:44 |
| jogo | clarkb: large-ops-testing was 175 but that didn't work | 21:44 |
| jogo | clarkb: I want to bump large-ops itself | 21:44 |
| jogo | large-ops testing was to find a good testing number | 21:45 |
| anteaya | mgagne: wasn't ignoring you, I didn't know the answer, thanks for sharing it | 21:45 |
| davidlenwell | anteaya: I would do that.. but we've been testing against it for months and it wasn't an issue until last week.. I'll add a few tests now that the api spec is finished. but my curriosity and questions stemed from the change in behavior | 21:45 |
| *** wenlock_ has quit IRC | 21:45 | |
| jogo | err good number that we could bump the value too | 21:45 |
| clarkb | jogo: ah gotcha | 21:45 |
| anteaya | davidlenwell: okay, I don't have the answer | 21:45 |
| jogo | large-ops started at 150 at first but cloud variance forced us to drop it | 21:45 |
| mgagne | in fact, it looks like the line "cherry picked from" is not added if the change you are cherry picking from is not yet merged. | 21:45 |
| jogo | clarkb: so I am thinking a few runs isn't enough data points to confirm the job is stable enough | 21:46 |
| jeblair | clarkb, fungi, mordred: https://review.openstack.org/96294 addresses the issue we saw with zuul/gear/nodepool | 21:46 |
| jogo | clarkb: which is why I was thinking silent queue and I can track it via graphite | 21:46 |
| anteaya | mgagne: cool, I didn't know that | 21:46 |
| clarkb | jogo: wfm | 21:47 |
| jeblair | clarkb, fungi, mordred: it's a substantial change. considering that gear primarily wants to be a client library, and the problem was predominantly with the server, i think we have a few options. | 21:47 |
| *** mrda-away is now known as mrda | 21:47 | |
| jeblair | clarkb, fungi, mordred: we should evaluate that change on whether it improves things for clients and at a reasonable complexity cost | 21:48 |
| *** Kevin_Bishop has joined #openstack-infra | 21:48 | |
| jeblair | clarkb, fungi, mordred: on balance, i think it might make things slightly better for clients in bad network situations or which are moving lots of data around. though it does make client calls like "send work data" non-blocking now. not sure if that's a good thing or not. | 21:48 |
| *** nati_ueno has quit IRC | 21:49 | |
| *** jvrbanac has joined #openstack-infra | 21:49 | |
| *** paul_glass has joined #openstack-infra | 21:49 | |
| jeblair | clarkb, fungi, mordred: i'm not sure whether it's more complicated or not. | 21:49 |
| clarkb | jeblair: client objects can still query the Connection object for its status right? | 21:50 |
| clarkb | so this isn't really fire and forget, it just becomes a bit more work to remember | 21:50 |
| dhellmann | lifeless: that sounds like a good plan for the specs | 21:51 |
| jeblair | clarkb: i don't think i made a reliable way to do that. i think it's more fire and forget. | 21:51 |
| anteaya | davidlenwell: https://review.openstack.org/#/c/92759/ | 21:51 |
| jvrbanac | Howdy all! Is this the right place to ask a question about an error message i'm seeing with OpenStack Gerrit? | 21:51 |
| jeblair | clarkb: so if you are a worker, and you sendWorkComplete and actually sending that fails, i don't think you'd know. | 21:52 |
| jeblair | whereas at the moment, that is synchronous | 21:52 |
| davidlenwell | anteaya: thank you for finding that for me | 21:52 |
| davidlenwell | makes sense now | 21:52 |
| anteaya | welcome | 21:52 |
| *** bcrochet is now known as bcrochet|g0ne | 21:53 | |
| anteaya | jvrbanac: what is the error you are seeing? | 21:53 |
| fungi | jeblair: i wonder whether the calls could be made to optionally block... or would that almost certainly increase complexity? | 21:53 |
| anteaya | jvrbanac: if it is more than two lines, please use paste.openstack.org | 21:54 |
| fungi | jeblair: and yeah, i guess that is a fairly big change, looking at it now | 21:54 |
| jeblair | clarkb, fungi, mordred: so if we decide that non-blocking is an anti-feature for clients, then i think we have two options: | 21:54 |
| jogo | clarkb: thanks | 21:55 |
| jeblair | clarkb, fungi, mordred: a) give geard its own poll loop (so it can do epoll edge-triggered non-blocking io without touching the client bits) | 21:55 |
| jeblair | clarkb, fungi, mordred: b) start hacking on the C server | 21:55 |
| *** maxbit has quit IRC | 21:55 | |
| * fungi likes how "b" is also "c" | 21:56 | |
| anteaya | one big ball of string | 21:56 |
| lifeless | jeblair: FWIW I am of the long-held opinion that all clients should be nonblocking, with blocking a thin shim on top. | 21:56 |
| jvrbanac | anteaya, Two of my devs (paul_glass & Kevin_Bishop) are trying to get setup with Gerrit and git review is telling them they need to have "current contact information"; however, when they attempt to update their information in Gerrit, they get a "Server Error - Cannot store contact information" | 21:56 |
| anteaya | great | 21:56 |
| anteaya | they haven't become foundation members yet | 21:57 |
| anteaya | what instructions are you following? | 21:57 |
| jeblair | clarkb, fungi, mordred: now that i have more or less the whole problem space in my head, i think (a) may be a much smaller change than this one, and generally not insane. | 21:57 |
| *** dhellmann is now known as dhellmann_ | 21:58 | |
| *** mriedem has quit IRC | 21:58 | |
| fungi | anteaya: jvrbanac: yeah, with the recent gerrit upgrade we lost the more descriptive error message we had hacked in there, so it's a little less instructive than it used to be. the faq entry for that is probably | 21:58 |
| *** dhellmann_ is now known as dhellmann | 21:58 | |
| fungi | https://wiki.openstack.org/wiki/CLA-FAQ#When_trying_to_sign_the_new_ICLA_and_include_contact_information.2C_why_am_I.27m_getting_an_error_message_saying_that_my_E-mail_address_doesn.27t_correspond_to_a_Foundation_membership.3F | 21:58 |
| *** dhellmann is now known as dhellmann_ | 21:58 | |
| nibalizer | krotscheck: im a bit confused, is this the code from the existing opentack-infra/config module or a new one? | 21:58 |
| *** wenlock_ has joined #openstack-infra | 21:59 | |
| fungi | jeblair: i think there's still merit in maintaining a pure python implementation of the server to use in tests anyway, so might as well make it usable for production | 21:59 |
| jvrbanac | anteaya, fungi Opps! I didn't see that when I did a quick google search. We'll give that a shot. Thanks! | 21:59 |
| fungi | jvrbanac: the narrative instructions (which list things needing to be done in the proper sequence) are at https://wiki.openstack.org/wiki/How_To_Contribute#If_you.27re_a_developer | 22:00 |
| clarkb | mordred: by the way 1.1 has a new failure mode | 22:01 |
| clarkb | mordred: we are getting 500 responses | 22:01 |
| clarkb | jvrbanac: sure | 22:01 |
| clarkb | jeblair: I think that gear should at least make you aware of that | 22:01 |
| clarkb | jeblair: maybe by togglign a flag on the connection object that can be checked | 22:01 |
| *** gondoi is now known as zz_gondoi | 22:01 | |
| *** ekarlso has quit IRC | 22:01 | |
| clarkb | I am doing a proper review of the change now | 22:01 |
| jeblair | clarkb: okay. as you can see, i consider the entire approach up for debate -- so keep that in mind as you review. :) | 22:02 |
| *** arnaud has quit IRC | 22:03 | |
| fungi | i have skimmed the change, but am going to flee to dinner and try to have a look at it when i get back | 22:03 |
| jvrbanac | fungi, that fixed it. Thanks a ton! Sorry to bug you about that. | 22:04 |
| fungi | jvrbanac: no worries. just glad it wasn't something severely broken we needed to fix! | 22:04 |
| * fungi -> out | 22:05 | |
| *** atiwari has joined #openstack-infra | 22:05 | |
| openstackgerrit | A change was merged to openstack-infra/pypi-mirror: No longer whitelist externally-hosted requirements https://review.openstack.org/94447 | 22:05 |
| *** yamahata has joined #openstack-infra | 22:06 | |
| *** nati_ueno has joined #openstack-infra | 22:07 | |
| *** greghaynes has quit IRC | 22:07 | |
| clarkb | jeblair: if a is much smaller and simpler maybe there is a better place to start? | 22:07 |
| clarkb | jeblair: then we can cut a 1.0 and start working on the less backward compatible client changes? | 22:07 |
| *** greghaynes has joined #openstack-infra | 22:09 | |
| jeblair | clarkb: it's an idea, though we'd end up with client and server being nonblocking, as they are in this change. basically, that's probably the route to go if we want both to be nonblocking and we want the server to be extra awesome. | 22:10 |
| openstackgerrit | Joe Gordon proposed a change to openstack-infra/config: Move large-ops-testing to silent queue for further testing https://review.openstack.org/96299 | 22:11 |
| jogo | that commit message took 10 times longer to write then the patch ^ | 22:11 |
| jeblair | jogo: and it's still longer than 50 chars :P | 22:12 |
| jogo | jeblair: heh | 22:12 |
| *** jgrimm has quit IRC | 22:13 | |
| jogo | on an unrelated note: http://logs.openstack.org/83/86983/3/gate/gate-nova-pep8/20690e0/console.html | 22:15 |
| jogo | is pypi.o.o having issues? | 22:15 |
| *** zehicle_at_dell has joined #openstack-infra | 22:15 | |
| *** bknudson has quit IRC | 22:16 | |
| *** dstanek is now known as dstanek_zzz | 22:19 | |
| *** pdmars has quit IRC | 22:19 | |
| *** dstanek_zzz is now known as dstanek | 22:20 | |
| *** whoops has quit IRC | 22:21 | |
| *** kaisero has quit IRC | 22:21 | |
| openstackgerrit | A change was merged to openstack-infra/elastic-recheck: Add query for 1320617 https://review.openstack.org/94080 | 22:23 |
| *** Kevin_Bishop has quit IRC | 22:24 | |
| *** zehicle_at_dell has quit IRC | 22:25 | |
| *** andreykurilin has quit IRC | 22:26 | |
| *** r-daneel has quit IRC | 22:26 | |
| *** paul_glass has left #openstack-infra | 22:26 | |
| *** ekarlso has joined #openstack-infra | 22:26 | |
| *** greghaynes has quit IRC | 22:27 | |
| *** greghaynes has joined #openstack-infra | 22:28 | |
| clarkb | jeblair: commented on the change. pep8 failed it for a few reasons | 22:28 |
| clarkb | jeblair: I think I am ok with that approach if we address my inline comment about setting a flag on the connection | 22:29 |
| clarkb | or a flag somewhere so that clients/workers can check if things have gone crazy | 22:29 |
| *** UtahDave has quit IRC | 22:29 | |
| *** mburned is now known as mburned_out | 22:29 | |
| clarkb | and maybe make that flag = errno | 22:29 |
| *** weshay has quit IRC | 22:29 | |
| jeblair | clarkb: this is so that clients/workers could see what's going on? | 22:31 |
| *** jp_at_hp has joined #openstack-infra | 22:31 | |
| clarkb | jeblair: ya | 22:31 |
| clarkb | though I was mostly thinking about this from a debugging/logging perspective rather than in code error handling | 22:32 |
| clarkb | and gear should do the logging for us right? | 22:32 |
| *** atiwari has quit IRC | 22:32 | |
| *** lakshmiS has quit IRC | 22:32 | |
| jeblair | clarkb: connection losses are always logged, yeah | 22:32 |
| *** greghaynes has quit IRC | 22:32 | |
| clarkb | so maybe that isn't as useful as I thought it might be | 22:33 |
| *** Ryan_Lane1 is now known as Ryan_Lane | 22:33 | |
| *** Ryan_Lane has quit IRC | 22:33 | |
| *** Ryan_Lane has joined #openstack-infra | 22:33 | |
| *** greghaynes has joined #openstack-infra | 22:34 | |
| *** mrodden has quit IRC | 22:34 | |
| *** ArxCruz has quit IRC | 22:34 | |
| *** rfolco has quit IRC | 22:35 | |
| *** thedodd has quit IRC | 22:35 | |
| *** marcoemorais has quit IRC | 22:36 | |
| *** marcoemorais has joined #openstack-infra | 22:36 | |
| openstackgerrit | craigtracey proposed a change to openstack-infra/config: Add nova-docker to the gerritbot config https://review.openstack.org/96192 | 22:36 |
| *** dangers_away is now known as dangers | 22:37 | |
| *** greghaynes has quit IRC | 22:37 | |
| *** greghaynes has joined #openstack-infra | 22:39 | |
| *** melwitt has quit IRC | 22:39 | |
| *** signed8b_ is now known as signed8bit_ZZZzz | 22:44 | |
| *** signed8bit_ZZZzz is now known as signed8b_ | 22:46 | |
| *** signed8b_ is now known as signed8bit | 22:46 | |
| *** dkliban is now known as dkliban_afk | 22:47 | |
| *** melwitt has joined #openstack-infra | 22:47 | |
| *** 14WAC1CKA has joined #openstack-infra | 22:49 | |
| *** 14WAC1CKA has quit IRC | 22:49 | |
| *** jhesketh has joined #openstack-infra | 22:49 | |
| jhesketh | Morning | 22:50 |
| jeblair | devananda: i posted a reply to your ironic ci thread | 22:51 |
| *** mmaglana has joined #openstack-infra | 22:51 | |
| *** dims has quit IRC | 22:51 | |
| clarkb | jhesketh: o/ | 22:52 |
| *** dims has joined #openstack-infra | 22:52 | |
| clarkb | jhesketh: so zuul is speaking swift now and I think I reviewed your JJB changes to put that into jobs, but I didn't see a job that uses it. Am I just not seeing that change? | 22:52 |
| jhesketh | clarkb: oh cool, thanks | 22:53 |
| jhesketh | but no, no job doing that yet | 22:53 |
| jhesketh | that's next I guess :-) | 22:53 |
| clarkb | ah ok, so I think thats next | 22:53 |
| clarkb | jhesketh: maybe add it to the config jobs | 22:53 |
| clarkb | jhesketh: and leave the scp in place so that we get two copies of logs for now | 22:54 |
| jhesketh | clarkb: btw, could you take a look at my reply to this please https://review.openstack.org/#/c/90883/ | 22:54 |
| clarkb | sure | 22:54 |
| jhesketh | yeah I was going to make up a fake job but that's a good idea | 22:54 |
| *** homeless has quit IRC | 22:54 | |
| clarkb | jhesketh: for 90883 what if we restrict the data available to regular node names | 22:55 |
| clarkb | jhesketh: rather then changing what subnodes know about | 22:55 |
| jhesketh | I'm not sure I follow sorry | 22:55 |
| clarkb | jhesketh: in the comment you mention subnodes don't have the full data set | 22:56 |
| clarkb | jhesketh: but we could go the other way and only allow self.node_id in launchNode | 22:56 |
| clarkb | jhesketh: then it would be consistent with launchSubNode | 22:57 |
| jhesketh | ah right | 22:57 |
| jhesketh | but then the names won't be able to be the same as already exists | 22:57 |
| *** signed8bit has quit IRC | 22:57 | |
| clarkb | oh, does that mean sub node names can't be the same as would exist today if we were using subnodes? | 22:57 |
| *** signed8bit has joined #openstack-infra | 22:57 | |
| *** arnaud has joined #openstack-infra | 22:58 | |
| clarkb | actually it looks like nodeid is all that we use | 22:58 |
| jhesketh | actually I just checked and it could be | 22:58 |
| jhesketh | yeah | 22:58 |
| clarkb | I think that would be better and will prevent drift in subnode and node names' | 22:58 |
| *** msabramo has quit IRC | 22:59 | |
| jhesketh | so we allow provider and label objects, it'd be a pity to miss out on accessing the node object, but I tend to agree | 22:59 |
| mattoliverau | Morning | 23:03 |
| clarkb | jhesketh: ya | 23:03 |
| *** flaper87 is now known as flaper87|afk | 23:03 | |
| *** BadCub has quit IRC | 23:04 | |
| *** msabramo has joined #openstack-infra | 23:07 | |
| *** W00dy__ has quit IRC | 23:07 | |
| *** wenlock_ has quit IRC | 23:08 | |
| *** yamahata has quit IRC | 23:09 | |
| *** prad has quit IRC | 23:14 | |
| davidlenwell | can someone please remind me which file in config indicates the voting things on my project? | 23:15 |
| clarkb | davidlenwell: modules/openstack_project/files/zuul/layout.yaml | 23:15 |
| *** Sukhdev has quit IRC | 23:15 | |
| openstackgerrit | Khai Do proposed a change to openstack-infra/jenkins-job-builder: remove misleading jjb example https://review.openstack.org/96309 | 23:17 |
| davidlenwell | thanks clarkb | 23:17 |
| openstackgerrit | David Lenwell proposed a change to openstack-infra/config: removed py26 testing from refstack https://review.openstack.org/96310 | 23:18 |
| clarkb | jeblair: fungi mordred 93950 adds a fun test to the infra/config project | 23:19 |
| openstackgerrit | Khai Do proposed a change to openstack-infra/jenkins-job-builder: remove misleading jjb example https://review.openstack.org/96309 | 23:20 |
| fungi | davidlenwell: keeping in mind that any project templates your project entry includes in that file are also going to be run | 23:20 |
| davidlenwell | fungi: that statement confuses me | 23:20 |
| *** amotoki has joined #openstack-infra | 23:20 | |
| clarkb | so I notice the redhat and hp tripleo clouds have significantly less fail going on | 23:20 |
| fungi | davidlenwell: which project? i can elucidate | 23:20 |
| clarkb | which is really cool | 23:20 |
| davidlenwell | restack | 23:21 |
| fungi | and by restack i assume refstack | 23:21 |
| davidlenwell | https://review.openstack.org/#/c/96310/ | 23:21 |
| davidlenwell | sorry | 23:21 |
| davidlenwell | typing is hard | 23:21 |
| *** mburned_out is now known as mburned | 23:22 | |
| fungi | davidlenwell: so... easy. refstack has no additional project templates at http://git.openstack.org/cgit/openstack-infra/config/tree/modules/openstack_project/files/zuul/layout.yaml#n4694 | 23:22 |
| davidlenwell | I didn't think so | 23:22 |
| *** hogepodge has quit IRC | 23:23 | |
| fungi | davidlenwell: so it's just a matter of looking in the jobs section of that file for patterns any of the job names mentioned in your check and gate pipelines | 23:23 |
| davidlenwell | sweet .. thankf for the info | 23:23 |
| fungi | davidlenwell: noting that at http://git.openstack.org/cgit/openstack-infra/config/tree/modules/openstack_project/files/zuul/layout.yaml#n556 voting is disabled for the gate-refstack-python33 job | 23:23 |
| *** aconrad has quit IRC | 23:24 | |
| fungi | and also for gate-refstack-python26 just below it | 23:24 |
| davidlenwell | yes .. as intended | 23:24 |
| davidlenwell | I removed that | 23:24 |
| *** salv-orlando has quit IRC | 23:24 | |
| davidlenwell | in that last commit | 23:24 |
| davidlenwell | I want non voting py33 but not py26 | 23:24 |
| fungi | so the rest are presumably voting, assuming no unduly complicated regular expressions which missed my simple keyword search | 23:24 |
| davidlenwell | yes | 23:25 |
| davidlenwell | thanks again fungi and clarkb | 23:25 |
| openstackgerrit | A change was merged to openstack-infra/config: Add serveraliases to cgit vhost template https://review.openstack.org/93870 | 23:25 |
| fungi | yw | 23:25 |
| *** salv-orlando has joined #openstack-infra | 23:29 | |
| fungi | davidlenwell: and the change you linked is definitely not a match for what it does. comments added inline | 23:29 |
| davidlenwell | oh .. okay.. looking at that now | 23:30 |
| *** radez is now known as radez_g0n3 | 23:30 | |
| openstackgerrit | David Lenwell proposed a change to openstack-infra/config: removed py26 testing from refstack https://review.openstack.org/96310 | 23:31 |
| davidlenwell | fungi: better ? ^^ | 23:31 |
| clarkb | jeblair: for 94440 (infra-specs initial commit) I think there are a few things that need addressing then we should probably get that in so that we can start working on real spec review and writing | 23:33 |
| clarkb | jeblair: also, I am a bit confused as to why we need a coveragerc and so on, but I think that came from cookiecutter so meh | 23:33 |
| fungi | davidlenwell: yep! +2 | 23:34 |
| davidlenwell | thank you! | 23:34 |
| *** dstanek is now known as dstanek_zzz | 23:37 | |
| *** jp_at_hp has quit IRC | 23:37 | |
| *** jp_at_hp has joined #openstack-infra | 23:38 | |
| *** zzelle has quit IRC | 23:38 | |
| clarkb | mordred: can you address the comment in 90234? | 23:38 |
| *** mmaglana has quit IRC | 23:40 | |
| clarkb | pleia2: 89928 would it be reasonable to ask for a mirror config that handled multiple ubuntu versions? | 23:40 |
| clarkb | pleia2: you should be able to make the variable an array then iterate over that in the erb file | 23:40 |
| clarkb | pleia2: that way we can mirror precise and trusty | 23:40 |
| openstackgerrit | A change was merged to openstack-infra/config: Add change-abandoned hook for gerrit in puppet https://review.openstack.org/93773 | 23:41 |
| clarkb | pleia2: nevermind I see it is a define now and we can have multiple defines | 23:41 |
| *** aconrad has joined #openstack-infra | 23:42 | |
| openstackgerrit | A change was merged to openstack-infra/config: removed py26 testing from refstack https://review.openstack.org/96310 | 23:45 |
| *** oomichi has joined #openstack-infra | 23:46 | |
| clarkb | pleia2: I commented on it with what I could figure out. Let me know what you think | 23:46 |
| *** dstanek_zzz is now known as dstanek | 23:47 | |
| *** salv-orlando has quit IRC | 23:52 | |
| *** gyee has joined #openstack-infra | 23:53 | |
| *** jaypipes has joined #openstack-infra | 23:54 | |
| fungi | jesusaurus: any objection to me rebasing https://review.openstack.org/93950 for you? | 23:56 |
| * clarkb runs home back in a bit | 23:56 | |
| clarkb | mordred: if you get back to a keyboard, I think we should follow up with the 1.1 stuff more aggressively. The failure mode appears to have changed and the amount of error is even greater | 23:57 |
| jesusaurus | fungi: oh, here, i'll take care of that in a sec | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!