lifeless | cool | 00:00 |
---|---|---|
lifeless | is it worth a test? like - staring at that I'm not sure I could predict that it will work | 00:01 |
lifeless | with our existing stuff | 00:01 |
mordred | lifeless: maybe? I tested that it continues to do what it was doing before by hand -- suggestions on what the test should look like? | 00:02 |
*** yamada-h has quit IRC | 00:02 | |
lifeless | mordred: I dunno. I ag:/ | 00:03 |
lifeless | :/ | 00:03 |
openstackgerrit | Jeremy Stanley proposed openstack-infra/bindep: Add a --version option https://review.openstack.org/178874 | 00:03 |
lifeless | mordred: it looks like we have little-to-no sphinx tests today | 00:03 |
lifeless | mordred: and I know we've had like three or four bugs in a row | 00:03 |
mordred | yah - it would be worth having tests for | 00:04 |
lifeless | mordred: I guess I figure its complex enough that we're in flip-flop mode, and need to start pinning down the user visible scenarios | 00:04 |
*** _nadya_ has joined #openstack-infra | 00:04 | |
openstackgerrit | Eric Brown proposed openstack/requirements: Add Bandit as a global test requirement https://review.openstack.org/167126 | 00:05 |
lifeless | fungi: does bindep already install-require pbr ? | 00:07 |
fungi | lifeless: setup_requires=['pbr'] | 00:08 |
fungi | do i need to add install_requires=['pbr'] in setup.py too? | 00:08 |
lifeless | fungi: in requirements.txt | 00:08 |
lifeless | fungi: just a pbr line | 00:08 |
openstackgerrit | Monty Taylor proposed openstack-dev/pbr: Add translations bones https://review.openstack.org/178879 | 00:08 |
fungi | oh, yep pbr>=0.6,!=0.7,<1.0 | 00:09 |
*** _nadya_ has quit IRC | 00:09 | |
*** sdake has quit IRC | 00:09 | |
mordred | lifeless: I went ahead and fixed the PTI bug ^^ while I was looking at the bug list | 00:09 |
lifeless | cool | 00:10 |
*** cody-somerville has quit IRC | 00:10 | |
mordred | lifeless: it has no functional application - there are no translated strings - but the commands work now | 00:10 |
lifeless | we'll get that in for 0.12 easily | 00:10 |
lifeless | I'm not inclined to rush non-criticals | 00:10 |
*** doude has quit IRC | 00:11 | |
*** otter768 has joined #openstack-infra | 00:12 | |
*** sabeen3 has quit IRC | 00:14 | |
lifeless | fungi: no, pbr reflects stuff into install_requires | 00:14 |
lifeless | fungi: I've reviewed ,2 | 00:14 |
*** smccully has quit IRC | 00:15 | |
fungi | thanks! | 00:15 |
*** otter768 has quit IRC | 00:17 | |
*** smccully has joined #openstack-infra | 00:18 | |
*** sarob_ has quit IRC | 00:18 | |
openstackgerrit | Jeremy Stanley proposed openstack-infra/bindep: Add a --version option https://review.openstack.org/178874 | 00:21 |
*** baoli has joined #openstack-infra | 00:21 | |
fungi | seems to work well | 00:22 |
*** gyee has quit IRC | 00:22 | |
fungi | oh, hrm. getting rid of the from bindep.depends import Depends indirection has exposed an untested regression | 00:24 |
lifeless | oh? | 00:29 |
*** pc_m has joined #openstack-infra | 00:30 | |
*** annegentle has joined #openstack-infra | 00:30 | |
*** annegentle is now known as Guest75589 | 00:30 | |
*** tsg has quit IRC | 00:30 | |
*** Guest75589 is now known as annegent_ | 00:30 | |
fungi | apparently need to import bindep.depends because importing bindep does not | 00:30 |
lifeless | oh | 00:31 |
lifeless | yeah | 00:31 |
openstackgerrit | Jeremy Stanley proposed openstack-infra/bindep: Add a --version option https://review.openstack.org/178874 | 00:31 |
fungi | manual testing shows that ^ works | 00:31 |
fungi | however i guess the testing is a little too synthetic there | 00:31 |
lifeless | actually its really hard to sanely show 'imports the right things' | 00:32 |
lifeless | because sys['modules'] is global state | 00:32 |
*** pc_m_ has joined #openstack-infra | 00:32 | |
*** mtanino has quit IRC | 00:33 | |
fungi | sort of schrödinger's bug situation | 00:33 |
lifeless | package authors have a choice of forcing everything to import -> ends up slow | 00:33 |
fungi | you change the nature of the thing by trying to test it | 00:33 |
lifeless | or having people opt into what youneed | 00:33 |
lifeless | yeah | 00:33 |
lifeless | fungi: also -1 | 00:33 |
lifeless | import foo.bar will import foo and then import foo.bar and bind the result to 'foo' in the current scope | 00:34 |
fungi | importing bindep.depends will cause __init__.py to get processed? okay | 00:34 |
openstackgerrit | Jeremy Stanley proposed openstack-infra/bindep: Add a --version option https://review.openstack.org/178874 | 00:34 |
*** pc_m has quit IRC | 00:35 | |
lifeless | ok https://review.openstack.org/#/c/178852/ is green | 00:35 |
lifeless | I need 2 x +2's please. | 00:35 |
fungi | looking | 00:35 |
lifeless | clarkb: / mordred: if you'd be so kind ^. | 00:36 |
*** camunoz has quit IRC | 00:36 | |
fungi | okay, so it was a lack of apt-get update | 00:36 |
lifeless | yeah | 00:36 |
lifeless | that was all that was needed | 00:36 |
fungi | i thought that looked familiar | 00:36 |
lifeless | devstack not running has shaved nearly 30m off it | 00:36 |
*** packet has joined #openstack-infra | 00:36 | |
lifeless | I'm going to guess at less contention on disk | 00:36 |
lifeless | with background work | 00:37 |
lifeless | and/or CPU | 00:37 |
fungi | yeah, all of the above | 00:37 |
lifeless | uhm, or node variation | 00:37 |
mordred | lifeless: +A | 00:37 |
*** camunoz has joined #openstack-infra | 00:37 | |
fungi | also often the case | 00:37 |
lifeless | last run was 1h in the gate. shrug. its faster. | 00:37 |
*** tiswanso has joined #openstack-infra | 00:40 | |
*** achanda has quit IRC | 00:41 | |
*** mwagner_lap has quit IRC | 00:46 | |
*** packet has quit IRC | 00:46 | |
openstackgerrit | Monty Taylor proposed openstack-infra/shade: Switch tasks vs put on a boolean config flag https://review.openstack.org/178886 | 00:46 |
mordred | SpamapS, clarkb, dtroyer ^^ the os-client-config patch is in service of this | 00:47 |
*** ivar-lazzaro has quit IRC | 00:47 | |
openstackgerrit | Monty Taylor proposed openstack-infra/shade: Switch tasks vs put on a boolean config flag https://review.openstack.org/178886 | 00:47 |
mordred | (added depends-on) | 00:48 |
*** tiswanso has quit IRC | 00:48 | |
*** zhiwei has joined #openstack-infra | 00:48 | |
*** sdake has joined #openstack-infra | 00:48 | |
*** tiswanso has joined #openstack-infra | 00:48 | |
openstackgerrit | Clark Boylan proposed openstack-infra/infra-specs: Perform puppet module functional testing https://review.openstack.org/178887 | 00:52 |
clarkb | mordred: nibalizer asselin_ ^ | 00:52 |
asselin_ | clarkb, cool will take a look | 00:52 |
fungi | speaking of the pbr sphinx entrypoint not being well tested, looks like we pin it <1.2 in pbr's test requirements but we pin global requirements <1.3 instead | 00:52 |
mordred | fungi: we should update that :) | 00:53 |
*** sdake has quit IRC | 00:53 | |
*** alexsyip has quit IRC | 00:53 | |
fungi | this was the culmination of my running down why the build_sphinx was vomiting crazy tracebacks for bindep | 00:54 |
openstackgerrit | Clark Boylan proposed openstack-infra/infra-specs: Perform puppet module functional testing https://review.openstack.org/178887 | 00:55 |
clarkb | that was a silly tyop | 00:55 |
mordred | asselin_: also, I -1'd a patch of yours not beacuse it's bad, but because I really want us to have that spec ^^ before we start growing ad-hoc functional testing | 00:55 |
nibalizer | clarkb: there's a bit of my feedback | 00:56 |
nibalizer | i'll try to circle around again and go deeper | 00:56 |
asselin_ | mordred, yeah, so that. fair enough. the patch itself was really to get the ball rolling on this topic while also being useful....seems it worked. :) | 00:57 |
openstackgerrit | Jeremy Stanley proposed openstack-infra/bindep: Use a pbr-compatible version of Sphinx https://review.openstack.org/178888 | 00:57 |
clarkb | nibalizer: you didn't leave any feedback :) you +1'd then removed the vote | 00:57 |
asselin_ | s/so/saw/ | 00:57 |
*** packet has joined #openstack-infra | 00:57 | |
openstackgerrit | Jeremy Stanley proposed openstack-infra/project-config: Test bindep harder https://review.openstack.org/178890 | 00:57 |
nibalizer | what no there was inline | 00:58 |
clarkb | it will probably come up but we can also use something like ansible running puppet in one container after another if/when we decide that one test.pp is insufficient. I didn't want to get too far down that path though as that should be mostly trivial once we decide on the other details | 00:58 |
*** Rockyg has quit IRC | 00:58 | |
nibalizer | ooooo you snuck patchset 2 and my drafts were on ps1 | 00:59 |
clarkb | nibalizer: ya I had a terrible tyop. | 00:59 |
nibalizer | anyways more on that later | 00:59 |
clarkb | look at the diff for much lols | 00:59 |
nibalizer | i trust you can find comments on ps1 | 00:59 |
nibalizer | not deep analysis hehe | 00:59 |
clarkb | nibalizer: yes I found them | 00:59 |
nibalizer | yay | 00:59 |
*** annegent_ has quit IRC | 01:00 | |
SpamapS | mordred: I don't think you want to dep on 178774 | 01:01 |
*** ivar-lazzaro has joined #openstack-infra | 01:02 | |
SpamapS | mordred: you probably want 178452 | 01:02 |
SpamapS | mordred: I was about to abandon 178774 | 01:02 |
SpamapS | mordred: it's not actually a good approach anyway, since it tries to do 1 and 2 at the same time | 01:02 |
*** ivar-lazzaro has quit IRC | 01:03 | |
*** mwagner_lap has joined #openstack-infra | 01:03 | |
*** Swami has quit IRC | 01:04 | |
mordred | SpamapS: abandon away - I can rebase | 01:04 |
mordred | nibalizer, clarkb I think the point about openstack-puppet and beaker is important - I'd love to align more than not align with them | 01:04 |
mordred | EmilienM: ^^ there is a conversation that would be worth having | 01:04 |
*** ivar-lazzaro has joined #openstack-infra | 01:05 | |
*** ivar-lazzaro has quit IRC | 01:05 | |
mordred | although I do have the opinion that using a tool that's built around assumptions that you want to run vagrant makes me itchy | 01:05 |
*** sigmavirus24 is now known as sigmavirus24_awa | 01:05 | |
mordred | but if we cna run it in a way that makes use of our existing nodes, then can be more ok with it | 01:05 |
clarkb | mordred: well its not just that you want to run vagrant its that the deployment and testing are super coupled | 01:05 |
mordred | clarkb: right | 01:06 |
clarkb | mordred: so you can't for example run your tests against production | 01:06 |
clarkb | which is silly to me | 01:06 |
mordred | clarkb: I agree - that is silly to me too | 01:06 |
*** ivar-lazzaro has joined #openstack-infra | 01:06 | |
clarkb | but I can add it to the options list rather than being down in alternatives | 01:06 |
mordred | clarkb: speaking of: https://review.openstack.org/#/c/177599/ patch to make sure I can run tests against production | 01:06 |
clarkb | since the idea here is to discuss it | 01:06 |
*** pc_m_ has quit IRC | 01:07 | |
clarkb | mordred: nice, reviewing that now | 01:07 |
*** pc_m has joined #openstack-infra | 01:07 | |
mtreinish | mordred: do you have thoughts on: https://review.openstack.org/#/c/164374/ it's been sitting for a while | 01:07 |
mordred | clarkb: I have used it to run shade's functional tests against rackspace | 01:07 |
mordred | mtreinish: looks great | 01:08 |
mtreinish | mordred: heh, at SpamapS's request I tried tempest's limited glance testing against rax this afternoon. Didn't go so well... | 01:08 |
mordred | clarkb: which I think given where it will sit for us is a good thing to have the ability to do | 01:08 |
mordred | mtreinish: heh | 01:09 |
*** bknudson has joined #openstack-infra | 01:09 | |
mtreinish | I wonder what happens when defcore testing comes around for that... | 01:10 |
*** stevemar has joined #openstack-infra | 01:11 | |
mordred | :) | 01:11 |
* hogepodge perks ears up, looks around | 01:11 | |
openstackgerrit | Merged openstack-infra/shade: Add tests and invalidation for glance v2 upload https://review.openstack.org/176024 | 01:11 |
*** markvoelker has joined #openstack-infra | 01:11 | |
hogepodge | mtreinish: I think that there are proponents who want to run shade for defcore | 01:11 |
clarkb | hogepodge: NO | 01:12 |
mordred | oh wow. we're getting popular | 01:12 |
hogepodge | (at least aspects of it) | 01:12 |
mtreinish | hogepodge: yeah, I'm with clarkb on that | 01:12 |
clarkb | hogepodge: because shade is basically working around the problems | 01:12 |
mordred | hogepodge: yeah - shade specifically hides the problems | 01:12 |
* hogepodge makes notes | 01:12 | |
clarkb | hogepodge: instead we should write tests that specifically targets the problems shade works around | 01:12 |
mordred | I think that defcore SHOULD look at shade | 01:12 |
clarkb | hogepodge: like rax keystone returning multiple compute options | 01:12 |
mordred | and anything that shade is working around | 01:12 |
mordred | should not be in defcore | 01:12 |
hogepodge | This is why I'm your friendly neighborhood defcore advocate | 01:12 |
stevemar | clarkb, "working as designed" | 01:13 |
mordred | hogepodge: so - if by "run shade for defcore" you mean "use shade as a source for negative information" - then YES | 01:13 |
SpamapS | hahaha nice | 01:13 |
mordred | stevemar: then it's a bad design | 01:13 |
SpamapS | if your cloud requires shade.. you're not compliant | 01:13 |
stevemar | mordred, forgot my <sarcasm> tags | 01:13 |
hogepodge | oh wait, shade is the compatbility library. I got it confused with the rax testing suite. | 01:13 |
mordred | stevemar: :) | 01:13 |
* hogepodge blushes | 01:13 | |
hogepodge | carry on. | 01:13 |
stevemar | there is so much sass going on | 01:13 |
clarkb | hogepodge: ya shade is basically "here are all the terrible things infra has to work around to use the clouds" lib | 01:13 |
mordred | hogepodge: well, no - please do use shade as a source of "this is where clouds are broken" | 01:14 |
clarkb | hogepodge: but now everyone else can work around them with us \o/ | 01:14 |
mtreinish | hogepodge: the thing I brought up was that rax will not pass the glance tempest tests | 01:14 |
*** rfolco has quit IRC | 01:14 | |
mtreinish | it just returns 413 on all the requests tempest makes | 01:14 |
clarkb | and its a great source of info on where things are bad for compatibility | 01:14 |
* hogepodge may be in SFO and three mules down. | 01:14 | |
mordred | hogepodge: also - please do NOT use the rackspace testing suite | 01:14 |
fungi | or mules | 01:15 |
mordred | hogepodge: beacuse they are, so far, the absolute worst at working like openstack - 95% of shade is there to deal with rackspace being bonghits | 01:15 |
hogepodge | mordred: why? Mostly curious | 01:15 |
mordred | hogepodge: oh, the litany of reason ... | 01:15 |
*** baoli has quit IRC | 01:15 | |
mordred | hogepodge: they are running a glance backend that you are required to interface with that is not landed upstream and works differently than upstream | 01:15 |
hogepodge | We're looking at opening up defcore to tests outside of tempest. The primary use case being swift, but once the gates are open... | 01:15 |
*** baoli has joined #openstack-infra | 01:16 | |
clarkb | hogepodge: swift has a functional test suite | 01:16 |
hogepodge | I have an open patch to start sourcing multiple test repositories. | 01:16 |
mordred | hogepodge: they expose tons of thinsg about their xenserver implementaiton that yo uhave to deal with including image formats that take patched tools | 01:16 |
mordred | hogepodge: their keystone catalog can contain two different compute entries | 01:16 |
*** sarob has quit IRC | 01:17 | |
clarkb | their keystone catalog also uses a different tenant for nova and swift (not sure if thats something that really needs to be enforced but its really weird for a first time user) | 01:17 |
mordred | hogepodge: I can go on - but quite literally if you remove rax from the matrix, all the rest of the clouds work VERY similarly | 01:17 |
openstackgerrit | Merged openstack-infra/shade: Do not cache unsteady state images https://review.openstack.org/177494 | 01:17 |
openstackgerrit | Merged openstack-infra/puppet-subunit2sql: Subscribe worker service to subunit2sql package https://review.openstack.org/164374 | 01:17 |
hogepodge | mordred: good to know | 01:17 |
mordred | hogepodge: so, great as their test suite may be - until they get theit shit together, I don't really want to be in the business of promulgating more of their differences into the world | 01:17 |
hogepodge | mordred: I'm intersted in seeing tests that force everyone to get in line. | 01:17 |
mordred | hogepodge: they exist | 01:17 |
mordred | hogepodge: rax can't pass tempest | 01:18 |
mordred | hogepodge: but I agree with your goal | 01:18 |
mordred | hogepodge: I would also like to humbly request that "run f-ing dhcp for your nodes" be added to the list of required interfaces even though it's not a REST API | 01:18 |
hogepodge | mordred: passing tempest is a very subjective statement. It's a big test suite | 01:18 |
mordred | hogepodge: sure | 01:18 |
mordred | hogepodge: well, it does very poorly at the glance tests :) | 01:19 |
*** mmedvede has quit IRC | 01:19 | |
clarkb | nibalizer: EmilienM are the openstack puppet modules using beaker or beaker-rspec? | 01:19 |
mordred | hogepodge: aren't you glad you showed up? | 01:19 |
hogepodge | mordred: We have lots of vendors passing the defcore tests, which admittedly is kind of a low bar, but there's a reason high jumpers start low. I want to raise it over the next year | 01:19 |
*** achanda has joined #openstack-infra | 01:19 | |
mordred | hogepodge: ++ | 01:19 |
clarkb | nibalizer: EmilienM the puppet wiki says use beaker-rspec for module testing so I am going with that | 01:19 |
hogepodge | mordred: as I said, three mules :-D | 01:19 |
mordred | hogepodge: :) | 01:19 |
mordred | hogepodge: if you want fun reading: http://git.openstack.org/cgit/openstack-infra/shade/tree/shade/__init__.py#n1115 | 01:20 |
hogepodge | I do have to say that so far I like what I've seen in the Kilo release. I've been playing around with the RCs for about a week, and aside from a horizon caching glitch I've seen stability improvements | 01:21 |
mordred | hogepodge: not especially the "this makes me want to die inside" parts | 01:21 |
hogepodge | This is, of course, anecdotal, so salt | 01:21 |
mordred | hogepodge: that's all of the places where an end user needs to do divergent tasks to be able to upload an image | 01:21 |
clarkb | hogepodge: I think several multinode related interactions are now much better now that we test it | 01:21 |
mordred | hogepodge: and relies on a few pieces of a-priori knowledge | 01:21 |
mtreinish | mordred: so on that note are you going to use upstream v2 glance for vanilla cloud? | 01:21 |
clarkb | hogepodge: a bunch of live migration and shelve bugs were fixed | 01:21 |
mordred | mtreinish: well, I've learned something | 01:21 |
mordred | mtreinish: which is v1 vs. v2 is not important | 01:22 |
clarkb | but ipv6 is broken with floating IPs, I should check in on those changes and see if we can backport yet | 01:22 |
mordred | mtreinish: v2 also has PUT for upoading images | 01:22 |
mordred | mtreinish: so for infra-cloud, we could run the v2 api but _not_ disable image upload | 01:22 |
hogepodge | clarkb: maybe we can sit down during or after the summit and talk about what shade is doing. I'd like to take it to the defcore committee and show what infra and qa is up against | 01:22 |
mordred | mtreinish: or we could run v1 and say meh | 01:22 |
mtreinish | yeah, that's what the tempest tests do. The json semantics are just a bit different | 01:22 |
clarkb | hogepodge: sure | 01:22 |
mordred | mtreinish: yeah - and I already have handling of the update_properties differences | 01:23 |
hogepodge | Have to pack up. Have a flight boarding in 5 minutes. | 01:23 |
mordred | hogepodge: enjoy! | 01:23 |
hogepodge | SFO -> PDX easy Zzzz | 01:23 |
hogepodge | thanks mordred | 01:23 |
*** dizquierdo_ has quit IRC | 01:24 | |
clarkb | sweet https://review.openstack.org/#/c/160856/ did merge which means the nova fix merged | 01:26 |
clarkb | mtreinish: ^ this means you can start writing some ipv6 tests into tempest, but keep in mind we also need to update cirros if the VMs are involved | 01:26 |
mtreinish | clarkb: there are already a bunch of ipv6 tests in tempest | 01:27 |
clarkb | mtreinish: they would not have worked prior to ^ aiui | 01:27 |
mtreinish | I do not know how they ran, but there are a bunch and they've apparently been running successfully | 01:27 |
*** ddieterly has quit IRC | 01:28 | |
mtreinish | clarkb: for example http://git.openstack.org/cgit/openstack/tempest/tree/tempest/scenario/test_network_v6.py | 01:28 |
clarkb | mtreinish: http://git.openstack.org/cgit/openstack/tempest/tree/tempest/scenario/test_network_v6.py#n41 would've skipped them prior to 160856 I think | 01:29 |
mtreinish | http://logs.openstack.org/86/176786/2/check/check-tempest-dsvm-neutron-full/d4bd986/console.html#_2015-04-29_01_07_27_102 | 01:29 |
clarkb | mtreinish: ya thats new enough to have had ipv6 enabled | 01:29 |
openstackgerrit | Monty Taylor proposed openstack-infra/shade: Switch tasks vs put on a boolean config flag https://review.openstack.org/178886 | 01:29 |
openstackgerrit | Merged openstack-infra/shade: Fix functional tests to run against live clouds https://review.openstack.org/177599 | 01:29 |
mtreinish | clarkb: from march: http://logs.openstack.org/66/166666/4/check/check-tempest-dsvm-neutron-full/f300879/console.html.gz#_2015-03-26_02_45_55_603 | 01:30 |
*** tqtran has quit IRC | 01:30 | |
mtreinish | clarkb: that flag defaults true I don't think devstack has the proper logic for setting it | 01:30 |
mtreinish | let me check | 01:30 |
clarkb | mtreinish: huh, see the change above its devstack changing the flag | 01:30 |
*** pc_m has quit IRC | 01:30 | |
clarkb | mtreinish: also when we turned on that flag a bunch of things failed, (see the depends on) one of the things was specifically ipv4 floating ip addrs which that test claims to check | 01:31 |
*** pc_m has joined #openstack-infra | 01:31 | |
*** ddieterly has joined #openstack-infra | 01:31 | |
mtreinish | clarkb: hmm yeah devstack sets that flag based on devstack's $IPV6_ENABLED which is true: http://logs.openstack.org/66/166666/4/check/check-tempest-dsvm-neutron-full/f300879/logs/tempest_conf.txt.gz | 01:31 |
openstackgerrit | Clark Boylan proposed openstack-infra/infra-specs: Perform puppet module functional testing https://review.openstack.org/178887 | 01:32 |
clarkb | mordred: ^ addressed your comments and nibalizer's | 01:32 |
clarkb | mtreinish: you mean tempest sets that flag based on devstack's ipv6 enabled? | 01:32 |
mordred | clarkb: woot | 01:32 |
clarkb | mtreinish: I wonder if this is a case where many ipv6 things work with neutron just as is because, and others do not | 01:34 |
mtreinish | clarkb: devstack sets that flag in the tempest config file which devstack is generatinghttp://git.openstack.org/cgit/openstack-dev/devstack/tree/lib/tempest#n374 | 01:34 |
mtreinish | oh, well that's actually that gets defined above in: http://git.openstack.org/cgit/openstack-dev/devstack/tree/lib/tempest#n81 | 01:34 |
clarkb | mtreinish: grepping says that IPV6_ENABLED isn't otherwise set so it just goes to the default of True | 01:35 |
clarkb | mtreinish: so tempest thinks its good but not all the neutron config happens? I am looking at what changes in neutron in devstack with the change bove | 01:36 |
mtreinish | clarkb: yeah it looks like devstack is telling tempest v6 is true, regardless of whether devstack is configuring neutron for v6 or not | 01:37 |
clarkb | mtreinish: yup, and the neutron side of the config is just setting up networks and routers, so if tempest does that for itself it would work | 01:37 |
clarkb | and now I know why only the pxe job for ironic failed, because they don't tempest iirc | 01:37 |
*** weshay has quit IRC | 01:38 | |
*** unicell has quit IRC | 01:38 | |
mtreinish | clarkb: yep tempest will create the things it needs as it dynamically creates isolated tenants for each test class | 01:39 |
*** alex_xu has quit IRC | 01:43 | |
*** pc_m has quit IRC | 01:46 | |
*** pc_m has joined #openstack-infra | 01:46 | |
*** dims has joined #openstack-infra | 01:48 | |
*** erikmwilson is now known as Guest79435 | 01:49 | |
*** erikmwilson has joined #openstack-infra | 01:49 | |
*** tsg has joined #openstack-infra | 01:50 | |
*** SumitNaiksatam has joined #openstack-infra | 01:52 | |
*** stevemar has quit IRC | 01:52 | |
openstackgerrit | Angus Salkeld proposed openstack-infra/project-config: Add missing ceilometer services to heat job https://review.openstack.org/178472 | 01:52 |
*** patrickeast has quit IRC | 01:52 | |
*** packet has quit IRC | 01:58 | |
*** pc_m has quit IRC | 02:01 | |
*** bhunter71 has quit IRC | 02:01 | |
*** SumitNaiksatam has quit IRC | 02:03 | |
openstackgerrit | Monty Taylor proposed openstack-infra/shade: Switch tasks vs put on a boolean config flag https://review.openstack.org/178886 | 02:04 |
*** yamada-h has joined #openstack-infra | 02:05 | |
*** tsg has quit IRC | 02:05 | |
*** smccully has quit IRC | 02:06 | |
*** baoli has quit IRC | 02:07 | |
*** baoli has joined #openstack-infra | 02:07 | |
*** jtriley has joined #openstack-infra | 02:10 | |
*** yamahata has quit IRC | 02:12 | |
*** otter768 has joined #openstack-infra | 02:13 | |
*** teran has quit IRC | 02:16 | |
*** signed8bit has joined #openstack-infra | 02:17 | |
*** otter768 has quit IRC | 02:18 | |
openstackgerrit | Merged openstack-infra/shade: Update flavor API for new get/list/search API https://review.openstack.org/176931 | 02:23 |
openstackgerrit | Merged openstack-infra/shade: Update secgroup API for new get/list/search API https://review.openstack.org/178807 | 02:24 |
*** stevemar has joined #openstack-infra | 02:25 | |
*** tsg has joined #openstack-infra | 02:25 | |
*** jtriley has quit IRC | 02:25 | |
*** ddieterl_ has joined #openstack-infra | 02:26 | |
*** ddieterly has quit IRC | 02:26 | |
*** annegent_ has joined #openstack-infra | 02:29 | |
*** dims has quit IRC | 02:31 | |
*** bhunter71 has joined #openstack-infra | 02:32 | |
openstackgerrit | Ian Wienand proposed openstack-infra/devstack-gate: Add errexit to setup & cleanup functions https://review.openstack.org/177017 | 02:34 |
openstackgerrit | Ian Wienand proposed openstack-infra/devstack-gate: Remove pip cache move https://review.openstack.org/178902 | 02:34 |
openstackgerrit | Ian Wienand proposed openstack-infra/devstack-gate: Make sysctl vm dump errexit safe https://review.openstack.org/178903 | 02:34 |
openstackgerrit | Ian Wienand proposed openstack-infra/devstack-gate: Check for mount before creating swap https://review.openstack.org/178904 | 02:34 |
openstackgerrit | Ian Wienand proposed openstack-infra/devstack-gate: Return command status in tsfilter https://review.openstack.org/178905 | 02:34 |
*** fifieldt has joined #openstack-infra | 02:36 | |
*** stevemar has quit IRC | 02:38 | |
*** ir2ivps has quit IRC | 02:39 | |
*** ir2ivps has joined #openstack-infra | 02:41 | |
*** yamada-h has quit IRC | 02:41 | |
openstackgerrit | Ian Wienand proposed openstack-infra/project-config: Remove update-git-cache.sh https://review.openstack.org/178906 | 02:42 |
*** yamada-h has joined #openstack-infra | 02:43 | |
*** yamahata has joined #openstack-infra | 02:43 | |
*** alex_xu has joined #openstack-infra | 02:44 | |
*** stevemar has joined #openstack-infra | 02:48 | |
*** heyongli has quit IRC | 02:49 | |
*** btully has quit IRC | 02:49 | |
*** otter768 has joined #openstack-infra | 02:50 | |
*** baoli has quit IRC | 02:51 | |
*** heyongli has joined #openstack-infra | 02:51 | |
*** baoli has joined #openstack-infra | 02:52 | |
*** baoli_ has joined #openstack-infra | 02:53 | |
*** baoli has quit IRC | 02:53 | |
*** harlowja has quit IRC | 02:56 | |
*** harlowja has joined #openstack-infra | 02:56 | |
*** harlowja has quit IRC | 02:57 | |
*** camunoz has quit IRC | 02:59 | |
*** sdake has joined #openstack-infra | 02:59 | |
*** annegent_ has quit IRC | 03:00 | |
*** yamada-h has quit IRC | 03:01 | |
*** camunoz has joined #openstack-infra | 03:02 | |
*** sdake_ has joined #openstack-infra | 03:02 | |
*** sdake has quit IRC | 03:02 | |
*** Krinkle is now known as Krinkle|detached | 03:10 | |
*** erlon has quit IRC | 03:11 | |
*** ashleighfarnham has quit IRC | 03:14 | |
*** zz_dimtruck is now known as dimtruck | 03:16 | |
*** packet has joined #openstack-infra | 03:20 | |
*** jtriley has joined #openstack-infra | 03:21 | |
*** tiswanso has quit IRC | 03:26 | |
*** jtriley has quit IRC | 03:26 | |
*** harlowja has joined #openstack-infra | 03:27 | |
*** bhunter71 has quit IRC | 03:33 | |
*** tsg has quit IRC | 03:34 | |
*** baoli_ has quit IRC | 03:36 | |
openstackgerrit | Qiming Teng proposed openstack/requirements: Bump python-openstacksdk version to 0.4.1 https://review.openstack.org/178923 | 03:37 |
*** SumitNaiksatam has joined #openstack-infra | 03:39 | |
*** otter768 has quit IRC | 03:44 | |
*** tjones1 has joined #openstack-infra | 03:45 | |
*** tjones1 has left #openstack-infra | 03:46 | |
*** fedexo has joined #openstack-infra | 03:46 | |
*** sdake_ has quit IRC | 03:52 | |
*** tsg has joined #openstack-infra | 03:57 | |
*** spzala has quit IRC | 04:00 | |
*** camunoz has quit IRC | 04:01 | |
*** camunoz has joined #openstack-infra | 04:01 | |
*** unicell has joined #openstack-infra | 04:05 | |
*** achanda has quit IRC | 04:11 | |
*** unicell1 has joined #openstack-infra | 04:14 | |
*** unicell has quit IRC | 04:15 | |
*** sdake has joined #openstack-infra | 04:24 | |
*** packet has quit IRC | 04:26 | |
lifeless | ttx: one thing you could do to do things much much faster | 04:32 |
lifeless | ttx: create a threadpool and do the assign + save for each task in a separate worker | 04:32 |
lifeless | ttx: concurrency of 20-40 should be ok | 04:32 |
*** ddieterl_ has quit IRC | 04:36 | |
openstackgerrit | Bharat Kumar Kobagana proposed openstack-infra/project-config: Skip test_volume_boot_pattern for GlusterFS until bug 1444806 is fixed https://review.openstack.org/173408 | 04:36 |
openstack | bug 1444806 in Cinder "test_volume_boot_pattern tempest test failure for glusterfs backend - Part 2" [Undecided,In progress] https://launchpad.net/bugs/1444806 - Assigned to Bharat Kumar Kobagana (bharat-kobagana) | 04:36 |
openstackgerrit | Merged openstack-dev/pbr: Issue #1450210: Preversioning rc tag handling https://review.openstack.org/178852 | 04:48 |
*** ajo has joined #openstack-infra | 04:49 | |
lifeless | yayayayya | 04:53 |
lifeless | we're now in shape | 04:53 |
*** ildikov has quit IRC | 04:54 | |
*** signed8bit has quit IRC | 05:02 | |
openstackgerrit | Merged openstack-infra/project-config: Add manila-ui to OpenStack https://review.openstack.org/175063 | 05:05 |
*** ildikov has joined #openstack-infra | 05:07 | |
*** yfried has quit IRC | 05:07 | |
*** achanda has joined #openstack-infra | 05:16 | |
*** jtriley has joined #openstack-infra | 05:16 | |
*** ivar-laz_ has joined #openstack-infra | 05:17 | |
*** ivar-laz_ has quit IRC | 05:17 | |
*** ibiris is now known as ibiris_away | 05:18 | |
*** emagana has joined #openstack-infra | 05:19 | |
*** ivar-lazzaro has quit IRC | 05:21 | |
*** openstackgerrit has quit IRC | 05:22 | |
*** openstackgerrit has joined #openstack-infra | 05:22 | |
*** harlowja is now known as harlowja_away | 05:23 | |
*** jtriley has quit IRC | 05:24 | |
*** jamespage_ has joined #openstack-infra | 05:25 | |
*** esker has quit IRC | 05:28 | |
openstackgerrit | Clark Boylan proposed openstack-infra/gearman-plugin: Fix a cause of livelock in worker work() method https://review.openstack.org/178929 | 05:28 |
clarkb | zaro: ^ thats a first pass at something, but basically I think we need to be driving IO as long as those two conditions hold | 05:29 |
*** btully has joined #openstack-infra | 05:29 | |
clarkb | we might also consider rewriting the state machine entirely such that it only ever processes one thing at a time then drives IO unconditionally | 05:29 |
clarkb | something like input state -> event -> output state (optionally submitting a task) -> drive IO | 05:30 |
openstackgerrit | lifeless proposed openstack-infra/release-tools: Possible parallel version. https://review.openstack.org/178930 | 05:32 |
lifeless | ttx: https://review.openstack.org/178930 may help you. Or it may make your life terrible. | 05:32 |
lifeless | ttx: its also entirely untested | 05:33 |
*** esker has joined #openstack-infra | 05:33 | |
*** dannywilson has joined #openstack-infra | 05:33 | |
*** dannywilson has quit IRC | 05:33 | |
*** dannywilson has joined #openstack-infra | 05:34 | |
clarkb | lifeless: those are the best changes "either your life will be amazing now or it will be very bad, good luck" | 05:35 |
*** ddieterly has joined #openstack-infra | 05:37 | |
StevenK | I'd recommend starting with 5 and ramping up, but eh | 05:37 |
*** jamespage_ has quit IRC | 05:38 | |
*** abregman has joined #openstack-infra | 05:40 | |
*** ddieterly has quit IRC | 05:41 | |
lifeless | StevenK: last I recall there were 120 service points on LP | 05:41 |
lifeless | StevenK: the stuff i'm worried about is going to kick in at c=2 | 05:41 |
lifeless | StevenK: its either going to break very very hard, or be great. | 05:42 |
StevenK | Oh, could be, I thought there were still 80 | 05:42 |
lifeless | same same | 05:42 |
lifeless | 40 << 80 | 05:42 |
lifeless | 4 24 core machines, running at slightly more than one process per core | 05:42 |
tchaypo | I’m impressed with how many of pip’s functional tests are failing on my machine | 05:43 |
lifeless | I still can't help but read preversion as perversion | 05:43 |
lifeless | tchaypo: they should all pass, assuming your internet actually exists | 05:44 |
lifeless | tchaypo: sadly many of them hit Das Net | 05:44 |
tchaypo | ah. I’m currently tethered. I think they should be allowed out, but perhaps they have too-short timeouts | 05:45 |
openstackgerrit | Ian Wienand proposed openstack-infra/project-config: Parameterize node to dsvm-full-plugin job template https://review.openstack.org/175625 | 05:45 |
openstackgerrit | Ian Wienand proposed openstack-infra/project-config: Add glusterfs/centos7 experimental job https://review.openstack.org/175626 | 05:45 |
lifeless | tchaypo: also, if you're working of my branch its possible some things are broken | 05:45 |
clarkb | lifeless: yes preversion reads as perversion for me too | 05:45 |
lifeless | tchaypo: I only check one or two before pushing up to let travis do it for me | 05:45 |
*** otter768 has joined #openstack-infra | 05:45 | |
*** ildikov has quit IRC | 05:45 | |
tchaypo | oh look | 05:46 |
lifeless | tchaypo: (if you're not basing off my branch, and you cause me merge conflicts, I will be unhappy at your adding work) | 05:46 |
tchaypo | I have a rule that prevents python from talking to pypi | 05:46 |
tchaypo | lifeless: nope, this is master | 05:46 |
lifeless | tchaypo: well, be ready for grumpy lifeless then, if it his is the union thing. | 05:47 |
lifeless | tchaypo: also, doing anything on master is pointless; 'develop' is the next release of pip. | 05:47 |
lifeless | master is for point releases | 05:47 |
tchaypo | lifeless: I don’t expect the union thing to be useful | 05:48 |
tchaypo | *checks* yep, develop is what I meant | 05:48 |
ianw | so http://logs.openstack.org/04/178904/1/check/gate-tempest-dsvm-large-ops/cc34726/logs/devstack-gate-setup-workspace-new.txt.gz | 05:48 |
ianw | 2015-04-30 02:46:30.383 | + git fetch http://zm06.openstack.org/p/openstack/nova refs/zuul/master/Z33d054bb372d4d529ef79b840915c27e | 05:49 |
ianw | 2015-04-30 03:06:38.242 | fatal: Couldn't find remote ref refs/zuul/master/Z33d054bb372d4d529ef79b840915c27e | 05:49 |
lifeless | tchaypo: I don't either, because it won't affect what is chosen something like 50% of the time | 05:49 |
*** claudiub has joined #openstack-infra | 05:49 | |
tchaypo | If I do the union thing it will be with the expectation that the code is going to be thrown away once we realise that it isnt at all useful | 05:49 |
*** dhritishikhar has joined #openstack-infra | 05:49 | |
ianw | ^ why so long? | 05:49 |
lifeless | tchaypo: (I gave an example of that in the thread with nicola | 05:49 |
lifeless | tchaypo: what are you hacking on then? | 05:49 |
tchaypo | and the times when it does affect what’s chosen, the most it can do is tell the user that there’s a paradox - it can’t even give hints about how to solve the problem, let alone solve it | 05:50 |
tchaypo | right now I’m just making sure I have a working test environment before I start hacking | 05:50 |
*** otter768 has quit IRC | 05:50 | |
lifeless | cool | 05:51 |
StevenK | Oh goody, I think I crashed zanata | 05:54 |
tchaypo | oh | 05:54 |
tchaypo | one thing that I do think might be useful is to look to see if we have tests that check pip’s behaviour with transitive dependencies (I’m fairly sure we don’t, since pip doesn’t handle them) and add some tests (marked as expected-to-fail for now) | 05:55 |
clarkb | ianw: its possible canonical has DOSed the git mirrors again | 05:55 |
* clarkb looks at cacti | 05:55 | |
lifeless | tchaypo: it handles transitive deps today | 05:55 |
*** ibiris_away is now known as ibiris | 05:56 | |
*** lucap has joined #openstack-infra | 05:57 | |
tchaypo | more specifically, the behaviour if a the same name is encountered >1 time with different version specifiers | 05:57 |
ianw | clarkb: http://cacti.openstack.org/cacti/graph_view.php?action=tree&tree_id=1&leaf_id=140 <- doesn't keep load stats? | 05:57 |
tchaypo | since pip currently just ignores subsequent specifiers I’m assuming there are not tests on what it’s doing | 05:57 |
* tchaypo except for maybe a test to check that it ignores subsequent specifiers.. | 05:57 | |
lifeless | tchaypo: there aren't tests that it resolves :) | 05:57 |
clarkb | ianw: see http://cacti.openstack.org/cacti/graph_view.php?action=tree&tree_id=1&leaf_id=140&page=2 | 05:58 |
lifeless | tchaypo: so thats certainly something that will conflict with stuff coming into my branch | 05:58 |
clarkb | ianw: the mirrors themselves look ok so this is isolated to the merger | 05:58 |
* clarkb goes to get key | 05:58 | |
lifeless | tchaypo: I'd really rather that such things be proprosed into my branch rather than in parallel to develop | 05:59 |
clarkb | ianw: we have seen it where one gearman worker will grab most of the merge jobs then get unhappy as it does mroe work | 05:59 |
tchaypo | yep, that makes sense | 05:59 |
clarkb | ianw: I can check for that easily once I get my ssh key | 05:59 |
tchaypo | but what do you mean by “that”? | 05:59 |
ianw | clarkb: http://cacti.openstack.org/cacti/graph_view.php?action=tree&tree_id=1&leaf_id=140&page=3 <-- suspicious swapping activity around that time | 05:59 |
ianw | http://cacti.openstack.org/cacti/graph.php?action=view&local_graph_id=2160&rra_id=all <- little blank spot, right around the 02:30 -> 03:00 mark ... something went a bit nuts i guess | 06:01 |
openstackgerrit | OpenStack Proposal Bot proposed openstack-infra/project-config: Normalize projects.yaml https://review.openstack.org/178938 | 06:01 |
*** mrunge has joined #openstack-infra | 06:03 | |
*** mrunge has quit IRC | 06:03 | |
*** mrunge has joined #openstack-infra | 06:03 | |
*** deepakcs has joined #openstack-infra | 06:04 | |
clarkb | 06 has actually had the least jobs which makes sense if it went sideways for a bit (gearman is supposed to be self regulating that way) | 06:04 |
clarkb | time to look at logs for that specific time period | 06:04 |
*** rbradfor has quit IRC | 06:05 | |
clarkb | ianw: that is a significant amount of iowait too, but may be due to swapping | 06:06 |
*** armax has quit IRC | 06:08 | |
clarkb | wow each nova merge is taking close to a minute | 06:08 |
clarkb | wow found one that is >7 minutes | 06:09 |
clarkb | now is that the cause or a symptom of this problem | 06:09 |
clarkb | syslog shows no oom | 06:10 |
ianw | it probably hit swap and got itself out of it ... stopping everything else in the process | 06:11 |
clarkb | ya I wonder if we should swappoff on those nodes or use cgorups to limit memory for the merger processes that way we kill one merge and not all of them | 06:11 |
clarkb | it does only have 2GB of ram but most of that is currently free or cache | 06:12 |
*** hdd has joined #openstack-infra | 06:12 | |
clarkb | I suppose it is possible that we need >2GB of ram to do some of the merges that show up | 06:13 |
clarkb | and just need to scale up | 06:13 |
*** arxcruz has joined #openstack-infra | 06:16 | |
tchaypo | lifeless: if you mean “tests that it ignores subsequent specifiers”, I can see how that would conflict, but I’d assume that a test to see that it’s resolving all the transitive deps shouldn’t conflict with any implemementation of a resolver (at least for simple cases ) | 06:17 |
lifeless | tchaypo: so you're expecting me to write a resolver without tests? | 06:18 |
lifeless | tchaypo: or you're expecting me to put those tests somewhere other than your test of the same thing? | 06:18 |
tchaypo | only if the tests don’t already exist | 06:19 |
lifeless | tchaypo: but they don't today | 06:19 |
tchaypo | yep. and if we both write one that’d be a conflict. | 06:19 |
lifeless | tchaypo: ... | 06:19 |
tchaypo | i understand what you meant now :) | 06:20 |
*** arxcruz has quit IRC | 06:21 | |
*** ildikov has joined #openstack-infra | 06:22 | |
*** hdd has quit IRC | 06:23 | |
*** erikmwilson has quit IRC | 06:24 | |
clarkb | apache logs look mostly fine too, the only thing I notice there is nova is transferring an order of magnitude more data than the other repos, but its megabytes vs kilobytes per request nothing too crazy | 06:24 |
*** dimtruck is now known as zz_dimtruck | 06:24 | |
openstackgerrit | Merged openstack-infra/project-config: Normalize projects.yaml https://review.openstack.org/178938 | 06:25 |
*** fawadkhaliq has joined #openstack-infra | 06:27 | |
*** mrmartin has joined #openstack-infra | 06:28 | |
*** spredzy|afk is now known as spredzy | 06:30 | |
*** tnovacik has joined #openstack-infra | 06:31 | |
*** Hal has joined #openstack-infra | 06:32 | |
*** Hal is now known as Guest8312 | 06:33 | |
*** shivharis has quit IRC | 06:33 | |
*** nfedotov has joined #openstack-infra | 06:33 | |
*** vlaza has joined #openstack-infra | 06:33 | |
*** shivharis has joined #openstack-infra | 06:34 | |
*** gary-smith has quit IRC | 06:34 | |
*** gary-smith has joined #openstack-infra | 06:34 | |
*** AJaeger_ has joined #openstack-infra | 06:35 | |
*** ssam2 has joined #openstack-infra | 06:35 | |
*** ddieterly has joined #openstack-infra | 06:37 | |
*** Guest8312 has quit IRC | 06:38 | |
*** camunoz has quit IRC | 06:39 | |
*** rbradfor has joined #openstack-infra | 06:41 | |
*** ddieterly has quit IRC | 06:43 | |
*** e0ne has joined #openstack-infra | 06:45 | |
*** stevemar has quit IRC | 06:46 | |
*** dtantsur|afk is now known as dtantsur | 06:48 | |
*** kevinbenton has quit IRC | 06:49 | |
*** sks has joined #openstack-infra | 06:49 | |
*** kevinbenton has joined #openstack-infra | 06:50 | |
*** Longgeek has joined #openstack-infra | 06:52 | |
*** zul has joined #openstack-infra | 06:53 | |
*** sushilkm has joined #openstack-infra | 06:55 | |
*** sushilkm has left #openstack-infra | 06:55 | |
*** zhiwei has quit IRC | 06:56 | |
*** scheuran has joined #openstack-infra | 06:58 | |
tchaypo | OSError: [Errno 2] No such file or directory: '/private/var/folders/_n/ft3sz29979590_nj7tj1dxhm0000gn/T/pytest-7/test_as_import0/home/Library/Caches/pip' | 07:00 |
tchaypo | I’m just going to restrict myself to testing pip on real computers | 07:00 |
*** HeOS has joined #openstack-infra | 07:01 | |
*** tsg has quit IRC | 07:02 | |
*** dtantsur has quit IRC | 07:02 | |
*** jcoufal has joined #openstack-infra | 07:03 | |
clarkb | ttx: everything is looking good from my end | 07:03 |
ttx | clarkb: awesome thx! | 07:03 |
clarkb | ttx: did you want to kick off the first release things with me still awake? | 07:03 |
ttx | clarkb: nah, should be fine, thx | 07:03 |
lifeless | tchaypo: where was that? | 07:03 |
tchaypo | lifeless: on a pre-release OSX version | 07:04 |
*** tsg_ has joined #openstack-infra | 07:05 | |
*** MaxV has joined #openstack-infra | 07:06 | |
*** AJaeger_ has quit IRC | 07:06 | |
*** dtantsur has joined #openstack-infra | 07:07 | |
*** Longgeek has quit IRC | 07:07 | |
*** afazekas has quit IRC | 07:08 | |
*** jamespage_ has joined #openstack-infra | 07:08 | |
*** yfried has joined #openstack-infra | 07:08 | |
*** mpaolino has joined #openstack-infra | 07:09 | |
*** e0ne has quit IRC | 07:09 | |
*** mpaolino has quit IRC | 07:10 | |
*** hdd has joined #openstack-infra | 07:10 | |
*** nfedotov1 has joined #openstack-infra | 07:10 | |
*** zul has quit IRC | 07:12 | |
*** nfedotov has quit IRC | 07:12 | |
*** zul has joined #openstack-infra | 07:12 | |
*** shivharis has quit IRC | 07:12 | |
lifeless | tchaypo: btw #pypa-dev for pip discussion | 07:13 |
*** vhoward has quit IRC | 07:13 | |
*** shivharis has joined #openstack-infra | 07:13 | |
*** dannywilson has quit IRC | 07:13 | |
*** sergsh has joined #openstack-infra | 07:14 | |
*** ociuhandu has joined #openstack-infra | 07:14 | |
*** mrmartin has quit IRC | 07:14 | |
*** Ala has joined #openstack-infra | 07:15 | |
*** vhoward has joined #openstack-infra | 07:15 | |
*** Longgeek has joined #openstack-infra | 07:22 | |
*** alexpilotti has joined #openstack-infra | 07:23 | |
*** teran has joined #openstack-infra | 07:25 | |
*** devvesa has joined #openstack-infra | 07:29 | |
*** mrmartin has joined #openstack-infra | 07:29 | |
*** mrmartin has quit IRC | 07:30 | |
*** jamespage_ has quit IRC | 07:30 | |
*** Hal has joined #openstack-infra | 07:31 | |
*** Hal is now known as Guest73021 | 07:31 | |
*** tsg_ has quit IRC | 07:33 | |
*** ddieterly has joined #openstack-infra | 07:38 | |
*** mrmartin has joined #openstack-infra | 07:39 | |
*** jlanoux has joined #openstack-infra | 07:40 | |
*** mpavone has joined #openstack-infra | 07:40 | |
*** yfried is now known as yfried|afk | 07:41 | |
*** chlong has quit IRC | 07:42 | |
*** mrmartin has quit IRC | 07:42 | |
*** nfedotov has joined #openstack-infra | 07:43 | |
*** nfedotov1 has quit IRC | 07:43 | |
*** ddieterly has quit IRC | 07:43 | |
*** yfried|afk is now known as yfried | 07:44 | |
*** otter768 has joined #openstack-infra | 07:46 | |
*** dhritishikhar has quit IRC | 07:47 | |
*** woodster_ has quit IRC | 07:50 | |
*** otter768 has quit IRC | 07:51 | |
*** fawadkhaliq has quit IRC | 07:52 | |
*** mase_x200 has joined #openstack-infra | 07:53 | |
*** hdd has quit IRC | 07:54 | |
*** HeOS has quit IRC | 07:55 | |
*** mase_x200 has quit IRC | 07:57 | |
*** mrmartin has joined #openstack-infra | 07:59 | |
*** mase_x200 has joined #openstack-infra | 08:01 | |
*** jistr has joined #openstack-infra | 08:04 | |
*** notnownikki has joined #openstack-infra | 08:05 | |
*** btully has quit IRC | 08:07 | |
*** dalgaaf has joined #openstack-infra | 08:09 | |
*** derekh has joined #openstack-infra | 08:10 | |
openstackgerrit | Thomas Bechtold proposed openstack/requirements: Relax Pillow requirement https://review.openstack.org/178970 | 08:12 |
*** mase_x200 has quit IRC | 08:13 | |
*** nfedotov1 has joined #openstack-infra | 08:16 | |
openstackgerrit | Thomas Bechtold proposed openstack/requirements: Relax Pillow requirement https://review.openstack.org/178970 | 08:17 |
*** nfedotov has quit IRC | 08:19 | |
openstackgerrit | Joshua Hesketh proposed openstack-infra/jenkins-job-builder: Adding publisher support to join-trigger plugin https://review.openstack.org/178836 | 08:20 |
*** jcoufal_ has joined #openstack-infra | 08:21 | |
*** erikmwilson has joined #openstack-infra | 08:25 | |
*** jcoufal has quit IRC | 08:25 | |
*** nfedotov1 has quit IRC | 08:26 | |
*** nfedotov has joined #openstack-infra | 08:26 | |
*** erikmwilson has quit IRC | 08:29 | |
*** arxcruz has joined #openstack-infra | 08:29 | |
*** niska has quit IRC | 08:32 | |
*** ajo has quit IRC | 08:34 | |
*** ajo_ has joined #openstack-infra | 08:34 | |
*** e0ne has joined #openstack-infra | 08:37 | |
*** zul has quit IRC | 08:38 | |
openstackgerrit | Renat Akhmerov proposed openstack-infra/project-config: Rename stackforge/mistral to openstack/mistral https://review.openstack.org/175328 | 08:38 |
*** ddieterly has joined #openstack-infra | 08:39 | |
*** ddieterly has quit IRC | 08:43 | |
*** salv-orl_ has joined #openstack-infra | 08:45 | |
*** salv-orlando has quit IRC | 08:46 | |
*** belmoreira has joined #openstack-infra | 08:46 | |
*** shardy has joined #openstack-infra | 08:47 | |
*** che-arne has joined #openstack-infra | 08:47 | |
*** niska has joined #openstack-infra | 08:48 | |
*** dizquierdo has joined #openstack-infra | 08:48 | |
jeblair | ttx: semi-online if you need anything | 08:51 |
ttx | jeblair: doing the first tag, things going well at this point | 08:52 |
ttx | jeblair: thx! | 08:52 |
openstackgerrit | Andrew Smith proposed openstack-infra/jenkins-job-builder: Add support for docker builder plugin https://review.openstack.org/178980 | 08:59 |
*** achanda has quit IRC | 09:01 | |
*** jamespage_ has joined #openstack-infra | 09:02 | |
*** fedexo has quit IRC | 09:03 | |
openstackgerrit | lifeless proposed openstack-dev/pbr: Parallelise integration tests. https://review.openstack.org/178982 | 09:04 |
*** doude has joined #openstack-infra | 09:05 | |
*** dtantsur is now known as dtantsur|brb | 09:06 | |
*** jtriley has joined #openstack-infra | 09:07 | |
*** HeOS has joined #openstack-infra | 09:09 | |
*** maishsk has joined #openstack-infra | 09:12 | |
*** jtriley has quit IRC | 09:13 | |
*** teran has quit IRC | 09:16 | |
*** Somay has joined #openstack-infra | 09:19 | |
*** mrmartin has quit IRC | 09:22 | |
*** dizquierdo has quit IRC | 09:23 | |
*** jamespage_ has quit IRC | 09:31 | |
*** jamespage_ has joined #openstack-infra | 09:33 | |
openstackgerrit | Claudiu Belu proposed openstack-infra/project-config: Adds pypi-jobs to networking-hyperv https://review.openstack.org/178993 | 09:33 |
*** fawadkhaliq has joined #openstack-infra | 09:34 | |
*** claudiub has quit IRC | 09:36 | |
*** jamespage_ has quit IRC | 09:36 | |
openstackgerrit | Samuel BARTEL proposed openstack-infra/project-config: add project fuel-plugin-tls https://review.openstack.org/178995 | 09:37 |
*** zul has joined #openstack-infra | 09:38 | |
*** emagana has quit IRC | 09:40 | |
*** dizquierdo has joined #openstack-infra | 09:41 | |
*** Somay has quit IRC | 09:43 | |
*** Somay has joined #openstack-infra | 09:44 | |
*** otter768 has joined #openstack-infra | 09:47 | |
*** e0ne is now known as e0ne_ | 09:48 | |
*** otter768 has quit IRC | 09:52 | |
*** dims has joined #openstack-infra | 09:56 | |
*** pelix has joined #openstack-infra | 09:57 | |
*** MaxV has quit IRC | 09:57 | |
*** e0ne_ is now known as e0ne | 09:58 | |
*** pblaho__ is now known as pblah | 09:58 | |
*** pblah is now known as pblaho | 09:58 | |
gilliard | Hi. Does infra (or anyone) have a tool for finding free rooms for IRC meetings? | 09:58 |
*** pblaho has quit IRC | 09:59 | |
*** pblaho has joined #openstack-infra | 09:59 | |
*** jlanoux has quit IRC | 09:59 | |
*** bookwar has left #openstack-infra | 10:02 | |
kragniz | gilliard: did you look at the ical with all the meetings? | 10:03 |
openstackgerrit | Andrew Smith proposed openstack-infra/jenkins-job-builder: Add support for docker builder plugin https://review.openstack.org/178980 | 10:03 |
*** jamespage_ has joined #openstack-infra | 10:04 | |
*** fawadkhaliq has quit IRC | 10:08 | |
*** dhritishikhar has joined #openstack-infra | 10:08 | |
*** dhritishikhar has quit IRC | 10:12 | |
*** mwagner_lap has quit IRC | 10:13 | |
*** e0ne is now known as e0ne_ | 10:13 | |
*** jlanoux has joined #openstack-infra | 10:13 | |
*** teran has joined #openstack-infra | 10:13 | |
*** dannywilson has joined #openstack-infra | 10:14 | |
*** openstackgerrit_ has joined #openstack-infra | 10:15 | |
gilliard | kragniz: looking... Oh, I didn't know I could add that to my google calendar. Neat. | 10:16 |
gilliard | Thanks | 10:16 |
*** e0ne_ is now known as e0ne | 10:16 | |
openstackgerrit | Max Rydahl Andersen proposed openstack-infra/jenkins-job-builder: Add option to skip certification check https://review.openstack.org/178794 | 10:16 |
openstackgerrit | lifeless proposed openstack-dev/pbr: Parallelise integration tests. https://review.openstack.org/178982 | 10:17 |
*** teran has quit IRC | 10:18 | |
openstackgerrit | Davide Guerri proposed openstack-infra/shade: Fix exception re-raise during task execution for py34 https://review.openstack.org/178107 | 10:18 |
openstackgerrit | Davide Guerri proposed openstack-infra/shade: Rename get_endpoint() to get_session_endpoint() https://review.openstack.org/178859 | 10:18 |
openstackgerrit | Davide Guerri proposed openstack-infra/shade: Add keystone services/endpoints methods https://review.openstack.org/177621 | 10:18 |
*** dannywilson has quit IRC | 10:18 | |
*** yfried is now known as yfried|afk | 10:20 | |
spredzy | fungi, ping https://review.openstack.org/#/c/175849/. Do you know how I can test it end to end ? Like simulating the whole workflow ? | 10:21 |
*** yfried|afk is now known as yfried | 10:23 | |
*** Guest73021 has quit IRC | 10:25 | |
*** erikmwilson has joined #openstack-infra | 10:25 | |
*** tjones1 has joined #openstack-infra | 10:27 | |
*** rlandy has joined #openstack-infra | 10:30 | |
*** erikmwilson has quit IRC | 10:30 | |
*** mwagner_lap has joined #openstack-infra | 10:30 | |
*** openstackgerrit_ has quit IRC | 10:31 | |
*** shivharis has quit IRC | 10:32 | |
*** shivharis has joined #openstack-infra | 10:32 | |
*** pc_m has joined #openstack-infra | 10:34 | |
*** frobware_ has quit IRC | 10:35 | |
*** pc_m_ has joined #openstack-infra | 10:35 | |
*** zul has quit IRC | 10:35 | |
*** yamahata has quit IRC | 10:36 | |
openstackgerrit | Sean Dague proposed openstack-infra/elastic-recheck: add web_server.py for testing web content https://review.openstack.org/178834 | 10:37 |
openstackgerrit | Sean Dague proposed openstack-infra/elastic-recheck: actually actually make window location work https://review.openstack.org/178835 | 10:37 |
*** BharatK_ has quit IRC | 10:37 | |
*** _dguerri is now known as dguerri | 10:38 | |
*** pc_m__ has joined #openstack-infra | 10:38 | |
*** ociuhandu has quit IRC | 10:40 | |
*** pc_m_ has quit IRC | 10:41 | |
*** jamespage_ has quit IRC | 10:41 | |
*** teran_ has joined #openstack-infra | 10:48 | |
*** pc_m__ has quit IRC | 10:52 | |
*** alexpilotti has quit IRC | 10:52 | |
*** salv-orlando has joined #openstack-infra | 10:53 | |
*** cdent has joined #openstack-infra | 10:53 | |
*** doude has quit IRC | 10:53 | |
*** dizquierdo_ has joined #openstack-infra | 10:54 | |
*** salv-orl_ has quit IRC | 10:56 | |
*** dizquierdo has quit IRC | 10:57 | |
*** e0ne has quit IRC | 11:05 | |
*** Somay has quit IRC | 11:07 | |
*** yfried is now known as yfried|afk | 11:08 | |
*** Somay has joined #openstack-infra | 11:09 | |
*** e0ne has joined #openstack-infra | 11:10 | |
*** yfried|afk is now known as yfried | 11:12 | |
*** _nadya_ has joined #openstack-infra | 11:13 | |
*** Scader has joined #openstack-infra | 11:18 | |
*** markvoelker has quit IRC | 11:18 | |
*** Scader has left #openstack-infra | 11:18 | |
*** hashar has joined #openstack-infra | 11:20 | |
*** jamielennox is now known as jamielennox|away | 11:20 | |
*** e0ne is now known as e0ne_ | 11:20 | |
*** e0ne_ is now known as e0ne | 11:21 | |
*** dtantsur|brb is now known as dtantsur | 11:23 | |
*** Somay has quit IRC | 11:23 | |
*** mwagner_lap has quit IRC | 11:24 | |
*** liam_ has joined #openstack-infra | 11:28 | |
*** liam_ is now known as Guest24325 | 11:29 | |
*** Longgeek_ has joined #openstack-infra | 11:31 | |
*** Longgeek has quit IRC | 11:32 | |
*** ociuhandu has joined #openstack-infra | 11:32 | |
fungi | spredzy: simulating the workflow would require running a zuul server and a jenkins master of your own (and optionally your own gerrit server, or you could try to trigger it off changes in review-dev or one of the sandbox repositories) | 11:32 |
*** ldnunes has joined #openstack-infra | 11:32 | |
fungi | spredzy: you're basically talking about testing configured interactions between several servers | 11:33 |
*** Guest24325 has quit IRC | 11:33 | |
*** Guest24325 has joined #openstack-infra | 11:35 | |
*** Guest24325 has quit IRC | 11:35 | |
*** alexpilotti has joined #openstack-infra | 11:36 | |
spredzy | fungi, yep, that is what I was afraid off. So there is not straight forward way to know if I works out of the box or not. How would you recommend to proceed ? | 11:39 |
spredzy | I mean I tested things separely, now having to test the whole thing sounds like a different story | 11:39 |
fungi | spredzy: at this point it's mostly a matter of getting more reviews on that change, and then once it merges we can check job logs to see if there's anything the manual testing missed | 11:41 |
*** ddieterly has joined #openstack-infra | 11:41 | |
fungi | assuming enough core reviewers can be convinced it's a worthwhile feature to support long-term | 11:41 |
fungi | ttx: everything going okay with kilo? let me know if there's anything you need my help with... i'm arounf | 11:42 |
fungi | arounD too | 11:42 |
*** derekh has quit IRC | 11:44 | |
ttx | fungi: all ok | 11:45 |
fungi | excellent | 11:45 |
*** derekh has joined #openstack-infra | 11:45 | |
jeblair | ++ | 11:45 |
*** ddieterly has quit IRC | 11:46 | |
spredzy | fungi, ok thank you. | 11:46 |
*** dizquierdo_ is now known as dizquierdo_afk | 11:47 | |
*** rfolco has joined #openstack-infra | 11:48 | |
*** otter768 has joined #openstack-infra | 11:48 | |
*** weshay has joined #openstack-infra | 11:48 | |
*** alexpilotti has quit IRC | 11:48 | |
*** zul has joined #openstack-infra | 11:50 | |
openstackgerrit | Sean Dague proposed openstack-infra/elastic-recheck: actually actually make window location work https://review.openstack.org/178835 | 11:51 |
openstackgerrit | Sean Dague proposed openstack-infra/elastic-recheck: add web_server.py for testing web content https://review.openstack.org/178834 | 11:51 |
*** jcoufal_ is now known as jcoufal | 11:51 | |
*** otter768 has quit IRC | 11:53 | |
*** baoli has joined #openstack-infra | 11:56 | |
*** woodster_ has joined #openstack-infra | 11:58 | |
*** ildikov has quit IRC | 11:59 | |
*** ZZelle has quit IRC | 12:02 | |
*** ZZelle has joined #openstack-infra | 12:02 | |
ttx | fungi: After release we'll push the .1 version bumps to stable/kilo and then we'll be able to revert to post-release ACLs for stable/kilo | 12:04 |
openstackgerrit | Sean Dague proposed openstack-infra/elastic-recheck: actually actually make window location work https://review.openstack.org/178835 | 12:04 |
openstackgerrit | Sean Dague proposed openstack-infra/elastic-recheck: add web_server.py for testing web content https://review.openstack.org/178834 | 12:04 |
ttx | I guess you can still prepare a WIP chnage | 12:04 |
*** kgiusti has joined #openstack-infra | 12:05 | |
*** claudiub has joined #openstack-infra | 12:05 | |
*** deepakcs has quit IRC | 12:06 | |
*** sks has quit IRC | 12:06 | |
*** Kiall has quit IRC | 12:08 | |
*** Kiall has joined #openstack-infra | 12:08 | |
*** chuck__ has joined #openstack-infra | 12:08 | |
*** mwagner_lap has joined #openstack-infra | 12:09 | |
*** zul has quit IRC | 12:10 | |
*** markvoelker has joined #openstack-infra | 12:11 | |
*** nfedotov has quit IRC | 12:11 | |
*** heyongli has quit IRC | 12:12 | |
*** nfedotov has joined #openstack-infra | 12:13 | |
*** heyongli has joined #openstack-infra | 12:13 | |
*** ildikov has joined #openstack-infra | 12:15 | |
*** dtantsur is now known as dtantsur|brb | 12:15 | |
*** bswartz has quit IRC | 12:15 | |
fungi | yep, doing now | 12:18 |
*** miqui has joined #openstack-infra | 12:22 | |
*** arxcruz has quit IRC | 12:24 | |
*** arxcruz has joined #openstack-infra | 12:25 | |
*** erikmwilson has joined #openstack-infra | 12:26 | |
*** erikmwilson has quit IRC | 12:31 | |
*** gordc has joined #openstack-infra | 12:32 | |
*** Longgeek has joined #openstack-infra | 12:32 | |
*** _nadya_ has quit IRC | 12:33 | |
*** Longgeek_ has quit IRC | 12:33 | |
*** bhunter71 has joined #openstack-infra | 12:33 | |
*** achanda has joined #openstack-infra | 12:36 | |
*** jamespage_ has joined #openstack-infra | 12:36 | |
*** yfried is now known as yfried|afk | 12:37 | |
*** Hal has joined #openstack-infra | 12:38 | |
*** Hal is now known as Guest4127 | 12:39 | |
openstackgerrit | Jeremy Stanley proposed openstack-infra/project-config: Revert "Temporarily copy proposed/* ACLs to stable/kilo" https://review.openstack.org/179045 | 12:39 |
*** jistr has quit IRC | 12:39 | |
fungi | ttx: ^ wip but will set ready for review when you're ready | 12:40 |
*** achanda has quit IRC | 12:41 | |
*** ddieterly has joined #openstack-infra | 12:42 | |
*** alexpilotti has joined #openstack-infra | 12:43 | |
*** Longgeek has quit IRC | 12:44 | |
*** dprince has joined #openstack-infra | 12:44 | |
*** jamespage_ has quit IRC | 12:44 | |
*** _nadya_ has joined #openstack-infra | 12:46 | |
*** ddieterly has quit IRC | 12:47 | |
*** yfried|afk is now known as yfried | 12:47 | |
*** Longgeek_ has joined #openstack-infra | 12:48 | |
*** erikmwilson has joined #openstack-infra | 12:49 | |
*** dtantsur|brb is now known as dtantsur | 12:50 | |
*** erikmwilson has quit IRC | 12:51 | |
*** erikmwilson_ is now known as erikmwilson | 12:51 | |
AJaeger | fungi, clarkb: I think we'Re ready for the next step with translation import "compression" - covering python projects. Could you review https://review.openstack.org/176947 , please? | 12:52 |
*** dkranz has joined #openstack-infra | 12:53 | |
*** chuck__ has quit IRC | 12:53 | |
*** bswartz has joined #openstack-infra | 12:53 | |
*** spzala has joined #openstack-infra | 12:54 | |
*** jistr has joined #openstack-infra | 12:54 | |
*** jistr is now known as jistr|biab | 12:55 | |
*** marcusvrn has joined #openstack-infra | 13:01 | |
*** jamespage_ has joined #openstack-infra | 13:02 | |
*** marcusvrn1 has joined #openstack-infra | 13:03 | |
*** maishsk has quit IRC | 13:04 | |
*** Longgeek_ has quit IRC | 13:04 | |
*** fifieldt has quit IRC | 13:04 | |
*** Longgeek has joined #openstack-infra | 13:05 | |
*** marcusvrn has quit IRC | 13:05 | |
*** ams__ has joined #openstack-infra | 13:07 | |
ams__ | In the jenkins job builder is there any way to optionally include a chunk of XML for a job, given a flag? | 13:08 |
*** radez_g0n3 is now known as radez | 13:08 | |
*** peristeri has joined #openstack-infra | 13:09 | |
*** ssam2 has quit IRC | 13:09 | |
*** jamespage_ has quit IRC | 13:09 | |
*** bknudson has quit IRC | 13:09 | |
*** MaxV has joined #openstack-infra | 13:10 | |
*** xyang1 has joined #openstack-infra | 13:10 | |
*** mrunge has quit IRC | 13:11 | |
*** dboik has quit IRC | 13:12 | |
*** bhunter71 has quit IRC | 13:12 | |
*** _nadya_ has quit IRC | 13:14 | |
*** MaxV has quit IRC | 13:15 | |
tristanC | *sighs*, elections period is now over, TC results are presented here: http://lists.openstack.org/pipermail/openstack-dev/2015-April/063000.html | 13:16 |
openstackgerrit | yolanda.robla proposed openstack-infra/puppet-cgit: Add puppet-cgit load balancer class https://review.openstack.org/179053 | 13:17 |
*** esker has quit IRC | 13:18 | |
*** esker has joined #openstack-infra | 13:18 | |
*** stevemar has joined #openstack-infra | 13:18 | |
*** _nadya_ has joined #openstack-infra | 13:19 | |
*** sslypushenko has quit IRC | 13:19 | |
*** esker has quit IRC | 13:19 | |
*** vlaza has left #openstack-infra | 13:19 | |
AJaeger | tristanC, pleia2 Thanks for doing all these elections! | 13:20 |
*** sslypushenko has joined #openstack-infra | 13:21 | |
*** dustins has joined #openstack-infra | 13:22 | |
*** eharney has quit IRC | 13:24 | |
fungi | tristanC: pleia2: awesome work!!! | 13:25 |
fungi | and congrats to ttx, jaypipes, jeblair, flaper87, markmcclain, dtroyer and lifeless! | 13:25 |
jaypipes | fungi: for what? what did I miss? | 13:25 |
openstackgerrit | yolanda.robla proposed openstack-infra/system-config: Move server outside git_backend class https://review.openstack.org/179058 | 13:25 |
jaypipes | oh... TC? | 13:25 |
fungi | jaypipes: your indentured servitude | 13:25 |
* jaypipes digs into email.. | 13:25 | |
jaypipes | fungi: lol | 13:26 |
* AJaeger joins fungi in the congrats! | 13:26 | |
*** mriedem has joined #openstack-infra | 13:26 | |
yolanda | congrats to all TC members | 13:27 |
flaper87 | fungi: thanks :) | 13:28 |
ttx | flaper87, dtroyer: welcome! lifeless: welcome back! | 13:28 |
*** lucap1 has joined #openstack-infra | 13:28 | |
*** lucap has quit IRC | 13:30 | |
*** dboik has joined #openstack-infra | 13:30 | |
markmcclain | fungi: thanks | 13:30 |
*** dboik has quit IRC | 13:30 | |
ttx | 548 voters... increase or not? | 13:31 |
*** amitgandhinz has joined #openstack-infra | 13:31 | |
*** dboik has joined #openstack-infra | 13:31 | |
ttx | 506/1893 last time | 13:31 |
*** Somay has joined #openstack-infra | 13:32 | |
fungi | 548/2169 (25%) this time | 13:32 |
fungi | 1.5% drop | 13:32 |
ttx | so +42 voters but 26.7% -> 25.2% | 13:32 |
ttx | 15.2% aquisition rate on new voters | 13:33 |
ttx | (only 42 new voters for 276 new voters) | 13:33 |
*** dizquierdo_afk is now known as dizquierdo | 13:34 | |
*** sdake_ has joined #openstack-infra | 13:34 | |
jaypipes | ttx: perhaps it's worth extending the TC election times? things were being debated/discussed on the ML up to and past the voting deadline... just a suggestion. | 13:34 |
ttx | jaypipes: most people vote on the 1st day | 13:35 |
ttx | we could extend the campaign duration I guess | 13:35 |
*** zul has joined #openstack-infra | 13:35 | |
fungi | yeah, maybe add a week between close of nominations and open voting | 13:36 |
*** tiswanso has joined #openstack-infra | 13:36 | |
fungi | with some candidates not throwing in their hat until the day before voting begins and lots of voters selecting them on the first day, the time for debate is sort of unnaturally compressed | 13:37 |
fungi | not that voters can't go back and change their ballots prior to conclusion of the election, but it leads to discussions going in parallel with the election itself which is potentially less useful | 13:38 |
*** sdake has quit IRC | 13:38 | |
*** jtriley has joined #openstack-infra | 13:38 | |
openstackgerrit | yolanda.robla proposed openstack-infra/system-config: Don't hardcode names on certs https://review.openstack.org/179067 | 13:39 |
*** packet has joined #openstack-infra | 13:39 | |
fungi | ideally candidates would have their platforms already written up and announce their candidacy on the first day of the nomination period, but why do today what you can put off until tomorrow? | 13:39 |
openstackgerrit | David Shrewsbury proposed openstack-infra/shade: Allow for int or string ID comparisons https://review.openstack.org/179069 | 13:40 |
*** zul has quit IRC | 13:40 | |
*** ajo_ is now known as ajo | 13:40 | |
*** prad-afk is now known as prad | 13:41 | |
openstackgerrit | Valeriy Ponomaryov proposed openstack-infra/project-config: Bind manila-ui project changes to IRC channel openstack-manila https://review.openstack.org/179070 | 13:42 |
*** bknudson has joined #openstack-infra | 13:43 | |
*** _nadya_ has quit IRC | 13:43 | |
*** sigmavirus24_awa is now known as sigmavirus24 | 13:44 | |
*** ddieterly has joined #openstack-infra | 13:44 | |
*** che-arne has quit IRC | 13:47 | |
*** sdake has joined #openstack-infra | 13:47 | |
jeblair | fungi: oh thanks :) | 13:49 |
*** otter768 has joined #openstack-infra | 13:49 | |
openstackgerrit | David Shrewsbury proposed openstack-infra/shade: Allow for int or string ID comparisons https://review.openstack.org/179069 | 13:49 |
*** sdake_ has quit IRC | 13:51 | |
*** ajmiller has joined #openstack-infra | 13:52 | |
*** jistr|biab is now known as jistr | 13:53 | |
*** otter768 has quit IRC | 13:53 | |
*** erlon has joined #openstack-infra | 13:56 | |
*** chlong has joined #openstack-infra | 13:56 | |
*** jtriley_ has joined #openstack-infra | 13:57 | |
*** eharney has joined #openstack-infra | 13:57 | |
*** armax has joined #openstack-infra | 13:59 | |
*** sergsh has quit IRC | 13:59 | |
*** jtriley has quit IRC | 14:00 | |
*** zul has joined #openstack-infra | 14:00 | |
*** signed8bit has joined #openstack-infra | 14:01 | |
*** esker has joined #openstack-infra | 14:02 | |
*** julim has joined #openstack-infra | 14:06 | |
*** mmedvede has joined #openstack-infra | 14:06 | |
*** mpaolino has joined #openstack-infra | 14:07 | |
*** dangers_away is now known as dangers | 14:09 | |
*** btully has joined #openstack-infra | 14:11 | |
*** AJaeger has quit IRC | 14:11 | |
*** unicell has joined #openstack-infra | 14:12 | |
*** unicell1 has quit IRC | 14:13 | |
*** lucap1 has quit IRC | 14:15 | |
*** lucap has joined #openstack-infra | 14:17 | |
*** heyongli has quit IRC | 14:18 | |
*** heyongli has joined #openstack-infra | 14:18 | |
*** mpaolino has quit IRC | 14:18 | |
*** rossella_s has quit IRC | 14:19 | |
openstackgerrit | Matthew Treinish proposed openstack-infra/subunit2sql: Add start and stop date options to subunit2sql-graph https://review.openstack.org/176114 | 14:19 |
openstackgerrit | Matthew Treinish proposed openstack-infra/subunit2sql: Add modular subunit2sql-graph entry point https://review.openstack.org/176115 | 14:19 |
*** Somay has quit IRC | 14:19 | |
openstackgerrit | Matthew Treinish proposed openstack-infra/subunit2sql: Add db api method to get status time series https://review.openstack.org/176042 | 14:20 |
openstackgerrit | Matthew Treinish proposed openstack-infra/subunit2sql: Add command to calculate failure count and graph it https://review.openstack.org/176043 | 14:20 |
*** rossella_s has joined #openstack-infra | 14:20 | |
openstackgerrit | Matthew Treinish proposed openstack-infra/subunit2sql: Improve run_time graph formatting https://review.openstack.org/178276 | 14:20 |
*** blahdeblah_ has joined #openstack-infra | 14:20 | |
fungi | wow, there's no option to yum to tell it to treat package names case-insensitively? that's a royal pain | 14:20 |
*** tonytan4ever has joined #openstack-infra | 14:21 | |
fungi | i mean, if you ask it to install package foo-bar it will happily tell you that it found no foo-bar but did find a foo-Bar instead. and then stop | 14:21 |
ameade | lol | 14:22 |
*** jgrimm is now known as zz_jgrimm | 14:22 | |
mtreinish | fungi: heh, yeah that's a fun yum feature, it's telling you there's a similar package name :) | 14:22 |
*** blahdeblah__ has joined #openstack-infra | 14:22 | |
fungi | this basically means that unless we want to rely on separate search-and-resolve logic in bindep, we're stuck having upper-case letters in our package lists | 14:23 |
*** doude has joined #openstack-infra | 14:23 | |
fungi | which means yet another patch to the parser validator to allow upper-case | 14:23 |
fungi | i'm fine with that as the _default_ yum install behavior, but even just having a --case-insensitive option would be nice | 14:24 |
*** e0ne is now known as e0ne_ | 14:25 | |
*** blahdeblah_ has quit IRC | 14:25 | |
mtreinish | fungi: hmm, a quick google search show many people complaining about the same thing | 14:25 |
fungi | yep | 14:25 |
mtreinish | maybe we should jsut infra patch yum to add the flag | 14:25 |
mtreinish | it is python :) | 14:25 |
fungi | "this is unix, filenames are case sensitive" | 14:25 |
*** zul has quit IRC | 14:26 | |
fungi | sure! but does that mean that logical names of packages need to be too? | 14:26 |
ameade | yums opensource, just submit a patch :P | 14:26 |
* ameade senses the rage | 14:26 | |
*** wenlock has joined #openstack-infra | 14:26 | |
mtreinish | ameade: it would probably be rejected based on the google hits... | 14:26 |
fungi | i'm currently looking to see if someone already has and been rejected | 14:27 |
fungi | but yeah, the yum maintainers responding to ml threads and bug reports at least seem to be fairly interested in keeping case-sensitivity in package names, even though they also seem to acknowledge that there's no real benefit to doing so | 14:27 |
*** mattfarina has joined #openstack-infra | 14:28 | |
*** dustins_ has joined #openstack-infra | 14:30 | |
*** e0ne_ is now known as e0ne | 14:30 | |
*** _nadya_ has joined #openstack-infra | 14:31 | |
fungi | ahh, the main counterargument seems to be that non-ascii is now allowed in package names, and there's no sane way to lowercase the codepoints for all glyphs in all charactersets | 14:31 |
*** eharney has quit IRC | 14:32 | |
*** dustins has quit IRC | 14:32 | |
fungi | e.g., print('Ä'.lower()) shows Ä not ä | 14:32 |
mtreinish | fungi: heh, but do they use snowmen in the package names? | 14:32 |
*** zz_dimtruck is now known as dimtruck | 14:33 | |
mtreinish | because saying you allow something, and that you actually do it are different | 14:33 |
*** sputnik13 has joined #openstack-infra | 14:34 | |
*** _nadya_ has quit IRC | 14:35 | |
*** dprince has quit IRC | 14:36 | |
*** wenlock has quit IRC | 14:39 | |
*** wenlock has joined #openstack-infra | 14:40 | |
*** sabeen1 has joined #openstack-infra | 14:40 | |
BobBall | Should the age specifier for gerrit's ssh query work? "gerrit query --format=JSON --patch-sets age:2w project:openstack/nova branch:master" gives changes that are over a month old (according to lastUpdated) | 14:41 |
openstackgerrit | Davide Guerri proposed openstack-infra/shade: Add keystone services/endpoints methods https://review.openstack.org/177621 | 14:41 |
*** wenlock_ has joined #openstack-infra | 14:41 | |
*** MarkAtwood has joined #openstack-infra | 14:42 | |
*** marcusvrn1 has quit IRC | 14:42 | |
openstackgerrit | Davide Guerri proposed openstack-infra/shade: Add keystone services/endpoints methods https://review.openstack.org/177621 | 14:43 |
*** dims has quit IRC | 14:44 | |
openstackgerrit | Jeremy Stanley proposed openstack-infra/project-config: Correct case on RPMs in bindep fallback https://review.openstack.org/179109 | 14:44 |
*** dims has joined #openstack-infra | 14:44 | |
*** emagana has joined #openstack-infra | 14:45 | |
*** sabeen2 has joined #openstack-infra | 14:45 | |
*** BharatK has joined #openstack-infra | 14:45 | |
openstackgerrit | Jeremy Stanley proposed openstack-infra/bindep: Allow mixed-case package names https://review.openstack.org/179111 | 14:46 |
*** eharney has joined #openstack-infra | 14:47 | |
*** sabeen1 has quit IRC | 14:47 | |
*** lucap has quit IRC | 14:48 | |
*** zul has joined #openstack-infra | 14:49 | |
jeblair | BobBall: yes, i believe that query would be "has not been updated in at least 2 weeks". I think if you do "-age:2w" or "NOT age:2w" you'll get "has been updated within the last 2 weeks" | 14:49 |
*** marcusvrn has joined #openstack-infra | 14:50 | |
*** bhunter71 has joined #openstack-infra | 14:50 | |
BobBall | Oh wow. Yes, I completely mis-read the documentation... Thanks. | 14:50 |
*** dustins_ has quit IRC | 14:51 | |
jeblair | BobBall: yeah, i think the docs could use a couple of examples :) | 14:51 |
jeblair | (lhf gerrit patch!) | 14:51 |
openstackgerrit | yolanda.robla proposed openstack-infra/puppet-cgit: Add cgit::ssh class to manage git over ssh https://review.openstack.org/179113 | 14:53 |
*** chlong has quit IRC | 14:53 | |
BobBall | Ah - that's great - it means I can do age:2d NOT age:1d to get everything that's between 1 and 2 days old. I like that. | 14:53 |
jeblair | BobBall: huh, let me know if that works or if gerrit gets confused :) | 14:54 |
fungi | yeah, that seems... useful! ;) | 14:54 |
fungi | and there's kilo! | 14:55 |
BobBall | Seems to work nicely indeed. | 14:55 |
*** nelsnelson has joined #openstack-infra | 14:56 | |
dhellmann | is anyone else seeing issues with gerrit? about every 3rd time I try to vote on a patch it gives me an error. | 14:57 |
*** roaet has quit IRC | 14:58 | |
*** roaet_ has joined #openstack-infra | 14:59 | |
*** dboik_ has joined #openstack-infra | 15:00 | |
*** tiswanso has quit IRC | 15:00 | |
*** tiswanso has joined #openstack-infra | 15:00 | |
*** emagana has quit IRC | 15:02 | |
*** BharatK has quit IRC | 15:02 | |
*** annegentle has joined #openstack-infra | 15:02 | |
*** dboik has quit IRC | 15:03 | |
fungi | dhellmann: what error? | 15:03 |
dhellmann | fungi: the web ui just says the server is "unavailable" or something, and then works on the next try. | 15:04 |
dhellmann | fungi: not a big deal, but with the recent issues I wondered if it was an indicator of a bigger problem about to happen | 15:04 |
fungi | that's a new error to me. i don't see anything out of the ordinary (yet) on the performance trending for the server but i'll take a look locally in case something has just started to cause trouble on it | 15:04 |
*** baoli has quit IRC | 15:05 | |
dhellmann | fungi: screen goes dark, error shows up, there's a button to clear the screen and then it goes back to the regular ui. I see it now and then, but hit it quite a lot today | 15:05 |
fungi | ahh, that often means problems with javascript calling back to the server i think (i see it often when my local network is flaky) | 15:06 |
fungi | but could also potentially indicate network problems near review.o.o if others are getting it right now too | 15:07 |
*** baoli has joined #openstack-infra | 15:07 | |
fungi | the server _seems_ healthy for the moment | 15:07 |
*** arxcruz has quit IRC | 15:08 | |
fungi | non-impacting network maintenance in rackspace dfw (where review.o.o sits) Thursday, April 30th - at 8:00AM Central to 9:00AM Central | 15:09 |
fungi | shouldn't supposedly have been noticeable though, and should have concluded an hour ago according to status.rackspace.com | 15:09 |
* BobBall loves assertions that maintenance is non-impacting | 15:09 | |
*** MarkAtwood has quit IRC | 15:10 | |
openstackgerrit | yolanda.robla proposed openstack-infra/puppet-cgit: Add cgit::ssh class to manage git over ssh https://review.openstack.org/179113 | 15:10 |
*** sergsh has joined #openstack-infra | 15:10 | |
fungi | over the many years i worked for a service provider, i was continually amused by the compulsion to announce non-impacting maintenance work. what we really always meant was "we think this won't break you, but we're not positive so here's a heads-up" | 15:11 |
*** spzala has quit IRC | 15:11 | |
openstackgerrit | Davide Guerri proposed openstack-infra/shade: Add keystone services/endpoints methods https://review.openstack.org/177621 | 15:11 |
*** krtaylor has quit IRC | 15:12 | |
fungi | for openstack infra maintenances, i usually try to phrase it in a more straightforward manner... since i don't have any nervous salespeople to hound me about the wording | 15:12 |
*** zz_jgrimm is now known as jgrimm | 15:17 | |
*** krtaylor has joined #openstack-infra | 15:18 | |
*** dannywilson has joined #openstack-infra | 15:18 | |
*** ujuc_ has joined #openstack-infra | 15:19 | |
*** dustins has joined #openstack-infra | 15:19 | |
*** dannywilson has quit IRC | 15:22 | |
*** pabelanger has joined #openstack-infra | 15:23 | |
*** belmoreira has quit IRC | 15:24 | |
*** ujuc_ has quit IRC | 15:24 | |
*** hdd has joined #openstack-infra | 15:25 | |
*** e0ne is now known as e0ne_ | 15:25 | |
*** dannywilson has joined #openstack-infra | 15:27 | |
*** MarkAtwood has joined #openstack-infra | 15:28 | |
*** scheuran has quit IRC | 15:29 | |
*** marun has joined #openstack-infra | 15:30 | |
*** ssam2 has joined #openstack-infra | 15:31 | |
*** liam_ has joined #openstack-infra | 15:31 | |
*** liam_ has quit IRC | 15:31 | |
*** pabelanger has quit IRC | 15:32 | |
*** pabelanger has joined #openstack-infra | 15:32 | |
*** pabelanger has joined #openstack-infra | 15:32 | |
*** hdd has quit IRC | 15:33 | |
*** e0ne_ is now known as e0ne | 15:35 | |
*** deepakcs has joined #openstack-infra | 15:35 | |
*** spzala has joined #openstack-infra | 15:35 | |
openstackgerrit | yolanda.robla proposed openstack-infra/puppet-cgit: Manage cgitrc config file https://review.openstack.org/179132 | 15:37 |
yolanda | i'm spamming puppet-cgit with changes today | 15:38 |
*** marcusvrn has quit IRC | 15:38 | |
*** rm_work|away is now known as rm_work | 15:38 | |
*** viktors is now known as viktors|afk | 15:39 | |
openstackgerrit | yolanda.robla proposed openstack-infra/puppet-cgit: Manage cgitrc config file https://review.openstack.org/179132 | 15:39 |
*** achanda has joined #openstack-infra | 15:39 | |
*** emagana has joined #openstack-infra | 15:39 | |
openstackgerrit | Filip Blaha proposed openstack-infra/project-config: Pylint job for Murano project https://review.openstack.org/172887 | 15:39 |
*** spzala has quit IRC | 15:40 | |
*** marcusvrn has joined #openstack-infra | 15:40 | |
*** Swami has joined #openstack-infra | 15:41 | |
*** yfried has quit IRC | 15:41 | |
*** achanda has quit IRC | 15:44 | |
*** jcoufal has quit IRC | 15:45 | |
*** ashleighfarnham has joined #openstack-infra | 15:46 | |
*** spredzy is now known as spredzy|afk | 15:48 | |
*** pblaho has quit IRC | 15:48 | |
*** yamahata has joined #openstack-infra | 15:49 | |
*** otter768 has joined #openstack-infra | 15:50 | |
*** ddieterly has quit IRC | 15:51 | |
*** annegentle has quit IRC | 15:53 | |
*** BharatK has joined #openstack-infra | 15:54 | |
*** otter768 has quit IRC | 15:55 | |
*** EmilienM is now known as EmilienM|afk | 15:55 | |
*** sushilkm has joined #openstack-infra | 15:56 | |
*** sushilkm has left #openstack-infra | 15:56 | |
*** emagana has quit IRC | 15:57 | |
*** amitgandhinz has quit IRC | 15:59 | |
*** emagana has joined #openstack-infra | 15:59 | |
*** alexsyip has joined #openstack-infra | 16:01 | |
*** hashar is now known as hasharAway | 16:01 | |
*** baoli has quit IRC | 16:02 | |
*** baoli_ has joined #openstack-infra | 16:03 | |
*** deepakcs has quit IRC | 16:03 | |
*** dboik_ has quit IRC | 16:04 | |
*** dboik has joined #openstack-infra | 16:04 | |
*** jlanoux has quit IRC | 16:07 | |
*** dizquierdo has quit IRC | 16:07 | |
*** gyee has joined #openstack-infra | 16:07 | |
*** luqas has quit IRC | 16:09 | |
*** ashleighfarnham has quit IRC | 16:11 | |
*** Ala has quit IRC | 16:11 | |
notmyname | when should I expect to see the stable kilo branch auto-merged to master (so the tags work properly)? I would have expected it to happen already | 16:12 |
*** Swami has quit IRC | 16:13 | |
fungi | notmyname: another casualty of the switch away from using proposed/.* branches... http://logs.openstack.org/04/045cff4b7cbee305d1a7ee688d0accb106bcdd66/release/swift-merge-release-tags/f9a5fa9/console.html#_2015-04-30_13_57_31_958 | 16:14 |
*** samuelBartel has quit IRC | 16:14 | |
pleia2 | oddly quiet morning in here | 16:14 |
fungi | now we need to brainstorm how to resolve the "merge release tags when pushed to the stable branch which used to be the proposed branch" with "don't merge release tags pushed to stable branches" | 16:14 |
fungi | ttx: ^ ideas if you're not already too wiped? | 16:15 |
fungi | dhellmann: mordred: ^ ? | 16:16 |
notmyname | fungi: I can manually propose a merge commit to master and land it | 16:17 |
fungi | maybe if the tag pushed sorts later than the most recent tag on master, then merge it | 16:17 |
fungi | notmyname: that's a fine workaround for the moment if you're in a hurry | 16:17 |
notmyname | yeah, I'd like to resolve it quickly. I'll do that | 16:17 |
notmyname | thanks | 16:17 |
fungi | i'm more concerned with how to fix the logic around deciding when to merge these | 16:17 |
*** salv-orlando has quit IRC | 16:17 | |
*** salv-orlando has joined #openstack-infra | 16:19 | |
*** dizquierdo has joined #openstack-infra | 16:19 | |
*** dizquierdo has quit IRC | 16:19 | |
*** sks has joined #openstack-infra | 16:19 | |
*** dizquierdo has joined #openstack-infra | 16:19 | |
*** ashleighfarnham has joined #openstack-infra | 16:20 | |
notmyname | fungi: done. `git co -b tag_merge && git merge -s ours stable/kilo && git amend && git review -R` | 16:22 |
*** julim has quit IRC | 16:22 | |
ttx | fungi: ah, fun. Another reason we had proposed/* | 16:22 |
fungi | notmyname: thanks and apologies for the complication. i'm working on a patch to the script which does that now | 16:22 |
openstackgerrit | Alexander Yip proposed openstack-infra/project-config: Add congress.yaml file https://review.openstack.org/178862 | 16:22 |
notmyname | no worries. thanks for looking into it | 16:23 |
fungi | ttx: well, i think the tag comparison can probably work in lieu of the stable branch check | 16:23 |
fungi | ttx: basically only merge a tag to master if it sorts after the most recent tag on master | 16:23 |
ttx | fungi: err, wouldn't that merge 2015.1.1 ? | 16:24 |
ttx | i.e. a stable branch tag ? | 16:24 |
fungi | oh, hrm... yeah | 16:24 |
*** krtaylor has quit IRC | 16:25 | |
fungi | latest pbr (not yet released) solves this i think, but we still need an alternate solution for non-pbr-using repos | 16:25 |
zaro | morning | 16:26 |
*** isviridov_away is now known as isviridov | 16:26 | |
fungi | oh, actually latest pbr still only solves it for projects declaring a next version somewhere | 16:26 |
*** nfedotov has quit IRC | 16:26 | |
fungi | i think | 16:27 |
fungi | i suppose we could only push if the first two components of the version number sort after the first two components of the latest tag on master | 16:28 |
*** sks has quit IRC | 16:29 | |
openstackgerrit | Alexander Yip proposed openstack-infra/project-config: Add congress.yaml file https://review.openstack.org/178862 | 16:29 |
fungi | so 2015.1.0 merges from stable/kilo to master, but 2015.1.(n>0) get skipped. then 2015.2.0 on stable/liberty merges because 2015.2 sorts after 2015.1 | 16:29 |
*** nfedotov has joined #openstack-infra | 16:29 | |
*** _nadya_ has joined #openstack-infra | 16:30 | |
* fungi is still trying to think through the release implications for libraries which may bump the first or second component on a stable branch, but that seems a poor choice regardless if semver stable is assumed to mean only the third component gets incremented) | 16:31 | |
*** sdake_ has joined #openstack-infra | 16:31 | |
ttx | fungi: so when https://review.openstack.org/#/q/branch:stable/kilo+topic:next-kilo,n,z all merged we can revert the stable ACL | 16:32 |
ttx | and I can remove my -2s | 16:33 |
*** tkelsey has joined #openstack-infra | 16:34 | |
fungi | sounds good | 16:34 |
ttx | I'll be back in a couple hours to do that, please recheck stuff that fails in the mean time (looks like mriedem is watching it too) | 16:34 |
fungi | yep, i'll keep an eye on them | 16:35 |
*** unicell has quit IRC | 16:35 | |
*** sdake has quit IRC | 16:35 | |
*** ashleighfarnham has quit IRC | 16:36 | |
jistr | hi :) i've had two similar patches for amending reviewer rights around puppet and tripleo, one got landed fine but this one seems to be a bit stuck... https://review.openstack.org/#/c/176226/ is it somehow controversial or something? :) | 16:36 |
*** dprince has joined #openstack-infra | 16:36 | |
*** gyee has quit IRC | 16:37 | |
openstackgerrit | Alexander Yip proposed openstack-infra/project-config: Add congress.yaml file https://review.openstack.org/178862 | 16:37 |
*** krtaylor has joined #openstack-infra | 16:38 | |
*** zul has quit IRC | 16:40 | |
*** mpavone has quit IRC | 16:40 | |
*** sdake has joined #openstack-infra | 16:40 | |
*** mmedvede has quit IRC | 16:41 | |
fungi | jistr: doesn't look controversial, just one of 72 open changes to that repo (out of the >100 repos our core review team is responsible for) and came in during the kilo release rush week | 16:41 |
pleia2 | jistr: got you sorted ^5 fungi | 16:41 |
openstackgerrit | Alexander Yip proposed openstack-infra/project-config: Add congress.yaml file https://review.openstack.org/178862 | 16:42 |
jistr | fungi, pleia2: awesome, thanks a lot! :) | 16:42 |
*** ociuhandu has quit IRC | 16:42 | |
*** sdake_ has quit IRC | 16:44 | |
clarkb | good morning | 16:45 |
*** alexpilotti has quit IRC | 16:46 | |
*** ssam2 has quit IRC | 16:46 | |
clarkb | looks like things went well? anything I should catch up on? | 16:48 |
*** derekh has quit IRC | 16:48 | |
fungi | clarkb: i'm trying an alternative solution for the tag merge script, other than that smooth sailing | 16:48 |
*** dims has quit IRC | 16:48 | |
openstackgerrit | Merged openstack-infra/project-config: Add tripleo-core to puppet-tripleo https://review.openstack.org/176226 | 16:49 |
*** dims has joined #openstack-infra | 16:49 | |
*** e0ne is now known as e0ne_ | 16:49 | |
*** e0ne_ is now known as e0ne | 16:50 | |
*** e0ne has quit IRC | 16:51 | |
openstackgerrit | Clark Boylan proposed openstack-infra/gearman-plugin: Fix a cause of livelock in worker work() method https://review.openstack.org/178929 | 16:53 |
clarkb | zaro: ^ if you get a chance to look at that, that would be good. I haven't been able to test it yet beyond what mvn package does, but figured getting some code up would be beneficial | 16:53 |
clarkb | fungi: now that I have pushed an update to ^ with the log data in the commit message anything I can do to help with the merge script? | 16:54 |
*** melwitt has joined #openstack-infra | 16:55 | |
*** shivharis has quit IRC | 16:55 | |
*** dizquierdo has quit IRC | 16:55 | |
openstackgerrit | Alexander Yip proposed openstack-infra/project-config: Add congress.yaml file https://review.openstack.org/178862 | 16:55 |
fungi | clarkb: i'll have something up for review in a sec and we can use it as a straw man | 16:55 |
*** aarefiev has quit IRC | 16:55 | |
*** patrickeast has joined #openstack-infra | 16:56 | |
*** amitgandhinz has joined #openstack-infra | 17:00 | |
*** emagana has quit IRC | 17:00 | |
*** whoops has joined #openstack-infra | 17:00 | |
*** mtanino has joined #openstack-infra | 17:01 | |
*** whoops is now known as sadams | 17:01 | |
*** sadams is now known as whoops | 17:01 | |
*** spzala has joined #openstack-infra | 17:01 | |
*** achanda has joined #openstack-infra | 17:02 | |
*** Sukhdev has joined #openstack-infra | 17:02 | |
*** notnownikki has quit IRC | 17:02 | |
*** bswartz has quit IRC | 17:03 | |
*** rm_work is now known as rm_work|away | 17:03 | |
*** dannywilson has quit IRC | 17:03 | |
*** patrickeast has quit IRC | 17:04 | |
*** unicell has joined #openstack-infra | 17:05 | |
*** sergsh has quit IRC | 17:05 | |
*** unicell has quit IRC | 17:05 | |
*** unicell has joined #openstack-infra | 17:05 | |
*** bswartz has joined #openstack-infra | 17:05 | |
*** unicell has quit IRC | 17:06 | |
*** unicell has joined #openstack-infra | 17:06 | |
*** emagana has joined #openstack-infra | 17:06 | |
*** yamahata has quit IRC | 17:07 | |
*** bswartz has quit IRC | 17:07 | |
*** bswartz has joined #openstack-infra | 17:07 | |
*** ashleighfarnham has joined #openstack-infra | 17:08 | |
*** dannywilson has joined #openstack-infra | 17:10 | |
*** samueldmq has joined #openstack-infra | 17:10 | |
*** dannywilson has quit IRC | 17:10 | |
samueldmq | so launchpad allows us to change username, but gerrit doesnt, is that right ? | 17:10 |
*** ayoung is now known as ayoung-afk | 17:10 | |
*** isviridov is now known as isviridov_away | 17:11 | |
*** harlowja_away is now known as harlowja | 17:11 | |
fungi | right | 17:11 |
*** radez is now known as radez_g0n3 | 17:12 | |
samueldmq | fungi, is it planned to add support for username changes ? | 17:12 |
fungi | samueldmq: gerrit supported changing usernames before i think release 2.3 when they removed it | 17:13 |
fungi | they have features which are predicated on usernames being immutable | 17:13 |
*** abregman has quit IRC | 17:13 | |
fungi | once set | 17:13 |
samueldmq | fungi, and I cant even unbind my lp account, and bind it to another gerrit account ? | 17:14 |
openstackgerrit | Jeremy Stanley proposed openstack-infra/project-config: Merge tags from stable branches to master https://review.openstack.org/179170 | 17:14 |
fungi | clarkb: ^ current idea | 17:14 |
clarkb | looking now | 17:16 |
*** salv-orlando has quit IRC | 17:16 | |
*** sabeen2 has quit IRC | 17:16 | |
*** salv-orlando has joined #openstack-infra | 17:16 | |
*** dtantsur is now known as dtantsur|pto | 17:17 | |
*** mmedvede has joined #openstack-infra | 17:18 | |
*** shivharis has joined #openstack-infra | 17:18 | |
shivharis | anteaya: hi | 17:19 |
*** cdent has quit IRC | 17:19 | |
fungi | samueldmq: not really, no. gerrit's openid integration and user management are pretty rudimentary, to put it politely | 17:20 |
*** Mmike has quit IRC | 17:20 | |
*** salv-orlando has quit IRC | 17:21 | |
*** sarob has joined #openstack-infra | 17:21 | |
*** salv-orlando has joined #openstack-infra | 17:21 | |
samueldmq | fungi, ack, wanted ot update my name there (samuel-z), I didnt know why in the world I chose this at the time hehe | 17:23 |
samueldmq | fungi, thanks | 17:23 |
fungi | Kiall: are you able to approve https://review.openstack.org/179127 ? | 17:23 |
clarkb | samueldmq: keep in mind that typically the only person that ever sees that is you | 17:24 |
samueldmq | clarkb, hmm, so basically my lp id is what's seen outside | 17:24 |
fungi | i sort of wish gerrit just used the account id numbers for ssh and api access to make it obvious that it's arbitrary | 17:25 |
samueldmq | fungi, yeah lp's id ++ | 17:25 |
samueldmq | thanks for clarifying | 17:26 |
fungi | gerrit actually lets you search/select accounts based on full name, e-mail address and account id number (in addition to the ssh/api username) | 17:27 |
fungi | dhellmann: are you around to approve https://review.openstack.org/179127 ? | 17:28 |
*** Krinkle|detached is now known as Krinkle | 17:28 | |
SpamapS | I love how Gerrit is so hyper-enabled on search. Google roots showing. ;) | 17:28 |
*** Mmike has joined #openstack-infra | 17:28 | |
fungi | yeah, it's definitely a google-originated project | 17:28 |
*** tsg_ has joined #openstack-infra | 17:29 | |
*** Mmike has quit IRC | 17:29 | |
*** Mmike has joined #openstack-infra | 17:29 | |
*** yamahata has joined #openstack-infra | 17:29 | |
fungi | the obsession with feature-rich-yet-unusable web interfaces is yet another sign | 17:29 |
*** ivar-lazzaro has joined #openstack-infra | 17:29 | |
zaro | clarkb: will take a look | 17:29 |
pleia2 | fortunately the search is good and mega useful | 17:30 |
*** mrmartin has joined #openstack-infra | 17:30 | |
*** pelix has quit IRC | 17:33 | |
*** teran_ has quit IRC | 17:34 | |
*** SumitNaiksatam has quit IRC | 17:34 | |
*** teran has joined #openstack-infra | 17:34 | |
*** tnovacik has quit IRC | 17:34 | |
*** emagana has quit IRC | 17:35 | |
clarkb | fungi: where does TAG_MINOR come from? | 17:35 |
fungi | clarkb: first two components of the tag version | 17:35 |
clarkb | oh I think there is an order issue, will comment | 17:36 |
fungi | oh, further up. yeah i refactored after writing | 17:36 |
fungi | i see it | 17:36 |
fungi | should just be $TAG there | 17:36 |
clarkb | fungi: also, do you mean should never match not release? | 17:36 |
openstackgerrit | Davide Guerri proposed openstack-infra/shade: Add keystone services/endpoints methods https://review.openstack.org/177621 | 17:37 |
*** dims_ has joined #openstack-infra | 17:37 | |
fungi | basically, but without relying on a guarantee that there will be a first release of .0 | 17:37 |
*** rkukura_ has joined #openstack-infra | 17:37 | |
*** dboik_ has joined #openstack-infra | 17:37 | |
*** jtriley has joined #openstack-infra | 17:37 | |
fungi | the problem statement, which i've maybe not made clear, is that we want to merge 2015.1.0 to master, but not 2015.1.1 | 17:37 |
*** signed8b_ has joined #openstack-infra | 17:38 | |
*** _nadya__ has joined #openstack-infra | 17:38 | |
fungi | effectively, if there's already a 2015.1.something as the latest tag in master, don't merge any other 2015.1.otherthings from stable to master | 17:38 |
*** doug-fish1 has joined #openstack-infra | 17:38 | |
*** sdake_ has joined #openstack-infra | 17:38 | |
*** teran has quit IRC | 17:39 | |
*** BobH1 has joined #openstack-infra | 17:39 | |
fungi | because we should only ever once merge a tag from a particular stable branch to the master branch | 17:39 |
*** miqui_ has joined #openstack-infra | 17:39 | |
*** asselin__ has joined #openstack-infra | 17:40 | |
*** vryzhenkin has joined #openstack-infra | 17:40 | |
*** megm_ has joined #openstack-infra | 17:40 | |
*** tobiash_ has joined #openstack-infra | 17:41 | |
*** GheRiver1 has joined #openstack-infra | 17:41 | |
*** sslypush1nko has joined #openstack-infra | 17:41 | |
*** mugsie_ has joined #openstack-infra | 17:42 | |
*** neoice_ has joined #openstack-infra | 17:42 | |
*** fabo_ has joined #openstack-infra | 17:42 | |
*** grue_pm has joined #openstack-infra | 17:42 | |
*** jeblair_ has joined #openstack-infra | 17:42 | |
clarkb | fungi: question, didn't we have a script that worked on stbale for last cycle? | 17:43 |
fungi | we merged from proposed/juno to master | 17:43 |
fungi | then cut stable/juno from proposed/juno tip | 17:43 |
clarkb | gotcha | 17:43 |
*** HeOS has quit IRC | 17:44 | |
fungi | so tagging the release on an already existing stable/kilo is running afoul of our previous safeguards against merging tags from stable point releases to master | 17:44 |
fungi | hence the rethinking | 17:44 |
*** rcarrill` has joined #openstack-infra | 17:44 | |
*** ildikov has quit IRC | 17:45 | |
*** dmsimard_ has joined #openstack-infra | 17:45 | |
*** coolsvapl has joined #openstack-infra | 17:45 | |
*** erikmwilson_ has joined #openstack-infra | 17:45 | |
*** purp_2 has joined #openstack-infra | 17:45 | |
*** cburgess_ has joined #openstack-infra | 17:45 | |
*** sdague_ has joined #openstack-infra | 17:45 | |
*** dims has quit IRC | 17:46 | |
*** _nadya_ has quit IRC | 17:46 | |
*** heyongli has quit IRC | 17:47 | |
*** jtriley_ has quit IRC | 17:47 | |
*** kgiusti has quit IRC | 17:47 | |
*** ZZelle has quit IRC | 17:47 | |
*** weshay has quit IRC | 17:47 | |
*** erikmwilson has quit IRC | 17:47 | |
*** ayoung-afk has quit IRC | 17:47 | |
*** rcarrillocruz has quit IRC | 17:47 | |
*** megm has quit IRC | 17:47 | |
*** GheRivero has quit IRC | 17:47 | |
*** greghaynes has quit IRC | 17:47 | |
*** jpeeler has quit IRC | 17:47 | |
*** fabo has quit IRC | 17:47 | |
*** tobiash has quit IRC | 17:47 | |
*** cburgess has quit IRC | 17:47 | |
*** dmsimard has quit IRC | 17:47 | |
*** sdake has quit IRC | 17:47 | |
*** dboik has quit IRC | 17:47 | |
*** signed8bit has quit IRC | 17:47 | |
*** sslypushenko has quit IRC | 17:47 | |
*** jistr has quit IRC | 17:47 | |
*** miqui has quit IRC | 17:47 | |
*** BobH has quit IRC | 17:47 | |
*** doug-fish has quit IRC | 17:47 | |
*** jeblair has quit IRC | 17:47 | |
*** mugsie has quit IRC | 17:47 | |
*** purp has quit IRC | 17:47 | |
*** rkukura has quit IRC | 17:47 | |
*** vponomaryov has quit IRC | 17:47 | |
*** bradm has quit IRC | 17:47 | |
*** asselin_ has quit IRC | 17:47 | |
*** sdague has quit IRC | 17:47 | |
*** grue_pm_ has quit IRC | 17:47 | |
*** neoice has quit IRC | 17:47 | |
*** coolsvap|afk has quit IRC | 17:47 | |
*** mjturek1 has quit IRC | 17:47 | |
*** freerunner has quit IRC | 17:47 | |
*** grantbow has quit IRC | 17:47 | |
*** salv-orlando has quit IRC | 17:47 | |
*** sputnik13 has quit IRC | 17:47 | |
*** samueldmq has quit IRC | 17:47 | |
*** bnemec has quit IRC | 17:47 | |
*** jogo has quit IRC | 17:47 | |
*** mkoderer has quit IRC | 17:47 | |
*** morganfainberg has quit IRC | 17:47 | |
*** gibi has quit IRC | 17:47 | |
*** asilenkov has quit IRC | 17:47 | |
*** jesusaurus has quit IRC | 17:47 | |
*** plol has quit IRC | 17:47 | |
*** jd__ has quit IRC | 17:47 | |
*** ptoohill has quit IRC | 17:47 | |
*** vipul has quit IRC | 17:47 | |
*** devananda has quit IRC | 17:47 | |
*** rkukura_ is now known as rkukura | 17:47 | |
*** dmsimard_ is now known as dmsimard | 17:47 | |
*** gary-smith has quit IRC | 17:47 | |
*** sdague_ is now known as sdague | 17:47 | |
*** marcusvrn1 has joined #openstack-infra | 17:47 | |
*** angela-s has joined #openstack-infra | 17:47 | |
*** dannywilson has joined #openstack-infra | 17:47 | |
*** dannywilson has quit IRC | 17:47 | |
*** _nadya__ has quit IRC | 17:47 | |
*** angela-s has left #openstack-infra | 17:48 | |
*** marcusvrn has quit IRC | 17:48 | |
*** salv-orlando has joined #openstack-infra | 17:49 | |
*** samueldmq has joined #openstack-infra | 17:49 | |
*** sputnik13 has joined #openstack-infra | 17:49 | |
*** bnemec has joined #openstack-infra | 17:49 | |
*** jogo has joined #openstack-infra | 17:49 | |
*** mkoderer has joined #openstack-infra | 17:49 | |
*** morganfainberg has joined #openstack-infra | 17:49 | |
*** gibi has joined #openstack-infra | 17:49 | |
*** asilenkov has joined #openstack-infra | 17:49 | |
*** jesusaurus has joined #openstack-infra | 17:49 | |
*** plol has joined #openstack-infra | 17:49 | |
*** jd__ has joined #openstack-infra | 17:49 | |
*** ptoohill has joined #openstack-infra | 17:49 | |
*** vipul has joined #openstack-infra | 17:49 | |
*** devananda has joined #openstack-infra | 17:49 | |
*** otter768 has joined #openstack-infra | 17:51 | |
*** ZZelle has joined #openstack-infra | 17:52 | |
*** vponomaryov has joined #openstack-infra | 17:53 | |
*** jpeeler has joined #openstack-infra | 17:53 | |
*** jpeeler has joined #openstack-infra | 17:53 | |
*** ayoung-afk has joined #openstack-infra | 17:53 | |
*** weshay has joined #openstack-infra | 17:53 | |
*** sdake has joined #openstack-infra | 17:53 | |
*** greghaynes has joined #openstack-infra | 17:54 | |
*** bradm has joined #openstack-infra | 17:54 | |
dansmith | anteaya: http://198.175.100.33/173995/2/console_status.log.gz | 17:54 |
dansmith | anteaya: is intel pci sick? | 17:54 |
*** otter768 has quit IRC | 17:55 | |
*** mjturek1 has joined #openstack-infra | 17:55 | |
*** maishsk has joined #openstack-infra | 17:55 | |
*** kgiusti has joined #openstack-infra | 17:56 | |
*** sdake_ has quit IRC | 17:56 | |
*** SumitNaiksatam has joined #openstack-infra | 17:56 | |
clarkb | fungi: `echo "1.2\n1.2"|sort -n` I am not sure your test will work right | 17:56 |
clarkb | fungi: to get a literal newline you have to escape it right? | 17:56 |
fungi | clarkb: yep, i'm currently muddling through the dance i need to make that actually work | 17:57 |
fungi | just looking for a conceptual good-or-not. i know the syntax is problematic still | 17:57 |
*** maishsk has quit IRC | 17:57 | |
fungi | more is the idea of skipping when the minor version matches or sorts earlier going to have the effect that we want, or are there corner cases in our release workflow i'm not thinking of? | 17:58 |
*** harlowja has quit IRC | 17:58 | |
clarkb | fungi: no I think the intent is correct, just the implementation doesn't quite get there | 17:59 |
clarkb | fungi: I left my comments | 17:59 |
fungi | i'll take a look once i have the errors i know about corrected and the script actually confirmed looking. thanks for the sounding board | 17:59 |
*** ddieterly has joined #openstack-infra | 18:00 | |
dhellmann | fungi: sorry, but it looks like ttx took care of it | 18:01 |
ttx | dhellmann: of what? | 18:01 |
*** e0ne has joined #openstack-infra | 18:02 | |
dhellmann | ttx: approving the designate version bump | 18:02 |
fungi | the designate change | 18:02 |
ttx | oh the missing +2a | 18:02 |
sdague | dansmith: I'm not sure I've ever seen it pass | 18:02 |
ttx | right, figured I'd take the shortvut | 18:02 |
ttx | c | 18:02 |
sdague | but I've only been looking for a little while | 18:02 |
dansmith | sdague: it definitely has | 18:02 |
ttx | sdague: removed my Nova -2s since the change merged in Nova | 18:02 |
dansmith | sdague: at least, it has reported success | 18:02 |
ttx | sdague: still waiting for ACL restore | 18:02 |
*** yfried has joined #openstack-infra | 18:03 | |
*** hdd has joined #openstack-infra | 18:03 | |
*** spredzy|afk is now known as spredzy | 18:04 | |
*** annegentle has joined #openstack-infra | 18:05 | |
clarkb | fungi: you can use sort -t to set a separator | 18:07 |
fungi | clarkb: that's to set field separator, not line separator | 18:07 |
fungi | for use with -k to specify which field to key the sort on | 18:07 |
clarkb | oh right | 18:07 |
*** sdake_ has joined #openstack-infra | 18:08 | |
*** yfried has quit IRC | 18:09 | |
*** sdake has quit IRC | 18:10 | |
*** sdake has joined #openstack-infra | 18:10 | |
openstackgerrit | Jeremy Stanley proposed openstack-infra/project-config: Merge tags from stable branches to master https://review.openstack.org/179170 | 18:11 |
*** bswartz has quit IRC | 18:11 | |
fungi | that's ^ slightly more polished now | 18:11 |
fungi | grr, i missed a couple of my intended edits. just a sec | 18:11 |
*** EmilienM|afk is now known as EmilienM | 18:11 | |
clarkb | also nested echos gj | 18:11 |
*** emagana has joined #openstack-infra | 18:12 | |
*** bswartz has joined #openstack-infra | 18:12 | |
*** erikmwilson_ is now known as erikmwilson | 18:12 | |
clarkb | and since we mostly semver the major.minor should always be numerically sortable | 18:13 |
fungi | the nested echoes are hacky, i know. open to alternatives. i couldn't find simple shell commands to sort a list of terms on one line (with an arbitrary separator or even just whitespace), nor could i recall a good tool for combining multiple lines into one space-separated line | 18:13 |
clarkb | fungi: I was going to suggest just writing to files | 18:13 |
fungi | oh, i could probably use heredocs for that | 18:13 |
fungi | would also be a little hacky | 18:14 |
openstackgerrit | Jeremy Stanley proposed openstack-infra/project-config: Merge tags from stable branches to master https://review.openstack.org/179170 | 18:14 |
clarkb | ya either way its hacky so not a big deal | 18:14 |
fungi | that's ^ with the missing comment correction | 18:14 |
*** sdake_ has quit IRC | 18:14 | |
fungi | the numeric-only assumption is the reason for that added sanity check at the top of the script too | 18:15 |
clarkb | fungi: doesn't this run in release always though? | 18:15 |
clarkb | fungi: I am still confused about that comment | 18:15 |
fungi | it does (_should_ at least) | 18:15 |
clarkb | right but comment says "should never match the release pipeline trigger" | 18:15 |
fungi | that comment was intended to mean that if we accidentally trigger this script on a non-numeric tag, safely bail out rather than potentially confusing the logic further down | 18:16 |
clarkb | my inline note has a suggested edit | 18:16 |
*** Swami has joined #openstack-infra | 18:16 | |
fungi | while acknowledging that the pattern matches we use for the release pipeline trigger should avoid us ever hitting that condition, but better safe than sorry | 18:16 |
*** tnovacik has joined #openstack-infra | 18:17 | |
openstackgerrit | Ben Swartzlander proposed openstack-infra/project-config: Allow manila-release group to push tags to manila-ui https://review.openstack.org/179194 | 18:17 |
clarkb | fungi: maybe "this tag should only ever match the release pipeline trigger regex, a series of >=1 numberic values separated by .'s | 18:17 |
clarkb | " | 18:17 |
fungi | i suppose i could have just been more verbose. ugly habit of trying to avoid multiline comments | 18:17 |
*** ivar-laz_ has joined #openstack-infra | 18:17 | |
clarkb | fungi: well its specifically says it should not match the release pipeline | 18:17 |
clarkb | fungi: except ti should only match the release pipeline | 18:17 |
fungi | a tag matching this exit condition should never match the release pipeline pattern either | 18:18 |
clarkb | ah, I read it as being our safety check is that the tag always matches the release pipeline trigger | 18:18 |
fungi | (hence redundant but validating assumptions elsewhere in the script) | 18:18 |
*** _nadya_ has joined #openstack-infra | 18:18 | |
*** tkelsey has quit IRC | 18:18 | |
fungi | i'll just use more words | 18:19 |
clarkb | and if it doesn't then we bail out | 18:19 |
*** Sukhdev has quit IRC | 18:20 | |
*** ivar-laz_ has quit IRC | 18:20 | |
*** baoli_ has quit IRC | 18:20 | |
*** Sukhdev has joined #openstack-infra | 18:20 | |
*** spzala has quit IRC | 18:21 | |
*** ivar-lazzaro has quit IRC | 18:21 | |
*** baoli_ has joined #openstack-infra | 18:21 | |
*** rm_work|away is now known as rm_work | 18:23 | |
openstackgerrit | Jeremy Stanley proposed openstack-infra/project-config: Merge tags from stable branches to master https://review.openstack.org/179170 | 18:24 |
fungi | better? | 18:24 |
*** _2_Pixie has joined #openstack-infra | 18:24 | |
*** ivar-lazzaro has joined #openstack-infra | 18:24 | |
clarkb | much thanks | 18:25 |
clarkb | also how do we trade git tags for diamonds? | 18:25 |
*** baoli_ has quit IRC | 18:25 | |
fungi | on the tag commodities exchange, obviously | 18:25 |
*** Somay has joined #openstack-infra | 18:25 | |
fungi | also corn bellies and hog futures | 18:26 |
clarkb | fungi: one last thing | 18:26 |
*** ajo has quit IRC | 18:26 | |
fungi | you sound like columbo | 18:26 |
clarkb | I posted the comment to the change, but the nested echos are not quite right | 18:27 |
*** _nadya_ has quit IRC | 18:27 | |
dtroyer | fungi, that means clarkb thinks you did it…. | 18:27 |
fungi | clarkb: nope, it's after a [ | 18:28 |
*** _2_Pixie has quit IRC | 18:28 | |
*** sushilkm has joined #openstack-infra | 18:29 | |
*** sushilkm has left #openstack-infra | 18:29 | |
clarkb | fungi: derp /me flips vote | 18:30 |
*** mrmartin has quit IRC | 18:30 | |
*** dannywilson has joined #openstack-infra | 18:30 | |
fungi | it runs echo -e "$MASTER_MINOR\n$TAG_MINOR"|sort -n which returns two lines as a string, then echoes that two-line string which coalesces them into one line, and the output of that results in the string wrapped in outer "" marks (i could have used escaped inner quoting but it actually winds up evaluating sufficiently without doing that) | 18:30 |
*** mattfarina has quit IRC | 18:31 | |
fungi | and then [] is comparing that = to the reference string | 18:32 |
clarkb | yup | 18:32 |
*** miqui_ has quit IRC | 18:32 | |
fungi | "figure out whether string a sorts after string b" turns out to be a mildly annoying problem to solve in shell. i'm sure someone has a saner implementation | 18:33 |
clarkb | use lexicographic ordering with < and > | 18:34 |
*** rm_work is now known as rm_work|away | 18:35 | |
clarkb | in this case it would work except for the clumps of digits being of arbitrary size | 18:35 |
*** esmute has quit IRC | 18:35 | |
clarkb | silly versions | 18:35 |
SpamapS | $ [[ "a" < "b" ]] && echo yay | 18:35 |
SpamapS | yay | 18:35 |
*** patrickeast has joined #openstack-infra | 18:35 | |
*** esmute has joined #openstack-infra | 18:35 | |
SpamapS | fungi: ^ ? | 18:36 |
fungi | oh, bash builtins can do quantitative comparisons of non-numeric strings? neat | 18:36 |
clarkb | ya but its the wrong sort order | 18:36 |
clarkb | I think | 18:36 |
clarkb | see [[ "1.2" < "11.2" ]] | 18:37 |
clarkb | that returns 1 | 18:37 |
SpamapS | $ [[ "A" < "b" ]] && echo yay | 18:37 |
SpamapS | yay | 18:37 |
SpamapS | it is lexical | 18:37 |
SpamapS | -gt is numeric only | 18:37 |
SpamapS | also decimals are not supported | 18:38 |
fungi | oh, actually i think my sort -n may also do the wrong thing with version subcomponents thinking they're floats. checking | 18:38 |
clarkb | oh right 1.2 < 1.20 would be false | 18:38 |
fungi | ah, sort -n dtrt | 18:39 |
fungi | good | 18:39 |
SpamapS | fungi: honestly, version comparison in shell is best done using dpkg --compare-versions .. ;) | 18:39 |
fungi | er, wait, no, it doesn't. poo | 18:39 |
clarkb | does sort -g work? | 18:39 |
fungi | SpamapS: especially if we assume we'll only run this on platforms where we install dpkg | 18:39 |
openstackgerrit | Khai Do proposed openstack-infra/project-config: Use zuul-cloner instead of git clone https://review.openstack.org/167816 | 18:40 |
fungi | clarkb: nope. i can't get 12.333 to sort after 12.34 | 18:40 |
clarkb | ya hrm | 18:40 |
fungi | i can work around it with a second sort -k2 -n -t'.' | 18:41 |
fungi | er, -t. | 18:41 |
SpamapS | fungi: dpkg is available on fedora :) | 18:42 |
SpamapS | ohh | 18:43 |
SpamapS | sort -V | 18:43 |
clarkb | oh! | 18:43 |
fungi | woah | 18:43 |
SpamapS | like holy crap | 18:43 |
clarkb | yup sort -V works | 18:43 |
fungi | nowai! | 18:43 |
SpamapS | where has this been all my life? | 18:43 |
* SpamapS drops the mic | 18:43 | |
fungi | it's even there on ubuntu 12.04 | 18:44 |
fungi | SCORE | 18:44 |
*** spzala has joined #openstack-infra | 18:44 | |
openstackgerrit | Jeremy Stanley proposed openstack-infra/project-config: Merge tags from stable branches to master https://review.openstack.org/179170 | 18:45 |
* fungi hands SpamapS a kewpie doll | 18:45 | |
SpamapS | I will love him and hug him and name him Stanley. | 18:46 |
fungi | now to figure out how i can shield the brain cells where i stored this useful trick against the impending onslaught of beer consumption | 18:47 |
*** rm_work|away is now known as rm_work | 18:48 | |
clarkb | fungi: :) you going to celebrate the release too? I am headed to breakside this afternoon | 18:48 |
* clarkb reviews latest patch | 18:48 | |
fungi | i plan to find out if the o'connor brewing "el guapo" agave ipa is still on tap at the cantina up the road | 18:48 |
*** maishsk has joined #openstack-infra | 18:48 | |
fungi | my current favorite | 18:49 |
fungi | brewed just a few dozen miles upshore from here in norfolk | 18:49 |
fungi | hoping to visit their taproom and catch a tour of the brewery next time i need to go to norfolk for a flight | 18:50 |
clarkb | does the agave go in before or after fermentation? | 18:50 |
fungi | that's what i want to find out ;) | 18:51 |
greghaynes | what, agave in fermentation? | 18:51 |
greghaynes | I guess yeast like sugar of any kind ;) | 18:51 |
fungi | definitely on the list of questions i plan to ask during a tour | 18:52 |
pleia2 | fungi: ooh, that sounds good | 18:52 |
*** annegentle has quit IRC | 18:52 | |
mriedem | fungi: https://review.openstack.org/#/q/branch:stable/kilo+topic:next-kilo,n,z is all merged now | 18:53 |
mriedem | per https://review.openstack.org/#/c/179045/ | 18:53 |
fungi | anyway, i was thrilled to find it on tap at the taco joint just up the road. goes great with fish burritos and jicama slaw | 18:53 |
*** nfedotov has quit IRC | 18:53 | |
fungi | mriedem: thanks! i'll get the acl revert un-wipped and reviewd | 18:53 |
lifeless | fungi: ttx: thanks | 18:54 |
*** Somay has quit IRC | 18:54 | |
clarkb | fungi: where is acl revert? | 18:54 |
mordred | ola all | 18:54 |
lifeless | is the release all finished? | 18:55 |
lifeless | ttx: ^ | 18:55 |
clarkb | lifeless: not quite, we have to get acls configured for stable branches then merge tags for release back into master | 18:55 |
fungi | clarkb: https://review.openstack.org/179045 | 18:55 |
lifeless | clarkb: ah... so as soon as it is let me know and I'll release pbr | 18:55 |
lifeless | dims_: tag - I have it | 18:55 |
clarkb | lifeless: can do | 18:56 |
dims_ | ++ lifeless | 18:56 |
lifeless | clarkb: thanks | 18:56 |
*** eharney has quit IRC | 18:56 | |
fungi | also bashate is displeased by our merge-tags script. now i also need to update it to match that view of reality... sigh | 18:56 |
*** nfedotov has joined #openstack-infra | 18:56 | |
*** maishsk has quit IRC | 18:58 | |
*** maishsk has joined #openstack-infra | 18:58 | |
openstackgerrit | Jeremy Stanley proposed openstack-infra/project-config: Merge tags from stable branches to master https://review.openstack.org/179170 | 18:58 |
openstackgerrit | David Shrewsbury proposed openstack-infra/shade: Test OperatorCloud doc https://review.openstack.org/179206 | 18:59 |
*** BharatK has quit IRC | 18:59 | |
fungi | mordred: while you're here, thoughts on https://review.openstack.org/179170 ? apparently our merge-tags-to-master job needs revisiting now that we're tagging the release on stable rather than proposed | 19:00 |
fungi | the trick is matching tags that would have originally appeared on the proposed branch without matching tags that would previously have only been done on stable post-release | 19:01 |
clarkb | fungi: acls lgtm, git grep doesn't show any other files with kilo acls | 19:01 |
*** dhp has joined #openstack-infra | 19:01 | |
*** abregman has joined #openstack-infra | 19:02 | |
*** cody-somerville has joined #openstack-infra | 19:02 | |
lifeless | clarkb: / fungi: / mordred: - http://logs.openstack.org/82/178982/2/check/check-pbr-installation-dsvm/e19a6d4/console.html#_2015-04-30_10_31_34_211 | 19:03 |
lifeless | opt/stack/new doesn't have everything from projects in it | 19:03 |
lifeless | is that expected? | 19:03 |
*** Somay has joined #openstack-infra | 19:03 | |
clarkb | pleia2: any chane you can review 179045 ? | 19:04 |
fungi | lifeless: yeah, the list of things to copy to /opt/stack/new is specified in devstack-gate i think | 19:04 |
*** jd__ has quit IRC | 19:04 | |
pleia2 | clarkb: yep, almost done | 19:05 |
fungi | lifeless: getting you a link now | 19:05 |
pleia2 | clarkb: actually, I'm looking at fungi's right now, then that one | 19:05 |
lifeless | ok, if its a known subset | 19:05 |
pleia2 | er, fungi's 179170 | 19:05 |
lifeless | I can just make discovery skip missing repos | 19:05 |
fungi | lifeless: hacking is not in the list at https://git.openstack.org/cgit/openstack-infra/devstack-gate/tree/devstack-vm-gate-wrap.sh#n35 | 19:05 |
clarkb | yes the PROJECTS list | 19:05 |
clarkb | we can add it to the PROJECTS list in the job too | 19:06 |
*** jd__ has joined #openstack-infra | 19:06 | |
lifeless | clarkb: so the script uses | 19:06 |
lifeless | PROJECTS=$* | 19:06 |
lifeless | and $* is from $(cat requirements/projects) in the job yaml | 19:06 |
lifeless | I don't particularly care if hacking is there or not, just that my replacement code needs to handle missing repos | 19:07 |
*** maishsk has quit IRC | 19:07 | |
clarkb | lifeless: that doesn't appear to be given to devstack-gate | 19:07 |
fungi | should probably clone/copy them from /opt/git if needed | 19:07 |
clarkb | https://git.openstack.org/cgit/openstack-infra/project-config/tree/jenkins/jobs/requirements.yaml#n48 is where it would need to be set | 19:08 |
*** maishsk has joined #openstack-infra | 19:08 | |
fungi | but yeah, the job could set that before calling into devstack-vm-gate-wrap.sh unless there's a chicken-and-egg problem there | 19:08 |
lifeless | if you look at | 19:08 |
lifeless | http://logs.openstack.org/52/178852/3/gate/gate-pbr-installation-dsvm/f8a531d/console.html | 19:08 |
lifeless | its currently skipping it | 19:08 |
lifeless | so there are two distinct things | 19:08 |
lifeless | a) the current job silently skips, my new code should do the same | 19:08 |
pabelanger | well, that was not too hard. Getting grafana to consume graphite data provided by -infra | 19:08 |
lifeless | b) should we test more - perhaps, but let me fix the perf issue first | 19:08 |
clarkb | lifeless: skipping what? | 19:09 |
fungi | lifeless: sounds good | 19:09 |
fungi | clarkb: previous iteration of the job skipped repos which weren't in /opt/stack/new | 19:09 |
clarkb | oh I see, but ya just set PROJECTS for d-g and the test consistently and problem should be solved | 19:09 |
fungi | so it was specifically only testing for breakage against things in the intersection of the requirements project list and default devstack-gate project list | 19:10 |
*** radez_g0n3 is now known as radez | 19:10 | |
lifeless | clarkb: clarkb http://logs.openstack.org/52/178852/3/gate/gate-pbr-installation-dsvm/f8a531d/console.html#_2015-04-30_04_12_07_612 | 19:10 |
lifeless | fungi: which is fine | 19:10 |
clarkb | lifeless: that is at the wrong place | 19:10 |
lifeless | I don't mind what we test I was just establishing the current behaviour | 19:10 |
clarkb | lifeless: /opt/stack/new is cloned by devstack-gate, so we have to export that variable for devstack-gate before running devstack-gate | 19:10 |
*** pblaho has joined #openstack-infra | 19:11 | |
fungi | sounded like that was his phase 2 plan | 19:11 |
lifeless | clarkb: sure, but the script handles it differently | 19:11 |
pleia2 | fungi, clarkb: +2 on 179045 & 179170, didn't +A yet in case we want more eyes or need to wait for any reason, but I can if we want to go ahead | 19:11 |
fungi | where phase 1 is to do what the old job did and not regress | 19:11 |
lifeless | I'm just aiming for bug compaitibility | 19:11 |
mordred | fungi: oh - lovely | 19:12 |
fungi | pleia2: we're probably fine to merge both. they're release wrap-up blockers, but getting mordred to skim the merge-tags script change first would help if he's got a moment | 19:12 |
*** xyang1 has quit IRC | 19:12 | |
clarkb | lifeless: differently to what? | 19:12 |
fungi | since previous iterations were his baileywick | 19:12 |
clarkb | I am either not keeping up or we are using too many implied references to nouns | 19:12 |
fungi | er, bailiwick | 19:13 |
pleia2 | fungi: ok great, just let 179045 go then | 19:13 |
lifeless | clarkb: don't worry, my q has been answered. | 19:13 |
mordred | fungi: master_minor is going to fail | 19:13 |
lifeless | clarkb: you have release stuff to do :) - don't page that out | 19:13 |
mordred | fungi: when we move from 2015.2 to 2016.1 | 19:13 |
mordred | fungi: no | 19:13 |
mordred | ? | 19:13 |
clarkb | mordred: it should handle that properly | 19:13 |
* clarkb goes back to reread script | 19:13 | |
fungi | mordred: 2016.1 should sort -V after 2015.2 and get merged | 19:14 |
*** ildikov has joined #openstack-infra | 19:14 | |
openstackgerrit | lifeless proposed openstack-dev/pbr: Parallelise integration tests. https://review.openstack.org/178982 | 19:14 |
clarkb | first check will be is 2016.1 on master already? and it will pass that | 19:14 |
openstackgerrit | David Shrewsbury proposed openstack-infra/shade: Add minor OperatorCloud documentation https://review.openstack.org/179206 | 19:14 |
openstackgerrit | lifeless proposed openstack-dev/pbr: Parallelise integration tests. https://review.openstack.org/178982 | 19:14 |
fungi | mordred: or did you miss that it's cut -f-2 not -f2? | 19:15 |
ttx | fungi: looks like we are a go for the ACL revert | 19:15 |
clarkb | then it will check if tag version sorts before master version but it won't because 2015.2 < 2016.1 | 19:15 |
mordred | fungi: AH | 19:15 |
pleia2 | ttx: good, because I just approved it :) | 19:15 |
mordred | fungi: the comment is confusing | 19:15 |
mordred | fungi: I do not need you to change it | 19:15 |
ttx | pleia2: haha | 19:15 |
fungi | mordred: my comments seem to have been plenty confusing today apparently so i'm not surprised by this ;) | 19:15 |
mordred | fungi: but a follow up patch that clarifies - "minor matches master" made me see cut -f2 | 19:16 |
mordred | since I thought it was trying to only match master | 19:16 |
mordred | the code looks good | 19:16 |
fungi | thanks for the double-check | 19:16 |
fungi | i'm lacking a good term for whatever "2015.1" represents rather than just the "1" part | 19:16 |
clarkb | mordred: basically sort -V is magic | 19:16 |
mordred | fungi: yah. I mean, yeah | 19:17 |
mordred | clarkb: also - yeah it is | 19:17 |
mordred | clarkb: I did not know about it until the scrollback here | 19:17 |
*** marun has quit IRC | 19:17 | |
fungi | i guess those variables hold major+minor (as opposed to major+minor+patch) | 19:17 |
fungi | so better naming might help too | 19:17 |
openstackgerrit | yolanda.robla proposed openstack-infra/jenkins-job-builder: Added parallelization options https://review.openstack.org/75514 | 19:18 |
fungi | mordred: i think we somehow all stepped into an alternate universe where sort -V has somehow existed for a long time. that's the only explanation i can find | 19:18 |
mordred | fungi: they actually are YEAR.COUNTER not MAJOR.MINOR | 19:18 |
mordred | fungi: which is why it's hard to find the right language :) | 19:19 |
fungi | mordred: for 2015.1 sure, but not for 1.2 | 19:19 |
mordred | yah | 19:19 |
fungi | and this script needs to deal with both (luckily that's easy) | 19:19 |
openstackgerrit | Merged openstack-infra/project-config: Revert "Temporarily copy proposed/* ACLs to stable/kilo" https://review.openstack.org/179045 | 19:19 |
fungi | we treat year as major and counter as minor | 19:19 |
clarkb | fungi: until we decide to stop using years | 19:19 |
clarkb | if we do that I vote for a version bump to 9000 | 19:20 |
clarkb | then the next release will be over 9000 | 19:20 |
fungi | once we cycle back around to h | 19:20 |
fungi | so we can have a hal 9000 release | 19:20 |
fungi | but then, yeah, subsequent releases will be "over 9000" | 19:20 |
clarkb | mordred: are you still reviewing 179170? | 19:21 |
*** ZZelle_ has joined #openstack-infra | 19:21 | |
clarkb | mordred: you said code looks good but didn't vote so double checking | 19:22 |
mordred | clarkb: +2 from me | 19:22 |
fungi | once we get that merged, it will get puppeted directly onto proposal.slave and we can retrigger the jobs for it | 19:22 |
clarkb | cool | 19:22 |
*** marun has joined #openstack-infra | 19:22 | |
clarkb | at this point should we +A it? | 19:22 |
fungi | thankfully no need to rebuild nodepool images for once | 19:22 |
clarkb | maybe sdague wants to review it too? | 19:22 |
fungi | i think so, yes | 19:22 |
fungi | if sdague is around, then definitely | 19:22 |
*** maishsk has quit IRC | 19:23 | |
fungi | i'll retrigger one job first with the updated script and keep an eye on the results before i blast through the rest | 19:23 |
clarkb | fungi: also looks like we have tags so we would just manually trigger it ya? | 19:23 |
fungi | yep. the jobs even already ran once, they just no-op'd | 19:23 |
clarkb | ++ to running through with one project | 19:23 |
pabelanger | Is there issues using git-review with the -D flag? | 19:23 |
fungi | panon our gerrit there is because we have crippled drafts | 19:24 |
pabelanger | getting cannot upload drafts rejection | 19:24 |
*** nfedotov has quit IRC | 19:24 | |
fungi | er, tab complete fail | 19:24 |
clarkb | pabelanger: we disable draft uploads because there are issues with drafts in gerrit | 19:24 |
pabelanger | clarkb, thanks, was trying to find info about it | 19:24 |
clarkb | pabelanger: they are insecure and cause headaches with missing changes and missing patchsts that result in confusion | 19:24 |
pabelanger | roger | 19:25 |
pabelanger | can you flag WIP from git-review directly? | 19:25 |
*** eharney has joined #openstack-infra | 19:25 | |
pabelanger | I assume no | 19:25 |
fungi | "disable" is a little too clean for what we do. we've got an acl in place which causes attempts to upload drafts to break horribly with a rejection | 19:25 |
openstackgerrit | Paul Belanger proposed openstack-infra/system-config: Initial commit for grafana https://review.openstack.org/179208 | 19:25 |
fungi | pabelanger: our "wip" (workflow -1) workaround isn't strictly speaking a gerrit feature, so we've resisted attempts to encode openstack-specific gerrit workflow asumptions into that tool | 19:26 |
clarkb | gerrit will tell you they are insecure but many users expect drafts to be truly private. Also if you have a series of commits and the parent is a draft nothing merges and no one knows why. Or if you have multiples patchsets and the latest is a draft no reviews can vote and no one knows why | 19:26 |
fungi | er, assumptions | 19:26 |
pabelanger | fungi, fair enough | 19:26 |
*** maishsk has joined #openstack-infra | 19:26 | |
*** sabeen1 has joined #openstack-infra | 19:26 | |
*** nfedotov has joined #openstack-infra | 19:26 | |
fungi | pabelanger: there's an effort underway to get a change state for "work in progress" implemented as a gerrit plug-in, and i expect git-review to support that once it's working | 19:27 |
openstackgerrit | greghaynes proposed openstack-infra/project-config: Move DIB test dep installation into DIB https://review.openstack.org/179209 | 19:27 |
openstackgerrit | Paul Belanger proposed openstack-infra/system-config: Initial commit for grafana https://review.openstack.org/179208 | 19:27 |
fungi | but in the interim, you can alias a command to make an api call leaving a workflow -1 vote on your change right after you submit it | 19:28 |
clarkb | fungi: I don't see sdague recently on IRC I say we should just approve it now | 19:28 |
fungi | or just use the webui, gertty, whatever to do it manually | 19:28 |
pabelanger | fungi, Oh, nice. Any docs on the API call? | 19:28 |
clarkb | I imagine sdague is celebrating the release a bit earlier than we are :) | 19:28 |
fungi | pabelanger: it's "gerrit review" | 19:28 |
pabelanger | ah | 19:28 |
fungi | pabelanger: https://review.openstack.org/Documentation/cmd-review.html | 19:29 |
pabelanger | fungi, great | 19:29 |
* clarkb calls it abd approves, we all want to get to the post release beers and pbr releases | 19:30 | |
mordred | woot! | 19:30 |
fungi | pabelanger: review --label Workflow=-1 <change_number> | 19:30 |
mordred | pabelanger: hey - have you chatted with jeblair_ and/or notnownikki about grafana? | 19:30 |
fungi | clarkb: as long as the beer and pbr are not synonymous in this case | 19:30 |
pleia2 | fungi: ++ | 19:30 |
clarkb | fungi: >_> | 19:30 |
mordred | pabelanger: I know they just chatted in person about a strategy aroudn that and may not have updated other fokls on it yet | 19:31 |
*** maishsk_ has joined #openstack-infra | 19:31 | |
pabelanger | So, I'm playing around with grafana, to allow users to write dashboards and such. However, first big issue is no support for external SSO (except github and google). Do we have any other o.o services that don't use SSO from launchpad? | 19:31 |
*** maishsk has quit IRC | 19:31 | |
*** maishsk_ is now known as maishsk | 19:31 | |
pabelanger | mordred, no | 19:31 |
pabelanger | err | 19:31 |
pabelanger | nope | 19:31 |
pabelanger | something I've been meaning to do | 19:31 |
*** dannywil_ has joined #openstack-infra | 19:31 | |
fungi | pabelanger: no general openid support i guess? | 19:31 |
*** dguerri is now known as _dguerri | 19:32 | |
*** akshai has joined #openstack-infra | 19:32 | |
pabelanger | mordred, I mentioned something yesterday about doing it. anteaya mentioned there was some talks, but nobody specific | 19:32 |
*** xyang1 has joined #openstack-infra | 19:32 | |
pabelanger | fungi, not that I see | 19:32 |
mordred | pabelanger: cool - so, the tl;dr from that is to not allow TTW graph creation - but instead write a little thing that lets us store graph descriptions in yaml and then on changes to the yaml file write the graphs to the mysql db directly | 19:32 |
*** heyongli has joined #openstack-infra | 19:32 | |
clarkb | mordred: wait what | 19:32 |
pabelanger | mordred, Interesting | 19:32 |
*** fabo_ is now known as fabo | 19:32 | |
mordred | that way we can manage the graph content like our other graph content | 19:32 |
* clarkb assumes that config is the database and moves along | 19:33 | |
*** ajo has joined #openstack-infra | 19:33 | |
pabelanger | ya, that would be nice | 19:33 |
openstackgerrit | yolanda.robla proposed openstack-infra/jenkins-job-builder: Added parallelization options https://review.openstack.org/75514 | 19:33 |
mordred | also means we don't have to solve SSO :) | 19:33 |
pabelanger | and help control ACLs | 19:33 |
*** kevinbenton has quit IRC | 19:33 | |
*** hamzy has quit IRC | 19:33 | |
*** wolsen has quit IRC | 19:33 | |
*** d0ugal has quit IRC | 19:33 | |
*** grantbow has joined #openstack-infra | 19:33 | |
mordred | yah | 19:33 |
*** wolsen_ has joined #openstack-infra | 19:33 | |
mordred | pabelanger: but I think your patch to get one running at all is relevant in either case | 19:33 |
*** grantbow has quit IRC | 19:33 | |
*** grantbow has joined #openstack-infra | 19:33 | |
*** trey has quit IRC | 19:33 | |
*** cburgess_ has quit IRC | 19:34 | |
pabelanger | mordred, ya, it was pretty easy to get the base going. Found upstream puppet module, and 2hrs later it is working | 19:34 |
mordred | \o/ | 19:34 |
pabelanger | still need to do http proxy / ssl | 19:34 |
pabelanger | but working | 19:34 |
*** dannywilson has quit IRC | 19:34 | |
*** cburgess has joined #openstack-infra | 19:34 | |
*** d0ugal has joined #openstack-infra | 19:34 | |
mordred | well, if we wind up going yaml route - ssl probably not needed | 19:34 |
pabelanger | /me nods | 19:34 |
mordred | althoguh http proxy always good idea | 19:34 |
mordred | because apache | 19:34 |
*** trey has joined #openstack-infra | 19:34 | |
*** hamzy has joined #openstack-infra | 19:35 | |
*** kevinbenton has joined #openstack-infra | 19:35 | |
*** esker has quit IRC | 19:36 | |
mordred | pabelanger: did I hear correctly that you'll be in vancouver? | 19:36 |
pabelanger | mordred, correct. Planning on shadowing -infra while I'm there. | 19:36 |
pabelanger | and having a beer or 3 | 19:36 |
mordred | woot! | 19:36 |
openstackgerrit | lifeless proposed openstack-dev/pbr: Parallelise integration tests. https://review.openstack.org/178982 | 19:36 |
mordred | we like both of those things | 19:36 |
pabelanger | excellent | 19:37 |
fungi | neat... so per the ml, someone actually took the jocular suggestions about reimplementing openstack in go seriously? | 19:37 |
pleia2 | fungi: it was only a matter of time | 19:38 |
lifeless | well | 19:38 |
lifeless | for all I hate go, its a better fit for the scale | 19:38 |
clarkb | lifeless: except that I am pretty sure it won't build as pushed <_< | 19:38 |
lifeless | unless we dogpile on mypy and a bunch more static analysis tooling | 19:39 |
mordred | I really want rust to have a release that the distros package - because I'd liek to suggest it - but given no installable toolchain without weird, it's kind of hard to | 19:39 |
clarkb | mordred: you don't like curl | bash? | 19:39 |
mordred | clarkb: I like it fine ... but _you_ don't :) | 19:39 |
openstackgerrit | yolanda.robla proposed openstack-infra/puppet-cgit: Manage cgitrc config file https://review.openstack.org/179132 | 19:40 |
*** maishsk has quit IRC | 19:40 | |
openstackgerrit | Merged openstack-infra/project-config: Merge tags from stable branches to master https://review.openstack.org/179170 | 19:40 |
openstackgerrit | lifeless proposed openstack/requirements: Add virtualenv to global requirements https://review.openstack.org/179212 | 19:40 |
fungi | i'm just waiting for the follow-up announcement that someone else is rewriting openstack swift in apple swift | 19:40 |
clarkb | fungi: were you planning on waiting for the next puppet run or doing that update by hand? | 19:40 |
fungi | clarkb: since it's merged, i'm planning to pull it down manually | 19:41 |
clarkb | fungi: just be careful puppet doesn't change it back | 19:41 |
clarkb | oh except its in project-config so maybe thats not an issue | 19:41 |
clarkb | system-config has that problem project-config does not | 19:41 |
fungi | yeah it won't | 19:42 |
fungi | if it were system-config i'd update teh git repo on puppetmaster before proceeding to puppet agent --test on the client | 19:42 |
fungi | but it's not | 19:42 |
fungi | /usr/local/jenkins/slave_scripts/merge_tags.sh is updated | 19:44 |
*** tonytan4ever has quit IRC | 19:44 | |
fungi | now to test retriggering a job for that | 19:44 |
* clarkb votes for not nova | 19:45 | |
fungi | heh, no | 19:45 |
openstackgerrit | yolanda.robla proposed openstack-infra/puppet-cgit: Manage cgitrc config file https://review.openstack.org/179132 | 19:45 |
pleia2 | cinerama: if you're around, I believe zanata bug 1214507 is mostly related to admin accounts, right? we need to follow up because I think we do need openid to stay locked down https://bugzilla.redhat.com/show_bug.cgi?id=1214507 | 19:45 |
openstack | bugzilla.redhat.com bug 1214507 in Authentication-OpenID "Can't use internal logins with a specified OpenID provider" [Unspecified,New] - Assigned to djansen | 19:45 |
openstack | bug 1196625 in software-center (Ubuntu) "duplicate for #1214507 software-center crashed with signal 5 in _XReadEvents()" [Medium,Triaged] https://launchpad.net/bugs/1196625 | 19:45 |
* pleia2 gives openstack a beer | 19:46 | |
*** mtanino has quit IRC | 19:47 | |
openstackgerrit | Jay Pipes proposed openstack-infra/project-config: Add vmware-dvs project into stackforge https://review.openstack.org/173289 | 19:48 |
*** hdd has quit IRC | 19:48 | |
*** hdd_ has joined #openstack-infra | 19:48 | |
*** devvesa has quit IRC | 19:49 | |
*** otter768 has joined #openstack-infra | 19:52 | |
*** sabeen2 has joined #openstack-infra | 19:52 | |
fungi | clarkb: argh, minor overisght | 19:52 |
bknudson | I thought you were going to say apple hypercard | 19:52 |
cinerama | pleia2: yes, i meant to follow up on that yesterday but coding stuff | 19:52 |
clarkb | fungi uh oh | 19:53 |
*** _nadya_ has joined #openstack-infra | 19:53 | |
fungi | clarkb: git describe on master reports 2015.1.0rc1-54-g325a3b6 | 19:53 |
clarkb | ha | 19:53 |
pleia2 | cinerama: no worries, I can follow-up | 19:53 |
fungi | clarkb: this is why i like regular human-ordered version numbers | 19:53 |
fungi | time for more thinkey-think | 19:54 |
*** marun has quit IRC | 19:54 | |
clarkb | i have caffeine and lunch now for brain fuel | 19:54 |
clarkb | we could use pbr | 19:54 |
cinerama | pleia2: i've updated | 19:55 |
*** sabeen1 has quit IRC | 19:55 | |
clarkb | or pkg resources | 19:55 |
fungi | clarkb: have the script install pbr to a virtualenv and perform version component comparisons? | 19:55 |
pleia2 | cinerama: oh great, thanks :) | 19:55 |
*** hdd_ has quit IRC | 19:55 | |
clarkb | fungi ya | 19:55 |
lifeless | fungi: clarkb: what versions do you need to compare? | 19:56 |
*** marun has joined #openstack-infra | 19:56 | |
lifeless | there's a generalised pep-440 library (packaging) that you can use - pbr doesn't because vendoring && no-deps. | 19:56 |
*** otter768 has quit IRC | 19:56 | |
fungi | lifeless: i need to decide whether the version that's been tagged on stable/foo sorts later than the most recent tag on master, but not if there's already a tag on master with the same major+minor | 19:56 |
lifeless | fungi: huh. why? | 19:57 |
fungi | e.g. we want to merge 2015.1.0 from stable/kilo to master, but when the stable branch managers tag 2015.1.1 we want to not merge that to master | 19:57 |
lifeless | huh | 19:58 |
fungi | we could bake in the assumption that any .0 release tag should be merged to master but not others | 19:58 |
lifeless | so however you describe it | 19:58 |
lifeless | that sounds like you want a little script to encode the business logic | 19:58 |
fungi | lifeless: we have it | 19:58 |
fungi | it's just wrong | 19:58 |
*** jd__ has quit IRC | 19:58 | |
lifeless | ah | 19:59 |
fungi | trying to figure out the best way to make it less-wrong | 19:59 |
lifeless | pointer to it? | 19:59 |
fungi | http://git.openstack.org/cgit/openstack-infra/project-config/tree/jenkins/scripts/merge_tags.sh | 19:59 |
fungi | the previous iteration assumed that if we tagged on proposed.* that should merge, but if we tagged on stable/.* it should not | 19:59 |
fungi | now we've gotten rid og proposed/.* branches altogether and need to adjust the logic | 20:00 |
lifeless | ack | 20:00 |
cinerama | pleia2: do you remember if transifex has the concept of "approval" for translations? i just want to make sure i'll be using the right stats for our hacky pull script | 20:00 |
lifeless | .0 seems like no less right than 'same major-minor' | 20:00 |
lifeless | both encode assumptions | 20:00 |
fungi | so the desire is to have the first release tag of a stable branch merge to master, but not any subsequent tags on a stable branch | 20:00 |
*** tiswanso has quit IRC | 20:01 | |
pleia2 | cinerama: I believe so, and only approved ones get pulled in for our current changes (right, clarkb?) | 20:01 |
*** tiswanso has joined #openstack-infra | 20:01 | |
cinerama | pleia2: so basically for each translation I have stat total="270" untranslated="52" needReview="0" translated="218" approved="0" rejected="0" translatedOnly="218" fuzzy="0" unit="MESSAGE" locale="it" lastTranslated="24/04/15 16:12 by stephane" | 20:01 |
pleia2 | gosh | 20:01 |
mordred | fungi: I thnk if ttx tags a release with other than .0 then we fire him | 20:01 |
*** e0ne has quit IRC | 20:01 | |
fungi | mordred: aye! | 20:01 |
fungi | out of a canon | 20:02 |
clarkb | ya transifex has proposed and set translations | 20:02 |
cinerama | pleia2: which would be awesome if i knew italian | 20:02 |
pleia2 | hehe | 20:02 |
fungi | er, cannon | 20:02 |
clarkb | fungi is that what happened to star wars EU? | 20:02 |
fungi | hah | 20:03 |
fungi | definitely out-of-canon | 20:03 |
cinerama | pleia2: oh yeah that's one other thing - wondering how we're meant to import this stuff. as you can see when i did the push it all goes under me | 20:03 |
*** baoli has joined #openstack-infra | 20:04 | |
pleia2 | cinerama: import into gerrit? | 20:05 |
cinerama | pleia2: doing the initial push of the translations into zanata | 20:05 |
pleia2 | or the move of stuff from transifex to zanata? | 20:05 |
pleia2 | ah, gotcha | 20:05 |
*** tsg_ has quit IRC | 20:05 | |
pleia2 | we should probably create a specific import account | 20:06 |
*** tsg_ has joined #openstack-infra | 20:06 | |
clarkb | oh that reminds me, do we need an account to do the pulls too? | 20:06 |
*** ayoung-afk is now known as ayoung | 20:06 | |
clarkb | iirc transifex needs that | 20:06 |
fungi | clarkb: lifeless: is it safe to assume releases will henceforth be tagged as x.y.0 rather than just x.y? that was the other possibility i was accommodating | 20:06 |
clarkb | if so we can jsut mae a single service account for it | 20:06 |
fungi | i guess i can still check for that | 20:06 |
*** baoli has quit IRC | 20:07 | |
clarkb | fungi: for anything using pbr it will be x.y.z | 20:07 |
clarkb | and as long as ttx is consistent it will be 0 | 20:07 |
pleia2 | cinerama: ah yeah, does the client require auth for stat & pulling down translations? | 20:07 |
lifeless | clarkb: not true | 20:07 |
fungi | i'm also going to put in a safety net to check that the branch starts with stable/ | 20:07 |
lifeless | clarkb: pbr doesn't do the tagging [yet] | 20:07 |
*** baoli has joined #openstack-infra | 20:07 | |
lifeless | clarkb: and pbr will accept '1' as a version as much as '1.0' or '1.0.0' | 20:07 |
lifeless | clarkb: it has to, historical versions. | 20:08 |
clarkb | lifeless: ah well we can have zuul enforce that bit | 20:08 |
lifeless | so, only ttx can answer the question about x.y vs x.y.0 | 20:08 |
clarkb | sorry tempest | 20:08 |
lifeless | clarkb: how would tempest enforce it? | 20:08 |
mordred | well, we moved to x.y.0 from x.y | 20:08 |
lifeless | clarkb: git tag; git push ; <- thats where it happens | 20:08 |
clarkb | lifeless: zuul would enforce it not tempest | 20:08 |
mordred | because stable releases increment the last number | 20:08 |
*** jddavila has joined #openstack-infra | 20:08 | |
mordred | and ttx did not like the idea of going from x.y to x.y.1 | 20:08 |
fungi | i'm going to have the script check for '^[0-9]\+\.[0-9]\+\(\.0\|\)$' and that the major+minor is as new or newer than master's | 20:09 |
clarkb | lifeless: because the script runs in a zuul pipeline that is triggered by the release tag which we specify | 20:09 |
mordred | so I doubt he would all of a sudden revert - he's pretty consistent | 20:09 |
lifeless | agreed | 20:09 |
*** jddavila has quit IRC | 20:09 | |
lifeless | I'm just saying, make sure he knows :) | 20:09 |
mordred | ++ | 20:09 |
clarkb | lifeless: tempest uses single digit version | 20:09 |
clarkb | lifeless: hence the sorry tempest | 20:09 |
*** doude has quit IRC | 20:10 | |
*** pblaho has quit IRC | 20:10 | |
cinerama | pleia2: auth for everything, over https | 20:10 |
*** gordc has quit IRC | 20:10 | |
pleia2 | clarkb: ^^ yeah, so we do need an account for each section, wfm to have import and proposal account to be the same | 20:10 |
*** hasharAway is now known as hashar | 20:11 | |
openstackgerrit | lifeless proposed openstack-dev/pbr: Parallelise integration tests. https://review.openstack.org/178982 | 20:13 |
*** kgiusti has left #openstack-infra | 20:14 | |
openstackgerrit | Jeremy Stanley proposed openstack-infra/project-config: Correct the tag merge script logic https://review.openstack.org/179227 | 20:14 |
*** tiswanso has quit IRC | 20:14 | |
fungi | that ^ should be safe | 20:14 |
fungi | checking the branch is probably fine to omit since any attempt to merge a tag from master back to master would not actually work, and our zuul layout should prevent us from running it for non-stable branches | 20:15 |
fungi | if it doesn't, i'll add it there in a sec | 20:16 |
clarkb | the release pipeline will run against master so it would have to be a job specific branch restriction | 20:17 |
fungi | yeah, i'm adding one | 20:17 |
*** sushilkm has joined #openstack-infra | 20:17 | |
*** HeOS has joined #openstack-infra | 20:17 | |
*** sushilkm has left #openstack-infra | 20:17 | |
fungi | pattern match on .*-merge-release-tags for the job name and ^stable/.* for the branch name | 20:17 |
*** gordc has joined #openstack-infra | 20:18 | |
*** erlon is now known as erlon_awaY | 20:19 | |
*** Sukhdev has quit IRC | 20:20 | |
*** harlowja has joined #openstack-infra | 20:20 | |
clarkb | fungi: question about script update, you are still doing the sort -V which is where I expected the previous run to have failed? | 20:20 |
*** dkranz has quit IRC | 20:21 | |
fungi | clarkb: that part is still fine. it's the check that the major+minor match that was the issue | 20:21 |
clarkb | fungi: does sort -V treat the rc version properly? | 20:22 |
fungi | we want to allow even when major+minor match as long as the tag is in the form of x.y or x.y.0 | 20:22 |
fungi | the rc doesn't appear in the sort. only the major+minor | 20:22 |
fungi | the rc appears beyond the second component so has been stripped | 20:22 |
clarkb | oh right duh | 20:23 |
clarkb | but if major minor are identical which they will be | 20:23 |
fungi | though i just realized, we can't restrict tag-triggered jobs by branch. that's nonsensical git-wise | 20:23 |
clarkb | then your check willfail | 20:23 |
fungi | oh, great point | 20:23 |
clarkb | fungi: oh right we don't used the change merged event we use ref updated | 20:23 |
*** Sukhdev has joined #openstack-infra | 20:24 | |
*** marcusvrn1 has quit IRC | 20:24 | |
fungi | i'll rework the script to do a better job with the matching major+minor and also add back the branch check from the old version (but invert it) | 20:24 |
*** pabelanger has quit IRC | 20:25 | |
*** maishsk has joined #openstack-infra | 20:27 | |
*** asahlin is now known as asahlin_afk | 20:27 | |
*** _nadya_ has quit IRC | 20:27 | |
*** sushilkm has joined #openstack-infra | 20:27 | |
*** sushilkm has left #openstack-infra | 20:27 | |
*** ajo has quit IRC | 20:28 | |
*** nfedotov has quit IRC | 20:29 | |
openstackgerrit | Jeremy Stanley proposed openstack-infra/project-config: Correct the tag merge script logic https://review.openstack.org/179227 | 20:29 |
fungi | okay, ho's that? ^ | 20:29 |
fungi | er, how's | 20:29 |
fungi | switching the order on the comparison and inverting the match result gets us the behavior we want | 20:30 |
*** emagana has quit IRC | 20:30 | |
clarkb | fungi: whats with the escaped ! ? | 20:31 |
fungi | also the test for not a stable branch has been restored from the original script but switch to a check for the not a stable branch | 20:31 |
fungi | clarkb: to keep the shell from interpreting the ! | 20:31 |
fungi | since ! is also a shell operator | 20:31 |
*** maishsk has quit IRC | 20:32 | |
clarkb | but its running in the shell? | 20:32 |
clarkb | you are negating the return code just like on line 24 | 20:32 |
lifeless | jenkins01 is unhappy | 20:32 |
*** marun has quit IRC | 20:32 | |
lifeless | The proxy server received an invalid | 20:32 |
lifeless | response from an upstream server. | 20:32 |
lifeless | The proxy server could not handle the request POST /job/check-pbr-installation-dsvm/17/logText/progressiveHtml. | 20:32 |
lifeless | https://jenkins01.openstack.org/job/check-pbr-installation-dsvm/17/console | 20:32 |
fungi | clarkb: the != is an inequality comparison for the /usr/bin/[ executable | 20:32 |
*** alexsyip has quit IRC | 20:33 | |
clarkb | fungi: sorry on line 18 | 20:33 |
fungi | clarkb: the others are using the ! shell operator to negate the return value, not as a comparison operator | 20:33 |
clarkb | fungi: so up a bit | 20:33 |
fungi | oh, good question | 20:33 |
*** hdd_ has joined #openstack-infra | 20:33 | |
fungi | typo ;) | 20:33 |
openstackgerrit | Jeremy Stanley proposed openstack-infra/project-config: Correct the tag merge script logic https://review.openstack.org/179227 | 20:34 |
fungi | and now gone | 20:34 |
fungi | lifeless: i'll take a look, probably thread leak yet once more | 20:34 |
clarkb | ya I am pulling up melody now | 20:34 |
clarkb | but its slow so probably that leak | 20:34 |
*** ajo has joined #openstack-infra | 20:34 | |
fungi | seems it's not been in that state for long. not much cruft yet | 20:35 |
clarkb | 4112 threads, I can shutdown mode it | 20:36 |
fungi | it let me auth. mind if i put it in prepare for shutdown? | 20:36 |
fungi | oh, go for it | 20:36 |
clarkb | also maybe we should restart 04 and put it back into service | 20:36 |
clarkb | I can do that | 20:36 |
fungi | thanks | 20:36 |
*** _dguerri is now known as dguerri | 20:36 | |
*** alexsyip has joined #openstack-infra | 20:37 | |
clarkb | fungi: latest script version lgtm | 20:37 |
fungi | oh, i should lighten up the error message on line 25 | 20:38 |
openstackgerrit | Jeremy Stanley proposed openstack-infra/project-config: Correct the tag merge script logic https://review.openstack.org/179227 | 20:39 |
sdague | clarkb: well, not celebrating quite yet, just working on a few things. Were there reviews you needed out of me? | 20:39 |
clarkb | sdague: the above change 179227 is what we are currently working on | 20:40 |
fungi | slightly corrected no-op text for that conditional now | 20:40 |
clarkb | sdague: its a script to merge the stable release tag back into master so master is aware of that history | 20:40 |
fungi | sdague: working out the new logic for merging release tags to master, since the safeties in the old version assumed they would not be present on stable branches | 20:41 |
sdague | ok, well I need to break soon because we're attempting drive in movie tonight | 20:41 |
*** jd__ has joined #openstack-infra | 20:41 | |
sdague | so I'm probably not going to be helpful on context there | 20:41 |
fungi | sdague: hope the weather's cooperative! | 20:41 |
sdague | but I'll look in the morning if it's still outstanding | 20:41 |
fungi | i think we're close. this is unfortunately take 2 but the good news is that we've been failing safe rather than failing by merging crap to projects | 20:42 |
sdague | fungi: yes, weather looks good | 20:42 |
clarkb | fungi: well its still git review'd so we have that safety net too | 20:42 |
fungi | if this checks out, i'll retrigger the same job i tested a few minutes ago and cross my fingers, then trigger the remainder | 20:42 |
openstackgerrit | Russell Bryant proposed openstack-infra/project-config: Add docs-on-rtfd to networking-ovn https://review.openstack.org/179236 | 20:43 |
fungi | clarkb: yeah, but annoying regardless | 20:43 |
*** achanda has quit IRC | 20:43 | |
fungi | oh, grrr, i cut-and-pasted the old comment in with the now reversed stable branch check | 20:44 |
clarkb | I am restarting jenkins04 now, and removing the list of slaves from it while it is down. Nodepool reports it knows of none of them | 20:44 |
openstackgerrit | Jeremy Stanley proposed openstack-infra/project-config: Correct the tag merge script logic https://review.openstack.org/179227 | 20:44 |
fungi | now the comment is correct, instead of the opposite of correct | 20:44 |
openstackgerrit | lifeless proposed openstack-dev/pbr: Parallelise integration tests. https://review.openstack.org/178982 | 20:44 |
*** harlowja has quit IRC | 20:45 | |
clarkb | pleia2: actually if you are around, want me to walk through the 04 restart? | 20:45 |
pleia2 | clarkb: sure | 20:45 |
*** harlowja has joined #openstack-infra | 20:45 | |
clarkb | pleia2: cool, so first step is putting jenkins04 in shutdown mode then waiting for all jobs to complete, this was already done a while back when it fell over due to the thread leak | 20:45 |
clarkb | pleia2: but you can confirm that via the web ui, no jobs running and has the red banner across the top | 20:46 |
lifeless | sdague: dhellmann: https://review.openstack.org/#/c/179212/ if you could - blocks a pbr test improvement | 20:46 |
clarkb | pleia2: next step is to check if nodepool knows about any of the slaves jenkins04 claims to know about, `sudo -H -u nodepool nodepool list | grep jenkins04` on nodepool.o.o | 20:46 |
sdague | lifeless: is there a minimum? | 20:46 |
lifeless | sdague: I could stick a pin in the current release | 20:47 |
pleia2 | clarkb: yep, looks good | 20:47 |
lifeless | sdague: but I'm only using one function, been around for ages | 20:47 |
clarkb | pleia2: there are only two nodes in nodepool for jenkins04 and both have been in delete state for forever so they will likely need to be dealt with separately. But this means we can clear out the slave list in jenkins config.xml after we shut it down | 20:47 |
* pleia2 logs into nodepool.o.o | 20:47 | |
sdague | lifeless: ok, well if trusty's shipped version works, then meh | 20:47 |
clarkb | pleia2: the reason we do this is offline nodes in jenkins will be brought online when jenkins boots, it does not preserve that state for some reason | 20:47 |
pleia2 | yep, both in delete state, good | 20:47 |
sdague | mostly the minimum ensures that a distro installed *really old thing* doesn't break people | 20:48 |
pleia2 | clarkb: gotcha | 20:48 |
lifeless | sdague: yah | 20:48 |
clarkb | pleia2: its also just a good opportunity to clean up after jenkins | 20:48 |
sdague | I'm ok if that risk seems small | 20:48 |
lifeless | sdague: (not that the set of 'hacks on pbr and runs the integration tests' is a large set of people) | 20:48 |
openstackgerrit | sebastian marcet proposed openstack-infra/openstackid-resources: OpenstackId resource server https://review.openstack.org/178853 | 20:48 |
lifeless | sdague: I think its vanishing, because ^ | 20:48 |
clarkb | pleia2: so now we are ready to stop the jenkins service. You will want to do a service stop, then check ps because with the thread leak jenkins has been stubborn and hangs around, if that is the case you can kill -9 it | 20:48 |
sdague | lifeless: sure | 20:49 |
clarkb | pleia2: let me know when jenkins on jenkins04.openstack.org is truly dead | 20:49 |
* pleia2 waits for it to stop | 20:49 | |
sdague | lifeless: is the patch up in pbr that would need this? | 20:49 |
*** dprince has quit IRC | 20:49 | |
pleia2 | clarkb: all good, no stubbornness | 20:50 |
clarkb | pleia2: cool, next step is to remove the slaves from /var/lib/jenkins/config.xml | 20:50 |
*** hdd_ has quit IRC | 20:50 | |
pleia2 | oops, actually, it is still there, sec | 20:50 |
clarkb | pleia2: as root or jenkins edit that file and remove everything between the <slaves> </slaves> tags | 20:50 |
lifeless | sdague: yes | 20:50 |
lifeless | sdague: https://review.openstack.org/#/c/178982/ | 20:50 |
*** cody-somerville has quit IRC | 20:51 | |
lifeless | sdague: drops test time from 1hr down to 20m for the check-pbr-installation-dsvm* jobs | 20:51 |
lifeless | sdague: its ugly as sin and likely going to get beaten up a chunk during review, but it does need virtualenv any which way :) | 20:51 |
pleia2 | clarkb: that's a whole bunch of slaves | 20:51 |
clarkb | pleia2: yup, but nodepool doesn't know about them and they are all offline | 20:52 |
pleia2 | I see, thanks | 20:52 |
clarkb | pleia2: I think what happens is when the thread leak happens nodepools calls to delete the nodes essentially get lost | 20:52 |
sdague | lifeless: also license import from elsewhere? | 20:52 |
clarkb | pleia2: so unfortunately that means we have to do some cleanup | 20:52 |
fungi | also we have a way to check for instances nodepool once booted in that tenant but neither it nor jenkins are aware of any longer | 20:52 |
sdague | probably should comment directly where the code was imported from for niceness there | 20:52 |
lifeless | sdague: license import ? | 20:52 |
sdague | Copyright (C) 2013 Association of Universities for Research in Astronomy | 20:53 |
sdague | # (AURA) | 20:53 |
pleia2 | clarkb: ok, I now have | 20:53 |
pleia2 | <slaves> | 20:53 |
pleia2 | </slaves> | 20:53 |
sdague | bsd license | 20:53 |
lifeless | sdague: oh ffs | 20:53 |
sdague | https://review.openstack.org/#/c/178982/7/pbr/tests/test_integration.py,cm | 20:53 |
lifeless | sdague: its new code,will sort that stuff out on next push | 20:53 |
clarkb | pleia2: perfect, now we can start the service back up again, then check the webui for jobs running | 20:53 |
sdague | lifeless: ok :) | 20:53 |
pleia2 | clarkb: great, so just a service start? | 20:53 |
*** bookwar has joined #openstack-infra | 20:53 | |
clarkb | pleia2: yup nothing specail for that | 20:53 |
lifeless | I blagged a asl2 header I thought. Fail. | 20:54 |
pleia2 | cool, doing that now | 20:54 |
clarkb | pleia2: you can also tail /var/log/jenkins/jenkins.log | 20:54 |
pleia2 | looking good | 20:54 |
lifeless | this is what the failures look like http://logs.openstack.org/82/178982/6/check/check-pbr-installation-dsvm/9f563ff/console.html#_2015-04-30_20_32_49_976 | 20:55 |
clarkb | yup its up for me, now we just wait for nodepool's allocator to give it nodes | 20:55 |
*** sdake_ has joined #openstack-infra | 20:55 | |
pleia2 | already showing some on nodepool | 20:56 |
pleia2 | and there's a bare-trusty in the web ui | 20:56 |
clarkb | and its running a job, I think thats that | 20:56 |
clarkb | pleia2: thanks | 20:56 |
pleia2 | clarkb: thanks! | 20:56 |
*** bswartz has quit IRC | 20:57 | |
clarkb | pleia2: also if you go to https://jenkins01.openstack.org/monitoring you can see the thread counts | 20:58 |
clarkb | pleia2: anything >1k is high and likely a sign of the thread leak, to double confirm you can grep for forremoting in the thread list and if there are thousands then that is the leak | 20:58 |
fungi | clarkb: pleia2: mordred: okay, the classic lint machine in zuul has greenlit 179227 if any of you are up for approving it | 20:58 |
*** sdake has quit IRC | 20:59 | |
*** marun has joined #openstack-infra | 20:59 | |
*** sdake has joined #openstack-infra | 20:59 | |
pleia2 | fungi: ah good, was just looking at this before jenkins04 restart | 20:59 |
fungi | if it merges soon i can retest it before i disappear for dinner | 20:59 |
*** annegentle has joined #openstack-infra | 21:00 | |
fungi | then someone with prior experience retriggering jobs can fire off the remainder | 21:00 |
clarkb | pleia2: probably the most important things in that process are making sure no jobs are running after putting it in shutdown mode first, then always doing stop, check ps, start | 21:00 |
fungi | assuming my first retry works as intended this time | 21:00 |
clarkb | pleia2: the slaves can be cleaned up out of band, this is just an easy opportunity to do it | 21:00 |
pleia2 | clarkb: I don't have admin | 21:00 |
clarkb | pleia2: oh hrm | 21:00 |
fungi | pleia2: we should fix that | 21:00 |
pleia2 | lyz is missing the Overall/Administer permission | 21:00 |
fungi | quick, to the launchpad! | 21:00 |
clarkb | ya we need to add you to the group then it should work I think | 21:01 |
pleia2 | :) | 21:01 |
clarkb | but reviewing the change first | 21:01 |
*** dannywilson has joined #openstack-infra | 21:01 | |
*** unicell has quit IRC | 21:01 | |
*** sdake has quit IRC | 21:01 | |
*** unicell has joined #openstack-infra | 21:01 | |
*** sdake has joined #openstack-infra | 21:01 | |
clarkb | fungi: its approved | 21:02 |
fungi | pleia2: what's your lp username? | 21:02 |
fungi | lyz | 21:02 |
*** sdake_ has quit IRC | 21:03 | |
pleia2 | fungi: that's it | 21:03 |
*** patrickeast has quit IRC | 21:03 | |
fungi | pleia2: log out of the jenkins webui and then back in but make sure to check the openid checkbox lp gives you for supplying group membership info | 21:03 |
*** dustins has quit IRC | 21:03 | |
*** patrickeast has joined #openstack-infra | 21:03 | |
pleia2 | k | 21:03 |
clarkb | and iirc you may have to go into your user profile then tick the box to make you a real admin | 21:04 |
cinerama | pleia2: can you look at the translations for a sec? i'm supposed to be getting some thumbs up/thumbs down buttons for approving but i can't see them - maybe a second pair of eyes would reveal what is wrong | 21:04 |
clarkb | or in system config? jenkins acls are weird | 21:04 |
fungi | quite possibly so | 21:04 |
*** dannywil_ has quit IRC | 21:04 | |
clarkb | but I remember being able to adminify myself which I thought was weird | 21:04 |
*** melwitt has quit IRC | 21:04 | |
*** ldnunes has quit IRC | 21:04 | |
*** melwitt has joined #openstack-infra | 21:04 | |
pleia2 | clarkb: great, can see /monitoring now | 21:05 |
openstackgerrit | sebastian marcet proposed openstack-infra/openstackid: Migration of resource server https://review.openstack.org/178854 | 21:05 |
clarkb | pleia2: oh good maybe you don't need the extra acl bump then | 21:05 |
*** rfolco has quit IRC | 21:05 | |
pleia2 | cinerama: logging in now, which project are you looking at? | 21:05 |
*** melwitt has quit IRC | 21:05 | |
*** melwitt has joined #openstack-infra | 21:05 | |
*** BobH1 has quit IRC | 21:05 | |
pleia2 | clarkb: yeah, all just worked with magic with re-login | 21:05 |
clarkb | oh I am not in the acl list, I bet that was something we had to do back when | 21:05 |
clarkb | pleia2: and you can see that 01 has ~4k threads | 21:06 |
cinerama | pleia2: any of 'em will do but you can look at stephane-test which has the most translations up | 21:06 |
fungi | i think the openstack-ci-admins administer permission at https://jenkins07.openstack.org/configureSecurity/ (take your pick of server) is all we need. nothing per-user | 21:06 |
fungi | should "just work" these days | 21:06 |
pleia2 | cinerama: oh yeah, this UI is sneaky | 21:07 |
*** harlowja has quit IRC | 21:07 | |
openstackgerrit | Doug Hellmann proposed openstack-infra/release-tools: Move highest_semver.py from oslo-incubator https://review.openstack.org/179243 | 21:07 |
openstackgerrit | Doug Hellmann proposed openstack-infra/release-tools: Add list_repos_by_project.py https://review.openstack.org/179244 | 21:07 |
openstackgerrit | Doug Hellmann proposed openstack-infra/release-tools: Add list_unreleased_changes.sh https://review.openstack.org/179245 | 21:07 |
pleia2 | clarkb: ah yes, Number = 4,170 | 21:08 |
clarkb | pleia2: so thats basically the thing to look for if jenkins gets weird, if you see that putting it in shutdown mode, waiting for jobs to complete then restarting will clear it up | 21:09 |
clarkb | pleia2: and we can do about 2 or 3 jenkins masters at a time if we need to | 21:09 |
pleia2 | clarkb: that == too many threads? | 21:10 |
clarkb | pleia2: yup | 21:10 |
pleia2 | got it | 21:10 |
clarkb | pleia2: I think two things happen to make jenkins unhappy in that case, the first is not enough cpu time to go around, the other is the stack memory space is actually quite small and each thread eats into it, eventually the garbage collector is spending a ton of time keeping up so we can make more threads | 21:10 |
openstackgerrit | Merged openstack-infra/project-config: Correct the tag merge script logic https://review.openstack.org/179227 | 21:11 |
pleia2 | clarkb: ah, fun | 21:11 |
*** ildikov has quit IRC | 21:12 | |
fungi | okay, giving this another shot | 21:13 |
pleia2 | cinerama: hmm, this has changed since I last looked at it | 21:14 |
cinerama | pleia2: might be the newer version | 21:15 |
fungi | clarkb: pleia2: https://review.openstack.org/179248 | 21:15 |
openstackgerrit | Doug Hellmann proposed openstack-infra/release-tools: Add list_unreleased_changes.sh https://review.openstack.org/179245 | 21:16 |
openstackgerrit | Doug Hellmann proposed openstack-infra/release-tools: Add list_repos_by_project.py https://review.openstack.org/179244 | 21:16 |
openstackgerrit | lifeless proposed openstack-dev/pbr: Parallelise integration tests. https://review.openstack.org/178982 | 21:17 |
cinerama | pleia2: i turned on some additional roles for you, maybe see if that changes anything | 21:17 |
pleia2 | cinerama: thanks, relogging in | 21:17 |
clarkb | fungi: now to check it outside of gerrit because gerrit isbad at showing those diffs | 21:17 |
clarkb | cinerama: I can't edit any translations on your test project's nova strings fwiw | 21:17 |
cinerama | argh | 21:18 |
fungi | pleia2: for reference, i first looked at the 2015.1.0 tag of an up-to-date designate clone and made sure that it only appeared in origin/stable/kilo not origin/master | 21:19 |
clarkb | fungi: 2015.1.0 is 5f2b243c087721204ada3b569bbfb0e5558fbf56, that change has parents Merge: 325a3b6 5f2b243 | 21:19 |
pleia2 | cinerama: yeah, so the buttons have moved (used to be listed vertically), but I *think* there really should be the thumbs up/down things there, and I don't have them either | 21:20 |
clarkb | fungi: and the existing master commit is 325a3b651b77d5067498afdca360c21a9cf697eb so that bit at least lgtm | 21:20 |
pleia2 | fungi: thanks, having a look | 21:20 |
fungi | pleia2: can be checked with git branch -r --contains 2015.1.0 | 21:20 |
clarkb | fungi: also git diff HEAD^ is empty | 21:20 |
cinerama | clarkb: try it now? | 21:20 |
clarkb | cinerama: will I need to log in and out? | 21:20 |
fungi | clarkb: that's exactly what we wanted--thanks! | 21:20 |
cinerama | clarkb: wouldn't hurt i think | 21:20 |
clarkb | fungi: git describe --always is 2015.1.0-55-g51a4d4d | 21:20 |
clarkb | fungi: so overall this is looking good (note that version will change once pbr releases) | 21:21 |
fungi | looks good | 21:21 |
fungi | yeah | 21:21 |
clarkb | er wait no | 21:21 |
clarkb | git describe won't change but the pbr generated ver won't be 2015.1.0 | 21:21 |
clarkb | anyways it looks good | 21:21 |
fungi | clarkb: yep | 21:21 |
fungi | pleia2: then go to https://jenkins.openstack.org/job/designate-merge-release-tags/ and you should see the most recent run for the job | 21:22 |
fungi | pleia2: you can review its parameters at the parameters link on the left | 21:22 |
*** mwagner_lap has quit IRC | 21:22 | |
fungi | pleia2: all the options that the trigger script wants are listed there (though there are a few extra listed there too you don't need to care about) | 21:22 |
pleia2 | fungi: ah I see | 21:23 |
* pleia2 nods | 21:23 | |
*** hashar has quit IRC | 21:23 | |
fungi | pleia2: anyway, i basically just repeated the parameters except for the LOG_PATH where i added a ,1 or ,2 after the 227448b | 21:23 |
clarkb | cinerama: there is a completed drop down with translated and approved states and it shows no approved translations, haven't figured out how to set them to be approved | 21:24 |
pleia2 | fungi: I see, so that's how there were 3 builds | 21:24 |
*** Longgeek has quit IRC | 21:24 | |
pleia2 | clarkb: I dug up the docs http://zanata.org/help/translation-review/ | 21:24 |
clarkb | cinerama: also the alpha editor seems to work but not the normal one? | 21:24 |
fungi | pleia2: yep, that conveniently puts all their logs at http://logs.openstack.org/a8/a8f8c575120817471257a2bf4c6a9a982785e47c/release/designate-merge-release-tags/ | 21:25 |
fungi | pleia2: so that you can compare them easily on the logserver | 21:25 |
pleia2 | fungi: cool | 21:25 |
cinerama | pleia2: yes that's where i'm getting the thumbs up/thumbs down thing | 21:25 |
*** ildikov has joined #openstack-infra | 21:26 | |
fungi | pleia2: so designate and swift are already taken care of (i did the first just now and notmyname did the latter manually), but we still need to do to check the other projects from ttx's kilo release announcement | 21:26 |
cinerama | btw, i move we refer to those like/not like buttons as "siskeleberts" as an homage to the masters of thumb updowning | 21:26 |
fungi | pleia2: if that's something you want to try to tackle, awesome. if not, i can give it a shot after dinner | 21:26 |
fungi | gotta jet now though | 21:26 |
fungi | bbl | 21:26 |
*** dkranz has joined #openstack-infra | 21:27 | |
pleia2 | fungi: alright, I'll poke around | 21:27 |
*** Longgeek has joined #openstack-infra | 21:28 | |
cinerama | so the siskeleberts are missing and their equivalent in the alpha doesn't seem to be there | 21:29 |
*** nelsnelson has quit IRC | 21:29 | |
*** unicell has quit IRC | 21:30 | |
cinerama | pleia2: oh, i don't know if you've seen this but it's neat: https://translate-dev.openstack.org/admin/monitoring | 21:31 |
*** sdake_ has joined #openstack-infra | 21:31 | |
pleia2 | cinerama: ooh, I had not | 21:31 |
clarkb | is that another melody like thing? | 21:31 |
cinerama | pleia2: okay that is just plain weird. so i tried doing the key shortcut in the alpha to approve the translation but it didn't do anything | 21:31 |
pleia2 | cinerama: it looks just like the page I was just on for jenkins :D | 21:31 |
cinerama | clarkb: it is actually | 21:31 |
pleia2 | cinerama: yes | 21:31 |
pleia2 | err | 21:32 |
pleia2 | clarkb: yes | 21:32 |
cinerama | very yes | 21:32 |
*** ajo has quit IRC | 21:32 | |
*** dboik has joined #openstack-infra | 21:33 | |
*** esker has joined #openstack-infra | 21:33 | |
*** cody-somerville has joined #openstack-infra | 21:33 | |
*** unicell has joined #openstack-infra | 21:33 | |
*** sdake has quit IRC | 21:34 | |
*** signed8b_ has quit IRC | 21:36 | |
openstackgerrit | lifeless proposed openstack-dev/pbr: Parallelise integration tests. https://review.openstack.org/178982 | 21:36 |
*** jtriley has quit IRC | 21:37 | |
clarkb | ok I need to step out for a bit now that things are working | 21:37 |
*** dboik_ has quit IRC | 21:37 | |
*** mmedvede has quit IRC | 21:37 | |
clarkb | pleia2: feel free to ask questions about the release jobs if you dig into that, but I may be a bit slow answering | 21:37 |
*** dboik has quit IRC | 21:37 | |
openstackgerrit | Brant Knudson proposed openstack/requirements: Remove discover https://review.openstack.org/178710 | 21:38 |
lifeless | [nagging sorry] hows the release wrapup going? | 21:38 |
pleia2 | clarkb: thanks | 21:38 |
mtreinish | lifeless: on 170268 yeah if you guys what to deprecate the pbr testr stuff I think that would be fine | 21:38 |
clarkb | lifeless: we just need to run the merge jobs now, they are done for swift and designate but have a handful of others to do | 21:38 |
pleia2 | cinerama: maybe even as admins we need to be added for review permission on the document? | 21:38 |
*** rlandy has quit IRC | 21:38 | |
lifeless | mtreinish: I'm wondering if rather than os-testr it should be testr-python or something | 21:38 |
clarkb | but both fungi and I need to step out for a bit now, not sure if pleia2 was going to jump on that in the mean time | 21:38 |
*** alexsyip has quit IRC | 21:38 | |
lifeless | mtreinish: as a generic here-is-python-glue-for-testr thing | 21:38 |
openstackgerrit | Michael Krotscheck proposed openstack-infra/storyboard-webclient: Result set pager now supports next and previous events. https://review.openstack.org/179256 | 21:39 |
clarkb | lifeless: mtreinish is there a reason it doesn't just go in testrepository? | 21:39 |
pleia2 | clarkb: I'll try to figure out what I'm doing here, but I might just stage until post-fungi dinner | 21:39 |
clarkb | pleia2: kk sounds good | 21:39 |
*** emagana has joined #openstack-infra | 21:39 | |
pleia2 | mmm mushroom dinner :d | 21:39 |
mtreinish | lifeless: sure, we can rename it if you'd like. I'm not attached to the name, it was just the first thing I thought of | 21:39 |
*** stevemar has quit IRC | 21:39 | |
lifeless | mtreinish: its just a thought | 21:39 |
lifeless | clarkb: thats a good question. I don't really know. I respond to PR's :) | 21:40 |
lifeless | clarkb: and I've offered commit access for folk that get involved and stay involved for a while | 21:40 |
*** sdake_ has quit IRC | 21:40 | |
mtreinish | clarkb: well some of it will probably be added back to testrepository, like the ui bits. But I didn't think things like subunit-trace or subunit2html belonged there | 21:40 |
lifeless | those two could go in subunit | 21:41 |
mtreinish | it was more just to replace the terrible-ness with pretty_tox.sh scripts copied everywhere | 21:41 |
lifeless | assuming compatible (c) terms can be found | 21:41 |
cinerama | pleia2: i've added you as a "maintainer", see if that does anything | 21:41 |
lifeless | clarkb: [release - thanks]. pleia2 - are you going to push that forward? | 21:41 |
pleia2 | cinerama: ah! go here: https://translate-dev.openstack.org/iteration/view/stephane-test/1/settings/translation | 21:42 |
pleia2 | cinerama: check "Require translation review" | 21:42 |
*** sdake has joined #openstack-infra | 21:42 | |
*** annegentle has quit IRC | 21:42 | |
lifeless | pleia2: (I want to release pbr, but not while there are CI things pending running that it might affect, that are part of the release. | 21:42 |
pleia2 | cinerama: that might be it? | 21:42 |
tchaypo | that awkward moment when you realise that trusty isn’t just one release behind latest, but two, because 15.04 got relased when you weren’t watching | 21:42 |
*** annegentle has joined #openstack-infra | 21:42 | |
cinerama | pleia2: good catch, let's see if that works. so i'm *assuming* we want to approve the translations | 21:42 |
pleia2 | sdake: worked \o/ | 21:43 |
pleia2 | err | 21:43 |
pleia2 | cinerama: worked | 21:43 |
pleia2 | wow, that's a very wrong tab-miss | 21:43 |
sdake | huh? | 21:43 |
sdake | oh | 21:43 |
pleia2 | sdake: sorry :) | 21:43 |
sdake | thanks :) | 21:43 |
cinerama | pleia2: wow great. so i'm going to "approve" these translations so i can continue developing the pull | 21:44 |
pleia2 | cinerama: thumbs up | 21:44 |
pleia2 | :D | 21:44 |
cinerama | pleia2: but i might take my afternoon break in a bit | 21:44 |
pleia2 | cinerama: yeah, no problem | 21:44 |
pleia2 | lifeless: sorry, this is my first time with these releases so I'm learning but would rather not pull the trigger on my own | 21:45 |
lifeless | ok | 21:45 |
lifeless | pleia2: can you tell me - do these jobs run tests | 21:45 |
cinerama | pleia2: oh another minor bug for them might be that "approved" doesn't appear in the drop-down for each translation in the alpha ui | 21:45 |
lifeless | pleia2: or is it just infra plumbing bits | 21:46 |
lifeless | pleia2: e.g. is there any way that a pbr release would break them? | 21:46 |
lifeless | clarkb: / fungi: ^ | 21:46 |
*** mriedem is now known as mriedem_away | 21:47 | |
*** annegentle has quit IRC | 21:48 | |
pleia2 | lifeless: I don't think so, but I'm not positive | 21:48 |
pleia2 | my husband just called to say the car broke down \o/ | 21:49 |
lifeless | trololololololol | 21:49 |
lifeless | ouch | 21:49 |
pleia2 | (no rescue attempt, we only have one car) | 21:49 |
lifeless | pleia2: can you point me at the jjb definitions for these jobs? | 21:50 |
*** alexsyip has joined #openstack-infra | 21:50 | |
*** dannywilson has quit IRC | 21:51 | |
*** ashleighfarnham has quit IRC | 21:51 | |
pleia2 | lifeless: line 20 https://git.openstack.org/cgit/openstack-infra/project-config/tree/jenkins/jobs/openstack-publish-jobs.yaml | 21:52 |
*** otter768 has joined #openstack-infra | 21:52 | |
*** achanda has joined #openstack-infra | 21:52 | |
*** amitgandhinz has quit IRC | 21:53 | |
clarkb | lifeless: pleia2 I think its more likely that these jobs would modify master $project such that pbr broke under them | 21:53 |
clarkb | they essentially tell master that the last version was 2015.1.0 | 21:53 |
clarkb | by applying the tag to master's history artificially | 21:53 |
*** jgrimm is now known as zz_jgrimm | 21:53 | |
lifeless | clarkb: that should be fine | 21:53 |
lifeless | clarkb: unless master has tagged a newer version prior to that tag | 21:54 |
lifeless | (and if it has we shouldn't be doing this merge!) | 21:54 |
lifeless | clarkb: so, I think it will be ok if we tag pbr. What do you think? | 21:54 |
lifeless | sooner-its-up-sooner-any-fallout-is-found | 21:54 |
clarkb | lifeless: ya its likely fine | 21:55 |
clarkb | lifeless: I don't think it would break, but if it does break I think it would be under the circumstances above | 21:56 |
*** dboik has joined #openstack-infra | 21:56 | |
lifeless | anything changed I should know about with release tagging? tag -s; push the tag? | 21:57 |
*** otter768 has quit IRC | 21:57 | |
*** dboik_ has joined #openstack-infra | 21:57 | |
clarkb | thats still the process | 21:57 |
*** Rockyg has joined #openstack-infra | 21:58 | |
*** peristeri has quit IRC | 21:58 | |
openstackgerrit | Richard Jones proposed openstack/requirements: Update pyScss and django-pyscss https://review.openstack.org/179260 | 21:58 |
*** emagana has quit IRC | 21:59 | |
*** dannywilson has joined #openstack-infra | 21:59 | |
*** abregman has quit IRC | 21:59 | |
lifeless | huh | 21:59 |
lifeless | apparently I don't have release bit for pbr anymore | 22:00 |
*** oomichi has joined #openstack-infra | 22:00 | |
lifeless | could someone push a tag of 0.11.0 -> a67e2c346c5193cf36e381694d8fddb0d826df6b ? | 22:00 |
*** dboik has quit IRC | 22:00 | |
mtreinish | lifeless: wait is subunit gpl? I don't think I ever bothered to look | 22:00 |
lifeless | mtreinish: asf + bsd dual | 22:00 |
mtreinish | ah ok | 22:01 |
*** bknudson has quit IRC | 22:01 | |
lifeless | same as testtools etc | 22:01 |
*** bhunter71 has quit IRC | 22:02 | |
*** Swami has quit IRC | 22:02 | |
lifeless | clarkb: ^ (I need other hands to do the release) | 22:03 |
lifeless | or someone to add me to the appropriate group :) | 22:03 |
clarkb | is dims not doing it? | 22:03 |
lifeless | I thought I could so let him off the hook | 22:04 |
clarkb | lifeless: fwiw I am not in that group either | 22:04 |
lifeless | dims_: still around? | 22:04 |
clarkb | so don't necessarily feel comfortable tagging myself or adding members | 22:04 |
lifeless | yeah, totally fair | 22:04 |
*** dims has joined #openstack-infra | 22:04 | |
lifeless | dims: hi | 22:04 |
clarkb | mordred: and dhellmann are in that group as well as oslo-release and oslo-core | 22:04 |
*** dims_ has quit IRC | 22:05 | |
lifeless | dims: good timing. turns out I'm not in the pbr release group | 22:05 |
lifeless | dims: just pbr-core, so - could you do a release of pbr master as 0.11.0 ? | 22:05 |
*** shivharis has quit IRC | 22:05 | |
*** shivharis has joined #openstack-infra | 22:05 | |
*** annegentle has joined #openstack-infra | 22:08 | |
openstackgerrit | lifeless proposed openstack-dev/pbr: Parallelise integration tests. https://review.openstack.org/178982 | 22:08 |
*** mmedvede has joined #openstack-infra | 22:08 | |
lifeless | hmmm | 22:08 |
lifeless | who else to nargh | 22:08 |
lifeless | huh | 22:09 |
lifeless | all oslo core can do pbr releases | 22:09 |
lifeless | that seems surprising | 22:09 |
lifeless | bnemec: around? | 22:09 |
lifeless | too early for mikal | 22:11 |
lifeless | russellb: ahha | 22:11 |
*** melwitt_ has joined #openstack-infra | 22:11 | |
* lifeless looks desperate :) | 22:11 | |
*** melwitt_ has quit IRC | 22:11 | |
*** melwitt has quit IRC | 22:12 | |
*** abregman has joined #openstack-infra | 22:13 | |
zaro | clarkb: i'm testing https://review.openstack.org/#/c/178929 when i send a job from my client the worker does not execute the job. | 22:13 |
zaro | clarkb: debugging now, haven't figured out the problem yet. | 22:13 |
*** gyee has joined #openstack-infra | 22:13 | |
clarkb | zaro: its entirely possible that my code is bad :) but it passed the built in tests | 22:14 |
clarkb | zaro: thansk for checking | 22:14 |
clarkb | zaro: its likely I will need to come back to that with fresh eyes | 22:15 |
zaro | clarkb: well i've confirmed that it's definately bad :( | 22:15 |
*** armax has quit IRC | 22:15 | |
*** jamielennox|away is now known as jamielennox | 22:16 | |
*** dizquierdo has joined #openstack-infra | 22:16 | |
zaro | clarkb: i'll try to figure out why but may take me a little while. that code gives me a headache. | 22:17 |
clarkb | zaro: me too :) oen thing I want to try is consolidating the state machine transitions to one function | 22:17 |
clarkb | zaro: but thats a much larger refactoring so haven't begun looking at that yet | 22:17 |
clarkb | zaro: but that should make it easier to reason about the states and their transitions I think | 22:17 |
*** Sukhdev has quit IRC | 22:19 | |
zaro | clarkb: or maybe ask jeblair since he wrote it? | 22:19 |
*** Sukhdev has joined #openstack-infra | 22:20 | |
*** melwitt has joined #openstack-infra | 22:20 | |
*** doug-fish1 has left #openstack-infra | 22:20 | |
*** erlon_awaY has quit IRC | 22:21 | |
dims | lifeless: pong | 22:26 |
openstackgerrit | Clint 'SpamapS' Byrum proposed openstack-infra/infra-manual: Add clarification on questions in reviews https://review.openstack.org/177364 | 22:27 |
openstackgerrit | Davide Guerri proposed openstack-infra/shade: Add keystone services/endpoints methods https://review.openstack.org/177621 | 22:27 |
*** sigmavirus24 is now known as sigmavirus24_awa | 22:27 | |
dims | lifeless: "pbr master as 0.11.0" means i'll be running "./release_library.sh liberty 0.11.0 HEAD pbr" | 22:27 |
*** akshai has quit IRC | 22:28 | |
dims | lifeless: cross-checking, no one has pushed the button yet (https://pypi.python.org/pypi/pbr/ shows 0.10.8) | 22:29 |
lifeless | dims: you'll get an error from git if someone else were to do so | 22:29 |
*** packet has quit IRC | 22:30 | |
lifeless | dims: (e.g. gerrit is our global lock for such operations) | 22:30 |
dims | lifeless: done - http://git.openstack.org/cgit/openstack-dev/pbr/ shows 0.11.0 | 22:32 |
dims | cross my fingers | 22:32 |
lifeless | dims: \o/ | 22:32 |
lifeless | BREAK THE WORLD. | 22:32 |
lifeless | oops, did that get out? | 22:32 |
dims | haha | 22:32 |
dims | so, i have to head out for 45 mins :) be back to check the carnage | 22:33 |
*** jpeeler has quit IRC | 22:34 | |
*** sarob has quit IRC | 22:34 | |
*** julim has joined #openstack-infra | 22:35 | |
clarkb | what does release_library.sh do? | 22:35 |
*** chlong has joined #openstack-infra | 22:36 | |
*** emagana has joined #openstack-infra | 22:36 | |
mtreinish | fungi, clarkb: did you guys change something to enable unicode in gerrit? I didn't think it used to work | 22:37 |
clarkb | we did, mostly was zaro | 22:38 |
pleia2 | mtreinish: yep, when we did a rename a few weeks back | 22:38 |
pleia2 | er, some renames | 22:38 |
mtreinish | pleia2: ah ok cool | 22:39 |
pleia2 | clarkb: ok, so we need to run the merge-release-tags for all the things in the release announcement... why? | 22:39 |
mtreinish | it's good that it works now | 22:39 |
*** dizquierdo has quit IRC | 22:39 | |
*** ddieterly has quit IRC | 22:40 | |
*** Somay_ has joined #openstack-infra | 22:40 | |
* pleia2 continues putting together trigger-job commands | 22:40 | |
clarkb | pleia2 it ties the git history of new release back into master so master sees it in its history | 22:40 |
*** Somay has quit IRC | 22:40 | |
clarkb | this way versions work properly | 22:40 |
pleia2 | clarkb: ok, so these jobs ran like 12 hours ago, and they were broken so fungi made all those changes we reviewed to fix them | 22:41 |
clarkb | yup | 22:41 |
clarkb | the failed runs nooped and bailed out | 22:41 |
pleia2 | clarkb: alright, I'm collecting my trigger-job.py commands here https://etherpad.openstack.org/p/kilo-merge-release-tags <-- fungi too | 22:42 |
clarkb | cool, I am on abus will pull it up from destination best I can | 22:42 |
*** esker has quit IRC | 22:42 | |
pleia2 | snagged fungi's command for designate from his .bash_history and digging up the details to filli n the rest | 22:42 |
*** emagana has quit IRC | 22:42 | |
*** annegentle has quit IRC | 22:43 | |
clarkb | you can look at the failed jobs' parameters in jenkins for info too | 22:43 |
clarkb | should have the refs and stuff for you | 22:43 |
clarkb | all should run on jenkins.o.o so should be quick to find | 22:44 |
pleia2 | clarkb: so looking at the designate job, it didn't fail https://jenkins.openstack.org/job/designate-merge-release-tags/1/ | 22:45 |
clarkb | I think it bailed out and nooped | 22:45 |
lifeless | clarkb: tags and sends a mail | 22:45 |
clarkb | since the branch did not exist | 22:45 |
pleia2 | clarkb: yeah, but didn't mark it as such, so not easy to find | 22:45 |
jhesketh | Morning | 22:46 |
*** emagana has joined #openstack-infra | 22:46 | |
pleia2 | anyway, easy enough to dig up manually | 22:47 |
asselin__ | anyone seen this error? pbr.version.SemanticVersion(4.0.1), but target version is pbr.version.SemanticVersion(4.0.0) http://paste.openstack.org/show/213977/ icehouse & juno | 22:47 |
lifeless | asselin__: ahha | 22:47 |
lifeless | asselin__: thats te symptom of a bad setup.cfg | 22:48 |
asselin__ | local issue or upstream issue? | 22:48 |
lifeless | asselin__: both :) | 22:48 |
lifeless | asselin__: so you've tagged 4.0.1 but setup.cfg says you are aiming at 4.0.0 | 22:49 |
lifeless | asselin__: setup.cfg needs to be fixed - either aim at a newer version, or just remove the version line entirely. | 22:49 |
lifeless | i'm writing up an announcement now | 22:49 |
lifeless | because i expect more than a few things will have fallout | 22:49 |
asselin__ | lifeless, I'm using devstack-gate....need to see what I'm missing | 22:49 |
asselin__ | this is tempest setting up it's venv... | 22:50 |
asselin__ | lifeless, master is failing on stack.sh | 22:51 |
clarkb | which projevt has that error? | 22:52 |
clarkb | oh tempest | 22:52 |
asselin__ | seems to be tempest | 22:52 |
clarkb | ya so we need to update tempest | 22:52 |
asselin__ | master devstack log: http://paste.openstack.org/show/213978/ | 22:52 |
lifeless | tempest shouldn't be preversioned should it? | 22:52 |
lifeless | mtreinish: ^ | 22:52 |
*** whoops has quit IRC | 22:53 | |
mtreinish | lifeless: the tempest tags are just integers | 22:53 |
mtreinish | no decimal points | 22:53 |
mtreinish | the current one is 4 | 22:53 |
mtreinish | (which I pushed yesterday) | 22:54 |
clarkb | ya but you have a version in setup.cfg that is too low | 22:54 |
clarkb | you can probably just remove it | 22:54 |
mtreinish | clarkb: no there shouldn't be one in the setup.cfg anymore | 22:54 |
mtreinish | I removed that I thought | 22:54 |
mtreinish | let me check | 22:54 |
mtreinish | clarkb: nope I was wrong, you were right | 22:54 |
mtreinish | I removed it on tempest-lib not tempest | 22:54 |
lifeless | mtreinish: remove it :) | 22:54 |
mtreinish | I'll fix that right now | 22:55 |
asselin__ | mtreinish, thanks | 22:55 |
*** Swami has joined #openstack-infra | 22:55 | |
*** annegentle has joined #openstack-infra | 22:58 | |
mtreinish | lifeless, asselin__, clarkb: https://review.openstack.org/#/c/179278/ | 22:58 |
*** gordc has quit IRC | 22:58 | |
*** Guest4127 has quit IRC | 22:58 | |
*** bswartz has joined #openstack-infra | 22:59 | |
lifeless | hopefully we won't be wedged by multiple projects in this state ;) | 23:00 |
*** achanda has quit IRC | 23:01 | |
*** annegentle has quit IRC | 23:03 | |
*** marun has quit IRC | 23:05 | |
*** baoli has quit IRC | 23:06 | |
*** oomichi has quit IRC | 23:06 | |
*** marun has joined #openstack-infra | 23:06 | |
fungi | okay, i'm back. you people were chatty! ;) | 23:06 |
*** melwitt has quit IRC | 23:07 | |
*** camunoz has joined #openstack-infra | 23:07 | |
pleia2 | fungi: https://etherpad.openstack.org/p/kilo-merge-release-tags | 23:08 |
*** annegentle has joined #openstack-infra | 23:08 | |
clarkb | pleia2: ok I have achieved tethering | 23:08 |
openstackgerrit | lifeless proposed openstack-dev/pbr: Parallelise integration tests. https://review.openstack.org/178982 | 23:09 |
* fungi did not have mushrooms for dinner, but did have fish in tortillas with jicama and also some cactus-beers | 23:09 | |
pleia2 | nice | 23:09 |
*** Sukhdev_ has joined #openstack-infra | 23:10 | |
* dims peeks | 23:10 | |
clarkb | hrm my laptop was not as charged as I thought | 23:10 |
*** claudiub has quit IRC | 23:10 | |
*** woodster_ has quit IRC | 23:10 | |
*** Sukhdev has quit IRC | 23:11 | |
*** dimtruck is now known as zz_dimtruck | 23:11 | |
clarkb | pleia2: the project for nova isn't substituted correctly | 23:11 |
*** marun has quit IRC | 23:11 | |
pleia2 | clarkb: fixed, thanks | 23:11 |
mtreinish | fungi: ooh cactus-beers those sound interesting | 23:11 |
pleia2 | mtreinish: he was saying they were agave ipas, I'm jealous :) | 23:11 |
pleia2 | fungi: oh, and designate wasn't listed in the release announcement, are there any others to look for? | 23:13 |
pleia2 | I just got my list from release announcement | 23:13 |
*** achanda has joined #openstack-infra | 23:14 | |
*** salv-orl_ has joined #openstack-infra | 23:14 | |
clarkb | ok I am going to go command by command and check the params and check off on them in the etherpad | 23:15 |
clarkb | pleia2: ^ would that help? | 23:15 |
pleia2 | clarkb: sure, thanks | 23:15 |
fungi | lifeless: the "bad setup.cfg" problem was _one_ of the reasons we emergency-branched pbr back when, i'm starting to remember now. at the time it was going to take too long to get everyone to approve the patches to various projects since it was holiday season. hopefully now we can work through it properly | 23:15 |
*** MarkAtwood has quit IRC | 23:15 | |
*** jgriffith has quit IRC | 23:16 | |
lifeless | fungi: we had actually fixed them all up previously, I did an audit over everything | 23:16 |
fungi | i think a few of them must have regressed | 23:17 |
lifeless | fungi: we could downgrade it to a warning and let pbr just calculate a new version regardless | 23:17 |
*** jgriffith has joined #openstack-infra | 23:17 | |
lifeless | fungi: with a whinge | 23:17 |
*** salv-orlando has quit IRC | 23:17 | |
fungi | well, i'm personally fine with telling people they're broken and fix their sh*t now | 23:17 |
*** weshay has quit IRC | 23:18 | |
pleia2 | clarkb: thanks, running the nova one now, let's see how it goes | 23:18 |
mtreinish | fungi: well we might want to fast path: https://review.openstack.org/#/c/179278/1 to unblock things | 23:18 |
lifeless | fungi: its possible that someone tagging two projects at once could setup a deadlock situation | 23:18 |
fungi | mtreinish: looking to see what's possible now | 23:19 |
*** woodster_ has joined #openstack-infra | 23:22 | |
mtreinish | fungi: cool thanks | 23:22 |
fungi | mtreinish: it's at the head of the gate now | 23:23 |
fungi | hopefully we get lucky and it passes what's left of the gauntlet | 23:23 |
mtreinish | fungi: heh, I don't know my luck with that hasn't been so hot this week | 23:24 |
fungi | i know how you feel | 23:24 |
*** tnovacik has quit IRC | 23:25 | |
clarkb | pleia2: fungi I haven't been checking any resultin changes yet | 23:25 |
*** ibiris is now known as ibiris_away | 23:25 | |
*** sabeen2 has quit IRC | 23:25 | |
pleia2 | fungi: want to have a look at changes? I'm pasting them in the etherpad | 23:26 |
* fungi neither. sort of still catching up | 23:26 | |
fungi | pleia2: yeah, looking. thanks--that helps! | 23:26 |
*** rm_work is now known as rm_work|away | 23:27 | |
*** rm_work|away is now known as rm_work | 23:27 | |
*** dalgaaf has quit IRC | 23:27 | |
*** _nadya_ has joined #openstack-infra | 23:28 | |
*** rm_work is now known as rm_work|away | 23:29 | |
*** wenlock_ has quit IRC | 23:29 | |
fungi | so far spot checks are looking good. the best sanity check is that the .gitreview "modification" shown by gerrit removes defaultbranch=stable/kilo | 23:29 |
*** wenlock has quit IRC | 23:32 | |
*** _nadya_ has quit IRC | 23:33 | |
lifeless | ah actually no, I remember, we're deadlock sage | 23:33 |
lifeless | safe because tagged version is ok | 23:33 |
fungi | in _interesting_ news, gertty doesn't handle these tag merge changes very well | 23:33 |
mtreinish | lifeless: heh, I think that I'd like to meet a deadlock sage. It would probably be an interesting experience :) | 23:33 |
fungi | (where "not very well" is defined as "explodes leaving traceback debris in its place") | 23:34 |
lifeless | unless we make temp commits during jobs for requirements updates, because that could cause two projects to conflict concurrently | 23:34 |
lifeless | from a known-good-state | 23:34 |
clarkb | pleia2: ok entire list lgtm | 23:35 |
*** melwitt has joined #openstack-infra | 23:36 | |
pleia2 | clarkb: thanks! | 23:36 |
pleia2 | just finished running the last of them | 23:36 |
pleia2 | this was all the ones from the release announcement, so if there are other sneaky ones, we'll have to do them (like designate) | 23:36 |
fungi | yep, those are all looking good | 23:36 |
*** annegentle has quit IRC | 23:36 | |
pleia2 | thanks fungi | 23:36 |
clarkb | pleia2: we can probably search for merge-tags on jenkins.o.o and see which ones ran the job | 23:37 |
fungi | pleia2: might not hurt to check against http://governance.openstack.org/reference/tags/release_at-6mo-cycle-end.html#application-to-current-projects | 23:37 |
*** blahdeblah__ has quit IRC | 23:38 | |
*** blahdeblah has joined #openstack-infra | 23:38 | |
pleia2 | fungi: oh dear, that's not all of them | 23:38 |
*** emagana has quit IRC | 23:38 | |
fungi | i don't know how many of the balance did or didn't get handled as part of the release process | 23:39 |
lifeless | yay Ran 63 tests in 328.769s | 23:39 |
lifeless | pbr's integration tests. \o/ | 23:39 |
pleia2 | fungi: 12 in the release announcement, 26 listed there | 23:39 |
pleia2 | so I guess it's time to see what was tagged | 23:40 |
fungi | bash for loop for the win | 23:40 |
* pleia2 clone-all | 23:40 | |
pleia2 | actually, I'll just look on one of the servers that has them | 23:40 |
cinerama | pleia2: i have hit another snag (and i didn't take my break yet :/) | 23:40 |
*** marun has joined #openstack-infra | 23:40 | |
fungi | i have a cron job on my workstation that keeps local git clones maintained for everything we host in gerrit and remote updates them daily | 23:41 |
*** emagana has joined #openstack-infra | 23:41 | |
fungi | comes in very handy when i want to quickly propose changes to a few hundred or so of them | 23:42 |
*** Alexandra_ has joined #openstack-infra | 23:43 | |
clarkb | ok my laptop just got assaulted by greghaynes mussels | 23:43 |
clarkb | so its time for me to put it away. anything else I should do first? | 23:43 |
openstackgerrit | lifeless proposed openstack-dev/pbr: Parallelise integration tests. https://review.openstack.org/178982 | 23:44 |
ianw | fungi / clarkb : if you have a minute, could you log into a hpcloud centos7 and do "cat /proc/sys/crypto/fips_enabled" | 23:44 |
*** krtaylor has quit IRC | 23:44 | |
lifeless | dims: do you have commit access to requirements? | 23:45 |
*** salv-orl_ has quit IRC | 23:46 | |
fungi | ianw: 0 | 23:46 |
fungi | clarkb: no, go enjoy your bivalves and beer | 23:47 |
cinerama | i would like some mussels pls | 23:47 |
ianw | fungi: hmm, thanks. http://logs.openstack.org/03/178903/1/check/check-tempest-dsvm-centos7/85b16a7/logs/devstacklog.txt.gz failed to setup mod_ssl correctly ... weird | 23:47 |
fungi | ianw: must be due to some other hilarious raisins | 23:48 |
*** mrmartin has joined #openstack-infra | 23:48 | |
*** pc_m has quit IRC | 23:50 | |
*** harlowja has joined #openstack-infra | 23:51 | |
ianw | according to logstash it has happened twice within 10 minutes, then never again. i wonder if a bad mod_ssl was pushed and quickly fixed up or something | 23:51 |
*** Swami_ has joined #openstack-infra | 23:51 | |
pleia2 | fungi: http://paste.openstack.org/show/214007/ | 23:51 |
pleia2 | cinerama: want to explain and call it a day? I don't know that I'll be free in the immediate future | 23:52 |
fungi | ianw: sounds like a reasonable hypothesis | 23:52 |
pleia2 | cinerama: i can follow up tomorrow :) | 23:52 |
pleia2 | fungi: I did end up just locally cloning the branches from the list that weren't in the release announcement | 23:52 |
fungi | pleia2: that looks probably correct | 23:53 |
pleia2 | I think this means I'm only halfway done (11 more to do!) and sahara-extra has gone wrong | 23:53 |
cinerama | pleia2: the tldr is i had THOUGHT we could download individual translations using the cli but i don't think we actually can so i need to write my own pull | 23:53 |
*** otter768 has joined #openstack-infra | 23:53 | |
fungi | pleia2: some of those may not have merge jobs on jenkins.o.o but that should be fast to double-check by munging urls | 23:53 |
*** SumitNaiksatam has quit IRC | 23:54 | |
cinerama | pleia2: so at this point we're basically writing our own client i think. we can still push with the cli | 23:54 |
pleia2 | fungi: ah, good point, I'll check then | 23:54 |
cinerama | pleia2: it all comes down to the percentage completion | 23:54 |
pleia2 | cinerama: uuugghh, maybe I should just follow up with carlos and see how likely getting that bug bumped in priority is | 23:55 |
*** Swami has quit IRC | 23:55 | |
cinerama | pleia2: it's not so bad as we are using rest but this has turned out to be a bit more work than i thought it was | 23:55 |
pleia2 | cinerama: yeah :\ | 23:56 |
*** krtaylor has joined #openstack-infra | 23:56 | |
cinerama | pleia2: and we can at least get the data we need from the server to do the percentages | 23:57 |
cinerama | pleia2: it could be a lot worse | 23:57 |
pleia2 | cinerama: indeed! just complicated, and slow | 23:57 |
cinerama | pleia2: fog of software development etc etc | 23:57 |
* pleia2 nods | 23:57 | |
pleia2 | I guess the percentage thing we do is unusual, most projects want to ship with as much as they can, even if it's a little bit | 23:57 |
cinerama | well, i think it's reasonable | 23:58 |
cinerama | a) transifex do it | 23:58 |
cinerama | i had a b) but i rejected it | 23:58 |
*** otter768 has quit IRC | 23:58 | |
pleia2 | yeah | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!