*** mtanino has quit IRC | 00:01 | |
gmann | mriedem: just replied, i think we need version bump otherwise it will be interop issue | 00:03 |
---|---|---|
*** itlinux has quit IRC | 00:03 | |
gmann | mriedem: cloud with >2.20 may or may not have this fix and it end up with different API behavior on clouds. | 00:04 |
*** nkorabli has quit IRC | 00:04 | |
*** ijw has quit IRC | 00:04 | |
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 |
mriedem | gmann: then we backport the fix | 00:05 |
mriedem | we've fixed regressions caused by microversions in the past and backported them | 00:05 |
*** jogo has quit IRC | 00:05 | |
*** jogo has joined #openstack-nova | 00:07 | |
*** takashin has joined #openstack-nova | 00:08 | |
*** takashin has left #openstack-nova | 00:08 | |
*** baoli_ has quit IRC | 00:10 | |
gmann | mriedem: and for mitaka? | 00:10 |
mriedem | gmann: we have all kinds of latent bugs that go back to releases that are long end of life by now | 00:10 |
*** winston-d_ has joined #openstack-nova | 00:10 | |
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:10 |
*** takashin has joined #openstack-nova | 00:12 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Add a functional test for 'removeFloatingIp' action https://review.openstack.org/453026 | 00:13 |
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:13 |
*** scottda has quit IRC | 00:15 | |
*** MasterOfBugs has joined #openstack-nova | 00:16 | |
mriedem | honestly i don't really see this as something to worry about | 00:16 |
gmann | alex_xu: can we get rid of ALIAS also now? - https://review.openstack.org/#/c/456478/1 | 00:16 |
mriedem | if you have a volume attached before 2.20 | 00:16 |
mriedem | and mountpoint isn't in there, it's completely bonkers | 00:16 |
mriedem | so i don't even know why that check is in there in the the first place | 00:17 |
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 |
mriedem | we're raising a 404 for something when we actually found the attachment | 00:17 |
gmann | yea i also confused about check, it could have just ignore 'device' or None | 00:18 |
*** ngupta has quit IRC | 00:19 | |
mriedem | gmann: that check goes way back 5 years https://review.openstack.org/#/c/7406/ | 00:20 |
mriedem | and for all intents and purposes, made no sense | 00:20 |
gmann | mriedem: yes and not sure why it was done for SHOW only and LIST just skip mountpoint. | 00:22 |
openstackgerrit | Takashi NATSUME proposed openstack/nova-specs master: Abort Cold Migration https://review.openstack.org/334732 | 00:22 |
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:25 |
*** artom has quit IRC | 00:26 | |
*** hongbin has quit IRC | 00:26 | |
*** artom has joined #openstack-nova | 00:26 | |
*** artom has quit IRC | 00:28 | |
*** artom has joined #openstack-nova | 00:28 | |
gmann | mriedem: but yes backporting makes good reason to avoid version bump if everyone is fine with that | 00:28 |
takashin | Nova-spec cores, would you review https://review.openstack.org/#/c/407237/ ? It got one +2. | 00:35 |
*** Qeas has joined #openstack-nova | 00:37 | |
*** Shunli has joined #openstack-nova | 00:39 | |
*** catintheroof has joined #openstack-nova | 00:41 | |
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:41 |
openstackgerrit | Bryan Song proposed openstack/nova master: Migrate VMs failed when anti-affinity group was removed from VMs. https://review.openstack.org/451664 | 00:44 |
*** baoli has joined #openstack-nova | 00:48 | |
*** baoli has quit IRC | 00:48 | |
*** dtp has quit IRC | 00:49 | |
*** baoli has joined #openstack-nova | 00:49 | |
*** huanxie has joined #openstack-nova | 00:51 | |
*** huanxie has quit IRC | 00:52 | |
*** zhenguo has joined #openstack-nova | 00:52 | |
*** links has joined #openstack-nova | 00:53 | |
*** ijw has joined #openstack-nova | 01:01 | |
*** cNilesh has joined #openstack-nova | 01:02 | |
openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Use plain routes list for flavors-access endpoint instead of stevedore https://review.openstack.org/456589 | 01:03 |
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:04 |
*** Apoorva_ has quit IRC | 01:04 | |
*** Apoorva has joined #openstack-nova | 01:04 | |
*** ijw has quit IRC | 01:10 | |
*** gyee has quit IRC | 01:22 | |
*** dixiaoli has joined #openstack-nova | 01:25 | |
*** ijw has joined #openstack-nova | 01:27 | |
*** ijw has quit IRC | 01:28 | |
*** thorst has quit IRC | 01:29 | |
*** ijw has joined #openstack-nova | 01:29 | |
*** harlowja has quit IRC | 01:29 | |
*** catintheroof has quit IRC | 01:30 | |
*** catintheroof has joined #openstack-nova | 01:30 | |
*** catintheroof has quit IRC | 01:31 | |
openstackgerrit | Zhaokun Fu proposed openstack/nova master: optimize url https://review.openstack.org/456797 | 01:39 |
openstackgerrit | Bryan Song proposed openstack/nova master: Migrate VMs failed when anti-affinity group was removed from VMs. https://review.openstack.org/451664 | 01:42 |
*** gjayavelu has quit IRC | 01:42 | |
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 |
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 |
*** MasterOfBugs has quit IRC | 01:45 | |
*** MasterOfBugs has joined #openstack-nova | 01:46 | |
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 |
openstackgerrit | Zhaokun Fu proposed openstack/nova master: fix url https://review.openstack.org/456798 | 01:46 |
*** artom has quit IRC | 01:52 | |
*** artom has joined #openstack-nova | 01:52 | |
*** iceyao has joined #openstack-nova | 01:53 | |
mriedem | gmann: sure, i'll take a look tomorrow | 01:55 |
*** vks1 has joined #openstack-nova | 01:55 | |
mriedem | too many late nights this week | 01:55 |
gmann | mriedem: sure, thanks | 01:55 |
*** thorst has joined #openstack-nova | 01:58 | |
*** vks1 has quit IRC | 01:58 | |
*** artom has quit IRC | 01:58 | |
*** thorst has quit IRC | 01:58 | |
*** artom has joined #openstack-nova | 01:58 | |
*** nkorabli has joined #openstack-nova | 02:01 | |
*** nkorabli has quit IRC | 02:05 | |
*** Shunli has quit IRC | 02:07 | |
*** Shunli has joined #openstack-nova | 02:07 | |
openstackgerrit | Zhaokun Fu proposed openstack/nova master: change http to https https://review.openstack.org/456802 | 02:09 |
*** ijw has quit IRC | 02:10 | |
*** crushil has joined #openstack-nova | 02:11 | |
*** Apoorva has quit IRC | 02:12 | |
*** esberglu has joined #openstack-nova | 02:13 | |
*** lyan has joined #openstack-nova | 02:13 | |
*** tbachman has joined #openstack-nova | 02:13 | |
*** esberglu has quit IRC | 02:13 | |
*** esberglu has joined #openstack-nova | 02:13 | |
*** esberglu has quit IRC | 02:17 | |
*** ngupta has joined #openstack-nova | 02:19 | |
openstackgerrit | Yasufumi Ogawa proposed openstack/nova-specs master: Merge "Libvirt options for DPDK" https://review.openstack.org/456601 | 02:25 |
openstackgerrit | Zhaokun Fu proposed openstack/nova master: optimize url https://review.openstack.org/456807 | 02:26 |
*** Apoorva has joined #openstack-nova | 02:28 | |
*** thorst has joined #openstack-nova | 02:29 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova-specs master: List/show all server migration types https://review.openstack.org/407237 | 02:29 |
*** Jack_Iv has joined #openstack-nova | 02:29 | |
*** zhurong has joined #openstack-nova | 02:31 | |
*** vks1 has joined #openstack-nova | 02:32 | |
openstackgerrit | Zhaokun Fu proposed openstack/nova master: fix url https://review.openstack.org/456810 | 02:33 |
*** Apoorva has quit IRC | 02:34 | |
*** artom has quit IRC | 02:34 | |
*** Jack_Iv has quit IRC | 02:34 | |
*** artom has joined #openstack-nova | 02:34 | |
*** MasterOfBugs has quit IRC | 02:36 | |
*** artom has quit IRC | 02:36 | |
openstackgerrit | Zhaokun Fu proposed openstack/nova master: fix url https://review.openstack.org/456813 | 02:36 |
*** artom has joined #openstack-nova | 02:36 | |
*** Kevin_Zheng has joined #openstack-nova | 02:36 | |
*** MasterOfBugs has joined #openstack-nova | 02:37 | |
*** vks1 has quit IRC | 02:38 | |
*** tbachman has quit IRC | 02:39 | |
*** ijw has joined #openstack-nova | 02:39 | |
*** yonglihe has joined #openstack-nova | 02:39 | |
*** vks1 has joined #openstack-nova | 02:42 | |
*** ijw has quit IRC | 02:44 | |
*** thorst has quit IRC | 02:48 | |
*** gouthamr has quit IRC | 02:49 | |
*** nkorabli has joined #openstack-nova | 03:02 | |
*** crushil has quit IRC | 03:04 | |
*** ngupta has quit IRC | 03:04 | |
*** ngupta has joined #openstack-nova | 03:04 | |
*** hamzy has joined #openstack-nova | 03:06 | |
*** nkorabli has quit IRC | 03:07 | |
*** ijw has joined #openstack-nova | 03:09 | |
*** fragatina has quit IRC | 03:09 | |
*** fragatina has joined #openstack-nova | 03:10 | |
openstackgerrit | Yasufumi Ogawa proposed openstack/nova-specs master: Add spec "Libvirt options for DPDK" https://review.openstack.org/456601 | 03:12 |
*** ijw has quit IRC | 03:13 | |
*** fragatina has quit IRC | 03:15 | |
openstackgerrit | Takashi NATSUME proposed openstack/python-novaclient master: Microversion 2.42 - Fix tag attribute disappearing https://review.openstack.org/429512 | 03:15 |
*** fragatina has joined #openstack-nova | 03:17 | |
alex_xu | gmann: good catch, we can remove that one! | 03:20 |
alex_xu | gmann: add your api-ref patch to the my list | 03:20 |
gmann | alex_xu: thanks :) | 03:20 |
alex_xu | gmann: thanks for that good catch also | 03:21 |
gmann | alex_xu: np!, i have removed from flavor patches | 03:22 |
*** fragatina has quit IRC | 03:22 | |
alex_xu | gmann: cool :) | 03:22 |
*** gjayavelu has joined #openstack-nova | 03:23 | |
*** ngupta has quit IRC | 03:23 | |
*** nicolasbock has quit IRC | 03:24 | |
*** ngupta has joined #openstack-nova | 03:24 | |
*** hongbin has joined #openstack-nova | 03:33 | |
*** hongbin has quit IRC | 03:33 | |
*** hongbin has joined #openstack-nova | 03:34 | |
*** ngupta has quit IRC | 03:36 | |
*** armax has joined #openstack-nova | 03:37 | |
*** ngupta has joined #openstack-nova | 03:37 | |
*** fragatina has joined #openstack-nova | 03:38 | |
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 |
openstackgerrit | Alex Xu proposed openstack/nova master: Use plain routes list for '/os-aggregates' endpoint instead of stevedore https://review.openstack.org/456825 | 03:40 |
*** Shunli has quit IRC | 03:41 | |
*** ngupta has quit IRC | 03:41 | |
*** Shunli has joined #openstack-nova | 03:42 | |
*** fragatina has quit IRC | 03:42 | |
*** harlowja has joined #openstack-nova | 03:44 | |
*** vks1 has quit IRC | 03:45 | |
*** vks1 has joined #openstack-nova | 03:50 | |
*** trinaths has joined #openstack-nova | 03:53 | |
*** gjayavelu has quit IRC | 03:56 | |
*** lyan has quit IRC | 03:57 | |
*** artom has quit IRC | 03:58 | |
*** artom has joined #openstack-nova | 03:58 | |
*** artom has quit IRC | 04:00 | |
*** artom has joined #openstack-nova | 04:00 | |
*** zhurong has quit IRC | 04:02 | |
*** faizy has joined #openstack-nova | 04:05 | |
*** Qeas has quit IRC | 04:12 | |
*** dixiaoli has quit IRC | 04:21 | |
*** gongysh has joined #openstack-nova | 04:23 | |
*** harlowja has quit IRC | 04:23 | |
*** psachin has joined #openstack-nova | 04:25 | |
*** hongbin has quit IRC | 04:27 | |
*** zhurong has joined #openstack-nova | 04:29 | |
*** gongysh has quit IRC | 04:31 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Add a functional test for 'removeFloatingIp' action https://review.openstack.org/453026 | 04:38 |
*** harlowja has joined #openstack-nova | 04:38 | |
*** mdnadeem has joined #openstack-nova | 04:42 | |
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 |
openstackgerrit | Alex Xu proposed openstack/nova master: Use plain routes list for '/servers' endpoint instead of stevedore https://review.openstack.org/445864 | 04:42 |
openstackgerrit | Alex Xu proposed openstack/nova master: Use plain routes list for flavor endpoint instead of stevedore[1] https://review.openstack.org/456549 | 04:42 |
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 |
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 |
openstackgerrit | Alex Xu proposed openstack/nova master: Use plain routes list for '/os-keypairs' endpoint instead of stevedore https://review.openstack.org/456478 | 04:43 |
*** thorst has joined #openstack-nova | 04:45 | |
*** ssurana has joined #openstack-nova | 04:46 | |
*** jdillaman has quit IRC | 04:53 | |
*** dillaman has quit IRC | 04:53 | |
*** prateek has joined #openstack-nova | 04:53 | |
alex_xu | gmann: ^ I put my patches depend on yours | 05:01 |
*** nkorabli has joined #openstack-nova | 05:03 | |
*** jdillaman has joined #openstack-nova | 05:05 | |
*** dillaman has joined #openstack-nova | 05:05 | |
*** nkorabli has quit IRC | 05:08 | |
*** rcernin has quit IRC | 05:08 | |
*** bmace has quit IRC | 05:11 | |
*** bmace has joined #openstack-nova | 05:11 | |
*** baoli has quit IRC | 05:13 | |
*** thorst has quit IRC | 05:14 | |
*** rcernin has joined #openstack-nova | 05:17 | |
*** gcb has quit IRC | 05:19 | |
gmann | alex_xu: thanks | 05:21 |
*** MasterOfBugs has quit IRC | 05:21 | |
*** MasterOfBugs has joined #openstack-nova | 05:21 | |
*** dixiaoli has joined #openstack-nova | 05:23 | |
*** MasterOfBugs has quit IRC | 05:24 | |
*** MasterOfBugs has joined #openstack-nova | 05:24 | |
*** prateek has quit IRC | 05:26 | |
*** gcb has joined #openstack-nova | 05:31 | |
*** pramodrj07 has joined #openstack-nova | 05:32 | |
*** pramodrj07 has quit IRC | 05:35 | |
*** MasterOfBugs has quit IRC | 05:35 | |
*** MasterOfBugs has joined #openstack-nova | 05:35 | |
*** ssurana has quit IRC | 05:37 | |
openstackgerrit | Ildiko Vancsa proposed openstack/nova master: Add Cinder V3 Detach calls https://review.openstack.org/438750 | 05:39 |
*** Jack_Iv has joined #openstack-nova | 05:41 | |
*** prateek has joined #openstack-nova | 05:41 | |
*** takashin has left #openstack-nova | 05:42 | |
*** Jack_Iv_ has joined #openstack-nova | 05:44 | |
*** trinaths has quit IRC | 05:45 | |
*** Jack_Iv has quit IRC | 05:46 | |
*** masayukig has quit IRC | 05:49 | |
*** masayukig has joined #openstack-nova | 05:49 | |
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 |
gmann | alex_xu: will be easy to read once we will have all plain url there | 05:51 |
gmann | alex_xu: like dir order in https://github.com/openstack/nova/tree/master/doc/api_samples | 05:52 |
openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Use plain routes list for '/os-keypairs' endpoint instead of stevedore https://review.openstack.org/456478 | 05:55 |
*** harlowja has quit IRC | 05:55 | |
alex_xu | gmann: +1 for sort order with url | 05:57 |
gmann | alex_xu: ok, ll do quickly for flavor one | 05:57 |
*** trinaths has joined #openstack-nova | 05:59 | |
alex_xu | gmann: thanks | 05:59 |
*** trinaths has quit IRC | 05:59 | |
*** voelzmo has joined #openstack-nova | 06:00 | |
*** gongysh has joined #openstack-nova | 06:00 | |
*** trinaths has joined #openstack-nova | 06:01 | |
*** voelzmo has quit IRC | 06:02 | |
*** MarkMielke has quit IRC | 06:10 | |
*** thorst has joined #openstack-nova | 06:11 | |
*** thorst has quit IRC | 06:17 | |
*** prateek has quit IRC | 06:20 | |
*** zhurong has quit IRC | 06:20 | |
*** armax has quit IRC | 06:21 | |
*** dixiaoli has quit IRC | 06:27 | |
*** zhurong has joined #openstack-nova | 06:31 | |
openstackgerrit | Ildiko Vancsa proposed openstack/nova master: Add Cinder V3 Detach calls https://review.openstack.org/438750 | 06:37 |
openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Use plain routes list for flavor endpoint instead of stevedore[1] https://review.openstack.org/456549 | 06:41 |
*** armax has joined #openstack-nova | 06:41 | |
*** ralonsoh has joined #openstack-nova | 06:45 | |
*** avolkov has joined #openstack-nova | 06:48 | |
*** damien_r has joined #openstack-nova | 06:57 | |
gmann | alex_xu: base patch server one need updates (server usage entry in unsed_ list) - https://review.openstack.org/#/c/445864/13 | 06:59 |
gmann | alex_xu: if you are not doing that, should i update ? | 06: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:00 |
*** amotoki has joined #openstack-nova | 07:01 | |
gmann | alex_xu: after that i can rebase other patch in one shot | 07:01 |
*** vks1 has quit IRC | 07:03 | |
*** abalutoiu has quit IRC | 07:08 | |
*** tesseract has joined #openstack-nova | 07:10 | |
*** thorst has joined #openstack-nova | 07:13 | |
*** Jack_Iv_ has quit IRC | 07:17 | |
*** thorst has quit IRC | 07:18 | |
*** vks1 has joined #openstack-nova | 07:18 | |
*** esberglu has joined #openstack-nova | 07:18 | |
*** Jack_Iv has joined #openstack-nova | 07:20 | |
openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Use plain routes list for '/servers' endpoint instead of stevedore https://review.openstack.org/445864 | 07:21 |
gmann | alex_xu: ^^ + changed url in sort order in router.py | 07:21 |
*** esberglu has quit IRC | 07:22 | |
*** dixiaoli has joined #openstack-nova | 07:24 | |
*** MasterOfBugs has quit IRC | 07:29 | |
*** bauzas is now known as bauwser | 07:33 | |
openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Use plain routes list for flavor endpoint instead of stevedore[1] https://review.openstack.org/456549 | 07:35 |
openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/nova master: Add Neutron port capabilities to devspec in request https://review.openstack.org/451777 | 07:42 |
openstackgerrit | Bryan Song proposed openstack/nova master: Migrate VMs failed when anti-affinity group was removed from VMs. https://review.openstack.org/451664 | 07:43 |
*** trinaths has left #openstack-nova | 07:50 | |
openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Use plain routes list for flavors-extraspecs endpoint instead of stevedore https://review.openstack.org/456585 | 07:50 |
*** zzzeek has quit IRC | 08:00 | |
*** zzzeek has joined #openstack-nova | 08:01 | |
openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Use plain routes list for flavors-access endpoint instead of stevedore https://review.openstack.org/456589 | 08:03 |
*** zenoway has joined #openstack-nova | 08:13 | |
*** thorst has joined #openstack-nova | 08:14 | |
*** zenoway has quit IRC | 08:15 | |
*** zenoway has joined #openstack-nova | 08:16 | |
*** Jack_Iv has quit IRC | 08:16 | |
*** efoley has joined #openstack-nova | 08:18 | |
*** efoley_ has joined #openstack-nova | 08:20 | |
*** nkorabli has joined #openstack-nova | 08:22 | |
*** efoley has quit IRC | 08:24 | |
*** cdent has joined #openstack-nova | 08:24 | |
*** armax has quit IRC | 08:26 | |
*** armax has joined #openstack-nova | 08:27 | |
*** vks1 has quit IRC | 08:28 | |
*** Jack_Iv has joined #openstack-nova | 08:29 | |
openstackgerrit | Ildiko Vancsa proposed openstack/nova master: Add Cinder v3 detach to local_cleanup https://review.openstack.org/456851 | 08:31 |
openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Use plain routes list for '/os-keypairs' endpoint instead of stevedore https://review.openstack.org/456478 | 08:32 |
*** thorst has quit IRC | 08:33 | |
alex_xu | gmann: thanks | 08:38 |
alex_xu | gmann: let me rebase the os-aggregates one | 08:39 |
gmann | alex_xu: just doing | 08:39 |
gmann | 5 sec ..... | 08:39 |
openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Use plain routes list for '/os-aggregates' endpoint instead of stevedore https://review.openstack.org/456825 | 08:40 |
gmann | alex_xu: ^^ | 08:40 |
gmann | alex_xu: check if all good. | 08:41 |
gmann | alex_xu: need to leave now. tty on monday. | 08:41 |
alex_xu | gmann: thanks :), see you on monday | 08:42 |
*** abalutoiu has joined #openstack-nova | 08:45 | |
*** kristian__ has joined #openstack-nova | 08:51 | |
*** kristian__ has quit IRC | 08:51 | |
*** liusheng has quit IRC | 09:12 | |
openstackgerrit | Hao Li proposed openstack/nova master: Update hypervisor host when multi-n-cpu with ironic driver do rebalancing https://review.openstack.org/456865 | 09:14 |
*** markvoelker_ has quit IRC | 09:17 | |
*** amotoki has quit IRC | 09:20 | |
openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Use deepcopy when process filters in db api https://review.openstack.org/456872 | 09:25 |
*** Jack_Iv has quit IRC | 09:27 | |
*** Shunli has quit IRC | 09:28 | |
*** gcb has quit IRC | 09:31 | |
*** Jack_Iv has joined #openstack-nova | 09:32 | |
*** links has quit IRC | 09:33 | |
*** liusheng has joined #openstack-nova | 09:38 | |
*** gcb has joined #openstack-nova | 09:44 | |
*** efoley_ is now known as efoley | 09:44 | |
*** vks1 has joined #openstack-nova | 09:46 | |
*** iceyao has quit IRC | 09:47 | |
*** cNilesh has quit IRC | 09:50 | |
*** ijw has joined #openstack-nova | 09:51 | |
*** ijw has quit IRC | 09:55 | |
*** CristinaPauna has quit IRC | 09:56 | |
*** vks1 has quit IRC | 10:00 | |
*** zhurong has quit IRC | 10:03 | |
*** dixiaoli has quit IRC | 10:04 | |
*** iceyao has joined #openstack-nova | 10:06 | |
openstackgerrit | Ildiko Vancsa proposed openstack/nova master: Add Cinder v3 detach to shutdown_instance https://review.openstack.org/456877 | 10:06 |
*** iceyao has quit IRC | 10:11 | |
*** Jack_Iv has quit IRC | 10:11 | |
*** markvoelker has joined #openstack-nova | 10:18 | |
*** vks1 has joined #openstack-nova | 10:18 | |
*** nicolasbock has joined #openstack-nova | 10:23 | |
*** markvoelker has quit IRC | 10:23 | |
*** yamahata has quit IRC | 10:24 | |
*** Jack_Iv has joined #openstack-nova | 10:27 | |
*** thorst has joined #openstack-nova | 10:30 | |
*** amotoki has joined #openstack-nova | 10:31 | |
*** gcb has quit IRC | 10:34 | |
*** thorst has quit IRC | 10:35 | |
*** lpetrut has joined #openstack-nova | 10:37 | |
*** zenoway has quit IRC | 10:54 | |
cdent | mriedem: you seen the collection of these yet: https://bugs.launchpad.net/nova/+bug/1682618 ? | 10:58 |
openstack | Launchpad bug 1682618 in OpenStack Compute (nova) "Official 1-800-213-2171 Pogo Games Customer Support U.S Contact Number" [Undecided,New] | 10:58 |
*** mvk has joined #openstack-nova | 10:59 | |
*** crushil has joined #openstack-nova | 11:01 | |
cdent | mriedem: took care of it: https://answers.launchpad.net/launchpad/+question/619107 | 11:02 |
*** vks1 has quit IRC | 11:02 | |
*** amotoki_ has joined #openstack-nova | 11:04 | |
*** amotoki has quit IRC | 11:07 | |
*** sdague has joined #openstack-nova | 11:15 | |
*** CristinaPauna has joined #openstack-nova | 11:16 | |
*** Jack_Iv has quit IRC | 11:21 | |
*** gongysh has quit IRC | 11:21 | |
*** Jack_Iv has joined #openstack-nova | 11:21 | |
*** yamamoto has quit IRC | 11:25 | |
*** spotz_ is now known as spotz | 11:32 | |
*** lpetrut has quit IRC | 11:35 | |
*** thorst has joined #openstack-nova | 11:42 | |
*** edmondsw has joined #openstack-nova | 11:42 | |
*** amotoki_ has quit IRC | 11:46 | |
*** amotoki has joined #openstack-nova | 11:46 | |
*** nkorabli has quit IRC | 11:48 | |
*** nkorabli has joined #openstack-nova | 11:49 | |
*** nkorabli_ has joined #openstack-nova | 11:51 | |
*** nkorabli has quit IRC | 11:51 | |
*** mnestratov has joined #openstack-nova | 11:51 | |
*** edmondsw has quit IRC | 11:54 | |
*** edmondsw has joined #openstack-nova | 11:55 | |
*** faizy_ has joined #openstack-nova | 11:56 | |
*** edmondsw has quit IRC | 11:56 | |
*** edmondsw has joined #openstack-nova | 11:56 | |
*** artom has quit IRC | 11:57 | |
*** artom has joined #openstack-nova | 11:58 | |
*** amotoki has quit IRC | 11:59 | |
*** iceyao has joined #openstack-nova | 12:00 | |
*** artom has quit IRC | 12:00 | |
*** artom has joined #openstack-nova | 12:00 | |
*** faizy has quit IRC | 12:00 | |
*** faizy has joined #openstack-nova | 12:03 | |
*** crushil has quit IRC | 12:04 | |
*** faizy_ has quit IRC | 12:06 | |
*** tbachman has joined #openstack-nova | 12:09 | |
*** zenoway has joined #openstack-nova | 12:14 | |
*** Jack_Iv has quit IRC | 12:16 | |
*** markvoelker has joined #openstack-nova | 12:20 | |
*** efoley has quit IRC | 12:21 | |
*** markvoelker has quit IRC | 12:24 | |
*** markvoelker has joined #openstack-nova | 12:25 | |
*** gouthamr has joined #openstack-nova | 12:26 | |
*** Jack_Iv has joined #openstack-nova | 12:31 | |
*** faizy_ has joined #openstack-nova | 12:31 | |
*** ildikov is now known as coffeecat | 12:32 | |
*** coffeecat is now known as coffee_cat | 12:32 | |
jaypipes | jianghuaw_, mriedem, peter-hamilton, sean-k-mooney, sfinucan: apologies, I've been ill for the last few days (still getting over it). Will try my best to get to all your requests today. I know it's past spec freeze day, but will still try to get spec reviews done. | 12:35 |
*** faizy has quit IRC | 12:35 | |
peter-hamilton | jaypipes: not a problem, hope you feel better soon | 12:35 |
sean-k-mooney | jaypipes: sorry to hear that, hope you are feeling better | 12:35 |
openstackgerrit | Ildiko Vancsa proposed openstack/nova master: Add Cinder v3 detach call to _terminate_volume_connections https://review.openstack.org/456896 | 12:36 |
*** smatzek has joined #openstack-nova | 12:36 | |
openstackgerrit | Hao Li proposed openstack/nova master: Update hypervisor host when multi-n-cpu with ironic driver do rebalancing https://review.openstack.org/456865 | 12:37 |
*** smatzek has quit IRC | 12:38 | |
*** smatzek has joined #openstack-nova | 12:38 | |
*** yamamoto has joined #openstack-nova | 12:39 | |
jianghuaw_ | jaypipes, sorry to hear that. Hope you will get recovered soon. | 12:40 |
jianghuaw_ | jaypipes, take care. | 12:40 |
openstackgerrit | Hao Li proposed openstack/nova master: Update hypervisor host when multi-n-cpu(ironic driver) do rebalancing https://review.openstack.org/456865 | 12:42 |
*** crushil has joined #openstack-nova | 12:43 | |
*** Jack_Iv has quit IRC | 12:44 | |
*** yamamoto has quit IRC | 12:44 | |
*** Jack_Iv has joined #openstack-nova | 12:46 | |
*** edmondsw has quit IRC | 12:49 | |
*** Guest52586 has quit IRC | 12:50 | |
*** cleong has joined #openstack-nova | 12:50 | |
*** lyan has joined #openstack-nova | 12:53 | |
*** faizy_ has quit IRC | 12:53 | |
efried | lifeless_ where's the "python/cpython issue tracker" | 12:57 |
*** zigo has joined #openstack-nova | 12:58 | |
*** jaypipes is now known as feeblepipes | 12:59 | |
*** xyang1 has joined #openstack-nova | 13:05 | |
mriedem | cdent: ha, no | 13:07 |
mriedem | i think i saw mention of that in the cinder channel last night though | 13:08 |
cdent | yeah, there's a few related reports | 13:08 |
cdent | at first I was "oh noes, lots of new bugs, something is very broken" | 13:08 |
*** cleong has quit IRC | 13:08 | |
*** ngupta has joined #openstack-nova | 13:09 | |
bauwser | hey folks, just a question, is it an US holiday today ? | 13:10 |
mriedem | bauwser: not for me | 13:10 |
bauwser | depending on your company, then ? | 13:10 |
mriedem | yes | 13:10 |
mriedem | easter is not a US holiday | 13:10 |
*** tbachman has quit IRC | 13:11 | |
mriedem | cdent: closing all of these bugs is going to do wonders for my LP karma | 13:11 |
*** dane-fichter has joined #openstack-nova | 13:12 | |
cdent | mriedem: my treatment of and respect for you is entirely based on your lp karma | 13:15 |
mriedem | i knew it | 13:15 |
*** mdrabe has joined #openstack-nova | 13:17 | |
*** cleong has joined #openstack-nova | 13:17 | |
*** baoli has joined #openstack-nova | 13:18 | |
*** esberglu has joined #openstack-nova | 13:19 | |
*** iceyao has quit IRC | 13:20 | |
smcginnis | mriedem: I reported it and someone with launchpad has been doing a decent job watching for them. | 13:21 |
smcginnis | It was mostly one account for awhile, but they've been switching over to a couple others. Probably as the one gets disabled. | 13:22 |
*** esberglu has quit IRC | 13:24 | |
*** vks1 has joined #openstack-nova | 13:31 | |
*** yamamoto has joined #openstack-nova | 13:31 | |
*** yamamoto has quit IRC | 13:32 | |
*** yamamoto has joined #openstack-nova | 13:32 | |
cdent | mriedem: I can't remember: Is it safe to assume that placement api will be upgraded before compute nodes? More specifically: can I make a change to the resource-tracker that requires a newer placement microversion? | 13:32 |
*** yamamoto_ has joined #openstack-nova | 13:33 | |
*** edmondsw has joined #openstack-nova | 13:35 | |
*** Dinesh_Bhor has quit IRC | 13:36 | |
*** yamamoto has quit IRC | 13:36 | |
mriedem | cdent: i thought we doc'ed something about that here? https://docs.openstack.org/developer/nova/placement.html#upgrade-notes | 13:37 |
mriedem | since we'd forget | 13:37 |
mriedem | cdent: either way you need to use microversions and be graceful about when it's not available | 13:37 |
mriedem | curtsy and whatnot | 13:37 |
cdent | meh. I was hoping to be able to delete 30 lines or so lines of code and replace it with < 10 | 13:38 |
* cdent adapts | 13:39 | |
*** vks1 has quit IRC | 13:40 | |
efried | This soft delete thing: can a guy "undelete" a soft-deleted instance? | 13:41 |
*** abalutoiu has quit IRC | 13:41 | |
*** edmondsw has quit IRC | 13:42 | |
mriedem | efried: yes | 13:43 |
mriedem | restore | 13:43 |
mriedem | efried: https://developer.openstack.org/api-ref/compute/#restore-soft-deleted-instance-restore-action | 13:43 |
efried | mriedem (melwitt) And by current design, does the quota get re-hit at that time? | 13:43 |
*** damien_r has quit IRC | 13:43 | |
* efried reading... | 13:43 | |
mriedem | efried: no | 13:43 |
mriedem | quota is decremented when we actually delete | 13:44 |
*** Jack_Iv has quit IRC | 13:44 | |
mriedem | it's one of the reasons the local delete code in the api is terrible | 13:44 |
mriedem | because we have to create a quota reservation to decrement in the api and then either roll that back or commit it in the compute | 13:44 |
efried | Ohh, but according to comments on https://review.openstack.org/#/c/386756/ it looks like we *also* decrement the quota when we do the soft delete. | 13:44 |
mriedem | if we lose the compute, well, oops! | 13:44 |
*** Jack_Iv has joined #openstack-nova | 13:45 | |
*** tesseract has quit IRC | 13:45 | |
efried | melwitt dansmith Am I misunderstanding ^^ ? | 13:46 |
*** tesseract has joined #openstack-nova | 13:46 | |
*** abalutoiu has joined #openstack-nova | 13:46 | |
mriedem | dan is out today | 13:46 |
*** Jack_Iv_ has joined #openstack-nova | 13:46 | |
mriedem | it's 6:46am for mel | 13:46 |
mriedem | and she's not a morning person i gather | 13:46 |
efried | :) | 13:47 |
*** jamesdenton has joined #openstack-nova | 13:47 | |
*** esberglu has joined #openstack-nova | 13:47 | |
*** Jack_Iv has quit IRC | 13:47 | |
*** abalutoiu_ has joined #openstack-nova | 13:47 | |
*** gongysh has joined #openstack-nova | 13:48 | |
mriedem | i left a comment in the patch | 13:49 |
*** damien_r has joined #openstack-nova | 13:51 | |
*** abalutoiu has quit IRC | 13:51 | |
mriedem | that change needs to be rebased anyway | 13:51 |
mriedem | the compute api code in there is wayyyyy old | 13:51 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Change soft delete resources to prevent DOS https://review.openstack.org/386756 | 13:51 |
efried | mriedem Cool, thanks. | 13:53 |
efried | I keep hoping that reviewing stuff I don't understand will eventually lead to some kind of enlightenment. | 13:53 |
efried | So far, it's like that flashlight in a horror movie. Dim... dimmer... gone. | 13:54 |
*** vks1 has joined #openstack-nova | 13:55 | |
*** gongysh has quit IRC | 13:55 | |
mriedem | well, that's true if you're reviewing anything related to local delete in the api or quotas | 13:55 |
mriedem | efried: you could triage some new bugs!? | 13:56 |
efried | mriedem BTW, I'm picking things to review based on the "query_high_prio_bug_fixes.py" result in the Pike Review Focus etherpad. | 13:56 |
efried | mriedem Sure, was gonna ask earlier if I could help with that. | 13:56 |
efried | But wasn't sure I'd be qualified - see flashlight analogy above. | 13:56 |
mriedem | like i came across this today https://bugs.launchpad.net/nova/+bug/1682060 | 13:56 |
openstack | Launchpad bug 1682060 in kolla-ansible "empty nova service and hypervisor list" [Undecided,New] | 13:56 |
mriedem | ^ is a thing that just landed which broke how kolla does cell setup | 13:56 |
mriedem | yay | 13:56 |
efried | mriedem Okay, looking. (And will have to find out wtf kolla is...) | 13:56 |
*** burt has joined #openstack-nova | 13:57 | |
mriedem | kolla is openstack on top of k8s | 13:57 |
mriedem | deploy openstack with k8s | 13:57 |
*** yamamoto_ has quit IRC | 13:57 | |
* cdent sympathizes and/or empthasizes with efried | 13:57 | |
efried | And k8s is...? | 13:57 |
* efried googles... | 13:57 | |
mriedem | kubernetes | 13:57 |
*** yamamoto has joined #openstack-nova | 13:57 | |
mriedem | dude, i'm still reeling from the shelve/unshelve stuff i was digging into for a few hours yesterday | 13:58 |
mriedem | i find new and terrible things every day it seems | 13:58 |
mriedem | well, "find" as in new to me, and then i find out someone else found the same thing out over a year ago | 13:58 |
mriedem | and it's not fixed yet | 13:58 |
mriedem | cdent: back to your point about dropping code, | 14:01 |
mriedem | i think we can do that after a release | 14:01 |
mriedem | cdent: so you request the new microversion and if it's there, great, else fallback, | 14:01 |
mriedem | cdent: and then in queens, we require the higher microversion - we've been signalling that in nova-status | 14:01 |
cdent | mriedem: okay, no problem, that's what I'm doing now: trying the new way. kablooie? okay do the old way | 14:01 |
mriedem | e.g. filter scheduler requires placement>=1.4 in pike, and nova-status checks that too | 14:02 |
*** larainema has quit IRC | 14:02 | |
mriedem | yeah https://github.com/openstack/nova/blob/ae637e70a72166067e09f3bbc3acb54e5d92771e/nova/cmd/status.py#L208 | 14:02 |
mriedem | cool | 14:02 |
*** larainema has joined #openstack-nova | 14:02 | |
*** yamamoto has quit IRC | 14:02 | |
mriedem | huh, never realized we were using pkg_resources there | 14:02 |
*** armax has quit IRC | 14:03 | |
cdent | mriedem: that's a relatively recent bugfix | 14:03 |
cdent | it was doing floats initially | 14:03 |
cdent | which would have broken relatively soon | 14:03 |
mriedem | i remember seeing something about that | 14:04 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Clarify wording in compute API _delete method https://review.openstack.org/456910 | 14:06 |
*** abalutoiu_ has quit IRC | 14:09 | |
*** xyang1 has quit IRC | 14:11 | |
*** xyang1 has joined #openstack-nova | 14:11 | |
sdague | mriedem: you know, shelve/unshelve being terrible might be a feature and not a bug, to convince no one to ever use it | 14:13 |
*** armax has joined #openstack-nova | 14:13 | |
mriedem | sdague: well, it's also fodder for why nova/openstack is no good | 14:15 |
mriedem | i'd rather drop things that don't work | 14:15 |
sdague | mriedem: I agree on the drop front | 14:16 |
mriedem | i think fixing the volume terminate_connection thing is simple, but that only fixes some backends, because of how cinder works | 14:16 |
mriedem | which is unfortunate, and what stalled fixing it over a year ago | 14:16 |
mriedem | because what really needs to be called is this internal remove_export function which only os-detach calls, but that's exactly what we don't want to use for shelve becaues we want the volume to stay attached | 14:17 |
sdague | mriedem: it would be nice if there was a way we could actually get some real data on whether anyone actually uses this bit | 14:17 |
smcginnis | +1 | 14:17 |
mriedem | so cinder exposes a new api, or changes terminate_connection to also call remove_export, of nova calls detach but then immediately also calls reserve_volume so the instance doesn't lose the volume, all of which aren't great | 14:17 |
mriedem | option 2 there is probably my favorite | 14:18 |
mriedem | since it abstracts everything within cinder | 14:18 |
mriedem | sdague: yeah but i don't really know how to get that | 14:18 |
sdague | mriedem: yeh, me either | 14:18 |
mriedem | polling the operators, i could ask if they disable it via policy | 14:18 |
mriedem | can't really ask users though | 14:18 |
mriedem | i also don't know if operators track incoming request metrics and could figure out how many calls are made to that api in a week/month | 14:19 |
sdague | mriedem: so... an idea I was thinking about the other day was to actually build something into nova that builds API stats in a format that's easy to annonymously dump back out | 14:19 |
cdent | turn if off and wait for someone to complain :D | 14:19 |
sdague | cdent: if the feedback cycle wasn't 1.5 years, that would work :) | 14:19 |
*** lucasxu has joined #openstack-nova | 14:20 | |
mriedem | we send an instance.shelve notification, which i'd think telemetry is consuming, but i'm not sure, | 14:20 |
cdent | we've all become quite patient for other reasons...this one more thing, no problem | 14:20 |
mriedem | but that also assumes people are using telemetry, which is a bad assumption | 14:20 |
cdent | yes it is a bad assumption | 14:20 |
*** felipemonteiro has joined #openstack-nova | 14:20 | |
cdent | it's also a bad assumption that even if they were, that they were recording that particular event | 14:20 |
cdent | s/recording/persisting/ | 14:20 |
openstackgerrit | Alex Xu proposed openstack/nova master: Use plain routes list for flavors-access endpoint instead of stevedore https://review.openstack.org/456589 | 14:20 |
openstackgerrit | Alex Xu proposed openstack/nova master: Use plain routes list for flavor endpoint instead of stevedore[1] https://review.openstack.org/456549 | 14:20 |
openstackgerrit | Alex Xu proposed openstack/nova master: Use plain routes list for '/os-aggregates' endpoint instead of stevedore https://review.openstack.org/456825 | 14:20 |
openstackgerrit | Alex Xu proposed openstack/nova master: Use plain routes list for flavors-extraspecs endpoint instead of stevedore https://review.openstack.org/456585 | 14:20 |
openstackgerrit | Alex Xu proposed openstack/nova master: Use plain routes list for '/os-keypairs' endpoint instead of stevedore https://review.openstack.org/456478 | 14:20 |
*** dimtruck is now known as zz_dimtruck | 14:21 | |
sdague | mriedem: so, just based on past interactions with folks, I honestly just wonder if marking shelve/unshelve as experimental in the API docs, say there are a number of issues with it, and move on might be best | 14:21 |
sdague | because it really really seems far on the edge | 14:21 |
sdague | and it gets a disproportunate amount of debugging time relative to use | 14:21 |
mriedem | i was hoping we could just pull from INFO logs in n-api for when a POST request to shelve comes in, but because it's an 'action' API and the bit we care about is in the body, we don't have that info... | 14:23 |
mriedem | i'm fine with noting limitations in the api, we do that for some other things as well, or should: https://review.openstack.org/#/c/452970/ | 14:24 |
mriedem | https://review.openstack.org/#/c/452970/2/api-ref/source/servers-admin-action.inc | 14:24 |
sdague | mriedem: yep, that's one of those issues. I do think it's elsewhere though. Anyway, need to afk for a bit. | 14:24 |
feeblepipes | mriedem: happen to know Jan Gutter's IRC nick? | 14:26 |
mriedem | jangutter | 14:26 |
feeblepipes | mriedem: k, thx. | 14:26 |
mriedem | sdague: on "an idea I was thinking about the other day was to actually build something into nova that builds API stats in a format that's easy to annonymously dump back out" - thinking about that, i think we already have that, it's instance actions | 14:26 |
feeblepipes | mriedem: pls see my -1 and comments from that netronome spec. I'm cool with it, but have concerns about the PCI device allocation code proposed for neutronv2/api.py module. | 14:27 |
mriedem | so compute_shelve_instance is an action event | 14:27 |
*** jangutter has joined #openstack-nova | 14:30 | |
mriedem | feeblepipes: they just mean this i think https://review.openstack.org/#/c/456481/1/nova/network/neutronv2/api.py | 14:31 |
mriedem | in addition to handling direct-physical, handle this other vnic type | 14:31 |
mriedem | that would need to be cleaned up a bit and non-vendorized, | 14:31 |
mriedem | but i think the spec notes that | 14:31 |
*** annegentle has joined #openstack-nova | 14:35 | |
feeblepipes | mriedem: yeah, not really a fan of that approach as it embeds way too much vendor/product specific stuff into the model. It shouldn't matter what vendor/product_id is used underneath the covers. all that should matter is what *functionality* the end user can derive from the particular vNIC. so the things that should be passed from the neutronv2 api should really only be feature requests/capabilities, not vendor/product information. | 14:35 |
feeblepipes | mriedem: make sense? | 14:35 |
cdent | feeblepipes++ | 14:36 |
*** Kevin_Zheng has quit IRC | 14:37 | |
mriedem | feeblepipes: so you're also saying you're not a fan of the existing direct-physical thing we already have in that code? | 14:37 |
mriedem | which i'm pretty sure doesn't work... | 14:37 |
*** Kevin_Zheng has joined #openstack-nova | 14:37 | |
mriedem | because a week after it landed, nikola created 2 bugs about it not working | 14:37 |
feeblepipes | mriedem: correct, I wasn't a fan of it when it entered the code and I'm still not a fan of it. you can check the review on that if you'd like :) | 14:38 |
mriedem | either way i'll totally defer here | 14:38 |
mriedem | i don't really know what the alternative implementation is, they are going to need some help there | 14:38 |
feeblepipes | mriedem: again, I'm not opposed to the netronome stuff at all. just have reservations (pun intended) about this particular part of th eproposal. | 14:38 |
*** coreywright has quit IRC | 14:39 | |
feeblepipes | mriedem: yeah, I'll wait for Jan to answer and think about alternatives to recommedn. | 14:39 |
feeblepipes | mriedem: also note that the direct-physical thing is *only* passing through a network tag (the physnet tag) from Neutron's port profile. It isn't passing a vendor-specific piece of information. | 14:40 |
feeblepipes | mriedem: or perhaps you were referring to a different part of the codebase w.r.t. the direct-physical thing? | 14:41 |
mriedem | no i was just pointing at that thing | 14:42 |
mriedem | this part dev_type = pci_request.DEVICE_TYPE_FOR_VNIC_TYPE.get(vnic_type) | 14:43 |
feeblepipes | right | 14:43 |
mriedem | pci_request.DEVICE_TYPE_FOR_VNIC_TYPE is only mapped to direct-physical | 14:43 |
feeblepipes | yes, understood. | 14:43 |
*** zz_dimtruck is now known as dimtruck | 14:43 | |
jangutter | Oh hey, thanks for taking the time to review! | 14:44 |
feeblepipes | jangutter: hi :) | 14:44 |
feeblepipes | jangutter: sorry for the delay. been quite ill last few days :( | 14:44 |
*** mtanino has joined #openstack-nova | 14:44 | |
jangutter | feeblepipes: not a problem, it took waaay too long on our side to jump the necessary hurdles. No-one to blame but me. | 14:45 |
jangutter | feeblepipes: yeah, regarding that allocation, the idea is not to put in a new place for it, but to allow port profiles that are "non-SR-IOV" to also request pci devices. | 14:46 |
jangutter | feeblepipes: and you're absolutely right, it is the crux of the spec. I had looked at other places, but most of them would happen after scheduling had already occurred. | 14:49 |
openstackgerrit | Ildiko Vancsa proposed openstack/nova master: Add Cinder v3 detach call to _terminate_volume_connections https://review.openstack.org/456896 | 14:49 |
feeblepipes | jangutter: see what I wrote to mriedem above about being pretty firm about keeping the information that we pass about the vNIC as vendor-agnostic as possible (and focusing instead on the *features* or functionality that the underlying technology supports) | 14:49 |
*** cdent has quit IRC | 14:49 | |
openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Idempotent PUT /resource_classes/{name} https://review.openstack.org/448791 | 14:49 |
openstackgerrit | Chris Dent proposed openstack/nova master: Update resource tracker to PUT custom resource classes https://review.openstack.org/456915 | 14:49 |
openstackgerrit | Jan Gutter proposed openstack/nova-specs master: Spec to enable Netronome SmartNIC support https://review.openstack.org/456386 | 14:50 |
*** sdague has quit IRC | 14:50 | |
feeblepipes | jangutter: what I'd like to see is a description of those features of the virtual switch technology that we could use instead of specific PCI vendor_id/product_id information. | 14:50 |
*** lpetrut has joined #openstack-nova | 14:51 | |
jangutter | Yep, so after a discussion yesterday with sean-k-mooney, it looks like it should be possible to push that check out to the OS-VIF side. | 14:52 |
feeblepipes | jangutter: another way to think about things... what is the difference between network_model.VNIC_TYPE_AGILIO_PASSTHROUGH and network_model.VNIC_TYPE_DIRECT or, frankly, network_model.VNIC_TYPE_DIRECT_PHYSICAL? | 14:52 |
*** coreywright has joined #openstack-nova | 14:52 | |
*** scottda has joined #openstack-nova | 14:52 | |
jangutter | feeblepipes: Those two AGILIO VNIC types were added to in an effort not to re-use internal OpenStack code: they should go completely away. | 14:54 |
*** dane-fichter has quit IRC | 14:55 | |
jangutter | feeblepipes: The idea is to have all the necessary whitelist info sent down via the os-vif port profile, so that the API will do matching. | 14:56 |
feeblepipes | jangutter: ok. that's cool with me. How exactly, however, do you plan to keep vendor_id/product_id information out of the os-vif modeling? In other words, what I'm looking for is an indication that you've thought of how to communicate the *functionality* that Agilio SmartNICs provide to the end user instead of passing a PCI vendor_id/product_id :) | 14:57 |
jangutter | feeblepipes: That's done in a couple of ways: a deployer runs a script that generates a nova.conf fragment, this adds the VF's to the compute node with in a full pci-whitelist format. | 14:58 |
feeblepipes | jangutter: sure, understood. that merely lets nova-compute "use" the Agilio NICs for tenants. | 14:59 |
*** yamamoto has joined #openstack-nova | 14:59 | |
jangutter | feeblepipes: Secondly, the OS-VIF object for the external plugin gets a set of fields. Likely this would be vendor/productids, we'll definitely have to coordinate with Mellanox on that. | 15:00 |
*** gjayavelu has joined #openstack-nova | 15:00 | |
jangutter | feeblepipes: so when the API code at that place hits, the port profile is available, and the pci devices has been registered. | 15:00 |
feeblepipes | jangutter: right, so that's the thing I'd be -1 on :) we should definitely work with Moshe and others from Mellanox to ensure vendor_id/product_id information isn't needed to be passed via os-vif models. | 15:01 |
feeblepipes | jangutter: what I'm looking for is an object model that says "hey, I need a VIF plug that supports feature X, Y, and Z", not an object model that says "hey, I need a VIF plug that is specific to Mellanox Indigo NPS" | 15:03 |
feeblepipes | jangutter: that make sense? | 15:03 |
jangutter | feeblepipes: hmmm... that's interesting... you just made me think quite a lot... | 15:03 |
jangutter | feeblepipes: that would basically mean a "capability set" sort-of? | 15:04 |
feeblepipes | jangutter: BINGO :) | 15:04 |
feeblepipes | jangutter: ala the traits/capabilities that have begun to emerge here: https://review.openstack.org/#/c/448283/4 | 15:05 |
jangutter | feeblepipes: I can understand the appeal, but WOW, the parameter space is _big_. | 15:06 |
feeblepipes | jangutter: so, as an example, the Neutron port profile and os-vif object model would, say, have a list of these acceleration traits: https://review.openstack.org/#/c/448283/4/os_traits/hw/nic/accel.py | 15:06 |
*** ijw has joined #openstack-nova | 15:06 | |
feeblepipes | that would be required to match to an available PCI device on the target host. | 15:06 |
*** zenoway has quit IRC | 15:06 | |
*** zenoway has joined #openstack-nova | 15:07 | |
feeblepipes | jangutter: and Netronome and Mellanox would add a set of traits/features that their NICs supported to https://review.openstack.org/#/c/448283/4/os_traits/hw/nic/sriov.py | 15:07 |
jangutter | feeblepipes: I'm just worried that the acceleration traits would not dovetail much. For example, our NIC's got an insane amount of flexibility. The same HW can support many different features. | 15:07 |
jangutter | feeblepipes: our internal lingo for the firmware running on the chip is "application" - the image I get when l think of negotiating capabilities is some insane duel. | 15:09 |
feeblepipes | jangutter: an end user, though, only wants to launch a VM on a host that has a NIC with certain features. an end user doesn't say "hey, I want a VM with a Netronome Agilio SmartNIC" (because, frankly, the end user doesn't *know* that the underlying hardware has Agilio NICs). Instead, the end user says "hey, give me a VM that has access to an SR-IOV virtual function that has a particular set of features" | 15:10 |
jangutter | feeblepipes: Ah, I've been confusing operators with end-users. | 15:11 |
feeblepipes | jangutter: no worries. it's a common NFV problem :) in NFV land, there is no end user. The only user of the system is a privileged administrator user embodied in the MANO/VNFM system. | 15:11 |
feeblepipes | jangutter: but Nova is for end users -- the people/systems that launch VMs. So, we're constantly trying to shape the new NFV-specific features into a world that "normal users" can understand and utilize via the Nova APIs. | 15:12 |
jangutter | feeblepipes: OK, let's take the current flat/vlan SR-IOV mechanism as an example. An end user creates a port on a particular network and specs "it should be direct". (I think "direct" tends to go with VF's and "direct-physical" tend to go with PFs). | 15:12 |
feeblepipes | ok, continue | 15:13 |
jangutter | feeblepipes: if there are a mix of NICs then some instances would pop up with one type of driver required, and others would pop up with another, correct? | 15:13 |
*** zenoway has quit IRC | 15:14 | |
*** zenoway has joined #openstack-nova | 15:14 | |
jangutter | feeblepipes: It should be possible to do some flavor/image filtering or something, to match the flavors with the drivers they support? | 15:15 |
feeblepipes | jangutter: if there are a mix of NICs, they should be differentiated by their feature set, not by their vendor_id/product_id. | 15:16 |
*** jamesdenton has quit IRC | 15:17 | |
feeblepipes | jangutter: so, for example, if a user launches a VM with nova boot --net port_id=<PORT_UUID> and that port has binding information that constructs a PCI request, the stuff in that PCI request should be a feature set, not a vendor_id/product_id. That's what I'm getting at :) | 15:17 |
*** burgerk has joined #openstack-nova | 15:17 | |
jangutter | feeblepipes: with iommu passthrough, the instances get a raw PCI device, with the vendor/product exposed. I've been looking at that tuple as "a feature" :-p | 15:17 |
feeblepipes | jangutter: and that feature set is what should be used by the scheduler and the nova-compute pci device manager to match a particular NIC among many on the host. | 15:18 |
feeblepipes | jangutter: what about having "iommu_passthrough" as the feature, then? | 15:18 |
jangutter | feeblepipes: well, iommu_passthrough won't tell us anything of the device type that's going to appear inside the VM. | 15:19 |
feeblepipes | jangutter: would an end-user say "I want a vNIC that is associated to a VF that has IOMMU passthrough features" or would the end user say "I want a vNIC that is associated with a VF that is a Netronome Agilio SmartNIC PF"? | 15:19 |
*** Qiming has quit IRC | 15:22 | |
jangutter | feeblepipes: my view of one usecase is this: an end-user says "I want a vNIC, associated with a VF, passed through via IOMMU, with <insert-hw-driver-here> PCI driver, additionally pin everything for me optimally, pretty please!" | 15:23 |
feeblepipes | :) | 15:23 |
feeblepipes | jangutter: ok, cool. we're getting somewhere here :) | 15:23 |
jangutter | feeblepipes: with DPDK applications being specifically compiled for specific PCI drivers, it's sometimes a _must_. | 15:23 |
*** crushil has quit IRC | 15:24 | |
feeblepipes | jangutter: the <insert-hw-driver-here> goes in the *image metadata*, though. | 15:24 |
*** crushil has joined #openstack-nova | 15:24 | |
feeblepipes | jangutter: because the image metadata is what describes the workload to be launched. that image metadata can describe the HW driver's installed on the image, and that information can/should be used by os-vif in setting up the VIF plug on the host. | 15:25 |
jangutter | feeblepipes: yeah, but that leads to the other bit of fun. Give me one vNIC on control, standard TAP, and vNIC 2 for my DPDK NFV application. | 15:25 |
feeblepipes | jangutter: so I think we're *almost* there. | 15:25 |
*** crushil has quit IRC | 15:25 | |
*** crushil has joined #openstack-nova | 15:25 | |
jangutter | feeblepipes: is it possible to spec stuff that fine-grained? | 15:25 |
*** crushil has quit IRC | 15:25 | |
feeblepipes | jangutter: ah, indeed. so perhaps that information (hw driver) does need to be in the port binding profile. | 15:26 |
feeblepipes | jangutter: yes, it is possible. via the port profile information. | 15:26 |
*** crushil has joined #openstack-nova | 15:26 | |
feeblepipes | jangutter: I still don't think we need to have vendor_id/product_id in there, though. :) | 15:26 |
feeblepipes | jangutter: in any case... I think this is also something we can hash out in the code rather than the spec. | 15:26 |
jangutter | feeblepipes: I am definitely not averse to having a "driver list". That's actually a pretty good idea from a user's perspective. | 15:27 |
feeblepipes | jangutter: I'm willing to +2 the spec on condition that you just place a note in the spec that we've discussed on IRC how to make the information passed between Neutron and Nova via os-vif as generic as possible. | 15:27 |
*** tomhambleton_ has quit IRC | 15:27 | |
feeblepipes | jangutter: if you can do that quick, I'll +2 it and mriedem can as well. | 15:27 |
*** tomhambleton_ has joined #openstack-nova | 15:27 | |
*** Qiming has joined #openstack-nova | 15:30 | |
mriedem | sfinucan: how do i reference another link within the same docs using rst again? | 15:30 |
mriedem | like i want to link to a section in the nova-manage man page from the nova-cells man page | 15:31 |
jangutter | I shall definitely do so: to summarize: It is undesirable to send the VF selection parameters to Nova as PCI vendor/product ID, further work should be done to develop a capability or driver list that would abstract this away from users. | 15:31 |
mriedem | oh maybe :ref: | 15:32 |
efried | mriedem Looks like that kolla thing got sorted. Can you throw me a link to whatever dashboard/view I should be using, so I can maybe pick off some lhf? | 15:32 |
mriedem | efried: not sorted yet, docs change coming soon | 15:32 |
efried | I meant triaged | 15:32 |
efried | That's the goal of the current sweep, right? | 15:33 |
efried | Not to resolve, but to get 'em out of 'New'? | 15:33 |
mriedem | efried: yeah | 15:33 |
mriedem | like 'unexpected api error' is usually always invalid | 15:33 |
mriedem | and a misconfiguration | 15:33 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Add release note and update cell install guide for multi-cell limitations https://review.openstack.org/456923 | 15:34 |
mriedem | melwitt: ^ | 15:34 |
*** imacdonn has joined #openstack-nova | 15:34 | |
*** edmondsw has joined #openstack-nova | 15:35 | |
mriedem | EmilienM: owalsh: ^ | 15:35 |
*** mdnadeem has quit IRC | 15:36 | |
*** rcernin has quit IRC | 15:37 | |
*** mtanino has quit IRC | 15:38 | |
efried | This thing about having to run a new discovery command to make stuff work... Could it not just be made to work out of the box if you only have one cell? | 15:38 |
*** john5223 has joined #openstack-nova | 15:44 | |
*** ijw has quit IRC | 15:44 | |
*** nkorabli_ has quit IRC | 15:45 | |
*** nkorabli has joined #openstack-nova | 15:45 | |
*** burt has quit IRC | 15:46 | |
efried | [irony: when --help doesn't automatically page, and you have to say --help|less] | 15:47 |
mriedem | efried: we have a config option if you want things discovered automatically, | 15:50 |
mriedem | but it's a bit heavy for large deployments | 15:50 |
mriedem | dan has a series up to improve that | 15:50 |
jangutter | Is there a way to refer to today's IRC chat in a spec? | 15:50 |
efried | mriedem Okay, neat. | 15:50 |
mriedem | jangutter: yeah, esc | 15:50 |
mriedem | *Sec | 15:50 |
efried | http://eavesdrop.openstack.org/irclogs/%23openstack-nova/ | 15:50 |
efried | jangutter ^^ | 15:50 |
mriedem | efried: https://blueprints.launchpad.net/nova/+spec/discover-hosts-faster | 15:50 |
mriedem | jangutter: http://eavesdrop.openstack.org/irclogs/%23openstack-nova/%23openstack-nova.2017-04-14.log.html#t2017-04-14T14:44:06 | 15:51 |
jangutter | Whatever you write is now memorialized forever. Or until cats reclaim their rightful place on top of the food chain. | 15:52 |
*** zenoway has quit IRC | 15:52 | |
*** zenoway has joined #openstack-nova | 15:53 | |
*** damien_r has quit IRC | 15:54 | |
mriedem | our cat can easily be defeated with a bag of chips | 15:54 |
*** phuongnh has joined #openstack-nova | 15:54 | |
openstackgerrit | Jan Gutter proposed openstack/nova-specs master: Spec to enable Netronome SmartNIC support https://review.openstack.org/456386 | 15:54 |
*** trinaths has joined #openstack-nova | 15:56 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Tell people that the nova-cells man page is for cells v1 https://review.openstack.org/456925 | 15:57 |
jangutter | Another newbie question. I've replied to a bunch of comments on an older review. They're in "draft" in gerrit now, am I too late? | 15:57 |
mriedem | jangutter: no, you can still publish those | 15:57 |
efried | jangutter No - go to that patch set and they'll show up. | 15:57 |
*** zenoway has quit IRC | 15:57 | |
*** Jack_Iv_ has quit IRC | 15:57 | |
*** psachin has quit IRC | 15:57 | |
efried | Still not entirely understanding why a compute node can't automatically 'discover' itself into a cell if there's only one cell... | 15:58 |
*** edmondsw has quit IRC | 15:58 | |
mriedem | efried: it would require an upcall from the compute to the api layer, which we don't want | 15:58 |
mriedem | upcalls are the devil | 15:58 |
mriedem | s/the/a/ | 15:58 |
openstackgerrit | Ildiko Vancsa proposed openstack/nova master: Add Cinder v3 detach call to _terminate_volume_connections https://review.openstack.org/456896 | 15:58 |
mriedem | efried: also, that would require either an upcall or putting the nova api db credentials in the compute config | 15:59 |
mriedem | another thing we don't want | 15:59 |
efried | mriedem And all of that is different from how a compute node registered itself B.C. (before cells)? | 16:00 |
mriedem | yes | 16:00 |
mriedem | in the before times, | 16:00 |
mriedem | there was no api db | 16:01 |
mriedem | there were no host mappings in the api db | 16:01 |
*** mlavalle has joined #openstack-nova | 16:01 | |
mriedem | when you start nova-compute, it creates it's own compute_nodes table record for that host (happens in ResourceTracker.update_available_resource eventually) | 16:01 |
mriedem | when you start the service it also creates a record in the services table for the nova-compute binary | 16:01 |
mriedem | those all get mangled up in the api somehow | 16:02 |
mriedem | feeblepipes loves it | 16:02 |
*** mdrabe has quit IRC | 16:02 | |
mriedem | but with no cells, i.e. regular old stock nova, there was just a single db and everything was in there | 16:02 |
mriedem | efried: are you going to be in boston? | 16:03 |
efried | mriedem Yessir. | 16:03 |
mriedem | then put https://www.openstack.org/summit/boston-2017/summit-schedule/events/18727/ on your schedule | 16:03 |
*** ralonsoh has quit IRC | 16:04 | |
efried | mriedem Roger wilco, and thanks. | 16:04 |
*** gyee has joined #openstack-nova | 16:05 | |
*** hongbin has joined #openstack-nova | 16:07 | |
*** mdrabe has joined #openstack-nova | 16:08 | |
*** dtp has joined #openstack-nova | 16:09 | |
*** yamamoto has quit IRC | 16:10 | |
*** larainema has quit IRC | 16:12 | |
EmilienM | mriedem: ack | 16:12 |
*** edmondsw has joined #openstack-nova | 16:14 | |
*** mwhahaha has quit IRC | 16:14 | |
*** mwhahaha has joined #openstack-nova | 16:15 | |
*** Apoorva has joined #openstack-nova | 16:16 | |
*** lucasxu has quit IRC | 16:19 | |
*** edmondsw has quit IRC | 16:21 | |
*** armax has quit IRC | 16:21 | |
*** armax has joined #openstack-nova | 16:21 | |
*** nkorabli_ has joined #openstack-nova | 16:23 | |
*** nkorabli has quit IRC | 16:23 | |
*** vks1 has quit IRC | 16:24 | |
*** lpetrut has quit IRC | 16:30 | |
*** amotoki has joined #openstack-nova | 16:31 | |
feeblepipes | jangutter: yes, your summary of the requested changes above is good. | 16:33 |
*** ssurana has joined #openstack-nova | 16:34 | |
jangutter | feeblepipes: if all else fails there's also that list of emulated vNICs that qemu uses: confusingly it's called "hw_vif_model" | 16:36 |
feeblepipes | jangutter: yeah, I know :( | 16:36 |
feeblepipes | peter-hamilton: approved. | 16:43 |
peter-hamilton | feeblepipes: awesome, thanks! | 16:43 |
feeblepipes | peter-hamilton: np. sorry again for the delay. | 16:43 |
peter-hamilton | feeblepipes: no problem, i hope you recover quickly | 16:44 |
feeblepipes | peter-hamilton: about 50% today. which is better than yesterday's 10% :) | 16:44 |
peter-hamilton | feeblepipes: yep, i've been there | 16:44 |
peter-hamilton | feeblepipes: thanks for the comment on the history btw | 16:45 |
peter-hamilton | feeblepipes: i find it helps to summarize decisions and the back-and-forth over the months | 16:45 |
*** catintheroof has joined #openstack-nova | 16:45 | |
feeblepipes | yeah, it actually makes it a bit easier for those of us kinda coming into a spec late in its lifetime. | 16:45 |
peter-hamilton | feeblepipes: no one else seems to use it for much | 16:45 |
feeblepipes | ++ | 16:45 |
*** catintheroof has quit IRC | 16:46 | |
*** crushil has quit IRC | 16:47 | |
*** catintheroof has joined #openstack-nova | 16:47 | |
*** Kevin_Zheng has quit IRC | 16:47 | |
*** robcresswell has quit IRC | 16:48 | |
*** robcresswell has joined #openstack-nova | 16:48 | |
*** ngupta has quit IRC | 16:49 | |
jangutter | Any more actions on https://review.openstack.org/456386 I need to take? | 16:52 |
*** edmondsw has joined #openstack-nova | 16:54 | |
feeblepipes | jangutter: did you push a new revision with wording around the "genericness" of parameters passed via os-vif/Neutron port binding profile? | 16:56 |
jangutter | feeblepipes: yeah, that's been updated, plus a link to the chat. | 16:57 |
*** hongbin has quit IRC | 16:58 | |
*** fragatina has joined #openstack-nova | 16:58 | |
feeblepipes | jangutter: awesome. reviewing again. | 16:58 |
*** fragatina has quit IRC | 16:59 | |
*** crushil has joined #openstack-nova | 16:59 | |
*** hoonetorg has quit IRC | 16:59 | |
*** fragatina has joined #openstack-nova | 16:59 | |
*** burgerk has quit IRC | 17:04 | |
*** ngupta has joined #openstack-nova | 17:06 | |
*** artom has quit IRC | 17:06 | |
*** lucasxu has joined #openstack-nova | 17:06 | |
*** artom has joined #openstack-nova | 17:06 | |
feeblepipes | jangutter: k, +2 from me. mriedem, feel free to give 456386 another spin. | 17:06 |
jangutter | feeblepipes: thanks! | 17:07 |
feeblepipes | jangutter: np | 17:07 |
feeblepipes | dansmith: did you see johnthetubaguy's request for your review on https://review.openstack.org/#/c/407237/? | 17:08 |
*** artom has quit IRC | 17:08 | |
*** artom has joined #openstack-nova | 17:08 | |
*** phuongnh has quit IRC | 17:09 | |
*** esberglu has quit IRC | 17:14 | |
*** sbezverk has joined #openstack-nova | 17:17 | |
*** lucasxu has quit IRC | 17:20 | |
*** trinaths has quit IRC | 17:21 | |
*** ngupta has quit IRC | 17:22 | |
*** esberglu has joined #openstack-nova | 17:25 | |
*** lucasxu has joined #openstack-nova | 17:26 | |
*** ngupta has joined #openstack-nova | 17:27 | |
*** damien_r has joined #openstack-nova | 17:28 | |
*** ijw has joined #openstack-nova | 17:31 | |
*** armax_ has joined #openstack-nova | 17:31 | |
*** tesseract has quit IRC | 17:33 | |
*** efried has quit IRC | 17:33 | |
*** armax has quit IRC | 17:35 | |
*** armax_ is now known as armax | 17:35 | |
owalsh | mriedem: yea, the related change in tripleo was to handle this | 17:37 |
*** damien_r has left #openstack-nova | 17:39 | |
*** yamahata has joined #openstack-nova | 17:41 | |
*** nkorabli_ has quit IRC | 17:41 | |
*** ngupta has quit IRC | 17:42 | |
*** nkorabli has joined #openstack-nova | 17:43 | |
*** efried has joined #openstack-nova | 17:45 | |
*** Jack_Iv has joined #openstack-nova | 17:46 | |
*** ngupta has joined #openstack-nova | 17:49 | |
*** ijw has quit IRC | 17:57 | |
*** sdague has joined #openstack-nova | 18:02 | |
*** baoli has quit IRC | 18:04 | |
*** baoli has joined #openstack-nova | 18:05 | |
*** oanson has quit IRC | 18:05 | |
*** oanson has joined #openstack-nova | 18:07 | |
mriedem | feeblepipes: jangutter: done | 18:08 |
feeblepipes | mriedem: thanks, Matt. and apologies for the delay on these after spec freeze. | 18:08 |
mriedem | p | 18:08 |
mriedem | *np | 18:08 |
mriedem | you get an exception because jay was sick | 18:08 |
mriedem | was/is | 18:08 |
feeblepipes | mriedem: I'm trying to get the GPU spec finalized and the one about listing migrations. | 18:08 |
mriedem | feeblepipes: i have not looked at the gpu spec and probably won't | 18:09 |
mriedem | i plan on looking at the listing migration types one though | 18:09 |
feeblepipes | mriedem: k | 18:09 |
mriedem | because that api annoys me | 18:09 |
feeblepipes | mriedem: I have a number of concerns on it, yes. | 18:09 |
*** burt has joined #openstack-nova | 18:09 | |
mriedem | lack of gpu complexity in nova doesn't annoy me | 18:09 |
jangutter | feeblepipes, mriedem, you have my eternal gratitude! | 18:09 |
mriedem | feeblepipes: e.g. https://bugs.launchpad.net/nova/+bug/1668747 | 18:10 |
openstack | Launchpad bug 1668747 in OpenStack Compute (nova) "api-ref: os-migrations is missing response details and information about v2.23" [Medium,Confirmed] | 18:10 |
feeblepipes | jangutter: kein Problem! | 18:10 |
feeblepipes | jangutter: or maybe "geen probleem!"? :) | 18:11 |
jangutter | It's the second one! | 18:11 |
feeblepipes | aha! | 18:11 |
*** harlowja has joined #openstack-nova | 18:12 | |
jangutter | Fair warning, don't let us Afrikaans people start teaching you phrases. You might end up learning the ultimate language to insult someone in. | 18:12 |
feeblepipes | :) | 18:13 |
*** cdent has joined #openstack-nova | 18:13 | |
*** armax has quit IRC | 18:18 | |
*** djohnsto has quit IRC | 18:20 | |
*** MasterOfBugs has joined #openstack-nova | 18:20 | |
*** jangutter has left #openstack-nova | 18:23 | |
openstackgerrit | Chris Dent proposed openstack/nova master: Update resource tracker to PUT custom resource classes https://review.openstack.org/456915 | 18:23 |
*** cdent has quit IRC | 18:24 | |
*** krtaylor has quit IRC | 18:30 | |
*** fragatina has quit IRC | 18:31 | |
mriedem | feeblepipes: while you're close to passing out or vomiting, let me throw this on you https://review.openstack.org/#/c/453272/ | 18:32 |
*** mnestratov has quit IRC | 18:33 | |
feeblepipes | mriedem: what's the difference between the instance action of evacuate vs the instance action of rebuild? | 18:35 |
mriedem | with evacuate the source compute host is dead | 18:36 |
mriedem | with rebuild it's not | 18:36 |
feeblepipes | mriedem: how would the user know which is which? | 18:36 |
mriedem | users don't evacuate | 18:36 |
mriedem | since it's a migratoin | 18:36 |
mriedem | *migratoin | 18:36 |
mriedem | damn | 18:36 |
mriedem | migration | 18:36 |
mriedem | admins evacuate | 18:36 |
feeblepipes | mriedem: and who rebuilds? | 18:36 |
mriedem | users that want the same attached volumes but a new image | 18:36 |
mriedem | you can specify a new image with rebuild | 18:37 |
feeblepipes | le sigh... ok. | 18:37 |
mriedem | basically it's a server create to over | 18:37 |
mriedem | same host though i believe | 18:37 |
feeblepipes | do-over you mean? | 18:37 |
mriedem | the rebuild happens on the same host that the instance already lives on | 18:37 |
mriedem | it would have to for the networking and volumes | 18:37 |
mriedem | now rebuild + volume-backed root disk.... | 18:38 |
mriedem | i'd have to investigate | 18:38 |
mriedem | we track evacuations with the migrations table since it's a type of move | 18:38 |
mriedem | that was part of dan's robustify evacuate changes in i think mitaka | 18:38 |
mriedem | so that when the old host comes back up it doesn't delete all of your instances that are now running somewhere else | 18:39 |
mriedem | because that was a real thing | 18:39 |
mriedem | and could wipe out your entire vcenter cluster | 18:39 |
mriedem | you're welcome | 18:39 |
mriedem | i.e. we check the migrations table to see if an instance was evacuated on startup of the old host | 18:39 |
mriedem | https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L635 | 18:40 |
feeblepipes | mriedem: +W'd that spec. working on mgagne's one now. | 18:42 |
mriedem | i guess i should read the list migration types one | 18:44 |
mriedem | sfinucan: the spec for this is approved now so you can drop the procedural -2 https://review.openstack.org/#/c/430608/ | 18:45 |
*** armax has joined #openstack-nova | 18:48 | |
*** claudiub|2 has quit IRC | 18:50 | |
*** krtaylor has joined #openstack-nova | 18:58 | |
*** baoli has quit IRC | 18:59 | |
*** baoli has joined #openstack-nova | 18:59 | |
openstackgerrit | Dan Peschman proposed openstack/nova master: Option to always create network metadata information as static https://review.openstack.org/336293 | 19:00 |
*** jbryce has quit IRC | 19:02 | |
*** jbryce has joined #openstack-nova | 19:02 | |
*** Jack_Iv has quit IRC | 19:10 | |
*** Jack_Iv has joined #openstack-nova | 19:15 | |
*** cleong has quit IRC | 19:18 | |
*** tbachman has joined #openstack-nova | 19:19 | |
*** rcernin has joined #openstack-nova | 19:20 | |
*** jamesdenton has joined #openstack-nova | 19:20 | |
*** fragatina has joined #openstack-nova | 19:22 | |
*** gyee has quit IRC | 19:22 | |
*** markvoelker has quit IRC | 19:22 | |
*** Jack_Iv has quit IRC | 19:27 | |
*** markvoelker has joined #openstack-nova | 19:30 | |
*** liusheng has quit IRC | 19:30 | |
*** chenying has quit IRC | 19:31 | |
*** armax_ has joined #openstack-nova | 19:32 | |
*** prateek has joined #openstack-nova | 19:33 | |
*** armax has quit IRC | 19:33 | |
*** armax_ is now known as armax | 19:33 | |
*** tbachman has quit IRC | 19:39 | |
*** edmondsw has quit IRC | 19:45 | |
*** edmondsw has joined #openstack-nova | 19:46 | |
*** stvnoyes has quit IRC | 19:46 | |
mriedem | feeblepipes: i left some comments in https://review.openstack.org/#/c/407237/ too | 19:49 |
mriedem | feeblepipes: i'm not sure why the various task state transitions were left in there, unless that was to show what in-progress meant for each type of migration? | 19:49 |
feeblepipes | yeah, maybe. | 19:49 |
mriedem | those don't line up with what the supported migration statuses are though | 19:49 |
mriedem | also, i've always found it odd that we filter what we return to just in-progress migrations | 19:49 |
mriedem | that is an artifact of when os-migrations was added, and i think it was only there for the cancel action | 19:50 |
mriedem | so those were tightly coupled | 19:50 |
*** edmondsw has quit IRC | 19:50 | |
mriedem | i don't think we should restrict the returned set to in-progress migrations by default, but rather allow a status filter parameter | 19:50 |
*** Jack_Iv has joined #openstack-nova | 19:51 | |
*** chenying has joined #openstack-nova | 19:53 | |
*** liusheng has joined #openstack-nova | 19:53 | |
mriedem | i might blast the ML on that one unless there is a good reason i'm missing | 19:54 |
mriedem | i'd also like operator feedback | 19:54 |
feeblepipes | cool with me. | 19:54 |
feeblepipes | almost done with the VGPU spec. | 19:55 |
sdague | mriedem: hmmm... that does make me wonder if I could pillage instance actions in a reasonable way then for statistics. Perhaps I'll ponder that next week | 20:01 |
mriedem | and that reminds me to ask about instance actions for shelve in the ops list | 20:01 |
*** catintheroof has quit IRC | 20:02 | |
sdague | I think it would mostly be useful if we put something into nova-manage that would build statistics based on it for dump | 20:05 |
*** stvnoyes has joined #openstack-nova | 20:06 | |
*** gyee has joined #openstack-nova | 20:08 | |
*** yamamoto has joined #openstack-nova | 20:08 | |
*** annegentle has quit IRC | 20:11 | |
mriedem | yeah | 20:11 |
mriedem | also, crap, but we return db primary keys for the migrations api | 20:11 |
feeblepipes | sfinucan: -1 from me for a few remaining niggles on https://review.openstack.org/#/c/450122/, but it's pretty close. | 20:11 |
mriedem | so when you want to cancel an in-progress live migration out of the api, you pass in the id but with multi-cell we won't know which cell it's in | 20:12 |
feeblepipes | mriedem: yeah... that needs to die in a fire. | 20:12 |
mriedem | feeblepipes: well that's dying for services and compute nodes in the api, but i missed migrations | 20:12 |
feeblepipes | mriedem: also, since migrations list runs in all cells, you get duplicate migration IDs. fun! | 20:12 |
mriedem | feeblepipes: true story | 20:12 |
mriedem | i don't think dansmith has fixed that one yet | 20:12 |
mriedem | which is why we didn't think about it earlier | 20:12 |
feeblepipes | mriedem: it's not terribly important really... now that we removed the top-level os-migrations/ REST resource, it's not like there's a global key. | 20:13 |
feeblepipes | mriedem: meaning it's always under servers/.../os-migrations | 20:13 |
*** yamamoto has quit IRC | 20:13 | |
feeblepipes | mriedem: so, no harm no foul. the primary key is within the server's cell. | 20:13 |
feeblepipes | mriedem: but still, annoying and should die in a fire. | 20:14 |
mriedem | feeblepipes: oh you're right yeah, we'll lookup the instance via uuid, and when we do that we target the context to the cell that the instance is in | 20:18 |
mriedem | so looking up the migration by id after that is ok | 20:19 |
mriedem | whew | 20:19 |
feeblepipes | ya | 20:19 |
*** abalutoiu has joined #openstack-nova | 20:19 | |
feeblepipes | #success mriedem said I was right. | 20:19 |
openstackstatus | feeblepipes: Added success to Success page | 20:19 |
*** smatzek has quit IRC | 20:19 | |
mriedem | ha | 20:26 |
feeblepipes | mriedem: so, quick question on mgagne's extend volume spec... | 20:33 |
feeblepipes | mriedem: we didn't add a microversion when we added the network-vif-deleted external event. why are we requiring one here? | 20:33 |
mriedem | feeblepipes: i think that predated microversions | 20:34 |
feeblepipes | mriedem: and the answer could easily be "we didn't have microversions then"... I just honestly don't know. | 20:34 |
feeblepipes | ah, ok. | 20:34 |
mriedem | that was like havana or icehouse | 20:34 |
feeblepipes | sdague: any memory of above? | 20:34 |
feeblepipes | mriedem: k. | 20:34 |
mriedem | i remember talking about that at the first midcycle in provo utah | 20:34 |
feeblepipes | mriedem: I'm going to approve this then. | 20:34 |
mriedem | which was icehouse | 20:34 |
mriedem | yeah https://github.com/openstack/nova/commit/ce936ea5f3ae0b4d3b816a7fe42d5f0100b20fca | 20:35 |
mriedem | f man, to the t | 20:35 |
mriedem | icehouse it was | 20:35 |
feeblepipes | mriedem: +W'd. | 20:35 |
*** gouthamr has quit IRC | 20:35 | |
*** ericyoung has quit IRC | 20:36 | |
feeblepipes | mriedem: any more high-priority specs you need reviewed? | 20:36 |
feeblepipes | mriedem: what about the "use service catalog for glance" one? I think that's actually a really good one to do and is a relatively small amount of work. | 20:37 |
feeblepipes | mriedem: well, technically it's not just glance, but all services. | 20:37 |
mriedem | there was a spec for that? | 20:37 |
mriedem | i thought we were just doing that | 20:37 |
mriedem | and had been | 20:37 |
feeblepipes | https://review.openstack.org/#/c/454983/ | 20:37 |
feeblepipes | looks new :) | 20:38 |
mriedem | heh, i don't think we really need a spec for that, | 20:38 |
feeblepipes | still has that new car smell. | 20:38 |
*** vishwanathj has quit IRC | 20:38 | |
mriedem | that's been part of the move to ksa for awhile now | 20:38 |
feeblepipes | ya | 20:38 |
*** vishwanathj has joined #openstack-nova | 20:38 | |
mriedem | let me read quick | 20:38 |
feeblepipes | mriedem: cburgess has responded positively on your suggestion for not limiting migrations in the listing. I say based on that feedback, we don't do it. :P | 20:41 |
mriedem | ha | 20:42 |
mriedem | oh chet | 20:42 |
mriedem | always the punching bag | 20:42 |
feeblepipes | indeed. | 20:42 |
mriedem | i'm going to pull the service catalog one down and fix some nits then +2 | 20:44 |
*** Jack_Iv has quit IRC | 20:44 | |
*** Jack_Iv has joined #openstack-nova | 20:45 | |
*** Jack_Iv has quit IRC | 20:49 | |
*** ijw has joined #openstack-nova | 20:50 | |
*** ijw has quit IRC | 20:50 | |
*** ijw has joined #openstack-nova | 20:50 | |
feeblepipes | mriedem: k, lemme know when you do and I'll review. | 20:51 |
*** dimtruck is now known as zz_dimtruck | 20:52 | |
feeblepipes | cfriesen: nice "does anyone actually use shelve?" :) | 20:53 |
feeblepipes | cfriesen: my feelings precisely. | 20:53 |
feeblepipes | should die in a fire :) | 20:53 |
openstackgerrit | Matt Riedemann proposed openstack/nova-specs master: Use service catalog to get endpoint URLs https://review.openstack.org/454983 | 20:53 |
mriedem | feeblepipes: ^ | 20:53 |
feeblepipes | danke | 20:53 |
mriedem | sdague: yeah nova-manage is probably what we have to use since we can't list instance actions out of the api for all tenants | 20:54 |
*** esberglu has quit IRC | 20:54 | |
mriedem | sdague: if you wanted to brain dump ideas for an interface in a blueprint it could be hacked on | 20:55 |
mriedem | like, list total number of all actions or something | 20:55 |
mriedem | maybe provide a time window | 20:55 |
*** gouthamr has joined #openstack-nova | 20:58 | |
feeblepipes | mriedem: actually, -1 from me. see review.. | 21:02 |
*** pramodrj07 has joined #openstack-nova | 21:03 | |
*** MasterOfBugs has quit IRC | 21:06 | |
mriedem | i commented on some of that in PS1 | 21:06 |
mriedem | looking something up | 21:07 |
mriedem | i guess ksa doesn't provided endoint_override | 21:08 |
mriedem | i know we have api_endpoint for ironic conf in nova and i thought that was coming from ksa | 21:08 |
mriedem | replied | 21:11 |
mriedem | feeblepipes: your turn to update that one? | 21:11 |
mriedem | i'm good with no endpoint_override backdoor, but also want to see the old options deprecated when we're using the service catalog | 21:12 |
*** lucasxu has quit IRC | 21:15 | |
cburgess | feeblepipes Have I wronged you in some way recently? :P | 21:15 |
*** claudiub|2 has joined #openstack-nova | 21:15 | |
cburgess | feeblepipes Yes shelve should die in a fire, preferably a very very hot one. | 21:16 |
*** lucasxu has joined #openstack-nova | 21:16 | |
mriedem | cburgess: you know, the lowest level of hell is actually ice | 21:17 |
cburgess | Hmmm I missed that. | 21:17 |
cburgess | Dante? | 21:17 |
mriedem | yup | 21:17 |
cburgess | Huh... not sure how I missed that. Didn't read it that closely though. | 21:17 |
*** prateek has quit IRC | 21:18 | |
cburgess | I haven't looked yet... is there any type of agenda for Boston? | 21:18 |
cburgess | I know its not typical summit format... | 21:19 |
mriedem | yes | 21:19 |
efried | cburgess If you get the app, you can see (and flag) all the sessions. | 21:19 |
mriedem | http://forumtopics.openstack.org/ are the sessions, but yeah they are in the schedule too now | 21:19 |
efried | So - a bp to deprecate and ultimately remove shelve? | 21:21 |
efried | And see who screams? | 21:21 |
efried | And if anyone gripes after we remove it, we can say we warned 'em? | 21:22 |
mriedem | i don't think we're there yet | 21:22 |
*** abalutoiu has quit IRC | 21:22 | |
efried | Just at the point where we can spend literally all day hatin on it. | 21:22 |
mriedem | i've only spent 1/8 of my day hating it | 21:23 |
mriedem | 3/8 yesterday | 21:23 |
efried | You multitasker, you. | 21:23 |
mriedem | cburgess: this is what you want i think https://www.openstack.org/summit/boston-2017/summit-schedule/global-search?t=Forum%3A%2BFishbowl%2BDiscussion | 21:23 |
mriedem | those are the things that take the place of the design summit now | 21:23 |
cburgess | mriedem Ahh cool thanks | 21:24 |
cburgess | mriedem Is its just 1 track of all of it now? Its all "The Forum" no break outs by project at all in this format? | 21:25 |
*** xinliang has quit IRC | 21:26 | |
mriedem | right | 21:26 |
mriedem | so think of how monday and tuesday used to work before | 21:26 |
cburgess | Hmmm I don't think I can afford donuts for that many people. | 21:26 |
mriedem | 2-3 tracks at a time | 21:26 |
cburgess | Yeah | 21:26 |
mriedem | but now it's 4 days | 21:26 |
mriedem | no vertical design summit | 21:26 |
feeblepipes | mriedem: yeah, gimme sec, will update in a bit. | 21:27 |
feeblepipes | cburgess: :P | 21:27 |
cburgess | feeblepipes What happened to leaky? Got tired of it? | 21:27 |
feeblepipes | cburgess: been sick as a pug of late :( | 21:28 |
cburgess | Lame. Sorry to hear. | 21:28 |
feeblepipes | cburgess: ya | 21:28 |
*** jamesdenton has quit IRC | 21:31 | |
*** crushil has quit IRC | 21:31 | |
*** artom has quit IRC | 21:36 | |
*** artom has joined #openstack-nova | 21:36 | |
*** ngupta has quit IRC | 21:37 | |
*** xinliang has joined #openstack-nova | 21:37 | |
*** xinliang has joined #openstack-nova | 21:37 | |
*** ngupta has joined #openstack-nova | 21:38 | |
*** artom has quit IRC | 21:38 | |
*** artom has joined #openstack-nova | 21:38 | |
*** ngupta has quit IRC | 21:42 | |
efried | mriedem Since I'm new to this bug triaging thing, can you please let me know whether I've done the right thing here, and/or if I could/should do more? https://bugs.launchpad.net/nova/+bug/1682195 | 21:48 |
openstack | Launchpad bug 1682195 in OpenStack Compute (nova) "Unexpected API Error" [Undecided,New] | 21:48 |
efried | I don't think I have permission to change the status. Also didn't see a way to mark a dup in an actual field. | 21:48 |
efried | ah, there it is. | 21:48 |
*** felipemonteiro has quit IRC | 21:49 | |
*** winston-d_ has quit IRC | 21:53 | |
feeblepipes | mriedem: with the cinderv2 thing, they have the service_type as cinderv2 :) | 21:55 |
*** mdrabe has quit IRC | 21:58 | |
*** amotoki has quit IRC | 21:58 | |
*** jogo has quit IRC | 21:59 | |
*** jogo has joined #openstack-nova | 21:59 | |
*** lucasxu has quit IRC | 22:01 | |
*** MasterOfBugs has joined #openstack-nova | 22:01 | |
*** gouthamr has quit IRC | 22:03 | |
*** pramodrj07 has quit IRC | 22:04 | |
*** burt has quit IRC | 22:05 | |
openstackgerrit | Jay Pipes proposed openstack/nova-specs master: Use service catalog to get endpoint URLs https://review.openstack.org/454983 | 22:09 |
feeblepipes | mriedem: ^ | 22:09 |
*** hongbin has joined #openstack-nova | 22:17 | |
*** feeblepipes has quit IRC | 22:20 | |
*** sdague has quit IRC | 22:25 | |
*** ngupta has joined #openstack-nova | 22:25 | |
*** zz_dimtruck is now known as dimtruck | 22:26 | |
mriedem | efried: ok, i don't know that https://bugs.launchpad.net/nova/+bug/1682195 is a duplicate of the other bug because the other bug is related to showing consoles | 22:27 |
openstack | Launchpad bug 1678493 in OpenStack Compute (nova) "duplicate for #1682195 oslo_messaging.exceptions.MessagingTimeout" [Undecided,Incomplete] | 22:27 |
mriedem | the former is just some error when launching an instance | 22:27 |
mriedem | bug 1682195 is using nova-network | 22:28 |
mriedem | probably not setup properly | 22:28 |
efried | mriedem Okay. Is it legal to mark as Incomplete out of hand when they don't have logs & configs attached? | 22:28 |
efried | Or is that just being lazy? | 22:29 |
mriedem | efried: as invalid if it's a completely a support request | 22:29 |
*** thorst has quit IRC | 22:30 | |
openstackgerrit | Ildiko Vancsa proposed openstack/nova master: Add Cinder v3 detach to swap_volume https://review.openstack.org/456971 | 22:32 |
mriedem | efried: i also base it on how much effort they put into the bug report | 22:33 |
mriedem | if it's just a "nova boot failed, 500, here is a stacktrace that doesn't help at all", i mark it as invalid | 22:34 |
*** ngupta has quit IRC | 22:34 | |
mriedem | i'll leave a comment about why it's not helpful and what to provide, but it's invalid | 22:34 |
*** lyan has quit IRC | 22:34 | |
mriedem | if they actually give a reasonable amount of information but i need more info, i ask the questions and mark it incomplete | 22:34 |
*** ngupta has joined #openstack-nova | 22:34 | |
efried | mriedem Okay. The same guy opened three separate bug reports with very similar stack traces, which probably indicates it's a config problem on his part. | 22:34 |
efried | I marked them all as dups of the one I referenced above, but perhaps more appropriate would be to mark the first one Invalid/Incomplete and dup the others, noting that it's probably his config. | 22:35 |
mriedem | i'd just mark all invalid | 22:36 |
*** edmondsw has joined #openstack-nova | 22:36 | |
mriedem | i don't duplicate super generic dead cat on your desk bugs as duplicates of other dead cat invalid bugs | 22:36 |
*** rcernin has quit IRC | 22:37 | |
*** ngupta has quit IRC | 22:38 | |
*** zul has quit IRC | 22:38 | |
*** edmondsw has quit IRC | 22:40 | |
*** shadow-x has joined #openstack-nova | 22:43 | |
*** gjayavelu has quit IRC | 22:48 | |
*** tbachman has joined #openstack-nova | 22:50 | |
*** Sukhdev has joined #openstack-nova | 22:55 | |
*** esberglu has joined #openstack-nova | 22:55 | |
*** tbachman has quit IRC | 22:55 | |
*** xyang1 has quit IRC | 22:57 | |
*** esberglu has quit IRC | 22:58 | |
*** esberglu has joined #openstack-nova | 22:58 | |
efried | mriedem Thanks for the tips. I marked https://bugs.launchpad.net/nova/+bug/1682087 and https://bugs.launchpad.net/nova/+bug/1682088 accordingly. | 23:07 |
openstack | Launchpad bug 1682087 in OpenStack Compute (nova) "<class 'oslo_messaging.exceptions.MessagingTimeout'>" [Undecided,Invalid] | 23:07 |
openstack | Launchpad bug 1682088 in OpenStack Compute (nova) "<class 'oslo_messaging.exceptions.MessagingTimeout'>" [Undecided,Invalid] | 23:07 |
*** esberglu has quit IRC | 23:07 | |
*** yamamoto has joined #openstack-nova | 23:11 | |
*** baoli has quit IRC | 23:13 | |
mriedem | efried: thanks for doing triage | 23:15 |
*** yamamoto has quit IRC | 23:16 | |
*** catintheroof has joined #openstack-nova | 23:16 | |
*** gouthamr has joined #openstack-nova | 23:20 | |
*** tuan_luong has joined #openstack-nova | 23:27 | |
*** tuanla_ has joined #openstack-nova | 23:28 | |
*** tuanla_ has quit IRC | 23:28 | |
*** tuan_luong has quit IRC | 23:28 | |
*** gjayavelu has joined #openstack-nova | 23:28 | |
*** gjayavelu has quit IRC | 23:29 | |
*** lyan has joined #openstack-nova | 23:49 | |
*** baoli has joined #openstack-nova | 23:50 | |
*** baoli has quit IRC | 23:55 | |
*** baoli has joined #openstack-nova | 23:58 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!