*** armax has quit IRC | 00:01 | |
*** hdd has quit IRC | 00:02 | |
fungi | i love how the operators list explodes with malcontent each time we eol a stable branch | 00:02 |
---|---|---|
mordred | hehe | 00:03 |
*** wenlock has quit IRC | 00:04 | |
*** baoli has quit IRC | 00:05 | |
*** tomoe has left #openstack-infra | 00:07 | |
*** todd_dsm has joined #openstack-infra | 00:08 | |
clarkb | fungi has that happened already? I am not seeing it if so | 00:08 |
*** tgohad has joined #openstack-infra | 00:09 | |
*** JayJ has joined #openstack-infra | 00:10 | |
fungi | clarkb: http://lists.openstack.org/pipermail/openstack-operators/2014-September/005229.html | 00:10 |
krotscheck | Hrmp. That didn’t work. Ahh well, I’ll poke it more in the morning. | 00:11 |
sdague | jogo: so you caught a real bug, but not the one you think :) | 00:11 |
*** tsg has quit IRC | 00:11 | |
*** baoli has joined #openstack-infra | 00:12 | |
*** yamamoto has joined #openstack-infra | 00:13 | |
openstackgerrit | Sean Dague proposed a change to openstack-infra/elastic-recheck: fix Launchpad link name https://review.openstack.org/124928 | 00:13 |
sdague | jogo: the link in ER that I'm adding isn't to launchpad, it's to that graph on the page | 00:13 |
sdague | so http://status.openstack.org/elastic-recheck/#1334109 will take you to that bug entry in the er page | 00:15 |
jlk | it's almost as if people don't understand that if they want long term maintenance of a software set, they should get said software from a long term maintenance vendor | 00:15 |
jlk | like, say Red Hat | 00:15 |
*** tgohad has quit IRC | 00:17 | |
fungi | jlk: well, we'd love it if red hat, ubuntu, suse and others dedicated more resources to maintaining our stable branches upstream longer, but the unfortunate truth is that after a time it's too much work and things are far simpler for them to just keep it working with whatever software dependencies they shipped with it when it released | 00:17 |
jlk | sure, it's on them to keep the old stuff working within their offering | 00:18 |
jlk | where they have more control over the userland | 00:18 |
jlk | but end users can't expect that to come for free | 00:18 |
*** SumitNaiksatam has quit IRC | 00:18 | |
fungi | jlk: i'm actually thrilled that they and others manage to keep support for our releases updated and fresh for as much as a year, and take care of more than one in parallel even | 00:19 |
*** xchu has joined #openstack-infra | 00:19 | |
*** marcoemorais has quit IRC | 00:19 | |
jlk | yeah, as you stated in the thread, that's well beyond what most upstreams can manage | 00:19 |
jlk | most are HEAD or GTFO | 00:19 |
fungi | supporting stable branches upstream for longer is mainly just a question of resources, and so far people only manage to step up and keep it going for roughly a year per release | 00:20 |
*** otherwiseguy has joined #openstack-infra | 00:20 | |
*** bdpayne has quit IRC | 00:21 | |
*** zz_dimtruck is now known as dimtruck | 00:21 | |
fungi | but it's not like we arbitrarily choose to end support for releases, and i find it amusing that there are so many people who just don't get that | 00:21 |
krotscheck | A question about jjb- when I set the flag of ‘keep hierarchy’ to false, does it flatten everything (the uploaded files and the path in the project directory) or does it only flatten out the latter? | 00:21 |
*** marcoemorais has joined #openstack-infra | 00:21 | |
*** bhuvan has quit IRC | 00:23 | |
mordred | krotscheck: yes | 00:23 |
krotscheck | mordred: I’m confused. | 00:24 |
krotscheck | mordred: I need my draft config file to be in the same directory as the js build output, and right now it’s putting it into ~/test/draft/. I was hoping if I fiddled the keep-hierarchy bit it’d just dump all the files into ~/ | 00:25 |
krotscheck | But if that’s not the case I need to figure something else out | 00:25 |
*** baoli has quit IRC | 00:26 | |
*** praneshp has quit IRC | 00:26 | |
*** andreaf has quit IRC | 00:26 | |
*** marcoemorais has quit IRC | 00:27 | |
*** andreaf has joined #openstack-infra | 00:27 | |
*** marcoemorais has joined #openstack-infra | 00:27 | |
*** yamahata has quit IRC | 00:28 | |
*** unicell has joined #openstack-infra | 00:29 | |
*** praneshp has joined #openstack-infra | 00:30 | |
*** praneshp has quit IRC | 00:30 | |
mordred | hrm | 00:31 |
mordred | krotscheck: you have keep-hierarchy set to true though? | 00:32 |
*** dims has joined #openstack-infra | 00:32 | |
mordred | krotscheck: https://review.openstack.org/#/c/124606/2/jenkins/jobs/javascript.yaml | 00:33 |
mordred | or? | 00:33 |
*** baoli has joined #openstack-infra | 00:33 | |
krotscheck | mordred: Yeah, so I’m getting a ‘~/dist/‘ folder with the compiled assets, and a ~/test/draft folder with the config file. Optimally I want them both in the same directory. | 00:33 |
krotscheck | mordred: http://paste.openstack.org/show/116914/ | 00:34 |
*** mmaglana has quit IRC | 00:35 | |
fungi | krotscheck: the short and unfortunate answer is that the scp-publisher plugin can either give you a flat dump of all the files, or preserve the original directory structure and root it at a location of your choosing, but there's no good path mangling solution for it | 00:36 |
fungi | krotscheck: so if possible, your job should arrange the files relative to your git clone working directory exactly how and where you expect to see them under the target directory | 00:37 |
*** todd_dsm has quit IRC | 00:39 | |
krotscheck | ALright, that should be doable | 00:39 |
krotscheck | Just not optimal | 00:39 |
*** bradjones has quit IRC | 00:40 | |
*** todd_dsm has joined #openstack-infra | 00:40 | |
fungi | yeah, the ftp publisher plugin can strip leading path parts from the source. a feature i really wish the scp publisher had | 00:40 |
fungi | you know, without giving your passwords to anyone who has a router between you and the server | 00:41 |
*** marcoemorais has quit IRC | 00:43 | |
*** gyee has quit IRC | 00:46 | |
openstackgerrit | patrick-crews proposed a change to openstack-infra/elastic-recheck: Adding query namespace to elastic-recheck https://review.openstack.org/101958 | 00:47 |
*** bradjones has joined #openstack-infra | 00:48 | |
*** bradjones has quit IRC | 00:48 | |
*** bradjones has joined #openstack-infra | 00:48 | |
mordred | fungi: or, we can just replace it all with swift | 00:48 |
mordred | fungi: OR | 00:49 |
mordred | fungi: wait for it | 00:49 |
mordred | fungi: wait for it | 00:49 |
mordred | fungi: wait for it | 00:49 |
mordred | fungi: AFS | 00:49 |
mordred | BOOM | 00:49 |
* fungi didn't see that coming at all | 00:49 | |
*** rwsu has quit IRC | 00:49 | |
* mordred drops amazing on everyone | 00:49 | |
*** gokrokve has joined #openstack-infra | 00:49 | |
*** salv-orlando has quit IRC | 00:49 | |
*** harlowja_away is now known as harlowja | 00:49 | |
mordred | fungi: we could write a bot that just responds to all questions with "use AFS" | 00:50 |
*** gokrokve has quit IRC | 00:51 | |
*** gokrokve_ has joined #openstack-infra | 00:51 | |
*** praneshp has joined #openstack-infra | 00:51 | |
*** chuckC has quit IRC | 00:53 | |
morganfainberg | mordred, lol! | 00:53 |
fungi | mordred: i thought that was already your core routine? | 00:54 |
mordred | fungi: use AFS! | 00:54 |
morganfainberg | mordred, so let me get this straight, AFS? | 00:56 |
*** esker has joined #openstack-infra | 00:57 | |
mordred | morganfainberg: use AFS! | 00:57 |
*** todd_dsm has quit IRC | 00:58 | |
openstackgerrit | A change was merged to openstack-dev/hacking: Stop using intersphinx https://review.openstack.org/121328 | 01:00 |
mordred | woot | 01:00 |
*** esker has quit IRC | 01:01 | |
*** marcoemorais has joined #openstack-infra | 01:01 | |
*** signed8bit has joined #openstack-infra | 01:02 | |
* nibalizer considers the benefits of an AFS hilight | 01:02 | |
mordred | nibalizer: you know the good topics have come up when AFS gets dropped | 01:02 |
nibalizer | mordred: this? http://www.openafs.org/ | 01:02 |
mordred | nibalizer: yup | 01:02 |
* nibalizer wonders if this is like kerberos and has competing impletmenations? | 01:02 | |
fungi | nibalizer: it relies heavily on kerberos | 01:03 |
fungi | and yeah, there are a few afs implementations | 01:03 |
nibalizer | oh look there are openafs packages in ubuntu | 01:03 |
mordred | yeah. it's been a mainstream thing for a WHILE | 01:04 |
fungi | nibalizer: http://en.wikipedia.org/wiki/Andrew_File_System#Implementations | 01:04 |
dstufft | mordred: I considered using AFS | 01:04 |
*** ChuckC_ has quit IRC | 01:04 | |
nibalizer | 'a fourth implementation, unnamed, has been randomly shoved into the kernel by RedHat' | 01:05 |
mordred | dstufft: corvus and I are going to spin up an AFS cell for infra when he gets back from vacation | 01:05 |
fungi | it'll be just like old times! ;) | 01:05 |
mordred | fungi: finally | 01:05 |
*** andreaf has quit IRC | 01:05 | |
fungi | between that and the nntp backend for our listserv... | 01:05 |
mordred | fungi: I'm really not kidding about that ... | 01:05 |
* fungi takes back the 90s | 01:06 | |
dstufft | mordred: instead we used glusterfs | 01:06 |
*** andreaf has joined #openstack-infra | 01:06 | |
*** otherwiseguy has quit IRC | 01:06 | |
dstufft | I'm not sure we made the right choice | 01:06 |
mordred | fungi: we could replace freenode IRC with a LambdaMOO server | 01:06 |
mordred | dstufft: I think they solve slightly different usecases | 01:06 |
fungi | mordred: tinymush ftw, luser! | 01:06 |
mordred | dstufft: glusterfs to me seems really great if you do want something that produces NFS mounts | 01:07 |
* fungi jokes... i mostly developed extensions to proper mud codebases, after all... mosly diku* | 01:07 | |
dstufft | mordred: glusterfs might be great if you have consitent IO | 01:08 |
mordred | yes. that too | 01:08 |
dstufft | It's not great for anything except frustration if you don't have consistent IO :( | 01:08 |
fungi | dstufft: on the other hand, if you stick it in a cloud... | 01:08 |
mordred | ugh | 01:08 |
*** shashankhegde has quit IRC | 01:09 | |
mordred | fungi: how fun would it be if instead of infra we had rooms in a MUD - so that if you came in to infra, you actually had an outer room with a dragon or something | 01:09 |
*** unicell has quit IRC | 01:09 | |
*** wenlock has joined #openstack-infra | 01:09 | |
mordred | fungi: and if we could ACTUALLY throw wet cats at people | 01:10 |
mordred | I realize I mistyped | 01:10 |
mordred | s/instead of infra/instead of IRC/ | 01:10 |
*** otherwiseguy has joined #openstack-infra | 01:10 | |
* fungi dons his complimentary set of +1 newbie armor | 01:11 | |
* fungi swings his sword at bug 1367907 | 01:11 | |
uvirtbot | Launchpad bug 1367907 in python3.4 "Segfault in gc with cyclic trash" [High,In progress] https://launchpad.net/bugs/1367907 | 01:11 |
* fungi misses, flailing wildly | 01:11 | |
dstufft | mordred: can I add like 12 dragons to participate in distutils-sig | 01:11 |
fungi | dstufft: and train them to eat anyone who wants to be able to silently reupload different package contents with the same version numbers? | 01:12 |
dstufft | yes | 01:13 |
dstufft | also other bad things | 01:13 |
fungi | because, you know, if you dictate sane release process to projects, the ones with crappy ideas about how to do releases will flee to github tarball distribution instead, leaving only good software on pypi | 01:13 |
fungi | and that makes the psf cry | 01:14 |
*** gokrokve_ has quit IRC | 01:14 | |
*** marcoemorais has quit IRC | 01:15 | |
*** rwsu has joined #openstack-infra | 01:15 | |
*** gokrokve has joined #openstack-infra | 01:15 | |
*** SumitNaiksatam has joined #openstack-infra | 01:15 | |
dstufft | fungi: I like the idea that people are fleeing PyPI | 01:15 |
clarkb | fungi++ | 01:16 |
dstufft | fungi: when in reality, this is what PyPI's usage looks like -> http://d.stufft.io/image/1f0j0k2u1r0Q | 01:16 |
fungi | dstufft: particularly the sorts of people who would get miffed if you told them they could no longer follow user-hostile release practices | 01:16 |
dstufft | (to be fair, that's downloads not uploads) | 01:16 |
*** kgavini has joined #openstack-infra | 01:17 | |
*** marcoemorais has joined #openstack-infra | 01:18 | |
dstufft | or like http://d.stufft.io/image/1O1K0j0n390s (total requests to PyPI per day) | 01:19 |
*** marcoemorais has quit IRC | 01:19 | |
*** gokrokve has quit IRC | 01:20 | |
*** marcoemorais has joined #openstack-infra | 01:20 | |
*** sdake_ has joined #openstack-infra | 01:20 | |
*** otherwiseguy has quit IRC | 01:23 | |
*** alexpilotti has quit IRC | 01:23 | |
*** otherwiseguy has joined #openstack-infra | 01:23 | |
*** marcoemorais has quit IRC | 01:24 | |
*** baoli has quit IRC | 01:24 | |
*** baoli has joined #openstack-infra | 01:25 | |
*** pcrews has quit IRC | 01:25 | |
openstackgerrit | Michael Krotscheck proposed a change to openstack-infra/storyboard-webclient: Added draft build https://review.openstack.org/124934 | 01:26 |
*** marcoemorais has joined #openstack-infra | 01:26 | |
*** signed8bit has quit IRC | 01:27 | |
*** pcrews has joined #openstack-infra | 01:27 | |
openstackgerrit | Michael Krotscheck proposed a change to openstack-infra/project-config: Call build:draft instead of build for js-draft build https://review.openstack.org/124936 | 01:28 |
*** yaguang has joined #openstack-infra | 01:29 | |
*** cyeoh has joined #openstack-infra | 01:31 | |
*** yamahata has joined #openstack-infra | 01:32 | |
*** mikedillion has joined #openstack-infra | 01:33 | |
*** ChuckC_ has joined #openstack-infra | 01:34 | |
*** chuckC has joined #openstack-infra | 01:35 | |
*** patrickeast has quit IRC | 01:38 | |
*** Sukhdev has quit IRC | 01:38 | |
*** sdake_ has quit IRC | 01:38 | |
*** marcoemorais has quit IRC | 01:39 | |
*** unicell has joined #openstack-infra | 01:42 | |
*** sdake_ has joined #openstack-infra | 01:44 | |
*** mikedillion has quit IRC | 01:47 | |
openstackgerrit | Matthew Treinish proposed a change to openstack-infra/subunit2sql: Add migration tests with data https://review.openstack.org/124938 | 01:50 |
openstackgerrit | Matthew Treinish proposed a change to openstack-infra/subunit2sql: Preseed data for migration tests until migration 163fd5aa1380 https://review.openstack.org/124939 | 01:50 |
*** baoli has quit IRC | 01:50 | |
*** tsg has joined #openstack-infra | 01:51 | |
openstackgerrit | Matthew Treinish proposed a change to openstack-infra/subunit2sql: Add order column to run table https://review.openstack.org/117861 | 01:51 |
openstackgerrit | Matthew Treinish proposed a change to openstack-infra/subunit2sql: Add db api function to get the latest run https://review.openstack.org/117882 | 01:51 |
openstackgerrit | Matthew Treinish proposed a change to openstack-infra/subunit2sql: Add session objects for get_all* db api calls https://review.openstack.org/118419 | 01:52 |
openstackgerrit | Matthew Treinish proposed a change to openstack-infra/subunit2sql: Add migration tests with data https://review.openstack.org/124938 | 01:53 |
openstackgerrit | Matthew Treinish proposed a change to openstack-infra/subunit2sql: Preseed data for migration tests until migration 163fd5aa1380 https://review.openstack.org/124939 | 01:53 |
openstackgerrit | Matthew Treinish proposed a change to openstack-infra/subunit2sql: Add session objects for get_all* db api calls https://review.openstack.org/118419 | 01:53 |
openstackgerrit | Matthew Treinish proposed a change to openstack-infra/subunit2sql: Add order column to run table https://review.openstack.org/117861 | 01:53 |
openstackgerrit | Matthew Treinish proposed a change to openstack-infra/subunit2sql: Add db api function to get the latest run https://review.openstack.org/117882 | 01:53 |
*** Ryan_Lane has quit IRC | 01:56 | |
*** nosnos has joined #openstack-infra | 01:56 | |
*** signed8bit has joined #openstack-infra | 01:56 | |
*** unicell has quit IRC | 01:59 | |
*** _buzz has joined #openstack-infra | 02:04 | |
*** ivar-laz_ has joined #openstack-infra | 02:05 | |
*** bdpayne has joined #openstack-infra | 02:07 | |
*** daya_k has joined #openstack-infra | 02:08 | |
*** ivar-lazzaro has quit IRC | 02:09 | |
*** aysyanne has quit IRC | 02:09 | |
*** ivar-laz_ has quit IRC | 02:09 | |
*** markmcclain has quit IRC | 02:10 | |
*** signed8bit has quit IRC | 02:11 | |
*** shashankhegde has joined #openstack-infra | 02:22 | |
*** unicell has joined #openstack-infra | 02:25 | |
*** dimtruck is now known as zz_dimtruck | 02:25 | |
*** arosen1 has joined #openstack-infra | 02:25 | |
*** otter768 has joined #openstack-infra | 02:26 | |
openstackgerrit | A change was merged to openstack-infra/subunit2sql: Stop using intersphinx https://review.openstack.org/123973 | 02:28 |
*** JayJ has quit IRC | 02:30 | |
*** arosen1 has left #openstack-infra | 02:33 | |
*** daya_k has quit IRC | 02:35 | |
*** melwitt1 has quit IRC | 02:37 | |
*** dims has quit IRC | 02:42 | |
*** dims has joined #openstack-infra | 02:43 | |
*** tsg has quit IRC | 02:43 | |
pleia2 | AJaeger: thanks :) | 02:44 |
*** mmaglana has joined #openstack-infra | 02:45 | |
*** dims has quit IRC | 02:47 | |
*** r1chardj0n3s is now known as r1chardj0n3s_afk | 02:47 | |
*** sdake_ has quit IRC | 02:47 | |
*** kgavini has quit IRC | 02:49 | |
*** shashankhegde has quit IRC | 02:54 | |
*** pcrews has quit IRC | 02:56 | |
*** pcrews has joined #openstack-infra | 02:57 | |
*** dims has joined #openstack-infra | 02:57 | |
*** daya_k has joined #openstack-infra | 02:58 | |
jhesketh | sdague: let me know when you're around to discuss the zuul status page stuff | 03:01 |
openstackgerrit | Matthew Treinish proposed a change to openstack-infra/subunit2sql: Add migration tests with data https://review.openstack.org/124938 | 03:01 |
openstackgerrit | Matthew Treinish proposed a change to openstack-infra/subunit2sql: Preseed data for migration tests until migration 163fd5aa1380 https://review.openstack.org/124939 | 03:02 |
openstackgerrit | Matthew Treinish proposed a change to openstack-infra/subunit2sql: Add session objects for get_all* db api calls https://review.openstack.org/118419 | 03:02 |
openstackgerrit | Matthew Treinish proposed a change to openstack-infra/subunit2sql: Add order column to run table https://review.openstack.org/117861 | 03:02 |
openstackgerrit | Matthew Treinish proposed a change to openstack-infra/subunit2sql: Add db api function to get the latest run https://review.openstack.org/117882 | 03:02 |
*** pcrews has quit IRC | 03:04 | |
*** andreaf has quit IRC | 03:04 | |
*** andreaf has joined #openstack-infra | 03:05 | |
*** dims has quit IRC | 03:07 | |
*** dims has joined #openstack-infra | 03:08 | |
*** mestery has joined #openstack-infra | 03:08 | |
*** koolhead17 has joined #openstack-infra | 03:10 | |
*** dims has quit IRC | 03:12 | |
*** bdpayne has quit IRC | 03:12 | |
openstackgerrit | Matthew Treinish proposed a change to openstack-infra/subunit2sql: Add order column to run table https://review.openstack.org/117861 | 03:16 |
*** pcrews has joined #openstack-infra | 03:21 | |
*** nosnos has quit IRC | 03:22 | |
*** nosnos has joined #openstack-infra | 03:23 | |
*** armax has joined #openstack-infra | 03:24 | |
*** harlowja is now known as harlowja_away | 03:26 | |
*** hdd has joined #openstack-infra | 03:26 | |
*** _buzz has quit IRC | 03:27 | |
*** nosnos has quit IRC | 03:27 | |
*** camunoz has quit IRC | 03:31 | |
*** harlowja_away is now known as harlowja | 03:32 | |
*** mestery has quit IRC | 03:33 | |
*** andreaf has quit IRC | 03:33 | |
*** andreaf has joined #openstack-infra | 03:34 | |
*** armax has quit IRC | 03:38 | |
*** adalbas has quit IRC | 03:38 | |
*** mestery has joined #openstack-infra | 03:40 | |
openstackgerrit | Elizabeth K. Joseph proposed a change to openstack-infra/project-config: Move python33 test for Keystone to experimental https://review.openstack.org/124563 | 03:42 |
*** r1chardj0n3s_afk is now known as r1chardj0n3s | 03:42 | |
*** zns has quit IRC | 03:42 | |
morganfainberg | pleia2, thanks for the spell-check :) | 03:43 |
*** zns has joined #openstack-infra | 03:43 | |
pleia2 | sure thing :) | 03:43 |
*** mestery has quit IRC | 03:47 | |
*** zns has quit IRC | 03:48 | |
*** otter768 has quit IRC | 03:48 | |
*** Ryan_Lane has joined #openstack-infra | 03:54 | |
*** Ryan_Lane has quit IRC | 03:58 | |
*** pcrews has quit IRC | 03:59 | |
*** rushiagr_away is now known as rushiagr | 04:01 | |
*** mrmartin has joined #openstack-infra | 04:02 | |
*** marcoemorais has joined #openstack-infra | 04:03 | |
*** marcoemorais1 has joined #openstack-infra | 04:05 | |
*** KanagarajM has joined #openstack-infra | 04:07 | |
*** marcoemorais has quit IRC | 04:08 | |
*** otherwiseguy has quit IRC | 04:08 | |
*** garyh has quit IRC | 04:10 | |
openstackgerrit | Matthew Treinish proposed a change to openstack-infra/subunit2sql: Add migration tests with data https://review.openstack.org/124938 | 04:10 |
openstackgerrit | Matthew Treinish proposed a change to openstack-infra/subunit2sql: Preseed data for migration tests until migration 163fd5aa1380 https://review.openstack.org/124939 | 04:10 |
openstackgerrit | Matthew Treinish proposed a change to openstack-infra/subunit2sql: Add session objects for get_all* db api calls https://review.openstack.org/118419 | 04:10 |
openstackgerrit | Matthew Treinish proposed a change to openstack-infra/subunit2sql: Add order column to run table https://review.openstack.org/117861 | 04:10 |
openstackgerrit | Matthew Treinish proposed a change to openstack-infra/subunit2sql: Add db api function to get the latest run https://review.openstack.org/117882 | 04:10 |
*** marcoemorais1 has left #openstack-infra | 04:11 | |
*** nosnos has joined #openstack-infra | 04:14 | |
*** otherwiseguy has joined #openstack-infra | 04:18 | |
*** shashankhegde has joined #openstack-infra | 04:20 | |
*** HeOS has joined #openstack-infra | 04:22 | |
*** ildikov_ has quit IRC | 04:28 | |
*** camunoz has joined #openstack-infra | 04:29 | |
*** otherwiseguy has quit IRC | 04:29 | |
*** yfried has quit IRC | 04:30 | |
*** KanagarajM has quit IRC | 04:35 | |
*** Daisy has joined #openstack-infra | 04:35 | |
*** unicell has quit IRC | 04:37 | |
*** yamahata has quit IRC | 04:40 | |
*** yamahata has joined #openstack-infra | 04:41 | |
*** mestery has joined #openstack-infra | 04:41 | |
*** mrmartin has quit IRC | 04:48 | |
*** mrmartin has joined #openstack-infra | 04:48 | |
openstackgerrit | Matthew Treinish proposed a change to openstack-infra/subunit2sql: Add order column to run table https://review.openstack.org/117861 | 04:48 |
*** mrmartin has quit IRC | 04:48 | |
openstackgerrit | Matthew Treinish proposed a change to openstack-infra/subunit2sql: Add db api function to get the latest run https://review.openstack.org/117882 | 04:49 |
*** rushiagr is now known as rushiagr_away | 04:51 | |
*** gokrokve has joined #openstack-infra | 04:56 | |
*** patrickeast has joined #openstack-infra | 04:57 | |
*** mestery has quit IRC | 04:57 | |
*** unicell has joined #openstack-infra | 04:58 | |
*** mestery has joined #openstack-infra | 04:59 | |
*** YorikSar has quit IRC | 05:01 | |
*** loki184 has joined #openstack-infra | 05:01 | |
*** hdd has quit IRC | 05:02 | |
*** YorikSar has joined #openstack-infra | 05:03 | |
*** KanagarajM has joined #openstack-infra | 05:03 | |
*** patrickeast has quit IRC | 05:04 | |
*** patrickeast has joined #openstack-infra | 05:05 | |
*** patrickeast has quit IRC | 05:09 | |
*** mmedvede has quit IRC | 05:12 | |
*** patrickeast has joined #openstack-infra | 05:13 | |
*** reed has quit IRC | 05:16 | |
*** ildikov has joined #openstack-infra | 05:19 | |
*** KanagarajM2 has joined #openstack-infra | 05:19 | |
*** KanagarajM has quit IRC | 05:20 | |
*** andreaf has quit IRC | 05:20 | |
*** andreaf has joined #openstack-infra | 05:21 | |
*** HeOS has quit IRC | 05:21 | |
*** unicell has quit IRC | 05:24 | |
*** yfried has joined #openstack-infra | 05:28 | |
*** sressot has joined #openstack-infra | 05:30 | |
*** rushiagr_away is now known as rushiagr | 05:32 | |
*** dalgaaf__ has joined #openstack-infra | 05:35 | |
*** timrc is now known as timrc-afk | 05:35 | |
*** mestery has quit IRC | 05:36 | |
*** waynr has quit IRC | 05:39 | |
*** patrickeast has quit IRC | 05:41 | |
*** cristov_mac has quit IRC | 05:42 | |
*** linuxmohan has joined #openstack-infra | 05:44 | |
*** linuxmohan has quit IRC | 05:44 | |
*** linuxmohan has joined #openstack-infra | 05:45 | |
*** andreaf has quit IRC | 05:46 | |
*** berendt has joined #openstack-infra | 05:47 | |
*** praneshp has quit IRC | 05:53 | |
*** ivar-lazzaro has joined #openstack-infra | 06:02 | |
*** ivar-lazzaro has joined #openstack-infra | 06:02 | |
*** harlowja is now known as harlowja_away | 06:05 | |
*** waynr has joined #openstack-infra | 06:06 | |
*** vigneshvar has joined #openstack-infra | 06:08 | |
*** unicell has joined #openstack-infra | 06:08 | |
*** rwsu has quit IRC | 06:09 | |
*** SumitNaiksatam_ has joined #openstack-infra | 06:11 | |
*** Ryan_Lane has joined #openstack-infra | 06:11 | |
*** SumitNaiksatam has quit IRC | 06:14 | |
*** SumitNaiksatam_ is now known as SumitNaiksatam | 06:14 | |
*** SumitNaiksatam has quit IRC | 06:21 | |
*** SumitNaiksatam has joined #openstack-infra | 06:22 | |
*** shashankhegde has quit IRC | 06:22 | |
*** stevemar has quit IRC | 06:24 | |
*** emagana has joined #openstack-infra | 06:26 | |
AJaeger | flaper87|afk, once you're back online, could you review https://review.openstack.org/#/c/116314/ again, please? | 06:26 |
*** praneshp has joined #openstack-infra | 06:27 | |
*** k4n0 has joined #openstack-infra | 06:27 | |
*** SumitNaiksatam has quit IRC | 06:29 | |
*** praneshp_ has joined #openstack-infra | 06:30 | |
*** ashaeron has joined #openstack-infra | 06:30 | |
*** SumitNaiksatam has joined #openstack-infra | 06:30 | |
*** praneshp has quit IRC | 06:32 | |
*** praneshp_ is now known as praneshp | 06:32 | |
*** emagana has quit IRC | 06:33 | |
*** emagana has joined #openstack-infra | 06:34 | |
*** SumitNaiksatam has quit IRC | 06:34 | |
*** SumitNaiksatam has joined #openstack-infra | 06:35 | |
*** emagana has quit IRC | 06:39 | |
*** nosnos has quit IRC | 06:39 | |
*** nosnos has joined #openstack-infra | 06:40 | |
*** nosnos has quit IRC | 06:41 | |
*** atuvenie has joined #openstack-infra | 06:41 | |
*** nosnos_ has joined #openstack-infra | 06:41 | |
*** Longgeek has joined #openstack-infra | 06:43 | |
*** Longgeek has quit IRC | 06:44 | |
*** wenlock has quit IRC | 06:44 | |
*** Longgeek has joined #openstack-infra | 06:46 | |
openstackgerrit | Adam Gandelman proposed a change to openstack-infra/elastic-recheck: Add query for 1375488 https://review.openstack.org/124913 | 06:46 |
*** jcoufal has joined #openstack-infra | 06:47 | |
*** jcoufal has quit IRC | 06:47 | |
*** Longgeek_ has joined #openstack-infra | 06:49 | |
*** camunoz has quit IRC | 06:51 | |
*** GheRivero has quit IRC | 06:52 | |
*** Longgeek has quit IRC | 06:52 | |
*** mmaglana has quit IRC | 06:55 | |
openstackgerrit | Yair Fried proposed a change to openstack-infra/devstack-gate: Adds tenant_network_conf to tempest.conf https://review.openstack.org/124768 | 06:58 |
*** GheRivero has joined #openstack-infra | 06:58 | |
*** xchu has quit IRC | 07:03 | |
*** hashar has joined #openstack-infra | 07:06 | |
*** arxcruz has joined #openstack-infra | 07:12 | |
*** arxcruz has quit IRC | 07:14 | |
*** mmagr has joined #openstack-infra | 07:16 | |
*** arxcruz has joined #openstack-infra | 07:17 | |
*** koolhead17 has quit IRC | 07:17 | |
*** _nadya_ has joined #openstack-infra | 07:18 | |
*** VijayTripathi has quit IRC | 07:18 | |
*** jcoufal has joined #openstack-infra | 07:22 | |
*** HeOS has joined #openstack-infra | 07:22 | |
*** _nadya_ has joined #openstack-infra | 07:23 | |
*** jcoufal has quit IRC | 07:23 | |
*** e0ne has joined #openstack-infra | 07:23 | |
*** jcoufal has joined #openstack-infra | 07:23 | |
*** e0ne has quit IRC | 07:24 | |
*** gokrokve has quit IRC | 07:26 | |
*** rdopieralski has joined #openstack-infra | 07:26 | |
*** Longgeek has joined #openstack-infra | 07:27 | |
*** ivar-lazzaro has quit IRC | 07:27 | |
*** Longgeek_ has quit IRC | 07:28 | |
*** _nadya_ has quit IRC | 07:28 | |
*** e0ne has joined #openstack-infra | 07:29 | |
*** yamamoto has quit IRC | 07:29 | |
*** Guest68510 has joined #openstack-infra | 07:30 | |
*** r1chardj0n3s is now known as r1chardj0n3s_afk | 07:31 | |
openstackgerrit | Mate Lakat proposed a change to openstack-infra/project-config: XenServer: Add XenServer support https://review.openstack.org/124730 | 07:32 |
*** jcoufal has quit IRC | 07:32 | |
*** ZZelle has quit IRC | 07:34 | |
*** jcoufal has joined #openstack-infra | 07:34 | |
*** ZZelle has joined #openstack-infra | 07:34 | |
*** koolhead17 has joined #openstack-infra | 07:35 | |
*** koolhead17 has quit IRC | 07:35 | |
*** koolhead17 has joined #openstack-infra | 07:35 | |
openstackgerrit | A change was merged to openstack-infra/infra-manual: Add an introduction paragraph to the developer guide https://review.openstack.org/107024 | 07:36 |
*** jp_at_hp has joined #openstack-infra | 07:36 | |
*** bogdando has quit IRC | 07:39 | |
*** jcoufal has quit IRC | 07:40 | |
*** sunrenjie6 has joined #openstack-infra | 07:40 | |
*** jcoufal has joined #openstack-infra | 07:40 | |
*** Longgeek has quit IRC | 07:40 | |
*** Longgeek has joined #openstack-infra | 07:42 | |
openstackgerrit | Thierry Carrez proposed a change to openstack-infra/release-tools: Fix rcdelivery.sh using wrong final version https://review.openstack.org/124963 | 07:42 |
*** ZZelle has quit IRC | 07:42 | |
*** ZZelle has joined #openstack-infra | 07:43 | |
openstackgerrit | Christoffer Skeppstedt proposed a change to openstack-infra/jenkins-job-builder: Serialize git-revision param for trigger_builds https://review.openstack.org/124965 | 07:46 |
*** bogdando has joined #openstack-infra | 07:46 | |
openstackgerrit | Joshua Hesketh proposed a change to openstack-infra/project-config: Run gate-{name}-docs on infra-publish-jobs https://review.openstack.org/124966 | 07:46 |
*** e0ne has quit IRC | 07:47 | |
*** skeppstedt has joined #openstack-infra | 07:50 | |
*** mpaolino has joined #openstack-infra | 07:51 | |
*** jistr has joined #openstack-infra | 07:55 | |
openstackgerrit | Jordan Pittier proposed a change to openstack-infra/project-config: Add pep8 gate job to stackforge/ospurge https://review.openstack.org/124729 | 07:57 |
*** andreykurilin_ has joined #openstack-infra | 07:57 | |
*** jlibosva has joined #openstack-infra | 07:58 | |
*** gokrokve has joined #openstack-infra | 07:58 | |
*** zz_johnthetubagu has joined #openstack-infra | 07:59 | |
*** zz_johnthetubagu is now known as johnthetubaguy | 07:59 | |
*** yamamoto has joined #openstack-infra | 07:59 | |
*** gokrokve has quit IRC | 08:00 | |
*** yamamoto has quit IRC | 08:00 | |
*** gokrokve has joined #openstack-infra | 08:00 | |
*** loki184 has quit IRC | 08:02 | |
*** e0ne has joined #openstack-infra | 08:02 | |
*** YorikSar has quit IRC | 08:02 | |
*** derekh has joined #openstack-infra | 08:03 | |
*** boden has joined #openstack-infra | 08:03 | |
*** YorikSar has joined #openstack-infra | 08:04 | |
*** gokrokve has quit IRC | 08:05 | |
*** ihrachyshka has joined #openstack-infra | 08:07 | |
*** e0ne has quit IRC | 08:13 | |
*** ociuhandu has quit IRC | 08:17 | |
*** e0ne has joined #openstack-infra | 08:17 | |
*** e0ne has quit IRC | 08:18 | |
*** afazekas has joined #openstack-infra | 08:18 | |
*** jgallard has joined #openstack-infra | 08:19 | |
*** MaxV has joined #openstack-infra | 08:19 | |
*** loki184 has joined #openstack-infra | 08:23 | |
*** cdent has joined #openstack-infra | 08:30 | |
*** amuller has joined #openstack-infra | 08:34 | |
*** doude has joined #openstack-infra | 08:39 | |
*** dizquierdo has joined #openstack-infra | 08:41 | |
*** andreykurilin_ has quit IRC | 08:44 | |
openstackgerrit | Mate Lakat proposed a change to openstack-infra/project-config: XenServer: Add XenServer support https://review.openstack.org/124730 | 08:50 |
*** andreaf_ is now known as andreaf | 08:59 | |
*** yfried is now known as yfried|afk | 08:59 | |
*** gokrokve has joined #openstack-infra | 08:59 | |
*** sdake_ has joined #openstack-infra | 09:00 | |
*** gokrokve has quit IRC | 09:04 | |
*** sdake_ has quit IRC | 09:05 | |
*** e0ne has joined #openstack-infra | 09:14 | |
*** ociuhandu has joined #openstack-infra | 09:14 | |
*** rlandy has joined #openstack-infra | 09:15 | |
*** mancdaz has joined #openstack-infra | 09:17 | |
*** che-arne has joined #openstack-infra | 09:24 | |
*** yfried|afk is now known as yfried | 09:26 | |
*** e0ne has quit IRC | 09:30 | |
*** alexpilotti has joined #openstack-infra | 09:33 | |
*** Daisy has quit IRC | 09:34 | |
*** praneshp has quit IRC | 09:34 | |
*** liusheng has quit IRC | 09:40 | |
*** e0ne has joined #openstack-infra | 09:43 | |
*** salv-orlando has joined #openstack-infra | 09:49 | |
openstackgerrit | Sergey Skripnick proposed a change to openstack-infra/project-config: Add rally-fakevirt performance job https://review.openstack.org/124996 | 09:52 |
sdague | jhesketh: up now | 09:55 |
*** koolhead17 has quit IRC | 09:58 | |
*** gokrokve has joined #openstack-infra | 09:59 | |
sbadia | \o hi infra | 10:01 |
sbadia | could I have a review on this two patchs ? ( https://review.openstack.org/#/c/124569/ for metadata.json on puppet-lint jobs, and https://review.openstack.org/#/c/124570/ for default jenkins builders for puppet-vswitch ) ? | 10:01 |
sbadia | thanks in advance =) | 10:02 |
openstackgerrit | Ricardo Carrillo Cruz proposed a change to openstack-infra/project-config: Add version-properties builder to macros.yaml https://review.openstack.org/124365 | 10:02 |
*** yfried_ has joined #openstack-infra | 10:02 | |
*** gokrokve has quit IRC | 10:03 | |
*** yfried has quit IRC | 10:03 | |
*** _nadya_ has joined #openstack-infra | 10:08 | |
*** yfried_ is now known as yfried|afk | 10:18 | |
*** hashar has quit IRC | 10:19 | |
*** sandywalsh has quit IRC | 10:20 | |
*** yfried|afk is now known as yfried_ | 10:22 | |
*** sandywalsh has joined #openstack-infra | 10:22 | |
*** jcoufal has quit IRC | 10:23 | |
*** jcoufal has joined #openstack-infra | 10:24 | |
*** pcm_ has joined #openstack-infra | 10:30 | |
*** pcm_ has quit IRC | 10:31 | |
*** pcm_ has joined #openstack-infra | 10:32 | |
*** yfried_ has quit IRC | 10:34 | |
*** pelix has joined #openstack-infra | 10:34 | |
openstackgerrit | Sean Dague proposed a change to openstack-infra/zuul: add support for getting status of individual changes https://review.openstack.org/124390 | 10:34 |
openstackgerrit | Sean Dague proposed a change to openstack-infra/zuul: add support for filtering changes in zuul webapp https://review.openstack.org/125002 | 10:34 |
*** yamahata has quit IRC | 10:35 | |
*** yfried has joined #openstack-infra | 10:40 | |
*** MindfulMonk has quit IRC | 10:46 | |
*** jgallard has quit IRC | 10:52 | |
*** MindfulMonk has joined #openstack-infra | 10:53 | |
*** habib__ has joined #openstack-infra | 10:54 | |
*** ildikov_ has joined #openstack-infra | 10:57 | |
*** MaxV has quit IRC | 10:57 | |
*** MaxV has joined #openstack-infra | 10:58 | |
*** gokrokve has joined #openstack-infra | 10:59 | |
*** ildikov has quit IRC | 11:01 | |
*** MaxV has quit IRC | 11:02 | |
*** loki184 has quit IRC | 11:02 | |
*** gokrokve has quit IRC | 11:04 | |
*** yfried_ has joined #openstack-infra | 11:04 | |
*** yfried__ has joined #openstack-infra | 11:05 | |
*** MaxV has joined #openstack-infra | 11:05 | |
*** yfried has quit IRC | 11:05 | |
*** hashar has joined #openstack-infra | 11:08 | |
*** dims has joined #openstack-infra | 11:09 | |
*** yfried_ has quit IRC | 11:09 | |
*** ildikov_ has quit IRC | 11:12 | |
*** ildikov has joined #openstack-infra | 11:12 | |
*** e0ne has quit IRC | 11:13 | |
*** signed8bit has joined #openstack-infra | 11:21 | |
*** zz_dimtruck is now known as dimtruck | 11:22 | |
*** mikal has quit IRC | 11:23 | |
*** loquacities has quit IRC | 11:23 | |
*** yfried__ is now known as yfried|afk | 11:28 | |
*** signed8bit has quit IRC | 11:29 | |
*** jistr is now known as jistr|english | 11:31 | |
openstackgerrit | Ricardo Carrillo Cruz proposed a change to openstack-infra/project-config: The current maven jobs in maven-plugin-jobs work fine for simple projects, but it cannot work for multi-module projects where multiple artifacts are generated with their corresponding pom.xml files. Also, it is not possible to pass project-specific settin https://review.openstack.org/125011 | 11:33 |
*** mrda-vacation has quit IRC | 11:33 | |
*** MaxV has quit IRC | 11:34 | |
*** MaxV has joined #openstack-infra | 11:34 | |
openstackgerrit | Ricardo Carrillo Cruz proposed a change to openstack-infra/project-config: Add support for multi-module maven jobs https://review.openstack.org/125011 | 11:34 |
*** mikal has joined #openstack-infra | 11:35 | |
jhesketh | sdague: left some comments on your patch.. I was going to help re-roll but wanted to check on your thoughts re the json structure (my main concern) | 11:36 |
sdague | jhesketh: so I respun | 11:36 |
sdague | based on your comments | 11:37 |
jhesketh | yep, just loading | 11:37 |
sdague | mostly I'd like to not get hung up on filtering structure to land the individual status patch | 11:37 |
*** timrc-afk is now known as timrc | 11:37 | |
sdague | because the individual status patch, once in, means I can write it into gerrit ui later the same day | 11:38 |
jhesketh | sdague: okay, sounds good to me | 11:40 |
sdague | I think that makes patch 1 landable, and patch 2 something to discuss | 11:40 |
jhesketh | sure | 11:40 |
sdague | because in the gate queue case, what would you expect to return for project=openstack/nova ? | 11:40 |
*** sressot has quit IRC | 11:40 | |
*** sandywalsh_ has joined #openstack-infra | 11:41 | |
*** yolanda has quit IRC | 11:41 | |
*** _nadya_ has quit IRC | 11:42 | |
jhesketh | sdague: what are your thoughts on having existing webapp tests into the new TestWebApp? | 11:42 |
sdague | jhesketh: I'd be happy to migrate them over | 11:42 |
sdague | I think it's a follow on patch though | 11:42 |
jhesketh | yeah, I'm not worried about it | 11:42 |
jhesketh | sdague: I was expecting it to return a json blob similar to how the javascript app filters things | 11:43 |
*** sandywalsh has quit IRC | 11:43 | |
*** sressot has joined #openstack-infra | 11:43 | |
sdague | jhesketh: ok, except that means that if you filter gate by openstack/nova | 11:43 |
*** yolanda has joined #openstack-infra | 11:43 | |
sdague | you get the whole gate queue | 11:43 |
sdague | which... isn't much filtering | 11:43 |
jhesketh | oh, I see your point | 11:43 |
jhesketh | sdague: well we wouldn't necessarily have to include the heads items (which is what I Thought the web app did, but testing it it appears to return the whole queue) | 11:44 |
*** kgiusti has joined #openstack-infra | 11:45 | |
sdague | yeh | 11:45 |
*** dhellmann has quit IRC | 11:48 | |
*** yfried|afk is now known as yfried__ | 11:48 | |
*** jhesketh_ has joined #openstack-infra | 11:49 | |
*** dhellmann has joined #openstack-infra | 11:50 | |
*** isviridov is now known as isviridov_away | 11:50 | |
*** jhesketh has quit IRC | 11:51 | |
ZZelle | adam_g, hi | 11:51 |
*** weshay has joined #openstack-infra | 11:53 | |
*** tsg has joined #openstack-infra | 11:54 | |
*** mbacchi has joined #openstack-infra | 11:57 | |
*** jhesketh has joined #openstack-infra | 11:58 | |
*** skeppste_ has joined #openstack-infra | 11:58 | |
*** yfried__ is now known as yfried|afk | 11:58 | |
openstackgerrit | Julien Danjou proposed a change to openstack-dev/hacking: Add a check for old style class declarations https://review.openstack.org/116846 | 11:58 |
*** skeppste_ has quit IRC | 11:58 | |
*** skeppste_ has joined #openstack-infra | 11:59 | |
*** jhesketh_ has quit IRC | 11:59 | |
*** baoli has joined #openstack-infra | 11:59 | |
*** gokrokve has joined #openstack-infra | 11:59 | |
*** jhesketh has quit IRC | 12:00 | |
*** skeppste_ has quit IRC | 12:00 | |
*** skeppst__ has joined #openstack-infra | 12:00 | |
*** gokrokve has quit IRC | 12:00 | |
*** jhesketh has joined #openstack-infra | 12:01 | |
*** gokrokve has joined #openstack-infra | 12:01 | |
*** skeppstedt has quit IRC | 12:01 | |
jhesketh | sdague: so I'm unsure on the filter thing.. maybe we need another opinion there? | 12:01 |
*** gus has quit IRC | 12:06 | |
*** gokrokve has quit IRC | 12:06 | |
*** tsg has quit IRC | 12:06 | |
*** yfried|afk has quit IRC | 12:06 | |
sdague | jhesketh: yep, I'm fine with holding that for now | 12:08 |
sdague | as long as we move forward on the single status case | 12:09 |
sdague | maybe we need to know what the use case is | 12:09 |
*** KanagarajM2 has quit IRC | 12:10 | |
jhesketh | sounds good to me | 12:11 |
jhesketh | sdague: looks like we were both working towards the same thing anyway... I actually had an overlay of zuul results onto gerrit from over 6 months back that kept getting pushed back for various reasons | 12:11 |
sdague | yeh | 12:12 |
sdague | so I think we can work this into the hideci.js code | 12:12 |
sdague | because ideally it would live above that | 12:12 |
sdague | honestly... I wouldn't replicate the whole test block from zuul | 12:12 |
sdague | just a small table to know that changes are currently running | 12:13 |
*** yfried|afk has joined #openstack-infra | 12:17 | |
*** dprince has joined #openstack-infra | 12:18 | |
*** dkranz has joined #openstack-infra | 12:19 | |
*** mmedvede has joined #openstack-infra | 12:19 | |
*** mattoliverau has quit IRC | 12:19 | |
*** zns has joined #openstack-infra | 12:19 | |
*** e0ne has joined #openstack-infra | 12:19 | |
*** zns has quit IRC | 12:22 | |
*** yamahata has joined #openstack-infra | 12:23 | |
*** bswartz has quit IRC | 12:24 | |
*** aysyanne has joined #openstack-infra | 12:25 | |
*** dizquierdo has quit IRC | 12:26 | |
*** ihre has joined #openstack-infra | 12:27 | |
*** dimtruck is now known as zz_dimtruck | 12:28 | |
*** dims has quit IRC | 12:29 | |
*** dims has joined #openstack-infra | 12:30 | |
*** yaguang has quit IRC | 12:30 | |
*** yfried|afk has quit IRC | 12:31 | |
*** waterkinfe has joined #openstack-infra | 12:33 | |
*** KanagarajM has joined #openstack-infra | 12:34 | |
jhesketh | sdague: yep, agreed | 12:34 |
*** waterkinfe has quit IRC | 12:34 | |
*** unicell has quit IRC | 12:35 | |
*** waterkinfe has joined #openstack-infra | 12:35 | |
*** adalbas has joined #openstack-infra | 12:35 | |
*** skeppst__ has quit IRC | 12:35 | |
sdague | addressing your comments in hideci.js now | 12:36 |
sdague | I have timing numbers, ci_parse_results is not nearly as expensive as people imagine :) | 12:36 |
openstackgerrit | Sean Dague proposed a change to openstack-infra/config: refactor of hideci for readability and modularity https://review.openstack.org/124632 | 12:41 |
sdague | so on a short review page it's 5ms | 12:41 |
*** dizquierdo has joined #openstack-infra | 12:41 | |
*** tsg has joined #openstack-infra | 12:41 | |
sdague | on this review - https://review.openstack.org/#/c/40467/ | 12:42 |
sdague | it was 85ms | 12:42 |
*** rdopieralski has left #openstack-infra | 12:42 | |
sdague | that has 103 change revisions | 12:42 |
gilliard | sdague: about the code being called twice - it may be browser dependent, but it definitely happens to me with firefox. | 12:42 |
sdague | which is the largest I've ever seen | 12:42 |
sdague | gilliard: hmmmm | 12:42 |
sdague | so how did it ever work before? | 12:42 |
sdague | because the toggle would have gotten called twice | 12:43 |
sdague | so everything would be displayed again | 12:43 |
gilliard | 1st time, the style attribute changes, 2nd time one of the aria-* attributes has changed. put a console.log in ci_page_loaded to see. | 12:44 |
sdague | so... why do we have the mutation observer anyway? | 12:48 |
openstackgerrit | A change was merged to openstack-infra/jenkins-job-builder: Add PMD publisher https://review.openstack.org/118312 | 12:48 |
*** mjturek has joined #openstack-infra | 12:48 | |
*** zz_dimtruck is now known as dimtruck | 12:48 | |
gilliard | It fires (a lot) when anything on the page changes, I think. In particular there, it is looking for the "waiting..." message to disappear. | 12:48 |
*** skeppstedt has joined #openstack-infra | 12:48 | |
sdague | oh... is this because .... gerrit is one big js bundle | 12:49 |
*** miqui has joined #openstack-infra | 12:49 | |
gilliard | nothing was toggled on page load. It just decided whether or not to show the button which does the toggle when clicked. | 12:49 |
sdague | gilliard: so, this is the way the code was previously | 12:49 |
gilliard | agreed. | 12:50 |
gilliard | Just pointing it out. I was planning to put up a patch for it in the next few hours if you'd like. | 12:50 |
sdague | if you have a fix for it, go for it :) | 12:50 |
sdague | ok | 12:50 |
gilliard | cool. | 12:50 |
sdague | I think I addressed the rest of the comments | 12:50 |
sdague | it's harder to test live now, because comments are cached | 12:51 |
*** rushiagr is now known as rushiagr_away | 12:51 | |
*** unicell has joined #openstack-infra | 12:51 | |
sdague | actually, let me do one more function change, then it's testable better | 12:52 |
*** changbl has quit IRC | 12:52 | |
sdague | gilliard: I wonder.... | 12:53 |
sdague | is there reason you are getting double firing because you are redefining things later? | 12:53 |
gilliard | I don't think so. | 12:54 |
*** _nadya_ has joined #openstack-infra | 12:54 | |
gilliard | I'll paste.o.o something... | 12:54 |
openstackgerrit | Christian Berendt proposed a change to openstack-infra/devstack-gate: Use $((EXPRESSION)) instead of $[EXPRESSION] https://review.openstack.org/125038 | 12:55 |
*** JayJ has joined #openstack-infra | 12:55 | |
*** jistr|english is now known as jistr | 12:56 | |
gilliard | http://paste.openstack.org/show/117111/ | 12:57 |
gilliard | style changes then aria-hidden changes. | 12:57 |
*** koolhead17 has joined #openstack-infra | 12:58 | |
gilliard | I don't know why we have to use this mutationObserver anyway, I assume that the JS GWT puts out is hostile enough to extension so we have to watch for side effects. I'll investigate & put up another patch later. | 12:58 |
*** ildikov has quit IRC | 12:59 | |
*** gokrokve has joined #openstack-infra | 12:59 | |
*** bswartz has joined #openstack-infra | 12:59 | |
*** ildikov has joined #openstack-infra | 12:59 | |
*** k4n0 has quit IRC | 13:01 | |
*** gokrokve has quit IRC | 13:03 | |
*** ildikov has quit IRC | 13:04 | |
*** cnesa15 has joined #openstack-infra | 13:04 | |
*** ildikov has joined #openstack-infra | 13:05 | |
*** isviridov_away is now known as isviridov | 13:06 | |
*** homeless has joined #openstack-infra | 13:06 | |
*** krtaylor has quit IRC | 13:06 | |
*** ppai has joined #openstack-infra | 13:06 | |
openstackgerrit | Sean Dague proposed a change to openstack-infra/config: refactor of hideci for readability and modularity https://review.openstack.org/124632 | 13:07 |
sdague | ok, so ... ^^^ should just fix it | 13:07 |
sdague | by not using toggle on page load | 13:07 |
*** krtaylor has joined #openstack-infra | 13:07 | |
sdague | but actually forcing the hide | 13:07 |
sdague | repurposing toggle was the wrong call anyway | 13:07 |
* gilliard looking | 13:08 | |
openstackgerrit | Sean Dague proposed a change to openstack-infra/config: refactor of hideci for readability and modularity https://review.openstack.org/124632 | 13:08 |
sdague | oops, typo in function name | 13:08 |
sdague | I used the old one that wasn't there | 13:08 |
gilliard | I'm trying to think of a way to test that the caching of "comments" var works as intended. | 13:08 |
sdague | yeh... me too | 13:09 |
sdague | it should | 13:09 |
*** julim has joined #openstack-infra | 13:09 | |
sdague | it's closures | 13:09 |
sdague | that being said, the caching is possibly completely overkill | 13:09 |
sdague | on the average gerrit review we're talking about 20ms | 13:10 |
sdague | the paint calls way outstrip that | 13:10 |
gilliard | What I want is an http proxy where I can inject this file, so that the window.onload stuff works as if it were live. | 13:10 |
*** denis_makogon has joined #openstack-infra | 13:11 | |
*** changbl has joined #openstack-infra | 13:11 | |
*** _buzz has joined #openstack-infra | 13:11 | |
sdague | yeh | 13:11 |
sdague | if you figure out how to do that, we should write up that testing procedure in the doc | 13:12 |
sdague | in the repo somewhere | 13:12 |
sdague | that being said, at a certain point, we probably just have to decide it's probably right, land it, then fix it if it isn't | 13:13 |
openstackgerrit | yolanda.robla proposed a change to openstack-infra/storyboard: Disable login for users https://review.openstack.org/112174 | 13:14 |
openstackgerrit | Alexander Kislitsky proposed a change to openstack-infra/project-config: Fuel-stats repo for Fuel https://review.openstack.org/124773 | 13:15 |
*** salv-orlando has quit IRC | 13:15 | |
*** changbl has quit IRC | 13:18 | |
*** dustins has joined #openstack-infra | 13:18 | |
*** esker has joined #openstack-infra | 13:19 | |
*** esker has quit IRC | 13:19 | |
*** esker has joined #openstack-infra | 13:20 | |
*** zns has joined #openstack-infra | 13:20 | |
*** zns has quit IRC | 13:21 | |
*** mriedem has joined #openstack-infra | 13:21 | |
*** zns has joined #openstack-infra | 13:22 | |
*** rushiagr_away is now known as rushiagr | 13:24 | |
*** juzuluag has joined #openstack-infra | 13:24 | |
*** eglynn-officeafk is now known as eglynn-office | 13:25 | |
openstackgerrit | yolanda.robla proposed a change to openstack-infra/storyboard: Disable login for users https://review.openstack.org/112174 | 13:26 |
*** dkliban_afk is now known as dkliban | 13:29 | |
*** zz_jgrimm is now known as jgrimm | 13:29 | |
*** vdo has joined #openstack-infra | 13:29 | |
openstackgerrit | yolanda.robla proposed a change to openstack-infra/storyboard: Disable login for users https://review.openstack.org/112174 | 13:31 |
*** doug-fish has joined #openstack-infra | 13:33 | |
*** KanagarajM has quit IRC | 13:33 | |
*** rushiagr is now known as rushiagr_away | 13:36 | |
*** yfried|afk has joined #openstack-infra | 13:37 | |
*** mfer has joined #openstack-infra | 13:39 | |
openstackgerrit | Denis M. proposed a change to openstack-infra/project-config: Added jobs for MongoDB and Percona datastores https://review.openstack.org/125057 | 13:40 |
*** skeppstedt has quit IRC | 13:40 | |
*** skeppstedt has joined #openstack-infra | 13:41 | |
*** jgallard has joined #openstack-infra | 13:43 | |
*** sdake has quit IRC | 13:44 | |
*** sdake has joined #openstack-infra | 13:44 | |
*** skeppstedt has quit IRC | 13:45 | |
openstackgerrit | A change was merged to openstack-infra/jenkins-job-builder: Adding support for the Powershell plugin. https://review.openstack.org/121142 | 13:45 |
*** dimtruck is now known as zz_dimtruck | 13:47 | |
*** radez_g0n3 is now known as radez | 13:49 | |
*** otherwiseguy has joined #openstack-infra | 13:51 | |
*** waterkinfe has quit IRC | 13:51 | |
*** skeppstedt has joined #openstack-infra | 13:52 | |
*** sdake_ has joined #openstack-infra | 13:53 | |
*** vigneshvar has quit IRC | 13:53 | |
*** dangers_away is now known as dangers | 13:54 | |
*** markmcclain has joined #openstack-infra | 13:54 | |
*** markmcclain has quit IRC | 13:54 | |
*** dkranz has quit IRC | 13:55 | |
*** nosnos_ has quit IRC | 13:59 | |
*** nosnos has joined #openstack-infra | 13:59 | |
*** gokrokve has joined #openstack-infra | 13:59 | |
*** zaphod42 has quit IRC | 13:59 | |
*** dustins has quit IRC | 14:00 | |
*** wenlock has joined #openstack-infra | 14:00 | |
*** sdake_ has quit IRC | 14:01 | |
*** dustins has joined #openstack-infra | 14:02 | |
*** dane_leblanc has joined #openstack-infra | 14:02 | |
*** zz_gondoi is now known as gondoi | 14:02 | |
*** bswartz has quit IRC | 14:03 | |
*** linuxmohan has quit IRC | 14:03 | |
*** nosnos has quit IRC | 14:04 | |
*** kashyap has quit IRC | 14:04 | |
*** gokrokve has quit IRC | 14:04 | |
*** Sincler has joined #openstack-infra | 14:05 | |
*** e0ne has quit IRC | 14:06 | |
*** daya_k has quit IRC | 14:06 | |
*** jergerber has joined #openstack-infra | 14:09 | |
*** andreaf_ has joined #openstack-infra | 14:10 | |
openstackgerrit | A change was merged to openstack-infra/jenkins-job-builder: Flatten attribute for publish plugins https://review.openstack.org/124081 | 14:11 |
*** andreaf has quit IRC | 14:13 | |
*** andreaf_ is now known as andreaf | 14:14 | |
*** andreaf_ has joined #openstack-infra | 14:14 | |
*** zz_dimtruck is now known as dimtruck | 14:15 | |
*** tonytan4ever has joined #openstack-infra | 14:16 | |
*** jergerber has quit IRC | 14:16 | |
*** dmsimard_away is now known as dmsimard | 14:16 | |
*** hdd has joined #openstack-infra | 14:17 | |
*** chuckC has quit IRC | 14:18 | |
*** BOKALDO has joined #openstack-infra | 14:18 | |
*** rushiagr_away is now known as rushiagr | 14:19 | |
*** pbelanyi_ is now known as pbelanyi | 14:21 | |
openstackgerrit | A change was merged to openstack-infra/jenkins-job-builder: ModuleRegistry now cache entry points group https://review.openstack.org/123163 | 14:22 |
anteaya | AJaeger: the job that adds more layout checks for zuul/layout.yaml: https://review.openstack.org/#/c/124593/1 this patch has the script for the checks and adds it to tox, what is your plan for having it run in check or gate? | 14:22 |
*** ildikov_ has joined #openstack-infra | 14:23 | |
*** ildikov has quit IRC | 14:23 | |
openstackgerrit | Sean Dague proposed a change to openstack-infra/elastic-recheck: also ignore icehouse partial grenade https://review.openstack.org/125070 | 14:24 |
*** alexpilotti has quit IRC | 14:24 | |
*** BOKALDO has quit IRC | 14:26 | |
*** andreaf has quit IRC | 14:26 | |
*** ChuckC_ has quit IRC | 14:27 | |
*** gokrokve has joined #openstack-infra | 14:27 | |
*** bradjones has quit IRC | 14:27 | |
*** xyang0 has joined #openstack-infra | 14:28 | |
*** cnesa15 has quit IRC | 14:29 | |
*** otherwiseguy has quit IRC | 14:29 | |
*** e0ne has joined #openstack-infra | 14:29 | |
*** gokrokve has quit IRC | 14:30 | |
*** gokrokve has joined #openstack-infra | 14:31 | |
*** eharney has joined #openstack-infra | 14:31 | |
*** jcoufal has quit IRC | 14:33 | |
*** datsun180b has joined #openstack-infra | 14:33 | |
*** mestery has joined #openstack-infra | 14:33 | |
anteaya | BobBall: when you get a chance can we have a bit of a chat about: https://review.openstack.org/#/c/124730/3 | 14:34 |
openstackgerrit | A change was merged to openstack-infra/elastic-recheck: also ignore icehouse partial grenade https://review.openstack.org/125070 | 14:34 |
BobBall | sure anteaya | 14:34 |
anteaya | these scripts appear to be specific to xenserver | 14:34 |
anteaya | thanks | 14:34 |
BobBall | They are | 14:34 |
anteaya | and we recently cleared out some code from master that was specific to hp and rackspace | 14:34 |
anteaya | help me understand why these scripts beneifit everyone using openstack | 14:35 |
anteaya | and should be in master | 14:35 |
*** emagana has joined #openstack-infra | 14:35 | |
BobBall | They only are useful for XenServer - we can host them elsewhere if needed. | 14:35 |
mordred | anteaya: I went on the hp/rackspace rampage because those are vendors who should just be compatible with openstack | 14:35 |
*** krtaylor has quit IRC | 14:36 | |
mordred | anteaya: for open source software drivers, I think we _do_ want to have support for them in our tree | 14:36 |
mordred | anteaya: similar to mysql vs. postgres | 14:36 |
anteaya | mordred: any objection to bob's scripts? | 14:36 |
*** wenlock has quit IRC | 14:36 | |
mordred | not at all | 14:37 |
anteaya | thank you | 14:37 |
anteaya | that was what I needed | 14:37 |
anteaya | thanks BobBall | 14:37 |
BobBall | ok, great | 14:37 |
mordred | I look forward to the day when we're as good at testing xenserver as we are with testing kvm | 14:37 |
*** krtaylor has joined #openstack-infra | 14:37 | |
*** jcoufal has joined #openstack-infra | 14:37 | |
mordred | BobBall: btw - did you see that the dib-nodepool stuff landed? | 14:37 |
BobBall | yes mordred | 14:37 |
mordred | cool. I have no idea if that makes things harder, easier, or indifferent | 14:37 |
BobBall | We've got a task in Q4 to look at moving to dib (well, not actually using DIB but a pre-built image) | 14:37 |
mordred | ++ | 14:38 |
BobBall | In fact, not using DIB at all, but extending nodepool to just be able to upload an image to glance with no DIB actions | 14:38 |
mordred | (strangely, it'll still probably use dib on our side - but the dib element will be VERY small) | 14:38 |
mordred | oh | 14:38 |
BobBall | (which it might be able to do ATM - not looked sufficiently) | 14:38 |
mordred | gotcha | 14:38 |
*** garyh has joined #openstack-infra | 14:38 | |
mordred | no, it has not - but your way is better than my way | 14:38 |
fungi | is xenserver actually 100% free software? i've used xen a bunch in the past, but never xenserver | 14:38 |
BobBall | The XenServer image is a right pain at the best of times... | 14:38 |
BobBall | yes fungi | 14:38 |
BobBall | 100% free and open source | 14:39 |
fungi | their "download for free" page requiring a login is a bit of a turn-off https://store.citrix.com/store/citrix/en_US/DisplayThreePgCheckoutAddressPaymentInfoPage/ThemeID.28169600 | 14:39 |
anteaya | BobBall: ah and that I didn't know | 14:39 |
anteaya | so I learned something | 14:39 |
BobBall | www.xenserver.org has _exactly_ the same builds as citrix.com | 14:39 |
mordred | BobBall: you're still probably going to need dib I think | 14:39 |
*** bknudson has joined #openstack-infra | 14:39 | |
BobBall | don't sue that one fungi :) | 14:39 |
*** reed has joined #openstack-infra | 14:39 | |
fungi | heh. is that the trap for suckers with money to waste? | 14:39 |
BobBall | Citrix want to make a difference between 'XenServer' and 'Citrix XenServer' (which is the thing you can buy support on) | 14:39 |
mordred | BobBall: because we're using dib to do the key injection stuff and to do the pre-caching of all the stuff we pre-cache | 14:39 |
*** david-lyle has joined #openstack-infra | 14:40 | |
BobBall | I like that suggestion fungi ;) | 14:40 |
mordred | BUT - I may be very wrong | 14:40 |
mordred | I'm still caffinating | 14:40 |
mordred | MySQL did the same thing back in the day | 14:40 |
fungi | BobBall: aha. got it. i was indeed conflating "xenserver" with "citrix xenserver." thanks for the info! | 14:40 |
mordred | may still do it | 14:40 |
BobBall | mordred: Understood. I hope we don't need to do that - it'll probably rule out DIB due to the convoluted architecture we have... | 14:40 |
mordred | MySQL Enterprise was ... well ... absolutely indistinguishable on a binary level from MySQL Community | 14:40 |
mordred | BobBall: I betcha lifeless will be happy to take that on as a challenge to solve if it is a problem :) | 14:41 |
*** _nadya_ has quit IRC | 14:41 | |
BobBall | Gotta love companies and the way they make money ;) | 14:41 |
*** andreaf has joined #openstack-infra | 14:42 | |
BobBall | Indeed - well, Mate will be looking at it in a few weeks | 14:42 |
fungi | but anyway, so just to simplify for my limited cranial capacity... xenserver is essentially a custom linux distro tuned for running/managing xen hypervisors? | 14:42 |
*** pcrews has joined #openstack-infra | 14:42 | |
BobBall | Bingo fungi | 14:42 |
*** jedimike has joined #openstack-infra | 14:42 | |
*** krtaylor has quit IRC | 14:42 | |
*** openstackgerrit has quit IRC | 14:42 | |
sdague | mordred: oh, so dib is currently blocking stable/icehouse requirements fixes | 14:43 |
sdague | mordred: if you want to noodle on why - https://review.openstack.org/#/c/124648/ | 14:43 |
*** bradjones has joined #openstack-infra | 14:44 | |
*** bradjones has quit IRC | 14:44 | |
*** bradjones has joined #openstack-infra | 14:44 | |
mordred | sdague: I'd love to noodle on why | 14:44 |
*** krtaylor has joined #openstack-infra | 14:44 | |
sdague | zuul change to expose individual patch set status - https://review.openstack.org/#/c/124390/ - with lots of tests :) | 14:44 |
mordred | sdague: why in the wORLD would anything break on that change? | 14:45 |
sdague | mordred: because dib is in projects.txt | 14:45 |
sdague | and probably doesn't have a stable/icehouse branch | 14:46 |
*** Sincler has quit IRC | 14:46 | |
mordred | sdague: so this actually has very little to do with pytz | 14:46 |
sdague | yep | 14:46 |
sdague | it has to do with dib | 14:46 |
mordred | somewhere we have an asymmetric gate on this | 14:46 |
mordred | SpamapS: ^^ you awake? | 14:46 |
*** rdopieralski has joined #openstack-infra | 14:47 | |
*** otherwiseguy has joined #openstack-infra | 14:47 | |
sdague | yeh, honestly, my first instinct is just remove dib from projects.txt | 14:47 |
*** krtaylor has quit IRC | 14:47 | |
*** markmcclain has joined #openstack-infra | 14:48 | |
mordred | sdague: so, I'm guessing that what happened is that the change to lande dib-utils in master happened and worked all fine | 14:48 |
sdague | yep | 14:48 |
mordred | sdague: and that there have been no stable/icehouse changes since then | 14:48 |
rdopieralski | hello, can I ask for adding the triple-o-core and horizon-core groups to the stackforge/tuskar-ui-extras-core group? https://review.openstack.org/#/admin/projects/stackforge/tuskar-ui-extras,access | 14:48 |
mordred | you know - honestly, this isn't just a dib thing, it's anything that doesn't have a stable branch, like the clients | 14:48 |
sdague | yep | 14:49 |
sdague | the clients have a job for this though | 14:49 |
rdopieralski | so that we can actually use that repository :) | 14:49 |
mordred | we should probably run a stable branch job on master changes to requirements or something | 14:49 |
mordred | sdague: do they? | 14:49 |
sdague | at least for icehouse | 14:49 |
sdague | I couldn't get the havana one to work, too much regression | 14:49 |
sdague | but.... | 14:49 |
sdague | I actually think that's the wrong thing | 14:49 |
mordred | whyfore? | 14:49 |
sdague | I think we should just remove things like dib from projects.txt | 14:50 |
mordred | and the clients as well? | 14:50 |
sdague | I haven't decided on the clients yet | 14:50 |
mordred | I mean, it's the same problem | 14:50 |
mordred | either we can solve it and it applies across the board, or we can't | 14:50 |
mordred | I see no difference between dib and python-novaclient | 14:50 |
*** mmagr has quit IRC | 14:51 | |
mordred | from a build system perspective | 14:51 |
*** thedodd has joined #openstack-infra | 14:51 | |
*** wenlock has joined #openstack-infra | 14:51 | |
sdague | oh hey, power outage | 14:51 |
mordred | whee | 14:52 |
sdague | rebooted now, but I probably missed the last 2 minutes of conversation | 14:52 |
mordred | sdague: since you left, I said: | 14:52 |
mordred | 14:50:32 mordred | I mean, it's the same problem | 14:52 |
mordred | 14:50:42 mordred | either we can solve it and it applies across the board, or we can't | 14:52 |
mordred | 14:50:53 mordred | I see no difference between dib and python-novaclient | 14:52 |
mordred | 14:51:03 mordred | from a build system perspective | 14:52 |
sdague | so... this opens the giant meta debate | 14:53 |
sdague | which projects have write authority for projects.txt | 14:53 |
sdague | is it everything? | 14:53 |
sdague | or is it the integrated release | 14:53 |
*** Sincler has joined #openstack-infra | 14:53 | |
*** skeppstedt has quit IRC | 14:53 | |
sdague | and if it's everything, why are we doing this :) | 14:54 |
*** skeppstedt has joined #openstack-infra | 14:54 | |
fungi | sdague: whatcha mean write authority? | 14:55 |
sdague | which projects can propose a requirement change | 14:55 |
anteaya | rdopieralski: I think you need to get fungi's attention for that request | 14:55 |
fungi | a change to the list of requirements, or a change to the list of projects on which the requirements list is enforced? | 14:55 |
*** rickerc has joined #openstack-infra | 14:56 | |
*** krtaylor has joined #openstack-infra | 14:56 | |
fungi | we've so far been letting any project be in the projects.txt file if they want, but additions/changes to global-requirements.txt can only be performed for the explicit benefit of official openstack projects | 14:57 |
*** dkranz has joined #openstack-infra | 14:58 | |
*** skeppstedt has quit IRC | 14:58 | |
*** mfer has quit IRC | 14:59 | |
sdague | right | 15:00 |
*** todd_dsm has joined #openstack-infra | 15:00 | |
sdague | ok, so it looks like we actually still don't have the thing that we think we want | 15:00 |
*** mestery has quit IRC | 15:00 | |
sdague | I'm reading the current requirements check job | 15:00 |
sdague | fungi: is there a way to know if a project has stable branch? | 15:00 |
mordred | git branch -a | grep stable | 15:01 |
rdopieralski | fungi: would you have a moment to add the triple-o and horizon -core and -release groups to the corresponding groups on https://review.openstack.org/#/admin/projects/stackforge/tuskar-ui-extras,access ? | 15:01 |
*** kmartin_away is now known as kmartin | 15:02 | |
*** MaxV has quit IRC | 15:03 | |
*** david-lyle has quit IRC | 15:03 | |
*** daya_k has joined #openstack-infra | 15:04 | |
*** MaxV has joined #openstack-infra | 15:04 | |
fungi | sdague: for p in `ssh -p 29418 review.openstack.org gerrit ls-projects | grep ^openstack | grep -v ^openstack/openstack$` ; do wget -qO/dev/null 'http://git.openstack.org/cgit/'$p'/tree/?h=stable/havana' && echo $p ; done | 15:04 |
*** MarkAtwood has joined #openstack-infra | 15:04 | |
fungi | rdopieralski: sure, just a sec | 15:04 |
*** AJaeger_ has joined #openstack-infra | 15:04 | |
*** markmcclain has quit IRC | 15:05 | |
AJaeger_ | anteaya: regarding https://review.openstack.org/#/c/124593 - good catch. i thought tox -e zuul would run it in the gate but that seems not to be the case. | 15:05 |
AJaeger_ | Let me enhance the test... | 15:05 |
*** dims has quit IRC | 15:06 | |
sdague | ok, I'm going to do a thing | 15:06 |
sdague | and I think the fall out will be a bunch of people will want to leave g-r on their own | 15:06 |
fungi | rdopieralski: done | 15:06 |
*** dims has joined #openstack-infra | 15:06 | |
fungi | sdague: that for loop was how i figured out which projects had a stable/havana branch (so i can delete them today) | 15:07 |
*** skeppstedt has joined #openstack-infra | 15:07 | |
*** andreaf has quit IRC | 15:07 | |
*** andreaf has joined #openstack-infra | 15:08 | |
*** david-lyle has joined #openstack-infra | 15:09 | |
AJaeger_ | fungi, anteaya: What about adding the test to gate-project-config-layout? is there a specific reason that tox -e zuul does more than the layout gate? | 15:10 |
*** dims has quit IRC | 15:10 | |
rdopieralski | fungi: thank you very much! | 15:12 |
*** dustins has quit IRC | 15:12 | |
AJaeger_ | fungi, seems somebody added me to project-config as core. Do you have any special requests or guidance for me? | 15:14 |
*** rdopieralski has left #openstack-infra | 15:14 | |
*** gokrokve_ has joined #openstack-infra | 15:15 | |
*** gokrokve_ has quit IRC | 15:15 | |
*** gokrokve_ has joined #openstack-infra | 15:16 | |
*** rwsu has joined #openstack-infra | 15:16 | |
*** flaviof_zzz is now known as flaviof | 15:16 | |
mordred | sdague: why do we want people to leave g-r ? | 15:16 |
*** bswartz has joined #openstack-infra | 15:17 | |
*** dustins has joined #openstack-infra | 15:18 | |
*** changbl has joined #openstack-infra | 15:18 | |
*** bswartz has quit IRC | 15:18 | |
*** atiwari has joined #openstack-infra | 15:18 | |
*** bswartz has joined #openstack-infra | 15:19 | |
fungi | AJaeger_: not sure why it was done that way. perhaps for simplifying the tox-driven job while keeping multiple checks running on one worker. couple perhaps be broken out into multiple new tox envs and then just call those from the zuul env currently there | 15:19 |
fungi | AJaeger_: it was me who added you... http://lists.openstack.org/pipermail/openstack-dev/2014-September/047480.html | 15:19 |
*** gokrokve has quit IRC | 15:19 | |
*** bswartz has quit IRC | 15:20 | |
fungi | AJaeger_: no specific advice, other than don't approve stuff unless you're sure it should be approved | 15:20 |
*** bswartz has joined #openstack-infra | 15:20 | |
*** e0ne has quit IRC | 15:21 | |
AJaeger_ | fungi: missed that email ;) thanks for welcoming us in! | 15:21 |
fungi | sdague: anteaya: ^ in case you don't have a direct e-mail feed into your brain holes, don't want the sudden extra review options to take you by surprise | 15:21 |
*** e0ne has joined #openstack-infra | 15:21 | |
AJaeger_ | fungi: so, let me +2 for now and let others approve ;) | 15:21 |
sdague | oh hey, gerrit bot might want to put project config in here | 15:21 |
AJaeger_ | sdague: it should be in here already | 15:22 |
fungi | sdague: great idea... did we miss that? | 15:22 |
sdague | yep | 15:22 |
fungi | sdague: i think openstackgerrit has simply gone on walkabout again | 15:22 |
*** ChuckC_ has joined #openstack-infra | 15:22 | |
sdague | oh, maybe | 15:22 |
sdague | https://review.openstack.org/125095 | 15:22 |
sdague | mordred / fungi ^^^ | 15:22 |
*** openstackgerrit has joined #openstack-infra | 15:24 | |
fungi | openstackgerrit has returned to us | 15:24 |
sdague | \o/ | 15:24 |
rcarrillocruz | zaro: hi there, i pushed a new change for the multimodule maven jobs, as the JJB repo is now in project-config. It's currently WIP, as I'd like to do some more testing but I'd appreciate if you could have a look | 15:25 |
*** hdd has quit IRC | 15:25 | |
*** salv-orlando has joined #openstack-infra | 15:25 | |
rcarrillocruz | it now uses curl, no maven for deploying | 15:25 |
sdague | so, I think with 125095 we're going to get the enforcement that we all think we should have | 15:25 |
*** unicell has quit IRC | 15:25 | |
*** chuckC has joined #openstack-infra | 15:25 | |
rcarrillocruz | https://review.openstack.org/#/c/125011/ | 15:25 |
sdague | and basically all master only projects will probably be requirements wedged | 15:25 |
sdague | but... at least it will be the thing that people seem to want | 15:26 |
*** mrmartin has joined #openstack-infra | 15:26 | |
*** jcoufal has quit IRC | 15:27 | |
mordred | sdague: yes. I agree with that patch | 15:27 |
mordred | sdague: and I do agree that some stuff may be wedged, but that's the life of something that doesn't have stable branches | 15:28 |
*** mfer has joined #openstack-infra | 15:30 | |
*** skeppstedt has quit IRC | 15:30 | |
*** skeppstedt has joined #openstack-infra | 15:31 | |
*** stevemar has joined #openstack-infra | 15:32 | |
clarkb | sdague why not git branch -a in requirements for SUPPORTED_BRANCHES | 15:32 |
*** coolsvap|afk is now known as coolsvap | 15:32 | |
clarkb | then we dont need to manually update multiple locations | 15:32 |
sdague | my git branch -a includes remotes/origin/stable/folsom | 15:34 |
sdague | remotes/origin/stable/grizzly | 15:34 |
sdague | remotes/origin/stable/havana | 15:34 |
sdague | remotes/origin/stable/icehouse | 15:34 |
sdague | so... that's why | 15:34 |
*** shashankhegde has joined #openstack-infra | 15:34 | |
clarkb | we can filter that | 15:34 |
sdague | by what? | 15:34 |
clarkb | and do in our image builds for example | 15:34 |
*** MarkAtwood has quit IRC | 15:34 | |
sdague | right, but is that any more complicated than changing 1 tupple every 6 months? | 15:35 |
*** signed8bit has joined #openstack-infra | 15:35 | |
*** skeppstedt has quit IRC | 15:35 | |
*** afazekas has quit IRC | 15:35 | |
sdague | this has the nice impact of being very explicit | 15:35 |
openstackgerrit | Denis M. proposed a change to openstack-infra/project-config: Added jobs for MongoDB and Percona datastores https://review.openstack.org/125057 | 15:36 |
stevemar | anteaya, AJaeger sdague congrats y'all :) | 15:36 |
*** gyee has joined #openstack-infra | 15:37 | |
*** krtaylor has quit IRC | 15:37 | |
*** gokrokve_ has quit IRC | 15:37 | |
*** gokrokve has joined #openstack-infra | 15:38 | |
*** krtaylor has joined #openstack-infra | 15:38 | |
sdague | apparently stable/icehouse devstack has been code unlandable for a month :) | 15:39 |
*** gokrokve has quit IRC | 15:39 | |
*** gokrokve has joined #openstack-infra | 15:39 | |
*** Daisy has joined #openstack-infra | 15:40 | |
*** todd_dsm has quit IRC | 15:41 | |
*** ihrachyshka has quit IRC | 15:42 | |
*** salv-orlando has quit IRC | 15:43 | |
pelix | zaro: wondering is it worth changing the tests to treat non-existing output candidates as empty strings? Would avoid adding a number of empty files for the duplicate jobs change | 15:44 |
*** gokrokve has quit IRC | 15:44 | |
AJaeger_ | stevemar: thanks1 | 15:44 |
openstackgerrit | Andreas Jaeger proposed a change to openstack-infra/project-config: Use tox -e zuul for layout test https://review.openstack.org/125101 | 15:44 |
*** atuvenie has quit IRC | 15:44 | |
AJaeger_ | anteaya: 125101 is the base for the gating on the alphabetized patch in https://review.openstack.org/124593 | 15:45 |
*** jgrimm is now known as zz_jgrimm | 15:46 | |
clarkb | sdague we have a lot of knobs to turn when adding new stable branches and one fewer should lead to fewer mistakes | 15:46 |
openstackgerrit | Darragh Bailey proposed a change to openstack-infra/jenkins-job-builder: Error on duplicate job names being found https://review.openstack.org/56734 | 15:48 |
*** VijayTripathi has joined #openstack-infra | 15:49 | |
sdague | so I just thought about the req thing again in the shower. I think my change is a mobius strip of terribleness | 15:53 |
openstackgerrit | A change was merged to openstack-dev/hacking: Add a check for old style class declarations https://review.openstack.org/116846 | 15:53 |
sdague | because here is the actual behavior we've never decided on | 15:53 |
openstackgerrit | patrick-crews proposed a change to openstack-infra/elastic-recheck: Make function for logstash query encoding https://review.openstack.org/122246 | 15:53 |
*** zz_jgrimm is now known as jgrimm | 15:53 | |
sdague | stable/icehouse: foo>0.1 | 15:53 |
sdague | master: foo>0.2 | 15:53 |
sdague | what is a master only project supposed to do? | 15:54 |
sdague | because if the answer is stable == master | 15:54 |
sdague | than I don't understand why g-r has branches | 15:54 |
*** dizquierdo has quit IRC | 15:55 | |
sdague | but if the answer isn't that, then is it that master only branches should actually get oldest stable syncs instead of master syncs? | 15:55 |
clarkb | pip needs that dependency resolver | 15:55 |
sdague | but that doesn't actually solve it either | 15:55 |
anteaya | fungi stevemar thank you | 15:55 |
sdague | because ... we do pip multiple times | 15:55 |
openstackgerrit | Andreas Jaeger proposed a change to openstack-infra/project-config: Add more checks for zuul/layout.yaml https://review.openstack.org/124593 | 15:55 |
clarkb | sdague it would though. just like apt yells in those cases | 15:56 |
sdague | clarkb: no, because pip doesn't care about what you did before | 15:56 |
clarkb | but a pip dep resolver should/would | 15:57 |
sdague | we're going to run pip 40 times | 15:57 |
clarkb | because the dep resolver must consider the world | 15:57 |
*** shashankhegde has quit IRC | 15:57 | |
clarkb | however I am sure python would say the actual way to fix this is virtualenvs | 15:57 |
sdague | so that would mean that some ubuntu package would have a pin on a thing | 15:58 |
*** emagana has quit IRC | 15:58 | |
anteaya | AJaeger_: sorry I was on the phone, just getting my head around what is happening in those patches | 15:58 |
sdague | and we'd never be able to test again unless we built python from source? | 15:58 |
*** ashaeron has quit IRC | 15:58 | |
sdague | because there is a lot of the world that we don't control | 15:58 |
*** emagana has joined #openstack-infra | 15:58 | |
clarkb | sdague or uninstall that thing via apt and install via pip | 15:58 |
*** yaguang has joined #openstack-infra | 15:58 | |
sdague | clarkb: like some of the system level tools? | 15:58 |
clarkb | but ya virtualenvs are likely the official answer | 15:59 |
clarkb | sdague yup | 15:59 |
AJaeger_ | anteaya: no problem.... | 15:59 |
sdague | yeh... so anyway, I think we remain in terribleness on all of that | 15:59 |
clarkb | we do this with things like requests and cloud init iirc | 15:59 |
sdague | we actually don't with requests | 15:59 |
clarkb | infra does | 15:59 |
*** markmcclain has joined #openstack-infra | 16:00 | |
*** dims has joined #openstack-infra | 16:00 | |
anteaya | fungi: so timing the creation of new projects with the creation of images | 16:00 |
*** amotoki has joined #openstack-infra | 16:00 | |
*** esker has quit IRC | 16:00 | |
clarkb | system requests is used by a system lib but its too old so we update via pip | 16:00 |
sdague | ok, on trusty nodes we don't | 16:00 |
anteaya | so if we approve new projects, do you want us to check with you first to ensure it doesn't interfere with image creation? | 16:00 |
*** emagana_ has joined #openstack-infra | 16:00 | |
sdague | and there is something else deep in that seems to drag it in, that I haven't unwound yet | 16:01 |
anteaya | since those two operations clash if I recall | 16:01 |
*** dims_ has joined #openstack-infra | 16:01 | |
clarkb | sdague ya I think we do this on centos6 and precise | 16:01 |
*** emagana has quit IRC | 16:01 | |
anteaya | AJaeger_: can I test this patch on my local env? https://review.openstack.org/#/c/124593/2 | 16:02 |
clarkb | anteaya they only clash if project creation fails | 16:02 |
clarkb | anteaya I wouldnt worry about it too much | 16:02 |
anteaya | which happens randomly for undetermined reasons, yes? | 16:02 |
AJaeger_ | anteaya: yes, you should | 16:02 |
anteaya | clarkb: k, thanks | 16:02 |
anteaya | AJaeger_: k, will attempt | 16:03 |
*** jgrimm is now known as zz_jgrimm | 16:04 | |
zaro | morning | 16:04 |
clarkb | sdague the problem with virtualenv per service is it likely wont be long before we break the distros | 16:04 |
clarkb | we could also stable branch everything | 16:05 |
sdague | clarkb: yeh... so I'm having a hard time caring a lot about that given that the distros aren't helping with the problem at all | 16:05 |
*** zaphod42 has joined #openstack-infra | 16:06 | |
*** sabeen3 has joined #openstack-infra | 16:06 | |
*** dims has quit IRC | 16:06 | |
*** sabeen has quit IRC | 16:06 | |
*** HeOS has quit IRC | 16:06 | |
AJaeger_ | anteaya: i think we should first integrate dhellmann's patch and then go test 124593... | 16:08 |
*** tothanhcong has joined #openstack-infra | 16:08 | |
*** gokrokve has joined #openstack-infra | 16:08 | |
*** tothanhcong has quit IRC | 16:08 | |
AJaeger_ | so, to untangle this, could somebody help reviewing and approving https://review.openstack.org/124759 and https://review.openstack.org/124517, please? | 16:09 |
*** tothanhcong has joined #openstack-infra | 16:09 | |
fungi | anteaya: adding new projects doesn't really break image creation unless the project addition itself breaks, or unless you get really lucky and it merges at just the wrong time | 16:10 |
zaro | hashar: need help with 124885 | 16:10 |
anteaya | fungi: awesome | 16:10 |
*** yamahata has quit IRC | 16:11 | |
*** eglynn-office is now known as eglynn-officeafk | 16:11 | |
*** tothanhcong has quit IRC | 16:11 | |
anteaya | AJaeger_: yes, the order of the patches seems to be important here in order to tox to do its thing | 16:11 |
*** ildikov_ has quit IRC | 16:11 | |
anteaya | morning zaro | 16:11 |
*** zz_jgrimm is now known as jgrimm | 16:12 | |
*** koolhead17 has quit IRC | 16:12 | |
anteaya | AJaeger_: did you want to merge dhellmann's patch first rather than work on the dependency dance? | 16:12 |
zaro | pelix: hey, welcome back. jjb release this week. is there something you really like to be included? | 16:12 |
AJaeger_ | anteaya: yes, let's get dhellmann's in (thanks sdague and clarkb for reviews!) and then I'll sort it out. | 16:12 |
AJaeger_ | Let me add some WIPs to mine ;) | 16:13 |
hashar | zaro: yeah I wasn't sure how to write the tests myself. I am in audio right now | 16:13 |
fungi | anteaya: probably the bigger thing which causes interruptions is adding new channels to eavesdrop/meetbot since that can interrupt meeting logging and leave stale topics, but conveniently we didn't move that one into project-config since it's handled as an array in a puppet manifest | 16:13 |
*** congtothanh has joined #openstack-infra | 16:13 | |
openstackgerrit | Darragh Bailey proposed a change to openstack-infra/jenkins-job-builder: Error on duplicate job names being found https://review.openstack.org/56734 | 16:13 |
*** yfried|afk has quit IRC | 16:13 | |
*** gokrokve has quit IRC | 16:13 | |
pelix | zaro: initial python3 compatibility? Seems like should try and get in place? | 16:13 |
clarkb | Ajaeger_ dont forget the parent of 124759 | 16:14 |
*** packet has joined #openstack-infra | 16:14 | |
anteaya | fungi: yay | 16:14 |
openstackgerrit | A change was merged to openstack-infra/project-config: Do not run unit test jobs for oslo-specs https://review.openstack.org/124517 | 16:14 |
*** packet is now known as Guest55921 | 16:14 | |
zaro | hashar: hmm, too bad msabram isn't around. | 16:15 |
zaro | pelix: ok. lets do it. which should i review? | 16:16 |
AJaeger_ | clarkb: I mixed up the ID of parent of 124759 but you all sorted it out - thanks a lot! | 16:17 |
hashar | zaro: I might have some free time tonight. Will play a bit with mock :] | 16:18 |
anteaya | AJaeger_: I need to have some lunch, let's see what happens with merge check on your patch | 16:18 |
pelix | zaro: just verifying after making the changes you recommended | 16:18 |
openstackgerrit | Darragh Bailey proposed a change to openstack-infra/jenkins-job-builder: Make JJB python 3 compatible https://review.openstack.org/118703 | 16:19 |
*** r-daneel has joined #openstack-infra | 16:19 | |
clarkb | is ^ an unrevert? | 16:19 |
*** yfried|afk has joined #openstack-infra | 16:19 | |
clarkb | did we sort out what broke? | 16:19 |
*** baoli has quit IRC | 16:19 | |
*** congtothanh has quit IRC | 16:19 | |
openstackgerrit | A change was merged to openstack-infra/project-config: Remove oslotest cross-project unit test jobs https://review.openstack.org/124758 | 16:19 |
openstackgerrit | A change was merged to openstack-infra/project-config: Fix up tests after repo reorganization https://review.openstack.org/124759 | 16:20 |
*** r-daneel has quit IRC | 16:20 | |
*** MaxV has quit IRC | 16:21 | |
*** IvanBerezovskiy has left #openstack-infra | 16:21 | |
clarkb | sdague would probably be reasonable to have a discussion around venv per service. dstufft is certainly in favor and tripleo does it for sanity | 16:21 |
sdague | yeh | 16:21 |
sdague | maybe that should be the ML thread :) | 16:21 |
clarkb | ya probably | 16:22 |
clarkb | or even cross project summit session | 16:22 |
mordred | clarkb: why? | 16:22 |
mordred | like, to solve which problem? | 16:22 |
mordred | hang on - lemme read the whole scrollback | 16:23 |
sdague | mordred: yeh, go back to mobius strip of terribleness | 16:23 |
*** Ryan_Lane has quit IRC | 16:23 | |
*** bradjones has quit IRC | 16:24 | |
mordred | ok. so - before we unwind all of the things | 16:24 |
mordred | the original question you posed was "sdague | what is a master only project supposed to do?" | 16:24 |
zaro | anybody else concerned that our docs are not getting updated in rtfd.org? | 16:25 |
mordred | and the idea is that a master only project is supposed to still work with stable releases | 16:25 |
mordred | zaro: no | 16:25 |
*** Longgeek has quit IRC | 16:25 | |
*** unicell has joined #openstack-infra | 16:25 | |
mordred | so regardless of how pip works or how virtualenvs or how g-r works | 16:25 |
mordred | we need to verify that master of python-novaclient still works with stable/icehouse | 16:25 |
zaro | ok. should we continue to upload to rtfd then? | 16:25 |
*** bradjones has joined #openstack-infra | 16:26 | |
*** bradjones has quit IRC | 16:26 | |
*** bradjones has joined #openstack-infra | 16:26 | |
mordred | and since dib is a thing that is following the same model, we need to also check that master dib works in a stable/icehouse build | 16:26 |
*** shashankhegde has joined #openstack-infra | 16:26 | |
mordred | we can rework all of the enforcement mechanics all day long, but we do need to verify those things | 16:26 |
zaro | i mean there's a doc-on-rtfd that tons of jobs are using, but it doesn't seem to work :( | 16:27 |
*** mrmartin has quit IRC | 16:27 | |
sdague | mordred: ok, so we do novaclient vs. stable/icehouse | 16:27 |
mordred | off the cuff, I do not believe that venv-per-service has anything to do with the above positive or negative - it's a thing we might consider doing, and I do not think it will solve anything in this discussion | 16:27 |
mordred | sdague: that's right | 16:28 |
*** r-daneel has joined #openstack-infra | 16:28 | |
mordred | and we don't actually co-gate dib with the integrated release, so we don't actually need to do a devstack test on it | 16:28 |
fungi | zaro: i have doubts that's what's broken. rtfd is also supposed to update itself daily, independent of that hook | 16:28 |
*** jistr has quit IRC | 16:28 | |
*** thedodd has quit IRC | 16:28 | |
*** Guest55921 has quit IRC | 16:28 | |
fungi | zaro: if projects aren't getting their docs updated there at all, then that points to a shift in rtfd's model/configuration or else perhaps rtdf itself has broken | 16:29 |
sdague | mordred: so in that case, why do we do a requirements check? | 16:29 |
fungi | tfdr... some combination of letters anyway | 16:29 |
mordred | but if the model holds for requirements from the client libs, then we should backport the dib-utils split to stable/icehouse g-r, and the porting of that patch should be enough to verify that installing master dib alongside icehouse does not break the world | 16:29 |
*** zns has quit IRC | 16:29 | |
*** Guest68510 has quit IRC | 16:30 | |
mordred | sdague: we do the requirements check to make sure that the contents of the requirements files describe reality and work | 16:30 |
sdague | so it seems like what we're actually saying | 16:30 |
sdague | "we should test that dib can work in an openstack environment" | 16:30 |
sdague | we don't do that | 16:30 |
*** mpaolino has quit IRC | 16:30 | |
mordred | Im not saying that | 16:30 |
mordred | at all | 16:30 |
sdague | ok | 16:30 |
*** zns has joined #openstack-infra | 16:31 | |
mordred | I'm saying that the addition of dib-utils to global-requirements, because dib is a master-only project, should have a corresponding change to add dib-utils to g-r:stable/icehouse. if that patch breaks something, then we should not land the corresponding one to master | 16:31 |
mordred | we are not testing that right now, but that is the consequences of having master-only projects at all | 16:31 |
mordred | and it holds for all of them | 16:31 |
sdague | except, what's requiring dib-utils ? | 16:32 |
mordred | dib | 16:32 |
sdague | and what's requring that? | 16:32 |
mordred | it's a top level thing | 16:32 |
*** AzherKhan has joined #openstack-infra | 16:32 | |
mordred | also, ironic | 16:32 |
sdague | it's not an integrated release thing | 16:32 |
mordred | the ironic devstack tests use dib | 16:32 |
derekh | --help | 16:32 |
fungi | derekh: unrecognized option '--help' | 16:33 |
openstackgerrit | Joe Gordon proposed a change to openstack-infra/elastic-recheck: Don't show ER patches under open reviews https://review.openstack.org/125117 | 16:34 |
*** amcrn has joined #openstack-infra | 16:34 | |
derekh | -? | 16:34 |
derekh | fungi: hmm, that didn't work either | 16:34 |
fungi | Segmentation Fault: core dumped | 16:34 |
sdague | so... this means that when evaluating whether we add a requirement to master, do we take into consideration the possibly disruptive impact on stable of adding new requirements? | 16:34 |
derekh | ;-) | 16:34 |
*** koolhead17 has joined #openstack-infra | 16:34 | |
mordred | sdague: we have to if we're adding the new requirement because of a master-only project | 16:35 |
mordred | integrated/not-integrated aside for a sec - it's the same thing with the recent splits of keystone middleware stuff | 16:35 |
zaro | fungi: so i guess we would need help from rtfd maintainers to debug? do you know who to ask? | 16:36 |
sdague | so master only projects that aren't part of the integrated release, get to add requirements to openstack that people need to backport package | 16:36 |
mordred | OR they don't | 16:36 |
sdague | that's the part that has always seemed odd to me | 16:36 |
mordred | it just needs to be considered | 16:36 |
mordred | it's possible that the answer is "no, that would break stable" | 16:36 |
mordred | in which case they can suck it | 16:37 |
sdague | ok, so, how do we know if they need to suck it? | 16:37 |
mordred | propose the backport of the patch to stable/icehouse and see if anything breaks | 16:37 |
sdague | that means ... we actually want to check master against oldest stable req right? | 16:37 |
*** atuvenie has joined #openstack-infra | 16:37 | |
clarkb | mordred venvs per project makes us care less about master foo working with icehouse bar because the coexist in venvs. its a bit of a heavyweighr solution yes and likely to cause distros pain | 16:38 |
*** baoli has joined #openstack-infra | 16:38 | |
mordred | I was just trying to think through the combinations and I'm not 100% sure there is a direct gate we can build | 16:38 |
pleia2 | good morning | 16:38 |
mordred | clarkb: that's right. and so far we've been considering the distros an important consumer | 16:38 |
mordred | so I would not be willing to be party to that change without a TC discussion | 16:38 |
sdague | mordred: yeh, especially with this idea that icehouse is getting an extra 6 months of support | 16:39 |
sdague | we're going to be juggling 4 branches at once | 16:39 |
clarkb | was just going to mention that | 16:39 |
*** patrickeast has joined #openstack-infra | 16:40 | |
mordred | we could completely ditch g-r and move to a completely independent list per-service | 16:40 |
mordred | but that's a pretty substantial change in thinking | 16:40 |
sdague | if we had venvs | 16:40 |
mordred | right | 16:40 |
sdague | yep | 16:40 |
mordred | I'm just saying - that's fairly massive. it may be the right choice | 16:40 |
mordred | but it's fairly massive | 16:40 |
sdague | sure | 16:40 |
openstackgerrit | Sean Dague proposed a change to openstack-infra/project-config: enforce stable requirements https://review.openstack.org/125095 | 16:40 |
mordred | it will also ABSOLUTELY screw the distros | 16:40 |
sdague | ok, so that's conceptually the thing | 16:40 |
mordred | almost immediately | 16:40 |
fungi | it also means we're saying "this is how you cd openstack now if you're not using distro packages" | 16:41 |
mordred | because we know how divergent we were in the projects per-convergence | 16:41 |
clarkb | yup | 16:41 |
sdague | well, looking at how they patch the f* out of a lot of these python packages | 16:41 |
*** hdd has joined #openstack-infra | 16:41 | |
* mordred is fine with that | 16:41 | |
mordred | btw | 16:41 |
fungi | and the docs team is just going to love us if we do that | 16:41 |
sdague | yeh, anyway, I have to go get food | 16:41 |
mordred | as all of the friday discussions around current state of distros | 16:41 |
mordred | but we can't just make that call here | 16:41 |
sdague | mordred: right 100% agreed | 16:42 |
fungi | summit session += | 16:42 |
sdague | I'm just trying to figure out if there is a sane way through this on the requirements side without that | 16:42 |
clarkb | the other alternative is stable branch everything | 16:42 |
mordred | sdague: what's wrong, for now, with proposing a backport of dib-utils to stable/icehouse | 16:42 |
sdague | which is why I'm challenging various things, hoping someone will just say "no silly, it works fine if yuo do A" | 16:42 |
*** hashar has quit IRC | 16:42 | |
mordred | doing that a few times by hand to see if patterns emerge | 16:42 |
sdague | mordred: we do that all the time | 16:43 |
mordred | and then trying to encapsulate what we learn in a suggestion? | 16:43 |
fungi | sdague: pretty sure none of it works fine, ever | 16:43 |
sdague | mordred: and it feels really weird every time we do to me | 16:43 |
fungi | our choice is between a variety of different sorts of broken compromises | 16:43 |
sdague | fungi: right | 16:43 |
mordred | yah | 16:43 |
mordred | remember we're trying to have our cake and eat it too all the time | 16:43 |
sdague | ok, lunch | 16:43 |
clarkb | I think if we want to make long term support of releases a thing and support distros everything needs a stable branch | 16:43 |
fungi | mmmcake | 16:44 |
mordred | I think less things need stable branches, not more :) | 16:44 |
* mordred says screw the distros and screw stable branches - which, btw, is what the original plan was before the distros showed up and whinned in boston | 16:44 | |
clarkb | then we shouldnt support things for 18 months post release | 16:45 |
*** lttrl has quit IRC | 16:45 | |
fungi | well, it's 15 months post release (final icehouse point release scheduled for july) | 16:45 |
*** _nadya_ has joined #openstack-infra | 16:45 | |
mordred | I keep getting told that people should get openstack from their vendors and not from us -that it's not our job to produce a product | 16:45 |
mordred | if we accepted that and adopted linus' thinking, we'd stop supporting anything and let ubuntu and redhat and piston all sort it out | 16:46 |
*** Daisy has quit IRC | 16:46 | |
*** Daisy has joined #openstack-infra | 16:46 | |
fungi | well, linus' model does have stable branches, fwiw | 16:46 |
mordred | I would advocate ditching releases as well though and just going to a completely CD-oriented pipeline | 16:46 |
*** yfried|afk has quit IRC | 16:46 | |
SpamapS | mordred: awake now. Whats up? | 16:47 |
mordred | SpamapS: dib screwed stable/icehouse | 16:47 |
mordred | SpamapS: and it has caused us all to go all meta about releases | 16:47 |
*** doude has quit IRC | 16:47 | |
zaro | jhesketh: still working on this? https://review.openstack.org/#/c/107724/ ? | 16:47 |
mordred | SpamapS: but - short version, dib added dib-utils in master g-r and so it doesn't work in stable/icehouse | 16:47 |
annegentle | honestly mordred I'd be happy to drop the install guide like a hot potato | 16:47 |
mordred | annegentle: I second that motion | 16:48 |
SpamapS | mordred: because stable/icehouse tests using dib master? | 16:48 |
annegentle | but board members approached me when I vented about it a few summits ago | 16:48 |
mordred | SpamapS: that's right - because there is no stable/icehouse of dib | 16:48 |
SpamapS | mordred: there are pypi releases... | 16:48 |
fungi | annegentle: we've got all new board members now ;) | 16:48 |
annegentle | but if the time is right after this release let's ditch | 16:48 |
SpamapS | mordred: so... <= in requirements.txt ? | 16:48 |
mordred | SpamapS: that is irrelevant | 16:48 |
annegentle | heh | 16:48 |
annegentle | nice | 16:48 |
annegentle | I like our board! and the members that approached me were super supportive | 16:48 |
*** _nadya_ has quit IRC | 16:49 | |
*** dims_ has quit IRC | 16:49 | |
mordred | SpamapS: I believe we use master of dib in devstack ... although I suppose you're suggesting that we just pip install dib? | 16:49 |
mordred | sdague: ^^ ? | 16:49 |
SpamapS | mordred: can't we just not pull things without stable branches via git? | 16:49 |
annegentle | so that's all cool. but honestly, looking at the web stats, the most widely read one is ubuntu and that's cuz they don't have any other doc | 16:49 |
fungi | well, we're also managing to sneak more and more tc members into board seats. it's only a matter of time until they become the majority voice ;) | 16:49 |
annegentle | fungi: hee | 16:49 |
*** hashar has joined #openstack-infra | 16:49 | |
*** dims has joined #openstack-infra | 16:49 | |
mordred | sdague: we may have a good specific suggestion here .... | 16:49 |
mordred | annegentle: did any of those board members offer doc writers to help? | 16:50 |
annegentle | mordred: yes they did | 16:50 |
mordred | oh. ok then | 16:50 |
annegentle | mordred: so yeah, money where mouth | 16:50 |
mordred | if people are ponying up humans to work on a thing, I'm much more in support of supporting that thing | 16:50 |
*** thedodd has joined #openstack-infra | 16:50 | |
annegentle | mordred: oh I'm meeting with the Eucalyptus writing team this afternoon! | 16:51 |
*** otherwiseguy has quit IRC | 16:51 | |
annegentle | mordred: not that they want to work on install docs :) | 16:51 |
annegentle | mordred: but it's cool to know those writers | 16:51 |
*** markmcclain has quit IRC | 16:52 | |
*** MarkAtwood has joined #openstack-infra | 16:52 | |
*** shashankhegde has quit IRC | 16:52 | |
mordred | annegentle: yah - I have a hunch we're all going to get to know them better ... | 16:52 |
*** tonytan4ever has quit IRC | 16:52 | |
anteaya | pleia2: how were penquins and wolverines? | 16:52 |
*** scottda has joined #openstack-infra | 16:53 | |
*** sarob has joined #openstack-infra | 16:53 | |
*** bhuvan has joined #openstack-infra | 16:54 | |
*** bodepd_ has quit IRC | 16:54 | |
mordred | sdague, SpamapS: https://review.openstack.org/125121 | 16:54 |
mordred | like that? | 16:55 |
clarkb | ok I am finally on a real computer now | 16:55 |
*** mwagner_lap has quit IRC | 16:55 | |
*** amuller_ has joined #openstack-infra | 16:55 | |
*** _nadya_ has joined #openstack-infra | 16:55 | |
mordred | clarkb: congratulations | 16:55 |
pleia2 | anteaya: they were a lot of fun | 16:55 |
*** laxity has quit IRC | 16:56 | |
anteaya | clarkb: are you at your portland home, with furniture and a computer? | 16:56 |
anteaya | pleia2: \o/ | 16:56 |
openstackgerrit | Darragh Bailey proposed a change to openstack-infra/jenkins-job-builder: Error on duplicate job names being found https://review.openstack.org/56734 | 16:56 |
clarkb | anteaya: yup. sitting on couch with laptop. The desk in office still needs to be put cobbled together. I need more wireless antennas | 16:56 |
*** laxity has joined #openstack-infra | 16:57 | |
*** amuller__ has joined #openstack-infra | 16:57 | |
*** emagana_ has quit IRC | 16:57 | |
* fungi imagines clarkb wandering about with various wireless antennae strapped on | 16:57 | |
*** ihrachyshka has joined #openstack-infra | 16:58 | |
anteaya | clarkb: yay for furtiture, in a house/home | 16:58 |
*** amuller_ has quit IRC | 16:58 | |
*** amuller has quit IRC | 16:58 | |
*** bodepd has joined #openstack-infra | 16:58 | |
*** derekh has quit IRC | 16:58 | |
*** amuller_ has joined #openstack-infra | 16:58 | |
*** otherwiseguy has joined #openstack-infra | 16:58 | |
anteaya | fungi: quite the visual there | 16:58 |
*** bodepd has quit IRC | 16:59 | |
*** mwagner_lap has joined #openstack-infra | 16:59 | |
*** marun has joined #openstack-infra | 16:59 | |
*** hashar has quit IRC | 16:59 | |
*** bodepd has joined #openstack-infra | 17:00 | |
*** sandywalsh_ has quit IRC | 17:01 | |
*** krtaylor has quit IRC | 17:01 | |
*** esker has joined #openstack-infra | 17:01 | |
*** armax has joined #openstack-infra | 17:01 | |
*** zns has quit IRC | 17:02 | |
*** _nadya_ has quit IRC | 17:02 | |
*** amuller__ has quit IRC | 17:02 | |
*** sandywalsh has joined #openstack-infra | 17:03 | |
anteaya | who is chairing today's infra meeting? | 17:03 |
anteaya | I was going to tidy the agenda | 17:03 |
*** amuller_ is now known as amuller | 17:03 | |
openstackgerrit | Darragh Bailey proposed a change to openstack-infra/jenkins-job-builder: Make JJB python 3 compatible https://review.openstack.org/118703 | 17:04 |
*** bhuvan_ has joined #openstack-infra | 17:04 | |
morgan_remote_ | Is anything weird going on with gerrit? I seem to be stuck in a login loop. Might just be mobile browser but it was working last night. | 17:04 |
fungi | anteaya: i am, and was about to go do that--i have a couple things i need to make sure i add anyway | 17:04 |
*** jedimike has quit IRC | 17:05 | |
clarkb | morgan_remote_: not that I have seen | 17:05 |
morgan_remote_ | I login and it redirects me to ubuntuone, redirects to gerrit then to Ubuntu one and requires login again. Huh weird. | 17:05 |
morgan_remote_ | And does that endlessly. | 17:05 |
* clarkb tries logging out and back in again | 17:06 | |
clarkb | morgan_remote_: no problem here. could your browser be caching stuff oddly? | 17:06 |
*** markmcclain has joined #openstack-infra | 17:06 | |
*** bhuvan has quit IRC | 17:06 | |
fungi | morgan_remote_: is your mobile browser an ubuntuphone? | 17:07 |
morgan_remote_ | No. iPhone | 17:07 |
*** atuvenie has quit IRC | 17:07 | |
*** tonytan4ever has joined #openstack-infra | 17:07 | |
morgan_remote_ | Will try clearing out cache etc | 17:07 |
*** Daisy has quit IRC | 17:08 | |
*** arosen-home2 has joined #openstack-infra | 17:08 | |
SpamapS | mordred: left a comment | 17:08 |
fungi | just figured it would be amusing if ubuntuphone had issues with ubuntu's openid | 17:09 |
*** thedodd has quit IRC | 17:09 | |
morgan_remote_ | clarkb: figured it out. 3rd party cookie blocking was turned on. | 17:09 |
*** gokrokve has joined #openstack-infra | 17:09 | |
* clarkb syncs project-config finally | 17:11 | |
*** kgavini has joined #openstack-infra | 17:11 | |
anteaya | fungi: I will get out of your way and do something else | 17:11 |
* anteaya does something else | 17:11 | |
*** emagana has joined #openstack-infra | 17:12 | |
*** jgallard has quit IRC | 17:13 | |
*** shashankhegde has joined #openstack-infra | 17:13 | |
*** lttrl has joined #openstack-infra | 17:14 | |
*** gokrokve has quit IRC | 17:14 | |
*** esker has quit IRC | 17:14 | |
*** bradjones has quit IRC | 17:15 | |
*** MaxV has joined #openstack-infra | 17:16 | |
* clarkb adds dib to the agenda if it isn't already there | 17:16 | |
*** gokrokve has joined #openstack-infra | 17:16 | |
openstackgerrit | A change was merged to openstack-infra/project-config: heat-dsvm-functional INSTALL_TESTONLY=1 https://review.openstack.org/124233 | 17:17 |
*** mrmartin has joined #openstack-infra | 17:17 | |
clarkb | oh good its still there udner priority efforts | 17:17 |
*** zns has joined #openstack-infra | 17:17 | |
*** mmaglana has joined #openstack-infra | 17:17 | |
*** harlowja_away is now known as harlowja | 17:18 | |
anteaya | AJaeger_: I think you can un-WIP this one now: https://review.openstack.org/#/c/125101/ I can get tox to work on my local environment | 17:20 |
zaro | clarkb: i don't think jjb compare-xml test is working anymore. do you know if project-config is at a point where compare-xml should be working yet? | 17:20 |
krotscheck | So, I have two reviews that undo-and-fix my CORS work from yesterday, https://review.openstack.org/#/c/124934/ and https://review.openstack.org/#/c/124936/ - anyone able to look at those for me? (The underlying issue is that the scp publisher does weird path things that break things) | 17:20 |
clarkb | zaro: I am not sure. anteaya and AJaeger_ were working on similar things earlier though. They may know | 17:21 |
*** amotoki has quit IRC | 17:22 | |
*** praneshp has joined #openstack-infra | 17:23 | |
annegentle | sdague: around? | 17:23 |
clarkb | krotscheck: I dont claim to fully understand the gruntfile | 17:23 |
anteaya | zaro: I think that tests on project-config either should be working now or we should be in the process of fixing what isn't working | 17:23 |
anteaya | zaro: how do we go about fixing compare-xml to make it work? I don't know that job very well | 17:24 |
clarkb | krotscheck: but it seems simple enough | 17:24 |
clarkb | copy:draft vs copy:dist to put files in different places | 17:24 |
*** esker has joined #openstack-infra | 17:25 | |
krotscheck | clarkb: Right, so the way grunt plugins work is that you can declare subcommands. So grunt copy:dist can be invoked, and grunt copy:draft can be invoked, and if you want to run both of them you can just call grunt copy. | 17:25 |
*** bhuvan_ has quit IRC | 17:25 | |
openstackgerrit | Darragh Bailey proposed a change to openstack-infra/jenkins-job-builder: Switch to new project-config repo https://review.openstack.org/125124 | 17:25 |
pelix | seems that updating the scripts in jenkins-job-builder to use the new repo for the job configs was forgotten with the reorganization | 17:25 |
krotscheck | clarkb: Now that we have two distinct copy subcommands though, I have to be specific about which one is invoked during which stage of the build. So I only need copy:draft when build:draft is invoked. | 17:25 |
krotscheck | clarkb: That | 17:26 |
*** bhuvan has joined #openstack-infra | 17:26 | |
pelix | zaro: ^^ we probably want that change in before allowing anything else through just to make sure we aren't breaking anything obvious | 17:26 |
krotscheck | clarkb: That’s why I had to modify other invocations of “copy” to say “copy:dist" | 17:26 |
clarkb | krotscheck: gotcha. +2 | 17:26 |
krotscheck | clarkb: And that’s basically it. | 17:26 |
AJaeger_ | zaro: are you running "tox -e zuul" locally? | 17:26 |
krotscheck | clarkb: Thanks :D | 17:26 |
anteaya | pelix: what scripts were forgotten? | 17:27 |
AJaeger_ | zaro, anteaya: it should work now with current HEAD | 17:27 |
anteaya | AJaeger_: the compare-xml job? | 17:27 |
pelix | anteaya: yeap | 17:27 |
pelix | the compare-xml script | 17:27 |
*** bradjones has joined #openstack-infra | 17:27 | |
*** flaviof is now known as flaviof_zzz | 17:27 | |
*** bradjones has quit IRC | 17:27 | |
*** bradjones has joined #openstack-infra | 17:27 | |
clarkb | fungi: https://review.openstack.org/#/c/119936/ would be a good one to get in if you have a chance to review it | 17:28 |
*** todd_dsm has joined #openstack-infra | 17:28 | |
* AJaeger_ double checks compare-xml | 17:28 | |
anteaya | pelix: where was the compare-xml script | 17:28 |
*** sacharya has joined #openstack-infra | 17:28 | |
pelix | anteaya: in the JJB git repo | 17:28 |
AJaeger_ | anteaya: project-config/tools/run-compare-xml | 17:28 |
pelix | tools/run-compare-xml.sh | 17:28 |
anteaya | ah in tools | 17:28 |
zaro | pelix: agreed, but i did run it manually with change 56734 and it looks good. | 17:28 |
anteaya | I wasn't asked to move it | 17:29 |
AJaeger_ | pelix: in JJB? | 17:29 |
anteaya | can folks talk in complete sentences please? | 17:29 |
zaro | anteaya: https://review.openstack.org/#/c/104004 | 17:29 |
anteaya | I am having trouble following the point | 17:29 |
zaro | anteaya: oops, i meant http://logs.openstack.org/34/56734/12/check/jenkins-job-builder-coverage/8689ed8/ | 17:29 |
*** baoli has quit IRC | 17:30 | |
AJaeger_ | zaro, I'm still confused - what is the relevance of the log? | 17:30 |
clarkb | (I am flushing project-config changes now) | 17:30 |
anteaya | clarkb: flushing them from what? | 17:30 |
zaro | anteaya, AJaeger_ : sorry, wrong link again. this one is what i really meant. http://logs.openstack.org/34/56734/12/check/jenkins-job-builder-compare-xml/1b97efb/ | 17:31 |
pelix | AJaeger: yes in jenkins-job-builder git repo. It's used by the jenkins-job-builder-compare-xml job to catch if any changes to JJB are going to cause problems with the existing openstack JJB configs, example of the problem http://logs.openstack.org/03/118703/4/check/jenkins-job-builder-compare-xml/4553e48/console.html | 17:31 |
zaro | it's the compare-xml test. | 17:31 |
*** baoli has joined #openstack-infra | 17:31 | |
anteaya | pelix: so you are saying that a script that was never identifying to be moved into the project-config repo is not in the project-config repo | 17:31 |
clarkb | anteaya: flushing out changes that are ready to merge | 17:31 |
anteaya | clarkb: ah okay, thanks | 17:31 |
clarkb | anteaya: I am treating older +1s from our new core members as +2s and approving things | 17:31 |
*** ZZelle_ has joined #openstack-infra | 17:31 | |
anteaya | clarkb: kk, thank you | 17:31 |
AJaeger_ | clarkb: thanks! | 17:31 |
pelix | anteaya: no, I'm saying it was referencing the old config repo to grab the yaml configs that were moved out into project-config and the script was never updated to point to the new repo. | 17:32 |
*** tonytan4ever has quit IRC | 17:32 | |
*** che-arne has quit IRC | 17:32 | |
openstackgerrit | A change was merged to openstack-infra/config: Removes zuul/layout-dev.yaml https://review.openstack.org/124898 | 17:32 |
*** yfried|afk has joined #openstack-infra | 17:32 | |
pelix | Don't think the script should have been moved, just that it wasn't updated to take into account the changes that occurred. | 17:32 |
anteaya | pelix: ah okay now I understand thank you, yes if the job points to the content which isnt' there, I can see it breaking | 17:32 |
*** AzherKhan has quit IRC | 17:33 | |
*** Ryan_Lane has joined #openstack-infra | 17:33 | |
anteaya | pelix: yes, can we edit that to point to where the content is now then? | 17:33 |
*** AzherKhan has joined #openstack-infra | 17:33 | |
zaro | anteaya, AJaeger_ : the checkout is done in tools/run-compar.sh it needs to point to new project. | 17:33 |
pelix | anteaya: see https://review.openstack.org/125124 | 17:33 |
zaro | anteaya, AJaeger_ : you guys want to fix? or i can do | 17:33 |
openstackgerrit | A change was merged to openstack-infra/project-config: Run a shell script instead of tox for osc functional https://review.openstack.org/124276 | 17:34 |
AJaeger_ | pelix, zaro: There's both a run-compare in jjb and in project-config. I was thinking you talk about project-config but you meant the jjb one. | 17:34 |
anteaya | pelix: +1, thank you | 17:34 |
anteaya | zaro: looks like pelix just needs your review on 125124 | 17:35 |
*** coolsvap is now known as coolsvap|afk | 17:35 | |
anteaya | pelix: is that accurate? | 17:35 |
AJaeger_ | clarkb, fungi could you review https://review.openstack.org/#/c/125124/ to fix jjb gating, please? | 17:35 |
pelix | yep, it appears to fix the job :) | 17:35 |
anteaya | yay | 17:35 |
openstackgerrit | A change was merged to openstack-infra/project-config: Run tripleo tests on horizon https://review.openstack.org/124333 | 17:36 |
* AJaeger_ is not confused anymore... | 17:36 | |
openstackgerrit | Andreas Jaeger proposed a change to openstack-infra/project-config: Group stackforge projects in layout.yaml https://review.openstack.org/124589 | 17:36 |
clarkb | AJaeger_: looking | 17:36 |
openstackgerrit | Andreas Jaeger proposed a change to openstack-infra/project-config: Add more checks for zuul/layout.yaml https://review.openstack.org/124593 | 17:36 |
openstackgerrit | Andreas Jaeger proposed a change to openstack-infra/project-config: Use tox -e zuul for layout test https://review.openstack.org/125101 | 17:36 |
* AJaeger_ was working on the three patches above - that also include run-compare ;9 | 17:36 | |
zaro | pelix: ohh, you are a step ahead! | 17:37 |
*** yaguang has quit IRC | 17:37 | |
clarkb | 125124 lgtm | 17:39 |
*** habib__ has quit IRC | 17:39 | |
clarkb | I checked the test output too | 17:39 |
AJaeger_ | thanks, clarkb | 17:39 |
*** mflobo_ has joined #openstack-infra | 17:40 | |
*** linuxmohan has joined #openstack-infra | 17:40 | |
openstackgerrit | A change was merged to openstack-infra/project-config: Gate Gnocchi against py34-postgresql https://review.openstack.org/124340 | 17:41 |
*** VijayTripathi has quit IRC | 17:41 | |
SpamapS | mordred: note that diskimage-builder and tripleo have developed fairly sophisticated (but not complex) tooling to allow packages vs. git ... seems like devstack might need the same. | 17:41 |
*** emagana has quit IRC | 17:41 | |
mordred | SpamapS: that might be a nice patch to devstack ... | 17:42 |
*** emagana has joined #openstack-infra | 17:42 | |
*** tonytan4ever has joined #openstack-infra | 17:42 | |
mordred | SpamapS: because I know that testing current pip python*client is desirable as is testing git | 17:42 |
mordred | sdague: ^^ or? | 17:42 |
*** daya_k has quit IRC | 17:42 | |
openstackgerrit | A change was merged to openstack-infra/config: Trim jenkins::slave package installations further https://review.openstack.org/119936 | 17:43 |
*** _nadya_ has joined #openstack-infra | 17:43 | |
clarkb | fungi: ty | 17:43 |
SpamapS | mordred: we don't want to fall too far down the testing-matrix-all-the-things hole, but having at least the extreme edges of the matrix handled does seem a worthy goal. | 17:43 |
*** mflobo has quit IRC | 17:43 | |
*** mflobo__ has joined #openstack-infra | 17:43 | |
mordred | SpamapS: well, if there is solid tooling, that seems like a good thing to have - whether we exercise that tooling in all possiblity permutations is orthogonal | 17:44 |
*** emagana_ has joined #openstack-infra | 17:45 | |
*** emagana has quit IRC | 17:45 | |
*** gokrokve has quit IRC | 17:45 | |
*** mflobo_ has quit IRC | 17:46 | |
*** gokrokve has joined #openstack-infra | 17:46 | |
*** arosen-home2 has quit IRC | 17:46 | |
*** arosen1 has joined #openstack-infra | 17:46 | |
*** arosen1 has quit IRC | 17:46 | |
*** arosen1 has joined #openstack-infra | 17:47 | |
pelix | zaro: ;) | 17:47 |
zaro | AJaeger_: you pointed out something that still confuses me, why run-compare-xml.sh in both project-config and jjb. i'm not sure one in project-config is used, might be opportunity to clean up? | 17:47 |
*** arosen1 has quit IRC | 17:47 | |
SpamapS | mordred: sure. I am thinking we should just basically turn your patch into an if on an environment variable or conf setting and keep that conf setting extremely coarse grained.. like if [ -n $install_non_openstack_things_from_pip ] ; then ... | 17:47 |
*** arosen1 has joined #openstack-infra | 17:48 | |
SpamapS | non_integrated things might be a better term | 17:48 |
AJaeger_ | zaro: We run it in project-config to see what has changed from one commit to another. We're looking at changes in project-config | 17:48 |
*** linuxmohan has quit IRC | 17:48 | |
AJaeger_ | zaro: it's run in jjb to check that there are no regressions with your patch to jjb | 17:48 |
*** sdake_ has joined #openstack-infra | 17:48 | |
*** rushiagr is now known as rushiagr_away | 17:48 | |
*** arosen1 has quit IRC | 17:49 | |
*** gokrokve has quit IRC | 17:49 | |
*** linuxmohan has joined #openstack-infra | 17:49 | |
*** gokrokve has joined #openstack-infra | 17:49 | |
*** Sukhdev has joined #openstack-infra | 17:51 | |
*** hashar has joined #openstack-infra | 17:51 | |
*** rushiagr_away is now known as rushiagr | 17:51 | |
*** sabeen has joined #openstack-infra | 17:52 | |
zaro | AJaeger_: ahh looks like a new test that wasn't in config. | 17:52 |
AJaeger_ | zaro, it was in config as well... | 17:53 |
*** reed has quit IRC | 17:53 | |
*** cnesa15 has joined #openstack-infra | 17:53 | |
openstackgerrit | A change was merged to openstack-infra/jenkins-job-builder: Switch to new project-config repo https://review.openstack.org/125124 | 17:53 |
*** reed has joined #openstack-infra | 17:53 | |
zaro | AJaeger_: was it named differently when in config? i didn't see it. | 17:53 |
*** _nadya_ has quit IRC | 17:54 | |
*** _nadya_ has joined #openstack-infra | 17:54 | |
zaro | clarkb: i think you should take a look at this one, https://review.openstack.org/#/c/118703/ | 17:55 |
*** pelix has quit IRC | 17:55 | |
*** sabeen3 has quit IRC | 17:55 | |
*** dangers is now known as dangers_away | 17:56 | |
AJaeger_ | zaro: see https://review.openstack.org/#/c/114933/ - and toggle ci to see the job was there as well | 17:56 |
AJaeger_ | "config-compare-xml Jenkins XML output has changed. in 2m 56s (non-voting)" | 17:56 |
zaro | AJaeger_: ohh, i see. i think it's actually only run in post pipeline. i only see it in merged changes. | 17:59 |
AJaeger_ | 114933 was not merged | 17:59 |
*** todd_dsm has quit IRC | 17:59 | |
*** arosen-home1 has joined #openstack-infra | 18:00 | |
clarkb | zaro: it should be rechecked once the xml comparison job is fixed | 18:01 |
*** zaphod42_ has joined #openstack-infra | 18:01 | |
arosen-home1 | How do i become part of an additional group in gerrit? For example congress-release https://github.com/openstack-infra/project-config/blob/master/gerrit/acls/stackforge/python-congressclient.config#L6 | 18:01 |
zaro | AJaeger_: then i'm confused. i don't see it in any of the open config changes. what determines when it gets run? | 18:01 |
*** MarkAtwood has quit IRC | 18:01 | |
clarkb | arosen-home1: two ways. if the group already has members they can add you. if there are no current members gerrit admins (basically infra core) can add the initial member | 18:02 |
*** MarkAtwood has joined #openstack-infra | 18:02 | |
AJaeger_ | zaro: it's not run since project-config was split up - you only see it in jobs before the split like 114933 | 18:03 |
*** johnthetubaguy is now known as zz_johnthetubagu | 18:03 | |
anteaya | pleia2: can you take a moment and help me understand this patch better? https://review.openstack.org/#/c/124569/1 | 18:04 |
*** zaphod42 has quit IRC | 18:04 | |
*** sarob has quit IRC | 18:04 | |
zaro | AJaeger_: ok. that clears it up. thanks. | 18:04 |
*** david-lyle is now known as david-lyle_afk | 18:05 | |
arosen-home1 | clarkb: yea looks like the group doesn't exist yet :/. Do i make the request through the infra-mailing list? Or i wonder if i should change the group to be congress-core ? | 18:05 |
*** hashar has quit IRC | 18:05 | |
clarkb | congress-release exists | 18:05 |
* arosen-home1 which does exist. | 18:05 | |
arosen-home1 | clarkb: anyone a part of it? | 18:05 |
clarkb | and no we usually do not want to mix those two groups | 18:05 |
clarkb | arosen-home1: nope. Who should be the initial member? I can add that person | 18:05 |
*** IAX has joined #openstack-infra | 18:05 | |
pleia2 | anteaya: sure | 18:05 |
arosen-home1 | Can you make me that person ? | 18:06 |
*** sdake_ has quit IRC | 18:06 | |
anteaya | pleia2: so basically it is using shell script to add a new conditional | 18:06 |
arosen-home1 | arosen is my launchpad id. | 18:06 |
clarkb | arosen-home1: sure | 18:06 |
arosen-home1 | clarkb: thanks! | 18:06 |
*** todd_dsm has joined #openstack-infra | 18:06 | |
anteaya | pleia2: and then it filters on each conditional, taking a different action based on the filters | 18:06 |
clarkb | arosen-home1: done. note we like to keep the release groups small to avoid having lots of people that can push tags | 18:06 |
clarkb | arosen-home1: since mispushed tags can be a giant pain to clean up | 18:06 |
arosen-home1 | clarkb: makes sense. | 18:06 |
anteaya | pleia2: and errors if the filters have no content | 18:06 |
anteaya | pleia2: yes? | 18:07 |
*** rmcall has joined #openstack-infra | 18:07 | |
*** otherwiseguy has quit IRC | 18:07 | |
anteaya | pleia2: and the action is the parsing of content to arrive at a name value for the MODULE variable | 18:08 |
*** _nadya_ has quit IRC | 18:08 | |
pleia2 | anteaya: so it checks to see whether we're using Modulefie (deprecated, see deprecation warning at: https://docs.puppetlabs.com/puppet/latest/reference/modules_publishing.html) or metadata.json (yay!) | 18:08 |
*** gokrokve has quit IRC | 18:08 | |
*** annegent_ has joined #openstack-infra | 18:09 | |
pleia2 | but yes, as you describe is correct | 18:09 |
anteaya | pleia2: great thanks | 18:09 |
*** gokrokve has joined #openstack-infra | 18:09 | |
sdague | mordred: ok, so that's a piece of things. However we also need to drop it from projects.txt after that, right? | 18:09 |
anteaya | pleia2: +2, thanks for taking the time to explain that to me | 18:10 |
*** gokrokve has quit IRC | 18:10 | |
mordred | sdague: no, it can still participate in projects.txt after that and it should have no ill effects on our gate | 18:10 |
pleia2 | anteaya: sure thing, you mostly got it yourself though :) | 18:10 |
*** gokrokve has joined #openstack-infra | 18:10 | |
mordred | the problem is that it was in projects.txt AND we were processing its git repo, which means that projects.txt syncing was going on with it | 18:11 |
*** jlibosva has quit IRC | 18:11 | |
pleia2 | anteaya: does also bring up the fact that we should switch to metadata.json at some point, but deprecated means we have some time ;) | 18:11 |
anteaya | pleia2: having you here helped me | 18:11 |
sdague | mordred: really? | 18:11 |
sdague | I thought the problem was it was in projects.txt | 18:11 |
*** sdake_ has joined #openstack-infra | 18:11 | |
sdague | in stable/icehouse | 18:11 |
anteaya | pleia2: ah the cushy pillow to fall back on | 18:11 |
sdague | so the pbr try to install things script fails | 18:12 |
mordred | sdague: that's only a problem if we are also processing its git repo in devstack | 18:12 |
*** _nadya_ has joined #openstack-infra | 18:12 | |
*** amuller has quit IRC | 18:12 | |
mordred | sdague: there are tons of things in projects.txt taht we do not test or care about | 18:12 |
mordred | sdague: dib was in a weird middle ground | 18:12 |
mordred | is | 18:12 |
mordred | sdague: where we install it from git as part of a normal devstack run | 18:12 |
mordred | which we do not do with other things in projects.txt that we don't care about | 18:13 |
*** todd_dsm has quit IRC | 18:14 | |
*** yolanda has quit IRC | 18:15 | |
sdague | mordred: yeh, but it does it by looping through /opt/stack/new right? | 18:15 |
sdague | so it doesn't matter what devstack does if the repo is there in d-g | 18:16 |
*** emagana_ has quit IRC | 18:16 | |
*** _nadya_ has quit IRC | 18:16 | |
*** hashar has joined #openstack-infra | 18:16 | |
*** emagana has joined #openstack-infra | 18:16 | |
mordred | sdague: right - but why would the repo be in /opt/stack/new ? | 18:16 |
*** HeOS has joined #openstack-infra | 18:17 | |
mordred | sdague: let me go look at code real quick | 18:17 |
*** gokrokve has quit IRC | 18:17 | |
sdague | because d-g puts it there | 18:17 |
sdague | devstack doesn't clone in the gate | 18:17 |
*** emagana has quit IRC | 18:17 | |
mordred | sdague: there are _several_ stackforge projects that are in projects.txt that subscribe to changes and tie themselves to what's in g-r | 18:19 |
*** linuxmohan has quit IRC | 18:19 | |
*** MarkAtwood has quit IRC | 18:19 | |
*** emagana has joined #openstack-infra | 18:19 | |
sdague | mordred: yeh, and I think they can all break us this way | 18:19 |
mordred | sdague: I disagree - and if they can, I think it's just a bug to fix | 18:19 |
sdague | they just haven't yet | 18:19 |
mordred | sdague: ok. it's like this. the problem is triggered by "setup_package_with_req_sync" | 18:20 |
mordred | sdague: that is called when we call setup_install on a directory | 18:20 |
mordred | we do not do that except for directories that we install from git as part of what we're doing | 18:20 |
mordred | since ironic uses dib, the way we had it installing was via setup_install from its git repo | 18:21 |
mordred | which meant we were reqs syncing it | 18:21 |
mordred | if we install from pip, we will not req sync it | 18:21 |
mordred | the other stackforge projects are not mentioned or installed from git as part of a normal intergrated run | 18:21 |
mordred | so they will not trigger it | 18:21 |
*** funzo has quit IRC | 18:21 | |
sdague | .... | 18:21 |
mordred | essentially, we are installing a tool using integrated gate mechanisms without it being part of teh integrated gate | 18:22 |
mordred | we need to install it not using integrated gate mechanisms | 18:22 |
sdague | ok | 18:23 |
sdague | so your patch actually as to propose to devstack icehouse then, correct? | 18:24 |
*** doude has joined #openstack-infra | 18:24 | |
mordred | yah. | 18:24 |
*** arxcruz has quit IRC | 18:24 | |
mordred | although I think we should investigate the other suggestion SpamapS was making | 18:24 |
mordred | of being able to enable via flag installing from git | 18:24 |
mordred | because that seems useful for devs | 18:24 |
EmilienM | mordred: hey, I'm a potential mentor for OPW and I'm trying to clarify https://wiki.openstack.org/wiki/OutreachProgramForWomen/Ideas#Community - hogepodge told me there ls some potential work in Puppet to do for openstack-infra. First question, do you have a blueprint or something listing what need to be done? (i.e. ask.o.o, etc) | 18:25 |
sdague | yeh... so that's probably part and parcel with the library proposal I have on the list | 18:25 |
mordred | sdague: yup | 18:25 |
fungi | EmilienM: we have a huge low-hanging-fruit one actually! | 18:25 |
mordred | sdague: apparently tripleo have written the logic in shell already | 18:25 |
bookwar | are there any docs/talks/slides explaining jenkins-job-builder parameters substitution? I mean at which point {smth} is substituted and what are the entry points and priorities for parameter values: project definition only, project + global defaults section, or project + global + job definition? I am trying to go through the sources but maybe there is a shortcut.. | 18:26 |
fungi | EmilienM: it's getting discussed at the infra team meeting in half an hour too | 18:26 |
EmilienM | fungi: the program takes 3 months afik | 18:26 |
*** xyang0 has quit IRC | 18:26 | |
fungi | EmilienM: yeah, this can be chunked up as needed i think. nibalizer is currently getting stuff lined up for it | 18:26 |
EmilienM | fungi: any pointer? | 18:26 |
fungi | EmilienM: and then we need volunteers for refactoring chunks | 18:26 |
*** mmedvede has quit IRC | 18:26 | |
SpamapS | mordred: we just have "packages" or "source" for element/install.d .. it may not be consumable for this exact case. | 18:27 |
* SpamapS goes on errand | 18:27 | |
EmilienM | fungi: what I need first is to identify a user story, like they have in other projects (i.e. https://wiki.openstack.org/wiki/OutreachProgramForWomen/Ideas#Glance_-_Swift_ranged_uploads) | 18:27 |
*** IAX has quit IRC | 18:27 | |
*** funzo has joined #openstack-infra | 18:27 | |
*** IAX has joined #openstack-infra | 18:28 | |
*** jp_at_hp has quit IRC | 18:28 | |
*** ivar-lazzaro has joined #openstack-infra | 18:28 | |
fungi | EmilienM: http://specs.openstack.org/openstack-infra/infra-specs/specs/puppet-modules.html is the one which comes to mind | 18:28 |
clarkb | bookwar: http://ci.openstack.org/jenkins-job-builder/definition.html something like that? | 18:28 |
EmilienM | fungi: this is awesome. | 18:28 |
*** ramashri has joined #openstack-infra | 18:28 | |
EmilienM | fungi: can I take it as example of an OPW internship? | 18:29 |
mordred | EmilienM: absolutely | 18:29 |
*** emagana has quit IRC | 18:29 | |
EmilienM | mordred: cool, thx | 18:29 |
sdague | SpamapS: direct git url might be handy, as there are a lot of tripleo repos, and I'm not familiar with them | 18:30 |
*** hashar has quit IRC | 18:30 | |
*** _buzz has quit IRC | 18:30 | |
fungi | EmilienM: though of course we have lots of other things which would also be good possibilities for opw work, i think. doesn't have to be puppet-specific | 18:30 |
openstackgerrit | A change was merged to openstack-infra/project-config: Fix indentation in rally-designate job config https://review.openstack.org/124346 | 18:31 |
EmilienM | fungi: feel free to give me specs links | 18:31 |
fungi | EmilienM: well, we have http://specs.openstack.org/openstack-infra/infra-specs/ approved and also https://review.openstack.org/#/q/status:open+project:openstack-infra/infra-specs,n,z proposed for consideration | 18:33 |
nibalizer | EmilienM: I'd be happy to help with that | 18:34 |
EmilienM | great! | 18:34 |
fungi | EmilienM: but there's tons more which we could probably mine out of our bug list and turn into low-level specs as well, for better clarity | 18:34 |
*** hashar has joined #openstack-infra | 18:35 | |
clarkb | sdague: fungi mordred any opposition to having the ironic sideways grenade job as a nonvoting check job on ironic, devstack, d-g, and grenade? | 18:36 |
sdague | nope | 18:36 |
EmilienM | fungi: I try to focus the internship on Puppet | 18:36 |
openstackgerrit | Andreas Jaeger proposed a change to openstack-infra/project-config: Group stackforge projects in layout.yaml https://review.openstack.org/124589 | 18:36 |
openstackgerrit | Andreas Jaeger proposed a change to openstack-infra/project-config: Add more checks for zuul/layout.yaml https://review.openstack.org/124593 | 18:36 |
openstackgerrit | Andreas Jaeger proposed a change to openstack-infra/project-config: Use tox -e zuul for layout test https://review.openstack.org/125101 | 18:36 |
*** kgavini has quit IRC | 18:36 | |
*** kgavini_ has joined #openstack-infra | 18:36 | |
SpamapS | sdague: https://git.openstack.org/cgit/openstack/diskimage-builder/tree/elements/base/extra-data.d/99-enable-install-types | 18:37 |
fungi | clarkb: no opposition from me | 18:37 |
clarkb | kk /me approves this change to make that a non voting job | 18:37 |
openstackgerrit | A change was merged to openstack-infra/project-config: Stop using deprecated TESTR_FULL option https://review.openstack.org/124404 | 18:37 |
nibalizer | EmilienM: yes, pulling out the modules, then maybe applying Modulesync to them, maybe writing tests, there is lots to do | 18:37 |
SpamapS | sdague: of course, we have things split into individual executables, rather than bash functions, so as I say, it might not help | 18:37 |
EmilienM | nibalizer: I like this one | 18:38 |
openstackgerrit | A change was merged to openstack-infra/infra-specs: HP owns everything. I own nothing. https://review.openstack.org/120188 | 18:38 |
*** tonytan_brb has joined #openstack-infra | 18:38 | |
*** tonytan4ever has quit IRC | 18:38 | |
* SpamapS does need to disappear for a bit | 18:38 | |
*** david-lyle_afk has quit IRC | 18:38 | |
AJaeger_ | clarkb, fungi, anteaya: Could you review my patch series soonish (starting at https://review.openstack.org/125101), please? I want to avoid more merge conflicts... | 18:38 |
clarkb | AJaeger_: sure | 18:39 |
*** cdent has quit IRC | 18:39 | |
clarkb | AJaeger_: these should be self testing right? | 18:40 |
openstackgerrit | A change was merged to openstack-infra/project-config: Move notifications for pycadf from oslo to keystone https://review.openstack.org/124449 | 18:40 |
AJaeger_ | clarkb: the last patch - 124593 - is the test | 18:40 |
AJaeger_ | clarkb: so far it only tests the ordering of stackforge projects. Let's do one step at a time ;9 | 18:41 |
*** thedodd has joined #openstack-infra | 18:41 | |
anteaya | nibalizer: ha ha ha | 18:41 |
anteaya | AJaeger_: looking | 18:41 |
openstackgerrit | A change was merged to openstack-infra/project-config: Promote sideways ironic grenade tests to non-voting checks https://review.openstack.org/124562 | 18:42 |
*** IAX has quit IRC | 18:42 | |
AJaeger_ | thanks, clarkb and anteaya | 18:42 |
nibalizer | anteaya: are you laughing because i implied we might write tests? | 18:42 |
*** mmedvede has joined #openstack-infra | 18:42 | |
AJaeger_ | thanks, fungi as well | 18:42 |
*** AzherKhan has quit IRC | 18:42 | |
*** khaazher has joined #openstack-infra | 18:43 | |
*** khaazher has quit IRC | 18:43 | |
*** khaazher has joined #openstack-infra | 18:43 | |
*** khaazher has quit IRC | 18:43 | |
anteaya | nibalizer: no that you admit you own nothing | 18:43 |
anteaya | nibalizer: your patch just merged in specs | 18:44 |
anteaya | ha ha ha | 18:44 |
fungi | anteaya: nibalizer: yeah, it seemed pointless to wait for a second +2 on that patch | 18:44 |
anteaya | nibalizer: I love that you write test, might write tests or imply that you might write tests | 18:44 |
anteaya | nibalizer: keep doing that | 18:44 |
openstackgerrit | A change was merged to openstack-infra/project-config: Move python33 test for Keystone to experimental https://review.openstack.org/124563 | 18:45 |
*** VijayTripathi has joined #openstack-infra | 18:45 | |
*** e0ne has quit IRC | 18:45 | |
anteaya | clarkb: I'll wait for you on 125101 | 18:45 |
openstackgerrit | Matthew Treinish proposed a change to openstack-infra/subunit2sql: Add manifest file to include base alembic config https://review.openstack.org/125151 | 18:45 |
*** yolanda has joined #openstack-infra | 18:46 | |
openstackgerrit | Steve Martinelli proposed a change to openstack-infra/config: Remove projects_alphabetized from tox.ini https://review.openstack.org/122600 | 18:46 |
*** VijayTripathi has quit IRC | 18:46 | |
openstackgerrit | Jeremy Stanley proposed a change to openstack-infra/project-config: Add more checks for zuul/layout.yaml https://review.openstack.org/124593 | 18:46 |
*** jgrimm is now known as zz_jgrimm | 18:46 | |
fungi | AJaeger_: corrected your commit message | 18:47 |
AJaeger_ | thanks, fungi | 18:47 |
*** MarkAtwood has joined #openstack-infra | 18:47 | |
sdague | SpamapS: yeh, this is probably different enough that it's not going to directly translate | 18:47 |
openstackgerrit | Steve Martinelli proposed a change to openstack-infra/config: Remove jjb from tox.ini https://review.openstack.org/125152 | 18:47 |
*** openstackgerrit has quit IRC | 18:47 | |
*** openstackgerrit has joined #openstack-infra | 18:48 | |
openstackgerrit | A change was merged to openstack-infra/project-config: macros: Manage metadata.json file in puppet-lint jobs https://review.openstack.org/124569 | 18:49 |
*** berendt has quit IRC | 18:49 | |
clarkb | AJaeger_: fungi: it looks like the merge template check is disabled | 18:51 |
clarkb | AJaeger_: fungi: should it be enabled? The commit message seems to imply that it is | 18:51 |
AJaeger_ | clarkb: it's disabled because it fails | 18:51 |
AJaeger_ | we need to merge first https://review.openstack.org/124588 | 18:51 |
AJaeger_ | sorry, I had a few too many loose ends coming together ;( | 18:52 |
fungi | AJaeger_: don't apologize. great work | 18:52 |
*** hdd has quit IRC | 18:53 | |
AJaeger_ | clarkb: I suggest to move the patch series in and I enhance 124588 to enable the merge check there | 18:53 |
arosen-home1 | Hi, i started seeing this weird nova failure in my CI but i think it's related to configuration: http://paste.openstack.org/show/117243/ . One thing that looks odd is nova complains saying: Nova requires libvirt version 0.9.11 or greater. | 18:53 |
arosen-home1 | If i'm using devstack-gate and nodepool with a ubuntu14.04 image i figured i'd already have that no? | 18:54 |
*** zz_jgrimm is now known as jgrimm | 18:54 | |
*** unicell has quit IRC | 18:56 | |
krotscheck | So, I want to contribute code that I wrote under the apache license to pecan, however pecan is licensed under BSD. What’s the appropriate approach there? | 18:56 |
clarkb | AJaeger_: sounds good | 18:57 |
krotscheck | i.e. There’s a part of StoryBoard that would be awesome for anyone using Pecan. | 18:57 |
fungi | krotscheck: they're compatible licenses, but pecan effectively becomes redistributed under the intersection of those licenses | 18:57 |
clarkb | or if you own the copyright on it then give it to pecan as bsd | 18:57 |
krotscheck | Hrm. | 18:58 |
clarkb | (assuming you are ok with those license terms) | 18:58 |
fungi | krotscheck: unless you can convince the copyright holder (probably your employer?) to relicense | 18:58 |
anteaya | clarkb: so do you want to +A those in the right order? | 18:58 |
krotscheck | Well, _I_ don’t care. | 18:58 |
fungi | yeah, what clarkb sais | 18:58 |
clarkb | anteaya: yup | 18:58 |
fungi | said | 18:58 |
krotscheck | But HP probably does. | 18:58 |
anteaya | clarkb: thanks | 18:58 |
AJaeger_ | thanks, clarkb ! | 18:58 |
* krotscheck wanders off to see who to ask about that. | 18:58 | |
anteaya | nice job AJaeger_ | 18:58 |
clarkb | krotscheck: lifeless | 18:58 |
fungi | er, union of those licenses, which leaves the intersection of their allowances i guess | 18:58 |
fungi | meetin' time, peoplez | 18:59 |
pleia2 | oh boy, time flies | 19:00 |
*** VijayTripathi has joined #openstack-infra | 19:00 | |
clarkb | so soon | 19:00 |
*** bcourt has quit IRC | 19:01 | |
*** andreykurilin_ has joined #openstack-infra | 19:02 | |
*** hashar has quit IRC | 19:02 | |
*** zaphod42_ has quit IRC | 19:02 | |
anteaya | clarkb: yeah I think it will be a bit before lifeless is online much, so perhaps someone else might be good to find | 19:02 |
*** rushiagr is now known as rushiagr_away | 19:02 | |
*** zaphod42 has joined #openstack-infra | 19:02 | |
*** chuckC has quit IRC | 19:03 | |
*** krtaylor has joined #openstack-infra | 19:03 | |
*** ChuckC_ is now known as ChuckC | 19:03 | |
clarkb | he is usually around this time of day | 19:03 |
*** chuckC_ has joined #openstack-infra | 19:03 | |
*** praneshp has quit IRC | 19:04 | |
*** gokrokve has joined #openstack-infra | 19:04 | |
*** annegentle has quit IRC | 19:04 | |
*** hashar has joined #openstack-infra | 19:04 | |
*** mmedvede has quit IRC | 19:05 | |
*** otherwiseguy has joined #openstack-infra | 19:05 | |
*** praneshp has joined #openstack-infra | 19:06 | |
openstackgerrit | A change was merged to openstack-infra/project-config: Use tox -e zuul for layout test https://review.openstack.org/125101 | 19:06 |
*** mrmartin has quit IRC | 19:07 | |
*** ramashri has quit IRC | 19:07 | |
*** tonytan_brb has quit IRC | 19:08 | |
*** praneshp has quit IRC | 19:08 | |
*** rfolco has quit IRC | 19:08 | |
openstackgerrit | A change was merged to openstack-infra/project-config: Group stackforge projects in layout.yaml https://review.openstack.org/124589 | 19:09 |
*** jgrimm is now known as zz_jgrimm | 19:10 | |
stevemar | AJaeger, haha, sorry about the can of worms! haha | 19:10 |
AJaeger_ | stevemar: thanks a lot for fixing these! | 19:13 |
*** dane_leblanc has quit IRC | 19:13 | |
openstackgerrit | Steve Martinelli proposed a change to openstack-infra/config: Remove projects_alphabetized from tox.ini https://review.openstack.org/122600 | 19:15 |
openstackgerrit | A change was merged to openstack-infra/project-config: Add more checks for zuul/layout.yaml https://review.openstack.org/124593 | 19:16 |
*** thedodd has quit IRC | 19:16 | |
AJaeger_ | clarkb: stevemar's is a cleanup patch for the project-config split... | 19:16 |
*** lttrl has quit IRC | 19:17 | |
*** bcourt has joined #openstack-infra | 19:17 | |
*** mmedvede has joined #openstack-infra | 19:17 | |
openstackgerrit | Steve Martinelli proposed a change to openstack-infra/config: Remove jjb from tox.ini https://review.openstack.org/125152 | 19:17 |
*** zz_jgrimm is now known as jgrimm | 19:18 | |
stevemar | AJaeger, why is there a tox env called 'upstream' | 19:18 |
stevemar | this doesn't seem to do anything... | 19:18 |
lifeless | clarkb: ? | 19:19 |
*** dane_leblanc has joined #openstack-infra | 19:19 | |
openstackgerrit | Steve Martinelli proposed a change to openstack-infra/config: Remove irc testenv from tox.ini https://review.openstack.org/125164 | 19:19 |
clarkb | lifeless: krotscheck had questions about licensing some storyboard apache2 licensed code as bsd udner pecan | 19:19 |
krotscheck | lifeless: See hipchat | 19:20 |
krotscheck | Or we can talk here. | 19:20 |
krotscheck | lifeless: This piece in particular -> http://git.openstack.org/cgit/openstack-infra/storyboard/tree/storyboard/api/middleware/cors_middleware.py | 19:21 |
*** ildikov has joined #openstack-infra | 19:21 | |
fungi | stevemar: i thought that one was intended to test whether specified "upstream" git repos for new projects were cloneable | 19:22 |
lifeless | where is pecans upstream ? | 19:23 |
openstackgerrit | Steve Martinelli proposed a change to openstack-infra/config: Remove undefined upstream environment from tox.ini https://review.openstack.org/125167 | 19:23 |
stevemar | fungi, my reasoning here ^ | 19:23 |
lifeless | krotscheck: I'm not signed into hipchat atm | 19:23 |
lifeless | krotscheck: I can if needed | 19:23 |
krotscheck | lifeless: I’ll paste. | 19:23 |
stevemar | fungi, upstream and projects_alphabetized where combined into 'projects' some time ago | 19:23 |
lifeless | krotscheck: that middleware looks eerily similar to r1chardj0n3s_afk's middleware | 19:23 |
fungi | stevemar: oh, yep | 19:23 |
stevemar | i don't know how that ever worked | 19:24 |
lifeless | except that yours is raw WSGI | 19:24 |
krotscheck | lifeless: Oh? | 19:24 |
stevemar | fungi, but they are no longer needed, so let's kill it with fire | 19:24 |
lifeless | krotscheck: https://review.openstack.org/#/c/120964/ | 19:24 |
fungi | stevemar: agreed | 19:24 |
openstackgerrit | Andreas Jaeger proposed a change to openstack-infra/project-config: Add merge-check to pylockfile, test for merge-check https://review.openstack.org/124588 | 19:24 |
krotscheck | Oh! | 19:24 |
krotscheck | NEat! | 19:24 |
*** baoli has quit IRC | 19:25 | |
*** melwitt has joined #openstack-infra | 19:25 | |
AJaeger_ | clarkb: 124588 enables the merge-check | 19:25 |
*** shashankhegde has quit IRC | 19:26 | |
lifeless | krotscheck: so to answer the question, contributing a patch or minor enhancement to bsd projects is I believe covered by the whitelist on 'does not need OSRB approval' | 19:26 |
*** david-lyle_afk has joined #openstack-infra | 19:26 | |
*** rfolco has joined #openstack-infra | 19:26 | |
*** kgavini_ has quit IRC | 19:26 | |
*** david-lyle_afk is now known as david-lyle | 19:26 | |
lifeless | krotscheck: I'd need to sign into the VPN to check the exact thing - but have a look on the OSRB web site you should find the flow chart pretty easily | 19:26 |
stevemar | sorry for all the patches AJaeger :\ | 19:27 |
openstackgerrit | A change was merged to openstack-infra/storyboard-webclient: Added draft build https://review.openstack.org/124934 | 19:27 |
krotscheck | lifeless: Excellent. I’ll go look at that. | 19:27 |
lifeless | krotscheck: if BSD isn't whitelisted then an OSRB proposal would be needed, but it would be trivial and actioned without issue | 19:27 |
lifeless | krotscheck: (also we should propose whitelisting BSD if its not) | 19:27 |
dougwig | anteaya: do you know if the brocade CI is supposed to be up right now? (i think it should). logs are 404. https://review.openstack.org/#/c/123468/ | 19:27 |
krotscheck | lifeless: I agree. Once the meeting’s over I have to switch over to the vpn anyway, so I’ll do that. | 19:28 |
AJaeger_ | stevemar: one big patch would have worked as well ;) But great to have it cleaned up! | 19:28 |
lifeless | krotscheck: that said, r1chardj0n3s_afk 's one looks like it will do what you need, and we can avoid having two of them L) | 19:28 |
lifeless | s/L/: | 19:28 |
anteaya | dougwig: I don't, can you check third-party-announce archives and tell me what the lastest says? | 19:28 |
stevemar | AJaeger, clean is good! | 19:28 |
*** [HeOS] has joined #openstack-infra | 19:29 | |
krotscheck | lifeless: Correct. Though I do feel that it’s generic enough to be useful for pecan, not just pecan + oslo. | 19:29 |
lifeless | sure | 19:30 |
*** baoli has joined #openstack-infra | 19:30 | |
lifeless | so is r1chardj0n3s_afk 's, modulo the use of oslo.config | 19:30 |
*** HeOS has quit IRC | 19:30 | |
krotscheck | lifeless: Though I think r1chardj0n3s_afk implementation is probably more thorough than mine. | 19:30 |
lifeless | anyhow, I don't have a dog in the fight | 19:30 |
dougwig | anteaya: looks like you disabled one of their ADX CI on 9/4 for the same reason. this is their vyatta CI. | 19:30 |
dougwig | nothing else in the archive. | 19:31 |
anteaya | kk thanks for looking | 19:31 |
anteaya | I appreciate it | 19:31 |
*** jergerber has joined #openstack-infra | 19:32 | |
openstackgerrit | Antoine Musso proposed a change to stackforge/python-jenkins: add get_version method https://review.openstack.org/124885 | 19:32 |
anteaya | clarkb: can you disable 10692 brocade-oss-service Brocade Vyatta CI DL-GRP-VYATTA-OSS@Brocade.com | 19:33 |
*** dprince has quit IRC | 19:34 | |
*** tonytan4ever has joined #openstack-infra | 19:35 | |
*** sdake_ has quit IRC | 19:35 | |
*** _nadya_ has joined #openstack-infra | 19:38 | |
fungi | anteaya: i can get it | 19:38 |
anteaya | fungi: thanks | 19:39 |
fungi | anteaya: disabled 10692 just now | 19:39 |
anteaya | thank you | 19:39 |
anteaya | will compose an email to -announce after infra meeting | 19:39 |
openstackgerrit | K Jonathan Harker proposed a change to openstack-infra/project-config: Clean up bashate failures https://review.openstack.org/124862 | 19:40 |
openstackgerrit | K Jonathan Harker proposed a change to openstack-infra/config: Clean up bashate failures https://review.openstack.org/118944 | 19:41 |
*** ramashri has joined #openstack-infra | 19:41 | |
openstackgerrit | Andreas Jaeger proposed a change to openstack-infra/project-config: Remove github-org from projects.yaml https://review.openstack.org/125169 | 19:42 |
*** praneshp has joined #openstack-infra | 19:42 | |
*** atiwari has quit IRC | 19:46 | |
krotscheck | In the interest of landing more storyboard things, I have this! https://review.openstack.org/#/c/124936/ | 19:46 |
bookwar | clarkb: I was curious about how to pass variables into macros definition. i mean whether variables defined for a project are global or not. It seems they aren't. In this example http://fpaste.org/137904/21043561/ , without lines 13-14 it doesn't work. | 19:48 |
*** cdent has joined #openstack-infra | 19:48 | |
clarkb | bookwar: ya they are not global | 19:49 |
*** e0ne has joined #openstack-infra | 19:50 | |
*** zehicle_at_dell has joined #openstack-infra | 19:51 | |
*** Sukhdev has quit IRC | 19:51 | |
*** annegent_ has quit IRC | 19:51 | |
openstackgerrit | Steven Dake proposed a change to openstack-infra/project-config: Add Kolla Project https://review.openstack.org/124453 | 19:51 |
anteaya | 3 | 19:53 |
*** shashankhegde has joined #openstack-infra | 19:53 | |
*** radez is now known as radez_g0n3 | 19:55 | |
*** zns has quit IRC | 19:56 | |
*** thedodd has joined #openstack-infra | 19:58 | |
*** changbl has quit IRC | 19:58 | |
*** zns has joined #openstack-infra | 19:58 | |
*** changbl has joined #openstack-infra | 19:59 | |
clarkb | asselin: git clone https://git.openstack.org/openstack-infra/project-config then make changes? | 20:00 |
*** rlandy has quit IRC | 20:00 | |
clarkb | so I have bad news on the dib front. we appear to have completely broken all image updates | 20:00 |
sdague | fungi: do you have a review up for pulling all the havana jobs? | 20:00 |
clarkb | no images have been updated in a week :( | 20:00 |
*** sarob has joined #openstack-infra | 20:00 | |
jesusaurus | clarkb: so maintain a fork? | 20:00 |
clarkb | jesusaurus: I mean thats what you are wanting to do... | 20:01 |
*** baoli has quit IRC | 20:01 | |
asselin | clarkb, yeah, I'm thinking I'll mirror that repo locally, and create a branch with my stuff, and periodically merge in master. | 20:01 |
clarkb | you are literally saying "I want your stuff + my changes" (a fork) | 20:01 |
AJaeger_ | clarkb, oops ;( is there no way to get notified about such failures? | 20:01 |
clarkb | AJaeger_: not today. I am trying to look into it, but other eyes would be helpful | 20:01 |
clarkb | mordred: fungi ^ | 20:01 |
asselin | clarkb, ok 'fork'. | 20:01 |
krtaylor | adalbas, ^^^ | 20:02 |
*** mfer has quit IRC | 20:02 | |
pleia2 | hogepodge: might be interested in the tail end of our meeting today, talked about 3rd party ci: http://eavesdrop.openstack.org/meetings/infra/2014/infra.2014-09-30-19.00.html | 20:02 |
*** annegentle has joined #openstack-infra | 20:02 | |
openstackgerrit | A change was merged to openstack-infra/elastic-recheck: add links to specific bugs https://review.openstack.org/124921 | 20:03 |
*** AJaeger_ has quit IRC | 20:04 | |
fungi | sdague: i think it already merged? | 20:04 |
fungi | just a sec | 20:04 |
sdague | I just had grenade-icehouse job run on tempest | 20:04 |
openstackgerrit | A change was merged to openstack-infra/elastic-recheck: fix Launchpad link name https://review.openstack.org/124928 | 20:04 |
*** sarob has quit IRC | 20:04 | |
*** baoli has joined #openstack-infra | 20:05 | |
*** JayJ has quit IRC | 20:06 | |
asselin | anteaya, krtaylor I won't be around for next monday's 3rd party meeting, but I think clarkb's answer makes sense. This is my plan: 'fork'. More specifically: " I'll mirror that repo locally, and create a branch with my stuff, and periodically merge in master." | 20:06 |
*** JayJ has joined #openstack-infra | 20:06 | |
*** IAX has joined #openstack-infra | 20:07 | |
anteaya | asselin: thanks for letting us know your status and plans | 20:07 |
anteaya | asselin: let us know how that workflow works for you | 20:07 |
anteaya | asselin: and if you have any suggestions for how to improve it | 20:07 |
anteaya | as well as documenting it | 20:07 |
asselin | ok, sure. probaly won't get to it until Oct 15th though. Yes, if there's a recommended place to document it, let me know. | 20:08 |
krtaylor | asselin, understood, we can bring it up Monday | 20:08 |
asselin | thanks. going to lunch | 20:09 |
krtaylor | anteaya, related, how about we discuss any possible third-party test system impact of system-config rename in Monday's third-party meeting? | 20:09 |
*** kgiusti has left #openstack-infra | 20:10 | |
*** mfer has joined #openstack-infra | 20:10 | |
*** marun has quit IRC | 20:10 | |
*** cnesa15 has quit IRC | 20:10 | |
clarkb | jesusaurus: asselin: in particular we wanted to separate the system configs from the project configs so that others doing ci similar to us can reconsume the system but have different project configs | 20:12 |
anteaya | krtaylor: sure, I may or may not know | 20:13 |
anteaya | krtaylor: since I don't run a third party ci | 20:13 |
jesusaurus | clarkb: yeah, it just wasnt clear to me if the intent was for project-config to be forked, mimicked, or thrown out the window by downstreams | 20:13 |
anteaya | krtaylor: but I will share what I know, if I know anything | 20:13 |
*** stevemar has quit IRC | 20:13 | |
*** ociuhandu has quit IRC | 20:13 | |
*** shashankhegde has quit IRC | 20:13 | |
anteaya | jesusaurus: I think the format should be used but no necessarily the values | 20:14 |
anteaya | s/no/not | 20:14 |
jesusaurus | but which of those is best is probably dependent on the downstream and what they are trying to do | 20:14 |
clarkb | fungi: mordred http://paste.openstack.org/show/117261/ finally found the exception. I am working on a fix | 20:14 |
clarkb | jesusaurus: yup agreed | 20:14 |
krtaylor | anteaya, well, I would expect that there wouldn't be as much impact for those systems that have already integrated or forked project-config | 20:15 |
krtaylor | anteaya, mainly a heads-up I guess | 20:15 |
krtaylor | anteaya, I'll add it to the agenda | 20:15 |
anteaya | krtaylor: can't hurt | 20:16 |
fungi | clarkb: thanks. i'm in far too many conversations at once to pay attention to what that was about just now | 20:16 |
fungi | oh, scrollback says image updates went stale. ick | 20:16 |
openstackgerrit | Clark Boylan proposed a change to openstack-infra/nodepool: Properly reference nodepool config in image update https://review.openstack.org/125186 | 20:16 |
clarkb | fungi: mordred ^ that should fix image updates | 20:16 |
sdague | fungi: can you see if the ER bot is lost? | 20:17 |
sdague | it should work now that ES is back | 20:17 |
*** IAX has quit IRC | 20:17 | |
*** kgavini has joined #openstack-infra | 20:18 | |
fungi | sdague: the er bot was dead since august when someone changed how its challen list is configured. did that get fixed yet? | 20:19 |
fungi | s/challen/channel/ | 20:19 |
sdague | fungi: oh... hmph | 20:19 |
*** Guest43089 has joined #openstack-infra | 20:20 | |
sdague | if you pastebin an error, I can fix post TC meeting | 20:20 |
*** annegent_ has joined #openstack-infra | 20:20 | |
fungi | sdague: k, i can do that post meeting | 20:20 |
*** Guest43089 is now known as annegentle_ | 20:20 | |
*** annegent_ has quit IRC | 20:20 | |
*** annegentle has quit IRC | 20:21 | |
*** homeless has quit IRC | 20:21 | |
*** mordred is now known as jeblair | 20:21 | |
*** jeblair is now known as mordred | 20:21 | |
*** homeless has joined #openstack-infra | 20:22 | |
clarkb | mordred: reviewing 125186 when you are done meeting would be great. also chown: invalid user: `jenkins:jenkins' is happening in my precise builds now. checking trusty real quick tpoo | 20:23 |
mordred | clarkb: sounds like jenkins user is not being added | 20:24 |
clarkb | mordred: ya, not sure why yet | 20:24 |
arosen-home1 | I'm trying to do a release of a python-client on pypi. I've added a tag 1.0.0a1 and pushed that to gerrit. Now, when i run: python setup.py sdist upload -r python-congressclient i get this: http://paste.openstack.org/show/117265/ | 20:25 |
arosen-home1 | am i missing something more? | 20:25 |
*** _nadya_ has quit IRC | 20:25 | |
mordred | arosen-home1: we actually usually just have infra publish things to pypi for you when you push a tag | 20:25 |
*** marun has joined #openstack-infra | 20:25 | |
arosen-home1 | mordred: ah i have to add infra to my pypi package i registered i guess? | 20:26 |
mordred | arosen-home1: I've never seen that error though | 20:26 |
mordred | arosen-home1: openstackci is the account you want | 20:26 |
clarkb | also don't use setup.py for that stuff. iirc it does so via http not https. its super secure | 20:26 |
*** shashankhegde has joined #openstack-infra | 20:26 | |
clarkb | we use that tool dstufft wrote to do it instead | 20:26 |
arosen-home1 | mordred: i added openstackci as a owner role. Will the openstackci eventually retry to upload the package? | 20:27 |
*** koolhead17 has quit IRC | 20:27 | |
*** ihrachyshka has quit IRC | 20:27 | |
clarkb | arosen-home1: no, you need to correct jobs in zuul and its a one shot thing | 20:27 |
*** julim has quit IRC | 20:27 | |
clarkb | if stuff fails we manually kick it | 20:27 |
fungi | clarkb: arosen-home1: "that tool dstufft wrote" is twine | 20:28 |
clarkb | ty I forgot the name | 20:28 |
*** kgavini__ has joined #openstack-infra | 20:28 | |
*** kgavini__ has quit IRC | 20:29 | |
arosen-home1 | is there a place i can look to see if this worked or failed? | 20:30 |
arosen-home1 | I guess if it failed I have to bug you guys? | 20:30 |
*** kgavini has quit IRC | 20:30 | |
fungi | arosen-home1: yeah, we have logs | 20:30 |
arosen-home1 | where would i see the logs for such a thing? | 20:31 |
clarkb | arosen-home1: I don't think you have the jobs | 20:31 |
fungi | arosen-home1: logs.openstack.org will have a log for the git sha of the tyag itself (you can find it by running git show-ref tagnum) | 20:31 |
clarkb | https://git.openstack.org/cgit/openstack-infra/project-config/tree/zuul/layout.yaml#n4732 doesn't list pypy jobs | 20:31 |
clarkb | and ya what fungi said http://logs.openstack.org/firsttwoofthatsha1/sha1 | 20:31 |
fungi | it'll appear in a pre-release or release subdirectory there | 20:32 |
*** zns has quit IRC | 20:32 | |
*** cdent has quit IRC | 20:33 | |
openstackgerrit | Aaron Rosen proposed a change to openstack-infra/project-config: Add publish-to-pypi job for python-congressclient https://review.openstack.org/125189 | 20:33 |
arosen-home1 | so once ^ merges will the pypi package be pushed ? | 20:33 |
arosen-home1 | or i guess the tag has to be done after this is setup. | 20:34 |
fungi | arosen-home1: no, but a subsequent tag would cause the next version to be | 20:34 |
arosen-home1 | k, thanks again guys. | 20:35 |
*** datsun180b has quit IRC | 20:35 | |
*** juzuluag has quit IRC | 20:36 | |
*** stevemar has joined #openstack-infra | 20:37 | |
*** koolhead17 has joined #openstack-infra | 20:37 | |
rfolco | clarkb, my forked openstack_functions.py has params['OFFLINE_NODE_WHEN_COMPLETE'] = '1', jenkins says the slaves are offline, but nodepool says the slaves are ready... so the job finishes and slaves are not deleted by nodepool. I can't find any evidences in the logs... could you give some advice to point me to the right direction ? | 20:39 |
*** andreaf has quit IRC | 20:39 | |
*** mmedvede has quit IRC | 20:39 | |
*** boden has quit IRC | 20:39 | |
*** sacharya has left #openstack-infra | 20:39 | |
*** andreaf has joined #openstack-infra | 20:39 | |
clarkb | rfolco: are you publishing jenkins zmq events to nodepool? you need those so that nodepool can know when to delete the nodes | 20:39 |
*** andreaf has quit IRC | 20:40 | |
clarkb | you also need them for nodepool to know a node is used so I think that explains it if it is missing | 20:40 |
*** andreaf has joined #openstack-infra | 20:40 | |
rfolco | clarkb, this one? | 20:40 |
rfolco | Jenkins Event Publisher (via ZMQ PUB SUB) | 20:40 |
rfolco | This plugin publishes Jenkins build events in JSON as a ZMQ Publisher. | 20:40 |
*** datsun180b has joined #openstack-infra | 20:40 | |
stevemar | fungi, err, question about test hooks for dsvm | 20:40 |
stevemar | do post_test_hook functions get run *after* devstack is torn down? | 20:41 |
clarkb | rfolco: ya | 20:42 |
rfolco | clarkb, I'll make sure I have 0.0.3 and re-rest | 20:42 |
*** hashar has quit IRC | 20:42 | |
clarkb | rfolco: nodepool needs that feed to know when to update the nodes | 20:42 |
*** bhuvan has quit IRC | 20:42 | |
*** zns has joined #openstack-infra | 20:43 | |
fungi | stevemar: not sure. i'd need to read through the devstack-gate scripts (unless you beat me to it) | 20:43 |
rfolco | clarkb, perhaps some configs set at http://git.openstack.org/cgit/openstack-infra/config/tree/modules/openstack_project/manifests/jenkins.pp do not exist in my fork :( -- thanks for the pointers. | 20:43 |
*** hashar has joined #openstack-infra | 20:43 | |
stevemar | fungi, hmmmm on it now... https://github.com/anteaya/project-config/blob/4cbe40aa954b7ac6bcd12b98039bfc1ae6e3247e/modules/openstack_project/files/jenkins_job_builder/config/devstack-gate.yaml | 20:44 |
stevemar | theres a pre, post, and just gate_hook | 20:44 |
*** ppai has left #openstack-infra | 20:45 | |
*** rmcall has quit IRC | 20:47 | |
*** sarob_away is now known as sarob | 20:48 | |
*** akerr has joined #openstack-infra | 20:48 | |
jogo | clarkb: when would tempest consume say nova unit tests? | 20:48 |
openstackgerrit | Khai Do proposed a change to stackforge/python-jenkins: generate a code coverage html report https://review.openstack.org/125194 | 20:49 |
clarkb | jogo: if/when nova unittests expose code to test nova in a particular way | 20:49 |
EmilienM | fungi: clarkb: when you got 2 min, could you have a look at the new project accepted by TC ? 'tripleo-puppet-elements into OpenStack' https://review.openstack.org/#/c/124560 | 20:49 |
clarkb | jogo: in exactly the same way hacking does it | 20:49 |
clarkb | jogo: test code is consumable code. | 20:49 |
*** yamahata has joined #openstack-infra | 20:49 | |
jogo | clarkb: I don't follow | 20:49 |
jogo | what is the hacking example? | 20:49 |
clarkb | jogo: hacking consumes pep8's unittests | 20:49 |
clarkb | or is it flake8? one of them | 20:50 |
jogo | hacking is a flake8 plugin | 20:50 |
clarkb | yes but hacking has tests | 20:50 |
clarkb | that specifically import another projects tests | 20:50 |
clarkb | to make use of them | 20:50 |
jogo | clarkb: it has a non-voting job that does that yes | 20:50 |
clarkb | jogo: it was voting last I saw | 20:50 |
jogo | clarkb: we must be thinking of different things | 20:51 |
jogo | clarkb: the case I envision is: I cleanup some nova unit tests. My patch only touches files in nova/tests/* | 20:51 |
jogo | what does running a dsvm job do for that case? | 20:51 |
*** bhuvan has joined #openstack-infra | 20:52 | |
clarkb | jogo: it makes sure that your changes to importable and consumable code do not break consumers of that code | 20:52 |
jogo | clarkb: nothing in dsvm should be consuming that code | 20:53 |
akerr | Hi infra. If anyone familiar with zuul pipeline layouts could take a look at my yaml it'd be really helpful. We're deploying a gerrit/zuul/jenkins/nodepool code review/gate system internally to vet our cinder driver code before we push out to the community and I can't seem to get this pipeline to trigger properly. I think it might have something to do with my "require" tags: http://paste.openstack.org/show/117267/ The idea is that this should | 20:53 |
akerr | trigger when someone Release+1 comments, if all the other required votes are present | 20:53 |
clarkb | jogo: maybe not today | 20:53 |
jogo | today we don't explicitly state that, but we should state that IMHO | 20:53 |
clarkb | jogo: I disagree | 20:54 |
jogo | nova/tests is NOT a stable API | 20:54 |
jogo | and consume at your own risk | 20:54 |
clarkb | jogo: then it should not be in the nova namespace | 20:54 |
clarkb | jogo: it is packaged and included whenever you install nova | 20:54 |
jogo | clarkb: not all of nova namespace is a stable API | 20:54 |
clarkb | which is intentional | 20:54 |
*** oanufriev has quit IRC | 20:54 | |
*** dimtruck is now known as zz_dimtruck | 20:55 | |
jogo | clarkb: we have a lot of code in nova namespace that isn't considered a stable API | 20:55 |
clarkb | jogo: yes and its the massive problem that nova development is currently facing | 20:55 |
clarkb | you dont' have stable apis so everything must be in tree going through one group of people | 20:55 |
jogo | clarkb: that is hardly a related issue | 20:56 |
jogo | yes they are both about APIs | 20:56 |
fungi | sdague: here's the traceback from the recheck bot, when you have a moment to go over it http://paste.openstack.org/show/117268/ (though the traceback i saw a few weeks ago was a parsing issue with the channels list in the yaml file, so this may be yet new bitrot on top) | 20:56 |
jogo | clarkb: we will never have a case where we say all the things are stable APIs | 20:56 |
jogo | in nova the only thing that is a true public API is the REST API toeay | 20:56 |
clarkb | jogo: thats fine. I am just trying to point otu that we explicitly do unittests this way for the reasons that hacking has for it | 20:56 |
jogo | today | 20:56 |
clarkb | jogo: and to compeltely shutdown that usecase is a change in status quo | 20:57 |
*** oanufriev has joined #openstack-infra | 20:57 | |
jogo | clarkb: I still don't see what hacking has to do with this | 20:57 |
clarkb | jogo: hacking consumes other projects unittests | 20:57 |
jogo | clarkb: its not a use case we should even allow | 20:57 |
clarkb | why not? | 20:57 |
clarkb | hacking does it | 20:57 |
jogo | clarkb: hacking just needs the code to be well python | 20:57 |
clarkb | and mordred has specifically fougth for this use case | 20:57 |
clarkb | unitests are useful code | 20:57 |
clarkb | make them consumeable | 20:58 |
jogo | clarkb: sure unit tests are useful code | 20:58 |
jogo | clarkb: but they are not a stable API | 20:58 |
jogo | we change them at will | 20:58 |
clarkb | jogo: the __init__.py has been pretty stable | 20:58 |
clarkb | jogo: and honestly has to be out of necessity so that the 10k tests in nova continue to work | 20:58 |
clarkb | or is it test.py in nova? | 20:59 |
jogo | just because something hasn't changed much doesn't mean its a public stable API | 20:59 |
mordred | jogo: more things need stable public apis | 20:59 |
mordred | that we keep saying that we dont' have stable apis is lazy | 20:59 |
clarkb | jogo: no but in this case it is because it must be stable | 20:59 |
clarkb | otherwise you have 10k flaky tests | 20:59 |
mordred | and it's time to do less of it, not more | 20:59 |
jogo | mordred: but unit tests? | 20:59 |
mordred | everything | 20:59 |
*** mfer has quit IRC | 20:59 | |
mordred | EVERYTHIN | 20:59 |
mordred | people are building giant systems on top of this | 21:00 |
mordred | for us to try to say we don't expect people to use our internal apis is bananas | 21:00 |
jogo | mordred: if everything is a stable API we cannot change anything | 21:00 |
mordred | sure we can | 21:00 |
mordred | we just do it sensibly | 21:00 |
clarkb | mordred: going to bug yo uabout https://review.openstack.org/#/c/125186/1 again :) | 21:00 |
jogo | mordred: we don't say that, we say this is not a public API it may change. but go for it | 21:00 |
*** hdd has joined #openstack-infra | 21:00 | |
mordred | jogo: we actively do much worse than that | 21:01 |
*** sarob is now known as sarob_away | 21:01 | |
krotscheck | Can a core take a look at this projects-config patch? It’s a minor tweak - https://review.openstack.org/#/c/124936/ | 21:01 |
jogo | mordred: sure | 21:01 |
mordred | jogo: we regularly change interface definitions and the unittests that use them | 21:01 |
*** baoli has quit IRC | 21:01 | |
mordred | jogo: this broke ironic several times this cycle | 21:01 |
jogo | mordred: all I am saying is that nova unit tests should not be deemed a public (and this stable) API | 21:01 |
mordred | and it's just pure laziness | 21:01 |
jogo | we can move around code at will in unit tests | 21:01 |
mordred | jogo: I agree, actually, I don't think unit tests need to be a public api | 21:01 |
mordred | BUT | 21:01 |
*** MarkAtwood has quit IRC | 21:02 | |
mordred | I do believe that making sure that a change to the unittets don't break the ability to consume the unittest SYSTEM is important | 21:02 |
clarkb | krotscheck: I would but I already have :) | 21:02 |
jogo | mordred: well ironic was using an interface that was *not* public | 21:02 |
krotscheck | clarkb: Clone yourself! | 21:02 |
mordred | jogo: that should embarrass a lot of people | 21:02 |
clarkb | krotscheck: if only | 21:02 |
jogo | mordred: sure | 21:02 |
mordred | since that's a CLEAR plugin point | 21:02 |
jogo | mordred: we sadly have plugins to unstable APIs | 21:02 |
jogo | why? I don't know | 21:03 |
mordred | that should stop. right. now. | 21:03 |
mordred | it's very easy to stop it | 21:03 |
mordred | we just stop it | 21:03 |
mordred | it's code | 21:03 |
mordred | we can make choices | 21:03 |
mordred | like not having unstable plugin apis | 21:03 |
mordred | whatever it is now? stable | 21:03 |
mordred | done | 21:03 |
mordred | now be careful to only add things, etc. | 21:03 |
jogo | mordred: I have to double check the entry points etc. that we have | 21:03 |
jogo | but we have a lot | 21:03 |
*** r1chardj0n3s_afk is now known as r1chardj0n3s | 21:03 | |
mordred | yes. we do. we became production software a long time ago | 21:04 |
mordred | those should all be considered defacto API contracts | 21:04 |
r1chardj0n3s | krotscheck: are you around? I just woke up and saw your discussion about my CORS middleware with lifeless :) | 21:04 |
sdague | all those things can be changed in a backwards compatible way, you just need to get into the mindset of it | 21:04 |
jogo | mordred: so going back to the nova unit test and running tempest example | 21:04 |
krotscheck | r1chardj0n3s: Nope. I am a turing machine. | 21:04 |
r1chardj0n3s | krotscheck: cool | 21:04 |
* krotscheck actually means yes. | 21:04 | |
r1chardj0n3s | krotscheck: I don't discriminate :) | 21:04 |
mordred | jogo: the only reason I'd personally fight to keep that | 21:04 |
fungi | krotscheck: i have this halting problem i'd like your help with ;) | 21:05 |
krotscheck | fungi: oh? | 21:05 |
clarkb | fungi++ | 21:05 |
mordred | jogo: is to catch places where someone does something stupid in a file that causes nova itself to not be runnable for $reason | 21:05 |
fungi | krotscheck: http://en.wikipedia.org/wiki/Halting_problem | 21:05 |
mordred | jogo: I'm fairly certain unittests themselves would catch that | 21:05 |
krotscheck | fungi: How do you feel about having this halting problem i’d like your help with [/eliza] | 21:05 |
mordred | jogo: however, I'm not sure we've got a great way to express that in zuul | 21:05 |
fungi | heh | 21:05 |
openstackgerrit | A change was merged to openstack-infra/project-config: Call build:draft instead of build for js-draft build https://review.openstack.org/124936 | 21:05 |
mordred | jogo: nor do I really believe it'll gain us a ton | 21:05 |
mordred | jogo: to warrant the development effort | 21:06 |
clarkb | jogo: mordred ya the benefits are minimal | 21:06 |
r1chardj0n3s | krotscheck: just happy to answer questions if you have then. there is also the related spec for oslo kilo https://review.openstack.org/#/c/119485/ | 21:06 |
clarkb | people don't even run pep8 before pusing code | 21:06 |
clarkb | so not running the harde rto run integration tests doesn't optimize anything | 21:06 |
krotscheck | r1chardj0n3s: Looking. | 21:06 |
*** zz_dimtruck is now known as dimtruck | 21:07 | |
jogo | mordred: so your saying you want to keep the integration jobs around for unit test only changes for the case where breaking a unit test actually breaks nova | 21:07 |
r1chardj0n3s | krotscheck: no idea whether it's even useful for your use-case as I don't know what that is ;) | 21:07 |
mordred | jogo: yah. since we don't actually test nova's unittests with tempest, I doubt running them on those changes will catch many other things | 21:07 |
sdague | jogo: so zuul doesn't support the kind of change logic you are proposing | 21:07 |
krotscheck | r1chardj0n3s: My use case is CORS support on the api at storyboard.openstack.org, ergo yes, it’s useful. | 21:07 |
mordred | and since we use discovery to look for tests, import errors should get caught | 21:08 |
sdague | you can't not run tests because of the output of diff | 21:08 |
r1chardj0n3s | krotscheck: ok, cool | 21:08 |
krotscheck | r1chardj0n3s: And using oslo is definitely an option, though I might suggest that CORS is something that pecan can benefit from too :) | 21:08 |
mordred | jogo: but my main concern is that it's not an easy fix | 21:08 |
*** otherwiseguy has quit IRC | 21:08 | |
mordred | jogo: because of what sdague is saying | 21:08 |
sdague | I've actually spoken with jeblair on that some | 21:08 |
jogo | sdague: how much work wound it be to support something like that somewhere (maybe outside of zuul?) | 21:08 |
r1chardj0n3s | krotscheck: pecan? | 21:08 |
mordred | jogo: it has to be in zuul | 21:08 |
sdague | jogo: you can't outside of zuul | 21:08 |
krotscheck | r1chardj0n3s: Touche | 21:08 |
jogo | sdague mordred: so that is a fair point, that it isn't a quick change | 21:08 |
krotscheck | r1chardj0n3s: Because middleware intercepts before pecan. | 21:09 |
*** otherwiseguy has joined #openstack-infra | 21:09 | |
* krotscheck will ponder this. | 21:09 | |
* r1chardj0n3s is ignorant of pecan | 21:09 | |
sdague | jogo: it actually hits one of my hot button issues of people throwing around "we should just change this part of our process" | 21:09 |
sdague | without understanding the tooling implication | 21:09 |
clarkb | sdague++ | 21:09 |
*** mmedvede has joined #openstack-infra | 21:09 | |
krotscheck | r1chardj0n3s: is a path->object mapper for wsgi | 21:09 |
mordred | krotscheck, r1chardj0n3s: dhellmann may have thoughts | 21:09 |
clarkb | sdague: we are very good at saying "X should just Y" | 21:09 |
r1chardj0n3s | krotscheck: ok, thanks | 21:09 |
HenryG | clarkb: Do you have a minute? I have a question about modules/openstack_project/manifests/slave_db.pp | 21:09 |
sdague | and not realizing that process without supporting tooling is just giant frustration, pain, and disillusionment | 21:10 |
fungi | anyone who wants to make sweeping proposals is welcome to throw together a rough draft patch to our tooling to explain how they expect it should work | 21:10 |
clarkb | mordred: trusty seems broken too | 21:10 |
krotscheck | dhellmann: Thoughts on CORS Middleware as a part of Oslo, Pecan, and/or other things? | 21:10 |
*** boden has joined #openstack-infra | 21:10 | |
jogo | I am uncomfortable with the logical extreme of this argument. That we shouldn't do X because our tooling today doesn't support it | 21:10 |
*** boden has quit IRC | 21:10 | |
ryanpetrello | from a maintainer of pecan standpoint, I'd prefer it *not* be in pecan | 21:10 |
clarkb | mordred: for dib building so might be related to config split. I am debugging | 21:10 |
mordred | clarkb: AWESOME | 21:10 |
r1chardj0n3s | krotscheck, dhellmann: I would prefer that CORS be unencumbered | 21:11 |
mordred | jogo: yah. agree | 21:11 |
ryanpetrello | I tend to shy away from implementing things in pecan's core if I can help it; I try to keep the feature set as sparse as possible | 21:11 |
clarkb | jogo: thats not the argument | 21:11 |
anteaya | HenryG: best to just ask the question | 21:11 |
*** boden has joined #openstack-infra | 21:11 | |
clarkb | jogo: the argument is we haev a problem where we say lets just do X ifnra | 21:11 |
clarkb | jogo: and the expectation is it will just happen tomorrow | 21:11 |
clarkb | jogo: see libvirt testing discussion | 21:11 |
r1chardj0n3s | krotscheck:, dhellmann: that is, it shouldn't require the deployer to have to worry about what pecan might be to use it | 21:11 |
*** boden has quit IRC | 21:11 | |
*** aysyanne has quit IRC | 21:11 | |
dhellmann | krotscheck: putting CORS in pecan would make it only available to the projects using pecan, which isn't a lot of them yet | 21:12 |
HenryG | anteaya: Yeah, I should know better by now :( | 21:12 |
jogo | clarkb: this is exactly why I am bringing this up here to see what the issues/implications etc are | 21:12 |
krotscheck | r1chardj0n3s, dhellmann: Ditto with Oslo. | 21:12 |
stevemar | fungi, ughhh, i don't think a lot of projects are doing functional tests properly :( | 21:12 |
fungi | or more to the point, some suggestions which can be described simply at a high level very well may entail extremely complex implementations | 21:12 |
anteaya | HenryG: :) | 21:12 |
*** boden has joined #openstack-infra | 21:12 | |
marun | sdague: then zuul is broken and needs to change | 21:12 |
mordred | stevemar: I'm certain of that | 21:12 |
mordred | marun: aroo? | 21:12 |
marun | re: we can't limit the jobs run based on the diff | 21:12 |
ryanpetrello | dhellmann: krotscheck: is there a way to accomplish the CORS from a strictly WSGI middleware approach (without something pecan-specific) | 21:12 |
dhellmann | krotscheck, r1chardj0n3s, ryanpetrello : middleware seems like the right approach here | 21:12 |
ryanpetrello | this would benefit the entire community | 21:12 |
jogo | broken is a strong word | 21:12 |
*** boden has quit IRC | 21:12 | |
dhellmann | ryanpetrello: https://review.openstack.org/#/c/119485/6/specs/kilo/cors-middware.rst | 21:12 |
marun | jogo: broken -> file a bug -> fix it | 21:12 |
fungi | stevemar: perhaps. i thought many of them were doing it with an overridden gate_hook? | 21:12 |
*** boden has joined #openstack-infra | 21:13 | |
jogo | marun: you should file a zuul bug then :) | 21:13 |
krotscheck | dhellmann, r1chardj0n3s: I agree that middleware’s the right approach. I’m questioning the oslo.config dependency. | 21:13 |
mordred | marun: I believe "doesn't have that feature at the moment because we quite literally have not ever defined that as being a use case we want" might be more what you meant | 21:13 |
*** krtaylor has quit IRC | 21:13 | |
stevemar | fungi, some are, some aren't - i think only oslo is doing it correctly | 21:13 |
mordred | it's not a bug | 21:13 |
mordred | it's not a bug | 21:13 |
mordred | it's not a bug | 21:13 |
*** boden has quit IRC | 21:13 | |
mordred | it's a very conscious choice | 21:13 |
jogo | mordred: why? | 21:13 |
HenryG | How do I apply modules/openstack_project/manifests/single_use_slave.pp to machines for the check-neutron-dsvm-functional job? | 21:13 |
dhellmann | krotscheck: ah, ok, I misunderstood the question | 21:13 |
mordred | because it's not a thing we've ever wanted to do | 21:13 |
jogo | mordred: what was the conscious choice | 21:13 |
marun | mordred: I think it's interesting to hear you say that it isn't a use case that has been considered | 21:13 |
r1chardj0n3s | krotscheck: it has to be configured *somewhere* and it makes sense to me to not add a new configuration file just for CORS | 21:13 |
mordred | and in fact | 21:13 |
jogo | mordred: why? | 21:13 |
mordred | most of the time | 21:13 |
mordred | we want to NOT do it | 21:13 |
*** boden has joined #openstack-infra | 21:13 | |
marun | mordred: It's a huge waste of time for everyone | 21:13 |
mordred | because doing it is an optimization | 21:13 |
*** sarob_away is now known as sarob | 21:13 | |
dhellmann | krotscheck: maybe we should talk about that on the spec review? | 21:14 |
stevemar | fungi, a lot of projects are kicking off a project specific post_gate_hook, but it's running tests, which don't make sense... since devstack has been torn down | 21:14 |
clarkb | akerr: sorry too much stuff going on right now. have you compared it to the upstream pipeline? | 21:14 |
r1chardj0n3s | dhellmann, krotscheck: that makes sense, yes | 21:14 |
mordred | and so far, in the project, we have tended to solve optimizations not by running less tests | 21:14 |
mordred | but by increasing our ability to run tests | 21:14 |
mordred | so | 21:14 |
krotscheck | r1chardj0n3s, dhellmann: Indeed. Off to gerrit! | 21:14 |
mordred | in general | 21:14 |
*** boden has quit IRC | 21:14 | |
* krotscheck rides off into the code review system. | 21:14 | |
mordred | a feature to limit what tests we run as a way to scale the system | 21:14 |
marun | mordred: ...and we're pretty screwed as a resuult | 21:14 |
mordred | is a completely different desgin | 21:14 |
mordred | and one to not be taken lightly | 21:14 |
HenryG | Darn. I mean, How do I apply modules/openstack_project/manifests/slave_db.pp to machines for the check-neutron-dsvm-functional job? | 21:14 |
clarkb | how are we screwed? | 21:14 |
mordred | marun: no | 21:14 |
jogo | mordred: aren't we at the scale where its time to do optimizations | 21:14 |
marun | mordred: yes | 21:14 |
*** boden has joined #openstack-infra | 21:14 | |
mordred | we are screwed because people are writing shitty code | 21:14 |
fungi | stevemar: some of them want devstack to configure things but not leave all the services running, i think | 21:14 |
marun | mordred: pushing patches sucks so damn hard | 21:14 |
mordred | and the code IS actually broken | 21:14 |
akerr | clarkb: let me compare them, thanks for the response | 21:15 |
marun | mordred: having merge queues that are so damn long | 21:15 |
mordred | so people should stop wanking with features and actually fix the damned bugs | 21:15 |
clarkb | akerr: we do very similar in the upstream gate pipeline so it would be a good one to compare to | 21:15 |
marun | mordred: patch velocity only due to a lack of reviewer resources? | 21:15 |
*** boden has quit IRC | 21:15 | |
anteaya | HenryG: are you asking in the context of your third party ci environment? | 21:15 |
marun | mordred: No, as much a result of the impossible task of trying to ensure overall openstack quality on .every.single.project.merge.proposal | 21:15 |
*** boden has joined #openstack-infra | 21:15 | |
fungi | patch velocity limited due to developers playing with pet features rather than fixing what's already broken in their projects | 21:15 |
marun | fungi: sure | 21:16 |
*** boden has quit IRC | 21:16 | |
fungi | and also due to integrating projects which were not stable enough to actually be plugged into one another | 21:16 |
mordred | marun: and you think quality woudl be better by increasing the number of less well tested patches we can land? | 21:16 |
HenryG | anteaya: No. This is for the check-neutron-dsvm-functional community job. | 21:16 |
marun | fungi: and that is as much due to openstack not requiring good testing | 21:16 |
*** mjturek has quit IRC | 21:16 | |
mordred | marun: or just that it would be easier for individual devs to land a patch? | 21:16 |
marun | fungi: pretending we can bolt it on with tempest and the gate | 21:16 |
fungi | absolutely | 21:16 |
*** boden has joined #openstack-infra | 21:16 | |
mordred | ++ | 21:16 |
anteaya | HenryG: do you have a patch in place that I can look at? | 21:16 |
marun | mordred: I want to see dramatically increased requirements around in-tree project testing | 21:16 |
mordred | marun: I do too | 21:16 |
anteaya | HenryG: or in process? | 21:16 |
*** boden has quit IRC | 21:17 | |
openstackgerrit | A change was merged to openstack-infra/subunit2sql: Add manifest file to include base alembic config https://review.openstack.org/125151 | 21:17 |
fungi | i think it was probably tractable to test it that way a while back, but we scaled past it being effective more quickly than anyone had an opportunity to implement more focused testing solutions | 21:17 |
HenryG | anteaya: Not yet, I am trying to figure out how. Not very good at puppet so not sure where to start. | 21:17 |
mordred | marun: I'm pushing back angrily because it would be a LOT of work to implement a "only test this patch when X changes" feature, and I'm pretty sure the amount of gain we'd get is nothing | 21:17 |
*** boden has joined #openstack-infra | 21:17 | |
marun | mordred: I want it to be harder to merge patches that aren't tested. But I also hope that we can all recognize the futility of running so many integrated jobs in the hopes that it ensures quality. | 21:17 |
marun | mordred: It doesn't. | 21:17 |
mordred | compared to the potential for rampant abuse | 21:17 |
marun | mordred: It increases developer friction with no real benefit | 21:17 |
anteaya | HenryG: let's work on me understanding what you are doing, what are you trying to accomplish? | 21:17 |
*** boden has quit IRC | 21:17 | |
clarkb | marun: so of that statement I am not so sure | 21:17 |
anteaya | HenryG: keep in mind everyone else is engaged in another topic and I'm slow | 21:18 |
mordred | marun: I disagree with "no real benefit" | 21:18 |
marun | clarkb: we are as likely to surface a race condition after a merge as before | 21:18 |
*** boden has joined #openstack-infra | 21:18 | |
clarkb | marun: if we take neutron for example. Quality there is much better than it was for many reasons, including the extra integration testing we do against it | 21:18 |
jogo | lets say each component of our system works 99% of the time. If we have 10 pieces to the system we get a success rate for the collective system of 90% | 21:18 |
marun | clarkb: really? | 21:18 |
clarkb | marun: we run double the tests and thankfully the whole suite now | 21:18 |
HenryG | anteaya: Sure, the bug is https://bugs.launchpad.net/neutron/+bug/1372981 | 21:18 |
clarkb | marun: yes neutron is double tempested | 21:18 |
uvirtbot | Launchpad bug 1372981 in neutron "Neutron functional tests can't target both postgres and mysql in the same test run" [Medium,Confirmed] | 21:18 |
clarkb | marun: and it appears to have helped | 21:18 |
marun | clarkb: and that sucks balls | 21:18 |
clarkb | significantly | 21:18 |
*** boden has quit IRC | 21:18 | |
*** bhuvan has quit IRC | 21:18 | |
anteaya | HenryG: thanks, that is a start | 21:18 |
clarkb | because in the last 6 months neutron has done what it has been trying to do for 2.5-3 years | 21:19 |
marun | clarkb: we now have twice as many chances to surface unrelated race conditions that block us from merging | 21:19 |
*** boden has joined #openstack-infra | 21:19 | |
anteaya | yay!! | 21:19 |
mordred | marun: so what? | 21:19 |
mordred | marun: so don't merge | 21:19 |
* anteaya throws a party at what clarkb just said | 21:19 | |
fungi | marun: recall that not too long ago there was basically no integration testing of neutron at all. just a stub because nothing worked. the amount of testing it's under now compared to them is tremendous | 21:19 |
mordred | so let's all stop wroking on new features and new vendor plugins | 21:19 |
marun | clarkb: I have a pretty good chance of surfacing that kind of condition with a tox.ini change that has zero functional implications | 21:19 |
marun | this is progress? | 21:19 |
*** boden has quit IRC | 21:19 | |
mordred | and let's do nothing but work on race bugs | 21:19 |
marun | mordred: fsck the race bugs | 21:19 |
fungi | er, compared to then | 21:19 |
marun | that's the symptom | 21:19 |
HenryG | anteaya: One way to do it would be to patch devstack so it can install both DB backends. | 21:20 |
*** boden has joined #openstack-infra | 21:20 | |
jogo | mordred: if you can make that happen awesome | 21:20 |
marun | let's write in-tree functional and integration tests that validate functionality for that service alone | 21:20 |
mordred | marun: yes. let's | 21:20 |
openstackgerrit | Steve Martinelli proposed a change to openstack-infra/project-config: Change OSC functional tests to gate_hook https://review.openstack.org/125202 | 21:20 |
mordred | PLEASE | 21:20 |
*** boden has quit IRC | 21:20 | |
mordred | marun: but let's not turn off what little we have until we've actually done that | 21:20 |
marun | let's cover off exceptional cases _before_ we try to add all the complexity of other services | 21:20 |
HenryG | anteaya: But the unit-test jobs install both backends via the puppet script I referred to above. | 21:20 |
mordred | because what we have may suck | 21:20 |
stevemar | fungi, ^ | 21:20 |
mordred | but it's FAR better than nothing | 21:20 |
marun | mordred: let's recognize the tradeoff involved too, though | 21:20 |
HenryG | anteaya: So I would like to try the same for the functional job. | 21:21 |
*** boden has joined #openstack-infra | 21:21 | |
jogo | mordred: you should try writing a tempest patch | 21:21 |
marun | mordred: MOOR TESTS BETTER is broken | 21:21 |
anteaya | HenryG: it looks like from the bug report you would be best served by spending some time with mtreinish in -qa | 21:21 |
marun | let's recognize that and find a good balance | 21:21 |
mordred | marun: totally | 21:21 |
mtreinish | anteaya: what did I do? | 21:21 |
*** boden has quit IRC | 21:21 | |
mordred | marun: all I'm saying is that we should not remove what we have until we have something new and better | 21:21 |
anteaya | mtreinish: you are just awesome | 21:21 |
anteaya | mtreinish: keep doing that | 21:21 |
*** boden has joined #openstack-infra | 21:21 | |
mordred | marun: because our current experience is that less tests == immediate breakage fail pain | 21:21 |
marun | mordred: I'm saying we should take a hard look at the cost/benefit of what we have, and adjust | 21:21 |
stevemar | fungi, thank you sirrrr | 21:22 |
HenryG | mtreinish: It may be hard to read the backlog here :D | 21:22 |
mordred | and that has been consistent every time something has failed positive over the last 2 years | 21:22 |
*** boden has quit IRC | 21:22 | |
marun | mordred: if the cost of running a given job has a potential payoff of catching a race/regression, great. | 21:22 |
marun | mordred: run it | 21:22 |
*** IAX has joined #openstack-infra | 21:22 | |
*** akerr has left #openstack-infra | 21:22 | |
*** boden has joined #openstack-infra | 21:22 | |
mtreinish | anteaya: cool, wfm :) | 21:22 |
marun | mordred: if there is a low chance that job has value - do not run it. | 21:22 |
mtreinish | HenryG: heh yeah that's why I asked? | 21:22 |
mordred | marun: this happens | 21:22 |
marun | mordred: save the resources, and more importantly the developer time | 21:22 |
mordred | marun: consistently | 21:22 |
marun | mordred: I think we've conflated 2 things that should be independent | 21:23 |
*** boden has quit IRC | 21:23 | |
marun | mordred: One is keeping a given project tree stable | 21:23 |
fungi | replace the current poorly-targeted tests with more focused testing? good. drop the current tests in hopes that better testing emerges before everything falls apart? _bad_ | 21:23 |
marun | mordred: The other is keeping openstack as a whole stable | 21:23 |
marun | These are different things | 21:23 |
*** boden has joined #openstack-infra | 21:23 | |
mordred | marun: right now we do neither | 21:23 |
mordred | well | 21:23 |
*** boden has quit IRC | 21:24 | |
marun | mordred: I'm saying, we need to consider these goals separately if we have any hope of achieving them | 21:24 |
mordred | sure | 21:24 |
marun | we're doing both, badly, right now | 21:24 |
jogo | marun: ++ | 21:24 |
sdague | marun: so help | 21:24 |
*** boden has joined #openstack-infra | 21:24 | |
anteaya | boden check your client | 21:24 |
*** bhuvan has joined #openstack-infra | 21:24 | |
marun | sdague: that's always your refraing, | 21:24 |
mordred | this is why I agree that we need functional tests | 21:24 |
marun | sdague: refrain | 21:24 |
mordred | marun: but why I believe that we need them FIRST | 21:24 |
marun | sdague: Do you honestly think I'm not doing all I can to move this forward? | 21:24 |
*** boden has quit IRC | 21:24 | |
marun | sdague: I pushed for how many months to get agreement that we needed to move in a different direction | 21:25 |
marun | ? | 21:25 |
clarkb | mordred: how do I break inside of a dib build? | 21:25 |
*** boden has joined #openstack-infra | 21:25 | |
mordred | clarkb: no clue. SpamapS ^^ ? | 21:25 |
marun | sdague: I hope you can trust at some point that I am working on this. | 21:25 |
mordred | lifeless: ^^ ? | 21:25 |
*** homeless has quit IRC | 21:25 | |
mordred | marun: I know you're working on it - but please trust in the opposite direction too | 21:25 |
*** boden has quit IRC | 21:25 | |
mordred | marun: and know that when you start with "zuul is broken" the response might not be too immediately welcome | 21:26 |
clarkb | anteaya: can your propose the cahnge you suggested in 124254? ianw is out this week | 21:26 |
*** homeless has joined #openstack-infra | 21:26 | |
*** boden has joined #openstack-infra | 21:26 | |
anteaya | clarkb: /me looks | 21:26 |
marun | mordred: it's qualified | 21:26 |
*** boden has quit IRC | 21:26 | |
marun | mordred: if zuul cannot dynamically determine the jobs to run based on a given patch, then a bug exists that needs to be fixed | 21:26 |
*** boden has joined #openstack-infra | 21:27 | |
marun | mordred: I think you know as well as I that saying something dramatic is sometimes done for the attention. | 21:27 |
mordred | :) | 21:27 |
mordred | what | 21:27 |
mordred | I never do that | 21:27 |
*** tonytan4ever has quit IRC | 21:27 | |
marun | hah | 21:27 |
anteaya | clarkb: yep, going to go for a walk first then will propose | 21:27 |
clarkb | anteaya: ty | 21:27 |
*** boden has quit IRC | 21:27 | |
mordred | marun: thing is - as I mentioned above - that's a conscious design decision | 21:27 |
sdague | marun: so the impact of that statement is basically people that do try to help with these things, like myself, decide it's completely not worth it because there is so much stone throwing from the outside | 21:28 |
*** boden has joined #openstack-infra | 21:28 | |
mordred | marun: and although I do think that the conversation about whether or not it's a useful design or whether we need a new one is a good discussion | 21:28 |
sdague | so if that's your intent, you win | 21:28 |
*** boden has quit IRC | 21:28 | |
marun | Ok | 21:28 |
marun | Let's air some dirty laundry then | 21:28 |
*** bcourt has quit IRC | 21:28 | |
*** _buzz has joined #openstack-infra | 21:28 | |
marun | and sdague, you can call it 'stone-throwing' all you want, but I think honesty is important | 21:28 |
*** boden has joined #openstack-infra | 21:28 | |
clarkb | mordred: fungi: for some reason pupept isn't finding openstack_project::single_use_slave in the dib builds, but it should be in the modulepath. so next step for me is to set up a breakpoint and figure it out | 21:29 |
*** boden has quit IRC | 21:29 | |
marun | I think infra is overloaded to the point that developer productivity doesn't really rate in the priority list | 21:29 |
mordred | clarkb: oh wow. that's amazing | 21:29 |
mordred | marun: I disagree very strongly | 21:29 |
mordred | developer producitivy is super important | 21:29 |
*** boden has joined #openstack-infra | 21:29 | |
marun | mordred: how much code do you push? | 21:29 |
*** doude has quit IRC | 21:29 | |
mordred | marun: a GIANT amount | 21:29 |
fungi | i think it can be implemented if we really want it. but i also question the effective gain... how many changes touch only non-code? and are those even changes worthy of prioritizing if the code itself is so broken it's painful to test? | 21:29 |
marun | mordred: how many hours do you waste waiting on gate queues? | 21:30 |
marun | mordred: on rechecks? | 21:30 |
marun | mordred: Do you think this is acceptable? | 21:30 |
*** boden has quit IRC | 21:30 | |
mordred | marun: I don't wait on gate queues - I tend to have at least 100 different tasks in my queue at any point in time | 21:30 |
marun | mordred: Or that contributes to developer productivity? | 21:30 |
clarkb | mordred: fungi actually I think its a bug in the build image script. I think I can sort this | 21:30 |
fungi | same here. i parallelize my work and pick up whatever's ready for a next step as opportunity allows | 21:30 |
*** boden has joined #openstack-infra | 21:30 | |
marun | mordred: So you've optimized your workflow to account for how broken our infrastructure is | 21:30 |
marun | mordred: good for you | 21:30 |
*** dhellmann is now known as dhellmann_ | 21:30 | |
mordred | marun: wow | 21:30 |
mordred | no | 21:30 |
marun | mordred: not everyone has that luxury | 21:30 |
mordred | our infrastructure is NOT broken | 21:30 |
lifeless | clarkb: mordred: Read the manual | 21:30 |
marun | mordred: that's a statement of opinion | 21:31 |
mordred | it does exactly what we ask it to very well | 21:31 |
clarkb | lifeless: :P I figured that was coming | 21:31 |
mordred | marun: openstack is broken | 21:31 |
*** boden has quit IRC | 21:31 | |
mordred | our infrastructure shows that quite regularly | 21:31 |
marun | mordred: it is broken from the perspective of developer productivity | 21:31 |
lifeless | clarkb: search for debug in README | 21:31 |
marun | mordred: we've optimized to try to keep openstack stable | 21:31 |
mordred | then I disagree with what you find productive | 21:31 |
*** boden has joined #openstack-infra | 21:31 | |
mordred | because "landing patches" is not necessarily productive | 21:31 |
fungi | our infrastructure will actually run jobs which are sanely written extremely well. the projects the code is coming from are full of bugs we're successfully identifying | 21:31 |
mordred | "fixing openstack" is | 21:31 |
marun | mordred: to the detriment of developer productivity | 21:31 |
mordred | nope | 21:31 |
clarkb | lifeless: ty | 21:31 |
mordred | not productive | 21:31 |
marun | mordred: there is no open discussion of the tradeoffs involved | 21:31 |
lifeless | clarkb: or break should hit it too :) | 21:31 |
*** boden has quit IRC | 21:32 | |
mordred | you're assuming that mean time to land patch should be minimized | 21:32 |
marun | mordred: so zuul is DESIGNED to avoid dynamic discovery of the jobs to run | 21:32 |
mordred | I disagree | 21:32 |
*** VijayTripathi has quit IRC | 21:32 | |
*** boden has joined #openstack-infra | 21:32 | |
*** e0ne has quit IRC | 21:32 | |
mordred | marun: zuul is designed to solve the problems we actually have, which is keeping up with the aggregate project velocity | 21:32 |
marun | mordred: and that indicates to me - and nothing you're saying is contradicting this view - that developer productivity is not considered important | 21:32 |
mordred | which is MASSIVE | 21:32 |
fungi | if developers on some projects would rather stick their collective heads in the sand and pretend it's okay to be completely broken, then i question the reason for even testing those projects (or including them in openstack) | 21:32 |
mordred | marun: I still disagree with yoru definition of productive | 21:32 |
*** boden has quit IRC | 21:32 | |
krotscheck | Random fun thing: When clicking on gate-storyboard-webclient-js-draft in gerrit you now get a fully functional client. (caveat - small bug in login that I’m fixing) | 21:32 |
mordred | marun: when the project in question is not broken, things run well | 21:33 |
marun | mordred: do you honestly think that your opinion is canonical here? | 21:33 |
krotscheck | Example: https://review.openstack.org/#/c/104004/ | 21:33 |
mordred | it's not my job to fix all the problem | 21:33 |
mordred | marun: no, I do not | 21:33 |
*** boden has joined #openstack-infra | 21:33 | |
marun | mordred: that I'm a lone voice, and that I'm uninformed? | 21:33 |
marun | I get that we need to converge on stability | 21:33 |
mordred | marun: I think taht you are representing a well known viewpoint that it's currently hard for developers to land a patch | 21:33 |
*** boden has quit IRC | 21:33 | |
marun | and that we are hamstrung in the absence of effective project-specific functional testing | 21:33 |
mordred | I agree with that | 21:33 |
* krotscheck pokes himself to speed things up a bit.... | 21:34 | |
*** bcourt has joined #openstack-infra | 21:34 | |
*** boden has joined #openstack-infra | 21:34 | |
mordred | we need much more of that and we need it to be much better | 21:34 |
fungi | also, zuul does make dynamic determination on what jobs to run and when. very dynamic decisions. it just doesn't have the necessary dynamism right now to vault your "fix all teh typoz" change through without running unit tests | 21:34 |
marun | But nowhere do I hear any suggestion that we can tune the amount of integration testing to the benefit derived | 21:34 |
*** e0ne has joined #openstack-infra | 21:34 | |
mordred | but you just described a failure in the projects, not in the automation of them | 21:34 |
*** boden has quit IRC | 21:34 | |
mordred | it is completely in the power of the projects to add good functional tests | 21:34 |
marun | our current infra is an accident | 21:34 |
*** boden has joined #openstack-infra | 21:35 | |
mordred | no. it is most certainly not | 21:35 |
mordred | and it is completely in our power to consistenlty run those tests | 21:35 |
marun | more than 2 years ago everyone realized that openstack was broken | 21:35 |
jogo | at the current scale of the integrated gate, I don't think we can ever get to the point where dsvm-tempest works 100% of the time | 21:35 |
*** boden has quit IRC | 21:35 | |
marun | gating began, first on devstack exercises | 21:35 |
mordred | nope | 21:35 |
marun | rather than fixing the problem - the projects were poorly tested | 21:35 |
mordred | that is factually inaccrate | 21:35 |
marun | we tried to bolt on quality | 21:35 |
marun | that much is _not_ factually inaccurate | 21:35 |
*** boden has joined #openstack-infra | 21:35 | |
mordred | the project started with gating from literally commit one | 21:35 |
marun | we've tried to bolt on quality for 2+ years | 21:36 |
mordred | and has always had it | 21:36 |
marun | and it doesn't work | 21:36 |
clarkb | anteaya: note 124254 needs an update too, I will push that as soon as I can test it | 21:36 |
marun | we get that now | 21:36 |
mordred | so we've been gating on tests for 4 years | 21:36 |
*** boden has quit IRC | 21:36 | |
marun | mordred: I play fast and loose with my dates because I've forgotten how long I've been here I guess. | 21:36 |
*** boden has joined #openstack-infra | 21:36 | |
mordred | marun: you are describing the advent of devstack | 21:36 |
mordred | before that, openstack was uninstallable | 21:37 |
lifeless | still is:P | 21:37 |
marun | lifeless++ | 21:37 |
fungi | zing! | 21:37 |
*** boden has quit IRC | 21:37 | |
mordred | and the devs would occasionally say "look, here is some puppet" | 21:37 |
mordred | but it wouldn't work | 21:37 |
lifeless | still do :P | 21:37 |
* lifeless isn't being helpful | 21:37 | |
* lifeless goes | 21:37 | |
*** boden has joined #openstack-infra | 21:37 | |
*** boden has quit IRC | 21:38 | |
*** e0ne has quit IRC | 21:38 | |
*** boden has joined #openstack-infra | 21:38 | |
mordred | marun: listen, we're in the weeds - I VERY MUCH agree and support the idea of much better targetted functional testing | 21:39 |
*** boden has quit IRC | 21:39 | |
mordred | marun: I think we'll die without it | 21:39 |
mordred | marun: and we probably should have had it quite a while ago | 21:39 |
*** boden has joined #openstack-infra | 21:39 | |
*** andreaf has quit IRC | 21:39 | |
clarkb | anteaya: so lets hold off on that second change for now | 21:39 |
*** boden has quit IRC | 21:39 | |
mwhudson | oh yes | 21:40 |
*** andreaf has joined #openstack-infra | 21:40 | |
fungi | (and for that matter, most projects could use some unit tests which are actually unit tests) | 21:40 |
*** boden has joined #openstack-infra | 21:40 | |
mwhudson | so last week i came in here and asked "how do i run tempest like gate does" | 21:40 |
mwhudson | and apparently this week, the answer is different? :) | 21:40 |
clarkb | mwhudson: no? | 21:40 |
*** boden has quit IRC | 21:40 | |
marun | mordred: I think we agree on a lot of this stuff. | 21:40 |
mwhudson | clarkb: something about moving to dib? | 21:40 |
*** boden has joined #openstack-infra | 21:41 | |
clarkb | mwhudson: yes, and we have been planning the move for at least half a year. But it isn't quite there yet and the old way continues to work | 21:41 |
fungi | mwhudson: our providers finally started running openstack! (well, started providing glance) | 21:41 |
marun | mordred: I think there are 2 areas we can improve in advance of our future functional test utopia. | 21:41 |
*** boden has quit IRC | 21:41 | |
marun | mordred: The first one is to allow dynamic determination of jobs to run. | 21:42 |
*** boden has joined #openstack-infra | 21:42 | |
mwhudson | clarkb: ah ok | 21:42 |
fungi | mwhudson: the current mechanism nodepool uses was, as much as anything, invented to work around teh fact that we couldn't upload our own server images into cloud providers | 21:42 |
*** boden has quit IRC | 21:42 | |
marun | mordred: The cost of running unnecessary integration jobs (i.e. on test-only changes) is not justified. | 21:42 |
mwhudson | fungi: ah | 21:42 |
*** boden has joined #openstack-infra | 21:42 | |
*** andreykurilin_ has quit IRC | 21:42 | |
mordred | marun: I understand and respect that position. I disagree with your premise that dynamic determination of jobs to run will help things for developers | 21:43 |
fungi | mwhudson: now that we can, that opens us up to be able to build an image locally and then push it to all providers/regions, so massive performance gain | 21:43 |
*** eharney has quit IRC | 21:43 | |
mordred | marun: I actually believe that it will make their experience worse | 21:43 |
marun | mordred: Why? | 21:43 |
mwhudson | fungi: are you building such images yet? | 21:43 |
mordred | marun: beacuse we do not have the functional tests yet | 21:43 |
*** boden has quit IRC | 21:43 | |
*** pcm_ has quit IRC | 21:43 | |
fungi | mwhudson: and will also make it possible for other parties (devs, third-party testing operators, et cetera) to consume/reuse the images we're building nightly | 21:43 |
marun | mordred: Hold on, we're not talking about the same thing. | 21:43 |
mordred | marun: so my confidence that targetted running will not lead to an increase in race bugs is low | 21:43 |
*** boden has joined #openstack-infra | 21:43 | |
mwhudson | fungi: right | 21:43 |
marun | mordred: This has zero to do with race bugs. | 21:43 |
mordred | marun: ah! excellent! I love it when that happens | 21:43 |
mordred | marun: when you say targetted running, what do you mean? | 21:44 |
*** boden has quit IRC | 21:44 | |
fungi | mwhudson: we are building images now, yes, and have nodes which can boot from them and run experimental jobs successfully | 21:44 |
mwhudson | fungi: well ultimately i care about 3rd party testing on arm64 so i guess i'll have to make my own images | 21:44 |
mwhudson | but that's ok | 21:44 |
marun | mordred: If I am making a change to the test tree (verifiably non-functional), the integration jobs should not run. | 21:44 |
*** boden has joined #openstack-infra | 21:44 | |
*** hashar has quit IRC | 21:44 | |
fungi | mwhudson: right, there may still be gains there as well, though we haven't really pondered how to do cross-architecture nodepool... however if you have an arm-based nodepool server | 21:44 |
clarkb | marun: how do you verify that though? | 21:45 |
clarkb | marun: today the answer is your run the tests | 21:45 |
*** boden has quit IRC | 21:45 | |
mwhudson | i'd also like to just run the tests myself on intel and see no failures, because i haven't managed that yet | 21:45 |
clarkb | marun: asserting in zuul config that it won't isn't quite the same | 21:45 |
marun | clarkb: look at the patch, if the changes are only to neutron/tests, don't run integration jobs | 21:45 |
*** boden has joined #openstack-infra | 21:45 | |
mordred | marun: right. but the generalized feature you're describing is "run tests less" | 21:45 |
marun | clarkb: it has to be dynamic. I know this isn't supported yet. | 21:45 |
clarkb | marun: so my argument earlier with jogo is that that is completely insufficient | 21:45 |
*** markmcclain has quit IRC | 21:45 | |
marun | mordred: no | 21:45 |
marun | mordred: it's being smart about what we run | 21:45 |
clarkb | at least with the assumptions we have today about our tests | 21:46 |
mordred | marun: in order to smartly run less tets | 21:46 |
marun | mordred: you're hearing 'run tests less' betrays your bias towards 'all test runs have value' | 21:46 |
clarkb | they are part of consumable code | 21:46 |
*** boden has quit IRC | 21:46 | |
marun | and they dont | 21:46 |
mordred | marun: no, I do _not_ have that bias | 21:46 |
clarkb | and it is completely fair game for nova to import its tests or tempest to import nova's tests | 21:46 |
marun | clarkb: Uh, what? | 21:46 |
*** boden has joined #openstack-infra | 21:46 | |
fungi | marun: so... you say this isn't about race conditions... yet... why do you care if the infra systems run a job guaranteed to pass because you're not modifying files affecting it? | 21:46 |
clarkb | marun: its python... you can do this. there are reasons to do this | 21:46 |
marun | clarkb: Why in _hell_ would that be allowable? | 21:46 |
clarkb | marun: because there may be useful code in one project's tests that another wants to consume | 21:46 |
*** boden has quit IRC | 21:47 | |
marun | clarkb: the code in question that does that s*** should break. | 21:47 |
clarkb | see hacking | 21:47 |
lifeless | ironic needed to do that for instance | 21:47 |
mordred | marun: maybe you're ceilometer and you want to check that doing your thing doesn't cause nova to regress | 21:47 |
lifeless | with the nova driver | 21:47 |
*** boden has joined #openstack-infra | 21:47 | |
mordred | marun: maybe you're ironic and need to check that your nova driver didn't break the existing nova unittests | 21:47 |
*** bswartz has quit IRC | 21:47 | |
marun | mordred: ffs | 21:47 |
marun | that's not a public api | 21:47 |
mordred | marun: and that's a lazy developer sentiment | 21:47 |
marun | if someone is using something that isn't for public consumption, fsck them! | 21:47 |
mordred | like the plugin interface? | 21:47 |
lifeless | so the public aspect has merit as an argument | 21:47 |
*** boden has quit IRC | 21:47 | |
marun | we define api's for a reason | 21:48 |
mordred | like the plugin interface? | 21:48 |
fungi | good thing most of our projects aren't actually for public consumption, as it turns out | 21:48 |
marun | if someone uses something private - buyer beware | 21:48 |
lifeless | but right now the test suites are not annotated as private | 21:48 |
r1chardj0n3s | thanks for the CORS spec review krotscheck, and your configuration point is valid. One way to solve the problem would be to require API authors to include the CORS middleware and configure it (apart from the allowed origins, of course). But that's a big mess... | 21:48 |
mordred | like the plugin interface? | 21:48 |
marun | lifeless: so let's change that | 21:48 |
*** boden has joined #openstack-infra | 21:48 | |
jogo | clarkb: no it is not fair game for tempest to import nova's unit tests unless we agree on a stable API to dotaht | 21:48 |
*** dustins has quit IRC | 21:48 | |
marun | mordred: we need to stabilize a public api | 21:48 |
marun | mordred: that's a battle we'll be fighting in neutron in kilo | 21:48 |
*** boden has quit IRC | 21:48 | |
marun | mordred: tests are arguably different, though | 21:48 |
mordred | marun: we need to admit that it's possible that anything in any of our projects has a contract in any way shape or form | 21:48 |
marun | mordred: we get to decide that, anyway | 21:48 |
*** boden has joined #openstack-infra | 21:49 | |
jogo | clarkb: hacking is not an example that helps your point as hacking has *no* co-gating | 21:49 |
marun | mordred: allowing something, because, that costs us, is the very definition of stupid | 21:49 |
krotscheck | r1chardj0n3s: Is that really a big mess? | 21:49 |
clarkb | jogo: but hackign does this with pep8 | 21:49 |
jogo | clarkb: link? | 21:49 |
clarkb | mordred: ^ did it not and didn't you yell at pep8 for breaking this? | 21:49 |
mordred | ok - two things | 21:49 |
r1chardj0n3s | krotscheck: it would require getting *all* the projects to agree to having the middleware included, which I think could be troublesome :) | 21:49 |
*** boden has quit IRC | 21:49 | |
mordred | A) I do not believe unit tests are a public API | 21:49 |
mordred | and I am not going to pretend that I do | 21:49 |
*** boden has joined #openstack-infra | 21:50 | |
jogo | clarkb: flake8 had some issues but they weren't hacking related and no co-gating was needed | 21:50 |
clarkb | jogo: you can't cogate with them | 21:50 |
*** flaviof_zzz is now known as flaviof | 21:50 | |
mordred | B) I argue that unit tests should ship because sometimes someone MIGHT want to do what marun mentioned is a buyer beware activity and reuse them | 21:50 |
clarkb | jogo: but you do consume their tests as a reconsumable thing | 21:50 |
*** boden has quit IRC | 21:50 | |
mordred | I do not think that we need to be concerned about that activity | 21:50 |
r1chardj0n3s | krotscheck: having said that, it would be much nicer than requiring deployers to do it for every api | 21:50 |
marun | mordred: in _production_ code? | 21:50 |
mordred | other than allowing someone to make that decision if it makes sense for them | 21:50 |
jogo | clarkb: oh that was something mordred did I think | 21:50 |
marun | mordred: wth | 21:50 |
mordred | marun: I'm talking about test suites | 21:50 |
jogo | clarkb: I have to double check if we still do that | 21:50 |
*** boden has joined #openstack-infra | 21:50 | |
*** mriedem has quit IRC | 21:50 | |
marun | mordred: if a given project is going to pick things up, the responsibility is for them to find the problem | 21:51 |
mordred | and devs who may want to do something with them in their own test suites | 21:51 |
mordred | yes | 21:51 |
mordred | I agree | 21:51 |
mordred | yikes | 21:51 |
mordred | clarkb is representing me as beieving something I do not beieve | 21:51 |
*** boden has quit IRC | 21:51 | |
mordred | believe | 21:51 |
clarkb | mordred: sorry. However I think this is a perfectly valid use case | 21:51 |
marun | clarkb: really | 21:51 |
marun | ? | 21:51 |
*** boden has joined #openstack-infra | 21:51 | |
clarkb | mordred: particularly since the tests must be internally consistent | 21:51 |
marun | clarkb: why in heck? | 21:51 |
clarkb | marun: because of the hacking thing | 21:51 |
clarkb | and the ironic thing | 21:52 |
mordred | I am now going to walk away from the laptop | 21:52 |
clarkb | your tests are reconsumable | 21:52 |
clarkb | unless you say don't reconsume this in a pythonic way | 21:52 |
jogo | clarkb: so the hacking thing. I think i know what it is now | 21:52 |
marun | clarkb: that makes absolutely zero sense. I challenge you to find any developer on an openstack project that supports that position | 21:52 |
*** boden has quit IRC | 21:52 | |
clarkb | marun: I think the ironic devs might | 21:52 |
marun | clarkb: and once you've identified them, I'll personally tar and feather them! | 21:52 |
jogo | clarkb: mordred wanted to use some code from pep8 that wasn't shipped since it was in there unit tests | 21:52 |
clarkb | because nova has burned them so hard | 21:52 |
*** boden has joined #openstack-infra | 21:52 | |
marun | clarkb: isn't that there problem? they do something dumb, they get burned. | 21:52 |
clarkb | marun: no, it isn't | 21:52 |
marun | clarkb: changing the rules so they don't get burned, not a great tradeoff | 21:53 |
clarkb | nova didn't say this is a private interface | 21:53 |
clarkb | import nova.tests | 21:53 |
clarkb | its a thing | 21:53 |
*** boden has quit IRC | 21:53 | |
clarkb | they are not marked private | 21:53 |
jogo | clarkb: everything in nova is private unless we say so | 21:53 |
marun | clarkb: it's common-fing sense, sorry | 21:53 |
clarkb | if you want to assert this stance then do it properly | 21:53 |
*** r1chardj0n3s is now known as r1chardj0n3s_afk | 21:53 | |
clarkb | jogo: no no no no | 21:53 |
*** boden has joined #openstack-infra | 21:53 | |
clarkb | jogo: that is really bad python then | 21:53 |
jogo | clarkb: private as in private API | 21:53 |
jogo | as in not stable | 21:53 |
*** boden has quit IRC | 21:54 | |
jogo | clarkb: go to the nova room and ask there | 21:54 |
marun | clarkb: on what planet does a project guarantee to clients that their _test suite_ is consumable and won't change in backwards-incompatible ways | 21:54 |
*** boden has joined #openstack-infra | 21:54 | |
jogo | or I can just get dansmith in here | 21:54 |
jogo | dansmith: ^ | 21:54 |
clarkb | jogo: marun I understand what you are saying.. | 21:54 |
clarkb | what I am saying is that you should be a bit more explicit about it if that is the stance | 21:54 |
*** zns has quit IRC | 21:54 | |
marun | clarkb: I consume tempest in the neutron poc, but I know they are going to break me at some point. | 21:54 |
*** boden has quit IRC | 21:54 | |
clarkb | from python's perspective you have done nothing to say don't do this | 21:54 |
dansmith | um, what? | 21:55 |
marun | python lets you do anything | 21:55 |
*** boden has joined #openstack-infra | 21:55 | |
jogo | dansmith: re: unit tests and stable whatnots | 21:55 |
*** bradjones has quit IRC | 21:55 | |
jogo | in backscroll | 21:55 |
*** sarob is now known as sarob_away | 21:55 | |
fungi | python also lets you identify private interfaces, and what isn't private is treated (by python) as public | 21:55 |
marun | clarkb: we rely on convention and commonsense to limit what we can do to a sane subset of what's possible | 21:55 |
marun | clarkb: it's the only way to work productively in a group | 21:55 |
*** boden has quit IRC | 21:55 | |
lifeless | I am quite sure the developer population consider the tests private | 21:55 |
lifeless | but as already said and agreed on they aren't annotated explicitly that way | 21:56 |
*** boden has joined #openstack-infra | 21:56 | |
marun | (because we're not using c++) | 21:56 |
clarkb | marun: you can move tests to the top level dir | 21:56 |
marun | I mean, is the argument that the ironic guys were in the right by trying to reuse nova's test suite and not expecting to be broken? | 21:56 |
*** boden has quit IRC | 21:56 | |
marun | Really? | 21:56 |
dansmith | the question is whether or not nova.tests.* is a stable interface? | 21:56 |
fungi | and there is some questionably legitimate use case when a project which is a plugin to another project wants to unit test the parent project with its code in place, that it may want to reuse those tests | 21:56 |
clarkb | dansmith: no | 21:57 |
*** boden has joined #openstack-infra | 21:57 | |
clarkb | dansmith: the question is should they be reconsumable in a sane way | 21:57 |
marun | And that it sets a precedent that we have to run the integrated jobs on every test change in case this happens again??? | 21:57 |
clarkb | dansmith: that may mean some bits are private others are stable | 21:57 |
clarkb | and not just nova | 21:57 |
*** boden has quit IRC | 21:57 | |
dansmith | clarkb: just to be clear, you mean someone other than nova importing nova.tests.something and using it? | 21:57 |
clarkb | dansmith: yes, or nova itself within nova | 21:57 |
*** boden has joined #openstack-infra | 21:58 | |
jogo | clarkb: but if it doesn't have a '_' in the method name then it should be stable and public? | 21:58 |
dansmith | well, those are two totally different things, right? | 21:58 |
clarkb | dansmith: not from the integration testing perspective | 21:58 |
*** boden has quit IRC | 21:58 | |
dansmith | I mean, nothing in nova imports anything from nova.tests, other than test code | 21:58 |
clarkb | dansmith: if nova imports nova.tests you still want to test it in an integration scenario to make sure that public nova behavior isn't broken. Similar to something else importing it | 21:58 |
clarkb | dansmith: sure, but it can... | 21:58 |
*** boden has joined #openstack-infra | 21:58 | |
dansmith | clarkb: it better not | 21:58 |
clarkb | there is nothing preventing it | 21:58 |
clarkb | is what I am saying. we have presented the tests in a reconsumable format | 21:59 |
jogo | dansmith: we can add a hacking rule to enforce that | 21:59 |
dansmith | jogo: yeah | 21:59 |
marun | clarkb: and for that reason we need to run integrated jobs on every commit? | 21:59 |
* jogo does that | 21:59 | |
*** boden has quit IRC | 21:59 | |
*** jaypipes has quit IRC | 21:59 | |
clarkb | marun: yes it is one way of catching it | 21:59 |
dansmith | clarkb: I don't think I agree with the assertion that just because it's a module it's intended to be consumable | 21:59 |
marun | clarkb: are you considering the cost of that? | 21:59 |
*** boden has joined #openstack-infra | 21:59 | |
*** sarob_away is now known as sarob | 21:59 | |
clarkb | marun: we do it today | 21:59 |
marun | clarkb: and it's f'ing stupid | 21:59 |
clarkb | marun: so yes, and teh cost isn't that insane | 21:59 |
marun | clarkb: for you maybe | 21:59 |
clarkb | we still merge a ton of code | 21:59 |
*** boden has quit IRC | 22:00 | |
clarkb | fungi: mordred I have sorted precise image builds | 22:00 |
marun | clarkb: you are not feeling the pain, clearly | 22:00 |
fungi | i think it was just being used as an example of "you can't assume a change to unit test code won't impact other code sharing the same git repository, or other projects with which that software interacts" | 22:00 |
clarkb | it is a bug in ianw's 124254 patch. pushing an update | 22:00 |
*** boden has joined #openstack-infra | 22:00 | |
marun | fungi: and it's a stupid example | 22:00 |
marun | fungi: we avoid those problems by forbidding people from reusing the damn test suite | 22:00 |
marun | fungi: a small price to pay for dramatically less gate cost / patch | 22:01 |
*** boden has quit IRC | 22:01 | |
marun | fungi: that's what I mean by tradeoffs | 22:01 |
clarkb | marun: then please forbid them from doing that | 22:01 |
marun | assuming that something is good without considering the costs -> not sustainable | 22:01 |
marun | clarkb: gladly! | 22:01 |
clarkb | marun: but the current stance has been to not do that | 22:01 |
marun | clarkb: so let's agree to that | 22:01 |
clarkb | and sorry if I misunderstood mordred on that front | 22:01 |
*** boden has joined #openstack-infra | 22:01 | |
dansmith | clarkb: what do you want us to do? yell at people? or put some technical block in place? | 22:01 |
marun | clarkb: I will make sure that nobody reuses the tests outside of the test tree | 22:01 |
clarkb | dansmith: a technical block | 22:01 |
*** boden has quit IRC | 22:01 | |
fungi | i would instead argue at a higher level that the perceived benefits from being able to avoid running integration test jobs for a project on changes to the fraction of its code which is not run-time functional is minimal | 22:01 |
clarkb | dansmith: all you have to do is move tests to top level dir | 22:01 |
clarkb | dansmith: but mordred is opposed to that | 22:02 |
marun | clarkb: And we can ensure update zuul to support dynamic job discovery so we don't run integrated jobs unnecessarily. | 22:02 |
*** boden has joined #openstack-infra | 22:02 | |
marun | fungi: and you're wrong | 22:02 |
*** ociuhandu has joined #openstack-infra | 22:02 | |
dansmith | is there a real problem here, or are we just arguing? | 22:02 |
lifeless | clarkb: no please don't suggeset top level tests dirs | 22:02 |
marun | fungi: you're not trying to merge those patches | 22:02 |
*** changbl has quit IRC | 22:02 | |
marun | fungi: I am | 22:02 |
clarkb | lifeless: ok | 22:02 |
dansmith | sorry if I missed it earlier in the scrollback | 22:02 |
lifeless | clarkb: that messes with testr and global unique test ids | 22:02 |
*** boden has quit IRC | 22:02 | |
marun | fungi: The cost is real to me and everyone else trying to revise the massively terrible neutron test suite | 22:02 |
clarkb | lifeless: can you or mordred explain where not top level dirs is useful outside of reconsumable tests | 22:02 |
lifeless | clarkb: we just went through the massive exercise to fix that most everwhere. | 22:02 |
lifeless | clarkb: reporting on aggregated tests | 22:03 |
clarkb | lifeless: I thoguht we were fixing it for the hacking reasons | 22:03 |
*** boden has joined #openstack-infra | 22:03 | |
fungi | at best, it will encourage more/faster contributions from people writing in-tree (but not in-line) documentation, or writing unit tests | 22:03 |
marun | fungi: good! | 22:03 |
lifeless | clarkb: I don't know all the reasons. I know the reason I care about :) | 22:03 |
clarkb | lifeless: rgr | 22:03 |
marun | fungi: easier/faster test contribution? | 22:03 |
marun | fungi: where is the problem? | 22:03 |
marun | fungi: we still have reviewer oversight. the tests still run | 22:03 |
*** boden has quit IRC | 22:03 | |
*** bhuvan has quit IRC | 22:03 | |
*** weshay has quit IRC | 22:03 | |
clarkb | marun we have merged 17.5k commits to openstack since october of last year | 22:03 |
marun | fungi: there is a real benefit here, whether you are personally affected or not | 22:03 |
marun | clarkb: at what cost? | 22:04 |
*** boden has joined #openstack-infra | 22:04 | |
clarkb | marun: I am not sure what you are using "cost" to measure here | 22:04 |
marun | clarkb: you can pull numbers all you want, but merging code in openstack SUCKS | 22:04 |
clarkb | marun: but I thought it was that code cannot merge | 22:04 |
marun | clarkb: developer productivity | 22:04 |
*** boden has quit IRC | 22:04 | |
clarkb | marun: merging 17.5k commits in one year is not productive? | 22:04 |
fungi | marun: i thought the current pain point was that the projects were broken and needed more people fixing those bugs in them? we'd effectively go to (rather a lot of) effort to make things easier for devs who aren't contributing to fixing that pain | 22:04 |
marun | clarkb: being roadblocked repeatedly by long gate queues | 22:04 |
*** boden has joined #openstack-infra | 22:04 | |
marun | fungi: we can't fix bugs particularly fast if a) we have limited gate resources and b) those gate resources are not being applied as effectively as they could be | 22:05 |
marun | fungi: this is part of the problem, too | 22:05 |
marun | it's multifaceted | 22:05 |
*** boden has quit IRC | 22:05 | |
*** boden has joined #openstack-infra | 22:05 | |
fungi | marun: yes, there are at least 10x as many facets to the problem as you're seeing from your perspective too, and we are already up to our eyeballs trying to solve them | 22:05 |
*** salv-orlando has joined #openstack-infra | 22:06 | |
*** boden has quit IRC | 22:06 | |
openstackgerrit | Clark Boylan proposed a change to openstack-infra/project-config: Port build-image.sh from old config repo https://review.openstack.org/124254 | 22:06 |
*** bhuvan has joined #openstack-infra | 22:06 | |
clarkb | anteaya: ^ is the fix that 124254 needed. you should be good to remove those files from the other repo | 22:06 |
fungi | so forgive me if i don't see avoiding running unit tests on non-code changes as a particularly high priority feature | 22:06 |
*** boden has joined #openstack-infra | 22:06 | |
marun | fungi: So you think it's ok to ignore my complaints because *you* don't think they are important. | 22:07 |
marun | great | 22:07 |
fungi | i don't disagree that there could be some gain | 22:07 |
stevemar | can I get one more review on https://review.openstack.org/#/c/125202/ i'm eager to get some test code in | 22:07 |
* marun gives up | 22:07 | |
*** boden has quit IRC | 22:07 | |
*** boden has joined #openstack-infra | 22:07 | |
fungi | marun: i will personally attempt to prioritize it appropriately along with everything else which i also know could possibly help | 22:07 |
marun | fungi: I'm not asking for you to do work | 22:08 |
*** boden has quit IRC | 22:08 | |
fungi | and alongside the higher priority things which i know are currently broken | 22:08 |
*** boden has joined #openstack-infra | 22:08 | |
marun | fungi: I'm asking for some appreciation that developer productivity matters and the impact of the current gate structure may not be the best tradeoff for openstack. | 22:08 |
*** bknudson has quit IRC | 22:08 | |
marun | fungi: where we go from there is entirely a separate issue | 22:08 |
*** boden has quit IRC | 22:09 | |
*** krtaylor has joined #openstack-infra | 22:09 | |
*** boden has joined #openstack-infra | 22:09 | |
fungi | marun: if you think for even a moment that we, as a group, are not taking the efficiency of our community development as the absolute top priority, we may as well all give up and leave | 22:09 |
fungi | because honestly, we're all failing if that's the case | 22:09 |
*** mbacchi has quit IRC | 22:09 | |
*** boden has quit IRC | 22:09 | |
marun | fungi: I think it's on both sides | 22:10 |
*** boden has joined #openstack-infra | 22:10 | |
marun | fungi: I recognize that infra is working impossibly hard to do an impossible job | 22:10 |
fungi | i can't actually remember one thing i've worked on which isn't specifically focused on smoothing and improving throughput for the development of openstack | 22:10 |
marun | fungi: I do think, though, that there is an insufficient appreciation for developer concerns. | 22:10 |
clarkb | fungi: re https://review.openstack.org/#/c/125186/ any opposition to self approving? was hoping mordred would weigh in but its a simple change | 22:10 |
*** boden has quit IRC | 22:10 | |
*** mattoliverau has joined #openstack-infra | 22:10 | |
fungi | marun: i too am a developer | 22:11 |
marun | fungi: And talking about this stuff here is an attempt to chart a different course in the future. | 22:11 |
*** boden has joined #openstack-infra | 22:11 | |
marun | fungi: let's be clear | 22:11 |
marun | fungi: do you work on critical-path features on a core openstack project? | 22:11 |
*** boden has quit IRC | 22:11 | |
*** david-lyle_ has joined #openstack-infra | 22:11 | |
fungi | marun: aside from fixing bugs in them sometimes, no | 22:11 |
*** david-lyle has quit IRC | 22:11 | |
*** boden has joined #openstack-infra | 22:12 | |
dstufft | fungi: wrt to your post about getting rid of external things in openstack - https://www.reddit.com/r/Python/comments/2hvi5m/looking_for_the_cdecimal_module_and_bytereeforg/ (cdecimal author is one of the main people against mandating things uploaded to PyPI) | 22:12 |
dstufft | :|| | 22:12 |
marun | fungi: I'm not questioning your technical capabilities. I'm saying the pain I'm seeing people facing in projects like neutron is crazy. | 22:12 |
fungi | and assuming you don't consider security support critical-path | 22:12 |
*** boden has quit IRC | 22:12 | |
*** david-lyle has joined #openstack-infra | 22:12 | |
*** mattoliverau has quit IRC | 22:12 | |
*** boden has joined #openstack-infra | 22:12 | |
marun | fungi: that is absolutely critical-path | 22:12 |
*** thedodd has quit IRC | 22:13 | |
fungi | and trust me, when we've got embargoes security fixes with advisory publication deadlines and can't get them merged because of bugs in openstack and poorly written tests, i'm very much aware of that pain | 22:13 |
marun | fungi: but presumably you can fast-track stuff that is so important | 22:13 |
marun | fungi: but for those whose constant pain is trying to get things through the gate? | 22:13 |
*** boden has quit IRC | 22:13 | |
marun | fungi: something has to give | 22:13 |
fungi | only to a point. we still won't merge security fixes if we can't get them to pass tests | 22:13 |
marun | fungi: I want things to be better. I want to help. | 22:13 |
*** boden has joined #openstack-infra | 22:13 | |
*** mattoliverau has joined #openstack-infra | 22:13 | |
*** boden has quit IRC | 22:14 | |
fungi | me too | 22:14 |
*** david_lyle__ has joined #openstack-infra | 22:14 | |
openstackgerrit | A change was merged to openstack-infra/nodepool: Properly reference nodepool config in image update https://review.openstack.org/125186 | 22:14 |
mordred | me too - even when I get unreasonably ragey | 22:14 |
*** boden has joined #openstack-infra | 22:14 | |
clarkb | fungi: thank you for approving | 22:14 |
fungi | dstufft: too fun | 22:15 |
*** boden has quit IRC | 22:15 | |
fungi | dstufft: we probably shouldn't be using cdecimal | 22:15 |
*** boden has joined #openstack-infra | 22:15 | |
*** boden has quit IRC | 22:15 | |
*** david-lyle_ has quit IRC | 22:16 | |
fungi | dstufft: and looks like we're not ;) | 22:16 |
mordred | dstufft: I completely do not grok the people who are oppoised to pypi upload | 22:16 |
*** boden has joined #openstack-infra | 22:16 | |
dstufft | mordred: especially when the thing they don't want to upload is BSD licensed? ;) | 22:16 |
*** boden has quit IRC | 22:16 | |
*** yamahata has quit IRC | 22:17 | |
dstufft | Normally one of the excuses is that PyPI's ToS are too onerous | 22:17 |
*** david-lyle has quit IRC | 22:17 | |
fungi | dstufft: i mainly wanted to point out to the detractors who claim the developer community will abandon pypi if it dictates sane release standards that there are also parts of the python community who could be driven away if it doesn't | 22:17 |
*** boden has joined #openstack-infra | 22:17 | |
*** yamahata has joined #openstack-infra | 22:17 | |
*** rmcall has joined #openstack-infra | 22:17 | |
*** boden has quit IRC | 22:17 | |
fungi | dstufft: and that perhaps the loss of one of those two groups would be less detrimental than the loss of the other | 22:17 |
dstufft | because they require you grant us an unlimited royality free <insert legal crap> license to redistribute forever and for anyone who uses PyPI to redistribute forrever | 22:17 |
*** dmsimard is now known as dmsimard_away | 22:18 | |
*** boden has joined #openstack-infra | 22:18 | |
mordred | egads. no. anything but that | 22:18 |
*** boden has quit IRC | 22:18 | |
dstufft | (notably there is no legal requirement that anyone be allowed to _use_ what you upload to PyPI, or modify it, only distribute) | 22:18 |
dstufft | the legal clause is entirely "make sure you can't sue the PSF and you can't sue mirror operators" | 22:19 |
fungi | so they'd like to somehow be able to retroactively revoke the abiliaty for anyone to redistribute things uploaded to pypi? that's a little pathological | 22:19 |
dstufft | fungi: funny thing, PyPI is growing in use | 22:19 |
dstufft | fungi: yes | 22:19 |
dstufft | they want to do that | 22:19 |
*** boden has joined #openstack-infra | 22:20 | |
* fungi tries to figure out how that would even work | 22:20 | |
*** david_lyle__ has quit IRC | 22:20 | |
dstufft | plus one of the people against it is in germany and he releases crypto stuff | 22:20 |
clarkb | fungi: thoughts on restarting nodepool shortly? | 22:20 |
dstufft | and he thinks the export laws in the US still matter | 22:20 |
fungi | clarkb: we can restart it now | 22:20 |
*** boden has quit IRC | 22:20 | |
clarkb | fungi: I can do it /me double checks new code is installed | 22:20 |
*** dims_ has joined #openstack-infra | 22:20 | |
clarkb | it appears to be installed | 22:20 |
*** boden has joined #openstack-infra | 22:20 | |
clarkb | fungi: no hope in a gracefull restart ya? | 22:20 |
fungi | stop it and then start it | 22:21 |
clarkb | kk /me does this | 22:21 |
*** boden has quit IRC | 22:21 | |
fungi | it will stop cleanly and quickly | 22:21 |
fungi | in my experience | 22:21 |
dstufft | fungi: this is what the total downloads per week looks like on PyPI since Jan 2014 http://d.stufft.io/image/1f0j0k2u1r0Q | 22:21 |
fungi | restart seems to hang for reasons i've not had time to debug | 22:21 |
dstufft | fungi: I think that we're doing *something* right | 22:21 |
clarkb | done | 22:21 |
jhesketh | Morning | 22:21 |
*** boden has joined #openstack-infra | 22:21 | |
*** r1chardj0n3s_afk is now known as r1chardj0n3s | 22:21 | |
clarkb | I think I will wait for tomorrow and the subsequent image builds before changing hpcloud* over to the dib devstack trusty image | 22:22 |
zaro | jhesketh: morning | 22:22 |
*** boden has quit IRC | 22:22 | |
fungi | dstufft: yep, i just hate seeing people try to stand in the way of progress because they're afraid of change | 22:22 |
*** boden has joined #openstack-infra | 22:22 | |
*** dims_ has quit IRC | 22:22 | |
*** boden has quit IRC | 22:23 | |
clarkb | marun: so going back to productivity. The measure you would like to see used is number of rechecks or time to merge post approval? | 22:23 |
zaro | jhesketh: are you still working on this one? https://review.openstack.org/#/c/107724/ | 22:23 |
*** dkehnm has quit IRC | 22:23 | |
*** gondoi is now known as zz_gondoi | 22:23 | |
*** dims_ has joined #openstack-infra | 22:23 | |
clarkb | marun: trying to understand what in particular you are finding problematic | 22:23 |
marun | clarkb: amount of time to merge | 22:23 |
*** boden has joined #openstack-infra | 22:23 | |
fungi | because some devs might stop using pypi if you tell them they can't reuse version numbers for new package uploads | 22:23 |
*** hdd has quit IRC | 22:23 | |
clarkb | marun: in total since proposed? | 22:23 |
*** jd__ has quit IRC | 22:23 | |
*** dmsimard_away is now known as dmsimard | 22:23 | |
marun | clarkb: no, since approved | 22:23 |
clarkb | kk | 22:23 |
*** dims has quit IRC | 22:23 | |
clarkb | jogo: did you run numbers like that for nova at some point? was it compiled anywhere? | 22:23 |
jogo | marun: that is measurable | 22:23 |
clarkb | jogo: curious if you have ways to collect that already | 22:24 |
*** boden has quit IRC | 22:24 | |
jogo | marun: I didn't do that part of the tseting | 22:24 |
marun | clarkb: assuming we can approve before the check has come back and that the gate won't start until it comes back | 22:24 |
*** boden has joined #openstack-infra | 22:24 | |
clarkb | marun: you can | 22:24 |
jogo | but I can share my scripts in github so you can add time to merge since first approval | 22:24 |
marun | clarkb: isn't that disabled during peak times? | 22:24 |
*** jd__ has joined #openstack-infra | 22:24 | |
clarkb | marun: no | 22:24 |
jhesketh | zaro: yep, still keen for it to merge. I'll take a look at it today | 22:24 |
*** boden has quit IRC | 22:24 | |
clarkb | marun: you always need a +1 | 22:24 |
clarkb | marun: but you can approve before you have that +1 and it will go into the gate after the +1 from jenkins arrives | 22:25 |
marun | clarkb: I seem to remember being warned not to approve until a check passed around FF | 22:25 |
*** boden has joined #openstack-infra | 22:25 | |
jhesketh | (I didn't realise there were comments) | 22:25 |
marun | clarkb: am I misremembering? | 22:25 |
clarkb | marun: I think that was last cycle before we implemented the clean check requirements | 22:25 |
*** boden has quit IRC | 22:25 | |
jogo | clarkb: so this is one big area of miscommunication: what the current gate behaviors are | 22:25 |
marun | clarkb: I swear I recall being told that had been turned off during FF for some reason | 22:26 |
*** boden has joined #openstack-infra | 22:26 | |
clarkb | marun: we can check git logs but pretty sure it wasn't | 22:26 |
marun | clarkb: arg. the whole neutron team was behaving like it was | 22:26 |
marun | :/ | 22:26 |
marun | we were holding off on approval until the check approval came in | 22:26 |
*** boden has quit IRC | 22:26 | |
clarkb | jogo: I think part of the problem there is that the number of devs that understand gating is small | 22:26 |
*** boden has joined #openstack-infra | 22:26 | |
clarkb | jogo: so when we try to communicate behaviors it creates more confusion (granted tis a thing that should be solved) | 22:27 |
marun | sdague: Do you remember anything about the clean check requirements being relaxed around FF for some reason | 22:27 |
marun | ? | 22:27 |
clarkb | marun: they were relaxed but not removed | 22:27 |
*** boden has quit IRC | 22:27 | |
marun | clarkb: what does 'relaxed' imply? | 22:27 |
sdague | marun: freshness checks were removed | 22:27 |
*** boden has joined #openstack-infra | 22:27 | |
clarkb | marun: we stopped running the <24 hour old +1 requirement | 22:27 |
clarkb | marun: whcih is different than requiring clean check | 22:28 |
jogo | clarkb: agreed, a doc that we can point folks to would be useful | 22:28 |
marun | sdague: ah, ok. and I guess we were being careful around that | 22:28 |
sdague | which I did explain in an email, and no one understood what it meant | 22:28 |
*** boden has quit IRC | 22:28 | |
marun | sdague: I'd just forgotten | 22:28 |
sdague | jogo: seriously, a doc will not help | 22:28 |
clarkb | jogo: ^ that is what I am talking about | 22:28 |
clarkb | gating is complex | 22:28 |
clarkb | and most devs don't understand the simple case | 22:28 |
clarkb | I have even had people ask why you would bother testing code before it merges... | 22:28 |
*** boden has joined #openstack-infra | 22:28 | |
marun | clarkb: Well, it works for google... | 22:28 |
fungi | clarkb: it's a reasonable question. many (most?) projects don't | 22:29 |
* marun stops trolling | 22:29 | |
*** boden has quit IRC | 22:29 | |
clarkb | marun: iirc google does gating on some projects | 22:29 |
jogo | sdague: making the current set of gate rules etc hard to discover isn't helping | 22:29 |
marun | clarkb: they always run lots of tests | 22:29 |
*** flaviof is now known as flaviof_zzz | 22:29 | |
clarkb | didn't google go to jeblair at linux con and say hey we do that | 22:29 |
*** boden has joined #openstack-infra | 22:29 | |
marun | clarkb: but they have more than one path to stability | 22:29 |
*** boden has quit IRC | 22:30 | |
clarkb | fungi: I think the fact that many projects don't is independent of "testing your code before merging it is a good thing" | 22:30 |
marun | clarkb: pre- and post-merge have their place | 22:30 |
clarkb | marun: agreed | 22:30 |
*** boden has joined #openstack-infra | 22:30 | |
marun | clarkb: it's questionable that our attempts to catch everything pre-merge are worth the cost | 22:30 |
marun | clarkb: a 2-pronged approach might also be viable | 22:30 |
*** boden has quit IRC | 22:30 | |
clarkb | marun: we have argued for 2 pronged... | 22:31 |
marun | clarkb: but we need better per-project testing of course :/ | 22:31 |
clarkb | marun: one of the things we want to happen is to stop testing postgres and cells and all these funky combos on every change | 22:31 |
*** boden has joined #openstack-infra | 22:31 | |
marun | clarkb++ | 22:31 |
*** praneshp has quit IRC | 22:31 | |
clarkb | and shift that to more of a post workflow | 22:31 |
*** ZZelle_ has quit IRC | 22:31 | |
clarkb | so that interested parties have the data they need to keep those features working without impacting the stability of common setups | 22:31 |
*** boden has quit IRC | 22:31 | |
clarkb | but that hasn't seemed to have moved very far since the idea happened | 22:32 |
*** boden has joined #openstack-infra | 22:32 | |
*** boden has quit IRC | 22:32 | |
*** boden has joined #openstack-infra | 22:33 | |
marun | clarkb: how about those jobs are moved out of the integrated gate? | 22:33 |
clarkb | marun: thats basically what would happen | 22:33 |
jogo | clarkb: speaking of post merge tests. what is the progress on that? | 22:33 |
*** boden has quit IRC | 22:33 | |
clarkb | jogo: I dno't think anyone has started working on it. we probably need bigger buy in | 22:33 |
marun | clarkb: and then the interested parties can devote resources to making post-merge happen | 22:33 |
clarkb | before we tell postgres and cells we are changing their workflow :) | 22:33 |
*** boden has joined #openstack-infra | 22:34 | |
marun | clarkb: right now they have no incentive - they already have what they want | 22:34 |
clarkb | marun: right | 22:34 |
*** boden has quit IRC | 22:34 | |
clarkb | sdague: you started that proposal, did it die early? | 22:34 |
clarkb | sdague: maybe we should pick that thread up again and say look we are doing this early in kilo | 22:34 |
sdague | I started it | 22:34 |
jogo | what kind of buy in is needed? | 22:35 |
*** boden has joined #openstack-infra | 22:35 | |
clarkb | jogo: we should probably make sure that ptls are on board | 22:35 |
sdague | jogo: no buy in needed, just workers | 22:35 |
jogo | I think I can get nova-core to agree we want that | 22:35 |
jogo | sdague: ahh thats what I thought | 22:35 |
clarkb | sdague: that works for me too :) | 22:35 |
*** boden has quit IRC | 22:35 | |
sdague | honestly, there is a ton of work that needs to happen for it | 22:35 |
clarkb | sdague: ya its not a simple change | 22:35 |
sdague | and, I'll be honest, I got burnt out on it | 22:35 |
*** boden has joined #openstack-infra | 22:35 | |
jogo | sdague: yeah in germany the list of steps was pretty big | 22:35 |
*** jgrimm is now known as zz_jgrimm | 22:36 | |
*** boden has quit IRC | 22:36 | |
*** homeless has quit IRC | 22:36 | |
*** praneshp has joined #openstack-infra | 22:36 | |
sdague | because the other thing is, when you spend time looking at gate issues a lot, what you actually want to do is remove +2 authority from all the core teams | 22:36 |
*** boden has joined #openstack-infra | 22:36 | |
marun | sdague: I can imagine that sentiment | 22:36 |
fungi | the rate at which we merge new bugs seems to be at least proportional to the rate at which we merge anything | 22:37 |
*** boden has quit IRC | 22:37 | |
marun | sdague: I see stuff merge in Neutron that shouldn't, all the time. :/ | 22:37 |
sdague | especially when you have critical bugs that no one looks at | 22:37 |
sdague | that are ruining the gate for everyone | 22:37 |
*** homeless has joined #openstack-infra | 22:37 | |
sdague | and no one cares | 22:37 |
*** boden has joined #openstack-infra | 22:37 | |
sdague | see... -meeting from an hour ago | 22:37 |
clarkb | marun: so the 2 pronged approach may be a good first step on reducing that friction. Then as the functional test suites become more robust (and actually exist) we can revisit the stop running tempest a lot question | 22:37 |
*** boden has quit IRC | 22:38 | |
fungi | and it's amusing/sad to look at timelines of varying levels of our breakage, to see that as soon as we fix enough bugs to make it faster for project to land new changes, they merge in new bugs more quickly to get back to the same pain threshold | 22:38 |
*** boden has joined #openstack-infra | 22:38 | |
marun | fungi, sdague: there is little incentive for people to write better quality features when they aren't directly responsible for fixing bugs. :/ | 22:38 |
*** boden has quit IRC | 22:39 | |
fungi | marun: that was the philosophy behind pre-merge testing, but it doesn't account for nondeterministic bugs | 22:39 |
*** boden has joined #openstack-infra | 22:39 | |
marun | The only way around this from my perspective is providing sufficient test infrastructure in-project and then mandating that all project-specific testing be present pre-merge | 22:39 |
*** david-lyle has joined #openstack-infra | 22:39 | |
*** boden has quit IRC | 22:39 | |
*** _buzz has quit IRC | 22:39 | |
sdague | right, and it definitely didn't account for developer behavior of not giving a crap about the project they work on and running 'recheck' 10 times to land their feature because their project manager wanted it in this release | 22:40 |
marun | We're making progress, slowly, on this in neutron. At least functional tests are possible. | 22:40 |
*** boden has joined #openstack-infra | 22:40 | |
*** boden has quit IRC | 22:40 | |
*** boden has joined #openstack-infra | 22:41 | |
*** ryanpetrello has quit IRC | 22:41 | |
*** boden has quit IRC | 22:41 | |
*** tsg has quit IRC | 22:41 | |
*** david-lyle_ has joined #openstack-infra | 22:41 | |
*** boden has joined #openstack-infra | 22:42 | |
marun | sdague, fungi, clarkb: Are most gate-blocking bugs race conditions? | 22:42 |
clarkb | marun: not always, but often yes | 22:42 |
*** boden has quit IRC | 22:42 | |
*** datsun180b has quit IRC | 22:42 | |
*** boden has joined #openstack-infra | 22:43 | |
clarkb | a non race condition that was making the gate run really slowly had to do with keystone performance under a single eventlet working process | 22:43 |
clarkb | it wasn't quite blocking but added 30 minutes to test run time | 22:43 |
clarkb | and no one noticed until I said hey guys whats going on here... | 22:43 |
*** boden has quit IRC | 22:43 | |
marun | clarkb: So, bugs that slipped through despite all the integrated jobs... | 22:43 |
fungi | the longer-lived ones are race conditions and/or other similar nondeterministic bugs in openstack or in the way tests are written | 22:43 |
marun | clarkb: yikes | 22:43 |
*** boden has joined #openstack-infra | 22:43 | |
*** armax has quit IRC | 22:44 | |
clarkb | marun: sort of. in many cases the issues were caught at least one in check or gate | 22:44 |
clarkb | marun: then they get rechecked until they merge | 22:44 |
*** boden has quit IRC | 22:44 | |
clarkb | marun: we have seen that happen on a few of these bugs | 22:44 |
*** david-lyle has quit IRC | 22:44 | |
fungi | there are also plenty of "gate-breaking" bugs which are due to external causes (dependency changes, incidents at service providers impacting our ci systems, et cetera) but those tend to get identified and addressed fare more quickly | 22:44 |
*** boden has joined #openstack-infra | 22:44 | |
sdague | marun: ... if only I'd given a talk on this subject ... ;) | 22:44 |
*** boden has quit IRC | 22:45 | |
marun | sdague: I'm wondering - if our biggest problem is race conditions that slip through despite our best efforts. | 22:45 |
sdague | marun: yes, because statistics are against you | 22:45 |
*** boden has joined #openstack-infra | 22:45 | |
*** hdd has joined #openstack-infra | 22:45 | |
fungi | and, irony of ironies, many of the longer-lived sources of failures in providers is *actually* nondeterministic bugs in openstack | 22:45 |
clarkb | ya we have seen manifestations of git bugs in our providers more than once | 22:45 |
sdague | given a complex system, if a failure were to happen 10% of the time | 22:46 |
clarkb | s/git/gate/ | 22:46 |
*** boden has quit IRC | 22:46 | |
fungi | things like loss of connectivity from workers to git mirrors, pypi mirrors, et cetera... actually openstack bugs impacting our providers | 22:46 |
*** yamahata has quit IRC | 22:46 | |
*** boden has joined #openstack-infra | 22:46 | |
marun | fungi: yikes | 22:46 |
*** dane_leblanc has quit IRC | 22:46 | |
*** beecee has joined #openstack-infra | 22:46 | |
*** boden has quit IRC | 22:46 | |
fungi | bugs which have slipped through testing, and yes, sometimes bugs where we already have failure log signatures and nobody's had time to run down the underlying causes | 22:47 |
marun | that implies, though, that the integrated gate serves more to slow down merging of new patches (new bugs) more than anything | 22:47 |
*** boden has joined #openstack-infra | 22:47 | |
marun | Since a good proportion of the problems have nothing to do with the patches at all | 22:47 |
*** boden has quit IRC | 22:47 | |
*** ryanpetrello has joined #openstack-infra | 22:47 | |
fungi | well, i'd wager that if we weren't doing the current base level of testing, providers wouldn't be using this at all | 22:47 |
marun | And race conditions are as likely to slip through as anything | 22:47 |
marun | fungi: I'm not suggesting that testing isn't required | 22:48 |
*** boden has joined #openstack-infra | 22:48 | |
fungi | race conditions and similar nondeterministic bugs are what slip through, yes, as well as bugs in code which isn't covered by tests of course | 22:48 |
marun | fungi: sure. | 22:48 |
*** boden has quit IRC | 22:48 | |
clarkb | marun: except that in many cases the patches that introduce the race actually failed tests at least once | 22:48 |
*** boden has joined #openstack-infra | 22:49 | |
clarkb | because the net is wide | 22:49 |
marun | fungi: it would seem, though, that we could accomplish a similar result by artificially limiting the merge rate and running a crapload of integrated job post-merge | 22:49 |
marun | I'm not saying that's a good idea | 22:49 |
marun | only that the result would effectively be the same | 22:49 |
*** boden has quit IRC | 22:49 | |
*** boden has joined #openstack-infra | 22:50 | |
sdague | marun: there are definitely lots of incremental ways to make this better | 22:50 |
sdague | but they mostly need people to work on them | 22:50 |
marun | clarkb: it can be hard for the average developer to discern a difference between a transient, infra bug, an unrelated race condition, or a failure they themselves are causing | 22:50 |
*** boden has quit IRC | 22:50 | |
marun | :/ | 22:50 |
clarkb | marun: I would say that is true for a chunk of them, but most peopel seem to completely ignore test results | 22:50 |
*** boden has joined #openstack-infra | 22:50 | |
*** JayJ has quit IRC | 22:51 | |
sdague | marun: I agree that debugging openstack is hard | 22:51 |
sdague | we should fix that | 22:51 |
*** boden has quit IRC | 22:51 | |
* marun waves his magic wand | 22:51 | |
clarkb | I know I am guilty of only looking at the SUCCESS v FAILURE reported back to gerrit | 22:51 |
marun | tada! | 22:51 |
*** boden has joined #openstack-infra | 22:51 | |
sdague | honestly, I think there is value in your idea about limitted testing support in zuul, honestly, I'd like it for docs/ trees (that's where I originally brought up the idea) | 22:52 |
*** boden has quit IRC | 22:52 | |
sdague | but ... someone needs to hack on it | 22:52 |
*** boden has joined #openstack-infra | 22:52 | |
sdague | so if you think it's a top fix to dev productivity, raise your hand and dive in, and I'm sure people will point you in the right direction | 22:53 |
*** boden has quit IRC | 22:53 | |
openstackgerrit | Clark Boylan proposed a change to openstack-infra/config: Add devstack-precise-dib image to nodepool https://review.openstack.org/125220 | 22:53 |
*** baoli has joined #openstack-infra | 22:53 | |
*** boden has joined #openstack-infra | 22:53 | |
marun | sdague: will do | 22:53 |
marun | sdague: I | 22:53 |
fungi | i would argue (and have) that troubleshooting integration test failures is basically the same as if you were a one-person operations team at a service provider troubleshooting your cloud falling over | 22:53 |
fungi | the range of possible ways it can go wrong is basically the same | 22:53 |
*** boden has quit IRC | 22:53 | |
marun | fungi: Can I assume that this is a bad thing? | 22:54 |
marun | fungi: or is it necessary complexity? | 22:54 |
sdague | well, we don't make it easy for people | 22:54 |
*** boden has joined #openstack-infra | 22:54 | |
sdague | seriously, staring at neutron logs for the last 2 hrs here | 22:54 |
sdague | we do not make it easy for people | 22:54 |
marun | sdague: arguably our lack of project-focused tests is at fault here | 22:54 |
fungi | i think it's necessary complexity, but i also think we as openstack are in a better position to make those bugs more shallow than $random_operator | 22:54 |
marun | sdague: most issues should not be debugged in an integrated env | 22:54 |
*** boden has quit IRC | 22:54 | |
marun | or rather, should be able to be debugged in isolation | 22:55 |
*** boden has joined #openstack-infra | 22:55 | |
sdague | marun: so I only partially agree | 22:55 |
*** boden has quit IRC | 22:55 | |
marun | sdague: me too :) | 22:55 |
sdague | if things go wrong, the system should be giving you enough info to figure it out | 22:56 |
*** boden has joined #openstack-infra | 22:56 | |
sdague | especially when we are running at DEBUG | 22:56 |
marun | sdague: I kind of agree | 22:56 |
fungi | some issues yes. some issues only surface under integration because they're bugs in how the projects interact with one another | 22:56 |
marun | sdague: but working at too high a level by choice seems unwise | 22:56 |
*** boden has quit IRC | 22:56 | |
marun | sdague: if we're developers, we can choose the level we want to work at | 22:56 |
sdague | sure | 22:56 |
fungi | defining and testing those interfaces would make a huge difference there, but we're a very long way from that being feasible from what i've seen | 22:56 |
marun | sdague: operators, though, we have to cover that demographic too. no question | 22:57 |
sdague | but we also need to make a system that operators have any hope of debugging | 22:57 |
*** boden has joined #openstack-infra | 22:57 | |
sdague | the gate is basically "lets play operator" | 22:57 |
*** koolhead17 has quit IRC | 22:57 | |
*** boden has quit IRC | 22:57 | |
*** andreaf has quit IRC | 22:57 | |
marun | sdague: that being the case, I don't think that every single patch should be triggering the 'operator' switch | 22:57 |
*** boden has joined #openstack-infra | 22:57 | |
*** stevemar has quit IRC | 22:57 | |
*** andreaf has joined #openstack-infra | 22:58 | |
marun | sdague: there is pain in the 'converge towards stability post-merge' approach | 22:58 |
*** jergerber has quit IRC | 22:58 | |
marun | sdague: no question | 22:58 |
*** boden has quit IRC | 22:58 | |
marun | sdague: but gating everyone on the weakest member is as bad an idea in software as it is in chains | 22:58 |
*** boden has joined #openstack-infra | 22:58 | |
*** boden has quit IRC | 22:59 | |
marun | reliable distributed software attempts to limit failure domains | 22:59 |
*** koolhead17 has joined #openstack-infra | 22:59 | |
*** koolhead17 has quit IRC | 22:59 | |
sdague | well, under that assumption, we'd just delete neutron as a project | 22:59 |
marun | our gate process conflates them | 22:59 |
*** boden has joined #openstack-infra | 22:59 | |
marun | sdague: I wouldn't argue :) | 22:59 |
openstackgerrit | Michael Krotscheck proposed a change to openstack-infra/storyboard-webclient: URL Util now maps full install path. https://review.openstack.org/125222 | 22:59 |
*** boden has quit IRC | 23:00 | |
*** boden has joined #openstack-infra | 23:00 | |
marun | sdague, clarkb, fungi, mordred: I have to run. Thank you for the constructive conversation, as always. | 23:00 |
*** AndroUser2 has joined #openstack-infra | 23:00 | |
openstackgerrit | Michael Krotscheck proposed a change to openstack-infra/storyboard-webclient: Subscription https://review.openstack.org/104004 | 23:00 |
*** boden has quit IRC | 23:00 | |
marun | and for bearing with me | 23:01 |
openstackgerrit | Clark Boylan proposed a change to openstack-infra/config: Switch hpcloud devstack-trusty to dib image https://review.openstack.org/125223 | 23:01 |
openstackgerrit | Clark Boylan proposed a change to openstack-infra/config: Put devstack-trusty-dib image in rax https://review.openstack.org/125224 | 23:01 |
clarkb | fungi: ^ I don't think I am ready to merge those yet. I want to see image updates working again tomorrow first | 23:01 |
clarkb | fungi: but that gievs you an idea of what I am thinking for the continung dibification of stuff | 23:01 |
*** boden has joined #openstack-infra | 23:01 | |
*** boden has quit IRC | 23:01 | |
*** boden has joined #openstack-infra | 23:02 | |
*** david-lyle_ has quit IRC | 23:02 | |
*** boden has quit IRC | 23:02 | |
*** boden has joined #openstack-infra | 23:03 | |
*** boden has quit IRC | 23:03 | |
*** esker has quit IRC | 23:03 | |
*** boden has joined #openstack-infra | 23:04 | |
*** koolhead17 has joined #openstack-infra | 23:04 | |
*** weshay has joined #openstack-infra | 23:04 | |
*** dkranz has quit IRC | 23:04 | |
*** koolhead17 has quit IRC | 23:04 | |
*** boden has quit IRC | 23:04 | |
openstackgerrit | K Jonathan Harker proposed a change to openstack-infra/nodepool: Reset the client object before creating the image https://review.openstack.org/74943 | 23:04 |
*** boden has joined #openstack-infra | 23:04 | |
*** boden has quit IRC | 23:05 | |
*** marun has quit IRC | 23:05 | |
clarkb | sdague: fungi: so I wonder if debugging the gate being so much like what happens in ops trenches is partially to blame for devs not debugging as much as we like? | 23:05 |
*** boden has joined #openstack-infra | 23:05 | |
clarkb | I think its the proper way to do things so that we feel the pain of operators | 23:05 |
clarkb | but it could explain part of the "hardness" of it | 23:06 |
clarkb | ops is different | 23:06 |
*** boden has quit IRC | 23:06 | |
*** boden has joined #openstack-infra | 23:06 | |
*** boden has quit IRC | 23:07 | |
clarkb | debugging large systems without running an attached debugger is very much an ops thing | 23:07 |
krotscheck | Can I get a sanity confirmation of something? Go to this review, click on ‘gate-storyboard-webclient-js-draft’, and try to log in? | 23:07 |
krotscheck | https://review.openstack.org/#/c/125222/ | 23:07 |
*** boden has joined #openstack-infra | 23:07 | |
clarkb | devs want to have stuff in as narrow a focus as possible with debugger attached | 23:07 |
clarkb | krotscheck: so I need to start logged out? | 23:08 |
*** boden has quit IRC | 23:08 | |
clarkb | oh wait | 23:08 |
clarkb | you want me to login to the thing linked gotcha | 23:08 |
*** boden has joined #openstack-infra | 23:08 | |
krotscheck | clarkb: RIght | 23:08 |
krotscheck | clarkb: Looks like it still can’t post because of a CORS header issue, but I can fix that... | 23:08 |
clarkb | it let me in | 23:09 |
jogo | clarkb: I do think that if all OpenStack devs had to deploy OpenStack from scratch (re: no devstack packages and such are OK) and debug an install, things would be better | 23:09 |
*** boden has quit IRC | 23:09 | |
krotscheck | clarkb: AWESOME | 23:09 |
*** boden has joined #openstack-infra | 23:09 | |
*** boden has quit IRC | 23:09 | |
clarkb | jogo: well I think it may just be that as individuals withd ifferent skill sets we thinkdifferentl | 23:09 |
clarkb | jogo: I don't tend to have problems jumping in and doing hands off debugging based on logs and such ebcause I do lots of ops | 23:10 |
clarkb | jogo: its the way we do things | 23:10 |
*** boden has joined #openstack-infra | 23:10 | |
clarkb | but if your background is always being attached to a debugger and needing to debug based on live events doing it the other way can be hard | 23:10 |
*** boden has quit IRC | 23:10 | |
*** baoli has quit IRC | 23:10 | |
mtreinish | jogo: I've done it, it wasn't too bad for me, only took me about a week... | 23:10 |
clarkb | mtreinish: jogo: I did it in an afternoon with diablo... | 23:11 |
mordred | I add print('dammit') in a lot | 23:11 |
*** boden has joined #openstack-infra | 23:11 | |
mordred | it's not super scientific | 23:11 |
*** sarob is now known as sarob_away | 23:11 | |
jogo | mtreinish: how recent was that? | 23:11 |
clarkb | annegentle_'s install guide was great :) | 23:11 |
jogo | mtreinish: that it took a week | 23:11 |
mtreinish | jogo: the week before I left IBM | 23:11 |
*** boden has quit IRC | 23:11 | |
*** weshay is now known as weshay_afk | 23:11 | |
jogo | mtreinish: wow, so fairly recently | 23:11 |
*** MaxV has quit IRC | 23:12 | |
clarkb | and having functional tests where you can attach directly to those live events will help significantly | 23:12 |
*** boden has joined #openstack-infra | 23:12 | |
openstackgerrit | Michael Krotscheck proposed a change to openstack-infra/storyboard: Added Content-Type header to CORS https://review.openstack.org/125227 | 23:12 |
clarkb | or at least I think that is what my theory is becoming | 23:12 |
*** atiwari has joined #openstack-infra | 23:12 | |
*** boden has quit IRC | 23:12 | |
mtreinish | clarkb: well I did it for a rack and a half | 23:12 |
clarkb | because that is an env that devs are comfortable in. But we shouldn't forget indirect debugging because operators everywhere deal with our tools that way | 23:12 |
*** sarob_away is now known as sarob | 23:12 | |
clarkb | mtreinish: ah, I only did a one node install | 23:12 |
*** boden has joined #openstack-infra | 23:12 | |
mtreinish | jogo: I didn't say I did a good job though :) a bunch of things were horribly misconfigured | 23:13 |
mtreinish | but it worked... | 23:13 |
jogo | mtreinish: haha | 23:13 |
*** achanda has joined #openstack-infra | 23:13 | |
*** boden has quit IRC | 23:13 | |
jogo | mtreinish: if it took the PTL of QA a week to just get it working .... | 23:13 |
*** dims_ has quit IRC | 23:13 | |
jogo | that is not good | 23:13 |
*** boden has joined #openstack-infra | 23:13 | |
mtreinish | jogo: oh and flatdhcp n-net I had enough sense to avoid trying neutron | 23:14 |
jogo | mtreinish: haha wise | 23:14 |
clarkb | jogo: it isn't but honestly it takes that long to install a vmware cluster of a rack and a half. stuff is complex | 23:14 |
*** boden has quit IRC | 23:14 | |
*** AndroUser2 has quit IRC | 23:14 | |
*** boden has joined #openstack-infra | 23:14 | |
jogo | clarkb: hmm I didn't realize vmware takes that long as well | 23:14 |
*** IAX has quit IRC | 23:15 | |
mordred | clarkb: I wonder how long a rack of infra would take | 23:15 |
*** boden has quit IRC | 23:15 | |
clarkb | jogo: people like to say it doesn't take that long, but when I did it a few years back it took ages | 23:15 |
clarkb | jogo: networking is a big hang up there too | 23:15 |
*** armax has joined #openstack-infra | 23:15 | |
*** boden has joined #openstack-infra | 23:15 | |
jogo | clarkb: but I do think the act of having more developers feel the operator/usability pain is a good thing | 23:15 |
clarkb | mordred: lifeless did it in what a week? | 23:15 |
jogo | one way or onother | 23:15 |
*** stevemar has joined #openstack-infra | 23:16 | |
*** boden has quit IRC | 23:16 | |
mtreinish | clarkb: yeah networking was my issue too. Luckily that was a replacement setup so I had learned all the tough lessons already | 23:16 |
*** boden has joined #openstack-infra | 23:16 | |
mtreinish | it took me a month to work through some of that the first time (with only 4 nodes) | 23:16 |
*** boden has quit IRC | 23:16 | |
*** baoli has joined #openstack-infra | 23:16 | |
*** loquacities has joined #openstack-infra | 23:17 | |
*** boden has joined #openstack-infra | 23:17 | |
*** boden has quit IRC | 23:17 | |
clarkb | tl;dr networking is hard | 23:17 |
clarkb | which is one of the reasons I like jeblair's philosophy of computers should just talk to each other :) | 23:18 |
*** boden has joined #openstack-infra | 23:18 | |
clarkb | it reduces a lot of complexity when you say that proxies and intermediate firewalls should diaf | 23:18 |
*** boden has quit IRC | 23:18 | |
*** boden has joined #openstack-infra | 23:19 | |
*** boden has quit IRC | 23:19 | |
*** boden has joined #openstack-infra | 23:20 | |
*** armax has quit IRC | 23:20 | |
*** boden has quit IRC | 23:20 | |
*** armax has joined #openstack-infra | 23:20 | |
*** boden has joined #openstack-infra | 23:20 | |
*** dkranz has joined #openstack-infra | 23:20 | |
*** boden has quit IRC | 23:21 | |
fungi | clarkb: jogo: as a point of reference, vce was (presumably still is) a large vmware integrator. they used an orchestration tool from emc, the name of which escapes me, to install vmware starting from bare metal, more or less turn-key... deploying several racks took on the order of a couple days of the orchestration software chugging nonstop | 23:21 |
*** boden has joined #openstack-infra | 23:21 | |
*** boden has quit IRC | 23:22 | |
*** boden has joined #openstack-infra | 23:22 | |
*** sarob is now known as sarob_away | 23:22 | |
clarkb | fungi: we did dense blade chassis at a slower rate mostly manually | 23:22 |
*** bswartz has joined #openstack-infra | 23:22 | |
clarkb | because automation didn't exist for various things in the stack | 23:23 |
*** MarkAtwood has joined #openstack-infra | 23:23 | |
*** boden has quit IRC | 23:23 | |
*** boden has joined #openstack-infra | 23:23 | |
jhesketh | clarkb: if you have time, I need some help debugging zuul swift stuff | 23:23 |
fungi | right. point being, fully automated, days to install a few racks. human intervention would obviously have multiplied that several times over | 23:23 |
clarkb | jhesketh: sure | 23:23 |
clarkb | jhesketh: I am mostly done with dib for the day | 23:23 |
*** boden has quit IRC | 23:23 | |
jhesketh | clarkb: awesome, thanks :-) | 23:24 |
jhesketh | clarkb: basically Jenkins isn't getting the correct swift values | 23:24 |
jhesketh | ie it hasn't got any env vars | 23:24 |
*** boden has joined #openstack-infra | 23:24 | |
fungi | i am mostly done for the day, with the usual feeling of having accomplished none of the things i set out to get done. there's always tomorrow! | 23:24 |
jhesketh | clarkb: are you able to see how swift is configured in zuul.conf? | 23:24 |
clarkb | jhesketh: is this happening with every job? | 23:24 |
jhesketh | clarkb: as far as I can tell, yes | 23:24 |
clarkb | jhesketh: I can check zuul.conf now | 23:24 |
*** boden has quit IRC | 23:24 | |
*** boden has joined #openstack-infra | 23:25 | |
*** boden has quit IRC | 23:25 | |
*** _nadya_ has joined #openstack-infra | 23:25 | |
*** boden has joined #openstack-infra | 23:26 | |
*** boden has quit IRC | 23:26 | |
*** boden has joined #openstack-infra | 23:27 | |
lifeless | clarkb: did what in a week? | 23:27 |
*** sarob_away is now known as sarob | 23:27 | |
clarkb | lifeless: deployed a set of openstack infra in not infra | 23:27 |
*** boden has quit IRC | 23:27 | |
clarkb | for that talk you ran through the things | 23:27 |
clarkb | jhesketh: I don't see anyting amiss there | 23:27 |
*** boden has joined #openstack-infra | 23:27 | |
lifeless | clarkb: are you referring to tripleo-ci ? or the testing-cabal deploy of zuul* ? | 23:27 |
clarkb | lifeless: testing cabal | 23:28 |
*** boden has quit IRC | 23:28 | |
openstackgerrit | K Jonathan Harker proposed a change to openstack-infra/nodepool: Reset the client object before creating the image https://review.openstack.org/74943 | 23:28 |
lifeless | a week of evenings got me most of the way yeah | 23:28 |
lifeless | clarkb: what is a 'rack of infra' though | 23:28 |
*** boden has joined #openstack-infra | 23:28 | |
lifeless | clarkb: are you suggesting an infra testing infra deployment? | 23:28 |
clarkb | lifeless: more just a rack somewhere that runs infra services | 23:29 |
*** boden has quit IRC | 23:29 | |
clarkb | that rack may be virtual I guess | 23:29 |
*** boden has joined #openstack-infra | 23:29 | |
*** _nadya_ has quit IRC | 23:30 | |
*** boden has quit IRC | 23:30 | |
*** boden has joined #openstack-infra | 23:30 | |
clarkb | jhesketh: have an example jenkins job link? | 23:31 |
*** boden has quit IRC | 23:31 | |
jhesketh | clarkb: https://jenkins01.openstack.org/job/gate-project-config-layout/14/injectedEnvVars/? | 23:31 |
*** boden has joined #openstack-infra | 23:31 | |
jhesketh | clarkb: actually I think we aren't setting up swift correctly in the layout file... let me check | 23:31 |
clarkb | oh that could be | 23:31 |
*** boden has quit IRC | 23:31 | |
*** boden has joined #openstack-infra | 23:32 | |
jhesketh | clarkb: yeah, that appears to be it.. I'll shoot through a fix | 23:32 |
clarkb | jhesketh: kk | 23:32 |
*** boden has quit IRC | 23:32 | |
*** boden has joined #openstack-infra | 23:33 | |
*** dmsimard is now known as dmsimard_away | 23:33 | |
*** boden has quit IRC | 23:33 | |
*** zz_jgrimm is now known as jgrimm | 23:33 | |
*** boden has joined #openstack-infra | 23:34 | |
*** camunoz has joined #openstack-infra | 23:34 | |
*** boden has quit IRC | 23:34 | |
*** otter768 has joined #openstack-infra | 23:34 | |
*** boden has joined #openstack-infra | 23:35 | |
openstackgerrit | Joshua Hesketh proposed a change to openstack-infra/project-config: Enable the swift logs env vars for all jobs https://review.openstack.org/125232 | 23:35 |
*** boden has quit IRC | 23:35 | |
jhesketh | clarkb: ^ | 23:35 |
*** stevemar has quit IRC | 23:35 | |
anteaya | clarkb: thanks | 23:36 |
*** mriedem has joined #openstack-infra | 23:36 | |
*** boden has joined #openstack-infra | 23:36 | |
clarkb | jhesketh: I think you need to update your local version of layout.yaml then repropose? that diff is huge | 23:37 |
*** boden has quit IRC | 23:37 | |
clarkb | jhesketh: and undoes AJaeger's changes from ealier fi I am reading correctly | 23:37 |
*** dmellado has joined #openstack-infra | 23:37 | |
*** boden has joined #openstack-infra | 23:37 | |
jhesketh | hmm, probably a git fail sorry | 23:37 |
clarkb | np :) | 23:37 |
*** boden_ has joined #openstack-infra | 23:38 | |
*** boden has quit IRC | 23:38 | |
openstackgerrit | Joshua Hesketh proposed a change to openstack-infra/project-config: Enable the swift logs env vars for all jobs https://review.openstack.org/125232 | 23:38 |
jhesketh | clarkb: okay, try that | 23:38 |
*** boden_ has quit IRC | 23:39 | |
*** boden has joined #openstack-infra | 23:39 | |
*** boden_ has joined #openstack-infra | 23:40 | |
*** boden has quit IRC | 23:40 | |
clarkb | jhesketh: muc better. is there an existing swift: section that can be removed? | 23:40 |
*** boden_ has quit IRC | 23:40 | |
jhesketh | clarkb: oh yeah, on the experimental job... but I'll shoot through another patch to remove all of that job when we no longer want it | 23:41 |
*** boden has joined #openstack-infra | 23:41 | |
clarkb | jhesketh: sounds good | 23:41 |
*** gus has joined #openstack-infra | 23:41 | |
*** boden has quit IRC | 23:41 | |
*** boden has joined #openstack-infra | 23:42 | |
sdague | clarkb: how do you feel about - https://review.openstack.org/#/c/124390/ ? | 23:42 |
*** boden has quit IRC | 23:42 | |
sdague | jhesketh: also, I fixed the hideci issues I believe - https://review.openstack.org/#/c/124632/ | 23:42 |
clarkb | sdague: reading the commit message Ithink that is a good idea | 23:42 |
sdague | that would be nice to get out there | 23:42 |
*** MaxV has joined #openstack-infra | 23:42 | |
clarkb | let me review it properly | 23:42 |
*** boden has joined #openstack-infra | 23:42 | |
lifeless | clarkb: isn't that what you have today? | 23:43 |
jhesketh | sdague: cool, I was just reading through it :-) | 23:43 |
lifeless | clarkb: is the idea to get away from cloud providers? | 23:43 |
*** boden has quit IRC | 23:43 | |
clarkb | lifeless: no no, I think mordred was just wondering since we were talking about how long it takes to install openstack | 23:43 |
clarkb | or isntall a vmware cluster | 23:43 |
*** boden has joined #openstack-infra | 23:43 | |
lifeless | clarkb: cause, you have an infra infra account on the tripleo clouds | 23:43 |
lifeless | ah | 23:43 |
lifeless | so yeah, depends heavily on what trunk is like | 23:44 |
lifeless | right now for instance we're two months into hp2 and its still not up | 23:44 |
*** boden has quit IRC | 23:44 | |
*** boden has joined #openstack-infra | 23:44 | |
*** Sincler has quit IRC | 23:44 | |
*** boden has quit IRC | 23:45 | |
*** boden has joined #openstack-infra | 23:45 | |
*** boden_ has joined #openstack-infra | 23:46 | |
*** boden has quit IRC | 23:46 | |
*** andreaf has quit IRC | 23:46 | |
*** JayJ has joined #openstack-infra | 23:46 | |
*** boden_ has quit IRC | 23:46 | |
*** andreaf has joined #openstack-infra | 23:47 | |
*** JayJ has joined #openstack-infra | 23:47 | |
*** boden has joined #openstack-infra | 23:47 | |
*** MaxV has quit IRC | 23:47 | |
*** boden has quit IRC | 23:47 | |
*** boden has joined #openstack-infra | 23:48 | |
*** boden has quit IRC | 23:48 | |
*** boden has joined #openstack-infra | 23:49 | |
*** ivar-laz_ has joined #openstack-infra | 23:49 | |
*** boden has quit IRC | 23:49 | |
*** boden has joined #openstack-infra | 23:50 | |
*** ivar-laz_ has quit IRC | 23:50 | |
*** boden has quit IRC | 23:50 | |
jhesketh | sdague: reviewed.. I think there's a bug with the way you're loading the comments | 23:50 |
*** boden has joined #openstack-infra | 23:50 | |
*** ivar-laz_ has joined #openstack-infra | 23:50 | |
*** ivar-lazzaro has quit IRC | 23:51 | |
sdague | mtreinish: https://review.openstack.org/#/q/status:open+project:openstack-dev/devstack+branch:stable/icehouse+topic:no_screen,n,z | 23:51 |
sdague | those are the devstack patches I need | 23:51 |
*** boden has quit IRC | 23:51 | |
*** ivar-laz_ has quit IRC | 23:51 | |
*** boden has joined #openstack-infra | 23:51 | |
openstackgerrit | A change was merged to openstack-infra/project-config: Enable the swift logs env vars for all jobs https://review.openstack.org/125232 | 23:51 |
fungi | jhesketh: if that ^ breaks the world, i won't be around to help debug ;) | 23:52 |
*** boden has quit IRC | 23:52 | |
clarkb | fungi: way to put us in the hot seat :P | 23:52 |
mtreinish | sdague: ok, I'll take a look | 23:52 |
*** ivar-lazzaro has joined #openstack-infra | 23:52 | |
*** boden has joined #openstack-infra | 23:52 | |
fungi | clarkb: you can always revert! | 23:52 |
clarkb | indeed | 23:53 |
*** boden has quit IRC | 23:53 | |
clarkb | sdague: do you want to fix the whitespace thing? | 23:53 |
*** boden has joined #openstack-infra | 23:53 | |
jhesketh | fungi: heh, I can never help debug when I merge stuff | 23:53 |
clarkb | in that zuul change? | 23:53 |
jhesketh | makes it scary | 23:53 |
clarkb | also not sure how comfortable I am merging zuul chagnes without jeblair around that are not bug fixes | 23:53 |
clarkb | but that change seems reasonable | 23:53 |
fungi | it was a zuul configuration change | 23:54 |
*** boden has quit IRC | 23:54 | |
fungi | so seemed fair game | 23:54 |
*** MarkAtwood has quit IRC | 23:54 | |
clarkb | fungi: that one is fine | 23:54 |
*** boden has joined #openstack-infra | 23:54 | |
clarkb | I am talking about https://review.openstack.org/#/c/124390/4 | 23:54 |
fungi | oh, heh no | 23:54 |
*** boden has quit IRC | 23:54 | |
*** boden has joined #openstack-infra | 23:55 | |
fungi | though i do need to re-review that one | 23:55 |
clarkb | sdague: particularly since we seem to not be doing the thing that jeblair suggested? | 23:55 |
*** boden_ has joined #openstack-infra | 23:56 | |
*** boden has quit IRC | 23:56 | |
*** boden_ has quit IRC | 23:56 | |
*** boden has joined #openstack-infra | 23:57 | |
sdague | clarkb: so I thought we were doing it | 23:57 |
mtreinish | sdague: do those test files get run anywhere? | 23:57 |
*** baoli has quit IRC | 23:57 | |
clarkb | sdague: it? | 23:57 |
*** boden has quit IRC | 23:57 | |
sdague | mtreinish: honestly, I don't know | 23:57 |
*** chuckC_ has quit IRC | 23:57 | |
*** boden has joined #openstack-infra | 23:57 | |
*** ivar-laz_ has joined #openstack-infra | 23:58 | |
*** boden has quit IRC | 23:58 | |
clarkb | sdague: "I like the cache approach used here, but I think we should consider the query-string and filtering options in Joshuas change. See more detailed comments there." is the jeblair comment i refer to | 23:58 |
*** boden has joined #openstack-infra | 23:58 | |
clarkb | or am I misreading what a query-string is in this case? | 23:58 |
* clarkb digs more | 23:58 | |
sdague | clarkb: so I think that's patch #2 | 23:59 |
*** boden has quit IRC | 23:59 | |
sdague | jhesketh: yep, you are correct, that's what I get for listening to people about optimizing :) | 23:59 |
clarkb | sdague: the project filter is missing in yours | 23:59 |
*** boden has joined #openstack-infra | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!