| *** sdake_ has joined #openstack-infra | 00:00 | |
| *** dannywilson has quit IRC | 00:00 | |
| lifeless | sigmavirus24_awa: I nearly do. It will be fun to see how many people my first release of mock breaks | 00:00 |
|---|---|---|
| lifeless | Things I never want to see in a commit message. " a mock function call. This will help to write better doctests. | 00:01 |
| *** esmute__ has joined #openstack-infra | 00:01 | |
| *** dims has joined #openstack-infra | 00:01 | |
| *** sdake has quit IRC | 00:02 | |
| ianw | do we know that etherpad seems to be a bit stuck? | 00:03 |
| clarkb | ianw: no | 00:03 |
| clarkb | ianw: https://etherpad.openstack.org/p/neutron-partial-upgrades is working for me | 00:04 |
| ianw | https://etherpad.openstack.org/p/oom-in-rax-centos7-CI-job is being weird | 00:04 |
| ianw | for me, anyway | 00:04 |
| *** esmute__ has quit IRC | 00:05 | |
| lifeless | ianw: ahha. pounce! - dunno if you saw my request for devstack +2A's | 00:05 |
| *** laughterwym has joined #openstack-infra | 00:05 | |
| jeblair | ianw: i think that one is working for me | 00:06 |
| ianw | i dunno, now it seems to work. it was stuck loading, then it flashed some sort of error page quickly (it said something like press f5 to refresh), now it works | 00:06 |
| openstackgerrit | David Shrewsbury proposed openstack-infra/shade: Raise exception for nova egress secgroup rule https://review.openstack.org/198871 | 00:06 |
| ianw | lifeless: is this the backports in the queue? | 00:06 |
| clarkb | hrm maybe it restarted | 00:07 |
| jeblair | ianw: http://cacti.openstack.org/cacti/graph_view.php?action=tree&tree_id=1&leaf_id=12 | 00:07 |
| jeblair | ianw: something definitiely is/was going on | 00:07 |
| *** _amrith_ is now known as amrith | 00:07 | |
| Shrews | mordred: greghaynes: turns out https://review.openstack.org/198871 had a genuine pep8 error (line too long). not sure where the output went | 00:07 |
| jeblair | clarkb: did you get a chance to test the gertty fix? | 00:07 |
| clarkb | jeblair: not yet let me do that now | 00:07 |
| lifeless | ianw: yes. Needed pre-conditions to enable fixing the check-*-requirements jobs in kilo and juno | 00:08 |
| *** chlong has quit IRC | 00:08 | |
| greghaynes | Shrews: psh, having fail output removes too much of the fun of debugging | 00:08 |
| *** sdake has joined #openstack-infra | 00:09 | |
| *** wenlock has quit IRC | 00:09 | |
| *** Sukhdev has quit IRC | 00:10 | |
| *** fallenpe_ has quit IRC | 00:11 | |
| clarkb | jeblair: I get some sql error now, but I think maybe it "corrupted" its db relative to where it expects it to be via the prvious fail | 00:11 |
| jeblair | clarkb: urg. sorry. i might be able to help you reverse that, or we could declare it fixed if you want to continue using your new db. | 00:12 |
| clarkb | I am happy to declare it fixed | 00:12 |
| openstackgerrit | Merged stackforge/gertty: Fix get_repo call even more https://review.openstack.org/198875 | 00:13 |
| jeblair | so declared | 00:13 |
| *** r-daneel has quit IRC | 00:13 | |
| *** sdake_ has quit IRC | 00:13 | |
| anteaya | its fixed | 00:13 |
| anteaya | so glad I was here for it | 00:13 |
| clarkb | I can confirm that that change works with my working db | 00:14 |
| *** baoli has quit IRC | 00:14 | |
| *** baoli has joined #openstack-infra | 00:15 | |
| *** yamamoto has joined #openstack-infra | 00:17 | |
| openstackgerrit | Paul Van Eck proposed openstack-infra/puppet-refstack: Add .gitreview & fix puppet-lint errors/warnings https://review.openstack.org/198910 | 00:17 |
| *** rfolco has joined #openstack-infra | 00:18 | |
| openstackgerrit | Paul Van Eck proposed openstack-infra/puppet-refstack: Add .gitreview & fix puppet-lint errors/warnings https://review.openstack.org/198910 | 00:20 |
| *** asettle has joined #openstack-infra | 00:20 | |
| *** asettle has quit IRC | 00:20 | |
| *** asettle has joined #openstack-infra | 00:21 | |
| *** yamamoto has quit IRC | 00:21 | |
| jeblair | clarkb: i wanted to make sure that https://review.openstack.org/189316 looked good before removing the d-g jobs (even though the depends-on goes the other way just for safety) | 00:22 |
| *** esmute__ has joined #openstack-infra | 00:22 | |
| jeblair | clarkb: there was stuff happening at the time which i think is cleared up now, so i have rechecked that, and assuming it comes back clean, i'll unwip the job removal change | 00:22 |
| *** esmute__ has quit IRC | 00:24 | |
| *** tsekiyama has quit IRC | 00:26 | |
| *** esmute__ has joined #openstack-infra | 00:27 | |
| *** HGarg has joined #openstack-infra | 00:27 | |
| *** HGarg has left #openstack-infra | 00:27 | |
| *** kgiusti has left #openstack-infra | 00:32 | |
| *** btully has joined #openstack-infra | 00:32 | |
| *** ddieterly has joined #openstack-infra | 00:32 | |
| jeblair | ianw: whatever was affecting etherpad, it seems to have been transient | 00:32 |
| *** esmute__ has quit IRC | 00:33 | |
| *** esmute__ has joined #openstack-infra | 00:34 | |
| *** heyongli has quit IRC | 00:35 | |
| *** heyongli has joined #openstack-infra | 00:35 | |
| *** tqtran has quit IRC | 00:39 | |
| lifeless | spekaing of gertty | 00:41 |
| lifeless | jeblair: when I review a change, it gets hidden (as I expect, I have show-reviewed-off), then it gets shown again | 00:42 |
| lifeless | jeblair: but the only change to the review I can see is that my comment has now been applied. | 00:42 |
| lifeless | jeblair: is this a known defect (do you agree that its a defect?) or would you like me to file a bug | 00:42 |
| jeblair | lifeless: ah, i noticed that too recently; i think it's something i overlooked in the change to add 'held changes' | 00:43 |
| jeblair | lifeless: i agree it's a defect; it's on my mental bug list; a formal bug would not hurt, but i'm also not likely to forget it if you don't feel like bothering | 00:43 |
| lifeless | thanks | 00:45 |
| *** mayurig has quit IRC | 00:46 | |
| lifeless | jeblair: https://storyboard.openstack.org/#!/story/2000313 | 00:46 |
| *** dims has quit IRC | 00:50 | |
| *** hdd has joined #openstack-infra | 00:51 | |
| *** fawadkhaliq has joined #openstack-infra | 00:53 | |
| *** jyuso1 has quit IRC | 00:54 | |
| *** jyuso1 has joined #openstack-infra | 00:54 | |
| lifeless | hmm | 00:55 |
| lifeless | I see some git yak shaving in my future | 00:55 |
| *** fawadkhaliq has quit IRC | 00:56 | |
| *** vnod has quit IRC | 00:57 | |
| *** Swami has quit IRC | 00:57 | |
| ianw | jeblair: yeah, page has been there for a while and i haven't been kicked off. glitch in the matrix | 00:58 |
| *** dims has joined #openstack-infra | 00:58 | |
| *** nelsnels_ has joined #openstack-infra | 00:59 | |
| *** nelsnels_ has quit IRC | 00:59 | |
| *** woodster_ has quit IRC | 01:01 | |
| *** nelsnelson has quit IRC | 01:02 | |
| *** amotoki has joined #openstack-infra | 01:02 | |
| *** zz_dimtruck is now known as dimtruck | 01:03 | |
| *** cody-somerville has quit IRC | 01:06 | |
| *** esmute__ has quit IRC | 01:07 | |
| *** erlon has quit IRC | 01:10 | |
| *** jasondotstar has quit IRC | 01:12 | |
| *** achanda has joined #openstack-infra | 01:13 | |
| *** gyee has quit IRC | 01:13 | |
| *** shashank_hegde has quit IRC | 01:14 | |
| *** Longgeek has joined #openstack-infra | 01:18 | |
| *** weshay has joined #openstack-infra | 01:21 | |
| *** mtanino has quit IRC | 01:22 | |
| *** yamamoto has joined #openstack-infra | 01:22 | |
| *** jasondotstar has joined #openstack-infra | 01:25 | |
| *** yamamoto has quit IRC | 01:27 | |
| *** chlong has joined #openstack-infra | 01:30 | |
| openstackgerrit | Merged openstack/diskimage-builder: Remove deprecated disk-image-get-kernel https://review.openstack.org/198810 | 01:37 |
| *** patrickeast has quit IRC | 01:38 | |
| *** patrickeast has joined #openstack-infra | 01:41 | |
| *** asettle has quit IRC | 01:45 | |
| jhesketh | fungi: ping | 01:47 |
| *** cody-somerville has joined #openstack-infra | 01:47 | |
| *** ari has joined #openstack-infra | 01:47 | |
| *** Longgeek has quit IRC | 01:51 | |
| *** tonytan4ever has joined #openstack-infra | 01:52 | |
| *** btully has quit IRC | 01:53 | |
| *** patrickeast has quit IRC | 01:55 | |
| openstackgerrit | fumihiko kakuma proposed openstack-infra/project-config: Make gate-networking-ofagent-python34 non-voting https://review.openstack.org/198657 | 02:00 |
| openstackgerrit | Merged openstack-infra/shade: Raise exception for nova egress secgroup rule https://review.openstack.org/198871 | 02:04 |
| *** patrickeast has joined #openstack-infra | 02:08 | |
| *** puranamr has quit IRC | 02:09 | |
| openstackgerrit | Chris Hoge proposed openstack-infra/project-config: Add openstack bot to #openstack-diversity channel https://review.openstack.org/198936 | 02:10 |
| *** yamahata has quit IRC | 02:10 | |
| openstackgerrit | Joshua Hesketh proposed openstack-infra/project-config: Fix ACL's on projects with puppet-system https://review.openstack.org/198938 | 02:12 |
| *** PinkFreud has joined #openstack-infra | 02:13 | |
| hogepodge | clarkb: the patch configures accessbot. Do I need to somehow invite it to the room too? | 02:17 |
| clarkb | nope just need to give it perms whoch chanserv says you have done | 02:17 |
| clarkb | but you also need meetbot if you want those services primarily logs | 02:18 |
| *** esmute__ has joined #openstack-infra | 02:18 | |
| clarkb | all the accessbot does us control who has access | 02:20 |
| clarkb | mostly ops | 02:20 |
| *** dboik_ has joined #openstack-infra | 02:24 | |
| *** spzala has quit IRC | 02:24 | |
| *** ddieterly has quit IRC | 02:25 | |
| clarkb | hogepodge what irc things are you wanting? | 02:27 |
| hogepodge | clarkb: meetbot and logging, and anything else I might me missing | 02:27 |
| hogepodge | clarkb: there's no repository, so gerritbot not needed | 02:28 |
| clarkb | the other common thibg is gerritbot ya ok | 02:28 |
| clarkb | so just need to add to the meetbot list. thats in system-config in eavesdrop.pp iirc | 02:28 |
| openstackgerrit | Chris Hoge proposed openstack-infra/system-config: Add eavesdrop to #openstack-diversity channel https://review.openstack.org/198941 | 02:29 |
| hogepodge | clarkb: :-) | 02:30 |
| *** otter768 has quit IRC | 02:30 | |
| *** dboik_ has quit IRC | 02:31 | |
| *** esmute__ has quit IRC | 02:32 | |
| *** yamamoto has joined #openstack-infra | 02:33 | |
| *** pal has joined #openstack-infra | 02:34 | |
| *** ari has quit IRC | 02:35 | |
| *** oomichi has joined #openstack-infra | 02:37 | |
| *** nijaba has joined #openstack-infra | 02:38 | |
| *** nijaba has joined #openstack-infra | 02:38 | |
| openstackgerrit | Merged openstack-infra/system-config: Rename pypi_slave class to release_slave https://review.openstack.org/198444 | 02:38 |
| *** esmute__ has joined #openstack-infra | 02:39 | |
| openstackgerrit | Merged openstack-infra/system-config: Add release.slave.openstack.org https://review.openstack.org/198445 | 02:44 |
| *** asselin_ has joined #openstack-infra | 02:45 | |
| lifeless | I always forget | 02:51 |
| lifeless | whats | 02:51 |
| *** BharatK has joined #openstack-infra | 02:51 | |
| lifeless | : | 02:51 |
| lifeless | on its own in a shell script do ? | 02:51 |
| *** patrickeast_ has joined #openstack-infra | 02:51 | |
| *** pal has quit IRC | 02:51 | |
| *** patrickeast has quit IRC | 02:51 | |
| *** patrickeast_ is now known as patrickeast | 02:51 | |
| larsks | lifeless: it does nothing, but arguments are evaluated, unlike a comment. It is sort of like the "true" command. | 02:55 |
| larsks | So while this does nothing: | 02:56 |
| larsks | # ${myvar:=default} | 02:56 |
| larsks | This will set a myvar to 'default' if it isn't set: | 02:57 |
| larsks | : ${myvar:=default} | 02:57 |
| *** tonytan4ever has quit IRC | 02:58 | |
| lifeless | larsks: what about -just- : ? | 02:58 |
| larsks | Then it does nothing. | 02:58 |
| lifeless | k, thanks. | 02:59 |
| *** weshay has quit IRC | 03:02 | |
| *** koolhead17 has joined #openstack-infra | 03:05 | |
| openstackgerrit | Sean M. Collins proposed stackforge/gertty: Catch errno.EPERM https://review.openstack.org/198945 | 03:11 |
| *** Longgeek has joined #openstack-infra | 03:18 | |
| *** ddieterly has joined #openstack-infra | 03:26 | |
| openstackgerrit | Guojian Shao proposed openstack-infra/project-config: test against neutron for osc gate job https://review.openstack.org/198948 | 03:26 |
| *** asettle has joined #openstack-infra | 03:28 | |
| *** Longgeek has quit IRC | 03:29 | |
| *** rkukura has joined #openstack-infra | 03:30 | |
| *** ddieterly has quit IRC | 03:30 | |
| *** julim has joined #openstack-infra | 03:31 | |
| *** rkukura has quit IRC | 03:33 | |
| *** dims has quit IRC | 03:39 | |
| *** heyongli has quit IRC | 03:44 | |
| *** heyongli has joined #openstack-infra | 03:45 | |
| *** mayurig has joined #openstack-infra | 03:46 | |
| *** mayurig has quit IRC | 03:51 | |
| *** ianw has quit IRC | 03:56 | |
| *** ianw has joined #openstack-infra | 03:56 | |
| *** koolhead17 has quit IRC | 04:07 | |
| openstackgerrit | Merged openstack-infra/irc-meetings: Add id and url for third party meetings. https://review.openstack.org/198722 | 04:08 |
| *** Krinkle_ is now known as Krinkle | 04:10 | |
| *** btully has joined #openstack-infra | 04:12 | |
| *** yamada-h has joined #openstack-infra | 04:14 | |
| *** pal has joined #openstack-infra | 04:17 | |
| *** julim has quit IRC | 04:19 | |
| *** xyang1 has quit IRC | 04:24 | |
| *** ddieterly has joined #openstack-infra | 04:27 | |
| *** laughterwym has quit IRC | 04:29 | |
| *** otter768 has joined #openstack-infra | 04:31 | |
| *** ddieterly has quit IRC | 04:31 | |
| *** yamada-h has quit IRC | 04:32 | |
| *** otter768 has quit IRC | 04:36 | |
| *** pal_ has joined #openstack-infra | 04:36 | |
| openstackgerrit | Merged openstack-infra/project-config: Add openstack bot to #openstack-diversity channel https://review.openstack.org/198936 | 04:38 |
| *** Sukhdev has joined #openstack-infra | 04:38 | |
| *** pal has quit IRC | 04:39 | |
| *** dims has joined #openstack-infra | 04:40 | |
| *** BharatK has quit IRC | 04:41 | |
| *** dims_ has joined #openstack-infra | 04:41 | |
| *** baoli has quit IRC | 04:43 | |
| *** dims has quit IRC | 04:45 | |
| *** GonZo2000 has joined #openstack-infra | 04:46 | |
| *** dims_ has quit IRC | 04:46 | |
| GonZo2000 | Hi can i have multiple jenkins servers configured on JJB ? | 04:50 |
| *** hdd has quit IRC | 04:51 | |
| *** ari has joined #openstack-infra | 04:55 | |
| clarkb | I dont think so, we do it by runnig it on each server independenrly | 04:59 |
| clarkb | but iirc the config ini file accepts just one | 04:59 |
| GonZo2000 | thanks clarkb that was my idea from reading the docs. | 05:00 |
| GonZo2000 | maybe i can use multiple config files | 05:00 |
| *** ari has quit IRC | 05:00 | |
| *** yfried has quit IRC | 05:01 | |
| *** deepakcs has joined #openstack-infra | 05:04 | |
| *** asselin_ has quit IRC | 05:05 | |
| *** devanand_ has joined #openstack-infra | 05:05 | |
| *** deepakcs has quit IRC | 05:06 | |
| *** BharatK has joined #openstack-infra | 05:08 | |
| *** patrickeast has quit IRC | 05:13 | |
| *** gildub has joined #openstack-infra | 05:15 | |
| *** madong_ is now known as larainema | 05:20 | |
| openstackgerrit | YAMAMOTO Takashi proposed openstack-infra/project-config: Add python3 jobs for neutron-lbaas https://review.openstack.org/198611 | 05:21 |
| *** e0ne has joined #openstack-infra | 05:22 | |
| *** _nadya_ has joined #openstack-infra | 05:23 | |
| *** achanda has quit IRC | 05:23 | |
| *** dannywilson has joined #openstack-infra | 05:23 | |
| *** ildikov has joined #openstack-infra | 05:23 | |
| *** deepakcs has joined #openstack-infra | 05:25 | |
| *** dannywilson has quit IRC | 05:25 | |
| *** dannywilson has joined #openstack-infra | 05:26 | |
| *** deepakcs has quit IRC | 05:26 | |
| *** Longgeek has joined #openstack-infra | 05:26 | |
| *** ddieterly has joined #openstack-infra | 05:28 | |
| openstackgerrit | YAMAMOTO Takashi proposed openstack-infra/project-config: Add python3-jobs for networking-midonet https://review.openstack.org/198960 | 05:28 |
| ianw | far too much debugging later ... random centos failures are because "devstack-centos7-1435932846.template.openstack.org.novalocal" is too long a hostname to put in a SSL certificate | 05:30 |
| rm_work | lol | 05:30 |
| rm_work | that's an awesome reason for failure | 05:30 |
| greghaynes | ianw: that might go away - we just released glean which will now be setting hostname to just devstack-centos7-1435932846 | 05:32 |
| greghaynes | once the images are rebuilt | 05:32 |
| *** ddieterly has quit IRC | 05:32 | |
| *** flepied has quit IRC | 05:32 | |
| *** esmute__ has quit IRC | 05:33 | |
| *** gmann has quit IRC | 05:33 | |
| ianw | greghaynes: yeah; i mean i guess really it's a rpm (mod_ssl) bug, that's trying to set this up | 05:33 |
| greghaynes | turns out when we make really long hostnames we break a whole lot of things ;) | 05:34 |
| ianw | greghaynes: so will glean be on all images or is it part of dib setup? | 05:35 |
| greghaynes | ianw: all of them | 05:36 |
| greghaynes | if the build and upload works ;) | 05:36 |
| greghaynes | oh, actually, IMBW | 05:36 |
| greghaynes | I think it might just be dib so the hp ones | 05:36 |
| *** laughterwym has joined #openstack-infra | 05:37 | |
| greghaynes | ianw: were you seeing that issue in rax too? | 05:38 |
| ianw | greghaynes: i do think it's only hpcloud | 05:39 |
| ianw | what's with the novalocal on the end? | 05:39 |
| greghaynes | domain? | 05:40 |
| ianw | greghaynes: yeah, but after templte.openstack.org | 05:41 |
| greghaynes | yea, so the issue is that glean is setting a full fqdn for the hostname which should really just be a bottom level domain, so its setting devstack-centos7-1435932846.template.openstack.org | 05:41 |
| greghaynes | so if something else has the idea that novalocal is the domain, itll append that too | 05:42 |
| *** BharatK has quit IRC | 05:42 | |
| *** unicell has joined #openstack-infra | 05:44 | |
| *** e0ne has quit IRC | 05:44 | |
| clarkb | novalocal is what nova metadata says is your domain | 05:45 |
| ianw | greghaynes: ok, so i'll file the mod_ssl bug, and keep an eye on the images tomorrow and see what's going on | 05:45 |
| *** juggler_ has joined #openstack-infra | 05:46 | |
| *** juggler has quit IRC | 05:49 | |
| *** Trozz_ is now known as Trozz | 05:50 | |
| *** yamahata has joined #openstack-infra | 05:50 | |
| *** ig0r_ has joined #openstack-infra | 05:52 | |
| *** Longgeek has quit IRC | 05:52 | |
| *** deepakcs has joined #openstack-infra | 05:52 | |
| *** heyongli has quit IRC | 05:53 | |
| *** heyongli has joined #openstack-infra | 05:54 | |
| *** _nadya_ has quit IRC | 05:54 | |
| openstackgerrit | BorneMace proposed openstack-infra/project-config: Added new validate-contents test for kolla https://review.openstack.org/198968 | 05:54 |
| *** abregman has joined #openstack-infra | 05:54 | |
| *** ig0r__ has quit IRC | 05:55 | |
| *** Longgeek has joined #openstack-infra | 05:56 | |
| *** unicell has quit IRC | 05:56 | |
| *** yamamoto has quit IRC | 05:57 | |
| *** Guest20028 has joined #openstack-infra | 05:58 | |
| *** unicell has joined #openstack-infra | 05:58 | |
| *** fallenpegasus has joined #openstack-infra | 05:59 | |
| *** heyongli has quit IRC | 05:59 | |
| *** yamamoto has joined #openstack-infra | 06:00 | |
| *** heyongli has joined #openstack-infra | 06:00 | |
| *** laughterwym_ has joined #openstack-infra | 06:01 | |
| *** Guest20028 has quit IRC | 06:01 | |
| *** kiran-r has joined #openstack-infra | 06:02 | |
| *** kdas_ has quit IRC | 06:03 | |
| *** laughterwym has quit IRC | 06:04 | |
| *** bmace has joined #openstack-infra | 06:05 | |
| bmace | hey folks, i tossed up a review a couple of mins ago and i am just trying to make sure i have the process / syntax right for this change. https://review.openstack.org/#/c/198968/ | 06:06 |
| *** kdas_ has joined #openstack-infra | 06:06 | |
| bmace | the entry is already added into the tox.ini file in the kolla codebase. | 06:06 |
| sdake | AJeager_ you happen to be around for that one? | 06:07 |
| bmace | it isn't super clear to me if i need to define it in jenkins/jobs/kolla.yaml or not. it doesn't need any extra args or anything. | 06:07 |
| *** BharatK has joined #openstack-infra | 06:08 | |
| sdake | bmace for questions you will probably have to try in the us timezones, for normal reviews, it will follow the normal review process - we don't ping infra unless we have some critical emergency for reviewing a busted-gate patch :) | 06:10 |
| sdake | so feel free to ask in the morning - but for the pinging on the review please let the folks address the reviews using their normal worflows k? | 06:11 |
| *** mrunge has joined #openstack-infra | 06:15 | |
| *** flepied has joined #openstack-infra | 06:16 | |
| *** gmann has joined #openstack-infra | 06:19 | |
| ianw | greghaynes: https://bugzilla.redhat.com/show_bug.cgi?id=1240495 | 06:19 |
| openstack | bugzilla.redhat.com bug 1240495 in httpd "mod_ssl install fails with a long hostname" [Unspecified,New] - Assigned to webstack-team | 06:19 |
| *** bmace is now known as bmace_away | 06:23 | |
| openstackgerrit | YAMAMOTO Takashi proposed openstack-infra/project-config: Remove Midonet v2 jobs https://review.openstack.org/198979 | 06:27 |
| *** _nadya__ has joined #openstack-infra | 06:27 | |
| *** ddieterly has joined #openstack-infra | 06:28 | |
| *** dtantsur|afk is now known as dtantsur | 06:28 | |
| *** armax has quit IRC | 06:29 | |
| *** pal_ has quit IRC | 06:30 | |
| openstackgerrit | Merged openstack-dev/pbr: This documents the use of comments that work between 2.6, 2.7 and 3.4 https://review.openstack.org/193927 | 06:31 |
| *** otter768 has joined #openstack-infra | 06:32 | |
| *** ddieterly has quit IRC | 06:33 | |
| *** Sukhdev has quit IRC | 06:34 | |
| *** dimtruck is now known as zz_dimtruck | 06:34 | |
| *** otter768 has quit IRC | 06:37 | |
| *** rossella_s has quit IRC | 06:37 | |
| *** rossella_s has joined #openstack-infra | 06:37 | |
| *** sshnaidm has joined #openstack-infra | 06:38 | |
| *** yfried has joined #openstack-infra | 06:38 | |
| *** yamamoto has quit IRC | 06:41 | |
| *** yfried_ has joined #openstack-infra | 06:43 | |
| *** yfried has quit IRC | 06:43 | |
| *** fifieldt has joined #openstack-infra | 06:43 | |
| *** kdas_ has quit IRC | 06:44 | |
| *** asettle has quit IRC | 06:45 | |
| *** scheuran has joined #openstack-infra | 06:45 | |
| clarkb | bmace the validate contents job template doesnt exist | 06:47 |
| *** kdas_ has joined #openstack-infra | 06:47 | |
| clarkb | that is why jenkins is -1 | 06:47 |
| clarkb | I dont understand us pep8 being replaced? | 06:48 |
| clarkb | why not just fix pep8? its all self testing so write a change or a stack of changes to get it where you want | 06:48 |
| clarkb | no new job necessary | 06:48 |
| *** afazekas has joined #openstack-infra | 06:50 | |
| *** jtomasek has joined #openstack-infra | 06:50 | |
| *** Murad has quit IRC | 06:53 | |
| *** btully has quit IRC | 06:54 | |
| *** k4n0 has joined #openstack-infra | 06:54 | |
| larainema | hi all, if someone can help, when I try to summit patchset to gerrit, I catch this error: http://paste.openstack.org/show/351046/ | 06:54 |
| clarkb | larainema you will need to set your contact info at the location provided | 06:55 |
| openstackgerrit | Dmitry Tantsur proposed openstack-infra/project-config: Add check-ironic-inspector-dsvm-nv to ironic experimental pipeline https://review.openstack.org/198381 | 06:56 |
| larainema | clarkb, i have checked all the info, i thought it correct, but still not work | 06:56 |
| clarkb | https://ask.openstack.org/en/question/56720/cannot-store-contact-information-when-updating-info-in-openstack-gerrit/ may be of help as well | 06:56 |
| *** e0ne has joined #openstack-infra | 06:57 | |
| *** fhubik has joined #openstack-infra | 06:57 | |
| *** soren has joined #openstack-infra | 07:00 | |
| openstackgerrit | Steve Kowalik proposed openstack-infra/project-config: Add a register-zanata-projects.py script and job https://review.openstack.org/198236 | 07:02 |
| clarkb | larainema can you see if that ask url sorts it out? | 07:03 |
| *** e0ne has quit IRC | 07:04 | |
| *** dannywilson has quit IRC | 07:04 | |
| larainema | clarkb, I can succefully connect over ssh, but git review can't work | 07:07 |
| *** shashank_hegde has joined #openstack-infra | 07:08 | |
| clarkb | yes you need to update your contact info at the url provided | 07:08 |
| tchaypo | update it to what? | 07:09 |
| clarkb | if you there and attempt to update it does that fix the issue? if not please see the ask url | 07:09 |
| clarkb | tchaypo whatevr you deem correct, we do t actaully have a way to check it | 07:09 |
| tchaypo | so as long as https://review.openstack.org/#/settings/contact has a non-zero value, it should be fine? | 07:10 |
| tchaypo | or does it need to match your launchpad ID or something like that? | 07:10 |
| *** mpavone has joined #openstack-infra | 07:10 | |
| clarkb | it is fine as long as you get the bit flipped to info updated | 07:11 |
| larainema | clarkb tchaypo, i can update the contact info | 07:11 |
| larainema | and after updated it works | 07:11 |
| *** yfried__ has joined #openstack-infra | 07:11 | |
| *** yfried_ has quit IRC | 07:11 | |
| clarkb | doing so requires a foundation member account with a primary email addr that matches the primary email addr in gerrit | 07:11 |
| clarkb | but thats the only mapping/check | 07:12 |
| clarkb | the ask answer should cover ^ | 07:12 |
| tchaypo | so there’s some check to ensure that the email address matches a foundation member, that only fires if you update the details in gerrit? | 07:13 |
| tchaypo | so if you provide details to gerrit first, then later become a foundation member, you have to make a no-op update in order to trigger a re-check of the details? | 07:13 |
| tchaypo | or am I misunderstanding? | 07:14 |
| clarkb | no it fails to update without a member account | 07:14 |
| clarkb | so its impossible to go that order | 07:14 |
| clarkb | in this case I think larainema just never provided contact info as part of cla signing | 07:15 |
| clarkb | so it just needes doing | 07:15 |
| tchaypo | and yet, if I’ve understood what he’s said, larainema seems to have got into a situation where https://review.openstack.org/#/settings/contact seemed to show info that matched the foundation member details, but didn’t allow for pushing reviews… | 07:16 |
| clarkb | nope it was never updated | 07:17 |
| larainema | tchaypo, I think the issue is i never update the contact info, i just thought only provide the email address | 07:17 |
| tchaypo | ah | 07:18 |
| larainema | thanks clarkb tchaypo for helping | 07:19 |
| *** nilasae has joined #openstack-infra | 07:20 | |
| *** yfried__ has quit IRC | 07:21 | |
| *** oomichi has quit IRC | 07:22 | |
| *** juggler_ is now known as juggler | 07:23 | |
| *** BharatK has quit IRC | 07:26 | |
| *** ParsectiX has joined #openstack-infra | 07:26 | |
| *** ddieterly has joined #openstack-infra | 07:29 | |
| *** dizquierdo has joined #openstack-infra | 07:30 | |
| *** binbincong has quit IRC | 07:31 | |
| tchaypo | Oh wait | 07:32 |
| tchaypo | this is the page with the in-real-life contact info, that’s stored in a way that the web app can’t decrypt | 07:32 |
| clarkb | yes | 07:32 |
| tchaypo | hence every time you look at the form it’s blank, so it’s hard to tell the difference between “I gave you my data” and “I haven’t given you data yet" | 07:32 |
| *** cdent has joined #openstack-infra | 07:33 | |
| clarkb | well it encrypts it but does not store it locally and it uses a last updated on to tell you when it was last updated | 07:33 |
| *** esmute__ has joined #openstack-infra | 07:34 | |
| tchaypo | I’m wondering if it’s worth a note in http://docs.openstack.org/infra/manual/developers.html#sign-the-appropriate-individual-contributor-license-agreement - it currently specifically says “make sure the primary email address …" | 07:34 |
| *** ddieterly has quit IRC | 07:34 | |
| *** arxcruz has joined #openstack-infra | 07:35 | |
| tchaypo | perhaps a .. note:: saying that you need to provide the other details as well and won’t be able to proceed unless there’s a last-updated date here would be helpful | 07:35 |
| *** fhubik is now known as fhubik_afk | 07:35 | |
| *** e0ne has joined #openstack-infra | 07:36 | |
| *** jistr has joined #openstack-infra | 07:37 | |
| *** larainema has quit IRC | 07:38 | |
| *** larainema has joined #openstack-infra | 07:38 | |
| *** heyongli has quit IRC | 07:39 | |
| *** esmute__ has quit IRC | 07:39 | |
| *** heyongli has joined #openstack-infra | 07:40 | |
| *** devanand_ has quit IRC | 07:41 | |
| *** dims has joined #openstack-infra | 07:43 | |
| *** btully has joined #openstack-infra | 07:43 | |
| *** binbincong has joined #openstack-infra | 07:45 | |
| *** laughterwym has joined #openstack-infra | 07:45 | |
| *** yamada-h has joined #openstack-infra | 07:45 | |
| *** btully has quit IRC | 07:48 | |
| *** dims has quit IRC | 07:48 | |
| *** HeOS has quit IRC | 07:50 | |
| *** BharatK has joined #openstack-infra | 07:54 | |
| *** yamada-h has quit IRC | 07:55 | |
| *** macjack has joined #openstack-infra | 07:56 | |
| *** Krinkle is now known as Krinkle_ | 07:57 | |
| *** heyongli has quit IRC | 08:00 | |
| *** shashank_hegde has quit IRC | 08:00 | |
| *** heyongli has joined #openstack-infra | 08:01 | |
| *** e0ne is now known as e0ne_ | 08:01 | |
| openstackgerrit | Flavio Percoco proposed openstack-infra/project-config: Move amqp1's devstack plugin under openstack https://review.openstack.org/199003 | 08:02 |
| *** fhubik_afk is now known as fhubik | 08:02 | |
| *** e0ne_ is now known as e0ne | 08:03 | |
| *** hashar has joined #openstack-infra | 08:03 | |
| eantyshev | hello! our CI on cinder has started failing to merge, with "error: Merge requires file-level merging" in merger logs. Anyone has any ideas? | 08:10 |
| *** koolhead17 has joined #openstack-infra | 08:11 | |
| clarkb | first check if that is a valid fail and attemlt to do the merge locally | 08:11 |
| clarkb | if that works then next check if the repo happy on the merger with a git fsck | 08:12 |
| clarkb | if that all looks good we likely need more specifics to hunt this down | 08:12 |
| *** chlong has quit IRC | 08:13 | |
| *** ZZelle has joined #openstack-infra | 08:13 | |
| *** bexelbie has joined #openstack-infra | 08:16 | |
| *** derekh has joined #openstack-infra | 08:17 | |
| eantyshev | clarkb: thank you, will start with that | 08:17 |
| *** notnownikki has joined #openstack-infra | 08:17 | |
| *** ildikov has quit IRC | 08:17 | |
| *** shardy has joined #openstack-infra | 08:17 | |
| *** gildub has quit IRC | 08:20 | |
| *** Krinkle_ is now known as Krinkle | 08:20 | |
| openstackgerrit | Merged openstack-infra/irc-meetings: Update neutron-drivers team meeting times. https://review.openstack.org/198806 | 08:20 |
| *** yfried__ has joined #openstack-infra | 08:21 | |
| *** larainema has quit IRC | 08:21 | |
| *** madong has joined #openstack-infra | 08:22 | |
| *** binbincong has quit IRC | 08:22 | |
| *** jcoufal has joined #openstack-infra | 08:22 | |
| *** dizquierdo has quit IRC | 08:23 | |
| *** madong is now known as larainema | 08:23 | |
| *** fhubik is now known as fhubik_afk | 08:23 | |
| *** larainema has quit IRC | 08:23 | |
| *** madong has joined #openstack-infra | 08:24 | |
| anteaya | eantyshev: missed you at the meeting, I ended it already, see you next week perhaps | 08:25 |
| anteaya | clarkb: everybody up or just you? | 08:25 |
| *** fhubik_afk is now known as fhubik | 08:25 | |
| anteaya | well I'm back to bed, g'night | 08:26 |
| *** mpaolino has joined #openstack-infra | 08:28 | |
| *** bexelbie_ has joined #openstack-infra | 08:29 | |
| clarkb | just waiting to feed twins the sleep hopefully | 08:30 |
| *** ddieterly has joined #openstack-infra | 08:30 | |
| *** pblaho has joined #openstack-infra | 08:30 | |
| *** kdas_ has quit IRC | 08:30 | |
| *** ildikov has joined #openstack-infra | 08:32 | |
| *** otter768 has joined #openstack-infra | 08:33 | |
| *** dpose has joined #openstack-infra | 08:34 | |
| *** ddieterly has quit IRC | 08:35 | |
| *** binbincong has joined #openstack-infra | 08:36 | |
| eantyshev | anteaya: I'm really sorry, I had urgent thing to do | 08:36 |
| *** otter768 has quit IRC | 08:38 | |
| *** bexelbie has quit IRC | 08:39 | |
| *** pelix has joined #openstack-infra | 08:39 | |
| *** bexelbie has joined #openstack-infra | 08:40 | |
| *** sambetts has quit IRC | 08:44 | |
| *** sambetts has joined #openstack-infra | 08:45 | |
| *** laughterwym has quit IRC | 08:51 | |
| *** laughterwym_ has quit IRC | 08:51 | |
| openstackgerrit | IWAMOTO Toshihiro proposed openstack/requirements: Add ryu https://review.openstack.org/154354 | 08:52 |
| *** Ala has joined #openstack-infra | 08:55 | |
| *** mfmcdonagh has quit IRC | 08:56 | |
| *** mcdonagm has quit IRC | 08:56 | |
| *** yamada-h has joined #openstack-infra | 08:57 | |
| *** flepied1 has joined #openstack-infra | 08:58 | |
| *** flepied2 has joined #openstack-infra | 08:58 | |
| *** yamamoto has joined #openstack-infra | 08:58 | |
| *** asahlin has quit IRC | 08:59 | |
| *** fhubik has quit IRC | 08:59 | |
| *** ParsectiX has quit IRC | 09:00 | |
| *** yamada-h has quit IRC | 09:00 | |
| *** dtantsur is now known as dtantsur|brb | 09:00 | |
| *** dizquierdo has joined #openstack-infra | 09:00 | |
| *** asahlin has joined #openstack-infra | 09:00 | |
| *** yamada-h has joined #openstack-infra | 09:00 | |
| *** shardy_ has joined #openstack-infra | 09:00 | |
| *** flepied has quit IRC | 09:01 | |
| *** ParsectiX has joined #openstack-infra | 09:02 | |
| *** shardy has quit IRC | 09:02 | |
| *** HeOS has joined #openstack-infra | 09:03 | |
| *** flepied1 has quit IRC | 09:03 | |
| *** yamada-h has quit IRC | 09:05 | |
| *** yamada-h has joined #openstack-infra | 09:05 | |
| *** shardy_ has quit IRC | 09:05 | |
| *** shardy has joined #openstack-infra | 09:06 | |
| *** yfried__ has quit IRC | 09:09 | |
| *** yamada-h has quit IRC | 09:09 | |
| *** yfried has joined #openstack-infra | 09:09 | |
| *** e0ne is now known as e0ne_ | 09:11 | |
| *** ParsectiX has quit IRC | 09:12 | |
| *** ParsectiX has joined #openstack-infra | 09:12 | |
| *** vikram has joined #openstack-infra | 09:16 | |
| *** madong has quit IRC | 09:16 | |
| *** madong has joined #openstack-infra | 09:17 | |
| *** madong has quit IRC | 09:17 | |
| openstackgerrit | Victor Stinner proposed openstack-infra/project-config: Move gate-keystone-python34 to check https://review.openstack.org/198350 | 09:18 |
| *** madong has joined #openstack-infra | 09:18 | |
| openstackgerrit | takehirokaneko proposed openstack-infra/project-config: Changed the gerrit acl of stackforge/rack https://review.openstack.org/199018 | 09:18 |
| *** yfried is now known as yfried|afk | 09:19 | |
| *** boris-42 has quit IRC | 09:22 | |
| *** e0ne_ is now known as e0ne | 09:24 | |
| *** laughterwym has joined #openstack-infra | 09:26 | |
| *** laughterwym_ has joined #openstack-infra | 09:26 | |
| openstackgerrit | Merged openstack/requirements: Adjust yaql version in upper-constraints https://review.openstack.org/198696 | 09:29 |
| openstackgerrit | Merged openstack/requirements: Adding python-mistralclient to the global-requirements https://review.openstack.org/195035 | 09:29 |
| *** ddieterly has joined #openstack-infra | 09:31 | |
| *** btully has joined #openstack-infra | 09:32 | |
| *** yfried|afk is now known as yfried | 09:34 | |
| *** ddieterly has quit IRC | 09:35 | |
| *** btully has quit IRC | 09:36 | |
| openstackgerrit | vikram.choudhary proposed openstack-infra/project-config: Adding networking-onos as a part of neutron project. https://review.openstack.org/198740 | 09:42 |
| openstackgerrit | Victor Stinner proposed openstack-infra/project-config: Make gate-os-brick-python34 voting and add gate https://review.openstack.org/199023 | 09:42 |
| *** dims has joined #openstack-infra | 09:44 | |
| *** dims_ has joined #openstack-infra | 09:45 | |
| *** dims__ has joined #openstack-infra | 09:46 | |
| openstackgerrit | Flavio Percoco proposed openstack-infra/project-config: Move amqp1's devstack plugin under openstack https://review.openstack.org/199003 | 09:46 |
| *** HeOS_ has joined #openstack-infra | 09:47 | |
| *** dims has quit IRC | 09:48 | |
| *** ociuhandu has joined #openstack-infra | 09:49 | |
| *** dims_ has quit IRC | 09:50 | |
| *** dims__ has quit IRC | 09:50 | |
| *** ZZelle has quit IRC | 09:55 | |
| *** notnownikki has quit IRC | 09:57 | |
| openstackgerrit | Merged openstack-infra/project-config: Run neutron multinode without dvr https://review.openstack.org/198906 | 10:06 |
| *** shardy_ has joined #openstack-infra | 10:06 | |
| *** e0ne is now known as e0ne_ | 10:07 | |
| *** shardy has quit IRC | 10:08 | |
| *** shardy_ has quit IRC | 10:12 | |
| eantyshev | jeblair, jhesketh: hello! I want to make modifications to zuul, made this https://review.openstack.org/c/198667 | 10:12 |
| *** shardy has joined #openstack-infra | 10:13 | |
| *** claudiub has joined #openstack-infra | 10:13 | |
| *** SotK_ has joined #openstack-infra | 10:14 | |
| jhesketh | eantyshev: oh cool that could be useful | 10:14 |
| jhesketh | eantyshev: I'll review it now | 10:15 |
| jhesketh | yolanda: ping | 10:15 |
| yolanda | hi | 10:16 |
| eantyshev | jhesketh: I'm in doubt how that fits zuulv3 spec, so I will be glad if you point me what to do exactly with this | 10:16 |
| jhesketh | eantyshev: yes, that's why I've pinged yolanda | 10:16 |
| eantyshev | yolanda: hello! | 10:16 |
| jhesketh | yolanda: why do you think 198667 should be part of zuul v3? | 10:17 |
| * jhesketh doesn't see any reason to block on the work | 10:17 | |
| yolanda | hi there, that change looked to me as it would fit there, when you talk about changes on mergers | 10:17 |
| yolanda | i don't want to block the work, but i thought it could be enclosed on that wider spec | 10:17 |
| *** ociuhandu has quit IRC | 10:17 | |
| yolanda | as you talk there about different tenants, changes needed in zuul mergers, and gitg | 10:18 |
| yolanda | git | 10:18 |
| *** SotK has quit IRC | 10:18 | |
| *** SotK_ is now known as SotK | 10:18 | |
| yolanda | i told eantyshev to read that spec, and see if that could fit there, and in that case, update the commit message to reflect that | 10:19 |
| jhesketh | hmm it may fit within the multi tenancy | 10:20 |
| jhesketh | we should consider how it may affect it | 10:21 |
| jhesketh | let me finish reviewing :-) | 10:21 |
| *** e0ne_ is now known as e0ne | 10:21 | |
| yolanda | cool, that was my point, that change seemed to fit very well in that section | 10:21 |
| jhesketh | eantyshev: if the git url is given to the merger with the appropraite gerrit details, wouldn't that allow a single merger to be attached to 2 zuul's? | 10:22 |
| eantyshev | jhesketh: If together with gerrit private key then yes | 10:23 |
| jhesketh | eantyshev: then why do you need this change? | 10:24 |
| *** dtantsur|brb is now known as dtantsur | 10:25 | |
| Kiall | Any requirements core about for a review? https://review.openstack.org/#/c/196633 - dnspython3 (py3 release of dnspython) | 10:25 |
| eantyshev | jhesketh: at the moment only username is given | 10:25 |
| openstackgerrit | Adam Coldrick proposed openstack-infra/storyboard: Fix entry point for storyboard-worker-daemon https://review.openstack.org/199042 | 10:26 |
| eantyshev | jhesketh: and merger uses private key from gerrit section of config | 10:26 |
| eantyshev | jhesketh: so options are either provide the key within the job, or to allow merger to use its own credentials | 10:27 |
| eantyshev | jhesketh: I chose the latter, since I saw no connection between credentials the merger uses and credentials used by zuul-server | 10:28 |
| *** viktors|afk is now known as viktors | 10:30 | |
| *** ddieterly has joined #openstack-infra | 10:31 | |
| jhesketh | eantyshev: I see... let me put a few comments in the review | 10:32 |
| *** otter768 has joined #openstack-infra | 10:34 | |
| *** ddieterly has quit IRC | 10:36 | |
| *** pc_m has joined #openstack-infra | 10:37 | |
| *** esmute__ has joined #openstack-infra | 10:38 | |
| *** koolhead17 has quit IRC | 10:38 | |
| jhesketh | eantyshev: I've added some comments :-) | 10:38 |
| *** otter768 has quit IRC | 10:39 | |
| eantyshev | thank you, jhesketh! | 10:39 |
| *** ericksonsantos has joined #openstack-infra | 10:40 | |
| *** koolhead17 has joined #openstack-infra | 10:41 | |
| *** esmute__ has quit IRC | 10:42 | |
| *** radez is now known as radez_g0n3 | 10:45 | |
| *** yfried is now known as yfried|afk | 10:45 | |
| *** dims has joined #openstack-infra | 10:46 | |
| *** Krinkle has left #openstack-infra | 10:47 | |
| *** katyafervent_ is now known as katyafervent_awa | 10:48 | |
| *** katyafervent_awa is now known as katyafervent | 10:51 | |
| *** dims has quit IRC | 10:51 | |
| *** koolhead_ has joined #openstack-infra | 10:53 | |
| openstackgerrit | Fathi Boudra proposed openstack-infra/jenkins-job-builder: general: add support for childCustomWorkspace https://review.openstack.org/97108 | 10:53 |
| *** ihrachyshka has joined #openstack-infra | 10:55 | |
| openstackgerrit | Zara proposed openstack-infra/storyboard-webclient: Bugfix: 'right click is interpreted as left click' https://review.openstack.org/199047 | 10:56 |
| *** koolhead17 has quit IRC | 10:58 | |
| *** koolhead_ has quit IRC | 10:58 | |
| *** yfried|afk is now known as yfried | 10:58 | |
| openstackgerrit | Flavio Percoco proposed openstack-infra/project-config: Move amqp1's devstack plugin under openstack https://review.openstack.org/199003 | 10:58 |
| *** yamada-h has joined #openstack-infra | 10:59 | |
| *** ociuhandu has joined #openstack-infra | 11:00 | |
| *** yamamoto has quit IRC | 11:01 | |
| *** yamamoto has joined #openstack-infra | 11:03 | |
| *** davideagnello has quit IRC | 11:04 | |
| *** yamada-h has quit IRC | 11:04 | |
| *** koolhead17 has joined #openstack-infra | 11:04 | |
| *** hashar has quit IRC | 11:05 | |
| *** dalgaaf has joined #openstack-infra | 11:08 | |
| *** Longgeek has quit IRC | 11:08 | |
| *** koolhead17 has quit IRC | 11:08 | |
| openstackgerrit | amrith proposed openstack-infra/project-config: Experiment with an increase time for gate-trove-functional job https://review.openstack.org/199050 | 11:09 |
| *** miqui has quit IRC | 11:11 | |
| openstackgerrit | Joshua Hesketh proposed openstack-infra/os-loganalyze: Set DocumentRoot for devstack plugin https://review.openstack.org/199053 | 11:12 |
| *** kashyap has joined #openstack-infra | 11:13 | |
| *** kashyap has left #openstack-infra | 11:14 | |
| openstackgerrit | Joshua Hesketh proposed openstack-infra/project-config: Test osloganalyze in devstack https://review.openstack.org/198728 | 11:14 |
| *** yamamoto has quit IRC | 11:15 | |
| *** kushal has joined #openstack-infra | 11:17 | |
| *** claudiub has quit IRC | 11:23 | |
| *** thorst has joined #openstack-infra | 11:24 | |
| *** e0ne is now known as e0ne_ | 11:25 | |
| openstackgerrit | Fathi Boudra proposed openstack-infra/jenkins-job-builder: builders: add 'publish over ssh' support as a build step https://review.openstack.org/98437 | 11:25 |
| *** vivcheri has joined #openstack-infra | 11:27 | |
| *** claudiub has joined #openstack-infra | 11:30 | |
| *** ddieterly has joined #openstack-infra | 11:32 | |
| openstackgerrit | Sean Dague proposed openstack-dev/pbr: Add wsgi_scripts support to PBR https://review.openstack.org/195292 | 11:35 |
| sdague | mordred / lifeless ok - https://review.openstack.org/195292 now has tests. They are a little janky, but they are a thing. | 11:36 |
| *** ddieterly has quit IRC | 11:37 | |
| *** ldnunes has joined #openstack-infra | 11:38 | |
| *** esmute__ has joined #openstack-infra | 11:39 | |
| *** yfried is now known as yfried|afk | 11:42 | |
| *** esmute__ has quit IRC | 11:43 | |
| *** e0ne_ is now known as e0ne | 11:45 | |
| *** Longgeek has joined #openstack-infra | 11:45 | |
| *** prad has joined #openstack-infra | 11:45 | |
| *** yfried|afk is now known as yfried | 11:52 | |
| openstackgerrit | Sergey Vilgelm proposed openstack/requirements: Use the 1.16.0 version of the oslo.messaging library https://review.openstack.org/199064 | 11:56 |
| *** jasondotstar has quit IRC | 11:57 | |
| *** baoli has joined #openstack-infra | 11:57 | |
| openstackgerrit | Sergey Vilgelm proposed openstack/requirements: Use the 1.16.0 version of the oslo.messaging library https://review.openstack.org/199064 | 11:58 |
| *** markvoelker has quit IRC | 11:59 | |
| *** markvoelker has joined #openstack-infra | 11:59 | |
| *** baoli has quit IRC | 12:01 | |
| *** derekh has quit IRC | 12:01 | |
| *** baoli has joined #openstack-infra | 12:01 | |
| *** yamamoto has joined #openstack-infra | 12:02 | |
| *** derekh has joined #openstack-infra | 12:02 | |
| *** dims has joined #openstack-infra | 12:02 | |
| *** jaypipes has joined #openstack-infra | 12:02 | |
| *** gordc has joined #openstack-infra | 12:02 | |
| sdague | clarkb: once you get around today, let me know. I want to talk through the right place to do the handoff between multinode d-g and grenade | 12:03 |
| sdague | there are a few things floating around in my head | 12:03 |
| *** bknudson has quit IRC | 12:03 | |
| *** weshay has joined #openstack-infra | 12:03 | |
| *** binbincong has quit IRC | 12:04 | |
| *** amrith is now known as _amrith_ | 12:04 | |
| *** flepied2 has quit IRC | 12:05 | |
| clarkb | I mean I am awake now but this is tail end of my day (I hope) | 12:05 |
| openstackgerrit | Dmitry Tantsur proposed openstack-infra/project-config: Add devstack check job for python-ironic-inspector-client https://review.openstack.org/199070 | 12:05 |
| *** chlong has joined #openstack-infra | 12:06 | |
| clarkb | I think we can do something very similar to what devstack did and run the grenade scripts wirh a constructed env via ansible | 12:06 |
| *** kiran-r has quit IRC | 12:06 | |
| *** koolhead17 has joined #openstack-infra | 12:06 | |
| sdague | clarkb: man, crazy hours :) | 12:07 |
| clarkb | so its set up controller, set up compute nodes, trigger tests, then trigger upgrade on cpntroller, upgrade on cpmpues,ru tests | 12:07 |
| sdague | so... the issue is, grenade needs a lot more slicing up to get to that state | 12:07 |
| *** dims has quit IRC | 12:07 | |
| sdague | because grenade handles the whole flow today | 12:07 |
| sdague | so... I'm wondering if we could do this slightly differently | 12:08 |
| sdague | and basically put a call out here - https://github.com/openstack-dev/grenade/blob/master/grenade.sh#L214 | 12:08 |
| clarkb | hrm my first reaction is grenade would need to change to either be multinode aware on its own or to be less in control | 12:08 |
| sdague | which could call the subnode script | 12:08 |
| *** yfried is now known as yfried|afk | 12:09 | |
| *** flepied has joined #openstack-infra | 12:09 | |
| *** yfried|afk is now known as yfried | 12:09 | |
| sdague | right, we'll need changes somewhere, I'm just trying to make it so it's something that's not irreconsilable with the current model, still lets you run things locally, and doesn't become such a mess that we all give up and go goat farming | 12:09 |
| clarkb | *diaper farming pretty sure there is money in that business | 12:10 |
| sdague | heh | 12:10 |
| clarkb | sdague ya since sibnode is basically just a second stack.sh run | 12:10 |
| sdague | right | 12:10 |
| clarkb | maybe hrm | 12:11 |
| *** koolhead_ has joined #openstack-infra | 12:11 | |
| clarkb | grenade could ask d-g to devstack | 12:11 |
| clarkb | and we will get nested i realky weird eays | 12:11 |
| sdague | so... if when we get to the subnode step, what if d-g just writes the ansible commands it would run to a script instead | 12:11 |
| sdague | and we modify grenade to have a post-stack phase | 12:12 |
| sdague | which runs that script | 12:12 |
| clarkb | this is prpbably a good reason to properly playboom this stuff | 12:12 |
| *** adduarte has quit IRC | 12:12 | |
| *** koolhead17 has quit IRC | 12:12 | |
| clarkb | so that d-g and grenade can both use it | 12:12 |
| *** duartea has joined #openstack-infra | 12:12 | |
| clarkb | mordred ^ this is probably the reason we needes | 12:12 |
| clarkb | so they both do ansible-playbook make a devstack | 12:13 |
| sdague | sure, though, honestly, I'd rather not make grenade ansible aware, just give it a script callout | 12:13 |
| sdague | though, I guess we could also make grenade take a configurable "stack" point | 12:13 |
| sdague | https://github.com/openstack-dev/grenade/blob/master/grenade.sh#L211-L214 | 12:13 |
| sdague | my concern though, is the gate code would end up so different from the base code, that it's not possible to local repro any thing | 12:14 |
| clarkb | thats why my change to ansible d-g used it for everything | 12:14 |
| *** Ala has quit IRC | 12:15 | |
| *** vikram has quit IRC | 12:15 | |
| clarkb | so multinide and single node d-g are the same | 12:15 |
| clarkb | there arent even any branches really | 12:15 |
| clarkb | just noop commands in single node | 12:15 |
| *** Ala has joined #openstack-infra | 12:15 | |
| *** dprince has joined #openstack-infra | 12:15 | |
| *** dizquierdo has quit IRC | 12:16 | |
| sdague | sure, but the run mode for grenade is a bit different | 12:17 |
| sdague | because it's also an orchestration layer to a degree | 12:17 |
| Shrews | mordred: do you plan to cut a 0.9.1 shade release? or 0.10.0... whatever | 12:17 |
| sdague | we could argue that grenade should become ansible, but that's a whole other thing beast | 12:18 |
| *** fabrice_ has quit IRC | 12:18 | |
| *** yamamoto_ has joined #openstack-infra | 12:18 | |
| sdague | clarkb: so I'm thinking something like this - https://review.openstack.org/199073 | 12:19 |
| *** vikram has joined #openstack-infra | 12:19 | |
| clarkb | ya same argument with d-g I personally find the shell with ansible as atool/cpmmand much easier to read | 12:19 |
| clarkb | sdague what if we gave grenade an inventory file and had it do its thing absed on that? | 12:20 |
| *** binbincong has joined #openstack-infra | 12:20 | |
| sdague | I find the ansible stuff pretty hard to read personally, the way it's done. because all the meat is buried deep in the command string | 12:20 |
| clarkb | and use a default local cpnnectio only inventory | 12:20 |
| *** fabrice_ has joined #openstack-infra | 12:20 | |
| sdague | so, today, grenade doesn't know anything about nodes | 12:20 |
| *** ddieterly has joined #openstack-infra | 12:21 | |
| sdague | which is kind of ok | 12:21 |
| clarkb | I think it has to be aware if it is orvhestration and you want multinode tests | 12:21 |
| sdague | clarkb: why? | 12:21 |
| clarkb | because orchestration across multiple nodes really wants to know the nodes exist | 12:22 |
| mordred | clarkb: nod | 12:22 |
| mordred | Shrews: yes | 12:22 |
| *** yamamoto has quit IRC | 12:22 | |
| sdague | right, but grenade doesn't really need to be aware of any of that | 12:22 |
| Shrews | \o/ | 12:22 |
| sdague | it is building a stack | 12:22 |
| sdague | and upgrading a stack on the one machine | 12:22 |
| sdague | the fact that there might be more workers, is kind of irrelevant | 12:23 |
| *** samuelBartel has joined #openstack-infra | 12:23 | |
| mordred | Shrews: done | 12:23 |
| sdague | and tempest is running against an API | 12:23 |
| mordred | sdague: so you're arguing that multi-machine awareness shoudl be only in d-g and not in grenade? | 12:24 |
| *** ari has joined #openstack-infra | 12:24 | |
| sdague | mordred: right. | 12:24 |
| sdague | especially with the partial upgrade case, where the reason for going down this path is that the workers should be left alone by grenade. | 12:24 |
| *** rossella_s has quit IRC | 12:25 | |
| sdague | and, I don't think full multinode grenade upgrade is a thing we're going to go near any time soon. This is just about the partial upgrade case, which more projects want | 12:25 |
| *** bknudson has joined #openstack-infra | 12:27 | |
| *** rossella_s has joined #openstack-infra | 12:28 | |
| *** bswartz has quit IRC | 12:28 | |
| *** aysyd has joined #openstack-infra | 12:28 | |
| *** kgiusti has joined #openstack-infra | 12:28 | |
| mordred | sdague: awesome: "my_ip = socket.gethostbyname(socket.gethostname())\nsocket.gaierror: [Errno -2] Name or service not known" | 12:29 |
| sdague | mordred: huh? | 12:29 |
| mordred | sdague: might be the same issue that hit multi-node that we cut a new glean for ... | 12:29 |
| sdague | what is the issue? | 12:30 |
| *** tlian has joined #openstack-infra | 12:30 | |
| mordred | sdague: well, we started setting hostnames on the hosts, but there was a bug, so it set the hostname to a truncated version | 12:30 |
| mordred | hrm | 12:30 |
| mordred | that can't possibly be this problem | 12:30 |
| mordred | (as I said it out loud ...) | 12:30 |
| *** markvoelker has quit IRC | 12:31 | |
| *** bexelbie has quit IRC | 12:31 | |
| sdague | yeh, socket.gethostbyname(socket.gethostname()) not working is beyond what I expected from these environments | 12:31 |
| *** bexelbie_ has quit IRC | 12:31 | |
| clarkb | so I still dont grok the demand for partial multinode grenade but thats a different issue | 12:32 |
| *** jlanoux has joined #openstack-infra | 12:32 | |
| clarkb | we can partial test single node pretty easily | 12:32 |
| sdague | clarkb: well, lets come back to that question, because that's the primary design point | 12:32 |
| clarkb | and will be much easier to debug | 12:33 |
| sdague | clarkb: so, here is the problem with single node partial | 12:33 |
| sdague | it's hyper synthetic | 12:33 |
| clarkb | thats fine | 12:33 |
| *** bexelbie has joined #openstack-infra | 12:33 | |
| sdague | the old code is only still working because it *probably* loaded all the libraries it needed before upgrade, and they are running in memory | 12:33 |
| clarkb | it tickles code paths, iseasier to debug and can be run lically | 12:34 |
| sdague | and it adds a lot of really tricky conditionals in the grenade and devstack bits that get kind of fragile | 12:34 |
| sdague | so making the barrier for upgrade be node level, makes all that a ton more clean | 12:34 |
| *** otter768 has joined #openstack-infra | 12:35 | |
| sdague | and means that the number of combinations of different kinds of partial upgrade models people want can be pretty unlimitted | 12:35 |
| sdague | because you just don't upgrade the worker | 12:35 |
| sdague | partial upgrade in a single node isn't easier to debug, because you can't stop the old side to add debug bits into it, because it's no longer old side code (it has different libraries) | 12:36 |
| *** miqui has joined #openstack-infra | 12:36 | |
| clarkb | use vrnvs or containers then | 12:36 |
| clarkb | this isnt an issue with single node its an issue with how we single node | 12:37 |
| mordred | sure. | 12:37 |
| mordred | but that would take reengineering all the thigns we do | 12:37 |
| mordred | which might be a thing to do think about | 12:38 |
| mordred | but probably isn't a thing to make a ton of progress on, like, soon | 12:38 |
| *** yamamoto_ has quit IRC | 12:38 | |
| clarkb | ya my poiny is either you orchrstrate and be node aware or you dont | 12:39 |
| *** dguerri` is now known as dguerri | 12:39 | |
| clarkb | first is use ansible second is singlw node | 12:39 |
| clarkb | both work | 12:39 |
| sdague | there are other options here | 12:39 |
| *** otter768 has quit IRC | 12:40 | |
| openstackgerrit | Dmitry Tantsur proposed openstack-infra/project-config: Do not run ironic-inspector-dsvm job on stable/1.1 branch https://review.openstack.org/199079 | 12:40 |
| *** markvoelker has joined #openstack-infra | 12:40 | |
| sdague | what is your objection to the post-stack.sh ? | 12:40 |
| clarkb | its hacky and doesnt make it easy to reproduce locally | 12:41 |
| sdague | no one reproduces grenade locally except me | 12:41 |
| sdague | in these partial cases | 12:41 |
| sdague | I don't even do the partial cases locally most of the time, because there are enough differences in the gate that it's not really valid | 12:42 |
| clarkb | then maybe we shpuld reduce them rather than add to them? | 12:42 |
| sdague | reduce which? | 12:42 |
| clarkb | the differneces with the gate | 12:43 |
| clarkb | and insert spwcial gate script here would be a big difference | 12:43 |
| clarkb | its also forces us into a very simple set of upgrade tests on multi ode | 12:43 |
| sdague | sure, so the first step there would be make something like devstack-vagrant to make it possible to exactly get the devstack-gate env up locally, including multinode | 12:44 |
| sdague | that's a whole other issue | 12:44 |
| clarkb | and to do proper upgrade testing on multinode would reqyire doing it properly anyways | 12:44 |
| *** fdegir has quit IRC | 12:44 | |
| clarkb | so why not just do it especually since effort is likely the same (cost is in making it work not writing the change) | 12:44 |
| sdague | I'm not sure I understand what should just be done? | 12:45 |
| clarkb | do orchestration of >1 node in grenade | 12:46 |
| clarkb | I think it can be done simply with absible and configurable inventory file | 12:47 |
| clarkb | grenade can explicitly affect subnodes properly if they are present that way | 12:47 |
| clarkb | allowing you control over full upgrade of partial(service) upgrades | 12:47 |
| clarkb | while using the same code paths locally and in the gate | 12:48 |
| sdague | but unlike devstack-gate it doesn't have a whole system around to allocate it nodes for you already | 12:48 |
| mordred | that's not how the ansible in d-g works though | 12:48 |
| sdague | and adding more complexity into grenade, with basically only 1 person that understands it, is just a black swan event waiting to happen | 12:48 |
| clarkb | mordred it is | 12:48 |
| mordred | clarkb: no, I mean what sdague said | 12:48 |
| clarkb | oh | 12:48 |
| mordred | clarkb: I was agreeing with you | 12:48 |
| mordred | sdague: the inventory file associates one or more hosts with one or more names. so the things you run are thigns like "run this on the controllers" "run this on the compute nodes" - if you have one node, you just have a simple inventory file that maps all the things to the one host | 12:50 |
| *** ParsectiX has quit IRC | 12:50 | |
| sdague | right, except I only ever want to run 1 thing on the compute nodes, stack.sh | 12:50 |
| sdague | once | 12:50 |
| sdague | after it was run on the controller | 12:51 |
| sdague | then never talk to the compute nodes ever again | 12:51 |
| *** yfried is now known as yfried|afk | 12:51 | |
| sdague | so rewrite all of grenade as ansible seems... a pretty big hammer to get that | 12:51 |
| mordred | well, I think "rewrite all of grenade as ansible" is likely not the suggestion | 12:52 |
| sdague | mordred: that's fair | 12:52 |
| clarkb | ya just the stack run | 12:52 |
| clarkb | whrre you linked | 12:52 |
| clarkb | I am saying make it configurable rather than $randomscripyable | 12:52 |
| clarkb | and ansible is my sugeestion for doi g that | 12:53 |
| *** fdegir has joined #openstack-infra | 12:53 | |
| *** yamamoto has joined #openstack-infra | 12:53 | |
| sdague | so that would make the communication boundary between d-g and grenade a bunch of ansible config | 12:54 |
| mordred | it would make it one inventory file, which it's already writing out | 12:54 |
| sdague | and require also being able to understand ansible to do grenade in the single node case | 12:54 |
| sdague | for people trying to work on things | 12:54 |
| sdague | which I'm not sure why that's better than the call out script | 12:54 |
| sdague | which is how devstack-gate hooks work for all our various jobs | 12:55 |
| *** rlandy has joined #openstack-infra | 12:55 | |
| *** BharatK has quit IRC | 12:55 | |
| clarkb | ya and hooks have been tons of troubel... | 12:55 |
| mordred | "being able to understand ansible" is going to become a requirement for touching pretty much any gate job in a few months | 12:55 |
| clarkb | I dunno I dislike the d-g hooks very much | 12:56 |
| mordred | so, not making value judgements on whether it's the right choice here - I don't think the learning curve of ansible should be a huge blocker | 12:56 |
| sdague | mordred: sure, but the whole point of pushing this back into grenade was to make it possible to replicate outside the gate | 12:56 |
| mordred | sure | 12:56 |
| *** bswartz has joined #openstack-infra | 12:56 | |
| clarkb | they are poorly declared interfaces so the natural progression of them has been to add more redundant unneeded ones to fit someones use casr | 12:56 |
| *** yfried|afk is now known as yfried | 12:56 | |
| clarkb | rather than grok interface and use it properly | 12:57 |
| mordred | but if we wind up reimplementing ansible in shell as a result of doing that, it's worth considering just using the tool | 12:57 |
| sdague | so.... https://review.openstack.org/#/c/199073/ doesn't seem like it's reimplementing ansible in shell | 12:57 |
| *** bexelbie has quit IRC | 12:58 | |
| *** yamamoto has quit IRC | 12:58 | |
| mordred | I agree. it does not seem like a reimplementation of ansible in shell :) | 12:59 |
| openstackgerrit | Samuel BARTEL proposed openstack-infra/project-config: add project fuel-plugin-availability-zones https://review.openstack.org/199084 | 12:59 |
| *** yamamoto has joined #openstack-infra | 13:00 | |
| *** yamamoto has quit IRC | 13:00 | |
| *** nilasae has quit IRC | 13:01 | |
| mordred | sdague: btw - this has ALMOST nothing to do with that - but I'm spinning up a devstack using devstack-vagrant right now | 13:01 |
| sdague | mordred: neat | 13:02 |
| mordred | and it strikes me, d-v uses puppet to do things ... which is what we use to configure the d-g nodes ... this has me wondering if it would be valuable to have a d-g-vagrant | 13:02 |
| mordred | that would run literally all of the exact things we run on d-g nodes except for repo caching | 13:02 |
| sdague | mordred: so... that was my end game for devstack-vagrant | 13:02 |
| sdague | I just never got there | 13:02 |
| mordred | nod | 13:02 |
| mordred | so, if I wrote a patch for that at some point, you would not scream and throw your arms in the air and whatnot | 13:03 |
| sdague | nope, I'd +A that like there was no tomorrow | 13:03 |
| mordred | cool | 13:03 |
| sdague | it even can do multinode | 13:03 |
| mordred | I've _also_ been thinking of taking a step (after dib-nodepool is finished, of course) to publishing our dib images to vagrant boxes and docker containers | 13:03 |
| sdague | that would be pretty awesome | 13:04 |
| mordred | since we already have a thing doing the work of making the images | 13:04 |
| *** ddieterly has quit IRC | 13:04 | |
| mordred | biggest challenge is that i think most people dont' want to download the image that has the caching in it | 13:04 |
| clarkb | 15GB vagrant images | 13:04 |
| mordred | yah | 13:04 |
| *** hashar has joined #openstack-infra | 13:04 | |
| sdague | yeh, trimming the cache might be worth while. | 13:05 |
| *** _nadya__ has quit IRC | 13:05 | |
| *** yamamoto has joined #openstack-infra | 13:06 | |
| *** eharney has joined #openstack-infra | 13:07 | |
| *** changbl has quit IRC | 13:08 | |
| *** sshnaidm has quit IRC | 13:08 | |
| *** jasondotstar has joined #openstack-infra | 13:09 | |
| *** jasondot_ has joined #openstack-infra | 13:10 | |
| openstackgerrit | Kyle Mestery proposed openstack-infra/project-config: Add stable ACLs for Neutron sub-projects https://review.openstack.org/198750 | 13:10 |
| openstackgerrit | Kyle Mestery proposed openstack-infra/project-config: Add a create ACL for networking-bgpvpn https://review.openstack.org/198751 | 13:10 |
| openstackgerrit | Kyle Mestery proposed openstack-infra/project-config: Update ACLs for Neutron sub-projects https://review.openstack.org/198749 | 13:10 |
| openstackgerrit | Kyle Mestery proposed openstack-infra/project-config: Update merge commit gerrit ACL for Neutron repositories https://review.openstack.org/199089 | 13:10 |
| *** _amrith_ is now known as amrith | 13:11 | |
| *** jsavak has joined #openstack-infra | 13:11 | |
| *** bexelbie has joined #openstack-infra | 13:12 | |
| mordred | sdague: yup. vagrant just crashed my laptop | 13:12 |
| mordred | sdague: I _have_ used it successfully that one time though | 13:13 |
| mordred | so that's good | 13:13 |
| *** notnownikki has joined #openstack-infra | 13:13 | |
| *** jasondot_ has quit IRC | 13:13 | |
| *** jasondotstar has quit IRC | 13:14 | |
| sdague | I've not had it crash my env in a long time, virtualbox up to date with your kernel? | 13:14 |
| *** jasondotstar has joined #openstack-infra | 13:14 | |
| *** skolekonov has quit IRC | 13:16 | |
| *** heyongli has quit IRC | 13:17 | |
| yolanda | mordred, can you update access on puppet-os_client_config-core, and allow puppet-cores to have rights there? | 13:18 |
| *** kiran-r has joined #openstack-infra | 13:18 | |
| yolanda | good afternoon btw :) | 13:18 |
| *** yamamoto has quit IRC | 13:18 | |
| *** heyongli has joined #openstack-infra | 13:19 | |
| *** dims has joined #openstack-infra | 13:19 | |
| *** dboik has joined #openstack-infra | 13:20 | |
| openstackgerrit | Sean Dague proposed openstack-infra/devstack-gate: POC: grenade partial multinode https://review.openstack.org/199091 | 13:21 |
| *** dizquierdo has joined #openstack-infra | 13:22 | |
| sdague | anyway, I'm sure that doesn't actually work, but that's kind of what I was thinking | 13:22 |
| mordred | yolanda: yup | 13:22 |
| mordred | yolanda: done | 13:23 |
| *** BharatK has joined #openstack-infra | 13:23 | |
| yolanda | cool, thx | 13:23 |
| Sam-I-Am | mornings | 13:23 |
| *** julim has joined #openstack-infra | 13:24 | |
| openstackgerrit | Zara proposed openstack-infra/storyboard-webclient: Stop 'about' and 'license' buttons overlapping others https://review.openstack.org/199095 | 13:25 |
| *** heyongli has quit IRC | 13:26 | |
| *** mriedem_away is now known as mriedem | 13:27 | |
| *** ddieterly has joined #openstack-infra | 13:28 | |
| *** heyongli has joined #openstack-infra | 13:28 | |
| *** ddieterly has quit IRC | 13:28 | |
| *** ddieterly has joined #openstack-infra | 13:29 | |
| openstackgerrit | Zara proposed openstack-infra/storyboard-webclient: Display 'about' and 'license' icons on higher zooms https://review.openstack.org/199097 | 13:30 |
| *** mtanino has joined #openstack-infra | 13:31 | |
| *** spzala has joined #openstack-infra | 13:31 | |
| *** BharatK has quit IRC | 13:32 | |
| *** heyongli has quit IRC | 13:32 | |
| *** amitgandhinz has joined #openstack-infra | 13:33 | |
| *** amotoki has quit IRC | 13:33 | |
| *** boris-42 has joined #openstack-infra | 13:33 | |
| *** heyongli has joined #openstack-infra | 13:36 | |
| *** mpavone has quit IRC | 13:36 | |
| *** mika has quit IRC | 13:37 | |
| *** e0ne is now known as e0ne_ | 13:37 | |
| *** mika has joined #openstack-infra | 13:37 | |
| *** bhunter71 has joined #openstack-infra | 13:37 | |
| *** yamamoto has joined #openstack-infra | 13:38 | |
| openstackgerrit | vikram.choudhary proposed openstack-infra/project-config: Adding networking-onos as a part of neutron project. https://review.openstack.org/198740 | 13:38 |
| *** e0ne_ is now known as e0ne | 13:39 | |
| *** dustins has joined #openstack-infra | 13:39 | |
| *** jasondotstar has quit IRC | 13:40 | |
| *** deepakcs has quit IRC | 13:42 | |
| *** annegentle has joined #openstack-infra | 13:42 | |
| *** _nadya_ has joined #openstack-infra | 13:42 | |
| *** ddieterly has quit IRC | 13:44 | |
| *** fallenpegasus has quit IRC | 13:44 | |
| *** heyongli has quit IRC | 13:44 | |
| *** AJaeger_ is now known as AJaeger | 13:45 | |
| *** blaedd has joined #openstack-infra | 13:46 | |
| openstackgerrit | Sean Dague proposed openstack-infra/project-config: add experimental grenade multinode jobs https://review.openstack.org/199103 | 13:46 |
| *** heyongli has joined #openstack-infra | 13:47 | |
| AJaeger | anteaya: yeah, I missed that. But what you wrote is not actionable for me. jeblair what do you want with reviews like https://review.openstack.org/#/c/196085/? anteaya said "made aware" which can mean many things. Adding you to the review? Waiting for your +2? Telling you on IRC? | 13:49 |
| openstackgerrit | BorneMace proposed openstack-infra/project-config: Added new validate-contents test for kolla https://review.openstack.org/198968 | 13:49 |
| *** smccully has joined #openstack-infra | 13:49 | |
| AJaeger | mordred: did you solve your po file issue for openstack-manuals? | 13:50 |
| *** mpavone has joined #openstack-infra | 13:50 | |
| mordred | AJaeger: I believe I learned that I did not need to solve my po file issue - which was even better | 13:50 |
| AJaeger | mordred: yeah, po files are all autogenerated - with the exception of initial setup. So, don't touch them ;) | 13:51 |
| *** claudiub has quit IRC | 13:52 | |
| pelix | any infra core reviewers happy to review/approve changes for JJB? | 13:53 |
| *** yamamoto has quit IRC | 13:53 | |
| pelix | https://review.openstack.org/181541 & https://review.openstack.org/179455 | 13:53 |
| openstackgerrit | Valeriy Ponomaryov proposed openstack-infra/project-config: Move experimental Manila jobs in Rally to check pipeline https://review.openstack.org/199108 | 13:55 |
| *** signed8bit_ZZZzz is now known as signed8__ | 13:55 | |
| *** r-daneel has joined #openstack-infra | 13:56 | |
| openstackgerrit | Valeriy Ponomaryov proposed openstack-infra/project-config: Move experimental Manila jobs in Rally to check pipeline https://review.openstack.org/199108 | 13:56 |
| *** annegentle has quit IRC | 13:57 | |
| openstackgerrit | Valeriy Ponomaryov proposed openstack-infra/project-config: Move experimental Manila jobs in Rally to check pipeline https://review.openstack.org/199108 | 13:57 |
| *** sigmavirus24_awa is now known as sigmavirus24 | 13:57 | |
| mordred | sdague: any chance you know what I need to go to get my devstack to have run with keystone v3? | 13:57 |
| mordred | sdague: ah - found it in stackrc | 13:59 |
| *** zz_dimtruck is now known as dimtruck | 13:59 | |
| sdague | yeh, it should always be there now | 13:59 |
| sdague | there is a flag to turn off v2 | 13:59 |
| mordred | well ... IDENTITY_API_VERSION=${IDENTITY_API_VERSION:-2.0} | 13:59 |
| Sam-I-Am | keystone v3 for middleware? | 13:59 |
| *** nelsnelson has joined #openstack-infra | 13:59 | |
| mordred | is what's set in stackrc in the thing I have locally | 13:59 |
| *** blaedd has quit IRC | 14:00 | |
| mordred | so I apparently need to set that to 3 - but I can do that | 14:00 |
| *** blaedd has joined #openstack-infra | 14:00 | |
| sdague | if [ "$ENABLE_IDENTITY_V2" == "False" ]; then | 14:00 |
| sdague | IDENTITY_API_VERSION=3 | 14:00 |
| sdague | fi | 14:00 |
| sdague | ok, that's pretty convoluted | 14:00 |
| mordred | yah. so - either disable v2 or enable v3 | 14:00 |
| samueldmq | mordred: sdague that's experimental ... | 14:01 |
| mordred | sdague: is there an easy way for me to pass a localrc to devstack-vagrant? | 14:01 |
| samueldmq | disabling v2 requires devstack to set up resources using keystone v3 only (of course) | 14:01 |
| mordred | samueldmq: I just need a keystone v3 running that I can run service and endpoint adding to - I don't actually need the rest of openstack to work with it :) | 14:01 |
| *** annegentle has joined #openstack-infra | 14:02 | |
| samueldmq | there are some patches to make that happen (starting at https://review.openstack.org/#/c/186681/) | 14:02 |
| *** jsavak has quit IRC | 14:02 | |
| samueldmq | mordred: it is already running you you run a simple devstack without setting anything else | 14:02 |
| mordred | samueldmq: really? | 14:02 |
| sdague | mordred: I don't think so, normally | 14:02 |
| sdague | normally I just work with an existing vagrant if I'm hacking on that, but that shouldn't be too bad to add in | 14:03 |
| *** jsavak has joined #openstack-infra | 14:03 | |
| Sam-I-Am | iirc, keystone v3 lives at the usual /v2.0 url | 14:03 |
| mordred | samueldmq: WARNING: unsupported identity-api-version 3, falling back to 2.0 | 14:03 |
| *** Kennan2 has joined #openstack-infra | 14:03 | |
| mordred | Authorization Failed: The resource could not be found. (HTTP 404) (Request-ID: req-b50d736c-6796-4503-8daa-4d2e7ee368ef) | 14:03 |
| mordred | wait | 14:03 |
| mordred | what? | 14:03 |
| samueldmq | mordred: yes it is https://github.com/openstack/keystone/blob/master/etc/keystone-paste.ini#L105 | 14:03 |
| mordred | you put keystone v3 at the v2.0 url? | 14:03 |
| mordred | *headdesk* | 14:03 |
| samueldmq | mordred: what did you do ? | 14:03 |
| samueldmq | Sam-I-Am: ? | 14:03 |
| Sam-I-Am | for kilo thats how it works | 14:03 |
| *** Kennan has quit IRC | 14:03 | |
| sdague | Sam-I-Am: ?? in what context | 14:03 |
| samueldmq | keystone v3 lives in :5000/v3 | 14:03 |
| sdague | yeh, definitely devstack uses /v3 | 14:04 |
| Sam-I-Am | sdague: might have been a well-needed change in liberty | 14:04 |
| Sam-I-Am | for the kilo install guide, nothing wanted to work right with /v3 (although it existed) | 14:04 |
| *** signed8__ is now known as signed8bit_ZZZzz | 14:04 | |
| mordred | ok. I agree - there is a keystone endpoint running on :5000/v3 | 14:05 |
| *** xyang1 has joined #openstack-infra | 14:05 | |
| samueldmq | Sam-I-Am: no, you can't access v3 API at v2.0 at all | 14:05 |
| Sam-I-Am | i think it was mostly client-side problems | 14:05 |
| mordred | however, WARNING: unsupported identity-api-version 3, falling back to 2.0 | 14:05 |
| Sam-I-Am | samueldmq: interesting. how am i getting v3 auth responses with /v2.0 endpoints? | 14:05 |
| samueldmq | mordred: you using the clients ? what command are you trying to run ? | 14:05 |
| Sam-I-Am | samueldmq: unless thats what changed | 14:05 |
| mordred | samueldmq: keystone catalog | 14:06 |
| Sam-I-Am | mordred: you need to use the openstack client for v3, iirc | 14:06 |
| *** dimtruck is now known as zz_dimtruck | 14:06 | |
| Sam-I-Am | keystone client was deprecated for v3 | 14:06 |
| mordred | Sam-I-Am: mordred@camelot:~/src/openstack-dev/devstack-vagrant$ openstack catalog list | 14:06 |
| mordred | ERROR: openstack 'NoneType' object does not support item assignment | 14:06 |
| dstanek | Sam-I-Am: v3 tokens or v3 api? | 14:06 |
| samueldmq | Sam-I-Am: v2.0 operations are equivalent in v3 using the default domain | 14:06 |
| samueldmq | dstanek: :-) | 14:06 |
| dstanek | samueldmq: yes. yes, we do make things too confusing. | 14:07 |
| Sam-I-Am | samueldmq: that makes sense. there's only one domain in this case. | 14:07 |
| Sam-I-Am | dstanek: tokens | 14:07 |
| *** annegentle has quit IRC | 14:07 | |
| dstanek | Sam-I-Am: that's different than the v3 api | 14:07 |
| samueldmq | mordred: ah you setting openstack client to use v3 ? | 14:08 |
| Sam-I-Am | dstanek: ah. | 14:08 |
| mordred | samueldmq: should be | 14:08 |
| mordred | samueldmq: OS_IDENTITY_API_VERSION=3 | 14:08 |
| Sam-I-Am | dstanek: is 'catalog' a v3 api thing? because that fails on my system. | 14:08 |
| mordred | catalog should work everywhere | 14:09 |
| mordred | "keystone catalog" -- "openstack catalog list" | 14:09 |
| mordred | should be the same thing | 14:10 |
| samueldmq | mordred: could you try something similar to openstack endpoint list --os-url $KEYSTONE_SERVICE_URI_V3 --os-identity-api-version 3 | 14:10 |
| *** bmace_away is now known as bmace | 14:10 | |
| mordred | ERROR: openstack 'NoneType' object does not support item assignment | 14:10 |
| samueldmq | mordred: I am scared .. | 14:10 |
| mordred | openstack endpoint list --os-url http://192.168.1.231:5000/v3 --os-identity-api-version 3 | 14:11 |
| samueldmq | mordred: that's how jamielennox is doing at https://review.openstack.org/#/c/186681/ | 14:11 |
| Sam-I-Am | ah, i see | 14:11 |
| Sam-I-Am | i had a problem with my openrc | 14:11 |
| mordred | ok. so - I used shade instead of the command line clients | 14:11 |
| samueldmq | mordred: let me get a vm up and see what I can do :) | 14:12 |
| mordred | and got this: keystoneclient.openstack.common.apiclient.exceptions.BadRequest: Expecting to find domain in project - the server could not comply with the request since it is either malformed or otherwise incorrect. The client is assumed to be in error. (HTTP 400) | 14:12 |
| mordred | so - I believe that's telling me "hi, you didn't specify a domain" | 14:12 |
| mordred | one would expect the error message elsewhere to be better than "'NoneType' object does not support item assignment"- just fwiw | 14:12 |
| samueldmq | mordred: ++ yes I agree | 14:13 |
| Sam-I-Am | mordred: here's what seems to work - http://paste.openstack.org/show/352089/ | 14:13 |
| Sam-I-Am | my keystone endpoints are /v2.0 though | 14:13 |
| Sam-I-Am | but openstack catalog list works now | 14:13 |
| samueldmq | mordred: in fact, that's what shade is for, right ? fix things openstack :) | 14:13 |
| mordred | heh | 14:13 |
| *** sdake has quit IRC | 14:14 | |
| mordred | samueldmq, Sam-I-Am: yes. adding OS_USER_DOMAIN_NAME=default and OS_PROJECT_DOMAIN_NAME=default makes it work. thank you both | 14:14 |
| *** sdake has joined #openstack-infra | 14:14 | |
| Sam-I-Am | mordred: os_project_domain_id ? | 14:15 |
| Sam-I-Am | yeah | 14:15 |
| * Sam-I-Am slow | 14:15 | |
| mordred | well, it's 'default', so I used name not id | 14:15 |
| Sam-I-Am | find moar coffee | 14:15 |
| mordred | although you are right, the id param _Does_ also accept names | 14:15 |
| clarkb | wouldnt ot be nice if there was a ... wait for it ...default domain? | 14:16 |
| mordred | btw - for anyone interested, I tested this via ansible and shade like this: | 14:16 |
| mordred | ansible localhost -m os_auth | 14:16 |
| *** signed8bit has joined #openstack-infra | 14:17 | |
| mordred | which helpfully printed out the service catalog for me | 14:17 |
| *** signed8bit_ZZZzz has quit IRC | 14:17 | |
| mordred | clarkb: right? | 14:18 |
| Sam-I-Am | clarkb: isnt the default domain default? ?) | 14:18 |
| Sam-I-Am | er, :) | 14:18 |
| Sam-I-Am | stupid fingers | 14:18 |
| *** heyongli has quit IRC | 14:18 | |
| *** haypo has joined #openstack-infra | 14:18 | |
| mordred | Sam-I-Am: it is... | 14:18 |
| bmace | hey folks, sorry if this is a silly question. my review failed last time so i think i fixed it and patched it appropriately, but it doesn't seem to be getting run through jenkins again? i checked zuul and there is no job for it and some others posted more recently seem to have finished. | 14:18 |
| clarkb | except mordred had to specify it | 14:18 |
| clarkb | which is not a default | 14:18 |
| mordred | Sam-I-Am: except yeah ^^ it didn't do a good job of being a default :) | 14:18 |
| Sam-I-Am | that was the joke | 14:18 |
| * mordred hands Sam-I-Am a fluffy bunny | 14:19 | |
| openstackgerrit | Paul Belanger proposed openstack-infra/project-config: Upload applytest folder gate-infra-puppet-apply-* https://review.openstack.org/199122 | 14:19 |
| Sam-I-Am | ha | 14:19 |
| clarkb | bmace link to change? | 14:19 |
| Sam-I-Am | does it contain caffeine? | 14:19 |
| haypo | hi. DevStack setup failed on "edit-constraints .../requirements/upper-constraints.txt -- keystone ..." with "pkg_resources.RequirementParseError: Missing distribution spec /opt/stack/new/oslo.utils" on my patch https://review.openstack.org/#/c/198026/ | 14:19 |
| *** mpaolino_ has joined #openstack-infra | 14:19 | |
| pabelanger | any chance for a some what fast review / +2 for above? ^ Make things easier when debugging puppet-apply jobs | 14:19 |
| clarkb | mordred btw if you really want to get people hand them twins | 14:19 |
| samueldmq | mordred: no you used the id .. | 14:19 |
| mordred | samueldmq: I did what? | 14:20 |
| *** mpaolino has quit IRC | 14:20 | |
| samueldmq | mordred: the default domain has id = 'defaul' :) | 14:20 |
| samueldmq | default* | 14:20 |
| haypo | i'm not more sure that lifeless told me that he's working on this issue. is anyone aware of this upper-constraints issue? | 14:20 |
| *** asselin_ has joined #openstack-infra | 14:20 | |
| bmace | hrm, and of course as soon as i mention it in here, from out of nowhere, jenkins marks it as verified though i saw nothing in zuul about it :/ https://review.openstack.org/#/c/198968/ | 14:20 |
| mordred | samueldmq: you realize I'm going to punch someone right? | 14:20 |
| *** heyongli has joined #openstack-infra | 14:20 | |
| *** sshnaidm has joined #openstack-infra | 14:20 | |
| AJaeger | clarkb: good morning. Could you add Robert Clark (Security PTL) as core to security-specs ACL, please? https://review.openstack.org/#/admin/groups/976,members | 14:20 |
| samueldmq | mordred: :-) | 14:20 |
| mordred | samueldmq: in a world where everything is UUIDs for ids and humand readable names for names .... *GAH* | 14:20 |
| bmace | clarkb thanks for asking. that must have been the magic to knock it loose :) | 14:20 |
| * mordred goes back to debugging the actual thing | 14:20 | |
| samueldmq | dstanek: is the id of default domain 'default' only in devstack ? or do we do that in our migration stuff ? | 14:21 |
| clarkb | AJaeger dobe | 14:21 |
| clarkb | er done | 14:21 |
| openstackgerrit | Zara proposed openstack-infra/storyboard-webclient: Display 'about' and 'license' icons on higher zooms https://review.openstack.org/199097 | 14:21 |
| clarkb | haypo I think thats the upgtafe pbr issue | 14:22 |
| AJaeger | clarkb: thanks! | 14:22 |
| clarkb | haypo you need to track down what is downgrading pbr | 14:22 |
| clarkb | arg more docs.o.o https fails | 14:24 |
| dstanek | samueldmq: not sure, but i assume it can be whatever | 14:24 |
| openstackgerrit | Paul Belanger proposed openstack-infra/project-config: Upload applytest folder gate-infra-puppet-apply-* https://review.openstack.org/199122 | 14:24 |
| *** btully has joined #openstack-infra | 14:24 | |
| pabelanger | nibalizer, ^ | 14:24 |
| samueldmq | mordred: dstanek it can be configurable .. but the it's default is 'default' https://github.com/openstack/keystone/blob/master/etc/keystone.conf.sample#L790 | 14:24 |
| haypo | clarkb: before the error, i see pbr 1.2 which looks correct | 14:25 |
| samueldmq | yes, this sentence is confusing including too many 'default' :-) | 14:25 |
| samueldmq | dstanek: thanks | 14:25 |
| mordred | so - there are more than one issue | 14:25 |
| mordred | one is that the ID is being set to default | 14:25 |
| mordred | that's silly | 14:25 |
| mordred | the _name_ should be set to default | 14:25 |
| mordred | and it shoudl have an ID that nobody should care about | 14:25 |
| *** chlong has quit IRC | 14:25 | |
| dstanek | samueldmq: yeah, i believe you still have to create it. so there isn't any magic. | 14:25 |
| mordred | also - region_name has a default value of RegionOne | 14:26 |
| mordred | but if that is the default region on the cloud, you do not have to specify it as a parameter | 14:26 |
| mordred | _that_ is how a default value for something global like that should behave | 14:26 |
| mordred | putting this: | 14:26 |
| mordred | user_domain_id: default | 14:26 |
| mordred | project_domain_id: default | 14:26 |
| mordred | into a config file for a user is just bonkers | 14:26 |
| samueldmq | mordred: if you don't specify anything, fall into the default thing | 14:27 |
| mordred | not that I don't love everyone, of course | 14:27 |
| samueldmq | hehe | 14:27 |
| samueldmq | mordred: I think that's for v2 (where the default domain is used).. in v3 domains should be set explicitely | 14:28 |
| AJaeger | clarkb: was https ever working on docs.openstack.org? | 14:28 |
| samueldmq | mordred: I think this is from security guys vs UX guys (what you're playing here I think) | 14:28 |
| mordred | samueldmq: I hear that - but I'd like to suggest that that is incorrect | 14:28 |
| mordred | samueldmq: nope. there is nothing security about this | 14:28 |
| mordred | if the cloud has one and only one domain | 14:28 |
| clarkb | AJaeger no | 14:28 |
| mordred | it should be ABSOLUTELY fine to have it behave as a default | 14:29 |
| clarkb | AJaeger https everywhere was previosuly at fault | 14:29 |
| mordred | this is especially important for the transition period from v2 to v3 | 14:29 |
| mordred | where people will have their old creds, and where their clouds will have only one domain | 14:29 |
| mordred | making everyone in the world pass the two strings "default" into the API | 14:29 |
| mordred | is _not_ security | 14:29 |
| openstackgerrit | Fathi Boudra proposed openstack-infra/jenkins-job-builder: builders: add 'publish over ssh' support as a build step https://review.openstack.org/98437 | 14:29 |
| clarkb | mordred: but making it impossible to use the api means no users which means you are most secure | 14:30 |
| samueldmq | mordred: you have interesting points, I'd love to see them discussed with other folks :) | 14:31 |
| mordred | clarkb: good point. I forgot about that | 14:31 |
| Sam-I-Am | mordred: this is one of the reasons i lose hair writing the install guide | 14:31 |
| Sam-I-Am | especially when keystone middleware changes without more docs than a blog post :/ | 14:31 |
| samueldmq | Sam-I-Am: hehe | 14:32 |
| *** annegentle has joined #openstack-infra | 14:32 | |
| samueldmq | mordred: clarkb I am kind of learning how things work in a UX vs security approach :-) | 14:32 |
| *** yamamoto has joined #openstack-infra | 14:32 | |
| samueldmq | when I wonder why certain things aren't different : | 14:32 |
| *** Ala has quit IRC | 14:34 | |
| Sam-I-Am | samueldmq: when i wonder, i just dive into neutron for a while :) | 14:35 |
| Sam-I-Am | then other things dont seem so crazy | 14:35 |
| samueldmq | Sam-I-Am: hehee I need to see more about neutron .. I am very keystone so far | 14:35 |
| samueldmq | Sam-I-Am: but willing to get a wider knowledge by contributing to/understanding other projects as well | 14:36 |
| *** otter768 has joined #openstack-infra | 14:36 | |
| Sam-I-Am | samueldmq: i sort of figured out its near impossible to know everything about every project, even just the 'core' ones | 14:36 |
| *** bmace has quit IRC | 14:36 | |
| Sam-I-Am | brain capacity limited | 14:36 |
| Sam-I-Am | or i keep killing brain cells | 14:37 |
| *** rossella_s has quit IRC | 14:37 | |
| *** BharatK has joined #openstack-infra | 14:37 | |
| *** rossella_s has joined #openstack-infra | 14:37 | |
| sdague | AJaeger: can you help me figure out what's going on with this failure - https://review.openstack.org/#/c/199103/ ? | 14:37 |
| AJaeger | sdague: let me check... | 14:37 |
| samueldmq | Sam-I-Am: sure, it's not about knowing everything about everyproject at all | 14:39 |
| *** woodster_ has joined #openstack-infra | 14:39 | |
| *** jsavak has quit IRC | 14:39 | |
| openstackgerrit | Sean Dague proposed openstack-infra/project-config: add experimental grenade multinode jobs https://review.openstack.org/199103 | 14:39 |
| sdague | AJaeger: oh, I think I figured it out | 14:39 |
| samueldmq | Sam-I-Am: I'm working on keystone for a while, and I am far from understanding everything | 14:39 |
| samueldmq | Sam-I-Am: that's just about having a better knowledge of openstack as a whole, if you know what I mean :) | 14:39 |
| *** fallenpegasus has joined #openstack-infra | 14:39 | |
| Sam-I-Am | samueldmq: sure | 14:40 |
| AJaeger | sdague: yeah, that looks fine now. | 14:40 |
| Sam-I-Am | writing the install guide helped a bit | 14:40 |
| *** jsavak has joined #openstack-infra | 14:40 | |
| *** otter768 has quit IRC | 14:40 | |
| openstackgerrit | Paul Belanger proposed openstack-infra/system-config: WIP: Migrate to puppet-httpd module https://review.openstack.org/198819 | 14:41 |
| openstackgerrit | Paul Belanger proposed openstack-infra/system-config: Actually test for puppet errors when we run puppet-apply https://review.openstack.org/199132 | 14:41 |
| *** esmute__ has joined #openstack-infra | 14:41 | |
| AJaeger | jeblair: https://review.openstack.org/#/c/199103/ is a change that touches devstack-gate. Is that something you like to review? | 14:42 |
| *** annegentle has quit IRC | 14:43 | |
| AJaeger | dhellmann: regarding https://review.openstack.org/#/c/197672, what's wrong with that one and it's depends-on? | 14:44 |
| *** otter768 has joined #openstack-infra | 14:44 | |
| *** austin81 has joined #openstack-infra | 14:44 | |
| *** austin81 has left #openstack-infra | 14:45 | |
| *** austin81 has joined #openstack-infra | 14:46 | |
| *** esmute__ has quit IRC | 14:46 | |
| *** BharatK has quit IRC | 14:46 | |
| *** yfried has quit IRC | 14:48 | |
| *** tonytan4ever has joined #openstack-infra | 14:49 | |
| dhellmann | AJaeger: oh, just that I didn't notice the depends-on | 14:50 |
| *** radez_g0n3 is now known as radez | 14:50 | |
| *** bmace has joined #openstack-infra | 14:50 | |
| dhellmann | AJaeger: actually, renaming isn't something we could block on with just a depends-on, right? it requires manual action and gerrit downtime? | 14:51 |
| *** vponomaryov has left #openstack-infra | 14:51 | |
| *** vponomaryov has joined #openstack-infra | 14:51 | |
| *** jsavak has quit IRC | 14:52 | |
| sdague | dhellmann: the way renames for existing projects in openstack occur is the governance change depends-on the project-config change | 14:52 |
| *** jsavak has joined #openstack-infra | 14:52 | |
| sdague | and the project-config change is just handled delicately | 14:52 |
| *** rlandy has quit IRC | 14:52 | |
| dhellmann | sdague: ok | 14:52 |
| sdague | so not approved until there is a window | 14:52 |
| sdague | if it's a "new" openstack project, the depends is the other way around | 14:52 |
| dhellmann | I guess there's no way to avoid the small window where the names in the 2 places don't match | 14:52 |
| sdague | so that the TC must ok it first | 14:53 |
| sdague | right | 14:53 |
| *** rlandy has joined #openstack-infra | 14:53 | |
| sdague | but we've got 2 TC members in infra, it just requires someone to twizzle it again post land on the project-config side | 14:53 |
| pabelanger | mordred, AJaeger any chance for some +2 action? https://review.openstack.org/#/c/199122/ | 14:55 |
| *** bradjones|away is now known as bradjones | 14:56 | |
| *** achanda has joined #openstack-infra | 14:56 | |
| *** hdd has joined #openstack-infra | 14:57 | |
| *** wenlock has joined #openstack-infra | 14:59 | |
| *** BharatK has joined #openstack-infra | 14:59 | |
| *** wenlock1 has joined #openstack-infra | 15:00 | |
| *** jsavak has quit IRC | 15:01 | |
| *** jsavak has joined #openstack-infra | 15:01 | |
| *** achanda has quit IRC | 15:02 | |
| *** ddieterly has joined #openstack-infra | 15:03 | |
| *** arxcruz has quit IRC | 15:03 | |
| *** cindyo has joined #openstack-infra | 15:03 | |
| *** nilasae has joined #openstack-infra | 15:03 | |
| *** annegentle has joined #openstack-infra | 15:04 | |
| *** laughterwym has quit IRC | 15:04 | |
| *** armax has joined #openstack-infra | 15:05 | |
| *** _kiran_ has joined #openstack-infra | 15:05 | |
| *** thedodd has joined #openstack-infra | 15:05 | |
| *** _kiran_ has quit IRC | 15:06 | |
| openstackgerrit | Russell Bryant proposed openstack/requirements: Stop adding extra newline after unparseable lines. https://review.openstack.org/199143 | 15:06 |
| *** zz_dimtruck is now known as dimtruck | 15:06 | |
| *** puranamr has joined #openstack-infra | 15:07 | |
| *** annegentle has quit IRC | 15:13 | |
| *** e0ne is now known as e0ne_ | 15:16 | |
| *** dannywilson has joined #openstack-infra | 15:16 | |
| *** shashank_hegde has joined #openstack-infra | 15:16 | |
| *** dims has quit IRC | 15:16 | |
| *** _nadya_ has quit IRC | 15:16 | |
| *** mpaolino has joined #openstack-infra | 15:18 | |
| *** mpaolino_ has quit IRC | 15:18 | |
| *** bexelbie has quit IRC | 15:20 | |
| *** dannywilson has quit IRC | 15:20 | |
| *** tsekiyama has joined #openstack-infra | 15:20 | |
| *** annegentle has joined #openstack-infra | 15:21 | |
| sdague | mordred / clarkb (who I know is probably asleep) can I get your look at - https://review.openstack.org/#/c/199091/ as an approach here | 15:22 |
| clarkb | I am not asleep yet, there is something terribly wrong here | 15:22 |
| sdague | AJaeger: hmmmm ... after addressing the duplicate entry thing, now I get a not defined error on https://review.openstack.org/#/c/199103/ | 15:24 |
| *** flepied has quit IRC | 15:26 | |
| *** sabeen1 has joined #openstack-infra | 15:26 | |
| *** e0ne_ is now known as e0ne | 15:26 | |
| *** dannywilson has joined #openstack-infra | 15:27 | |
| *** cdent has quit IRC | 15:28 | |
| openstackgerrit | Russell Bryant proposed openstack-infra/project-config: Publish docs for networking-ovn. https://review.openstack.org/199153 | 15:29 |
| *** mpaolino has quit IRC | 15:29 | |
| openstackgerrit | yolanda.robla proposed openstack-infra/puppet-gerrit: cron to clear gerrit logs https://review.openstack.org/199154 | 15:30 |
| * mriedem has spent entirely too much time in the guts of pbr / sphinx / autodoc this morning | 15:30 | |
| *** changbl has joined #openstack-infra | 15:31 | |
| *** shashank_hegde has quit IRC | 15:32 | |
| anteaya | eantyshev: oh no problem I just didn't want you to be surprised if you popped in | 15:32 |
| anteaya | AJaeger: ah I didn't know you would only consider my comments if they were framed as an action item | 15:33 |
| *** kiran-r has quit IRC | 15:33 | |
| anteaya | AJaeger: yes, let's hear from jeblair what he would like, as I am motivated by his wishes to be informed | 15:33 |
| *** bexelbie has joined #openstack-infra | 15:33 | |
| AJaeger | anteaya: you misunderstood me. I missed that when doing my +2A. Still, I wonder what's the proper work flow is. | 15:33 |
| openstackgerrit | Dmitry Tyzhnenko proposed openstack-infra/git-review: Add reviewers by group alias on upload https://review.openstack.org/195043 | 15:34 |
| *** sabeen3 has joined #openstack-infra | 15:34 | |
| anteaya | well I think part of my reluctance to review is the lack of acknowledgement my reviews get | 15:34 |
| anteaya | as I feel only a green check mark, if it is exists is acknowledged | 15:34 |
| *** sabeen1 has quit IRC | 15:35 | |
| anteaya | comments I leave tend to get glossed over, which I find tiring | 15:35 |
| AJaeger | anteaya: And that was the only complaint there - you gave a comment which I should have read properly and come back to IRC and ask: What shall be done? | 15:35 |
| AJaeger | anteaya: If I would have read it properly, I wouldn't have approved it ;( Sorry. | 15:36 |
| anteaya | it is okay, it is a small thing | 15:37 |
| anteaya | but it is indicative of something i have been feeling for a while | 15:37 |
| sdague | AJaeger: any thoughts on https://review.openstack.org/#/c/199103/ because now I'm more confused | 15:37 |
| anteaya | many small things | 15:37 |
| AJaeger | sdague: you have "pipeline: check" in projects.yaml but zuula/layout.yaml uses "gate-..." | 15:37 |
| *** soren has quit IRC | 15:37 | |
| sdague | ah.... | 15:37 |
| sdague | AJaeger: thanks | 15:38 |
| *** davidlenwell has quit IRC | 15:38 | |
| AJaeger | anteaya: the problem is that when I'm giving my reviews, the input box does not show any previous review comments. | 15:38 |
| AJaeger | anteaya: and if I didn't read them properly before, I might act on what I see. That happens to several of us... | 15:39 |
| AJaeger | anteaya: so, what I do in such case: Give my comment with just saying "LGTM, I'll only +2 once this or that is done." | 15:39 |
| *** shardy_ has joined #openstack-infra | 15:40 | |
| AJaeger | anteaya: and for others where it's really critical, I even -2 like project renames: https://review.openstack.org/#/c/195341/ | 15:41 |
| *** shardy has quit IRC | 15:41 | |
| *** dims has joined #openstack-infra | 15:43 | |
| openstackgerrit | Sean Dague proposed openstack-infra/project-config: add experimental grenade multinode jobs https://review.openstack.org/199103 | 15:43 |
| sdague | hmmm... is there a hostname truncation issue - http://logs.openstack.org/91/199091/1/experimental/check-tempest-dsvm-multinode-full/e519323//console.html#_2015-07-07_15_11_42_302 ? | 15:45 |
| openstackgerrit | Darragh Bailey proposed openstack-infra/jenkins-job-builder: Exclude .test directory from pep8 check https://review.openstack.org/199159 | 15:45 |
| clarkb | sdague: yes there is, image builds from this morning should correct it | 15:45 |
| *** shardy_ has quit IRC | 15:45 | |
| clarkb | sdague: this was the other thing I was sying about multinode, it doesn't work at all currently and would be good to get working before we try to add a bunch of complexity | 15:45 |
| *** spredzy is now known as spredzy|afk | 15:45 | |
| clarkb | hopefully that is the only thing that has broken int he last month or so | 15:45 |
| anteaya | AJaeger: the input box does not show any previous review comments, that sounds like the problem right there | 15:46 |
| *** shardy has joined #openstack-infra | 15:46 | |
| *** asselin_ has quit IRC | 15:46 | |
| *** whoops has joined #openstack-infra | 15:47 | |
| *** sdake_ has joined #openstack-infra | 15:48 | |
| *** sdake has quit IRC | 15:48 | |
| sdague | it looks like the key local regression is brctl isn't in the subnodes - http://logs.openstack.org/32/189632/6/check/check-tempest-dsvm-multinode-smoke/b6214f4/logs/subnode-2/devstacklog.txt.gz | 15:48 |
| *** BharatK has quit IRC | 15:49 | |
| hogepodge | how often does the puppet scripts that manages the openstack bot run? | 15:49 |
| clarkb | hogepodge: every half hour to 45 minutes depending on how slow the previous puppet run is | 15:49 |
| *** unicell has quit IRC | 15:49 | |
| sdague | clarkb: yes, I do understand that point. I also understand that 0 people besides me look at grenade race failures (that often show up more often in the partial jobs), even when they are a 40% failure rate. So making that part more complicated isn't a win either :) | 15:49 |
| fungi | hogepodge: basically it's fired from cron every 15 minutes, but happens under a global lock to prevent concurrent runs, and currently takes around 30 minutes to complete | 15:50 |
| clarkb | sdague: I think the hostname -f is the failure there | 15:50 |
| clarkb | sdague: not sure if brctl is related to that or not, I thought that we were explicitly installing the bridge utils package prior to any bridge setup | 15:50 |
| fungi | hogepodge: which can mean every 30-60 minutes depending on how long the previous run takes | 15:50 |
| sdague | clarkb: oh, good point | 15:50 |
| hogepodge | fungi: hmm, I'm wondering why openstack bot hasn't joined #openstack-diversity yet. That change merged last night. https://review.openstack.org/#/c/198936/1 | 15:50 |
| sdague | clarkb: right, we might be, that's exposing in the worlddump | 15:50 |
| fungi | hogepodge: i'll have a look | 15:51 |
| sdague | clarkb: so the hostname -f failure is because of the hostname truncation? | 15:51 |
| clarkb | sdague: yes, /etc/hosts has the full name but the hostname is truncated so the name lookup fails | 15:51 |
| clarkb | because foo != foobar | 15:51 |
| fungi | hogepodge: that's the change to add access permissions for our bots and infrastructure admins, not a change make the openstack bot join the channel | 15:52 |
| sdague | clarkb: gotcha | 15:52 |
| hogepodge | fungi ah ha. | 15:52 |
| fungi | hogepodge: did you have a corresponding change to the openstack_project::eavesdrop class in the openstack-infra/system-config repo? | 15:52 |
| *** hichihara has quit IRC | 15:53 | |
| hogepodge | fungi: yes, but it hasn't merged yet https://review.openstack.org/#/c/198941/ | 15:53 |
| *** akshai has joined #openstack-infra | 15:53 | |
| *** davidlenwell has joined #openstack-infra | 15:53 | |
| openstackgerrit | Zara proposed openstack-infra/storyboard-webclient: Display 'about' and 'license' icons on higher zooms https://review.openstack.org/199097 | 15:54 |
| fungi | hogepodge: lgtm, but we generally avoid approving those while meetings are going on since that's also our meetbot and the eavesdrop class restarts it on config updates, which can be disruptive mid-meeting | 15:54 |
| *** kushal has quit IRC | 15:54 | |
| *** davidlenwell is now known as daviddavid | 15:54 | |
| *** daviddavid is now known as davidlenwell | 15:54 | |
| hogepodge | fungi: I understand. personally, I'd just wait for the infra meeting to restart it ;-) | 15:55 |
| pelix | Anyone on the infra team available to approve a change to enable some more tests in JJB? https://review.openstack.org/181541 | 15:55 |
| fungi | if we ever find a vict^H^H^Holunteer for reimplementing our irc bots in errbot we should dig into ways to make that sort of thing more graceful | 15:55 |
| *** annegentle has quit IRC | 15:56 | |
| *** ivar-lazzaro has joined #openstack-infra | 15:56 | |
| zaro | pelix: can do | 15:56 |
| zaro | morning | 15:56 |
| *** ivar-lazzaro has quit IRC | 15:57 | |
| pelix | zaro: you've already +2'ed it ;) | 15:57 |
| clarkb | sdague: so I think that worlddump brctl can't happen if we fail early enough to have not installed brctl | 15:57 |
| *** ivar-lazzaro has joined #openstack-infra | 15:57 | |
| clarkb | sdague: so in theory that problem goes away with the new images today | 15:57 |
| *** ddieterly has quit IRC | 15:57 | |
| anteaya | zaro: morning | 15:57 |
| *** hashar is now known as hasharMeeting | 15:58 | |
| *** davidlenwell has quit IRC | 15:58 | |
| *** davidlenwell has joined #openstack-infra | 15:58 | |
| *** salv-orlando has joined #openstack-infra | 15:58 | |
| openstackgerrit | Merged openstack-infra/jenkins-job-builder: Enable tests of builder object https://review.openstack.org/181541 | 15:59 |
| zaro | pelix: i would have obliged for tests only changes :) | 15:59 |
| fungi | clarkb: are you working toward a once-every-two-days sleep schedule now? :/ | 16:00 |
| *** jistr has quit IRC | 16:00 | |
| *** BharatK has joined #openstack-infra | 16:00 | |
| zaro | that would be impressive | 16:00 |
| *** jtomasek has quit IRC | 16:01 | |
| clarkb | fungi: I hope not, I just got caught being up last night and now its cool so time to get stuff done | 16:01 |
| yolanda | fungi, mordred, i don't seem to have core rights to puppet-refstack , can you please check that? | 16:01 |
| clarkb | will probably check out after hte meeting if I make it that far | 16:01 |
| *** aswadr has joined #openstack-infra | 16:02 | |
| clarkb | have to take advantage of all sub 25C hours | 16:02 |
| fungi | yolanda: fixing, we probably missed adding our puppet core group to that once it was created | 16:02 |
| anteaya | clarkb: I'm for you catching a nap anytime you can get one | 16:02 |
| yolanda | mm, if i look into the acl, i just see "inherit from all-projects", i cannot see any special puppet group there | 16:03 |
| clarkb | part of my problem is I have always been bad at taking naps | 16:03 |
| *** ddieterly has joined #openstack-infra | 16:03 | |
| clarkb | so I end up awake far too long then just crash and burn and thankfully have enough help that that isn't too terrible | 16:03 |
| fungi | yolanda: oh! that acl never applied. something may be broken with it | 16:03 |
| fungi | looking now to see if i can tell what | 16:03 |
| *** annegentle has joined #openstack-infra | 16:03 | |
| anteaya | clarkb: yeah you don't strike me as a good napper | 16:03 |
| yolanda | maybe some typo in the acl | 16:04 |
| anteaya | clarkb: I can offer lessons | 16:04 |
| fungi | yolanda: it's better than that. there is no acl file for it in project-config | 16:04 |
| fungi | i wonder if there was an acl in the change which added the repo | 16:04 |
| *** dtantsur is now known as dtantsur|afk | 16:04 | |
| yolanda | was that added recently? | 16:04 |
| fungi | yolanda: https://review.openstack.org/182437 | 16:05 |
| yolanda | so no acl | 16:05 |
| fungi | yolanda: oh, it's got an acl-config specified in project.yaml though | 16:05 |
| yolanda | mm, puppet-system is the generic one, right? | 16:05 |
| sdague | clarkb: cool, when do we expect new images? | 16:06 |
| *** laughterwym_ has quit IRC | 16:06 | |
| fungi | yolanda: nope, there is no gerrit/acls/openstack-infra/puppet-system.config in the project-config repo. it's a nonexistent file i guess | 16:06 |
| clarkb | sdague: prior to me disappearing for forever they were showing up across the board by around noon pdt, so about 3 hours from now | 16:07 |
| yolanda | i remember having used or seen that puppet-system sometime | 16:07 |
| *** scheuran has quit IRC | 16:07 | |
| *** ddieterly has quit IRC | 16:07 | |
| *** jistr has joined #openstack-infra | 16:07 | |
| fungi | yolanda: looks like it might have been intended to use system-config.config instead | 16:07 |
| anteaya | it is in openstack | 16:07 |
| anteaya | not openstack-infra | 16:07 |
| anteaya | the path is wrong | 16:07 |
| yolanda | anteaya, thx, i remember having seen that somewhere | 16:08 |
| *** davidlenwell has quit IRC | 16:08 | |
| *** ddieterly has joined #openstack-infra | 16:08 | |
| fungi | anteaya: i don't see one there either. can you link to it? | 16:08 |
| *** ashleighfarnham has joined #openstack-infra | 16:08 | |
| *** jlanoux has quit IRC | 16:08 | |
| *** davidlenwell has joined #openstack-infra | 16:08 | |
| fungi | anteaya: are you thinking of puppet-modules.config in openstack? | 16:08 |
| anteaya | sorry you are correct | 16:08 |
| *** ashleighfarnham has quit IRC | 16:08 | |
| anteaya | yes I was thinking of puppet-mondules in openstack | 16:08 |
| *** ashleighfarnham has joined #openstack-infra | 16:09 | |
| anteaya | as that seems to be the catchall acl for puppety things | 16:09 |
| fungi | anteaya: anyway, puppet-refstack is an openstack-infra project so likely wouldn't use the puppet-openstack team's acls | 16:09 |
| anteaya | ah | 16:09 |
| anteaya | so yeah, no acl | 16:09 |
| yolanda | fungi, is there some generic acl for openstack-infra puppet projects? | 16:09 |
| *** ddieterly has quit IRC | 16:09 | |
| *** kushal has joined #openstack-infra | 16:09 | |
| sdague | clarkb: coolio | 16:10 |
| fungi | yolanda: no, they're separate. i think the fix is to remove the acl-config entry from projects.yaml and add a puppet-refstack.config acl | 16:10 |
| *** annegentle has quit IRC | 16:10 | |
| yolanda | ok, i can do it in a moment | 16:10 |
| *** k4n0 has quit IRC | 16:11 | |
| yolanda | fungi, that puppet-system is appearing in more projects | 16:11 |
| yolanda | openstack-infra/beaker-localhost, beaker-nodepool.. | 16:12 |
| *** davidlenwell is now known as omnibus7 | 16:12 | |
| *** omnibus7 is now known as davidlenwell | 16:12 | |
| yolanda | so at some point there was some intention to have that acl ? | 16:12 |
| fungi | yolanda: i wonder if someone has cargo culted someone else's typo | 16:12 |
| *** davidlenwell is now known as david_enwell | 16:12 | |
| anteaya | yolanda: the intention to have a common acl has to come from puppet | 16:13 |
| *** kgiusti has left #openstack-infra | 16:13 | |
| zaro | pelix: you didn't answer my question regarding folder support. | 16:13 |
| yolanda | mm, so i wonder if that beaker projects introduced the bug | 16:14 |
| fungi | anteaya: yolanda: found. it used to exist and was removed by https://review.openstack.org/186217 | 16:14 |
| fungi | anteaya: yolanda: i think people keep copying old reviews to add new projects | 16:14 |
| anteaya | ah ha | 16:14 |
| anteaya | you are the best detective | 16:14 |
| fungi | anteaya: i owe it all to git log -p | 16:15 |
| anteaya | :) | 16:15 |
| pelix | zaro: which patchset was that on? | 16:15 |
| *** fawadkhaliq has joined #openstack-infra | 16:15 | |
| zaro | pelix: https://review.openstack.org/#/c/180185/ | 16:15 |
| yolanda | fungi, ok, so then better to remove that puppet-system from the 3 projects, and add individual acls for them, right? | 16:16 |
| *** MarkAtwood has joined #openstack-infra | 16:17 | |
| fungi | yolanda: yep, they're all broken. they should have separate acls each | 16:18 |
| *** mayurig has joined #openstack-infra | 16:18 | |
| *** flepied has joined #openstack-infra | 16:18 | |
| openstackgerrit | yolanda.robla proposed openstack-infra/project-config: Fix puppet-refstack acl https://review.openstack.org/199177 | 16:18 |
| fungi | also reviewers might want to keep an eye out for new acolytes of that particular cargo cult | 16:18 |
| openstackgerrit | Vladimir Kozhukalov proposed openstack-infra/project-config: Add project fuel-agent repo https://review.openstack.org/199178 | 16:18 |
| yolanda | ok, change for puppet refstack | 16:18 |
| openstackgerrit | Merged openstack-infra/elastic-recheck: Remove obsolete cinder LVM no space querie https://review.openstack.org/188782 | 16:19 |
| fungi | yolanda: thanks! | 16:19 |
| *** david_enwell is now known as davidlenwell | 16:19 | |
| yolanda | about the other 2 projects, they are more related to test, doesn't seem to be puppet modules, so not sure about the right acl. i guess no release is needed | 16:19 |
| *** sshnaidm has quit IRC | 16:20 | |
| *** mpavone has quit IRC | 16:20 | |
| *** davidlenwell has quit IRC | 16:20 | |
| *** soren has joined #openstack-infra | 16:20 | |
| *** dpose has quit IRC | 16:20 | |
| *** davidlenwell has joined #openstack-infra | 16:20 | |
| *** hogepodge has quit IRC | 16:21 | |
| anteaya | fungi: thank you | 16:21 |
| *** BharatK has quit IRC | 16:21 | |
| *** austin81 has left #openstack-infra | 16:22 | |
| *** kdas_ has joined #openstack-infra | 16:23 | |
| *** bexelbie has quit IRC | 16:23 | |
| *** kushal has quit IRC | 16:23 | |
| *** unicell has joined #openstack-infra | 16:23 | |
| pleia2 | good morning | 16:23 |
| *** davidlenwell has quit IRC | 16:23 | |
| *** davidlenwell has joined #openstack-infra | 16:24 | |
| yolanda | the beaker ones seem to be from nibalizer, i'll check with him | 16:24 |
| *** ildikov has quit IRC | 16:25 | |
| *** amrith is now known as _amrith_ | 16:26 | |
| *** davidlenwell has quit IRC | 16:27 | |
| *** davidlenwell has joined #openstack-infra | 16:28 | |
| *** unicell has quit IRC | 16:28 | |
| anteaya | morning pleia2 | 16:28 |
| *** unicell has joined #openstack-infra | 16:28 | |
| *** jcoufal has quit IRC | 16:28 | |
| *** armax has quit IRC | 16:28 | |
| *** ddieterly has joined #openstack-infra | 16:29 | |
| clarkb | good morning | 16:29 |
| *** ddieterly has quit IRC | 16:30 | |
| *** davidlenwell has quit IRC | 16:31 | |
| zaro | clarkb: any interest in updating gearman-plugin with your change? | 16:31 |
| *** notnownikki has quit IRC | 16:31 | |
| greghaynes | 'morning | 16:32 |
| *** e0ne has quit IRC | 16:33 | |
| nibalizer | Ya I created the beaker repos | 16:33 |
| *** armax has joined #openstack-infra | 16:34 | |
| *** jistr has quit IRC | 16:35 | |
| *** sdake_ is now known as sdae | 16:35 | |
| *** davidlenwell has joined #openstack-infra | 16:35 | |
| *** davidlenwell is now known as davidlenwell_ | 16:36 | |
| *** luqas has quit IRC | 16:37 | |
| anteaya | morning greghaynes | 16:38 |
| *** davidlenwell_ is now known as davidlenwell | 16:38 | |
| *** abregman is now known as abregman|brb | 16:38 | |
| *** armax has quit IRC | 16:38 | |
| *** Swami has joined #openstack-infra | 16:39 | |
| *** kdas_ has quit IRC | 16:39 | |
| *** haypo has quit IRC | 16:39 | |
| *** sdae is now known as sdake | 16:39 | |
| zaro | clarkb: i'm refering to https://review.openstack.org/#/c/180371 | 16:39 |
| *** haypo has joined #openstack-infra | 16:40 | |
| *** annegentle has joined #openstack-infra | 16:40 | |
| clarkb | zaro: re your comments the block at line 249 is supposed to handle that | 16:41 |
| *** haypo has left #openstack-infra | 16:41 | |
| clarkb | at least I think that was the intent, but if we don't call register after we close in all cases that wont work | 16:41 |
| *** annegentle has quit IRC | 16:44 | |
| *** patrickeast has joined #openstack-infra | 16:45 | |
| clarkb | zaro: I likely wont get to that today | 16:45 |
| clarkb | but feel free to push a new patchset if you like | 16:45 |
| *** claudiub has joined #openstack-infra | 16:46 | |
| zaro | clarkb: ahh, i missed that, but it definately doesn't seem to handle all cases on reset. | 16:47 |
| *** Longgeek has quit IRC | 16:47 | |
| *** shashank_hegde has joined #openstack-infra | 16:47 | |
| zaro | clarkb: i'll take a look again to see if i can get that to work for all cases. | 16:47 |
| *** rockyg has joined #openstack-infra | 16:47 | |
| *** kiran-r has joined #openstack-infra | 16:47 | |
| *** wenlock has quit IRC | 16:48 | |
| zaro | clarkb: i didn't mean to try it today. i'm just wondering if you were interested in updating the plugin at some point in the future since the plugin seems to be working without problems now. | 16:48 |
| *** tonytan4ever has quit IRC | 16:48 | |
| *** duartea has quit IRC | 16:49 | |
| clarkb | its a less pressing change | 16:49 |
| clarkb | but would probably be good | 16:49 |
| openstackgerrit | Vipul Sabhaya proposed openstack-infra/project-config: Move Cue and related repos to OpenStack namespace https://review.openstack.org/196268 | 16:51 |
| *** kdas_ has joined #openstack-infra | 16:51 | |
| openstackgerrit | Vipul Sabhaya proposed openstack-infra/project-config: Move Cue and related repos to OpenStack namespace https://review.openstack.org/196268 | 16:52 |
| *** claudiub has quit IRC | 16:52 | |
| zaro | clarkb: ok, i just didn't want to put more working into it than necessary :) | 16:52 |
| *** claudiub has joined #openstack-infra | 16:52 | |
| *** ZZelle has joined #openstack-infra | 16:53 | |
| *** hasharMeeting has quit IRC | 16:53 | |
| *** davideagnello has joined #openstack-infra | 16:56 | |
| AJaeger | yolanda, fungi : There's already https://review.openstack.org/#/c/198938/1/gerrit/projects.yaml | 16:58 |
| *** shashank_hegde has quit IRC | 16:59 | |
| *** jsavak has quit IRC | 16:59 | |
| *** ddieterly has joined #openstack-infra | 17:00 | |
| fungi | AJaeger: good find. i guess that could be updated to add acl files | 17:01 |
| *** afazekas has quit IRC | 17:01 | |
| *** esmute__ has joined #openstack-infra | 17:01 | |
| *** shashank_hegde has joined #openstack-infra | 17:02 | |
| *** wenlock has joined #openstack-infra | 17:02 | |
| AJaeger | fungi: yeah, I'll approved yolanda's and will update the other now on top of it | 17:02 |
| *** ociuhandu has quit IRC | 17:04 | |
| fungi | AJaeger: thanks | 17:04 |
| fungi | i also commented on 198938 | 17:04 |
| *** ihrachyshka has quit IRC | 17:05 | |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config: Fix ACL's on projects with puppet-system https://review.openstack.org/198938 | 17:06 |
| * AJaeger updated 198938... | 17:06 | |
| *** wenlock has quit IRC | 17:06 | |
| *** aswadr has quit IRC | 17:07 | |
| *** ihrachyshka has joined #openstack-infra | 17:07 | |
| AJaeger | fungi: Now there're no more acl-config lines in openstack-infra ;) | 17:07 |
| fungi | AJaeger: thanks! | 17:07 |
| openstackgerrit | Merged openstack-infra/project-config: Fix puppet-refstack acl https://review.openstack.org/199177 | 17:07 |
| *** cindyo has quit IRC | 17:07 | |
| pabelanger | fungi, AJaeger: sorry for the noise, but do you mind checking out 199122 ? It's gating me for some git and puppet stuff | 17:08 |
| pabelanger | uploads log files basically | 17:08 |
| *** salv-orl_ has joined #openstack-infra | 17:08 | |
| *** ildikov has joined #openstack-infra | 17:08 | |
| *** e0ne has joined #openstack-infra | 17:09 | |
| AJaeger | pabelanger: +2. | 17:09 |
| *** yamahata has quit IRC | 17:09 | |
| *** jsavak has joined #openstack-infra | 17:09 | |
| pabelanger | AJaeger, danke | 17:10 |
| *** vahidh has joined #openstack-infra | 17:10 | |
| *** salv-orlando has quit IRC | 17:11 | |
| *** HeOS has quit IRC | 17:11 | |
| AJaeger | speaking about uploads, I'd like some advise on https://review.openstack.org/#/c/198802/2/jenkins/jobs/puppet-module-jobs.yaml . The change copies over devstack-logs macro but this is not a devstack setup. So, should it use the macro - or should it use the full definition? fungi, clarkb, do you have any style comments, please? | 17:11 |
| AJaeger | pabelanger: gern geschehen! | 17:11 |
| *** gyee has joined #openstack-infra | 17:12 | |
| *** derekh has quit IRC | 17:12 | |
| *** ddieterly has quit IRC | 17:12 | |
| AJaeger | fungi: Do you want to add krotscheck to puppet-refstack acl? Or wait until he asks for it? ;) | 17:13 |
| krotscheck | AJaeger: Oh god please no. | 17:13 |
| * krotscheck is FAR more productive if he doesn't have to worry about reviewing projects. | 17:13 | |
| AJaeger | krotscheck: ok, we leave puppet-refstack without any cores for now ;) | 17:14 |
| krotscheck | AJaeger: I think hogepoge is the right person to go to for that. | 17:14 |
| fungi | AJaeger: i leave anything beyond including the infra-core and infra-puppet-core groups up to jeblair to formally delegate | 17:14 |
| rockyg | AJaeger, krotscheck Yes, hogepodge and maybe davidlenwell? | 17:15 |
| rockyg | Or pvanek? | 17:15 |
| davidlenwell | I'm here | 17:15 |
| fungi | AJaeger: we have a core review team for our puppet modules. it just needs to be added as an included group | 17:15 |
| fungi | not sure why this devolved into a pingfest | 17:15 |
| AJaeger | fungi: Agreed, let's add the usual setup and let rockyg discuss with jeblair... | 17:16 |
| rockyg | davidlenwell, it's a discussion on whether puppet-refstack has any refstack cores. I put forth you, chris, paul | 17:16 |
| fungi | infra puppet modules have infra-core and infra-puppet-core review teams added to the initial core review group. any additional core reviewers are up to the infra ptl to formally delegate | 17:16 |
| davidlenwell | we can just match the refstack-core group | 17:16 |
| *** austin81 has joined #openstack-infra | 17:16 | |
| davidlenwell | or add the refstack-core group as core | 17:17 |
| *** dizquierdo is now known as dizquierdo_afk | 17:17 | |
| fungi | rockyg: davidlenwell: krotscheck: apologies for you getting pinged on this. it's not something that needs addressing right now | 17:17 |
| davidlenwell | cool | 17:18 |
| davidlenwell | thanks | 17:18 |
| krotscheck | kk | 17:18 |
| rockyg | davidlenwell, I think the question is, who from Refstack should be proposed for the puppet module maintenance/changes | 17:18 |
| *** bdemers has quit IRC | 17:18 | |
| rockyg | 'nuff said | 17:18 |
| fungi | and "nobody" is a perfectly fine answer to that question | 17:18 |
| *** e0ne is now known as e0ne_ | 17:18 | |
| *** bdemers has joined #openstack-infra | 17:18 | |
| AJaeger | mordred: are you around? I have a question on the Meiji release name... | 17:19 |
| anteaya | I'm curious how it should be pronounced | 17:19 |
| davidlenwell | fungi: nobody it is ;) | 17:19 |
| *** david-lyle has quit IRC | 17:19 | |
| anteaya | me-jee | 17:19 |
| anteaya | may-jee | 17:19 |
| krotscheck | "Tasty Chocolate Treat"? | 17:19 |
| fungi | anteaya: mostly the latter | 17:19 |
| AJaeger | anteaya: I see everybody in Tokyo at the first keynote learn to pronounce it ;) | 17:19 |
| anteaya | fungi: thanks, I'll work with that | 17:20 |
| *** abregman|brb is now known as abregman | 17:20 | |
| anteaya | AJaeger: yeah no kidding | 17:20 |
| anteaya | I need a brain pattern though to start to attach things too | 17:20 |
| anteaya | I'll go with may-jee until I learn differently | 17:20 |
| *** austin81 has left #openstack-infra | 17:20 | |
| rockyg | AJaeger, yah. Not so much. No one in the US has been able to pronounce Huawei, even when we work with them on it :P | 17:20 |
| anteaya | I'm going to get a walk in before the meeting | 17:21 |
| *** pradk has joined #openstack-infra | 17:21 | |
| *** e0ne_ is now known as e0ne | 17:21 | |
| AJaeger | rockyg: it's not better in Germany - and pronouncing SUSE has also it's challenges. But so is pronouncing Linux ;) | 17:21 |
| fungi | rockyg: i still don't understand why "wah-way" is hard to remember | 17:21 |
| *** jtomasek has joined #openstack-infra | 17:22 | |
| pleia2 | fungi: it has none of the same letters! ;) (ok, a couple) | 17:22 |
| *** pblaho_ has joined #openstack-infra | 17:22 | |
| rockyg | fungi, Mirkans. Furrin words are hard How-way, Hoo-ah-way, haw-way | 17:22 |
| *** pblaho_ has quit IRC | 17:22 | |
| krotscheck | anteaya: That video that mordred posted has the pronunciation. The last three seconds of this movie have it as well: https://www.youtube.com/watch?v=Cuw5VK4ZvxM | 17:22 |
| krotscheck | anteaya: It's three syllables. | 17:23 |
| *** kiran-r has quit IRC | 17:23 | |
| *** jklare has quit IRC | 17:23 | |
| *** _amrith_ is now known as amrith | 17:24 | |
| rockyg | anteaya, consider it a Hawaiian style pronounciation ;-) | 17:24 |
| *** _nadya_ has joined #openstack-infra | 17:25 | |
| *** achanda has joined #openstack-infra | 17:25 | |
| *** e0ne is now known as e0ne_ | 17:26 | |
| fungi | yeah, "meh-ee-jee" is a little closer, though the first two syllables are sort of a diphthong and get slurred together a bit | 17:26 |
| *** e0ne_ is now known as e0ne | 17:27 | |
| AJaeger | anteaya, mordred : My question was whether the release name Meiji is named after the emperoror or after a location. This came up in https://review.openstack.org/#/c/199060/2/doc/glossary/glossary-terms.xml | 17:27 |
| fungi | japanese pronunciation gets a little interesting around distinguishing syllables. for example the n sound is treated as a distinct syllable, and is more akin to a vowel than a consonant in the way it's used in word construction | 17:28 |
| *** wenlock has joined #openstack-infra | 17:28 | |
| fungi | AJaeger: technically it's named after the location | 17:28 |
| AJaeger | fungi: The shrine or the park? Or either? ;) | 17:29 |
| *** mrunge has quit IRC | 17:29 | |
| rockyg | fungi, and no Japanes word starts with "en" | 17:29 |
| *** annegentle has joined #openstack-infra | 17:30 | |
| krotscheck | Anyone remember off the top of their head what the appropriate bitwise operator is to check if a file is readable? I've got the mode, need to check if I can read it. | 17:30 |
| clarkb | 444 | 17:30 |
| clarkb | in octal | 17:31 |
| AJaeger | "The code name comes from the locations Meiji shrine and Meiji park which are named after the emperor Meiji." | 17:31 |
| fungi | AJaeger: our wiki is a little vague on which (or whether both) but since we name based on locations (kilo being a bit of an exception) it's definitely a place name rather than intended as a direct reference to a person or a historical time period | 17:31 |
| clarkb | there is a defined val for all three bits you should use though | 17:31 |
| clarkb | I would need to read docs for that though | 17:31 |
| AJaeger | fungi: thanks. So, my wording above should be fine, isn't it? | 17:31 |
| fungi | AJaeger: looks right to me | 17:32 |
| fungi | AJaeger: you might extend that to "Meiji shrine and Meiji park in Tokyo" since that was the relevant selection criteria | 17:33 |
| fungi | (i.e. the candidate names were distilled to only locations in tokyo for the final poll) | 17:33 |
| *** e0ne is now known as e0ne_ | 17:33 | |
| smoser | hey, is there something that i can just give a url to a review and have it run 'git fetch ... git branch' for me ? | 17:33 |
| smoser | ie, i'd like: | 17:33 |
| jroll | smoser: git review -d 123456 | 17:34 |
| jroll | will check out that review | 17:34 |
| jroll | s/-d/-x/ if you want to cherry-pick | 17:34 |
| AJaeger | thanks, fungi! | 17:34 |
| smoser | jroll, thats exactly what i was looking for. thanks. | 17:35 |
| jroll | np :) | 17:35 |
| *** annegentle has quit IRC | 17:35 | |
| krotscheck | clarkb: I was looking for &, ^ or |, but it turns out your answer helped me figure it out. Thanks! | 17:35 |
| *** shashank_hegde has quit IRC | 17:37 | |
| smoser | the one thing i'd like is for it to take a url and scrape the review number out of it . cause i'm too lazy, and i have the review url in my browser. | 17:37 |
| smoser | but oh well. | 17:37 |
| openstackgerrit | Khai Do proposed openstack-infra/jenkins-job-builder: Improve delete all job performance https://review.openstack.org/199193 | 17:38 |
| fungi | smoser: i just cut/paste the review number out of the url for that, but i suppose you could whip up a wrapper that just does a regex to pull the value out of a url string | 17:39 |
| *** mayurig has quit IRC | 17:39 | |
| smoser | yeah, thats what i'md oding fungi . you're just less lazy than me. | 17:39 |
| *** e0ne_ is now known as e0ne | 17:39 | |
| greghaynes | clarkb: so this nodepool gearman race - somehow the nodepool gear client is trying to connect to the totally wrong gearman server port | 17:40 |
| *** jklare has joined #openstack-infra | 17:40 | |
| AJaeger | pleia2: could you check https://review.openstack.org/#/c/198236/ , please? The scripts look fine but need some local testing. And I think this needs to be part of the post queue, not the gate queue. | 17:40 |
| clarkb | krotscheck & | 17:40 |
| greghaynes | clarkb: we do the same kind of config templating trick as we do for the db name in test setup to do the gearman server... so its wierd there could be some kind of race there | 17:41 |
| *** austin81 has joined #openstack-infra | 17:41 | |
| fungi | smoser: i dunno, i'm pretty darned lazy. but i'm too lazy to compete with you on who's the laziest | 17:41 |
| smoser | touche | 17:41 |
| clarkb | greghaynes its not just open port 0 ask socket for port? | 17:41 |
| pleia2 | AJaeger: added to my list, thanks for bumping it up in my queue | 17:42 |
| greghaynes | clarkb: that is basically what happens, yes, then we template that out in the nodepool config so the clients use it | 17:42 |
| AJaeger | thanks, pleia2 ! | 17:43 |
| fungi | greghaynes: are the clients reusing the same generated config/same gearman server or are you creating them per-test? | 17:43 |
| *** mayurig has joined #openstack-infra | 17:44 | |
| greghaynes | fungi: it gets made per test, we copy it to a temp location | 17:44 |
| *** yamahata has joined #openstack-infra | 17:44 | |
| *** pcrews has quit IRC | 17:46 | |
| jeblair | smoser: if you use gertty, you can just hit 'x' (cherry-pick) or 'c' (checkout) while looking at the change | 17:46 |
| *** armax has joined #openstack-infra | 17:46 | |
| *** _nadya_ has quit IRC | 17:46 | |
| fungi | greghaynes: hrm, i wonder if the temp location for the configs is maybe not unique and clients end up reading configs not meant for their test? | 17:47 |
| AJaeger | fungi, jeblair: Speaking about new repos: Do you have any guidance on https://review.openstack.org/#/c/198905/ - stackforge/xstatic-angular-gettext? All the other xstatics are in the stackforge namespace but should they stay there or should we encourage them moving to openstack? | 17:47 |
| openstackgerrit | greghaynes proposed openstack-infra/nodepool: Set gearman client_id for nodepool client https://review.openstack.org/199200 | 17:47 |
| *** dguerri is now known as dguerri` | 17:47 | |
| greghaynes | fungi: hrmmm | 17:48 |
| jeblair | AJaeger: i'd keep them together; better for developers, and the marginal cost for us to move one is small | 17:48 |
| AJaeger | jeblair: thanks, will review and approve... | 17:49 |
| fungi | AJaeger: i agree with jeblair. but also one of the issues that resulted in xstatic packages being in stackforge and not officially governed by horizon in the past was a question of licensing (since it's common for the xstatic package to duplicate the license of the software it's packaging) | 17:49 |
| *** esmute__ has quit IRC | 17:49 | |
| openstackgerrit | greghaynes proposed openstack-infra/nodepool: Run geard for nodepool tests https://review.openstack.org/194928 | 17:50 |
| AJaeger | fungi: thanks, good to know! | 17:50 |
| openstackgerrit | greghaynes proposed openstack-infra/nodepool: Trigger image builds using gearman https://review.openstack.org/195175 | 17:50 |
| jeblair | AJaeger, anteaya: i don't have to block devstack-gate job changes, sdague and other cores there can help evaluate those | 17:50 |
| openstackgerrit | greghaynes proposed openstack-infra/nodepool: Trigger image uploads using gearman https://review.openstack.org/197722 | 17:52 |
| openstackgerrit | greghaynes proposed openstack-infra/nodepool: Trigger image delete using gearman https://review.openstack.org/198114 | 17:52 |
| AJaeger | jeblair: so, who should review devstack-gate as well? One of sdague, jeblair, clarkb, fungi? | 17:53 |
| *** Longgeek has joined #openstack-infra | 17:53 | |
| fungi | AJaeger: yep https://review.openstack.org/#/admin/groups/267,members | 17:53 |
| fungi | AJaeger: specifically (at the moment) infra-core plus sdague | 17:54 |
| *** yfried has joined #openstack-infra | 17:54 | |
| *** yfried has quit IRC | 17:54 | |
| AJaeger | So, in other words: Anteaya and myself shouldn't be the only +2s when it gets approved right now ;) | 17:54 |
| AJaeger | (that's easier for me to remember ;) | 17:54 |
| fungi | AJaeger: the goal being that we have some oversight against the (thankfully infrequent) trend to want to add any job that runs devstack-gate as also voting on devstack-gate changes | 17:54 |
| jeblair | AJaeger: well, if the change comes from sdague, probably okay ;) | 17:55 |
| AJaeger | jeblair: ;) | 17:55 |
| jeblair | but yes, what fungi said | 17:55 |
| fungi | the qa team is in a similar situation with people proposing jobs for devstack and tempest just because they happen to run one or both | 17:56 |
| AJaeger | fungi, jeblair: I'll keep both in mind when reviewing. Thanks! | 17:58 |
| openstackgerrit | Merged openstack-infra/jenkins-job-builder: Exclude .test directory from pep8 check https://review.openstack.org/199159 | 17:59 |
| *** reed_ has quit IRC | 17:59 | |
| *** abregman has quit IRC | 17:59 | |
| *** esmute__ has joined #openstack-infra | 18:01 | |
| *** jsavak has quit IRC | 18:02 | |
| *** jsavak has joined #openstack-infra | 18:03 | |
| *** julim_ has joined #openstack-infra | 18:03 | |
| *** marzif_ has joined #openstack-infra | 18:03 | |
| *** Longgeek has quit IRC | 18:04 | |
| *** tonytan4ever has joined #openstack-infra | 18:04 | |
| *** coolsvap|away is now known as coolsvap | 18:04 | |
| *** changbl has quit IRC | 18:05 | |
| *** julim has quit IRC | 18:05 | |
| *** ihrachyshka has quit IRC | 18:07 | |
| *** shashank_hegde has joined #openstack-infra | 18:09 | |
| anteaya | jeblair: thank you | 18:09 |
| openstackgerrit | Merged openstack-infra/project-config: Add Zanata support for openstack-manuals projects https://review.openstack.org/198221 | 18:11 |
| openstackgerrit | Merged openstack-infra/project-config: Create a repository for stackforge/xstatic-angular-gettext https://review.openstack.org/198905 | 18:11 |
| openstackgerrit | Merged openstack-infra/project-config: midonet: Specify Q_PLUGIN=midonet in local.conf https://review.openstack.org/192531 | 18:11 |
| *** hogepodge has joined #openstack-infra | 18:11 | |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade: Add CRUD methods for Keystone domains https://review.openstack.org/190396 | 18:11 |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade: Support project/tenant and domain vs. None https://review.openstack.org/190442 | 18:11 |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade: Allow use of admin tokens in keystone https://review.openstack.org/199209 | 18:11 |
| mordred | Shrews, fungi, jeblair: ^^ enjoy | 18:12 |
| greghaynes | oo fun | 18:12 |
| *** esmute__ has quit IRC | 18:12 | |
| mordred | greghaynes: yah. | 18:12 |
| *** vnod has joined #openstack-infra | 18:13 | |
| Shrews | mordred: i will enjoy so much | 18:13 |
| openstackgerrit | YAMAMOTO Takashi proposed openstack-infra/project-config: Remove Midonet v2 jobs https://review.openstack.org/198979 | 18:15 |
| *** jsavak has quit IRC | 18:15 | |
| *** yamamoto has quit IRC | 18:16 | |
| anteaya | krotscheck: thanks, sounds like may-he-gee to my ears | 18:16 |
| *** kgiusti has joined #openstack-infra | 18:16 | |
| *** soren has quit IRC | 18:16 | |
| *** ZZelle has quit IRC | 18:16 | |
| *** jsavak has joined #openstack-infra | 18:16 | |
| *** _nadya_ has joined #openstack-infra | 18:17 | |
| *** e0ne is now known as e0ne_ | 18:17 | |
| *** hogepodge has quit IRC | 18:20 | |
| *** marzif_ has quit IRC | 18:21 | |
| smoser | jeblair, i did not know of gertty. thanks for that too. | 18:22 |
| *** salv-orl_ has quit IRC | 18:22 | |
| *** _nadya_ has quit IRC | 18:26 | |
| Shrews | mordred: i still would like for 190396 to keep the get/list/search methods with the others to at least keep *some* code organization in that large __init__.py file | 18:26 |
| Shrews | mordred: but i'm ocd'ish that way. if i'm being overly stupid, please say so | 18:27 |
| jeblair | i try to aim for just the right amount of stupid in any given situation | 18:28 |
| mordred | Shrews: oh - piddle. one sec | 18:28 |
| mordred | Shrews: we actually don't seem to have the methods sorted that way in OperatorCloud | 18:29 |
| Shrews | jeblair: it is a fine line :) | 18:29 |
| jeblair | Shrews: are you calling me stupid? | 18:29 |
| Shrews | mordred: those aren't in OperatorCloud | 18:29 |
| mordred | Shrews: yah they are | 18:30 |
| Shrews | mordred: check again :) | 18:30 |
| Shrews | they are inherited | 18:30 |
| mordred | Shrews: oh - actually - list/search/get are in OpenStack - create/delete are in operator | 18:31 |
| Shrews | mordred: we have had this problem often enough, i'm going to split that class out into a separate file, i think | 18:31 |
| Shrews | jeblair: certainly not | 18:31 |
| mordred | Shrews: and now I remember - I needed to rework this to use the new methods that the keystone team added into ksc for us | 18:31 |
| mordred | Shrews: I agree | 18:31 |
| jeblair | Shrews: ;) | 18:31 |
| mordred | Shrews: let's do it after this keystone pile though | 18:31 |
| Shrews | mordred: oh, totally | 18:31 |
| mordred | Shrews: ok - so - there are two ways to get a domain list | 18:33 |
| mordred | Shrews: one is via ksc.auth.domains() - which gives you the list of domains your token is scoped to | 18:33 |
| mordred | Shrews: the other is ksc.domains.list() - which gives you all the domains and is admin only | 18:34 |
| mordred | Shrews: I think that list_domains() should call ksc.auth.domains() for sure | 18:34 |
| *** rockyg has quit IRC | 18:34 | |
| mordred | Shrews: shoudl we put an override in OperatorCloud that calls the unscoped one? | 18:34 |
| *** john-davidge_ has joined #openstack-infra | 18:34 | |
| mordred | Shrews: or leave that for another day? | 18:34 |
| *** pcrews has joined #openstack-infra | 18:34 | |
| mordred | jeblair: (you may also have design thoughts there) | 18:35 |
| Shrews | mordred: that sounds right to me | 18:35 |
| mordred | Shrews: I asked an either/or? | 18:35 |
| Shrews | mordred: but, it can be done later | 18:36 |
| Shrews | mordred: add the override | 18:36 |
| mordred | cool | 18:36 |
| mordred | so - override, but do it later | 18:36 |
| Shrews | yeah | 18:36 |
| mordred | ossum | 18:36 |
| jeblair | that seems sensible | 18:36 |
| *** sdake has quit IRC | 18:36 | |
| *** esmute__ has joined #openstack-infra | 18:36 | |
| *** signed8bit is now known as signed8bit_ZZZzz | 18:37 | |
| *** rossella_s has quit IRC | 18:37 | |
| *** rossella_s has joined #openstack-infra | 18:37 | |
| *** esmute__ has quit IRC | 18:38 | |
| *** esmute__ has joined #openstack-infra | 18:38 | |
| *** esmute__ has quit IRC | 18:39 | |
| *** e0ne_ is now known as e0ne | 18:39 | |
| *** heyongli has quit IRC | 18:39 | |
| *** koolhead_ has quit IRC | 18:39 | |
| john-davidge_ | is jenkins failing on python34 currently a known issue? | 18:40 |
| fungi | john-davidge_: you'll have to be a _lot_ more specific | 18:41 |
| fungi | john-davidge_: an example is probably the best place to start your questions | 18:41 |
| john-davidge_ | fungi: haha, sure. Here’s 3 examples: https://review.openstack.org/#/c/199164/ https://review.openstack.org/#/c/165023/ https://review.openstack.org/#/c/158697/ | 18:42 |
| john-davidge_ | fungi: from my quick look through reviews these failures seem to be intermittent | 18:43 |
| fungi | john-davidge_: those all look like recent neutron test failures. i know there was a release of the "routes" dependency that broke python 3 compatibility in the past week-ish. let me find you a bug report and you can see if that matches what you're getting | 18:44 |
| john-davidge_ | fungi: thanks | 18:44 |
| fungi | or it's possible we bumped the upper cap on routes to include a broken version | 18:45 |
| *** heyongli has joined #openstack-infra | 18:45 | |
| *** Sukhdev has joined #openstack-infra | 18:45 | |
| pabelanger | jeblair, anything else specific you want to see for the Host trystack.o.o site spec before adding it into the spec approval section for the meeting? review 195098 | 18:45 |
| fungi | john-davidge_: upstream report seems to be https://github.com/bbangert/routes/issues/51 | 18:46 |
| fungi | haypo's not around right now or i'd ask for a shortcut to related bug reports | 18:47 |
| john-davidge_ | fungi: Ok thanks, looks like its a legit failure then. will hold off on the rechecks for now :) | 18:49 |
| *** salv-orlando has joined #openstack-infra | 18:49 | |
| *** heyongli has quit IRC | 18:50 | |
| *** pcrews has quit IRC | 18:50 | |
| *** e0ne has quit IRC | 18:50 | |
| sc68cal | anteaya: quick update on the linuxbridge job - we've got https://review.openstack.org/198833 in the pipe which fixes another bug we've seen in the LB CI job | 18:50 |
| anteaya | yay another bug fix | 18:51 |
| anteaya | sc68cal: thank you! | 18:51 |
| fungi | john-davidge_: hrm... https://review.openstack.org/182746 blacklists routes 2.1 and is still failing its python34 job, so might be something new | 18:51 |
| sc68cal | anteaya: do you have any thoughts on merging https://review.openstack.org/195653 next week or so? | 18:51 |
| *** achanda has quit IRC | 18:51 | |
| *** sdake has joined #openstack-infra | 18:52 | |
| *** baoli has quit IRC | 18:52 | |
| anteaya | sc68cal: let's see where we are after 198833 gets some exercise | 18:52 |
| anteaya | but yes would be happy to look at it even at the end of this week | 18:53 |
| anteaya | sc68cal: thank you | 18:53 |
| john-davidge_ | fungi: :/ | 18:53 |
| *** dims has quit IRC | 18:53 | |
| fungi | russellb: you commented on https://review.openstack.org/182746 about routes breaking python 3. can you elaborate? | 18:55 |
| pelix | zaro: I believe I have a way to test the JJB timeout capability, though it's currently using gevent.server.StreamServer which is probably a bit heavy for the need | 18:55 |
| *** mriedem has quit IRC | 18:55 | |
| fungi | russellb: i see that change is blacklisting routes 2.1 which has an open github issue about wanting a 2.2 release so that python 3 fixes will be available | 18:55 |
| pelix | basically just creates a server in a thread to open a port but ignores all connection attempts | 18:56 |
| *** baoli has joined #openstack-infra | 18:56 | |
| sc68cal | anteaya: that sounds good, thank you | 18:56 |
| *** annegentle has joined #openstack-infra | 18:56 | |
| *** sdake_ has joined #openstack-infra | 18:56 | |
| jhesketh | Morning | 18:57 |
| *** heyongli has joined #openstack-infra | 18:57 | |
| fungi | russellb: the corresponding requirements change https://review.openstack.org/196867 suggests that routes 2.1 was blacklisted because of bug 1469883 | 18:57 |
| openstack | bug 1469883 in OpenStack Compute (nova) "Unit tests fail with Routes==2.1" [Medium,Fix committed] https://launchpad.net/bugs/1469883 - Assigned to Davanum Srinivas (DIMS) (dims-v) | 18:57 |
| *** dims has joined #openstack-infra | 18:58 | |
| anteaya | jhesketh: hey | 18:59 |
| anteaya | sc68cal: thank you | 18:59 |
| *** sdake has quit IRC | 19:00 | |
| pleia2 | meeting time | 19:00 |
| *** hogepodge has joined #openstack-infra | 19:00 | |
| fungi | aha... russellb: john-davidge_: dims: seems i was previously aware of this and then drank those braincells away https://github.com/bbangert/routes/issues/37 | 19:00 |
| dims | haha fungi :) | 19:01 |
| fungi | dims: did you maybe have a cached wheel? | 19:01 |
| *** ildikov has quit IRC | 19:02 | |
| *** _nadya_ has joined #openstack-infra | 19:02 | |
| fungi | dims: wondering now if pip autobuilding and caching wheels is bringing this issue back | 19:02 |
| *** harlowja_ has joined #openstack-infra | 19:02 | |
| *** harlowja has quit IRC | 19:03 | |
| *** armax has quit IRC | 19:03 | |
| dims | fungi: something made me believe that haypo took care of it in https://github.com/haypo/routes/commit/10400ba8a269cd3d000db4f47543b9238cc01d55 | 19:03 |
| dims | so will wait for next rev of Routes | 19:03 |
| *** ildikov has joined #openstack-infra | 19:03 | |
| *** heyongli has quit IRC | 19:03 | |
| *** ociuhandu has joined #openstack-infra | 19:05 | |
| *** heyongli has joined #openstack-infra | 19:05 | |
| *** Rockyg has joined #openstack-infra | 19:06 | |
| *** mriedem has joined #openstack-infra | 19:06 | |
| *** jsavak has quit IRC | 19:08 | |
| *** kdas_ has quit IRC | 19:08 | |
| *** armax has joined #openstack-infra | 19:10 | |
| openstackgerrit | Scott Moser proposed openstack-infra/git-review: support urls as change-id input https://review.openstack.org/199225 | 19:10 |
| *** heyongli has quit IRC | 19:10 | |
| openstackgerrit | Doug Hellmann proposed openstack-infra/infra-specs: cleanups for centralize-release-tagging https://review.openstack.org/199226 | 19:10 |
| *** puranamr has quit IRC | 19:10 | |
| *** radez is now known as radez_g0n3 | 19:10 | |
| *** soren has joined #openstack-infra | 19:10 | |
| *** cindyo has joined #openstack-infra | 19:11 | |
| *** jsavak has joined #openstack-infra | 19:11 | |
| *** esmute__ has joined #openstack-infra | 19:12 | |
| openstackgerrit | Emilien Macchi proposed openstack-infra/project-config: specs: fix puppet specs URL https://review.openstack.org/199228 | 19:13 |
| *** zz_ja is now known as zz_zz_ja | 19:13 | |
| *** ayoung is now known as ayoung-afk | 19:13 | |
| *** signed8bit_ZZZzz is now known as signed8bit | 19:14 | |
| *** yamamoto has joined #openstack-infra | 19:16 | |
| *** heyongli has joined #openstack-infra | 19:17 | |
| fungi | dims: got it. apparently that's blocking requirements sync to neutron since neutron's gating on python 3 so was working with routes 2.1 but not with the routes 1.x that they end up with after blacklisting 2.1 | 19:19 |
| *** pcrews has joined #openstack-infra | 19:20 | |
| dims | ah i see. we weave a really tangled web :) | 19:20 |
| fungi | seriously :/ | 19:20 |
| *** yamamoto has quit IRC | 19:21 | |
| *** changbl has joined #openstack-infra | 19:22 | |
| *** heyongli has quit IRC | 19:24 | |
| *** mriedem has quit IRC | 19:25 | |
| *** bdemers has quit IRC | 19:25 | |
| pelix | zaro: I'm thinking that although I have this working on JJB at the moment, that I should move it to python-jenkins | 19:26 |
| dims | fungi: would something like this work for them? http://paste.openstack.org/show/352789/ | 19:26 |
| dims | (and all of us!) | 19:26 |
| pelix | since that is where we really want to test the timeout behaviour and with JJB only should care that we set the correct properties | 19:26 |
| *** heyongli has joined #openstack-infra | 19:27 | |
| *** signed8bit is now known as signed8bit_ZZZzz | 19:28 | |
| *** abregman has joined #openstack-infra | 19:28 | |
| *** signed8bit_ZZZzz is now known as signed8bit | 19:28 | |
| fungi | dims: possibly. you could try amending 182746 to see if it passes | 19:29 |
| fungi | since the requirements jobs aren;t going to check unit tests for individual projects | 19:29 |
| *** mriedem has joined #openstack-infra | 19:29 | |
| *** linkedinyou has joined #openstack-infra | 19:30 | |
| *** shardy_ has joined #openstack-infra | 19:31 | |
| *** heyongli has quit IRC | 19:32 | |
| *** shardy has quit IRC | 19:32 | |
| *** bswartz has quit IRC | 19:32 | |
| *** vnod has quit IRC | 19:32 | |
| *** bswartz has joined #openstack-infra | 19:32 | |
| pleia2 | mordred: you seem tight with the openstack ansible folks, want to tell them to get their rename patch in? they're still over in stackforge | 19:34 |
| pleia2 | and it's been a while :) | 19:34 |
| mordred | pleia2: sure! | 19:35 |
| pleia2 | would be great to see them moved over on friday | 19:35 |
| *** shardy_ has quit IRC | 19:36 | |
| *** tonytan4ever has quit IRC | 19:37 | |
| *** achanda has joined #openstack-infra | 19:37 | |
| *** shardy has joined #openstack-infra | 19:37 | |
| *** luqas has joined #openstack-infra | 19:38 | |
| *** rlandy has quit IRC | 19:38 | |
| *** john-davidge_ has quit IRC | 19:38 | |
| *** smccully has quit IRC | 19:38 | |
| *** heyongli has joined #openstack-infra | 19:40 | |
| *** bdemers has joined #openstack-infra | 19:41 | |
| lifeless | morganfainberg: and yeah - loook at the content of the reply to me on the git list :/ | 19:42 |
| *** tonytan4ever has joined #openstack-infra | 19:42 | |
| lifeless | morganfainberg: 99% 'fit into our community norms', 1% actual review | 19:42 |
| *** luqas has quit IRC | 19:42 | |
| *** austin81 has left #openstack-infra | 19:45 | |
| *** heyongli has quit IRC | 19:45 | |
| *** austin81 has joined #openstack-infra | 19:45 | |
| morganfainberg | lifeless: will it male | 19:46 |
| morganfainberg | Me | 19:46 |
| morganfainberg | Make me* sad? | 19:46 |
| * morganfainberg throws phone smudgy little screen across the room. | 19:47 | |
| Clint | it will mail you sad | 19:47 |
| openstackgerrit | Sean Dague proposed openstack-dev/pbr: Add wsgi_scripts support to PBR https://review.openstack.org/195292 | 19:47 |
| openstackgerrit | Emilien Macchi proposed openstack-infra/project-config: puppet & tripleo: use devstack-logs publisher for logs https://review.openstack.org/198802 | 19:47 |
| EmilienM | AJaeger: ^ | 19:47 |
| *** dims has quit IRC | 19:48 | |
| zaro | pelix: cool and agreed! | 19:48 |
| *** rcarrillocruz has joined #openstack-infra | 19:49 | |
| rcarrillocruz | asselin: heya, it's a shame tomorrow i'll be prepping my vaca (leaving on Thu), won't be attending the sprint | 19:49 |
| rcarrillocruz | and i really wanted to be there | 19:49 |
| rcarrillocruz | :/ | 19:49 |
| *** _nadya_ has quit IRC | 19:49 | |
| *** maishsk has joined #openstack-infra | 19:49 | |
| pleia2 | asselin: care to send out a reminder about the puppet virtual sprint tomorrow+thursday? | 19:50 |
| asselin | rcarrillocruz, np | 19:50 |
| asselin | pleia2, sure will do | 19:50 |
| pleia2 | thanks :) | 19:50 |
| sdague | lifeless: so I'm not going to say I'm proud of the tests added to https://review.openstack.org/#/c/195292/, but there are some now. And they do test the critical functionality. But it took a lot of jiggery pokery to come up with a pattern that didn't just deadlock python. | 19:50 |
| asselin | rcarrillocruz, enjoy your vacation | 19:50 |
| morganfainberg | Clint: i see what you did there. | 19:50 |
| *** david-lyle has joined #openstack-infra | 19:50 | |
| *** esmute__ has quit IRC | 19:52 | |
| *** ayoung-afk is now known as ayoung | 19:52 | |
| *** heyongli has joined #openstack-infra | 19:52 | |
| rcarrillocruz | thx sir | 19:53 |
| openstackgerrit | Scott Moser proposed openstack-infra/git-review: support urls as change-id input https://review.openstack.org/199225 | 19:53 |
| *** e0ne has joined #openstack-infra | 19:54 | |
| *** esmute__ has joined #openstack-infra | 19:55 | |
| *** heyongli has quit IRC | 19:57 | |
| *** heyongli has joined #openstack-infra | 19:59 | |
| *** austin81 has quit IRC | 20:00 | |
| *** jtomasek has quit IRC | 20:00 | |
| pleia2 | not in time for the meeting, but worth mentioning, we're at the midpoint with our Outreachy intern, taron has done an admirable job getting up to speed and we've discussed & reported such to the outreachy community for the midterm update, eyes on hound changes continue to be appreciated :) | 20:00 |
| *** wshao has joined #openstack-infra | 20:00 | |
| mordred | pleia2: w00t | 20:00 |
| crinkle | \o/ | 20:00 |
| persia | jeblair: What's the best way to send you the names/email of the folk in question? Email? IRC? | 20:00 |
| *** Longgeek has joined #openstack-infra | 20:01 | |
| taron | should be more patches to review later today, just need to do some reshuffling so I have an ubuntu test vm again | 20:01 |
| thingee | fungi: going to be doing a tag release soon for cinderclient with the revert of version discovery. putting together the release notes, should be quiet otherwise. | 20:01 |
| pleia2 | taron: yay! | 20:01 |
| jeblair | persia: either is good | 20:01 |
| fungi | thingee: thanks for the heads up | 20:01 |
| taron | go has the strangest, ugliest way of compiling things I have ever seen | 20:01 |
| openstackgerrit | Darragh Bailey proposed openstack-infra/jenkins-job-builder: Allow JJB to set connection timeout to jenkins server https://review.openstack.org/151472 | 20:01 |
| fungi | taron: that's excellent news! | 20:02 |
| fungi | well, i mean not the ugliness of go compilation. that's not news at all | 20:02 |
| taron | hahaha | 20:02 |
| nibalizer | taron: \o/ | 20:02 |
| pleia2 | hehe | 20:02 |
| fungi | just sort of a fact of life | 20:02 |
| taron | vcsrepo { '/opt/hound/src/github.com/etsy/hound': | 20:02 |
| taron | died a little inside while writing that | 20:02 |
| pelix | zaro: I've modified the timeout patch to abstract from socket and confirmed it works as expected, also updated to support floats which are supported. | 20:03 |
| nibalizer | pleia2: fungi could I have core on beaker-localhost and beaker-nodepool | 20:03 |
| anteaya | taron pleia2 fungi is there a gerrit topic selected for hound changes? | 20:03 |
| Clint | /opt is pretty bad | 20:03 |
| pelix | zaro: be tomorrow or the next day when I get around to adding the tests to python-jenkins for timeout behaviour | 20:03 |
| nibalizer | assuming jeblair doesn't want to pop those out since it did go without his +1 | 20:03 |
| pleia2 | anteaya: no, but the work is happening in puppet-hound (eventually system-config for our specific stuff) | 20:03 |
| pelix | zaro: just need to convert from using gevent to something more lightweight | 20:03 |
| pleia2 | puppet-hound repo, I mean | 20:04 |
| yolanda | fungi, jeblair, i was wondering if you could cut some release for gerritlib. I have a patch for manage-projects, to add descriptions of projects, blocked for very long time, and just depends on some code that landed in gerritlib | 20:04 |
| taron | Clint: that's about the one part of that path I can modify if you suggest it being placed somewhere else | 20:04 |
| anteaya | pleia2: oh so just anything to the puppet-hound repo | 20:04 |
| taron | suppose I could do it in a service user's homedir or something instead | 20:04 |
| Clint | taron: you should wait several months before baiting me to express an opinion on that | 20:04 |
| *** puranamr has joined #openstack-infra | 20:04 | |
| pleia2 | anteaya: yep! | 20:04 |
| anteaya | pleia2: okay thanks | 20:04 |
| taron | heh, k | 20:04 |
| pleia2 | anteaya: the boilerplate landed yesterday | 20:05 |
| anteaya | okay | 20:05 |
| anteaya | good | 20:05 |
| *** pelix has quit IRC | 20:05 | |
| lifeless | sdague: thanks, I'll look soon | 20:05 |
| nibalizer | anteaya: taron 'code-search' is the topic the spec states should be our topic for the hound stuff | 20:05 |
| pleia2 | nibalizer: oh, thanks | 20:05 |
| *** chlong has joined #openstack-infra | 20:06 | |
| *** jtomasek has joined #openstack-infra | 20:07 | |
| *** marzif_ has joined #openstack-infra | 20:07 | |
| anteaya | nibalizer: ah thank you | 20:07 |
| *** armax has quit IRC | 20:08 | |
| lifeless | morganfainberg: dunno, does http://www.spinics.net/lists/git/msg255543.html make you sad ? | 20:08 |
| *** reazem_ has quit IRC | 20:09 | |
| fungi | nibalizer: those are the projects yolanda was talking about having been originally configured to point to a nonexistent acl | 20:09 |
| *** jtomasek has quit IRC | 20:09 | |
| *** heyongli has quit IRC | 20:09 | |
| *** jtomasek has joined #openstack-infra | 20:10 | |
| morganfainberg | lifeless: yes. Can haz gerrit. Or at least something with pull requests barring a nice tool for bigger communities? :( | 20:10 |
| fungi | nibalizer: https://review.openstack.org/198938 needs to be approved to fix them | 20:10 |
| *** amrith is now known as _amrith_ | 20:11 | |
| *** austin81 has joined #openstack-infra | 20:11 | |
| pleia2 | on our way | 20:11 |
| *** heyongli has joined #openstack-infra | 20:14 | |
| *** fawadkhaliq has quit IRC | 20:15 | |
| *** puranamr has quit IRC | 20:15 | |
| *** marzif__ has joined #openstack-infra | 20:16 | |
| *** armax has joined #openstack-infra | 20:17 | |
| *** marzif_ has quit IRC | 20:18 | |
| *** ivar-laz_ has joined #openstack-infra | 20:18 | |
| *** jtomasek has quit IRC | 20:19 | |
| *** puranamr has joined #openstack-infra | 20:19 | |
| openstackgerrit | Merged openstack-infra/project-config: Fix ACL's on projects with puppet-system https://review.openstack.org/198938 | 20:19 |
| *** spzala has quit IRC | 20:19 | |
| *** salv-orlando has quit IRC | 20:20 | |
| *** kgiusti has left #openstack-infra | 20:21 | |
| *** ivar-lazzaro has quit IRC | 20:21 | |
| *** Longgeek has quit IRC | 20:24 | |
| *** marzif__ has quit IRC | 20:26 | |
| *** akraminsky has joined #openstack-infra | 20:28 | |
| akraminsky | I'm looking for a file called multinode_setup.sh which is used as the ready-script for multinode projects. I can't seem to locate it, does anyone know off the top of their head where I can find it? | 20:28 |
| fungi | akraminsky: http://git.openstack.org/cgit/openstack-infra/project-config/tree/nodepool/scripts/multinode_setup.sh | 20:29 |
| akraminsky | fungi, thanks! | 20:30 |
| fungi | akraminsky: you're welcome | 20:31 |
| mgagne | so nodepool thinks it's a good idea to delete my node while a job is still running on it. Is there a known bug or broken python-jenkins/jenkins versions in the wild? | 20:33 |
| *** dprince has quit IRC | 20:34 | |
| fungi | mgagne: do you have info from the nodepool debug log? it should normally base that decision on receipt of a zeromq message from the jenkins master stating that the job has completed | 20:35 |
| fungi | mgagne: is it possible the worker is not being put into single-use mode and has started running a new job before nodepool processes the deletion? | 20:36 |
| mgagne | fungi: I don't know how to check that info. How can I check? | 20:36 |
| akraminsky | mgagne: you can tail -f /var/log/nodepool.debug.log while running the job | 20:37 |
| fungi | mgagne: see if a running job has an OFFLINE_NODE_WHEN_COMPLETE parameter being set | 20:38 |
| akraminsky | mgagne: err sorry /var/log/nodepool/debug.log | 20:38 |
| mgagne | forgot about that file, checking now | 20:38 |
| mgagne | OFFLINE_NODE_WHEN_COMPLETE is not found in the file | 20:38 |
| fungi | mgagne: the parameter value should be 1 to tell jenkins to set the node offline so that it won't pick up a second job while it awaits deletion | 20:38 |
| fungi | mgagne: i mean in the job parameters in jenkins | 20:38 |
| *** coolsvap is now known as coolsvap|away | 20:39 | |
| fungi | mgagne: like https://jenkins01.openstack.org/job/gate-puppet-keystone-puppet-unit-4.0/80/parameters/ | 20:39 |
| mgagne | :O wth is this parameter :O | 20:39 |
| *** heyongli has quit IRC | 20:39 | |
| fungi | mgagne: we use a zuul parameter function to set it for our jobs that run on single-use workers | 20:40 |
| mgagne | sure but I never read anything about it in nodepool docs if you see what I mean | 20:40 |
| fungi | mgagne: it's a jenkins+zuul detail not anything specific to nodepool | 20:40 |
| mgagne | hmmm, I'm not using zuul or gearman | 20:41 |
| mgagne | only zeromq plugin | 20:41 |
| *** jamielennox is now known as jamielennox|away | 20:41 | |
| fungi | mgagne: aha, the triggering on that parameter is done within the jenkins-gearman plugin so that might be a coupling concern we didn't foresee | 20:41 |
| *** madong_ has joined #openstack-infra | 20:42 | |
| mgagne | node is marked "USED" as soon as the job starts | 20:42 |
| fungi | mgagne: yep, and then marked as delete when the job completes. but if it takes a few seconds for nodepool to remove it from jenkins then it might start running a second job | 20:43 |
| mgagne | yep, that's what happened | 20:43 |
| fungi | and then get deleted in the beginning of that second job | 20:43 |
| *** madong has quit IRC | 20:43 | |
| *** e0ne has quit IRC | 20:44 | |
| *** heyongli has joined #openstack-infra | 20:44 | |
| fungi | jeblair: ianw: clarkb: zaro: ^ any ideas on how/if that can be worked around, short of running zuul? like set the parameter in job configuration and install jenkins-gearman but point it at a dummy gearman on localhost or something? | 20:44 |
| mgagne | fungi: could it be related to that bug? https://review.openstack.org/#/c/193600/ | 20:45 |
| fungi | wondering if there could be a stripped-down jenkins-gearman plugin which just does the OFFLINE_NODE_WHEN_COMPLETE bit | 20:45 |
| *** smccully has joined #openstack-infra | 20:45 | |
| *** Sukhdev has quit IRC | 20:45 | |
| *** erlon has joined #openstack-infra | 20:46 | |
| fungi | mgagne: maybe that could decrease the incidence, but the underlying issue (and reason we have OFFLINE_NODE_WHEN_COMPLETE in the jenkins-gearman plugin) is so that jenkins itself can avoid a race condition between job completion and node deletion | 20:47 |
| mgagne | true | 20:47 |
| mgagne | so unless I use gearman plugin or write my own to offline the node, there isn't much I can do for now | 20:48 |
| fungi | mgagne: yeah, i mean the routines and framework are there in the jenkins-gearman plugin source already, so it might not be hard to strip out the bulk of the things you're not using to just have an offline-when-complete plugin for jenkins | 20:48 |
| mgagne | or I move the cleanup cron to a time where jobs aren't running and accept the fact that a node could be used multiple times before it's cleaned up | 20:50 |
| fungi | though i'm not well versed in java, so consider that a guess on my part | 20:50 |
| *** jasondotstar has joined #openstack-infra | 20:50 | |
| fungi | mgagne: that's a possible workaround too, i think. we have a bunch of jobs where rerunning them on the same worker more than once will just outright fail, so that's not really something we've tried | 20:51 |
| *** jamielennox|away is now known as jamielennox | 20:52 | |
| mgagne | our jobs aren't that destructive (yet) so it's a compromise I can accept for now | 20:52 |
| *** rtom has joined #openstack-infra | 20:54 | |
| *** bswartz has quit IRC | 20:54 | |
| *** heyongli has quit IRC | 20:55 | |
| *** ivar-laz_ has quit IRC | 20:57 | |
| *** ivar-lazzaro has joined #openstack-infra | 20:57 | |
| *** HeOS has joined #openstack-infra | 20:58 | |
| *** marzif__ has joined #openstack-infra | 20:58 | |
| *** puranamr has quit IRC | 21:01 | |
| *** puranamr has joined #openstack-infra | 21:03 | |
| jeblair | mgagne: you want to run nodepool without zuul? | 21:03 |
| mgagne | yea | 21:03 |
| jeblair | why? | 21:03 |
| *** wshao has quit IRC | 21:03 | |
| *** thorst has quit IRC | 21:03 | |
| mgagne | jeblair: yea, I am already and it's working fine (except when a race occurs) | 21:03 |
| mgagne | jeblair: because I don't have time to setup all that stuff | 21:03 |
| mgagne | jeblair: I'm working on it on borrowed time | 21:04 |
| *** julim_ has quit IRC | 21:04 | |
| jeblair | mgagne: well, it's not designed for that, as you have found. you would need to add a new plugin to jenkins to do the locking that the gearman plugin does. it's not easy, which is why it's implemented the way it is. | 21:05 |
| mgagne | jeblair: well, nodepool docs doesn't mention zuul much and let you understand that it's possible to run it without zuul | 21:05 |
| mgagne | jeblair: which I'm currently doing | 21:06 |
| mgagne | http://docs.openstack.org/infra/nodepool/installation.html#zuul | 21:06 |
| jeblair | mgagne: i'd recommend either using zuul (puppet-zuul and puppet-openstackci are intended to make it easier, but might not be there yet), or using something like jclouds which is designed for jenkins generally | 21:06 |
| openstackgerrit | Cody Herriges proposed openstack-infra/project-config: Move "bundle install" for puppet-modules-jobs. https://review.openstack.org/199291 | 21:06 |
| jeblair | that might be very old :/ | 21:07 |
| *** maishsk has quit IRC | 21:07 | |
| *** aysyd has quit IRC | 21:07 | |
| *** dustins has quit IRC | 21:07 | |
| mgagne | nodepool is working fine except for that race which I "fixed" by moving the cleanup cron to 4am when there is no job running | 21:08 |
| *** heyongli has joined #openstack-infra | 21:08 | |
| openstackgerrit | Kyle Mestery proposed openstack-infra/project-config: Add stable ACLs for Neutron sub-projects https://review.openstack.org/198750 | 21:08 |
| openstackgerrit | Kyle Mestery proposed openstack-infra/project-config: Add a create ACL for networking-bgpvpn https://review.openstack.org/198751 | 21:08 |
| openstackgerrit | Kyle Mestery proposed openstack-infra/project-config: Update merge commit gerrit ACL for Neutron repositories https://review.openstack.org/199089 | 21:08 |
| openstackgerrit | Kyle Mestery proposed openstack-infra/project-config: Update ACLs for Neutron sub-projects https://review.openstack.org/198749 | 21:08 |
| mgagne | zuul might come later, not now | 21:08 |
| asselin | pleia2, comments on reminder e-mail? https://etherpad.openstack.org/p/common-ci-sprint-reminder | 21:08 |
| jeblair | mgagne: it's not deleting nodes after jobs run? | 21:08 |
| mgagne | jeblair: it's something I can accept with my kind of jobs | 21:08 |
| jeblair | mgagne: oh i see | 21:09 |
| mgagne | jeblair: we already have permanent slaves | 21:09 |
| openstackgerrit | Scott Moser proposed openstack-infra/git-review: support urls as change-id input https://review.openstack.org/199225 | 21:09 |
| jeblair | so you're not trying to use it to make single-use slaves? | 21:09 |
| mgagne | jeblair: just trying to move to nodepool to automate slaves update and move away from puppet | 21:09 |
| *** rtom has quit IRC | 21:09 | |
| openstackgerrit | Cody Herriges proposed openstack-infra/project-config: Move "bundle install" for puppet-modules-jobs. https://review.openstack.org/199291 | 21:09 |
| mgagne | jeblair: if it works, yes. otherwise I don't mind much for now | 21:09 |
| jeblair | i think i get the picture | 21:09 |
| pleia2 | asselin: reviewing, and looking up ref re: timezones now | 21:10 |
| pleia2 | (I think it was at a recent meeting) | 21:10 |
| openstackgerrit | Cody Herriges proposed openstack-infra/project-config: Move "bundle install" for puppet-modules-jobs. https://review.openstack.org/199291 | 21:10 |
| mgagne | if I'm *really* annoyed by garbage on slaves and need *real* single-use, I'll check zuul or just write a plugin for jenkins | 21:10 |
| asselin | pleia2, yeah...not sure how to search for that (easily) | 21:10 |
| pleia2 | asselin: found and adding | 21:11 |
| *** abregman has quit IRC | 21:11 | |
| asselin | pleia2, did you login and grep the file systm? | 21:12 |
| *** dims has joined #openstack-infra | 21:13 | |
| *** austin81 has quit IRC | 21:13 | |
| pleia2 | asselin: no, I just opened up recent meeting logs until I found it | 21:13 |
| *** ociuhandu has quit IRC | 21:13 | |
| taron | mordred: how're you going about generating the config file for hound.o.o? trying to come up with a plan of attack for the module | 21:13 |
| *** wshao has joined #openstack-infra | 21:13 | |
| pleia2 | used the standard logs for the link, since that allows me to link to a timestamp <3 | 21:13 |
| asselin | pleia2, thanks | 21:13 |
| taron | it builds which is all well and good until you try to run it and it dies from lack of config.json | 21:13 |
| pleia2 | asselin: email looks good to me now | 21:14 |
| jeblair | fungi, mgagne: i don't think the suggestion from fungi at 20:44 will work -- i believe that parameter is only considered by jenkins gearman plugin if it comes with a gearman job start request | 21:14 |
| mordred | taron: I wrote a patch to jeepyb ... one sec | 21:14 |
| asselin | pleia2, ok...need to go to a meetgin...will send it off when I get back (about 30 mins) | 21:14 |
| fungi | jeblair: ahh, yep that seems likely. so it really would need a separate plugin or an actual zuul | 21:14 |
| mgagne | jeblair: yes. However I'm suggesting writing a new plugin replicating this behavior | 21:14 |
| EmilienM | hello infra, I hope sun is shinning for you, can we have a review on https://review.openstack.org/#/c/199291/ to fix our upgrade job please ? | 21:15 |
| mordred | taron: https://review.openstack.org/#/c/178474/ | 21:15 |
| jeblair | mgagne: yes, that may work. however, i believe that will be more work than setting up zuul. | 21:15 |
| taron | mordred: much obliged | 21:15 |
| mordred | taron: idea is that we have a projects.yaml already - so we can just tranform that information | 21:16 |
| *** jasondotstar has quit IRC | 21:16 | |
| *** maishsk has joined #openstack-infra | 21:16 | |
| jeblair | (but with a payoff for the community, and the gearman plugin could have a cleaner design; so not a bad idea) | 21:16 |
| *** ldnunes has quit IRC | 21:16 | |
| *** heyongli has quit IRC | 21:17 | |
| *** armax has quit IRC | 21:17 | |
| taron | mordred: yeah, that's how I assumed it would work for our purposes, generalizing it for the module may take a bit more thinking | 21:17 |
| taron | either that or it could just not manage config.json, but that seems a bit odd | 21:17 |
| *** annegentle has quit IRC | 21:18 | |
| taron | something toggleable, perhaps | 21:18 |
| fungi | taron: also you can crib from where we install jeepyb in http://git.openstack.org/cgit/openstack-infra/system-config/tree/modules/openstack_project/manifests/git_backend.pp to run create-cgitrepos | 21:18 |
| fungi | taron: and also from that same puppet class you can see how to get projects.yaml on the system | 21:18 |
| taron | oh cool | 21:19 |
| * taron will do some reading | 21:19 | |
| fungi | taron: though a lot of the same glue can be found in the hound class from monty's poc change https://review.openstack.org/#/c/178488/5/modules/openstack_project/manifests/hound.pp (which is linked in the code-search spec) | 21:19 |
| *** dizquierdo_afk is now known as dizquierdo | 21:20 | |
| fungi | so some viable examples to work from | 21:20 |
| *** sdake_ is now known as sdake | 21:20 | |
| * taron needs to reread the spec, evidently | 21:20 | |
| taron | indeed | 21:20 |
| fungi | taron: for reference, mentioned in work item #2 at http://specs.openstack.org/openstack-infra/infra-specs/specs/code-search.html#work-items | 21:21 |
| *** MarkAtwood has quit IRC | 21:21 | |
| *** cindyo has quit IRC | 21:21 | |
| *** bmace has quit IRC | 21:21 | |
| *** salv-orlando has joined #openstack-infra | 21:22 | |
| *** fallenpegasus has quit IRC | 21:22 | |
| *** heyongli has joined #openstack-infra | 21:24 | |
| *** dboik has quit IRC | 21:25 | |
| *** marzif_ has joined #openstack-infra | 21:26 | |
| *** marzif__ has quit IRC | 21:26 | |
| *** austin81 has joined #openstack-infra | 21:26 | |
| *** miqui has quit IRC | 21:27 | |
| *** heyongli has quit IRC | 21:28 | |
| *** heyongli has joined #openstack-infra | 21:29 | |
| *** eharney has quit IRC | 21:29 | |
| *** salv-orlando has quit IRC | 21:29 | |
| *** salv-orlando has joined #openstack-infra | 21:30 | |
| *** dims has quit IRC | 21:30 | |
| *** jsavak has quit IRC | 21:30 | |
| *** esmute__ has quit IRC | 21:32 | |
| *** marzif_ has quit IRC | 21:32 | |
| *** pradk has quit IRC | 21:32 | |
| *** esmute__ has joined #openstack-infra | 21:33 | |
| *** heyongli has quit IRC | 21:33 | |
| *** bmace has joined #openstack-infra | 21:33 | |
| *** heyongli has joined #openstack-infra | 21:35 | |
| anteaya | pleia2: ah just reading the email you sent out about the renames | 21:35 |
| *** dims has joined #openstack-infra | 21:35 | |
| greghaynes | clarkb: so I think I fugured out the race | 21:35 |
| anteaya | pleia2: the compass ones | 21:35 |
| *** bhunter71 has quit IRC | 21:35 | |
| *** dims has quit IRC | 21:35 | |
| *** Guest7393 has joined #openstack-infra | 21:35 | |
| anteaya | pleia2: compass was discussed at the tc meeting today and I belief the agreement was they have some things to work on yet | 21:35 |
| greghaynes | clarkb: I wasnt naming the nodepool builder thread, so it was getting a default name of Thread- which is in out wait_for_thread whitelist | 21:36 |
| pleia2 | anteaya: I just copied from https://wiki.openstack.org/wiki/Meetings/InfraTeamMeeting#Upcoming_Project_Renames | 21:36 |
| anteaya | s/belief/believe | 21:36 |
| anteaya | pleia2: yup I understand | 21:36 |
| pleia2 | anteaya: so we can update that if needed :) | 21:36 |
| anteaya | I'm thinking that entry bears some further scrutiny | 21:36 |
| krotscheck | I remember conversations about migrating to a different IRC bot. Which one did we want to migrate to? | 21:36 |
| pleia2 | the list is in flux, so we'll just want to make sure it's up to date on Thursday when I do prep | 21:36 |
| jeblair | krotscheck: undecided, potentially errbot, but needs some more thought and a plan | 21:37 |
| pleia2 | krotscheck: I think we were looking at eir | 21:37 |
| *** mriedem is now known as mriedem_away | 21:37 | |
| pleia2 | well, something with an e and an r anyway! | 21:38 |
| krotscheck | jeblair, pleia2: Thanks. Was going to put together an ansible script to create an info bot for a private team channel, because OMG organizational knowledge. Wanted to make sure whatever I ended up using was the same thing in use in infra. | 21:38 |
| krotscheck | Hrm. errbot is GPLv3 :/ | 21:38 |
| *** heyongli has quit IRC | 21:39 | |
| mordred | krotscheck: yup | 21:39 |
| fungi | nothing complex about running a gplv3 tool unless you need to modify its to get it to do what you want it to do | 21:39 |
| mordred | fungi: nothing complex about that either | 21:40 |
| krotscheck | fungi: Yeah, but if we want to write a plugin, that's considered derivative according to the apache foundation. | 21:40 |
| mordred | fungi: the only thing that's complex about it is if you want to ship a proprietary product based on that code | 21:40 |
| fungi | mordred: other than if you need your employer to okay you contributing changes to a gplv3 project on work time | 21:40 |
| krotscheck | the GPLv3 people consider even linking to it a derivative work. | 21:40 |
| jeblair | krotscheck: when you say "private team channel", i'm assuming your talking about something extra-curricular to openstack, yeah? | 21:40 |
| mordred | krotscheck: the openstack foundation's legal council has ok'd the infra project having and producing GPLv3 code | 21:40 |
| jeblair | and yes, if we were to use it, i would expect that if we wrote a plugin, it would be gplv3 for that reason | 21:41 |
| krotscheck | mordred: Oh good. | 21:41 |
| *** _amrith_ is now known as amrith | 21:41 | |
| *** amitgandhinz has quit IRC | 21:41 | |
| mordred | krotscheck: so - yeah - I think errbot is our best theory so far - but we do not yet have concrete plans | 21:41 |
| fungi | yep, i'm not _personally_ concerned about working on gplv3 projects. just noting that some people may not be allowed by their employers to contribute as part of their jobs. though the company i'm thinking of has at least a couple of people here that likely would mention if that were really a problem ;) | 21:42 |
| Clint | that is also true of Apache-2.0 | 21:42 |
| fungi | indeed it is | 21:43 |
| mordred | fungi: are any of those companies companies that I work for? | 21:43 |
| *** heyongli has joined #openstack-infra | 21:43 | |
| fungi | mordred: possibly, yes | 21:43 |
| mordred | fungi: well, theoretically, it's not a problem | 21:43 |
| mordred | hypothetically of course | 21:43 |
| *** rlandy has joined #openstack-infra | 21:44 | |
| fungi | though i do think we would need to _not_ turn on cla enforcement for any gplv3 project we're directly maintaining in our gerrit, because that would just be silly | 21:44 |
| mordred | fungi: funny story ... | 21:44 |
| Clint | pretty sure the company i work for distributes gplv3 code | 21:44 |
| fungi | dco would likely be fine. our icla is pretty apache-specific though | 21:44 |
| * fungi makes ianal handsign to ward off evil eye | 21:45 | |
| mordred | fungi: I believe the terms of the apache license are such that code contributed under it could be added to a gplv3 final work | 21:45 |
| jeblair | fungi: when i dig deep -- really really deep -- for a reason for openstack to have a cla, the _only_ thing i can come up with is "so that the foundation can, in the future, decide to release openstack under gplv3". | 21:45 |
| openstackgerrit | sebastian marcet proposed openstack-infra/openstackid: [smarcet] https://review.openstack.org/198458 | 21:46 |
| jeblair | (i mean, after it changes its bylaws) | 21:46 |
| Clint | it could do that without a cla, no? | 21:46 |
| mordred | fungi: I'm not saying I think this is necessary, of course, but I dont' think having it on breaks anything more than it breaks other things | 21:46 |
| mordred | Clint: it can do literally everything without a CLA that it thinks it needs a CLA to do | 21:46 |
| fungi | dual-license apl2/gpl3 would be an interesting combo | 21:46 |
| jeblair | mordred: er, it could not relicense code written by others | 21:46 |
| *** fallenpegasus has joined #openstack-infra | 21:47 | |
| mordred | jeblair: no need - it could release a gplv3 licensed derivative work | 21:47 |
| openstackgerrit | sebastian marcet proposed openstack-infra/openstackid: OIDC - OpenId Connect Implementation https://review.openstack.org/198458 | 21:47 |
| jeblair | (but of course, apache is liberal so that's pretty eassy to do in effect anyway) | 21:47 |
| mordred | yah. that's my point | 21:47 |
| jeblair | mordred: agreed, the outcome would be effectively the same | 21:47 |
| mordred | now - the foundation being allowed to make that decision would involved a fascnatingly large amount of bylaws changes I believe | 21:48 |
| krotscheck | For those of us not on the private channel where I just answered jeblair's question: It's a channel of HP people who do a lot of infra work, and we need infobot things for all the crazy links and forms and hoops. I'd rather just use the thing we're likely going to use in public. | 21:48 |
| mordred | there are SO MANY CRAZY LINKS AND FORMS AND HOOPS | 21:48 |
| fungi | sounds like lots of fun i don't want to know about ;) | 21:48 |
| mordred | wait. no there aren't... you shold all come work for us | 21:48 |
| *** sdake_ has joined #openstack-infra | 21:48 | |
| zaro | mgagne: sorry i was afk, did you figure out a workaround? | 21:48 |
| jeblair | krotscheck: anyway, back to your question; we currently run supybot, so investment in using a supybot infobot thingy might not be a waste (in that we'd be converting eventually anyway) | 21:49 |
| mgagne | zaro: yea, just delete the used node at night instead of every minutes and let jobs reuse them (which I'm fine with) | 21:49 |
| krotscheck | jeblair: Aight. I can do that. | 21:49 |
| EmilienM | fungi, mordred: can I get a final +A on https://review.openstack.org/#/c/199291/ please ? | 21:50 |
| jeblair | krotscheck: i'm not actually sure which would be easier/better. hth. | 21:50 |
| mordred | jeblair: based on a thing Clint said - I want to start geotagging all of my git commits ... | 21:51 |
| krotscheck | jeblair: ¯\_(ツ)_/¯ | 21:51 |
| *** heyongli has quit IRC | 21:51 | |
| *** wshao has quit IRC | 21:51 | |
| jeblair | mordred: and add that to stackalytics? | 21:51 |
| jeblair | krotscheck: pretty much :) | 21:51 |
| mordred | yup | 21:51 |
| *** Longgeek has joined #openstack-infra | 21:51 | |
| mordred | jeblair: preferrably with some time data to show paths of patches for a person through a cycle | 21:51 |
| *** sdake has quit IRC | 21:52 | |
| *** tonytan4ever has quit IRC | 21:52 | |
| *** nilasae has quit IRC | 21:53 | |
| *** whoops has quit IRC | 21:54 | |
| openstackgerrit | Merged openstack-infra/release-tools: Close milestones for post-version releases https://review.openstack.org/195130 | 21:54 |
| mordred | jeblair: ZOMG. THIS IS ACTUALLY A THING!!!!! | 21:54 |
| mordred | jeblair: pip install geocommit | 21:54 |
| openstackgerrit | Merged openstack-infra/release-tools: Handle more errors when updating bugs https://review.openstack.org/195242 | 21:54 |
| *** pc_m has quit IRC | 21:55 | |
| *** heyongli has joined #openstack-infra | 21:55 | |
| openstackgerrit | Merged openstack-infra/release-tools: Catch errors checking for zuul status https://review.openstack.org/195319 | 21:56 |
| *** tonytan4ever has joined #openstack-infra | 21:56 | |
| fungi | neat. it just stores the location data in git notes | 21:56 |
| openstackgerrit | greghaynes proposed openstack-infra/nodepool: Trigger image builds using gearman https://review.openstack.org/195175 | 21:57 |
| *** annegentle has joined #openstack-infra | 21:57 | |
| *** Longgeek has quit IRC | 21:57 | |
| greghaynes | how does that work with patchsets though - do you just get the last location you worked added? | 21:57 |
| greghaynes | really you want a list of locations | 21:57 |
| mordred | greghaynes: that's an excellent point | 21:58 |
| greghaynes | so maybe git-review should support geocommit | 21:59 |
| fungi | "all the pubs^H^H^H^Hplaces where i worked on this commit" | 21:59 |
| greghaynes | oh yes! we can also add in a BAC feature | 21:59 |
| mordred | greghaynes: clearly this is what we shoudl all drop everything to work on | 21:59 |
| *** changbl has quit IRC | 21:59 | |
| *** heyongli has quit IRC | 21:59 | |
| greghaynes | mordred: I thought thats why we have priority efforts - for when stuff like this comes along | 22:00 |
| mordred | greghaynes: there is also a thing called "doko" which is a command line python thing for getting you your GPS coordinates | 22:01 |
| openstackgerrit | Merged openstack-infra/project-config: Move "bundle install" for puppet-modules-jobs. https://review.openstack.org/199291 | 22:01 |
| openstackgerrit | Merged openstack-infra/release-tools: Fix --stable flag https://review.openstack.org/196712 | 22:01 |
| *** heyongli has joined #openstack-infra | 22:01 | |
| greghaynes | mordred: heh, I may be a nerd about that and have a beaglebone hooked up to a SPI gps that parses that out using pynmea | 22:02 |
| mordred | greghaynes: I may want one of those | 22:03 |
| *** bmace has left #openstack-infra | 22:03 | |
| greghaynes | hrm, I bet you could find a serial gps, then just usb -> serial and do it on your laptop | 22:03 |
| fungi | or bt it from your phone | 22:04 |
| greghaynes | oh right, we do all have gps things on us nowdays dont we ;) | 22:04 |
| fungi | i see several smartphone apps for different platforms that let you query your gps data over bluetooth from a computer | 22:05 |
| *** wshao has joined #openstack-infra | 22:05 | |
| fungi | no idea whether there are free software client implementations or if you'd need to reverse-engineer a hackish and unversioned high-level protocol | 22:05 |
| *** heyongli has quit IRC | 22:06 | |
| openstackgerrit | Spencer Krum proposed openstack-infra/system-config: Test the apply test https://review.openstack.org/199321 | 22:06 |
| *** heyongli has joined #openstack-infra | 22:07 | |
| *** thorst has joined #openstack-infra | 22:07 | |
| *** dimtruck is now known as zz_dimtruck | 22:09 | |
| *** unicell has quit IRC | 22:10 | |
| *** unicell has joined #openstack-infra | 22:10 | |
| *** heyongli has quit IRC | 22:12 | |
| *** heyongli has joined #openstack-infra | 22:13 | |
| *** ari has quit IRC | 22:13 | |
| *** esmute__ has quit IRC | 22:15 | |
| mordred | nibalizer: best commit message eer | 22:16 |
| *** esmute__ has joined #openstack-infra | 22:16 | |
| *** prad has quit IRC | 22:16 | |
| nibalizer | heh | 22:16 |
| nibalizer | I want to make sure it fails when it should be failing, pabelanger has me all worried | 22:17 |
| nibalizer | looks like it does | 22:17 |
| mordred | nibalizer: he'll do that | 22:17 |
| mordred | nibalizer: wily canadians | 22:17 |
| *** thorst has quit IRC | 22:18 | |
| mordred | lifeless: you think we do not need the oscc patch? | 22:19 |
| *** austin81 has quit IRC | 22:19 | |
| *** heyongli has quit IRC | 22:19 | |
| *** esmute__ has quit IRC | 22:19 | |
| lifeless | mordred: I think we do not need it. Its not solving a problem that I can see, and it creates inconsistency and surprises for developers. | 22:20 |
| *** wshao has quit IRC | 22:20 | |
| *** esmute__ has joined #openstack-infra | 22:20 | |
| lifeless | as well as likely throwing crap into stdout during the test suite since I very much doubt it was developed entirely cleanly so far [in that you've had the belts-and-braces, which stops you seeing cruft] | 22:20 |
| mordred | lifeless: you are correct on this | 22:21 |
| mordred | lifeless: I'm happy to abandon the change in oscc ... I just mainly want to make sure the oscc-based fixture is in oslotest so that everyone can use it in their functional tests | 22:21 |
| *** heyongli has joined #openstack-infra | 22:22 | |
| *** fawadkhaliq has joined #openstack-infra | 22:22 | |
| lifeless | so a fixture that uses oscc makes sense to me | 22:22 |
| lifeless | is it in a dedicated file? | 22:22 |
| lifeless | I'd make the dependency from oscc optional I think | 22:22 |
| dhellmann | if it's a fixture, why not just put it in oscc? | 22:22 |
| lifeless | e.g. in setup.cfg | 22:22 |
| lifeless | [extras] | 22:23 |
| lifeless | os-client-config = | 22:23 |
| dhellmann | we're doing similar things in the other oslo libs, for similar reasons | 22:23 |
| lifeless | os-client-config | 22:23 |
| lifeless | or as dhellmann says just have the fixture inside os-client-config | 22:23 |
| lifeless | folk that need it have to know that they're using os-client-config one way or another | 22:23 |
| dhellmann | right | 22:23 |
| *** fallenpegasus has quit IRC | 22:24 | |
| mordred | no - they don't | 22:24 |
| mordred | that's the entire point | 22:24 |
| *** linkedinyou has quit IRC | 22:24 | |
| mordred | is that they do NOT need to know that they're using oscc, and are, in fact, unlikely to know much about it per-se | 22:25 |
| mordred | it's used in a general setup fixture for getting cloud credentials in a devstack-based functional test | 22:25 |
| *** fallenpegasus has joined #openstack-infra | 22:26 | |
| mordred | what it winds up returning is a python dictionary with expected auth info in it | 22:27 |
| lifeless | ok | 22:27 |
| *** austin81 has joined #openstack-infra | 22:27 | |
| dhellmann | ok. Why don't we want them to know where that comes from? Just to make it easier? | 22:27 |
| *** puranamr has quit IRC | 22:28 | |
| mordred | well, I mean, they could - but they already depend on oslotest, so it's an easy way to just get them the functionality now that devstack exports it | 22:28 |
| *** linkedinyou has joined #openstack-infra | 22:28 | |
| *** armax has joined #openstack-infra | 22:28 | |
| mordred | I mean, I could also just go patch-bomb all the projects ... my patch count _is_ getting low :) | 22:28 |
| *** annegentle has quit IRC | 22:29 | |
| *** puranamr has joined #openstack-infra | 22:29 | |
| dhellmann | I'm just trying to make sure I understand. We get a lot of "but there are so many extra dependencies" about some of these libs that folks start not wanting to use them. If this is a thing they need, then making that dependency explicit elsewhere may be educational. | 22:29 |
| mordred | dhellmann: I can see taht | 22:30 |
| dhellmann | *I* want oslotest to have stuff like this, I'm just not sure its users do. | 22:30 |
| dhellmann | also, with this circular dependency issue... | 22:30 |
| lifeless | well | 22:30 |
| *** heyongli has quit IRC | 22:30 | |
| lifeless | its not a circular dependency | 22:30 |
| dhellmann | although I guess it's a test dep, so -- yeah | 22:30 |
| lifeless | neither setup_requires the other | 22:31 |
| *** rlandy has quit IRC | 22:31 | |
| lifeless | so you can build them both sanely to binary form. | 22:31 |
| *** bknudson has quit IRC | 22:31 | |
| dhellmann | oslotest does require oscc, no? | 22:31 |
| lifeless | You can run the tests for the other with the prior version | 22:31 |
| dhellmann | oh, I see | 22:31 |
| dhellmann | install_requires | 22:31 |
| lifeless | and once you've done that you can do the other, and then install them both | 22:31 |
| lifeless | no races | 22:31 |
| * dhellmann doesn't know how those things apply to the way the debs are being packaged | 22:31 | |
| lifeless | no binary-blob-from-deity-to-get-started | 22:32 |
| lifeless | dhellmann: Debian is right now evolving its tooling to better support such nuance. | 22:32 |
| lifeless | dhellmann: but since they're all already in the archive,there should be no drama | 22:32 |
| * dhellmann hates it when we all spend so much time on things that shouldn't be an issue | 22:33 | |
| *** dizquierdo has quit IRC | 22:33 | |
| dhellmann | so should we just leave everything alone? because even making oscc an optional dependency of oslotest means we have to tell users they need it to use some of the oslotest features, which I thought was what mordred wanted to avoid | 22:33 |
| *** esmute__ has quit IRC | 22:33 | |
| openstackgerrit | Khai Do proposed openstack-infra/puppet-openstackci: Add Jenkins Job Builder manifest https://review.openstack.org/184919 | 22:34 |
| *** esmute__ has joined #openstack-infra | 22:34 | |
| *** Guest15874 has joined #openstack-infra | 22:34 | |
| lifeless | dhellmann: I think we should just leave well enough alone | 22:35 |
| lifeless | dhellmann: until we've really understood zigo's issue | 22:35 |
| dhellmann | lifeless: ++ | 22:35 |
| lifeless | if its what I think it is, I think its a non-issue | 22:35 |
| lifeless | if its something else | 22:35 |
| zigo | What issue? | 22:35 |
| lifeless | lets work through it and not knee-jerk | 22:36 |
| lifeless | zigo: see the last 40 or so lines of backscroll | 22:36 |
| * dhellmann heads off to dinner | 22:36 | |
| zigo | Oh, about the build-dependency loop? | 22:36 |
| *** gordc has quit IRC | 22:36 | |
| zigo | I think it's not too bad, as removing os-client-config from the build-depends is ok. | 22:36 |
| zigo | Is it really used? | 22:37 |
| zigo | Either it's not, or the unit tests aren't using it at least. | 22:37 |
| zigo | But really, if it was in use in the unit tests, it would be. | 22:39 |
| zigo | I just hope it never will. | 22:40 |
| zigo | Which is why I sent a mail to the list. | 22:40 |
| *** esmute__ has quit IRC | 22:42 | |
| lifeless | zigo: I don't understand why its a build-dependency loop | 22:42 |
| lifeless | zigo: its not upstream | 22:42 |
| openstackgerrit | Khai Do proposed openstack-infra/system-config: Use third party ci manifest to setup jenkins job builder. https://review.openstack.org/199335 | 22:42 |
| zigo | lifeless: oslotest has os-client-config in requirements.txt, so I've put it in Build-Depends-Indep. But in fact, it doesn't use it during tests, it seems, so I finally removed it. Then os-client-config uses oslotest during its unit tests... | 22:45 |
| *** Sukhdev has joined #openstack-infra | 22:45 | |
| zigo | So, it's a non-issue if oslotest *never will* use os-client-config during its unit tests. | 22:45 |
| *** chlong has quit IRC | 22:46 | |
| lifeless | zigo: requirements.txt is for install requirements, not build requirements. | 22:46 |
| lifeless | zigo: all openstack packages have one and only one build requirement: pbr. | 22:47 |
| *** sabeen3 has quit IRC | 22:47 | |
| lifeless | (unless you have a different definition of build than I'm expecting) | 22:47 |
| zigo | lifeless: In most cases, all stuff in requirements.txt are also build dependencies required to run unit tests. | 22:47 |
| lifeless | you can run unit tests with the prior version of runtime dependencies | 22:47 |
| zigo | I do run unit tests on build, always... | 22:47 |
| lifeless | we know this because we sequence all our commits in one at a time | 22:47 |
| *** tonytan4ever has quit IRC | 22:48 | |
| *** _nadya_ has joined #openstack-infra | 22:50 | |
| openstackgerrit | greghaynes proposed openstack-infra/nodepool: Trigger image uploads using gearman https://review.openstack.org/197722 | 22:51 |
| asselin | zaro, thanks for the updates....will take a look | 22:52 |
| *** esmute__ has joined #openstack-infra | 22:54 | |
| *** _nadya_ has quit IRC | 22:54 | |
| *** Rockyg has quit IRC | 22:57 | |
| hogepodge | jeblair: fungi: fits-list is nothing but a big spam dump now, and is replaced by defcore-committee. What can I do to help shut that list down? | 22:57 |
| *** esmute__ has quit IRC | 22:57 | |
| *** r-daneel has quit IRC | 23:00 | |
| *** fawadkhaliq has quit IRC | 23:02 | |
| *** Guest15874 has left #openstack-infra | 23:02 | |
| fungi | hogepodge: i don't think we've (yet) closed down a mailing list on our mailman instance. it's been a while since i've done that. i suspect the options there (you seem to be an admin for it) are to remove all the subscribers and adjust the moderation mode so that all new posts will be rejected with a message saying the list is retired (or just black-hole them) | 23:03 |
| *** esmute__ has joined #openstack-infra | 23:04 | |
| *** soren has quit IRC | 23:04 | |
| pleia2 | we shut down and deleted the non-announce third party one | 23:04 |
| jeblair | hogepodge, fungi: yeah, i think either something like that, or we could see about adding something to puppet to tell exim to ignore some lists (and therefore, msgs addressed there would be rejected) | 23:04 |
| mordred | ++ | 23:04 |
| mtreinish | fungi: we were going to close down the openstack-qa list, but until the fabled dashboard exists we still need it for periodic job notifications | 23:05 |
| jeblair | pleia2: yeah, i think there were mitigating circumstances for that one that led us to think we could delete it (such as, no substantial messages were sent to it) | 23:05 |
| pleia2 | yeah, we didn't need to keep anything, it was just deleted | 23:05 |
| *** esmute__ has quit IRC | 23:06 | |
| *** heyongli has joined #openstack-infra | 23:08 | |
| lifeless | fungi: so | 23:09 |
| *** puranamr has quit IRC | 23:09 | |
| *** ashleighfarnham has quit IRC | 23:10 | |
| lifeless | fungi: http://logs.openstack.org/periodic/propose-requirements-constraints-master/50fc7d4/console.html#_2015-07-07_06_05_57_926 | 23:10 |
| fungi | lifeless: i think we're hoping for additional reviews on https://review.openstack.org/198926 | 23:10 |
| *** esmute__ has joined #openstack-infra | 23:11 | |
| *** sridhar_ram has joined #openstack-infra | 23:11 | |
| hogepodge | fungi: jeblair: pleia2: I'd be happy with just removing the subscribers and rejecting all new posts. I can set it up, and set a message out saying the list is about to be shut down. Then after some point I can ping you again to delete it | 23:11 |
| *** thedodd has quit IRC | 23:11 | |
| lifeless | there's a -1 there already | 23:12 |
| lifeless | mordred: https://review.openstack.org/#/c/198926/ <- can you please either +2A or -1 ? | 23:12 |
| *** annegentle has joined #openstack-infra | 23:12 | |
| *** heyongli has quit IRC | 23:13 | |
| *** esmute__ has quit IRC | 23:13 | |
| *** wenlock has quit IRC | 23:13 | |
| fungi | jhesketh: or if you're up to revisit your current -1 | 23:13 |
| *** esmute__ has joined #openstack-infra | 23:14 | |
| fungi | unclear whether you -1'd for the lack of a cleanup patch (which i've tested and should not need to be set as a dependent patch from what i can tell) or for the choice of requirements file location, or both | 23:15 |
| jhesketh | fungi: yep, let me look | 23:15 |
| mordred | I +A'd it. I think the cleanups are thingks we can do with a followup | 23:15 |
| fungi | i actually think that adding the cleanup patch after will be safer anyway | 23:15 |
| fungi | because this lets us be absolutely certain it's working before we potentially rip it off our nodepool images and wind up with a fire drill scramble to delete new images and recheck a bunch of failed changes | 23:16 |
| jhesketh | fungi: ah it was because I wasn't a fan of putting a reqs file in etc. In fact I don't think we need one at all and we could just pip install into the venv | 23:17 |
| *** hichihara has joined #openstack-infra | 23:17 | |
| fungi | once it ends up applied to the proposal/pypi job workers i'll propose the cleanup for our prep scripts/dib elements | 23:17 |
| *** wenlock1 has quit IRC | 23:17 | |
| *** akshai has quit IRC | 23:18 | |
| fungi | jhesketh: perfectly fair, however we already have this patch which should work and it's blocking getting requirements constraints proposed to many, many repos. we can always improve the mechanism we use to install it later | 23:18 |
| openstackgerrit | Sridhar Ramaswamy proposed openstack-infra/project-config: Add tacker-horizon to gerritt IRC notifications https://review.openstack.org/199341 | 23:18 |
| jhesketh | Sure, it's certainly pretty harmless as is so I'm fine with that :-) | 23:18 |
| fungi | that will allow us to take our time on a more aesthetically pleasing modification to use a pip exec rather than a requirements file | 23:19 |
| *** bswartz has joined #openstack-infra | 23:19 | |
| *** nelsnelson has quit IRC | 23:20 | |
| openstackgerrit | Merged openstack-infra/system-config: Install zuul-env venv with puppet https://review.openstack.org/198926 | 23:20 |
| jhesketh | Sounds good to me | 23:20 |
| *** bhunter71 has joined #openstack-infra | 23:22 | |
| *** linkedinyou has quit IRC | 23:23 | |
| *** esmute__ has quit IRC | 23:23 | |
| *** esmute__ has joined #openstack-infra | 23:23 | |
| *** yamada-h has joined #openstack-infra | 23:26 | |
| *** gildub has joined #openstack-infra | 23:26 | |
| *** salv-orlando has quit IRC | 23:26 | |
| *** krtaylor has quit IRC | 23:29 | |
| *** heyongli has joined #openstack-infra | 23:30 | |
| lifeless | nuts | 23:34 |
| lifeless | http://logs.openstack.org/28/184328/5/check/gate-oslo.db-requirements/55e3703/console.html#_2015-07-07_22_41_43_227 | 23:34 |
| *** annegentle has quit IRC | 23:34 | |
| lifeless | I need to iterate again the project-config script there | 23:34 |
| *** heyongli has quit IRC | 23:34 | |
| lifeless | I wish more of it was in the requirement repo and unit tested | 23:34 |
| *** austin81 has left #openstack-infra | 23:35 | |
| *** heyongli has joined #openstack-infra | 23:37 | |
| *** krtaylor has joined #openstack-infra | 23:41 | |
| *** Longgeek has joined #openstack-infra | 23:44 | |
| *** heyongli has quit IRC | 23:44 | |
| *** heyongli has joined #openstack-infra | 23:45 | |
| *** yamada-h has quit IRC | 23:46 | |
| mordred | lifeless: ++ | 23:48 |
| *** Longgeek has quit IRC | 23:49 | |
| *** heyongli has quit IRC | 23:50 | |
| *** erlon has quit IRC | 23:50 | |
| *** jasondotstar has joined #openstack-infra | 23:51 | |
| *** gyee has quit IRC | 23:51 | |
| *** heyongli has joined #openstack-infra | 23:52 | |
| greghaynes | Ng: In hp2 theres some nodes in the machine-information doc that say youre reserving them, im guessing thats a thing of the past and no longer relavent? | 23:52 |
| *** baoli has quit IRC | 23:53 | |
| *** heyongli has quit IRC | 23:56 | |
| *** camunoz__ has joined #openstack-infra | 23:57 | |
| *** camunoz__ is now known as camunoz_mtg | 23:57 | |
| *** laughterwym has joined #openstack-infra | 23:58 | |
| mordred | greghaynes: I hear he's making a nice lemon cream sauce with them | 23:58 |
| *** dalgaaf has quit IRC | 23:58 | |
| greghaynes | oh, I should definitely leave them be | 23:59 |
| *** camunoz has quit IRC | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!