| praneshp_ | jeblair: pinky swear :) | 00:00 |
|---|---|---|
| clarkb | praneshp_: do you pinky swear? and can you confirm the name of the tag you want removed? | 00:00 |
| *** igor_ has joined #openstack-infra | 00:00 | |
| praneshp_ | clarkb: stackforge/entropy, tag name is 5.19 | 00:00 |
| praneshp_ | its the only tag | 00:00 |
| praneshp_ | morganfainberg: sure! | 00:00 |
| bknudson | is there some reason gerrit seems to become unresponsive at this time every day? | 00:01 |
| clarkb | praneshp_: done | 00:01 |
| clarkb | praneshp_: it may take a few minutes to propogate to the mirrors and github | 00:01 |
| praneshp_ | clarkb: thanks a lot | 00:01 |
| praneshp_ | i already see it has vanished from github | 00:02 |
| praneshp_ | sorry for the trouble | 00:02 |
| clarkb | bknudson: it is responsive to me, we do have a scheduled DB backup though | 00:02 |
| morganfainberg | bknudson, i'm not having issues on this end. | 00:02 |
| * clarkb checks time for DB backup | 00:02 | |
| bknudson | ok, I always seem to be making a comment and gerrit gets stuck with "working"... then I check the time and it's the same every day. | 00:03 |
| bknudson | only lasts for a minute or so so not a big deal | 00:03 |
| clarkb | yup backups happen at 0000 UTC daily | 00:04 |
| *** igor_ has quit IRC | 00:04 | |
| clarkb | we can shift that to be more middle of the night PST which is the deadest time for us | 00:04 |
| bknudson | clarkb: thanks! at least I have an explanation | 00:04 |
| jeblair | ah yeah, that might be a mysql lock | 00:04 |
| clarkb | jeblair: http://cacti.openstack.org/cacti/graph.php?action=view&local_graph_id=34&rra_id=all looks funny | 00:05 |
| jeblair | yeah, it pushed a lot of data today | 00:05 |
| clarkb | wondering if something is hitting our APIs quite a bit more | 00:06 |
| jeblair | clarkb: outbound eth0 matchen inbound eth1 so it seems like it was moving a lot of data from the db to the Internet | 00:07 |
| clarkb | ah yup | 00:07 |
| clarkb | it would be funny if that is all gertty :P | 00:08 |
| *** bhuvan has quit IRC | 00:09 | |
| *** SumitNaiksatam has joined #openstack-infra | 00:10 | |
| jerryz | clarkb: i find that gerrit hook is not updating bugs i just reported | 00:14 |
| *** Sukhdev has joined #openstack-infra | 00:16 | |
| *** sarob has quit IRC | 00:17 | |
| *** sarob has joined #openstack-infra | 00:17 | |
| openstackgerrit | Nachi Ueno proposed a change to openstack-infra/reviewstats: Add Review Stat WebUI https://review.openstack.org/94288 | 00:18 |
| *** fbo has quit IRC | 00:18 | |
| *** dstanek_zzz is now known as dstanek | 00:18 | |
| jeblair | clarkb: i wonder if the zuul loop was part of it | 00:21 |
| *** sarob has quit IRC | 00:22 | |
| *** paul--_ has quit IRC | 00:22 | |
| *** msabramo has quit IRC | 00:23 | |
| *** _nadya_ has quit IRC | 00:25 | |
| boris-42 | jeblair ping | 00:27 |
| *** fbo has joined #openstack-infra | 00:28 | |
| *** Sukhdev has quit IRC | 00:30 | |
| *** maxbit has joined #openstack-infra | 00:34 | |
| openstackgerrit | Nachi Ueno proposed a change to openstack-infra/reviewstats: Add Review Stat WebUI https://review.openstack.org/94288 | 00:38 |
| *** maxbit has quit IRC | 00:38 | |
| *** maxbit has joined #openstack-infra | 00:38 | |
| *** melwitt has joined #openstack-infra | 00:42 | |
| *** dstanek is now known as dstanek_zzz | 00:44 | |
| *** crc32 has quit IRC | 00:46 | |
| *** hipster has joined #openstack-infra | 00:47 | |
| *** maxbit has quit IRC | 00:48 | |
| *** maxbit has joined #openstack-infra | 00:48 | |
| *** msabramo has joined #openstack-infra | 00:51 | |
| *** gilliard_ has joined #openstack-infra | 00:51 | |
| *** hipster has quit IRC | 00:51 | |
| *** _nadya_ has joined #openstack-infra | 00:52 | |
| *** hogepodge has quit IRC | 00:54 | |
| *** briancurtin has joined #openstack-infra | 00:58 | |
| *** igor_ has joined #openstack-infra | 01:00 | |
| *** gilliard_ has quit IRC | 01:01 | |
| *** msabramo has quit IRC | 01:02 | |
| *** UtahDave has quit IRC | 01:04 | |
| *** saschpe has quit IRC | 01:04 | |
| *** igor_ has quit IRC | 01:05 | |
| *** dstanek_zzz is now known as dstanek | 01:05 | |
| openstackgerrit | Nachi Ueno proposed a change to openstack-infra/reviewstats: Add Review Stat WebUI https://review.openstack.org/94288 | 01:05 |
| *** maxbit has quit IRC | 01:05 | |
| boris-42 | mordred ping | 01:05 |
| *** saschpe has joined #openstack-infra | 01:06 | |
| *** msabramo has joined #openstack-infra | 01:07 | |
| *** krotscheck has quit IRC | 01:12 | |
| *** paul--_ has joined #openstack-infra | 01:14 | |
| *** mriedem has joined #openstack-infra | 01:15 | |
| *** dstanek is now known as dstanek_zzz | 01:15 | |
| *** krotscheck has joined #openstack-infra | 01:15 | |
| jogo | so far both keynote speakers here at linuxcon japan mentioned openstack | 01:20 |
| openstackgerrit | A change was merged to openstack-infra/elastic-recheck: Delete fingerprint for 1224001 https://review.openstack.org/94276 | 01:23 |
| *** msabramo has quit IRC | 01:24 | |
| *** gus_ has joined #openstack-infra | 01:24 | |
| *** msabramo has joined #openstack-infra | 01:25 | |
| *** marcoemorais has quit IRC | 01:26 | |
| *** nati_ueno has quit IRC | 01:27 | |
| SlickNik | jogo: the OpenDaylight keynote and the OpenCloud panel I take it? | 01:28 |
| jogo | well the guy doing the intro and the opendaylight | 01:28 |
| jogo | now apache | 01:28 |
| jogo | wondering if they will hit it too ;) | 01:28 |
| *** maxbit has joined #openstack-infra | 01:29 | |
| *** mattoliverau has quit IRC | 01:30 | |
| *** mattoliverau has joined #openstack-infra | 01:33 | |
| jogo | jeblair: https://github.com/jcrocholl/pep8/issues/236 ohh pep8 -- it is now taking our rules | 01:35 |
| *** dstanek_zzz is now known as dstanek | 01:37 | |
| jogo | Alex_Gaynor: https://github.com/jcrocholl/pep8/issues/236 | 01:39 |
| *** aconrad has quit IRC | 01:42 | |
| Alex_Gaynor | jogo: what'd I do? | 01:42 |
| harlowja | Alex_Gaynor thx for the help on doc8 :-P | 01:42 |
| Alex_Gaynor | harlowja: no problem! Thanks for working on it, we've already started using it in cryptography | 01:43 |
| *** maxbit has quit IRC | 01:43 | |
| harlowja | Alex_Gaynor https://review.openstack.org/#/c/94236/ also | 01:43 |
| harlowja | might as well just move it off my own repo, lol | 01:43 |
| harlowja | i also refactored alot of it last night, lol | 01:43 |
| harlowja | much better structure now @ https://github.com/harlowja/doc8/tree/master/doc8 | 01:43 |
| harlowja | *simple but good enough, haha | 01:44 |
| jogo | Alex_Gaynor: you got pep8 the project to enforce a hacking rule that isn't in pep8 itself | 01:44 |
| *** lcheng has joined #openstack-infra | 01:44 | |
| jogo | Alex_Gaynor: which is very confusing IMHO | 01:45 |
| *** nosnos has joined #openstack-infra | 01:47 | |
| *** hipster has joined #openstack-infra | 01:47 | |
| *** _nadya_ has quit IRC | 01:50 | |
| *** hipster has quit IRC | 01:51 | |
| *** malini is now known as malini_afk | 01:53 | |
| *** dkehnx has left #openstack-infra | 01:54 | |
| *** igor_ has joined #openstack-infra | 02:01 | |
| *** saschpe has quit IRC | 02:01 | |
| *** aconrad has joined #openstack-infra | 02:03 | |
| *** saschpe has joined #openstack-infra | 02:03 | |
| *** igor_ has quit IRC | 02:05 | |
| *** marun has quit IRC | 02:09 | |
| *** yjiang has joined #openstack-infra | 02:10 | |
| *** dstanek is now known as dstanek_zzz | 02:10 | |
| *** lcheng has quit IRC | 02:11 | |
| *** _nadya_ has joined #openstack-infra | 02:17 | |
| *** cody-somerville has joined #openstack-infra | 02:17 | |
| *** aconrad has quit IRC | 02:23 | |
| *** gus_ is now known as gus | 02:26 | |
| *** crc32 has joined #openstack-infra | 02:33 | |
| *** crc32 has quit IRC | 02:33 | |
| *** crc32 has joined #openstack-infra | 02:34 | |
| *** dstanek_zzz is now known as dstanek | 02:35 | |
| *** amcrn has quit IRC | 02:37 | |
| *** hipster has joined #openstack-infra | 02:39 | |
| *** yjiang has quit IRC | 02:43 | |
| *** esker has quit IRC | 02:43 | |
| *** esker has joined #openstack-infra | 02:44 | |
| *** hipster has quit IRC | 02:44 | |
| *** hipster_ has joined #openstack-infra | 02:46 | |
| *** esker has quit IRC | 02:48 | |
| *** smarcet has joined #openstack-infra | 02:50 | |
| *** mriedem has quit IRC | 02:51 | |
| *** marcoemorais has joined #openstack-infra | 02:58 | |
| *** esker has joined #openstack-infra | 03:00 | |
| *** praneshp_ has quit IRC | 03:01 | |
| *** igor_ has joined #openstack-infra | 03:02 | |
| *** lcheng has joined #openstack-infra | 03:03 | |
| *** praneshp_ has joined #openstack-infra | 03:04 | |
| *** praneshp_ has quit IRC | 03:05 | |
| *** melwitt has quit IRC | 03:05 | |
| *** nosnos has quit IRC | 03:07 | |
| *** igor_ has quit IRC | 03:07 | |
| *** marcoemorais has quit IRC | 03:14 | |
| *** praneshp_ has joined #openstack-infra | 03:15 | |
| praneshp_ | clarkb jeblair: ping | 03:15 |
| praneshp_ | I;m trying to help someone sign the ICLA | 03:15 |
| clarkb | ok | 03:16 |
| praneshp_ | and keep seeing an error saying cannot store contact derails | 03:16 |
| praneshp_ | is there a known workaround about this | 03:16 |
| clarkb | make sure they have joined the foundation and the email address there matches the email address in gerrit | 03:17 |
| praneshp_ | clarkb: yeah, doing that | 03:17 |
| praneshp_ | we just found a launchpad bug about this | 03:17 |
| praneshp_ | thanks | 03:17 |
| *** esker has quit IRC | 03:19 | |
| *** esker has joined #openstack-infra | 03:19 | |
| *** praneshp_ has quit IRC | 03:20 | |
| *** marcoemorais has joined #openstack-infra | 03:20 | |
| *** jhesketh has joined #openstack-infra | 03:22 | |
| *** hipster_ has quit IRC | 03:23 | |
| *** arnaud_ has quit IRC | 03:24 | |
| *** arnaud has joined #openstack-infra | 03:25 | |
| *** esker has quit IRC | 03:25 | |
| *** marcoemorais has quit IRC | 03:26 | |
| *** nati_ueno has joined #openstack-infra | 03:29 | |
| *** arnaud has quit IRC | 03:29 | |
| *** hipster has joined #openstack-infra | 03:32 | |
| *** esker has joined #openstack-infra | 03:33 | |
| *** gus has quit IRC | 03:33 | |
| *** msabramo has quit IRC | 03:39 | |
| *** ccit has joined #openstack-infra | 03:40 | |
| *** msabramo has joined #openstack-infra | 03:40 | |
| *** lcheng has quit IRC | 03:40 | |
| *** nosnos has joined #openstack-infra | 03:47 | |
| *** crc32 has quit IRC | 03:48 | |
| *** gus has joined #openstack-infra | 03:48 | |
| *** morganfainberg is now known as morganfainberg_Z | 03:49 | |
| *** vogxn has joined #openstack-infra | 03:52 | |
| *** moted has quit IRC | 03:52 | |
| *** gyee has quit IRC | 03:54 | |
| *** nati_ueno has quit IRC | 03:55 | |
| *** hipster has quit IRC | 03:57 | |
| *** Sukhdev has joined #openstack-infra | 03:59 | |
| *** nosnos has quit IRC | 04:01 | |
| *** gokrokve has quit IRC | 04:03 | |
| *** dstanek is now known as dstanek_zzz | 04:04 | |
| *** nati_ueno has joined #openstack-infra | 04:05 | |
| *** praneshp_ has joined #openstack-infra | 04:05 | |
| *** esker has quit IRC | 04:06 | |
| *** esker has joined #openstack-infra | 04:07 | |
| *** praneshp__ has joined #openstack-infra | 04:10 | |
| *** paul--_ has quit IRC | 04:10 | |
| *** esker has quit IRC | 04:11 | |
| *** praneshp_ has quit IRC | 04:12 | |
| *** Longgeek has joined #openstack-infra | 04:13 | |
| *** gokrokve has joined #openstack-infra | 04:19 | |
| *** paul--_ has joined #openstack-infra | 04:25 | |
| *** dstanek_zzz is now known as dstanek | 04:27 | |
| *** hipster has joined #openstack-infra | 04:27 | |
| *** yfried has quit IRC | 04:28 | |
| *** hipster_ has joined #openstack-infra | 04:29 | |
| *** nosnos has joined #openstack-infra | 04:30 | |
| *** hipster has quit IRC | 04:33 | |
| *** hipster_ has quit IRC | 04:33 | |
| *** odyssey4me has quit IRC | 04:33 | |
| *** mkoderer has joined #openstack-infra | 04:34 | |
| *** aconrad has joined #openstack-infra | 04:43 | |
| *** marcoemorais has joined #openstack-infra | 04:47 | |
| *** paul--_ has quit IRC | 04:48 | |
| *** nati_ueno has quit IRC | 04:49 | |
| *** paul--_ has joined #openstack-infra | 04:49 | |
| *** marcoemorais1 has joined #openstack-infra | 04:49 | |
| *** dstanek is now known as dstanek_zzz | 04:49 | |
| *** ccit has quit IRC | 04:49 | |
| *** nati_ueno has joined #openstack-infra | 04:50 | |
| *** lcheng has joined #openstack-infra | 04:50 | |
| *** marcoemorais has quit IRC | 04:52 | |
| *** Longgeek has quit IRC | 04:55 | |
| *** igor_ has joined #openstack-infra | 05:04 | |
| *** hipster has joined #openstack-infra | 05:07 | |
| *** igor_ has quit IRC | 05:08 | |
| *** yfried has joined #openstack-infra | 05:09 | |
| *** hipster has quit IRC | 05:12 | |
| *** medieval1 has joined #openstack-infra | 05:14 | |
| *** pcrews_ has quit IRC | 05:15 | |
| *** medieval1_ has quit IRC | 05:17 | |
| *** Ryan_Lane has quit IRC | 05:19 | |
| *** e0ne has joined #openstack-infra | 05:19 | |
| *** Ryan_Lane has joined #openstack-infra | 05:21 | |
| *** Ryan_Lane has quit IRC | 05:25 | |
| *** smarcet has quit IRC | 05:30 | |
| *** andreykurilin_ has joined #openstack-infra | 05:32 | |
| *** e0ne has quit IRC | 05:32 | |
| *** jamielennox is now known as jamielennox|away | 05:32 | |
| *** e0ne has joined #openstack-infra | 05:33 | |
| jogo | looks like the gate is down http://jogo.github.io/gate/ | 05:34 |
| jogo | http://logs.openstack.org/34/94034/1/check/check-tempest-dsvm-postgres-full/7f008a2/logs/devstacklog.txt.gz | 05:34 |
| StevenK | jogo: It's strange seeing you in a reasonable timezone | 05:35 |
| * jogo is in Tokyo | 05:36 | |
| jogo | StevenK: haha | 05:36 |
| jogo | ohh so trove doesn't gate on postgres https://review.openstack.org/#/c/92845/4 | 05:40 |
| * jogo files a bug and begins pushing patches | 05:40 | |
| jogo | any infra cores around: fungi clarkb etc | 05:41 |
| *** changbl has quit IRC | 05:45 | |
| *** nati_uen_ has joined #openstack-infra | 05:46 | |
| *** doude has quit IRC | 05:46 | |
| *** hipster has joined #openstack-infra | 05:47 | |
| *** nati_ueno has quit IRC | 05:49 | |
| *** Ryan_Lane has joined #openstack-infra | 05:50 | |
| *** _nadya_ has quit IRC | 05:51 | |
| openstackgerrit | Joe Gordon proposed a change to openstack-infra/config: Gate on Postrgres in trove. https://review.openstack.org/94307 | 05:52 |
| *** hipster has quit IRC | 05:52 | |
| *** nati_uen_ has quit IRC | 05:52 | |
| *** yjiang has joined #openstack-infra | 05:52 | |
| *** nati_ueno has joined #openstack-infra | 05:53 | |
| *** Longgeek has joined #openstack-infra | 05:54 | |
| chmouel | fungi: if you want to +2A this patch https://review.openstack.org/#/c/92988/ now that notmyname has +1 it | 05:54 |
| *** Ryan_Lane has quit IRC | 05:55 | |
| *** _nadya_ has joined #openstack-infra | 05:56 | |
| openstackgerrit | Chmouel Boudjnah proposed a change to openstack-infra/config: Add swift specs repository https://review.openstack.org/94140 | 05:57 |
| *** Sukhdev has quit IRC | 05:59 | |
| *** _nadya_ has quit IRC | 05:59 | |
| praneshp__ | jogo: ha, you have given me good news :) | 05:59 |
| praneshp__ | one of my changes in glance has been failing forever on postgres | 06:00 |
| openstackgerrit | Chmouel Boudjnah proposed a change to openstack-infra/config: Add swift specs repository https://review.openstack.org/94140 | 06:00 |
| jogo | praneshp__: well now we need a trove core to approve my fix | 06:00 |
| jogo | err revert | 06:00 |
| praneshp__ | sorry, I’m not one ;) | 06:00 |
| openstackgerrit | Joe Gordon proposed a change to openstack-infra/elastic-recheck: Add fingerprint for bug 1321093 https://review.openstack.org/94308 | 06:01 |
| uvirtbot | Launchpad bug 1321093 in trove "*-tempest-dsvm-postgres-full failing on trove-manage db_sync" [Undecided,New] https://launchpad.net/bugs/1321093 | 06:01 |
| *** e0ne has quit IRC | 06:03 | |
| *** e0ne has joined #openstack-infra | 06:03 | |
| *** igor has joined #openstack-infra | 06:04 | |
| yfried | guys, I want to ssh to localhost, how can I get user and pass/key during runtime? | 06:08 |
| yfried | ^ during tempest test for debugging | 06:08 |
| *** gokrokve has quit IRC | 06:09 | |
| yfried | fungi: ping | 06:09 |
| *** igor has quit IRC | 06:10 | |
| *** _nadya_ has joined #openstack-infra | 06:12 | |
| *** andreykurilin_ has quit IRC | 06:14 | |
| *** vogxn1 has joined #openstack-infra | 06:21 | |
| *** vogxn has quit IRC | 06:21 | |
| *** vogxn1 has quit IRC | 06:21 | |
| *** vogxn has joined #openstack-infra | 06:21 | |
| *** igor has joined #openstack-infra | 06:24 | |
| *** arnaud_ has joined #openstack-infra | 06:25 | |
| *** e0ne has quit IRC | 06:25 | |
| *** igor has quit IRC | 06:26 | |
| *** harlowja is now known as harlowja_away | 06:28 | |
| *** crc32 has joined #openstack-infra | 06:28 | |
| *** crc32 has quit IRC | 06:29 | |
| *** arnaud_ has quit IRC | 06:29 | |
| *** AJaeger has joined #openstack-infra | 06:36 | |
| *** cody-somerville has quit IRC | 06:39 | |
| *** gokrokve has joined #openstack-infra | 06:40 | |
| *** gokrokve_ has joined #openstack-infra | 06:42 | |
| *** gokrokve has quit IRC | 06:45 | |
| *** nati_ueno has quit IRC | 06:45 | |
| *** dkranz has quit IRC | 06:46 | |
| *** hipster has joined #openstack-infra | 06:47 | |
| *** gokrokve_ has quit IRC | 06:47 | |
| *** jcoufal has joined #openstack-infra | 06:48 | |
| *** vtapia has joined #openstack-infra | 06:48 | |
| *** lcheng has quit IRC | 06:48 | |
| *** gus has quit IRC | 06:49 | |
| *** _nadya_ has quit IRC | 06:50 | |
| *** dstanek_zzz is now known as dstanek | 06:50 | |
| *** hipster has quit IRC | 06:52 | |
| *** Ryan_Lane has joined #openstack-infra | 06:52 | |
| *** Ryan_Lane has quit IRC | 06:57 | |
| *** nati_ueno has joined #openstack-infra | 06:58 | |
| openstackgerrit | Joe Gordon proposed a change to openstack-infra/config: Gate on Postrgres in trove https://review.openstack.org/94307 | 06:58 |
| openstackgerrit | Joe Gordon proposed a change to openstack-infra/config: Add tempest-dsvm-postgres-full into the integrated gate https://review.openstack.org/94311 | 06:58 |
| *** dstanek is now known as dstanek_zzz | 07:00 | |
| *** dkranz has joined #openstack-infra | 07:02 | |
| *** che-arne has quit IRC | 07:08 | |
| openstackgerrit | Joe Gordon proposed a change to openstack-infra/devstack-gate: Save sql logs https://review.openstack.org/94314 | 07:09 |
| *** flaper87|afk is now known as flaper87 | 07:10 | |
| *** praneshp__ has quit IRC | 07:11 | |
| *** afazekas has quit IRC | 07:13 | |
| *** w_ has joined #openstack-infra | 07:14 | |
| *** andreaf has quit IRC | 07:17 | |
| *** olaph has quit IRC | 07:17 | |
| *** paul--_ has quit IRC | 07:18 | |
| *** odyssey4me has joined #openstack-infra | 07:19 | |
| *** ihrachyshka has joined #openstack-infra | 07:24 | |
| *** arnaud_ has joined #openstack-infra | 07:25 | |
| jogo | SergeyLukjanov: ping | 07:26 |
| *** pblaho has joined #openstack-infra | 07:27 | |
| *** igor has joined #openstack-infra | 07:28 | |
| *** arnaud_ has quit IRC | 07:29 | |
| *** ihrachyshka_ has joined #openstack-infra | 07:31 | |
| *** e0ne has joined #openstack-infra | 07:31 | |
| *** igor has quit IRC | 07:32 | |
| *** miqui__ has joined #openstack-infra | 07:32 | |
| *** gus has joined #openstack-infra | 07:33 | |
| *** jgallard has joined #openstack-infra | 07:33 | |
| *** hashar has joined #openstack-infra | 07:35 | |
| *** Hal_ has joined #openstack-infra | 07:37 | |
| *** ihrachyshka has quit IRC | 07:37 | |
| *** vogxn has quit IRC | 07:37 | |
| *** miqui_ has quit IRC | 07:37 | |
| *** isviridov|away has quit IRC | 07:37 | |
| *** odyssey4me has quit IRC | 07:40 | |
| *** gokrokve has joined #openstack-infra | 07:42 | |
| *** isviridov has joined #openstack-infra | 07:43 | |
| *** alff has joined #openstack-infra | 07:44 | |
| *** gokrokve has quit IRC | 07:47 | |
| *** _nadya_ has joined #openstack-infra | 07:50 | |
| *** ilyashakhat has quit IRC | 07:51 | |
| *** hughsaunders has quit IRC | 07:51 | |
| *** esmute has quit IRC | 07:52 | |
| *** ilyashakhat has joined #openstack-infra | 07:52 | |
| *** hughsaunders has joined #openstack-infra | 07:52 | |
| *** Ryan_Lane has joined #openstack-infra | 07:53 | |
| *** jlibosva has joined #openstack-infra | 07:54 | |
| *** marcoemorais1 has quit IRC | 07:55 | |
| *** nati_ueno has quit IRC | 07:56 | |
| *** jpich has joined #openstack-infra | 07:57 | |
| *** Ryan_Lane has quit IRC | 07:57 | |
| *** esmute has joined #openstack-infra | 07:58 | |
| *** nati_uen_ has joined #openstack-infra | 07:58 | |
| *** derekh_ has joined #openstack-infra | 08:00 | |
| *** pelix has joined #openstack-infra | 08:03 | |
| *** andreaf has joined #openstack-infra | 08:25 | |
| *** arnaud_ has joined #openstack-infra | 08:26 | |
| *** rwsu has quit IRC | 08:26 | |
| *** aconrad has quit IRC | 08:27 | |
| *** jooools has joined #openstack-infra | 08:27 | |
| *** igor has joined #openstack-infra | 08:28 | |
| *** dizquierdo has joined #openstack-infra | 08:30 | |
| *** arnaud_ has quit IRC | 08:30 | |
| *** igor has quit IRC | 08:33 | |
| *** gus has quit IRC | 08:33 | |
| *** ihrachyshka_ is now known as ihrachyshka | 08:41 | |
| *** gokrokve has joined #openstack-infra | 08:42 | |
| *** vogxn has joined #openstack-infra | 08:43 | |
| *** gokrokve has quit IRC | 08:47 | |
| *** unicell has joined #openstack-infra | 08:48 | |
| *** Ryan_Lane has joined #openstack-infra | 08:54 | |
| *** Ryan_Lane has quit IRC | 08:54 | |
| *** Ryan_Lane has joined #openstack-infra | 08:54 | |
| *** ominakov has joined #openstack-infra | 08:56 | |
| *** Ryan_Lane has quit IRC | 08:58 | |
| skraynev | Could anybody explain, what happens with check-tempest-dsvm-postgres-full gate ? | 08:58 |
| *** doude has joined #openstack-infra | 09:01 | |
| *** Hal_ has quit IRC | 09:06 | |
| *** Hal_ has joined #openstack-infra | 09:06 | |
| *** nati_uen_ has quit IRC | 09:10 | |
| *** jp_at_hp has joined #openstack-infra | 09:11 | |
| therve | skraynev, https://review.openstack.org/#/c/94315/ | 09:12 |
| therve | And https://bugs.launchpad.net/trove/+bug/1321093 | 09:12 |
| uvirtbot | Launchpad bug 1321093 in trove "*-tempest-dsvm-postgres-full failing on trove-manage db_sync" [Undecided,New] | 09:12 |
| therve | I wonder if some trove guys are around | 09:12 |
| SlickNik | therve: We're working on a fix | 09:13 |
| therve | SlickNik, A different one? | 09:14 |
| SlickNik | Nope: https://review.openstack.org/#/c/94315 | 09:14 |
| SlickNik | same one. | 09:14 |
| SlickNik | I'm smoke testing it before I approve to unblock the gate. | 09:14 |
| therve | Okay | 09:15 |
| *** jhesketh has quit IRC | 09:17 | |
| *** andreykurilin_ has joined #openstack-infra | 09:21 | |
| *** igor has joined #openstack-infra | 09:24 | |
| skraynev | therve: thanks ;) | 09:26 |
| openstackgerrit | Nikita Konovalov proposed a change to openstack-infra/storyboard-webclient: Refactor preference provider https://review.openstack.org/94330 | 09:26 |
| *** igor has quit IRC | 09:28 | |
| openstackgerrit | Nikita Konovalov proposed a change to openstack-infra/storyboard-webclient: Make Timeline configurable https://review.openstack.org/93433 | 09:37 |
| *** andreykurilin_ has quit IRC | 09:39 | |
| *** flaper87 has quit IRC | 09:41 | |
| *** gokrokve has joined #openstack-infra | 09:42 | |
| *** jhesketh has joined #openstack-infra | 09:44 | |
| *** andreykurilin_ has joined #openstack-infra | 09:45 | |
| *** gokrokve has quit IRC | 09:47 | |
| *** yjiang has quit IRC | 09:47 | |
| openstackgerrit | Nikita Konovalov proposed a change to openstack-infra/storyboard-webclient: Make Timeline configurable https://review.openstack.org/93433 | 09:47 |
| *** hipster has joined #openstack-infra | 09:47 | |
| *** unicell has quit IRC | 09:48 | |
| *** unicell has joined #openstack-infra | 09:48 | |
| openstackgerrit | Nikita Konovalov proposed a change to openstack-infra/storyboard-webclient: Small refactor for preferences https://review.openstack.org/94332 | 09:50 |
| *** _nadya_ has quit IRC | 09:50 | |
| *** hipster has quit IRC | 09:51 | |
| *** _nadya_ has joined #openstack-infra | 09:52 | |
| *** andreykurilin_ has quit IRC | 09:53 | |
| *** Ryan_Lane has joined #openstack-infra | 09:54 | |
| *** che-arne has joined #openstack-infra | 09:55 | |
| *** hashar has quit IRC | 09:55 | |
| *** igor has joined #openstack-infra | 09:59 | |
| *** Ryan_Lane has quit IRC | 09:59 | |
| *** rlandy has joined #openstack-infra | 10:09 | |
| *** jcoufal has quit IRC | 10:09 | |
| *** jcoufal has joined #openstack-infra | 10:09 | |
| *** andreykurilin_ has joined #openstack-infra | 10:10 | |
| *** jgallard has quit IRC | 10:20 | |
| *** vogxn has quit IRC | 10:26 | |
| *** arnaud_ has joined #openstack-infra | 10:27 | |
| openstackgerrit | Radomir Dopieralski proposed a change to openstack/requirements: Add xtstatic and xtstatic-jquery for Horizon https://review.openstack.org/94337 | 10:27 |
| *** Longgeek has quit IRC | 10:29 | |
| *** arnaud_ has quit IRC | 10:31 | |
| *** afazekas has joined #openstack-infra | 10:39 | |
| *** prad has quit IRC | 10:40 | |
| *** NithyaG has joined #openstack-infra | 10:41 | |
| *** gokrokve has joined #openstack-infra | 10:42 | |
| *** rdopieralski has joined #openstack-infra | 10:42 | |
| *** nithya_g has quit IRC | 10:44 | |
| rdopieralski | hello there, I've heard that there is a way to use jslint/jshint for Horizon on the gate without adding it to our requirements or including in the project's tree, how would I go about setting that up? | 10:45 |
| *** _nadya_ has quit IRC | 10:45 | |
| rdopieralski | (we can't depend on it because of license issues) | 10:46 |
| *** gokrokve has quit IRC | 10:47 | |
| *** hipster has joined #openstack-infra | 10:47 | |
| *** Longgeek has joined #openstack-infra | 10:51 | |
| jpich | rdopieralski: I'm not familiar with the details but I believe storyboard is the project that uses it, maybe you'll find some more information looking at the jobs they're running and how | 10:51 |
| jpich | while waiting for someone more knowledgeable to wake up and provide actually useful information | 10:52 |
| *** hipster has quit IRC | 10:52 | |
| *** Ryan_Lane has joined #openstack-infra | 10:55 | |
| rdopieralski | jpich: thanks | 10:56 |
| rdopieralski | aah, I see it's just a tox job that installs all the js stuff | 10:58 |
| *** NithyaG has quit IRC | 10:59 | |
| *** Ryan_Lane has quit IRC | 10:59 | |
| *** andreykurilin_ has quit IRC | 11:06 | |
| *** Longgeek has quit IRC | 11:11 | |
| *** andreykurilin_ has joined #openstack-infra | 11:16 | |
| *** _nadya_ has joined #openstack-infra | 11:16 | |
| *** Longgeek has joined #openstack-infra | 11:18 | |
| openstackgerrit | Thanassis Parathyras proposed a change to openstack-infra/config: Adding Greek Language mailing list in the corresponding puppet module. https://review.openstack.org/94342 | 11:20 |
| *** alff has quit IRC | 11:23 | |
| *** yfried_ has joined #openstack-infra | 11:24 | |
| *** yfried has quit IRC | 11:25 | |
| *** NithyaG has joined #openstack-infra | 11:27 | |
| *** arnaud_ has joined #openstack-infra | 11:28 | |
| *** weshay has joined #openstack-infra | 11:30 | |
| *** Hal_ has quit IRC | 11:32 | |
| *** arnaud_ has quit IRC | 11:32 | |
| *** yfried__ has joined #openstack-infra | 11:33 | |
| *** yfried_ has quit IRC | 11:33 | |
| *** w__ has joined #openstack-infra | 11:38 | |
| *** NithyaG has quit IRC | 11:39 | |
| *** GheRivero has quit IRC | 11:39 | |
| *** w_ has quit IRC | 11:40 | |
| *** matsuhashi has quit IRC | 11:41 | |
| *** gokrokve has joined #openstack-infra | 11:42 | |
| *** NithyaG has joined #openstack-infra | 11:42 | |
| *** andreykurilin_ has quit IRC | 11:43 | |
| *** gokrokve has quit IRC | 11:47 | |
| *** Hal_ has joined #openstack-infra | 11:47 | |
| *** mbacchi has joined #openstack-infra | 11:50 | |
| *** GheRivero has joined #openstack-infra | 11:50 | |
| *** HenryG has quit IRC | 11:53 | |
| *** Ryan_Lane has joined #openstack-infra | 11:56 | |
| *** hashar has joined #openstack-infra | 11:57 | |
| *** Ryan_Lane has quit IRC | 12:01 | |
| *** pdmars has joined #openstack-infra | 12:03 | |
| *** afazekas has quit IRC | 12:05 | |
| *** alff has joined #openstack-infra | 12:05 | |
| *** ArxCruz has joined #openstack-infra | 12:06 | |
| *** ominakov_ has joined #openstack-infra | 12:06 | |
| *** yfried__ has quit IRC | 12:07 | |
| *** yfried__ has joined #openstack-infra | 12:07 | |
| *** ominakov has quit IRC | 12:08 | |
| *** dhellmann_ is now known as dhellmann | 12:16 | |
| *** salv-orlando_ has joined #openstack-infra | 12:18 | |
| *** salv-orlando has quit IRC | 12:20 | |
| *** salv-orlando_ is now known as salv-orlando | 12:20 | |
| *** afazekas has joined #openstack-infra | 12:20 | |
| *** aysyd has joined #openstack-infra | 12:21 | |
| *** doude_ has joined #openstack-infra | 12:22 | |
| *** doude has quit IRC | 12:22 | |
| *** lcostantino has joined #openstack-infra | 12:23 | |
| *** mriedem has joined #openstack-infra | 12:23 | |
| *** dprince has joined #openstack-infra | 12:23 | |
| *** dizquierdo has quit IRC | 12:24 | |
| SergeyLukjanov | jogo, pong | 12:24 |
| *** ominakov_ has quit IRC | 12:26 | |
| *** flaper87|afk has joined #openstack-infra | 12:27 | |
| *** flaper87|afk is now known as flaper87 | 12:27 | |
| SergeyLukjanov | jeblair, I'll take alook, thx! | 12:27 |
| *** ominakov has joined #openstack-infra | 12:28 | |
| *** arnaud_ has joined #openstack-infra | 12:28 | |
| rcarrillocruz | zaro: saw your message about getting rid of drafts in gerrit this morning? Should I mark https://bugs.launchpad.net/openstack-ci/+bug/1078231 as invalid then? | 12:29 |
| uvirtbot | Launchpad bug 1078231 in openstack-ci "Draft shouldn't update Launchpad before being Published" [Low,In progress] | 12:29 |
| *** yfried__ is now known as yfried | 12:32 | |
| *** arnaud_ has quit IRC | 12:32 | |
| *** plars has quit IRC | 12:35 | |
| openstackgerrit | Aleksandra Fedorova proposed a change to openstack-infra/config: New fuel-specs repo for Fuel designs and specs https://review.openstack.org/93560 | 12:36 |
| phschwartz | Morning infra | 12:36 |
| SergeyLukjanov | phschwartz, hey | 12:38 |
| *** ildikov has joined #openstack-infra | 12:39 | |
| SergeyLukjanov | fungi, clarkb, jeblair, mordred, folks, I'll not be able to attend todays infra meeting - I'll be on plane, we'll be completely available at Thursday | 12:39 |
| SergeyLukjanov | zaro, anteaya, ^^ | 12:39 |
| SergeyLukjanov | I'll be here for next 1-2 hours | 12:40 |
| bookwar | anteaya, we already discussed this acl issue, could you please re-review https://review.openstack.org/93560 ? | 12:40 |
| *** eharney has joined #openstack-infra | 12:41 | |
| *** dstanek_zzz is now known as dstanek | 12:41 | |
| openstackgerrit | Thanassis Parathyras proposed a change to openstack-infra/config: Adding Greek Language mailing list in the corresponding puppet module https://review.openstack.org/94342 | 12:41 |
| *** gokrokve has joined #openstack-infra | 12:42 | |
| *** rfolco has joined #openstack-infra | 12:43 | |
| openstackgerrit | Thanassis Parathyras proposed a change to openstack-infra/config: Adding Greek Language mailing list in the corresponding puppet module https://review.openstack.org/94342 | 12:43 |
| anteaya | rcarrillocruz: it might be a bit of time before we can get rid of drafts, but it is a goal | 12:45 |
| anteaya | rcarrillocruz: let's leave that bug as it is for the moment | 12:46 |
| *** flaper87 has quit IRC | 12:46 | |
| *** flaper87 has joined #openstack-infra | 12:46 | |
| anteaya | SergeyLukjanov: ack, and safe flight | 12:46 |
| anteaya | phschwartz: morning | 12:46 |
| SergeyLukjanov | anteaya, thx! | 12:46 |
| anteaya | bookwar: hi, yes, I saw the new patchset, just finding my post summit self, I have it on my list of patches to review | 12:46 |
| phschwartz | anteaya: How are you this morning? | 12:47 |
| anteaya | phschwartz: well rested and smiling, thanks for asking | 12:47 |
| anteaya | how about yourself? | 12:47 |
| phschwartz | :) | 12:47 |
| anteaya | :D | 12:47 |
| *** gokrokve has quit IRC | 12:47 | |
| phschwartz | anteaya: Doing good. Today is my 7th wedding anniversary so it is a really good day. | 12:47 |
| bookwar | anteaya: thanks | 12:49 |
| anteaya | phschwartz: congratulations, and happy anniversary to you both | 12:50 |
| anteaya | phschwartz: any plans? | 12:50 |
| phschwartz | anteaya: Going to this nice gourmet restaurant that we haven't tried before. http://www.sixtablesbocaraton.com | 12:50 |
| *** yfried has quit IRC | 12:53 | |
| openstackgerrit | Cyril Roelandt proposed a change to openstack/requirements: django_compressor: bump to 1.4 https://review.openstack.org/94357 | 12:54 |
| anteaya | that looks like a lovely night out | 12:54 |
| anteaya | I hope you have fun | 12:54 |
| *** andreykurilin_ has joined #openstack-infra | 12:54 | |
| *** nati_ueno has joined #openstack-infra | 12:55 | |
| *** esker has joined #openstack-infra | 12:55 | |
| *** julim has joined #openstack-infra | 12:58 | |
| *** esker has quit IRC | 12:59 | |
| *** jgrimm has joined #openstack-infra | 13:00 | |
| *** changbl has joined #openstack-infra | 13:00 | |
| openstackgerrit | Sean Dague proposed a change to openstack-infra/jeepyb: add dashboard support to jeepyb https://review.openstack.org/94260 | 13:01 |
| *** andreykurilin_ has quit IRC | 13:01 | |
| openstackgerrit | Cyril Roelandt proposed a change to openstack/requirements: Netifaces: bump to 0.10.4 https://review.openstack.org/94358 | 13:01 |
| *** nosnos has quit IRC | 13:02 | |
| *** nati_ueno has quit IRC | 13:04 | |
| *** heyongli has joined #openstack-infra | 13:04 | |
| *** changbl has quit IRC | 13:05 | |
| *** plars has joined #openstack-infra | 13:05 | |
| *** miqui__ is now known as miqui_ | 13:07 | |
| *** smarcet has joined #openstack-infra | 13:07 | |
| *** w__ has quit IRC | 13:07 | |
| *** dstanek is now known as dstanekafk | 13:09 | |
| *** pblaho has quit IRC | 13:10 | |
| *** dstanekafk is now known as dstanekafk_zzz | 13:11 | |
| *** yamahata has joined #openstack-infra | 13:11 | |
| *** mwagner_lap has quit IRC | 13:11 | |
| *** zz_gondoi is now known as gondoi | 13:11 | |
| *** pcrews_ has joined #openstack-infra | 13:12 | |
| *** afazekas has quit IRC | 13:12 | |
| *** ildikov has quit IRC | 13:13 | |
| *** bknudson has quit IRC | 13:17 | |
| *** e0ne_ has joined #openstack-infra | 13:19 | |
| *** signed8bit has joined #openstack-infra | 13:20 | |
| *** w__ has joined #openstack-infra | 13:22 | |
| rdopieralski | is there an official openstack user on the pypi? | 13:22 |
| *** nati_ueno has joined #openstack-infra | 13:22 | |
| *** e0ne has quit IRC | 13:22 | |
| *** dstanekafk_zzz is now known as dstanek | 13:23 | |
| rdopieralski | openstackci, I assume? | 13:24 |
| *** afazekas has joined #openstack-infra | 13:25 | |
| fungi | rdopieralski: yes, make openstackci a package index owner | 13:26 |
| fungi | rdopieralski: that will allow our release automation to upload packages and trove metadata | 13:26 |
| anteaya | hey fungi | 13:26 |
| fungi | anteaya: hi there | 13:26 |
| anteaya | how was your weekend? | 13:27 |
| *** dhellmann is now known as dhellmann_ | 13:27 | |
| anteaya | I see you took one, well done | 13:27 |
| rdopieralski | fungi: thanks | 13:28 |
| fungi | anteaya: i was staying in a lodge up on a mountainside with no cell phone service | 13:29 |
| fungi | deep in a national park | 13:29 |
| fungi | it was... relaxing | 13:29 |
| *** arnaud has joined #openstack-infra | 13:29 | |
| *** hipster has joined #openstack-infra | 13:29 | |
| BobBall | I know it's only the second day back after the conference, so I'm very happy to be told to wait - but is https://bugs.launchpad.net/openstack-ci/+bug/1319184 in the right place to get me set as the PTL for xenapi-os-testing? | 13:29 |
| uvirtbot | Launchpad bug 1319184 in openstack-ci "Request initial member: xenapi-os-testing-core" [Undecided,New] | 13:29 |
| *** doug-fish has joined #openstack-infra | 13:29 | |
| fungi | BobBall: sure, just a sec and i'll add you | 13:29 |
| BobBall | oh perfect! :D | 13:30 |
| *** monester has joined #openstack-infra | 13:30 | |
| fungi | BobBall: actually, looks like there may have been a problem with the acl for stackforge/xenapi-os-testing (i'm digging into it now) | 13:31 |
| anteaya | fungi: oh I'm so glad | 13:32 |
| *** hipster has quit IRC | 13:32 | |
| anteaya | that sounds awesome | 13:32 |
| *** miqui has quit IRC | 13:32 | |
| BobBall | hmmm OK - let me know if you need me to do anything. I'll be honest I was just following without understanding so it's quite possible I got something wrong. | 13:32 |
| *** nati_ueno has quit IRC | 13:33 | |
| *** arnaud has quit IRC | 13:33 | |
| openstackgerrit | Nikita Konovalov proposed a change to openstack-infra/storyboard: Refresh token support https://review.openstack.org/94363 | 13:34 |
| fungi | nothing jumps out at me as being syntactically wrong... checking the logs on the server for any related errors just after 2014-05-09 19:00 UTC | 13:35 |
| *** jgrimm has quit IRC | 13:35 | |
| openstackgerrit | Sean Dague proposed a change to openstack-infra/zuul: Implement Depends-On: IdempotentID https://review.openstack.org/94365 | 13:35 |
| fungi | which should correspond with the merging of https://review.openstack.org/87680 | 13:35 |
| sdague | now I need jeblair to be up to give me ideas about why that is just generating alarm events in my unit tests | 13:35 |
| *** homeless has joined #openstack-infra | 13:36 | |
| *** jgrimm has joined #openstack-infra | 13:36 | |
| AJaeger | fungi, that sounds like an excellent weekend! | 13:36 |
| fungi | AJaeger: well, i wasn't there for long, and was in town primarily to help set up/break down/participate in my brother's wedding. but to be able to get away from family in between, i booked a room at a lodge in the middle of a national forest about 45 minutes away from them ;) | 13:39 |
| *** timrc is now known as timrc-afk | 13:39 | |
| AJaeger | ;) | 13:39 |
| *** timrc-afk is now known as timrc | 13:40 | |
| *** bknudson has joined #openstack-infra | 13:41 | |
| openstackgerrit | Nikita Konovalov proposed a change to openstack-infra/storyboard-webclient: Fix StoryBoard spelling https://review.openstack.org/93406 | 13:41 |
| *** gokrokve has joined #openstack-infra | 13:42 | |
| jeblair | sdague: welcome to the wonderful world of testr and zuul. :( | 13:43 |
| jeblair | sdague: an alarm probably means you're hitting a test timeout, and test timeouts don't come with any logs | 13:43 |
| jeblair | sdague: it's the most maddening thing | 13:44 |
| jeblair | sdague: so the process is something like this: | 13:44 |
| sdague | hmmm... I thought we got those in tempest | 13:44 |
| jeblair | sdague: a) guess which tests might be timing out -- sometimes this involves reading subunit streams. if you're lucky, you already know which test timed out. | 13:44 |
| jeblair | sdague: b) run that test in the foreground (there are some commands in TESTING that do this) | 13:45 |
| *** wenlock has joined #openstack-infra | 13:45 | |
| sdague | I have no idea what timed out | 13:45 |
| sdague | it's hitting it on 4 threads | 13:45 |
| *** andreaf has quit IRC | 13:45 | |
| jeblair | sdague: we have to use a hard timeout -- the soft timeout doesn't actually kill the test, and i believe it's the hard timeouts that lose logs | 13:46 |
| sdague | ok | 13:46 |
| *** pcm__ has joined #openstack-infra | 13:46 | |
| jeblair | sdague: so yeah, i think the best process people have come up with in this case is to examine the subunit log and see which test started but did not finish | 13:46 |
| *** gokrokve has quit IRC | 13:46 | |
| jeblair | sdague: or, since this is likely to be reliably reproducible (eg, not racy), you might be able to run all the tests sequentially and print their names | 13:47 |
| jeblair | sdague: i think there's a command in TESTING that illustrates that too | 13:47 |
| sdague | jeblair: yeh, honestly, I'll probably pull the tempest subunit filter over, that makes life readable, even if only in my local tree | 13:47 |
| fungi | mordred: clarkb: jeblair: yay new (old?) manage-projects issue... stackforge/xenapi-os-testing was created on may 9th but didn't have an acl applied. i dug through the gerrit error/ssh logs (syslog had been rotated into oblivion weeks ago) and found no smoking gun. manually re-running manage-projects for that project seemed to add the acl just fine :/ | 13:48 |
| jeblair | sdague: that works too. though the commands under "Hanging Tests" in the TESTING.rst file are the ones i'm thinking of that do something similar | 13:49 |
| *** _nadya_ has quit IRC | 13:49 | |
| anteaya | fungi: :/ | 13:49 |
| *** andreykurilin_ has joined #openstack-infra | 13:50 | |
| jeblair | sdague: fyi tests.test_scheduler.TestScheduler.test_jobs_launched is sort of the 'hello world' test; it's a good one to just run and tell you if you've broken everything | 13:50 |
| jeblair | fungi: well, we do have a LOT of new project creations coming up in the spec repos | 13:50 |
| anteaya | fungi: do we have any logging around the acl application event currently? | 13:50 |
| jeblair | so we may be able to get more data | 13:51 |
| sdague | fungi: so I was in that code | 13:51 |
| sdague | my guess is that it raced on gerrit building that file | 13:51 |
| sdague | and just bailed out early silently | 13:51 |
| *** eglynn has joined #openstack-infra | 13:52 | |
| fungi | while i was out, did any additional logging of manage-projects get added to a not-syslog location which might still be hanging around weeks later? | 13:52 |
| openstackgerrit | Radomir Dopieralski proposed a change to openstack/requirements: Add xstatic and xstatic-jquery for Horizon https://review.openstack.org/94337 | 13:53 |
| fungi | but yeah, hopefully we can spot this quickly enough to have a record of what happened next time | 13:53 |
| anteaya | we have one error message: http://git.openstack.org/cgit/openstack-infra/jeepyb/tree/jeepyb/cmd/manage_projects.py#n487 | 13:53 |
| sdague | I've got a clean up here - https://review.openstack.org/#/c/94196/3 - but if that's all just syslog logging, it won't change that | 13:53 |
| jeblair | anyone know if Radomir is on irc? | 13:55 |
| fungi | whee! more launchpad bug update timeouts. "Server error, please contact an administrator." | 13:55 |
| anteaya | I don't know who that person is, so not me | 13:56 |
| *** nati_ueno has joined #openstack-infra | 13:56 | |
| anteaya | fungi: where are you seeing that? | 13:56 |
| anteaya | is that in our gerrit launchpad hooks? | 13:57 |
| fungi | anteaya: adding a comment to bug 1319184 before i mark it fixed | 13:57 |
| uvirtbot | Launchpad bug 1319184 in openstack-ci "Request initial member: xenapi-os-testing-core" [Medium,In progress] https://launchpad.net/bugs/1319184 | 13:57 |
| *** e0ne_ has quit IRC | 13:57 | |
| fungi | anteaya: nope, just a plain old user clicking his browser in launchpad | 13:57 |
| *** e0ne has joined #openstack-infra | 13:57 | |
| anteaya | fungi: the timeout in plain sight | 13:58 |
| fungi | anteaya: according to StevenK the number of "administrators" to contact on lp problems is rapidly approaching the null set | 13:58 |
| anteaya | yeah, heyho storyboard | 13:58 |
| fungi | yay! only took four tries to add my comment | 13:58 |
| anteaya | well done | 13:58 |
| sdague | jeblair: so if I dump over the pretty tox from tempest, I get the following - http://paste.openstack.org/show/81000/ | 13:59 |
| fungi | BobBall: you should be all set now | 13:59 |
| sdague | which is that 7 workers hung? | 13:59 |
| BobBall | that's wonderful - thanks fungi. | 13:59 |
| fungi | chmouel: i just noticed that https://review.openstack.org/92988 should also be adding that job to the gate pipeline when making it voting, so i'll push a quick patchset to fix that before approving | 14:01 |
| anteaya | sdague: line 179: https://review.openstack.org/#/c/94196/3/jeepyb/cmd/manage_projects.py | 14:01 |
| sdague | anteaya: old or new side? | 14:01 |
| anteaya | the logic changes and matches the logic on line 177 | 14:01 |
| anteaya | new side | 14:01 |
| anteaya | help me understand what is happening | 14:01 |
| sdague | diff --quiet returns > 0 if there are changes | 14:02 |
| sdague | if there are not changes, we throw an exception so that no further processing inside the try happens | 14:02 |
| jeblair | sdague: sounds about right; most of those tests exercise multiple items in a dependent pipeline | 14:03 |
| BobBall | sdague: return value from that push_acl_config seems to be ignored? | 14:03 |
| anteaya | okay so we evaluate status, adjust the value of status, and evaluate status again | 14:03 |
| anteaya | and if the value never adjusted we stop | 14:03 |
| sdague | BobBall: yep, it was before as well | 14:03 |
| sdague | anteaya: right | 14:04 |
| anteaya | sdague: great, thanks | 14:04 |
| BobBall | that could be where the issue hit by xenapi-os-testing was though | 14:04 |
| * anteaya continues to review | 14:04 | |
| sdague | BobBall: I actually expect the issue is the copy | 14:04 |
| sdague | copy_acl_config really depends on gerrit creating an initial version | 14:05 |
| *** wenlock has quit IRC | 14:05 | |
| BobBall | OK | 14:05 |
| sdague | because it's assuming the acl is an already versioned file | 14:05 |
| sdague | so there is definitely a race there | 14:05 |
| openstackgerrit | Jeremy Stanley proposed a change to openstack-infra/config: Make swiftclient python3 job as voting https://review.openstack.org/92988 | 14:05 |
| *** mattoliverau has quit IRC | 14:06 | |
| *** miqui_ is now known as miqui | 14:06 | |
| fungi | sdague: good point. maybe a busy-wait with a timeout+raise there would help | 14:07 |
| sdague | fungi: it could | 14:07 |
| fungi | sdague: i think clarkb added something similar for group creation too | 14:07 |
| sdague | oh, maybe in group creation I was reading about that | 14:07 |
| anteaya | this was posted this morning, a netifaces bump: https://review.openstack.org/#/c/94358/ since this created an issue last time I just wanted folks to know | 14:08 |
| anteaya | I don't know how to test this myself, yet | 14:08 |
| fungi | anteaya: looks like we got that version in our mirror yesterday, so should be fine... http://pypi.openstack.org/openstack/netifaces/ | 14:10 |
| *** jamielennox|away has quit IRC | 14:11 | |
| anteaya | was that the issue last time, it wasn't in our mirror? | 14:11 |
| *** ianw has quit IRC | 14:12 | |
| fungi | anteaya: well, "the issue" is a little vague (we've had plenty of mirror/reqs/integration issues) | 14:12 |
| fungi | anteaya: also, https://pypi.python.org/simple/netifaces/ has tarballs now, so we should be able to take it out of our whitelist in the integration test script in the pbr repo | 14:12 |
| anteaya | I thought there was breakage, but I could be remembering wrong | 14:12 |
| *** dkehn_ is now known as dkehn | 14:13 | |
| *** ianw has joined #openstack-infra | 14:13 | |
| *** danpb has joined #openstack-infra | 14:13 | |
| *** ianw has quit IRC | 14:14 | |
| *** mattoliverau has joined #openstack-infra | 14:14 | |
| fungi | oh, not pbr, but run-mirror is where we have that whitelisted now... http://git.openstack.org/cgit/openstack-infra/pypi-mirror/tree/pypi_mirror/cmd/run_mirror.py | 14:14 |
| Alex_Gaynor | jeblair: ping | 14:14 |
| *** ianw has joined #openstack-infra | 14:14 | |
| danpb | i'm guessing this has probably been raised before.... is there a reason why the wiki SSO session timeout is so short ? | 14:14 |
| *** ianw has quit IRC | 14:14 | |
| danpb | it is frustrating to get logged out of the wiki multiple times a day, while gerrit happily leaves us logged in for much longer | 14:15 |
| *** mkerrin has quit IRC | 14:15 | |
| *** ianw has joined #openstack-infra | 14:15 | |
| *** ianw has quit IRC | 14:15 | |
| openstackgerrit | Alex Gaynor proposed a change to openstack-infra/config: Re-enfranchise the PyPy jobs https://review.openstack.org/94373 | 14:15 |
| fungi | danpb: i think we have an open bug against that and Ryan_Lane was looking into it... let me see if i can find it | 14:15 |
| *** ianw has joined #openstack-infra | 14:15 | |
| chmouel | fungi: okay | 14:16 |
| fungi | danpb: bug 1172753 | 14:16 |
| uvirtbot | Launchpad bug 1172753 in openstack-ci "Wiki timeout too short" [Medium,Triaged] https://launchpad.net/bugs/1172753 | 14:16 |
| danpb | fungi: thanks for the info | 14:17 |
| *** _nadya_ has joined #openstack-infra | 14:17 | |
| fungi | danpb: seems to have been rotting in its current state for roughly a year :/ | 14:17 |
| *** heyongli has quit IRC | 14:17 | |
| *** jamielennox|away has joined #openstack-infra | 14:18 | |
| openstackgerrit | A change was merged to openstack-infra/config: Make swiftclient python3 job as voting https://review.openstack.org/92988 | 14:18 |
| fungi | chmouel: ^ | 14:18 |
| eglynn | another potential gerrit issue introduced by the upgrade? | 14:18 |
| eglynn | mere mortals seem to have lost their non-binding +1 ability on goverance patches | 14:19 |
| chmouel | fungi: sweet! | 14:19 |
| eglynn | e.g. https://review.openstack.org/87526 | 14:19 |
| *** jgallard has joined #openstack-infra | 14:19 | |
| eglynn | where "mere mortal" == !"TC member" | 14:19 |
| fungi | eglynn: i believe that has always been the case (i've never been able to +1/-1 governance changes, only comment) | 14:19 |
| jeblair | Alex_Gaynor: pong | 14:20 |
| *** ianw has quit IRC | 14:20 | |
| openstackgerrit | Radomir Dopieralski proposed a change to openstack/requirements: Add pyscss and django_pyscss for Horizon https://review.openstack.org/94376 | 14:20 |
| Alex_Gaynor | jeblair: For the pypy builds, if we want the new package that was just released, we need to build a new image? | 14:20 |
| *** ianw has joined #openstack-infra | 14:21 | |
| *** msabramo has quit IRC | 14:21 | |
| eglynn | fungi: a-ha, ok, thanks! ... nijaba, jd__ & /me recalled being able to +1, but maybe we were mistaken | 14:21 |
| fungi | eglynn: tc members get -1..+1 code review on that repository, and the tc chair gets -2..+2 code review, -1..+1 workflow on it | 14:21 |
| eglynn | fungi: cool, thanks ... apologies for the noise :) | 14:22 |
| jeblair | eglynn, fungi: correct; however, we may be able to add a new column to governance for non-binding votes. please do leave comments though, we definitely want them | 14:22 |
| jeblair | eglynn: (it was merely a technical limitation of gerrit 2.4 that caused us to disallow votes) | 14:22 |
| eglynn | jeblair: thanks, we prolly just confused it with the policy on the other repo for which we're not core | 14:22 |
| sdague | jeblair: if my current random debugging is correct, the real issue seems to be that self.sched.wake_event.wait(0.1) doesn't actually timeout | 14:22 |
| eglynn | *repos | 14:23 |
| jeblair | sdague: that doesn't sound right | 14:23 |
| jeblair | Alex_Gaynor: i believe so | 14:24 |
| Alex_Gaynor | jeblair: what does it take to rebuild that image? | 14:24 |
| jeblair | Alex_Gaynor: happens daily, may currently be in progress. hpcloud fails 98% of the time and may take weeks to propagate there. | 14:24 |
| *** gondoi is now known as zz_gondoi | 14:24 | |
| fungi | Alex_Gaynor: also, is the new package in the ppa we're pulling from? | 14:25 |
| Alex_Gaynor | fungi: yup; ina legacy system this would just be apt-get update && apt-get upgrade | 14:25 |
| sdague | jeblair: nope, sorry, it's earlier, it's blocking trying to get a lock | 14:25 |
| jeblair | Alex_Gaynor: http://nodepool.openstack.org/image.log | 14:25 |
| jeblair | Alex_Gaynor: that's the full log from all the builds (interleaved, so grep for the one you're interested in) | 14:25 |
| *** zz_gondoi is now known as gondoi | 14:26 | |
| jeblair | Alex_Gaynor: eg "grep rax-ord.py3k-precise image.log" | 14:26 |
| fungi | Alex_Gaynor: worth noting, there are 33 images in the midst of being built right now | 14:27 |
| jeblair | so yeah, in progress as we speak | 14:27 |
| jeblair | started about 10 mins ago | 14:27 |
| fungi | 6 are py3k images | 14:27 |
| sdague | self.fake_gerrit.event_queue.join() that's where the hang is for me (even on concurency 1) | 14:27 |
| Alex_Gaynor | Ok, so sounds like we just wait a bit and then we can land https://review.openstack.org/#/c/94373/ ? | 14:27 |
| fungi | (which is where the pypy jobs get run) | 14:28 |
| jeblair | Alex_Gaynor: we'll want to make sure they all updated successfully | 14:28 |
| *** dims has joined #openstack-infra | 14:28 | |
| Alex_Gaynor | jeblair: okey doke | 14:29 |
| *** arnaud has joined #openstack-infra | 14:29 | |
| *** nati_ueno has quit IRC | 14:32 | |
| *** jcoufal has quit IRC | 14:33 | |
| *** arnaud has quit IRC | 14:33 | |
| *** yfried has joined #openstack-infra | 14:35 | |
| *** ihrachyshka has quit IRC | 14:37 | |
| *** Hal_ has quit IRC | 14:39 | |
| *** hipster has joined #openstack-infra | 14:39 | |
| *** wenlock has joined #openstack-infra | 14:40 | |
| *** david-lyle has joined #openstack-infra | 14:40 | |
| jeblair | sdague: that probably means one of the loops errored out, therefore the test never finished. at this point it's probably worth capturing the full log output and looking for an exception | 14:40 |
| jeblair | sdague: (so run the test alone in the foreground and redirect stdout/stderr) | 14:40 |
| *** wenlock has quit IRC | 14:40 | |
| *** wenlock has joined #openstack-infra | 14:41 | |
| *** maxbit has joined #openstack-infra | 14:42 | |
| *** gokrokve has joined #openstack-infra | 14:42 | |
| rcarrillocruz | anteaya: you think it's worthwhile working on a fix then? I started a branch on that draft gerrit integration, depending on how far we will get rid of it I would continue working on it... | 14:42 |
| *** sarob has joined #openstack-infra | 14:42 | |
| *** Hal_ has joined #openstack-infra | 14:44 | |
| *** msabramo has joined #openstack-infra | 14:45 | |
| phschwartz | Is the process for requesting gerrit groups still through launchpad | 14:46 |
| *** gokrokve has quit IRC | 14:46 | |
| *** mwagner_lap has joined #openstack-infra | 14:48 | |
| sdague | jeblair: ok, well, same thing, it's hanging on that join. Which is not true before my patch. So any chance at looking at the patch for where I might be doing a bad thing? | 14:48 |
| *** esker has joined #openstack-infra | 14:48 | |
| *** dprince has quit IRC | 14:48 | |
| *** esker has quit IRC | 14:48 | |
| jeblair | sdague: which test are you running? | 14:48 |
| *** esker has joined #openstack-infra | 14:48 | |
| sdague | tests.test_scheduler.TestScheduler.test_build_configuration | 14:49 |
| jeblair | sdague: run: .tox/py27/bin/python -m testtools.run tests.test_scheduler.TestScheduler.test_build_configuration | 14:50 |
| mriedem | clarkb: i've got a query for bug https://bugs.launchpad.net/keystone/+bug/1320670 but it's only hitting on experimental queue jobs on master, but it does show up in stable/icehouse jobs in grenade, | 14:51 |
| uvirtbot | Launchpad bug 1320670 in grenade "404 on GET /v3/OS-SIMPLE-CERT/ca at grenade" [Undecided,In progress] | 14:51 |
| *** nati_ueno has joined #openstack-infra | 14:51 | |
| mriedem | clarkb: wondering if i'm missing something? or the query just won't hit stable/icehouse grenade jobs for some other reason, i.e. not indexed or something | 14:51 |
| jeblair | sdague: you should see the error | 14:51 |
| anteaya | found the netifaces thing: http://eavesdrop.openstack.org/irclogs/%23openstack-infra/%23openstack-infra.2014-05-03.log timestamp 2014-05-03T19:36:17 | 14:51 |
| sdague | ok, got it | 14:52 |
| anteaya | rcarrillocruz: well it all depends on when we are getting rid of drafts in gerrit | 14:52 |
| *** gondoi is now known as zz_gondoi | 14:52 | |
| openstackgerrit | Antoine Musso proposed a change to stackforge/python-jenkins: Support node configuration https://review.openstack.org/89604 | 14:52 |
| anteaya | rcarrillocruz: I had thought that was going to happen when we upgrade to gerrit 2.9, and as far as I know we don't have a time for when we are moving to 2.9 | 14:53 |
| anteaya | since we just moved to 2.8 | 14:53 |
| anteaya | rcarrillocruz: I think we should hear from zaro and others | 14:53 |
| openstackgerrit | Antoine Musso proposed a change to stackforge/python-jenkins: Support node configuration https://review.openstack.org/89604 | 14:53 |
| *** derekh_ has quit IRC | 14:54 | |
| openstackgerrit | Antoine Musso proposed a change to stackforge/python-jenkins: Support node startup with the JNLP Launcher https://review.openstack.org/89603 | 14:54 |
| openstackgerrit | Antoine Musso proposed a change to stackforge/python-jenkins: Support node configuration https://review.openstack.org/89604 | 14:54 |
| *** mattoliverau has quit IRC | 14:55 | |
| *** esker has quit IRC | 14:55 | |
| *** esker has joined #openstack-infra | 14:55 | |
| *** sarob has quit IRC | 14:57 | |
| *** mfer has joined #openstack-infra | 14:57 | |
| *** sarob has joined #openstack-infra | 14:57 | |
| *** zz_gondoi is now known as gondoi | 14:57 | |
| anteaya | phschwartz: what gerrit group do you need created? | 14:57 |
| *** Ryan_Lane has joined #openstack-infra | 14:58 | |
| anteaya | hey Ryan_Lane | 14:58 |
| *** mrodden has joined #openstack-infra | 14:58 | |
| *** james_li has joined #openstack-infra | 14:59 | |
| phschwartz | anteaya: The ones for vinz that you also noted in your review of my patchset | 14:59 |
| *** cody-somerville has joined #openstack-infra | 14:59 | |
| *** kgriffs|afk is now known as kgriffs | 14:59 | |
| anteaya | phschwartz: I am not sure if the groups get created automatically by manage-projects or if they are created manually after the patch merges | 15:00 |
| anteaya | but having them in the patch is sufficient for you communicating that they need to be created | 15:00 |
| *** mattoliverau has joined #openstack-infra | 15:00 | |
| anteaya | I am assuming you are the vinz-ptl? | 15:00 |
| *** mkerrin has joined #openstack-infra | 15:01 | |
| phschwartz | anteaya: The docs say after the merge to crete a bug in lp for it. Just wanted to make sure that it was needed that way still | 15:01 |
| anteaya | hmmm, I don't know | 15:01 |
| anteaya | fungi: any input? | 15:01 |
| phschwartz | anteaya: Probably to start yes. And after we get established and moving forward with more people working on it, it would probably be up for a vote like other projects. | 15:01 |
| *** sarob has quit IRC | 15:01 | |
| anteaya | kk | 15:02 |
| *** gondoi is now known as zz_gondoi | 15:02 | |
| *** Ryan_Lane has quit IRC | 15:03 | |
| *** malini_afk is now known as malini | 15:03 | |
| anteaya | jhesketh: when you are next online, I have commented back to you here: https://review.openstack.org/#/c/93562/3 | 15:04 |
| *** dizquierdo has joined #openstack-infra | 15:05 | |
| *** Longgeek has quit IRC | 15:05 | |
| *** mattoliverau has quit IRC | 15:06 | |
| *** zz_gondoi is now known as gondoi | 15:06 | |
| *** e0ne has quit IRC | 15:08 | |
| *** sandywalsh_ has quit IRC | 15:08 | |
| *** sandywalsh has joined #openstack-infra | 15:08 | |
| *** Hal_ has quit IRC | 15:08 | |
| *** moted has joined #openstack-infra | 15:09 | |
| anteaya | a repo in the openstack-infra space should require a contributor agreement, should it not? | 15:09 |
| anteaya | repos in stackforge don't need to and spec repos don't need to, but I think openstack-infra/puppet-storyboard should | 15:09 |
| anteaya | notmyname: why does swift need a new repo for a ceph backend? https://review.openstack.org/#/c/93102/ | 15:11 |
| phschwartz | krotscheck: so get this. Accessing the REST API for Gerrit 2.8.4 using HTTP requests works fine for me, but js from mod_proxy was a cluster F*** still | 15:11 |
| *** marun has joined #openstack-infra | 15:11 | |
| *** zul has quit IRC | 15:11 | |
| anteaya | notmyname: obviously I am missing something here | 15:11 |
| *** zul has joined #openstack-infra | 15:11 | |
| *** nati_ueno has quit IRC | 15:11 | |
| *** zns has joined #openstack-infra | 15:12 | |
| notmyname | anteaya: that's a community-developed thing (like the zeroVM and glusterFS ones). I don't know much about it, but it seems the devs there want it to be in stackforge | 15:12 |
| notmyname | anteaya: similar to how the swift3 middleware is in stackforge | 15:12 |
| * anteaya looks at what is in stackforge | 15:13 | |
| chmouel | anteaya: that's a stackforge project | 15:13 |
| *** mattoliverau has joined #openstack-infra | 15:13 | |
| anteaya | notmyname chmouel okay, I will review for accuracy of creating the repo and would be good to have your input on the patch from a "should this exist" point of view | 15:14 |
| *** virmitio has joined #openstack-infra | 15:15 | |
| *** e0ne has joined #openstack-infra | 15:15 | |
| notmyname | anteaya: to be very clear, it should not exist inside of swift. and I don't care if it exists in stackforge. that is, I support it's development in the community, but I don't have any opinions on where they should host their code | 15:15 |
| *** atiwari has joined #openstack-infra | 15:16 | |
| *** gondoi is now known as zz_gondoi | 15:16 | |
| *** changbl has joined #openstack-infra | 15:16 | |
| anteaya | notmyname: *nod | 15:16 |
| chmouel | anteaya: +1 to notmyname, i don't think we have the right to give opinion about "should this exist" on stackforge projects | 15:17 |
| chmouel | (althought this is made by my colleagues so obv involved with it) | 15:18 |
| *** lcheng has joined #openstack-infra | 15:18 | |
| *** zns has quit IRC | 15:18 | |
| anteaya | chmouel: okay, thanks for bringing me up to date | 15:19 |
| *** mattoliverau has quit IRC | 15:19 | |
| *** zns has joined #openstack-infra | 15:19 | |
| *** andreykurilin_ has quit IRC | 15:19 | |
| openstackgerrit | A change was merged to openstack-dev/specs-cookiecutter: Update readme to refer to specs-cookiecutter https://review.openstack.org/90451 | 15:19 |
| msabramo | zaro: I would love to hear your thoughts on https://review.openstack.org/90114 whenever you have the time | 15:20 |
| *** Ryan_Lane has joined #openstack-infra | 15:21 | |
| *** markwash has joined #openstack-infra | 15:22 | |
| *** wenlock has quit IRC | 15:22 | |
| *** gokrokve has joined #openstack-infra | 15:24 | |
| *** mattoliverau has joined #openstack-infra | 15:24 | |
| *** dripton has quit IRC | 15:25 | |
| openstackgerrit | Sean Dague proposed a change to openstack-infra/zuul: Implement Depends-On: IdempotentID https://review.openstack.org/94365 | 15:25 |
| sdague | jeblair: not quite passing existing tests, but it's closer, and at least blows up in a way that one can work through test issues | 15:26 |
| *** mestery has quit IRC | 15:26 | |
| *** dripton has joined #openstack-infra | 15:26 | |
| *** mestery has joined #openstack-infra | 15:26 | |
| rcarrillocruz | anteaya: ok, thx | 15:27 |
| *** atiwari has quit IRC | 15:28 | |
| jeblair | sdague: controlled detonation as a development model. i like it. :) | 15:28 |
| sdague | :) | 15:28 |
| *** cody-somerville has quit IRC | 15:29 | |
| sdague | so there is no way to make the thread explosions return to testr in the general case? | 15:29 |
| *** wenlock has joined #openstack-infra | 15:29 | |
| *** msabramo has quit IRC | 15:29 | |
| *** andreaf has joined #openstack-infra | 15:30 | |
| *** arnaud_ has joined #openstack-infra | 15:30 | |
| *** resker has joined #openstack-infra | 15:31 | |
| sdague | I also believe I integrated all your feedback in the jeepyb patch yesterday - https://review.openstack.org/#/c/94196/3 | 15:31 |
| sdague | as well as putting a dashboard support patch on top of it | 15:31 |
| openstackgerrit | Chmouel Boudjnah proposed a change to openstack-infra/config: Add swift specs repository https://review.openstack.org/94140 | 15:31 |
| fungi | phschwartz: anteaya: (sorry, was afk, skimmed just now) if i understand what you're asking, we probably want 1) a review to add the skeleton openstack-infra/vinz project (probably already exists--i'm behind on reviews), an infra spec for vinz (once the openstack-infra/infra-specs repo exists, if it doesn't already) and a storyboard project for vinz | 15:31 |
| *** atiwari has joined #openstack-infra | 15:33 | |
| jeblair | fungi: ++ | 15:33 |
| fungi | phschwartz: as for how to request gerrit groups associated with the vinz code review repo, requesting them in here, in an lp bug against openstack-ci or on the openstack-infra ml are all fine | 15:33 |
| *** esker has quit IRC | 15:34 | |
| jeblair | fungi: https://review.openstack.org/#/c/93929/ probably worth prioritizing that one | 15:34 |
| fungi | phschwartz: however, it works best to request them once the project is created, so that we don't forget to do it later ;) | 15:34 |
| phschwartz | fungi: awesome, that makes it easy. Yes, the review to add the skeleton is up. https://review.openstack.org/#/c/93953/, nothing for the spec yet. | 15:34 |
| *** arnaud_ has quit IRC | 15:34 | |
| fungi | jeblair: next on my agenda. thanks | 15:34 |
| *** mattoliverau has quit IRC | 15:35 | |
| phschwartz | fungi, jeblair, anteaya: I have some things todo internal post summit and once done with those, I will start on the spec. | 15:35 |
| anteaya | chmouel: no tests now for you? | 15:35 |
| anteaya | chmouel: I'm fine either way, just seems like a change of heart from yesterday | 15:36 |
| phschwartz | fungi: for note, the groups for gerrit will be vinz-webclient-core and vinz-ptl | 15:36 |
| jeblair | phschwartz: no problem; that gives us time to set up the specs repo. :) | 15:37 |
| james_li | clarkb, fungi: could you pls take a look at https://review.openstack.org/#/c/92655/ ? | 15:38 |
| *** hogepodge has joined #openstack-infra | 15:38 | |
| zaro | morning | 15:40 |
| *** afazekas has quit IRC | 15:40 | |
| phschwartz | zaro: Morning | 15:40 |
| *** dkliban is now known as dkliban_afk | 15:40 | |
| *** dprince has joined #openstack-infra | 15:41 | |
| *** rdopieralski has quit IRC | 15:41 | |
| *** zns has quit IRC | 15:41 | |
| *** UtahDave has joined #openstack-infra | 15:42 | |
| *** msabramo has joined #openstack-infra | 15:42 | |
| anteaya | zaro: morning | 15:42 |
| *** praneshp_ has joined #openstack-infra | 15:42 | |
| *** mattoliverau has joined #openstack-infra | 15:42 | |
| anteaya | zaro: rcarrillocruz has some questions about our timing around getting rid of gerrit drafts | 15:42 |
| anteaya | zaro: what information do you have on that? | 15:43 |
| *** mrodden has quit IRC | 15:43 | |
| jeblair | sdague: https://review.openstack.org/#/c/88289/ | 15:43 |
| sdague | jeblair: do we know the current pass rate of that job? | 15:44 |
| phschwartz | As part of my work internal here at Rax I created a set of ansible playbooks to automate creating the puppetmaster and launching nodes as needed. If there is interest in them, I can put them up as a WIP review for you guys to look at and if liked, we can work them into the config repo. | 15:44 |
| sdague | salv-orlando you feeling comfortable with that change? | 15:44 |
| zaro | rcarrillocruz, anteaya : actually the right word is 'disable' drafts in gerrit. will probably happen in by end of week. i think will will talk about it today in infra meeting. i would recommend to stop work on 1078231. | 15:46 |
| anteaya | zaro: awesome thank you | 15:47 |
| * anteaya notes the use of the word disable | 15:47 | |
| fungi | phschwartz: while i'm not opposed to ansible (and what brief introduction/demos i've had do make it seem more attractive a choice than puppet), i'd rather not consider integrating ansible into our current infrastructure unless we had some consensus that we want to start replacing puppet with it (which would be a massive undertaking). worth discussing however | 15:47 |
| anteaya | phschwartz: some folks have deployment tool files in the stackforge space | 15:48 |
| phschwartz | fungi: I am actually not looking at it to replace puppet for infra. But in the same manner we use it here at Rax to add orchestration outside of puppet as you see with things like the launch_node.py and other such scripts. | 15:48 |
| anteaya | chef cookbooks and such | 15:48 |
| *** atiwari has quit IRC | 15:48 | |
| rcarrillocruz | k, thx | 15:49 |
| anteaya | fungi: have you had a chance to look at my feedback on https://review.openstack.org/#/c/93065/1 | 15:49 |
| jeblair | fungi, phschwartz: i think mordred did a POC of using ansible for the ssh puppet trigger | 15:49 |
| anteaya | the normalize acl script patch | 15:49 |
| anteaya | fungi: I'm just wondering if you have any thoughts on my feedback | 15:50 |
| phschwartz | jeblair: That is basically what I did, lol. Should have asked as that would have saved some time. But I have been using it to replace launch_node.py (and created a playbook for automating the creation of the puppet-master.) | 15:50 |
| krotscheck | phschwartz: Reading scrollback right now. That makes some sense, though it might be a mod_proxy configuration flag that’s missing. | 15:52 |
| *** praneshp_ is now known as praneshp | 15:52 | |
| jeblair | jesusaurus: are you around to babysit 89708? | 15:53 |
| zaro | msabramo : sorry, i've been avoiding the python 3 stuff because openstack does not support it yet. however it looks like python 3 support is coming soon, so good time to dive in. will give it a look soon. | 15:53 |
| msabramo | zaro: no problem | 15:53 |
| *** gyee has joined #openstack-infra | 15:54 | |
| *** atiwari has joined #openstack-infra | 15:54 | |
| msabramo | zaro: Victor Stinner seems to be pushing a lot for more Python 3 support in openstack | 15:54 |
| *** e0ne has quit IRC | 15:57 | |
| *** nati_ueno has joined #openstack-infra | 15:58 | |
| *** yjiang5 has quit IRC | 15:58 | |
| clarkb | o/ | 15:59 |
| clarkb | morning | 15:59 |
| anteaya | hey clarkb | 15:59 |
| *** amcrn has joined #openstack-infra | 16:00 | |
| openstackgerrit | Khai Do proposed a change to openstack-infra/config: update gerrit app on review-dev.o.o to same version as on review.o.o https://review.openstack.org/94413 | 16:00 |
| james_li | clarkb: hey morning | 16:00 |
| *** hashar is now known as hasharConfCall | 16:01 | |
| *** marcoemorais has joined #openstack-infra | 16:01 | |
| *** cody-somerville has joined #openstack-infra | 16:01 | |
| jeblair | nice, i have 70% packet loss to review over v6 | 16:02 |
| jeblair | seems to be he.net | 16:03 |
| jeblair | i noticed that because gertty keeps dropping into offline mode. | 16:03 |
| anteaya | gertty works over ipv6? | 16:04 |
| clarkb | jeblair: is there a specific proposed gertty change you would recommend I install? it looks like you branched the change tree | 16:05 |
| *** mattoliverau has quit IRC | 16:05 | |
| *** ominakov has quit IRC | 16:06 | |
| jeblair | i did? huh, lemme see | 16:06 |
| chmouel | anteaya: it was a discussion we had on #openstack-swift and I lost :) | 16:06 |
| jeblair | clarkb: no, https://review.openstack.org/#/c/94097/ is the tip | 16:07 |
| jeblair | clarkb: https://review.openstack.org/#/c/92747/ has a branch, but one of them is abandoned | 16:07 |
| *** hipster_ has joined #openstack-infra | 16:07 | |
| jeblair | clarkb: so it should be a straight line from HEAD to 94097 | 16:08 |
| clarkb | thanks | 16:08 |
| *** dkliban_afk is now known as dkliban | 16:08 | |
| *** thomasbiege has joined #openstack-infra | 16:08 | |
| chmouel | anteaya: i mean lost to convince that tests in specs are useful :) | 16:08 |
| *** nati_ueno has quit IRC | 16:09 | |
| anteaya | chmouel: :( I think tests in specs are useful | 16:10 |
| anteaya | chmouel: I have +1'd the patches you offered since they will create the repo, but up to swift to decide on tests | 16:10 |
| anteaya | I don't think we have had a general conversation on tests for spec repos | 16:11 |
| jeblair | there's one on the -dev list now. | 16:11 |
| *** hipster has quit IRC | 16:11 | |
| openstackgerrit | Sean Dague proposed a change to openstack-infra/zuul: Implement Depends-On: IdempotentID https://review.openstack.org/94365 | 16:11 |
| sdague | ok, going to get out and ride for a bit, will try to stare a zuul some more later | 16:12 |
| jeblair | i think they should all have the -docs gate job, at a minimum. i don't have an opinion currently beyond that. | 16:12 |
| zaro | jeblair: tried to release jjb today, but was not able to because i don't have permission in gerrit. | 16:12 |
| boris-42 | jeblair hi there | 16:12 |
| *** danpb has left #openstack-infra | 16:12 | |
| anteaya | chmouel: http://lists.openstack.org/pipermail/openstack-dev/2014-May/035256.html | 16:12 |
| zaro | jeblair: need to do 2nd part of this https://review.openstack.org/#/c/56823/ | 16:12 |
| anteaya | sdague: enjoy the bike ride | 16:12 |
| zaro | jeblair: you think you can help with that? | 16:12 |
| boris-42 | jeblair could you remove -1 pls from here https://review.openstack.org/#/c/90402/ ? | 16:13 |
| pleia2 | good morning | 16:13 |
| anteaya | chmouel: so you have that thread and jeblair's opinion that specs should at least have a docs gate job | 16:13 |
| zaro | pleia2: morning | 16:13 |
| anteaya | chmouel: try again | 16:13 |
| jeblair | zaro: done | 16:13 |
| anteaya | morning pleia2 | 16:13 |
| jeblair | zaro: who else did we want to add? | 16:13 |
| *** otherwiseguy has joined #openstack-infra | 16:13 | |
| *** pelix has quit IRC | 16:14 | |
| *** jgrimm has quit IRC | 16:14 | |
| *** pelix has joined #openstack-infra | 16:14 | |
| chmouel | anteaya: any chances that you can mention it on the review so other can coment there? | 16:14 |
| anteaya | I will do so now | 16:14 |
| zaro | jeblair: not sure atm. | 16:14 |
| jeblair | boris-42: done | 16:15 |
| openstackgerrit | A change was merged to openstack-infra/config: Set KEEP_LOCALRC for enabling CEDARISH in solum https://review.openstack.org/92655 | 16:15 |
| *** changbl has quit IRC | 16:15 | |
| boris-42 | jeblair thank you | 16:15 |
| *** mattoliverau has joined #openstack-infra | 16:16 | |
| fungi | boris-42: i removed mordred's -1 workflow (wip) too | 16:17 |
| boris-42 | fungi thank you=) | 16:17 |
| anteaya | chmouel: let me know if you need more | 16:17 |
| fungi | since it looked like he was setting it to wip for the same reason | 16:17 |
| boris-42 | fungi yep | 16:17 |
| chmouel | anteaya: cool tks | 16:17 |
| boris-42 | fungi because of John's -1 | 16:17 |
| anteaya | chmouel: go get some tests | 16:17 |
| *** yamahata has quit IRC | 16:18 | |
| zaro | rcarrillocruz: hey, since your doing the LP update, would you like to take ownership of this issue? https://bugs.launchpad.net/openstack-ci/+bug/1011705 | 16:19 |
| uvirtbot | Launchpad bug 1011705 in openstack-ci "update_bug should use review owner instead of latest commiter" [Low,Confirmed] | 16:19 |
| rcarrillocruz | sure i can | 16:20 |
| rcarrillocruz | done | 16:20 |
| rcarrillocruz | jeblair , fungi, clarkb: hello there. if you have some time, could you please review https://review.openstack.org/#/c/93773/ ? | 16:21 |
| openstackgerrit | A change was merged to openstack-infra/config: Create an infra-specs repo https://review.openstack.org/93929 | 16:21 |
| *** zz_gondoi is now known as gondoi | 16:23 | |
| *** mkerrin has quit IRC | 16:23 | |
| jeblair | rcarrillocruz: my review queue is 135 items long. your change is at position #58. please feel free to review some infra changes if you have time. | 16:24 |
| *** MarkAtwood has joined #openstack-infra | 16:24 | |
| jeblair | dprince: are you up for creating a puppet-yum module/repo as suggested by jesusaurus in 90874 ? | 16:27 |
| dprince | jeblair: yes, I'm happy to make that change | 16:28 |
| rcarrillocruz | oops, sorry :/ | 16:28 |
| jeblair | dprince: cool, thx | 16:28 |
| zaro | jeblair: just made a jjb release, do you happen to know why the overview info is missing in pypi? https://pypi.python.org/pypi/jenkins-job-builder/0.8.0 | 16:29 |
| zaro | jeblair: ver 0.7.0 has the info. | 16:30 |
| pleia2 | dprince: I spoke to morganfainberg about this yesterday, any objections? | 16:30 |
| *** rwsu has joined #openstack-infra | 16:30 | |
| pleia2 | dprince: https://review.openstack.org/#/c/94266/ | 16:30 |
| *** morganfainberg_Z is now known as morganfainberg | 16:30 | |
| *** hogepodge_ has joined #openstack-infra | 16:30 | |
| *** arnaud_ has joined #openstack-infra | 16:31 | |
| dprince | pleia2: I have mixed feelings I suppose. Some people do like to see WIPs I think. | 16:31 |
| *** hogepodge has quit IRC | 16:31 | |
| *** hogepodge_ is now known as hogepodge | 16:31 | |
| zaro | are we no longer mirroring to github? | 16:32 |
| dprince | pleia2: I'm probably okay removing them for now if they are problematic though... | 16:32 |
| fungi | zaro: we are trying, but sometimes project setup there breaks in jeepyb due to api throttling | 16:32 |
| fungi | zaro: did you spot one which broke/. | 16:32 |
| fungi | ? | 16:32 |
| pleia2 | dprince: not problematic really, just the reason for reviewday seems to be to find important reviews to focus on, and WIP doesn't seem like it should be there :) | 16:32 |
| *** hasharConfCall has quit IRC | 16:32 | |
| fungi | Alex_Gaynor: fyi, the only nodepool region which failed to build a fresh precise-py3k image a couple hours ago was hpcloud-az2 | 16:33 |
| zaro | fungi: ohh forgot i was looking at github.org/openstack-ci instead of openstack-infra | 16:33 |
| dprince | pleia2: yeah. well the idea was they should always be ranked near the bottom. Fair enough though, not showing them is probably fine | 16:33 |
| Alex_Gaynor | fungi: so do we kick that builder again, or just wait until it runs automatically? | 16:33 |
| * dprince has trouble finding WIPs sometimes though | 16:33 | |
| * dprince although the big red X should help! | 16:34 | |
| pleia2 | :) | 16:34 |
| *** otherwiseguy has quit IRC | 16:34 | |
| jeblair | zaro: i don't know off hand why it's missing; you may want to check the release jobs and see if there's anything there. | 16:34 |
| dprince | pleia2: we missed you in Atlanta BTW | 16:34 |
| *** lcostantino has quit IRC | 16:34 | |
| zaro | fungi, jeblair : i think the info on pypi still points to github.com/openstack-ci instead of openstack-infra and maybe that's why no overview info? | 16:34 |
| pleia2 | dprince: thanks, missed you all too! | 16:34 |
| jeblair | zaro: no, that's built from the setup.py/setup.cfg files | 16:35 |
| jeblair | fungi: http://logs.openstack.org/0a71ef9/post/reddwarf-coverage/6/cover/ should all have been deleted, right? | 16:35 |
| *** openstackgerrit has quit IRC | 16:35 | |
| *** arnaud_ has quit IRC | 16:35 | |
| *** openstackgerrit has joined #openstack-infra | 16:36 | |
| zaro | jeblair: ahh thanks. i think hashar made a change to that recently. | 16:37 |
| *** jpich has quit IRC | 16:37 | |
| *** jpich has joined #openstack-infra | 16:37 | |
| *** aconrad has joined #openstack-infra | 16:37 | |
| *** vtapia has quit IRC | 16:38 | |
| fungi | Alex_Gaynor: i've started a while loop over rebuilding the image in that region, so hopefully we'll get a fresh one some time in the next day or so if we're lucky (the current most recent image there is over a month old) | 16:38 |
| *** _nadya_ has quit IRC | 16:39 | |
| *** _nadya_ has joined #openstack-infra | 16:39 | |
| fungi | jeblair: yes, those are hella-old. i'll see if i can spot why they stuck around | 16:39 |
| *** Ryan_Lane has quit IRC | 16:39 | |
| openstackgerrit | A change was merged to openstack-infra/storyboard-webclient: Added resolveSessionState method https://review.openstack.org/92639 | 16:40 |
| openstackgerrit | A change was merged to openstack-infra/storyboard-webclient: Created new Dashboard module https://review.openstack.org/92640 | 16:40 |
| openstackgerrit | A change was merged to openstack-infra/storyboard-webclient: Removed new story button on index https://review.openstack.org/92645 | 16:40 |
| *** jp_at_hp has quit IRC | 16:41 | |
| anteaya | krotscheck phschwartz http://git.openstack.org/cgit/openstack-infra/infra-specs/ | 16:41 |
| *** jp_at_hp has joined #openstack-infra | 16:41 | |
| jeblair | anteaya: we need to populate it with a template first | 16:41 |
| jeblair | i will do that | 16:41 |
| *** ihrachyshka has joined #openstack-infra | 16:41 | |
| krotscheck | YAAAY | 16:42 |
| *** BobBall is now known as BobBallAway | 16:43 | |
| *** pelix has quit IRC | 16:43 | |
| Alex_Gaynor | fungi: fun times | 16:43 |
| *** mattoliverau has quit IRC | 16:44 | |
| anteaya | great, since I don't know what you are looking for in a template | 16:44 |
| zaro | jeblair: "home page" section for jjb on pypi still points to github.com/openstack-ci. This doesn't look like it comes from setup.cfg. is that an account config setting? | 16:44 |
| jeblair | zaro: it all comes from the setup file | 16:45 |
| openstackgerrit | A change was merged to openstack-infra/storyboard-webclient: Ajust tox.ini https://review.openstack.org/93405 | 16:46 |
| anteaya | I can see things now in gertty | 16:46 |
| anteaya | by things I mean diffs, yay | 16:46 |
| *** jgallard has quit IRC | 16:46 | |
| *** aconrad has quit IRC | 16:48 | |
| Alex_Gaynor | fungi: ok, so the CR I sent is blocked until that's resolved? | 16:48 |
| zaro | jeblair: yeah, i see it now. | 16:48 |
| krotscheck | What’s the appropriate way to handle “Hey we got 2 x +2 on this but the patch failed because a different merge included an alembic migration with the same # so I had to rename a file to make the patch clear”? | 16:48 |
| krotscheck | Approve it, or let it go through another round of review? | 16:49 |
| krotscheck | (91675) | 16:49 |
| zaro | krotscheck: usually another round. | 16:49 |
| krotscheck | zaro: Got it | 16:49 |
| krotscheck | tx | 16:50 |
| *** mattoliverau has joined #openstack-infra | 16:50 | |
| anteaya | I think I have time to get a walk in before meeting season starts | 16:50 |
| anteaya | back in about an hour | 16:50 |
| pleia2 | enjoy | 16:50 |
| anteaya | thanks | 16:50 |
| *** sarob has joined #openstack-infra | 16:50 | |
| fungi | Alex_Gaynor: yeah, for safety reasons, we really don't want jobs breaking at random depending on where they run | 16:51 |
| Alex_Gaynor | fungi: that's reasonable :-) | 16:51 |
| *** otherwiseguy has joined #openstack-infra | 16:52 | |
| clarkb | the logstash queue has taken off :( | 16:52 |
| *** mkerrin has joined #openstack-infra | 16:52 | |
| *** andreykurilin_ has joined #openstack-infra | 16:53 | |
| *** Ryan_Lane has joined #openstack-infra | 16:55 | |
| *** Hal_ has joined #openstack-infra | 16:55 | |
| *** mattoliverau has quit IRC | 16:56 | |
| clarkb | seems to be falling now. I have no idea what happened there. maybe just post summit rush? | 16:56 |
| clarkb | I will keep an eye on it today I guess | 16:57 |
| clarkb | (everything looks healthy) | 16:57 |
| zaro | does anyone know how the overview page on pypi works? | 16:58 |
| fungi | zaro: its contents are built from the data in a project's setup.cfg file | 16:58 |
| zaro | does it read the README.rst from a project then just display it? | 16:58 |
| *** mattoliverau has joined #openstack-infra | 16:58 | |
| *** markmcclain has joined #openstack-infra | 16:59 | |
| jesusaurus | jeblair: yeah, im around to babysit now. i will want to manually rebase before it's approved to make sure that we don't lose any slave_scripts that might have been added in the past week | 16:59 |
| jeblair | jesusaurus: okay, why don't you go ahead and do that and let me know when you're ready | 16:59 |
| zaro | fungi: what is the result of the setup.cfg? can i look at that resulting file from a build? | 16:59 |
| *** pelix has joined #openstack-infra | 16:59 | |
| fungi | zaro: no, it's a file in the project's git repository | 17:00 |
| fungi | zaro: such as http://git.openstack.org/cgit/openstack-infra/jenkins-job-builder/tree/setup.cfg | 17:00 |
| fungi | zaro: if i understand from scrollback, you want to adjust the metadata.home-page parameter there to no longer be https://github.com/openstack-ci/jenkins-job-builder | 17:01 |
| *** gondoi is now known as zz_gondoi | 17:02 | |
| zaro | fungi: so what about the overview (or stuff from README.rst) here https://pypi.python.org/pypi/jenkins-job-builder/0.7.0 | 17:02 |
| *** bhuvan has joined #openstack-infra | 17:02 | |
| fungi | zaro: so you'd just submit a change in gerrit to alter that line, and then it will be correct on pypi the next time jenkins-job-builder tags a release | 17:02 |
| *** lcheng has quit IRC | 17:02 | |
| *** zz_gondoi is now known as gondoi | 17:02 | |
| *** sarob has quit IRC | 17:02 | |
| *** sarob has joined #openstack-infra | 17:03 | |
| *** resker has quit IRC | 17:03 | |
| *** gondoi is now known as zz_gondoi | 17:03 | |
| fungi | zaro: yes, it also embeds the contents of README.rst there any time a new version of the package is released | 17:03 |
| *** esker has joined #openstack-infra | 17:04 | |
| *** amcrn has quit IRC | 17:04 | |
| *** sarob has quit IRC | 17:04 | |
| zaro | fungi: ok. do you happen to know why README.rst is missing in 0.8.0? that probably has nothing to do with setup.cfg, does it? | 17:04 |
| *** sarob has joined #openstack-infra | 17:05 | |
| zaro | fungi: build job looks ok to me https://jenkins.openstack.org/job/jenkins-job-builder-pypi-upload/1/console | 17:06 |
| clarkb | jeblair: https://review.openstack.org/#/c/92753/2/gertty/sync.py has onion layers of db session context management. Does sqlite handle that ok? I think it should because it is multi thread that it fails at while this is still single thread | 17:06 |
| *** atiwari has quit IRC | 17:07 | |
| fungi | zaro: in 0.7.0 the pypi metadata seems to have been included directly in setup.py instead of setup.cfg, so maybe something got missed in the change which ported it over? | 17:08 |
| jeblair | clarkb: where are the layers? | 17:08 |
| jeblair | clarkb: i only see one session block | 17:08 |
| zaro | fungi: ahh, ok. let me check that. | 17:08 |
| *** zns has joined #openstack-infra | 17:08 | |
| * eglynn doesn't want to be a pest | 17:08 | |
| eglynn | ... buuut | 17:08 |
| eglynn | ... it would be great to get the ceilo-specs patch landed | 17:09 |
| fungi | zaro: looks like it's missing metadata.description-file like http://git.openstack.org/cgit/openstack-infra/zuul/tree/setup.cfg#n4 has | 17:09 |
| clarkb | jeblair: SyncChangeTask has a with on line 183 and it dispatches a SyncChangebyCommitTask with on 163 | 17:09 |
| openstackgerrit | K Jonathan Harker proposed a change to openstack-infra/config: Move openstack-specific config out of ::jenkins https://review.openstack.org/89708 | 17:09 |
| dstufft | fungi to be specific, if there is no long descriptionn pypi will attempt to read a readme.rst | 17:09 |
| eglynn | ... so that we can start churning out detailed BP specs while the goings-on at summit are still fresh | 17:09 |
| eglynn | https://review.openstack.org/94044 | 17:09 |
| dstufft | and probably some other files | 17:09 |
| dstufft | to discover one | 17:09 |
| *** jcoufal has joined #openstack-infra | 17:09 | |
| fungi | dstufft: good to know--thanks! | 17:09 |
| *** _nadya_ has quit IRC | 17:09 | |
| jeblair | eglynn: it will be a few more hours at a mininum, but we'll try to land those asap | 17:09 |
| eglynn | jeblair: excellent! thank you sir! | 17:09 |
| fungi | zaro: i suspect the addition of metadata.description masked it from trying to read the README.rst file | 17:10 |
| jeblair | i'd like to bring up the question of repo naming conventions at the project meeting | 17:10 |
| *** rlandy has quit IRC | 17:10 | |
| jeblair | clarkb: that just enqueues a task, it doesn't actually run it | 17:10 |
| dstufft | fungi: dunno if it's useful for y'all or not, but sooner or later there wil lbe a lib to verify that pypi can render your description | 17:10 |
| jeblair | clarkb: (that == submitTask) | 17:11 |
| clarkb | jeblair: does something prevent a context switch to running it right there? does the dispatcher run in the same thread? | 17:11 |
| fungi | dstufft: that would be nice. i can imagine us adding a gate job for projects configured to release to pypi | 17:11 |
| dstufft | (sorry, my typing is bad i have a brace on my hand) | 17:11 |
| jeblair | clarkb: yeah, tasks aren't threads -- there is a single thread that runs each task in series | 17:11 |
| dstufft | fungi: github.com/pypa/readme is how warehouse renders, which is slighly diff than pypi. but it's already there for warehouse... just need to make a cli interface | 17:12 |
| clarkb | jeblair: got it thanks | 17:12 |
| fungi | dstufft: similarly, i'd like to add a gate job which would catch things like being unable to build a tarball, since we get a lot of false failures which turn out to be incomplete release configuration within a given project source tree, so i can see adding checks like that to the same job | 17:12 |
| dstufft | fungi: you might like this project, one sec | 17:12 |
| jeblair | fungi: we build a tarball for every test | 17:13 |
| jeblair | that's one of the reasons we switched to tox | 17:13 |
| dstufft | fungi: https://warehouse.python.org/project/check-manifest/ | 17:13 |
| clarkb | new tox worked with zuul btw \o/ | 17:13 |
| clarkb | so once there is a new tox release we can upgrade to it | 17:14 |
| jeblair | clarkb: yay | 17:14 |
| fungi | dstufft: awesome! taking notes | 17:14 |
| dstufft | fungi: i use check-manifest in warehouse's tests | 17:14 |
| dstufft | it's pretty ok | 17:14 |
| dstufft | https://github.com/pypa/warehouse/blob/master/tox.ini#L25-L28 | 17:14 |
| dstufft | saves you from the mistake of forgetting to add some package data | 17:14 |
| *** dizquierdo has quit IRC | 17:15 | |
| fungi | dstufft: agreed, that would be helpful. does it also catch when people mistype or make up nonexistent trove classifiers which the pypi upload interface is going to reject? | 17:15 |
| pleia2 | jeblair: was there time at the summit to talk about debian&fedora mirrors? (saw your update to one of the fedora patches re: putting it in a separate module) | 17:15 |
| *** arnaud_ has joined #openstack-infra | 17:15 | |
| *** harlowja_away is now known as harlowja | 17:16 | |
| *** zz_gondoi is now known as gondoi | 17:16 | |
| fungi | pleia2: yeah, we chatted with some of the tripleo folk in the dev lounge about it | 17:16 |
| dstufft | fungi: no | 17:17 |
| dstufft | fungi: sounds like a use thing to make tho | 17:17 |
| pleia2 | fungi: I have a debian patch+bug hanging out there, so I'll need to get up to speed at some point | 17:17 |
| krotscheck | Design question on OAuth - should refresh_tokens expire? | 17:17 |
| fungi | dstufft: yeah, that's been one of the other common opaque failure modes for our release automation... inability to guess what pypi will reject | 17:17 |
| * krotscheck is pondering 94363 | 17:18 | |
| dstufft | fungi: some day pypi will reject invalid markup | 17:18 |
| jeblair | morganfainberg: thoughts on krotscheck's question? | 17:18 |
| fungi | pleia2: if memory serves, the rough consensus was that infra would like puppetry to build package mirrors for platforms on which we run gate jobs, since we may also want to embed some in our various providers/regions/zones | 17:18 |
| morganfainberg | jeblair, looking at the qeustion. | 17:19 |
| fungi | pleia2: but if tripleo plans to run their own outside of infra systems management, then it should go in a separate tenant at a minimum | 17:19 |
| pleia2 | fungi: ok, thanks | 17:20 |
| morganfainberg | krotscheck, what would the impact be if a refresh_token expires? need to log in again? | 17:20 |
| krotscheck | morganfainberg: Well, it would set two expiration dates - expiration of the access_token and expiration of the refresh_token some time after that. | 17:20 |
| krotscheck | morganfainberg: User impact would be that they’d have to log in again. | 17:20 |
| krotscheck | morganfainberg: Dev impact would be that we’d have to handle an additional edge case. | 17:21 |
| *** aconrad has joined #openstack-infra | 17:21 | |
| krotscheck | morganfainberg: Security impact would be… dunno, that’s why I’m asking. | 17:21 |
| *** atiwari has joined #openstack-infra | 17:21 | |
| morganfainberg | krotscheck, right. | 17:22 |
| *** changbl has joined #openstack-infra | 17:22 | |
| morganfainberg | krotscheck, from a pure security standpoint, yes refresh_token should expire. Otherwise it would be possible to re-use something waaaaaay down the line | 17:22 |
| *** jgrimm has joined #openstack-infra | 17:23 | |
| phschwartz | morganfainberg: Agree to that completely | 17:23 |
| phschwartz | anteaya: perfect, I have a place to put the spec when it is ready | 17:24 |
| krotscheck | morganfainberg: kk, that changes a few comments on my code review. | 17:24 |
| phschwartz | do we have a specific format for the specs for infra or is copying the nova format good to start | 17:24 |
| *** nati_ueno has joined #openstack-infra | 17:24 | |
| jeblair | phschwartz: working on it now | 17:25 |
| *** amcrn has joined #openstack-infra | 17:25 | |
| jeblair | phschwartz: it will be different | 17:25 |
| *** crc32 has joined #openstack-infra | 17:25 | |
| jeblair | anteaya: did infra-specs get use-storyboard:True? | 17:26 |
| openstackgerrit | Khai Do proposed a change to openstack-infra/jenkins-job-builder: update URL references to project. https://review.openstack.org/94430 | 17:26 |
| phschwartz | jeblair: awesome, I will keep an eye out. | 17:26 |
| *** eglynn has quit IRC | 17:26 | |
| morganfainberg | krotscheck, cool. i am not sure on what a sane refresh period is (might be in the spec) | 17:26 |
| *** jpich has quit IRC | 17:27 | |
| *** mriedem has quit IRC | 17:27 | |
| openstackgerrit | James E. Blair proposed a change to openstack-infra/config: Use storyboard in infra-specs https://review.openstack.org/94431 | 17:29 |
| *** gondoi is now known as zz_gondoi | 17:29 | |
| jeblair | clarkb, fungi: ^ speedy approve? | 17:29 |
| *** james_li has quit IRC | 17:30 | |
| clarkb | speedy approved | 17:31 |
| *** nati_ueno has quit IRC | 17:31 | |
| jesusaurus | is there a known ongoing issue with nodepool? I'm just noticing a "Looks like the node went offline during the build." failure from half an hour ago | 17:32 |
| fungi | oops, double-approved ;) | 17:32 |
| *** james_li has joined #openstack-infra | 17:32 | |
| openstackgerrit | Joshua Harlow proposed a change to openstack-infra/config: Add a doc8 stackforge project https://review.openstack.org/94236 | 17:32 |
| jeblair | jesusaurus: was the test reported to gerrit, or was it one that zuul aborted? | 17:32 |
| jesusaurus | no, zuul reported as FAILURE | 17:33 |
| jesusaurus | patch 7 on https://review.openstack.org/#/c/89708/ | 17:34 |
| *** igor has quit IRC | 17:34 | |
| *** changbl has quit IRC | 17:35 | |
| *** igor_ has joined #openstack-infra | 17:35 | |
| fungi | and that one was hpcloud-az2 again | 17:36 |
| *** dangers_away is now known as dangers | 17:36 | |
| *** Hal_ has quit IRC | 17:36 | |
| clarkb | ya last I checked az2 was the only one exhibiting that symptom | 17:37 |
| clarkb | you can ask logstash for that info | 17:37 |
| fungi | it really seems like we see a disproportionately higher ratio of these sorts of failures there vs overall quota | 17:37 |
| *** melwitt has joined #openstack-infra | 17:37 | |
| jeblair | yeah, that's starting to look like jenkins marked it offline, probably due to a loss of connectivity | 17:37 |
| jeblair | nodepool logs look normal | 17:37 |
| jeblair | resolution: stop using az2 | 17:38 |
| *** mriedem has joined #openstack-infra | 17:38 | |
| *** zns has quit IRC | 17:38 | |
| *** melwitt has quit IRC | 17:38 | |
| *** melwitt has joined #openstack-infra | 17:38 | |
| jesusaurus | logstash shows that my quoted error above occurred 3 times in the past 4 hours. 2 of them were in az2, the other was hpcloud-az1 | 17:40 |
| *** _nadya_ has joined #openstack-infra | 17:40 | |
| fungi | jesusaurus: clarkb: i think it may be hard to differentiate those from aborted/deleted runs when only analyzing the console log? | 17:41 |
| openstackgerrit | Khai Do proposed a change to openstack-infra/jenkins-job-builder: add overview info to pypi https://review.openstack.org/94434 | 17:41 |
| *** timrc is now known as timrc-afk | 17:41 | |
| fungi | jesusaurus: clarkb: or do they always present distinctly? | 17:41 |
| phschwartz | Who ever looks at https://review.openstack.org/#/c/93953/ hold off on approving it please. | 17:42 |
| clarkb | fungi: I think you are correct. Specifically it depends on when in the run it happens | 17:42 |
| fungi | phschwartz: mark it work in progress (workflow -1) | 17:42 |
| clarkb | fungi: I think that a job that runs past normal jenkins inity stuff says something along the lines of node was marked offline and that is dstinct | 17:42 |
| clarkb | but if it happens early in the job run it is a dice roll on message | 17:42 |
| jesusaurus | fungi: i think that logstash will have build_status:"FAILURE" vs build_status:"ABORTED", but I only see FAILURE in my results | 17:42 |
| phschwartz | fungi: Was doing that as I messaged the channel. | 17:42 |
| *** krtaylor has quit IRC | 17:42 | |
| fungi | phschwartz: party on | 17:42 |
| *** wenlock has quit IRC | 17:43 | |
| zaro | fungi: thanks for the help. what do you suggest i do with release 0.8.0 that has already gone out to pypi? it doesn't show README info and it has references to github.com/openstack-ci | 17:43 |
| phschwartz | Need to work out how things need to be done for things like auth. Do we share with storyboard or move out to a common oauth endpoint some where, etc.. | 17:43 |
| *** timrc-afk is now known as timrc | 17:43 | |
| *** reed has joined #openstack-infra | 17:43 | |
| zaro | fungi: have create 2 patches to fix it, not sure if i should make another release to fix that up? | 17:44 |
| *** jcoufal has quit IRC | 17:44 | |
| fungi | zaro: if it's urgent then a 0.8.1 release, otherwise i'd just leave it that way until 0.9.0 or whatever | 17:44 |
| *** maxbit has quit IRC | 17:45 | |
| mordred_ | phschwartz: if we ran a keystone, we could register storyboard and vinz in it as services... | 17:45 |
| *** Ajaeger1 has joined #openstack-infra | 17:45 | |
| clarkb | a wild mordred has appeared | 17:45 |
| clarkb | mordred_: lifeless has pbr version semver questions for you | 17:45 |
| fungi | phschwartz: probably openstack-infra/openstackid (soon to be hosted at openstackid.org) | 17:45 |
| clarkb | mordred_: do you have scrollback from sunday? | 17:45 |
| phschwartz | mordred_: would we prefer to do that over oauth directly? | 17:46 |
| * mordred_ is in a car right now | 17:46 | |
| harlowja | qq guys, for the irc review notification bot, is there anyway to make it not publish notifications for WIP reviews? | 17:46 |
| mordred_ | phschwartz: unsure. just pondering | 17:46 |
| fungi | harlowja: probably not easily, since the patchset is uploaded before someone will set it to wip | 17:46 |
| zaro | fungi: i prefer 0.8.1, would it be possible to fast track these to do 0.8.1 release? https://review.openstack.org/#/c/94430/ and https://review.openstack.org/#/c/94434/ | 17:46 |
| mordred_ | clarkb: have scroll back, haven't fully digested it yet | 17:46 |
| harlowja | fungi hmmm | 17:46 |
| phschwartz | fungi: That is my thoughts, but I want to share code with storyboard as much as possible and their oauth is controlled from a rest endpoint in the storyboard api server. I am thinking breaking that out into a separate rest server and plugging things pertaining to projects like storyboard, vinz, etc into it might be a good idea. | 17:47 |
| jesusaurus | jeblair: clarkb: 89708 is rebased and ready | 17:47 |
| fungi | harlowja: we'd likely have to introduce a delay to see whether it's set wip by someone before a specified timeout | 17:47 |
| krotscheck | So, wait. | 17:47 |
| harlowja | fungi or if its -2 would that accomplish the same (except it wouldn't be lost) | 17:47 |
| harlowja | *the WIP wouldn't be lost | 17:47 |
| mordred_ | phschwartz: also, you could just make it a thing that both programs could import ... like a pecan plugin or something | 17:48 |
| phschwartz | That also brings up the need no matter what for a vinz project as well as the vinz-webclient to provide the backend requirements for oauth token storage and such that can't be done from the webclient directly. | 17:48 |
| krotscheck | An ‘other’ service would start issuing API auth tokens for storyboard, and storyboard would have to check that ‘other’ service to see whether a token is valid? | 17:48 |
| fungi | phschwartz: i believe the upshot of the storyboard design session was that its eventual openid endpoint would point to openstackid | 17:48 |
| krotscheck | On every API request? | 17:48 |
| krotscheck | I like pecan plugins. | 17:48 |
| dstufft | mordred_: you may (or may not) find this useful/interesting - https://github.com/dstufft/packaging/pull/1 | 17:49 |
| phschwartz | krotscheck: very good point, that would be horrific. | 17:49 |
| phschwartz | krotscheck: I also like the idea of doing it as a pecan plugin | 17:49 |
| *** pelix has quit IRC | 17:49 | |
| mordred_ | dstufft: nice | 17:49 |
| mordred_ | phschwartz, krotscheck : may need to sort right way to do token storage in a plugin ... | 17:50 |
| phschwartz | mordred_: agreed, but I think that is the best way to go forward with that. | 17:50 |
| dstufft | mordred_: trying decide if I want the "suggest" thing or not :/ 95.26% compatiblity w/ pypi w/o it... so fa I have up to 98.78% compat but it's starting to feel real janky :/ | 17:50 |
| *** andreaf has quit IRC | 17:51 | |
| *** markmcclain has quit IRC | 17:51 | |
| *** _nadya_ has quit IRC | 17:51 | |
| krotscheck | FYI, devananda has expressed interest for a common pecan auth plugin, but it would need to be configurable for keystone as well as openid. | 17:51 |
| fungi | phschwartz: oh, ignore me. i thought you were talking about just the external authentication itself, not how it gets consumed | 17:52 |
| *** gokrokve has quit IRC | 17:52 | |
| *** markmcclain has joined #openstack-infra | 17:52 | |
| krotscheck | Although devananda was drunk at the time, so I may be putting words in his mouth. | 17:52 |
| phschwartz | fungi: I figured as much. But I think the goal is to move from LP oauth to openstackid oauth no matter what. | 17:52 |
| fungi | phschwartz: agreed | 17:52 |
| openstackgerrit | A change was merged to openstack-infra/config: Use storyboard in infra-specs https://review.openstack.org/94431 | 17:53 |
| *** gokrokve has joined #openstack-infra | 17:54 | |
| *** changbl has joined #openstack-infra | 17:54 | |
| krotscheck | Incidentally, ttx pointed out that storyboard got some press this week: http://www.eweek.com/cloud/openstack-building-storyboard-for-the-software-defined-economy.html | 17:54 |
| mordred_ | wow | 17:54 |
| clarkb | jeblair: no module named alembic. I think there is a missing dependency somewhere | 17:55 |
| jeblair | clarkb: yep | 17:55 |
| clarkb | adding it to my venv gets past that error | 17:55 |
| clarkb | now having issues with somethin related to palettes | 17:55 |
| *** Hal_ has joined #openstack-infra | 17:56 | |
| zaro | clarkb, fungi, jeblair, mordred_ : puppet agent isn't running on review-dev.o.o. you have any problems with me starting the service? | 17:56 |
| *** andreykurilin_ has quit IRC | 17:56 | |
| mordred_ | zaro: we don't use agent anymore | 17:56 |
| zaro | same with jenkins-dev. | 17:56 |
| fungi | zaro: we don't run puppet-agent as a daemon any longer | 17:56 |
| fungi | zaro: it gets runs triggered directly via ssh from the puppet master | 17:57 |
| clarkb | jeblair: looks like palettes are required now because we iterate over them | 17:57 |
| zaro | i htought that's what allow it to report ot puppetboard? | 17:57 |
| fungi | zaro: it reports to puppetboard *when* the puppet master triggers an agent run on it | 17:57 |
| jeblair | clarkb: that's a bug; there should be sane defaults | 17:58 |
| clarkb | jeblair: ok do you mind if I test a fix and push an update to your change? | 17:58 |
| jeblair | did anyone fix infra docs publishing? | 17:58 |
| jeblair | clarkb: please and thanks | 17:58 |
| clarkb | jeblair: yes I think i fixed docs publishing | 17:58 |
| clarkb | for infra | 17:58 |
| zaro | fungi: ohh, looks like jenkins-dev and review-dev are no longer communicating to puppetboard. | 17:58 |
| *** ramashri has joined #openstack-infra | 17:58 | |
| krotscheck | phschwartz: I’m going to be focused on getting storyboard up to infra-spec in the near future, but a common auth plugin is something I really want. Maybe after we ship 1.1 I can focus on that. | 17:59 |
| *** kgriffs has left #openstack-infra | 17:59 | |
| zaro | so i guess i can't fix. will leave it up to admins to look at the master then. | 17:59 |
| *** kgriffs has joined #openstack-infra | 17:59 | |
| jeblair | clarkb: hrm, i don't see the stuff about disabling puppet | 17:59 |
| anteaya | krotscheck: thanks for adding infra-specs to storyboard | 17:59 |
| anteaya | krotscheck: now I have to learn how to use storyboard | 17:59 |
| krotscheck | anteaya: Urm, that was jeblair, but I’ll take credit for it :) | 18:00 |
| phschwartz | krotscheck: Sounds good if I don't get to it first as we will need it for vinz and no point just doing a cut and past of the current storyboard code. | 18:00 |
| clarkb | jeblair: http://ci.openstack.org/puppet.html#disabling-puppet-on-nodes | 18:00 |
| krotscheck | phschwartz: You might want to loop NikitaKonovalov in to it, he wrote most of the code for storyboard | 18:00 |
| anteaya | krotscheck: ah okay, thanks jeblair | 18:01 |
| phschwartz | krotscheck: will do when I am done with my internal gerrit setup here. | 18:01 |
| anteaya | and I can see it in storyboard now | 18:01 |
| jeblair | clarkb: thx | 18:01 |
| *** ptippett[A] has quit IRC | 18:02 | |
| phschwartz | stepping away for lunch. will look at scroll back when I am done. | 18:02 |
| clarkb | woot gertty is syncing. Will push my fixes shortly | 18:03 |
| *** jamielennox|away is now known as jamielennox | 18:03 | |
| anteaya | woot | 18:03 |
| openstackgerrit | A change was merged to openstack-infra/config: Add rally-cli job https://review.openstack.org/86331 | 18:03 |
| *** cdub has joined #openstack-infra | 18:04 | |
| *** sarob has quit IRC | 18:04 | |
| *** packet has joined #openstack-infra | 18:04 | |
| *** jlibosva has quit IRC | 18:05 | |
| *** hipster_ has quit IRC | 18:06 | |
| *** zz_gondoi is now known as gondoi | 18:07 | |
| comstud | Hmm... | 18:07 |
| *** dprince has quit IRC | 18:07 | |
| comstud | So, I'm seeing -1's sticking around patchsets, it seems | 18:07 |
| *** ptippett[A] has joined #openstack-infra | 18:07 | |
| comstud | is that by design? | 18:07 |
| comstud | i'm also seeing +1 and +2s sticking around, despite no mention of 'trivial rebase' | 18:08 |
| zaro | comstud: yes | 18:08 |
| *** aconrad has quit IRC | 18:08 | |
| comstud | -1s now carry over to new patchsets? | 18:08 |
| comstud | i didn't think I read that with the new gerrit release | 18:08 |
| *** wenlock has joined #openstack-infra | 18:08 | |
| openstackgerrit | Joshua Harlow proposed a change to openstack-infra/gerritbot: Allow skipping patchsets with WIP subjects https://review.openstack.org/94438 | 18:09 |
| zaro | comstud: only for code-review vote | 18:09 |
| harlowja | fungi what do u think about ^ | 18:09 |
| *** thomasbiege has quit IRC | 18:09 | |
| comstud | zaro: Ok, what about +1, +2.. those should certainly reset, right? | 18:09 |
| comstud | :) | 18:09 |
| *** thomasbiege has joined #openstack-infra | 18:09 | |
| fungi | comstud: gerrit 2.8 (unlike 2.4) no longer adds comments when it detects a trivial rebase, it just carries over the votes transparently. it also now does this on patchsets which are only commit message edits as well | 18:09 |
| comstud | ahh okay | 18:09 |
| jeblair | zaro: really? approvals are copied even for non-trivial changes? | 18:10 |
| comstud | that explains that | 18:10 |
| comstud | because this should have been a trivial rebase | 18:10 |
| Ajaeger1 | harlowja: how will this work? I push something and only set WIP a minute later. | 18:10 |
| comstud | but there was no comment saying so | 18:10 |
| anteaya | yay for new tox working!! | 18:10 |
| harlowja | Ajaeger1 this is in the commit subject | 18:10 |
| jeblair | oh ok, we're talking about trivial changes, nm. | 18:10 |
| harlowja | not in the WIP button on gerrit | 18:10 |
| comstud | But, the -1 that carried over in a different patchset was not a trivial rebase | 18:10 |
| comstud | so I was more concerned about that... but if that's by design, then cool | 18:10 |
| Ajaeger1 | harlowja: Ah! Sorry, should have read better... | 18:10 |
| zaro | jeblair: no, i don't think so. non-trivial change should reset | 18:11 |
| jeblair | comstud: are you talking about two different instances? | 18:11 |
| fungi | comstud: its definition of a trivial change may also differ slightly from what our previous hook detected | 18:11 |
| comstud | yeah, sorry, i'm being confusing, probably | 18:11 |
| comstud | jeblair: ^ :) | 18:11 |
| fungi | comstud: do you have a link for the non-trivial change which exhibited this behavior? | 18:11 |
| comstud | review #1: +1 and +2 carried over on a trivial rebase (or should have been trivial), but no trivial rebase comment was left. | 18:11 |
| jeblair | comstud: if there's an instance of a vote carrying over on a non-trivial change, we should take a look | 18:11 |
| comstud | That appears to be 'okay' | 18:11 |
| comstud | jeblair: In review #2, I have a -1 carry over on something that was NOT a trivial rebase | 18:12 |
| jeblair | comstud: agreed regarding review#1 | 18:12 |
| comstud | Just wanted to make sure that is by design | 18:12 |
| jeblair | comstud: i don't think it is; why don't you give us a link and we can take a look | 18:12 |
| comstud | https://review.openstack.org/#/c/93995/ | 18:12 |
| comstud | That's the one with -1 that carried over | 18:12 |
| comstud | -1 is on patch set 7 | 18:13 |
| comstud | but i'm on #9 now | 18:13 |
| comstud | and I know I changed the commit reason and 1 line in the code since #7. | 18:13 |
| anteaya | jeblair: sorry I was walking when you asked if infra-specs was using storyboard and thanks for fixing that | 18:15 |
| comstud | hm, well, maybe this patch set didn't have a line of code changed, but the commit reason changed | 18:15 |
| jeblair | yeah, i don't think that should happen. | 18:15 |
| anteaya | man, a lot happens in an hour | 18:15 |
| comstud | yeah okay, I'm confusing patch sets... | 18:15 |
| devananda | krotscheck: i probably said something like that. ironic uses the keyustoneclient.middleware.auth_token today | 18:15 |
| comstud | This one didn't change a line of code | 18:15 |
| jeblair | commit message might be considered trivial | 18:15 |
| comstud | but I *did* change the commit message. | 18:15 |
| comstud | yeha | 18:15 |
| comstud | apparently it did | 18:15 |
| openstackgerrit | Clark Boylan proposed a change to stackforge/gertty: Perform http calls outside of the db session https://review.openstack.org/94097 | 18:15 |
| openstackgerrit | Clark Boylan proposed a change to stackforge/gertty: Add custom palettes and commentlinks https://review.openstack.org/94094 | 18:15 |
| comstud | it's unclear since there's no comment about it anymore | 18:15 |
| clarkb | jeblair: ^ fixed | 18:15 |
| comstud | :) | 18:15 |
| pleia2 | setup.py is making me feel dumb, any idea how to add README.md to the list of data_files https://git.openstack.org/cgit/openstack-infra/reviewday/tree/setup.py ? tried ('', ['README.md']), and about 25 other variations and it still won't land in my sdist package | 18:15 |
| jeblair | i think our old script would have considered that a non-trivial change, but it would not surprise me if gerrit 2.8 considers it trivial | 18:16 |
| *** hcc has joined #openstack-infra | 18:16 | |
| comstud | yeah, a commit reason change before would reset votes | 18:16 |
| *** hcc is now known as hdd_ | 18:16 | |
| comstud | IIRC | 18:16 |
| fungi | comstud: jeblair: yes, i have witnessed gerrit 2.8 carry votes over on commit-message-update-only patchsets | 18:16 |
| comstud | like, I'm not sure a +2 should carry over if you change commit reason. | 18:16 |
| comstud | although it's less concerning than the code changing | 18:17 |
| comstud | :) | 18:17 |
| clarkb | I think this is configurable | 18:17 |
| jeblair | this may be colliding with the intended ease of use of the TTW commit message editing feature as well | 18:17 |
| *** _nadya_ has joined #openstack-infra | 18:17 | |
| * clarkb looks | 18:17 | |
| *** dims has quit IRC | 18:17 | |
| *** eglynn has joined #openstack-infra | 18:17 | |
| zaro | pleia2: this? http://git.openstack.org/cgit/openstack-infra/zuul/tree/setup.cfg | 18:18 |
| openstackgerrit | James E. Blair proposed a change to openstack-infra/infra-specs: Initial commit https://review.openstack.org/94440 | 18:18 |
| comstud | anyway, not really complaining about anything, but just making sure there's not unintended behavior here | 18:18 |
| zaro | pleia2: description-file reference | 18:18 |
| jeblair | comstud: thanks | 18:18 |
| fungi | comstud: previously we were doing this via an external script called from a gerrit hook, so we had a lot more control over nuance (such as detecting commit message changes, whitespace-only changes, et cetera). when gerrit folded this functionality in they seem to have made its behavior relatively non-configurable. i'd consider it something worth trying to get improved upstream | 18:18 |
| comstud | fungi: nod | 18:18 |
| clarkb | https://review.openstack.org/Documentation/config-labels.html | 18:18 |
| clarkb | copyAllScoresIfNoCodeChange is what we want | 18:19 |
| pleia2 | zaro: thanks, I'll give that a shot | 18:19 |
| comstud | The loss of 'trivial rebase' comment is kinda sucky, but I guess you can infer it from votes carrying over | 18:19 |
| *** che-arne has quit IRC | 18:19 | |
| clarkb | set that to false and set trivial rebase to true | 18:19 |
| zaro | removing is same as set to false | 18:20 |
| fungi | clarkb: it claims to default to false... perhaps we enabled it? | 18:20 |
| clarkb | fungi: ya | 18:20 |
| fungi | the down side is that it doesn't immediately point out that the new patchset only changed the commit message, but i suppose selecting the patchset delta view makes that fairly obvious | 18:21 |
| *** aconrad has joined #openstack-infra | 18:21 | |
| pleia2 | zaro: this seems to be a much different format for setup.py (older maybe?) than the one for zuul | 18:21 |
| fungi | we are still missing identification of whitespace-only changes however (i suspect those may be getting treated as trivial rebases) | 18:21 |
| clarkb | pleia2: I think it is a dprince special :) | 18:22 |
| lifeless | clarkb: thanks | 18:22 |
| lifeless | mordred_: yohi | 18:22 |
| clarkb | or was it ttx? in any case it was never part of the standardized setup.py stuff | 18:22 |
| *** alff has quit IRC | 18:22 | |
| zaro | fungi: it's documented that we set those to true http://ci.openstack.org/gerrit.html | 18:22 |
| pleia2 | clarkb: yeah dprince, maybe I should just rewrite the whole thing ;) | 18:22 |
| clarkb | fungi: the documentation indicates that they shouldn't be | 18:22 |
| clarkb | fungi: since it says the diff must be identical | 18:22 |
| * pleia2 seeks snack first | 18:23 | |
| fungi | clarkb: oh, hrm... that's good then i suppose. the original hook script (before we modified it) considered them the same and i had to do some substitution magic to make it not. but it was relying on git patch-id to decide what was "identical" (and that treats whitespace changes as irrelevant) | 18:24 |
| zaro | pleia2: ohh setup.py, i think setup.cfg is for pbr. let me see if i can find something. | 18:24 |
| fungi | from the manpage: A "patch ID" is nothing but a SHA-1 of the diff associated with a patch, with whitespace and line numbers ignored. | 18:25 |
| *** mrodden has joined #openstack-infra | 18:25 | |
| fungi | (i believe this choice stemmed from git being used initially for the kernel, thus tracking patches for c, where whitespace really is usually irrelevant) | 18:25 |
| *** jp_at_hp has quit IRC | 18:26 | |
| clarkb | hrm then maybe the docs are wrong | 18:26 |
| openstackgerrit | Andreas Jaeger proposed a change to openstack-infra/config: Publish translated localized API documents https://review.openstack.org/91132 | 18:27 |
| fungi | clarkb: well, their internal implementation may no longer use git patch-id to accomplish it | 18:28 |
| *** jp_at_hp has joined #openstack-infra | 18:28 | |
| anteaya | jeblair: a missing ) in https://review.openstack.org/#/c/94440/1/README.rst | 18:28 |
| *** otherwiseguy has quit IRC | 18:30 | |
| jeblair | anteaya: thx | 18:30 |
| openstackgerrit | James E. Blair proposed a change to openstack-infra/infra-specs: Initial commit https://review.openstack.org/94440 | 18:30 |
| *** otherwiseguy has joined #openstack-infra | 18:31 | |
| *** resker has joined #openstack-infra | 18:31 | |
| anteaya | great thanks, +1 | 18:31 |
| anteaya | krotscheck phschwartz, you're up: https://review.openstack.org/#/c/94440/ | 18:31 |
| Ajaeger1 | jeblair, clarkb: I see you're back in the reviewing cycle - thanks! I would appreciate a review on https://review.openstack.org/92997 | 18:31 |
| *** nati_ueno has joined #openstack-infra | 18:32 | |
| krotscheck | anteaya: Already in it. | 18:32 |
| phschwartz | anteaya: looking | 18:32 |
| zaro | pleia2: sorry, can't help you further. i just took a look at nose-html-output but there's no reference to README. however it does appear on pypi. http://git.openstack.org/cgit/openstack-infra/nose-html-output/tree/setup.py | 18:32 |
| openstackgerrit | Cyril Roelandt proposed a change to openstack-infra/config: django_openstack_auth: make the Python 3.3 gate voting https://review.openstack.org/94442 | 18:32 |
| openstackgerrit | Nachi Ueno proposed a change to openstack-infra/reviewstats: Add Review Stat WebUI https://review.openstack.org/94288 | 18:33 |
| zaro | pleia2: ^ maybe clarkb can help because he's the author. | 18:33 |
| jeblair | Ajaeger1: that's at position #11 in my queue | 18:33 |
| anteaya | krotscheck phschwartz great thanks | 18:33 |
| pleia2 | zaro: if it was a .rst or just README it would automatically be included, README.md is not :\ | 18:33 |
| fungi | zaro: earlier dstufft said that if a metadata.description is not specified, then pypi tried to figure one out by including readme or other files | 18:34 |
| dstufft | ahhhh | 18:34 |
| dstufft | I went to bitbucket to find the code, and bibucket changed their design QQ | 18:34 |
| openstackgerrit | A change was merged to openstack-infra/config: Add configuration option to disable top level menu for javamelody plugin https://review.openstack.org/91176 | 18:34 |
| krotscheck | Oh man, I can’t wait until this process is automated… | 18:34 |
| zaro | README.rst wasn't auto included in the last jjb release | 18:34 |
| *** esker has quit IRC | 18:34 | |
| fungi | zaro: so my guess as to why jjb 0.7.0 had the readme displayed on pypi while 0.8.0 did not is that 0.8.0 added a description field in the setup.cfg | 18:34 |
| Ajaeger1 | jeblair: that's high enough for me ;) | 18:34 |
| Ajaeger1 | jeblair: thanks! | 18:34 |
| krotscheck | …wait, what? We use LaTeX? | 18:35 |
| *** Hal_ has quit IRC | 18:35 | |
| dstufft | fungi: https://bitbucket.org/pypa/pypi/src/929afe98295ac47528bcff7fca8e82526dd50a89/description_utils.py?at=default#cl-172 | 18:35 |
| fungi | krotscheck: if only. i miss tex | 18:35 |
| dstufft | there is code | 18:35 |
| *** rfolco has quit IRC | 18:37 | |
| * ptippett[A] is now away - Reason : Auto-Away after 30 minutes | 18:37 | |
| *** _nadya_ has quit IRC | 18:38 | |
| *** sarob has joined #openstack-infra | 18:38 | |
| fungi | dstufft: and it only does that if metadata.description is unspecified, right? | 18:38 |
| krotscheck | Should setup.cfg’s license be Apache or Creative Commons? | 18:38 |
| dstufft | fungi: well if long_description is blank, I assume that maps to metadata.description | 18:38 |
| jeblair | ptippett[A]: Hi, this is the third time we've asked you to disable your away message. Please feel free to rejoin the channel after you have done so. | 18:38 |
| anteaya | what are the options for getting rid of ptippett[A]'s away message? addressing this person directly doesn't seem to be effective | 18:39 |
| *** ChanServ sets mode: +o jeblair | 18:39 | |
| *** ptippett[A] was kicked by jeblair (ptippett[A]) | 18:39 | |
| jeblair | anteaya: that. | 18:39 |
| *** jeblair sets mode: -o jeblair | 18:39 | |
| anteaya | that is effective | 18:39 |
| *** krtaylor has joined #openstack-infra | 18:40 | |
| zaro | dstufft, fungi : this one has a description but yet still renders the README http://git.openstack.org/cgit/openstack-infra/nose-html-output/tree/setup.py | 18:40 |
| dstufft | long_description | 18:41 |
| dstufft | description isn't a field | 18:41 |
| openstackgerrit | A change was merged to openstack-infra/config: Add mistral-dashboard project to stackforge https://review.openstack.org/91296 | 18:41 |
| dstufft | or well it might be summary | 18:41 |
| * dstufft is hurting so a little out of it | 18:41 | |
| dstufft | yea | 18:41 |
| dstufft | description is the short summary | 18:41 |
| dstufft | ~1 sentence or so | 18:41 |
| *** sarob_ has joined #openstack-infra | 18:42 | |
| dstufft | https://pypi.python.org/pypi/requests/ "Python HTTP for Humans" is description | 18:42 |
| dstufft | long_description is the long descripion, where you can shove multi line stuff and gets rendered rst and whatever | 18:42 |
| pleia2 | https://git.openstack.org/cgit/openstack-infra/reviewday/tree/setup.py is what originally started this, this script fails now (used to work fine, pep8 change?) because README.md, the long_description is no longer in the sdist file so it can find it and dies | 18:42 |
| dstufft | yea becase you have open("README.md") | 18:43 |
| pleia2 | so all reviewday checks are failing | 18:43 |
| *** dims has joined #openstack-infra | 18:43 | |
| dstufft | you need to include that in the package | 18:43 |
| dstufft | MANIFEST.in it | 18:43 |
| Ajaeger1 | pleia2: don't you need to add it to MANIFEST.in? | 18:43 |
| pleia2 | dstufft: ah MAIFEST.in | 18:43 |
| Ajaeger1 | "include README.rst" | 18:43 |
| pleia2 | I was trying to add it to data_files | 18:43 |
| Ajaeger1 | pleia2: Just a single line that contains the include | 18:43 |
| dstufft | MANIFEST.in controls what fiels get added to the datball, data_files/package_data cotrols what gets installed | 18:43 |
| pleia2 | aha, thanks guys | 18:44 |
| *** resker has quit IRC | 18:44 | |
| *** esker has joined #openstack-infra | 18:44 | |
| *** gokrokve has quit IRC | 18:45 | |
| mordred_ | pleia2: or, just pbr it | 18:45 |
| anteaya | bookwar: I have reviewed 93560 | 18:46 |
| *** dkranz has quit IRC | 18:46 | |
| anteaya | dstufft: what did you do to your hand? | 18:47 |
| dstufft | anteaya: oh, it's just an RSI, but i've been ignoring it b/c the braces make it hard to type and i didn't wanna wear them | 18:47 |
| dstufft | now it hurts and i'm weaing the braces and chicken pecking my keyboard -.- | 18:48 |
| anteaya | dstufft: megadoes of vitamin B help you at all? | 18:48 |
| anteaya | megadoses? | 18:48 |
| dstufft | dunno never tried | 18:48 |
| *** sarob_ has quit IRC | 18:48 | |
| anteaya | it turns your urine neon yellow which is interesting | 18:48 |
| bookwar | anteaya: thanks | 18:48 |
| anteaya | bookwar: np | 18:49 |
| anteaya | bookwar: now you just have to track down jhesketh, he is on Aussie time | 18:49 |
| *** sarob has quit IRC | 18:50 | |
| *** sarob has joined #openstack-infra | 18:51 | |
| *** e0ne has joined #openstack-infra | 18:52 | |
| fungi | jeblair: so... turns out that http://git.openstack.org/cgit/openstack-infra/config/tree/modules/openstack_project/files/log_archive_maintenance.sh#n11 only deletes files which are at least 120 days old *and* end in ".gz" | 18:53 |
| sdague | jeblair / pleia2: got to go do a CSA pickup before the TC meeting, might be able to hit the end of the infra meeting. Especially interested in talking about the dashboard patch for jeepyb | 18:54 |
| fungi | jeblair: i think if we want to start deleting more than just .gz files, we'll need to repeat (or refactor) some of the compression exclusion logic in the earlier conditional stanza | 18:54 |
| * anteaya envisions lots of garlic scapes in sdague's CSA basket | 18:55 | |
| openstackgerrit | Elizabeth Krumbach Joseph proposed a change to openstack-infra/reviewday: Add MANIFEST.in to include README.md in sdist pkg https://review.openstack.org/94446 | 18:55 |
| *** sarob has quit IRC | 18:55 | |
| jeblair | fungi: now that it's a standalone script, that doesn't trouble me very much | 18:56 |
| jeblair | sdague: ack | 18:56 |
| *** hogepodge has quit IRC | 18:56 | |
| *** crc32 has quit IRC | 18:56 | |
| fungi | jeblair: cleaning up the files for the commit you pointed out frees up 192k, though en masse that might add up to a measurable amount of space (and there could be jobs leaving other larger files around which are ignored due to similar logic) | 18:57 |
| * pleia2 always forgets bug comments in commit messages | 18:58 | |
| *** dkranz has joined #openstack-infra | 18:59 | |
| * pleia2 writes a git hook to echo "is there a bug related to this?" | 18:59 | |
| pleia2 | ;) | 18:59 |
| mordred_ | ++ | 18:59 |
| jeblair | pleia2: it may be easier to remember when we have "bugs" (stories) for more things. | 18:59 |
| pleia2 | jeblair: yeah | 18:59 |
| fungi | meeting time | 19:00 |
| *** amcrn has quit IRC | 19:00 | |
| *** ianw has quit IRC | 19:00 | |
| fungi | (once keystone wraps up) | 19:00 |
| jeblair | yeah | 19:00 |
| *** ianw has joined #openstack-infra | 19:00 | |
| fungi | which they just did | 19:00 |
| *** ramashri has quit IRC | 19:02 | |
| openstackgerrit | Jeremy Stanley proposed a change to openstack-infra/pypi-mirror: No longer whitelist externally-hosted requirements https://review.openstack.org/94447 | 19:03 |
| *** dims has quit IRC | 19:03 | |
| openstackgerrit | Elizabeth Krumbach Joseph proposed a change to openstack-infra/reviewday: Add MANIFEST.in to include addl files in sdist pkg https://review.openstack.org/94446 | 19:03 |
| fungi | mordred_: dstufft: ^ (yay!) | 19:03 |
| *** pblaho has joined #openstack-infra | 19:04 | |
| *** kgriffs is now known as kgriffs|afk | 19:04 | |
| *** crc32 has joined #openstack-infra | 19:07 | |
| *** Sukhdev has joined #openstack-infra | 19:08 | |
| *** otherwiseguy has quit IRC | 19:17 | |
| *** resker has joined #openstack-infra | 19:18 | |
| *** esker has quit IRC | 19:21 | |
| *** markwash has quit IRC | 19:22 | |
| *** amcrn has joined #openstack-infra | 19:25 | |
| *** andreaf has joined #openstack-infra | 19:25 | |
| *** alugovoi has joined #openstack-infra | 19:25 | |
| *** resker has quit IRC | 19:26 | |
| *** simonmcc has quit IRC | 19:27 | |
| *** esker has joined #openstack-infra | 19:27 | |
| *** simonmcc has joined #openstack-infra | 19:27 | |
| *** lxsli has quit IRC | 19:27 | |
| *** gokrokve has joined #openstack-infra | 19:29 | |
| *** e0ne has quit IRC | 19:29 | |
| *** lxsli has joined #openstack-infra | 19:29 | |
| *** e0ne has joined #openstack-infra | 19:30 | |
| openstackgerrit | David Shrewsbury proposed a change to openstack-infra/devstack-gate: Change Ironic bm flavor to use ephemeral partition https://review.openstack.org/94456 | 19:31 |
| *** hashar has joined #openstack-infra | 19:32 | |
| *** andreaf has quit IRC | 19:33 | |
| *** e0ne has quit IRC | 19:33 | |
| openstackgerrit | Michael Krotscheck proposed a change to openstack-infra/storyboard-webclient: Textareas now autoresize their height. https://review.openstack.org/92939 | 19:33 |
| *** andreaf has joined #openstack-infra | 19:33 | |
| *** hipster has joined #openstack-infra | 19:33 | |
| *** hipster has quit IRC | 19:33 | |
| *** hipster has joined #openstack-infra | 19:34 | |
| *** kgriffs|afk is now known as kgriffs | 19:35 | |
| *** gyee has quit IRC | 19:36 | |
| *** sarob has joined #openstack-infra | 19:37 | |
| *** annegent^ has quit IRC | 19:37 | |
| *** andreaf_ has joined #openstack-infra | 19:38 | |
| *** _nadya_ has joined #openstack-infra | 19:38 | |
| *** e0ne has joined #openstack-infra | 19:39 | |
| *** andreaf has quit IRC | 19:39 | |
| *** annegentle_ has joined #openstack-infra | 19:40 | |
| *** _nadya_ has quit IRC | 19:44 | |
| *** _nadya_ has joined #openstack-infra | 19:45 | |
| *** sarob has quit IRC | 19:47 | |
| *** lxsli has quit IRC | 19:47 | |
| *** sarob has joined #openstack-infra | 19:47 | |
| *** lxsli has joined #openstack-infra | 19:48 | |
| *** krotscheck has quit IRC | 19:49 | |
| *** krotscheck has joined #openstack-infra | 19:50 | |
| *** marcoemorais has quit IRC | 19:50 | |
| *** jp_at_hp has quit IRC | 19:50 | |
| *** sarob_ has joined #openstack-infra | 19:51 | |
| openstackgerrit | Andreas Jaeger proposed a change to openstack-infra/config: Publish troveclient documentation https://review.openstack.org/92562 | 19:53 |
| *** smarcet has quit IRC | 19:53 | |
| ianw | am i going nuts or is my reverify command not getting picked up on https://review.openstack.org/#/c/92748/? | 19:54 |
| *** stevebaker has quit IRC | 19:54 | |
| *** stevebaker has joined #openstack-infra | 19:54 | |
| jesusaurus | ianw: infra is in a meeting, should be done in the next couple minutes | 19:55 |
| fungi | ianw: recheck bug 1321093 is what you probably want there | 19:55 |
| uvirtbot | Launchpad bug 1321093 in trove "*-tempest-dsvm-postgres-full failing on trove-manage db_sync" [Critical,Triaged] https://launchpad.net/bugs/1321093 | 19:55 |
| *** jeblair has quit IRC | 19:55 | |
| *** jeblair has joined #openstack-infra | 19:55 | |
| *** jeblair has quit IRC | 19:55 | |
| *** sarob has quit IRC | 19:55 | |
| *** pblaho has quit IRC | 19:56 | |
| ianw | fungi: doh, check, verify ... i'll just claim it's too much to differentiate at 5am :) thanks | 19:56 |
| openstackgerrit | A change was merged to openstack-infra/config: Only download nearly finished translated files https://review.openstack.org/92997 | 19:56 |
| Alex_Gaynor | dstufft: happy dance: | 19:58 |
| Alex_Gaynor | https://review.openstack.org/#/c/94447/ | 19:58 |
| dstufft | oh hell yea | 19:58 |
| *** marcoemorais has joined #openstack-infra | 19:59 | |
| * dstufft gives it a +1 although my +1 will have little actual affect but still +1 | 20:00 | |
| pleia2 | I'd really appreciate pushing this through soon, all reviewday checks are broken until this is in, including the one to fix reviewday itself (it's broken, hasn't updated since yesterday) https://review.openstack.org/#/c/94446/ | 20:01 |
| krotscheck | So, the a bunch of UX people are having a pretty active storyboard-related discussion in a non-gerrit side channel here - http://invis.io/TUULORSC - Now, I don’t want to get caught between two groups that end up reaching different kinds of consensus, any suggestions on how I can make everyone happy? | 20:01 |
| clarkb | pleia2: +2 | 20:02 |
| *** jeblair has joined #openstack-infra | 20:02 | |
| fungi | pleia2: i take it reviewday isn't pbr-ized | 20:02 |
| sc68cal | soo, ipv6 | 20:02 |
| sdague | I seem to have completely been over ambitious in thinking I could complete the CSA outing in < 1 hr. Will go back through the infra meeting pings post TC meeting | 20:02 |
| anteaya | krotscheck: can everyone involved have a common place to talk? | 20:02 |
| pleia2 | fungi: right | 20:03 |
| anteaya | krotscheck: can you suggest they come here? | 20:03 |
| clarkb | jeblair: gertty has a bunch of infra/config changes it knows about with the refs in the git repos. Is the simplest way to fix that to move my db and git repos aside? | 20:03 |
| clarkb | jeblair: I can also give you a traceback if you want | 20:03 |
| anteaya | sc68cal: yes, we are listening | 20:03 |
| anteaya | sc68cal: where were we? | 20:03 |
| dstufft | pleia2: I gave you a comment | 20:03 |
| dstufft | I think your MANIFEST.in is wrong | 20:04 |
| sc68cal | haha sort of a bum rush at the end of that meeting | 20:04 |
| dstufft | but it's distutils so who actually knows | 20:04 |
| clarkb | good thing I wasn't the only one reviewing that | 20:04 |
| krotscheck | anteaya: I’m not certain that IRC provides the kind of asynchronous, out-of-band discussion & annotation on screenshots that UX/UI reviews require. | 20:04 |
| krotscheck | anteaya: I can suggest it, yes. | 20:04 |
| fungi | dstufft: pleia2: oh, unapproved ;) | 20:04 |
| sc68cal | So - I've been pushing a couple patches into Gerrit around Tempest and DevStack | 20:04 |
| sc68cal | prepping for ipv6 testing | 20:04 |
| anteaya | krotscheck: what format do they usually use? | 20:04 |
| dstufft | -> bus stop to pick up daughter | 20:04 |
| *** aveiga has joined #openstack-infra | 20:04 | |
| anteaya | krotscheck: in person? | 20:04 |
| krotscheck | anteaya: Take a look: http://invis.io/TUULORSC | 20:04 |
| clarkb | krotscheck: neat it took me half a second to realize what was going on there | 20:05 |
| fungi | sdague: yeah, we had a few items in there you probably want to provide feedback on | 20:05 |
| krotscheck | anteaya: Though now that I think about it, the discussion can be linked from a spec. | 20:05 |
| anteaya | sc68cal: yes, we have to stick to our hour | 20:05 |
| clarkb | krotscheck: can we -1 the windows logo? | 20:05 |
| clarkb | as far as coalescing the review body goes, this is a hard problem and one of the reasons we looked at phabricator way back when | 20:05 |
| *** lyxus has joined #openstack-infra | 20:06 | |
| *** bhuvan_ has joined #openstack-infra | 20:06 | |
| anteaya | yeah I'm -1 on the windows logo too | 20:06 |
| sc68cal | sorry - trying to juggle a couple different things and the chainsaws are starting to fall on the floor | 20:06 |
| anteaya | sc68cal: yup | 20:06 |
| fungi | sc68cal: yeah, easy way to lose your hands | 20:06 |
| clarkb | krotscheck: maybe the thing to do is tie a spec to the mock up? | 20:06 |
| anteaya | can you link to the patches you have up? | 20:06 |
| *** kgriffs is now known as kgriffs|afk | 20:06 | |
| krotscheck | clarkb anteaya : you can -1 all you want :) | 20:06 |
| clarkb | krotscheck: then at least both sides of the discussion can see it | 20:06 |
| krotscheck | clarkb: That’s actually what I’m thinking. | 20:06 |
| anteaya | ha ha ha | 20:06 |
| clarkb | until we have something better | 20:06 |
| openstackgerrit | Elizabeth Krumbach Joseph proposed a change to openstack-infra/reviewday: Add MANIFEST.in to include addl files in sdist pkg https://review.openstack.org/94446 | 20:07 |
| pleia2 | dstufft: there you go ^^ | 20:07 |
| *** dims has joined #openstack-infra | 20:07 | |
| *** _nadya_ has quit IRC | 20:08 | |
| *** amcrn has quit IRC | 20:08 | |
| pleia2 | fungi: agreed re: pbr, I'll toss up a bug for now so I can look at it when I get my head around pbr ;) | 20:08 |
| zaro | aconrad: new jjb release has been created | 20:08 |
| *** bhuvan has quit IRC | 20:09 | |
| *** smarcet has joined #openstack-infra | 20:09 | |
| fungi | sc68cal: so, just to recap. if there are specific hardware requirements for things you want to test, then that will likely have to be third party. however, anything we can test to ensure working ipv6 within a devstack environment using virtual networks (keeping in mind that this could include multiple devstack virtual machines connected via gre tunnels or the like) we ought to try to do upstream | 20:09 |
| aveiga | sc68cal: are your devstack patches tied to provider networks? | 20:10 |
| clarkb | sc68cal: aveiga keep in mind that anything you do upstream is work that you don't have to do and developers must make it pass | 20:10 |
| lyxus | I was wondering why do download the fedora qcow in devstack | 20:10 |
| aveiga | clarkb: totally agreed. We just don't want to mistakenly break someone's work while we get it right | 20:10 |
| clarkb | and ipv6 should be base functionality that we should make work | 20:10 |
| aveiga | the guy who breaks gate is the guy who never commits again | 20:11 |
| clarkb | aveiga: huh | 20:11 |
| sc68cal | aveiga: I have some patches in devstack regarding provider networking | 20:11 |
| clarkb | aveiga: the infra team probably breaks the gate more than anyone else :) | 20:11 |
| fungi | aveiga: that can be solved simply by having a separate non-voting test until it's sane enough to be voting | 20:11 |
| clarkb | lyxus: because it is needed for heat testing iirc | 20:11 |
| aveiga | heh, I'd be afraid to commit again | 20:11 |
| sc68cal | that'd be used for some of the provider network scenario tests that we will author | 20:11 |
| sc68cal | since we do not deploy the l3 agent | 20:11 |
| aveiga | ok, I can attempt to configure some bits of my lab to do gre and see if the incoming patches work with l3-agent | 20:11 |
| sdague | jeblair: in looking at scroll back, I'm curious what feedback I missed here - https://review.openstack.org/#/c/94196/ - I thought I'd had it covered | 20:12 |
| *** marcoemorais has quit IRC | 20:12 | |
| *** marcoemorais has joined #openstack-infra | 20:12 | |
| fungi | sc68cal: aveiga: yeah, unless i'm missing something, there shouldn't be anything specific about provider networks, as a feature, which would preclude upstream testing (e.g. by configuring virtual provider networks to make that work) | 20:13 |
| *** lxsli has quit IRC | 20:13 | |
| lyxus | clarkb,, I will need to mirror it in someway downloading 200M everytime for CI is too much ! | 20:13 |
| sc68cal | fungi: true, but we'd need to add some stuff to devstack to simulate a hardware device | 20:13 |
| aveiga | fungi: is there something that emulates a provider net somewhere? The only thing our provider net does is act as a .1q switch and provide RAs | 20:13 |
| fungi | lyxus: we cache it on our nodepool images and devstack has logic to prevent downloading if it already exists | 20:13 |
| lyxus | fungi, so if i put the file where it's supposed to be downloaded it. it will not do it again ? | 20:15 |
| clarkb | sc68cal: aveiga you can't use an openvswitch setup on the test node(s)? | 20:15 |
| *** zns has joined #openstack-infra | 20:15 | |
| *** sarob_ has quit IRC | 20:15 | |
| aveiga | OVS doesn't generate packets. Something has to act as a router and issue the router advertisement packets | 20:15 |
| fungi | sc68cal: aveiga: i'm not super familiar with what's already there for emulated provider networks in devstack, but it should be able to be added i would think, and would not likely be rejected by the devstack developers | 20:15 |
| sc68cal | clarkb: this might help explain - https://review.openstack.org/#/c/88043/ | 20:15 |
| clarkb | aveiga: you can always attach a process to a port on the switch | 20:16 |
| jeblair | krotscheck: i can't wait until we can have that in storyboard itself | 20:16 |
| sc68cal | http://docs-draft.openstack.org/43/88043/8/check/gate-neutron-specs-docs/fd0a1d7/doc/build/html/specs/juno/ipv6-provider-nets-slaac.html | 20:16 |
| clarkb | unless I am missing something I don't see why hardware is necessary | 20:16 |
| aveiga | clarkb: true, we could bolt radvd onto it | 20:16 |
| fungi | aveiga: installing and configuring rdadvd for example | 20:16 |
| fungi | er, radvd | 20:16 |
| aveiga | :) | 20:16 |
| fungi | yeah | 20:16 |
| sc68cal | Yes - we just need to add those pieces to devstack to simulate | 20:16 |
| aveiga | ok, let me ponder this setup and try it in the lab. If we can get it to run the same tests as our prov-net devstack, we can upstream it | 20:17 |
| anteaya | sc68cal: so the topic for 88043 is very specific to that patch, if you used something like ipv6 or ipv6-testing, it would be easier for us all to find | 20:17 |
| clarkb | aveiga: great | 20:17 |
| sc68cal | anteaya: Agree - that's why i registerd a BP on the tempest side | 20:17 |
| anteaya | sc68cal: as a for instance: https://review.openstack.org/#/q/status:open+project:openstack/neutron-specs+branch:master+topic:ipv6-provider-nets-slaac,n,z | 20:17 |
| anteaya | sc68cal: I don't see a bp link in 88043 | 20:18 |
| *** otherwiseguy has joined #openstack-infra | 20:18 | |
| sc68cal | bear with me fetching it from launchpad | 20:18 |
| anteaya | thanks | 20:18 |
| sc68cal | https://blueprints.launchpad.net/openstack/?searchtext=neutron-provider-networking | 20:19 |
| *** jaypipes has quit IRC | 20:19 | |
| openstackgerrit | A change was merged to openstack-infra/reviewday: Add MANIFEST.in to include addl files in sdist pkg https://review.openstack.org/94446 | 20:19 |
| sc68cal | I tried to explain it on the devstack side - https://blueprints.launchpad.net/devstack/+spec/neutron-provider-networking | 20:19 |
| *** jaypipes has joined #openstack-infra | 20:20 | |
| sc68cal | obviously a next piece would need to be to launch a radvd process and simulate an upstream router | 20:21 |
| aveiga | sc68cal: I think I can help there | 20:22 |
| sc68cal | right now it's just that we have a lab environemnt that has worked very well for us, and we're trying to pull more of it into devstack and then into software | 20:22 |
| sc68cal | from the hardware layout we currnetly have | 20:22 |
| *** markwash has joined #openstack-infra | 20:22 | |
| fungi | sc68cal: yeah, i think that working on that is likely to be time far better spent than setting up a 3rd-party test system. we can easily help you run your new virtual network configuration in a non-voting test for however long it takes to shake out any serious bugs, and then it's an easy toggle to start enforcing it on all new changes | 20:24 |
| clarkb | jeblair: https://storyboard.openstack.org/#!/story/91 I am going to try moving my db and repos aside now and resync | 20:24 |
| *** julim has quit IRC | 20:25 | |
| sc68cal | fungi: ok - I will work on the devstack & tempest patches | 20:26 |
| sc68cal | I feel bad bugging anteaya and sdague at the summit about it but I've got tons of patches that never landed in icehouse - really don't feel like adding more on top of the pile. | 20:27 |
| anteaya | sc68cal: you needed to talk to us | 20:27 |
| fungi | sc68cal: don't feel bad. we (infra) have a vested interest in ipv6 support in neutron, after all | 20:27 |
| anteaya | and talking to us about something like this is important, it isn't bugging | 20:27 |
| anteaya | openstack needs ipv6 | 20:27 |
| anteaya | so we need to be able to test it | 20:27 |
| openstackgerrit | Nachi Ueno proposed a change to openstack-infra/reviewstats: Add Review Stat WebUI https://review.openstack.org/94288 | 20:28 |
| *** sarob has joined #openstack-infra | 20:28 | |
| *** zns has quit IRC | 20:28 | |
| *** bhuvan_ has quit IRC | 20:28 | |
| anteaya | and yeah, if your workflow is blocked we need to help you get it flowing again | 20:28 |
| fungi | i (selfishly) need neutron to support ipv6, because i want openstack infra to be able to make use of ipv6 within our donated provider resources without relying on their hacky non-openstack bolt-on solutions which differ from one to the next | 20:29 |
| *** krtaylor has quit IRC | 20:29 | |
| anteaya | :D | 20:30 |
| *** marcoemorais has quit IRC | 20:31 | |
| sc68cal | cool. I'll talk with everyone tomorrow once I get things back on track. Have a ton of paperwork that was neglected while at the summit | 20:31 |
| *** Ajaeger1 has quit IRC | 20:31 | |
| clarkb | ohman me too | 20:31 |
| * anteaya nods | 20:31 | |
| clarkb | expense reports | 20:31 |
| clarkb | I should do those | 20:31 |
| *** stevebaker has quit IRC | 20:32 | |
| anteaya | sc68cal do you and aveiga share workspace? | 20:32 |
| anteaya | or separate lab environments here? | 20:32 |
| *** zns has joined #openstack-infra | 20:32 | |
| *** dstanek is now known as dstanek_zzz | 20:33 | |
| sc68cal | Same rack, or adjacent rack | 20:33 |
| anteaya | cool | 20:34 |
| sc68cal | aveiga does QA on the packages we build, from the patches I test out on our fork of devstack | 20:34 |
| anteaya | ah nice | 20:35 |
| *** dstanek_zzz is now known as dstanek | 20:35 | |
| openstackgerrit | Clark Boylan proposed a change to openstack-infra/config: Configure swift credentials for workers to push to https://review.openstack.org/92683 | 20:36 |
| clarkb | mordred_: fungi ^ that addresses your concern | 20:37 |
| clarkb | would be great to move that along in the near future | 20:37 |
| lifeless | mordred: o/ | 20:39 |
| fungi | sdague: do we need to be renaming grenade logs to make them more consumable in browser, and so that our compression scripts catch them? for example, see http://logs.openstack.org/74/93874/1/check/check-grenade-dsvm-neutron/af51965/logs/ | 20:39 |
| clarkb | fungi: ++ we only need the non dated file | 20:39 |
| fungi | sdague: i suspect we should add some log handling logic for those in d-g cleanup | 20:39 |
| clarkb | er files | 20:39 |
| fungi | clarkb: well, we have four files there... we should just keep the .log and .log.summary and append or substitute .txt suffixes on them? | 20:40 |
| *** eharney has quit IRC | 20:41 | |
| mordred | lifeless: I ALMOST have brainspace for you | 20:41 |
| *** ramashri has joined #openstack-infra | 20:41 | |
| clarkb | fungi: yes | 20:41 |
| fungi | clarkb: okay, i'll throw in a d-g change shortly for that | 20:42 |
| markwash | hi hi folks. May I get glance-ptl added as an "included group" in glance-specs-core for gerrit / review.openstack.org ? | 20:43 |
| fungi | markwash: yep, just a sec | 20:44 |
| markwash | I think that is the one step that is needed so I can self-administer | 20:44 |
| markwash | (gerrit, not alcohol) | 20:44 |
| markwash | ((heck, both)) | 20:44 |
| fungi | markwash: done | 20:45 |
| fungi | see if that works for you | 20:45 |
| fungi | though there may be additional changes needed before you can self-administer alcohol | 20:45 |
| lifeless | mordred: yay. PLEASE. | 20:46 |
| markwash | fungi: seems to be working, thanks! | 20:47 |
| sdague | fungi: yes, we need to redo the grenade artifacts to make them more logviewer friendly | 20:47 |
| mordred | lifeless: I read the scrollback - but do you have a brief tl;dr ? when I read it, I was still post-summit braindead | 20:47 |
| mordred | lifeless: something about pre-release OR dev release precedence, right? | 20:47 |
| clarkb | anteaya: I just approved the third party doc requiring utc timestamp | 20:48 |
| anteaya | yay | 20:48 |
| anteaya | thank you | 20:48 |
| clarkb | anteaya: we should write a follow up change soon that documents the other things we decided on | 20:48 |
| anteaya | yes | 20:48 |
| anteaya | I'm hoping to sort out that stuff this week and document the best I can and have it someplace for review | 20:48 |
| anteaya | gerrit or wiki | 20:48 |
| anteaya | so we can get the changes communicated | 20:49 |
| lifeless | mordred: what version should teh 5th commit after an alpha tag have | 20:49 |
| lifeless | e.g. git tag 1.2.3.0a4 && git commit && git commit ... && setup.py sdist -> ??? | 20:49 |
| lifeless | mordred: there are two options; 1.2.3.4.0a4.dev4.gabcdef and 1.2.3.4.dev4.gabcdef | 20:50 |
| *** marcoemorais has joined #openstack-infra | 20:50 | |
| fungi | derekh: check-nova-docker-dsvm-f20 seems to archive a lot of files which we're not compressing and which aren't easily viewable in a browser. what's lacking to allow us to add .txt extensions on those (and are some of those files unnecessary)? | 20:50 |
| lifeless | mordred: semver.rst seems to imply the latter. | 20:50 |
| dstufft | hm | 20:50 |
| mordred | hrm | 20:50 |
| fungi | oh, derekh isn't around :/ | 20:51 |
| mordred | lifeless: so, since an alpha tag is a pre-release tag, I would think it doesn't want to persist for the comits between a4 and a5 | 20:52 |
| mordred | lifeless: BUT - maybe that's overthinking | 20:52 |
| mordred | and we should just always apply post-versioning rules to the most recent tag in the history? | 20:53 |
| mordred | consistently? | 20:53 |
| lifeless | mordred: the question is - do we want to be able to tell that hash abcdef has some relation to alpha4 | 20:53 |
| lifeless | mordred: or only to 1.2.3.4 | 20:53 |
| lifeless | mordred: oh, my question above was buggy. let me fix | 20:54 |
| dstufft | are you all aware that those versions aren't PEP440 compliant, or maybe I already told you that | 20:54 |
| lifeless | mordred: what version should teh 5th commit after an alpha tag have | 20:54 |
| lifeless | e.g. git tag 1.2.3.0a4 && git commit && git commit ... && setup.py sdist -> ??? | 20:54 |
| mordred | dstufft: because of the git sha? yeah, we know | 20:54 |
| lifeless | mordred: there are two options; 1.2.3.0a4.dev4.gabcdef and 1.2.3.dev4.gabcdef | 20:54 |
| *** ArxCruz has quit IRC | 20:54 | |
| lifeless | mordred: now, for a tag 1.2.3, the options are | 20:55 |
| mordred | lifeless: 1.2.3.0a4.dev1.gabcdef and 1.2.3.dev5.gabcdef, no? | 20:55 |
| lifeless | 1.2.4.dev4.gabcdef and 1.2.4.dev4.gabcdef | 20:55 |
| lifeless | so thats easy | 20:55 |
| dstufft | mordred: yea, at some point in the future pip will either split out --pre and --pep440 or just stop accepting things that we can't make pep440, but no idea when that'll be but yea b/c of the sha | 20:55 |
| mordred | dstufft: yah - but I keep getting promised some way to put the sha into a metadata somewhere at some point | 20:55 |
| lifeless | mordred: my scenario was tag; then 5 commits. | 20:55 |
| mordred | we'll use that as soon as it's deady | 20:56 |
| mordred | dstufft: ready | 20:56 |
| dstufft | mordred: makes sense | 20:56 |
| mordred | lifeless: k. | 20:56 |
| dstufft | metadata 2.0 has a "source version" or some such thing which is meant for that, PEP440 also has a local version syntax that could be abused for it | 20:56 |
| mordred | lifeless: yes. I agree with the tag 1.2.4 scenario | 20:56 |
| lifeless | mordred: there are four sorts of tags. non-versions; release versions < 1.0.0, release versions >= 1.0.0 and pre-release versions. | 20:56 |
| openstackgerrit | A change was merged to openstack-infra/config: Configure swift credentials for workers to push to https://review.openstack.org/92683 | 20:56 |
| lifeless | non-versions we should ignore; release < 1.0.0 and release >= 1.0.0 we should treat the same for this case; pre-release versions are the one that I need input on | 20:57 |
| clarkb | fungi: jeblair anteaya everyone else, for the new repos should we go ahead and approve at will or do we want to do it in a controlled manner for debugging? | 20:59 |
| jeblair | clarkb, fungi: hold off for a bit | 20:59 |
| jeblair | i want to ask about project-specs vs program-specs at the project meeting today | 20:59 |
| clarkb | jeblair: rgr | 20:59 |
| anteaya | clarkb where is sdague's patch? | 20:59 |
| anteaya | I'd like the extra logging if it is close | 21:00 |
| clarkb | anteaya: it is still up for review. jeblair indicated it hadn't addressed all of his concerns but I don't grok them in that case | 21:00 |
| sdague | jeblair: if you could give me a little more direction on what I missed, I'll respin it. I actually thought I'd addressed the issues. | 21:01 |
| fungi | jeblair: good point. so far it's seemed very ad-hoc, which is thus inconsistent and not easily discovered as a result | 21:01 |
| fungi | (and may subsequently pose challenges for automated publication as well) | 21:02 |
| *** aconrad has quit IRC | 21:03 | |
| *** marcoemorais has quit IRC | 21:04 | |
| *** simonmcc has quit IRC | 21:04 | |
| *** jraim has quit IRC | 21:04 | |
| *** ramashri has quit IRC | 21:04 | |
| *** jraim has joined #openstack-infra | 21:05 | |
| *** simonmcc has joined #openstack-infra | 21:05 | |
| *** pdmars has quit IRC | 21:05 | |
| openstackgerrit | A change was merged to openstack-infra/config: Adding a utc timestamp requirement for 3rd party test logs https://review.openstack.org/77376 | 21:07 |
| *** markmcclain has quit IRC | 21:07 | |
| *** bhuvan has joined #openstack-infra | 21:08 | |
| clarkb | jeblair: can you see the comments on 88432? | 21:08 |
| lifeless | mordred: so | 21:08 |
| mordred | lifeless: I think the most important thing is that the sequence is followable | 21:08 |
| mordred | lifeless: I don't think it's important that commits after 0a4 express relationship to 0a4 per se | 21:08 |
| *** Sukhdev has quit IRC | 21:08 | |
| mordred | except in as much as it may or may not break the sequence nature of the commits | 21:08 |
| mordred | (like, se break the sequence now which is lame and people don't like, so if we're going to change how we do things a bit, we should go ahead and fix taht) | 21:09 |
| mordred | s/commits/versions/ | 21:09 |
| lifeless | mordred: ok so how about we do just .dev.gabcdef ? | 21:10 |
| *** sarob has quit IRC | 21:10 | |
| lifeless | mordred: semver says there is no comparison possible between dev versions | 21:10 |
| mordred | lifeless: ++ | 21:10 |
| *** stevebaker has joined #openstack-infra | 21:10 | |
| *** sarob has joined #openstack-infra | 21:11 | |
| lifeless | the reason I suggest this is that if there is 1.2.3.0a4 and 1.2.3.0a5 | 21:11 |
| lifeless | you get multiple .dev1 versions - the first commit after each | 21:11 |
| mordred | lifeless: if someone wanted to make debs or rpms per commit and put them into repos | 21:11 |
| *** ramashri has joined #openstack-infra | 21:11 | |
| *** e0ne has quit IRC | 21:11 | |
| clarkb | jesusaurus: I am guessing your jenkins change should go in before 88477? | 21:11 |
| mordred | lifeless: what mechanical tranform can we apply to get versions that would work for that | 21:11 |
| lifeless | mordred: right, so this is why I started down the path of doing 1.2.3.0a4.devN so that there *was* a sortable relation between those builds | 21:12 |
| mordred | nod. I think _having_ a sortable relation would be nice, even if we do it in such a way that we tell people to not imply meaning from it | 21:12 |
| lifeless | I wonder | 21:12 |
| openstackgerrit | A change was merged to openstack-infra/config: Add database-api publishing jobs to trove https://review.openstack.org/88050 | 21:13 |
| lifeless | what if we could dev from the last release | 21:13 |
| lifeless | so there is a 1.2.2 tag | 21:13 |
| lifeless | and a 1.2.3.a0 tag 10 commits up | 21:13 |
| lifeless | the first commit after 1.2.3.0a0 gets 1.2.3.dev11 | 21:13 |
| *** aysyd has quit IRC | 21:13 | |
| *** amcrn has joined #openstack-infra | 21:13 | |
| jesusaurus | clarkb: let me take a peek at 88477 | 21:13 |
| mordred | then the 0a0 commit jumps the sequence for the CD packaging repo people | 21:14 |
| lifeless | mordred: thoughts? also brb | 21:14 |
| mordred | lifeless: I think we might _have_ to do 1.2.3.0a4.devN | 21:14 |
| jesusaurus | clarkb: yeah, there is going to be a conflict there | 21:15 |
| mordred | it seems like the only thing that is consistent and understandable | 21:15 |
| *** sarob has quit IRC | 21:15 | |
| clarkb | jesusaurus: thanks, I think we should get yours in first because yours is the more difficult of the two | 21:16 |
| lifeless | mordred: the CD folk would perhaps be better with a way to say 'ignore pre-release tags' | 21:17 |
| lifeless | mordred: so they get .dev10 for that | 21:17 |
| *** aconrad has joined #openstack-infra | 21:18 | |
| *** mwagner_lap has quit IRC | 21:19 | |
| clarkb | jeblair: 90402, are we ready to approve that since jgriffith +1'd? | 21:20 |
| jesusaurus | clarkb: if you feel like approving 89708 i can babysit puppetboard | 21:20 |
| *** smarcet has quit IRC | 21:20 | |
| * clarkb looks | 21:21 | |
| clarkb | jesusaurus: and you checked that the rebase is clean? | 21:22 |
| * jgriffith hopes he doesn't regret :) | 21:22 | |
| jesusaurus | yeah, i did that a little while before the meeting | 21:22 |
| jesusaurus | and then jeblair got busy | 21:22 |
| clarkb | ok approving | 21:22 |
| *** marun has quit IRC | 21:23 | |
| clarkb | gertty should post that shortly | 21:23 |
| * jesusaurus watches puppetboard and zuul for errors | 21:23 | |
| *** ihrachyshka has quit IRC | 21:24 | |
| *** eharney has joined #openstack-infra | 21:30 | |
| *** zns has quit IRC | 21:30 | |
| clarkb | apparently my gertty is behind | 21:31 |
| *** bhuvan has quit IRC | 21:31 | |
| clarkb | jeblair: maybe gertty reviews should be high priority tasks | 21:31 |
| *** krtaylor has joined #openstack-infra | 21:31 | |
| *** hashar has quit IRC | 21:32 | |
| SergeyLukjanov | clarkb, jeblair, /me'd like to have gertty for iPad :( | 21:32 |
| *** doude_ has quit IRC | 21:33 | |
| *** marun has joined #openstack-infra | 21:33 | |
| *** lcheng_ has joined #openstack-infra | 21:35 | |
| clarkb | jesusaurus: I went ahead and approved in web ui | 21:35 |
| clarkb | didn't want to wait for gertty much longer | 21:35 |
| *** andreaf_ has quit IRC | 21:36 | |
| *** mfer has quit IRC | 21:36 | |
| jesusaurus | ugh, and it needs to rebase again | 21:37 |
| *** bhuvan has joined #openstack-infra | 21:37 | |
| *** Sukhdev has joined #openstack-infra | 21:38 | |
| clarkb | fun | 21:38 |
| *** dangers is now known as dangers_away | 21:40 | |
| *** jooools has quit IRC | 21:41 | |
| openstackgerrit | K Jonathan Harker proposed a change to openstack-infra/config: Move openstack-specific config out of ::jenkins https://review.openstack.org/89708 | 21:44 |
| jhesketh | Morning | 21:44 |
| anteaya | morning jhesketh | 21:45 |
| anteaya | when you have time, you have a comment on a patch of mine, a -1 actually: https://review.openstack.org/#/c/93562/ | 21:46 |
| jhesketh | will do | 21:46 |
| anteaya | i have thrown it back to you, if you have some suggestions for wording | 21:46 |
| anteaya | after coffee of course | 21:46 |
| anteaya | thanks | 21:46 |
| *** nati_ueno has quit IRC | 21:46 | |
| jhesketh | heh, you read my thoughts! | 21:46 |
| * jhesketh 's kettle just finished boiling | 21:47 | |
| anteaya | :D | 21:47 |
| lifeless | mordred: what do you think? | 21:48 |
| *** gokrokve has quit IRC | 21:49 | |
| *** jergerber has joined #openstack-infra | 21:49 | |
| *** bhuvan has quit IRC | 21:52 | |
| *** nati_ueno has joined #openstack-infra | 21:56 | |
| jesusaurus | clarkb: jeblair can you give 89708 another review? | 21:58 |
| *** harlowja is now known as harlowja_away | 21:59 | |
| openstackgerrit | Gaudenz Steinlin proposed a change to openstack-infra/elastic-recheck: Add e-r query for bug 1320670 https://review.openstack.org/94486 | 21:59 |
| uvirtbot | Launchpad bug 1320670 in grenade "404 on GET /v3/OS-SIMPLE-CERT/ca at grenade" [Undecided,In progress] https://launchpad.net/bugs/1320670 | 21:59 |
| *** bhuvan has joined #openstack-infra | 22:00 | |
| *** gyee has joined #openstack-infra | 22:01 | |
| jeblair | okay, word from the project meeting, after 30 minutes of bikeshedding, is program-specs and we should rename the existing project-specs repos | 22:01 |
| jeblair | fungi, clarkb, mordred: ^ so we should -1 the pending specs repos that use project names | 22:01 |
| *** dstanek is now known as dstanek_zzz | 22:01 | |
| *** pcm__ has quit IRC | 22:02 | |
| anteaya | well at least we don't have to rename | 22:03 |
| *** markwash has quit IRC | 22:03 | |
| jeblair | anteaya: or tripleo or qa | 22:03 |
| anteaya | I'm going to have to refer to this a lot of find what I am looking for: http://git.openstack.org/cgit/openstack/governance/tree/reference/programs.yaml | 22:04 |
| mattoliverau | Morning all | 22:04 |
| anteaya | morning mattoliverau | 22:04 |
| mattoliverau | Hey anteaya, did you get home ok?.. has the noise stopped? | 22:04 |
| clarkb | jesusaurus: yup expense report is done switching back to reviews | 22:05 |
| jeblair | anteaya: oh good point; we did reckon we could use the short names, like 'object' instead of object-storage. not sure those are in there, but should give you the idea. | 22:05 |
| mordred_ | lifeless brilliant. I love it | 22:05 |
| openstackgerrit | Devananda van der Veen proposed a change to openstack-infra/config: Create baremetal-specs git repo https://review.openstack.org/94113 | 22:06 |
| *** james_li has quit IRC | 22:06 | |
| anteaya | mattoliverau: I am home, the noise has stopped | 22:07 |
| anteaya | 15 hours of sleep last night | 22:07 |
| anteaya | jeblair: yay for short names | 22:07 |
| *** harlowja_away is now known as harlowja | 22:08 | |
| clarkb | jeblair: jesusaurus: the conflict was a thing for translation updates that I approved without mapping as a conflict. sorry about that. The change has my +2 in its current state | 22:10 |
| * anteaya 's homemade pizza is ready | 22:10 | |
| *** bknudson has quit IRC | 22:10 | |
| *** james_li has joined #openstack-infra | 22:11 | |
| jesusaurus | clarkb: yeah, i was surprised to see that that change conflicted, but git was able to auto-merge and i didn't see anything wrong with the resulting tree | 22:12 |
| *** lxsli has joined #openstack-infra | 22:12 | |
| *** sarob has joined #openstack-infra | 22:13 | |
| *** hdd_ is now known as hdd | 22:13 | |
| *** packet has quit IRC | 22:14 | |
| clarkb | jesusaurus: jgit just being silly | 22:14 |
| openstackgerrit | Cyril Roelandt proposed a change to openstack-infra/config: django_openstack_auth: make the Python 3.3 gate voting https://review.openstack.org/94442 | 22:15 |
| Alex_Gaynor | jogo: pep8 now says to prefer `is not` over `not ... is` | 22:15 |
| *** weshay has quit IRC | 22:15 | |
| jeblair | sdague: i think i placed my inline comment on the wrong line on your change | 22:16 |
| openstackgerrit | A change was merged to openstack-infra/jenkins-job-builder: Test creation of multiple jobs from templates https://review.openstack.org/91960 | 22:18 |
| jeblair | sdague: i have left updated comments on https://review.openstack.org/#/c/94196/ sorry about the confusion. | 22:21 |
| jeblair | clarkb: ^ | 22:21 |
| jogo | Alex_Gaynor: wow that was fast | 22:21 |
| fungi | jeblair: thanks for the heads-up on program-specs. does "infra-specs" qualify or do we need to rename to "infrastructure-specs" (or similar)? | 22:22 |
| clarkb | jeblair: thanks | 22:22 |
| jeblair | fungi: i think our short name is infra; openstack-infra is significant prior evidence | 22:22 |
| *** dstanek_zzz is now known as dstanek | 22:23 | |
| fungi | programs.yaml needs a short-name key ;) | 22:23 |
| jeblair | fungi: ++ | 22:23 |
| fungi | programs.telemetry.short-name=ceilo and win! | 22:24 |
| jeblair | also, just because i happened to notice it... infra has nearly half the number of git repos that openstack has. :) though that won't last long since openstack is about to get several more. | 22:24 |
| jeblair | 48% | 22:24 |
| clarkb | I am reviewing d-g changes right now to avoid conflicts with jesusaurus' config change | 22:25 |
| clarkb | ianw: I think I just approved a good chunk of your fedora support changes | 22:25 |
| fungi | clear evidence we're falling down on the job. time for a few frivilous new projects | 22:25 |
| *** doug-fish has left #openstack-infra | 22:25 | |
| *** eglynn has quit IRC | 22:25 | |
| jesusaurus | jeblair: but infra will also get a few more repos if we get around to breaking up the puppet modules | 22:25 |
| jeblair | fungi: and to move melange :) | 22:25 |
| *** mwagner_lap has joined #openstack-infra | 22:26 | |
| fungi | oof, friday here we come | 22:26 |
| jeblair | jesusaurus: push us over the edge! | 22:26 |
| ianw | clarkb: thanks, if you could also approve https://review.openstack.org/#/c/92748/ that would be great, others depend on that | 22:26 |
| *** wenlock has quit IRC | 22:28 | |
| *** gondoi is now known as zz_gondoi | 22:28 | |
| fungi | "rough data point... it requires as many new projects to run openstack as there are within openstack" (who wants to write another blog post?) | 22:28 |
| *** sarob has quit IRC | 22:28 | |
| *** wenlock has joined #openstack-infra | 22:29 | |
| * jeblair is starting to see another talk materialize... | 22:29 | |
| *** HenryG has joined #openstack-infra | 22:29 | |
| notmyname | I've noticed some 500 errors from paste.o.o a few times recently. eg now. http://paste.openstack.org/show/81020/ | 22:29 |
| clarkb | ianw: done | 22:29 |
| * notmyname is hoping that has nothing to do with using trove for it now | 22:30 | |
| anteaya | notmyname: no it is browser caching | 22:30 |
| anteaya | clear your browser cache and try again | 22:30 |
| anteaya | that link renders for me | 22:30 |
| fungi | notmyname: so far we've seen those clear if you delete old cookies for paste.o.o (if you have already done that and still see them, let us know) | 22:31 |
| notmyname | loading now. but why would it have a 500 response anyway? | 22:31 |
| clarkb | fungi: I hit the 500 too | 22:31 |
| clarkb | so this is different htan the cookie thing | 22:31 |
| clarkb | and it does load for me now | 22:31 |
| fungi | oh, fun | 22:31 |
| anteaya | it is? | 22:31 |
| anteaya | hmmmm | 22:31 |
| ianw | clarkb / sdague : awesome thanks! i'll let those filter through, then rebase a fix for https://review.openstack.org/#/c/92981/ and fingers crossed ... should clear all known issues | 22:31 |
| fungi | this is probably an unanticipated race from moving paste's db to an external source | 22:32 |
| *** dstanek is now known as dstanek_zzz | 22:33 | |
| *** lcheng_ has quit IRC | 22:33 | |
| ianw | clarkb: https://review.openstack.org/#/c/92981/ <- not showing as approved? | 22:33 |
| fungi | ianw: also not showing as passing tests | 22:34 |
| clarkb | ianw: that one is -1'd | 22:35 |
| clarkb | I don't see where lodgeit may be doing any logging | 22:35 |
| *** dims has quit IRC | 22:35 | |
| ianw | clarkb: bah, sorry, copy paste errror, i meant https://review.openstack.org/92748 but yes, that's gone through now, thanks! | 22:35 |
| *** changbl has quit IRC | 22:37 | |
| clarkb | mtreinish: you still around? | 22:38 |
| anteaya | this patch has a broken link in the comment, which apparently is intentional and I disagree with it: https://review.openstack.org/#/c/94044/ | 22:38 |
| clarkb | mtreinish: your change to grab the failure logs should rename them to .txt files | 22:38 |
| anteaya | anyone else want to weigh in? | 22:38 |
| anteaya | broken link in the commit message | 22:38 |
| *** HenryG has quit IRC | 22:38 | |
| *** HenryG has joined #openstack-infra | 22:38 | |
| *** lxsli has quit IRC | 22:39 | |
| openstackgerrit | Mathew Odden proposed a change to openstack-dev/cookiecutter: Fix documentation template issues https://review.openstack.org/94492 | 22:39 |
| *** yamahata has joined #openstack-infra | 22:40 | |
| *** david-lyle has quit IRC | 22:43 | |
| *** james_li has quit IRC | 22:43 | |
| fungi | anteaya: glimpsed it briefly, and i'm not personally as concerned about commit messages--a url in a commit message doesn't *necessarily* have to link to something real. while i agree the commit message could probably be written just as clearly omitting it, i'm also not particularly picky about it | 22:44 |
| *** homeless has quit IRC | 22:45 | |
| fungi | others, of course, are welcome to (and almost certainly do in some cases) disagree | 22:45 |
| krotscheck | So if I want to create a new spec, does that go into specs/storyboard? specs/implemented? specs/implemented/storyboard? | 22:46 |
| zaro | msabramo: have you requested a pbr release for this? https://review.openstack.org/#/c/87382 | 22:47 |
| *** otherwiseguy has quit IRC | 22:49 | |
| *** mriedem has quit IRC | 22:49 | |
| anteaya | fungi: k thanks | 22:50 |
| openstackgerrit | Clark Boylan proposed a change to openstack-infra/devstack-gate: localize variables https://review.openstack.org/85309 | 22:50 |
| anteaya | I think links should work | 22:50 |
| anteaya | and samples should be samples | 22:50 |
| clarkb | sdague: ^ I took the liberty of fixing that change for you | 22:50 |
| clarkb | it needed a non trivial rebase | 22:50 |
| *** atiwari has quit IRC | 22:51 | |
| *** sarob has joined #openstack-infra | 22:51 | |
| *** bknudson has joined #openstack-infra | 22:54 | |
| *** jaypipes has quit IRC | 22:54 | |
| *** nati_ueno has quit IRC | 22:54 | |
| *** sarob has quit IRC | 22:56 | |
| *** msabramo has quit IRC | 23:00 | |
| clarkb | mordred_: sdague SergeyLukjanov jeblair fungi 92736 is something we should probably have an inclusive discussion about | 23:01 |
| clarkb | gertty should post an initial comment from me shortly | 23:01 |
| jeblair | krotscheck: according to the docs in https://review.openstack.org/#/c/94440/ just 'specs/'. but that's a good idea, feel free to -1 that and suggest it. | 23:03 |
| *** amcrn_ has joined #openstack-infra | 23:04 | |
| jeblair | clarkb: i'm eagerly awaiting your comment on that! what's your sync counter? :) | 23:05 |
| *** msabramo has joined #openstack-infra | 23:05 | |
| *** jgrimm has quit IRC | 23:06 | |
| *** amcrn has quit IRC | 23:06 | |
| clarkb | sync counter is 241 | 23:07 |
| clarkb | it got to 1.7k earlier today and I restarted it | 23:07 |
| clarkb | I think it doesn't trend to 0 for some reason | 23:07 |
| fungi | comment just appeared | 23:08 |
| msabramo | zaro: No I haven't requested a pbr release; don't know what the process is for that | 23:08 |
| anteaya | msabramo: I think the process is bug mordred | 23:09 |
| jeblair | clarkb: that's disturbing and worth looking into | 23:09 |
| anteaya | do we have something more formalized than that? | 23:09 |
| fungi | anteaya: "bug mordred" isn't formal? | 23:09 |
| msabramo | :-) | 23:09 |
| clarkb | jeblair: ya, I ran it under debug mode once but I don't think it happened then | 23:10 |
| mordred | anteaya: what's the thing we need a release for? | 23:10 |
| fungi | and no, "bug someone" is about the most formalized solution we have for requesting releases | 23:10 |
| clarkb | jeblair: do you tend to float around 0? | 23:10 |
| mordred | (also, lifeless and I are close to landing the version update code) | 23:10 |
| mordred | so I'd like to just do one release instead of two | 23:10 |
| jeblair | clarkb: yes; i also generally run in debug mode | 23:10 |
| msabramo | mordred: pbr, specifically jjb is interested in https://review.openstack.org/#/c/87382 | 23:10 |
| openstackgerrit | Eoghan Glynn proposed a change to openstack-infra/config: Create telemetry-specs git repo https://review.openstack.org/94044 | 23:10 |
| msabramo | oh oops that's the jjb bug | 23:11 |
| msabramo | https://review.openstack.org/89205 is the pbr task | 23:11 |
| jeblair | clarkb: i only have 14 projects susbscribed right now | 23:11 |
| mordred | msabramo: I support releasing based on that | 23:11 |
| *** virmitio has quit IRC | 23:11 | |
| clarkb | jeblair: I think I have more than that | 23:11 |
| clarkb | jeblair: also, debug logs were huge and I didn't want to make my ssd have a sad :) | 23:11 |
| msabramo | mordred: cool! | 23:12 |
| msabramo | that will let JJB start to measure coverage | 23:12 |
| jeblair | clarkb: moar ssds. mine is 67M since i started it this morning. | 23:13 |
| *** nati_ueno has joined #openstack-infra | 23:14 | |
| Shrews | clarkb: you want i should change the commit msg for you on 94456? huh? huh? huh? :) | 23:15 |
| clarkb | jeblair: I have closer to 50 projects subscirbed | 23:15 |
| clarkb | Shrews: I don't think it is super necessary | 23:15 |
| clarkb | Shrews: but a lot of commit messages do leave out the why :) | 23:15 |
| *** flaper87 is now known as flaper87|afk | 23:15 | |
| *** sarob has joined #openstack-infra | 23:15 | |
| openstackgerrit | Eoghan Glynn proposed a change to openstack-infra/config: Create telemetry-specs git repo https://review.openstack.org/94044 | 23:16 |
| Shrews | clarkb: k. but short answer - we want to test 'nova rebuild --preserve-ephemeral' | 23:16 |
| clarkb | jeblair: I am trimming my list of subscriptions now | 23:17 |
| *** aconrad has quit IRC | 23:18 | |
| clarkb | Alex_Gaynor: for 93458 maybe we can just stop using nose for ypthon 3? | 23:19 |
| *** dstanek_zzz is now known as dstanek | 23:19 | |
| Alex_Gaynor | clarkb: that's reasonably, I honestly don't have an opinion... do we even use nose? I don't know how testtools testrepository works | 23:19 |
| clarkb | Alex_Gaynor: some projects still use nose | 23:20 |
| clarkb | I think they should stop using nose :) | 23:20 |
| mordred | Alex_Gaynor: nose is deprecated, but still in use | 23:20 |
| Alex_Gaynor | That's reasonably, I think this was probably swift | 23:20 |
| clarkb | yes swift is one of them | 23:20 |
| mordred | what's the issue? | 23:20 |
| clarkb | I really want that nose plugin to die | 23:20 |
| clarkb | mordred: patches against nose html output | 23:20 |
| mordred | oh. weird | 23:20 |
| mordred | well, I mean, if someone wants to own it, I don't care | 23:21 |
| clarkb | well we own | 23:21 |
| clarkb | * own it | 23:21 |
| Alex_Gaynor | I definitely dont' want to own it :-) | 23:21 |
| mordred | we're about to do renames ... why not put it on stackforge and give it away | 23:21 |
| Alex_Gaynor | Swift test-requirements.txt require that, and it doesn't work on py3 | 23:21 |
| *** eglynn has joined #openstack-infra | 23:21 | |
| jeblair | mordred: do we use it? | 23:21 |
| clarkb | mordred: that requires a group of people that care about it | 23:21 |
| clarkb | yes swift requires it for html output for nose runs | 23:21 |
| mordred | clarkb: right, I'm saying, if we're not willing to take patches for it, and someone wants to write patches for it | 23:21 |
| mordred | then we should reconsider its ownership structure | 23:22 |
| mordred | it it's important that we own it, then we should take patches | 23:22 |
| clarkb | mordred: I think its ok to not take patches that make it future compatible | 23:22 |
| jeblair | mordred: we don't have to take every patch given. this is starting to turn into a strange conversation. | 23:22 |
| clarkb | fixing bugs for today is a different story | 23:22 |
| clarkb | and python3 for swift is a future thing | 23:22 |
| mordred | jeblair: yes. I think I may have been trying to nudge it that way | 23:22 |
| mordred | without meaning to | 23:23 |
| mordred | clarkb: that's fair | 23:23 |
| mordred | but if they're working on getting swift python3 compat and this is a blocker - and also there is no work on getting swift on to testr ... | 23:23 |
| eglynn | jeblair: note s/ceilometer-specs/telemetry-specs/ as you discussed earlier with the PTL group ... https://review.openstack.org/94044 | 23:23 |
| *** paul--_ has joined #openstack-infra | 23:24 | |
| mordred | I mean, I dunno. maybe we just make a stronger stance about testr there? | 23:24 |
| jeblair | eglynn: cool, thanks | 23:24 |
| jeblair | mordred, SergeyLukjanov: fyi i just sent mail about reviewing changes for new specs repos. | 23:25 |
| mordred | jeblair: awesome. will go read | 23:25 |
| sdague | jeblair: was at dinner, reading scrollback | 23:26 |
| openstackgerrit | Michael Krotscheck proposed a change to openstack-infra/infra-specs: New specification for StoryBoard UX overhaul https://review.openstack.org/94499 | 23:26 |
| clarkb | mordred: Alex_Gaynor I left a comment with my position. I am happy to move it to stackforge if people want to care and feed for the things | 23:26 |
| sdague | jeblair: I can change that part back to true/false. I could also use a distinct exception which we catch and pass for just the NothingToDo case | 23:26 |
| sdague | I was mostly trying to unnest the logic, as that seemed pretty error prone | 23:27 |
| clarkb | is 76796 safe to approve for the reason I mention? | 23:27 |
| clarkb | jeblair: sdague jhesketh ^ | 23:27 |
| jeblair | sdague: i'm not generally keen on the "exception for normal conditions" pattern, so i think it might be worth keeping that one level of nesting. | 23:28 |
| openstackgerrit | Eoghan Glynn proposed a change to openstack-infra/config: Create telemetry-specs git repo https://review.openstack.org/94044 | 23:28 |
| jeblair | sdague: i don't feel strongly about that though. | 23:28 |
| jeblair | clarkb: i have no independent knowledge of that but trust your evaluation, so +1 | 23:30 |
| clarkb | woo gertty fails on binary files | 23:30 |
| jhesketh | clarkb: sorry, haven't been following.. what was the question there? | 23:30 |
| openstackgerrit | A change was merged to openstack-infra/publications: Updated projects list to match today https://review.openstack.org/92837 | 23:30 |
| jeblair | jhesketh: it's in the review | 23:30 |
| *** aconrad has joined #openstack-infra | 23:31 | |
| jhesketh | jeblair: the nonexistant config file? | 23:31 |
| clarkb | ya | 23:31 |
| sdague | jeblair: ok, that's fair. I was trying to not mix the styles, but I don't feel strongly either. | 23:32 |
| mordred | jeblair, clarkb: HP has asked if we can make another try to move our workload to their new area- do we feel comfortable attempting that again? | 23:32 |
| sdague | clarkb: if you can baby sit it, 76796 seems fine. I really want to see it tried on a real env | 23:32 |
| *** gus has joined #openstack-infra | 23:32 | |
| openstackgerrit | Sean Dague proposed a change to openstack-infra/jeepyb: use exceptions instead of return True/False https://review.openstack.org/94196 | 23:32 |
| jeblair | mordred: i think you have the most knowledge about failure rates; if you are okay with it, i think it's great; az2 has been making us very unhappy. | 23:32 |
| sdague | jeblair: ok, how about that | 23:32 |
| jhesketh | jeblair, clarkb: that rounds right to me (re config not existing) | 23:33 |
| *** MarkAtwood has quit IRC | 23:33 | |
| clarkb | sdague: I can babysit | 23:33 |
| * clarkb rereads change really quickly then will approve | 23:33 | |
| mordred | jeblair: currently, 1.1 and ORD have the best rates | 23:33 |
| sdague | clarkb: it will be really interesting to see how it performs once it's out there somewhere | 23:34 |
| mordred | jeblair: 1.1 doesn't have great build times during peaks | 23:34 |
| *** markwash has joined #openstack-infra | 23:36 | |
| *** melwitt has quit IRC | 23:37 | |
| *** melwitt has joined #openstack-infra | 23:37 | |
| *** Sukhdev has quit IRC | 23:38 | |
| mordred | jeblair: now that we've got stats on stuff, I think it's worth a try | 23:38 |
| *** melwitt has quit IRC | 23:39 | |
| clarkb | jhesketh: sdague jeblair approved | 23:39 |
| clarkb | and it couldn't be merged apparently | 23:39 |
| clarkb | oh well, jhesketh want ot look into that? | 23:40 |
| openstackgerrit | Eoghan Glynn proposed a change to openstack-infra/config: Create telemetry-specs git repo https://review.openstack.org/94044 | 23:41 |
| openstackgerrit | Michael Krotscheck proposed a change to openstack-infra/infra-specs: New specification for StoryBoard UX overhaul https://review.openstack.org/94499 | 23:42 |
| *** yamahata has quit IRC | 23:43 | |
| *** malini is now known as malini_afk | 23:43 | |
| *** lyxus has quit IRC | 23:43 | |
| clarkb | jeblair: after cutting my subscription list in halfish I trend to 0 now | 23:46 |
| openstackgerrit | Monty Taylor proposed a change to openstack-infra/config: Revert "Revert "Stop using hpcloud az1-az3 in favor of region-b"" https://review.openstack.org/94503 | 23:47 |
| openstackgerrit | Monty Taylor proposed a change to openstack-infra/config: Revert "Revert "Stop using hpcloud az1-az3 in favor of region-b"" https://review.openstack.org/94503 | 23:48 |
| openstackgerrit | Monty Taylor proposed a change to openstack-infra/config: Revert "Revert "Stop using hpcloud az1-az3 in favor of region-b"" https://review.openstack.org/94503 | 23:48 |
| *** Ryan_Lane has quit IRC | 23:48 | |
| openstackgerrit | Sean Dague proposed a change to openstack-infra/jeepyb: add dashboard support to jeepyb https://review.openstack.org/94260 | 23:49 |
| *** nati_ueno has quit IRC | 23:50 | |
| mordred | jeblair, clarkb ^^ there's a change to move stuff back to HP 1.1 | 23:50 |
| *** nati_ueno has joined #openstack-infra | 23:51 | |
| *** wenlock has quit IRC | 23:52 | |
| *** eglynn has quit IRC | 23:52 | |
| *** Ryan_Lane has joined #openstack-infra | 23:52 | |
| sdague | pleia2: you still about? | 23:53 |
| *** homeless has joined #openstack-infra | 23:53 | |
| sdague | I'm thinking about - https://review.openstack.org/#/c/93608/1/modules/openstack_project/files/status/common.js and the conversation at summit | 23:53 |
| pleia2 | sdague: hey | 23:54 |
| pleia2 | sdague: yeah, we talked about it at our meeting today too, let me grab the link | 23:54 |
| sdague | right, i skimmed that | 23:54 |
| sdague | so my pov is that current rechecks page is broken, we shouldn't fix it, and we should just replace it completely with elastic recheck. | 23:54 |
| sdague | The question is whether we build back the bulk queries | 23:55 |
| *** mbacchi has quit IRC | 23:55 | |
| * pleia2 nods | 23:55 | |
| sdague | to use as possible bug indications for people classifying | 23:55 |
| sdague | but I think was can actually do that independantly | 23:55 |
| clarkb | https://etherpad.openstack.org/p/disabling-gerrit-drafts | 23:55 |
| clarkb | jeblair: zaro fungi mordred SergeyLukjanov and everyone else ^ | 23:55 |
| clarkb | how does that look? I am not sold on the date I just picked one very conservatively | 23:56 |
| sdague | pleia2: so I'd actually not add Elastic Recheck per say, i'd replace Rechecks with the Elastic Recheck page | 23:56 |
| pleia2 | sdague: since it's broken, is there any reason to hold off on replacement? | 23:56 |
| sdague | pleia2: no | 23:56 |
| jogo | pleia2: agreed | 23:56 |
| openstackgerrit | Marc Abramowitz proposed a change to stackforge/python-jenkins: Use pbr, pep8 checks & fixes, add "doc" tox target https://review.openstack.org/90455 | 23:56 |
| sdague | but my -1 on https://review.openstack.org/#/c/93608/1/modules/openstack_project/files/status/common.js would be basically, we should also remove the rechecks link | 23:57 |
| pleia2 | ok, I'll abandon the one change and adjust the other accordingly | 23:57 |
| pleia2 | thanks sdague and jogo | 23:57 |
| * jogo is happpy to see the old rechecks page go away | 23:57 | |
| sdague | pleia2: cool, thanks for doing it | 23:57 |
| sdague | I'm trying to get my bulk query hacky scripts to do the right thing here | 23:57 |
| jeblair | clarkb: one change to drafts email, otherwise, lgtm. | 23:57 |
| *** Ryan_Lane has quit IRC | 23:59 | |
| clarkb | thanks. How do people feel about the date? I could do this friday as well but we are already doing renaming then | 23:59 |
| *** msabramo has quit IRC | 23:59 | |
| clarkb | the change itself is tiny and easy to backout so I am not super worried about it | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!