dansmith | yep | 00:00 |
---|---|---|
clarkb | mordred: I think we agree on the goal | 00:00 |
mordred | clarkb: now, the mechanism for doing that ... | 00:00 |
clarkb | mordred: we just don't agree on the mechanism | 00:00 |
* krotscheck is blatantly abusing the short zuul queue | 00:00 | |
krotscheck | rebase all the things! | 00:00 |
mordred | krotscheck: now's the time! | 00:00 |
jogo | clarkb: eww | 00:00 |
mordred | clarkb: if docker were an option (stupid iscsi) I'd actually just be sugesting that we cook up a way to have folks be able to collaborate on dockerfiles | 00:00 |
mordred | clarkb: but it's not, beacuse of iscsi | 00:00 |
jogo | mordred: to not using an exernal distro for deciding the libvirt version | 00:01 |
mordred | but the nice thing about it is that it's potentially quite straightforward to express the thing someone wants | 00:01 |
*** dims_ has quit IRC | 00:01 | |
mordred | which making our own packaging repos where people can edit a debian control file or an rpm spec ... not so much | 00:01 |
mordred | there will be like 10 people who hack in those ever and it'll be truly a nightmare | 00:01 |
*** dims_ has joined #openstack-infra | 00:01 | |
clarkb | mordred: ya | 00:02 |
openstackgerrit | Michael Krotscheck proposed a change to openstack-infra/storyboard-webclient: Removed nodeenv dependency https://review.openstack.org/106191 | 00:02 |
mordred | clarkb: which leaves me sort of at a mix between what you were saying and what I'd been thinking ... | 00:02 |
mordred | clarkb: what if we did an overlay ports tree like thing on top of a regular distro | 00:02 |
clarkb | jogo: I can't find the cahnges in gerrit anymore | 00:02 |
clarkb | mordred: hrm, thats not insane | 00:03 |
dansmith | clarkb: don't flatter him | 00:03 |
dansmith | mordred: the best I can say about that thing is that it's not insane | 00:03 |
mordred | it may not be "sexy" ... but like devstack, it might be unsexy enough to actually work | 00:03 |
clarkb | mordred: we might even be able to take arch ports at a snapshot in time and update it periodically | 00:03 |
clarkb | mordred: rather than rolling | 00:03 |
mordred | and also unsexy enough to CLEARLY be something you should not run in production | 00:03 |
mordred | clarkb: ++ | 00:03 |
lifeless | dhellmann: I'm lost - what docstring correctly ? in https://review.openstack.org/#/c/94108/ | 00:03 |
clarkb | except arch is different enough to not work on ubuntu | 00:03 |
mordred | clarkb: you know ... afs has this feature ... | 00:03 |
*** amotoki has joined #openstack-infra | 00:03 | |
clarkb | mordred: I was waiting for afs to solve the problems | 00:04 |
mordred | clarkb: well, it's still the overlay | 00:04 |
lifeless | mordred: you could use nix | 00:04 |
lifeless | :) | 00:04 |
lifeless | its designed directly at the problem space we have here, I think. | 00:04 |
clarkb | lifeless: nix would have the same problem as debs though | 00:04 |
mordred | clarkb: but it does support the idea of installing software into the afs space and then sanely mounting in an arch-dependent way locally | 00:04 |
openstackgerrit | Michael Krotscheck proposed a change to openstack-infra/storyboard: Assocate StoryTag with Story via mapping https://review.openstack.org/113635 | 00:04 |
clarkb | lifeless: its designed to solve this problem but isn't any more approachable than debs imo | 00:04 |
clarkb | lifeless: of course I haven't used it and should probably fix that | 00:05 |
*** marun has joined #openstack-infra | 00:05 | |
mordred | yah. too esoteric for our devs I fear | 00:05 |
*** SumitNaiksatam has joined #openstack-infra | 00:06 | |
*** dims_ has quit IRC | 00:06 | |
lifeless | clarkb: I'm confused about https://review.openstack.org/#/c/107859/; zuul claimed it started stuff, but I can't see it in the zuul queue | 00:06 |
clarkb | lifeless: zuul was just restarted | 00:06 |
clarkb | lifeless: your event may have been eaten | 00:07 |
*** unicell has joined #openstack-infra | 00:07 | |
lifeless | ok, poked again | 00:08 |
clarkb | for the record I gave up on that thread when it turned into a infra go do all this crazy work that I am waving my hands about | 00:08 |
mordred | clarkb: there was a thread? | 00:08 |
clarkb | so I really do think we need to think in concrete possibilities and not wave our hands around what we want here | 00:08 |
clarkb | mordred: yes fair standards for all hypervisor drivers | 00:08 |
mordred | oh. that | 00:08 |
mordred | yeah | 00:08 |
clarkb | mordred: I asked for concrete ways of achieving nova's things | 00:08 |
clarkb | mordred: was hand waved at | 00:09 |
*** amotoki has quit IRC | 00:09 | |
mordred | so, luckily the problem is more general that hypervisors | 00:09 |
mordred | and I think we probably know everythin we need to know at this point to solve it | 00:09 |
mordred | as in, I don't think more data or input will help | 00:09 |
mordred | but I do agree, it's time for concrete proposals | 00:09 |
clarkb | I think it would be worth an experimental arch job | 00:10 |
clarkb | if only to show people that one extreme is not doable (like me) | 00:11 |
clarkb | of course devstack doesn't pacman | 00:11 |
clarkb | so that is probably not worth the effort | 00:11 |
mordred | clarkb: you could get the same effect with a debian sid job | 00:12 |
clarkb | mordred: no arch is miles more stable than debian sid | 00:12 |
mordred | clarkb: but devstack does deb | 00:12 |
krotscheck | mordred: You know tox and node things, mind taking a look at https://review.openstack.org/#/c/106191 ? | 00:12 |
mordred | krotscheck: looking | 00:12 |
*** david-lyle has quit IRC | 00:12 | |
mordred | krotscheck: have you seen dox? | 00:13 |
clarkb | we can also run our own deb and rpm repos with packages we curate (I haev zero interest in this for the reasons above, it shifts all of the responsibility to us and we already have enogh to worry about) | 00:13 |
krotscheck | mordred: I have not. | 00:13 |
salv-orlando | fungi: I and rhsu have found the root cause of the issue. With our great shame we missed the mailing list post about the fact that ^(recheck|reverify).* namespace is off limits. We are now compliant with the recheck syntax. | 00:13 |
clarkb | we can run nix based packages. this has the benefit of allowing us to do version selection at job runtime | 00:13 |
krotscheck | mordred: ANd google hates me | 00:13 |
clarkb | I really like nix for that reason, thought we will have to do the work above | 00:13 |
mordred | krotscheck: thing I've been hacking on recently | 00:13 |
salv-orlando | fungi: maybe if you can indicate me how I can prove that I’ll be happy to do that. | 00:13 |
openstackgerrit | Spencer Krum proposed a change to openstack-infra/puppet-storyboard: Put a default section header above the TTL https://review.openstack.org/100328 | 00:14 |
openstackgerrit | Spencer Krum proposed a change to openstack-infra/puppet-storyboard: StoryBoard module bounces apache on update https://review.openstack.org/100329 | 00:14 |
openstackgerrit | Spencer Krum proposed a change to openstack-infra/puppet-storyboard: Add Storyboard puppet module https://review.openstack.org/100321 | 00:14 |
openstackgerrit | Spencer Krum proposed a change to openstack-infra/puppet-storyboard: Added Authorization Header flag to storyboard module https://review.openstack.org/100322 | 00:14 |
openstackgerrit | Spencer Krum proposed a change to openstack-infra/puppet-storyboard: Make storyboard run over ssl https://review.openstack.org/100323 | 00:14 |
openstackgerrit | Spencer Krum proposed a change to openstack-infra/puppet-storyboard: Load storyboard projects from projects.yaml https://review.openstack.org/100324 | 00:14 |
openstackgerrit | Spencer Krum proposed a change to openstack-infra/puppet-storyboard: Load storyboard superusers from yaml file https://review.openstack.org/100325 | 00:14 |
openstackgerrit | Spencer Krum proposed a change to openstack-infra/puppet-storyboard: Storyboard token expiration increased https://review.openstack.org/100326 | 00:14 |
openstackgerrit | Spencer Krum proposed a change to openstack-infra/puppet-storyboard: Make storyboard idempotent https://review.openstack.org/100327 | 00:14 |
openstackgerrit | Spencer Krum proposed a change to openstack-infra/puppet-storyboard: Modularized StoryBoard Module https://review.openstack.org/113724 | 00:14 |
openstackgerrit | Spencer Krum proposed a change to openstack-infra/puppet-storyboard: Fixed location of storyboard configuration file https://review.openstack.org/113725 | 00:14 |
openstackgerrit | Spencer Krum proposed a change to openstack-infra/puppet-storyboard: Create storyboard configuration directory https://review.openstack.org/113726 | 00:14 |
salv-orlando | I’ve tried it with my personal account but that can’t post comments for “verified” | 00:14 |
krotscheck | ooooOOOooooo | 00:14 |
clarkb | we can use a ports based solution where our image builds build things from source | 00:14 |
mordred | krotscheck: https://github.com/emonty/dox <-- it's like tox, except docker instead of virtualenv | 00:14 |
clarkb | since we already build images doing stuff from source is not super scary | 00:14 |
clarkb | but isn't really dev friendly | 00:14 |
mordred | clarkb: althuogh that means we have to install more things into the iage like gcc and whatnot that could pollute things | 00:15 |
clarkb | mordred: yes, but I think having a dev env on what are basically de vboxes is ok | 00:15 |
mordred | clarkb: also not dev friendly | 00:15 |
krotscheck | mordred: Huhn. That’s pretty neat | 00:15 |
clarkb | mordred: yup | 00:15 |
*** lyxus has joined #openstack-infra | 00:15 | |
lifeless | dhellmann: found it | 00:16 |
*** sdake has quit IRC | 00:16 | |
clarkb | mordred: if we want to be honest te most dev friendly thing is using what the distros give us | 00:16 |
clarkb | mordred: then you don't need anythin special beyond an install of X distro | 00:16 |
openstackgerrit | lifeless proposed a change to openstack-dev/pbr: Teach pbr about post versioned dev versions. https://review.openstack.org/80449 | 00:16 |
lifeless | dhellmann: fixed. | 00:16 |
openstackgerrit | lifeless proposed a change to openstack-dev/pbr: Raise an error if preversion versions are too low https://review.openstack.org/108269 | 00:16 |
openstackgerrit | lifeless proposed a change to openstack-dev/pbr: Look for and process sem-ver pseudo headers in git https://review.openstack.org/108270 | 00:17 |
openstackgerrit | lifeless proposed a change to openstack-dev/pbr: Introduce a SemanticVersion object https://review.openstack.org/94108 | 00:17 |
mordred | krotscheck: that patch does not seem crazy | 00:17 |
krotscheck | mordred: And it runs, too. | 00:17 |
krotscheck | mordred: And the build time is cut in half. | 00:17 |
mordred | clarkb: well, but that's not dev friendly if they want to propose that openstack needs a new version of libvirt | 00:17 |
openstackgerrit | A change was merged to openstack-infra/storyboard-webclient: Updated story list interface. https://review.openstack.org/110165 | 00:17 |
nibalizer | krotscheck: clarkb mordred jeblair that ^^ is the big pile-o storyboard | 00:17 |
mordred | krotscheck: woot! | 00:17 |
nibalizer | so if we can merge that stuff that would be awwwwesome | 00:17 |
nibalizer | i shoud check if tests are passing | 00:18 |
nibalizer | looks like they are :) | 00:18 |
clarkb | mordred: so we are assuming then that some dev cost to have special env for testing is acceptable | 00:18 |
clarkb | mordred: ebcause the value of doing that thing is greater than the value of keeping it simple | 00:18 |
mordred | ya | 00:18 |
mordred | or, at least, that's the theory I'm working from | 00:19 |
lifeless | I think there are competing definitions of 'dev friendly' here | 00:19 |
lifeless | perhaps a couple of personas might help | 00:19 |
mordred | yes | 00:19 |
mordred | because be "dev" I mean "python dev who does not understand distro packaging" | 00:19 |
mordred | which has a subtype of "redhat dev who does not understand debian pacakging" and "debian dev who does not understand redhat packaging" | 00:20 |
lifeless | which is not the same as 'redhat dev' or 'canonical dev' or 'docker dev' | 00:20 |
lifeless | jinx :) | 00:20 |
mordred | :) | 00:20 |
krotscheck | nibalizer: That’s a TON of storyboard thigns. | 00:20 |
mordred | so "dev who does not understand the ins and outs of all existing distro packaging systems" | 00:20 |
mordred | as opposed to "distro packager" | 00:20 |
*** shashankhegde has joined #openstack-infra | 00:20 | |
salv-orlando | fungi, clarkb, jogo, dansmith, mordred, rhsu: so vmware minesweeper triggered hell upstream because I forgot to adhere to the new grammar for 3rd party rechecks | 00:21 |
mordred | salv-orlando: oops :) | 00:21 |
mordred | lifeless: the sucky part is - we have exactly all of the same challenges as people making linux distros, except that we're not making a linux distro | 00:21 |
openstackgerrit | A change was merged to openstack-infra/storyboard-webclient: Updated project list interface. https://review.openstack.org/110169 | 00:21 |
openstackgerrit | A change was merged to openstack-infra/storyboard-webclient: Point search requests to search endpoints https://review.openstack.org/102536 | 00:21 |
salv-orlando | mordred: yeah a bit oops and it’s more oops if you think that a lot of 3rd party cis arent | 00:21 |
openstackgerrit | A change was merged to openstack-infra/storyboard-webclient: Sort stories and projects. https://review.openstack.org/112426 | 00:21 |
salv-orlando | mordred even if they are unlikely to recheck 300 patches in a row, they will still be adding to the upstream load | 00:22 |
*** david-lyle has joined #openstack-infra | 00:23 | |
salv-orlando | and incidentally this whole thing might have surface the floating ip issue we were discussiong with arosen mordred and clarkb | 00:23 |
*** arnaud has quit IRC | 00:23 | |
salv-orlando | but I’m just speculating here | 00:23 |
mordred | salv-orlando: hehe. actually, that was the cause of yesterday's issues | 00:24 |
*** david-lyle has quit IRC | 00:24 | |
salv-orlando | yes, but even yesterday you probably had over 20 ci systems rechecking patches into upstream still this is just speculation... | 00:25 |
*** david-lyle has joined #openstack-infra | 00:25 | |
nibalizer | krotscheck: its just a roll down then a roll up | 00:25 |
salv-orlando | anyway now we’ve sorted out the issue… what’s the process to reenable the service account? I don’t know how to prove I’ve fixed the issue without it ;) | 00:25 |
nibalizer | so all the commits in infra/config that touched the storyboard files, in one place | 00:25 |
krotscheck | nibalizer: Yeah, I recall from the first time you went through this. | 00:26 |
*** vermarah_ has quit IRC | 00:26 | |
nibalizer | proposed as new commits, in order, to the puppet-storyboard mod | 00:26 |
krotscheck | nibalizer: So my goal is to move the rabbitmq things onto that chain? | 00:26 |
*** che-arne|2 has joined #openstack-infra | 00:27 | |
nibalizer | krotscheck: ya | 00:27 |
clarkb | mordred: I really cant help but think rolling release is what we need | 00:27 |
clarkb | then anyone can grab the same rolling release update to latest and be on their way | 00:28 |
nibalizer | i would put something in the commit message that says 'warning, this hasn't landed in config, please actually review it' | 00:28 |
nibalizer | or maybe give it the workflow -1 | 00:28 |
clarkb | no special repos or ports or anything | 00:28 |
mordred | clarkb: but that doesn't solve any of my issues that I want to solve | 00:28 |
openstackgerrit | Salvatore Orlando proposed a change to openstack-infra/config: Add neutron full job to integrated gate https://review.openstack.org/113562 | 00:28 |
*** che-arne has quit IRC | 00:29 | |
mordred | clarkb: which is that it does not enable our developers to have control of the versions of things they want in the same way they have with pip requirements | 00:29 |
*** dims_ has joined #openstack-infra | 00:29 | |
clarkb | mordred only nix does that of the proposed solutions | 00:29 |
mordred | clarkb: amusingly enough - the reason I keep coming back to docker on this point, is that publishing docker base images to a public index is really easy for _us_ to do | 00:29 |
*** david-lyle has quit IRC | 00:29 | |
openstackgerrit | A change was merged to openstack-infra/storyboard-webclient: Removed nodeenv dependency https://review.openstack.org/106191 | 00:30 |
mordred | so a dev saying "docker run -t -i infra/gate" is pretty much all they'd need | 00:30 |
clarkb | maybe we should give nix a go | 00:30 |
openstackgerrit | Michael Krotscheck proposed a change to openstack-infra/storyboard-webclient: Added pagesize to browse service. https://review.openstack.org/109372 | 00:31 |
mordred | clarkb: dude. I just looked at the packaging format | 00:31 |
mordred | https://github.com/NixOS/nixpkgs/blob/master/pkgs/shells/bash/default.nix | 00:31 |
mordred | I dare you to tell our dev community to do that ^^ | 00:31 |
clarkb | oh its terrible | 00:32 |
mordred | yah | 00:32 |
clarkb | but it does the pip like thing | 00:32 |
clarkb | that nothibg else really does | 00:32 |
mordred | clarkb: https://gist.github.com/emonty/5456592ee9d01578e7d1 | 00:33 |
mordred | clarkb: there's a base infra image :) | 00:33 |
openstackgerrit | Michael Krotscheck proposed a change to openstack-infra/storyboard-webclient: Added pagesize to browse service. https://review.openstack.org/109372 | 00:33 |
clarkb | that doesnt pin anything | 00:33 |
clarkb | or build libvirt | 00:33 |
mordred | clarkb: it doesn't need - I didn't say it's a nova image | 00:33 |
openstackgerrit | Spencer Krum proposed a change to openstack-infra/config: Use split-out storyboard module https://review.openstack.org/113731 | 00:34 |
*** reed has quit IRC | 00:34 | |
openstackgerrit | Michael Krotscheck proposed a change to openstack-infra/storyboard-webclient: Added pagesize to browse service. https://review.openstack.org/109372 | 00:34 |
mordred | I'm just saying it's pretty simple to understand | 00:34 |
clarkb | sure docker is shiny but solves a different problem | 00:34 |
mordred | I disagre | 00:34 |
mordred | you just said "nixos does the pip thing liek nothing else really does" - with the concern being ease of devs interacting with the chosen environment | 00:34 |
jroll | 17:30:20 mordred | so a dev saying "docker run -t -i infra/gate" is pretty much all they'd need <- as a dev. this. please. | 00:35 |
mordred | yah. that ^^ | 00:35 |
mordred | dev interaction is _super_ easy and very optimized | 00:35 |
mordred | now, again - iscsi | 00:35 |
clarkb | no the selection of versions | 00:35 |
mordred | but in looking at user interface and what a dev experience can be ... ^^ that is pretty spectacular | 00:35 |
clarkb | docker does not solve that | 00:35 |
openstackgerrit | Michael Krotscheck proposed a change to openstack-infra/storyboard-webclient: Rearranged admin menu https://review.openstack.org/112381 | 00:35 |
openstackgerrit | Michael Krotscheck proposed a change to openstack-infra/storyboard-webclient: Added user administration interface https://review.openstack.org/109439 | 00:35 |
clarkb | nix does (not nixos either) | 00:35 |
mordred | clarkb: docker run -t -i infra/gate:v1.0.0 | 00:36 |
jroll | clarkb: versions of what? | 00:36 |
jroll | yeah, that | 00:36 |
mordred | clarkb: it allows for arbitrary tagging of arbitrary image versions | 00:36 |
clarkb | versions of libvirt | 00:36 |
mordred | clarkb: so, yeah, we can _totally_ publish a ton of different image results and let devs use them | 00:36 |
mordred | yah | 00:36 |
jroll | yeah | 00:36 |
clarkb | docker doesnt do that... | 00:36 |
jroll | so | 00:36 |
mordred | we can't use docker | 00:36 |
clarkb | you are confusing two things | 00:36 |
mordred | I'm pointing out good ui | 00:36 |
jroll | cat Dockerfile | 00:36 |
jroll | from infra/gate | 00:37 |
jroll | apt-get install latest-lib-virt | 00:37 |
jroll | done | 00:37 |
clarkb | jroll what builds that package? | 00:37 |
mordred | jroll: we're talking about where latest-libvirt comes from | 00:37 |
jroll | (simplified but you get the idea) | 00:37 |
mordred | that's the probelm at hand | 00:37 |
clarkb | jroll it does not fix the problem | 00:37 |
clarkb | it makes other things nice | 00:37 |
jroll | ah, make a dockerfile that builds it with infra/gate as the base | 00:37 |
*** dims_ is now known as dims | 00:37 | |
*** bdpayne has quit IRC | 00:37 | |
clarkb | that doesnt fix it either | 00:37 |
clarkb | because that isnt dev friendly | 00:38 |
mordred | we still need to encode the build instructions | 00:38 |
mordred | in a modular fashion | 00:38 |
jroll | right | 00:38 |
mordred | the other thing docker doesn't do is it's not composible | 00:38 |
mordred | or, rather, not building block style | 00:38 |
mordred | and that's by design | 00:38 |
*** bhuvan_ has quit IRC | 00:38 | |
mordred | and I think they're smart to be specific about that | 00:38 |
jroll | I don't have an idea immediately on that... but regardless can I get an infra/gate docker image? :P | 00:38 |
clarkb | so this is why I am resistant to docker | 00:39 |
mordred | jroll: I'm working on it ... but don't hold your breath | 00:39 |
clarkb | everyone pretends it is a fountain of youth | 00:39 |
jroll | heh :) | 00:39 |
clarkb | its great it wint make you youbger please stop hand waving | 00:39 |
mordred | clarkb: I don't - I think ti's EXCEPTIONALLY good at the one thing it dose | 00:39 |
clarkb | this is why I ignores the ml thread | 00:39 |
mordred | which is easy redistribution of fully put together single purpose environments | 00:40 |
mordred | it does NOT help putting them together | 00:40 |
mordred | and it does NOT help packaging software | 00:40 |
clarkb | that dont need iscsi or a host of other things | 00:40 |
mordred | yah | 00:40 |
mordred | I think it has a place in our toolchain same as virtualenv | 00:40 |
mordred | but it doesn't solve this | 00:40 |
clarkb | jroll ^ is one reason no infra docker | 00:40 |
mordred | clarkb: I strongly want to repalce virtualenv with docker | 00:41 |
clarkb | jroll docker doesnt actually work for the majority of our tests | 00:41 |
mordred | because they do the exact same thing, except docker does it better | 00:41 |
mordred | for all of our current virtualenv usecases | 00:41 |
mordred | it does not solve a bunch of stuff, but neither does virtualenv | 00:41 |
cody-somerville | <mordred> the other thing docker doesn't do is it's not composible | 00:41 |
jroll | clarkb: iscsi etc? | 00:41 |
clarkb | yes it is a decent standin for virtualenv | 00:42 |
clarkb | jroll yup | 00:42 |
cody-somerville | mordred: From the way timrc was talking the other day, he made it sound like docker did allow that | 00:42 |
mordred | cody-somerville: you can't have a docker unit that defines libvirt, and another that defines nova-compute | 00:42 |
mordred | cody-somerville: and the combine them into a third thing | 00:42 |
jroll | clarkb: why can't it do iscsi? | 00:42 |
mordred | you can have a base image. you can have an image that consumes that and adds libvirt, you can have an image that consumes that and adds nova-compute | 00:42 |
clarkb | iscsi isnt namespaced | 00:42 |
jroll | mmmm | 00:42 |
clarkb | so iscsid dies | 00:42 |
mordred | jroll: like, it's a fundametnal container issue | 00:43 |
jroll | yeah | 00:43 |
mordred | or, rather, it's an iscsi issue | 00:43 |
mordred | but it manifests fundamentally in containers | 00:43 |
jroll | right | 00:43 |
mordred | as I understand it, the folks who looked in to fixing iscsi ran away screaming | 00:43 |
* timrc blinks | 00:43 | |
cody-somerville | I'm sure someone from redhat will fix it sooner or later | 00:43 |
adam_g | same with lvm | 00:43 |
jroll | mordred: I'm shocked | 00:43 |
mordred | cody-somerville: iscsi? or docker composability? | 00:44 |
mordred | adam_g: ++ | 00:44 |
cody-somerville | iscsi in docker | 00:44 |
mordred | cody-somerville: yeah. igiven their recent forray into docker, I'm expecting them to bite the bullet | 00:44 |
mordred | but until they do, we cannot run devstack in it fully | 00:44 |
cody-somerville | and was just going to say, probably the latter too | 00:44 |
mordred | which means it's not a complete solution for any of our current problems or perceived problems | 00:45 |
*** r1chardj0n3s is now known as r1chardj0n3s_afk | 00:46 | |
*** homeless has joined #openstack-infra | 00:46 | |
krotscheck | Can someone hop onto storyboard.o.o for me and see why the client tarball isn’t coming down? | 00:46 |
* krotscheck is going to go check the tarball. | 00:46 | |
mordred | krotscheck: on it | 00:48 |
krotscheck | Thanks | 00:48 |
krotscheck | It’s the right tarball :/ | 00:48 |
timrc | mordred, Maybe to do as you describe you would have to export each container to tarball, combine, and then re-import into Docker... that doesn't sound pleasant though | 00:49 |
*** rhsu has quit IRC | 00:49 | |
openstackgerrit | Ian Wienand proposed a change to openstack-infra/config: Remove old stats from whisper https://review.openstack.org/113733 | 00:50 |
*** bdpayne has joined #openstack-infra | 00:50 | |
*** markmcclain has quit IRC | 00:50 | |
dims | timrc: reminds me about this article on optimizing docker images on my read queue - http://www.centurylinklabs.com/optimizing-docker-images/ | 00:53 |
jeblair | jhesketh: can you clarify why you left a -1 on https://review.openstack.org/#/c/112411/ ? | 00:53 |
jeblair | jhesketh: the only comments i see from you on that change are about whitespace, which i'm assuming you left as "+0" comments (eg, fix if updating a patchset) | 00:53 |
krotscheck | mordred: I think I may have to add a -N flag to puppi:netinstall | 00:54 |
jeblair | jhesketh: you did leave a comment about the parent change, but i'm interpreting that as "let's continue further in this direction", which i'm happy to do, but since i don't think it needs to happen before this change i'd like to avoid depending on it | 00:54 |
jhesketh | jeblair: the -1 is probably more for the parent patchset, but because of those nits I put it on that patch too | 00:54 |
jhesketh | jeblair: I'm also curious how pep8 didn't catch those long lines | 00:55 |
jeblair | clarkb, mordred, fungi: can you also review https://review.openstack.org/#/c/112411/ ? it does address two ongoing problems we have, so i think merging it soon could help a lot (it may eventually remove the need for the freshness check) | 00:56 |
jeblair | jhesketh: i believe the addition of "select=H231" has completely broken pep8 for zuul. ie, it now _only_ checks H231. | 00:56 |
*** r1chardj0n3s_afk is now known as r1chardj0n3s | 00:57 | |
clarkb | arg | 00:57 |
jeblair | jhesketh: i think the solution to that is to, in a separate patch, fix the config and clean things up. i'm not interested in holding any legitimate changes up based on pep8 issues. | 00:57 |
jhesketh | sure, that makes sense | 00:58 |
jeblair | jhesketh: so regarding the parent -- do you think we can build on what's there and keep going, or do we need to revise it? based on what you wrote, i think we're probably beginning to imagine the same end state here | 00:59 |
jhesketh | yeah, I was just parsing your comments | 01:00 |
jhesketh | we probably could do it in a later change if we needed to get these in more urgently | 01:00 |
timrc | dims, Indeed. I think the problem with flattening images is the losing of meta information due to the export / import. I think flattening would be a nice optimization. I haven't encountered a situation personally w/ Docker where composition solved a problem for me that couldn't be solved with inheritance. | 01:01 |
jhesketh | jeblair: in regards to your source that isn't a trigger question... do you think it'd ever be possible to have a pure git source... where you have an upstream git and a forked repo to pull from | 01:01 |
jhesketh | I guess the trigger would need to know about the forks... | 01:01 |
jeblair | jhesketh: i'd say it's not critical, but high. one of the features lets us fix changes that aren't getting pulled into the gate; and the other one lets us potentially reduce or eliminate freshness checks | 01:02 |
jeblair | jhesketh: i think you could have a pure git source, particularly with a timer | 01:03 |
jeblair | jhesketh: (eg, traditional ci on specific branches/repos) | 01:03 |
mordred | krotscheck: I do not see any particular message about the file in the puppet run on storyboard. | 01:03 |
jhesketh | right | 01:03 |
krotscheck | mordred: Yeah, I think it’s the lack of a -N flag for wget… will test on a VM. | 01:03 |
mordred | krotscheck: cool | 01:04 |
jeblair | jhesketh: but i don't think a pure-git source would have a trigger itself | 01:04 |
jhesketh | jeblair: perhaps we should separate out sources/triggers completely in this change and then I could tackle separating out gerrit in another change | 01:04 |
jeblair | jhesketh: i think you'd need github or gerrit, etc for that | 01:04 |
jhesketh | (ie allowing multiple gerrits) | 01:04 |
timrc | mordred, Have you run into issues with, say, trying to get mysql running inside a Docker container :)? | 01:05 |
jeblair | jhesketh: i guess the question is -- do they need to be separate, or do they just need to implement the right interface to be used in the right way? | 01:05 |
clarkb | timrc issues with /etc/hosts :) | 01:05 |
clarkb | er resolv.conf | 01:05 |
mordred | timrc: I have not - but I havent' spent an awful lot of time poking at it. | 01:05 |
mordred | timrc: as clarkb says - the main issues we hit was /etc/hosts and /etc/resolv.conf | 01:05 |
timrc | clarkb, UGH | 01:06 |
timrc | clarkb, I had to use a very ugly hack to work around that | 01:06 |
jhesketh | jeblair: I feel like it makes a better distinction... it allows the timer trigger to queue changes from a gerrit source, for example | 01:06 |
jhesketh | and also avoids having triggers raise not implemented exceptions on things like canMerge | 01:07 |
jeblair | jhesketh: i think since you would want to use the same account/server settings for the source, trigger, and reporter with gerrit, i'm not sure you'd need three different objects for that | 01:07 |
jeblair | jhesketh: did you see my suggestion of treating the union of them as a 'connector' in the config file? | 01:07 |
*** arnaud has joined #openstack-infra | 01:08 | |
jhesketh | nope, where was that? | 01:08 |
timrc | iptables issues too that rely on the presence of /lib/modules/`uname -r`/modules.dep | 01:08 |
jeblair | jhesketh: https://review.openstack.org/#/c/97391/3/doc/source/reporters.rst | 01:08 |
jeblair | jhesketh: i think if you had a class for each connector driver (eg, gerrit, timer, zull, github, smtp), you could instantiate one instance of that class for each definition in the config file. then the layout could specify that instance be used in any one of the three roles | 01:09 |
*** oanufriev has quit IRC | 01:09 | |
jeblair | jhesketh: to avoid the exception on canMerge problem, we could have them implement 3 interfaces, and have the config parser check that the driver supported the correct interface | 01:10 |
jeblair | so you'd find that early | 01:10 |
*** nosnos has joined #openstack-infra | 01:10 | |
*** yamahata has joined #openstack-infra | 01:10 | |
mordred | phschwartz: around? | 01:10 |
openstackgerrit | Ian Wienand proposed a change to openstack-infra/config: Remove old stats from whisper https://review.openstack.org/113733 | 01:10 |
mordred | phschwartz: you should be involved in this conversatino ^^ | 01:11 |
phschwartz | I am | 01:11 |
jhesketh | jeblair: oh nice, I was just about to try and flesh out an idea that looks just like the connector stuff (in irc) | 01:11 |
mordred | jeblair: phschwartz was earlier musing to me about drives and opportunistic loading based on config file | 01:11 |
*** shashankhegde has quit IRC | 01:11 | |
mordred | phschwartz: you may not want to wait until next monday to post what you've got - even if it's ugly, I think you should push up your patches if you've got some stuff | 01:11 |
phschwartz | mordred: I talked with jeblair about it and he rather have it be directly in the code | 01:12 |
mordred | k. awesome | 01:12 |
jhesketh | jeblair: so would the driver implement the features of sources/triggers/reporters? | 01:12 |
jeblair | yeah, happy to have github driver be in-tree (and we'll use it! :) | 01:12 |
mordred | phschwartz: mainly - it seems like your'e hacking in the same space | 01:12 |
*** oanufriev has joined #openstack-infra | 01:12 | |
jeblair | jhesketh: that's what i'm thinking | 01:12 |
clarkb | mordred is libvirt on github? | 01:13 |
clarkb | maybe we can just test libvirt that way | 01:13 |
clarkb | and keep life easy for devs | 01:13 |
openstackgerrit | Ian Wienand proposed a change to openstack-infra/config: Remove old stats from whisper https://review.openstack.org/113733 | 01:13 |
mordred | clarkb: ++ | 01:13 |
*** otherwiseguy has joined #openstack-infra | 01:13 | |
jhesketh | jeblair: how would that look for something like the timer... we'd have to have a connector that uses the timer driver and then only lists that under triggers? | 01:15 |
phschwartz | mordred: jeblair: I will try to get the patch out tomorrow I I can fit it in. | 01:16 |
phschwartz | *if | 01:16 |
jeblair | jhesketh: good point, and same for the newly proposed 'zuul' -- they have no external connectivity that needs defining. perhaps they just get registered as internal singletons? | 01:17 |
clarkb | jeblair is review of that change tomorrow ok? or are you wanting it tonight? | 01:17 |
clarkb | I need to afk now for dinner will check back after | 01:18 |
jeblair | clarkb: tomorrow is fine :) | 01:18 |
jhesketh | jeblair: maybe, although I feel like the drivers will start to look funny if some are just loaded for you etc | 01:18 |
openstackgerrit | Oscar Romero (homeless) proposed a change to openstack-infra/jeepyb: Fix close_pull_requests from jeepyb https://review.openstack.org/113598 | 01:18 |
*** SumitNaiksatam has quit IRC | 01:19 | |
jhesketh | jeblair: what if we had a large degree of separation and had drivers+sources+triggers+reporters on their own, then in layout.yaml you specify the driver by connector name under each gerrit trigger or source etc | 01:20 |
jeblair | jhesketh: https://etherpad.openstack.org/p/cviAgfbKV6 | 01:21 |
jeblair | jhesketh: maybe you could fill that example out | 01:21 |
clarkb | how is driver different than trigger? | 01:21 |
clarkb | cant help myself discussion so interesting | 01:22 |
jeblair | clarkb: trigger is something that causes a zuul enqueue, reporter is something that zuul does after completion, source is where it gets the change from | 01:24 |
jeblair | clarkb: so a driver, or connector, or something, is about implementing something that supports those things with the right abstraction | 01:24 |
jhesketh | jeblair: filled out, but I'm not a huge fan of it... | 01:24 |
*** trinaths has joined #openstack-infra | 01:25 | |
jeblair | jhesketh: yeah, i feel like there should be only one object representing the openstack-gerrit connection, because it's associated with an ssh connection that can be used for all three functions | 01:25 |
jeblair | jhesketh: (well, actually 2 of the three functions, and the third needs a separate ssh connection, but that's a detail, and the concept still holds) | 01:26 |
clarkb | ya I think that is what I was getting at | 01:26 |
*** melwitt has left #openstack-infra | 01:26 | |
clarkb | the line there seemed too fuzzy | 01:26 |
jhesketh | jeblair: Sure, so I guess in this example that object is passed around by defining the connector | 01:27 |
jeblair | jhesketh: ok, i see | 01:27 |
jeblair | jhesketh: well, it's dinnertime here; let's bang on this some more later. thanks :) | 01:27 |
jhesketh | okay sounds good :-) | 01:28 |
lifeless | Warning: Variable access via 'servers_real' is deprecated. Use '@servers_real' instead. template[/etc/puppet/modules/ntp/templates/ntp.conf.el.erb]:25 | 01:31 |
Alex_Gaynor | face | 01:34 |
Alex_Gaynor | whoops, wrong window | 01:34 |
asselin__ | has anyone seen an issue w/ nodepool getting a floating ip. nova error is: Caught error: Cannot load 'instance' in the base class | 01:34 |
asselin__ | NotImplementedError: Cannot load 'instance' in the base class | 01:34 |
*** andreaf has quit IRC | 01:35 | |
imcsk8 | hello, 3 hours ago i approved this patch: https://review.openstack.org/#/c/109997/ and nothing has happened, i checked here http://status.openstack.org/zuul/ but couldnt find the review id. can someone give me a hand please? | 01:35 |
*** mmaglana has quit IRC | 01:35 | |
*** andreaf has joined #openstack-infra | 01:35 | |
openstackgerrit | A change was merged to openstack/requirements: Updated version requirement for websockify https://review.openstack.org/102215 | 01:36 |
mordred | lifeless: that does look like a warning | 01:37 |
mordred | imcsk8: there was a zuul restart a few hours ago because of $evil | 01:38 |
mordred | imcsk8: you may have to re-approve it | 01:38 |
lifeless | mordred: so how about that review ;) | 01:38 |
mordred | lifeless: your code is too dense - can you go back and write it in very small words? :) | 01:39 |
lifeless | mordred: uhm | 01:39 |
imcsk8 | mordred: thanks! i'll do that | 01:40 |
*** andreaf has quit IRC | 01:40 | |
*** jyuso has joined #openstack-infra | 01:40 | |
*** gokrokve has joined #openstack-infra | 01:40 | |
mordred | lifeless: just kidding | 01:41 |
*** andreaf has joined #openstack-infra | 01:41 | |
mordred | lifeless: it's looked generally good each time I've looked at it - but I haven't had the time to dive all the way in, which I cannot do with a superficial read | 01:41 |
*** [1]trinaths has joined #openstack-infra | 01:41 | |
lifeless | mordred: so, is there a third person with +A on pbr - or is it blocked on you? | 01:43 |
lifeless | mordred: if its blocked on you, I'd -really- like this in before you go burn | 01:43 |
*** Ryan_Lane has quit IRC | 01:43 | |
lifeless | erm fourth person with +A | 01:43 |
*** oanufriev has quit IRC | 01:44 | |
*** andreaf has quit IRC | 01:44 | |
*** andreaf_ has joined #openstack-infra | 01:44 | |
*** gokrokve_ has quit IRC | 01:44 | |
*** trinaths has quit IRC | 01:44 | |
*** [1]trinaths is now known as trinaths | 01:44 | |
*** oanufriev has joined #openstack-infra | 01:45 | |
*** andreaf_ has quit IRC | 01:45 | |
imcsk8 | mordred: i changed che comment to 0 and then +1 and +2 is that ok? https://review.openstack.org/#/c/109997/ | 01:45 |
*** gokrokve has quit IRC | 01:45 | |
*** andreaf_ has joined #openstack-infra | 01:45 | |
*** andreaf_ has quit IRC | 01:46 | |
*** trinaths has quit IRC | 01:47 | |
*** daya_k has joined #openstack-infra | 01:48 | |
*** trinaths has joined #openstack-infra | 01:48 | |
*** otter768 has joined #openstack-infra | 01:49 | |
*** mbacchi has quit IRC | 01:51 | |
*** trinaths has quit IRC | 01:52 | |
mordred | imcsk8: yup | 01:54 |
mordred | lifeless: there's tons of people with +A | 01:54 |
mordred | lifeless: all of infra core has +A on it | 01:54 |
mordred | lifeless: and all of oslo-core | 01:54 |
*** bdpayne has quit IRC | 01:54 | |
*** crc32 has quit IRC | 01:55 | |
*** yaguang has joined #openstack-infra | 01:55 | |
lifeless | mordred: ok | 01:55 |
lifeless | mordred: I will nag more folk then :) | 01:55 |
mordred | lifeless: reading now | 01:55 |
lifeless | mordred: the haveged patch failed jenkins I believe :( | 01:55 |
mordred | lifeless: btw - I'm going to kill you for test_equality and test_ordering | 01:56 |
mordred | lifeless: I get it - but dude brain melt | 01:56 |
*** weshay_afk has quit IRC | 01:56 | |
homeless | hi guys when you have time can you review this one, https://review.openstack.org/#/c/113598/ | 01:58 |
*** Krast has joined #openstack-infra | 02:00 | |
*** oanufriev has quit IRC | 02:01 | |
*** ramashri has quit IRC | 02:03 | |
*** oanufriev has joined #openstack-infra | 02:03 | |
*** dims has quit IRC | 02:04 | |
*** amotoki has joined #openstack-infra | 02:04 | |
*** dims has joined #openstack-infra | 02:05 | |
*** dims is now known as Guest48695 | 02:05 | |
mordred | StevenK, lifeless: in the mean time - you wanna check out https://review.openstack.org/#/c/113624/ for me - it's launchpad related ... | 02:05 |
*** salv-orlando has quit IRC | 02:06 | |
*** Sukhdev has joined #openstack-infra | 02:07 | |
imcsk8 | mordred: regarding https://review.openstack.org/#/c/109997/, still does not appear on http://status.openstack.org/zuul/ :( | 02:08 |
lifeless | mordred: sure,once I finish this latest mandatory training course for HP | 02:09 |
*** Guest48695 has quit IRC | 02:09 | |
*** amotoki has quit IRC | 02:09 | |
mordred | lifeless: ok. +2 from me | 02:11 |
lifeless | mordred: done a quick glance at the script | 02:12 |
lifeless | mordred: its going to be horrendously slow as-written, hint for fixing showed | 02:12 |
*** med_ has joined #openstack-infra | 02:13 | |
*** med_ has quit IRC | 02:13 | |
*** med_ has joined #openstack-infra | 02:13 | |
*** amcrn has quit IRC | 02:13 | |
openstackgerrit | Monty Taylor proposed a change to openstack-infra/storyboard: Add loader subcommand to sync from launchpad https://review.openstack.org/113624 | 02:15 |
mordred | lifeless: awesome. thanks! it IS horribly slow | 02:17 |
lifeless | mordred: every object ref in the restful API ORM has a matching id attribute | 02:18 |
lifeless | mordred: you can cache the objects e.g. people = {} # id -> person object | 02:18 |
lifeless | mordred: and avoid traversing the .owner etc attributes at all (except on cache misses when you populate the cache) | 02:18 |
mordred | lifeless: awesome | 02:19 |
mordred | lifeless: so instead of message.owner.name, I should grab message.ownerID ? | 02:19 |
*** mmaglana has joined #openstack-infra | 02:20 | |
arosen | mordred: clarkb jogo jeblair this should solve your floatingip leaking issue in nodepool if you're backed by neutron: https://review.openstack.org/113754 | 02:22 |
lifeless | mordred: yeah, and then people[ownerID].name will be entirely local and fast | 02:22 |
mordred | message.ownerID gives me nothing | 02:22 |
arosen | I think we can probably back port this as well and hopefully hp can deploy it in their cloud. | 02:22 |
lifeless | mordred: it might be owner_id or something, its been a couple years :> | 02:22 |
lifeless | mordred: dir(message) should list it | 02:23 |
*** Krast has quit IRC | 02:23 | |
mordred | arosen: awesome | 02:24 |
mordred | lifeless: there are no id anythings ... maybe they removed it | 02:24 |
openstackgerrit | A change was merged to openstack-infra/config: Make sure we actually run haveged on our slaves https://review.openstack.org/113401 | 02:25 |
mordred | lifeless: ^^ | 02:26 |
*** trinaths has joined #openstack-infra | 02:26 | |
mordred | lifeless: that will not take effect until tomorrow when we re-build the images | 02:26 |
StevenK | lifeless: Those are API Entry() responses, I don't think id's are exported | 02:26 |
fungi | https://review.openstack.org/#/c/112411/ ? it does address two | 02:26 |
*** [1]trinaths has joined #openstack-infra | 02:27 | |
mordred | jeblair, jhesketh, phschwartz: just for the record, I'm excited about 112411 and then our subsequent github webhook overlords | 02:27 |
*** trinaths has quit IRC | 02:30 | |
*** [1]trinaths is now known as trinaths | 02:30 | |
fungi | er, mispaste, sorry all | 02:31 |
cody-somerville | mordred: I can also show you how to safely do threaded launchpadlib operations. | 02:32 |
openstackgerrit | Aaron Rosen proposed a change to openstack-infra/nodepool: Fix potential floating ip leakage issue https://review.openstack.org/113659 | 02:33 |
*** homeless has quit IRC | 02:33 | |
*** arnaud has quit IRC | 02:34 | |
*** trinaths has quit IRC | 02:34 | |
cody-somerville | :O | 02:34 |
cody-somerville | I knew it. | 02:34 |
cody-somerville | jesusaurus: We should watch https://review.openstack.org/113659 and merge it down once it gets accepted. | 02:35 |
*** zz_dimtruck is now known as dimtruck | 02:36 | |
*** mriedem has quit IRC | 02:36 | |
jesusaurus | cody-somerville: yep | 02:37 |
*** arnaud has joined #openstack-infra | 02:38 | |
*** Krast has joined #openstack-infra | 02:42 | |
*** changbl has joined #openstack-infra | 02:45 | |
*** pcm_ has quit IRC | 02:49 | |
*** r1chardj0n3s is now known as r1chardj0n3s_afk | 02:53 | |
*** armax has quit IRC | 02:54 | |
*** adalbas has quit IRC | 02:55 | |
*** shashankhegde has joined #openstack-infra | 03:00 | |
*** aysyd has joined #openstack-infra | 03:02 | |
*** Sukhdev_ has joined #openstack-infra | 03:04 | |
*** Sukhdev has quit IRC | 03:07 | |
*** Sukhdev_ has quit IRC | 03:08 | |
*** rainya has joined #openstack-infra | 03:11 | |
clarkb | I dont think that change helps anything | 03:14 |
clarkb | the nova side leak is the same as when we do it | 03:15 |
*** pballand has quit IRC | 03:16 | |
openstackgerrit | Gregory Haynes proposed a change to openstack-infra/config: Add tripleo to specs.o.o https://review.openstack.org/111588 | 03:20 |
*** r1chardj0n3s_afk is now known as r1chardj0n3s | 03:27 | |
jogo | arosen: awesome, once that passes jenkins that will be on the top of my review list | 03:29 |
*** AlexF has joined #openstack-infra | 03:30 | |
*** shardy has quit IRC | 03:31 | |
*** shashankhegde has quit IRC | 03:31 | |
*** nosnos has quit IRC | 03:35 | |
*** kashyap has quit IRC | 03:39 | |
*** marun has quit IRC | 03:43 | |
lifeless | dhellmann: (for when you return - review 94108 typo fixed) | 03:49 |
*** praneshp has quit IRC | 03:55 | |
*** marun has joined #openstack-infra | 03:56 | |
*** otter768 has quit IRC | 03:57 | |
*** otter768 has joined #openstack-infra | 04:01 | |
*** lttrl has quit IRC | 04:01 | |
*** arnaud has quit IRC | 04:02 | |
*** jroll|dupe has joined #openstack-infra | 04:03 | |
mikal | How do I reverify a change these days? | 04:04 |
mikal | Reapproving didn't work like I expected | 04:04 |
*** arnaud has joined #openstack-infra | 04:05 | |
*** jroll has quit IRC | 04:05 | |
*** jroll|dupe is now known as jroll | 04:05 | |
*** SumitNaiksatam has joined #openstack-infra | 04:06 | |
*** flaviof is now known as flaviof_zzz | 04:08 | |
*** arnaud has quit IRC | 04:08 | |
clarkb | mikal: `recheck` | 04:08 |
mikal | recheck will reverify now? | 04:09 |
clarkb | yup. the clean check setup has essentially merged the two | 04:09 |
*** garyh has quit IRC | 04:09 | |
*** trinaths has joined #openstack-infra | 04:10 | |
mikal | Huh, ok | 04:10 |
clarkb | reverify will work too | 04:10 |
mikal | Fair enough | 04:10 |
clarkb | they are now synonyms | 04:10 |
*** andreaf has joined #openstack-infra | 04:12 | |
*** AlexF has quit IRC | 04:12 | |
clarkb | mikal I think it works if you do +0 then +W | 04:15 |
*** otter768 has quit IRC | 04:15 | |
clarkb | it has to do with how gerrit outputs events | 04:15 |
*** andreaf has quit IRC | 04:15 | |
*** arnaud has joined #openstack-infra | 04:18 | |
*** rainya has quit IRC | 04:25 | |
*** aysyd has quit IRC | 04:29 | |
*** david-lyle has joined #openstack-infra | 04:31 | |
*** nosnos has joined #openstack-infra | 04:32 | |
*** praneshp_ has joined #openstack-infra | 04:32 | |
*** mrmartin has joined #openstack-infra | 04:35 | |
*** gokrokve has joined #openstack-infra | 04:36 | |
*** ildikov has quit IRC | 04:36 | |
*** timrc is now known as timrc-afk | 04:36 | |
*** marun has quit IRC | 04:39 | |
*** garyh has joined #openstack-infra | 04:40 | |
*** rhsu has joined #openstack-infra | 04:41 | |
openstackgerrit | Steve Baker proposed a change to openstack-infra/config: Define gate-heat-dsvm-functional job https://review.openstack.org/111087 | 04:41 |
rhsu | fungi: hello thar | 04:45 |
*** dimtruck is now known as zz_dimtruck | 04:46 | |
rhsu | anteaya: ping | 04:46 |
*** chandankumar has joined #openstack-infra | 04:50 | |
*** Sukhdev has joined #openstack-infra | 04:51 | |
*** e0ne has joined #openstack-infra | 04:55 | |
*** e0ne has quit IRC | 04:57 | |
*** jgrimm has quit IRC | 05:01 | |
openstackgerrit | patrick-crews proposed a change to openstack-infra/elastic-recheck: Add crm114 logstash queries to uncategorized page https://review.openstack.org/113771 | 05:02 |
*** chandankumar has quit IRC | 05:02 | |
pcrews | clarkb: ^ your crm114 wish :) | 05:03 |
*** mrmartin has quit IRC | 05:05 | |
*** chandankumar has joined #openstack-infra | 05:12 | |
*** gema_ has joined #openstack-infra | 05:13 | |
*** yfried__ has joined #openstack-infra | 05:13 | |
*** ominakov has quit IRC | 05:13 | |
*** gema has quit IRC | 05:14 | |
*** marun has joined #openstack-infra | 05:17 | |
*** mmaglana has quit IRC | 05:20 | |
openstackgerrit | Johnson Dantis proposed a change to openstack-infra/elastic-recheck: recheck bug 1270710 https://review.openstack.org/113774 | 05:21 |
uvirtbot | Launchpad bug 1270710 in openstack-ci "sporadic pip timeouts during download" [Medium,Incomplete] https://launchpad.net/bugs/1270710 | 05:21 |
*** gema has joined #openstack-infra | 05:21 | |
*** marun has quit IRC | 05:21 | |
openstackgerrit | patrick-crews proposed a change to openstack-infra/elastic-recheck: Add crm114 logstash queries to uncategorized page https://review.openstack.org/113771 | 05:22 |
*** marun has joined #openstack-infra | 05:22 | |
*** otherwiseguy has quit IRC | 05:23 | |
*** gema_ has quit IRC | 05:24 | |
*** tonyb has joined #openstack-infra | 05:25 | |
*** rcarrill` has joined #openstack-infra | 05:25 | |
*** flwang_ has joined #openstack-infra | 05:25 | |
tonyb | So I'm trying to sign the ICLA. After I click 'submit' I get an error screen saying it couldn't save my information. Any idea what I've done wrong? | 05:26 |
clarkb | pcrews \o/ | 05:27 |
clarkb | tonyb have you joined the openstack foundation? | 05:27 |
*** rcarrillocruz has quit IRC | 05:27 | |
tonyb | clarkb: no. I guess that answers that ;P | 05:27 |
clarkb | tonyb if not you need to do that. also ensurs your primary email address there matches the primary email address in gerrit | 05:28 |
*** SumitNaiksatam has quit IRC | 05:28 | |
clarkb | tonyb https://wiki.openstack.org/wiki/How_To_Contribute should cover it | 05:28 |
openstackgerrit | patrick-crews proposed a change to openstack-infra/elastic-recheck: Add crm114 logstash queries to uncategorized page https://review.openstack.org/113771 | 05:29 |
*** tjones has joined #openstack-infra | 05:29 | |
tonyb | clarkb: Thanks. I just signed in to gerrit and started following the bounching ball. | 05:29 |
*** marun has quit IRC | 05:29 | |
tonyb | clarkb: Thanks for the help. | 05:29 |
*** k4n0 has joined #openstack-infra | 05:30 | |
*** emagana has joined #openstack-infra | 05:30 | |
clarkb | no problem | 05:31 |
rhsu | clarkb: sorry to bother you so late, but was wondering if you saw salv-orlando's message on the mailing list about VMware CI | 05:34 |
rhsu | clarkb: we have taken corrective actions and things are back to normal | 05:34 |
clarkb | rhsu I did but am without ssh keys to enable the account. can do it in the morning | 05:34 |
rhsu | clarkb: ah ok, thanks so much. i'm extremely sorry about all trouble that was caused.. | 05:35 |
*** SumitNaiksatam has joined #openstack-infra | 05:36 | |
*** mmaglana has joined #openstack-infra | 05:37 | |
*** rlandy has joined #openstack-infra | 05:37 | |
tonyb | clarkb: all fixed now. | 05:38 |
*** r1chardj0n3s is now known as r1chardj0n3s_afk | 05:38 | |
tjones | clarkb: we (vmw) have a pretty big queue stacked up for review (300 now). anyone else up who has the keys? again sorry about the issue - we didn't realize the upstream trigger changed…. | 05:41 |
clarkb | tjones: jhesketh and sergeylukjanov | 05:42 |
tjones | clarkb: thx | 05:43 |
*** gokrokve_ has joined #openstack-infra | 05:44 | |
*** yamamoto has quit IRC | 05:45 | |
*** ildikov has joined #openstack-infra | 05:46 | |
tjones | jhesketh: SergeyLukjanov: minesweeper is back to normal. we were rechecking on "recheck-vmware" which caused it to appear we were spamming due to a change in the upstream recheck trigger. we have changed to vmware-recheck. can you please re-enable minesweeper? | 05:46 |
*** Ryan_Lane has joined #openstack-infra | 05:47 | |
tjones | rhsu: please clarify if i was unclear ^^ | 05:47 |
*** gokrokve has quit IRC | 05:47 | |
clarkb | well wait | 05:47 |
clarkb | that isnt the fix we wanted | 05:47 |
*** pcrews has quit IRC | 05:47 | |
clarkb | the bug here is comment spam on 500+ changes | 05:48 |
clarkb | that triggering jenkins is a side effect that changing the magic string doesnt fully fix | 05:48 |
*** gokrokve_ has quit IRC | 05:48 | |
clarkb | if you need to test against 500+ changes all at once you really should be triggering that directly in the ci system | 05:49 |
rhsu | clarkb: yes, we're going to do it internall from now on. no longer relying on recheck comments. | 05:49 |
rhsu | clarkb: the change to "vmware-recheck" is to conform to the recheck pattern guidelines. this way, we won't be re-triggering upstream Jenkins when someone requests a vmware recheck. | 05:51 |
clarkb | rhsu but that isnt a thibg | 05:52 |
clarkb | there is no guideline. we actually do want things to be retested across the board in most cases | 05:52 |
clarkb | fungi has an explanation on the ml somewhere | 05:52 |
rhsu | clarkb: agreed. the new pattern will still allow for that. we just wanted to make sure when someone explicitly requests a vmware recheck that it wont clash with the upstream jenkins recheck pattern. | 05:54 |
*** AlexF has joined #openstack-infra | 05:56 | |
rhsu | so from now on, developers can still issue "vmware-recheck" to request a recheck but whenever we need to do a mass recheck, we will be doing this using internal means so we don't spam everyone's patches. | 05:56 |
clarkb | it wont clash | 05:56 |
clarkb | we never intended to have complex recheck syntaxes | 05:57 |
clarkb | the idea was we expect all ci to pass together so just rerun them | 05:57 |
clarkb | this isnt an issue in normal day to day operations | 05:58 |
rhsu | understood. so the main issue was in how we do mass recheck then. i guess this should be resolved by just not posting recheck-comments in bulk then? | 05:59 |
clarkb | correct | 05:59 |
clarkb | for two reasons. first jenkins matches recheck* | 06:00 |
clarkb | but also because commenting on an idle change causes an upstream recheck too | 06:00 |
clarkb | I imagine a good chunk of those vhanges would have rechecked regardless of comment content | 06:01 |
*** Sukhdev has quit IRC | 06:01 | |
*** r1chardj0n3s_afk is now known as r1chardj0n3s | 06:02 | |
rhsu | that makes sense. so next time I need to do this, i'll be doing this from the CI. sorry, if i didnt make that clear earlier. | 06:03 |
*** Longgeek has joined #openstack-infra | 06:03 | |
*** ido_ has joined #openstack-infra | 06:03 | |
ido_ | hi guys, can anyonehelp woth a problem on jenkins-job-builder? | 06:03 |
ido_ | hi guys, can anyone help woth a problem on jenkins-job-builder? | 06:04 |
rhsu | i actually didn't need to retrigger via comments today, but we had done this a few times before without issues (but back then it didn't trigger upstream CI) so ended up going with the comments | 06:04 |
rhsu | clarkb: i hope i've given enough proof that these things shouldnt happen again :) | 06:07 |
*** dtantsur|afk is now known as dtantsur | 06:08 | |
ido_ | hi guys, is it the rogt place to get help with jenkins-job-builder? | 06:09 |
ido_ | hi guys, is it the right place to get help with jenkins-job-builder? | 06:09 |
*** HenryG_ has joined #openstack-infra | 06:09 | |
*** Azher has joined #openstack-infra | 06:11 | |
*** HenryG has quit IRC | 06:12 | |
*** Alexei_987 has quit IRC | 06:14 | |
*** yamamoto has joined #openstack-infra | 06:14 | |
*** loki184 has joined #openstack-infra | 06:16 | |
jhesketh | ido_: Not a bad place to try. Leave your question here and if anybody knows the answer they'll chime in | 06:19 |
ido_ | well, basically, | 06:22 |
*** tonyb has left #openstack-infra | 06:22 | |
ido_ | well, basically, jenkins-jobs --conf /path/to/jenkins_jobs.ini -l debug test /path/to/foo.yaml ** does nothing | 06:24 |
ido_ | it prints the following debugging output: DEBUG:root:Reading config from /var/lib/jenkins/jobs/bootstrap/workspace/jenkins-ci/jenkins_jobs.ini DEBUG:root:Config: <ConfigParser.ConfigParser instance at 0x133def0> INFO:root:Updating jobs in /var/lib/jenkins/jobs/bootstrap/workspace/jenkins-ci/config/example.yaml ([]) DEBUG:jenkins_jobs.builder:Parsing YAML file /var/lib/jenkins/jobs/bootstrap/workspace/jenkins-ci/config/example.y | 06:24 |
ido_ | but outputs nada to stdout | 06:24 |
ido_ | the ([]) print is what disturbing me | 06:25 |
rhsu | jhesketh: while you're on, can i trouble you to re-enable our account on gerrit? | 06:27 |
jhesketh | rhsu: Sorry I don't have those rights (afaik) | 06:28 |
jhesketh | you need somebody on the root team | 06:28 |
rhsu | jhesketh: ah ok, i know clarkb said he would do it in the morning | 06:29 |
openstackgerrit | Johnson Dantis proposed a change to openstack-infra/elastic-recheck: Add e-r query for bug 1350237 https://review.openstack.org/113788 | 06:29 |
uvirtbot | Launchpad bug 1350237 in horizon "Networks tab does not show external network configuration" [Wishlist,In progress] https://launchpad.net/bugs/1350237 | 06:29 |
rhsu | jhesketh: i assume everyone else is asleep by now? | 06:30 |
jhesketh | Unless SergeyLukjanov is around, most likely | 06:30 |
rhsu | thanks very much jhesketh, i'll try to check in with him before i'm off for the night | 06:32 |
*** rdopieralski has joined #openstack-infra | 06:33 | |
*** gokrokve has joined #openstack-infra | 06:37 | |
*** zz_dimtruck is now known as dimtruck | 06:37 | |
*** rcarrillocruz has joined #openstack-infra | 06:38 | |
*** rcarrill` has quit IRC | 06:40 | |
*** jcoufal has joined #openstack-infra | 06:40 | |
*** gokrokve has quit IRC | 06:42 | |
*** stevemar has quit IRC | 06:42 | |
*** stevemar has joined #openstack-infra | 06:42 | |
*** yamamoto has quit IRC | 06:44 | |
*** flaper87|afk is now known as flaper87 | 06:44 | |
*** emagana has quit IRC | 06:46 | |
*** che-arne|2 has quit IRC | 06:46 | |
*** emagana has joined #openstack-infra | 06:46 | |
*** dimtruck is now known as zz_dimtruck | 06:47 | |
*** stevemar has quit IRC | 06:47 | |
*** yamamoto has joined #openstack-infra | 06:48 | |
*** arnaud has quit IRC | 06:48 | |
*** emagana has quit IRC | 06:51 | |
*** rhsu has quit IRC | 06:53 | |
*** evgenyf has joined #openstack-infra | 06:54 | |
*** arnaud has joined #openstack-infra | 06:57 | |
*** Ryan_Lane has quit IRC | 06:58 | |
*** zz_dimtruck is now known as dimtruck | 07:00 | |
openstackgerrit | Guido Günther proposed a change to openstack-infra/jenkins-job-builder: git: Support timeout option https://review.openstack.org/111280 | 07:03 |
*** salv-orlando has joined #openstack-infra | 07:04 | |
*** amcrn has joined #openstack-infra | 07:10 | |
*** mmaglana has quit IRC | 07:10 | |
*** dimtruck is now known as zz_dimtruck | 07:12 | |
*** e0ne has joined #openstack-infra | 07:12 | |
*** mrmartin has joined #openstack-infra | 07:14 | |
*** imcsk8 has quit IRC | 07:19 | |
*** imcsk8 has joined #openstack-infra | 07:19 | |
*** trinaths has quit IRC | 07:19 | |
*** ihrachyshka has joined #openstack-infra | 07:20 | |
*** mpaolino has joined #openstack-infra | 07:20 | |
*** mugsie has quit IRC | 07:21 | |
*** ihrachyshka has quit IRC | 07:23 | |
*** Guest35108 has joined #openstack-infra | 07:24 | |
*** sballe has joined #openstack-infra | 07:29 | |
*** david-lyle has quit IRC | 07:29 | |
*** david-lyle has joined #openstack-infra | 07:29 | |
*** mrmartin has quit IRC | 07:29 | |
*** _nadya_ has joined #openstack-infra | 07:32 | |
*** david-lyle has quit IRC | 07:34 | |
*** rwsu has quit IRC | 07:34 | |
*** flwang_ has quit IRC | 07:35 | |
*** gokrokve has joined #openstack-infra | 07:36 | |
ido_ | hi guys, is it the right place to get help with jenkins-job-builder? | 07:37 |
*** shardy has joined #openstack-infra | 07:37 | |
*** gokrokve has quit IRC | 07:37 | |
*** gokrokve has joined #openstack-infra | 07:38 | |
ido_ | reading (debugging) the code I think I understand that a projects file is a must, and without it, no jobs are generated, am I right? | 07:38 |
*** mmaglana has joined #openstack-infra | 07:41 | |
mattoliverau | well time to go and cook dinner, so calling it a night, night all! | 07:41 |
*** hashar has joined #openstack-infra | 07:42 | |
*** gokrokve has quit IRC | 07:43 | |
*** linkid has joined #openstack-infra | 07:43 | |
*** mmaglana_ has joined #openstack-infra | 07:43 | |
*** mmaglan__ has joined #openstack-infra | 07:44 | |
*** yamamoto has quit IRC | 07:45 | |
*** mmaglana has quit IRC | 07:46 | |
*** Longgeek has quit IRC | 07:46 | |
*** Longgeek has joined #openstack-infra | 07:46 | |
*** mmaglana_ has quit IRC | 07:47 | |
*** mmaglan__ has quit IRC | 07:49 | |
*** mika has quit IRC | 07:50 | |
*** Longgeek has quit IRC | 07:51 | |
*** sballe has quit IRC | 07:51 | |
*** rwsu has joined #openstack-infra | 07:52 | |
*** jpich has joined #openstack-infra | 07:53 | |
*** afazekas has joined #openstack-infra | 07:54 | |
*** Longgeek has joined #openstack-infra | 07:54 | |
*** e0ne has quit IRC | 07:55 | |
*** ling-yun has joined #openstack-infra | 07:56 | |
ling-yun | hi stackers. If meeting problem with manual, which irc channel do you use to ask for help? | 07:56 |
*** mugsie has joined #openstack-infra | 07:57 | |
*** _nadya_ has quit IRC | 07:58 | |
*** e0ne has joined #openstack-infra | 07:59 | |
*** skolekonov has joined #openstack-infra | 08:00 | |
*** _nadya_ has joined #openstack-infra | 08:01 | |
openstackgerrit | Flavio Percoco proposed a change to openstack/requirements: Add glance.store to global requirements https://review.openstack.org/113806 | 08:03 |
*** derekh has joined #openstack-infra | 08:04 | |
*** daya_k has quit IRC | 08:05 | |
*** jistr has joined #openstack-infra | 08:08 | |
*** yamamoto has joined #openstack-infra | 08:09 | |
*** Longgeek has quit IRC | 08:10 | |
*** sballe has joined #openstack-infra | 08:11 | |
*** jlibosva has joined #openstack-infra | 08:11 | |
*** arnaud has quit IRC | 08:12 | |
*** cristov_mac has joined #openstack-infra | 08:17 | |
*** cristov_mac has quit IRC | 08:17 | |
*** daya_k has joined #openstack-infra | 08:19 | |
daya_k | server sydney.irc.ibm.com | 08:19 |
*** maxbit has quit IRC | 08:22 | |
*** e0ne has quit IRC | 08:25 | |
*** amuller has joined #openstack-infra | 08:25 | |
*** ihrachyshka has joined #openstack-infra | 08:26 | |
*** leifmadsen has quit IRC | 08:28 | |
*** e0ne has joined #openstack-infra | 08:30 | |
openstackgerrit | Alexis Lee proposed a change to openstack-infra/config: Add firewall traffic on port 27410 https://review.openstack.org/108737 | 08:31 |
*** sballe has quit IRC | 08:32 | |
*** leifmadsen has joined #openstack-infra | 08:33 | |
*** AlexF has quit IRC | 08:36 | |
*** zz_dimtruck is now known as dimtruck | 08:37 | |
*** gokrokve has joined #openstack-infra | 08:37 | |
ihrachyshka | afazekas: hey. do you by chance know why we could miss rpm/yum for 'periodic-{name}-python26{branch-name}' (for havana) even though it's set for 'node: bare-centos6'? | 08:40 |
ihrachyshka | we get this py26 job constantly failing for havana branches. It seems we have some hacks to install libffi-devel there via rpm and yum, and for some reason, console log shows they fail due to 'rpm: command not found' | 08:41 |
*** gokrokve has quit IRC | 08:41 | |
*** dimtruck is now known as zz_dimtruck | 08:46 | |
*** enikanorov has quit IRC | 08:48 | |
*** enikanorov has joined #openstack-infra | 08:48 | |
afazekas | ihrachyshka: is the rpm command in the current PATH ? | 08:49 |
*** rdekel has joined #openstack-infra | 08:50 | |
*** Longgeek has joined #openstack-infra | 08:50 | |
*** p5ntangl_ has quit IRC | 08:52 | |
*** p5ntangle has joined #openstack-infra | 08:53 | |
rdekel | I'm looking for cirrOS information - specifically nc and other communication CMD's. 10x in advance | 08:54 |
ihrachyshka | afazekas: how can I tell that from jenkins logs? | 08:59 |
ihrachyshka | afazekas: this is all I have: http://logs.openstack.org/periodic-stable/periodic-cinder-python26-havana/093cf3d/console.html | 08:59 |
*** ZZelle has joined #openstack-infra | 09:00 | |
afazekas | it is expected to be in the /usr/bin/ which expected to be in the path even without login shell.. | 09:04 |
*** jp_at_hp has joined #openstack-infra | 09:07 | |
*** pblaho has joined #openstack-infra | 09:08 | |
*** sballe has joined #openstack-infra | 09:10 | |
*** pblaho has quit IRC | 09:11 | |
*** pblaho has joined #openstack-infra | 09:11 | |
*** ihrachyshka has quit IRC | 09:11 | |
*** ociuhandu has quit IRC | 09:13 | |
*** Longgeek has quit IRC | 09:15 | |
*** jgallard has joined #openstack-infra | 09:15 | |
*** pelix has joined #openstack-infra | 09:17 | |
*** jedimike has joined #openstack-infra | 09:19 | |
*** sballe has quit IRC | 09:20 | |
*** yamamoto has quit IRC | 09:21 | |
*** sballe has joined #openstack-infra | 09:22 | |
*** sballe_ has joined #openstack-infra | 09:24 | |
*** katyafervent is now known as katyafervent_awa | 09:25 | |
*** sballe has quit IRC | 09:27 | |
*** dmsimardz has joined #openstack-infra | 09:29 | |
*** dmsimardz is now known as dmsimard | 09:30 | |
*** david-lyle has joined #openstack-infra | 09:30 | |
*** _nadya_ has quit IRC | 09:30 | |
*** dmsimard_away has quit IRC | 09:31 | |
*** _nadya_ has joined #openstack-infra | 09:31 | |
*** amuller has quit IRC | 09:32 | |
*** renlt has joined #openstack-infra | 09:32 | |
*** AlexF has joined #openstack-infra | 09:34 | |
openstackgerrit | afazekas proposed a change to openstack-infra/devstack-gate: [DO NOT MERGE] test full tempest run on f20 https://review.openstack.org/112739 | 09:34 |
*** david-lyle has quit IRC | 09:35 | |
*** gokrokve has joined #openstack-infra | 09:37 | |
openstackgerrit | Peter Belanyi proposed a change to openstack-infra/config: Add jshint job for horizon as non-voting https://review.openstack.org/113825 | 09:38 |
*** david-lyle has joined #openstack-infra | 09:38 | |
*** gokrokve has quit IRC | 09:38 | |
*** gokrokve has joined #openstack-infra | 09:39 | |
openstackgerrit | Sergey Lukjanov proposed a change to openstack-infra/config: Use sahara-specs-core for sahara-specs acl https://review.openstack.org/110927 | 09:40 |
*** jp_at_hp has quit IRC | 09:40 | |
*** mika has joined #openstack-infra | 09:41 | |
SergeyLukjanov | jhesketh, rhsu, I'm here now | 09:43 |
*** david-lyle has quit IRC | 09:43 | |
*** gokrokve has quit IRC | 09:43 | |
*** katyafervent_awa is now known as katyafervent | 09:44 | |
*** mugsie has quit IRC | 09:45 | |
*** sballe__ has joined #openstack-infra | 09:46 | |
*** amcrn has quit IRC | 09:47 | |
*** andreaf has joined #openstack-infra | 09:47 | |
*** flwang_ has joined #openstack-infra | 09:48 | |
SergeyLukjanov | anteaya, sorry, it was already 2am in my tz and I was sleeping / or filling dishwasher :) | 09:48 |
*** mugsie has joined #openstack-infra | 09:49 | |
*** sballe_ has quit IRC | 09:50 | |
openstackgerrit | A change was merged to openstack-infra/config: Add specific python-saharaclient acls https://review.openstack.org/105712 | 09:50 |
*** Longgeek has joined #openstack-infra | 09:52 | |
openstackgerrit | Andreas Jaeger proposed a change to openstack/requirements: Rename bash8 to bashate https://review.openstack.org/113080 | 09:52 |
*** flwang_ has quit IRC | 09:52 | |
*** yfried has joined #openstack-infra | 09:53 | |
*** yfried__ has quit IRC | 09:54 | |
*** med_ has quit IRC | 10:00 | |
*** gyee has quit IRC | 10:00 | |
*** bookwar has quit IRC | 10:00 | |
*** jeblair has quit IRC | 10:00 | |
*** SpamapS has quit IRC | 10:00 | |
*** jamespage has quit IRC | 10:00 | |
*** bnemec has quit IRC | 10:00 | |
*** unicell has quit IRC | 10:00 | |
*** zul has quit IRC | 10:00 | |
*** pblaho is now known as pblaho|afk | 10:01 | |
*** AlexF has quit IRC | 10:01 | |
*** zul has joined #openstack-infra | 10:01 | |
*** jgallard has quit IRC | 10:04 | |
*** sballe has joined #openstack-infra | 10:06 | |
*** imeerovi has joined #openstack-infra | 10:07 | |
*** ihrachyshka has joined #openstack-infra | 10:07 | |
*** med_ has joined #openstack-infra | 10:07 | |
*** gyee has joined #openstack-infra | 10:07 | |
*** bookwar has joined #openstack-infra | 10:07 | |
*** jeblair has joined #openstack-infra | 10:07 | |
*** SpamapS has joined #openstack-infra | 10:07 | |
*** jamespage has joined #openstack-infra | 10:07 | |
*** bnemec has joined #openstack-infra | 10:07 | |
*** imeerovi has quit IRC | 10:07 | |
*** imeerovi has joined #openstack-infra | 10:07 | |
*** ihrachyshka has quit IRC | 10:07 | |
*** ihrachyshka has joined #openstack-infra | 10:07 | |
*** sballe__ has quit IRC | 10:07 | |
openstackgerrit | Sergey Lukjanov proposed a change to openstack-infra/config: sahara-* repos aren't part of integrated release https://review.openstack.org/113833 | 10:10 |
*** pcm_ has joined #openstack-infra | 10:13 | |
*** mpaolino has quit IRC | 10:13 | |
*** pcm__ has joined #openstack-infra | 10:15 | |
openstackgerrit | Sergey Lukjanov proposed a change to openstack-infra/config: Create refs permission isn't needed for sahara now https://review.openstack.org/113834 | 10:15 |
*** jyuso has quit IRC | 10:17 | |
*** AJaeger has joined #openstack-infra | 10:18 | |
*** pcm_ has quit IRC | 10:18 | |
*** jcoufal has quit IRC | 10:19 | |
*** sballe_ has joined #openstack-infra | 10:21 | |
openstackgerrit | Sergey Lukjanov proposed a change to openstack-infra/config: Use more strict regex for sahara tags acl https://review.openstack.org/113836 | 10:21 |
AJaeger | SergeyLukjanov, bash8 was renamed to bashate and thus I changed my patch: https://review.openstack.org/113828 | 10:21 |
*** sballe has quit IRC | 10:21 | |
AJaeger | But looking at keystone - the only user of bash8 that I could find - I think we need to do the following: | 10:21 |
AJaeger | Add bashate; merge keystone rename;remove bash8 | 10:21 |
*** hashar has quit IRC | 10:22 | |
AJaeger | Here's the requirements patch https://review.openstack.org/113080 - what do you think? | 10:22 |
*** yamamoto has joined #openstack-infra | 10:22 | |
SergeyLukjanov | AJaeger, already +2'd | 10:23 |
SergeyLukjanov | AJaeger, I've missed that bash8 has been added to it | 10:23 |
AJaeger | I think I need to change it in two patches as above... | 10:23 |
AJaeger | will be back in 30mins... | 10:23 |
*** dtantsur is now known as dtantsur|lunch | 10:23 | |
*** ihrachyshka has quit IRC | 10:23 | |
*** imeerovi has quit IRC | 10:23 | |
*** med_ has quit IRC | 10:23 | |
*** gyee has quit IRC | 10:23 | |
*** bookwar has quit IRC | 10:23 | |
*** jeblair has quit IRC | 10:23 | |
*** SpamapS has quit IRC | 10:23 | |
*** jamespage has quit IRC | 10:23 | |
*** bnemec has quit IRC | 10:23 | |
openstackgerrit | Nikita Konovalov proposed a change to openstack-infra/storyboard: Assocate StoryTag with Story via mapping https://review.openstack.org/113635 | 10:24 |
*** sballe_ has quit IRC | 10:25 | |
*** sballe has joined #openstack-infra | 10:26 | |
*** yamamoto has quit IRC | 10:27 | |
*** ido_ has quit IRC | 10:29 | |
*** imeerovi has joined #openstack-infra | 10:30 | |
*** ihrachyshka has joined #openstack-infra | 10:30 | |
*** med_ has joined #openstack-infra | 10:30 | |
*** gyee has joined #openstack-infra | 10:30 | |
*** bookwar has joined #openstack-infra | 10:30 | |
*** jeblair has joined #openstack-infra | 10:30 | |
*** SpamapS has joined #openstack-infra | 10:30 | |
*** jamespage has joined #openstack-infra | 10:30 | |
*** bnemec has joined #openstack-infra | 10:30 | |
*** imeerovi has quit IRC | 10:30 | |
*** imeerovi has joined #openstack-infra | 10:30 | |
*** ihrachyshka has quit IRC | 10:30 | |
*** ihrachyshka has joined #openstack-infra | 10:30 | |
*** mika has quit IRC | 10:34 | |
*** flaviof_zzz is now known as flaviof | 10:35 | |
*** markmc has joined #openstack-infra | 10:36 | |
*** mika has joined #openstack-infra | 10:36 | |
*** sballe has quit IRC | 10:36 | |
*** sballe has joined #openstack-infra | 10:37 | |
*** gokrokve has joined #openstack-infra | 10:37 | |
*** oanufriev has quit IRC | 10:38 | |
*** yfried has quit IRC | 10:38 | |
openstackgerrit | Flavio Percoco proposed a change to openstack/requirements: Add glance.store to global requirements https://review.openstack.org/113806 | 10:38 |
*** amuller has joined #openstack-infra | 10:38 | |
*** david-lyle has joined #openstack-infra | 10:39 | |
*** Longgeek has quit IRC | 10:41 | |
*** gokrokve has quit IRC | 10:41 | |
*** Guest35108 has quit IRC | 10:42 | |
*** yamahata has quit IRC | 10:42 | |
*** mmaglana has joined #openstack-infra | 10:43 | |
*** david-lyle has quit IRC | 10:43 | |
*** sballe_ has joined #openstack-infra | 10:44 | |
*** sballe has quit IRC | 10:44 | |
*** Azher has quit IRC | 10:47 | |
*** mmaglana has quit IRC | 10:48 | |
*** heyongli has joined #openstack-infra | 10:48 | |
*** heyongli has quit IRC | 10:49 | |
*** oanufriev has joined #openstack-infra | 10:51 | |
openstackgerrit | Dina Belova proposed a change to openstack-infra/config: Make py33 job running only for Ceilometer master branch https://review.openstack.org/113842 | 10:52 |
*** andreykurilin_ has joined #openstack-infra | 10:52 | |
openstackgerrit | Andreas Jaeger proposed a change to openstack/requirements: Rename bash8 to bashate (part 1) https://review.openstack.org/113080 | 10:56 |
openstackgerrit | Andreas Jaeger proposed a change to openstack/requirements: Rename bash8 to bashate (part 1) https://review.openstack.org/113843 | 10:56 |
salv-orlando | jhesketh: sorry to bother you, did you forget to write the reason for the -1 here? https://review.openstack.org/#/c/113562/ | 10:56 |
openstackgerrit | Le Tian Ren proposed a change to openstack-infra/config: Add create access right on "refs/heads/*". https://review.openstack.org/113844 | 10:59 |
openstackgerrit | Andreas Jaeger proposed a change to openstack/requirements: Rename bash8 to bashate (part 2) https://review.openstack.org/113843 | 11:00 |
*** renlt has quit IRC | 11:01 | |
AJaeger | SergeyLukjanov, I split it up, we need to do this in two steps... | 11:02 |
*** dims has joined #openstack-infra | 11:03 | |
jhesketh | salv-orlando: Heh, sorry, I had clicked the wrong vote! | 11:03 |
*** dims is now known as Guest25021 | 11:03 | |
*** pblaho|afk is now known as pblaho | 11:03 | |
*** ociuhandu has joined #openstack-infra | 11:03 | |
*** Guest25021 has quit IRC | 11:04 | |
*** yfried has joined #openstack-infra | 11:05 | |
*** dims_ has joined #openstack-infra | 11:05 | |
*** loki184 has quit IRC | 11:06 | |
*** bookwar has quit IRC | 11:06 | |
*** miqui has joined #openstack-infra | 11:06 | |
*** yaguang has quit IRC | 11:07 | |
*** oanufriev has quit IRC | 11:08 | |
*** p5ntangle has quit IRC | 11:09 | |
salv-orlando | jhesketh: that’s ok… thanks for the +1 | 11:09 |
*** jcoufal has joined #openstack-infra | 11:09 | |
*** derekh has quit IRC | 11:09 | |
*** bookwar has joined #openstack-infra | 11:11 | |
*** yolanda has quit IRC | 11:13 | |
*** [1]evgenyf has joined #openstack-infra | 11:14 | |
*** yfried_ has joined #openstack-infra | 11:16 | |
*** evgenyf has quit IRC | 11:16 | |
*** [1]evgenyf is now known as evgenyf | 11:16 | |
*** yfried has quit IRC | 11:16 | |
openstackgerrit | Davanum Srinivas (dims) proposed a change to openstack-infra/elastic-recheck: Add query for Postgres startup failure https://review.openstack.org/113634 | 11:17 |
*** p5ntangle has joined #openstack-infra | 11:17 | |
*** yfried__ has joined #openstack-infra | 11:18 | |
*** yfried_ has quit IRC | 11:21 | |
*** dims_ has quit IRC | 11:22 | |
*** yamamoto has joined #openstack-infra | 11:22 | |
*** dims_ has joined #openstack-infra | 11:23 | |
*** sballe_ has quit IRC | 11:24 | |
*** Longgeek has joined #openstack-infra | 11:26 | |
*** dims__ has joined #openstack-infra | 11:26 | |
*** yamamoto has quit IRC | 11:26 | |
*** dkranz has quit IRC | 11:27 | |
*** dims_ has quit IRC | 11:27 | |
*** mpaolino has joined #openstack-infra | 11:28 | |
*** yolanda has joined #openstack-infra | 11:29 | |
*** pblaho is now known as pblaho|afk | 11:31 | |
*** andreaf_ has joined #openstack-infra | 11:32 | |
*** andreaf has quit IRC | 11:34 | |
*** dalgaaf_ has joined #openstack-infra | 11:35 | |
*** gokrokve has joined #openstack-infra | 11:37 | |
*** andreaf_ has quit IRC | 11:38 | |
*** pblaho|afk is now known as pblaho | 11:39 | |
*** david-lyle has joined #openstack-infra | 11:40 | |
*** rfolco has quit IRC | 11:42 | |
*** gokrokve has quit IRC | 11:42 | |
*** david-lyle has quit IRC | 11:44 | |
mkoderer | hi folks - the grenade jobs seems to have problem https://bugs.launchpad.net/openstack-ci/+bug/1356310 | 11:45 |
uvirtbot | Launchpad bug 1356310 in openstack-ci "Grenade test fails while using a deprecated glance command" [Undecided,New] | 11:45 |
*** unicell has joined #openstack-infra | 11:50 | |
*** yfried has joined #openstack-infra | 11:52 | |
*** flwang_ has joined #openstack-infra | 11:53 | |
*** rfolco has joined #openstack-infra | 11:54 | |
*** nosnos has quit IRC | 11:54 | |
openstackgerrit | Radomir Dopieralski proposed a change to openstack/requirements: Update xstatic-jquery-ui for Horizon https://review.openstack.org/113184 | 11:55 |
rdopieralski | so it looks like we won't be splitting Horizon this cycle | 11:55 |
*** yfried__ has quit IRC | 11:56 | |
rdopieralski | but we still want to do it as early as possible in K | 11:56 |
*** andreykurilin_ has quit IRC | 11:56 | |
*** flwang_ has quit IRC | 11:57 | |
*** _nadya_ has quit IRC | 11:59 | |
*** Guest35108 has joined #openstack-infra | 12:01 | |
*** jamielennox is now known as jamielennox|away | 12:02 | |
*** salv-orlando has quit IRC | 12:02 | |
*** adalbas has joined #openstack-infra | 12:03 | |
*** dims__ has quit IRC | 12:04 | |
*** dims_ has joined #openstack-infra | 12:05 | |
russellb | dansmith: btw i went with a new repo-based approach for the new libvirt testing (not source anymore) so the dep concerns shouldn't be a big deal | 12:05 |
russellb | fungi: clarkb ^^^ since you were mentioned in the comment i was responding to in scrollback | 12:06 |
russellb | (a repo maintained by fedora already that is) | 12:07 |
russellb | https://review.openstack.org/#/c/113349/1 | 12:08 |
*** dims_ has quit IRC | 12:09 | |
*** esker has quit IRC | 12:10 | |
*** dprince has joined #openstack-infra | 12:15 | |
*** mpaolino has quit IRC | 12:17 | |
*** dkranz has joined #openstack-infra | 12:20 | |
*** boris-42 has quit IRC | 12:21 | |
imeerovi | hi, I want to run unitests for my jjb patch (https://review.openstack.org/#/c/113348/), I can see that many patches are uploaded with relevant tests/scm/fixtures/<....> stuff. i tried to google about a way to do it and didn't find some specific documentation, can somebody point me to relevant stuff? | 12:21 |
*** dims_ has joined #openstack-infra | 12:22 | |
*** yamamoto has joined #openstack-infra | 12:22 | |
*** amotoki has joined #openstack-infra | 12:22 | |
*** leifmadsen has quit IRC | 12:23 | |
*** leifmadsen has joined #openstack-infra | 12:23 | |
*** mbacchi has joined #openstack-infra | 12:25 | |
*** gargola has joined #openstack-infra | 12:26 | |
imeerovi | afazekas, hi Its Ilia (we were at past at Elena's team), I uploaded my first patch to jjb and i have some question: I want to run unitests for my jjb patch (https://review.openstack.org/#/c/113348/), I can see that many patches are uploaded with relevant tests/scm/fixtures/<....> stuff. i tried to google about a way to do it and didn't find some specific documentation, maybe you can point me to relevant stuff? | 12:26 |
*** weshay has joined #openstack-infra | 12:27 | |
*** yamamoto has quit IRC | 12:27 | |
*** pdmars has joined #openstack-infra | 12:28 | |
*** pdmars has quit IRC | 12:29 | |
*** pdmars has joined #openstack-infra | 12:29 | |
*** dkliban is now known as dkliban_afk | 12:30 | |
*** mpaolino has joined #openstack-infra | 12:30 | |
*** _nadya_ has joined #openstack-infra | 12:30 | |
*** aysyd has joined #openstack-infra | 12:31 | |
*** chandankumar has quit IRC | 12:32 | |
*** bknudson1 has quit IRC | 12:32 | |
*** boris-42 has joined #openstack-infra | 12:32 | |
openstackgerrit | Andreas Jaeger proposed a change to openstack/requirements: Rename bash8 to bashate (part 1) https://review.openstack.org/113080 | 12:33 |
openstackgerrit | Andreas Jaeger proposed a change to openstack/requirements: Rename bash8 to bashate (part 2) https://review.openstack.org/113843 | 12:33 |
*** chandankumar has joined #openstack-infra | 12:33 | |
*** sballe has joined #openstack-infra | 12:34 | |
openstackgerrit | A change was merged to openstack-dev/cookiecutter: Use spaces instead of tabs for indentation https://review.openstack.org/103869 | 12:35 |
*** gokrokve has joined #openstack-infra | 12:37 | |
*** dims_ has quit IRC | 12:39 | |
*** dims_ has joined #openstack-infra | 12:40 | |
*** flaviof is now known as flaviof_zzz | 12:40 | |
*** david-lyle has joined #openstack-infra | 12:40 | |
*** sballe_ has joined #openstack-infra | 12:41 | |
*** gokrokve has quit IRC | 12:41 | |
*** dims_ has quit IRC | 12:44 | |
*** sballe has quit IRC | 12:44 | |
*** david-lyle has quit IRC | 12:45 | |
*** harlowja_at_home has joined #openstack-infra | 12:48 | |
*** yfried has quit IRC | 12:50 | |
*** julim has joined #openstack-infra | 12:50 | |
*** _nadya_ has quit IRC | 12:50 | |
*** ominakov has joined #openstack-infra | 12:51 | |
*** AlexF has joined #openstack-infra | 12:51 | |
*** julim has quit IRC | 12:52 | |
*** bknudson has joined #openstack-infra | 12:52 | |
*** salv-orlando has joined #openstack-infra | 12:53 | |
*** julim has joined #openstack-infra | 12:53 | |
*** radez_g0n3 is now known as radez | 12:54 | |
*** ominakov has quit IRC | 12:54 | |
*** dustins has joined #openstack-infra | 13:00 | |
*** mfer has joined #openstack-infra | 13:02 | |
*** salv-orlando has quit IRC | 13:02 | |
*** daya_k has quit IRC | 13:04 | |
*** sandywalsh has joined #openstack-infra | 13:04 | |
*** rlandy_ has joined #openstack-infra | 13:05 | |
openstackgerrit | Louis Taylor proposed a change to openstack-infra/gitdm: Add Louis Taylor to HP https://review.openstack.org/113873 | 13:06 |
*** dims_ has joined #openstack-infra | 13:10 | |
*** rlandy_ has quit IRC | 13:11 | |
*** ominakov has joined #openstack-infra | 13:11 | |
*** AlexF has quit IRC | 13:12 | |
*** rlandy_ has joined #openstack-infra | 13:12 | |
*** dkliban_afk is now known as dkliban_ | 13:12 | |
*** dkliban_ is now known as dkliban | 13:12 | |
*** andreaf has joined #openstack-infra | 13:14 | |
*** flaviof_zzz is now known as flaviof | 13:16 | |
openstackgerrit | A change was merged to openstack-dev/pbr: Introduce a SemanticVersion object https://review.openstack.org/94108 | 13:16 |
mtreinish | mkoderer: I think https://review.openstack.org/#/c/113878/ should fix it | 13:16 |
*** rlandy_ has quit IRC | 13:17 | |
*** ominakov has quit IRC | 13:17 | |
mtreinish | mkoderer: although this identified a gap in coverage around grenade, stable branches, and clients | 13:18 |
mkoderer | dalgaaf_: ^^ | 13:20 |
*** rlandy_ has joined #openstack-infra | 13:20 | |
mkoderer | mtreinish: ok thx .. I didn't see that there are some glance commands in setup too | 13:20 |
*** homeless has joined #openstack-infra | 13:21 | |
mkoderer | ah, it's in stable .. ok ;) | 13:21 |
*** yamamoto has joined #openstack-infra | 13:22 | |
openstackgerrit | afazekas proposed a change to openstack-infra/config: Not use firewalld on the jenkins slaves https://review.openstack.org/113884 | 13:23 |
*** rlandy_ has quit IRC | 13:24 | |
*** sballe_ has quit IRC | 13:24 | |
fungi | mkoderer: based on your update, i moved that bug from openstack-ci to grenade | 13:24 |
mkoderer | fungi: yep thx | 13:24 |
fungi | (in case that was a non-obvious status change) | 13:24 |
*** stevemar has joined #openstack-infra | 13:24 | |
*** rlandy_ has joined #openstack-infra | 13:25 | |
*** mflobo has joined #openstack-infra | 13:25 | |
*** andreaf_ has joined #openstack-infra | 13:25 | |
openstackgerrit | A change was merged to openstack-infra/config: Add create access right on "refs/heads/*". https://review.openstack.org/113844 | 13:26 |
*** dtantsur|lunch is now known as dtantsur | 13:26 | |
*** yamamoto has quit IRC | 13:27 | |
*** rlandy_ has quit IRC | 13:27 | |
dansmith | russellb: ah, cool | 13:28 |
*** andreaf has quit IRC | 13:28 | |
*** mriedem has joined #openstack-infra | 13:29 | |
openstackgerrit | Ilia Meerovich proposed a change to openstack-infra/jenkins-job-builder: Fixing exception formatting https://review.openstack.org/113888 | 13:29 |
*** rlandy_ has joined #openstack-infra | 13:31 | |
*** andreaf_ has quit IRC | 13:31 | |
*** andreaf has joined #openstack-infra | 13:32 | |
openstackgerrit | A change was merged to openstack-infra/zuul: Report correctly when dequeuing dependent changes https://review.openstack.org/111479 | 13:32 |
*** rlandy_ has quit IRC | 13:33 | |
*** rlandy_ has joined #openstack-infra | 13:34 | |
*** andreaf has quit IRC | 13:34 | |
*** rlandy_ has quit IRC | 13:34 | |
*** andreaf has joined #openstack-infra | 13:35 | |
*** mmaglana has joined #openstack-infra | 13:35 | |
*** rlandy_ has joined #openstack-infra | 13:35 | |
*** maxbit has joined #openstack-infra | 13:35 | |
openstackgerrit | A change was merged to openstack/requirements: Add glance.store to global requirements https://review.openstack.org/113806 | 13:36 |
*** vhoward has left #openstack-infra | 13:36 | |
*** gokrokve has joined #openstack-infra | 13:37 | |
*** rlandy_ has quit IRC | 13:39 | |
openstackgerrit | Ilia Meerovich proposed a change to openstack-infra/jenkins-job-builder: Implements wrapper for ownership plugin. This wrapper sets "Inject ownership variables into environment" https://review.openstack.org/113348 | 13:39 |
homeless | fungi: could you help to review this, https://review.openstack.org/#/c/113598/, there is now what you guys suggested | 13:39 |
*** rdekel has quit IRC | 13:40 | |
*** mmaglana has quit IRC | 13:40 | |
*** ilyashakhat_ has quit IRC | 13:40 | |
*** radez is now known as radez_g0n3 | 13:40 | |
openstackgerrit | Nikita Konovalov proposed a change to openstack-infra/storyboard: Tags db api https://review.openstack.org/113889 | 13:41 |
openstackgerrit | A change was merged to openstack-infra/elastic-recheck: Add query for Postgres startup failure https://review.openstack.org/113634 | 13:41 |
*** gokrokve has quit IRC | 13:42 | |
*** pcrews has joined #openstack-infra | 13:42 | |
*** mmaglana has joined #openstack-infra | 13:43 | |
*** ominakov has joined #openstack-infra | 13:43 | |
*** chandankumar has quit IRC | 13:44 | |
*** armax has joined #openstack-infra | 13:44 | |
*** rfolco has quit IRC | 13:48 | |
*** mmaglana has quit IRC | 13:48 | |
fungi | homeless: jhesketh makes a very good point. rereading the commit message and the change now, i'm questioning what the original problem was. options.has_pull_requests is what we want to skip the closer on. if there's no options section then we need to assume we should close pull requests for it. i don't think the original conditional is incorrect after all, instead you need to trap for nonexistent | 13:48 |
fungi | github projects right? | 13:48 |
openstackgerrit | A change was merged to openstack-infra/elastic-recheck: Add query for ceilometer py26 bug 1355983 https://review.openstack.org/113612 | 13:49 |
uvirtbot | Launchpad bug 1355983 in ceilometer "gate-ceilometer-python26 failing with "Could not find mongod command"" [Undecided,New] https://launchpad.net/bugs/1355983 | 13:49 |
*** sballe has joined #openstack-infra | 13:51 | |
homeless | right fungi, my last comment I explained a little more, but basically the problem becomes when any of your projects doesnt exists in github, then everything breaks, because the current if statement doesnt skip the entry | 13:51 |
openstackgerrit | A change was merged to openstack-dev/pbr: add docs env to tox https://review.openstack.org/113529 | 13:51 |
*** pdmars has quit IRC | 13:51 | |
*** andreaf_ has joined #openstack-infra | 13:52 | |
fungi | homeless: do you have a stacktrace for where it breaks? we might just need to handle an appropriate exception for that instead | 13:53 |
*** marun has joined #openstack-infra | 13:53 | |
*** denis_makogon_ has joined #openstack-infra | 13:53 | |
homeless | sure look at this, this is my projects.yaml http://paste.forj.io/show/314/ | 13:53 |
*** sballe has quit IRC | 13:53 | |
homeless | and with that data, I got this http://paste.forj.io/show/315/ | 13:54 |
*** flwang_ has joined #openstack-infra | 13:54 | |
fungi | ahh, yep i just checked your comment and saw the link | 13:54 |
homeless | because the first project on the projects.yaml (forj-config) doesnt exists on github, so the script should skip it | 13:54 |
*** shashankhegde has joined #openstack-infra | 13:54 | |
*** harlowja_at_home has quit IRC | 13:54 | |
fungi | but i don't think we can infer that from whether the project has an options section in your projects.yaml | 13:55 |
*** denis_makogon has quit IRC | 13:55 | |
*** andreaf has quit IRC | 13:55 | |
*** denis_makogon_ is now known as denis_makogon | 13:55 | |
fungi | which is what your proposed change would effectively do | 13:55 |
*** dmakogon_ has joined #openstack-infra | 13:55 | |
*** sballe has joined #openstack-infra | 13:56 | |
fungi | homeless: we probably should just wrap the get_repo() call in a try block and except github.GithubException.UnknownObjectException with a continue (and a comment stating that this probably means the project does not exist on github) | 13:56 |
homeless | I think my change is what you are trying to do, but with the wrong conditions, and I think you never get that failure because all your projects exists on github | 13:56 |
fungi | homeless: no, if there's no options section, we still want to close pull requests | 13:57 |
homeless | then the option "has-pull-request" is not needed | 13:57 |
fungi | homeless: if it was options.close-pull-requests then that would be true | 13:57 |
fungi | but if there's no options section, and thus no options.has-pull-requests then we should close any we find | 13:58 |
openstackgerrit | A change was merged to openstack-infra/config: Clear out stray github-org parameters https://review.openstack.org/113332 | 13:58 |
*** flwang_ has quit IRC | 13:58 | |
homeless | got it, then the options.has-pull-request becomes useless right | 13:59 |
*** prad has joined #openstack-infra | 14:00 | |
openstackgerrit | A change was merged to openstack-infra/config: Adds Poppy CDN Project https://review.openstack.org/111095 | 14:00 |
homeless | because you will close pull requests anyways.. then that is my problem, because I dont want to do it for all, just the ones that I specify o nthe yaml | 14:00 |
homeless | right fungi? | 14:01 |
*** rfolco has joined #openstack-infra | 14:01 | |
*** andreaf_ has quit IRC | 14:02 | |
fungi | homeless: yeah, the assumption with jeepyb (note that it's a phonetic pronunciation of gpb, the initials for "gerrit project builder") is that you use gerrit for most things, so in some very few cases you might set options.has-pull-requests=true to make the closer skip just that one project | 14:02 |
*** sballe has quit IRC | 14:02 | |
*** pdmars has joined #openstack-infra | 14:02 | |
fungi | but by default, it assumes you want to close pull requests and tell people to use your gerrit instead | 14:02 |
SergeyLukjanov | flaper87, please, take a look on my comments for https://review.openstack.org/#/c/111244/4 | 14:02 |
*** andreaf_ has joined #openstack-infra | 14:02 | |
SergeyLukjanov | fungi, good morning! | 14:02 |
fungi | because why else would you put the project in gerrit, basically | 14:03 |
*** dustins_ has joined #openstack-infra | 14:03 | |
openstackgerrit | A change was merged to openstack-infra/config: launch-node.py: request flavors/images by ID or name https://review.openstack.org/112371 | 14:03 |
*** dustins has quit IRC | 14:03 | |
openstackgerrit | Simon McCartney proposed a change to openstack-infra/jenkins-job-builder: Add support for the SSH Agent Plugin as a build wrapper https://review.openstack.org/113222 | 14:04 |
fungi | homeless: but what you're saying you want is to be able to indicate whether some projects are in github at all, and skip trying to close pull requests for them | 14:04 |
homeless | fungi: yes | 14:04 |
fungi | homeless: stepping back, it seems to me that what you actually *need* though is for the pull request closer not to break when some of your projects aren't in github, and teh pragmatic approach to that is to just skip trying to close pull requests for any project where the github api raises an exception indicating that the project isn't there | 14:05 |
openstackgerrit | A change was merged to openstack-infra/config: Remove devnull from post regex https://review.openstack.org/112057 | 14:05 |
*** hashar has joined #openstack-infra | 14:05 | |
homeless | you got it | 14:05 |
openstackgerrit | A change was merged to openstack-infra/config: Use storyboard for gearman-plugin project. https://review.openstack.org/113368 | 14:06 |
fungi | otherwise you're doing dual bookkeeping of projects in github by mentioning what's replicated both in your gerrit replication config and in your jeepyb projects,yaml, which is likely to diverge over time | 14:06 |
fungi | good evening SergeyLukjanov | 14:06 |
flaper87 | SergeyLukjanov: thanks for the heads up, I'll do that asap | 14:08 |
homeless | another approach could be that I dont want to close pull request for an specific project, then I should skip it | 14:09 |
*** reed has joined #openstack-infra | 14:09 | |
*** _nadya_ has joined #openstack-infra | 14:09 | |
homeless | fungi: right, because right now it doesnt care about the options, I just close the pull request for all the projects on the yaml | 14:10 |
*** wenlock has joined #openstack-infra | 14:10 | |
flaper87 | SergeyLukjanov: done | 14:10 |
openstackgerrit | Flavio Percoco proposed a change to openstack-infra/config: Rename Marconi to Zaqar https://review.openstack.org/111244 | 14:10 |
fungi | homeless: right, you could just put options.has-pull-requests on any project which isn't actually in github, as a workaround, but the pythonic fix is to ask forgiveness not permission | 14:10 |
openstackgerrit | A change was merged to openstack-dev/pbr: cleanup tox.ini https://review.openstack.org/113553 | 14:11 |
fungi | SergeyLukjanov: most recent example of a rename maintenance plan is https://etherpad.openstack.org/p/repo-renaming-2014-06-28 | 14:11 |
fungi | SergeyLukjanov: that should provide a decent template to build from | 14:12 |
*** salv-orlando has joined #openstack-infra | 14:12 | |
*** medieval1_ is now known as medieval1 | 14:12 | |
SergeyLukjanov | fungi, great, I'll try to make the same for the upcoming renaming later today | 14:12 |
SergeyLukjanov | flaper87, thx | 14:12 |
*** timrc-afk is now known as timrc | 14:12 | |
*** Krast has quit IRC | 14:13 | |
*** radez_g0n3 is now known as radez | 14:13 | |
*** Krast has joined #openstack-infra | 14:14 | |
fungi | SergeyLukjanov: a tip... i usually write a quick little shell script to generate the sql update queries and directory modifications from the list of projects to be renamed, so that it's less prone to individual typographical errors or omissions | 14:14 |
openstackgerrit | afazekas proposed a change to openstack-infra/config: Not use firewalld on the jenkins slaves https://review.openstack.org/113884 | 14:14 |
fungi | particularly useful for maintenances where we have a lot of projects to do | 14:14 |
*** rlandy_ has joined #openstack-infra | 14:15 | |
*** lttrl has joined #openstack-infra | 14:16 | |
*** zz_gondoi is now known as gondoi | 14:16 | |
*** openstackgerrit has quit IRC | 14:16 | |
*** rlandy__ has joined #openstack-infra | 14:17 | |
*** mriedem has quit IRC | 14:17 | |
*** openstackgerrit has joined #openstack-infra | 14:17 | |
*** reed has quit IRC | 14:18 | |
*** rlandy_ has quit IRC | 14:19 | |
*** dkranz has quit IRC | 14:19 | |
*** yamamoto has joined #openstack-infra | 14:22 | |
*** yamamoto has quit IRC | 14:24 | |
*** yamamoto has joined #openstack-infra | 14:24 | |
*** wenlock has quit IRC | 14:24 | |
openstackgerrit | A change was merged to openstack-infra/config: Add python3-jobs to os-refresh-config https://review.openstack.org/112493 | 14:26 |
*** mflobo has quit IRC | 14:26 | |
*** mriedem has joined #openstack-infra | 14:26 | |
*** gokrokve has joined #openstack-infra | 14:27 | |
*** Krast has quit IRC | 14:28 | |
*** Krast has joined #openstack-infra | 14:28 | |
*** yamamoto has quit IRC | 14:28 | |
*** Longgeek has quit IRC | 14:29 | |
*** homeless has quit IRC | 14:30 | |
*** Longgeek has joined #openstack-infra | 14:30 | |
*** Krast has quit IRC | 14:31 | |
*** Krast has joined #openstack-infra | 14:31 | |
*** jgallard has joined #openstack-infra | 14:34 | |
*** julim has quit IRC | 14:36 | |
openstackgerrit | patrick-crews proposed a change to openstack-infra/elastic-recheck: Add crm114 logstash queries to uncategorized page https://review.openstack.org/113771 | 14:37 |
*** julim has joined #openstack-infra | 14:37 | |
*** pdmars has quit IRC | 14:40 | |
*** pdmars has joined #openstack-infra | 14:40 | |
*** mmaglana has joined #openstack-infra | 14:41 | |
*** lyxus has quit IRC | 14:41 | |
*** dangers_away is now known as dangers | 14:41 | |
*** mmaglana has quit IRC | 14:45 | |
*** mmaglana has joined #openstack-infra | 14:46 | |
*** dkranz has joined #openstack-infra | 14:46 | |
*** tnurlygayanov has quit IRC | 14:46 | |
*** sdake has joined #openstack-infra | 14:46 | |
*** sdake has quit IRC | 14:46 | |
*** sdake has joined #openstack-infra | 14:46 | |
*** andreykurilin_ has joined #openstack-infra | 14:46 | |
*** tnurlygayanov has joined #openstack-infra | 14:47 | |
*** markmcclain has joined #openstack-infra | 14:48 | |
*** sballe has joined #openstack-infra | 14:49 | |
*** wenlock has joined #openstack-infra | 14:50 | |
*** gokrokve has quit IRC | 14:50 | |
*** gokrokve has joined #openstack-infra | 14:50 | |
*** markmcclain1 has joined #openstack-infra | 14:51 | |
*** flaper87 is now known as flaper87|afk | 14:54 | |
*** datsun180b has joined #openstack-infra | 14:54 | |
*** pballand has joined #openstack-infra | 14:54 | |
*** markmcclain has quit IRC | 14:55 | |
*** gokrokve has quit IRC | 14:55 | |
*** mmaglana has quit IRC | 14:56 | |
*** pblaho is now known as pblaho|afk | 14:58 | |
*** otherwiseguy has joined #openstack-infra | 14:58 | |
*** UtahDave has joined #openstack-infra | 14:59 | |
*** UtahDave has left #openstack-infra | 14:59 | |
anteaya | SergeyLukjanov: np, I figured as much | 15:00 |
anteaya | SergeyLukjanov: if you have time to join #openstack-cinder some folks have some questions | 15:00 |
*** jgrimm has joined #openstack-infra | 15:01 | |
*** gondoi is now known as zz_gondoi | 15:01 | |
phschwartz | morning infra | 15:01 |
anteaya | rhsu hello | 15:01 |
anteaya | salv-orlando: thanks I did, my first night of blues swing | 15:03 |
anteaya | salv-orlando: not sure how many kinds of swing there are | 15:03 |
anteaya | salv-orlando: every night is a new experience | 15:03 |
salv-orlando | I have no idea at all. | 15:03 |
anteaya | morning phschwartz salv-orlando | 15:03 |
salv-orlando | it’s probably still early for rhsu. It might be a hour. | 15:03 |
anteaya | salv-orlando: sure | 15:04 |
salv-orlando | and btw goodmorning to you as well, anteaya - even if it’s 5 pm here. | 15:04 |
anteaya | salv-orlando: good evening to you | 15:04 |
anteaya | salv-orlando: how is dinner coming along | 15:04 |
*** evgenyf has quit IRC | 15:04 | |
salv-orlando | anteaya: still early for that. I’ll do reviews for an hour or two | 15:05 |
*** rhsu has joined #openstack-infra | 15:06 | |
*** ildikov has quit IRC | 15:06 | |
anteaya | salv-orlando: ah, patch owners will be happy | 15:06 |
*** MarkAtwood has joined #openstack-infra | 15:07 | |
*** funzo has quit IRC | 15:10 | |
*** funzo has joined #openstack-infra | 15:10 | |
*** mmaglana has joined #openstack-infra | 15:10 | |
*** pblaho|afk is now known as pblaho | 15:10 | |
*** zz_dimtruck is now known as dimtruck | 15:11 | |
funzo | clarkb: if you get a couple minutes, would you mind reviewing https://review.openstack.org/#/c/113333/ | 15:11 |
*** jgallard has quit IRC | 15:11 | |
*** mmaglana has quit IRC | 15:12 | |
*** yfried has joined #openstack-infra | 15:12 | |
*** Sukhdev has joined #openstack-infra | 15:12 | |
*** yolanda has quit IRC | 15:13 | |
*** eharney has joined #openstack-infra | 15:14 | |
*** shashankhegde has quit IRC | 15:17 | |
*** cody-somerville has quit IRC | 15:18 | |
*** rlandy has quit IRC | 15:18 | |
*** yamahata has joined #openstack-infra | 15:19 | |
*** homeless has joined #openstack-infra | 15:21 | |
*** rlandy__ has quit IRC | 15:22 | |
*** yamamoto has joined #openstack-infra | 15:22 | |
*** daya_k has joined #openstack-infra | 15:23 | |
*** andreykurilin_ has quit IRC | 15:26 | |
*** yamamoto has quit IRC | 15:27 | |
*** baoli has joined #openstack-infra | 15:27 | |
jeblair | anteaya: ping | 15:30 |
anteaya | hello there jeblair | 15:31 |
mordred | jeblair, clarkb: http://grafana.org/ just got pointed at this - it | 15:32 |
anteaya | thanks for the +A on the single quote patch | 15:32 |
mordred | jeblair, clarkb: it's apparently related to kibana but is a frontend for graphite | 15:32 |
jeblair | anteaya: hi. do you have links to the wiki pages for third party contact information? | 15:32 |
*** imeerovi has quit IRC | 15:32 | |
anteaya | jeblair: yes, https://wiki.openstack.org/wiki/Category:ThirdPartySystems | 15:32 |
jeblair | anteaya: is there documentation that third party systems should set that up? | 15:33 |
anteaya | I have been waiting for the naming to settle down before pointing many people at it | 15:33 |
anteaya | since I wanted to try to avoid many redirects | 15:33 |
anteaya | no, to be honest I got caught in the mud of the renaming | 15:34 |
anteaya | and I have worked with a few people to set up their pages | 15:34 |
openstackgerrit | Michael Krotscheck proposed a change to openstack-infra/puppet-storyboard: Added RabbitMQ to StoryBoard https://review.openstack.org/113935 | 15:34 |
openstackgerrit | A change was merged to openstack-infra/config: Added a missing single quote https://review.openstack.org/113655 | 15:34 |
*** annegent_ has joined #openstack-infra | 15:34 | |
anteaya | but not enough to know what they need to be told to have success | 15:34 |
jeblair | anteaya: is there a pending patch to add that documentation? | 15:34 |
anteaya | no, I haven't even attempted to put together the documentation for the wikipage set up | 15:35 |
anteaya | the naming has really bogged me down | 15:35 |
jeblair | anteaya: okay, i'm going to do that very quickly. back in a few. | 15:35 |
anteaya | jeblair: very good | 15:35 |
*** mmaglana has joined #openstack-infra | 15:39 | |
*** gokrokve has joined #openstack-infra | 15:41 | |
*** rpodolyaka_afk has left #openstack-infra | 15:42 | |
openstackgerrit | Michael Krotscheck proposed a change to openstack-infra/puppet-storyboard: StoryBoard Deferred Processors https://review.openstack.org/113937 | 15:42 |
openstackgerrit | James E. Blair proposed a change to openstack-infra/config: Add instructions to register third-party CI in wiki https://review.openstack.org/113938 | 15:42 |
jeblair | anteaya: ^ that look good to you? | 15:42 |
* anteaya clicks | 15:42 | |
zaro | orning | 15:43 |
*** atiwari has joined #openstack-infra | 15:43 | |
zaro | jeblair: it's unclear what you are suggesting in review of https://review.openstack.org/#/c/92268 | 15:44 |
*** mmaglana has quit IRC | 15:45 | |
*** zz_gondoi is now known as gondoi | 15:45 | |
zaro | jeblair: could you please clarify for me? | 15:45 |
anteaya | jeblair: yes, one suggestion | 15:45 |
nibalizer | mordred: i was playing with grafana a cpl months ago | 15:46 |
anteaya | morning zaro | 15:46 |
nibalizer | pretty sweet, since it stores graphs in ES so you can easily share them out with people | 15:46 |
jeblair | zaro: publishing a draft should not remove a change from the queue. | 15:46 |
nibalizer | and if you already have people working with ELK then giving a kibana view into graphite helps prevent retrain | 15:46 |
nibalizer | however, the codebases have diverged significantly and there aren't plans to ever really support grafana and elk on the same web page, short of embedding | 15:47 |
mordred | nibalizer: darn | 15:47 |
mordred | nibalizer: that was kinda what I was hoping ... was that there would be a magical kibana/graphite super-dashboard thing | 15:47 |
nibalizer | i mean everyone wants it | 15:48 |
nibalizer | so its comming | 15:48 |
mordred | nibalizer: but - the "save graphs and dashboards in elasticsearch" thing is cool | 15:48 |
nibalizer | grafana supports multiple graphites | 15:48 |
openstackgerrit | James E. Blair proposed a change to openstack-infra/config: Add instructions to register third-party CI in wiki https://review.openstack.org/113938 | 15:48 |
mordred | nibalizer: what does that even mean? | 15:48 |
nibalizer | multiple graphite urls | 15:48 |
nibalizer | so graphite-dev and graphite-prod | 15:48 |
mordred | nibalizer: no, I mean, I undersatnd your words | 15:48 |
mordred | ok. thanks | 15:48 |
mordred | I was lokoing for "why would I want more tahn one graphite" | 15:48 |
jeblair | mordred: graphite supports saving graphs; but that kind of suggests a user-authentication system that we have not put in place | 15:49 |
nibalizer | if they are different servers | 15:49 |
nibalizer | jeblair: ya you have to add user auth | 15:49 |
anteaya | jeblair: +1 | 15:49 |
jeblair | mordred, nibalizer: i expect something different will face a similar problem | 15:49 |
jeblair | mordred: re-review https://review.openstack.org/113938 pls? | 15:49 |
mordred | jeblair: yah. it's less interesting of a thing to look in to if it _doesn't_ act as both graphite frontend _And_ kibana | 15:49 |
mordred | jeblair: on it | 15:49 |
nibalizer | jeblair: grafana takes the 'paste/etherpad' approach of letting anonymous users carve out namespaces and we hope no one horribly abuses it | 15:50 |
jeblair | nibalizer: ok | 15:50 |
mordred | jeblair: done | 15:50 |
jeblair | fungi: if you are around, can you aprv https://review.openstack.org/#/c/113938/ please? | 15:51 |
zaro | jeblair: what should it do? just add another change to the queue? | 15:51 |
jeblair | zaro: it should be recognized as a trigger event, but that's it. | 15:51 |
* anteaya really hopes she can keep enough of jeblair's attention for https://review.openstack.org/#/c/95743/ to be merged, the js patch for ci filtering | 15:51 | |
jeblair | anteaya: working up to that | 15:51 |
anteaya | jeblair: yup | 15:51 |
anteaya | just kind of announcing to the room | 15:52 |
* anteaya elbows some space | 15:52 | |
*** lemongrabber has quit IRC | 15:52 | |
jeblair | anteaya: i prepared this this morning based on your earlier etherpad: https://etherpad.openstack.org/p/32PBpaudDt | 15:52 |
*** lemongrab has joined #openstack-infra | 15:52 | |
* anteaya clicks | 15:52 | |
*** lemongrab is now known as Guest48423 | 15:53 | |
*** yfried has quit IRC | 15:53 | |
jeblair | anteaya: i intend to: 1) merge the doc change i just wrote; 2) run those renames; 3) send an email announcing the renames and doc change; 4) merge the JS changes. all before lunch. :) | 15:53 |
*** Guest48423 has quit IRC | 15:54 | |
anteaya | jeblair: yay | 15:54 |
anteaya | reviewing your etherpad for step 2 now | 15:54 |
*** afshaff has joined #openstack-infra | 15:54 | |
*** ramashri has joined #openstack-infra | 15:55 | |
*** dprince has quit IRC | 15:55 | |
*** flwang_ has joined #openstack-infra | 15:55 | |
*** yamamoto has joined #openstack-infra | 15:55 | |
*** pcm__ has quit IRC | 15:57 | |
*** tjones has left #openstack-infra | 15:58 | |
*** ihrachyshka has quit IRC | 15:58 | |
jeblair | anteaya: also, is the change to add a recheck syntax for third-party systems still around? | 15:59 |
anteaya | jeblair: question: ssh review gerrit set-account 10692 --full-name '"Brocade Vyatta CI"' this is currently Brocade OSS CI -- I'm fine with the change, I guess we can change it back if it upsets them | 15:59 |
anteaya | yes | 15:59 |
*** flwang_ has quit IRC | 15:59 | |
anteaya | jeblair: https://review.openstack.org/#/c/109565/ | 15:59 |
jeblair | anteaya: the change to vyatta was in your earlier etherpad | 16:00 |
fungi | jeblair: i just spotted something maybe unimportant on 95743 (+2'd anyway) but wanted to make sure you saw it... easy to possibly overlook since i didn't downvote on it | 16:00 |
*** jlibosva has quit IRC | 16:01 | |
openstackgerrit | Daniel Shirley proposed a change to openstack-infra/jenkins-job-builder: Added support for JClouds Blobstore https://review.openstack.org/107515 | 16:01 |
anteaya | jeblair: ah okay | 16:01 |
*** k4n0 has quit IRC | 16:01 | |
*** pballand has quit IRC | 16:01 | |
*** yfried has joined #openstack-infra | 16:01 | |
jeblair | fungi: i agree a link would be good. however, i'm not certain that puppet won't copy new versions of that file. | 16:01 |
fungi | jeblair: nor am i ;) | 16:02 |
jeblair | (i can never remember) | 16:02 |
*** hashar is now known as hasharConfcall | 16:02 | |
*** emagana has joined #openstack-infra | 16:02 | |
anteaya | jeblair: I had thought we weren't going to use storage, have I mis-remembered? ssh review gerrit set-account 11611 --full-name '"Nimble Storage CI" | 16:02 |
*** gyee_ has joined #openstack-infra | 16:02 | |
jeblair | anteaya: it's the name of their company | 16:02 |
*** dustins_ has quit IRC | 16:02 | |
anteaya | okay, so the same for Pure Storage CI | 16:03 |
jeblair | http://www.nimblestorage.com/ | 16:03 |
anteaya | since Pure Storage is the name of their company | 16:03 |
anteaya | what about HP Storage CI? | 16:03 |
jeblair | yep. and i think both of those are better than "nimble ci" and "pure ci" :) | 16:03 |
jeblair | anteaya: got a better name? | 16:03 |
jeblair | anteaya: hp has a storage division they call "hp storage" http://www8.hp.com/us/en/products/data-storage/ | 16:04 |
fungi | nibalizer: out of curiosity, does ensure:present on a file with another local file as its source automatically replace if the source file changes or is there more magic needed? (related to my comment on the https://review.openstack.org/#/c/95743/10/modules/openstack_project/manifests/gerrit.pp patch) | 16:04 |
*** gargola has quit IRC | 16:04 | |
fungi | hogepodge: ^ ? | 16:04 |
* fungi just spams pings at random people for fun | 16:04 | |
anteaya | jeblair: I agree with Nimble Storage and Pure Storage | 16:04 |
nibalizer | PING CANNON | 16:04 |
fungi | hold on, reloading... ;) | 16:04 |
anteaya | jeblair: okay, no I don't have a better name right now | 16:05 |
anteaya | let's go with it then | 16:05 |
jeblair | easy to change if we do :) | 16:05 |
anteaya | true | 16:05 |
nibalizer | im pretty sure its gonna md5 both sides and overwrite if it finds diffs | 16:05 |
*** sabeen3 has quit IRC | 16:05 | |
nibalizer | but let me fire up a quick test | 16:05 |
*** daya_k has quit IRC | 16:05 | |
*** p5ntangle has quit IRC | 16:05 | |
anteaya | how were you able to confirm the bots are bots? | 16:05 |
fungi | nibalizer: awesome. if so, it's just what we want there | 16:05 |
*** sabeen1 has joined #openstack-infra | 16:05 | |
anteaya | jeblair: I was never able to figure out how | 16:05 |
jeblair | anteaya: i checked to make sure they had not reviewed anything recently. search for "reviewer:<accountnumber>" | 16:06 |
*** gargola has joined #openstack-infra | 16:06 | |
*** Guest35108 has quit IRC | 16:06 | |
anteaya | kk | 16:07 |
nibalizer | fungi: confirmed it keeps em in sync | 16:07 |
*** dustins has joined #openstack-infra | 16:07 | |
fungi | nibalizer: great--thanks! | 16:07 |
*** pballand has joined #openstack-infra | 16:07 | |
asselin__ | nodepool works with openstack-master provider, right? | 16:07 |
*** andreykurilin_ has joined #openstack-infra | 16:07 | |
*** reed has joined #openstack-infra | 16:08 | |
fungi | asselin__: i believe so... we have tripleo cloud regions as providers in nodepool.openstack.org's deployment and they follow tip of master for server projects (or very close to it modulo bugs being worked around while fixes are pending) | 16:09 |
anteaya | okay I confirm the accounts named bot have not reviewed anything in a while | 16:09 |
*** yfried has quit IRC | 16:09 | |
anteaya | jeblair: I'm good with step 2 | 16:09 |
anteaya | jeblair: I'm also good with 3 and 4 | 16:09 |
*** praneshp has joined #openstack-infra | 16:09 | |
*** juzuluag has joined #openstack-infra | 16:09 | |
*** praneshp_ has quit IRC | 16:09 | |
anteaya | standing by if you want me to view your email for 3 in an etherpad | 16:09 |
jeblair | anteaya: thx, writing it now | 16:10 |
anteaya | kk | 16:10 |
nibalizer | so i have all the storyboard history proposed as reviews to the infra/puppet-storyboard module, starting here: https://review.openstack.org/#/c/100370/ | 16:10 |
nibalizer | and going on through dependencies for about 15 commits | 16:10 |
*** annegent_ has quit IRC | 16:10 | |
nibalizer | then this; https://review.openstack.org/#/c/113731/, which is -2 workflow right now, is the actual commit to flip it over | 16:10 |
krotscheck | The chain ends here: https://review.openstack.org/#/c/113935/ | 16:11 |
krotscheck | (as in, that’s the first new code in that repo) | 16:11 |
nibalizer | once the rest of the storyboard stuff lands, we should see the puppet-apply-precise test start passing again | 16:12 |
asselin__ | we have an interesting nodepool/jenkins use case where we would like at most 1 job of a certain type to run per provider. Is there an easy way to configure that? | 16:12 |
nibalizer | krotscheck: nice | 16:12 |
*** vermarah has joined #openstack-infra | 16:13 | |
asselin__ | currently it seems we need to create 2 providers for the same 1 provider, with the extra provider set to max-server: 1 for that special job | 16:13 |
SergeyLukjanov | fungi, thanks for the tips | 16:13 |
jroll | can I please bug a devstack core to take a look at these two reviews? trying to get check jobs going for a new ironic driver. :) https://review.openstack.org/#/c/112095/ and https://review.openstack.org/#/c/108457/ | 16:14 |
*** mmaglana has joined #openstack-infra | 16:14 | |
*** andreykurilin_ has quit IRC | 16:15 | |
*** hdd has joined #openstack-infra | 16:17 | |
*** ling-yun-1 has joined #openstack-infra | 16:17 | |
*** dprince has joined #openstack-infra | 16:18 | |
jeblair | anteaya: draft email at bottom of same etherpad: https://etherpad.openstack.org/p/32PBpaudDt | 16:19 |
* anteaya reads | 16:19 | |
*** mmaglana has quit IRC | 16:20 | |
*** jcoufal has quit IRC | 16:20 | |
*** ling-yun has quit IRC | 16:21 | |
anteaya | jeblair: two suggestions in the etherpad | 16:21 |
*** arnaud has joined #openstack-infra | 16:21 | |
*** mmaglana has joined #openstack-infra | 16:21 | |
*** e0ne has quit IRC | 16:21 | |
jeblair | anteaya: doen | 16:21 |
jeblair | done | 16:21 |
anteaya | thanks | 16:21 |
jeblair | fungi: lgty? | 16:21 |
*** rhsu has quit IRC | 16:22 | |
*** amuller has quit IRC | 16:23 | |
*** praneshp has quit IRC | 16:23 | |
*** skolekonov has quit IRC | 16:24 | |
fungi | jeblair: yep, though i also agree with anteaya's suggested clarifications there | 16:25 |
*** yamahata has quit IRC | 16:26 | |
jeblair | fungi: thanks. made your update too. | 16:26 |
jeblair | i have renamed all the ci accounts now | 16:26 |
*** fungi has quit IRC | 16:26 | |
anteaya | yay!! | 16:27 |
*** rdopieralski has quit IRC | 16:27 | |
*** fungi has joined #openstack-infra | 16:27 | |
anteaya | fungi: welcome back | 16:28 |
mkoderer | jogo: hi ;) we need you +A for https://review.openstack.org/#/c/113878/ | 16:28 |
mkoderer | otherwise check-grenade-dsvm-icehouse doesn't work | 16:29 |
fungi | yeah, whoops. my fingers substituted /quit on a channel when my brain wanted /part | 16:29 |
anteaya | ha ha ha | 16:30 |
*** hasharConfcall has quit IRC | 16:30 | |
*** mmaglana_ has joined #openstack-infra | 16:30 | |
*** jpich has quit IRC | 16:31 | |
openstackgerrit | James E. Blair proposed a change to openstack-infra/config: Add CI results table to Gerrit https://review.openstack.org/105860 | 16:32 |
jeblair | SergeyLukjanov, fungi: ^ i'd like to aprv that now as well | 16:32 |
*** mmaglana has quit IRC | 16:33 | |
SergeyLukjanov | jeblair, it's already tested, isn't it? | 16:33 |
*** mmaglana_ has quit IRC | 16:33 | |
SergeyLukjanov | jeblair, on review-dev | 16:33 |
jeblair | SergeyLukjanov: yes, it should still be running there (unless it was accidentally reverted). the only change i made since then was whitespace and removing that debug statement | 16:34 |
jeblair | (i'll go ahead and put the latest version on there) | 16:34 |
SergeyLukjanov | jeblair, yeah, see my own comments ;) | 16:34 |
*** _nadya_ has quit IRC | 16:34 | |
jeblair | SergeyLukjanov: yeah, i meant i just made those changes because of your comments :) | 16:35 |
jeblair | review-dev is restarting with latest hideci | 16:36 |
*** andreaf_ has quit IRC | 16:36 | |
* anteaya goes to review-dev | 16:37 | |
*** fifieldt has quit IRC | 16:37 | |
openstackgerrit | A change was merged to openstack-infra/config: Add button that shows/hides CI comments in Gerrit https://review.openstack.org/95743 | 16:38 |
clarkb | do much mail and scrollback | 16:38 |
clarkb | *so | 16:38 |
anteaya | perhaps I am thick, I am not seeing a table for the ci results | 16:38 |
anteaya | can anyone suggest a url? | 16:38 |
hogepodge | fungi if you specify a source ensure => present will always sync the file to the source. | 16:38 |
SergeyLukjanov | jeblair, fungi, I'm ok with approving https://review.openstack.org/#/c/105860/ | 16:39 |
fungi | hogepodge: good to know. for some reason it never dawned on me that it would also do that for local sources | 16:39 |
*** daya_k has joined #openstack-infra | 16:39 | |
*** atiwari has quit IRC | 16:40 | |
fungi | but in retrospect, makes perfect sense | 16:40 |
fungi | SergeyLukjanov: jeblair: me too. +2'd | 16:40 |
afshaff | Hey guys, I've got a question about Jenkins Job Builder. I'm building out some job templates and I would like to include a file with a templated filename (ie. !include-raw {name}-build-script.sh) I've tried using macros, YAML node references, and digging through the source code a bit, but can't seem to figure anything out. Can anyone here help me out? Thanks! | 16:41 |
jeblair | anteaya: i think i didn't get all of the necessary files installed; one sec | 16:41 |
anteaya | kk | 16:42 |
*** afazekas has quit IRC | 16:42 | |
fungi | afshaff: have you tried escaping the parameter by doubling the braces? e.g. {{name}} | 16:42 |
fungi | afshaff: oh, i think i may have misread what you want to do there... | 16:43 |
fungi | afshaff: we may have an example--looking now | 16:43 |
*** mmaglana has joined #openstack-infra | 16:44 | |
*** fifieldt has joined #openstack-infra | 16:44 | |
jeblair | anteaya: https://review-dev.openstack.org/#/c/5269/ | 16:45 |
afshaff | awesome thanks, with the single brace, it tries to find a file named {repo}-build-script.sh, and with two braces it throws a parse expection | 16:45 |
anteaya | jeblair: ah beautiful | 16:45 |
*** imeerovi has joined #openstack-infra | 16:45 | |
anteaya | beautiful, beautiful | 16:45 |
clarkb | fungi the no rpm and no yum found errors on dfw nodes dont appear to be related to the hpcloud-b1 issue from yesterday (wrong base image) | 16:46 |
*** mmaglana_ has joined #openstack-infra | 16:46 | |
jeblair | i'm really hoping we can get something like that into gerrit proper with _david_'s plugin | 16:46 |
clarkb | fungi have you come up with any new theories for those fails? | 16:47 |
fungi | clarkb: agreed. i strongly suspect they're related to the too-old run-pep8.sh failures | 16:47 |
anteaya | jeblair: that will be awesome | 16:47 |
fungi | clarkb: on my list for today is to start analyzing what snapshots we have hanging around in that region | 16:47 |
clarkb | I will try holding a node today | 16:47 |
*** juzuluag has quit IRC | 16:47 | |
*** juzuluag_ has joined #openstack-infra | 16:47 | |
fungi | clarkb: current best theory is a bug in either nodepool, novaclient or nova at rax causing a wrong (very outdated) snapshot to get selected under some circumstances (maybe bad image cache on a particular nova compute host?) | 16:48 |
*** rhsu has joined #openstack-infra | 16:49 | |
*** mmaglana has quit IRC | 16:49 | |
*** sballe has quit IRC | 16:50 | |
jogo | mkoderer: done | 16:50 |
zaro | i just signed out of review-dev, but cannot sign back in. anybody know why? | 16:51 |
anteaya | jeblair was updating review-dev, zaro, perhaps that might be playing a role | 16:52 |
anteaya | zaro: https://review-dev.openstack.org/#/c/5269/ | 16:52 |
anteaya | zaro: note the pretty table of ci results, separate from comments | 16:52 |
*** mjturek has left #openstack-infra | 16:52 | |
fungi | afshaff: here's an example of embedding a built-in job parameter within a shell builder in a job template... http://git.openstack.org/cgit/openstack-infra/config/tree/modules/openstack_project/files/jenkins_job_builder/config/oslo.yaml#n27 | 16:53 |
*** gyee_ has quit IRC | 16:53 | |
*** praneshp has joined #openstack-infra | 16:53 | |
jeblair | zaro: i just signed into review-dev | 16:54 |
*** ling-yun-1 has quit IRC | 16:56 | |
*** pcm_ has joined #openstack-infra | 16:56 | |
zaro | ok, i'm able to login now. | 16:56 |
*** ling-yun-1 has joined #openstack-infra | 16:56 | |
*** Longgeek has quit IRC | 16:57 | |
*** alexpilotti has joined #openstack-infra | 16:57 | |
afshaff | fungi: ahh ok, that isn't quite what we were hoping to do though. because our build scripts could look vastly different based on what {repo} we are trying to build, I don't think it'I wanted to manage the build scripts separately and use the !include syntax to "plug-in" the correct build script based on the repo. | 16:58 |
afshaff | fungi: *I don't think it's possible to only substitute a few variables here and there into an inline script* | 16:59 |
phschwartz | fungi: clarkb: mordred: jeblair: Just wanted to give you guys a follow up. Our internal Neutron devs are looking to see what is causing issues while instances are being deleted. Neutron looks to be the issue that is causing the error+deleting state. | 16:59 |
fungi | phschwartz: right, we see it in random devstack runs ourselves... neutron erroring on port removal i think | 16:59 |
phschwartz | fungi: That would be the issue. lol | 16:59 |
clarkb | there is probably an e-r bug tracking it | 17:00 |
fungi | almost certainly. any collaboration on that issue would help massively upstream as well as downstream | 17:00 |
jeblair | one way or another, this system is tests openstack. :) | 17:00 |
jeblair | s/is// | 17:00 |
fungi | or put another way, we get hit coming and going | 17:00 |
jeblair | haha | 17:01 |
*** chuckC has quit IRC | 17:01 | |
*** annegent_ has joined #openstack-infra | 17:02 | |
*** AlexF has joined #openstack-infra | 17:03 | |
*** bhuvan has joined #openstack-infra | 17:03 | |
fungi | phschwartz: maybe bug 1313794 or bug 1334109 (those may be dupes)? | 17:04 |
uvirtbot | Launchpad bug 1313794 in neutron "lock_wait timeout caused by plug_vip_port" [Medium,In progress] https://launchpad.net/bugs/1313794 | 17:04 |
uvirtbot | Launchpad bug 1334109 in neutron "Lock wait timeout updating router's gw_port_id" [Medium,In progress] https://launchpad.net/bugs/1334109 | 17:04 |
clarkb | fungi: :) | 17:04 |
*** juzuluag_ has quit IRC | 17:05 | |
fungi | caught between an upstream rock and a downstream hard place? | 17:05 |
fungi | that doesn't roll off the tongue quite so well | 17:05 |
*** annegent_ has quit IRC | 17:05 | |
phschwartz | lol | 17:05 |
krotscheck | phschwartz: FYI https://review.openstack.org/#/c/112128/ | 17:06 |
clarkb | fungi: looks like devstack-trusty in hpcloud-b1 built a just under a couple hours ago so that has been sorted | 17:06 |
*** annegent_ has joined #openstack-infra | 17:06 | |
*** krtaylor has quit IRC | 17:06 | |
openstackgerrit | A change was merged to openstack-infra/config: Add instructions to register third-party CI in wiki https://review.openstack.org/113938 | 17:07 |
fungi | clarkb: presumably, if we're not seeing the shmmax/postgres issue resurface there anyway | 17:07 |
phschwartz | krotscheck: looks good to me | 17:07 |
*** Ryan_Lane has joined #openstack-infra | 17:07 | |
clarkb | fungi: dtroyer: I do still think that postgres should bump those numbers | 17:07 |
*** YorikSar has quit IRC | 17:07 | |
clarkb | but that change should probably come from someone that understands postgres | 17:07 |
krotscheck | As much as I hate asking for a rubberstamp, are there any infra-core available to help nibalizer and I move the storyboard puppet module history moved over to infra/puppet-storyboard? https://review.openstack.org/#/c/100370/ | 17:08 |
*** MarkAtwood has quit IRC | 17:08 | |
krotscheck | phschwartz: Thanks :) | 17:08 |
krotscheck | phschwartz: I’m not _quite_ ready to land the storyboard one yet since I have a different priority today, but I’m hopeful. | 17:08 |
fungi | afshaff: unless i misunderstand your comment, we do pass per-project parameters into job templates when instantiating them, like http://git.openstack.org/cgit/openstack-infra/config/tree/modules/openstack_project/files/jenkins_job_builder/config/projects.yaml#n1498 (tons of examples in there) | 17:08 |
clarkb | krotscheck: nibalizer: are we not doing that as a git filter branch pushed into the repo? | 17:09 |
phschwartz | krotscheck: I am hoping soon I will have our internal systems here stable so I can start on vinz. :( darn hassel of broken things | 17:09 |
nibalizer | clarkb: i submitted it as reviews | 17:09 |
clarkb | krotscheck: nibalizer: it looks like we are pusing code up to make the switch | 17:09 |
clarkb | nibalizer: why not git filter branch? | 17:09 |
nibalizer | i dont have push access to the repo afaik | 17:09 |
*** YorikSar has joined #openstack-infra | 17:09 | |
*** arborism has joined #openstack-infra | 17:09 | |
*** andreykurilin_ has joined #openstack-infra | 17:09 | |
*** arborism is now known as amcrn | 17:09 | |
clarkb | nibalizer: right, you should be doing it as part of the initial repo import | 17:09 |
nibalizer | clarkb: i used subtree but its basically the same as i understnd it? | 17:09 |
fungi | phschwartz: hasselhoff broken things?!? | 17:09 |
krotscheck | clarkb: Isn’t it too late to do that now? | 17:10 |
*** shashankhegde has joined #openstack-infra | 17:10 | |
clarkb | krotscheck: yes, I am wondering why we got here | 17:10 |
anteaya | fungi: ohhhh | 17:10 |
* anteaya wonders who else got it | 17:10 | |
clarkb | krotscheck: this is our guinea pig, I have an interest in doing it right :) | 17:10 |
* fungi can't let some typos go | 17:10 | |
phschwartz | fungi: lol, yeah. stupid spellcheck failure on OSx | 17:10 |
krotscheck | clarkb: Well, I think in this case the repo preceded the guinea pig. | 17:11 |
*** yamahata has joined #openstack-infra | 17:11 | |
clarkb | krotscheck: ya that seems to be the case | 17:12 |
*** Sukhdev has quit IRC | 17:12 | |
clarkb | what do people think should we still filter branch and push it in from the side (does that repo already have history?) | 17:12 |
krotscheck | clarkb: Ok, so a good thing to note for guinea pig v2 | 17:12 |
openstackgerrit | Aishwarya Thangappa proposed a change to openstack-infra/storyboard: Added a subscription_events table & subscription hander https://review.openstack.org/113016 | 17:12 |
clarkb | this is probably a bad guinea pig | 17:12 |
clarkb | nibalizer: krotscheck: is there divergent history in that repo now? I am assuming yes for things like .gitreview files | 17:14 |
nibalizer | yea the whole history of the subtree was replayed on top of .gitreview and a testing rakefile | 17:14 |
clarkb | nibalizer: :( | 17:15 |
nibalizer | clarkb: why does divergent history matter ? | 17:15 |
*** praneshp has quit IRC | 17:15 | |
clarkb | nibalizer: because I thought the process we agreed on, was filter branch out a module into a standaloen repo, put that in gerrit, setup infra/config to use that repo and remove the in tree version. | 17:16 |
clarkb | nibalizer: can't do that with puppet-storyboard without forcing everyone to git reset --hard | 17:16 |
nibalizer | clarkb: i dont think we're communicating | 17:16 |
clarkb | nibalizer: we can still do the above process with storyboard | 17:17 |
nibalizer | so i split out the storyboard module | 17:17 |
clarkb | nibalizer: doing so will force anyone with puppet-storyboard clone to git reset --hard | 17:17 |
krotscheck | clarkb: Oh, that’s not what’s going on. We started with a clean repo, added .gitreview and Rakefile, and then applied all the subtree patches against it. | 17:17 |
clarkb | krotscheck: right which is the wrong way to do it | 17:17 |
nibalizer | and resubmitted all those commits against puppet-storyboard | 17:17 |
clarkb | krotscheck: we have never split repos that way | 17:17 |
nibalizer | it doesn't require anyone to reset --hard | 17:17 |
clarkb | nibalizer: the process I describe does | 17:18 |
clarkb | nibalizer: I don'twant to do what you did to every one of our modules | 17:18 |
krotscheck | clarkb: Why would that require a reset —hard? Everything’s in there, we’re not rewriting the history | 17:18 |
clarkb | nibalizer: the review overhead is too high | 17:18 |
fungi | correcting it to preserve the original history from the config repo at this stage will cause people who cloned the previous state of the puppet-storyboard repo to need to reset | 17:18 |
nibalizer | well its all code thats already been accepted it | 17:18 |
anteaya | jeblair: I think you have done all the steps but I don't think https://review.openstack.org/#/c/105860/ will merge quickly, so many in the check queue | 17:18 |
nibalizer | the current state of the puppet-storyboard repo only has a .gitreview file in it | 17:18 |
clarkb | nibalizer: right, which is why I just want ot push it into the repo with the original history | 17:18 |
*** yamahata has quit IRC | 17:18 | |
anteaya | jeblair: and it is before lunch | 17:18 |
nibalizer | okay, so how do you want it to be done | 17:18 |
nibalizer | clarkb: okay | 17:19 |
anteaya | jeblair: well except the email, I haven't seen it in my inbox yet | 17:19 |
nibalizer | but i dont have those acls, so i did what I could | 17:19 |
clarkb | nibalizer: you do have the ability to do this | 17:19 |
clarkb | nibalizer: let me describe the process | 17:19 |
*** amotoki has quit IRC | 17:19 | |
nibalizer | okay please do | 17:19 |
nibalizer | because im not understanding it | 17:19 |
nibalizer | especially around the 'push it into' part | 17:19 |
clarkb | nibalizer: git filter branch module out of infra/config into a standalone repo with NO history changed. Do not rebase .gitreview under this repo | 17:20 |
afshaff | fungi: the gotcha with what I'm trying to do is that I'm trying to pass parameters into an !include tag so that it can dynamically figure out what filename it needs to include | 17:20 |
clarkb | nibalizer: add this repo to gerrit using jeepyb and review.projects.yaml. If you point the upstream at this standalone repo it will be imported into gerrit as is | 17:20 |
fungi | afshaff: oh... where !include is a yaml directive not a jjb templating expansion | 17:20 |
jeblair | anteaya: waiting for ci-docs to run to publish the docs change | 17:21 |
fungi | afshaff: right, i think that's probably going to get ugly. instead we put shell blobs like that in self-contained builder macros and then call those from a template's builders list | 17:21 |
clarkb | nibalizer: at this point you have the repo in git with all of its existing history unchanged. Once that is done you update infra/config to consume that repo and git rm the modules from ifnra/config to preserve the history of the delete in infra/config | 17:21 |
clarkb | nibalizer: then you put .gitreview in the module repo on top of the existing history | 17:21 |
anteaya | jeblair: ah yes | 17:21 |
nibalizer | so use the 'upstream' keyword ? | 17:21 |
fungi | afshaff: and you could certainly use parameters to select a particular builder that way | 17:21 |
clarkb | nibalizer: yes | 17:21 |
nibalizer | and a github repo or something as an intermediary | 17:22 |
*** Ajaeger1 has joined #openstack-infra | 17:22 | |
krotscheck | clarkb: So how would we do that if the repo already exists? | 17:22 |
krotscheck | clarkb: i.e. it was already created without an upstream in review.projects.yaml | 17:22 |
clarkb | nibalizer: its too late to do that now for puppet storyboard. So an infra root can do it manually, however it is too late to do that for this repo without forcing everyone who has cloned it to git reset --hard | 17:22 |
*** praneshp has joined #openstack-infra | 17:22 | |
nibalizer | okay well we can do that for all the other ones | 17:22 |
fungi | clarkb: well, technically... | 17:23 |
fungi | clarkb: we can pull the filtered repo into a clone of the current one as a local branch, merge from it into master and then force-push that so it's ff-able | 17:23 |
*** mpaolino has quit IRC | 17:23 | |
clarkb | fungi: I think that is more work than just reviewing those changes | 17:23 |
fungi | clarkb: agreed. thus the "technically" ;) | 17:24 |
clarkb | my concern is that I actually want to do the above process with our guinea pig | 17:24 |
nibalizer | also those changes were already reviewed | 17:24 |
*** vermarah has quit IRC | 17:24 | |
clarkb | nibalizer: no they werent | 17:24 |
krotscheck | clarkb fungi: it’s an option to keep in mind if this happens again though. | 17:24 |
clarkb | nibalizer: the rebasing stuff under them makes gerrit treat them as different | 17:24 |
clarkb | nibalizer: I get that the patches were reviewed but I still have to review them | 17:24 |
nibalizer | okay | 17:24 |
nibalizer | well thats gonna be a giant pain | 17:24 |
nibalizer | also | 17:24 |
nibalizer | if you have any feedback, thats not really the place to provide it | 17:24 |
clarkb | why not? its a new repo? | 17:25 |
nibalizer | so the act of viewing and reading and commenting doesn't make sense | 17:25 |
clarkb | it totally does | 17:25 |
nibalizer | on commit 7 out of 15? | 17:25 |
clarkb | sure | 17:25 |
* krotscheck only really wants the puppet module to get used so he can get rabbit onto storyboard before he leaves on vacation. | 17:25 | |
clarkb | the key thing is we ditched history | 17:25 |
clarkb | so the new history is udner review and scrutiny imo | 17:25 |
nibalizer | the history is copied | 17:26 |
nibalizer | krotcheck is still the author | 17:26 |
nibalizer | im just the comitter on most of those | 17:26 |
clarkb | nibalizer: I understand | 17:26 |
nibalizer | clarkb: the way i see it is you're being inconsistent | 17:26 |
clarkb | nibalizer: but from a git/gerrit perspective its fresh unreviewed changes | 17:26 |
*** markmc has quit IRC | 17:27 | |
*** arosen1 has joined #openstack-infra | 17:27 | |
*** arosen1 is now known as arosen-home | 17:27 | |
nibalizer | they way you want to do this, huck it all in with a shovel, gives no opportunity for code review. but now you're saying that you want to rigourously review and ask for changes on the storyboard history | 17:27 |
clarkb | so if we are building a new history in review making changes is fair game | 17:27 |
fungi | this is, for example, similar to cherry-picked backports between projects or between branches... we re-review those | 17:27 |
clarkb | I am not saying that will happen just that I don't have an issue with it | 17:27 |
afshaff | fungi: I started playing around with the builder macros, and I guess my hope was to have each {repo}'s build script in a separate file. But if I had one large yaml file in the config directory, I could put all of my per-repo build scripts as builder macros in that file and then just use template variables normally to select the correct builder? It sounds like that is what you're suggesting | 17:27 |
clarkb | nibalizer: the other way preserves exactly the tree that was reviewed previously | 17:27 |
clarkb | nibalizer: there is nothing to rereview in that case | 17:27 |
fungi | nibalizer: most of the issue is that the git shas change, so there's no way to be *sure* they're identical without a fair amount of extra effort | 17:28 |
nibalizer | so what is the difference between that three and this tree | 17:28 |
krotscheck | clarkb: not necessarily, since once you pull things into a separate independent repo the owner has the power to rewrite all kinds of things in the history. | 17:28 |
clarkb | nibalizer: the difference is you put stuff under the tree | 17:28 |
nibalizer | fungi: the git shas change under subtree and and filter branch too | 17:28 |
jesusaurus | it sounds like the storyboard module is not a very good guinea pig because it already exists as its own project | 17:28 |
clarkb | jesusaurus: right | 17:28 |
clarkb | jesusaurus: and thats all I am really saying | 17:29 |
fungi | nibalizer: yep, but it's one command anyone can run on their local clone of the tree to compare, and the resulting shas should match | 17:29 |
*** amcrn has quit IRC | 17:29 | |
* krotscheck patiently waits for the argument to settle so we can get back to getting rabbit onto storyboard. | 17:30 | |
nibalizer | haha | 17:30 |
clarkb | nibalizer: krotscheck: I think we live with puppet-storyboard as is, the new code gets reviewed and we moev on | 17:30 |
fungi | it's also a lot faster for the original proposal than trying to hunt down the specific patches which made up those files | 17:30 |
clarkb | nibalizer: krotscheck but we don't treat this as a guinea pig for the process | 17:30 |
clarkb | because this will be a snowflake | 17:30 |
nibalizer | okay | 17:30 |
*** cwchang has joined #openstack-infra | 17:30 | |
jeblair | clarkb: yeah, i'm pretty sure we agreed on that at the meeting where we discussed this. | 17:30 |
nibalizer | we can use github and the 'upstream' keyword on the next one | 17:30 |
*** AlexF has quit IRC | 17:31 | |
cwchang | clarkb: I | 17:31 |
clarkb | jeblair: oh I thought in the last meeting we said this was the one | 17:31 |
*** Ryan_Lane has quit IRC | 17:31 | |
nibalizer | jeblair: i think i had a different plan than you all did, and we didn't discuss in great enough detail to make that clear | 17:31 |
jeblair | nope, here's the process described in detail, as it should be, in the spec: https://review.openstack.org/#/c/99990/2/specs/puppet-modules.rst | 17:32 |
jeblair | that's clearly not the the process for storyboard, since it already exists | 17:32 |
cwchang | clarkb: hi, i have cleaned up the source repository for https://review.openstack.org/#/c/102662/ | 17:32 |
jeblair | what we discussed in the meeting was basically "stabilize it in-tree, port all the changes to the external tree, switch" | 17:32 |
krotscheck | Ok, sooooooo about that review chain for puppet-storyboard? https://review.openstack.org/#/c/100370/ :D | 17:33 |
nibalizer | ah oaky well my bad, sorry guys | 17:33 |
openstackgerrit | A change was merged to openstack/requirements: Add oslo.serialization to projects.txt https://review.openstack.org/112993 | 17:33 |
openstackgerrit | Oscar Romero (homeless) proposed a change to openstack-infra/jeepyb: Fix close_pull_requests from jeepyb https://review.openstack.org/113598 | 17:33 |
clarkb | cwchang: great, just remove your WIP vote when you are ready for it to be reviewed | 17:33 |
jeblair | krotscheck: yes, so let's proceed with the move to the module, and discuss and agree on the procedure for future extractions in the spec review | 17:33 |
jeblair | clarkb, nibalizer: ^ | 17:33 |
clarkb | ++ | 17:34 |
jesusaurus | jeblair: ++ | 17:35 |
nibalizer | ++ | 17:35 |
cwchang | clarkb: are we still waiting on waiting for fungi's https://review.openstack.org/#/c/103355/15 to resolve conflict ? | 17:36 |
clarkb | cwchang: that change merged, but your change was in enough of a holding pattern that we didn't push a rebase | 17:36 |
fungi | cwchang: it merged a week ago | 17:36 |
jeblair | clarkb, fungi, nibalizer, jesusaurus: who's reviewing the puppet-storboard series? | 17:37 |
*** e0ne has joined #openstack-infra | 17:38 | |
clarkb | cwchang: it may actually not need a rebase, but it would be nice to remove the redundant github-org field | 17:38 |
*** atiwari has joined #openstack-infra | 17:38 | |
clarkb | jeblair: I was willing to review it | 17:38 |
nibalizer | I submitted it, so generally I wouldn't be reviewing it | 17:38 |
clarkb | jeblair: I reviewed a few of the original patches infra/config side | 17:38 |
jeblair | clarkb: okay; i'm going to do a cursory review and +2 it; it sounds like the +3 is up to you. | 17:39 |
nibalizer | i also want to apologize for getting a little heated up there ^^ it comes down to i didn't read the spec very closely and now i feel like an idiot :/ | 17:39 |
*** hdd has quit IRC | 17:39 | |
clarkb | jeblair: wfm | 17:39 |
clarkb | jeblair: if you are reviewing things the grenade sideways changes are all up | 17:40 |
clarkb | I think sdague is happy with them now too so ready to go | 17:40 |
cwchang | clarkb: how do I remove WIP vote and "githu-org" ? | 17:40 |
jeblair | clarkb: ok thanks. i'm not quite there yet but will try to soon. | 17:40 |
*** ramashri has quit IRC | 17:40 | |
clarkb | cwchang: you have to push a new patchset to update the projects.yaml file to remove that line. To remove the WIP vote you need to comment on the change and set your vote to 0 instead of -1 work in progress. | 17:41 |
Ajaeger1 | Alex_Gaynor, dhellmann or any other requirements core: Please approve https://review.openstack.org/#/c/113080/ to unbreak the requirements post job. It has already two +2s. | 17:41 |
dhellmann | Ajaeger1: +2a | 17:42 |
*** ramashri has joined #openstack-infra | 17:42 | |
Ajaeger1 | dhellmann: Thanks! | 17:42 |
krotscheck | clarkb jeblair: Everything up to the rabbitmq patch is old history. Anything after that needs a real review. | 17:42 |
clarkb | krotscheck: thanks | 17:44 |
*** pelix has quit IRC | 17:44 | |
*** e0ne has quit IRC | 17:44 | |
openstackgerrit | A change was merged to openstack-infra/puppet-storyboard: Adding Rakefile https://review.openstack.org/100370 | 17:45 |
jeblair | krotscheck: why no default for $rabbitmq_user_password in application.pp? | 17:45 |
jeblair | (there is one for mysql password) | 17:46 |
clarkb | jeblair: I made a comment about that. not having a default forces you to provide a value | 17:46 |
cwchang | clarkb: I looked at my project.yaml file section for puppet-n1k-vsm... right now the github-org says stackforge. I am seeing openstack openstack-dev openstack-infra for different values.. | 17:47 |
clarkb | jeblair: that is better than possibly setting a public password imo | 17:47 |
clarkb | cwchang: if you update to the latest version of that fiel you will see they have been removed | 17:47 |
cwchang | clarkb: which should be the right value there ? | 17:47 |
jeblair | clarkb: okay, it's an unexplained pattern break though. | 17:47 |
openstackgerrit | A change was merged to openstack-infra/puppet-storyboard: Add Storyboard puppet module https://review.openstack.org/100321 | 17:47 |
cwchang | clarkb: aha | 17:47 |
*** tonytan4ever has joined #openstack-infra | 17:47 | |
jeblair | clarkb: and the default invocation of it provides a default | 17:48 |
jeblair | clarkb: it's starting to look like an omission | 17:48 |
*** linkid has left #openstack-infra | 17:48 | |
tonytan4ever | Hi all, I am trying to store contact information for my openstack account, and I cannot. | 17:48 |
clarkb | nibalizer: ^ this is why it is hard to review :) | 17:49 |
jeblair | clarkb: also, i don't see your comment on it | 17:49 |
tonytan4ever | can somebody help me with that ? It prevents me from submitting PR | 17:49 |
clarkb | jeblair: its on the infra/config change | 17:49 |
clarkb | tonytan4ever: have you joined the openstack foundation | 17:49 |
jeblair | oh, i thought this was the first new change | 17:49 |
clarkb | tonytan4ever: doing so is a requirement | 17:49 |
clarkb | jeblair: I think it is new in that the infra/config change did not merge | 17:49 |
clarkb | jeblair: but it has been reviewed over there | 17:49 |
tonytan4ever | I don't recall that. How can I join ? | 17:49 |
jroll | can I please bug a devstack core to take a look at these two reviews? trying to get check jobs going for a new ironic driver. :) https://review.openstack.org/#/c/112095/ and https://review.openstack.org/#/c/108457/ | 17:50 |
jeblair | clarkb: it appears to be under a different change id | 17:50 |
nibalizer | clarkb: sure, and the next one will be slurped in | 17:50 |
nibalizer | my bad | 17:50 |
*** annegent_ has quit IRC | 17:50 | |
clarkb | jeblair: yes, now you know why I was a bit confused earlier :) | 17:50 |
clarkb | tonytan4ever: https://wiki.openstack.org/wiki/How_To_Contribute#If_you.27re_a_developer describes the process | 17:50 |
*** radez is now known as radez_g0n3 | 17:51 | |
jeblair | clarkb, krotscheck: okay, we landed https://review.openstack.org/113616 which is what we agreed on at the meeting; i think we're doing the cutover now, so i am going to -2 infra/config changes to the storyboard module | 17:51 |
openstackgerrit | A change was merged to openstack-infra/puppet-storyboard: Added Authorization Header flag to storyboard module https://review.openstack.org/100322 | 17:51 |
jeblair | clarkb: can you port your comments over to the new repo? | 17:51 |
clarkb | jeblair: sure | 17:52 |
krotscheck | jeblair: I think I already abandoned those. | 17:53 |
*** amcrn has joined #openstack-infra | 17:53 | |
jeblair | krotscheck: you did indeed | 17:53 |
jeblair | thanks | 17:54 |
*** chuckC has joined #openstack-infra | 17:54 | |
*** annegent_ has joined #openstack-infra | 17:54 | |
*** erlon has quit IRC | 17:55 | |
*** Alec-A2 has quit IRC | 17:55 | |
*** portante has quit IRC | 17:55 | |
*** annegen__ has joined #openstack-infra | 17:55 | |
*** erlon has joined #openstack-infra | 17:56 | |
*** Alec-A2 has joined #openstack-infra | 17:56 | |
jeblair | clarkb: ah, it's unset because you asked that it be. we now have contradictory -1s. | 17:56 |
*** flwang_ has joined #openstack-infra | 17:56 | |
clarkb | jeblair: yes | 17:56 |
*** esker has joined #openstack-infra | 17:56 | |
clarkb | jeblair: I want there to be no default password | 17:56 |
tonytan4ever | clarkb: I registered to openstack foundation, and it still don't work for me. | 17:56 |
openstackgerrit | A change was merged to openstack-infra/puppet-storyboard: Make storyboard run over ssl https://review.openstack.org/100323 | 17:56 |
*** radez_g0n3 is now known as radez | 17:56 | |
clarkb | tonytan4ever: you need to ensure that your email address in your foundation profile matches your address in gerrit | 17:56 |
clarkb | tonytan4ever: that is the key used to match entries | 17:57 |
jeblair | clarkb, krotscheck: i'm okay with the 'leave unset to avoid a default password' pattern. however, i think it should be a real pattern applied everywhere. i'd recommend we reinstate the default password for rabbitmq, and then do a new change that does that comprehensively. | 17:57 |
clarkb | jeblair: the problem with default passwords is you say "changeme" but nothing enforces that. Not having a password does enforce the value be set | 17:57 |
*** bhuvan_ has joined #openstack-infra | 17:57 | |
jeblair | clarkb: yep. i'm cool with that. but just dropping that one instance of it does not actually accomplish the goal, and i would rather not fool ourselves into thinking we have. | 17:58 |
clarkb | jeblair: ok | 17:58 |
jeblair | clarkb: (specifically because the very invocation of it within the module supplies a default, so most users will never notice that) | 17:58 |
clarkb | jeblair: I think I did mention consistency in my comments too saying I didn't expect tis to happen in that change | 17:58 |
jeblair | woo, we actually agree :) | 17:58 |
*** datsun180b has quit IRC | 17:59 | |
tonytan4ever | clarkb: OK. tried with the correct email addres and it worked. Thanks. | 17:59 |
clarkb | tonytan4ever: great | 17:59 |
*** annegent_ has quit IRC | 17:59 | |
*** arnaud has quit IRC | 17:59 | |
*** e0ne has joined #openstack-infra | 17:59 | |
*** ociuhandu has quit IRC | 17:59 | |
*** flwang_ has quit IRC | 18:00 | |
*** bhuvan has quit IRC | 18:01 | |
clarkb | krotscheck: nibalizer in https://review.openstack.org/#/c/100325/7 where does the superusersfile come from? | 18:01 |
*** maxbit has quit IRC | 18:01 | |
*** maxbit has joined #openstack-infra | 18:01 | |
*** shayneburgess has joined #openstack-infra | 18:01 | |
clarkb | I guess that is just a utility change to allow you to hcange it | 18:01 |
openstackgerrit | A change was merged to openstack-infra/puppet-storyboard: Load storyboard projects from projects.yaml https://review.openstack.org/100324 | 18:02 |
*** portante has joined #openstack-infra | 18:02 | |
openstackgerrit | A change was merged to openstack-infra/puppet-storyboard: Load storyboard superusers from yaml file https://review.openstack.org/100325 | 18:03 |
*** hdd has joined #openstack-infra | 18:04 | |
mriedem | what is this new 'toggle ci' button? | 18:06 |
openstackgerrit | A change was merged to openstack-infra/config: Add CI results table to Gerrit https://review.openstack.org/105860 | 18:07 |
openstackgerrit | A change was merged to openstack-infra/puppet-storyboard: Storyboard token expiration increased https://review.openstack.org/100326 | 18:07 |
openstackgerrit | A change was merged to openstack-infra/puppet-storyboard: Make storyboard idempotent https://review.openstack.org/100327 | 18:07 |
*** andreykurilin_ has quit IRC | 18:08 | |
jeblair | mriedem: it hides the results from third-party ci systems | 18:08 |
clarkb | mriedem: its js to filter out CI comments so that reviewers can focus on human interaction. It will still give you a handy collapsed set of info from ci | 18:08 |
fungi | mriedem: also, you are very quick to notice changes ;) | 18:08 |
openstackgerrit | A change was merged to openstack-infra/puppet-storyboard: Put a default section header above the TTL https://review.openstack.org/100328 | 18:08 |
*** jistr has quit IRC | 18:08 | |
openstackgerrit | A change was merged to openstack-infra/puppet-storyboard: StoryBoard module bounces apache on update https://review.openstack.org/100329 | 18:08 |
reed | what a week! | 18:08 |
mriedem | fungi: when did it show up? | 18:08 |
*** yfried has joined #openstack-infra | 18:08 | |
jeblair | clarkb, fungi: should we change 'Jenkins' to 'OpenStack CI', or add 'Jenkins' as a special case to the ci results table? | 18:09 |
fungi | jeblair: i think maybe first the second, then second the first (adter some warning) | 18:09 |
*** pblaho has quit IRC | 18:09 | |
fungi | s/adter/after? | 18:09 |
fungi | gah, fingers no tyep gud | 18:09 |
clarkb | ya I think we want to be OpenStack CI eventually but doing so right now might cause confusion | 18:10 |
clarkb | so second then first seems good | 18:11 |
krotscheck | clarkb: puppet:///modules/openstack_project/storyboard/superusers.yaml | 18:11 |
fungi | basically lots of people, myself included) have that name hard-coded in personal places | 18:11 |
clarkb | krotscheck: gotcha thanks | 18:11 |
clarkb | fungi: jeblair I haev held 1450575 in an effort to debug dfw centos6 | 18:12 |
fungi | (mail filters particularly) | 18:12 |
krotscheck | clarkb: Also, that was moved in the big refactor into being a standalone class. | 18:12 |
clarkb | it is still running a test though I think | 18:12 |
jeblair | oh, actually we don't need to special case it... | 18:13 |
fungi | oh? | 18:13 |
jeblair | we only need to do that if we want its comments hidden; it shows up in the table based on whether it has matching result comments | 18:13 |
fungi | true. i assumed you meant to be able to hide its comments | 18:13 |
fungi | but perhaps we just announce that we'll rename it at a certain date/time and live with that until | 18:14 |
jeblair | i thought it was more urgent (i thought we were about to hide the comments from it and _not_ display the results in the table) | 18:14 |
clarkb | krotscheck: ya thats the change I am reviewing now | 18:14 |
clarkb | krotscheck: it is big | 18:14 |
clarkb | jeblair: oh :) | 18:14 |
clarkb | that would be bad | 18:15 |
openstackgerrit | James E. Blair proposed a change to openstack-infra/config: Special case 'Jenkins' in hideci. https://review.openstack.org/113984 | 18:15 |
jeblair | but i don't think that's the case | 18:15 |
jeblair | anyway, there's the change if we want to go ahead and do it. | 18:15 |
fungi | clarkb: so... on the snapshots front we only have two each showing in nova image-list for bare-centos6, bare precise and bare-trusty within rax-dfw. if it's using an old snapshot it's not one the api is telling us exists | 18:17 |
clarkb | fungi: shiny | 18:17 |
clarkb | fungi: so maybe a hypervisor cache is biting us | 18:18 |
fungi | clarkb: right, doesn't rule that out obviously | 18:18 |
clarkb | fungi: the node I held looks fine though.. at least from a which yum front and uname -a | 18:18 |
* clarkb looks at pep8 script | 18:18 | |
fungi | but this means we want to see if we can dig up enough detail on each instance where a failure was observed and see whether phschwartz can map them to a common compute node | 18:18 |
clarkb | run-pep8.sh looks good too | 18:19 |
*** dims_ has quit IRC | 18:19 | |
fungi | i'll fish instance uuids out of the nodepool logs for these nodes | 18:19 |
*** dims_ has joined #openstack-infra | 18:19 | |
krotscheck | clarkb: Yeah, well, I felt it was necessary once putting it on puppetforge became a possibility :) | 18:19 |
jesusaurus | clarkb: fungi: jeblair: feel like giving https://review.openstack.org/#/c/111600/ a quick review? it has 2 +2s | 18:20 |
clarkb | fungi: ok | 18:20 |
jeblair | anteaya: the 3rd party docs update has been published; sending email now | 18:20 |
*** dims_ has quit IRC | 18:21 | |
*** Alexei_987 has joined #openstack-infra | 18:21 | |
clarkb | jesusaurus: I think they already have timestamps, unless we are providing a different logging config. that could explain it | 18:21 |
fungi | jesusaurus: at the risk of asking a stupid question, why is that needed? i see timestamps on our nodepool logs | 18:21 |
fungi | jinx | 18:21 |
*** dims has joined #openstack-infra | 18:21 | |
* fungi looks at the logging config | 18:22 | |
fungi | yep | 18:22 |
fungi | [formatter_simple] | 18:22 |
fungi | format=%(asctime)s %(levelname)s %(name)s: %(message)s | 18:22 |
*** _nadya_ has joined #openstack-infra | 18:22 | |
fungi | that a'splains it | 18:22 |
*** hashar has joined #openstack-infra | 18:22 | |
jeblair | i believe these are for cases that aren't handled by that | 18:22 |
clarkb | ya | 18:22 |
fungi | agreed | 18:23 |
openstackgerrit | ChingWei Chang proposed a change to openstack-infra/config: new project puppet-n1k-vsm creation https://review.openstack.org/102662 | 18:23 |
jeblair | for nodepoold, if there is no logging config file, this sets the default | 18:23 |
jeblair | for nodepool commands, there is no logging config, so it always applies there | 18:23 |
fungi | seems like a sane enough default behavior too | 18:23 |
jeblair | i wonder though if it should not match what we have configured? | 18:23 |
*** krotscheck has left #openstack-infra | 18:23 | |
openstackgerrit | A change was merged to openstack-infra/puppet-storyboard: Modularized StoryBoard Module https://review.openstack.org/113724 | 18:23 |
*** krotscheck has joined #openstack-infra | 18:24 | |
jeblair | i think ours is a more typical python log line | 18:24 |
fungi | jeblair: yeah, would allow us to remove out formatter sections too possiblt | 18:24 |
fungi | my t key is in a gravity well today or something | 18:24 |
*** krotscheck has quit IRC | 18:24 | |
openstackgerrit | A change was merged to openstack-infra/puppet-storyboard: Fixed location of storyboard configuration file https://review.openstack.org/113725 | 18:24 |
openstackgerrit | A change was merged to openstack-infra/puppet-storyboard: Create storyboard configuration directory https://review.openstack.org/113726 | 18:25 |
*** krotscheck has joined #openstack-infra | 18:25 | |
clarkb | we lost krotscheck, I have ogtten through the storyboard changes up to the rabbitmq change | 18:25 |
clarkb | now to review the zuul trigger change | 18:25 |
*** _nadya_ has quit IRC | 18:26 | |
jeblair | what's this? http://puppetboard.openstack.org/node/ipa.openstack.org | 18:26 |
*** arnaud has joined #openstack-infra | 18:26 | |
clarkb | I do not know | 18:27 |
krotscheck | I’m back! | 18:27 |
fungi | i can log into it | 18:27 |
clarkb | it is a thing | 18:27 |
fungi | up 38 days | 18:27 |
clarkb | krotscheck: got through the storyboard stack | 18:27 |
clarkb | krotscheck: up to the rabbitmq change | 18:27 |
krotscheck | clarkb: Saw that, thanks :) | 18:27 |
*** otherwiseguy has quit IRC | 18:27 | |
*** imeerovi has quit IRC | 18:28 | |
fungi | running fedora 20 | 18:28 |
fungi | something testing ironic python agent? | 18:28 |
jesusaurus | clarkb: fungi: yeah, it adds timestamps to the default format rather than requiring a logging config on the system to manage that | 18:28 |
krotscheck | clarkb: Ok, so we’re going to set a default password, and then add a patch later? | 18:28 |
clarkb | krotscheck: ya | 18:28 |
jeblair | there is a command in root's bash history on ci-puppetmaster to create it | 18:28 |
krotscheck | kk | 18:28 |
krotscheck | on it | 18:28 |
fungi | mordred has been ssh'ing into it a bunch | 18:29 |
clarkb | krotscheck: that way the module is internally consistent | 18:29 |
fungi | back on july 6 | 18:29 |
clarkb | looks like when it was booted? | 18:29 |
fungi | yeah | 18:29 |
fungi | perhaps its name stands for internet protocol ale. it's been fermenting for over a month now | 18:30 |
jeblair | mordred: care to share? | 18:30 |
krotscheck | clarkb jeblair: Default applied. I also have a few other things I want to land in there, so I’ll handle the default password change while I’m in there. | 18:32 |
*** dims_ has joined #openstack-infra | 18:32 | |
*** tonytan_brb has joined #openstack-infra | 18:32 | |
*** praneshp_ has joined #openstack-infra | 18:33 | |
*** pcm__ has joined #openstack-infra | 18:34 | |
jeblair | i'm going to restart gerrit now to pick up the hideci changes | 18:34 |
*** rakhmerov_ has joined #openstack-infra | 18:34 | |
*** rakhmerov has quit IRC | 18:34 | |
*** miqui_ has joined #openstack-infra | 18:34 | |
*** froek_ has joined #openstack-infra | 18:35 | |
*** tonytan_brb has quit IRC | 18:36 | |
*** julim_ has joined #openstack-infra | 18:36 | |
jeblair | done. check out https://review.openstack.org/#/c/112733/4 | 18:36 |
krotscheck | clarkb: For some reason melody is now blocking port 29418 (gerrit) | 18:36 |
*** Hefeweiz1n has joined #openstack-infra | 18:36 | |
jeblair | krotscheck: can you provide more context? | 18:37 |
*** rakhmerov_ is now known as rakhmerov | 18:37 | |
fungi | clarkb: what was the centos6 dfw bug number? | 18:38 |
*** dustins_ has joined #openstack-infra | 18:38 | |
clarkb | fungi: I don't remember but can look for it | 18:39 |
fungi | clarkb: no worries, i'll find it | 18:39 |
krotscheck | jeblair: Sorry, it’s an in-hp-office only thing. Probably not relevant to this channel. | 18:39 |
*** dmsimard_ has joined #openstack-infra | 18:40 | |
clarkb | krotscheck: talk to chris | 18:40 |
clarkb | he sits on the south corner of the building near yazz iirc | 18:40 |
jeblair | krotscheck: ok. though, double check -- it's an interesting coincidence that you reported that right as i was restarting gerrit. :) | 18:40 |
krotscheck | clarkb: Already did, things have been fixed. | 18:40 |
krotscheck | Oh! | 18:40 |
krotscheck | Well then | 18:40 |
krotscheck | That was probably why | 18:40 |
*** portante_ has joined #openstack-infra | 18:40 | |
*** arnaud has quit IRC | 18:41 | |
*** tonytan4ever has quit IRC | 18:41 | |
*** praneshp has quit IRC | 18:41 | |
*** dustins has quit IRC | 18:41 | |
*** julim has quit IRC | 18:41 | |
*** dmsimard has quit IRC | 18:41 | |
*** froek has quit IRC | 18:41 | |
*** dims has quit IRC | 18:41 | |
*** portante has quit IRC | 18:41 | |
*** pcm_ has quit IRC | 18:41 | |
*** openstackgerrit has quit IRC | 18:41 | |
*** miqui has quit IRC | 18:41 | |
*** mrodden has quit IRC | 18:41 | |
*** Hefeweizen has quit IRC | 18:41 | |
*** timrc has quit IRC | 18:41 | |
*** praneshp_ is now known as praneshp | 18:41 | |
*** dmsimard_ is now known as dmsimard | 18:41 | |
*** froek_ is now known as froek | 18:41 | |
fungi | clarkb: for reference, bug 1355907 | 18:41 |
uvirtbot | Launchpad bug 1355907 in openstack-ci ""sudo: yum: command not found" in py26 jobs (bare-centos6-rax* nodes)" [Undecided,New] https://launchpad.net/bugs/1355907 | 18:41 |
clarkb | jeblair: toggling works! | 18:42 |
*** mrodden has joined #openstack-infra | 18:42 | |
clarkb | jeblair: I assume we don't see a summary of the various third party CIs because they don't follow the report comment format? | 18:42 |
pleia2 | jeblair: neat | 18:42 |
clarkb | jeblair: on that change you linked specifically | 18:42 |
russellb | so that must be new? | 18:43 |
*** timrc has joined #openstack-infra | 18:43 | |
russellb | just saw it, cool :) | 18:43 |
*** sabeen3 has joined #openstack-infra | 18:43 | |
clarkb | oh wait their it is | 18:43 |
clarkb | I was looking in the wrong place | 18:43 |
clarkb | so shiny | 18:43 |
*** sabeen1 has quit IRC | 18:44 | |
clarkb | jeblair: your special case jenkins change lgtm | 18:44 |
fungi | clarkb: and also bug 1354829 | 18:45 |
uvirtbot | Launchpad bug 1354829 in tempest ""sudo: 3 incorrect password attempts" in host setup" [Undecided,Invalid] https://launchpad.net/bugs/1354829 | 18:45 |
clarkb | fungi: thanks | 18:45 |
clarkb | fungi: do you want me to keep the centos6 node i haev held? I will delete it shortly if not | 18:45 |
clarkb | fungi: I don't think it has much info to give us | 18:45 |
*** hashar has quit IRC | 18:46 | |
fungi | clarkb: probably not. it seems holding a random node there only tells us it's not all nodes in that region | 18:46 |
clarkb | fungi: ya | 18:46 |
dansmith | jeblair: dare I ask the chance of seeing that on the new review format view? :) | 18:47 |
*** jistr has joined #openstack-infra | 18:47 | |
*** openstackgerrit has joined #openstack-infra | 18:50 | |
clarkb | dansmith: I think you just need to do a conditional switch in the js based on which page it is loaded in, then toggle the correct objects | 18:50 |
clarkb | dansmith: but I know little of js in browsers | 18:50 |
phschwartz | fungi: If you can get me a list of uuid's I can look up info on them | 18:50 |
jeblair | dansmith: it's supposed to; if it isn't showing up it's a bug | 18:51 |
fungi | phschwartz: yep, assembling that now | 18:51 |
clarkb | phschwartz: fungi: 9351cb5a-f399-4788-9809-e10d35921d0b is a uuid of a good node | 18:51 |
dansmith | jeblair: lemme make sure I'm not caching | 18:51 |
clarkb | jeblair: dansmith though you may need to hard refresh | 18:51 |
phschwartz | fungi: and sorry for a delay, I went to get lunch :) | 18:51 |
dansmith | jeblair: yeah, there it is | 18:51 |
dansmith | jeblair: if you were here, I'd hug you inappropriately | 18:52 |
clarkb | phschwartz: I should do that too. I still have left over pulled pork | 18:52 |
fungi | phschwartz: i do that sometimes too--don't apologize! ;) | 18:52 |
clarkb | been eating it for 3 days now | 18:52 |
clarkb | dansmith: woah there | 18:52 |
phschwartz | clarkb: I wish I had pulled pork. lol. I was lazy and ran to Chipotle | 18:52 |
russellb | dansmith: and you wonder why i'm against so many meetups | 18:52 |
dansmith | russellb: heh | 18:52 |
jeblair | dansmith: thanks :) we should also thank Radoslav Gerganov <rgerganov@vmware.com> who wrote the toggle button (i just did the table) | 18:53 |
*** hashar has joined #openstack-infra | 18:54 | |
*** krotscheck has quit IRC | 18:54 | |
fungi | phschwartz: so... here's the list of instance uuids i see associated with bug 1355167 and the approximate dates/times they were booted... http://paste.openstack.org/show/94553 | 18:54 |
uvirtbot | Launchpad bug 1355167 in openstack-ci "run-pep8.sh usage now being output" [Critical,In progress] https://launchpad.net/bugs/1355167 | 18:54 |
fungi | phschwartz: if my theory is correct, we can repeat it with the couple other bugs i think might have a similar underlying cause | 18:54 |
dansmith | jeblair: cool, yeah, awesome to see this evolution | 18:54 |
*** krotscheck has joined #openstack-infra | 18:55 | |
*** _nadya_ has joined #openstack-infra | 18:55 | |
*** tonytan4ever has joined #openstack-infra | 18:56 | |
clarkb | jeblair: the zuul trigger tests are in a separate test file and class :) | 18:56 |
jeblair | clarkb: neat huh? :) (thanks!) | 18:57 |
fungi | clarkb: i'm assuming that's because emacs threw an error about the old file containing too many lines | 18:57 |
jeblair | fungi: on the contrary, i seem to be the only one who doesn't mind 10,000 line files :) | 18:57 |
fungi | heh | 18:57 |
*** mmaglana_ has quit IRC | 18:58 | |
clarkb | fungi: 10k line files is how emacs gets to eight megs and counting | 18:59 |
clarkb | jeblair: note I am reviewing these two chanes with a bit of last nights context missing as I wasn't quite able to follow the entire discussion | 19:01 |
*** ildikov has joined #openstack-infra | 19:02 | |
phschwartz | fungi: so it looks like for most of those the last update of the image in glance was on the 9th or 10th | 19:03 |
phschwartz | hmm, that is interesting considering the instances where on the 7th. | 19:03 |
phschwartz | I think nodepool created new images from an older bad image | 19:04 |
fungi | phschwartz: for a little background, we're seeing random instances boot from snapshots containing files which shouldn't have been in that state for more than a week prior | 19:04 |
phschwartz | That would be all that I could think of. Used the one on the 7th to create a new image that was used for the next couple of days | 19:04 |
phschwartz | hmm | 19:05 |
fungi | phschwartz: did they all boot on the same nova compute host by any chance? | 19:05 |
phschwartz | sec | 19:05 |
fungi | phschwartz: basically none of the snapshots we see in nova image-list are old enough to have contained those files, and most instances booted from them in that region have the correct (newer) contents | 19:06 |
phschwartz | so far no, they look like different computes in different cells | 19:06 |
clarkb | but they also do weird tings like not have yum in /usr/bin | 19:06 |
fungi | clarkb: yeah, that first uuid list was only for the ones which hit outdated run-pep8.sh contents | 19:07 |
fungi | i was going to do separate lists for the other two possibly related bugs in case they're actually separate problems/causes | 19:08 |
*** ildikov has quit IRC | 19:08 | |
fungi | the pep8 one was just a fairly obvious first case because it's a smaller list and it's really got to be something causing an old version of that file to exist | 19:08 |
*** dmsimard has quit IRC | 19:10 | |
*** ildikov has joined #openstack-infra | 19:10 | |
phschwartz | All of the uuid's in that list span 3 cells and no 2 are on the same compute | 19:11 |
*** dmsimard_away has joined #openstack-infra | 19:11 | |
*** dmsimard_away has quit IRC | 19:11 | |
phschwartz | And all apear to be spawned from a different image ref id | 19:11 |
*** dmsimard_away has joined #openstack-infra | 19:12 | |
*** dmsimard_away is now known as dmsimard | 19:12 | |
phschwartz | Let me look closer at glance | 19:13 |
clarkb | jeblair: zuul changes reviewed. -1 is not for the break thing, it is for the unnecessary trigger method | 19:13 |
*** emagana has quit IRC | 19:13 | |
clarkb | jeblair: fungi: going to find food now. then hopefully back into reviews (I feel productive ish today | 19:14 |
*** emagana has joined #openstack-infra | 19:14 | |
clarkb | fungi: I have marked my held node for deletion | 19:14 |
jroll | ianw, jeblair, can I bug y'all to take a look at these two reviews? trying to get check jobs going for the ironic-python-agent driver. :) https://review.openstack.org/#/c/112095/ and https://review.openstack.org/#/c/108457/ | 19:15 |
*** yamamoto has quit IRC | 19:17 | |
clarkb | jeblair: pleia2: are we happy enough with the solution for local accounts to demo pootle 2.6.0-rc1? pleia2 have you run into any more problems? | 19:17 |
clarkb | jeblair: pleia2: mostly asking because i can change the DNS records to point at the new host if we are happy with it | 19:18 |
fungi | phschwartz: yeah, some of those were booted from precise snapshots and some from trusty snapshots, so wouldn't all be from the same snapshot | 19:18 |
*** emagana has quit IRC | 19:18 | |
phschwartz | even if for 2 that were both precise, they were different image id's so looks like different snapshots | 19:19 |
fungi | phschwartz: but the only explanation i can reasonably imagine for the errors we saw on them is that they were booted from older snapshots than we requested through the nova api, and in fact snapshots that we couldn't even see in nova image-list at all | 19:19 |
phschwartz | I have asked one of our glance devs to take a look with me as it looks correct and like there shouldn't have been an issue | 19:19 |
mordred | jeblair: that was me poking at freeipa - can be deleted - sorry, meant to clean that up a while ago | 19:20 |
*** ociuhandu has joined #openstack-infra | 19:23 | |
*** oanufriev has joined #openstack-infra | 19:24 | |
*** esker has quit IRC | 19:25 | |
pleia2 | clarkb: I think we're good to go, the only way to add more admins is by manual mysql update, right? | 19:25 |
*** vhoward has joined #openstack-infra | 19:25 | |
*** esker has joined #openstack-infra | 19:25 | |
*** e0ne has quit IRC | 19:25 | |
*** med_ has quit IRC | 19:26 | |
fungi | phschwartz: so just to double-check assumptions... the one which we saw today for instance 97078adc-8020-4ae5-b896-81548d323c4c should have been booted from image 98f9aff3-699f-4c33-a163-b41b8dc5483c | 19:26 |
clarkb | pleia2: ya, or I can add them on the command line if I register them. imo best to just have people sign up and I can toggle teh db | 19:26 |
pleia2 | clarkb: I'll need dwayne and daisy to have admin accounts, but they're both vacationing this week and our demo is sunday night our time | 19:26 |
clarkb | pleia2: so let me know if you need that done | 19:26 |
*** daya_k has quit IRC | 19:27 | |
clarkb | pleia2: all infra rooters have access to the machine too and can update accounts_user set is_superuser=1 where id=X; or some such to do it | 19:27 |
phschwartz | fungi: and according to the logs and statistics it was built from that image. | 19:27 |
pleia2 | ok, I'll email them both now to create accounts so hopefully they can get it done at a sane US time | 19:27 |
clarkb | pleia2: thanks | 19:28 |
pleia2 | hmm, my calendar invite is not right, this should actually be monday night | 19:29 |
fungi | phschwartz: and yet our logs indicate it booted with a file on it which was more than a week older content than other instances booted from that same snapshot | 19:30 |
*** radez is now known as radez_g0n3 | 19:30 | |
*** esker has quit IRC | 19:30 | |
*** armax has quit IRC | 19:31 | |
fungi | phschwartz: and so far the only common thread between the failures matching this pattern is that we only see them in dfw (equivalent systems in iad and ord don't exhibit this behavior) | 19:31 |
mordred | fungi: wow. the issue you are looking in to and the scrollback on it is ... fascinating | 19:31 |
*** e0ne has joined #openstack-infra | 19:31 | |
fungi | mordred: several seemingly disconnected bugs about files on nodepool nodes in states they should not be in, but all happening only in dfw. very odd indeed | 19:32 |
clarkb | and across node types | 19:32 |
clarkb | so no single image build on our side is to blame | 19:32 |
fungi | right, seen do far on bare-centos6, bare-precise and bare-trusty | 19:32 |
fungi | er, so far | 19:33 |
*** dtantsur is now known as dtantsur|afk | 19:33 | |
phschwartz | fungi: are the images that the uuid's are built off of not displaying in the image-list? | 19:33 |
openstackgerrit | A change was merged to openstack-infra/config: Add neutron full job to integrated gate https://review.openstack.org/113562 | 19:33 |
*** portante_ is now known as portante | 19:34 | |
fungi | phschwartz: the images we expect them to be booted from display in image-list, but every so often the instances have filesystem content inconsistent with what is on those images | 19:34 |
*** shashankhegde has quit IRC | 19:34 | |
fungi | phschwartz: observed so far: 1. some have a week-old copy of the run-pep8.sh script which we install on the snapshot, 2. some fail to be able to sudo as a user who is listed in the sudoers file on the snapshot, 3. some claim yum/rpm aren't installed | 19:36 |
fungi | phschwartz: all happening infrequently but for roughly a week as far as we can tell, and only in dfw | 19:36 |
krotscheck | nibalizer: I did a recheck no bug on https://review.openstack.org/#/c/113731/ and it passed | 19:36 |
openstackgerrit | A change was merged to openstack-infra/gitdm: Add Louis Taylor to HP https://review.openstack.org/113873 | 19:39 |
*** e0ne has quit IRC | 19:39 | |
openstackgerrit | Andreas Jaeger proposed a change to openstack-infra/config: Templatize puppet projects in layout.yaml https://review.openstack.org/114001 | 19:41 |
*** marun has quit IRC | 19:41 | |
*** Ryan_Lane has joined #openstack-infra | 19:43 | |
clarkb | actually I think for the afternoon I will be testing the non oslo prefixed projects on trusty and doing more tox 1.7.2 testing | 19:43 |
clarkb | so that I am ready for next week | 19:43 |
fungi | clarkb: a great idea | 19:43 |
clarkb | but do point out things that need review and I can context switch will I wait for tox to do its thing | 19:43 |
*** gema_ has joined #openstack-infra | 19:44 | |
phschwartz | fungi: ok, no answer yet and not sure how long it would be. We are pulling in a wider group of people to look at and discuss the issue | 19:44 |
clarkb | fungi: phschwartz: would it maybe help on our end if we deleted all of our dfw images and just start over? | 19:45 |
anteaya | jeblair: k, thanks | 19:45 |
clarkb | fungi: phschwartz then if it happens we have a pretty good idea of where it could be beraking | 19:45 |
fungi | phschwartz: understood. we're in no real hurry, it's just a very perplexing set of symptoms and seems like something you (rackspace) should be aware of since it likely has wider implications than just our tenant | 19:45 |
phschwartz | fungi: are you able to provide a break down for each of the uuid's that have an issue including which image id was given to build it from. | 19:45 |
phschwartz | fungi: That is our exact concern that it might be a wider issue. | 19:46 |
phschwartz | clarkb: that is an option to see if it corrects the issue, but not sure if you want to take the delay in check/gating to let it work | 19:46 |
fungi | phschwartz: mapping them back to image uuids will be harder. getting the instance uuids is something i can repeat fairly easily for the other two presumed related bugs | 19:47 |
*** gema has quit IRC | 19:47 | |
clarkb | phschwartz: ya it will render dfw unusable for us for about an hour | 19:47 |
clarkb | phschwartz: but if it helps debug that might be good? | 19:47 |
clarkb | fungi: jeblair ^ | 19:47 |
fungi | image uuid mapping isn't something we explicitly record, so would require time series analysis comparing our image update logs with our node management logs | 19:47 |
openstackgerrit | Monty Taylor proposed a change to openstack-infra/storyboard: Add loader subcommand to sync from launchpad https://review.openstack.org/113624 | 19:48 |
*** krtaylor has joined #openstack-infra | 19:48 | |
anteaya | and yay!! it looks great | 19:48 |
jeblair | clarkb: why will it render dfw unusable? | 19:49 |
clarkb | jeblair: because we won't have any images there | 19:49 |
jeblair | oh, didn't read back far enough; you're asking about deleting images | 19:49 |
clarkb | jeblair: yup. Basically start from a clean slate so that we have a bit more known data | 19:49 |
*** shayneburgess has left #openstack-infra | 19:50 | |
phschwartz | clarkb: it would provide a good baseline to see if it happens again or if it was just a fluke when we were having load that caused it | 19:50 |
phschwartz | I am a little weired out that it is only one region though | 19:50 |
jeblair | clarkb, fungi: dfw provides such a small portion of our nodes currently, i think that would be fine | 19:50 |
jeblair | (side question, i wonder why?) | 19:51 |
jeblair | gah, because i mistyped the command :) | 19:51 |
fungi | seems we've all come down with the typing flu today | 19:52 |
openstackgerrit | Christian Fetzer proposed a change to openstack-infra/jenkins-job-builder: Add support for the Jenkins Valgrind publisher plugin. https://review.openstack.org/113117 | 19:52 |
clarkb | my typing driver is buggy | 19:52 |
* mordred no typie goood | 19:52 | |
jeblair | it would have an impact, but oh well, we should do all we can to debug anyway | 19:52 |
*** andreaf_ has joined #openstack-infra | 19:53 | |
jeblair | clarkb: can we do one image at a time? | 19:53 |
jeblair | which images are affected? | 19:53 |
*** andreaf_ is now known as andreaf | 19:53 | |
clarkb | jeblair: so far bare-* are known affected | 19:53 |
fungi | jeblair: so far we have bugs impacting at least bare-centos6, bare-precise and bare-trusty | 19:53 |
jeblair | clarkb: so could we drop all the bare images first but leave devstack? | 19:53 |
*** lsell has joined #openstack-infra | 19:53 | |
clarkb | jeblair: ya we could do that | 19:53 |
jeblair | then when bare have rebuilt, drop devstack? | 19:53 |
jeblair | that way we maintain the capacity | 19:54 |
clarkb | wfm | 19:54 |
*** otherwiseguy has joined #openstack-infra | 19:54 | |
phschwartz | ++ | 19:54 |
anteaya | jeblair: +1 and a suggestion on https://review.openstack.org/#/c/113984/ the special case hideci regex for Jenkins | 19:54 |
clarkb | ok I can go ahead and do that now | 19:54 |
jeblair | clarkb: cool | 19:54 |
*** shashankhegde has joined #openstack-infra | 19:54 | |
clarkb | starting with bare-* on rax-dfw | 19:54 |
mordred | anteaya: what does your suggestion mean? I don't follow the sentence? | 19:55 |
lsell | does anyone have a few minutes to help me with a mailing list issue? | 19:55 |
*** achuprin has joined #openstack-infra | 19:55 | |
fungi | lsell: sure--which mailing list? | 19:56 |
mordred | lsell: what sort of issue? | 19:56 |
lsell | great thank you. there's an email sitting in moderation for the user-committee list | 19:56 |
lsell | but tim bell and ryan lane are the only ones with credentials | 19:56 |
fungi | lsell: if it's for the user-committee ml, i reset the admin password earlier today and sent it to fifieldt | 19:56 |
fungi | lsell: ahh, yep | 19:56 |
mordred | jeblair: any reason I should hold back on +A'ing the hideci change? | 19:56 |
lsell | ah, great, we must have missed each other in time zones | 19:56 |
clarkb | ok all three deleted and new images building | 19:56 |
Ryan_Lane | isviridov: I don't have them | 19:56 |
Ryan_Lane | well, I have some, but they don't work | 19:56 |
lsell | ryan_lane: that was going to be my next message :) | 19:57 |
Ryan_Lane | there's a global mailman password | 19:57 |
clarkb | Ryan_Lane: that is supposed to be changed :) | 19:57 |
*** flwang_ has joined #openstack-infra | 19:57 | |
Ryan_Lane | I'm not actually on the user committee anymore | 19:57 |
Ryan_Lane | I sent in a resignation a few weeks ago | 19:57 |
openstackgerrit | Salvatore Orlando proposed a change to openstack-infra/config: Make neutron full job voting (neutron gate only) https://review.openstack.org/88289 | 19:57 |
clarkb | ya | 19:57 |
*** andreaf has quit IRC | 19:57 | |
clarkb | Ryan_Lane: we miss you | 19:57 |
Ryan_Lane | :) | 19:57 |
fungi | Ryan_Lane: and we're all sorry to see you go too--it was such great help! | 19:58 |
clarkb | ok now really I am eating lunch | 19:58 |
Ryan_Lane | thinking of help, I should probably upgrade the wiki soon | 19:58 |
fungi | lsell: anyway, did you need the pw for it too, or just wait until fifieldt is awake and let him handle it? | 19:58 |
lsell | fungi: i would prefer to get the email out sooner if possible, because i want to make people aware of a meeting happening tomorrow, so yes... | 19:59 |
anteaya | mordred: my suggestion means that some third party ci accounts have wanted to use the word Jenkins in their name | 20:00 |
lsell | Ryan_Lane: thank you also for checking it out earlier, I appreciate it! | 20:00 |
fungi | lsell: or i can pop in and moderate it in the interim. i'll see if that works real quick | 20:00 |
anteaya | mordred: as a preventative step, I suggest including the comment to ensure it is clear that the Jenkins regex applies to OpenStack Infra only | 20:01 |
jeblair | mordred: no reason not to merge it | 20:01 |
*** annegen__ has quit IRC | 20:01 | |
mordred | anteaya: ah, nod | 20:01 |
*** flwang_ has quit IRC | 20:01 | |
*** andreaf has joined #openstack-infra | 20:02 | |
*** unicell has quit IRC | 20:02 | |
*** baoli has quit IRC | 20:02 | |
fungi | lsell: i just released a message from chris kemp. there are others waiting but i think they're all spam unless i skimmed over one | 20:02 |
lsell | fungi: thank you!! that's the one. it's also a meeting I'd love for you to join, although i know it's late your time | 20:03 |
fungi | lsell: i'll see what i can arrange! | 20:03 |
*** dustins_ has quit IRC | 20:04 | |
*** _nadya_ has quit IRC | 20:04 | |
*** armax has joined #openstack-infra | 20:04 | |
*** sabeen has joined #openstack-infra | 20:04 | |
*** amcrn has quit IRC | 20:05 | |
*** _nadya_ has joined #openstack-infra | 20:05 | |
*** sabeen3 has quit IRC | 20:05 | |
fungi | jeblair: mordred: clarkb: some/all of you may be interested in joining http://lists.openstack.org/pipermail/user-committee/2014-August/000265.html for the same reasons | 20:05 |
fungi | (we do, sometimes, make use of an openstack cloud or two) | 20:06 |
openstackgerrit | Andreas Jaeger proposed a change to openstack-infra/config: Templatize puppet projects in layout.yaml https://review.openstack.org/114001 | 20:06 |
*** gema_ has quit IRC | 20:07 | |
*** hashar has quit IRC | 20:08 | |
clarkb | I can attend | 20:08 |
nibalizer | krotscheck: awesome! | 20:08 |
*** gema has joined #openstack-infra | 20:08 | |
*** baoli has joined #openstack-infra | 20:08 | |
nibalizer | i'll remove my workflow -1 | 20:08 |
*** _nadya_ has quit IRC | 20:08 | |
clarkb | is it something I call into? | 20:09 |
fungi | clarkb: i haven't figured out what a meetingburner is yet, so no idea | 20:09 |
fungi | might be some sort of corporateyness | 20:09 |
*** marun has joined #openstack-infra | 20:10 | |
fungi | clarkb: ahh, yep, after registering for the meeting it gives you the dial-in info | 20:11 |
openstackgerrit | A change was merged to openstack-infra/config: Special case 'Jenkins' in hideci. https://review.openstack.org/113984 | 20:11 |
*** emagana has joined #openstack-infra | 20:13 | |
*** jistr has quit IRC | 20:16 | |
*** pdmars has quit IRC | 20:16 | |
*** pdmars has joined #openstack-infra | 20:16 | |
*** gema_ has joined #openstack-infra | 20:17 | |
*** cwchang has quit IRC | 20:19 | |
marcusvrn | Hey guys, now I'm facing some troubles trying to run a job on jenkins slave....I'm trying to build the dsvm-tempest tests, but it's returning | 20:19 |
marcusvrn | Job timeout set to: 135 minutes | 20:19 |
marcusvrn | 17:17:06 Traceback (most recent call last): | 20:19 |
marcusvrn | 17:17:06 File "./test-matrix.py", line 21, in <module> | 20:19 |
marcusvrn | 17:17:06 import yaml | 20:19 |
marcusvrn | 17:17:06 ImportError: No module named yaml | 20:19 |
anteaya | marcusvrn: do you know how to use paste.openstack.org? | 20:19 |
marcusvrn | nope | 20:19 |
anteaya | http://paste.openstack.org/ | 20:20 |
marcusvrn | hmmmm!!! awesome! thx and sry for the paste here | 20:20 |
*** gema has quit IRC | 20:20 | |
anteaya | np | 20:20 |
anteaya | thanks for listening | 20:20 |
*** e0ne has joined #openstack-infra | 20:21 | |
krotscheck | Looking for reviews on switching the StoryBoard puppet module that we just landed. https://review.openstack.org/#/c/113731/ | 20:22 |
marcusvrn | http://paste.openstack.org/show/94599/ | 20:23 |
*** yamamoto has joined #openstack-infra | 20:24 | |
*** gondoi is now known as zz_gondoi | 20:27 | |
*** julim_ has quit IRC | 20:28 | |
*** yamamoto has quit IRC | 20:29 | |
*** gema has joined #openstack-infra | 20:31 | |
marcusvrn | And my jenkins slave is not synchronized with the jenkins server...Is this a problem? | 20:32 |
marcusvrn | I mean, time synchronization | 20:33 |
*** gema_ has quit IRC | 20:34 | |
*** AlexF has joined #openstack-infra | 20:34 | |
anteaya | marcusvrn: looks like you are missing yaml | 20:35 |
*** pdmars has quit IRC | 20:37 | |
openstackgerrit | Eoghan Glynn proposed a change to openstack-infra/config: Revert "Temporarily ignore Ceilometer Python 2.6 job" https://review.openstack.org/114018 | 20:37 |
*** crc32 has joined #openstack-infra | 20:37 | |
fungi | phschwartz: so here's a breakdown of the instances where we saw odd, potentially related behavior, all of which has been isolated to dfw... http://paste.openstack.org/show/94616 | 20:38 |
*** e0ne has quit IRC | 20:39 | |
marcusvrn | anteaya: I installed it and now I'm trying again :) | 20:40 |
*** maxbit_ has joined #openstack-infra | 20:41 | |
anteaya | marcusvrn: great | 20:41 |
*** Ajaeger1 has quit IRC | 20:42 | |
*** mmaglana has joined #openstack-infra | 20:44 | |
*** datsun180b has joined #openstack-infra | 20:44 | |
*** maxbit has quit IRC | 20:45 | |
fungi | jeblair: did the ci table and toggle get reverted? i don't see them in gerrit now... | 20:45 |
anteaya | fungi: I confirm | 20:46 |
anteaya | perhaps the regex change did something other than we expected? | 20:46 |
*** gema_ has joined #openstack-infra | 20:47 | |
*** stevemar has quit IRC | 20:47 | |
*** dkranz has quit IRC | 20:48 | |
*** annegent_ has joined #openstack-infra | 20:49 | |
lifeless | mordred: has your haveged fix landed? | 20:49 |
*** AlexF has quit IRC | 20:49 | |
*** datsun180b has quit IRC | 20:50 | |
*** annegent_ has quit IRC | 20:50 | |
fungi | lifeless: yep, as of 02:25 utc | 20:50 |
*** gema has quit IRC | 20:51 | |
fungi | centos/fedora slaves built from today's nodepool images (jobs started after ~1500 utc) should have it enabled | 20:51 |
lifeless | fungi: thanks! rechecking https://review.openstack.org/#/c/80449/ | 20:51 |
*** stevebaker has quit IRC | 20:52 | |
*** stevebaker has joined #openstack-infra | 20:52 | |
*** gema has joined #openstack-infra | 20:54 | |
SergeyLukjanov | fungi, jeblair, looks like I should send note re gerrit maintenance this weekend to the ML? | 20:55 |
clarkb | fungi: wait we reverted the ci stuff? | 20:55 |
openstackgerrit | Matthew Treinish proposed a change to openstack-infra/config: Make acls for QA program projects consistent https://review.openstack.org/114024 | 20:56 |
anteaya | clarkb: I think the hideci patch might have broken it | 20:56 |
*** marun has quit IRC | 20:56 | |
mtreinish | jogo: ^^^ that should take care of most of what we discussed the other day | 20:56 |
mtreinish | although I think bits of it require manual intervention | 20:56 |
mtreinish | which I outlined in the commit msg | 20:56 |
*** gema_ has quit IRC | 20:57 | |
clarkb | bare-trusty in rax-dfw is the last one we are waiting on. as soon as it goes ready I will delete the other images there | 20:57 |
clarkb | anteaya: the jenkins one? it should require a gerrit restart to pull that in | 20:57 |
fungi | SergeyLukjanov: that would be helpful, yes | 20:57 |
clarkb | ++ | 20:58 |
SergeyLukjanov | fungi, ack, I'll do it | 20:58 |
*** lsell has left #openstack-infra | 20:58 | |
clarkb | fungi: jeblair I haev held 1453997 for trusty and tox testing | 20:58 |
*** yamamoto has joined #openstack-infra | 20:59 | |
*** dprince has quit IRC | 21:00 | |
clarkb | dhellmann: I am testing taskflow, stevedore, and pbr now with trusty for unitests, docs, and pep8. And will check them against new tox | 21:01 |
clarkb | dhellmann: are there any others under the oslo umbrella not named oslo*? | 21:02 |
*** emagana has quit IRC | 21:02 | |
*** emagana has joined #openstack-infra | 21:02 | |
*** ominakov has quit IRC | 21:03 | |
dims_ | clarkb: pycadf, cliff | 21:03 |
clarkb | dims_: thanks | 21:03 |
*** gema_ has joined #openstack-infra | 21:04 | |
clarkb | arg the node i held apparently raced a delete and got deleted. hodling a different node now | 21:06 |
*** miqui_ is now known as miqui | 21:06 | |
clarkb | holding 1454425 now | 21:06 |
clarkb | jeblair: did you see the comments on the zuul changes? | 21:07 |
*** gema has quit IRC | 21:07 | |
*** dkliban is now known as dkliban_afk | 21:08 | |
*** e0ne has joined #openstack-infra | 21:08 | |
*** achuprin has quit IRC | 21:09 | |
*** _nadya_ has joined #openstack-infra | 21:09 | |
*** aysyd has quit IRC | 21:11 | |
jeblair | clarkb, fungi: it seems that if the hideci file changes, gerrit needs to be restarted | 21:11 |
jeblair | the cache-busting token it appends to the url seems to break it | 21:11 |
*** Alex_Gaynor_ has joined #openstack-infra | 21:11 | |
jeblair | restarting gerrit now | 21:11 |
clarkb | kk | 21:12 |
*** a2hill has joined #openstack-infra | 21:12 | |
*** zz_johnthetubagu has quit IRC | 21:12 | |
*** Alexei_987 has quit IRC | 21:12 | |
*** bookwar has quit IRC | 21:12 | |
*** ptoohill has quit IRC | 21:12 | |
*** medieval1 has quit IRC | 21:12 | |
*** Svedrin has quit IRC | 21:12 | |
*** rfolco has quit IRC | 21:12 | |
*** Alex_Gaynor has quit IRC | 21:12 | |
*** IAX has quit IRC | 21:12 | |
krotscheck | Gerrit went down! The sky is falling! | 21:12 |
krotscheck | :D | 21:12 |
*** IAX has joined #openstack-infra | 21:12 | |
*** krtaylor has quit IRC | 21:13 | |
*** YorikSar has quit IRC | 21:13 | |
*** fifieldt has quit IRC | 21:13 | |
*** changbl has quit IRC | 21:13 | |
*** mancdaz has quit IRC | 21:13 | |
*** Alex_Gaynor_ is now known as Alex_Gaynor | 21:13 | |
*** medieval1 has joined #openstack-infra | 21:13 | |
*** rfolco has joined #openstack-infra | 21:13 | |
*** krtaylor has joined #openstack-infra | 21:13 | |
*** bookwar has joined #openstack-infra | 21:13 | |
*** fifieldt has joined #openstack-infra | 21:13 | |
krotscheck | clarkb: I went ahead and made that change y’all asked fro on https://review.openstack.org/#/c/113935/ - was there anything else? | 21:13 |
jeblair | krotscheck: i really appreciate that when you couldn't connect to gerrit, you assumed the network was broken. :) that sort of thing makes my inner sysadmin happy. | 21:14 |
*** YorikSar has joined #openstack-infra | 21:14 | |
*** changbl has joined #openstack-infra | 21:14 | |
*** mancdaz has joined #openstack-infra | 21:14 | |
krotscheck | jeblair: When in doubt, blame IT :) | 21:14 |
*** hdd has quit IRC | 21:14 | |
jeblair | clarkb, fungi: hideci is back | 21:14 |
jeblair | and hiding jenkins now | 21:14 |
*** zz_johnthetubagu has joined #openstack-infra | 21:15 | |
*** zz_johnthetubagu is now known as johnthetubaguy | 21:15 | |
openstackgerrit | Khai Do proposed a change to openstack-infra/zuul: add support for draft-published event https://review.openstack.org/92268 | 21:15 |
krotscheck | nibalizer: I | 21:15 |
krotscheck | Argh | 21:15 |
*** Alexei_987 has joined #openstack-infra | 21:15 | |
*** Svedrin has joined #openstack-infra | 21:15 | |
*** mfer has quit IRC | 21:16 | |
clarkb | krotscheck: looks like we figured out how to remove admin perms? | 21:16 |
krotscheck | nibalizer: I’m trying to figure out a dependency chain in puppet-storyboard. Basically, if the user doesn’t configure an SSL certificate, it tries to use snakeoil, but that requires Apache, which creates a loop :/ | 21:16 |
*** amcrn has joined #openstack-infra | 21:16 | |
clarkb | nibalizer: puppet question. If you ensure a package is absent but that package does not exist at all on a distro will puppet complain? | 21:16 |
nibalizer | requires the apache class? | 21:16 |
*** e0ne has quit IRC | 21:16 | |
krotscheck | nibalizer: yep. I’m pondering just getting rid of the default ssl install in the first place. | 21:17 |
*** marun has joined #openstack-infra | 21:17 | |
nibalizer | clarkb: well i just tested it | 21:17 |
nibalizer | and it is fine with ensuring absent on a package that doesn't exist | 21:17 |
nibalizer | im not sure that question implies you're writing good code | 21:17 |
krotscheck | clarkb: Context? | 21:17 |
*** yamamoto has quit IRC | 21:18 | |
nibalizer | krotscheck: ya writing modules with multiple entry points and pivoted use cases is really hard (TM) | 21:18 |
clarkb | krotscheck: that rabbitmq user | 21:18 |
clarkb | nibalizer: it doesn't and the change has been -1'd for other reasons | 21:18 |
clarkb | nibalizer: 113884 | 21:18 |
ianw | any feelings on https://review.openstack.org/#/c/113884/2 <- (Not use firewalld on the jenkins slaves) . turns out current libvirt runs ~18x slower with firewalld | 21:19 |
clarkb | dtroyer: jeblair ianw 113884 will probably interest you as devstack cores | 21:19 |
clarkb | ianw: yup that is the one I just -1'd | 21:19 |
ianw | clarkb: heh :) | 21:19 |
clarkb | ianw: devstack should fix it and not the underlying base image so that individuals using devstack get the same fix | 21:19 |
krotscheck | clarkb: Yeah, took me a while to get it set up, but it appears that the admin flag is really only useful if accessing using rabbitmqadmin and rabbitmq web manager | 21:19 |
clarkb | ianw: is it sufficient to have devstack stop firewalld? | 21:20 |
clarkb | or can we configure libvirt to use iptables instead? | 21:20 |
*** arnaud__ has joined #openstack-infra | 21:20 | |
fungi | clarkb: any more, i just tend to always have http://git.openstack.org/cgit/openstack/governance/tree/reference/programs.yaml up somewhere as a reference | 21:20 |
jeblair | clarkb: i agree that would be the most desirable route | 21:20 |
openstackgerrit | Steve Baker proposed a change to openstack-infra/config: Define check-heat-dsvm-functional job https://review.openstack.org/111087 | 21:21 |
ianw | clarkb: afazekas proposed that, have to find the review. eharney made the point that disabling firewall on a developers box isn't great | 21:21 |
clarkb | ianw: we already do all kinds of terrible with devstack | 21:21 |
clarkb | everything from git reset --hard to installing all the things | 21:21 |
ianw | https://review.openstack.org/#/c/113856/ | 21:21 |
clarkb | if devstack can't fix this then we should use slow firewalld imo | 21:22 |
clarkb | ianw: we are very careful to not encode devstack bug fixes in our images | 21:22 |
ianw | clarkb: the problem is it is so slow that it causes the f20 tests to timeout in the gate | 21:22 |
nibalizer | oh my goodness, the ci report at the top is sooooo gooooooo | 21:22 |
clarkb | ianw: right, and that bug exists for all people doing devstack + firewalld | 21:23 |
ianw | (btw, i haven't 100% confirmed that ... last night .au time daniel suggested this is the issue) | 21:23 |
clarkb | ianw: so we need to fix it for them too | 21:23 |
jeblair | SergeyLukjanov: i just sent mail to jraim asking about kite. | 21:23 |
SergeyLukjanov | jeblair, okay, if he'll confirm, I'll add kite to renaming | 21:24 |
*** denis_makogon has quit IRC | 21:24 | |
clarkb | ianw: ideally we would be able to tell libvirt to not use firewalld and leave the service running | 21:24 |
ianw | clarkb: agree; but declaring shields down unconditionally may not be great. so will 1) confirm firewalld off speeds things up and 2) think more about how we can disable it safely | 21:24 |
jeblair | clarkb, ianw: yeah, i mean, devstack _takes over_ the box. afaic there is no assumption that the box should do anything other than what devstack says. sudo... etc. | 21:24 |
clarkb | ianw: but it doesn't appear that libvirt supports this? | 21:25 |
ianw | clarkb: can't say off the top of my head if libvirt can ignore firewalld. will investigate | 21:25 |
dtroyer | Honestly I'm find with dumping it, it isn't the first time we've removed stuff to keep things working. Is it an issue on all platforms? | 21:26 |
dtroyer | firewalld ^^^ | 21:26 |
clarkb | 113733 is also interesting | 21:26 |
clarkb | jeblair: fungi ^ | 21:26 |
clarkb | dtroyer: yes, it is an issue between libvirt and firewalld so would affect all platforms | 21:27 |
clarkb | dtroyer: we just don't use firewalld anywhere else | 21:27 |
clarkb | (I think firewalld must be default on fedora?) | 21:27 |
dtroyer | so how about a conditional that True in d-g and not otherwise? | 21:27 |
ianw | check-tempest-dsvm-f20 SUCCESS in 29m 29s (non-voting) | 21:27 |
jeblair | clarkb: is the statsd thing urgent? | 21:27 |
dtroyer | we have soooo mant of those already, I know | 21:28 |
ianw | that is i think significantly faster than other changes | 21:28 |
clarkb | jeblair: no, just an interesting thing | 21:28 |
ianw | check-tempest-dsvm-f20SUCCESS in 1h 03m 55s (non-voting) | 21:28 |
phschwartz | clarkb: did you clear the images in dfw? | 21:28 |
ianw | ^ yep | 21:28 |
jeblair | clarkb: yes. it's not a good change. but i'd like to prioritize reviewing it appropriately. | 21:28 |
clarkb | jeblair: ok | 21:28 |
openstackgerrit | A change was merged to openstack-infra/config: Make py33 job running only for Ceilometer master branch https://review.openstack.org/113842 | 21:28 |
clarkb | phschwartz: I did but only for the bare-* images | 21:28 |
* jeblair thinks all changes are interesting | 21:28 | |
clarkb | phschwartz: those images are all ready so I am going to delete the devstack images now | 21:29 |
phschwartz | clarkb: k, let me know if we see anything similar to before | 21:29 |
anteaya | just for info, this table is missing results from 4 CI systems: https://review.openstack.org/#/c/96630/ | 21:30 |
anteaya | anyone have any thoughts as to why that might be? | 21:30 |
*** annegent_ has joined #openstack-infra | 21:32 | |
anteaya | open contrail, hyper v, Artista, plumgrid, nuage, so 5 | 21:32 |
*** annegent_ has quit IRC | 21:32 | |
clarkb | anteaya: to be in the table they need to follow the same report comment format as jenkins | 21:32 |
openstackgerrit | Salvatore Orlando proposed a change to openstack-infra/config: Make neutron full job voting https://review.openstack.org/88289 | 21:32 |
fungi | phschwartz: it's worth noting that we were already expiring images there ~every 48 hours as we built new snapshots to replace them and rotated them to backup status | 21:33 |
openstackgerrit | Joel Coffman proposed a change to openstack/requirements: Add pykmip https://review.openstack.org/114037 | 21:33 |
clarkb | ok all remaining images in rax-dfw are being rebuilt | 21:34 |
jeblair | clarkb, ianw: responded on https://review.openstack.org/#/c/113856/ | 21:34 |
krotscheck | I don’t suppose there’s an infra-core available to assist me with getting the new rabbitmq patch for storyboard not only landed, but onto storyboard.o.o using the new puppet module? Before, I join mordred in the desert and Ish__ disappears back to her grad program. | 21:34 |
clarkb | jeblair: thanks | 21:35 |
clarkb | krotscheck: sure | 21:35 |
krotscheck | clarkb: THANK YOU | 21:35 |
fungi | krotscheck: oh, are you on you going to go be a burnout? exciting! | 21:35 |
*** zengyunling__ has joined #openstack-infra | 21:35 | |
anteaya | clarkb: ah, hmmmm | 21:35 |
clarkb | krotscheck: so what needs to be done? we need to approve the change to the module, and approve a chaneg to the infra/config repo that switches to using the module? | 21:35 |
* anteaya toggles ci | 21:35 | |
clarkb | krotscheck: I have the first chaneg open, is the second laying around somewhere | 21:36 |
krotscheck | clarkb: Three things.... | 21:36 |
mordred | krotscheck: Ish__ is going back to gradschool? that's terrible | 21:36 |
krotscheck | mordred: She is! She couldn’t get her department to sign off on another internship waiver because of their own stupid policies, so she’s gone next thursday | 21:37 |
krotscheck | clarkb: https://review.openstack.org/#/c/113731/ | 21:37 |
anteaya | hmmm so they need log ip Success|Failure in time elapsed | 21:37 |
mordred | krotscheck: blech | 21:37 |
krotscheck | mordred: No kidding :/ | 21:37 |
SergeyLukjanov | jeblair, fungi, clarkb, etherpad for upcoming renaming - https://etherpad.openstack.org/p/repo-renaming-2014-08-16 | 21:37 |
krotscheck | mordred: It’s one of those immigration things. | 21:37 |
krotscheck | clarkb: 2 is this one: https://review.openstack.org/#/c/113935/ | 21:38 |
mordred | krotscheck: sigh | 21:38 |
krotscheck | And then 3 is sortof like the first one, except that the sha hash needs to be updated. | 21:38 |
krotscheck | clarkb: Let me put the 3rd together for you. | 21:38 |
*** ling-yun-1 has quit IRC | 21:38 | |
clarkb | krotscheck: why don't we combine 1 and 3? | 21:38 |
clarkb | krotscheck: or do you prefer switching to it nowwithout rabbitmq? | 21:38 |
clarkb | then adding rabbitmq? | 21:39 |
krotscheck | clarkb: Because I’d like to break one thing at a time. | 21:39 |
clarkb | krotscheck: thats fair | 21:39 |
ianw | dtroyer: i agree with FIREWALLD_DISABLE ; because this is supposed to be fixed in f21. so when we transition later we can just drop that flag and stay closer to what the distro is shipping | 21:40 |
openstackgerrit | Cedric Brandily proposed a change to openstack-infra/git-review: test https://review.openstack.org/114038 | 21:40 |
ianw | not sure if the affects centos7... | 21:40 |
openstackgerrit | Khai Do proposed a change to openstack-infra/zuul: add support for draft-published event https://review.openstack.org/92268 | 21:41 |
*** emagana has quit IRC | 21:42 | |
openstackgerrit | Khai Do proposed a change to openstack-infra/zuul: add support for draft-published event https://review.openstack.org/92268 | 21:42 |
*** emagana has joined #openstack-infra | 21:43 | |
*** arosen-home has quit IRC | 21:43 | |
clarkb | krotscheck: ok, first thing to break is approved | 21:43 |
clarkb | krotscheck: I am going to review rabbitmq change now as that can be approved independently | 21:43 |
dtroyer | ianw: BTW, I amended that comment to add this should be in tools/fixup_stuff.sh | 21:43 |
krotscheck | clarkb: yay! | 21:43 |
*** emagana has quit IRC | 21:43 | |
SergeyLukjanov | fungi, thx for note re kite | 21:44 |
*** emagana has joined #openstack-infra | 21:44 | |
jeblair | dtroyer: i'd rather the gate behavior be the default behavior | 21:44 |
fungi | SergeyLukjanov: i also slightly tweaked the mysql command, though i cleared the authorship colors again after i did | 21:44 |
openstackgerrit | Cedric Brandily proposed a change to openstack-infra/git-review: test https://review.openstack.org/114038 | 21:44 |
jeblair | dtroyer: i'm getting really frustrated with people thinking the gate is something special they can't replicate; we really should just "run devstack" | 21:44 |
dtroyer | jeblair: I think I'm neutral here which should be the default on this | 21:45 |
SergeyLukjanov | fungi, I've right simple py script to generate commands | 21:45 |
SergeyLukjanov | fungi, yup, I see the changes | 21:45 |
jeblair | dtroyer: i'm totally cool with there being different ways of running it (postgres/mysql) but not so much (work/not work) :) | 21:45 |
ianw | dtroyer: question i'm asking is what happens if you remove firewalld on a fresh install anyway ... does it even have default rules setup? | 21:45 |
ianw | (booting rax f20 image to confirm) | 21:46 |
phschwartz | jeblair: is there a rest service as part of zuul currently? I haven't dug threw it all to see. | 21:46 |
*** hdd has joined #openstack-infra | 21:46 | |
*** hdd has quit IRC | 21:46 | |
*** krtaylor has quit IRC | 21:46 | |
*** crc32 has quit IRC | 21:47 | |
jeblair | phschwartz: there's a really simple webserver which we can continue to abuse (or even improve and turn it into a webservice); i think jhesketh may have a change outstanding that does some of that. | 21:47 |
fungi | phschwartz: there's a status json dataset you can retrieve from it, other than that just the gearman queues (you can connect and issue admin commands) and some rpc client examples in the command-line utilities | 21:47 |
*** bhuvan_ is now known as bhuvan | 21:47 | |
fungi | also what jeblair said about patches welcome et cetera | 21:48 |
phschwartz | jeblair: I will check with him when he is on tonight. I want to refine my github webhook more and it is better to do it over rest or create a separate running service that is | 21:48 |
phschwartz | a gearman client | 21:48 |
openstackgerrit | Michael Krotscheck proposed a change to openstack-infra/config: Update storyboard version to include rabbitmq https://review.openstack.org/114039 | 21:48 |
dtroyer | jeblair: agreed…I think devstack may rediscover some of the opinions it once had | 21:48 |
jeblair | dtroyer: excellent :) | 21:49 |
clarkb | krotscheck: and now rabbitmq approved | 21:49 |
clarkb | krotscheck: so ready for third change whenever you are | 21:49 |
krotscheck | clarkb: Awesome | 21:49 |
krotscheck | clarkb: The patch is here https://review.openstack.org/114039 - but let’s wait to make sure that the first change lands. | 21:49 |
clarkb | krotscheck: rgr | 21:49 |
fungi | phschwartz: right now the implemented zuul trigger examples are an internal periodic schedule and a stream event watcher for gerrit. if github has something stream-like you can use to watch for things like pull requests that might be the easiest to emulate | 21:50 |
jeblair | phschwartz: i don't think it needs to be a separate service; but then, maybe it should be and so should all the other triggers :) | 21:50 |
clarkb | krotscheck: and I need to generate a rabbitmq password and put it in hiera | 21:50 |
openstackgerrit | A change was merged to openstack-infra/config: Use split-out storyboard module https://review.openstack.org/113731 | 21:50 |
krotscheck | clarkb: Right! | 21:50 |
clarkb | krotscheck: any guidelines for that? random 32 char string good? | 21:50 |
openstackgerrit | A change was merged to openstack-infra/puppet-storyboard: Added RabbitMQ to StoryBoard https://review.openstack.org/113935 | 21:51 |
jeblair | phschwartz: but i think the internal-only option is the way to go; i think an external process using gearman is a stretch for this and likely to bog us down | 21:51 |
krotscheck | clarkb: I think rabbit only accepts passwords that have been blessed by the priests of hermes on a full moon. | 21:51 |
phschwartz | fungi: I am not too keen on creating a poller to poll the github api (doing it currently, but it is not the best and the api key is only allowed a set number of polls per day) | 21:51 |
fungi | phschwartz: yeah, that's why i suggested looking for stream-like interfaces | 21:51 |
krotscheck | clarkb: (No clue. 32 character alphanumeric should be fine) | 21:51 |
SergeyLukjanov | oh, my, I have ipv6 at home O_O | 21:51 |
fungi | phschwartz: active notification channel of some sort from github perhaps? | 21:51 |
jeblair | phschwartz, fungi: i don't think github has a streaming thing; i think instead we provide a url it hits. so i think we could hook something into the existing webserver to receive those hits, then put them on an internal queue just like the gerrit trigger does | 21:51 |
*** jaypipes has joined #openstack-infra | 21:52 | |
clarkb | krotscheck: I can turn on futurama when I generate the password (wait wrong hermes) | 21:52 |
*** ling-yun-1 has joined #openstack-infra | 21:52 | |
phschwartz | fungi: currently they offer webhooks for pr creation and merges and such or the api. | 21:52 |
clarkb | jeblair: very similar to how jenkins callbacks worked | 21:52 |
fungi | yeah, that's what i had in mind for a notification channel. is that what travis uses? | 21:52 |
jeblair | phschwartz, fungi: (the stream-events processor in gerrit has a dedicated queue where it puts its events, then processes them and turns them into zuul events and puts them on the zuul event queue) | 21:52 |
jeblair | clarkb: yep | 21:52 |
phschwartz | jeblair: That is what I was thinking and why I was asking about a rest service as part of zuul | 21:52 |
openstackgerrit | Sergey Lukjanov proposed a change to openstack-infra/config: Corrections to the project renaming guide https://review.openstack.org/114041 | 21:53 |
ianw | http://paste.openstack.org/show/94644/ <- i remember the days when a 163 lines of iptables rules wasn't the default | 21:53 |
anteaya | dougwig: hello, I see you created a third party system wiki page entitled Cisco Neutron CI, there is no account named Cisco Neutron CI: http://paste.openstack.org/show/94642/ | 21:53 |
fungi | while a "github bridge daemon" isn't something zuul should really grow, a rest api service which runs as a daemon and communicates events into zuul may not be a totally terrible idea | 21:53 |
SergeyLukjanov | fungi, https://review.openstack.org/114041 | 21:53 |
*** jaypipes has quit IRC | 21:53 | |
anteaya | dougwig: can you shed some light on your choice? the wikipages are intended to map to third party ci accounts | 21:53 |
*** crc32 has joined #openstack-infra | 21:53 | |
dougwig | err, wasn't it A10 Networks CI? | 21:54 |
jeblair | fungi: yeah, i think that's where jhesketh wanted to head with that. and i like it. | 21:54 |
anteaya | https://wiki.openstack.org/w/index.php?title=ThirdPartySystems&diff=0&oldid=57683 | 21:54 |
fungi | ianw: you really need recalibrate your brain onto a logarithmic scale. this is openstack after all | 21:54 |
anteaya | is the Cisco change not you? dougwig? | 21:54 |
phschwartz | fungi: That was my thought, either as part of zuul or a separate process that enqueues into zuul on request | 21:54 |
phschwartz | I will takl with jhesketh tonight | 21:54 |
*** mmaglana has quit IRC | 21:55 | |
*** zengyunling__ has quit IRC | 21:55 | |
dougwig | checking history, i thought i just added a10 | 21:55 |
dougwig | the way my day has been going though... | 21:55 |
dougwig | it was dane that made the cisco change. | 21:56 |
fungi | SergeyLukjanov: thanks!!! | 21:56 |
anteaya | dougwig: okay I am wrong | 21:56 |
anteaya | dougwig: thanks I will talk to dane | 21:56 |
dougwig | no worries, thanks. :) | 21:56 |
anteaya | I don't seem him online | 21:56 |
anteaya | I'll see if Henry knows where he is, thanks | 21:57 |
ianw | fungi: after calibrating my brain to understand how ping packet got from a vm to the world and back with neutron, nothing much can surprise me :) | 21:57 |
marcusvrn | anteaya: now...that problem was solved...but when run devstack, this error occurs: cannot stat `/opt/stack/new/tempest/nosetests*.xml': No such file or directory | 21:57 |
*** HenryG_ is now known as HenryG | 21:57 | |
fungi | ianw: in a broader sense, this is the internet. most days i'm surprised any of it works AT ALL | 21:58 |
*** flwang_ has joined #openstack-infra | 21:58 | |
fungi | pleasantly surprised, mind you | 21:58 |
ianw | marcusvrn: i think that's a red-herring, i always see that | 21:58 |
*** arosen1 has joined #openstack-infra | 21:58 | |
*** mriedem has quit IRC | 21:58 | |
dtroyer | ianw: funny, my default f20 image doesn't have firewalld, its based on the published F20 cloud image, not one from hpcloud or rax | 21:59 |
*** flaviof is now known as flaviof_zzz | 21:59 | |
ianw | dtroyer: yeah, i'm thinking that the rax image is the outlier with it turned on by default | 21:59 |
arosen1 | does log stash only go back ~11 days? | 21:59 |
fungi | dtroyer: ianw: i can hold an f20 devstack image and see what's installed, if that helps | 21:59 |
fungi | ianw: yes, rackspace still rolls their own i think. hpcloud seems to have switched over to using distro-provided cloud images instead | 22:00 |
fungi | arosen1: approximately, i believe | 22:00 |
arosen1 | thx | 22:00 |
fungi | arosen1: due to capacity issues, if memory serves | 22:01 |
arosen1 | makes sense. | 22:01 |
*** weshay has quit IRC | 22:01 | |
fungi | arosen1: yeah, the indexes take a crazy amount of space compared to the raw log data | 22:01 |
marcusvrn | ianw: but how can I make the dsvm-tempest-full complete with no errors? | 22:01 |
dtroyer | fungi, ianw: so lets just fix it in fixup_stuff.sh and move on… | 22:01 |
fungi | dtroyer: seems like a pragmatic choice | 22:01 |
jeblair | ianw, dtroyer: if one of our providers is doing something that's not in the base os, i'm more open to fixing that in infra. (though it still means people using that provider may hit a problem) | 22:01 |
ianw | fungi: it's ok, i just booted one up. for all the noise in http://paste.openstack.org/show/94644/ i think by default it's just allowing 5353 and ssh | 22:01 |
clarkb | arosen1: we float around 3-5TB of index for 10 days | 22:01 |
jeblair | not trying to dissuade you -- just saying the hard line is softer in that case :) | 22:02 |
*** yamamoto has joined #openstack-infra | 22:02 | |
dtroyer | jeblair: either way, we already have a place to do this | 22:02 |
*** flwang_ has quit IRC | 22:02 | |
jeblair | wfm | 22:02 |
jeblair | also, soon we hope we will stop using provider-supplied images altogether | 22:02 |
dtroyer | ++++++ | 22:03 |
fungi | clarkb: hopefully the steadily falling log volume will allow us to scale logstash retention back out sometime soonish? | 22:03 |
clarkb | fungi: ya | 22:03 |
*** emagana has quit IRC | 22:03 | |
* jeblair just +2d change 46482 (Build images using diskimage-builder) | 22:03 | |
clarkb | fungi: though the window is much smaller so may not affect us greatly | 22:03 |
clarkb | fungi: unless you mean we can use more volume on es? | 22:03 |
clarkb | jeblair: nice! | 22:03 |
*** emagana has joined #openstack-infra | 22:03 | |
fungi | clarkb: well, that too | 22:03 |
fungi | clarkb: when jhesketh's swift efforts finally pay off | 22:03 |
clarkb | I should rereview that change then | 22:04 |
clarkb | fungi: ya | 22:04 |
*** maxbit_ has quit IRC | 22:04 | |
fungi | also, afs | 22:04 |
HenryG | anteaya: hi | 22:05 |
anteaya | hi | 22:05 |
clarkb | fungi: tseting these things on trusty is so simple now that our scrpts don't need a ton of args | 22:05 |
anteaya | jeblair: so HenryG would like to discuss Cisco Unknown Function CI | 22:05 |
clarkb | fungi: I am very happy you went and cleaned that up | 22:05 |
anteaya | HenryG: what does this account verify? | 22:05 |
anteaya | HenryG: do you know? | 22:05 |
fungi | clarkb: meh. thank mordred. i was merely the (long-term) shepherd | 22:06 |
HenryG | anteaya: Let me pick a neutron review to give you an idea ... | 22:06 |
fungi | he was the one to start that change | 22:06 |
anteaya | HenryG: thanks | 22:06 |
krotscheck | clarkb: So… it looks like the SB module ran, but I don’t really know how frequently config is updated on puppetmaster. | 22:07 |
clarkb | krotscheck: every 15 minutes. I can check it directly | 22:07 |
fungi | krotscheck: cron job every 15 minutes | 22:07 |
krotscheck | clarkb fungi: Awesome, thanks. | 22:07 |
clarkb | krotscheck: module is installed in /etc/puppet/modules | 22:08 |
fungi | should have run on the hour since the change merged before | 22:08 |
clarkb | krotscheck: it was installed at 2200 UTC. So any puppet run after that time will have used it | 22:08 |
clarkb | krotscheck: so check puppetboard and if green after 2200 we should be set | 22:08 |
clarkb | (2200 was 8 minutes ago) | 22:08 |
krotscheck | clarkb: That seems correct. http://puppetboard.openstack.org/report/storyboard.openstack.org/9a50d69fa201d25ff9b0b82eaf74b72640cfc388 | 22:09 |
jeblair | clarkb, fungi, krotscheck: puppet updates and runs are tied together now | 22:09 |
fungi | yep | 22:09 |
jeblair | clarkb, fungi, krotscheck: the cron job updates puppet, installs modules, then runs puppet | 22:09 |
clarkb | jeblair: oh right | 22:09 |
clarkb | perfect | 22:09 |
clarkb | krotscheck: so you ready for the next change? | 22:09 |
krotscheck | Neat! | 22:09 |
HenryG | anteaya: https://review.openstack.org/113751 | 22:09 |
krotscheck | Yep! | 22:09 |
krotscheck | Let’s kick the tires :) | 22:09 |
anteaya | HenryG: great, so guide me to the salient points | 22:09 |
anteaya | HenryG: what are you verifying with this account | 22:09 |
HenryG | anteaya: Note that the bot reports itself as "Cisco Neutron CI". I'll get Dane to change that part soon. | 22:10 |
*** ominakov has joined #openstack-infra | 22:10 | |
clarkb | krotscheck: what do you want to call the storyboard rabbitmq user? | 22:10 |
*** maxbit has joined #openstack-infra | 22:10 | |
* clarkb grabs hiera data file lock | 22:10 | |
*** Ryan_Lane has quit IRC | 22:11 | |
openstackgerrit | James E. Blair proposed a change to openstack-infra/zuul: Add a Zuul trigger https://review.openstack.org/112411 | 22:11 |
krotscheck | clarkb: Hrm… is there a convention? I like to use appname_servicename for my own things. | 22:11 |
anteaya | HenryG: you can't | 22:11 |
krotscheck | so storyboard_rabbit | 22:11 |
mattoliverau | Morning | 22:11 |
jeblair | it's our first rabbit deployment | 22:11 |
clarkb | krotscheck: looks like the mysql user is just 'storyboard' | 22:11 |
clarkb | I am tempted to use tat | 22:11 |
HenryG | anteaya: Oh. Sorry. Where is the name coming from? | 22:11 |
krotscheck | I don’t see a reason why you shouldn't. | 22:12 |
anteaya | HenryG: yes where is that name coming from | 22:12 |
anteaya | fungi: can you find out about Cisco Neutron CI and move it into the third party group? | 22:12 |
clarkb | krotscheck: thats the default too so I am going with it | 22:12 |
fungi | anteaya: sure... hunting | 22:13 |
HenryG | anteaya: And it should be called "Cisco OpenStack CI"? | 22:13 |
clarkb | ok hiera lock released | 22:14 |
openstackgerrit | Khai Do proposed a change to openstack-infra/config: add a turbo hipster worker for testing https://review.openstack.org/107331 | 22:14 |
anteaya | HenryG: ah we aren't going to be using OpenStack in third party CI names either | 22:15 |
fungi | anteaya: index account_id is 10192 fyi | 22:15 |
anteaya | HenryG: so let's go back to what the account is verifying | 22:15 |
clarkb | jeblair: fungi 114039 is the change krotscheck needs to complete this thing | 22:15 |
fungi | anteaya: no openid url, so we created it directly | 22:15 |
clarkb | hiera data is in and change lgtm | 22:15 |
anteaya | HenryG: I see a reference to nexus? | 22:15 |
anteaya | fungi: kk, thanks | 22:15 |
HenryG | anteaya: Yes, there are (currently) five different jobs. All are triggered only on neutron changes. | 22:16 |
jeblair | clarkb, nibalizer: can we put "latest" in as a value? | 22:16 |
jeblair | clarkb: +3 | 22:16 |
anteaya | HenryG: what are you verifying? | 22:16 |
anteaya | HenryG: are you verifying that openstack works with a driver called nexus? | 22:17 |
jeblair | clarkb, nibalizer, jesusaurus: because i think it would be nice to have all of our "internal" modules use the latest versions, with the option of pinning them to a specific version if we need to do some breaking changes | 22:17 |
fungi | anteaya: i've added them to the group... creation date on their account was february, so not sure how they got left out unless that was on the cusp of when we were removing them from the group to disable voting | 22:17 |
fungi | feb 3, so maybe | 22:17 |
anteaya | fungi: thanks for fixing that | 22:17 |
HenryG | anteaya: Sorry for being slow, I am trying to understand the question. Bear with me ... | 22:18 |
fungi | anteaya: what should they be renamed to, if anything? | 22:18 |
anteaya | HenryG: okay so now your account is in the third party group, sorry it had been left out before, which is why I couldn't find it | 22:18 |
krotscheck | clarkb: Thanks :) | 22:18 |
anteaya | HenryG: np, I'm catching you by surprise | 22:18 |
*** shashankhegde has quit IRC | 22:18 | |
anteaya | fungi: working on that | 22:18 |
krotscheck | jeblair: I believe you can set “HEAD”, right? | 22:18 |
HenryG | anteaya: So let's start with nexus (one of the current five jobs) | 22:18 |
clarkb | jeblair: we can do 'master' | 22:18 |
anteaya | HenryG: awesome so Cisco Nexus CI? | 22:18 |
krotscheck | or that | 22:19 |
anteaya | HenryG: can we live with that? | 22:19 |
HenryG | anteaya: No | 22:19 |
anteaya | no | 22:19 |
*** dkranz has joined #openstack-infra | 22:19 | |
* anteaya waits | 22:19 | |
HenryG | anteaya: Nexus is just one of the many products that is tested | 22:19 |
anteaya | okay | 22:19 |
anteaya | what other products are tested? | 22:19 |
fungi | HenryG: anteaya: and is there a common word which describes the class of things being tested? | 22:20 |
HenryG | anteaya: For that job, neutron is configured with the ML2 mechanism driver for nexus, and the job is run on a system connected to a physical Nexus switch. | 22:20 |
fungi | (e.g. not neutron, but what's being tested in interactions with neutron) | 22:20 |
HenryG | anteaya: fungi: Anything wrong with "Cisco CI" ? | 22:20 |
nibalizer | jeblair: not sure, would have to read insatll_modules.sh | 22:21 |
anteaya | HenryG: yes | 22:21 |
nibalizer | and after what bode did to it, its really hard to read | 22:21 |
anteaya | HenryG: Cisco has more than one CI | 22:21 |
fungi | HenryG: the chance that there's more than one department at cisco with their own separate ci testing things | 22:21 |
*** rwsu has quit IRC | 22:21 | |
openstackgerrit | A change was merged to openstack-infra/config: Update storyboard version to include rabbitmq https://review.openstack.org/114039 | 22:21 |
anteaya | HenryG: so we are talking about the Cisco Neutron CI account, yes? or the or the Cisco Unknown Function CI? | 22:22 |
*** jaypipes has joined #openstack-infra | 22:22 | |
ianw | dtroyer / jeblair : proposed removal in https://review.openstack.org/#/c/113856/ | 22:22 |
*** dmsimard is now known as dmsimard_away | 22:22 | |
HenryG | anteaya: I have no idea what that last one is? | 22:23 |
krotscheck | clarkb: Thanks :) | 22:23 |
fungi | anteaya: worth noting, "cisco unknown function ci" doesn't seem to have weighed in on any reviews ever? https://review.openstack.org/#/q/reviewer:%22Cisco+Unknown+Function+CI+%253Ccisco-openstack-ci%2540external.cisco.com%253E%22,n,z | 22:24 |
clarkb | krotscheck: that should update in about 7 minutes then run in about 11 | 22:24 |
krotscheck | clarkb: I’ll keep my eye on it, thanks :) | 22:24 |
anteaya | HenryG: neither do we: http://paste.openstack.org/show/94672/ | 22:24 |
anteaya | fungi: thanks | 22:24 |
fungi | anteaya: created back on november 5 | 22:25 |
anteaya | awesome | 22:25 |
fungi | anteaya: associated with the cisco-openstack-ci@external.cisco.com address | 22:25 |
HenryG | fungi: anteaya: It has the same username and email as the "normal" one | 22:26 |
anteaya | HenryG: any chance you can find someone at cisco who can tell us we can disable this ci account? | 22:26 |
anteaya | HenryG: we sure would appreciate an opportunity to clean up the cruft | 22:26 |
jeblair | ianw: lgtm, thx | 22:26 |
fungi | HenryG: well, the ssh username on it is cisco-openstack-ci | 22:26 |
*** SumitNaiksatam has quit IRC | 22:27 | |
sarob | brother can you spare a +2 https://review.openstack.org/#/c/112423/ | 22:27 |
*** shashankhegde has joined #openstack-infra | 22:28 | |
*** SumitNaiksatam has joined #openstack-infra | 22:28 | |
fungi | sarob: you're not going to use it to buy booze are you? | 22:28 |
jeblair | jhesketh: so i think on https://etherpad.openstack.org/p/cviAgfbKV6 the main difference between your suggestion and mine is whether in the layout the name of the source/trigger/reporter is the name of the connector (mine), or is the name of the driver with an additional field specifying the connector | 22:29 |
jeblair | (yours) | 22:29 |
HenryG | fungi: How is the "ssh username" related to the "ci account"? What is the unique identifier for a ci account? | 22:29 |
*** maxbit has quit IRC | 22:29 | |
anteaya | HenryG: the id number | 22:30 |
anteaya | so 10192 and 9270 | 22:30 |
fungi | HenryG: ssh username, e-mail addresses and account numeric id are all unique-ish (gerrit attempts to throw errors if they conflict anyway) | 22:31 |
HenryG | anteaya: I don't see 10192 in your paste list? | 22:31 |
anteaya | but ssh username can be edited and so can email address | 22:31 |
anteaya | HenryG: right | 22:31 |
*** prad has quit IRC | 22:31 | |
anteaya | since it wasn't in the third party gerrit group when I created taht paste | 22:31 |
anteaya | fungi just added it | 22:31 |
anteaya | HenryG: if you run ssh -p 29418 anteaya@review.openstack.org gerrit ls-members "Third-Party\ CI" --recursive | 22:32 |
anteaya | with your username in place of mine | 22:32 |
anteaya | you should see it in the list now | 22:32 |
*** crc32 has quit IRC | 22:33 | |
HenryG | anteaya: yes, I do | 22:34 |
anteaya | HenryG: great | 22:34 |
reed | is the PSTN number listed on https://wiki.openstack.org/wiki/Infrastructure/Conferencing still valid? | 22:34 |
anteaya | so now it will show up as a member of the third party gerrit group, it had been missed previously, sorry about that | 22:34 |
jeblair | reed: probably? if it doesn't work, we may need to check on asterisk | 22:35 |
anteaya | HenryG: so, homework, can you help us confirm we can deprecate the Cisco Unknown Function CI gerrit account? | 22:35 |
reed | jeblair, I tried calling it but I had a 'busy' tone | 22:35 |
anteaya | HenryG: and we have to think of a word taht isn't openstack or neutron that describes what your ci system is verifying | 22:35 |
fungi | reed: i get the same. maybe its account expired | 22:36 |
anteaya | HenryG: do sleep on it and get back to me | 22:36 |
reed | jeblair, not a priority | 22:36 |
*** rwsu has joined #openstack-infra | 22:36 | |
fungi | reed: i hadn't tried it in a while, mainly due to lack of anyone expressing interest in using it at all | 22:36 |
reed | I'll update the wiki page, mention that we need a maintainer | 22:36 |
jeblair | reed: er | 22:36 |
jeblair | not sure we need a maintainer | 22:36 |
krotscheck | clarkb: BOOM http://puppetboard.openstack.org/report/storyboard.openstack.org/3a110b645acc8951b782eecc8024fc62d58ce72b | 22:37 |
reed | jeblair, yeah, wrong word... | 22:37 |
clarkb | krotscheck: and no red! | 22:37 |
anteaya | fungi: well the nova ml thread indicates that folks might like a higher bandwidth option for interactions | 22:37 |
HenryG | anteaya: I will get sleep, but Dane won't. :) | 22:37 |
clarkb | krotscheck: I imagine you can confirm the new functionality is in storyboard now? | 22:37 |
anteaya | so interest may be moving towards asterisk | 22:37 |
jeblair | reed: i'm happy to help with it, and i think so is russelb... i think no one has noticed the problem because it hasn't been used lately | 22:37 |
anteaya | HenryG: :D poor Dane | 22:37 |
krotscheck | clarkb: Not yet - we didn’t want to land that patch until we had a working rabbit. | 22:37 |
jeblair | russellb even :) | 22:37 |
reed | jeblair, I meant to say that we need to promote it | 22:37 |
fungi | anteaya: i was reading that into it as well... though "telepresence system" was mentioned... not quite the same thing as a conference call | 22:37 |
clarkb | krotscheck: gotcha | 22:37 |
ianw | jeblair : mind having a look at https://review.openstack.org/#/c/111637/ (Exact match rhel6) this is keeping centos7 images from working | 22:38 |
anteaya | fungi: true | 22:38 |
*** emagana has quit IRC | 22:38 | |
russellb | reed: jeblair yeah i'm happy to help fix stuff ... haven't used it myself in a long time, so didn't know it was busted | 22:38 |
anteaya | fungi: but perhaps the group doesn't know we did have asterisk up at one point | 22:38 |
fungi | anteaya: yep | 22:38 |
russellb | maybe file a bug and assign it to me? | 22:38 |
dhellmann | clarkb: thanks, with the ones dims_ mentioned I think you have them all | 22:38 |
reed | yeah | 22:38 |
krotscheck | clarkb: Thanks for all your help :) | 22:38 |
russellb | i don't mind being the primary maintainer | 22:38 |
anteaya | russellb: what do you think, do you think folks might use if if they knew about it? | 22:38 |
*** emagana has joined #openstack-infra | 22:38 | |
russellb | jeblair and others have enough on their plate | 22:38 |
jeblair | russellb: yeah, i'm doing some quick triage | 22:39 |
russellb | jeblair: k | 22:39 |
anteaya | russellb: would you like to mention it on the thread and gauge interest? | 22:39 |
* russellb hasn't seen that thread | 22:39 | |
reed | my main concern with Asterisk is that I don't have a SIP client, i'm too lazy to install stuff from extra ppa | 22:39 |
clarkb | dhellmann: great | 22:39 |
russellb | jeblair: maybe just msg or email your context if you don't resolve it and i can look tomorrow | 22:39 |
clarkb | dhellmann: so far trusty looks safe (onto taskflow now) | 22:39 |
anteaya | russellb: the nova core thread | 22:39 |
clarkb | dhellmann: going to upgrade tox shortly and run through them again | 22:39 |
pleia2 | I often used the dial in number for it | 22:39 |
anteaya | russellb: you have replied to it | 22:39 |
pleia2 | but I guess US-only :\ | 22:40 |
anteaya | russellb: discussion options to in person meetings | 22:40 |
russellb | reed: guess we should document more clients that are easy to install from normal repos | 22:40 |
reed | russellb, if there are any... looks like the situation is pretty bad | 22:40 |
reed | like... bad | 22:40 |
russellb | i think most people just use google hangouts for this sort of thing though | 22:40 |
clarkb | ubuntu comes with one right? | 22:40 |
*** dguerri`afk is now known as dguerri | 22:40 | |
reed | clarkb, nope | 22:40 |
russellb | used to | 22:40 |
jeblair | pleia2: we can add non-us numbers if there's an interest | 22:41 |
* fungi shudders | 22:41 | |
clarkb | I prefer using my phone for phone stuff | 22:41 |
jeblair | pleia2: they cost a bit more money | 22:41 |
clarkb | which makes me weird apparently | 22:41 |
russellb | clarkb: yeah, same | 22:41 |
jeblair | so i didn't want to add a bunch until we knew we needed them | 22:41 |
pleia2 | empathy does sip, in theory | 22:41 |
* fungi was shuddering at google hangouts, not international trunk lines | 22:41 | |
reed | clarkb, not weird, I use the phone too... because software SIP clients are bad | 22:41 |
pleia2 | I've not used it though | 22:41 |
russellb | the conferencing support in asterisk we're using supports both HD audio and video, if you use the right client | 22:41 |
pleia2 | should do, since that comes with ubuntu | 22:41 |
russellb | and call in via SIP and not the dial-in # | 22:41 |
clarkb | russellb: oh it does video too? | 22:42 |
nibalizer | mumble server? | 22:42 |
clarkb | nibalizer: no no asterisk > mumble | 22:42 |
russellb | clarkb: yeah, basic video anyway | 22:42 |
reed | russellb, jitsi is the suggested client, right? | 22:42 |
clarkb | russellb: thats cool, didn't know that | 22:42 |
russellb | reed: it was just the one i tried when i set it up | 22:42 |
russellb | any SIP client should be fine | 22:42 |
*** shayneburgess has joined #openstack-infra | 22:42 | |
reed | russellb, apparently there isn't much choice, that's what I meant :) | 22:43 |
clarkb | dhellmann: yup they are clean on trusty. do you want me to switch them to trusty now or just wait for next week? | 22:43 |
clarkb | dhellmann: slight chance for regression if we wait but less work >_> | 22:43 |
russellb | well there's more ... that one is just kinda pretty | 22:43 |
fungi | at some point i should figure out a webcam/mic integration with some sip client and see what works. presumably v4l driver streams are legitimate video sources? | 22:43 |
dhellmann | clarkb: I'm OK with waiting if that's less work for you. | 22:43 |
reed | russellb, I and people at FSFE have investigated the A/V conferencing situation, not good results and no changes in more recent times either | 22:43 |
*** emagana has quit IRC | 22:43 | |
dhellmann | clarkb: I think the chance of regressions is pretty small given the rate we're landing significant patches | 22:44 |
clarkb | dhellmann: cool | 22:44 |
russellb | if everyone uses the same video codec, asterisk will do a video conference that switches based on who is talking | 22:44 |
*** jgrimm has quit IRC | 22:44 | |
russellb | you don't get the film strip of everyone else that's there like hangouts though | 22:44 |
dhellmann | clarkb: and thank you for doing this; my testing effort stalled out and then I got distracted by on-boarding and other stuff | 22:44 |
fungi | now we just need the world to settle on a single, non-patent-encumbered video codec. no problem | 22:44 |
pleia2 | heh | 22:44 |
russellb | right | 22:44 |
russellb | turns out, video transcoding for conferencing is quite CPU intensive | 22:44 |
russellb | and a patent minefield | 22:44 |
russellb | so we punted | 22:45 |
clarkb | dhellmann: no problem. I figured I would end up doing this for random projects and oslo projects are the most important ones left | 22:45 |
fungi | as cpus get faster, so do patent lawyers | 22:45 |
*** bknudson has quit IRC | 22:45 | |
clarkb | what happened to that google sponsored one | 22:45 |
fungi | google likes to forget that they're working on free software after a little while (no offense to people working on google-funded free software) | 22:46 |
clarkb | fungi: well ti wasn't free software | 22:46 |
clarkb | it was just not patent encumbered | 22:46 |
russellb | so, some friends of mine at my old company are working on a webrtc PaaS, that does audio / video / filesharing / screensharing | 22:47 |
fungi | ahh. also, did nobody like ogg theora? | 22:47 |
jeblair | there is an authn issue with the sip account | 22:47 |
russellb | wonder if we could build something on that for our needs | 22:47 |
russellb | but that sounds like work | 22:47 |
russellb | jeblair: ah, that would do it | 22:47 |
jeblair | i'm wondering if i botched something during heartbleed | 22:48 |
fungi | jeblair: it was a bit of a scramble, so i wouldn't be surprised | 22:48 |
russellb | doubt that's using TLS ... | 22:48 |
*** sabeen has quit IRC | 22:48 | |
jeblair | russellb: we changed all our passwords because they were potentially in the memory space of a vulnerable tls server | 22:48 |
russellb | unless you mean cycling passwords | 22:48 |
russellb | ah right | 22:48 |
fungi | russellb: we could have simply botched some credentials (reset them in one place but not anotherm et cetera) | 22:48 |
jeblair | russellb: it's was extremely painful | 22:48 |
*** homeless has quit IRC | 22:48 | |
russellb | :( | 22:49 |
*** crc32 has joined #openstack-infra | 22:49 | |
* fungi is depressed that the latest entry on http://www.theora.org/news/ is now almost 5 years old | 22:50 | |
reed | fungi, didn't they publicly announce the defeat and started working on next gen codecs? | 22:51 |
clarkb | dhellmann: and those projects apepars tox==1.7.2 clean. going to check oslo* against tox==1.7.2 now | 22:51 |
fungi | reed: wouldn't surprise me. i have only so many hours in the day to read things, so probably missed it | 22:52 |
*** markmcclain1 has quit IRC | 22:52 | |
jeblair | russellb, fungi: yep, i did indeed botch it; fixing | 22:52 |
russellb | jeblair: yay thanks | 22:52 |
* reed trying ekiga to connect to pbx.openstack.org, it requires a username | 22:52 | |
*** emagana has joined #openstack-infra | 22:52 | |
russellb | yeah, lots of clients assume you need an account somewhere | 22:52 |
jeblair | reed, russellb: i think 'openstack' will work | 22:52 |
reed | last time i tried, all VoIP clients on Android have the same issue: they require an account | 22:52 |
jeblair | i think we made a simple account to deal with that | 22:53 |
*** andreaf has quit IRC | 22:53 | |
russellb | jeblair: we did talk about adding a generic account at one point didn't we ... | 22:53 |
reed | jeblair, tried openstack, Unauthorized | 22:53 |
* russellb checks the config | 22:53 | |
russellb | reed: openstack, pw: openstack | 22:53 |
*** dangers is now known as dangers_away | 22:54 | |
reed | could not register, forbidden | 22:54 |
reed | different error than leaving the pwd empty | 22:54 |
jeblair | reed: i see a "Wrong password" entry in the log | 22:55 |
reed | probably that's related to what jeblair is doing | 22:55 |
russellb | yeah, you don't want it to register | 22:55 |
russellb | ideally | 22:55 |
russellb | but it can use that auth to make calls | 22:55 |
russellb | jeblair: needs host=dynamic to allow register ... but it's going to get confused and weird if more than 1 is trying to do that | 22:55 |
reed | ekiga doesn't give me other options | 22:56 |
reed | ok, I'll bite the bullet and get jitsi | 22:56 |
openstackgerrit | Cedric Brandily proposed a change to openstack-infra/git-review: Define -T/--notopic to disable review submit with topic https://review.openstack.org/114038 | 22:56 |
jeblair | russellb, reed: pstn works now | 22:57 |
krotscheck | clarkb: Ok, so for some reason rabbit is rejecting all connections from users who are not admins. | 22:58 |
*** chuckC has quit IRC | 22:58 | |
*** Ish__ has joined #openstack-infra | 22:58 | |
russellb | jeblair: yay thanks | 22:58 |
krotscheck | clarkb: I’m going to hop over to #rabbitmq and ask them wtf is up. | 22:58 |
*** chuckC has joined #openstack-infra | 22:58 | |
reed | jeblair, yeah, cool | 22:58 |
clarkb | krotscheck: ok | 22:59 |
*** andreaf has joined #openstack-infra | 22:59 | |
HenryG | Can someone clue me in here? I made this change: https://review.openstack.org/112326 but it does not seem to have the desired effect. | 22:59 |
fungi | yep, no more fast busy on that dial-in number | 22:59 |
russellb | how about slow busy? | 23:00 |
clarkb | HenryG: are the files located in the proper location? | 23:00 |
russellb | :-p | 23:00 |
clarkb | HenryG: it will only copy them if they are in the workspace root | 23:00 |
*** Ryan_Lane has joined #openstack-infra | 23:00 | |
*** Ryan_Lane has joined #openstack-infra | 23:00 | |
*** otherwiseguy has quit IRC | 23:01 | |
*** chuckC has quit IRC | 23:01 | |
HenryG | clarkb: Don't know, investigating ... | 23:01 |
* anteaya notes how awesome it is to have jenkins results by themselves at the top of the page | 23:02 | |
*** krotscheck has quit IRC | 23:03 | |
*** krotscheck has joined #openstack-infra | 23:03 | |
*** andreaf_ has joined #openstack-infra | 23:04 | |
clarkb | HenryG: they need to be named properly too, they are the same files as copied by unittests to subunit_log.txt iirc and results.html | 23:04 |
HenryG | clarkb: Looking in python-jobs.yaml, which is the bit that renames/moves the files? | 23:06 |
*** andreaf has quit IRC | 23:07 | |
HenryG | clarkb: The functional job should be very much the same as the python27 unit test job. | 23:07 |
clarkb | HenryG: the run-unittests.sh script is the thing that renames/moves the files | 23:08 |
*** markmcclain has joined #openstack-infra | 23:08 | |
clarkb | HenryG: you might even want to use that script to do your tox run | 23:08 |
ZZelle | Hi everyone | 23:10 |
ZZelle | Great new gerrit layout! | 23:10 |
ZZelle | anteaya, hi | 23:10 |
*** dguerri is now known as dguerri`afk | 23:11 | |
clarkb | dhellmann: https://review.openstack.org/114052 do we backport incubator changes? | 23:11 |
clarkb | dims_: ^ | 23:12 |
clarkb | dhellmann: dims_ we don't do stable branches in incubator right? | 23:13 |
dims_ | clarkb: right | 23:13 |
clarkb | thanks | 23:15 |
clarkb | oslo.messaging has an issue too but its only one test so I think I can just fix it correclty | 23:15 |
openstackgerrit | A change was merged to openstack/requirements: Rename bash8 to bashate (part 1) https://review.openstack.org/113080 | 23:15 |
HenryG | clarkb: I think run-unittests.sh will work if I am allowed to tweak it. The venv doesn't start with "py". | 23:16 |
clarkb | HenryG: oh right, no we shouldn't tweak that at least not without some planning | 23:17 |
clarkb | HenryG: we will need to update a lot of things and do it in multiple steps to make that change without issues | 23:17 |
openstackgerrit | OpenStack Proposal Bot proposed a change to openstack-infra/devstack-gate: Updated from global requirements https://review.openstack.org/114054 | 23:17 |
clarkb | huh when did d-g grow test-requierments.txt? | 23:18 |
clarkb | maybe we shouldn't remove it from teh requirements project list then | 23:18 |
anteaya | ZZelle: hi I am being kicked out of the building at cinder sprint | 23:18 |
anteaya | tty tomorrow | 23:18 |
*** eharney has quit IRC | 23:18 | |
ZZelle | anteaya, ? | 23:19 |
*** dimtruck is now known as zz_dimtruck | 23:19 | |
ZZelle | anteaya, could we assume that all third party CIs name ends with " CI" ? | 23:20 |
*** gargola has quit IRC | 23:20 | |
openstackgerrit | ChingWei Chang proposed a change to openstack-infra/config: new project puppet-n1k-vsm creation https://review.openstack.org/102662 | 23:21 |
fungi | ZZelle: that's the intent with the rename | 23:22 |
fungi | ZZelle: i hope to be able to clean up my e-mail delivery filters soon, taking advantage of that | 23:22 |
*** armax has quit IRC | 23:23 | |
ZZelle | fungi, me too :), great job on the gerrit layout! | 23:23 |
*** otherwiseguy has joined #openstack-infra | 23:23 | |
*** tonytan4ever has quit IRC | 23:23 | |
*** jedimike has quit IRC | 23:24 | |
clarkb | dims_: dhellmann https://review.openstack.org/114079 | 23:26 |
openstackgerrit | Cedric Brandily proposed a change to openstack-infra/git-review: Define -T/--notopic to disable review submit with topic https://review.openstack.org/114038 | 23:29 |
reed | fungi, since https://review.openstack.org/#/c/113345/ does it mean that openstackid now points to the 'real life' members db? | 23:32 |
reed | whoever modified Gerrit UI today: thank you! | 23:32 |
fungi | reed: i don't know... i was told there were new credentials/database server to point the dev server at, but as to whether that's the live silverstripe backend db, a copy of it, or some dummy data i couldn't guess | 23:33 |
reed | fungi, thanks, I'll ask jimmy | 23:33 |
clarkb | dhellmann: dims_ ok those were the only two oslo projects that got cranky under tox1.7.2 with random python hash seed | 23:36 |
dims_ | nice! thanks | 23:36 |
*** chuckC has joined #openstack-infra | 23:37 | |
* fungi must transition to evening tasks now... dance card is oversubscribed already | 23:37 | |
clarkb | fungi: have fun | 23:38 |
fungi | thx | 23:38 |
lifeless | clarkb: sadface, py26 failed again with http://logs.openstack.org/49/80449/23/check/gate-pbr-python26/ac41f37/ TimeoutException | 23:43 |
lifeless | mordred: ^ | 23:43 |
lifeless | mordred: can you log into a node again and see what the next bit of upness is? | 23:43 |
* krotscheck nukes rabbitmq from orbit | 23:44 | |
clarkb | krotscheck: :) I figured you would eventually but not this soon | 23:44 |
*** emagana has quit IRC | 23:44 | |
clarkb | lifeless: is it possible there is a bug in the test? | 23:44 |
krotscheck | clarkb: The complete lack of any kind of community support is what’s getting me. | 23:44 |
clarkb | lifeless: and can we just give it a gpg key as a fixture and move on? | 23:44 |
*** emagana has joined #openstack-infra | 23:45 | |
*** ominakov has quit IRC | 23:45 | |
*** Sukhdev has joined #openstack-infra | 23:45 | |
lifeless | clarkb: its passing on py27 | 23:46 |
lifeless | clarkb: it passed for mordred when he poked at it directly w/haveged running | 23:46 |
clarkb | lifeless: oh, so maybe haveged isn't running now? even though we tried to make it so (if eg you ran on an old image build) | 23:47 |
lifeless | clarkb: right | 23:47 |
lifeless | clarkb: thats what I'm wondering | 23:47 |
clarkb | hrm image is 8 hours old | 23:47 |
clarkb | test ran 3 hours ago | 23:47 |
mordred | krotscheck: it seems that it seems much better than it actually is | 23:48 |
clarkb | I doubt that it ran on the old image | 23:48 |
mordred | krotscheck: but I've never used it in anger | 23:48 |
krotscheck | mordred: Basically, I cannot connect to rabbit unless the user is an admin. | 23:48 |
lifeless | clarkb: I really don't like the idea of shipping an actual private key in the source | 23:48 |
lifeless | clarkb: so I'd like to try to avoid that | 23:48 |
clarkb | lifeless: even a dummy key? | 23:48 |
lifeless | clarkb: its still a key | 23:49 |
lifeless | clarkb: users do dumb things | 23:49 |
mordred | krotscheck: maybe that's a feature | 23:49 |
lifeless | krotscheck: cannot connect to rabbit how - rabbitmqctl or amqp ? | 23:49 |
*** emagana has quit IRC | 23:49 | |
krotscheck | lifeless: amqp | 23:49 |
lifeless | krotscheck: have you created the users you need? | 23:50 |
krotscheck | yep | 23:50 |
mordred | krotscheck: lifeless can fix your problems | 23:50 |
lifeless | krotscheck: have you granted them access to your vhost? | 23:50 |
lifeless | krotscheck: and to the queues / exchanges you want them to use? | 23:50 |
krotscheck | lifeless: lemme show you where that’s all set up... | 23:50 |
clarkb | oh you know what | 23:50 |
* clarkb looks at haveged change | 23:50 | |
krotscheck | lifeless: So there’s this - http://git.openstack.org/cgit/openstack-infra/puppet-storyboard/tree/manifests/rabbit.pp | 23:51 |
*** alexandra_ has joined #openstack-infra | 23:51 | |
clarkb | mordred: the haveged change won't actually ensure haveged is running | 23:51 |
*** jamielennox|away is now known as jamielennox | 23:51 | |
mordred | clarkb: it won't? | 23:51 |
krotscheck | lifeless: Which, from the documentation, tells me that the rabbitmq_user should be able to connect to virtualhost / and create all exchanges that it needs to. | 23:51 |
clarkb | mordred: lifeless: you are ensuring that state at image build time, this has nothing to do with what it does after a reboot | 23:51 |
clarkb | I think you need to enable=> true orwhatever | 23:52 |
mordred | o for the love of GOD | 23:52 |
clarkb | basically tell centos init to start that service at boot which is different than starting the service with puppet | 23:52 |
lifeless | mordred: love of haveged please | 23:52 |
clarkb | I am going to confirm by logging onto a host | 23:52 |
mordred | k. patch coming | 23:52 |
*** alexandra_ is now known as asettle | 23:52 | |
clarkb | mordred: yup thats the issue | 23:53 |
clarkb | no haveged running | 23:53 |
clarkb | jesusaurus: haev you run into admin only being able to connect to rabbitmq? | 23:54 |
jesusaurus | clarkb: no, i havent | 23:54 |
krotscheck | lifeless: Any ideas? I’ve tried pretty much everything I can think of. | 23:54 |
openstackgerrit | Monty Taylor proposed a change to openstack-infra/config: Enable haveged https://review.openstack.org/114082 | 23:55 |
lifeless | krotscheck: check the output of rabbitmqctl list_permissions | 23:55 |
lifeless | krotscheck: lets me sure it has actually worked ;) | 23:55 |
clarkb | I suppose you need me to do that? | 23:55 |
lifeless | krotscheck: also make sure you're connecting to the rabbit you think you are, not e.g. some other one on the same machine | 23:55 |
mordred | clarkb: ^^ | 23:56 |
krotscheck | lifeless: storyboard.*.*.* | 23:56 |
NobodyCam | what is the correct pip version for mac 1.5.6 ? | 23:56 |
*** markmcclain1 has joined #openstack-infra | 23:56 | |
clarkb | krotscheck: oh you are able to do this? | 23:56 |
lifeless | krotscheck: ok, and password is correct? | 23:56 |
*** markmcclain has quit IRC | 23:56 | |
clarkb | krotscheck: I couldn't remember if you have account | 23:56 |
krotscheck | lifeless: And I think I’m connecting to the correct one, given that it all works if I set the user to admin, but it doesn’t if I don't. | 23:56 |
mordred | NobodyCam: 1.5.6 is great | 23:56 |
NobodyCam | getting odd error : http://paste.openstack.org/show/94707/ | 23:56 |
NobodyCam | hey mordred :) | 23:56 |
jesusaurus | mordred: "enable" means start-on-boot, but won't make sure that its actually running | 23:56 |
clarkb | jesusaurus: ya thats what we want | 23:56 |
krotscheck | clarkb: I’m on a VM | 23:56 |
clarkb | krotscheck: oh that works | 23:57 |
*** imcsk8_ has joined #openstack-infra | 23:57 | |
clarkb | jesusaurus: we don't run puppet on the test slaves, we only do it at image build time | 23:57 |
clarkb | jesusaurus: so we want to make sure haveged is enabled on boot | 23:57 |
*** unicell has joined #openstack-infra | 23:58 | |
mordred | NobodyCam: try "pip install wheel" | 23:58 |
mordred | NobodyCam: and try again | 23:58 |
clarkb | mordred: approved | 23:58 |
jesusaurus | clarkb: mordred: which slaves do you want that running on? | 23:58 |
mordred | NobodyCam: also, are you sure taht you're running pip 1.5.6 ? | 23:58 |
clarkb | jesusaurus: all of them | 23:58 |
mordred | jesusaurus: all of them | 23:58 |
clarkb | jesusaurus: its an entropy building service, vms have bad entropy so we want it everywhere | 23:58 |
mordred | jesusaurus: it's already running on ubuntu because ubuntu enables service on install | 23:58 |
mordred | jesusaurus: but centos does not enable on install | 23:58 |
*** shayneburgess has quit IRC | 23:59 | |
*** gokrokve has quit IRC | 23:59 | |
*** flwang_ has joined #openstack-infra | 23:59 | |
jesusaurus | clarkb: mordred: then you want slave_common, slave looks like its only included in pypi_slave and proposal_slave | 23:59 |
*** imcsk8 has quit IRC | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!