| *** esmute_ has quit IRC | 00:00 | |
| *** flepied has quit IRC | 00:00 | |
| * mordred may have included too many references in one joke | 00:00 | |
| jeblair | filed a bug with traceback: https://storyboard.openstack.org/#!/story/2000335 | 00:00 |
|---|---|---|
| jeblair | mordred: since zuul is stuck, now is a great time to restart nodepool. i will do it. | 00:01 |
| mordred | jeblair: awesome | 00:01 |
| jeblair | done | 00:01 |
| jeblair | now i will restart zuul | 00:01 |
| *** esmute_ has joined #openstack-infra | 00:01 | |
| jeblair | 2015-08-11 23:01:11,323 DEBUG zuul.reporter.gerrit.Reporter: Report change <Change 0x7fa19c80a110 211775,1>, params {'verified': 1}, message: Build succeeded (check pipeline). | 00:02 |
| jeblair | is the event in question | 00:02 |
| jeblair | so we'll end up losing events since then | 00:02 |
| mordred | Shrews: cross your fingers, jeblair restarted nodepool with 0.12 | 00:02 |
| *** david-lyle has quit IRC | 00:03 | |
| * Shrews crosses all the things | 00:03 | |
| SlickNik | thanks jeblair! | 00:03 |
| * jhesketh is concerned by the monkey on his back | 00:03 | |
| * SlickNik crosses all things other than the balinese monkey, tiger, jhesketh. That could be…dangerous! | 00:04 | |
| jeblair | #status notice Zuul was restarted due to an error; events (such as approvals or new patchsets) since 23:01 UTC have been lost and affected changes will need to be rechecked | 00:04 |
| openstackstatus | jeblair: sending notice | 00:04 |
| *** wenlock has quit IRC | 00:05 | |
| jeblair | i love that it sat in "self._cv.wait(timeout)" for an hour | 00:05 |
| *** puranamr has quit IRC | 00:05 | |
| -openstackstatus- NOTICE: Zuul was restarted due to an error; events (such as approvals or new patchsets) since 23:01 UTC have been lost and affected changes will need to be rechecked | 00:05 | |
| jeblair | perhaps that value is adjustable | 00:05 |
| openstackstatus | jeblair: finished sending notice | 00:06 |
| *** bharath has quit IRC | 00:07 | |
| *** puranamr has joined #openstack-infra | 00:07 | |
| *** esmute_ has quit IRC | 00:07 | |
| *** esmute_ has joined #openstack-infra | 00:08 | |
| *** madhu_ak has quit IRC | 00:09 | |
| *** otter768 has joined #openstack-infra | 00:12 | |
| *** puranamr has quit IRC | 00:12 | |
| *** puranamr has joined #openstack-infra | 00:12 | |
| *** esmute_ has quit IRC | 00:13 | |
| *** minwang2 has quit IRC | 00:14 | |
| nibalizer | krotscheck: mordred when i ask puppet to do something, I expect it to do that thing, schedule being like "I already did that today" is weird to me | 00:14 |
| *** hichihara has joined #openstack-infra | 00:14 | |
| nibalizer | now I've never really used schedule so maybe its awesome | 00:15 |
| nibalizer | I think we should be able to make the npm upgrade exec idempotent so that it doesn't cost anything | 00:15 |
| mordred | nibalizer: I think of puppet in terms of asking it to ensure that somethign exists/has been done | 00:15 |
| mordred | nibalizer: there are plenty of times where puppet skips doing things because they're already dnoe | 00:15 |
| nibalizer | right but it inspects that they need to be done | 00:16 |
| mordred | nibalizer: in this case, it's a way to express "please ensure that this has been run at least one time today" | 00:16 |
| mordred | to me | 00:16 |
| mordred | I've also never used it, so I could be wrong | 00:16 |
| *** otter768 has quit IRC | 00:16 | |
| jeblair | mordred: yeah, though i'm not certain that's what we really want | 00:16 |
| jeblair | maybe if we were doing bank-style batch processing with puppet | 00:16 |
| jeblair | but for 'upgrade npm', do we actually want to attempt it only once per day? | 00:17 |
| jeblair | i could easily see us wondering why we're not getting the security update they just released | 00:17 |
| mtreinish | jeblair, mordred, anteaya, et all: if you could take a sec to just +W (it's already been approved) https://review.openstack.org/210941 to kick into the gate queue, it's stuck because of a depends on (which merged this afternoon) | 00:17 |
| mordred | dunno. I was more arguing the general usefulness of the construct | 00:17 |
| mtreinish | I'd like to get that job up before the nightlies kick off | 00:17 |
| jeblair | mordred: yeah, that's why i didn't address that, i agree, it seems like a thing one might want to do | 00:17 |
| jeblair | mordred: so i'm exploring the idea that maybe we're talking around the idea that maybe that's not actually what we want it to do :) | 00:18 |
| *** esmute_ has joined #openstack-infra | 00:18 | |
| nibalizer | krotscheck: I also wonder if what you're trying to do could be done with this module cleaner | 00:19 |
| nibalizer | https://github.com/puppet-community/puppet-nodejs | 00:19 |
| nibalizer | kinda looks like it can install node from packages | 00:19 |
| jeblair | nibalizer, mordred: 'constantly idempotently ensure this is the latest version' is the pattern we usually use | 00:19 |
| nibalizer | or use npm to globaly update *things* | 00:19 |
| mordred | "On SUSE, ArchLinux, FreeBSD, OpenBSD and Gentoo, native packages are used. On Darwin, the MacPorts package is used. On Windows the packages are installed via Chocolatey." | 00:19 |
| jeblair | nibalizer: we do need a newer version of node than what's in trusty apparently | 00:20 |
| jeblair | er | 00:20 |
| mordred | that is the most ridiculous list of platforms | 00:20 |
| jeblair | nibalizer: s/node/npm/ | 00:20 |
| mordred | nibalizer: I do not have confidence in that module | 00:20 |
| mordred | nibalizer: purely because it does not support either of the top 2 linux distros | 00:20 |
| anteaya | mtreinish: yeah sorry I had that open earlier and than had to disappear due to weather, looking now | 00:20 |
| mordred | nibalizer: but it does support the world of esoteric things | 00:20 |
| mordred | nibalizer: I'm pretty sure the author is trying to out-hipster the world | 00:21 |
| crinkle | it does support normal distros https://github.com/puppet-community/puppet-nodejs#limitations | 00:21 |
| mtreinish | anteaya: no worries, I just need someone to kick it so zuul will push it to the gate | 00:21 |
| anteaya | yup | 00:21 |
| anteaya | I understand | 00:21 |
| *** mrodden has quit IRC | 00:21 | |
| mordred | crinkle: gotcha. so it's not uber-hipster - it's just a docs bug up above | 00:21 |
| nibalizer | ya we do beaker CI on that module | 00:21 |
| nibalizer | I mean, when my power supply isn't busted | 00:21 |
| fungi | i think it's "on platforms that just compile our source code straight to install" (and also suse?!?) | 00:22 |
| *** tiswanso has joined #openstack-infra | 00:22 | |
| openstackgerrit | Merged openstack-infra/irc-meetings: change compass irc meeting time https://review.openstack.org/211775 | 00:22 |
| fungi | hence arch, gentoo and *bsd ports (including mac os/darwin) | 00:22 |
| mordred | fungi: "On ... FreeBSD ... native packages are used" | 00:22 |
| nibalizer | also https://github.com/puppet-community/puppet-nodejs/blob/master/metadata.json#L10 | 00:22 |
| mordred | fungi: that's the silliest sentence | 00:23 |
| fungi | indeed | 00:23 |
| nibalizer | mordred: pkg_get -r is pretty good | 00:23 |
| fungi | "packages" | 00:23 |
| *** oomichi has joined #openstack-infra | 00:23 | |
| nibalizer | no I mean really bsd has binary packages now | 00:23 |
| nibalizer | the future is cometh | 00:23 |
| mordred | so I really _should_ start running freebsd? | 00:23 |
| fungi | as long as configure&&make&&make install works, they update their binary packages in an automated fashion | 00:23 |
| mtreinish | mordred: wait, what is chocolatey? | 00:23 |
| mordred | fungi: hahahaha | 00:23 |
| * mtreinish wants a candy bar | 00:24 | |
| mordred | mtreinish: https://chocolatey.org/ | 00:24 |
| lifeless | mordred: you should give dragonfly a go | 00:24 |
| anteaya | mtreinish: in the gate | 00:24 |
| mtreinish | anteaya: great thanks a lot | 00:24 |
| mordred | "Chocolatey NuGet is a Machine Package Manager, somewhat like apt-get, but built with Windows in mind." | 00:24 |
| fungi | basically *bsd binary packages are the result of make install followed by tar | 00:24 |
| *** nelsnelson has quit IRC | 00:24 | |
| mordred | "iex ((new-object net.webclient).DownloadString('https://chocolatey.org/install.ps1'))" | 00:25 |
| fungi | it's like the bsds reinvented slackware | 00:25 |
| mordred | that's how you install it | 00:25 |
| mtreinish | fungi: heh | 00:25 |
| lifeless | fungi: which is totally unlike .deb's.... | 00:25 |
| nibalizer | its the curl | bash of powershell | 00:25 |
| fungi | lifeless: touche! | 00:25 |
| openstackgerrit | Sridhar Ramaswamy proposed openstack-infra/project-config: Fix up tacker stackforge projects https://review.openstack.org/211789 | 00:25 |
| fungi | lifeless: heck, they even use makefiles | 00:25 |
| fungi | oh, but ar instead of tar! | 00:25 |
| lifeless | fungi: Debian, a more greybeard BSD than BSD. Now with added BSD support. | 00:26 |
| fungi | see, totally different | 00:26 |
| mtreinish | mordred: ugh, well I think I really want a chocolate bar now to recover from seeing that | 00:26 |
| anteaya | ha ha ha | 00:26 |
| fungi | lifeless: ar is also much more pirate | 00:26 |
| Shrews | mordred: OMG. How can they title that "Easy Install!"? | 00:26 |
| fungi | Shrews: remember, it's windows | 00:26 |
| anteaya | and they are high on chocolate | 00:27 |
| fungi | Shrews: their target audience all have a serious case of stockholm syndrome | 00:27 |
| * Shrews searches for a snopes article on "chocolatey"... gotta be a joke | 00:27 | |
| *** enikanorov2 has quit IRC | 00:27 | |
| *** otter768 has joined #openstack-infra | 00:27 | |
| openstackgerrit | Merged openstack-infra/project-config: Add periodic/experimental tempest job for using test accounts https://review.openstack.org/210941 | 00:28 |
| lifeless | Shrews: its not | 00:28 |
| fungi | if only | 00:28 |
| lifeless | Shrews: http://blogs.technet.com/b/windowsserver/archive/2014/04/03/windows-management-framework-v5-preview.aspx | 00:28 |
| anteaya | ha ha ha ha mtreinish moar fort collins | 00:28 |
| Shrews | yeah yeah... i was being flippant :) | 00:29 |
| mtreinish | anteaya: heh, cody-somerville suggested it and I've never been to CO :) | 00:29 |
| mtreinish | I also had limited time to schedule because I'm a slacker, so it was easy | 00:29 |
| anteaya | mtreinish: yes it is easy | 00:30 |
| anteaya | mtreinish: rent a car for sure | 00:30 |
| *** bhunter71 has joined #openstack-infra | 00:30 | |
| mtreinish | anteaya: heh, yep | 00:31 |
| *** stevemar has joined #openstack-infra | 00:31 | |
| anteaya | fort collins is nice | 00:32 |
| *** tiswanso has quit IRC | 00:33 | |
| *** stevemar has quit IRC | 00:34 | |
| *** claudiub has quit IRC | 00:34 | |
| mordred | ++ | 00:35 |
| *** achanda_ has joined #openstack-infra | 00:35 | |
| *** melaks has quit IRC | 00:36 | |
| *** thorst has joined #openstack-infra | 00:36 | |
| *** achanda has quit IRC | 00:38 | |
| *** gildub has quit IRC | 00:38 | |
| *** laughterwym has joined #openstack-infra | 00:38 | |
| *** achanda_ has quit IRC | 00:39 | |
| *** stevemar has joined #openstack-infra | 00:40 | |
| *** nelsnelson has joined #openstack-infra | 00:40 | |
| *** sharikapongubala has quit IRC | 00:40 | |
| *** stevemar has quit IRC | 00:41 | |
| *** Azherkhan has joined #openstack-infra | 00:44 | |
| *** xnox has quit IRC | 00:47 | |
| *** tiswanso has joined #openstack-infra | 00:48 | |
| *** woodster_ has quit IRC | 00:50 | |
| *** tsekiyam_ has joined #openstack-infra | 00:50 | |
| *** tiswanso has quit IRC | 00:51 | |
| *** bharath has joined #openstack-infra | 00:51 | |
| *** xyang1 has quit IRC | 00:51 | |
| *** tiswanso has joined #openstack-infra | 00:52 | |
| *** puranamr has quit IRC | 00:52 | |
| *** samueldmq has joined #openstack-infra | 00:52 | |
| *** mtanino has quit IRC | 00:53 | |
| *** tsekiyama has quit IRC | 00:53 | |
| *** andersonvom has quit IRC | 00:53 | |
| *** david-lyle has joined #openstack-infra | 00:54 | |
| mriedem | anteaya: i emailed btopol about the ibm people that said they were using vpnaas to see if he knows who they are and what they are doing, and why they aren't doing it upstream | 00:54 |
| mriedem | anteaya: some services team, i looked her up internally, never heard of her | 00:54 |
| anteaya | mriedem: thank you | 00:55 |
| *** xnox has joined #openstack-infra | 00:55 | |
| *** tsekiyam_ has quit IRC | 00:55 | |
| anteaya | because if they are, hey we could use some help sorting it out | 00:55 |
| mriedem | could be like the same people that were working on nova-docker and see how great that's going :) | 00:55 |
| mriedem | https://review.openstack.org/#/q/status:open+project:stackforge/nova-docker,n,z | 00:55 |
| anteaya | mriedem: wonderful | 00:55 |
| anteaya | woooo look at that chorus line of -1's | 00:56 |
| *** bharath has quit IRC | 00:56 | |
| *** rlandy has quit IRC | 00:57 | |
| *** rlandy has joined #openstack-infra | 00:58 | |
| *** andersonvom has joined #openstack-infra | 00:58 | |
| *** gildub has joined #openstack-infra | 00:58 | |
| *** sabeen has joined #openstack-infra | 00:59 | |
| *** skylerberg has quit IRC | 00:59 | |
| *** vlaza has quit IRC | 01:01 | |
| *** Azherkhan has quit IRC | 01:02 | |
| openstackgerrit | Clint 'SpamapS' Byrum proposed openstack-infra/project-config: Add rustygear to projects https://review.openstack.org/211796 | 01:02 |
| *** spzala has joined #openstack-infra | 01:04 | |
| *** erikwilson_ has quit IRC | 01:11 | |
| *** erikwilson_ has joined #openstack-infra | 01:12 | |
| *** baoli has joined #openstack-infra | 01:12 | |
| *** sridhar_ram has quit IRC | 01:12 | |
| *** madong_ is now known as larainema | 01:13 | |
| *** thorst has quit IRC | 01:16 | |
| jasondotstar | hi there | 01:16 |
| *** erikwilson_ has quit IRC | 01:16 | |
| jasondotstar | flirting with the idea of trying to use a vagrant box w/ the system-config project... I noticed that the .gitignore doesn't include the .vagrant dir or ignoring Vagrantfile | 01:17 |
| jasondotstar | has anyone tried setting up a local dev environment for system-config using vagrant? | 01:17 |
| *** alexsyip has quit IRC | 01:18 | |
| jasondotstar | not seeing it in there leads me to believe that perhaps something else is being done for local developement | 01:18 |
| *** puranamr has joined #openstack-infra | 01:20 | |
| clarkb | jasondotstar: we work on clouds so tend to use them for "local" dev | 01:20 |
| lifeless | jasondotstar: I don't imagine that a couple of git ignores would be contentious | 01:21 |
| jasondotstar | just wondering if I want to test out the etherpad example in the dox | 01:21 |
| *** skylerberg has joined #openstack-infra | 01:21 | |
| jasondotstar | in vagrant, for example.... | 01:21 |
| jasondotstar | it'd be decent to ignore the file and the hidden dir.... | 01:22 |
| jasondotstar | but to get back to clarkb's suggestion... what "cloud" are you using for your dev enviro ? | 01:22 |
| *** tqtran_ has quit IRC | 01:24 | |
| clarkb | I use rackspace | 01:24 |
| clarkb | but also have used hpcloud | 01:24 |
| jasondotstar | ah i c | 01:29 |
| jasondotstar | I guess I *could* spawn an instance or two in one of those and test out things... | 01:30 |
| *** sergmelikyan has joined #openstack-infra | 01:34 | |
| *** andersonvom has quit IRC | 01:35 | |
| *** michchap has quit IRC | 01:39 | |
| openstackgerrit | Jay_Clark proposed openstack-infra/system-config: adding Vagrantfile to .gitignore https://review.openstack.org/211805 | 01:41 |
| *** michchap has joined #openstack-infra | 01:43 | |
| *** nelsnelson has quit IRC | 01:44 | |
| *** mriedem is now known as mriedem_movie | 01:49 | |
| *** baoli has quit IRC | 01:49 | |
| *** spzala has quit IRC | 01:50 | |
| *** bknudson has quit IRC | 01:51 | |
| *** ruagair has quit IRC | 01:53 | |
| *** ruagair has joined #openstack-infra | 01:54 | |
| *** larainema has quit IRC | 01:55 | |
| *** larainema has joined #openstack-infra | 01:55 | |
| *** erlon has quit IRC | 02:03 | |
| *** patrickeast has quit IRC | 02:04 | |
| *** baoli has joined #openstack-infra | 02:04 | |
| *** laughterwym_ has joined #openstack-infra | 02:05 | |
| *** ajmiller has joined #openstack-infra | 02:06 | |
| *** nelsnelson has joined #openstack-infra | 02:06 | |
| *** marzif has quit IRC | 02:06 | |
| *** laughterwym__ has joined #openstack-infra | 02:07 | |
| *** laughterwym has quit IRC | 02:08 | |
| *** stevemar has joined #openstack-infra | 02:09 | |
| *** yamamoto has joined #openstack-infra | 02:09 | |
| *** laughterwym_ has quit IRC | 02:10 | |
| *** sergmelikyan has quit IRC | 02:14 | |
| *** yamahata has quit IRC | 02:15 | |
| *** samueldmq has quit IRC | 02:15 | |
| *** nelsnelson has quit IRC | 02:20 | |
| *** yamamoto has quit IRC | 02:23 | |
| *** laughterwym__ has quit IRC | 02:27 | |
| *** gyee has quit IRC | 02:27 | |
| *** laughterwym has joined #openstack-infra | 02:27 | |
| *** mdbooth has quit IRC | 02:29 | |
| *** david-lyle has quit IRC | 02:33 | |
| *** mdbooth has joined #openstack-infra | 02:34 | |
| *** funzo has joined #openstack-infra | 02:36 | |
| *** nelsnelson has joined #openstack-infra | 02:38 | |
| *** funzo has quit IRC | 02:40 | |
| *** nelsnelson has quit IRC | 02:42 | |
| *** bharath has joined #openstack-infra | 02:44 | |
| *** rlandy has quit IRC | 02:45 | |
| *** angdraug has quit IRC | 02:46 | |
| *** wendar has quit IRC | 02:46 | |
| *** wendar has joined #openstack-infra | 02:46 | |
| *** puranamr has quit IRC | 02:48 | |
| *** bharath has quit IRC | 02:49 | |
| *** BharatK has joined #openstack-infra | 02:53 | |
| *** otter768 has quit IRC | 02:54 | |
| *** patrickeast has joined #openstack-infra | 02:55 | |
| *** Sree has joined #openstack-infra | 02:55 | |
| *** Sree has quit IRC | 03:00 | |
| *** Sree has joined #openstack-infra | 03:00 | |
| *** Sree has quit IRC | 03:01 | |
| *** Sree has joined #openstack-infra | 03:02 | |
| *** yamamoto has joined #openstack-infra | 03:03 | |
| tchaypo | Nakato: https://github.com/jamezpolley/ansible-stackviz is my role for installing stackviz | 03:03 |
| openstackgerrit | Ian Wienand proposed openstack/diskimage-builder: Remove grub2 in redhat-common/pre-install.d/15-remove-grub https://review.openstack.org/211124 | 03:08 |
| jlvillal | tonyb, Regarding: https://review.openstack.org/#/c/211273/ Do you know why it isn't merging? Am I missing something? It seems like it should kick off a merge. | 03:08 |
| *** rm_work|away is now known as rm_work | 03:10 | |
| *** erikwilson_ has joined #openstack-infra | 03:12 | |
| jroll | jlvillal: there was a zuul restart, probably got caught in that | 03:13 |
| *** mriedem_movie has quit IRC | 03:14 | |
| jlvillal | jroll, I was thinking so. I did do a recheck. Maybe that will help? It did do the test job. But no gate :( | 03:14 |
| jroll | yeah, dunno | 03:15 |
| *** patrickeast has quit IRC | 03:15 | |
| jlvillal | jroll, Shouldn't you be out being a tourist? | 03:15 |
| jroll | jlvillal: what about you :P | 03:16 |
| * jlvillal is almost ready for sleep... | 03:16 | |
| jroll | just checking on things, I've been out and about already | 03:16 |
| jlvillal | Good. See you tomorrow morning. | 03:16 |
| *** erikwilson_ has quit IRC | 03:17 | |
| *** sergmelikyan has joined #openstack-infra | 03:19 | |
| *** fawadkhaliq has joined #openstack-infra | 03:22 | |
| *** baoli has quit IRC | 03:23 | |
| *** Sree has quit IRC | 03:23 | |
| *** Sree has joined #openstack-infra | 03:24 | |
| tonyb | jhesketh: Could the zuul restart mean that https://review.openstack.org/#/c/211273/ which is +W has missed the gate queue and need a hand? | 03:24 |
| *** vahidh has quit IRC | 03:25 | |
| *** dan has quit IRC | 03:26 | |
| *** adreznec has quit IRC | 03:26 | |
| *** vahidh has joined #openstack-infra | 03:26 | |
| *** dan has joined #openstack-infra | 03:26 | |
| *** achanda has joined #openstack-infra | 03:27 | |
| openstackgerrit | Colleen Murphy proposed openstack-infra/puppet-cgit: Add beaker-rspec gem to Gemfile https://review.openstack.org/211829 | 03:27 |
| openstackgerrit | Colleen Murphy proposed openstack-infra/puppet-cgit: Fix git clone for local test runs https://review.openstack.org/211830 | 03:27 |
| *** esmute_ has quit IRC | 03:27 | |
| *** adreznec has joined #openstack-infra | 03:27 | |
| *** baoli has joined #openstack-infra | 03:28 | |
| *** Sree has quit IRC | 03:28 | |
| jhesketh | tonyb: yep, if it's not in the queue then it'll need a 'recheck' | 03:29 |
| *** david-lyle has joined #openstack-infra | 03:30 | |
| jlvillal | jhesketh, I did a 'recheck' on it | 03:30 |
| jlvillal | jhesketh, It ran through the 'check' queue and passed. But nothing since. | 03:30 |
| tonyb | jhesketh: it whent through check but seems to dislike gate ? | 03:30 |
| jlvillal | tonyb, I wasn't sure if it would make any difference if you removed your +W and then put it back? I'm making wild guesses :) | 03:31 |
| jhesketh | hmm, it should have gone to the gate after the check | 03:31 |
| tonyb | jlvillal: I don't think you can remove *my* +W but if you add a -W that would block it. (I think) | 03:32 |
| *** heyongli has quit IRC | 03:32 | |
| jhesketh | tonyb: you should be able to remove your +w and then add it again | 03:32 |
| jlvillal | tonyb, Not me removing your +w. You remove it. | 03:32 |
| jhesketh | if that doesn't queue it I'll dig through the logs | 03:33 |
| jlvillal | tonyb, I did try doing a -1 and then 0. But no change. | 03:33 |
| tonyb | jlvillal, jhesketh: okay that did it | 03:34 |
| jlvillal | jhesketh, tonyb: I see it in the gate queue now | 03:34 |
| jlvillal | Thanks! | 03:34 |
| tonyb | jlvillal: okay I mis parsed you sentence. sorry | 03:34 |
| *** heyongli has joined #openstack-infra | 03:34 | |
| tonyb | jhesketh: Thanks. | 03:34 |
| tonyb | jhesketh: sake++ (this could end badly) | 03:35 |
| jlvillal | tonyb, Understandable. Thanks for your help. | 03:35 |
| jlvillal | jhesketh, Thanks also! | 03:35 |
| openstackgerrit | Merged openstack-infra/irc-meetings: Delete ux-team-meeting. Change ironic meeting to weekly. https://review.openstack.org/211273 | 03:35 |
| jhesketh | I wonder if it was because the check comment from jenkins didn't change the verified vote | 03:35 |
| jhesketh | might be worth looking closer at | 03:35 |
| *** sdake_ has joined #openstack-infra | 03:35 | |
| tonyb | jhesketh: I don't think so. Jenkins passed it (and added +V) days ago | 03:36 |
| tonyb | jhesketh: I think it was just that I +W'd it whiel zuul was "slow" and so that got lost | 03:36 |
| jhesketh | tonyb: right, but the trigger rules for queuing back into the gate requires jenkins to leave a +1 | 03:36 |
| tonyb | jhesketh: but clearly I'm just guessing | 03:36 |
| tonyb | jhesketh: Ahh okay. | 03:37 |
| jhesketh | tonyb: your original +W, yes | 03:37 |
| jhesketh | but the trigger after the recheck only looks for your +W as a requirement | 03:37 |
| jhesketh | I'll need to look at what the event stream from gerrit outputs on identical votes | 03:38 |
| * jhesketh adds it to his todo | 03:38 | |
| tonyb | jhesketh: okay. That's a long list. | 03:38 |
| jhesketh | tonyb: it's okay, I sort the 'take a nap' to the top of the list | 03:38 |
| *** sdake has quit IRC | 03:39 | |
| tonyb | jhesketh: :) | 03:39 |
| *** BharatK has quit IRC | 03:39 | |
| *** fedexo has joined #openstack-infra | 03:40 | |
| *** heyongli has quit IRC | 03:41 | |
| *** heyongli has joined #openstack-infra | 03:43 | |
| *** stevemar has quit IRC | 03:44 | |
| *** tsg has joined #openstack-infra | 03:44 | |
| anteaya | take a nap is always at teh top of my list | 03:45 |
| *** harlowja_at_home has joined #openstack-infra | 03:45 | |
| *** sdake_ is now known as sdake | 03:46 | |
| *** tjones has joined #openstack-infra | 03:48 | |
| *** bharath_ has joined #openstack-infra | 03:51 | |
| *** BharatK has joined #openstack-infra | 03:52 | |
| mordred | jasondotstar: your employer should give you a free account :) | 03:52 |
| *** esmute_ has joined #openstack-infra | 03:53 | |
| anteaya | mordred: how is the jungle? | 03:53 |
| anteaya | more monkies today? | 03:53 |
| *** tiswanso has quit IRC | 03:54 | |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/requirements: Bump oslo.utils to 2.0.0 https://review.openstack.org/204904 | 03:55 |
| *** baoli has quit IRC | 03:55 | |
| *** heyongli has quit IRC | 03:55 | |
| *** bharath_ has quit IRC | 03:56 | |
| *** heyongli has joined #openstack-infra | 03:56 | |
| *** fedexo has quit IRC | 04:00 | |
| *** BharatK has quit IRC | 04:00 | |
| *** stevemar has joined #openstack-infra | 04:00 | |
| openstackgerrit | Merged openstack-infra/project-config: Add Manila-GlusterFS experimental jobs https://review.openstack.org/207425 | 04:01 |
| mordred | anteaya: nope. but found a lovely river | 04:06 |
| *** funzo has joined #openstack-infra | 04:06 | |
| *** unicell has joined #openstack-infra | 04:06 | |
| *** sergmelikyan has quit IRC | 04:08 | |
| anteaya | mordred: nice | 04:08 |
| harlowja_at_home | mordred, u still up for possibly coming to yahoo in september to talk about openstack-infra to yahoo-infra peps? | 04:10 |
| * harlowja_at_home has to follow up with that guy internally | 04:10 | |
| harlowja_at_home | *eventually* | 04:10 |
| *** funzo has quit IRC | 04:11 | |
| *** BharatK has joined #openstack-infra | 04:15 | |
| *** sergmelikyan has joined #openstack-infra | 04:19 | |
| *** Sukhdev_ has joined #openstack-infra | 04:19 | |
| *** esmute_ has quit IRC | 04:20 | |
| *** BharatK has quit IRC | 04:22 | |
| *** morgan_404 is now known as morgan_410 | 04:26 | |
| *** morgan_410 is now known as morgan_404 | 04:26 | |
| *** BharatK has joined #openstack-infra | 04:33 | |
| *** puranamr has joined #openstack-infra | 04:38 | |
| *** madong_ has joined #openstack-infra | 04:40 | |
| *** larainema has quit IRC | 04:40 | |
| *** minwang2 has joined #openstack-infra | 04:41 | |
| *** bhunter71 has quit IRC | 04:42 | |
| *** links has joined #openstack-infra | 04:42 | |
| *** tjones has quit IRC | 04:43 | |
| *** abregman has joined #openstack-infra | 04:45 | |
| *** rlandy has joined #openstack-infra | 04:51 | |
| *** otter768 has joined #openstack-infra | 04:55 | |
| *** tjones has joined #openstack-infra | 04:56 | |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config: Remove XML only checks from security-doc https://review.openstack.org/211843 | 04:58 |
| *** otter768 has quit IRC | 04:59 | |
| *** enikanorov2 has joined #openstack-infra | 05:04 | |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config: Remove XML only checks from security-doc https://review.openstack.org/211843 | 05:07 |
| *** deepakcs has joined #openstack-infra | 05:12 | |
| *** erikwilson_ has joined #openstack-infra | 05:13 | |
| *** flepied has joined #openstack-infra | 05:14 | |
| *** tjones has quit IRC | 05:15 | |
| *** BharatK has quit IRC | 05:17 | |
| *** BharatK has joined #openstack-infra | 05:18 | |
| *** erikwilson_ has quit IRC | 05:18 | |
| jamielennox | lifeless: is this stuck somewhere: https://review.openstack.org/#/c/210988/ the recheck didn't go through to merge and i don't see it on status.o.o/zuul | 05:20 |
| jamielennox | (nor did a reverify show it up) | 05:20 |
| lifeless | jamielennox: no idea :/ | 05:21 |
| jamielennox | normally goes through normal gate job? | 05:21 |
| lifeless | yup | 05:23 |
| jamielennox | it is a mystery then | 05:24 |
| *** harlowja_at_home has quit IRC | 05:24 | |
| lifeless | jamielennox: its in zuul | 05:24 |
| lifeless | http://status.openstack.org/zuul/ | 05:24 |
| lifeless | as of 1m ago. | 05:24 |
| jamielennox | oh damn, just showed up back in the check queue | 05:25 |
| jamielennox | i've been waiting 3 weeks for a glance-store release to run a job, i guess it can wailt for tomorrow | 05:25 |
| *** tsg has quit IRC | 05:27 | |
| morgan_404 | jamielennox: sometimes a recheck gets backed up in the event queue | 05:27 |
| jamielennox | morgan_404: the recheck worked, it just move to the gate queue | 05:28 |
| morgan_404 | oh | 05:28 |
| morgan_404 | interesting | 05:28 |
| jamielennox | just didn't move | 05:28 |
| jamielennox | gave it a reverify this time, i'm not sure that's any different any more | 05:28 |
| openstackgerrit | Spencer Krum proposed openstack-infra/system-config: Nanliu-git is now Puppet-community git https://review.openstack.org/201815 | 05:29 |
| *** sergmelikyan has quit IRC | 05:29 | |
| *** Sree has joined #openstack-infra | 05:29 | |
| *** ayoung has quit IRC | 05:30 | |
| *** skylerberg has quit IRC | 05:32 | |
| *** fawadkhaliq has quit IRC | 05:38 | |
| *** minwang2 has quit IRC | 05:38 | |
| *** bharath has joined #openstack-infra | 05:40 | |
| *** andersonvom has joined #openstack-infra | 05:42 | |
| *** bharath has quit IRC | 05:45 | |
| *** ihrachyshka has joined #openstack-infra | 05:47 | |
| SlickNik | jamielennox: zuul seems to be stuck. I uploaded a new patchset, and it didn't make it to the check queue. | 05:51 |
| SlickNik | It was moving really slowly earlier today, and jeblair had to restart it — I wonder if it's in the same state again? | 05:51 |
| jamielennox | SlickNik: i'm not sure, i expect the reason this patch didn't merge hours ago is because it was over that restart window | 05:52 |
| *** hdd has quit IRC | 05:53 | |
| SlickNik | huh, strange. Let me keep an eye out on the queue and see if I see any other strangeness. | 05:53 |
| jamielennox | i guess we have to wait and see what the infra team says tomorrow | 05:53 |
| openstackgerrit | Ihar Hrachyshka proposed openstack-infra/devstack-gate: Revert "Cleanup unneeded bug workarounds" https://review.openstack.org/211853 | 05:54 |
| SlickNik | Okay seems moving again now for me. | 05:55 |
| *** yonglihe has joined #openstack-infra | 05:56 | |
| *** yonglihe has quit IRC | 05:56 | |
| *** yonglihe has joined #openstack-infra | 05:57 | |
| *** Sukhdev_ has quit IRC | 05:58 | |
| openstackgerrit | Jesse Pretorius proposed openstack-infra/project-config: Add docs check to os-ansible-deployment https://review.openstack.org/210498 | 05:58 |
| *** fawadkhaliq has joined #openstack-infra | 05:59 | |
| openstackgerrit | Jesse Pretorius proposed openstack-infra/project-config: Add docs check to os-ansible-deployment https://review.openstack.org/210498 | 06:00 |
| *** nagyz has quit IRC | 06:04 | |
| openstackgerrit | Spencer Krum proposed openstack-infra/system-config: Small changes to get closer to p4 compatiblity https://review.openstack.org/211854 | 06:05 |
| *** yfried has joined #openstack-infra | 06:07 | |
| *** puranamr has quit IRC | 06:09 | |
| *** nadya has joined #openstack-infra | 06:10 | |
| *** mrmartin has joined #openstack-infra | 06:11 | |
| openstackgerrit | Spencer Krum proposed openstack-infra/project-config: Add nonvoting beaker tests to infra/puppet* https://review.openstack.org/208799 | 06:12 |
| *** hdd has joined #openstack-infra | 06:18 | |
| *** madong_ has quit IRC | 06:18 | |
| *** madong_ has joined #openstack-infra | 06:19 | |
| openstackgerrit | greghaynes proposed openstack/diskimage-builder: Sanitize distro elements for minimal vs cloud https://review.openstack.org/211859 | 06:20 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/requirements: Updated from generate-constraints https://review.openstack.org/210988 | 06:20 |
| openstackgerrit | greghaynes proposed openstack/diskimage-builder: Sanitize distro elements for minimal vs cloud https://review.openstack.org/211859 | 06:21 |
| openstackgerrit | Spencer Krum proposed openstack-infra/puppet-httpd: Allow httpd::vhost to take a content parameter https://review.openstack.org/211860 | 06:21 |
| greghaynes | clarkb: SpamapS morgan_404 ^ I did a thing | 06:21 |
| greghaynes | clarkb: You would be interested because that adds ssh to an element which uses -minimal | 06:21 |
| openstackgerrit | Spencer Krum proposed openstack-infra/puppet-httpd: Allow httpd::vhost to take a content parameter https://review.openstack.org/211860 | 06:22 |
| openstackgerrit | Spencer Krum proposed openstack-infra/puppet-httpd: Add vim swp files to gitignore https://review.openstack.org/211861 | 06:22 |
| *** e0ne has joined #openstack-infra | 06:22 | |
| * morgan_404 is not found. | 06:23 | |
| greghaynes | er, woops | 06:23 |
| greghaynes | mordred: ^ | 06:23 |
| * greghaynes shakes fist at mor<tab> | 06:24 | |
| *** zz_dimtruck is now known as dimtruck | 06:26 | |
| *** ParsectiX has joined #openstack-infra | 06:28 | |
| *** laughterwym has quit IRC | 06:28 | |
| *** unicell1 has joined #openstack-infra | 06:28 | |
| *** unicell has quit IRC | 06:28 | |
| *** mrunge has joined #openstack-infra | 06:33 | |
| *** vlaza has joined #openstack-infra | 06:35 | |
| *** sdake_ has joined #openstack-infra | 06:35 | |
| *** stevemar has quit IRC | 06:36 | |
| *** stevemar has joined #openstack-infra | 06:36 | |
| *** e0ne has quit IRC | 06:39 | |
| *** ihrachyshka has quit IRC | 06:39 | |
| *** sdake has quit IRC | 06:39 | |
| *** stevemar has quit IRC | 06:40 | |
| *** e0ne has joined #openstack-infra | 06:40 | |
| *** scheuran has joined #openstack-infra | 06:40 | |
| *** e0ne has quit IRC | 06:45 | |
| *** sabeen has quit IRC | 06:45 | |
| *** e0ne has joined #openstack-infra | 06:49 | |
| *** zeih has joined #openstack-infra | 06:53 | |
| *** e0ne has quit IRC | 06:54 | |
| *** otter768 has joined #openstack-infra | 06:56 | |
| *** yonglihe has quit IRC | 06:57 | |
| *** ildikov has quit IRC | 07:00 | |
| *** otter768 has quit IRC | 07:00 | |
| *** heyongli has quit IRC | 07:02 | |
| *** bharath has joined #openstack-infra | 07:02 | |
| *** devvesa has joined #openstack-infra | 07:03 | |
| *** heyongli has joined #openstack-infra | 07:06 | |
| *** madong_ is now known as larainema | 07:08 | |
| *** heyongli has quit IRC | 07:10 | |
| *** fawadkhaliq has quit IRC | 07:12 | |
| *** fawadk has joined #openstack-infra | 07:13 | |
| *** isviridov_away is now known as isviridov | 07:13 | |
| *** zeih has quit IRC | 07:14 | |
| *** erikwilson_ has joined #openstack-infra | 07:14 | |
| *** ig0r_ has joined #openstack-infra | 07:15 | |
| *** k4n0 has joined #openstack-infra | 07:15 | |
| *** isviridov is now known as isviridov_away | 07:16 | |
| *** arxcruz has joined #openstack-infra | 07:17 | |
| *** erikwilson_ has quit IRC | 07:19 | |
| *** ifarkas has joined #openstack-infra | 07:19 | |
| *** yamamoto has quit IRC | 07:20 | |
| *** hdd has quit IRC | 07:20 | |
| *** dizquierdo has joined #openstack-infra | 07:23 | |
| *** yamahata has joined #openstack-infra | 07:24 | |
| *** markus_z has joined #openstack-infra | 07:26 | |
| Nakato | clarkb: diff looks fine to me. | 07:27 |
| tonyb | If I make a (stable/juno) nova patch 'Depend-On' a (stable/juno) devstack patch will that run the (nova) check queue with my (hopefully) fixed devstack? | 07:30 |
| *** achanda has quit IRC | 07:31 | |
| *** heyongli has joined #openstack-infra | 07:31 | |
| *** afazekas has joined #openstack-infra | 07:32 | |
| *** achanda has joined #openstack-infra | 07:36 | |
| *** heyongli has quit IRC | 07:36 | |
| *** stevemar has joined #openstack-infra | 07:37 | |
| *** HeOS has quit IRC | 07:38 | |
| *** rossella_s has joined #openstack-infra | 07:38 | |
| *** ig0r__ has joined #openstack-infra | 07:38 | |
| *** stevemar has quit IRC | 07:40 | |
| *** ig0r_ has quit IRC | 07:41 | |
| *** andersonvom has quit IRC | 07:43 | |
| *** jlanoux has joined #openstack-infra | 07:45 | |
| yfried | hi, who can help me with JJB questions? | 07:45 |
| *** achanda has quit IRC | 07:46 | |
| *** dizquierdo has quit IRC | 07:49 | |
| *** boris-42 has quit IRC | 07:50 | |
| *** dimtruck is now known as zz_dimtruck | 07:50 | |
| *** yamamoto has joined #openstack-infra | 07:51 | |
| *** mikap has joined #openstack-infra | 07:52 | |
| *** ajmiller has quit IRC | 07:56 | |
| *** dtantsur|afk is now known as dtantsur | 07:57 | |
| *** gildub has quit IRC | 08:02 | |
| *** claudiub has joined #openstack-infra | 08:03 | |
| *** sdake_ has quit IRC | 08:04 | |
| *** yamahata has quit IRC | 08:08 | |
| *** mikap has quit IRC | 08:08 | |
| *** ildikov has joined #openstack-infra | 08:08 | |
| *** dguerri` is now known as dguerri | 08:09 | |
| *** ajmiller has joined #openstack-infra | 08:12 | |
| *** ig0r__ has quit IRC | 08:13 | |
| *** ig0r_ has joined #openstack-infra | 08:17 | |
| *** ig0r_ has quit IRC | 08:19 | |
| *** HeOS has joined #openstack-infra | 08:20 | |
| *** jistr has joined #openstack-infra | 08:21 | |
| *** tsg has joined #openstack-infra | 08:24 | |
| *** tsg has quit IRC | 08:30 | |
| *** nadya has quit IRC | 08:30 | |
| openstackgerrit | Merged openstack-infra/puppet-openstackid: Add Gemfile and puppet 4 checks https://review.openstack.org/211762 | 08:30 |
| *** yamamoto has quit IRC | 08:31 | |
| *** yamamoto has joined #openstack-infra | 08:32 | |
| electrical | yfried: i can try to help/ | 08:33 |
| *** fhubik has joined #openstack-infra | 08:33 | |
| yfried | electrical: hi | 08:36 |
| *** dizquierdo has joined #openstack-infra | 08:37 | |
| yfried | electrical: I'm adding a job to a project and I would like to test/update only that job and not the whole project/tree | 08:37 |
| yfried | electrical: how do I do that? | 08:38 |
| electrical | yfried: you can add the job name to the command at the very end. so you can do 'jenkins-jobs test jobname' | 08:39 |
| *** andersonvom has joined #openstack-infra | 08:39 | |
| *** samuelBartel has quit IRC | 08:40 | |
| openstackgerrit | Merged openstack-infra/puppet-os_client_config: Add Gemfile and puppet 4 checks https://review.openstack.org/211763 | 08:43 |
| *** andersonvom has quit IRC | 08:44 | |
| *** ruagair has quit IRC | 08:44 | |
| *** samuelBartel has joined #openstack-infra | 08:46 | |
| *** ruagair has joined #openstack-infra | 08:47 | |
| *** esmute_ has joined #openstack-infra | 08:49 | |
| *** ruagair has quit IRC | 08:52 | |
| *** ruagair has joined #openstack-infra | 08:53 | |
| *** hichihara has quit IRC | 08:53 | |
| *** nadya has joined #openstack-infra | 08:55 | |
| openstackgerrit | Bartłomiej Piotrowski proposed openstack-infra/project-config: Add fuel-plugin-detach-keystone project to stackforge https://review.openstack.org/211910 | 08:55 |
| *** otter768 has joined #openstack-infra | 08:57 | |
| *** b10n1k has joined #openstack-infra | 08:57 | |
| electrical | yfried: did that answer your question ? | 09:00 |
| *** otter768 has quit IRC | 09:01 | |
| *** nadya has quit IRC | 09:02 | |
| Kennan2 | hi, is infra any issue about this ? | 09:03 |
| Kennan2 | Went to status ERROR due to "Message: No valid host was found. There are not enough hosts available. | 09:03 |
| Kennan2 | I found in jenkins jobs failed for that | 09:03 |
| openstackgerrit | Bartłomiej Piotrowski proposed openstack-infra/project-config: Add fuel-plugin-detach-database project to stackforge https://review.openstack.org/211914 | 09:04 |
| *** yfried is now known as yfried|afk | 09:05 | |
| *** ihrachyshka has joined #openstack-infra | 09:05 | |
| *** nadya has joined #openstack-infra | 09:06 | |
| *** nadya has quit IRC | 09:07 | |
| *** pelix has joined #openstack-infra | 09:07 | |
| *** nadya has joined #openstack-infra | 09:08 | |
| *** redixin has joined #openstack-infra | 09:10 | |
| openstackgerrit | Bartłomiej Piotrowski proposed openstack-infra/project-config: Add fuel-plugin-detach-keystone project to stackforge https://review.openstack.org/211910 | 09:11 |
| openstackgerrit | Bartłomiej Piotrowski proposed openstack-infra/project-config: Add fuel-plugin-detach-rabbitmq project to stackforge https://review.openstack.org/211915 | 09:11 |
| openstackgerrit | Ihar Hrachyshka proposed openstack-infra/devstack-gate: Revert "Cleanup unneeded bug workarounds" https://review.openstack.org/211853 | 09:12 |
| openstackgerrit | Andrei V. Ostapenko proposed openstack-infra/project-config: Move from stackforge/magnetodb to openstack/magnetodb https://review.openstack.org/203524 | 09:13 |
| openstackgerrit | Andrei V. Ostapenko proposed openstack-infra/project-config: Changes jobs description for MagnetoDB https://review.openstack.org/204478 | 09:13 |
| *** erikwilson_ has joined #openstack-infra | 09:15 | |
| openstackgerrit | Ricardo Carrillo Cruz proposed openstack-infra/project-config: Add infra-ansible project to OpenStack Infra https://review.openstack.org/211923 | 09:19 |
| *** erikwilson_ has quit IRC | 09:20 | |
| *** GheRivero is now known as GheIrcCloud | 09:22 | |
| *** GheRivero has joined #openstack-infra | 09:25 | |
| *** yfried|afk is now known as yfried | 09:25 | |
| ihrachyshka | SergeyLukjanov, hi. neutron is blocked by new db driver. There is a revert that should fix the gate for it: https://review.openstack.org/#/c/211853/ | 09:27 |
| openstackgerrit | Ihar Hrachyshka proposed openstack-infra/devstack-gate: Revert "Cleanup unneeded bug workarounds" https://review.openstack.org/211853 | 09:31 |
| ihrachyshka | sdague, clarkb, lyz, fungi, mordred ^^ | 09:32 |
| *** yfried is now known as yfried|afk | 09:35 | |
| *** esmute_ has quit IRC | 09:37 | |
| *** larainema has quit IRC | 09:37 | |
| *** larainema has joined #openstack-infra | 09:38 | |
| *** stevemar has joined #openstack-infra | 09:38 | |
| *** yfried|afk is now known as yfried | 09:39 | |
| openstackgerrit | Ricardo Carrillo Cruz proposed openstack-infra/infra-specs: Create Nodepool REST API spec https://review.openstack.org/141016 | 09:39 |
| *** stevemar has quit IRC | 09:41 | |
| Kennan2 | hi clarkb: fungi: | 09:41 |
| Kennan2 | I found our jenkins failed for | 09:41 |
| Kennan2 | NoValidHost: No valid host was found. There are not enough hosts available. | 09:41 |
| Kennan2 | is that infra not resource issue? | 09:41 |
| *** Kennan2 is now known as Kennan | 09:42 | |
| *** arxcruz has quit IRC | 09:44 | |
| *** arxcruz has joined #openstack-infra | 09:44 | |
| *** kushal has joined #openstack-infra | 09:46 | |
| *** Somay has joined #openstack-infra | 09:46 | |
| *** bharath has quit IRC | 09:47 | |
| *** bharath has joined #openstack-infra | 09:49 | |
| *** larainema has quit IRC | 09:49 | |
| *** madong_ has joined #openstack-infra | 09:49 | |
| *** yamamoto has quit IRC | 09:50 | |
| *** yamamoto has joined #openstack-infra | 09:51 | |
| *** bharath has quit IRC | 09:53 | |
| yfried | electrical: not fully | 09:53 |
| yfried | electrical: the name is really complicated and constructed from a large matrix. Is there a way to do this based on a regex? | 09:54 |
| electrical | yfried: yeah. you can use * for example my-job-* | 09:54 |
| yfried | electrical: nice. I'll try | 09:55 |
| *** mdbooth has quit IRC | 09:55 | |
| *** mikap has joined #openstack-infra | 09:56 | |
| *** madong_ has quit IRC | 09:56 | |
| *** madong_ has joined #openstack-infra | 09:56 | |
| openstackgerrit | Ricardo Carrillo Cruz proposed openstack-infra/puppet-jenkins: Add sane defaults for SSL certificates https://review.openstack.org/211943 | 09:56 |
| SergeyLukjanov | ihrachyshka, hi, looking on it now | 10:01 |
| *** mdbooth has joined #openstack-infra | 10:01 | |
| *** claudiub has quit IRC | 10:03 | |
| ihrachyshka | SergeyLukjanov, thnkas! | 10:05 |
| *** marzif has joined #openstack-infra | 10:05 | |
| *** jcoufal has joined #openstack-infra | 10:05 | |
| *** ogelbukh has quit IRC | 10:07 | |
| *** b10n1k has quit IRC | 10:09 | |
| *** ig0r_ has joined #openstack-infra | 10:10 | |
| *** madong_ has quit IRC | 10:14 | |
| *** mikap has left #openstack-infra | 10:14 | |
| *** ogelbukh has joined #openstack-infra | 10:15 | |
| openstackgerrit | Derek Higgins proposed openstack-infra/tripleo-ci: Fix PIP_INDEX_URL https://review.openstack.org/211948 | 10:18 |
| *** claudiub has joined #openstack-infra | 10:19 | |
| *** jkraj has joined #openstack-infra | 10:19 | |
| *** samueldmq has joined #openstack-infra | 10:22 | |
| openstackgerrit | Somay Jain proposed openstack-infra/jenkins-job-builder: Support for link-to-last-build option in html publisher plugin https://review.openstack.org/211951 | 10:27 |
| openstackgerrit | Merged openstack/requirements: Updated from generate-constraints https://review.openstack.org/210988 | 10:28 |
| *** zeih has joined #openstack-infra | 10:30 | |
| *** jcoufal has quit IRC | 10:35 | |
| *** zeih has quit IRC | 10:36 | |
| sdague | ihrachyshka: +A | 10:37 |
| *** b10n1k has joined #openstack-infra | 10:37 | |
| sdague | sorry about that, but we'll fast revert it and get things running | 10:37 |
| ihrachyshka | sdague, thanks! I think we neutron people should have go this route first step, then try to find the guilty patches, so we were slow ourselves here. | 10:38 |
| sc68cal | sdague: we're in the process of putting some suspenders on as well - https://review.openstack.org/#/c/211947/ | 10:39 |
| *** rguillebert has joined #openstack-infra | 10:39 | |
| sdague | sc68cal: gotcha | 10:39 |
| *** andersonvom has joined #openstack-infra | 10:40 | |
| *** fawadk has quit IRC | 10:41 | |
| *** rguillebert has quit IRC | 10:44 | |
| Daviey | lifeless (or anyone else): Seen, "error in setup command: Invalid environment marker: (python_version=='2.7' # MIT)" on g-r propagations? | 10:44 |
| *** andersonvom has quit IRC | 10:45 | |
| *** b10n1k has quit IRC | 10:45 | |
| *** sergmelikyan has joined #openstack-infra | 10:45 | |
| *** ig0r_ has quit IRC | 10:46 | |
| *** larainema has joined #openstack-infra | 10:47 | |
| *** kmartin has quit IRC | 10:48 | |
| *** bharath has joined #openstack-infra | 10:49 | |
| *** pc_m has joined #openstack-infra | 10:50 | |
| *** sergmelikyan has quit IRC | 10:50 | |
| *** bharath has quit IRC | 10:54 | |
| *** alexpilotti has joined #openstack-infra | 10:55 | |
| *** fhubik is now known as fhubik_brb | 10:57 | |
| *** otter768 has joined #openstack-infra | 10:57 | |
| *** rguillebert has joined #openstack-infra | 10:57 | |
| *** yfried is now known as yfried|afk | 11:00 | |
| *** jcoufal has joined #openstack-infra | 11:00 | |
| *** rossella_s has quit IRC | 11:00 | |
| *** jcoufal has quit IRC | 11:01 | |
| *** jcoufal has joined #openstack-infra | 11:01 | |
| *** otter768 has quit IRC | 11:02 | |
| *** jcoufal has quit IRC | 11:02 | |
| *** mwagner_lap has quit IRC | 11:02 | |
| *** jcoufal has joined #openstack-infra | 11:02 | |
| *** oomichi has quit IRC | 11:05 | |
| *** markus_z has quit IRC | 11:05 | |
| *** cdent has joined #openstack-infra | 11:07 | |
| *** yfried|afk is now known as yfried | 11:08 | |
| *** yamamoto has quit IRC | 11:08 | |
| *** juggler has quit IRC | 11:13 | |
| therve | Is there something going with logstach? It doesn't find results I'm expecting to find | 11:13 |
| *** jcoufal has quit IRC | 11:15 | |
| *** erikwilson_ has joined #openstack-infra | 11:16 | |
| openstackgerrit | Sean M. Collins proposed openstack-infra/project-config: [WIP] IPv6 only neutron experimental job https://review.openstack.org/209663 | 11:17 |
| *** erikwilson_ has quit IRC | 11:20 | |
| *** markus_z has joined #openstack-infra | 11:21 | |
| openstackgerrit | Clint Adams proposed openstack-infra/release-tools: python3-compatible syntax https://review.openstack.org/211972 | 11:25 |
| lifeless | Daviey: I haven't seen that before | 11:26 |
| Daviey | lifeless: I haven't dug into it yet.. but for interest, https://review.openstack.org/#/c/192319/ | 11:27 |
| openstackgerrit | Somay Jain proposed openstack-infra/jenkins-job-builder: BugFix: Target field in Job-DSL plugin was not being populated https://review.openstack.org/211973 | 11:28 |
| *** marcusvrn has joined #openstack-infra | 11:28 | |
| lifeless | Daviey: hah, 2.6 | 11:29 |
| lifeless | Daviey: http://logs.openstack.org/19/192319/11/check/gate-python-keystoneclient-kerberos-python26/8453d0c/console.html | 11:29 |
| lifeless | Daviey: I rather suspect that job needs to be removed | 11:29 |
| lifeless | (or the marker in openstack/requirements made more broad) | 11:30 |
| lifeless | Daviey: may be a pbr bug; I suspect thats the only comment-on-a-marker-using-line we have | 11:31 |
| lifeless | Daviey: (s/may be/probably/) | 11:31 |
| lifeless | night all | 11:31 |
| *** fhubik_brb is now known as fhubik | 11:32 | |
| *** baoli has joined #openstack-infra | 11:33 | |
| *** baoli has quit IRC | 11:35 | |
| openstackgerrit | Marian Horban proposed openstack/requirements: Add pyinotify to global-requirements.txt https://review.openstack.org/211975 | 11:35 |
| *** baoli has joined #openstack-infra | 11:35 | |
| *** thorst has joined #openstack-infra | 11:37 | |
| openstackgerrit | Ye-hong Fang proposed openstack/diskimage-builder: Add debian support to iso element https://review.openstack.org/211976 | 11:38 |
| *** julim has joined #openstack-infra | 11:38 | |
| *** ldnunes has joined #openstack-infra | 11:38 | |
| *** stevemar has joined #openstack-infra | 11:39 | |
| *** stevemar has quit IRC | 11:42 | |
| openstackgerrit | Merged openstack-infra/devstack-gate: Revert "Cleanup unneeded bug workarounds" https://review.openstack.org/211853 | 11:46 |
| *** gordc has joined #openstack-infra | 11:49 | |
| *** kushal has quit IRC | 11:53 | |
| Daviey | lifeless: thanks, nn | 11:55 |
| *** yfried has quit IRC | 11:56 | |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config: Fix check-syntax manuals check https://review.openstack.org/211988 | 11:56 |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config: Fix check-syntax manuals check https://review.openstack.org/211988 | 11:57 |
| *** dkranz has joined #openstack-infra | 11:58 | |
| *** AJaeger has joined #openstack-infra | 11:58 | |
| *** Somay has quit IRC | 11:58 | |
| *** Sree has quit IRC | 12:02 | |
| AJaeger | I have a few changes for the Docs team: this fixes a too tight check https://review.openstack.org/211988 and this removes some jobs: https://review.openstack.org/211843 . Timely reviews of these are welcome! ianw, sdague, fungi, could you do this today, please? | 12:06 |
| yolanda | jeblair, when you come online, there is a change on governance needing ptl +1 for puppet cookiecutter. Also another +2 +w needed for the creation of the project | 12:06 |
| *** hdd has joined #openstack-infra | 12:06 | |
| openstackgerrit | Nicko proposed openstack-infra/jenkins-job-builder: Customized pollscm trigger https://review.openstack.org/211993 | 12:07 |
| *** yamamoto has joined #openstack-infra | 12:09 | |
| Daviey | AJaeger: Can i suggest making the commit msg clearer for 211843? Seeing "Remove..." but +6, -0 blew my mind until i worked out what was happening | 12:10 |
| *** fhubik is now known as fhubik_brb | 12:10 | |
| *** fhubik_brb is now known as fhubik | 12:10 | |
| *** fhubik is now known as fhubik_brb | 12:11 | |
| fungi | jhesketh: so that you don't end up researching unnecessarily, yes gerrit does fail to emit vote info in the event stream if an existing vote is replaced by the same value. needs a new approval vote to get things moving in that case | 12:12 |
| openstackgerrit | Christian Berendt proposed openstack-infra/groups: Use drush version 0.10 in bootstrap script https://review.openstack.org/211996 | 12:13 |
| *** markus_z has quit IRC | 12:13 | |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config: Remove XML only checks from security-doc https://review.openstack.org/211843 | 12:13 |
| AJaeger | Daviey: Better? | 12:13 |
| Daviey | AJaeger: best earned +1 ever. Thanks | 12:14 |
| *** markus_z has joined #openstack-infra | 12:14 | |
| jhesketh | fungi: oh cool, thanks for that | 12:15 |
| jhesketh | I can knock that off my list :-) | 12:15 |
| * jhesketh doesn't understand how people keep up with the entire scrollback | 12:15 | |
| AJaeger | thanks, Daviey ! | 12:15 |
| fungi | Kennan: NoValidHost is raised by nova, not our infrastructure | 12:16 |
| *** deepakcs has quit IRC | 12:21 | |
| openstackgerrit | Merged openstack-infra/subunit2sql: Update API docs to include alternative db layer initialization https://review.openstack.org/209960 | 12:21 |
| *** funzo has joined #openstack-infra | 12:22 | |
| *** aysyd has joined #openstack-infra | 12:23 | |
| fungi | AJaeger: problem on 211843 | 12:24 |
| *** yamamoto has quit IRC | 12:25 | |
| dgonzalez | Hi all, is this the right place to talk about devstack? I can't find any dedicated devstack channel :P | 12:26 |
| AJaeger | fungi: thanks for catching that! You're right. | 12:26 |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config: Remove XML only checks from security-doc https://review.openstack.org/211843 | 12:26 |
| *** funzo has quit IRC | 12:27 | |
| dgonzalez | I stumpled over a problem with multi-region setups in devstack and proposed a fix for it: https://review.openstack.org/#/c/211689/ Maybe somebody could have a look? | 12:28 |
| AJaeger | fungi, thanks for the reviews! | 12:28 |
| fungi | dgonzalez: devstack is part of the qa project, so #openstack-qa is the most topical channel for it | 12:29 |
| dgonzalez | fungi: Ok thanks! i will go there to ask :) | 12:29 |
| *** fdegir has quit IRC | 12:30 | |
| *** erlon has joined #openstack-infra | 12:31 | |
| *** fdegir has joined #openstack-infra | 12:31 | |
| *** spzala has joined #openstack-infra | 12:31 | |
| *** bapalm has joined #openstack-infra | 12:31 | |
| *** rossella_s has joined #openstack-infra | 12:34 | |
| *** bnemec has joined #openstack-infra | 12:35 | |
| *** rfolco has joined #openstack-infra | 12:37 | |
| openstackgerrit | Nicko proposed openstack-infra/jenkins-job-builder: Added the possibility to configure stash publisher's credentials through jenkins_jobs.ini file https://review.openstack.org/212004 | 12:38 |
| *** bharath has joined #openstack-infra | 12:38 | |
| *** notnownikki has joined #openstack-infra | 12:40 | |
| *** larainema has quit IRC | 12:40 | |
| *** larainema has joined #openstack-infra | 12:40 | |
| *** andersonvom has joined #openstack-infra | 12:41 | |
| *** bharath has quit IRC | 12:43 | |
| *** radez is now known as radez_g0n3 | 12:43 | |
| *** akshai has joined #openstack-infra | 12:43 | |
| *** alexpilotti has quit IRC | 12:44 | |
| *** alexpilotti has joined #openstack-infra | 12:45 | |
| openstackgerrit | Nicko proposed openstack-infra/jenkins-job-builder: Customized pollscm trigger https://review.openstack.org/211993 | 12:45 |
| *** andersonvom has quit IRC | 12:45 | |
| *** jcoufal has joined #openstack-infra | 12:46 | |
| *** jcoufal has quit IRC | 12:46 | |
| *** jsavak has joined #openstack-infra | 12:46 | |
| *** jcoufal has joined #openstack-infra | 12:46 | |
| *** Somay has joined #openstack-infra | 12:47 | |
| *** placko_ has joined #openstack-infra | 12:47 | |
| *** ParsectiX has quit IRC | 12:49 | |
| placko_ | hello, is here somebody working on/with jenkins job builder? | 12:49 |
| *** tlian has joined #openstack-infra | 12:50 | |
| *** yamamoto has joined #openstack-infra | 12:51 | |
| *** ParsectiX has joined #openstack-infra | 12:52 | |
| Shrews | fungi: morning. qq... do you know where openrc gets sourced before running a job? | 12:54 |
| *** yamamoto has quit IRC | 12:54 | |
| *** zz_dimtruck is now known as dimtruck | 12:55 | |
| *** thiagop has joined #openstack-infra | 12:55 | |
| openstackgerrit | Thiago Paiva Brito proposed openstack-infra/project-config: Adds python-oneviewclient project https://review.openstack.org/211301 | 12:56 |
| *** otter768 has joined #openstack-infra | 12:58 | |
| *** mase_x200 has joined #openstack-infra | 12:59 | |
| *** akshai has quit IRC | 12:59 | |
| *** bhunter71 has joined #openstack-infra | 13:00 | |
| *** nelsnelson has joined #openstack-infra | 13:01 | |
| *** nelsnelson has quit IRC | 13:01 | |
| *** nelsnelson has joined #openstack-infra | 13:02 | |
| *** otter768 has quit IRC | 13:03 | |
| Shrews | fungi: nm. found it | 13:06 |
| fungi | placko_: the primary reviewers for jjb in this channel at the moment are pelix, zaro, mgagne and waynr | 13:07 |
| *** flepied has quit IRC | 13:07 | |
| fungi | Shrews: cool, because i was about to ask you an even stupider question ;) | 13:07 |
| Shrews | fungi: i am the master of stupid questions | 13:07 |
| fungi | is openrc the config file for python-openstackclient? | 13:08 |
| fungi | or something else? | 13:08 |
| *** flepied has joined #openstack-infra | 13:08 | |
| timrc | jeblair, So I just go through reading the meeting chat log. Regarding something like infra-ansible, which I'm thinking was a complete surprise to a lot of folks upstream. One of the concerns raised was to wait for consensus to start working on a problem. Does upstream have a way of supporting skunkworks projects? Stackforge does not seem to be the appropriate namespace. | 13:10 |
| timrc | got through* | 13:10 |
| timrc | While people are agreeing on architecture, I don't think that should block folks from experience and gaining intelligence around the problem. | 13:11 |
| timrc | Argh... typing... bad. | 13:11 |
| timrc | s/experience/experimenting/ | 13:11 |
| *** Somay has quit IRC | 13:11 | |
| *** andersonvom has joined #openstack-infra | 13:13 | |
| *** Somay has joined #openstack-infra | 13:13 | |
| *** ilyashakhat has quit IRC | 13:14 | |
| *** skraynev has quit IRC | 13:15 | |
| *** erikwilson_ has joined #openstack-infra | 13:15 | |
| *** MF2 has joined #openstack-infra | 13:17 | |
| *** mase_x200 has quit IRC | 13:17 | |
| *** mase_x200 has joined #openstack-infra | 13:17 | |
| MF2 | Hi, I have 2 homogeneous cloud platform (openstack) with 2 dashboard, I want to integrate that 2 dashboard with one dashboard, can I do that? | 13:17 |
| openstackgerrit | David Shrewsbury proposed openstack-infra/shade: Add flavor access API https://review.openstack.org/211710 | 13:18 |
| Shrews | fungi: it's the devstack user env file | 13:18 |
| *** BharatK has quit IRC | 13:19 | |
| *** Sree has joined #openstack-infra | 13:19 | |
| Shrews | fungi: turns out it is sourced in post_test_hook | 13:19 |
| *** mwagner_lap has joined #openstack-infra | 13:19 | |
| jasondotstar | pabelanger: as I was asking... what are you thoughts about how we should go about testing the restore-from-backup process? | 13:19 |
| jasondotstar | s/you/your | 13:19 |
| pabelanger | jasondotstar: fungi: Ya, so I'm curious how people currently see it working. I mean, right now it is a manual process if I understand correctly. Perhaps people would like to see this process automated and consuming backups more frequently | 13:20 |
| *** xyang1 has joined #openstack-infra | 13:21 | |
| *** amitgandhinz has joined #openstack-infra | 13:22 | |
| *** ilyashakhat has joined #openstack-infra | 13:23 | |
| *** skraynev has joined #openstack-infra | 13:23 | |
| fungi | MF2: i think you are likely asking in the wrong channel. we're the sysadmins for the developer community infrastructure | 13:23 |
| Clint | i wonder why pypi doesn't have launchpadlib 1.10.3 | 13:23 |
| fungi | Shrews: thanks! adding to "things i learned today..." ;) | 13:23 |
| placko_ | fungi: thanks! | 13:24 |
| MF2 | fungi: thanks, which channel, can i ask my question? opnstack? | 13:24 |
| fungi | pabelanger: jasondotstar: it's not _even_ a manual process at the moment. let's start by making one, even if it's brief-lived (even if it only gets used once or twice) so we can refine the process some before thinking about the potential for automation | 13:25 |
| fungi | Clint: i think they release it on launchpad instead? :/ | 13:25 |
| fungi | MF2: #openstack likely, yes | 13:25 |
| *** yamamoto has joined #openstack-infra | 13:25 | |
| *** jcoufal has quit IRC | 13:25 | |
| Clint | i'm going to have words with barry or dmitri asap | 13:26 |
| jasondotstar | are we thinking a bash script + cron or something of the sort ? | 13:26 |
| fungi | Clint: i'm verifying that assumption right now, but it seems likely anyway | 13:26 |
| *** MF2 has left #openstack-infra | 13:26 | |
| jasondotstar | fungi, pabelanger ^ | 13:26 |
| fungi | jasondotstar: i'm thinking an etherpad with the plan written out for how to consistently test it manually | 13:27 |
| fungi | jasondotstar: premature automation is akin to premature optimization | 13:27 |
| jasondotstar | fungi: agreed. the pre-planning. gotcha. | 13:27 |
| *** Somay has quit IRC | 13:27 | |
| pabelanger | jasondotstar: fungi: Ya, etherpad would be good start. Been a while since I did backup management, going to need to google a bit to see how others do it today. | 13:28 |
| pabelanger | I mean, can't go wrong with mysqldump and crontab, but must be some new hotness | 13:29 |
| pabelanger | :) | 13:29 |
| placko_ | pelix, zaro, mgagne, waynr: hi, could some of you help me with 405 error I get when creating new job? http://pastebin.com/q4GYb9gn | 13:29 |
| *** mriedem has joined #openstack-infra | 13:29 | |
| fungi | Clint: confirmed. 1.10.3 released to launchpad, but 1.10.2 is the latest uploaded to pypi (so you have to jump through the hoops to reenable link spidering in modern pip if you want it to find the latest release) | 13:30 |
| *** puranamr has joined #openstack-infra | 13:31 | |
| *** tiswanso has joined #openstack-infra | 13:32 | |
| fungi | jasondotstar: pabelanger: i recommend starting by deconstructing our puppet-bup module and reading a little of the bup documentation first. also see http://docs.openstack.org/infra/system-config/sysadmin.html#backups | 13:32 |
| jasondotstar | ack | 13:32 |
| *** BharatK has joined #openstack-infra | 13:32 | |
| pabelanger | fungi: are you invisioning using something like bacula to handle the process or something more home grown? | 13:35 |
| pabelanger | envisioning* | 13:35 |
| *** alivigni has joined #openstack-infra | 13:35 | |
| fungi | pabelanger: we already have a backup mechanism in place, so probably don't need something else with so much similar functionality | 13:35 |
| *** ayoung has joined #openstack-infra | 13:36 | |
| *** puranamr has quit IRC | 13:36 | |
| pabelanger | fungi: ack, let me check out what we have then | 13:36 |
| fungi | pabelanger: mainly just wanting 1) a plan for manually validating our current backups, 2) thoughts about how we might be able to perform the same validation in an automated manner in the future | 13:36 |
| pelix | placko_: what version of jenkins and python-jenkins are you using? | 13:36 |
| *** test has joined #openstack-infra | 13:37 | |
| *** test is now known as Guest16237 | 13:37 | |
| *** dtantsur is now known as dtantsur|brb | 13:37 | |
| Odd_Bloke | With stackforge/ being deprecated, will existing projects be required to move over to openstack/? Or is it just that new projects will only be created there? | 13:37 |
| *** puranamr has joined #openstack-infra | 13:38 | |
| *** funzo has joined #openstack-infra | 13:38 | |
| pabelanger | Odd_Bloke: projects that want to be under big tent, will be moved to openstack/. Other wise, project will eventually have to move out of stackforge to some other location | 13:39 |
| Odd_Bloke | pabelanger: Ack, understood. Thanks. :) | 13:39 |
| *** stevemar has joined #openstack-infra | 13:39 | |
| *** alivigni has quit IRC | 13:39 | |
| openstackgerrit | Merged openstack-infra/project-config: Add experimental postgres functional job for Ceilometer https://review.openstack.org/211646 | 13:40 |
| fungi | Odd_Bloke: it's more subtle. new projects (in the governance sense of the term) will create their repos in the openstack namespace. existing projects have the option of creating new repos in the stackforge namespace for now if they already have one or more in that namespace. any project that wants to schedule its repos to be renamed into the openstack namespace are free to do so without governance | 13:40 |
| fungi | involvement. at _some_ undefined point in the future, we'll likely rename the existing stackforge projects into the common openstack namespace | 13:40 |
| fungi | pabelanger: what "other location?" | 13:41 |
| Odd_Bloke | I assumed that 'other location' was 'not on OpenStack infra at all'. | 13:41 |
| pabelanger | fungi: something outside of openstack control? I was under the assumption that stackforge will eventually go dark (read only mode) some time down the road | 13:41 |
| AJaeger | pabelanger: that was basically the first draft but the one that was approved is different | 13:42 |
| *** akshai has joined #openstack-infra | 13:42 | |
| AJaeger | pabelanger: see http://governance.openstack.org/resolutions/20150615-stackforge-retirement.html for the final version | 13:42 |
| *** jaypipes has joined #openstack-infra | 13:42 | |
| pabelanger | AJaeger: Ah, thanks for the heads up. Reading now | 13:42 |
| fungi | pabelanger: Odd_Bloke: ahh, yes so that was an earlier iteration of the plan, but the tc took a pass on deciding whether stackforge itself will ultimately "go away." for now all that's been decided is that the stackforge git namespace should be deprecated | 13:42 |
| *** funzo has quit IRC | 13:43 | |
| *** stevemar has quit IRC | 13:43 | |
| *** alivigni has joined #openstack-infra | 13:43 | |
| fungi | so that at some point in the future we can stop renaming projects between namespaces | 13:43 |
| fungi | you no longer need to be an "official" (big tent) openstack project to have repos in the openstack git namespace | 13:44 |
| *** pradk has joined #openstack-infra | 13:44 | |
| *** pradk has quit IRC | 13:44 | |
| dstufft | fungi: clarkb whoever else I forget who hates the old cffi thing, cryptography 1.0 is out which no longer does any runtime magic via cffi | 13:45 |
| fungi | dstufft: yay! the future is now | 13:46 |
| dstufft | all the magic is in the build process now :D | 13:46 |
| dstufft | also imports are way faster | 13:46 |
| dstufft | and less memory | 13:46 |
| dstufft | ~25MB of ram on import is gone, and imports happen in a handful of ms instead of > 1s | 13:47 |
| AJaeger | fungi, could you have a look at your change https://review.openstack.org/203099 and answer jeblair's question, please? | 13:48 |
| pabelanger | fungi: clarkb: Can we enable lint checks again for system-config? Since nibalizer and yolanda smacked my hand for not aligning arrows. | 13:48 |
| pabelanger | puppet* lint | 13:48 |
| *** mrunge has quit IRC | 13:49 | |
| *** smoser has joined #openstack-infra | 13:49 | |
| openstackgerrit | Paul Belanger proposed openstack-infra/system-config: Add serveraliases for trystack.org https://review.openstack.org/211205 | 13:49 |
| fungi | pabelanger: pretty sure we decided _not_ to lint the system-config repo because we don't care about things like arrow alignment in it (we're not going to publish that one to the forge) | 13:49 |
| fungi | nibalizer: yolanda: ^ ? | 13:49 |
| fungi | let's make sure we're all on the same page with regards to review consistency | 13:50 |
| pabelanger | fungi: that was my thoughts too. however more a few times -1 reviews have resulted in unaligned arrows | 13:50 |
| *** tsekiyama has joined #openstack-infra | 13:50 | |
| pabelanger | I perfer to lint honestly | 13:50 |
| fungi | yeah, rule of thumb is, if we're not style-checking a repo, don't -1 for style nits | 13:50 |
| notnownikki | hi all, little change we've been using downstream to make puppet apply failures easier to read :) https://review.openstack.org/#/c/211249/ | 13:51 |
| *** links has quit IRC | 13:51 | |
| fungi | and if we _are_ style-checking a repo, the job should -1 any egregious style issues, so also don't -1 for style nits | 13:51 |
| fungi | basically, never -1 for style nits | 13:51 |
| fungi | pointing out and fixing actual bugs is a much more productive way to spend our collective time | 13:52 |
| pabelanger | I don't mind people pointing them out, but we have gate checks that could do it. Which, is more efficient then human eyes :) | 13:53 |
| yolanda | fungi, there is an effort on that by Colleen/Spencer, and some efforts and reviews to force and fix style, even for system-config | 13:54 |
| *** fhubik_brb is now known as fhubik | 13:54 | |
| fungi | yolanda: exciting, though we should get some consensus from our core review team that it's desired in that case | 13:54 |
| Daviey | fungi: Hmm, why is -1 so bad? If it is something that needs fixing then -1 seems appropriate (-2 inappropriate for your reasons stated).. Setting -1 allows me to easily filter out reviews i've already looked at which is scrubbed at the next push.. which is needed anyway to fix Verification. | 13:55 |
| fungi | the last time the discussion came up that i remember, we were collectively against conforming to puppet style conventions, but conceded that we would for repos we're planning to publish on puppetforge | 13:55 |
| yolanda | fungi, pabelanger, so can we wait a bit longer, for nibalizer or crinkle to join, as they are the ones driving that effort? | 13:55 |
| *** bswartz has quit IRC | 13:56 | |
| *** annegentle has joined #openstack-infra | 13:56 | |
| fungi | Daviey: -1 is fine for pointing out actual problems. using -1 to ask open-ended questions or quibble over non-semantic whitespace is a waste of lots of people's time | 13:56 |
| *** redixin has quit IRC | 13:57 | |
| *** jistr is now known as jistr|mtg | 13:57 | |
| pabelanger | yolanda: we can revisit, that is fine. I got the first lint checks online 2.5 years ago for openstack-infra/config. So, don't mind having them. Like I said, removing the lint gate to only have code reviews nit over, lint issues, doesn't make sense to me. | 13:57 |
| fungi | pabelanger: what happened was that puppet-lint got a lot more strict and added all sorts of new rules after we originally implemented it, some of which made things less readable. we figured in the end that not linting system-config was simpler all around and dropped it there | 13:58 |
| fungi | if i find time later, i'll pull up the conversation from around when that decision was made | 13:59 |
| *** sridhar_ram has joined #openstack-infra | 14:00 | |
| yolanda | fungi, pabelanger, i think that we should be consistent on that rules. If we do for the modules, do the same for system-config. I agree that right now having that lint rules on system-config cannot be done because there is lot of tech debt, but it could be done in the future | 14:00 |
| *** radez_g0n3 is now known as radez | 14:00 | |
| *** sdake_ has joined #openstack-infra | 14:00 | |
| yolanda | maybe add it at non-voting as first step? | 14:00 |
| *** mriedem has quit IRC | 14:01 | |
| *** rossella_ has joined #openstack-infra | 14:01 | |
| pabelanger | yolanda: that is part of the problem, we did have it before. And was removed :( | 14:01 |
| *** Somay has joined #openstack-infra | 14:01 | |
| pabelanger | IIRC, issues like code changes that affected git blame were part of the issue | 14:01 |
| *** hdd has quit IRC | 14:02 | |
| yolanda | so i think that it should be enabled, i'm newer to puppet but i think we should be coherent | 14:02 |
| *** hdd has joined #openstack-infra | 14:02 | |
| *** rossella_s has quit IRC | 14:02 | |
| pabelanger | But, yes. Let us revist, discuss, and have everybody agree. And if yes, lint check. Get the gate back going | 14:02 |
| fungi | yes, having to reindent an entire block because one variable name was added is terrible for tracking down actual git history | 14:03 |
| *** tsekiyama has quit IRC | 14:03 | |
| yolanda | really i care much more about functionality than style but if we have that style on puppet modules, we should have that on system-config at the long term | 14:03 |
| *** ajmiller has quit IRC | 14:03 | |
| *** nadya has quit IRC | 14:03 | |
| yolanda | fungi, but then we'll have same problem on puppet modules,and as we are progressing, we are having more functionality on modules, less on system-config | 14:03 |
| pabelanger | yolanda: that was my standpoint about 5 months ago. But was shot down :) | 14:03 |
| fungi | if we could just get rid of the arrow alignment check, that alleviate ~90% of my concern. it's an absolutely silly (and harmful) style requirement | 14:03 |
| *** barnaby has quit IRC | 14:03 | |
| yolanda | ++ | 14:03 |
| yolanda | arrow alignment is the worse for me | 14:04 |
| yolanda | i always fail on that honestly and it's a pain | 14:04 |
| pabelanger | fungi: that is easy | 14:04 |
| yolanda | but if we do, let's do everywhere, i don't see why system-config is an exception | 14:04 |
| pabelanger | PuppetLint.configuration.send('disable_arrow_alignment') | 14:04 |
| fungi | people closer to upstream puppet/ruby however said it was very necessary for anything we want to upload to the forge, so we agreed to keep running it on other modules | 14:04 |
| *** nadya has joined #openstack-infra | 14:05 | |
| fungi | anyway, let's hold the conversation until more of the original participants are around and awake | 14:05 |
| openstackgerrit | Kurt Taylor proposed openstack-infra/infra-specs: Host an initial CI monitoring dashboard service https://review.openstack.org/194437 | 14:06 |
| yolanda | sure, i was not really involved on that, but want to follow the established rules for puppet style | 14:06 |
| *** mriedem has joined #openstack-infra | 14:06 | |
| *** regXboi has joined #openstack-infra | 14:06 | |
| fungi | since i don't want to misrepresent their positions | 14:07 |
| *** ParsectiX has quit IRC | 14:07 | |
| yolanda | yes, i cannot talk about community needs or requirements | 14:07 |
| fungi | alternatively, it could make more sense to relocate the discussion to our mailing list | 14:09 |
| *** Somay has quit IRC | 14:10 | |
| *** austin81 has joined #openstack-infra | 14:11 | |
| odyssey4me | jeblair when you're back online, we'd appreciate your final nod on https://review.openstack.org/210498 | 14:11 |
| *** w_ has joined #openstack-infra | 14:12 | |
| *** abregman is now known as abregman|brb | 14:12 | |
| openstackgerrit | Clint Adams proposed openstack-infra/release-tools: python3-compatible syntax https://review.openstack.org/211972 | 14:13 |
| *** sigmavirus24_awa is now known as sigmavirus24 | 14:13 | |
| *** olaph has quit IRC | 14:13 | |
| *** bhunter71 has quit IRC | 14:14 | |
| *** yamahata has joined #openstack-infra | 14:14 | |
| *** Sree__ has joined #openstack-infra | 14:15 | |
| openstackgerrit | Paul Belanger proposed openstack-infra/system-config: Upgrade grafana.o.o to 2.1.0 https://review.openstack.org/209218 | 14:15 |
| yolanda | fungi, i think that posting to the mailinig list is a good idea, it will help everyone to follow the conversation. pabelanger, do you want to send? i'll answer with my opinion | 14:16 |
| placko_ | pelix: Jenkins Job Builder version: 1.2.0 and jenkins 1.609.2 | 14:17 |
| openstackgerrit | Paul Belanger proposed openstack-infra/system-config: Upgrade grafana.o.o to 2.1.0 https://review.openstack.org/209218 | 14:17 |
| openstackgerrit | Paul Belanger proposed openstack-infra/system-config: Bump puppetlabs-apt to 2.1.0 https://review.openstack.org/209195 | 14:17 |
| fungi | yolanda: agreed, mailing list is much better for asynchronous discussion | 14:17 |
| *** Sree has quit IRC | 14:18 | |
| pabelanger | yolanda: ya, I'll fire something other today | 14:18 |
| yolanda | ++ | 14:18 |
| pabelanger | s/other/out/ | 14:19 |
| yolanda | understood :) | 14:20 |
| *** dkranz has quit IRC | 14:20 | |
| pabelanger | Always makes sense in my head, but fingers do something else :) | 14:20 |
| *** puranamr has quit IRC | 14:20 | |
| yolanda | ah, tell me that, half of my sentences are bad :) | 14:20 |
| *** barnaby has joined #openstack-infra | 14:20 | |
| *** wznoinsk has joined #openstack-infra | 14:22 | |
| *** aveiga has left #openstack-infra | 14:22 | |
| *** puranamr has joined #openstack-infra | 14:22 | |
| *** Sree__ has quit IRC | 14:24 | |
| *** Sree has joined #openstack-infra | 14:24 | |
| placko_ | pelix: and it's same with latest version from git | 14:26 |
| pelix | placko_: same version of python-jenkins? | 14:26 |
| pelix | or latest version of python-jenkins? | 14:26 |
| *** wenlock has joined #openstack-infra | 14:26 | |
| *** Sree has quit IRC | 14:27 | |
| *** bharath has joined #openstack-infra | 14:27 | |
| placko_ | pelix: latest version, I just cloned and installed it | 14:27 |
| *** Sree has joined #openstack-infra | 14:27 | |
| pelix | I'm trying to see where abouts the change to switch to use to the POST method for creating jobs occurred in python-jenkins, since I believe the old way may have used a GET request incorrectly and support for that method was removed from jenkins | 14:27 |
| openstackgerrit | Benjamin Koep proposed openstack-infra/jenkins-job-builder: Added gitlab plugin https://review.openstack.org/211491 | 14:30 |
| *** mtanino has joined #openstack-infra | 14:30 | |
| *** stevemar has joined #openstack-infra | 14:30 | |
| pelix | placko_: btw, you still didn't indicate what version of python-jenkins are you using since that is where the API method for communication would be selected | 14:31 |
| *** bharath has quit IRC | 14:32 | |
| placko_ | pelix: ah, sry, 0.4.7 | 14:32 |
| *** Sree has quit IRC | 14:32 | |
| *** puranamr has quit IRC | 14:32 | |
| *** vlaza has quit IRC | 14:32 | |
| *** ildikov has quit IRC | 14:33 | |
| *** yamahata has quit IRC | 14:33 | |
| *** dkranz has joined #openstack-infra | 14:33 | |
| *** puranamr has joined #openstack-infra | 14:34 | |
| *** jistr|mtg is now known as jistr | 14:38 | |
| *** mase_x200 has quit IRC | 14:38 | |
| *** ajmiller has joined #openstack-infra | 14:38 | |
| *** bharath has joined #openstack-infra | 14:39 | |
| *** thedodd has joined #openstack-infra | 14:39 | |
| pelix | definitely new enough | 14:40 |
| placko_ | pelix: I also tried now to install python-jenkins from git, still same error | 14:40 |
| *** annegentle has quit IRC | 14:41 | |
| thiagop | Somebody could take a look at this new project config? https://review.openstack.org/#/c/211301/ | 14:42 |
| *** BharatK has quit IRC | 14:43 | |
| *** spzala has quit IRC | 14:43 | |
| *** mriedem1 has joined #openstack-infra | 14:43 | |
| *** mdenny has joined #openstack-infra | 14:44 | |
| *** mriedem has quit IRC | 14:44 | |
| *** afazekas has quit IRC | 14:44 | |
| pelix | placko_: I would suggest running "curl -s -D - <jenkins_url> -o /dev/null" and confirm that you actually see X-Jenkins in the headers listed | 14:44 |
| pelix | or from python python -c 'import jenkins; j = jenkins.Jenkins("<jenkins-url>"); print(j.get_version())' | 14:44 |
| *** abregman|brb is now known as abregman | 14:46 | |
| *** EricGonczer_ has joined #openstack-infra | 14:46 | |
| pelix | placko_: basically you'll have to confirm your requests are reaching a valid jenkins , then look to determine why a POST request is getting rejected | 14:46 |
| pelix | It could be a permissions issue, but I didn't think that jenkins returned that type of message | 14:48 |
| *** sabeen1 has joined #openstack-infra | 14:48 | |
| pelix | for when you didn't have access to create or reconfigure a job | 14:48 |
| regXboi | ping - need a little help tracking down how a job in the neutron check pipeline gets triggered ... | 14:48 |
| *** doug-fish has joined #openstack-infra | 14:50 | |
| openstackgerrit | Masayuki Igawa proposed openstack-infra/subunit2sql: Fix env command path https://review.openstack.org/212052 | 14:50 |
| *** tonytan4ever has joined #openstack-infra | 14:50 | |
| *** jsavak has quit IRC | 14:51 | |
| *** dannywilson has joined #openstack-infra | 14:51 | |
| *** tsekiyama has joined #openstack-infra | 14:51 | |
| pelix | placko_: or perhaps you have apache or another webserver in front of jenkins that is rejecting POST requests? | 14:51 |
| regXboi | n/m - figured it out | 14:51 |
| *** jsavak has joined #openstack-infra | 14:51 | |
| *** warpc__ has joined #openstack-infra | 14:52 | |
| *** larainema has quit IRC | 14:53 | |
| placko_ | pelix: I don't think that's the case, because I'm able to add new job via rest api with curl | 14:53 |
| openstackgerrit | Benjamin Koep proposed openstack-infra/jenkins-job-builder: Added gitlab plugin https://review.openstack.org/211491 | 14:54 |
| pabelanger | fungi: yolanda: email sent to -infra ML | 14:54 |
| *** larainema has joined #openstack-infra | 14:54 | |
| pelix | placko_: can you share the command line that worked? | 14:55 |
| pabelanger | opps, wrong email address | 14:55 |
| pabelanger | .trolo | 14:55 |
| *** minwang2 has joined #openstack-infra | 14:55 | |
| pelix | minus auth header of course ;) | 14:55 |
| *** jsavak has quit IRC | 14:56 | |
| placko_ | curl -X POST "https://username:pwd@server/createitem?name=jobname" --data-binary "@job-file.xml" -H "Content-Type: text/xml" | 14:56 |
| *** jsavak has joined #openstack-infra | 14:56 | |
| pelix | placko_: could you confirm that you were able to get the version with the python request? | 14:56 |
| yolanda | pabelanger, cool, i'll answer with my opinion there | 14:57 |
| openstackgerrit | Ryan Moats proposed openstack-infra/project-config: Replace gate-tempest-dsvm-neutron-full with multinode version https://review.openstack.org/212058 | 14:57 |
| placko_ | pelix: with python request I had no output, same for curl command you sent | 14:57 |
| *** zeih has joined #openstack-infra | 14:57 | |
| placko_ | what was actually expected? | 14:57 |
| pelix | curl request should have dumped the response headers to the stdout | 14:57 |
| *** dizquierdo has quit IRC | 14:58 | |
| *** bswartz has joined #openstack-infra | 14:58 | |
| *** otter768 has joined #openstack-infra | 14:59 | |
| *** zeih has quit IRC | 14:59 | |
| pelix | try 'curl -D - <jenkins_url>' instead and see if it reports some errors | 14:59 |
| *** patrickeast has joined #openstack-infra | 15:00 | |
| *** rkukura has joined #openstack-infra | 15:01 | |
| placko_ | pelix: headers are there, I forgot to add -k --ciphers rsa_aes_128_sha option | 15:02 |
| placko_ | so that's why I had no output, seems erver needs some update | 15:03 |
| *** alexpilotti has quit IRC | 15:03 | |
| pelix | hmm, I wonder if we don't support insecure properly | 15:04 |
| *** otter768 has quit IRC | 15:04 | |
| crinkle | yolanda: pabelanger: fungi: I was told by jeblair that we decided we are okay with the arrow alignment checks http://eavesdrop.openstack.org/irclogs/%23openstack-infra/%23openstack-infra.2015-07-14.log.html#t2015-07-14T16:17:07 | 15:04 |
| *** sridhar_ram has quit IRC | 15:05 | |
| yolanda | wow... we need to do this publicly, arrive to a consensus, either on mailing list or in meetings | 15:05 |
| crinkle | the gemfiles that I've been adding install a newer version of puppet-lint that properly checks for arrow alignment, which the old version we've been using (0.3.2) did not | 15:05 |
| *** DuncanT has quit IRC | 15:05 | |
| *** isviridov_away has quit IRC | 15:05 | |
| *** vaidy has quit IRC | 15:05 | |
| *** rm_work has quit IRC | 15:05 | |
| yolanda | i just follow trend on that, and -1 on lint as i saw in other reviews, but we need to have an official agreement | 15:05 |
| crinkle | to disable the checks we'd need to specifically disable it in the rakefile | 15:05 |
| *** matt_____ has joined #openstack-infra | 15:06 | |
| crinkle | I thought there was a patch a while ago where it was discussed? | 15:06 |
| *** thingee has quit IRC | 15:06 | |
| * crinkle will look for it | 15:06 | |
| placko_ | pelix: can be, I also turn of certificate verification because although it's present in bundle file which python reads, it just ignores it, so I turn verification of globaly | 15:06 |
| *** GheRivero has quit IRC | 15:06 | |
| *** mattoliverau has quit IRC | 15:06 | |
| *** gus has quit IRC | 15:06 | |
| *** freyes has quit IRC | 15:06 | |
| pabelanger | crinkle: yes, but what about system-config? That is where this discussion is coming from. Check -infra mailing list for my reply / thoughts | 15:07 |
| *** DuncanT has joined #openstack-infra | 15:08 | |
| *** abregman has quit IRC | 15:08 | |
| pabelanger | FWIW: Our current openstack-infra/puppet-* lint checks and gate is fine to me. | 15:08 |
| pabelanger | we should enable them for system-config too | 15:08 |
| *** gus has joined #openstack-infra | 15:08 | |
| yolanda | i agree with pabelanger on that | 15:08 |
| pelix | placko_: the other alternative is whether we handle username/password in the url correctly | 15:08 |
| yolanda | if we have lint, let's do it everywhere | 15:08 |
| crinkle | I'm working on doing it everywhere | 15:08 |
| crinkle | it takes a lot of time | 15:08 |
| jeblair | pabelanger: actually the problem is that a human should never -1 for a whitespace nit | 15:08 |
| *** funzo has joined #openstack-infra | 15:09 | |
| jeblair | pabelanger: so you are right -- you should not have to deal with the uncertainty there | 15:09 |
| *** thingee has joined #openstack-infra | 15:09 | |
| jeblair | but the solution is for all of us to stop wasting time on useless little things like that | 15:09 |
| jeblair | that means reviewers too | 15:09 |
| jeblair | code review is for finding problems before they happen | 15:09 |
| jeblair | rocketship alignment is not a problem | 15:10 |
| *** freyes has joined #openstack-infra | 15:10 | |
| jeblair | "this won't work because...." is a problem | 15:10 |
| *** GheRivero has joined #openstack-infra | 15:10 | |
| *** rm_work has joined #openstack-infra | 15:10 | |
| *** rm_work has quit IRC | 15:10 | |
| *** rm_work has joined #openstack-infra | 15:10 | |
| pabelanger | Right. I think this is some of the issue. We don't have a clear policy for system-config. Even though it is a puppet module, it handles lint checks different then other puppet modules | 15:11 |
| pelix | placko_: but I would presume so since the plugin info should have been retrieved before the first job is attempted to be created, so it still comes back to GET requests succeed while POST requests get a 405 | 15:11 |
| pabelanger | I can go either way. I just need to know what that way is :) | 15:11 |
| *** vaidy has joined #openstack-infra | 15:11 | |
| yolanda | we have to arrive on an agreement for it really, but maintain same criteria in all reviews | 15:12 |
| AJaeger | pabelanger: and then we should document it so that we don't have this discussion in a few months again ;) | 15:12 |
| jeblair | pabelanger: yep, fair. we do have a clear policy, and it is in place for each and every one of the infra repos: do not -1 for nitpicks. do not leave reviews of whitespace. | 15:12 |
| pabelanger | AJaeger: Yup, sounds like I'm doing a infra-spec | 15:12 |
| *** isviridov_away has joined #openstack-infra | 15:12 | |
| openstackgerrit | Marian Horban proposed openstack/requirements: Add pyinotify to global-requirements.txt https://review.openstack.org/211975 | 15:13 |
| *** funzo has quit IRC | 15:13 | |
| jeblair | pabelanger: i'm writing something now | 15:14 |
| *** GheRivero is now known as GheRivero2 | 15:14 | |
| *** GheIrcCloud is now known as GheRivero | 15:14 | |
| *** kmartin has joined #openstack-infra | 15:15 | |
| *** GheRivero is now known as Gheirccloud | 15:15 | |
| *** GheRivero2 is now known as GheRivero | 15:15 | |
| rcarrillocruz | otoh, i really really wish Gerrit would notify appropiately 0 scores | 15:15 |
| rcarrillocruz | to me , i see a lot of times things that are nits that I refuse to -1 | 15:16 |
| rcarrillocruz | i put a 0 | 15:16 |
| yolanda | yes, some of the -1 will be just a comment | 15:16 |
| yolanda | but then people don't noticed them, it's a very common issue | 15:16 |
| rcarrillocruz | and because that has nil visibility to the committer it goes ignored | 15:16 |
| *** GheRivero is now known as GheRivero2 | 15:16 | |
| *** Gheirccloud is now known as GheRivero | 15:16 | |
| rcarrillocruz | GheRivero: so fancy you, irccloud huh? | 15:16 |
| rcarrillocruz | :P | 15:16 |
| jeblair | rcarrillocruz: i don't find that's the case; let me know if you see instances of people ignoring your comments and i'll talk to them | 15:17 |
| openstackgerrit | Merged openstack-infra/subunit2sql: Fix env command path https://review.openstack.org/212052 | 15:17 |
| rcarrillocruz | well not a personal thing, is just that is easy to not notice 0 scores... | 15:17 |
| GheRivero | rcarrillocruz: stupid nickserv and invitations | 15:17 |
| openstackgerrit | Merged openstack-infra/openstackid: Correct the openstackid.org URL https://review.openstack.org/209100 | 15:17 |
| rcarrillocruz | which is why i think some reviewers would -1 those nits ( i want to think that's the cause :-) ) | 15:18 |
| *** woodster_ has joined #openstack-infra | 15:18 | |
| *** GheRivero has left #openstack-infra | 15:18 | |
| jeblair | rcarrillocruz: the score only makes sense in context of comments, all of which should be read anyway. if someone isn't reading comments then they aren't doing what we're asking of them | 15:18 |
| yolanda | jeblair, yes, it's quite common for reviewers to put a -1 for this reason, even if that's a question. I cannot remember any particular case now but that happens | 15:18 |
| fungi | rcarrillocruz: i think part of the problem is that there's a desire for a review score between -1 and +1, but gerrit overloads review +/-0 to mean "hasn't been reviewed" | 15:18 |
| rcarrillocruz | yep | 15:19 |
| *** tonytan4ever has quit IRC | 15:19 | |
| *** GheRivero2 is now known as GheRivero | 15:19 | |
| jeblair | yolanda: then let's call it out and ask them to stop. i put a 0 for general questions; i put a -1 for questions where things could be bad if the answer is one way or the other (and i suspect it's the bad answer) | 15:19 |
| fungi | so i can understand the "i'd like to differentiate in the votes table whether i've reviewed this even though i scored it 0" | 15:19 |
| jeblair | fungi: true, i wrote gertty to deal with that. but even so, externalizing one's own workflow onto the contributor is bad mojo. | 15:20 |
| fungi | i tend to use -1 for "i think this needs more work before i can +1/+2" and -2 for "i think this change is unneeded/harmful/in the wrong repo" | 15:20 |
| pabelanger | I tend to do 0 for straight comments, mind you I do notice people not replying to them more often then not. | 15:20 |
| rcarrillocruz | jeblair: well, maybe it's time for us to start using gertty | 15:21 |
| yolanda | pabelanger, same here | 15:21 |
| pabelanger | not sure why. If that is because they don't get notification or just ignore them | 15:21 |
| rcarrillocruz | i have to admit i used a long ago and got some freeze issues | 15:21 |
| fungi | rcarrillocruz: absolutely! ;) | 15:21 |
| jeblair | pabelanger, yolanda: that's not okay -- please call people out or let me know | 15:21 |
| rcarrillocruz | must be working now, since it's the hip thing around | 15:21 |
| * rcarrillocruz goes to clone gertty | 15:21 | |
| fungi | rcarrillocruz: it's been extremely stable for me | 15:21 |
| notnownikki | hi all, little change we've been using downstream to make puppet apply failures easier to read - https://review.openstack.org/#/c/211249/ | 15:21 |
| *** puranamr has quit IRC | 15:21 | |
| *** puranamr has joined #openstack-infra | 15:21 | |
| *** pc_m is now known as pc_m_AWAY | 15:22 | |
| AJaeger | rcarrillocruz: see also http://docs.openstack.org/infra/manual/developers.html#peer-review with point 7 on questions | 15:22 |
| pabelanger | notnownikki: do you have the output some place? | 15:22 |
| notnownikki | pabelanger, sure, will pastebin it, one moment... | 15:22 |
| openstackgerrit | Julia Kreger proposed openstack-infra/shade: WIP: Initial support for ironic enroll state https://review.openstack.org/212070 | 15:22 |
| jeblair | notnownikki, pabelanger: that change should be self | 15:23 |
| jeblair | notnownikki, pabelanger: that change should be self-testing and we can look at the output from zuul's runs, right? | 15:23 |
| *** dguerri is now known as dguerri` | 15:23 | |
| rcarrillocruz | oh, that's even documented the 0 score | 15:23 |
| rcarrillocruz | ++ | 15:23 |
| pabelanger | jeblair: ya, should have mentioned for a failure. Sorry | 15:23 |
| jeblair | pabelanger: ah, heh :) | 15:24 |
| *** angdraug has joined #openstack-infra | 15:24 | |
| jeblair | pabelanger, notnownikki: could have an intentional failure "depends-on" that patch if you wanted to generate one | 15:24 |
| AJaeger | There was some discussion a few months ago on openstack-dev on this tropic and that's why step 7 got added | 15:24 |
| pabelanger | jeblair: Ya, actually doing that now :) Figured I see if notnownikki had something handy | 15:24 |
| *** AJaeger has quit IRC | 15:24 | |
| notnownikki | jeblair, it is indeed, you can see the success style already, I could put through a failing change as a test to keep it in that review? | 15:25 |
| fungi | AJaeger: rcarrillocruz: yep, the history is in the commit message/review too if you git blame what added that section | 15:25 |
| fungi | has, i believe, links back to the related ml thread | 15:25 |
| fungi | notnownikki: that also works, just leave a comment referring to the intentionally broken patchset | 15:26 |
| openstackgerrit | Paul Belanger proposed openstack-infra/system-config: [WIP] Fail, fail, fail! https://review.openstack.org/212071 | 15:26 |
| fungi | we use that trick a lot to demonstrate things like regression fixes (add patchset with regression tests only, then follow up with a patchset including the bug fix which makes the test pass) | 15:27 |
| rcarrillocruz | pleia2: just following http://princessleia.com/journal/?p=9785 for gertty | 15:27 |
| *** tiswanso has quit IRC | 15:27 | |
| *** annegentle has joined #openstack-infra | 15:28 | |
| *** k4n0 has quit IRC | 15:28 | |
| fungi | rcarrillocruz: gertty has also made huge progress since that blog post | 15:29 |
| *** radez is now known as radez_g0n3 | 15:30 | |
| *** tiswanso has joined #openstack-infra | 15:30 | |
| fungi | though skimming, it looks like it's still basically accurate | 15:30 |
| fungi | rcarrillocruz: i recommend installing it into a virtualenv in your homedir and running it from that. makes upgrades and testing branch tip/proposed gertty changes a little easier too | 15:31 |
| rcarrillocruz | hmm, good call... | 15:31 |
| fungi | rcarrillocruz: _though_ it's also present in some distros too (for example if you run debian/testing the package there is a fairly current 1.2.1) | 15:32 |
| openstackgerrit | Vitaly Gridnev proposed openstack/requirements: Bump saharaclient to the latest version https://review.openstack.org/212072 | 15:32 |
| rcarrillocruz | i just pulled from pip, but yeah, i rather do that in a virtualenv | 15:33 |
| fungi | in fact, i say fairly current... 1.2.1 is the most recent release | 15:33 |
| rcarrillocruz | Syncing projects... | 15:33 |
| *** tvanderwiel has joined #openstack-infra | 15:33 | |
| *** tsg has joined #openstack-infra | 15:33 | |
| *** alivigni has quit IRC | 15:33 | |
| openstackgerrit | James E. Blair proposed openstack-infra/system-config: Add section on review criteria https://review.openstack.org/212073 | 15:34 |
| jeblair | fungi, yolanda, pabelanger: ^ | 15:34 |
| *** tvanderwiel has left #openstack-infra | 15:34 | |
| timrc | "<jeblair> pabelanger: yep, fair. we do have a clear policy, and it is in place for each and every one of the infra repos: do not -1 for nitpicks. do not leave reviews of whitespace." TIL - Sorry! | 15:35 |
| notnownikki | jeblair, fungi, pabelanger, example of error output - apply is pretty clean here so grepped errors aren't *that* useful, but where you have a lot of warnings, it can be useful to separate out the error lines into a separate section - http://logs.openstack.org/71/212071/1/check/gate-infra-puppet-apply-bare-precise/bc3899f/console.html#_2015-08-12_15_29_23_528 | 15:35 |
| jeblair | timrc: is okay, now we're writing it down! :) | 15:35 |
| rcarrillocruz | hmm, otoh, i need to adjust my mind for keystrokes, i a vi user and those keys seem to be used | 15:35 |
| * fungi is very much in favor of recording our intent | 15:36 | |
| timrc | jeblair, If only we could do lint checking on review comments before the post ;P | 15:36 |
| pabelanger | notnownikki: left comments in review. In general like the idea, however don't link the fact we are duplicating error output | 15:36 |
| timrc | they* | 15:36 |
| jeblair | timrc: this is your big opportunity to enhance gerrit ;) | 15:36 |
| fungi | rcarrillocruz: as a vi user myself, i had no trouble getting accustomed to gertty's keybindings, though also you can just rebind whatever keys you prefer via configuration | 15:36 |
| pabelanger | timrc: all good :) | 15:36 |
| *** tqtran has joined #openstack-infra | 15:37 | |
| *** salv-orl_ has joined #openstack-infra | 15:37 | |
| timrc | jeblair, Looks like I'm nearing the zenith of my career! | 15:37 |
| notnownikki | pabelanger, cool, I'll carry on enhancing it downstream and push up another change when it's polished | 15:37 |
| *** radez_g0n3 is now known as radez | 15:37 | |
| fungi | notnownikki: pabelanger: definitely agreed that we need to improve something there. i've been resorting to a grep for "exit [^0]" | 15:38 |
| rcarrillocruz | erm | 15:38 |
| rcarrillocruz | so | 15:38 |
| rcarrillocruz | you move over to the tests run | 15:38 |
| timrc | puppet apply tests are the worst... | 15:38 |
| rcarrillocruz | press enter | 15:38 |
| rcarrillocruz | and browser opens with the job logs | 15:38 |
| rcarrillocruz | this is very very cool | 15:38 |
| eantyshev | jhesketh: Hi! I've left a comment for you in https://review.openstack.org/#/c/207094 | 15:38 |
| openstackgerrit | greghaynes proposed openstack/diskimage-builder: Sanitize distro elements for minimal vs cloud https://review.openstack.org/211859 | 15:38 |
| pabelanger | fungi: Yup, that's what I do today | 15:38 |
| *** olaph has joined #openstack-infra | 15:38 | |
| fungi | rcarrillocruz: i invoke gertty like `BROWSER=lynx gertty` | 15:38 |
| pabelanger | notnownikki: Ya, anything to improve it is helpful | 15:38 |
| rcarrillocruz | hahaha | 15:39 |
| rcarrillocruz | terminal ftw | 15:39 |
| fungi | rcarrillocruz: lynx does an excellent job of browsing our job logs from gertty | 15:39 |
| notnownikki | great, i've got a few comments on our downstream version so I'll carry on polishing and make something useful | 15:39 |
| fungi | rcarrillocruz: note that asking firefox to load a 10k-line console log can lead to terrible performance, and lynx hardly sneezes at it | 15:40 |
| rcarrillocruz | ++ | 15:40 |
| rcarrillocruz | whenever my laptop crawls | 15:40 |
| *** salv-orlando has quit IRC | 15:40 | |
| rcarrillocruz | i know it's firefox that is updating jenkins UI jobs page | 15:40 |
| eantyshev | jeblair: please, take a look at https://review.openstack.org/#/c/207094 | 15:40 |
| notnownikki | rcarrillocruz, yolanda, timrc, I noticed the puppet apply tests on r.o.o have separate output for each test, so if we do that and only output fails and grepped errors/warnings with the name of the test that failed, we should have something quite nice | 15:40 |
| *** w_ has quit IRC | 15:41 | |
| rcarrillocruz | notnownikki: well, yeah, we should just do that | 15:41 |
| *** barnaby has quit IRC | 15:41 | |
| yolanda | yes, it's good | 15:41 |
| openstackgerrit | greghaynes proposed openstack/diskimage-builder: Sanitize distro elements for minimal vs cloud https://review.openstack.org/211859 | 15:42 |
| * notnownikki will get on it, as I've just delivered two big things and have a day to work on this stuff :) | 15:42 | |
| *** ifarkas has quit IRC | 15:42 | |
| *** jsavak has quit IRC | 15:42 | |
| timrc | gertty is a little funky... is there a key combination I can press to get a table of all the key bindings including the ones defined in .gertty.yaml? | 15:43 |
| jeblair | timrc: f1 or h | 15:43 |
| fungi | or ? | 15:43 |
| timrc | Hm... I tried "h" | 15:43 |
| *** scheuran has quit IRC | 15:43 | |
| *** marzif has quit IRC | 15:43 | |
| jeblair | sorry, ? | 15:43 |
| jeblair | not h | 15:43 |
| *** jsavak has joined #openstack-infra | 15:43 | |
| timrc | Sometimes I resort to mashing my keypad to see what happens. | 15:43 |
| timrc | Which actually probably explains some of my reviews ;-) | 15:43 |
| fungi | also note that it may be longer than your screen, so try hitting the down arrow to see more keybindings | 15:43 |
| fungi | when you see the <OK> button you're at the bottom | 15:44 |
| *** marzif has joined #openstack-infra | 15:44 | |
| timrc | fungi, jeblair Perfect! | 15:44 |
| *** geoffarnold has joined #openstack-infra | 15:44 | |
| *** arxcruz has quit IRC | 15:47 | |
| *** akshai has quit IRC | 15:47 | |
| nibalizer | oh are we having chatter about arrow alignment again? | 15:48 |
| jeblair | nibalizer: we didn't write it down last time | 15:48 |
| *** bapalm has quit IRC | 15:49 | |
| *** bapalm has joined #openstack-infra | 15:49 | |
| bdemers | are the ‘get-tripleo-ironic-*’ jobs stable ? | 15:50 |
| bdemers | I’ve seen some failures the past couple days: error: could not revert 7f8128f... Add the swap and ephemeral BDMs if needed | 15:50 |
| openstackgerrit | David Shrewsbury proposed openstack-infra/shade: Change functional testing to use clouds.yaml https://review.openstack.org/212083 | 15:54 |
| *** barnaby has joined #openstack-infra | 15:54 | |
| *** morganfainberg_ has quit IRC | 15:55 | |
| *** BharatK has joined #openstack-infra | 15:56 | |
| jeblair | fungi, clarkb, anteaya: i'm missing context because of zendesk, but i see a response from tom to the eventbrite folks that says: Can you please provide further details on the research you conducted that concluded "there was no negative impact on conversion rates for events". I would be particularly interested in details on the results you saw in testing from a "technical" demographic. | 15:57 |
| annegentle | sdague: If you can take a look at https://review.openstack.org/#/c/209669/ and let me know if that's the way forward for developer.openstack.org RST docs, I can get going on the publish jobs. | 15:58 |
| openstackgerrit | Adam Coldrick proposed openstack-infra/storyboard-webclient: Change the page title when different things are being viewed https://review.openstack.org/212085 | 15:58 |
| jeblair | fungi, clarkb, anteaya: i'm filling in the blanks as "yes, eventbrite changed that to increase their registered user base and don't think it has any impact" | 15:58 |
| *** fhubik is now known as fhubik_brb | 15:58 | |
| fungi | jeblair: that's pretty terrible | 15:58 |
| jeblair | i'm writing back privately to tom | 15:59 |
| *** radez is now known as radez_g0n3 | 15:59 | |
| *** jcoufal has joined #openstack-infra | 15:59 | |
| openstackgerrit | greghaynes proposed openstack/diskimage-builder: WIP: create growroot element https://review.openstack.org/206636 | 16:00 |
| *** skylerberg has joined #openstack-infra | 16:00 | |
| *** gyee has joined #openstack-infra | 16:00 | |
| *** nadya has quit IRC | 16:00 | |
| greghaynes | :( | 16:01 |
| greghaynes | re: eventbrite | 16:01 |
| *** akshai has joined #openstack-infra | 16:02 | |
| *** matt_____ is now known as mattoliverau | 16:03 | |
| *** jsavak has quit IRC | 16:03 | |
| fungi | seems like yet again the foundation choosing a non-open/proprietary service for something is unnecessarily excluding members of our community by doing so | 16:03 |
| openstackgerrit | greghaynes proposed openstack/diskimage-builder: create growroot element https://review.openstack.org/206636 | 16:03 |
| *** jsavak has joined #openstack-infra | 16:04 | |
| placko_ | pelix: so do have any suggestion about it? | 16:04 |
| jeblair | fungi: is there a free signup thingy? i mean, most of the openstack stuff is free anyway, it doesn't even need to process payments. | 16:04 |
| *** puranamr has quit IRC | 16:04 | |
| *** alivigni has joined #openstack-infra | 16:04 | |
| jeblair | (i used both meanings of free ^ ) | 16:05 |
| anteaya | jeblair: thanks for the update and :( | 16:05 |
| *** puranamr has joined #openstack-infra | 16:05 | |
| greghaynes | free as in open source beer | 16:05 |
| jeblair | greghaynes: cheers! | 16:06 |
| *** jlanoux has quit IRC | 16:06 | |
| *** mrmartin has quit IRC | 16:06 | |
| *** puranamr has quit IRC | 16:06 | |
| *** puranamr has joined #openstack-infra | 16:07 | |
| *** mrmartin has joined #openstack-infra | 16:07 | |
| openstackgerrit | Eric Brown proposed openstack/requirements: Bump version of bandit to 0.13.1 https://review.openstack.org/210787 | 16:07 |
| *** mrmartin has quit IRC | 16:07 | |
| *** mrmartin has joined #openstack-infra | 16:08 | |
| *** akshai has quit IRC | 16:09 | |
| *** funzo has joined #openstack-infra | 16:09 | |
| *** deepakcs has joined #openstack-infra | 16:10 | |
| *** mdbooth has quit IRC | 16:10 | |
| *** jkraj has quit IRC | 16:11 | |
| *** devvesa has quit IRC | 16:11 | |
| *** mdbooth has joined #openstack-infra | 16:12 | |
| fungi | i've closed my eventbrite account so that it will be a little more obvious when changes like this start to get introduced | 16:12 |
| *** dtantsur|brb is now known as dtantsur | 16:12 | |
| *** jistr has quit IRC | 16:13 | |
| *** sdake_ is now known as sdake | 16:13 | |
| fungi | on the account termination survey, one of the possible reasons for leaving was actually "i don't want eventbrite to save my information" | 16:13 |
| *** ivar-lazzaro has joined #openstack-infra | 16:14 | |
| *** funzo has quit IRC | 16:14 | |
| *** ivar-lazzaro has quit IRC | 16:14 | |
| morgan_404 | fungi: ping - have some questions re ATC codes. [I'm sure you've been asked this from the other side and now I'm just getting looped into it] | 16:14 |
| fungi | morgan_404: not found | 16:14 |
| morgan_404 | fungi: ^_^ | 16:15 |
| *** fhubik_brb is now known as fhubik | 16:15 | |
| fungi | morgan_404: i pinged you a few days ago if it's what i think it's about. finding now... | 16:15 |
| *** ivar-lazzaro has joined #openstack-infra | 16:15 | |
| morgan_404 | fungi: i was in australia / jetlagged | 16:15 |
| fungi | i figured | 16:15 |
| morgan_404 | fungi: this is the co-authored-by thing | 16:15 |
| anteaya | morgan_404: are you finished being jet lagged yet? this is just going on and on for me | 16:16 |
| *** sergmelikyan has joined #openstack-infra | 16:16 | |
| anteaya | the non stop construction doesn't help | 16:16 |
| morgan_404 | anteaya: i was mostly recovered by monday | 16:16 |
| morgan_404 | 2 days of bike rides helped | 16:16 |
| anteaya | the neighbours are tag teaming | 16:16 |
| * morgan_404 needs to go for another ride today soon. | 16:16 | |
| anteaya | morgan_404: ah good strategy | 16:16 |
| morgan_404 | anteaya: yeah exercise = tired-ish and sleeping more normally | 16:16 |
| fungi | morgan_404: http://eavesdrop.openstack.org/irclogs/%23openstack-infra/%23openstack-infra.2015-08-07.log.html#t2015-08-07T16:51:58 | 16:16 |
| morgan_404 | anteaya: long walks in beautiful areas away from construction i hear helps too :) | 16:17 |
| *** flepied has quit IRC | 16:17 | |
| anteaya | I wish I could find that in my life | 16:17 |
| anteaya | hopefully palo alto will be nice and quiet | 16:17 |
| morgan_404 | anteaya: eh, probably more quiet than construction zone | 16:17 |
| jeblair | neat, my hpcloud account doesn't work | 16:17 |
| anteaya | jeblair: wooooo | 16:18 |
| morgan_404 | fungi: ah ok. I'll go propose that... [or make someone else in keystone do it and +1 it] | 16:18 |
| anteaya | jeblair: you are just having all the luck | 16:18 |
| anteaya | morgan_404: one can hope | 16:18 |
| * morgan_404 hasn't tried to VPN yet. | 16:18 | |
| *** sigmavirus24 is now known as sigmavirus24_awa | 16:18 | |
| *** tonytan4ever has joined #openstack-infra | 16:18 | |
| jeblair | anteaya: the force of entropy is strong with hp; constant work is required to repair it. | 16:18 |
| morgan_404 | fungi: thnx i was seriously confused about this for a bit. | 16:18 |
| anteaya | jeblair: good thing there are many entropy fighters | 16:19 |
| morgan_404 | fungi: typically I view extra ATC as something for folks who legitimately couldn't land a patch [spec work only/comments/reviewers/bug triage] | 16:19 |
| anteaya | sigh, I think I should eat before I do internal things | 16:19 |
| anteaya | morgan_404: I agree, is that not the case in this case? | 16:19 |
| morgan_404 | fungi: since landing patches is pretty easy if you've done the work. | 16:20 |
| morgan_404 | anteaya: this is folks who co-authored patches and didn't get caught up in the ATC thing because they only co-authored | 16:20 |
| *** sergmelikyan has quit IRC | 16:20 | |
| morgan_404 | anteaya: this is a one off i'm willing to do. | 16:20 |
| fungi | morgan_404: yes, i personally think it's in terrible, terrible taste and looks bad for intel. you, as the ptl, are perfectly welcome to _not_ add them to extra-atcs | 16:20 |
| morgan_404 | fungi: let me mull on this for a bit actually | 16:21 |
| morgan_404 | fungi: i just am baffled they didn't do a simple.. i mean typo fix? | 16:21 |
| fungi | morgan_404: however, it's not my call. i just made sure they were told to go through the proper channels | 16:21 |
| anteaya | morgan_404: well how about talk to them and ask them what contribution they actually made | 16:21 |
| morgan_404 | its not that hard to land patches. | 16:21 |
| anteaya | morgan_404: riding a commit for a summit pass isn't great | 16:21 |
| fungi | morgan_404: it's a one-line change that removes the four characters "bin/" from a developer document | 16:21 |
| anteaya | actually participating in the design of something is a contribution | 16:22 |
| *** mrmartin has quit IRC | 16:22 | |
| morgan_404 | anteaya: sure. but it's a low barrier to entry | 16:22 |
| anteaya | ha ha ha | 16:22 |
| *** mrmartin has joined #openstack-infra | 16:22 | |
| anteaya | that's just funny | 16:22 |
| morgan_404 | anteaya: i mean... low. | 16:22 |
| anteaya | did they remove one character a piece | 16:22 |
| anteaya | and then throw fingers for who removed the / | 16:23 |
| morgan_404 | oh you know what no. oh hell now | 16:23 |
| morgan_404 | no* | 16:23 |
| morgan_404 | a 1 line change with co-authors | 16:23 |
| morgan_404 | .... | 16:23 |
| * morgan_404 just starts looking at it | 16:23 | |
| openstackgerrit | Merged openstack-infra/tripleo-ci: Fix PIP_INDEX_URL https://review.openstack.org/211948 | 16:23 |
| zaro | morning | 16:24 |
| fungi | morgan_404: yeah, it's absurd. 3x$600 discount codes for a 4-line change... that comes out to $450 per character removed | 16:24 |
| *** unicell1 has quit IRC | 16:25 | |
| fungi | er, 4-character change | 16:25 |
| *** bharath has quit IRC | 16:25 | |
| morgan_404 | fungi: can i get paid that much | 16:25 |
| morgan_404 | plx | 16:25 |
| morgan_404 | ;) | 16:25 |
| *** bharath has joined #openstack-infra | 16:25 | |
| anteaya | morning zaro | 16:25 |
| *** sigmavirus24_awa is now known as sigmavirus24 | 16:25 | |
| *** dolphm has left #openstack-infra | 16:26 | |
| *** alexsyip has joined #openstack-infra | 16:27 | |
| jeblair | morgan_404: yeah, forget using "x" as a variable. that's only $450. "outer_loop_iteration_counter" is a good $12,600. | 16:27 |
| jeblair | that's a days work. | 16:27 |
| * jeblair packs up | 16:27 | |
| anteaya | ha ha ha | 16:28 |
| fungi | jeblair: for intel anyway | 16:28 |
| anteaya | so yeah given that some folks want to ride the commit to the summit, I'm not a fan of auto extra ATC for anyone who puts up a fuss | 16:28 |
| anteaya | contributions? by all means | 16:28 |
| anteaya | taking advantage of a technicality, no | 16:29 |
| fungi | morgan_404: the only reason i didn't just tell them "oh hell no" myself is it sounds like they might be highschool summer interns with intel, so at least they can plead their value to the ptl of the project to which they "contributed" that change. it's bound to be a learning experience for them | 16:29 |
| krotscheck | nibalizer: I lost backscroll, did you get around to answering my question from yesterday? | 16:29 |
| jeblair | i think this may be a learning experience regardless | 16:30 |
| jeblair | krotscheck: in case that question wasn't rhetorical, there are logs available: http://eavesdrop.openstack.org/irclogs/%23openstack-infra/ | 16:31 |
| * morgan_404 just sent a deny email | 16:31 | |
| krotscheck | jeblair: It's not, thanks for the link | 16:31 |
| morgan_404 | saying that i set the "high high bar" for extra atc | 16:31 |
| jeblair | cool | 16:31 |
| odyssey4me | jeblair mind casting a final eye on https://review.openstack.org/210498 ? | 16:31 |
| morgan_404 | i mean.. extra ATC is a really high bar because landing a patch is such a low bar. | 16:31 |
| *** btran has joined #openstack-infra | 16:32 | |
| jeblair | odyssey4me: lgtm -- i don't _think_ you need the change to 'node' -- do you know of a reason for it? | 16:33 |
| krotscheck | jeblair, nibalizer, mordred: So, an alternative approach would be to add the nodejs PPA to the release node _only_ | 16:34 |
| odyssey4me | jeblair it's not necessary - but I added it to open the pool of nodes available for the lint/docs checks | 16:34 |
| odyssey4me | it can't hurt, right? | 16:34 |
| krotscheck | And then we'd be able to just use the package{} ]resource | 16:34 |
| jeblair | odyssey4me: i'm pretty sure they only run on trusty anyway | 16:34 |
| odyssey4me | jeblair meh :) | 16:35 |
| jeblair | odyssey4me: my thoughts too :) | 16:35 |
| *** pc_m_AWAY is now known as pc_m | 16:35 | |
| jeblair | krotscheck: is it like well maintained and get security updates and whatnot? | 16:35 |
| fungi | morgan_404: yeah, if they take exception to needing to submit one more patch apiece, then i'm happy for them to have the foundation board member from intel explain why this is necessary and how intel has fallen on such hard times | 16:35 |
| odyssey4me | thanks jeblair :) | 16:35 |
| anteaya | fungi: ha ha ha | 16:36 |
| krotscheck | jeblair: Googling. I think the answer is yet, but I'll confirm. | 16:37 |
| openstackgerrit | Merged openstack-infra/project-config: Add docs check to os-ansible-deployment https://review.openstack.org/210498 | 16:37 |
| *** jsavak has quit IRC | 16:37 | |
| pelix | placko_: other than trying to use the python-jenkins api directly with python calls to narrow down what might be occurring, I'm a bit stumped | 16:37 |
| morgan_404 | fungi: if it had been a significant code contribution i'd have seriously considered it | 16:37 |
| morgan_404 | fungi: but... 1 line? | 16:37 |
| *** tiswanso has quit IRC | 16:37 | |
| *** jsavak has joined #openstack-infra | 16:37 | |
| pelix | placko_: you did get the one line python script to work eventually? | 16:37 |
| annegentle | morgan_404: in keystone docs? | 16:37 |
| morgan_404 | annegentle: yeah | 16:38 |
| morgan_404 | annegentle: https://review.openstack.org/#/c/198924/ | 16:38 |
| annegentle | morgan_404: interesting. If they are high school interns it's a good lesson | 16:38 |
| *** sabeen3 has joined #openstack-infra | 16:38 | |
| pelix | placko_: as in python -c 'import jenkins; j = jenkins.Jenkins("<url>"); print(j.get_version())' | 16:38 |
| annegentle | (other said that too, just agreeing) | 16:38 |
| *** sabeen1 has quit IRC | 16:38 | |
| *** arosen has quit IRC | 16:38 | |
| *** tiswanso has joined #openstack-infra | 16:38 | |
| placko_ | pelix: I didn't try, but I guess the reason I didn't get an output is that I didn't specify cipher | 16:38 |
| morgan_404 | annegentle: nod. | 16:39 |
| openstackgerrit | Paul Belanger proposed openstack-infra/glean: Fix path for documentation URL https://review.openstack.org/212096 | 16:39 |
| placko_ | pelix: I will play with it and eventually report a bug if I will be able to narrow it down | 16:39 |
| *** salv-orl_ has quit IRC | 16:40 | |
| *** salv-orlando has joined #openstack-infra | 16:41 | |
| *** pradk has joined #openstack-infra | 16:42 | |
| *** persia has quit IRC | 16:42 | |
| *** w_ has joined #openstack-infra | 16:43 | |
| *** rfolco has quit IRC | 16:45 | |
| *** akshai has joined #openstack-infra | 16:46 | |
| *** olaph has quit IRC | 16:46 | |
| *** EricGonc_ has joined #openstack-infra | 16:47 | |
| *** dkranz has quit IRC | 16:47 | |
| *** minwang2 has quit IRC | 16:47 | |
| *** akshai has quit IRC | 16:47 | |
| *** pblaho has quit IRC | 16:48 | |
| *** rfolco has joined #openstack-infra | 16:48 | |
| *** unicell has joined #openstack-infra | 16:48 | |
| *** EricGonczer_ has quit IRC | 16:50 | |
| *** arosen has joined #openstack-infra | 16:52 | |
| *** rossella_ has quit IRC | 16:52 | |
| *** bapalm has quit IRC | 16:52 | |
| *** dkranz has joined #openstack-infra | 16:54 | |
| *** salv-orlando has quit IRC | 16:54 | |
| *** salv-orlando has joined #openstack-infra | 16:54 | |
| openstackgerrit | Merged openstack-infra/subunit2sql: Add a DB API method to get failing test_ids by run metadata https://review.openstack.org/210855 | 16:55 |
| *** ashleighfarnham has joined #openstack-infra | 16:56 | |
| * Zara looks at the number of reviewers for the infamous 4-character patch and wishes there was that much interest in storyboard! :) | 16:58 | |
| *** FallenPegasus has joined #openstack-infra | 16:58 | |
| *** ivar-lazzaro has quit IRC | 16:58 | |
| *** AJaeger has joined #openstack-infra | 16:59 | |
| *** ivar-lazzaro has joined #openstack-infra | 16:59 | |
| openstackgerrit | Travis Tripp proposed openstack-infra/project-config: Add python3-jobs for searchlight https://review.openstack.org/212103 | 16:59 |
| *** arosen has quit IRC | 16:59 | |
| AJaeger | odyssey4me: the infra change merged 20 mins ago, I'm not sure whether all nodes are updated for your change, you might need to retry again in an hour... | 16:59 |
| odyssey4me | AJaeger yep, I see the change taking effect :) thank you so much for your help | 17:00 |
| *** otter768 has joined #openstack-infra | 17:00 | |
| AJaeger | odyssey4me: Ah, zuul has it queued - great! | 17:00 |
| *** nadya has joined #openstack-infra | 17:01 | |
| *** sergmelikyan has joined #openstack-infra | 17:01 | |
| odyssey4me | AJaeger hmm, what's up with this? http://logs.openstack.org/76/207576/3/check/gate-os-ansible-deployment-docs/88e2c4f/console.html | 17:02 |
| dhellmann | Clint: I'm working on some other changes to the release tools that are going to be hard to rebase. Do you have any other work in progress? | 17:02 |
| *** esmute_ has joined #openstack-infra | 17:02 | |
| openstackgerrit | Adam Coldrick proposed openstack-infra/storyboard: Allow browsing stories by creator_id https://review.openstack.org/212104 | 17:02 |
| *** bexelbie has joined #openstack-infra | 17:02 | |
| openstackgerrit | Adam Coldrick proposed openstack-infra/storyboard-webclient: Show stories created by the current user on the dashboard https://review.openstack.org/212105 | 17:03 |
| *** tiswanso has quit IRC | 17:03 | |
| *** esker has joined #openstack-infra | 17:03 | |
| *** tiswanso has joined #openstack-infra | 17:04 | |
| *** nadya has quit IRC | 17:05 | |
| *** placko_ has quit IRC | 17:05 | |
| *** otter768 has quit IRC | 17:05 | |
| *** ybathia has joined #openstack-infra | 17:06 | |
| *** olaph has joined #openstack-infra | 17:08 | |
| *** ihrachyshka has quit IRC | 17:09 | |
| *** tsg has quit IRC | 17:10 | |
| *** w_ has quit IRC | 17:10 | |
| *** deepakcs has quit IRC | 17:10 | |
| *** tsg has joined #openstack-infra | 17:10 | |
| *** rguillebert has quit IRC | 17:10 | |
| *** fhubik has quit IRC | 17:11 | |
| *** isq_ has quit IRC | 17:12 | |
| AJaeger | odyssey4me: do you have a tox.ini file? | 17:12 |
| *** pvaneck has joined #openstack-infra | 17:13 | |
| AJaeger | odyssey4me: you need to be able to run locally "tox -evenv -- python setup.py build_sphinx" | 17:13 |
| *** achanda has joined #openstack-infra | 17:13 | |
| *** isq_ has joined #openstack-infra | 17:13 | |
| *** marzif has quit IRC | 17:14 | |
| *** patrickeast has quit IRC | 17:16 | |
| *** pelix has quit IRC | 17:17 | |
| *** barnaby has quit IRC | 17:17 | |
| *** vinay_yadhav has joined #openstack-infra | 17:19 | |
| *** yamamoto has quit IRC | 17:20 | |
| krotscheck | jeblair: There are two current PPA's, one of which is actively supported by a consulting company that pins its business model on node-for-enterprise. THe other's some guy, and is a few versions out of date. Neither is hosted by the actual node foundation. | 17:20 |
| odyssey4me | AJaeger oh dear, I thought we had it ready - working on an in-repo patch | 17:20 |
| *** dahoo has joined #openstack-infra | 17:21 | |
| *** dahoo is now known as jerryz | 17:22 | |
| zaro | timrc: ping | 17:24 |
| *** persia has joined #openstack-infra | 17:24 | |
| *** persia has quit IRC | 17:24 | |
| *** persia has joined #openstack-infra | 17:24 | |
| *** minwang2 has joined #openstack-infra | 17:25 | |
| odyssey4me | AJaeger is https://review.openstack.org/207576 perhaps hitting nodes that aren't updated yet, it seems that zuul is not showing it in the queue | 17:26 |
| timrc | zaro, Hi. | 17:26 |
| zaro | timrc: so about your gearman-plugin change. i was thinking about it again and I don't think i really undestand your use case for that change. | 17:27 |
| timrc | zaro, I'd like to submit metrics from zuul by node type. | 17:27 |
| zaro | timrc: i think what you are after is something like 'ubunutSlaves' as a grouping of ubuntu machines? | 17:27 |
| AJaeger | odyssey4me: "Queue lengths: 843 events, 285 results." at the top of status.o.o/zuul | 17:27 |
| AJaeger | odyssey4me: have patience with busy Zuul... | 17:27 |
| timrc | zaro, No I want to know zuul metrics for jobs running on "bare-precise" vs "bare-trusty" | 17:27 |
| odyssey4me | AJaeger okie dokey :) nice zuul, pretty zuul | 17:28 |
| zaro | timrc: so yeah, you want to know which type of node the build ran on? | 17:28 |
| odyssey4me | AJaeger oh dear, queue length just doubled | 17:29 |
| zaro | timrc: so there's a problem, jobs registered as 'build:fooProject' can run on any node type. | 17:29 |
| zaro | timrc: you won't get that node type from the gearman registration. | 17:30 |
| timrc | zaro, Yeah I thought about that too... so how would I go about getting the label? | 17:30 |
| anteaya | odyssey4me: could be a gate reset, give it a minute | 17:30 |
| *** unicell has left #openstack-infra | 17:31 | |
| krotscheck | jeblair, sdague: So, because of the ongoing discussion on how to get npm to the correct version on the release slave, the npm publish jobs are effectively blocked. I would like to peel those two patches apart, one that handles the "package and upload the tarball to tarballs.openstack.org", and one that handles the "run npm publish on the release slave" bit, so that I have something useful to give TheJulia and the bifrost team. Then only the | 17:31 |
| krotscheck | second half would be blocked. Would you two be willing to prioritize that review today? It'd basically be a split of https://review.openstack.org/#/c/199715/ into two reviews. | 17:31 |
| timrc | zaro, My change should at least capture node labels for jobs that define "bare-trusty || bare-precise" right? | 17:31 |
| *** patrickeast has joined #openstack-infra | 17:31 | |
| zaro | timrc: also when a gearman client request to run a job on a node type (i.e. build:fooProjet:bar-precise) the client already has the node type info. | 17:31 |
| anteaya | odyssey4me: yeah, the first 9 or so gate patches have finished, zuul is working on recalculating | 17:32 |
| timrc | zaro, But I'm trying to solve for the "node-a || node-b" case | 17:32 |
| odyssey4me | anteaya thanks, I'll continue to watch the numbers in awe and wonder :p | 17:32 |
| anteaya | odyssey4me: a hot cup of tea helps too, I find | 17:33 |
| zaro | timrc: i believe zuul only makes requests like build:fooProject:bar-precise or build:fooProject:bare-trusty so zuul will know which 'node type' the job will run on. | 17:33 |
| odyssey4me | anteaya hahaha, I just had a double espresso... oh dear | 17:34 |
| anteaya | odyssey4me: ha ha ha | 17:34 |
| zaro | timrc: zuul has to know that. | 17:34 |
| timrc | zaro, I don't think that's the case actually... | 17:34 |
| *** vinay_yadhav has quit IRC | 17:34 | |
| *** jsavak has quit IRC | 17:35 | |
| *** FallenPegasus has quit IRC | 17:35 | |
| zaro | timrc: then i'm not sure how zuul adds a job to the gearman queue. | 17:35 |
| zaro | timrc: it must know about the functions that have been registered. | 17:35 |
| *** amuller has joined #openstack-infra | 17:35 | |
| zaro | timrc: in anycase, Jenkins always dynamically allocates the next available node to run a job on. | 17:36 |
| timrc | zaro, I think zuul sends demand to nodepool. | 17:36 |
| fungi | zaro: timrc: zuul does add one of build:fooProject:bar-precise or build:fooProject:bare-trusty to the gearman queue. "node-a || node-b" in the job definition is merely telling jenkins what node types to register for that job | 17:36 |
| *** barnaby has joined #openstack-infra | 17:36 | |
| timrc | fungi, But zuul won't actually know which type of node the job is actually running on will it? | 17:36 |
| jeblair | it can also run simply "build:fooProject" and let jenkins decide | 17:36 |
| timrc | Yeah.. | 17:37 |
| fungi | timrc: so we may register a job for multiple labels, but zuul will "choose" which label it's targeting with a specific work request | 17:37 |
| pabelanger | fungi: who do you think can assist with https://review.openstack.org/#/c/211703/ ? | 17:37 |
| timrc | that's the case I'm trying to solve for jeblair | 17:37 |
| fungi | or what jeblair said, in the default case it may just request build:fooProject | 17:37 |
| zaro | timrc: I think you need to ask Jenkins which node the build ran on then put that on the gearman data packet. My original suggestion | 17:37 |
| timrc | Right which is why I need gearman-plugin to tell zuul what label it used. | 17:37 |
| timrc | zaro, So that would mean I'd have to parse it out of the hostname? | 17:37 |
| timrc | I _really_ want to avoid that solution if possible. | 17:38 |
| jeblair | timrc: there can be more than one match in jenkins | 17:38 |
| fungi | timrc: there's no guarantee the hostname correlates to the label | 17:38 |
| timrc | fungi, Right. | 17:38 |
| timrc | I definitely don't want to make that assumption. | 17:38 |
| fungi | it does in our systems because we choose for it to | 17:38 |
| jeblair | krotscheck: does the repo in question have a post job that builds branch tip tarballs? | 17:38 |
| zaro | timrc: actually probably not, may you can ask which node it ran on then request the label on that node? | 17:38 |
| jeblair | zaro: labels | 17:38 |
| amuller | The neutron docs job is passing at the gate but failing locally for me. Can anyone tell me where is the %s-docs job itself defined? | 17:38 |
| jeblair | nodes have multiple labels | 17:38 |
| *** jsavak has joined #openstack-infra | 17:39 | |
| krotscheck | jeblair: That sentence requires translation? | 17:39 |
| krotscheck | jeblair: It's this -> https://deb.nodesource.com/node_0.12/dists/ | 17:39 |
| amuller | here's a local failure of the neutron docs job: http://paste.openstack.org/show/412743/ | 17:39 |
| jeblair | krotscheck: no, your repo | 17:39 |
| krotscheck | jeblair: Oh! No. | 17:39 |
| krotscheck | jeblair: Tag only | 17:39 |
| timrc | zaro, At line 150... https://review.openstack.org/#/c/205097/4/src/main/java/hudson/plugins/gearman/ExecutorWorkerThread.java -- does the conditional there not satisfy the case where zuul lets Jenkins decide? | 17:40 |
| fungi | amuller: http://git.openstack.org/cgit/openstack-infra/project-config/tree/jenkins/jobs/python-jobs.yaml#n152 which runs http://git.openstack.org/cgit/openstack-infra/project-config/tree/jenkins/scripts/run-docs.sh | 17:40 |
| jeblair | krotscheck: would adding that job be a better stop-gap? | 17:40 |
| amuller | fungi: right on thank you | 17:40 |
| *** andersonvom has quit IRC | 17:40 | |
| fungi | amuller: actually, for the full call chain, the job-template runs this macro http://git.openstack.org/cgit/openstack-infra/project-config/tree/jenkins/jobs/macros.yaml#n119 which in turn calls that script | 17:40 |
| amuller | fungi: since this job is shared across projects I would guess other projects are affected as well | 17:41 |
| *** pblaho has joined #openstack-infra | 17:41 | |
| krotscheck | jeblair: Lemme ask TheJulia | 17:41 |
| krotscheck | She ran away | 17:41 |
| jeblair | amuller: it looks like you may be running "tox -e docs". that's not what the docs jobs do | 17:41 |
| fungi | amuller: totally depends on what you're running | 17:41 |
| amuller | yeah I'm running tox -e docs | 17:41 |
| zaro | timrc: i think jenkins always decides, depending on the label applied to the project and node. | 17:41 |
| fungi | amuller: tox -evenv -- python setup.py build_sphinx | 17:42 |
| jeblair | and yeah, it looks like the docs env runs sphinx with warnings treated as errors, which the standard sphinx command does not do | 17:42 |
| fungi | amuller: per line 18 of that script (i reduced the venv=venv and $venv in it) | 17:42 |
| krotscheck | jeblair: I guess the real question is: Do we want to run that tarball job _anyway_ regardless of the tag tarball? | 17:43 |
| krotscheck | To which I say: How much harddrive space do we have? | 17:43 |
| *** yamamoto has joined #openstack-infra | 17:43 | |
| jeblair | krotscheck: the post-pipeline branch-tip tarball jobs generally build a tarball for every commit, but use the same name "eg foo-master.tgz" and overwrite it each time | 17:44 |
| jeblair | krotscheck: so there's not much additional space used | 17:44 |
| *** annegentle has quit IRC | 17:44 | |
| *** abregman has joined #openstack-infra | 17:45 | |
| fungi | krotscheck: but also in answer to your question, http://cacti.openstack.org/cacti/graph.php?action=view&local_graph_id=1055&rra_id=all (cacti truncates the word "tarballs" in the devname, but that's the right one) | 17:45 |
| *** dtantsur is now known as dtantsur|afk | 17:45 | |
| jeblair | amuller: the correct way to ask sphinx to treat warnings as errors so that the standard "python setup.py build_sphinx" picks it up is this: https://review.openstack.org/212122 | 17:46 |
| timrc | zaro, So are you saying I can only know the node label the job runs on after the build is complete? | 17:46 |
| jeblair | amuller: that should cause the docs job to fail in the same way you have been seeing | 17:46 |
| * timrc has a love/hate relationship with information. | 17:46 | |
| jeblair | amuller: note that i have no interest in actually fixing the warnings in neutron; feel free to take over that patch | 17:46 |
| amuller | jeblair: yeah I have a patch for that | 17:46 |
| amuller | jeblair: will that change in setup.cfg affect only sphinx? | 17:47 |
| jeblair | amuller: ah, then if you rebase my patch on it, it will pass (but otherwise it will fail) | 17:47 |
| fungi | timrc: i have a feeling this will get much easier to track post-jenkins when all our workers talk directly to gearman | 17:47 |
| *** earlephilhower has quit IRC | 17:47 | |
| amuller | jeblair: I want to see it fail :) | 17:47 |
| jeblair | amuller: yes, that's a hook in pbr that solves the fact that otherwise you have to switch to sphinx-build, which is not generally the way we expect docs to be built. | 17:47 |
| amuller | jeblair: fungi: Thanks for your help! | 17:48 |
| zaro | timrc: Jenkins might provide it after it kicks off the job. i'de have to read the api again | 17:48 |
| jeblair | amuller: all yours then; feel free to let it run, fail, then rebase | 17:48 |
| amuller | jeblair: that's the plan | 17:48 |
| zaro | timrc: but yeah, i think the most consistant way to find that info is request it from jenkins. | 17:48 |
| *** crc32 has joined #openstack-infra | 17:49 | |
| zaro | timrc: or maybe the only way to know. | 17:50 |
| timrc | fungi, Yeah... post-jenkins... *dream* | 17:52 |
| fungi | timrc: come help us make it a reality! | 17:53 |
| *** jcoufal_ has joined #openstack-infra | 17:53 | |
| * anteaya hands out balloons | 17:53 | |
| timrc | fungi, I'd like too. | 17:53 |
| *** abregman is now known as abregman|afk | 17:54 | |
| jeblair | timrc: if you want to know how many jobs ran on a particular node type, why not count the transitions to used for that type in nodepool? | 17:54 |
| timrc | jeblair, I want to know zuul metrics (like wait time between "build this job" to when the actual job starts building by node type. | 17:55 |
| jeblair | though i guess that could count re-launches | 17:55 |
| * timrc tosses a ) in there some where | 17:55 | |
| *** ihrachyshka has joined #openstack-infra | 17:55 | |
| timrc | zaro, So if I ask Jenkins there is a possibility of getting back multiple labels right? | 17:56 |
| zaro | timrc: yes, that is true. | 17:56 |
| mmmpork | I'm walking through https://review.openstack.org/#/c/179840/ and I have a question about where the nodepool.yaml file should go | 17:56 |
| *** mrmartin has quit IRC | 17:56 | |
| *** HeOS has quit IRC | 17:56 | |
| *** shashank_hegde has joined #openstack-infra | 17:56 | |
| mmmpork | it says it should go in "/path/to/nodepool/things/nodepool.yaml" | 17:56 |
| timrc | zaro, So okay jeblair and I talked about this before... I'll try to rework to get those labels and then just submit the metrics for all of them. I don't think I can do any better than that.... | 17:57 |
| *** jcoufal has quit IRC | 17:57 | |
| mmmpork | i followed the instructions to use virtualenv to install nodepool, so i have a directory called venv | 17:57 |
| timrc | I was really hoping I could get the node label by a node label -> gearman working mapping. | 17:57 |
| timrc | s/working/worker/ | 17:57 |
| mmmpork | and i've got venv/bin/nodepool | 17:57 |
| anteaya | so been trying to wait my turn and now I have to get on the phone, but I would like to figure out a way to have fewer bots reporting bugs in channels | 17:58 |
| anteaya | and welcome advice to have that | 17:58 |
| anteaya | one report per bug mention is what I am going for here | 17:58 |
| *** amuller has left #openstack-infra | 17:58 | |
| *** markus_z has quit IRC | 17:58 | |
| *** ildikov has joined #openstack-infra | 17:59 | |
| jeblair | bug 1 | 17:59 |
| openstack | bug 1 in Ubuntu Malaysia LoCo Team "Microsoft has a majority market share" [Critical,In progress] https://launchpad.net/bugs/1 - Assigned to MFauzilkamil Zainuddin (apogee) | 17:59 |
| *** salv-orl_ has joined #openstack-infra | 17:59 | |
| zaro | timrc: nope unfortunately the way we've setup function registrations don't map well to that solution. | 17:59 |
| jeblair | anteaya: we only have 1 here... is there something elsewhere? | 18:00 |
| *** earlephilhower has joined #openstack-infra | 18:00 | |
| anteaya | in -meeting | 18:00 |
| zaro | timrc: so actually i took a look at the labels on our nodes and it seems like mostly it's just one label per node. so that solution to query jenkins may work out pretty well. | 18:00 |
| anteaya | one from our bot and two from uvirbot | 18:00 |
| *** sridhar_ram has joined #openstack-infra | 18:01 | |
| anteaya | uvirtbot | 18:01 |
| openstackgerrit | Merged openstack-infra/subunit2sql: Update the changelog in the readme https://review.openstack.org/210945 | 18:01 |
| jeblair | anteaya: i thought uvirtbot was being removed. if it isn't we can ban it. | 18:01 |
| anteaya | hey there is an easy fix | 18:01 |
| *** salv-orlando has quit IRC | 18:01 | |
| anteaya | thanks | 18:01 |
| fungi | mmmpork: our puppet modules put it at /etc/nodepool/nodepool.yaml | 18:02 |
| timrc | zaro, Okay. I will figure out what that API call is and where I need to make it. | 18:02 |
| fungi | jeblair: yeah, soren agreed to delist all the channels our meetbot joins, but i don't think he did (or it got undone). i kicked it from this channel a while back | 18:03 |
| fungi | anteaya: ^ | 18:03 |
| timrc | zaro, Thanks much for your help. I'm definitely out of my element in the Java / Jenkins plugin world. | 18:03 |
| *** FallenPegasus has joined #openstack-infra | 18:03 | |
| mmmpork | fungi: thanks i just dug around in the other docs and found it in configuration.rst | 18:03 |
| zaro | timrc: np, me too :) | 18:03 |
| mmmpork | definitely going to put a link to configuration.rst or something, that was a bit confusing for me | 18:04 |
| anteaya | thx | 18:04 |
| *** dannywilson has quit IRC | 18:04 | |
| zaro | timrc: also i referenced the jenkins api when i made the suggestion in PS 1 | 18:04 |
| timrc | Will eliminating Jenkins effectively eliminate Java from infra? | 18:04 |
| fungi | timrc: not unless we replace gerrit too | 18:04 |
| zaro | timrc: no :) | 18:04 |
| timrc | Oh yeah. | 18:04 |
| timrc | Gerrit. | 18:04 |
| fungi | also zanata now | 18:04 |
| *** dolphm has joined #openstack-infra | 18:04 | |
| zaro | java is too big to beat down | 18:05 |
| timrc | I'm surprised that hasn't been rewritten in Go yet, honestly. | 18:05 |
| timrc | So is ruby apparently :/ | 18:05 |
| fungi | timrc: give google time. they can't rewrite all their existing software in their new language overnight | 18:05 |
| timrc | fungi, They need to acquire Canonical, then :) | 18:05 |
| fungi | timrc: that sounds like poor financial advice to me, but what do i know? | 18:06 |
| *** akshai has joined #openstack-infra | 18:06 | |
| timrc | fungi, Haha. | 18:06 |
| timrc | touche. | 18:06 |
| *** abregman|afk has quit IRC | 18:07 | |
| *** esmute_ has quit IRC | 18:10 | |
| *** angdraug has quit IRC | 18:12 | |
| *** hdd has quit IRC | 18:13 | |
| *** bapalm_ has joined #openstack-infra | 18:14 | |
| davideagnello | hello, this patch needs one more core reviewer: https://review.openstack.org/#/c/201285/2 | 18:14 |
| *** tvanderwiel has joined #openstack-infra | 18:15 | |
| *** tvanderwiel has left #openstack-infra | 18:15 | |
| *** esmute_ has joined #openstack-infra | 18:15 | |
| *** arxcruz has joined #openstack-infra | 18:17 | |
| *** tonytan4ever has quit IRC | 18:17 | |
| *** hdd has joined #openstack-infra | 18:18 | |
| *** tonytan4ever has joined #openstack-infra | 18:19 | |
| *** dolphm has left #openstack-infra | 18:20 | |
| *** esmute_ has quit IRC | 18:21 | |
| *** esmute_ has joined #openstack-infra | 18:22 | |
| *** bapalm has joined #openstack-infra | 18:23 | |
| minwang2 | likewise, this patch needs core reviewers : https://review.openstack.org/#/c/211319/ | 18:23 |
| *** notnownikki has quit IRC | 18:24 | |
| *** bapalm has quit IRC | 18:24 | |
| *** olaph has quit IRC | 18:24 | |
| *** olaph has joined #openstack-infra | 18:24 | |
| *** radez_g0n3 is now known as radez | 18:27 | |
| *** ayoung has quit IRC | 18:32 | |
| *** jcoufal has joined #openstack-infra | 18:32 | |
| *** nadya has joined #openstack-infra | 18:32 | |
| *** hdd has quit IRC | 18:33 | |
| *** yamamoto has quit IRC | 18:33 | |
| *** jcoufal__ has joined #openstack-infra | 18:35 | |
| *** akshai has quit IRC | 18:35 | |
| *** jcoufal_ has quit IRC | 18:36 | |
| *** skylerberg has quit IRC | 18:36 | |
| *** esmute_ has quit IRC | 18:36 | |
| zaro | minwang2: nice slip in | 18:36 |
| *** nadya has quit IRC | 18:37 | |
| *** Sukhdev_ has joined #openstack-infra | 18:37 | |
| *** sergmelikyan has quit IRC | 18:37 | |
| *** jcoufal has quit IRC | 18:38 | |
| openstackgerrit | Paul Belanger proposed openstack-infra/project-config: Add puppet-modulesync-configs hook https://review.openstack.org/210517 | 18:38 |
| *** esmute_ has joined #openstack-infra | 18:40 | |
| *** enikanorov2 has quit IRC | 18:42 | |
| *** esmute_ has quit IRC | 18:43 | |
| *** erikwilson_ has quit IRC | 18:45 | |
| *** erikwilson_ has joined #openstack-infra | 18:45 | |
| *** salv-orl_ has quit IRC | 18:46 | |
| mmmpork | whoohoo nodepool tox tests passed on the first attempt! | 18:47 |
| mmmpork | now to test against a devstack... :) | 18:47 |
| *** EricGonc_ has quit IRC | 18:48 | |
| *** sripriya has joined #openstack-infra | 18:49 | |
| *** salv-orlando has joined #openstack-infra | 18:49 | |
| *** jsavak has quit IRC | 18:49 | |
| *** jsavak has joined #openstack-infra | 18:50 | |
| *** erikmwilson_ has joined #openstack-infra | 18:50 | |
| *** EricGonczer_ has joined #openstack-infra | 18:51 | |
| *** erikwilson_ has quit IRC | 18:53 | |
| *** baoli has quit IRC | 18:53 | |
| *** nadya has joined #openstack-infra | 18:53 | |
| *** baoli has joined #openstack-infra | 18:54 | |
| *** boris-42 has joined #openstack-infra | 18:54 | |
| *** hdd has joined #openstack-infra | 18:54 | |
| *** HeOS has joined #openstack-infra | 18:54 | |
| *** enikanorov2 has joined #openstack-infra | 18:55 | |
| *** sergmelikyan has joined #openstack-infra | 18:56 | |
| *** mrmartin has joined #openstack-infra | 18:57 | |
| *** jsavak has quit IRC | 18:57 | |
| *** jsavak has joined #openstack-infra | 18:58 | |
| *** arxcruz has quit IRC | 18:59 | |
| *** dannywilson has joined #openstack-infra | 18:59 | |
| *** esker has quit IRC | 18:59 | |
| *** otter768 has joined #openstack-infra | 19:01 | |
| *** esmute_ has joined #openstack-infra | 19:01 | |
| openstackgerrit | Matthew Treinish proposed openstack-infra/subunit2sql: Update the in-tree TODO list https://review.openstack.org/212150 | 19:02 |
| *** tqtran is now known as tqtran-afk | 19:03 | |
| openstackgerrit | Khai Do proposed openstack-infra/jenkins-job-builder: Add a JJB quick start guide https://review.openstack.org/208037 | 19:03 |
| *** w_ has joined #openstack-infra | 19:04 | |
| *** jsavak has quit IRC | 19:05 | |
| zaro | waynr: wondering if you coiuld take a quick look at https://review.openstack.org/#/c/206311 would like that one and https://review.openstack.org/#/c/208037/ merged. both are basically just Doc updates. | 19:06 |
| *** jsavak has joined #openstack-infra | 19:06 | |
| *** otter768 has quit IRC | 19:06 | |
| *** Sukhdev__ has joined #openstack-infra | 19:07 | |
| akraminsky_ | clarkb, so the gate64 model requires the host cpu to support those features, but my nodes do not support them when they are created. Was this an issue upstream? | 19:07 |
| *** olaph has quit IRC | 19:07 | |
| akraminsky_ | The features being ssse3 and vme | 19:08 |
| *** w_ has quit IRC | 19:08 | |
| openstackgerrit | Armando Migliaccio proposed openstack-infra/devstack-gate: Drop these config switches as they are no longer being used https://review.openstack.org/199672 | 19:09 |
| *** Sukhdev_ has quit IRC | 19:11 | |
| *** barnaby has quit IRC | 19:11 | |
| *** Sukhdev_ has joined #openstack-infra | 19:13 | |
| openstackgerrit | Michael Krotscheck proposed openstack-infra/project-config: Added npm tarball upload job https://review.openstack.org/212153 | 19:13 |
| openstackgerrit | Michael Krotscheck proposed openstack-infra/project-config: Switched npm-run-test to npm-test https://review.openstack.org/212154 | 19:13 |
| *** hdd has quit IRC | 19:13 | |
| *** Sukhdev__ has quit IRC | 19:14 | |
| *** mrmartin has quit IRC | 19:14 | |
| *** angdraug has joined #openstack-infra | 19:14 | |
| openstackgerrit | Merged openstack-infra/project-config: Add support for pulling translations from Zanata https://review.openstack.org/203928 | 19:14 |
| *** hdd has joined #openstack-infra | 19:16 | |
| *** erikmwilson_ has quit IRC | 19:16 | |
| *** bpokorny has joined #openstack-infra | 19:16 | |
| openstackgerrit | Sridhar Ramaswamy proposed openstack-infra/project-config: Fix up tacker stackforge projects https://review.openstack.org/211789 | 19:16 |
| *** shashank_hegde has quit IRC | 19:16 | |
| openstackgerrit | Merged openstack-infra/system-config: Add puppet-os_client_config to modules.env https://review.openstack.org/204313 | 19:17 |
| *** bexelbie has quit IRC | 19:19 | |
| *** mriedem1 is now known as mriedem | 19:24 | |
| *** funzo has joined #openstack-infra | 19:25 | |
| *** hdd has quit IRC | 19:25 | |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config: Merge tacker ACL files https://review.openstack.org/212156 | 19:27 |
| openstackgerrit | Merged openstack-infra/puppet-gerrit: Add class to handle welcome rsa key https://review.openstack.org/209929 | 19:28 |
| openstackgerrit | Merged openstack-infra/system-config: Add a reminder about grafana and puppet-archive conflict https://review.openstack.org/209946 | 19:29 |
| *** funzo has quit IRC | 19:30 | |
| *** bexelbie_ has joined #openstack-infra | 19:32 | |
| *** yamamoto has joined #openstack-infra | 19:33 | |
| *** rcarrillocruz has left #openstack-infra | 19:33 | |
| *** milligana1 has joined #openstack-infra | 19:35 | |
| *** milligana has quit IRC | 19:35 | |
| nibalizer | krotscheck: so can the nodejs module not provide the granularity of installation you desire? | 19:38 |
| nibalizer | I'm also ok with a ppa... provided we think it's reliably updated | 19:38 |
| *** yamamoto has quit IRC | 19:38 | |
| nibalizer | thats essentially what we do with puppet, except it is a ppa updated by the software publisher | 19:38 |
| openstackgerrit | Merged openstack-infra/puppet-user: Add a sane default value for key_id https://review.openstack.org/195618 | 19:39 |
| *** annegentle has joined #openstack-infra | 19:40 | |
| *** bexelbie_ has quit IRC | 19:41 | |
| krotscheck | nibalizer: I am hesitant to introduce that module into system-config in general, because it might leak over to other builds? We need a somewhat more recent version of node for the release slave only. | 19:43 |
| nibalizer | having that module in the modulepath isn't bad | 19:43 |
| nibalizer | doesn't on its own change anything | 19:43 |
| krotscheck | nibalizer: But for everything else, we should stick with distro version | 19:43 |
| nibalizer | well ideally the module allows you to specify things | 19:44 |
| nibalizer | like 'use distro' 'use bleeding edge' 'use repo' | 19:44 |
| *** sc68cal has quit IRC | 19:44 | |
| nibalizer | but yea a PPA resource followed by a package resource might be the way to go | 19:44 |
| nibalizer | simple, fast, reliable | 19:44 |
| openstackgerrit | Merged openstack-infra/puppet-zuul: Allow for ability to block status.json referers https://review.openstack.org/171420 | 19:44 |
| *** EricGonc_ has joined #openstack-infra | 19:45 | |
| *** esker has joined #openstack-infra | 19:46 | |
| *** sc68cal has joined #openstack-infra | 19:46 | |
| krotscheck | nibalizer: Ok, so the only question is whether the PPA is something we like. | 19:46 |
| *** esker has quit IRC | 19:46 | |
| nibalizer | ya, can you evaluate that? | 19:47 |
| nibalizer | or link to the ppa? | 19:47 |
| *** esker has joined #openstack-infra | 19:47 | |
| openstackgerrit | Merged openstack-infra/jenkins-job-builder: Document that jjb supports json job definitions https://review.openstack.org/206311 | 19:48 |
| *** tonytan4ever has quit IRC | 19:48 | |
| fungi | we have used ppas for other things besides puppet in the past (python 3, pypy, uca is sort of a ppa too), though keep in mind the intent should always be to _stop_ using a ppa as soon as there is an available lts that provides the features we need instead | 19:48 |
| *** EricGonczer_ has quit IRC | 19:49 | |
| waynr | zaro: queued the first for merge, reviewed the other--both look good to me! | 19:49 |
| openstackgerrit | Michael Krotscheck proposed openstack-infra/project-config: Switched npm-run-test to npm-test https://review.openstack.org/212154 | 19:49 |
| openstackgerrit | Michael Krotscheck proposed openstack-infra/project-config: Added npm tarball upload job https://review.openstack.org/212153 | 19:49 |
| *** ayoung has joined #openstack-infra | 19:49 | |
| fungi | nibalizer: krotscheck: just worth noting that in this case we're basically handing the ppa owner control over the machine which uploads our releases of everything (or optionally just our releases of things to npm if we decide that's an out) | 19:50 |
| *** radez is now known as radez_g0n3 | 19:50 | |
| *** esker has quit IRC | 19:50 | |
| *** swat30 has quit IRC | 19:50 | |
| krotscheck | fungi: Yep, and I do not put it beyond the node community to decide that they're going to take their node in-branch patches, apply them to their libs (like openssl), put that package into their PPA, and roll their node on top of that. | 19:51 |
| *** shashank_hegde has joined #openstack-infra | 19:51 | |
| fungi | i'm starting to think that between the upload to tarballs.o.o and the upload of that to traditional publication channels, a separate very-locked-down system with our artifact signing keys might be warranted | 19:52 |
| krotscheck | fungi: I like the words in that sentence, but I do not quite understand their meaning. | 19:52 |
| krotscheck | fungi: As long as it doesn't hold up my attempts to get us to push to npm :) | 19:52 |
| nibalizer | fungi: yea wow | 19:53 |
| fungi | 1. single-use worker produces release artifact and allows that to be slurped to tarballs.o.o; 2. signing worker retrieves that, signs it and uploads detached sig to tarballs.o.o; 3. release worker retrieves artifact and signature from tarballs.o.o and uploads to pypi/maven central/jenkins-ci.org/npm/wherever | 19:54 |
| fungi | nibalizer: nah, just a hardening enhancement | 19:54 |
| greghaynes | mordred: I just noticed something annoying re: glean - I think we need to have a glean lock | 19:54 |
| greghaynes | mordred: since were running once per interface, and were doing things like i/o to the same ssh key file | 19:55 |
| *** BharatK has quit IRC | 19:56 | |
| fungi | nibalizer: krotscheck: that would at least protect the signing key from compromise by things run on the release worker, which would provide an extra safety check to be able to tell what releases are "valid" and what were made purely with the compromised upload credentials | 19:56 |
| *** tqtran-afk has quit IRC | 19:56 | |
| krotscheck | fungi: That makes sense. It won't protect the upload credentials to the various package managers, but it'll protect the signing keys. | 19:56 |
| *** geoffarnold has quit IRC | 20:00 | |
| *** jsavak has quit IRC | 20:00 | |
| *** jsavak has joined #openstack-infra | 20:01 | |
| krotscheck | fungi, nibalizer: Ok, so I'm looking for alternatives to getting an updated version of NPM onto the release slave in a way that makes all the stakeholders happy | 20:01 |
| krotscheck | Right now the effort is in a bit of a limbo. | 20:01 |
| *** geoffarnold has joined #openstack-infra | 20:01 | |
| krotscheck | Using a PPA is not really an option, because who knows what additional package overrides might occur. | 20:01 |
| nibalizer | so the npm in trusty can't perform a publish? | 20:02 |
| krotscheck | Using exec is less than optimal, because it's not actually a cheap command. | 20:02 |
| nibalizer | so I followed the instructions in that patch on my laptop | 20:02 |
| *** FallenPegasus has quit IRC | 20:02 | |
| nibalizer | and my npm is now quite sad, by the way | 20:02 |
| krotscheck | nibalizer: the version of npm in trusty does not support the --ignore-scripts command, which is something we need to ensure no privileged code is executed that could expose credentials. | 20:02 |
| *** FallenPegasus has joined #openstack-infra | 20:02 | |
| krotscheck | nibalizer: That's odd, because my vagrant VM is happy as a clam. | 20:02 |
| nibalizer | npm just exits every time | 20:03 |
| krotscheck | nibalizer: Ultimately, we need to execute `npm publish --ignore-scripts <tarball>` | 20:03 |
| krotscheck | nibalizer: No stacktrace? | 20:03 |
| *** amitgandhinz has quit IRC | 20:03 | |
| openstackgerrit | Merged openstack-infra/puppet-zuul: Set zuul server and merger services with ensure running https://review.openstack.org/168306 | 20:05 |
| krotscheck | nibalizer: Anyway, can we switch back to getting NPM updated solutions? | 20:05 |
| *** HeOS has quit IRC | 20:06 | |
| nibalizer | krotscheck: sure | 20:06 |
| fungi | fbo_: nibalizer: jhesketh: yolanda: wow, really? our puppet modules are going to unilaterally start services now? https://review.openstack.org/168306 it wasn't an accident that they didn't | 20:06 |
| nibalizer | I mean we have to trust code from someone...s | 20:06 |
| nibalizer | fungi: oh? | 20:07 |
| *** jsavak has quit IRC | 20:07 | |
| nibalizer | fungi: we can revert if we don't want that, I didn't realize that was the policy when I approved | 20:07 |
| openstackgerrit | David Shrewsbury proposed openstack-infra/shade: Change functional testing to use clouds.yaml https://review.openstack.org/212083 | 20:07 |
| fungi | fbo_: nibalizer: jhesketh: yolanda: though it does seem that change has been hanging out there since march with no objections. i should have noticed it long before now | 20:07 |
| openstackgerrit | Spencer Krum proposed openstack-infra/puppet-zuul: Revert "Set zuul server and merger services with ensure running" https://review.openstack.org/212176 | 20:08 |
| *** jsavak has joined #openstack-infra | 20:08 | |
| nibalizer | fungi: https://review.openstack.org/#/c/212176/ | 20:08 |
| fungi | nibalizer: well, it was intentional in the past | 20:08 |
| fungi | and so i assumed it still was | 20:08 |
| nibalizer | so I spent a few minutes reading throguth the puppet code to try to find the default | 20:08 |
| nibalizer | fungi: lets revert and have a conversation about it then | 20:08 |
| fungi | nibalizer: yep, thanks! | 20:08 |
| nibalizer | and it wasn't at all clear what the puppet default behaviour was at least on 3.8.x | 20:09 |
| *** skylerberg has joined #openstack-infra | 20:10 | |
| fungi | fbo_: nibalizer: jhesketh: yolanda: maybe this is also one worth discussing a little on the mailing list too before we potentially reintroduce it (or approve other similar behavior changes) | 20:11 |
| openstackgerrit | greghaynes proposed openstack-infra/glean: Create systemwide lock for glean https://review.openstack.org/212177 | 20:11 |
| krotscheck | argh, sec. | 20:11 |
| *** ivar-lazzaro has quit IRC | 20:11 | |
| nibalizer | fungi: i agree | 20:11 |
| *** akshai has joined #openstack-infra | 20:12 | |
| *** alivigni has quit IRC | 20:12 | |
| fungi | the basic argument against doing that was that something like zuul can make a right mess if just fired up without some care | 20:12 |
| *** akshai has quit IRC | 20:12 | |
| greghaynes | mordred: TheJulia ^ glean change to add locking, curious what youall think | 20:13 |
| fungi | nibalizer: and many systems administrators (us included) like to be in control of whether services are started. particularly stopping the service to perform some activity shouldn't require disabling puppet agent while you do so | 20:13 |
| nibalizer | fungi: I totally see where you're comming from on that | 20:14 |
| nibalizer | I suspect it does not hold true for the apache things? | 20:14 |
| openstackgerrit | David Shrewsbury proposed openstack-infra/shade: Add flavor access API https://review.openstack.org/211710 | 20:14 |
| fungi | nibalizer: apache is fairly stateless, though i'd also love to have puppet not start that either | 20:14 |
| nibalizer | ya we have | 20:14 |
| nibalizer | service { 'httpd': | 20:15 |
| nibalizer | ensure => running, | 20:15 |
| nibalizer | not even a parameter in our module for disabling the 'always on' | 20:15 |
| *** achanda has quit IRC | 20:15 | |
| fungi | we've sort of compromised on some of those, or ignored them, but having zuul start back up without you expecting it can be pretty detrimental | 20:15 |
| nibalizer | sure | 20:15 |
| nibalizer | and I think we explicitly (clark i mean) disabled puppet on etherpad during the summit so that some local changes wouldn't be messed with | 20:16 |
| nibalizer | I think that might have been config file related not service restart though | 20:16 |
| fungi | ultimately though, the operating system has mechanisms in place for starting and stopping services, determining whether to start them on boot and in what order, and available service managers for when you really want to make sure something always stays running. this is one of those places where puppet trying to get into orchestration can potentially ruin your (sysadmin-)day | 20:17 |
| *** tonytan4ever has joined #openstack-infra | 20:18 | |
| regXboi | Ajaeger: ping? | 20:19 |
| openstackgerrit | Merged openstack-infra/puppet-zuul: Revert "Set zuul server and merger services with ensure running" https://review.openstack.org/212176 | 20:20 |
| fungi | nibalizer: i'm happy to revisit that, but i think we'd need to get a lot more careful about using config management to enable and disable services explicitly rather than doing so manually (and not sure if that's taking things a bit _too_ far into the config management realm) | 20:20 |
| yolanda | fungi why don't we want to start zuul? | 20:20 |
| regXboi | AJaeger: ping? | 20:20 |
| fungi | yolanda: it's mostly that we don't want puppet to start zuul if we stop zuul | 20:20 |
| yolanda | is not a good idea to disable puppet in that case? | 20:20 |
| nibalizer | fungi: totally understand | 20:21 |
| AJaeger | regXboi: what's up? Next time please say directly what you need instead of using a naked ping... | 20:21 |
| nibalizer | its also complex with apache and any debian services because of the start-on | 20:21 |
| *** tonytan4ever has quit IRC | 20:21 | |
| *** tonytan4ever has joined #openstack-infra | 20:21 | |
| regXboi | AJaeger: apologies - trying to figure out what's the cleanest way to keep gate-tempest-dsvm-neutron-full running only on the juno branch | 20:21 |
| fungi | yolanda: it just needs further discussion, and i unfortunately missed the review that was adding that until now (i know, it's been proposed for ~4.5 months) | 20:21 |
| regXboi | but not anywhere else | 20:22 |
| nibalizer | I thinnk fbo_ probably rna the puppet module and was suprised that zuul didn't start and wanted to fix it | 20:22 |
| yolanda | yes, it took a long time and seemed logical to me | 20:22 |
| yolanda | nibalizer, i was as suprised as fbo_ | 20:22 |
| yolanda | and i missed that on that module, i thought it was a problem | 20:22 |
| regXboi | can I just use the gate-tempest-dsvm-neutron-full-juno tag? | 20:22 |
| yolanda | fungi, our procedure downstream when we want to do live changes in an instance is just stop puppet | 20:23 |
| nibalizer | and the way I usually operate services with puppet is to have puppet ensure them then disable puppet when I need to do surgery, but I'm fine with the other way that the infra-rooters do things | 20:23 |
| yolanda | same as nibalizer | 20:23 |
| fungi | yolanda: say i want to do something locally with zuul stopped, pull in the pending puppeted config changes without having zuul started... hand-patch? | 20:24 |
| AJaeger | regXboi: As commented in gerrit: Create a new job like "gate-tempest-dsvm-neutron-full-juno" and add conditions to it. | 20:24 |
| yolanda | yes, hand patch if you are operating manual | 20:24 |
| yolanda | manually | 20:24 |
| AJaeger | regXboi: I'm going offline now, best ask fungi or jeblair how to do it properly. | 20:24 |
| regXboi | AJaeger: ack | 20:24 |
| regXboi | AJaeger: and thanks | 20:24 |
| AJaeger | regXboi: You're welcome | 20:25 |
| *** AJaeger has quit IRC | 20:25 | |
| openstackgerrit | David Shrewsbury proposed openstack-infra/shade: Add a developer coding standards doc https://review.openstack.org/211148 | 20:25 |
| yolanda | fungi, what about passing a parameter to ensure there and make it optional? | 20:25 |
| yolanda | so we have a compromise | 20:25 |
| openstackgerrit | Madhusudhan Kandadai proposed openstack-infra/project-config: Set experimental job for all neutron-vpnaas tests https://review.openstack.org/211767 | 20:25 |
| fungi | yolanda: point being, this is a pretty significant behavior change. i _at_least_ want the people managing our zuul server (besides me and jhesketh) to be aware that it's happened | 20:25 |
| openstackgerrit | Henry Gessau proposed openstack-infra/elastic-recheck: Update query for Bug #1477192 https://review.openstack.org/212180 | 20:26 |
| openstack | bug 1477192 in neutron "neutron test_multi_prefix_slaac failing in the gate with ping failures starting around 7/22" [High,Confirmed] https://launchpad.net/bugs/1477192 - Assigned to Henry Gessau (gessau) | 20:26 |
| fungi | yolanda: having it default to not start, forcing that default through the system-config repo, then switching the default in the puppet-zuul module seems like a safe compromise | 20:26 |
| fungi | yolanda: though it may take other downstream users of the module by surprise as well | 20:26 |
| yolanda | fungi, on the other side, puppet keeps services up if something fails. Haven't you had that issue some time with zuul-merger for example? | 20:26 |
| yolanda | fungi, i agree with that, a parameter wth default not running, and we can enable it if needed | 20:27 |
| nibalizer | puppet as a service monitor is the wrong tool for the job | 20:27 |
| yolanda | fungi, do you want to revert, or iterate over it? | 20:27 |
| fungi | yolanda: that's a behavior of puppet modules i very much dislike. puppet is a great configuration manager. it's a terrible service management daemon and even worse orchestration engine | 20:27 |
| nibalizer | I think puppet's ensure => running is the most valuable when puppet is doing an initial bring up, but only when the service is very simple | 20:28 |
| nibalizer | fungi: however if we were running zuul from packages ( I hear it has those now) puppet will install the package and debian will be like 'better start you' anyways | 20:28 |
| *** ihrachyshka has quit IRC | 20:28 | |
| regXboi | jeblair, fungi: ola - I'm working on https://review.openstack.org/#/c/212058 and trying to put gate-tempest-dsvm-neutron-full back as part of integrated-gate-neutron but only for the juno branch. AJaeger suggested using gate-tempest-dsvm-neutron-full-juno, but ... | 20:29 |
| yolanda | it's a bit strange , at least for first boot, to deploy your zuul instance and see that nothing runs | 20:29 |
| fungi | nibalizer: not even then. on debian derivatives most daemons start when the package is installed so puppet shouldn't need to second-guess it. and on red hat derivatives, sysadmins explicitly _don't_ expect services to start automatically unless they configure that behavior | 20:29 |
| regXboi | that symbol is already used elsewhere in the layout and I don't want to adversely impact those other uses | 20:29 |
| yolanda | so either we go with that parameter, that looks as a good compromise to me, or document it on the module, to avoid thinking it's a mistake (i thought it was) | 20:29 |
| nibalizer | so leaving the default to unmanaged is fine with me | 20:30 |
| yolanda | then you need a manual intervention, or other orchestration layer on top, to spin it up | 20:30 |
| nibalizer | I think we can accomplish that by passing undef to ensure | 20:30 |
| yolanda | fungi, do you think it's worth reverting, or iterate on top? | 20:30 |
| nibalizer | and then a parameter can be opened up in the module to open that up | 20:30 |
| fungi | nibalizer: yolanda: i'm fine with digging deeper into potential options, but would like clarkb and jeblair to participate in the discussion (like me they seem to have missed that review and they don't seem to be around at the moment) | 20:31 |
| fungi | documenting that it's intentional that the module doesn't start services sounds like a great place to start | 20:31 |
| nibalizer | yep, I liked your idea of heading to the ML | 20:31 |
| yolanda | fungi, ok, if you want to send something to the mailing list, we can discuss there as well. Honestly, i prefer that, as i'll be able to follow even if the conversation happens at times where i'm not here | 20:31 |
| fungi | yolanda: yep, i'll go ahead and start a discussion thread on it | 20:32 |
| yolanda | great | 20:32 |
| yolanda | going back with children, i'll follow tomorrow | 20:32 |
| fungi | and thanks for bearing with us. adding lots of new core reviewers is awesome because it means we get a lot more review bandwidth, but it definitely also means there are undocumented assumptions the original reviewers were around for that the new reviewers lack context on | 20:33 |
| yolanda | fungi yes, i feel i miss that bits, i know the modules from the downstream usage, but i'm missing the general infra context | 20:34 |
| *** hdd has joined #openstack-infra | 20:34 | |
| regXboi | jeblair, fungi: alternatively, I could just leave integated-gate-neutron alone and not use it in the openstack/neutron path ... | 20:34 |
| *** mwagner_lap has quit IRC | 20:35 | |
| fungi | i definitely appreciate you reviewing (and approving!) changes, so i don't want to discourage that. but there will be some where we revert and extend discussion (and then possibly reinstate once the discussion is had) | 20:35 |
| yolanda | fungi, i'm not normally approving anything conflictive on my timezone because of that, i leave nibalizer the honours :) | 20:36 |
| yolanda | don't want to break anything without infra roots to fix | 20:36 |
| jeblair | yeah, the thing about having puppet enforce that something is running is that then there is no way to have puppet apply a configuration change while the service is down without also starting it | 20:36 |
| jeblair | personally, i think puppet ought to ensure that something is configured to start at boot (ensure it is enabled), but not ensure it is running. that's a transient thing that, if automated at all, should be done via something like ansible | 20:37 |
| krotscheck | nibalizer: Sorry for bouncing out, had to have a conversation. | 20:38 |
| fungi | regXboi: i'm about out of runway for new topics today but am trying to wrap my head around your dilemma now. i'll follow up on 212058 with comments there | 20:38 |
| * krotscheck is distracted by midcycle things :/ | 20:38 | |
| *** barnaby has joined #openstack-infra | 20:38 | |
| regXboi | fungi: thanks - I'm thinking way outside the box now | 20:38 |
| nibalizer | krotscheck: no worries me too man | 20:38 |
| regXboi | fungi: but I'm thinking this idea may have to wait until juno is retired | 20:39 |
| *** gyee has quit IRC | 20:39 | |
| krotscheck | nibalizer: Ok, so. PPA is not a good idea. npm running every 15 minutes is too much overhead for me, but using schedule{} is not idempotent enough for you. | 20:39 |
| *** madong_ has joined #openstack-infra | 20:40 | |
| *** larainema has quit IRC | 20:40 | |
| *** bswartz has quit IRC | 20:40 | |
| krotscheck | nibalizer: Using the puppet node module is... well, I'm leery of introducing that into infra, because it's a small step there from "We're just using it on the release slave" to "Oops we tested all javascript in non-distro node" | 20:40 |
| jeblair | fungi: have you proposed a revert? | 20:41 |
| fungi | jeblair: already approved it, nibalizer proposed it | 20:41 |
| nibalizer | jeblair: https://review.openstack.org/#/c/212176/ | 20:41 |
| krotscheck | nibalizer: What other options do we have? | 20:41 |
| openstackgerrit | Madhusudhan Kandadai proposed openstack-infra/project-config: Create experimental job for all neutron-vpnaas tests https://review.openstack.org/211767 | 20:41 |
| nibalizer | krotscheck: iduno let me headscratch for a bit | 20:41 |
| openstackgerrit | Henry Gessau proposed openstack-infra/elastic-recheck: Delete query for Bug #1477192 https://review.openstack.org/212180 | 20:41 |
| openstack | bug 1477192 in neutron "neutron test_multi_prefix_slaac failing in the gate with ping failures starting around 7/22" [High,Confirmed] https://launchpad.net/bugs/1477192 - Assigned to Henry Gessau (gessau) | 20:41 |
| fungi | regXboi: yeah, i think the solution is simple there, just need to double-check something before i potentially provide an incomplete answer | 20:42 |
| nibalizer | sorry for merging the zuul service change by the way | 20:42 |
| krotscheck | nibalizer: If there was an easy version-compare script I could use in "unless"? sometihng like "unless npm --version | versioncompare < 1.3.18" | 20:42 |
| fungi | nibalizer: no need to apologize. the fault is on me and others for not seeing that change for many months | 20:43 |
| regXboi | fungi: thx, I'll read it later | 20:43 |
| *** tiswanso has quit IRC | 20:44 | |
| crinkle | krotscheck: nibalizer there is already a change to add the nodejs module if you hadn't seen it https://review.openstack.org/#/c/98029/ | 20:45 |
| nibalizer | ya we needed that for jquery in zuul iirc | 20:45 |
| crinkle | krotscheck: nibalizer just having the module installed on the puppet master won't affect anything on its own but comment if you feel strongly it shouldn't be there | 20:45 |
| crinkle | krotscheck: nibalizer I haven't followed the whole discussion but it might be worth seeing if we can add the functionality we need to that module | 20:46 |
| crinkle | being open source citizens and all | 20:46 |
| *** pc_m has quit IRC | 20:46 | |
| nibalizer | crinkle: ++ | 20:46 |
| *** akshai has joined #openstack-infra | 20:47 | |
| *** mriedem is now known as mriedem_away | 20:47 | |
| openstackgerrit | David Shrewsbury proposed openstack-infra/shade: Change functional testing to use clouds.yaml https://review.openstack.org/212083 | 20:47 |
| fungi | regXboi: commented | 20:48 |
| regXboi | fungi: ack | 20:48 |
| *** mriedem_away has quit IRC | 20:48 | |
| * regXboi trying to understand why a unit test run locally that passed, failed in the pipeline :( | 20:48 | |
| fungi | regXboi: what change did you see it on? | 20:49 |
| regXboi | fungi: one I pushed about 40 minutes ago - I'm thinking I did something colossally stupid | 20:49 |
| regXboi | yep | 20:50 |
| regXboi | I did | 20:50 |
| *** hdd has quit IRC | 20:52 | |
| *** hdd has joined #openstack-infra | 20:52 | |
| *** swat30 has joined #openstack-infra | 20:54 | |
| krotscheck | crinkle: Urm. Well. Let's jus tdo that then. | 20:55 |
| krotscheck | That's a really old patch | 20:55 |
| *** rfolco has quit IRC | 20:55 | |
| *** stevemar has quit IRC | 20:57 | |
| openstackgerrit | Michael Krotscheck proposed openstack-infra/system-config: Add nodejs to the source modules https://review.openstack.org/98029 | 20:58 |
| krotscheck | nibalizer, crinkle ^^ | 20:58 |
| *** HeOS has joined #openstack-infra | 20:58 | |
| *** krtaylor has quit IRC | 20:58 | |
| nibalizer | krotscheck: so does that do what you need? | 20:59 |
| openstackgerrit | Tim Buckley proposed openstack-infra/devstack-gate: Collect DStat CSV logs if they exist. https://review.openstack.org/212189 | 20:59 |
| nibalizer | what is the NodeSource repository that it claims to use? | 20:59 |
| krotscheck | nibalizer: I have no idea. | 20:59 |
| krotscheck | nibalizer: Oh. That's the PPA I was mentioning earlier | 20:59 |
| *** sharikapongubala has joined #openstack-infra | 20:59 | |
| krotscheck | _SIGHHH_ | 20:59 |
| krotscheck | nibalizer: So it's effectively no different than using the ppa. | 21:01 |
| *** achanda has joined #openstack-infra | 21:01 | |
| krotscheck | I suppose at least this way, it's centralized and we can evict it if necessary | 21:01 |
| nibalizer | 0uh | 21:01 |
| nibalizer | https://deb.nodesource.com/setup | 21:01 |
| nibalizer | look they made us something to curl | bash | 21:02 |
| nibalizer | I am filled with unconfidence | 21:02 |
| jeblair | is nodesource the enterprisey one? | 21:02 |
| fungi | nibalizer: it's curl | sudo bash! | 21:02 |
| *** otter768 has joined #openstack-infra | 21:02 | |
| krotscheck | jeblair: Yes. Over beers, let me tell you _exactly_ what that means. | 21:02 |
| nibalizer | jeblair: yep https://nodesource.com/ | 21:02 |
| * krotscheck had lunch with their founders a few weeks back. | 21:03 | |
| openstackgerrit | Merged openstack-infra/subunit2sql: Update the in-tree TODO list https://review.openstack.org/212150 | 21:03 |
| * jeblair is unsure whether 'enterprise' is being used in its complimentary of pejorative sense. | 21:03 | |
| krotscheck | Don't get me wrong, I love that they're pushing the node community towards being saner in their deployment model. | 21:03 |
| jeblair | s/of/or/ | 21:03 |
| jeblair | though weirdly, that sentence also makes sense the way it was written | 21:04 |
| krotscheck | jeblair: I think in this case it's meant in the "We're a startup and need to earn money" sense | 21:04 |
| krotscheck | They're good people. I feel like their business model is going to be eaten by open source in the long run. | 21:04 |
| nibalizer | it almost sounds like they are shipping 'their' node and not 'the' node | 21:04 |
| nibalizer | which is concerning | 21:04 |
| krotscheck | nibalizer: They do have a version of node that is "hardened for enterprise", however they charge money for that. | 21:05 |
| * nibalizer has flashbacks to ree | 21:05 | |
| krotscheck | I can check with joe to make sure that's node's HEAD | 21:05 |
| lifeless | http://bugs.python.org/issue17703 is reported as affecting 3.4: have we seen it at all ? | 21:07 |
| *** otter768 has quit IRC | 21:07 | |
| *** olaph has joined #openstack-infra | 21:07 | |
| *** barnaby has quit IRC | 21:08 | |
| krotscheck | nibalizer, jeblair: It's pure upstream, from the horses mouth. | 21:10 |
| krotscheck | I can unpack the .deb to make sure if you're paranoid | 21:10 |
| nibalizer | krotscheck: so I'm running time npm install npm -g --upgrade and it seems to be 28 seconds | 21:10 |
| nibalizer | but not idempotent | 21:11 |
| nibalizer | any exec without an unless or a creates can't converge | 21:11 |
| *** krtaylor has joined #openstack-infra | 21:11 | |
| krotscheck | nibalizer: ok. | 21:11 |
| krotscheck | nibalizer: Yep. | 21:11 |
| fungi | lifeless: not to my knowledge, we're mostly just plagued by http://bugs.python.org/issue21435 in the 3.4 in trusty | 21:11 |
| krotscheck | nibalizer: As I said, something like `npm --version | versioncompare.sh 1.3.2` or something would make that idempotent | 21:11 |
| nibalizer | krotscheck: well but | 21:12 |
| nibalizer | then you have to know what version | 21:12 |
| nibalizer | and you didn't like that idea? | 21:12 |
| krotscheck | nibalizer: So, puppet has a versioncmp() method | 21:12 |
| *** Sukhdev_ has quit IRC | 21:13 | |
| krotscheck | nibalizer: And I know we want at least 1.3.18 | 21:13 |
| *** yamamoto has joined #openstack-infra | 21:13 | |
| nibalizer | unless => 'npm --version | grep 1.3.10' would seem to work | 21:13 |
| krotscheck | nibalizer: And that provides -1, 0, 1 (I think) | 21:13 |
| nibalizer | well npm --version prints out the version we have locally | 21:13 |
| nibalizer | not the version that we /could/ install | 21:13 |
| krotscheck | nibalizer: So, can we use that? | 21:14 |
| nibalizer | we can't asses the availablity and suitability of a component without going out over the network | 21:14 |
| nibalizer | s/component/candidate/ | 21:14 |
| *** ldnunes has quit IRC | 21:14 | |
| krotscheck | nibalizer: I was thinking ... versioncmp('npm --version', 1.3.18) > 0 | 21:15 |
| krotscheck | nibalizer: So: Is the current version larger than what we need? If so, great. Otherwise, go install it. | 21:15 |
| nibalizer | ya you can't shell out like that in puppet | 21:15 |
| krotscheck | nibalizer: I know, it's pseudocode. | 21:15 |
| krotscheck | nibalizer: no way to capture the return value of that command then, eh? | 21:15 |
| nibalizer | nope | 21:15 |
| *** thorst has quit IRC | 21:16 | |
| krotscheck | nibalizer: let's just migrate to ansible. | 21:16 |
| nibalizer | okay | 21:16 |
| clarkb | you can exec and trigger on that based on success fail right? | 21:17 |
| clarkb | with a dependency | 21:17 |
| clarkb | we did it for file hash checking | 21:17 |
| *** jalarpo has joined #openstack-infra | 21:18 | |
| openstackgerrit | Doug Hellmann proposed openstack-infra/release-tools: use zuul-cloner to check out files https://review.openstack.org/211340 | 21:18 |
| *** bswartz has joined #openstack-infra | 21:18 | |
| *** sharikapongubala has quit IRC | 21:18 | |
| crinkle | yes you can onlyif => "shell commands" | 21:19 |
| *** yamamoto has quit IRC | 21:19 | |
| *** aysyd has quit IRC | 21:19 | |
| fungi | gotta disappear for a couple hours. bbl | 21:19 |
| adreznec | anteaya: are you the right person to talk to about finding the correct reviewers for a proposed project rename and getting it on the list of scheduled renames? | 21:19 |
| anteaya | enjoy | 21:19 |
| anteaya | adreznec: well I am one | 21:20 |
| nibalizer | so I think I have a ifx | 21:20 |
| *** sergmelikyan has quit IRC | 21:20 | |
| *** sharikapongubala has joined #openstack-infra | 21:21 | |
| *** zul has quit IRC | 21:21 | |
| anteaya | adreznec: create your patch and add the url to this section of the infra meeting agenda: https://wiki.openstack.org/wiki/Meetings/InfraTeamMeeting#Upcoming_Project_Renames | 21:21 |
| anteaya | adreznec: once a rename is scheduled, the patches will be reviewed so be available on the week the rename is scheduled | 21:21 |
| adreznec | anteaya: Okay, will do. I have a patch up for review now to move a neutron driver from stackforge into the openstack namespace following the neutron stadium model. the governance change was approved today | 21:22 |
| anteaya | renames are scheduled in infra weekly meetings | 21:22 |
| *** barnaby has joined #openstack-infra | 21:22 | |
| anteaya | adreznec: great add the url for the project-config patch to the rename section of the wiki page | 21:22 |
| anteaya | and ensure the governance patch is linked in the commit message of the project-config patch | 21:22 |
| anteaya | with Depends-On: <change id for governance patch> | 21:23 |
| adreznec | anteaya: Already did the latter. I'll get that up on the infra meeting page here. Thanks! | 21:23 |
| anteaya | thank you | 21:23 |
| krotscheck | nibalizer: oookay? | 21:23 |
| morgan_404 | krotscheck: kayooo? | 21:23 |
| krotscheck | morgan_404: ENOTENGLISH | 21:24 |
| morgan_404 | NotFound | 21:24 |
| openstackgerrit | Matthew Treinish proposed openstack-infra/elastic-recheck: Add support to filter results by failure test_ids https://review.openstack.org/209712 | 21:24 |
| nibalizer | krotscheck: so your npm works? This is the second time my npm has been rendered unusable by npm install -g --upgrade npm | 21:24 |
| krotscheck | nibalizer: Yep. | 21:24 |
| krotscheck | nibalizer: Like a charm. | 21:24 |
| mtreinish | clarkb: ^^^ I switched to using the all in one db api method I added in that rev | 21:25 |
| nibalizer | krotscheck: you logged out and logged back in? | 21:25 |
| krotscheck | nibalizer: Well, I killed the vm and restarted it. That counts, right? | 21:25 |
| nibalizer | ya | 21:25 |
| nibalizer | ill try to get a really clean reproduction for you | 21:25 |
| clarkb | can we just nodenv whatever you are trying to do? | 21:25 |
| clarkb | works great in my experience | 21:26 |
| krotscheck | clarkb: Not a python project. | 21:26 |
| clarkb | no its node | 21:26 |
| krotscheck | All I'm trying to do is execute `npm publish --ignore-scripts <tarball>` | 21:26 |
| krotscheck | That's it. | 21:26 |
| clarkb | and nodeenv nodes | 21:26 |
| krotscheck | Why is this so hard? | 21:26 |
| clarkb | because node | 21:27 |
| krotscheck | No, because debian | 21:27 |
| clarkb | I mean I currenrly build from scratch for etherpad | 21:27 |
| *** dkranz has quit IRC | 21:27 | |
| clarkb | well debian isnt treating node special | 21:27 |
| jeblair | krotscheck: should we revisit the "curl" idea? | 21:27 |
| nibalizer | hehe | 21:28 |
| nibalizer | I kindof like that idea | 21:28 |
| clarkb | and many other interpreters/compilers run on it just fine including those that run js (iceweasel) | 21:28 |
| nibalizer | I also am more okay with using the NodeSource repo if they are GoodPeople(tm) | 21:28 |
| krotscheck | jeblair: We could, if someone is interested in reverse engineering the npm repository API. It's not documented anywhere. | 21:28 |
| *** pblaho has quit IRC | 21:28 | |
| nibalizer | I spent some time looking for docs, and at the npm source, the 'how to' was not forthcomming | 21:29 |
| krotscheck | So what's wrong with schedule{}? | 21:29 |
| krotscheck | nibalizer: Honestly, I think I trust the nodesource people more than I trust the npm people. | 21:30 |
| krotscheck | (not by much) | 21:30 |
| nibalizer | heh | 21:30 |
| nibalizer | so what I don't like about 210601, two things | 21:31 |
| nibalizer | 1) schedule is weird, we shouldn't need to do stuff like this only once a day | 21:31 |
| *** tlian has quit IRC | 21:31 | |
| nibalizer | as jeblair said, we'd be tapping our foot waiting for a security update befor we'd rememebr that we'd invoked 'schedule' | 21:31 |
| nibalizer | 2) as written, that exec can never converge | 21:31 |
| nibalizer | it will just run every time (or once a day) | 21:31 |
| jeblair | my order of preference is: curl; npm-self-upgrade-without-schedule; nodesource ppa; npm-self-upgrade-with-schedule | 21:32 |
| openstackgerrit | sebastian marcet proposed openstack-infra/openstackid: OIDC - OpenId Connect Implementation https://review.openstack.org/198458 | 21:32 |
| *** rkukura has quit IRC | 21:32 | |
| nibalizer | now we could fix that in a few different ways, the easiest of which is unless => 'npm --version | grep <versionnumber>' and someone has to manually update <versionnumber> periodically | 21:32 |
| *** w_ has joined #openstack-infra | 21:34 | |
| *** zul has joined #openstack-infra | 21:34 | |
| jeblair | nibalizer: i don't like that -- part of why we use distros is because we want them to do some of this work for us | 21:35 |
| nibalizer | jeblair: i agree | 21:35 |
| *** olaph has quit IRC | 21:36 | |
| openstackgerrit | Jeff Peeler proposed openstack-infra/project-config: Kolla: add non-voting build job https://review.openstack.org/212194 | 21:37 |
| nibalizer | so the publish job itself, could curl https://nodejs.org/dist/v0.12.7/node-v0.12.7-linux-x64.tar.gz; unpack, ./bin/npm publish; rm | 21:37 |
| nibalizer | that link is just the latest link but could be hardcoded or not | 21:38 |
| krotscheck | Is updating to vivid an option on the release slave? | 21:38 |
| * krotscheck is assuming that the version in npm updated. | 21:38 | |
| *** akshai has quit IRC | 21:39 | |
| krotscheck | (seems to be recent enough | 21:40 |
| krotscheck | That satisfies the "Let the distros do that work for us" requirement | 21:40 |
| nibalizer | moving the release slave to vivid isn't a good call because that just signs us up for upgrading a server every 9 months | 21:40 |
| *** funzo has joined #openstack-infra | 21:41 | |
| *** jalarpo has quit IRC | 21:41 | |
| *** mwagner_lap has joined #openstack-infra | 21:41 | |
| *** jsavak has quit IRC | 21:41 | |
| nibalizer | krotscheck: http://paste.ubuntu.com/12065921/ | 21:41 |
| krotscheck | nibalizer: p node - Amateur Packet Radio Node program (transitional package) | 21:43 |
| krotscheck | nibalizer: What you want is `apt get install nodejs npm nodejs-legacy` | 21:43 |
| krotscheck | Also, purge node | 21:43 |
| *** fawadkhaliq has joined #openstack-infra | 21:43 | |
| krotscheck | Because naming things is hard | 21:44 |
| openstackgerrit | Merged openstack-infra/jenkins-job-builder: Add a JJB quick start guide https://review.openstack.org/208037 | 21:45 |
| *** funzo has quit IRC | 21:45 | |
| nibalizer | krotscheck: aha thank you | 21:46 |
| openstackgerrit | Colleen Murphy proposed openstack-infra/puppet-cgit: Add basic beaker tests https://review.openstack.org/212198 | 21:46 |
| krotscheck | nibalizer: That's actually a good catch, because the npm package from debian knows to go to the debian-named executable, while the npm-self-installed version goes to the default 'node' exec. | 21:47 |
| krotscheck | So any self upgraded npm requires the nodejs-legacy package | 21:47 |
| nibalizer | ok | 21:48 |
| nibalizer | also note that this is setting my npm version considerably higher than I think you anticipated | 21:48 |
| nibalizer | root@derpderp8383838:~# npm --version | 21:48 |
| nibalizer | 2.13.4 | 21:48 |
| openstackgerrit | Matthew Treinish proposed openstack-infra/elastic-recheck: Add support to filter results by failure test_ids https://review.openstack.org/209712 | 21:49 |
| *** fawadkhaliq has quit IRC | 21:50 | |
| krotscheck | nibalizer: That is correct. | 21:50 |
| krotscheck | nibalizer: And expected. | 21:50 |
| nibalizer | haha so when excuting 'npm search npm' I run out of memory on my vm | 21:50 |
| openstackgerrit | Merged openstack-infra/release-tools: document release_postversion.sh https://review.openstack.org/211688 | 21:50 |
| krotscheck | I think I have a solution everyone will like, just a moment | 21:52 |
| *** amitgandhinz has joined #openstack-infra | 21:53 | |
| *** jcoufal__ has quit IRC | 21:54 | |
| *** annegentle has quit IRC | 21:54 | |
| nibalizer | krotscheck: what if we change the exec to look like this http://paste.openstack.org/show/412792/ | 21:55 |
| krotscheck | nibalizer: Maybe. I was thinking more along the lines of `npm install -g --upgrade npm@2.13.4` | 21:56 |
| *** yamamoto has joined #openstack-infra | 21:57 | |
| nibalizer | well then we are setting the version number again | 21:57 |
| krotscheck | And then your version comparison will just work | 21:57 |
| *** tsg has quit IRC | 21:58 | |
| krotscheck | Hrm. | 21:58 |
| krotscheck | Point | 21:58 |
| krotscheck | And your version is technically idempotent | 21:58 |
| *** bpokorny_ has joined #openstack-infra | 21:58 | |
| krotscheck | I think it still checks the cache though | 21:58 |
| *** warpc__ has quit IRC | 21:58 | |
| krotscheck | Let me see how that works in my vm. | 21:58 |
| *** amitgandhinz has quit IRC | 21:58 | |
| krotscheck | btw, were you able to fix your npm install on your laptop? | 21:59 |
| nibalizer | ya | 21:59 |
| openstackgerrit | Merged openstack-infra/zuul: Fix two typos on zuul documentation https://review.openstack.org/206535 | 21:59 |
| *** otter768 has joined #openstack-infra | 22:01 | |
| *** bpokorny has quit IRC | 22:01 | |
| openstackgerrit | Merged openstack-infra/zuul: Ask for commit-message on current-patchset https://review.openstack.org/187268 | 22:02 |
| krotscheck | nibalizer: That seems to work! | 22:04 |
| *** regXboi has quit IRC | 22:05 | |
| openstackgerrit | Michael Krotscheck proposed openstack-infra/system-config: Keep npm up-to-date. https://review.openstack.org/210601 | 22:05 |
| krotscheck | Let's see if gerrit agrees | 22:05 |
| *** madong_ is now known as larainema | 22:12 | |
| *** gyee has joined #openstack-infra | 22:14 | |
| jeblair | wow https://review.openstack.org/175359 is not a typo, it is a serious bug | 22:14 |
| krotscheck | nibalizer: https://review.openstack.org/#/c/210601/ passed, your comments would be much appreciated :) | 22:16 |
| *** flepied has joined #openstack-infra | 22:16 | |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul: Remove ready_ahead from scheduler https://review.openstack.org/175359 | 22:17 |
| *** sthillma has joined #openstack-infra | 22:18 | |
| *** pradk has quit IRC | 22:18 | |
| *** w_ has quit IRC | 22:19 | |
| *** tonytan4ever has quit IRC | 22:19 | |
| *** w_ has joined #openstack-infra | 22:19 | |
| openstackgerrit | Tim Buckley proposed openstack-infra/devstack-gate: Install and run StackViz during devstack-gate host cleanup. https://review.openstack.org/212207 | 22:24 |
| *** cdent has quit IRC | 22:25 | |
| openstackgerrit | Colleen Murphy proposed openstack-infra/puppet-cgit: Add basic beaker tests https://review.openstack.org/212198 | 22:26 |
| *** bpokorny has joined #openstack-infra | 22:29 | |
| *** krtaylor has quit IRC | 22:30 | |
| *** gordc has quit IRC | 22:31 | |
| *** bpokorny_ has quit IRC | 22:33 | |
| *** marcusvrn has quit IRC | 22:33 | |
| thiagop | Hey guys, if you could kindly take a look on this review to approve our new project on the infra: https://review.openstack.org/#/c/211301/ | 22:34 |
| thiagop | it already has a +2 :) | 22:34 |
| *** sharikapongubala has quit IRC | 22:35 | |
| sdake | hey folks quick question about custom jobs in the gate | 22:36 |
| *** marekd is now known as marekd_404 | 22:36 | |
| sdake | in https://review.openstack.org/#/c/212194/1/jenkins/jobs/kolla.yaml, we are going to end up with very minor variations of this job with tox -e images-centos-rdo, images-centos-src, etc | 22:37 |
| *** thiagop is now known as tpborion_afk | 22:37 | |
| sdake | we will have a total of 12 jobs when done | 22:37 |
| sdake | 6 distros * (src+rdo) | 22:37 |
| sdake | isome (src) will be voting others (bin) will not be over time | 22:38 |
| sdake | what is the best way to do that so we dont have to repeat the above kolla.yaml job-template 12 times | 22:38 |
| sdake | note we can't just use the tox thing we use elsewhere in our gating because this is not a tox job | 22:39 |
| *** dguerri` is now known as dguerri | 22:39 | |
| *** achanda has quit IRC | 22:40 | |
| *** bnemec has quit IRC | 22:41 | |
| *** ivar-lazzaro has joined #openstack-infra | 22:42 | |
| *** olaph has joined #openstack-infra | 22:43 | |
| *** dguerri is now known as dguerri` | 22:43 | |
| *** erlon has quit IRC | 22:43 | |
| *** annegentle has joined #openstack-infra | 22:44 | |
| sdake | basically what I'm after is something like: | 22:44 |
| sdake | 'gate-{name}-tox-{envlist}':4943 | 22:44 |
| sdake | 4944 envlist: validate-contents envlist: validate-contents | 22:44 |
| sdake | sorry cut to much | 22:44 |
| *** devlaps has joined #openstack-infra | 22:44 | |
| sdake | gate-{name}-tox-{envlist} | 22:44 |
| sdake | but I don't quite know where to look as to how that is implemented | 22:45 |
| *** annegentle has quit IRC | 22:45 | |
| *** w_ has quit IRC | 22:45 | |
| sdake | https://review.openstack.org/#/c/212194/1/jenkins/jobs/projects.yaml | 22:45 |
| sdake | and it needs to work like this - with the pipeline and node arguments | 22:45 |
| *** annegentle has joined #openstack-infra | 22:46 | |
| *** tpborion_afk has quit IRC | 22:47 | |
| *** hdd has quit IRC | 22:47 | |
| jhesketh | Morning | 22:48 |
| anteaya | morning jhesketh | 22:50 |
| *** austin81 has left #openstack-infra | 22:50 | |
| jhesketh | o/ | 22:51 |
| *** annegentle has quit IRC | 22:52 | |
| *** dimtruck is now known as zz_dimtruck | 22:53 | |
| *** camunoz has joined #openstack-infra | 22:54 | |
| *** yamamoto has quit IRC | 22:54 | |
| lifeless | sdake: project-config/jenkins/python-jobs IIRC | 22:55 |
| lifeless | sdake: grep for r'gate-{name}-tox-{envlist}' | 22:55 |
| sdake | lifeless thanks i'll look there | 22:55 |
| sdake | i dont totally get how these parameter things work | 22:55 |
| openstackgerrit | Merged openstack/requirements: Bump saharaclient to the latest version https://review.openstack.org/212072 | 22:58 |
| *** FallenPegasus has quit IRC | 23:00 | |
| sdake | lifeless unpeeling the union that has | 23:04 |
| sdake | - tox: | 23:04 |
| sdake | envlist: '{envlist}' | 23:04 |
| sdake | suggestions for where tox job is located? | 23:04 |
| sdake | python-jobs? | 23:04 |
| sdake | tox is used in a bajillion places in the repo | 23:04 |
| sdake | not sure grep will help | 23:04 |
| * timrc wonders if he should do an #openstack-infra BBQ at his house for the Openstack Summit in Austin. Not sure I could smoke enough brisket. Would have to cater. | 23:06 | |
| nibalizer | or start smoking now | 23:06 |
| *** EricGonczer_ has joined #openstack-infra | 23:07 | |
| lifeless | sdake: I think so | 23:09 |
| sthillma | Hi Infra team, need to modify the core members for a stackforge group (puppet-n1k-vsm-core/puppet-n1k-vsm-release specifically), how would I go about doing that? | 23:10 |
| *** EricGonc_ has quit IRC | 23:11 | |
| sdake | sthillma if its a new project you have to get someone with appropriate permissions to add you to the gerrit group | 23:11 |
| sdake | fungi usually does the job for me when I ask | 23:11 |
| sdake | if someone is already in the group, they add the others | 23:11 |
| sdake | its self-managed | 23:11 |
| *** hichihara has joined #openstack-infra | 23:11 | |
| sthillma | sdake: ah ok, is there docs on how to do that somewhere? | 23:12 |
| sthillma | it is an existing group | 23:12 |
| sdake | sec | 23:12 |
| sdake | finding ink | 23:12 |
| sdake | oh a fellow cisco dude, hey :) | 23:12 |
| sdake | https://review.openstack.org/#/admin/groups/420,members | 23:12 |
| sdake | someone in that group needs to do the changing | 23:13 |
| *** esmute_ has quit IRC | 23:13 | |
| sdake | you got the 420 group lol | 23:13 |
| sdake | if someone logs in there is a button add remove if they are in th ecore group | 23:13 |
| sthillma | sdake: yep, thanks! :) lol! nice! also on a related note, I need to add the create permission to our ACL is any bug required for that, or do i just open a review? | 23:14 |
| sdake | there is also a release group for tagging | 23:14 |
| sdake | open review - project config doesn't erquire bug tracking afaik | 23:14 |
| *** markvoelker has quit IRC | 23:14 | |
| sdake | to find groups, click "People" "List Groups" type name in filter, click | 23:15 |
| mmmpork | hey folks, i'm running into another issue with the testing.rst instructions i've been working on for nodepool | 23:16 |
| mmmpork | i'm hoping someone with more openstack knowledge can help me troubleshoot this | 23:16 |
| *** claudiub has quit IRC | 23:16 | |
| *** achanda has joined #openstack-infra | 23:16 | |
| sthillma | sdake found it, thanks so much for the quick response! | 23:16 |
| mmmpork | when i launch nodepool, i keep seeing the following error "devstack does not have image ubuntu for label ubuntu" | 23:16 |
| mmmpork | when i do nova image-list i see ubuntu in the list | 23:17 |
| *** ashleighfarnham has quit IRC | 23:17 | |
| mmmpork | i also tried to manually create an instance and that worked ok too | 23:17 |
| *** sripriya has left #openstack-infra | 23:17 | |
| *** zul has quit IRC | 23:17 | |
| mmmpork | i'll link to the instructions i'm referring to for how i created the image | 23:17 |
| mmmpork | and also to my yaml file | 23:17 |
| mmmpork | here's the yaml file: https://github.com/missaugustina/my_nodepool/blob/master/nodepool.yaml | 23:18 |
| mmmpork | here's the debug log when i start up nodepool: http://paste.openstack.org/show/412802/ | 23:18 |
| *** esmute_ has joined #openstack-infra | 23:19 | |
| mmmpork | this is the best i can do for linking to the changes, https://review.openstack.org/#/c/179840/6/doc/source/testing.rst | 23:20 |
| openstackgerrit | Steven Hillman proposed openstack-infra/project-config: Add create permission for puppet-n1k-vsm core group https://review.openstack.org/212221 | 23:20 |
| clarkb | mmmpork what does nova image-list show? | 23:20 |
| clarkb | you do have to manually upload that image | 23:20 |
| *** doug-fish has left #openstack-infra | 23:21 | |
| mmmpork | oh wait i'm looking for the instructions i used | 23:21 |
| mmmpork | clarkb: this is what nova image-list shows http://paste.openstack.org/show/412804/ | 23:22 |
| *** hdd has joined #openstack-infra | 23:22 | |
| mmmpork | i used the instructions in the README.rst i think | 23:23 |
| mmmpork | for setting up the image | 23:23 |
| *** esmute_ has quit IRC | 23:24 | |
| mmmpork | oh wait no they aren't in there, sorry still trying to find it | 23:24 |
| clarkb | its in the proposed testing doc | 23:24 |
| mmmpork | yeah it is :) | 23:25 |
| mmmpork | i just missed it for some reason | 23:25 |
| mmmpork | yeah so i followed those instructions verbatim | 23:25 |
| clarkb | so I think thats a nodepool.yaml config issue somehwere not a glance/nova issue | 23:25 |
| *** chlong has quit IRC | 23:25 | |
| mmmpork | yeah that's what i was wondering | 23:25 |
| mmmpork | i've been looking at the yaml config and nothing is popping out at me | 23:25 |
| mmmpork | unless a username is wrong somewhere | 23:26 |
| mmmpork | i'm just using demo | 23:26 |
| clarkb | I think it is saying that the image ubunut isnt found under provider images in nodepool config | 23:26 |
| mmmpork | right | 23:27 |
| clarkb | can you double check and possibly paste tge exact nodepool.yaml just remove the password | 23:27 |
| mmmpork | this is the exact one: https://github.com/missaugustina/my_nodepool/blob/master/nodepool.yaml | 23:27 |
| mmmpork | i just set the password to "password" since i made the repo public | 23:28 |
| mmmpork | on my vm i created a symbolic link in etc to that repo in my home dir | 23:28 |
| mmmpork | so i could track my config changes in git | 23:29 |
| clarkb | is password actually password? usually its secret iurc | 23:29 |
| clarkb | hrm is he dburi missing the password? | 23:30 |
| *** sridhar_ram1 has joined #openstack-infra | 23:31 | |
| mmmpork | i haven't been prompted for one | 23:31 |
| mmmpork | currently it's just vanilla setup with no root password | 23:31 |
| sdake | lifeless its in macros.yaml | 23:31 |
| sdake | (tox) | 23:31 |
| *** sridhar_ram has quit IRC | 23:32 | |
| mmmpork | yeah password is just a plain text password | 23:32 |
| mmmpork | i just used "password" becuse that's how the devstack docs had you do it in the example | 23:32 |
| clarkb | ya you need username:password@localhost/nodepool | 23:32 |
| clarkb | thats not in your linked file | 23:32 |
| mmmpork | ah ok | 23:32 |
| mmmpork | i thought that was for talking to mysql | 23:32 |
| mmmpork | so i thought i didn't need one? | 23:32 |
| clarkb | it is for mysql and you should need one if using the devstack configured db | 23:33 |
| mmmpork | kk | 23:33 |
| mmmpork | i think i just left it out and figured something would complain | 23:33 |
| mmmpork | the comment said if it wasn't provided i would be prompted for it? | 23:33 |
| mmmpork | maybe i misunderstood? | 23:33 |
| clarkb | devstack will prompt you for it | 23:34 |
| clarkb | when you build your devstack cloud | 23:34 |
| mmmpork | ok | 23:35 |
| mmmpork | should the user for that be nodepool? | 23:35 |
| mmmpork | or should it be the root/combo that's in the example? | 23:35 |
| mmmpork | root:yourpasswordhere combo | 23:35 |
| clarkb | I think its the one from the example | 23:35 |
| clarkb | thats what devstack uses | 23:35 |
| *** salv-orl_ has joined #openstack-infra | 23:35 | |
| *** esmute_ has joined #openstack-infra | 23:37 | |
| *** salv-orlando has quit IRC | 23:37 | |
| mmmpork | hrm so when i change that i'm getting access denied because i never set up passwords for those users, nodepool or root | 23:37 |
| *** sridhar_ram1 has quit IRC | 23:38 | |
| mmmpork | but with the original config i didn't see any errors in the logs | 23:38 |
| mmmpork | but i'll keep trying | 23:38 |
| *** FallenPegasus has joined #openstack-infra | 23:38 | |
| *** esmute_ has quit IRC | 23:38 | |
| clarkb | when you run devstack it asks you for that password so its whatever you used or set that env var to prior to running devstack | 23:39 |
| mmmpork | alright | 23:39 |
| clarkb | you can check novas config to see what it was set too as well | 23:39 |
| mmmpork | and the user should be root? | 23:39 |
| clarkb | I belive so, check nova | 23:39 |
| *** Alexandra_ has joined #openstack-infra | 23:41 | |
| *** geoffarnold has quit IRC | 23:41 | |
| mmmpork | ahh ok it's under [database] | 23:41 |
| mmmpork | ok it's not complaining now | 23:42 |
| mmmpork | but it's still not finding the image | 23:42 |
| mmmpork | but that's progress | 23:42 |
| mmmpork | i think | 23:42 |
| mmmpork | hahaha | 23:42 |
| clarkb | it has to build it | 23:42 |
| clarkb | do you see logs for the build? also nodepool image-lisy | 23:43 |
| clarkb | *image-list | 23:43 |
| mmmpork | zomgz i see it! | 23:43 |
| mmmpork | :D | 23:43 |
| * mmmpork does a happy dance | 23:43 | |
| mmmpork | dammittall i knew it was something i screwed up in the config | 23:44 |
| mmmpork | i'm going to put a comment about checking for that value in the nova config in that doc | 23:44 |
| clarkb | I wouldnt | 23:44 |
| clarkb | I would tell people to set it explicitly with that devstack co fig var | 23:45 |
| clarkb | its more direct that way | 23:45 |
| *** geoffarnold has joined #openstack-infra | 23:45 | |
| *** geoffarnold has quit IRC | 23:45 | |
| mmmpork | ok that's fair | 23:46 |
| mmmpork | i don't care much for passwords in plain text in config files personally | 23:46 |
| clarkb | its for developmwnt and testing we set them like that everywhere :) | 23:47 |
| mmmpork | oh yeah me too | 23:47 |
| mmmpork | but it's not the best habit ;) | 23:47 |
| clarkb | the alternative is very complicated fir dev/testing | 23:47 |
| clarkb | not sure the tradeoff is worthwhile "here go learn gpg have fun" | 23:48 |
| mmmpork | yeah, i've worked on things that had different ways of doing it, it was definitely a pain | 23:48 |
| clarkb | "lol hate me yet? sorry" | 23:48 |
| mmmpork | on the flip side, i've worked at too many places where devs didn't know any better | 23:48 |
| mmmpork | so being consistent was kind of nice, even if it was painful ;) | 23:48 |
| mmmpork | anyways that's cool, thanks for your help | 23:48 |
| clarkb | in theory something like barbican helps here but that requires a service be run | 23:49 |
| mmmpork | i'll improve the comment on that so in the event someone as newbie as me is doing this they know what those values are referring to | 23:49 |
| clarkb | (though you still need to deal with barbican cred its all turtles) | 23:50 |
| *** zz_dimtruck is now known as dimtruck | 23:50 | |
| *** oomichi has joined #openstack-infra | 23:51 | |
| *** baoli has quit IRC | 23:52 | |
| *** yamamoto has joined #openstack-infra | 23:55 | |
| olaph | i like turtles | 23:55 |
| anteaya | mmmpork: we do encourage folks to choose non-swearing words in this channel | 23:55 |
| *** baoli has joined #openstack-infra | 23:56 | |
| Kennan | fungi: yes, it was raised by nova, but sometimes resource problem can cause that nova to raise that function | 23:56 |
| Kennan | the jenkins passed now, very strange | 23:56 |
| *** wenlock has quit IRC | 23:58 | |
| *** FallenPegasus has quit IRC | 23:58 | |
| *** minwang2 has quit IRC | 23:58 | |
| *** crc32 has quit IRC | 23:58 | |
| *** achanda has quit IRC | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!