| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-01-20 | |||
| 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 | |
| 18:56:22 | melwitt | sdague: mriedem_afk has this change up to add logging to help with investigating that https://review.openstack.org/#/c/421221/ | |
| 18:59:21 | sdague | melwitt: +A | |
| 18:59:53 | melwitt | sweet | |
| 19:05:02 | melwitt | mriedem: ohay, I just commented on your d-g change, I think maybe I found what's wrong | |
| 19:05:11 | mriedem | melwitt: i just updated it for that thanks | |
| 19:05:20 | melwitt | oh, cool | |
| 19:05:24 | mriedem | that doesn't really explain why it's trying to run things on stable though | |
| 19:05:34 | mriedem | since: test -f discover_hosts.sh won't exist on stable | |
| 19:05:38 | mriedem | so it shouldn't try the sudo part | |
| 19:06:38 | melwitt | mriedem: oh, yeah sorry I found something else that I just commented on a sec ago, the permissions on the /logs dir | |
| 19:06:54 | melwitt | I dunno about the stable thing though | |
| 19:08:15 | mriedem | the logs permissions are changed before we run discover_host | |
| 19:08:23 | mriedem | so i'm thinking that's not it | |
| 19:09:09 | melwitt | I thought it's only doing that in the other code path | |
| 19:09:26 | melwitt | in the "else" | |
| 19:12:12 | mriedem | gdi | |
| 19:12:20 | mriedem | melwitt: do you ever get tired of being right? | |
| 19:13:00 | sdague | mriedem: is that the t-shirt slogan for Boston? | |
| 19:13:48 | melwitt | haha, well, we'll see if that actually works. d-g looks like magic to me | |
| 19:14:29 | oomichi | alex_xu: johnthetubaguy: gmann: Can you take a look at https://review.openstack.org/#/c/406377 ? | |
| 19:15:18 | mriedem | sdague: i think the slogan is going to be "wait, what if i upgrade to ocata THIS way" | |
| 19:16:08 | melwitt | haha, yes that | |
| 19:16:11 | sdague | people didn't realize that ocata is actually a puzzle room | |
| 19:16:18 | melwitt | lol | |
| 19:23:03 | mriedem | bauzas: ok +W on the placement fixture | |
| 19:23:47 | mriedem | sdague: so before laura gets home and explodes her travel anxiety on me, can you take a quick peek at https://review.openstack.org/#/c/423005/ and let me know if that's an OK track for this thing with cells v2 in d-g | |
| 19:24:18 | mriedem | the point of putting discover_hosts into a script in devstack is to avoid needing ugly branch logic in d-g itself | |
| 19:34:45 | sdague | mriedem: I think so... | |