| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-01-20 | |||
| 16:58:20 | bauzas | mriedem: the fact that the get method is not asking for 1.4 | |
| 16:58:28 | robcresswell | mriedem: We don't use them, so 2.1 I'd assume. | |
| 16:58:44 | mriedem | robcresswell: then i don't know why you'd be broken | |
| 16:58:46 | robcresswell | Its just because of the 7.0.0 release I believe | |
| 16:58:47 | bauzas | so when the scheduler asks for resources, the placement service says "meh, what are you asking?" | |
| 16:58:52 | robcresswell | oo, interesting | |
| 16:59:01 | mriedem | bauzas: i know | |
| 16:59:22 | bauzas | oh, sorry, I thought your "what microversion are you breaking on?" was related to me :) | |
| 16:59:41 | bauzas | anyway, back to that problem | |
| 16:59:59 | mriedem | nope | |
| 16:59:59 | sdague | robcresswell: a well formed bug is probably important here, because at this point we're just all guessing :) | |
| 17:00:15 | mriedem | i'm going to lunch and running errands, so be back in a couple of hours | |
| 17:00:20 | robcresswell | sdague: Yeah, I'll give it a go. | |
| 17:00:23 | mriedem | when you guys will have figured and fixed everything | |
| 17:01:54 | openstackgerrit | Arnaud Morin proposed openstack/nova: Set default value to HTTP_X_FORWARDED_PROTO https://review.openstack.org/423375 | |
| 17:05:17 | dims | sdague : just 3 lines to review if you get a chance please - https://review.openstack.org/#/c/421911/ | |
| 17:05:36 | dims | leakypipes : ^^ | |
| 17:05:57 | sdague | dims: +2 | |
| 17:06:16 | dims | thanks sdague ! | |
| 17:06:23 | bauzas | Whipped | |
| 17:06:33 | dims | thanks bauzas ! | |
| 17:06:36 | leakypipes | dims: done. | |
| 17:06:42 | dims | and leakypipes ! | |
| 17:06:51 | leakypipes | fastest patch approval EVAH. | |
| 17:06:58 | dims | i know!!!! | |
| 17:07:14 | bauzas | leakypipes: https://review.openstack.org/#/c/423353/ in case you wanna beat my timings :) | |
| 17:07:33 | leakypipes | bauzas: I'm already reviewing that one :) | |
| 17:07:49 | bauzas | leakypipes: http://paste.openstack.org/show/595821/ for seeing the result | |
| 17:09:32 | bauzas | leakypipes: FWIW, I'm planning to write a specific change for https://review.openstack.org/#/c/385693/27/nova/scheduler/client/report.py | |
| 17:09:39 | bauzas | because it's a bug | |
| 17:10:02 | leakypipes | bauzas: what's a bug? | |
| 17:10:08 | bauzas | and we need that in order to merge the scheduler change | |
| 17:10:12 | dims | update for all of you - sdague, leakypipes, bauzas, mriedem_afk : am down to one flaky issue in Nova and one persistent failure in neutron in that py35/tempest job | |
| 17:10:16 | bauzas | leakypipes: the fact that we're stuck with 1.0 | |
| 17:10:21 | sdague | dims: cool | |
| 17:10:30 | sdague | dims what is the flakey issue? | |
| 17:10:48 | openstack | Launchpad bug 1658148 in python-novaclient "Horizon/Nova interactions broken " [Undecided,New] | |
| 17:10:48 | robcresswell | sdague, mriedem_afk: https://bugs.launchpad.net/python-novaclient/+bug/1658148 Bug report so far, I'll continue to add more. | |
| 17:10:57 | robcresswell | Sorry for the vague title | |
| 17:11:14 | leakypipes | bauzas: you're saying you are pushing a patch that only adds the OpenStack-APi-Version header to the reporting client. is that correct? | |
| 17:11:17 | dims | sdague : tempest.api.compute.admin.test_servers.ServersAdminTestJSON.test_list_servers_by_admin_with_specified_tenant | |
| 17:11:18 | leakypipes | bauzas: if so, cool by me. | |
| 17:11:28 | bauzas | leakypipes: yup, zactly | |
| 17:11:43 | leakypipes | bauzas: I have separate three patches waiting each with that same code block :) | |
| 17:11:48 | bauzas | heh | |
| 17:11:52 | bauzas | and mine too | |
| 17:12:08 | bauzas | all my tempest jobs are red because of that | |
| 17:12:15 | dims | the neutron failure is tempest.api.compute.admin.test_auto_allocate_network.AutoAllocateNetworkTest.test_server_multi_create_auto_allocate for the record | |
| 17:12:31 | bauzas | FWIW, 7th coffee of the day | |
| 17:12:44 | bauzas | I'm nearly close to beat my personal record | |
| 17:13:02 | bauzas | still 1 | |
| 17:13:11 | bauzas | but the day is not over | |
| 17:15:55 | johnthetubaguy | melwitt: I think I got my head more around the first two quota patches now | |
| 17:16:01 | johnthetubaguy | melwitt: added some follow up comments | |
| 17:16:35 | melwitt | johnthetubaguy: I saw, thanks, much appreciated. I replied to your one about duplicates a sec ago. I was thinking of the create_limit API when I say 'duplicates' | |
| 17:16:40 | johnthetubaguy | melwitt: I think the outstanding question in my head is a nuts edge case, delete, if you delete in the new DB, but the old one wasn't deleted, it might come back to life | |
| 17:16:47 | sdague | robcresswell: added comments to the bug, nova-api logs would be useful there for that request id | |
| 17:17:06 | sdague | the 403 makes it seem like there is something going on differently on the auth side | |
| 17:17:15 | johnthetubaguy | melwitt: I was thinking we only get duplicates from the migration | |
| 17:17:20 | melwitt | johnthetubaguy: yeah, have to think about that one more. thanks for all the good points | |
| 17:17:43 | johnthetubaguy | melwitt: no worries, this stuff really mashes your brain, somehow | |
| 17:19:12 | melwitt | johnthetubaguy: yeah. exact duplicates most likely. but I was thinking of if someone already has a 'instances' limit in the main DB, if they create an 'instances' limit before migrations were run, they would have two, one in API DB and one in main DB | |
| 17:19:59 | johnthetubaguy | melwitt: oh right, you get a "newer one" at the top | |
| 17:20:17 | melwitt | yeah. again I don't see why anyone would do that but that could happen | |
| 17:20:36 | johnthetubaguy | well, depends if you automate your user onboarding | |
| 17:20:49 | johnthetubaguy | thats probably more likely that I would like | |
| 17:20:55 | johnthetubaguy | s/that/than/ | |
| 17:22:36 | johnthetubaguy | hmm, thats mostly nonsense probably, I need to look at the API | |
| 17:32:42 | openstackgerrit | Eric Fried proposed openstack/nova: Introduce base PowerVM driver (cfg drv) https://review.openstack.org/409404 | |
| 17:33:36 | robcresswell | sdague: Updated the bug report with logs from nova-api. Seems to not recognise the project id. | |
| 17:36:16 | sdague | robcresswell: hmmm | |
| 17:36:40 | sdague | robcresswell: do you have a simple test case to reproduce it? That all seems very weird. | |
| 17:40:05 | robcresswell | sdague: Horizon master would be the simplest thing to do, if you have a devstack machine about | |
| 17:40:32 | robcresswell | Had multiple other confirmations in the Horizon channel before heading over hear :) | |
| 17:40:39 | robcresswell | here* | |
| 17:42:01 | sdague | robcresswell: ok, yeh, I need to find some lunch first, but can probably look this afternoon | |
| 17:42:37 | robcresswell | sdague: Cool, no problem. If you have any suggestions, please leave them in the bug report - its Friday evening here and has been a loooong week :) | |
| 17:43:06 | sdague | robcresswell: yeh, no worries, enjoy the weekend | |
| 17:43:58 | robcresswell | sdague: Thanks, you too | |
| 17:44:05 | diana_clarke | About the horizon/python-novaclient thing: the tests all pass but it doesn't work in real life (aka devstack), correct? | |
| 17:46:43 | robcresswell | diana_clarke: Seems that way, yep | |
| 17:47:48 | robcresswell | diana_clarke: Haven't done much investigation, mainly just marched over here with my pitchfork. I'll be around again Monday if there are some suggestions, otherwise I'll just poke at Horizon until it works. | |
| 17:49:14 | robcresswell | Primarily a time constraint thing though so far, will have more time to dedicate next week. PTl isn't around much right now, so I'm probably the easiest person to contact :) | |
| 17:49:53 | openstackgerrit | Matthew Booth proposed openstack/nova: libvirt: Remove more imagecache info file code https://review.openstack.org/423392 | |
| 17:50:43 | diana_clarke | robcresswell: If I had to guess (without much digging), I suspect it has to do with the deprecations that were done here, specially if it's project id related: https://review.openstack.org/#/q/project:openstack/python-novaclient+branch:master+topic:session | |
| 17:51:05 | diana_clarke | *especially | |
| 17:51:11 | diana_clarke | (but I could be wrong) | |
| 17:52:27 | diana_clarke | perhaps this, in particular: https://review.openstack.org/#/c/350106/14/releasenotes/notes/clarify-project-id-variable-5832698315000438.yaml | |
| 17:52:34 | robcresswell | diana_clarke: Would you mind leaving suggestions on https://bugs.launchpad.net/python-novaclient/+bug/1658148 ? I'm happy to spend the time looking, but this is a busy channel so I'll likely lose track if suggestions are posted here. | |
| 17:52:34 | openstack | Launchpad bug 1658148 in OpenStack Dashboard (Horizon) "Horizon/Nova interactions broken " [Critical,New] - Assigned to Rob Cresswell (robcresswell) | |
| 17:53:23 | openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: [WIP] ignore me - trying experimental jobs https://review.openstack.org/421209 | |
| 17:53:35 | diana_clarke | robcresswell: I'm mostly just talking out loud at this point. If I find something legit I'll be sure to update that bug, I'm on PTO this afternoon though, so I probably won't look much until Monday either. | |
| 17:54:55 | robcresswell | diana_clarke: Ah cool, not a problem. Even educated guesses are probably more valuable than my blundering around novaclient code though. Have a good weekend, and thanks for the help. | |
| 18:15:33 | openstackgerrit | Matthew Booth proposed openstack/nova: libvirt: Fix races in glusterfs mount/umount https://review.openstack.org/423402 | |
| 18:16:33 | mdbooth | johnthetubaguy: For context with the nfs volume driver, with that in place GlusterFS took me about 30 minutes ^^^ | |
| 18:23:47 | openstackgerrit | Anusha Unnam proposed openstack/nova: Add keypairs field to InstancePayload https://review.openstack.org/419730 | |
| 18:30:10 | openstackgerrit | Matthew Booth proposed openstack/nova: libvirt: Remove more imagecache info file code https://review.openstack.org/423392 | |
| 18:48:28 | sdague | the fail rate on one of the notification tests in the functional bucket is pretty problematic - http://logs.openstack.org/29/421529/1/gate/gate-nova-tox-db-functional-ubuntu-xenial/20c5e9c/console.html#_2017-01-20_18_13_50_140452 | |
| 18:48:59 | sdague | we've got 2 gate resets right now on that | |
| 18:49:03 | sdague | http://logs.openstack.org/09/407309/16/gate/gate-nova-tox-db-functional-ubuntu-xenial/5680644/console.html#_2017-01-20_17_53_41_192031 as well | |