| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-05-24 | |||
| 18:52:23 | sdague | mriedem: I'm trying to understand why you think it isn't :) | |
| 18:52:43 | mriedem | https://github.com/openstack/nova/blob/stable/mitaka/nova/api/openstack/compute/hide_server_addresses.py#L62 is looking up the instance in the cache using the id parameter to the show() method | |
| 18:52:52 | mriedem | which is passed in via the wsgi extension processing code, right? | |
| 18:52:56 | melwitt | mysql is case-insensitive by default, right? it's the request instance cache in the REST API that is case-sensitive (python) | |
| 18:53:05 | mriedem | and i'm assuming is what came in on the request, the {server_id} part that is upper case | |
| 18:53:19 | sdague | mriedem: sure | |
| 18:53:40 | mriedem | sdague: if you look at all of the other extensions, they do something like resp_obj.obj['server']['id'] | |
| 18:53:49 | sdague | oh, right | |
| 18:53:53 | mriedem | which is presumably the id for the server obj that's coming out of the db | |
| 18:53:56 | mriedem | which is why it matches | |
| 18:54:02 | sdague | right, makes sense | |
| 18:54:13 | mriedem | please state that i'm correct for the record | |
| 18:54:17 | mriedem | etc | |
| 18:54:17 | mriedem | let the record show | |
| 18:54:25 | sdague | heh | |
| 18:55:27 | mriedem | tidwellr: can you get a bug reported? | |
| 18:55:36 | mriedem | we can backport to newton, but you're on your own for mitaka | |
| 18:55:56 | tidwellr | mriedem: fair enough, I'll report the bug | |
| 18:57:10 | sdague | mriedem: you want to whip up the fix? or should I? | |
| 18:57:16 | tidwellr | mriedem sdague: thanks for looking into this | |
| 18:57:52 | mriedem | sdague: i can wip it | |
| 18:57:57 | mriedem | btw, way to break s390x guys | |
| 18:58:10 | mriedem | "it's all fun and games until s390 shows up" | |
| 18:58:15 | mriedem | where is jlk when we need him? | |
| 18:58:56 | sdague | tidwellr: no prob. I would love to know why the client application was upper casing things | |
| 18:59:41 | sdague | mriedem: that the etcd thing? | |
| 18:59:51 | mriedem | yeah | |
| 19:00:04 | sdague | it's a good thing that wasn't discussed on the mailing list for the last month | |
| 19:00:11 | mriedem | but also something quotable from jlk back at the first meetup at intel, talking about zvm | |
| 19:00:25 | mriedem | sdague: it actually caught a lot of people by surprise, | |
| 19:00:27 | mriedem | non-dev people | |
| 19:01:20 | sdague | mriedem: sure, but if you are going to be maintaining a non primary architecture, it seems like you should keeping an eye on things | |
| 19:01:53 | mriedem | i don't actually care, | |
| 19:01:59 | mriedem | i just thought it was kind of amusing | |
| 19:02:01 | sdague | heh | |
| 19:02:11 | mriedem | now that i'm no longer paid to care | |
| 19:02:14 | mriedem | \oo/ | |
| 19:02:16 | mriedem | whoa | |
| 19:02:18 | mriedem | that came out wrong | |
| 19:02:21 | sdague | so, to be fair, there was all this work in the devstack patch that looked multiarch | |
| 19:02:30 | sdague | but, then when I started to add sha256sums this morning | |
| 19:02:38 | sdague | I found that nothing except amd64 existed | |
| 19:02:43 | sdague | but ship had sailed | |
| 19:03:52 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Provide original fault message when BFV fails https://review.openstack.org/467715 | |
| 19:05:54 | tidwellr | sdague: well, it turns out the user has an app that was forcing the UUID to upper case | |
| 19:06:03 | sdague | tidwellr: heh | |
| 19:06:15 | melwitt | there's an app for that | |
| 19:07:38 | sdague | tidwellr: is that changable? | |
| 19:07:53 | sdague | or is it going to be easier for you to patch openstack? | |
| 19:08:20 | tidwellr | sdague: yes, I had them change their app to force everything to lower case and all is right in the world | |
| 19:08:32 | sdague | heh | |
| 19:08:51 | sdague | tidwellr: nice | |
| 19:09:02 | tidwellr | sadgue: it would be nice if that was sanitized by nova, but that's pretty low on my priority list :) | |
| 19:09:07 | sdague | tidwellr: fwiw, this is the mitaka patch that would fix things https://gist.github.com/sdague/863d31d4dba44193c6095b21566cd7fa | |
| 19:09:48 | sdague | tidwellr: it actually came up recently, our uuid validation is much looser than people realized, this is probably another instance of that | |
| 19:13:50 | openstack | Launchpad bug 1446850 in python-novaclient "flavor-show of mixed case flavorid fails" [High,Fix released] - Assigned to melanie witt (melwitt) | |
| 19:13:50 | melwitt | be careful with forcing everything to lowercase in clients though, we had a bug a long time ago in novaclient where it tried to lowerc flavor names but they could have been created with mixed case and could cause them not to be found in request cache https://bugs.launchpad.net/python-novaclient/+bug/1446850 | |
| 19:14:10 | mriedem | melwitt: what about mogan indeed http://git.openstack.org/cgit/openstack/mogan/tree/mogan/network/api.py#n120 | |
| 19:14:13 | mriedem | any of that look familiar? | |
| 19:14:39 | tidwellr | sdague: thanks, I'll get a bug filed by EOD | |
| 19:15:33 | sdague | tidwellr: cool | |
| 19:15:57 | melwitt | mriedem: yes ... surprised they wanted to use that, heh | |
| 19:17:50 | mriedem | finding out how to write a unit test for this hide_server_addresses thing is going to be the hard part | |
| 19:19:44 | sdague | mriedem: yeh, you can't use the real test condition, because sqlite goes case sensitive | |
| 19:22:36 | sdague | I don't remember if we ever landed the patch to run real tests with mysql backend for the other case sensitive issues | |
| 19:26:02 | openstack | Launchpad bug 1693335 in OpenStack Compute (nova) "UUID case sensitivity causing unfriendly and unexpected API errors" [Undecided,New] | |
| 19:26:02 | tidwellr | sdague mriedem: https://bugs.launchpad.net/nova/+bug/1693335 | |
| 19:29:13 | mriedem | thanks | |
| 19:33:28 | ckopper | that adds a flavor extra_spec `hw:hide_kvm` to set the correct options through libvirt and was wondering what the correct procedure to get this proposed for upstream was. | |
| 19:33:28 | ckopper | Hi! I am new to nova and openstack development, so I apologize in advance if this is the wrong place to ask or if there is a certain procedure for this. I have a local use case for OpenStack where we want to use PCI passthrough for consumer GPUs which works well in principle. However the NVidia driver checks if it is run within a VM and refuses to load for consumer cards if you don't set a certain KVM/qemu flag (`kvm=off`). I have a local patch | |
| 19:33:33 | ckopper | The patch itself is pretty trivial (mostly tests): https://gist.github.com/claudiok/5dd9fd01b9628e35f0b48884471e58cf | |
| 19:34:02 | ckopper | I assume this needs a spec and blueprint and then submission to gerrit for review but I thought I would check here first | |
| 19:34:16 | openstackgerrit | Sean Dague proposed openstack/nova master: WIP: request_id passing to neutron https://review.openstack.org/467242 | |
| 19:38:07 | mriedem | ckopper: lucky for you someone is already working on that https://review.openstack.org/#/c/459753/ | |
| 19:38:55 | ckopper | Haha - I have to admit I searched for this but apparently missed this review | |
| 19:39:08 | mriedem | sdague: yup, i have a functional test but b/c of sqlite, doing a GET /servers/id with upper-case returns 404 :( | |
| 19:39:35 | sdague | mriedem: yep | |
| 19:39:46 | mriedem | so, i'm not sure how to hack this | |
| 19:40:22 | sdague | so, there is the right way, which is figure out how to be able to run some tests against a mysql backend directly | |
| 19:40:32 | mriedem | i could stub the pull from the db to change the case on what is returned from the db so what's in the cache is different from what's in the request | |
| 19:40:40 | sdague | right, please don't do that | |
| 19:41:02 | sdague | honestly, any energy doing crazy stuff like that should be put into real mysql support in the test suite | |
| 19:41:23 | sdague | the biggest issue was the way the magic oslo.db stuff worked for that iirc | |
| 19:41:30 | mriedem | i'll poke | |
| 19:41:33 | sdague | it's honestly I think better to just ship the patch | |
| 19:41:40 | mriedem | maybe i can just extend test_base.MySQLOpportunisticTestCase | |
| 19:41:59 | sdague | mriedem: there was something weird about that when I tried | |
| 19:42:08 | melwitt | too bad there's not a sqlite pragma for that | |
| 19:43:32 | sdague | mriedem: https://review.openstack.org/#/c/283364/ that's a restore of an attempt to go down that path | |
| 19:43:39 | sdague | maybe it was working, but the fix never landed | |
| 19:44:04 | sdague | mriedem: the issue was about tricking engine fascade I think | |
| 19:44:39 | mriedem | trying it | |
| 19:44:42 | sdague | mriedem: anyway, that might be useful to build the pattern from | |
| 19:45:13 | mriedem | which mysql package do we need? | |
| 19:45:16 | mriedem | mysql-server? | |
| 19:46:10 | mriedem | bindep has both the server and client | |
| 19:46:16 | mriedem | but for some reason they fail to install on 16.04 | |
| 19:46:50 | mriedem | ah here we go | |
| 19:49:01 | ckopper | mriedem: concerning the img_hide_hypervisor_id feature: there one more flag that needs to be set to hide the hypervisor for Windows VMs (not necessary for Linux). (<hyperv><vendor_id state="on" value="FakeID"/></hyperv>). Is this something that should go into a separate patch or would it be appropriate to add this as a comment to the current code review? | |
| 19:49:18 | ckopper | (thanks again for pointing me to it!) | |