| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-04-13 | |||
| 22:59:00 | mriedem | https://review.openstack.org/#/c/442162/ was pike | |
| 22:59:21 | dansmith | oh yeaq | |
| 22:59:25 | dansmith | I was thinking the p-1 | |
| 22:59:31 | mriedem | ok, | |
| 22:59:33 | dansmith | you'd rather do that? | |
| 22:59:43 | dansmith | I think I have this changed, fwiw | |
| 22:59:44 | mriedem | i thought red hat backported multi-cell support for rhosp ocata :) | |
| 22:59:51 | mriedem | how about we do the safe thing now | |
| 22:59:57 | mriedem | and look at the lazy way later | |
| 22:59:58 | claudiub|2 | mriedem: hellou. just peeking around here for a bit. this blueprint was approved in ocata, and had the implementation up for review, but it didn't merge in ocata: https://blueprints.launchpad.net/nova/+spec/hyper-v-nested-virtualization | |
| 23:00:08 | melwitt | it seems like doing that would be weird. though it is doing like Forbidden and Invalid that way but not NotFound. dunno the history on that | |
| 23:00:10 | openstackgerrit | Dan Smith proposed openstack/nova master: Fix hypervisors api missing HostMappingNotFound handlers https://review.openstack.org/456227 | |
| 23:00:15 | claudiub|2 | can it get re-approved? :D | |
| 23:00:22 | mriedem | claudiub|2: for you? anything. | |
| 23:00:25 | claudiub|2 | <3 | |
| 23:00:37 | claudiub|2 | seems like an early easter. :) | |
| 23:00:38 | mriedem | melwitt: i think it's basically just a catch all for global types of failures | |
| 23:00:43 | mriedem | which hostmappingnotfound kind of is | |
| 23:00:55 | dansmith | mriedem: yeah that one is pretty easy to catch and always do the right thing | |
| 23:01:04 | mriedem | claudiub|2: jesus hasn't come up and seen his shadow yet so don't count your chickens | |
| 23:01:05 | dansmith | it's fairly specific about what happened and what should be done | |
| 23:01:07 | melwitt | yeah. I was just noting it for whatever reason intentionally left out the base class for NotFound | |
| 23:01:07 | mriedem | this easter could be different | |
| 23:01:19 | dansmith | mriedem: could be super confusing for someone trying to track down the 404 though | |
| 23:01:25 | mriedem | dansmith: yeah i totally agree | |
| 23:01:39 | mriedem | dansmith: i never knew that existed until about a year ago when laski pointed it out | |
| 23:01:45 | mriedem | ^ is generally my story for everything | |
| 23:01:46 | claudiub|2 | well, it is a bit cloud-y lately | |
| 23:01:51 | melwitt | laski, that rascal | |
| 23:03:29 | claudiub|2 | also, i've submitted the sr-iov bp: https://blueprints.launchpad.net/nova/+spec/hyperv-sriov no implementation yet i'm afraid, apparently there's a whole guide to follow just to configure the nics properly. | |
| 23:04:24 | mriedem | claudiub|2: sounds about right | |
| 23:04:31 | mriedem | claudiub|2: do you get a long weekend? | |
| 23:05:05 | claudiub|2 | well, technically i'm free on monday, i'll still be around | |
| 23:05:16 | claudiub|2 | i don't like vacations too much. :D | |
| 23:05:23 | mriedem | you can read the sriov manual | |
| 23:05:42 | claudiub|2 | yeah, that is in my plan. :D | |
| 23:06:14 | claudiub|2 | and question: can we live-migrate vms with sr-iov? | |
| 23:06:19 | mriedem | claudiub|2: no | |
| 23:06:23 | mriedem | resource tracker issue | |
| 23:06:24 | claudiub|2 | hm, interesting | |
| 23:06:26 | mriedem | *issues | |
| 23:06:31 | mriedem | cfriesen knows about it | |
| 23:06:40 | claudiub|2 | is that a nova limitation, or a hypervisor limitation? | |
| 23:09:47 | claudiub|2 | so, alaski is like beetlejuice - say his name too many times and he shows up. :D | |
| 23:10:53 | alaski | I'm glad I appear to have explained things to folks in the past, because I don't remember much these days | |
| 23:15:26 | melwitt | you missed out on some cerebral shelve feature talk today | |
| 23:20:37 | alaski | I hope it was good. I am very much not proud of introducing that feature. | |
| 23:25:38 | melwitt | hm ... define good. mriedem found that upon shelve offload, volumes are not disconnected and then also when network is cleaned up, that only does something for nova-net through the network_api, the neutron implementation is a no-op pass | |
| 23:27:00 | melwitt | so it pretty much relies on the unshelve putting the instance back on the same host it was shelved offloaded from, which seems really unlikely | |
| 23:29:15 | alaski | yeah, I would not expect that to happen much in practice | |
| 23:33:17 | gmann | mriedem: thank i almost forgot about that bug, ll check patch | |
| 23:37:04 | cfriesen | claudiub|2: the first issue is that we need to get resource tracking in general working over live migration. (see https://review.openstack.org/#/c/244489 and the follow-on patches) | |
| 23:39:23 | cfriesen | do people actually use shelve? If so, why hasn't anyone noticed this? | |
| 23:40:22 | alaski | maybe not using volumes, or neutron? | |
| 23:40:41 | alaski | it did seem to have a small following, despite discouragement about using it | |
| 23:42:53 | melwitt | yeah, I would guess similar. I have heard of it being used and it would work fine for nova-net and no volumes | |
| 23:53:20 | mriedem | cfriesen: someone has, long time ago | |
| 23:53:23 | mriedem | the stratoscale guys | |
| 23:53:26 | mriedem | from the bug in the ML | |
| #openstack-nova - 2017-04-14 | |||
| 00:03:10 | gmann | mriedem: just replied, i think we need version bump otherwise it will be interop issue | |
| 00:04:00 | gmann | mriedem: cloud with >2.20 may or may not have this fix and it end up with different API behavior on clouds. | |
| 00:05:00 | gmann | mriedem: we could have avoided version bump by backporting fix since 2.20 was in but that was before mitaka which is EOL now | |
| 00:05:33 | mriedem | gmann: then we backport the fix | |
| 00:05:45 | mriedem | we've fixed regressions caused by microversions in the past and backported them | |
| 00:10:17 | gmann | mriedem: and for mitaka? | |
| 00:10:29 | mriedem | gmann: we have all kinds of latent bugs that go back to releases that are long end of life by now | |
| 00:10:45 | mriedem | just because we can't backport something to the release that introduced the bug doesn't mean we can't fix it and backport to what's supported | |
| 00:13:19 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Add a functional test for 'removeFloatingIp' action https://review.openstack.org/453026 | |
| 00:13:34 | gmann | mriedem: yea we can do for supported branches one but my concern with cloud (mitaka) and cloud(mitaka+) where app/sdk would face different behavior issue even both cloud have >2.20 version | |
| 00:16:39 | mriedem | honestly i don't really see this as something to worry about | |
| 00:16:46 | gmann | alex_xu: can we get rid of ALIAS also now? - https://review.openstack.org/#/c/456478/1 | |
| 00:16:47 | mriedem | if you have a volume attached before 2.20 | |
| 00:16:59 | mriedem | and mountpoint isn't in there, it's completely bonkers | |
| 00:17:08 | mriedem | so i don't even know why that check is in there in the the first place | |
| 00:17:30 | mriedem | i really don't think we should bend over backward and create a new microversion for a thing in the API that makes really no sense to be validating on | |
| 00:17:38 | mriedem | we're raising a 404 for something when we actually found the attachment | |
| 00:18:01 | gmann | yea i also confused about check, it could have just ignore 'device' or None | |
| 00:20:00 | mriedem | gmann: that check goes way back 5 years https://review.openstack.org/#/c/7406/ | |
| 00:20:06 | mriedem | and for all intents and purposes, made no sense | |
| 00:22:25 | gmann | mriedem: yes and not sure why it was done for SHOW only and LIST just skip mountpoint. | |
| 00:22:25 | openstackgerrit | Takashi NATSUME proposed openstack/nova-specs master: Abort Cold Migration https://review.openstack.org/334732 | |
| 00:25:35 | gmann | mriedem: i understand your point of fixing it for shelve offload at least where it will always 404 but m still worried about this fix discoverability as it change API code | |
| 00:28:36 | gmann | mriedem: but yes backporting makes good reason to avoid version bump if everyone is fine with that | |
| 00:35:36 | takashin | Nova-spec cores, would you review https://review.openstack.org/#/c/407237/ ? It got one +2. | |
| 00:41:25 | Qeas | Hi everyone, I have a problem with block device mappings in my environment and would really appreciate any advice on this. The problem: iSCSI mappings have changed on the backend, though the data is untouched. All the VMs were shutdown before that. Is there a way for Nova to re-attach those same volumes but with their new iSCSI LUNs? I was thinking of cleaning connection_info in mysql, but scared of consequences. | |
| 00:44:07 | openstackgerrit | Bryan Song proposed openstack/nova master: Migrate VMs failed when anti-affinity group was removed from VMs. https://review.openstack.org/451664 | |
| 01:03:04 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Use plain routes list for flavors-access endpoint instead of stevedore https://review.openstack.org/456589 | |
| 01:04:11 | gmann | mriedem: alex_xu oomichi can you check this, need this to fill api-ref for security group rule - https://review.openstack.org/#/c/452055/ | |
| 01:39:52 | openstackgerrit | Zhaokun Fu proposed openstack/nova master: optimize url https://review.openstack.org/456797 | |
| 01:42:09 | openstackgerrit | Bryan Song proposed openstack/nova master: Migrate VMs failed when anti-affinity group was removed from VMs. https://review.openstack.org/451664 | |
| 01:43:36 | openstackgerrit | Bryan Song proposed openstack/nova master: Migrate VMs failed when anti-affinity group was removed from VMs. https://review.openstack.org/451664 | |
| 01:45:36 | openstackgerrit | Bryan Song proposed openstack/nova master: Migrate VMs failed when anti-affinity group was removed from VMs. https://review.openstack.org/451664 | |
| 01:46:08 | openstackgerrit | Bryan Song proposed openstack/nova master: Migrate VMs failed when anti-affinity group was removed from VMs. https://review.openstack.org/451664 | |
| 01:46:32 | openstackgerrit | Zhaokun Fu proposed openstack/nova master: fix url https://review.openstack.org/456798 | |
| 01:55:12 | mriedem | gmann: sure, i'll take a look tomorrow | |
| 01:55:17 | mriedem | too many late nights this week | |
| 01:55:35 | gmann | mriedem: sure, thanks | |
| 02:09:49 | openstackgerrit | Zhaokun Fu proposed openstack/nova master: change http to https https://review.openstack.org/456802 | |
| 02:25:29 | openstackgerrit | Yasufumi Ogawa proposed openstack/nova-specs master: Merge "Libvirt options for DPDK" https://review.openstack.org/456601 | |
| 02:26:28 | openstackgerrit | Zhaokun Fu proposed openstack/nova master: optimize url https://review.openstack.org/456807 | |