Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-04-13
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
02:29:24 openstackgerrit Takashi NATSUME proposed openstack/nova-specs master: List/show all server migration types https://review.openstack.org/407237
02:33:24 openstackgerrit Zhaokun Fu proposed openstack/nova master: fix url https://review.openstack.org/456810
02:36:22 openstackgerrit Zhaokun Fu proposed openstack/nova master: fix url https://review.openstack.org/456813
03:12:00 openstackgerrit Yasufumi Ogawa proposed openstack/nova-specs master: Add spec "Libvirt options for DPDK" https://review.openstack.org/456601
03:15:52 openstackgerrit Takashi NATSUME proposed openstack/python-novaclient master: Microversion 2.42 - Fix tag attribute disappearing https://review.openstack.org/429512
03:20:25 alex_xu gmann: good catch, we can remove that one!
03:20:45 alex_xu gmann: add your api-ref patch to the my list
03:20:55 gmann alex_xu: thanks :)
03:21:40 alex_xu gmann: thanks for that good catch also
03:22:11 gmann alex_xu: np!, i have removed from flavor patches
03:22:23 alex_xu gmann: cool :)
03:40:06 openstackgerrit Alex Xu proposed openstack/nova master: Use plain routes list for '/os-keypairs' endpoint instead of stevedore https://review.openstack.org/456478
03:40:06 openstackgerrit Alex Xu proposed openstack/nova master: Use plain routes list for '/os-aggregates' endpoint instead of stevedore https://review.openstack.org/456825
04:38:16 openstackgerrit Takashi NATSUME proposed openstack/nova master: Add a functional test for 'removeFloatingIp' action https://review.openstack.org/453026
04:42:58 openstackgerrit Alex Xu proposed openstack/nova master: Use plain routes list for flavors-access endpoint instead of stevedore https://review.openstack.org/456589
04:42:59 openstackgerrit Alex Xu proposed openstack/nova master: Use plain routes list for '/servers' endpoint instead of stevedore https://review.openstack.org/445864
04:42:59 openstackgerrit Alex Xu proposed openstack/nova master: Use plain routes list for flavor endpoint instead of stevedore[1] https://review.openstack.org/456549
04:43:00 openstackgerrit Alex Xu proposed openstack/nova master: Use plain routes list for '/os-aggregates' endpoint instead of stevedore https://review.openstack.org/456825
04:43:00 openstackgerrit Alex Xu proposed openstack/nova master: Use plain routes list for flavors-extraspecs endpoint instead of stevedore https://review.openstack.org/456585
04:43:01 openstackgerrit Alex Xu proposed openstack/nova master: Use plain routes list for '/os-keypairs' endpoint instead of stevedore https://review.openstack.org/456478
05:01:35 alex_xu gmann: ^ I put my patches depend on yours
05:21:36 gmann alex_xu: thanks
05:39:12 openstackgerrit Ildiko Vancsa proposed openstack/nova master: Add Cinder V3 Detach calls https://review.openstack.org/438750
05:51:30 gmann alex_xu: how about maintaining ROUTE_LIST in sort order with url - https://review.openstack.org/#/c/456589/3/nova/api/openstack/compute/routes.py
05:51:42 gmann alex_xu: will be easy to read once we will have all plain url there
05:52:48 gmann alex_xu: like dir order in https://github.com/openstack/nova/tree/master/doc/api_samples
05:55:02 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Use plain routes list for '/os-keypairs' endpoint instead of stevedore https://review.openstack.org/456478
05:57:03 alex_xu gmann: +1 for sort order with url
05:57:34 gmann alex_xu: ok, ll do quickly for flavor one
05:59:18 alex_xu gmann: thanks
06:37:52 openstackgerrit Ildiko Vancsa proposed openstack/nova master: Add Cinder V3 Detach calls https://review.openstack.org/438750
06:41:52 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Use plain routes list for flavor endpoint instead of stevedore[1] https://review.openstack.org/456549
06:59:16 gmann alex_xu: base patch server one need updates (server usage entry in unsed_ list) - https://review.openstack.org/#/c/445864/13
06:59:50 gmann alex_xu: if you are not doing that, should i update ?
07:00:59 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Use plain routes list for flavors-extraspecs endpoint instead of stevedore https://review.openstack.org/456585
07:01:59 gmann alex_xu: after that i can rebase other patch in one shot
07:21:18 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Use plain routes list for '/servers' endpoint instead of stevedore https://review.openstack.org/445864
07:21:20 gmann alex_xu: ^^ + changed url in sort order in router.py
07:35:49 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Use plain routes list for flavor endpoint instead of stevedore[1] https://review.openstack.org/456549

Earlier   Later