| *** thorst_ has joined #openstack-nova | 00:03 | |
| *** sacharya has joined #openstack-nova | 00:03 | |
| *** jamielennox|away is now known as jamielennox | 00:05 | |
| *** voelzmo has joined #openstack-nova | 00:06 | |
| *** ducttape_ has joined #openstack-nova | 00:06 | |
| *** thorst_ has quit IRC | 00:08 | |
| *** voelzmo has quit IRC | 00:11 | |
| *** diana_clarke has joined #openstack-nova | 00:15 | |
| *** tbachman has joined #openstack-nova | 00:19 | |
| *** jamesdenton has joined #openstack-nova | 00:22 | |
| *** gouthamr has quit IRC | 00:24 | |
| *** sacharya has quit IRC | 00:27 | |
| *** sacharya has joined #openstack-nova | 00:27 | |
| *** gouthamr has joined #openstack-nova | 00:33 | |
| *** ducttape_ has quit IRC | 00:37 | |
| *** ducttape_ has joined #openstack-nova | 00:37 | |
| *** ducttape_ has quit IRC | 00:37 | |
| *** ducttape_ has joined #openstack-nova | 00:38 | |
| *** hfu has joined #openstack-nova | 00:39 | |
| *** ducttape_ has joined #openstack-nova | 00:42 | |
| *** edmondsw has joined #openstack-nova | 00:43 | |
| *** mtanino has quit IRC | 00:43 | |
| *** edmondsw has quit IRC | 00:48 | |
| *** thorst_ has joined #openstack-nova | 00:50 | |
| openstackgerrit | Digambar proposed openstack/nova-specs: Add placement nofications using versionedObjects. https://review.openstack.org/423872 | 00:51 |
|---|---|---|
| *** claudiub has quit IRC | 00:53 | |
| *** thorst_ has quit IRC | 00:53 | |
| *** hoangcx has joined #openstack-nova | 00:54 | |
| *** sacharya has quit IRC | 00:57 | |
| *** alexpilotti has joined #openstack-nova | 01:00 | |
| *** jamesdenton has quit IRC | 01:00 | |
| *** alexpilotti has quit IRC | 01:04 | |
| *** jamesdenton has joined #openstack-nova | 01:06 | |
| *** phuongnh has joined #openstack-nova | 01:07 | |
| *** ijw has joined #openstack-nova | 01:09 | |
| *** abalutoiu__ has joined #openstack-nova | 01:10 | |
| *** abalutoiu_ has quit IRC | 01:13 | |
| *** yuntongjin has joined #openstack-nova | 01:38 | |
| *** dikonoor has quit IRC | 01:39 | |
| *** jamesdenton has quit IRC | 01:45 | |
| openstackgerrit | zhangyujun proposed openstack/nova: move 'MEMPAGES_*' to fields 'fileds.MemoryPageSize' https://review.openstack.org/417742 | 01:47 |
| *** mingyu has joined #openstack-nova | 01:49 | |
| *** thorst_ has joined #openstack-nova | 01:55 | |
| *** thorst_ has quit IRC | 01:56 | |
| *** ijw_ has joined #openstack-nova | 01:56 | |
| *** ijw has quit IRC | 01:59 | |
| *** baoli has quit IRC | 02:05 | |
| *** voelzmo has joined #openstack-nova | 02:07 | |
| *** voelzmo has quit IRC | 02:12 | |
| openstackgerrit | Digambar proposed openstack/nova-specs: Add placement nofications using versionedObjects https://review.openstack.org/423872 | 02:13 |
| *** tiendc has joined #openstack-nova | 02:16 | |
| *** artom_ has quit IRC | 02:16 | |
| openstackgerrit | Digambar proposed openstack/nova-specs: Add placement nofications using versionedObjects https://review.openstack.org/423872 | 02:19 |
| *** esmiurium has quit IRC | 02:20 | |
| *** esmiurium has joined #openstack-nova | 02:26 | |
| *** yuntongjin has quit IRC | 02:29 | |
| *** mingyu has quit IRC | 02:31 | |
| *** yuntongjin has joined #openstack-nova | 02:37 | |
| *** dave-mccowan has joined #openstack-nova | 02:38 | |
| *** sacharya has joined #openstack-nova | 02:43 | |
| *** jamesdenton has joined #openstack-nova | 02:45 | |
| *** mingyu has joined #openstack-nova | 02:46 | |
| *** thorst_ has joined #openstack-nova | 02:48 | |
| *** thorst_ has quit IRC | 02:48 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/os-vif: [py35] Switch filter to list comprehensions https://review.openstack.org/423718 | 02:53 |
| *** jerrygb has joined #openstack-nova | 02:55 | |
| *** tbachman has quit IRC | 02:56 | |
| *** amotoki has quit IRC | 02:59 | |
| *** Apoorva has joined #openstack-nova | 03:00 | |
| *** Apoorva has quit IRC | 03:03 | |
| *** gongysh has joined #openstack-nova | 03:05 | |
| *** amotoki has joined #openstack-nova | 03:10 | |
| *** amotoki has quit IRC | 03:15 | |
| *** david-lyle has joined #openstack-nova | 03:15 | |
| *** kevinz has joined #openstack-nova | 03:16 | |
| *** amotoki has joined #openstack-nova | 03:20 | |
| *** david-lyle has quit IRC | 03:21 | |
| *** baoli has joined #openstack-nova | 03:26 | |
| *** ducttape_ has quit IRC | 03:28 | |
| *** ducttape_ has joined #openstack-nova | 03:28 | |
| *** ducttape_ has quit IRC | 03:29 | |
| *** ducttape_ has joined #openstack-nova | 03:29 | |
| *** baoli has quit IRC | 03:30 | |
| *** sudipto has joined #openstack-nova | 03:31 | |
| *** sudipto_ has joined #openstack-nova | 03:31 | |
| *** amotoki_ has joined #openstack-nova | 03:33 | |
| *** takashin has joined #openstack-nova | 03:35 | |
| *** amotoki has quit IRC | 03:35 | |
| gmann | oomichi: looks nice just few query/comment - https://review.openstack.org/#/c/406377/3 | 03:36 |
| *** yuntongjin has quit IRC | 03:36 | |
| *** zz_dimtruck is now known as dimtruck | 03:39 | |
| *** amotoki_ has quit IRC | 03:42 | |
| *** esmiurium has quit IRC | 03:42 | |
| *** jamesdenton has quit IRC | 03:45 | |
| *** esmiurium has joined #openstack-nova | 03:49 | |
| *** baoli has joined #openstack-nova | 03:53 | |
| *** dave-mccowan has quit IRC | 03:53 | |
| *** esmiurium has quit IRC | 03:54 | |
| *** dikonoor has joined #openstack-nova | 03:54 | |
| *** baoli has quit IRC | 04:00 | |
| *** jerrygb has quit IRC | 04:03 | |
| *** Sukhdev has joined #openstack-nova | 04:03 | |
| *** dave-mccowan has joined #openstack-nova | 04:04 | |
| *** abalutoiu__ has quit IRC | 04:05 | |
| Kevin_Zheng | gmann: https://bugs.launchpad.net/nova/+bug/1658571 | 04:07 |
| openstack | Launchpad bug 1658571 in OpenStack Compute (nova) "Microversion 2.37 break 2.32 usage" [Undecided,New] - Assigned to Zhenyu Zheng (zhengzhenyu) | 04:07 |
| Kevin_Zheng | looks like I found a critical bug here | 04:08 |
| *** sree has joined #openstack-nova | 04:09 | |
| *** sree has joined #openstack-nova | 04:09 | |
| *** tbachman has joined #openstack-nova | 04:09 | |
| *** ducttape_ has joined #openstack-nova | 04:10 | |
| *** ducttape_ has quit IRC | 04:11 | |
| *** ducttape_ has joined #openstack-nova | 04:12 | |
| *** ducttape_ has quit IRC | 04:12 | |
| *** ducttape_ has joined #openstack-nova | 04:12 | |
| *** Dinesh_Bhor has joined #openstack-nova | 04:13 | |
| *** ducttape_ has quit IRC | 04:13 | |
| *** ducttape_ has joined #openstack-nova | 04:13 | |
| gmann | Kevin_Zheng: ohh, yea you are right. it overridden the tag field. :( | 04:14 |
| *** ducttape_ has quit IRC | 04:14 | |
| *** nicolasbock has quit IRC | 04:14 | |
| gmann | Kevin_Zheng: i agree we should backport that | 04:14 |
| Kevin_Zheng | will try to fix this after launch | 04:15 |
| gmann | Kevin_Zheng: very nice catch | 04:15 |
| Kevin_Zheng | :) | 04:15 |
| gmann | Kevin_Zheng: +1 | 04:15 |
| *** amotoki has joined #openstack-nova | 04:16 | |
| *** jwcroppe has quit IRC | 04:17 | |
| *** mdnadeem has joined #openstack-nova | 04:22 | |
| *** tbachman has quit IRC | 04:23 | |
| *** psachin has joined #openstack-nova | 04:24 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: Refactor a test method including 7 test cases https://review.openstack.org/395436 | 04:25 |
| *** dikonoor has quit IRC | 04:26 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: Enable cold migration with target host(1/2) https://review.openstack.org/408955 | 04:26 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: Enable cold migration with target host(2/2) https://review.openstack.org/408964 | 04:26 |
| openstackgerrit | Takashi NATSUME proposed openstack/python-novaclient: Microversion 2.42 - Enable cold migration with target host https://review.openstack.org/406707 | 04:27 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: Add functional tests for cold migration to same host https://review.openstack.org/414926 | 04:27 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: api-ref: Fix path parameters in os-hypervisors.inc https://review.openstack.org/416911 | 04:27 |
| *** sofie has joined #openstack-nova | 04:27 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: api-ref: Fix deprecated proxy API parameters https://review.openstack.org/416882 | 04:27 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: api-ref: Fix keypair API parameters https://review.openstack.org/420626 | 04:27 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: api-ref: Add parameters in cold migrate action https://review.openstack.org/410042 | 04:27 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: Avoid forcing translation on logging calls https://review.openstack.org/413876 | 04:27 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: Refactor a test method including 3 test cases https://review.openstack.org/394738 | 04:27 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: api-ref: Fix parameters in os-hosts.inc https://review.openstack.org/416444 | 04:27 |
| *** dave-mccowan has quit IRC | 04:35 | |
| *** gongysh has quit IRC | 04:36 | |
| *** mingyu has quit IRC | 04:43 | |
| *** esmiurium has joined #openstack-nova | 04:45 | |
| *** mdnadeem has quit IRC | 04:46 | |
| *** thorst_ has joined #openstack-nova | 04:49 | |
| *** thorst_ has quit IRC | 04:54 | |
| *** esmiurium has quit IRC | 05:01 | |
| *** udesale has joined #openstack-nova | 05:02 | |
| *** yuntongjin has joined #openstack-nova | 05:03 | |
| *** gouthamr has quit IRC | 05:07 | |
| *** esmiurium has joined #openstack-nova | 05:07 | |
| *** ayogi has joined #openstack-nova | 05:08 | |
| *** voelzmo has joined #openstack-nova | 05:09 | |
| *** voelzmo has quit IRC | 05:14 | |
| *** esmiurium has quit IRC | 05:14 | |
| *** diga has joined #openstack-nova | 05:15 | |
| *** prateek has joined #openstack-nova | 05:15 | |
| *** gongysh has joined #openstack-nova | 05:16 | |
| *** ducttape_ has joined #openstack-nova | 05:23 | |
| *** ratailor has joined #openstack-nova | 05:24 | |
| *** ecelik has joined #openstack-nova | 05:25 | |
| *** ecelik has left #openstack-nova | 05:25 | |
| *** hongbin has quit IRC | 05:28 | |
| *** ducttape_ has quit IRC | 05:29 | |
| *** esmiurium has joined #openstack-nova | 05:31 | |
| *** gongysh has quit IRC | 05:32 | |
| *** ijw_ has quit IRC | 05:32 | |
| *** guchihiro has joined #openstack-nova | 05:38 | |
| *** ratailor is now known as ratailor|afk | 05:47 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: Use ListOfIntegersField in oslo.versionedobjects https://review.openstack.org/423912 | 05:53 |
| *** ratailor|afk is now known as ratailor | 06:01 | |
| *** jerrygb has joined #openstack-nova | 06:04 | |
| *** jerrygb has quit IRC | 06:08 | |
| *** sacharya has quit IRC | 06:10 | |
| *** edmondsw has joined #openstack-nova | 06:13 | |
| *** lpetrut has joined #openstack-nova | 06:14 | |
| *** mdnadeem has joined #openstack-nova | 06:14 | |
| openstackgerrit | zhangyanxian proposed openstack/nova-specs: Delete the unnecessary word in pcs-support.rst https://review.openstack.org/423927 | 06:17 |
| *** edmondsw has quit IRC | 06:17 | |
| openstackgerrit | zhangyanxian proposed openstack/nova-specs: Delete the unnecessary word in pcs-support.rst https://review.openstack.org/423927 | 06:17 |
| *** bkopilov has joined #openstack-nova | 06:23 | |
| *** moshele has joined #openstack-nova | 06:26 | |
| openstackgerrit | Hussain Chachuliya proposed openstack/nova: Fix 500 error while passing 4-byte unicode data https://review.openstack.org/407514 | 06:31 |
| *** dikonoor has joined #openstack-nova | 06:31 | |
| *** ekuris_ has joined #openstack-nova | 06:34 | |
| *** claudiub has joined #openstack-nova | 06:38 | |
| *** claudiub|2 has joined #openstack-nova | 06:40 | |
| *** claudiub has quit IRC | 06:42 | |
| *** sridharg has joined #openstack-nova | 06:45 | |
| *** thorst_ has joined #openstack-nova | 06:50 | |
| *** ducttape_ has joined #openstack-nova | 06:54 | |
| *** thorst_ has quit IRC | 06:55 | |
| openstackgerrit | zhurong proposed openstack/python-novaclient: Fix client can not auth with token https://review.openstack.org/423946 | 06:56 |
| *** ducttape_ has quit IRC | 06:59 | |
| *** markus_z has joined #openstack-nova | 06:59 | |
| *** avolkov has joined #openstack-nova | 07:04 | |
| *** ircuser-1 has quit IRC | 07:06 | |
| *** hfu has quit IRC | 07:07 | |
| *** aojea has quit IRC | 07:07 | |
| *** hfu has joined #openstack-nova | 07:07 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Fix device taging after API microversion 2.37 https://review.openstack.org/423952 | 07:08 |
| *** dikonoor has quit IRC | 07:09 | |
| *** sacharya has joined #openstack-nova | 07:11 | |
| *** dikonoor has joined #openstack-nova | 07:11 | |
| *** Oku_OS-away is now known as Oku_OS | 07:15 | |
| *** Sukhdev has quit IRC | 07:15 | |
| *** sacharya has quit IRC | 07:15 | |
| *** sahid has joined #openstack-nova | 07:17 | |
| *** voelzmo has joined #openstack-nova | 07:18 | |
| *** aojea has joined #openstack-nova | 07:20 | |
| *** ircuser-1 has joined #openstack-nova | 07:24 | |
| *** tesseract has joined #openstack-nova | 07:31 | |
| *** yuntongjin has quit IRC | 07:33 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add sort_key white list for server list/detail https://review.openstack.org/415142 | 07:34 |
| *** yonglihe has joined #openstack-nova | 07:45 | |
| *** hfu has quit IRC | 07:48 | |
| openstackgerrit | Wang Qing wu proposed openstack/nova: WIP(5): Introduce base PowerVM driver (ovs vif) https://review.openstack.org/422512 | 07:52 |
| *** takashin has left #openstack-nova | 07:58 | |
| *** lpetrut has quit IRC | 08:00 | |
| *** nkrinner_afk is now known as nkrinner | 08:02 | |
| snikitin | BobBall: Hi, Bob! Could you please look at Xen diagnostics again? https://review.openstack.org/#/c/399613/ Citrix XenServer CI is finally green! https://review.openstack.org/#/c/413606/ | 08:02 |
| *** brault has joined #openstack-nova | 08:07 | |
| *** lpetrut has joined #openstack-nova | 08:08 | |
| openstackgerrit | gengchc2 proposed openstack/nova: Improved-the-judgment-of-the-service-status-in-memcache https://review.openstack.org/423754 | 08:10 |
| openstackgerrit | Pawel Koniszewski proposed openstack/nova: live-mig: Add claims and proper resource tracking https://review.openstack.org/244489 | 08:12 |
| openstackgerrit | Pawel Koniszewski proposed openstack/nova: libvirt: pass libvirt reference methods to migration https://review.openstack.org/328997 | 08:12 |
| *** amoralej|off is now known as amoralej | 08:12 | |
| openstackgerrit | Pawel Koniszewski proposed openstack/nova: libvirt: live-migrate updates NUMA and cpus in the XML https://review.openstack.org/286744 | 08:12 |
| *** swamireddy has quit IRC | 08:15 | |
| *** dikonoor has quit IRC | 08:17 | |
| *** lpetrut has quit IRC | 08:17 | |
| *** pcaruana has joined #openstack-nova | 08:17 | |
| *** dikonoor has joined #openstack-nova | 08:18 | |
| *** ducttape_ has joined #openstack-nova | 08:25 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Fix device taging after API microversion 2.37 https://review.openstack.org/423952 | 08:28 |
| *** ducttape_ has quit IRC | 08:30 | |
| *** swamireddy has joined #openstack-nova | 08:34 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Fix device taging after API microversion 2.37 https://review.openstack.org/423952 | 08:34 |
| openstackgerrit | sahid proposed openstack/nova: numa: add numa constraints for emulator threads policy https://review.openstack.org/385023 | 08:38 |
| openstackgerrit | sahid proposed openstack/nova: libvirt: return a CPU overhead if isolate emulator threads requested https://review.openstack.org/385364 | 08:38 |
| openstackgerrit | sahid proposed openstack/nova: objects: add new field cpuset_reserved in NUMACell https://review.openstack.org/400874 | 08:38 |
| openstackgerrit | sahid proposed openstack/nova: numa: fit instance topology with cpus reserved onto cores https://review.openstack.org/400875 | 08:38 |
| openstackgerrit | sahid proposed openstack/nova: virt: update overhead to take into account vCPUs https://review.openstack.org/385024 | 08:38 |
| openstackgerrit | sahid proposed openstack/nova: libvirt: make emulator threads to run on the reserved pCPU https://review.openstack.org/400878 | 08:39 |
| openstackgerrit | sahid proposed openstack/nova: numa: take into account cpus reserved https://review.openstack.org/400876 | 08:39 |
| openstackgerrit | sahid proposed openstack/nova: numa: update numa usage incording the cpu reserved https://review.openstack.org/400877 | 08:39 |
| *** jpena|off is now known as jpena | 08:41 | |
| *** a-pugachev has joined #openstack-nova | 08:42 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add sort_key white list for server list/detail https://review.openstack.org/415142 | 08:44 |
| *** satyar has joined #openstack-nova | 08:47 | |
| *** thorst_ has joined #openstack-nova | 08:51 | |
| *** pkoniszewski_ is now known as pkoniszewski | 08:52 | |
| *** ralonsoh has joined #openstack-nova | 08:54 | |
| *** guchihiro has quit IRC | 08:55 | |
| *** thorst_ has quit IRC | 08:56 | |
| openstackgerrit | Istvan Imre proposed openstack/python-novaclient: Pass relevant parameters to Token based authentication https://review.openstack.org/419441 | 08:59 |
| *** zzzeek has quit IRC | 09:00 | |
| *** zzzeek has joined #openstack-nova | 09:01 | |
| *** ekuris_ has quit IRC | 09:04 | |
| *** mjura has joined #openstack-nova | 09:05 | |
| *** mlakat has joined #openstack-nova | 09:11 | |
| *** acabot_ has joined #openstack-nova | 09:12 | |
| *** ekuris_ has joined #openstack-nova | 09:20 | |
| *** slaweq has quit IRC | 09:20 | |
| *** slaweq has joined #openstack-nova | 09:22 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Fix device taging after API microversion 2.37 https://review.openstack.org/423952 | 09:26 |
| *** yarkot has quit IRC | 09:29 | |
| *** efoley has joined #openstack-nova | 09:29 | |
| *** abalutoiu has joined #openstack-nova | 09:33 | |
| *** yassine has joined #openstack-nova | 09:37 | |
| *** yassine is now known as Guest17268 | 09:37 | |
| *** lennyb has joined #openstack-nova | 09:37 | |
| *** mingyu has joined #openstack-nova | 09:37 | |
| openstackgerrit | gengchc2 proposed openstack/nova: Improved-the-judgment-of-the-service-status-in-memcache https://review.openstack.org/423754 | 09:42 |
| *** amotoki has quit IRC | 09:44 | |
| openstackgerrit | Duong Ha-Quang proposed openstack/nova: Remove mox in nova/tests/unit/compute/test_shelve.py (3) https://review.openstack.org/342648 | 09:46 |
| *** derekh has joined #openstack-nova | 09:47 | |
| *** iceyao has joined #openstack-nova | 09:49 | |
| sfinucan | Morning, mdbooth. Any chance you could take a gawk at https://review.openstack.org/#/c/392643/ when you have a chance? | 09:49 |
| openstackgerrit | Rikimaru Honjo proposed openstack/nova: Consolidate unit tests for shelve API https://review.openstack.org/421220 | 09:49 |
| *** edmondsw has joined #openstack-nova | 09:50 | |
| *** lpetrut has joined #openstack-nova | 09:54 | |
| *** edmondsw has quit IRC | 09:54 | |
| alex_xu | johnthetubaguy: gmann Kevin_Zheng, I guess we need some discussion on https://review.openstack.org/#/c/415330/16/nova/policies/servers.py for making progress, are you guys around? | 09:55 |
| pkoniszewski | hello, can I have some eyes on this series https://review.openstack.org/#/q/topic:lm-downtime ? Freeze is near, those 3 are simple patches to increase live migration success rate and reduce configuration complexity | 09:55 |
| Kevin_Zheng | alex_xu: you can represent me :), I'm about to driving | 09:55 |
| *** ducttape_ has joined #openstack-nova | 09:55 | |
| alex_xu | Kevin_Zheng: yea, just focus on driving :) | 09:56 |
| *** ekuris_ has quit IRC | 09:56 | |
| *** iceyao_ has joined #openstack-nova | 09:57 | |
| mikal | were | 09:58 |
| mikal | Doh, sorry | 09:58 |
| *** iceyao_ has quit IRC | 09:58 | |
| *** hachi has joined #openstack-nova | 09:59 | |
| *** iceyao has quit IRC | 10:00 | |
| *** ducttape_ has quit IRC | 10:00 | |
| snikitin | BobBall: thank you for review! | 10:03 |
| BobBall | snikitin: Thank you for your perseverence with the CI ;) | 10:03 |
| gmann | alex_xu: hi | 10:04 |
| gmann | alex_xu: actually problem is we cannot not know whether rule is overridden or not right? | 10:04 |
| bauzas | johnthetubaguy: hola, got a sec ? | 10:04 |
| bauzas | johnthetubaguy: not sure you saw my ML thread about the placement API | 10:05 |
| gmann | alex_xu: it can be with same default value RULE_ADMIN_API and operator can claim this was our overridden value why you broke us | 10:05 |
| bauzas | johnthetubaguy: http://lists.openstack.org/pipermail/openstack-dev/2017-January/110661.html | 10:05 |
| alex_xu | gmann: I thought we have some way, just like this https://github.com/openstack/oslo.policy/blob/master/oslo_policy/generator.py#L216 | 10:05 |
| bauzas | johnthetubaguy: but I'd love your operator insights, in particular wrt the CachingScheduler | 10:06 |
| *** sambetts|afk is now known as sambetts | 10:06 | |
| *** sapcc-bot has quit IRC | 10:06 | |
| *** dgonzalez_ has joined #openstack-nova | 10:06 | |
| *** sapcc-bot has joined #openstack-nova | 10:06 | |
| gmann | alex_xu: yea but if they have overridden with RULE_ADMIN_API then they expect to work on same way which makes 1st and 2nd case same | 10:07 |
| openstackgerrit | Duong Ha-Quang proposed openstack/nova: Remove mox in nova/tests/unit/compute/test_shelve.py (3) https://review.openstack.org/342648 | 10:08 |
| *** dgonzalez_ has quit IRC | 10:08 | |
| gmann | alex_xu: making old rule to RULE_ANY breaks the old operator as johnthetubaguy pointed | 10:09 |
| *** ekuris_ has joined #openstack-nova | 10:09 | |
| alex_xu | gmann: so we expect 1st and 2nd cases work as old behaviour? | 10:11 |
| lyarwood | reviews on the following findmnt removal change would be appreciated if anyone has time, currently hitting EL based distros pretty hard - https://review.openstack.org/#/c/420146/ | 10:12 |
| *** sapcc-bot has quit IRC | 10:12 | |
| gmann | alex_xu: only 1st but it is hard to differentiate those cases | 10:12 |
| *** sapcc-bot has joined #openstack-nova | 10:12 | |
| *** mvk has quit IRC | 10:12 | |
| *** dgonzalez_ has joined #openstack-nova | 10:12 | |
| alex_xu | gmann: so we expect 2nd case as new behaviour | 10:12 |
| gmann | alex_xu: yea | 10:13 |
| alex_xu | gmann: if we can check whether there is override, then we can implement all three cases. | 10:14 |
| *** dgonzalez_ has quit IRC | 10:14 | |
| gmann | alex_xu: even override with default value we consider that as 1st case right ? | 10:15 |
| alex_xu | gmann: yes, I think it is fine | 10:15 |
| alex_xu | gmann: that is way for operator to enforce a old behaviour | 10:16 |
| gmann | alex_xu: so how to check if overridden ? | 10:16 |
| alex_xu | gmann: i'm thinking this way https://github.com/openstack/oslo.policy/blob/master/oslo_policy/generator.py#L176 | 10:16 |
| *** slaweq has quit IRC | 10:17 | |
| alex_xu | gmann: if the rule in the enforcer.file_rules, then it should mean the rule in the policy.json | 10:17 |
| *** kevinz has quit IRC | 10:17 | |
| *** gszasz has joined #openstack-nova | 10:17 | |
| *** CristinaPauna has quit IRC | 10:18 | |
| *** slaweq has joined #openstack-nova | 10:19 | |
| gmann | alex_xu: but i think we say, generate the policy with all default rules and then override if needed, it means all rules will be in policy.json ?(not sure how operator prepare policy.json) | 10:19 |
| sahid | jaypipes: could you ack https://review.openstack.org/#/c/400874 i repin it to change the variable name | 10:19 |
| *** mariusv has quit IRC | 10:19 | |
| gmann | i am just assuming how people use config file | 10:19 |
| *** CristinaPauna has joined #openstack-nova | 10:20 | |
| *** phuongnh has quit IRC | 10:20 | |
| *** mariusv has joined #openstack-nova | 10:20 | |
| *** mariusv has quit IRC | 10:20 | |
| *** mariusv has joined #openstack-nova | 10:20 | |
| alex_xu | gmann: we expect the policy.json empty. the policy.json should only have override rule | 10:24 |
| gmann | alex_xu: like this - http://docs-draft.openstack.org/30/415330/16/check/gate-nova-docs-ubuntu-xenial/13f51ca//doc/build/html/sample_policy.html | 10:24 |
| gmann | alex_xu: and operator just override the interested one | 10:24 |
| gmann | alex_xu: but m not sure about that this is how they use or they have only overriden rules in policy.json | 10:24 |
| *** ociuhandu has quit IRC | 10:26 | |
| alex_xu | gmann: if the user really have override rule, there will be a warning message :) | 10:27 |
| *** Hazelesque_ is now known as Hazelesque | 10:27 | |
| *** hoangcx has quit IRC | 10:27 | |
| *** abalutoiu has quit IRC | 10:27 | |
| *** dikonoor has quit IRC | 10:28 | |
| gmann | alex_xu: yea that case is fine as you said but if anyone assuming overridden rule even with default value only and expect to work same way | 10:28 |
| gmann | they do not believe the nova policy default values and want to make sure by overriding the rule even with same default value | 10:29 |
| claudiub|2 | sfinucan: moshele: hello. If you have a tiny bit of time, can you take a look at the hyper-v pci passthrough patch? it is quite small too: https://review.openstack.org/#/c/420614/ | 10:30 |
| *** satyar has quit IRC | 10:31 | |
| sfinucan | claudiub|2: Just back from vacation so I'm chowing down on (a crazy amount of) mails and other reviews. If you stick me on the review though, I'll aim to get around to it today or tomorrow | 10:32 |
| claudiub|2 | sfinucan: sure, ty. :) | 10:32 |
| *** claudiub|2 is now known as claudiub | 10:32 | |
| alex_xu | gmann: emmm....maybe johnthetubaguy and sdague can ensure that whether we expect a empty policy.json for new installation | 10:34 |
| mdbooth | sfinucan: Looking now. | 10:35 |
| mdbooth | sfinucan: Incidentally, this one needs to get into Ocata: https://review.openstack.org/#/c/423392/ . It's a continuation of a previous code removal. Needs to be phased for upgrades, hence need for Ocata. | 10:36 |
| mdbooth | It's very simple. | 10:36 |
| sfinucan | mdbooth: Sure thing. I'll chuck that on my review queue :) | 10:37 |
| *** owalsh-afk has joined #openstack-nova | 10:38 | |
| *** owalsh-afk is now known as owalsh | 10:38 | |
| gmann | alex_xu: yea, that will be nice to know for me too | 10:41 |
| *** cdent has joined #openstack-nova | 10:42 | |
| *** purplerbot has quit IRC | 10:46 | |
| *** mvk has joined #openstack-nova | 10:46 | |
| moshele | claudiub|2: sure I will take a look later today | 10:47 |
| *** purplerbot has joined #openstack-nova | 10:48 | |
| *** abalutoiu has joined #openstack-nova | 10:49 | |
| *** dikonoor has joined #openstack-nova | 10:51 | |
| cdent | bauzas, jaypipes : was just reading back through the log about something to do with noauthmiddleware and fixtures and placement, but can't really piece together the gist. Were you able to work things out? | 10:51 |
| bauzas | cdent: I fixed the problem | 10:51 |
| *** ratailor has quit IRC | 10:51 | |
| bauzas | cdent: it was merged on Friday by adding a PlacementFixture | 10:52 |
| *** sahid has quit IRC | 10:52 | |
| *** thorst_ has joined #openstack-nova | 10:52 | |
| cdent | bauzas: In any case there is a functional test of placement api that does not use gabbi: https://github.com/openstack/nova/blob/master/nova/tests/functional/api/openstack/placement/test_report_client.py | 10:52 |
| bauzas | cdent: now I have another question about having resource usage call being optionals given the existing filters | 10:52 |
| bauzas | cdent: I know | 10:52 |
| cdent | bauzas: and the noauthmiddleware is _not_ something I wrote special, it is the nova one adapted | 10:53 |
| cdent | do you have a link to the new fixture? | 10:53 |
| cdent | (I mean the review that made it) | 10:53 |
| bauzas | cdent: the problem was with the existing other functional tests that call the scheduler | 10:53 |
| bauzas | cdent: see nova.fixtures.PlacementFixture | 10:53 |
| cdent | Yeah, I've seen the mail about the optionals, it's next on my list to look at more closely | 10:53 |
| bauzas | cdent: I'll be pushing a new PS by today | 10:53 |
| cdent | cool | 10:54 |
| bauzas | that would +1 the functional tests | 10:54 |
| bauzas | I had a shit ton of problems related to the functional tests, but that's done | 10:55 |
| *** udesale has quit IRC | 10:55 | |
| *** thorst_ has quit IRC | 10:57 | |
| cdent | bauzas: did you consider using wsgi-intercept (as in the pre-existing functional test) instead of doing a server.run()? | 11:02 |
| bauzas | yup | 11:02 |
| cdent | and that wouldn't work? | 11:02 |
| bauzas | but we agreed with sdague on using the Server directly | 11:03 |
| bauzas | cdent: honestly, that looks a nit | 11:03 |
| cdent | I'm not complaining, I'm just trying to understand the decision process | 11:03 |
| bauzas | cdent: given the short deadline, I don't want to discuss again about it | 11:03 |
| bauzas | yeah, I can understand | 11:03 |
| bauzas | but sometimes you need to find solutions that work easily | 11:03 |
| *** tbachman has joined #openstack-nova | 11:06 | |
| hachi | hello | 11:06 |
| bauzas | cdent: just one point | 11:06 |
| bauzas | cdent: I'm not sure I like https://github.com/openstack/nova/blob/master/nova/tests/functional/api/openstack/placement/test_report_client.py#L89-L92 | 11:06 |
| bauzas | cdent: because we are mocking an internal attribute here | 11:06 |
| hachi | how i can use the policy: | 11:06 |
| hachi | "owner" : "user_id:%(user_id)s" | 11:06 |
| hachi | in nova api 2.1 | 11:06 |
| bauzas | cdent: in case keystone changes its interface, we wouldn't have mock complaning at all | 11:07 |
| hachi | with newton release | 11:07 |
| bauzas | cdent: but then we would have huge problems | 11:07 |
| bauzas | cdent: I rather preferred to use the keystone public way of avoiding the SC | 11:07 |
| *** efoley_ has joined #openstack-nova | 11:07 | |
| cdent | keystoneauth1 has made a commitment to not change the interface on its public methods without changing the entire name of the package? | 11:08 |
| bauzas | but like I said previously, that looks a nit to me and we could discuss on that later on | 11:08 |
| bauzas | cdent: sure, we can trust them | 11:08 |
| bauzas | cdent: that's just riskier, that's it | 11:08 |
| bauzas | anyway | 11:08 |
| cdent | and even if it did break, we could just fix it when it broke: the tests will have done their job in that case | 11:09 |
| bauzas | I'm totally up discussing on that after FF :) | 11:09 |
| bauzas | probably we could accept not to fake our scheduler client methods and rather mock the Service object | 11:09 |
| bauzas | but I think we could still use the Server proxy to really run an wsgi server and not intercept | 11:10 |
| bauzas | but that's my thoughts - just coming along while tests are running on my local desktop and that I have a bit of time :) | 11:10 |
| *** efoley has quit IRC | 11:11 | |
| *** dikonoor has quit IRC | 11:11 | |
| cdent | Out of curiosity why do you prefer the server proxy? | 11:11 |
| *** dikonoor has joined #openstack-nova | 11:12 | |
| bauzas | cdent: because that's running our internals | 11:14 |
| cdent | What are "our internals"? | 11:15 |
| bauzas | internal piece of code | 11:15 |
| bauzas | and we're not emulating a socket, we *run* it | 11:15 |
| bauzas | which makes possible to debug the app outside of the test | 11:16 |
| *** pkoniszewski has quit IRC | 11:16 | |
| bauzas | but meh. | 11:16 |
| bauzas | anyway, back to work | 11:16 |
| *** andreas_s has joined #openstack-nova | 11:17 | |
| cdent | okay, but when the placement api runs for real, it doesn't use any of the nova Server() handling | 11:17 |
| *** ducttape_ has joined #openstack-nova | 11:22 | |
| openstackgerrit | Bob Ball proposed openstack/nova: Use physical utilisation for cached images https://review.openstack.org/353862 | 11:23 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: hyper-v: Adds support for VMs with nested virtualization https://review.openstack.org/398509 | 11:25 |
| cdent | bauzas: I've got a couple questions on the CoreFilter issue that are probably easier to track here than email. You got time to answer them now or should I just post a response to the email? | 11:26 |
| bauzas | cdent: just throw me a bit of them | 11:27 |
| cdent | In the world before Placement API existed, if I disabled the CoreFilter, that means that select_destinations could return some compute nodes that would reject a "placement" and cause a retry because it didn't have enough cores. Is that right? | 11:29 |
| *** tbachman has quit IRC | 11:30 | |
| *** krtaylor has quit IRC | 11:31 | |
| openstackgerrit | Alex Xu proposed openstack/nova: Limit the min length of string for integer JSON-Schema https://review.openstack.org/415103 | 11:31 |
| cdent | bauzus: If it is right, then asking the placement api for vcpu resources is simply short circuiting that retry and we should do that? | 11:31 |
| *** moshele has quit IRC | 11:31 | |
| johnthetubaguy | alex_xu: I would expect no policy.json file in a new install, empty I guess is OK too. | 11:31 |
| *** dikonoor has quit IRC | 11:31 | |
| alex_xu | johnthetubaguy: got it, thanks | 11:32 |
| johnthetubaguy | alex_xu: in the future I was thinking a fully commented out yaml file, a bit like what we do for config | 11:32 |
| alex_xu | johnthetubaguy: ok | 11:32 |
| *** krtaylor has joined #openstack-nova | 11:33 | |
| alex_xu | johnthetubaguy: I probably figure out a way to implement the 3 cases you pointed out. so I will work with Kevin_Zheng to update the patch. | 11:33 |
| johnthetubaguy | alex_xu: Kevin_Zheng: let me know how that goes, for me the warning message is an optional extra, as we could leave the old rules ill we can get that into olso.policy | 11:35 |
| *** rfolco has joined #openstack-nova | 11:36 | |
| *** tbachman has joined #openstack-nova | 11:38 | |
| bauzas | cdent: sorry, missed your question | 11:40 |
| bauzas | cdent: in a world where an operator disables the CoreFilter, we don't verify VCPUs | 11:40 |
| *** tbachman has quit IRC | 11:41 | |
| *** ducttape_ has quit IRC | 11:41 | |
| bauzas | cdent: most of them for example are disabiling the DiskFilter for the exact reason it doesn't count correctly | 11:41 |
| *** ducttape_ has joined #openstack-nova | 11:41 | |
| cdent | we don't verify vcpus at the scheduler level but do at the compute node level, or we don't very vcpus _anywhere_? | 11:41 |
| bauzas | cdent: if you don't run the CoreFilter, we don't return a limit | 11:41 |
| *** ducttape_ has quit IRC | 11:42 | |
| alex_xu | hachi: we didn't suggest to use user_id based policy check. That was added back to the code just for backward-compatible, it may be removed in the future | 11:42 |
| bauzas | cdent: if so, that's coming to the compute, and then we verify the claim without verifying the limit | 11:42 |
| *** ducttape_ has joined #openstack-nova | 11:42 | |
| bauzas | cdent: which means no allocation ratio | 11:42 |
| cdent | interesting | 11:42 |
| *** ducttape_ has quit IRC | 11:42 | |
| *** ducttape_ has joined #openstack-nova | 11:43 | |
| *** nicolasbock has joined #openstack-nova | 11:43 | |
| bauzas | cdent: so, I'm a bit afraid that a lot of operators have disabled the DiskFilter | 11:43 |
| *** ducttape_ has quit IRC | 11:43 | |
| bauzas | cdent: if now we check the disk size directly without having the aggregates created automatically for the RPs, then they could have problmes | 11:43 |
| cdent | so your idea is to remove resource classes from the request to /resource_providers based on which filters are disabled | 11:44 |
| *** ducttape_ has joined #openstack-nova | 11:44 | |
| bauzas | cdent: yup | 11:44 |
| bauzas | cdent: lemme upload the change this afternoon and you'll see | 11:44 |
| cdent | and the change would solely be on the nova-scheduler side | 11:44 |
| bauzas | yup | 11:44 |
| *** ducttape_ has quit IRC | 11:44 | |
| *** haplo37_ has quit IRC | 11:44 | |
| cdent | cool. It makes sense. My main concern was that we should change something on the placement server-side based on nova's config | 11:45 |
| bauzas | I just have a problem with our FakeDriver creating the same hypervisor hostname for all our computes | 11:45 |
| bauzas | so, once I fix that, I'll upload | 11:45 |
| *** dikonoor has joined #openstack-nova | 11:45 | |
| openstackgerrit | Chris Dent proposed openstack/nova: Optional separate database for placement API https://review.openstack.org/362766 | 11:46 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: Fix and add some notes to the cells v2 first time setup doc https://review.openstack.org/421421 | 11:49 |
| *** erlon has joined #openstack-nova | 11:49 | |
| *** karimb has joined #openstack-nova | 11:49 | |
| *** kevinz has joined #openstack-nova | 11:52 | |
| *** tiendc has quit IRC | 11:52 | |
| *** efoley__ has joined #openstack-nova | 11:53 | |
| openstackgerrit | Alvaro Lopez Garcia proposed openstack/nova: Ensure that periodic reclaim cleans DB deleted instances https://review.openstack.org/323250 | 11:55 |
| *** haplo37_ has joined #openstack-nova | 11:56 | |
| *** sree has quit IRC | 11:56 | |
| *** efoley_ has quit IRC | 11:57 | |
| *** sree has joined #openstack-nova | 11:59 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Fix device taging after API microversion 2.37 https://review.openstack.org/423952 | 12:00 |
| *** thorst_ has joined #openstack-nova | 12:03 | |
| *** sree has quit IRC | 12:04 | |
| *** thorst_ has quit IRC | 12:04 | |
| *** dave-mccowan has joined #openstack-nova | 12:05 | |
| *** diga has quit IRC | 12:09 | |
| *** ducttape_ has joined #openstack-nova | 12:12 | |
| *** mingyu has quit IRC | 12:15 | |
| *** amotoki has joined #openstack-nova | 12:15 | |
| *** ayogi has quit IRC | 12:17 | |
| *** mdnadeem has quit IRC | 12:19 | |
| *** mingyu has joined #openstack-nova | 12:20 | |
| *** karimb has quit IRC | 12:22 | |
| *** sdague has joined #openstack-nova | 12:22 | |
| *** ociuhandu has joined #openstack-nova | 12:22 | |
| *** prateek_ has joined #openstack-nova | 12:23 | |
| *** dave-mccowan has quit IRC | 12:25 | |
| *** prateek has quit IRC | 12:26 | |
| *** Guest17268 has quit IRC | 12:27 | |
| openstackgerrit | Wang Qing wu proposed openstack/nova: WIP(5): Introduce base PowerVM driver (ovs vif) https://review.openstack.org/422512 | 12:28 |
| *** bvanhav has joined #openstack-nova | 12:29 | |
| *** ducttape_ has quit IRC | 12:32 | |
| *** amotoki has quit IRC | 12:33 | |
| *** mdnadeem has joined #openstack-nova | 12:35 | |
| *** diga has joined #openstack-nova | 12:36 | |
| *** baoli has joined #openstack-nova | 12:38 | |
| *** mingyu has quit IRC | 12:39 | |
| openstackgerrit | Wang Qing wu proposed openstack/nova: WIP(5): Introduce base PowerVM driver (ovs vif) https://review.openstack.org/422512 | 12:39 |
| *** baoli has quit IRC | 12:39 | |
| *** Guest17268 has joined #openstack-nova | 12:40 | |
| *** ansiwen_ is now known as ansiwen | 12:43 | |
| *** iceyao has joined #openstack-nova | 12:44 | |
| *** dave-mccowan has joined #openstack-nova | 12:45 | |
| *** catintheroof has joined #openstack-nova | 12:45 | |
| openstackgerrit | Alex Szarka proposed openstack/nova: Transform instance-live_migration_abort notification https://review.openstack.org/420453 | 12:48 |
| *** patriciadomin_ is now known as patriciadomin | 12:49 | |
| *** catintheroof has quit IRC | 12:50 | |
| *** pkoniszewski has joined #openstack-nova | 12:50 | |
| *** psachin has quit IRC | 12:50 | |
| lyarwood | sfinucan: thanks for the review on the findmnt change, https://review.openstack.org/#/c/418371/ is another straight forward one adding a missing releasenote if you have time today | 12:55 |
| *** tbachman has joined #openstack-nova | 12:57 | |
| *** catintheroof has joined #openstack-nova | 12:58 | |
| *** tblakes has joined #openstack-nova | 12:58 | |
| snikitin | johnthetubaguy: Hi John! can you look at instance diagnostics implementation for Xen please? https://review.openstack.org/#/c/399613/ | 13:02 |
| *** markus_z has quit IRC | 13:03 | |
| *** tbachman_ has joined #openstack-nova | 13:04 | |
| *** tbachman has quit IRC | 13:04 | |
| *** tbachman_ is now known as tbachman | 13:04 | |
| *** kylek3h_away has quit IRC | 13:04 | |
| *** ducttape_ has joined #openstack-nova | 13:05 | |
| hachi | alex_xu thank you. | 13:06 |
| hachi | but i don't want my users within the same project do accidently delete the VM(s) of each other. | 13:06 |
| hachi | how i can do that without user_id policy ? | 13:07 |
| bhagyashris | cdent: Hi, | 13:09 |
| alex_xu | hachi: you can lock the instance | 13:11 |
| *** xyang1 has joined #openstack-nova | 13:11 | |
| cdent | bhagyashris: hi, what's up? | 13:14 |
| dims | sean-k-mooney : jaypipes : https://review.openstack.org/#/c/423718/ does the trick for getting os-vif's linuxbridge working | 13:14 |
| *** smatzek has joined #openstack-nova | 13:15 | |
| hachi | alex_xu thank you for the tip. I will try that. | 13:15 |
| amrith | andreykurilin, ping. I'm having a little issue with nova's 7.0.0 client, would you have a couple of minutes to help out? thx | 13:15 |
| amrith | one part of this is a change you submitted, https://review.openstack.org/#/c/412497/ | 13:16 |
| amrith | appears that something changed in 7.0.0 which I'm not quite grok'ing. pl ping me if you have a minute andreykurilin. thx | 13:17 |
| *** markus_z has joined #openstack-nova | 13:17 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add new policy for server list/detail with all_tenants https://review.openstack.org/415330 | 13:18 |
| *** edmondsw has joined #openstack-nova | 13:19 | |
| openstackgerrit | Julian proposed openstack/nova: Create cmd/script that prints JSON Schema of Notification Payload Classes https://review.openstack.org/411836 | 13:19 |
| alex_xu | hachi: np | 13:21 |
| *** jwcroppe has joined #openstack-nova | 13:25 | |
| *** amoralej is now known as amoralej|luch | 13:26 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add sort_key white list for server list/detail https://review.openstack.org/415142 | 13:27 |
| *** sofie has quit IRC | 13:28 | |
| *** owalsh is now known as owalsh-lunch | 13:29 | |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: WIP: Scheduler calling the Placement API https://review.openstack.org/417961 | 13:31 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Fix the FakeDriver using same hypervisor names https://review.openstack.org/424110 | 13:31 |
| bauzas | cdent: johnthetubaguy: dansmith: latest versions for the high-prio scheduler placement changes ^ | 13:32 |
| * cdent enqueues | 13:33 | |
| bauzas | cdent: you'll see now the conditional branch for checking the filters before calling the placement service | 13:33 |
| *** mdrabe has joined #openstack-nova | 13:34 | |
| bhagyashris | cdent: I have few questions in my mind regarding Scheduler calling the Placement API [1] https://review.openstack.org/#/c/417961. 1. As it updates allocation information at the placement db so is there a need of compute node table for resource tracking? | 13:34 |
| cdent | bhagyashris: the resource tracker and the nova-scheduler will carry on needing to know about compute nodes, yes. Is that what you're asking? The placement services itself will not need to know about the compute node table, just the uuids of those compute nodes, represented as resource providers in the placement api | 13:37 |
| *** satyar has joined #openstack-nova | 13:37 | |
| openstackgerrit | Mikhail Feoktistov proposed openstack/nova: libvirt: ephemeral disk support for virtuozzo containers https://review.openstack.org/312488 | 13:37 |
| *** lpetrut has quit IRC | 13:38 | |
| *** iceyao has quit IRC | 13:38 | |
| *** mingyu has joined #openstack-nova | 13:39 | |
| *** iceyao has joined #openstack-nova | 13:39 | |
| *** iceyao has quit IRC | 13:39 | |
| *** ducttape_ has quit IRC | 13:39 | |
| *** karimb has joined #openstack-nova | 13:40 | |
| *** kylek3h_away has joined #openstack-nova | 13:41 | |
| *** jpena is now known as jpena|lunch | 13:41 | |
| *** cleong has joined #openstack-nova | 13:41 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova: hyper-v: Do not allow instances with pinned CPUs to spawn https://review.openstack.org/424116 | 13:42 |
| *** mingyu has quit IRC | 13:44 | |
| *** nicolasbock has quit IRC | 13:46 | |
| *** mnestratov has joined #openstack-nova | 13:47 | |
| sfinucan | lyarwood: Done and done https://review.openstack.org/#/c/418371/1 | 13:47 |
| sfinucan | johnthetubaguy: ^ that looks like a straightforward +W - could you take a glance if you have a moment? | 13:47 |
| *** mfeoktistov has joined #openstack-nova | 13:48 | |
| lyarwood | sfinucan: cheers | 13:49 |
| *** jheroux has joined #openstack-nova | 13:50 | |
| bhagyashris | cdent: I know , i am asking lot of questions :) I just want to know if we are using the placement apis then will compute node table be needed in that case? If it will used then in which case it will be used and for what? | 13:50 |
| *** nicolasbock has joined #openstack-nova | 13:51 | |
| diga | jaypipes: Hi | 13:51 |
| cdent | bhagyashris: it will carry on being used for keeping track of the compute nodes because nova still needs to know about them | 13:51 |
| johnthetubaguy | sfinucan: ack, thanks | 13:51 |
| *** lpetrut has joined #openstack-nova | 13:51 | |
| cdent | diga: jaypipes is on PTO, maybe somebody else can help? | 13:51 |
| diga | cdent: Hi, sure | 13:52 |
| diga | cdent: https://review.openstack.org/#/c/423872/ can you review this spec - placement notification | 13:52 |
| cdent | yeah, sure, will stick that on my list for today | 13:53 |
| *** satyar has quit IRC | 13:53 | |
| *** kylek3h_away is now known as kylek3h | 13:54 | |
| diga | cdent: :) thank you! | 13:54 |
| diga | cdent: we have meeting now right ? | 13:54 |
| cdent | yeah, in five minutes | 13:54 |
| *** tlian has joined #openstack-nova | 13:54 | |
| diga | okay | 13:55 |
| edleafe | diga: in #openstack-meeting-alt | 13:55 |
| diga | edleafe: Sure, will join | 13:55 |
| *** vladikr has joined #openstack-nova | 13:55 | |
| *** mfeoktistov|2 has joined #openstack-nova | 13:55 | |
| andreykurilin | amrith: hi | 13:56 |
| amrith | hi andreykurilin | 13:56 |
| amrith | did you see the email I just sent you | 13:56 |
| andreykurilin | yes | 13:56 |
| johnthetubaguy | bauzas: you say only ironic needs the host field, isn't that what we use for the RPC call? | 13:56 |
| amrith | the issue I'm facing is that if I use the v6.0.0 client things work fine | 13:57 |
| *** gouthamr has joined #openstack-nova | 13:57 | |
| amrith | if I switch to the v7.0.0 client, it bombs out | 13:57 |
| bauzas | johnthetubaguy: I meant the problem is that host != node for Ironic | 13:57 |
| johnthetubaguy | bauzas: or is that the node vs host thing | 13:57 |
| bauzas | yup that | 13:57 |
| amrith | am not sure why that would be the case | 13:57 |
| johnthetubaguy | bauzas: ah, gotcha | 13:57 |
| *** artom has joined #openstack-nova | 13:57 | |
| bauzas | maybe my comment is unclear, sorry if so | 13:57 |
| andreykurilin | amrith: I have just several minutes before meeting and will able to help you in ~30 minutes | 13:57 |
| bauzas | but I can't say that I could just use the RP name | 13:57 |
| bauzas | given it's the nodename which can be different for ironic | 13:58 |
| amrith | andreykurilin, I'll be here. | 13:58 |
| andreykurilin | amrith: but now I have one question: What is user.tenant https://github.com/openstack/trove/blob/master/trove/tests/util/__init__.py#L170 here? Is it tenant name or id? | 13:58 |
| bauzas | while the other drivers (except the Fake driver, but that's fixed in a bottom patch) are using host == node | 13:58 |
| bauzas | johnthetubaguy: ^ | 13:58 |
| amrith | looking | 13:58 |
| *** nicolasbock has quit IRC | 13:58 | |
| *** amoralej|luch is now known as amoralej | 13:58 | |
| johnthetubaguy | bauzas: the custom resource class stuff should fix that, I am assuming | 13:59 |
| bauzas | johnthetubaguy: yeah, but it could be merged by Pike | 14:00 |
| bauzas | johnthetubaguy: hence me using the new object method I created | 14:00 |
| *** kevinz has quit IRC | 14:00 | |
| bauzas | we could just cut that once the above BP you mentioned is fully merged | 14:00 |
| *** AAR has joined #openstack-nova | 14:00 | |
| bauzas | but AFAICS, that's still not done yet hence my big fat comment | 14:01 |
| johnthetubaguy | bauzas: yeah, no biggie, just getting my head straight on that | 14:01 |
| *** AAR has left #openstack-nova | 14:01 | |
| bauzas | np | 14:01 |
| bauzas | johnthetubaguy: I have a bigger concern that I wrote in a ML thread this week-end | 14:01 |
| edleafe | bauzas: scheduler meeting? | 14:02 |
| bauzas | edleafe: I can do, for 20 mins | 14:02 |
| edleafe | cool | 14:02 |
| amrith | andreykurilin, one second, you are looking at the tests. I wasn't running test code | 14:02 |
| bauzas | edleafe: but I'm pretty down given all the work I'm doing atm | 14:02 |
| andreykurilin | ok... | 14:02 |
| amrith | I was running a straight trove command which tried to hit nova | 14:02 |
| amrith | I will find out what that is | 14:02 |
| andreykurilin | I just look at first failure at gates:) | 14:02 |
| andreykurilin | *looked | 14:03 |
| amrith | but for what I've been trying to do, the code is at https://github.com/openstack/trove/blob/master/trove/common/remote.py#L101 | 14:03 |
| *** bvanhav_ has joined #openstack-nova | 14:03 | |
| amrith | ah, sorry | 14:03 |
| amrith | I am running trove locally to debug | 14:03 |
| *** bvanhav has quit IRC | 14:03 | |
| amrith | here, we're making a client with version 2.12 | 14:03 |
| amrith | which comes from https://github.com/openstack/trove/blob/master/trove/common/cfg.py#L67 | 14:03 |
| amrith | and we're passing in a bunch of parameters | 14:04 |
| amrith | my specific issue (and like I said in the email, trove may be doing something wrong here) | 14:04 |
| amrith | we're making the call like https://github.com/openstack/trove/blob/master/trove/common/remote.py#L101 | 14:04 |
| amrith | and if I understand compatibility right, this should work unchanged with the v7.0.0 client from nova | 14:04 |
| andreykurilin | amrith: need to go to meeting, sorry. PS: it should help(with depends-on) ttps://review.openstack.org/412497 | 14:04 |
| amrith | but it isn't | 14:04 |
| amrith | nope, doesn't help | 14:05 |
| amrith | tried it | 14:05 |
| amrith | something else is broken, not sure where | 14:05 |
| amrith | the issue I'm facing is that nova returns a 403 (forbidden) on a simple thing like flavor list | 14:05 |
| andreykurilin | amrith: I added depends-on there | 14:05 |
| bhagyashris | cdent: Ok. Thank you. :) | 14:06 |
| andreykurilin | amrith: did you check it with pulling not-merged changed from novcliane? | 14:06 |
| amrith | oh, sorry | 14:06 |
| amrith | different change | 14:06 |
| amrith | let me look | 14:06 |
| amrith | NO, I did not check with pulling not merged change in novaclient | 14:06 |
| amrith | that may be it. let me check | 14:06 |
| *** nicolasbock has joined #openstack-nova | 14:06 | |
| amrith | catch yo after your meeting | 14:06 |
| amrith | thanks! | 14:06 |
| alex_xu | johnthetubaguy: sdague, hope you can take a look at this patch https://review.openstack.org/#/c/423952/5, we break some old microversion in newton release. cdent have a good point on it. I'm not sure we can fix it directly, maybe another microversion? | 14:07 |
| johnthetubaguy | alex_xu: yeah, sounds like it should be added back in a the next microversion | 14:08 |
| *** thorst_ has joined #openstack-nova | 14:08 | |
| *** esberglu has joined #openstack-nova | 14:09 | |
| *** mfeoktistov has left #openstack-nova | 14:09 | |
| alex_xu | johnthetubaguy: yea, we already bump few versions after break that. | 14:10 |
| *** mfeoktistov|2 has left #openstack-nova | 14:10 | |
| *** cjvolzka has joined #openstack-nova | 14:10 | |
| *** amotoki has joined #openstack-nova | 14:11 | |
| *** mfeoktistov has joined #openstack-nova | 14:12 | |
| *** lpetrut has quit IRC | 14:13 | |
| *** jamesdenton has joined #openstack-nova | 14:13 | |
| *** moshele has joined #openstack-nova | 14:13 | |
| *** jerrygb has joined #openstack-nova | 14:14 | |
| sdague | alex_xu: so, before fixing it, is there something we can learn about how to not let something like that fall through review again? | 14:14 |
| sdague | no one seemed to catch that over a lot of review iterations (64 revisions on that patch, that bug showed up in revision 44) | 14:15 |
| *** ducttape_ has joined #openstack-nova | 14:15 | |
| sdague | https://review.openstack.org/#/c/304510/44/nova/api/openstack/compute/block_device_mapping.py | 14:15 |
| *** jerrygb_ has joined #openstack-nova | 14:15 | |
| diana_clarke | amrith, andreykurilin: Horizon is reporting a similar issue with python-novaclient 7.0.0. https://bugs.launchpad.net/python-novaclient/+bug/1658148 | 14:15 |
| openstack | Launchpad bug 1658148 in OpenStack Dashboard (Horizon) "Horizon/Nova interactions broken " [Critical,New] - Assigned to Rob Cresswell (robcresswell) | 14:15 |
| alex_xu | sdague: i'm thinking is there any test can cover that. we probably only run the test on specific version, not in the future version. | 14:16 |
| alex_xu | but even the test can run in the future version, but the bug only can be found when we bump a new version | 14:17 |
| *** ducttape_ has quit IRC | 14:17 | |
| *** jerrygb__ has joined #openstack-nova | 14:18 | |
| sdague | alex_xu: right, I was wonder if we could iterate through the schema and compare at various versions | 14:18 |
| *** jerrygb has quit IRC | 14:19 | |
| sdague | including a fake new version after the one being committed | 14:19 |
| sdague | and if we see a blip and back, that's probably wrong | 14:19 |
| andreykurilin | amrith: ok, I returned | 14:19 |
| *** baoli has joined #openstack-nova | 14:19 | |
| bauzas | jaypipes: I thought you were on PTO | 14:19 |
| alex_xu | sdague: yea, that should works | 14:19 |
| *** owalsh-lunch has quit IRC | 14:20 | |
| *** hongbin has joined #openstack-nova | 14:20 | |
| alex_xu | sdague: actually that patch fixes two bugs, another one break the old microversion in the later patch. | 14:20 |
| *** jerrygb_ has quit IRC | 14:20 | |
| alex_xu | maybe we should run the api sample tests on the specific version and the lastest version. it is fine to find bug before we bump next version | 14:21 |
| sdague | alex_xu: yeh | 14:22 |
| sdague | that might be good enough as well, run everything at n+1 | 14:22 |
| sdague | I don't know, just trying to figure out how to make it easier for folks | 14:23 |
| sdague | honestly, == is probably always the wrong answer for microversion comparison | 14:23 |
| alex_xu | yes, that is totally wrong. | 14:24 |
| *** owalsh-lunch has joined #openstack-nova | 14:25 | |
| *** amotoki has quit IRC | 14:25 | |
| sdague | if version stops being a string, and is actually a version object when it gets to get_server_create_version then it would be harder to make it an equality | 14:26 |
| *** lpetrut has joined #openstack-nova | 14:26 | |
| *** owalsh-lunch is now known as owalsh | 14:26 | |
| sdague | version.between('2.32', None) | 14:26 |
| *** ducttape_ has joined #openstack-nova | 14:27 | |
| sdague | if you really want an equal, then do between('2.32','2.32') | 14:27 |
| sdague | that will be a lot more obvious to reviewers that it is wrong | 14:27 |
| sdague | that might be more effective than complicated testing | 14:28 |
| *** catinthe_ has joined #openstack-nova | 14:29 | |
| *** abalutoiu_ has joined #openstack-nova | 14:30 | |
| *** smatzek has quit IRC | 14:31 | |
| *** mdnadeem has quit IRC | 14:31 | |
| *** ducttape_ has quit IRC | 14:31 | |
| *** voelzmo has quit IRC | 14:32 | |
| *** catintheroof has quit IRC | 14:32 | |
| *** dikonoor has quit IRC | 14:32 | |
| *** abalutoiu has quit IRC | 14:33 | |
| amrith | hi andreykurilin | 14:34 |
| amrith | let's watch the change you proposed (with the dependency) | 14:34 |
| amrith | and go from there | 14:34 |
| amrith | I'm looking at the test you mentioned | 14:34 |
| amrith | which also failed | 14:34 |
| amrith | last time around | 14:34 |
| *** sridharg has quit IRC | 14:35 | |
| andreykurilin | amrith: actually, it looks like I have a workaround (depends-on patch looks like not ready yet and require some additional work). I'll propose one more change | 14:35 |
| amrith | ok, thx andreykurilin | 14:35 |
| *** diga has quit IRC | 14:36 | |
| *** voelzmo has joined #openstack-nova | 14:36 | |
| *** moshele has quit IRC | 14:36 | |
| alex_xu | sdague: yea, we can pass the object instead of string. but the object support __eq__ :) | 14:36 |
| *** satyar has joined #openstack-nova | 14:38 | |
| *** voelzmo has quit IRC | 14:38 | |
| *** hongbin has quit IRC | 14:39 | |
| *** hongbin has joined #openstack-nova | 14:40 | |
| alex_xu | johnthetubaguy: sdague, so that bug need to be fixed by microversion, is that something we can done before release, or it will be a Pike thing. And have a reno 'issue' note in Ocata? | 14:40 |
| *** cdent has quit IRC | 14:40 | |
| *** mingyu has joined #openstack-nova | 14:40 | |
| *** liverpooler has joined #openstack-nova | 14:41 | |
| sdague | alex_xu: I think we should make the object stop supporting eq natively | 14:41 |
| johnthetubaguy | I am with sdague on stopping the route cause here urgently | 14:42 |
| alex_xu | sdague johnthetubaguy : ok, I will check that tomorrow | 14:42 |
| sdague | alex_xu: so, what happens when 2.33 is used? | 14:42 |
| andreykurilin | amrith: what is context.token here https://github.com/openstack/trove/blob/master/trove/common/remote.py#L102 ?) | 14:43 |
| *** slaweq_ has joined #openstack-nova | 14:44 | |
| *** mingyu has quit IRC | 14:45 | |
| alex_xu | sdague: emm...looks like there is no 'addtionalProperties: False', that means the tag still work in the 2.33, just lose validation, I need to check that | 14:45 |
| *** liverpooler has quit IRC | 14:45 | |
| *** eharney has joined #openstack-nova | 14:45 | |
| *** xinliang has quit IRC | 14:45 | |
| *** liverpooler has joined #openstack-nova | 14:45 | |
| jroll | edleafe: one more question, did we figure out what an ironic flavor using resource classes looks like yet? | 14:45 |
| jroll | jaypipes: ^^ | 14:45 |
| sdague | alex_xu: it would be extremely useful to me to get a mailing list post out about what went wrong, what this allows, and what the fall out from various fixing options would be | 14:45 |
| sdague | because otherwise everyone needs to build that up in their own head from scratch, and it's hard to agree on path forward unless we all agree where we stand | 14:46 |
| *** mingyu has joined #openstack-nova | 14:46 | |
| edleafe | jroll: good question | 14:46 |
| alex_xu | sdague: i see now. I will investigate more, then send out an email | 14:47 |
| jroll | edleafe: I'd like to have that in my docs for the ironic install guide, I guess | 14:47 |
| jroll | though, if it doesn't work until pike, idk | 14:47 |
| edleafe | jroll: I'm still not clear on whether we need to add the resource class to the flavor | 14:47 |
| jroll | edleafe: how else do we map a flavor to said resource class? | 14:47 |
| edleafe | jroll: or if there is some other mapping | 14:47 |
| sdague | alex_xu: thank you | 14:47 |
| edleafe | jroll: jinx | 14:47 |
| alex_xu | sdague: np | 14:48 |
| *** jpena|lunch is now known as jpena | 14:48 | |
| edleafe | jroll: my understanding is that the flavor will be: CUSTOM_IRONIC_CLASS: 1 | 14:48 |
| * alex_xu also learned the way to resolve problem. | 14:48 | |
| edleafe | jroll: and that placement will find a provider for that class | 14:49 |
| jroll | edleafe: where, in the flavor? e.g. how do I create said flavor? | 14:49 |
| edleafe | jroll: IOW, no disk/ram/cpu BS like now | 14:49 |
| openstackgerrit | Pawel Koniszewski proposed openstack/nova: live-mig: Add claims and proper resource tracking https://review.openstack.org/244489 | 14:50 |
| openstackgerrit | Pawel Koniszewski proposed openstack/nova: libvirt: pass libvirt reference methods to migration https://review.openstack.org/328997 | 14:50 |
| openstackgerrit | Pawel Koniszewski proposed openstack/nova: libvirt: live-migrate updates NUMA and cpus in the XML https://review.openstack.org/286744 | 14:50 |
| edleafe | jroll: well, like any other flavor, but not sure what changes the custom RC stuff will need | 14:50 |
| jroll | edleafe: sure, so just nova flavor-key my-flavor set CUSTOM_FOO_THING=1? | 14:50 |
| jroll | right | 14:50 |
| edleafe | jroll: jaypipes had all this on his mental whiteboard | 14:50 |
| jroll | edleafe: so here's the thing, if we require this to be set up for a smooth upgrade to pike, then we need to be able to make a working flavor in ocata | 14:51 |
| edleafe | jroll: agreed | 14:51 |
| jroll | edleafe: and we're three days from feature freeze and this is but a butterfly in jaypipes' brain, correct? | 14:51 |
| edleafe | jroll: unless that butterfly laid some eggs in cdent's brain, yet | 14:52 |
| *** eharney has quit IRC | 14:52 | |
| edleafe | s/yet/yes | 14:52 |
| *** udesale has joined #openstack-nova | 14:52 | |
| edleafe | jroll: He may have written it up somewhere - I can dig for it | 14:52 |
| jroll | edleafe: I mean, if it isn't written down at this point, I don't believe it will happen, which means this can't be mandatory fora pike upgrade | 14:52 |
| jroll | okm, | 14:52 |
| jroll | ok* | 14:52 |
| jroll | thanks | 14:53 |
| jroll | all I've seen is maybe an extra spec | 14:53 |
| edleafe | jroll: I'll let you know what I find | 14:53 |
| jroll | edleafe: thank you | 14:53 |
| *** dimtruck is now known as zz_dimtruck | 14:54 | |
| *** zz_dimtruck is now known as dimtruck | 14:54 | |
| *** ducttape_ has joined #openstack-nova | 14:55 | |
| *** tbachman has quit IRC | 14:55 | |
| *** jaosorior has joined #openstack-nova | 14:56 | |
| sfinucan | vladikr: Comments left on https://review.openstack.org/#/c/421186/2 | 14:56 |
| *** smatzek has joined #openstack-nova | 14:57 | |
| *** xinliang has joined #openstack-nova | 14:57 | |
| *** smatzek_ has joined #openstack-nova | 14:58 | |
| vladikr | sfinucan, thanks | 14:58 |
| *** _ducttape_ has joined #openstack-nova | 14:58 | |
| *** tbachman has joined #openstack-nova | 14:58 | |
| *** Tahvok has joined #openstack-nova | 14:59 | |
| amrith | andreykurilin, the token there is something that the client (trove client) would have created and passed to the server. so the auth and token creation is done on the trove client side and the API service (or TM) on the trove control plane uses those credentials. That would be a keystone token. | 15:00 |
| *** hongbin has quit IRC | 15:00 | |
| *** hongbin_ has joined #openstack-nova | 15:00 | |
| andreykurilin | amrith: oh, sorry. I wrote the wrong question :( it should be : what is tenant there | 15:01 |
| amrith | project_id | 15:01 |
| dansmith | sdague: sfinucan: johnthetubaguy: bauzas: The two patches at the head of this series have been +1 from jenkins for a while and have seen little review. They're not super complicated and I'd really like to see them get in while we work on the third, which is the major goal for ocata for cells: https://review.openstack.org/#/c/396775 | 15:01 |
| *** smatzek has quit IRC | 15:01 | |
| *** ducttape_ has quit IRC | 15:01 | |
| andreykurilin | amrith: nice | 15:02 |
| bauzas | dansmith: okay, I can take a look now | 15:02 |
| dansmith | thanks | 15:02 |
| sfinucan | dansmith: Likewise | 15:02 |
| *** _ducttape_ has quit IRC | 15:03 | |
| *** dimtruck is now known as zz_dimtruck | 15:03 | |
| dansmith | also thanks :) | 15:04 |
| *** eharney has joined #openstack-nova | 15:06 | |
| ildikov | johnthetubaguy Hi, Can you make it to the meeting today? | 15:06 |
| *** hongbin_ has quit IRC | 15:08 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova: scheduler: Don't modify RequestSpec.numa_topology https://review.openstack.org/419479 | 15:12 |
| *** voelzmo has joined #openstack-nova | 15:13 | |
| sfinucan | dansmith, sahid: If ye could take another look at ^, I'd be mighty appreciative | 15:13 |
| bauzas | dansmith: comments in your bottom change | 15:13 |
| dansmith | bauzas: thanks | 15:14 |
| dansmith | sfinucan: yep | 15:14 |
| bauzas | dansmith: just wanted to make sure we have a limit but not a marker, right? | 15:15 |
| dansmith | bauzas: the marker is handled in that code | 15:16 |
| bauzas | dansmith: mmm? lemme look, I maybe missed it | 15:17 |
| dansmith | bauzas: we pass the marker down to all the sub-list commands and honor the limit at the top | 15:17 |
| *** mtanino has joined #openstack-nova | 15:19 | |
| *** voelzmo has quit IRC | 15:19 | |
| *** knikolla has joined #openstack-nova | 15:20 | |
| *** knikolla has quit IRC | 15:20 | |
| bauzas | dansmith: I'm seeing it now | 15:20 |
| *** knikolla has joined #openstack-nova | 15:20 | |
| bauzas | dansmith: but I actually wonder how it works with 2 cells not having the same number of instances in there | 15:20 |
| bauzas | dansmith: but lemme look at that | 15:20 |
| dansmith | bauzas: how what works? | 15:20 |
| bauzas | my brain is just fried by the scheduler upcall so I could be misunderstanding | 15:21 |
| bauzas | dansmith: so, you construct a global limit and you pass it down to the cell-related instance.get_all() call | 15:21 |
| bauzas | with leaving the marker, right? | 15:21 |
| dansmith | the global limit is just to make sure that we don't keep querying cells for more instances | 15:22 |
| dansmith | so if you limit=100 and we return 90 from the first cell, then the second cell should only fetch 10, and the third cell 0 | 15:22 |
| dansmith | sfinucan: why no test on that filter change/ | 15:23 |
| bauzas | dansmith: right, I understood that | 15:24 |
| sfinucan | dansmith: Because I couldn't figure out where to slot it in 🙈 | 15:24 |
| bauzas | I'm thinking of the limit=50 and cell1 having 60 | 15:24 |
| *** zz_dimtruck is now known as dimtruck | 15:24 | |
| dansmith | bauzas: then we'll limit=50 on the first cell, and then limit will be zero afterwards, an we will never call the next one | 15:25 |
| bauzas | in that case, we get the first 50 elements and we set the marker to the 50th injstance | 15:25 |
| dansmith | bauzas: and return 50 instances | 15:25 |
| bauzas | right | 15:25 |
| bauzas | so, next call would get the cell1 marker to the 50th instance, so a new limit of 50 would return the 10 left + 40 from cell2 if we have enough, right? | 15:25 |
| bauzas | I think I see the logic | 15:26 |
| bauzas | we consume all instances from cell to cell | 15:26 |
| bauzas | and the marker is just related to the cell currently half-checked | 15:26 |
| dansmith | right | 15:26 |
| bauzas | dansmith: because there is huge assumption that we get once the cells, and we don't get again | 15:28 |
| dansmith | bauzas: I'm not sure what you mean | 15:28 |
| bauzas | dansmith: you only set your global cells value once, right? | 15:28 |
| dansmith | bauzas: you mean we never re-query the list of cells currently | 15:28 |
| *** mingyu has quit IRC | 15:29 | |
| dansmith | ? | 15:29 |
| bauzas | yeah, your TODO | 15:29 |
| bauzas | I mean your FIXME sorry | 15:29 |
| jroll | edleafe: line 70 here is the major reason I need to figure out what a flavor looks like https://review.openstack.org/#/c/345081/6/install-guide/source/include/configure-nova-flavors.rst | 15:29 |
| *** mingyu has joined #openstack-nova | 15:29 | |
| dansmith | bauzas: that's correct -- what does that have to do with the limit/marker thing? | 15:29 |
| *** prateek_ has quit IRC | 15:29 | |
| bauzas | dansmith: because we could imagine that we look over 3 cells | 15:29 |
| *** baoli has quit IRC | 15:30 | |
| bauzas | have the marker in the middle of the 2nd cell | 15:30 |
| bauzas | and then all instances newly created in cell1 would never appear | 15:30 |
| bauzas | in a further call | 15:30 |
| edleafe | jroll: yeah, I see that | 15:30 |
| dansmith | sfinucan: test_numa_topology_filters.py ? | 15:30 |
| edleafe | jroll: looking through the flavor docs, they are super-VM centric. So I think extra-specs will have to be used for now | 15:31 |
| *** ducttape_ has joined #openstack-nova | 15:31 | |
| dansmith | bauzas: that's the sorting problem we've already identified, right? either way, pagination is always lossy and if you are sorted on something like uuid you'd not see those new instances even in a single cell currently | 15:31 |
| edleafe | jroll: the other resource classes are hard-coded into flavors | 15:31 |
| jroll | edleafe: bleh | 15:32 |
| edleafe | jroll: which I wanted flavors to DIAFF long ago | 15:32 |
| edleafe | s/which/which is | 15:32 |
| bauzas | dansmith: sure, I just wanted to make sure we are at the same page | 15:32 |
| jroll | edleafe: now I'm wondering how nova behaves if nothing is set but resource classes :P | 15:32 |
| jroll | I guess this is why we CI | 15:32 |
| * jroll continues working on that | 15:32 | |
| dansmith | bauzas: yes, we're on the same marker :) | 15:32 |
| edleafe | jroll: cdent is away ATM, but I'll pick his brain when he gets back | 15:33 |
| openstackgerrit | Maciej Szankin proposed openstack/nova: conf: Resolve TODOs in 'database' https://review.openstack.org/393695 | 15:33 |
| jroll | edleafe: yeah, fair, thanks | 15:33 |
| *** ekuris_ has quit IRC | 15:33 | |
| *** beekneemech is now known as bnemec | 15:33 | |
| bauzas | dansmith: since you're working on a new PS, maybe writing this scenario into a unittest could be nice to me | 15:34 |
| bauzas | dansmith: like, create 3 cells, iterate over 1 and then the 2nd, add a new instance to cell1 and make sure that it's never read | 15:34 |
| bauzas | dansmith: because I think that it could be something important for operators wanting to have multi-cells | 15:35 |
| bauzas | and they could come to us and say 'why can't I see my new instances?" | 15:35 |
| dansmith | bauzas: ...really? | 15:35 |
| bauzas | dansmith: well, at least documenting it then | 15:35 |
| dansmith | bauzas: but it's a marker/limit query and that situation exists today | 15:35 |
| bauzas | something saying that we know our limitations | 15:35 |
| dansmith | but it's exactly the way it behaves today :) | 15:36 |
| sfinucan | So I'm clear, the marker is applied before the limit, right? | 15:37 |
| bauzas | dansmith: okay then nevermind | 15:37 |
| *** cdent has joined #openstack-nova | 15:37 | |
| vladikr | dansmith, I have reworked the "sriov-pf-passthrough-neutron-port-vlan" series according to your suggestion to get the vlan from the info_cache, could you please take a look when you have the time, before it hits the code freeze ? https://review.openstack.org/#/q/topic:bp/sriov-pf-passthrough-neutron-port-vlan | 15:37 |
| sfinucan | I'm curious what would happen if there was nothing matching the marker in the first cell but we hadn't indexed any others yet. I'm guessing '_get_instances_by_filters' will return nothing and limit wouldn't be deprecated, but just making sure :) | 15:37 |
| *** udesale has quit IRC | 15:37 | |
| *** voelzmo has joined #openstack-nova | 15:37 | |
| *** efoley has joined #openstack-nova | 15:40 | |
| *** efoley__ has quit IRC | 15:41 | |
| cdent | jroll, edleafe: re [t 22sr] no catepillars, just maggots | 15:41 |
| purplerbot | <edleafe> jroll: unless that butterfly laid some eggs in cdent's brain, yet [2017-01-23 14:52:19] [n 22sr] | 15:41 |
| *** baoli has joined #openstack-nova | 15:41 | |
| jroll | :| | 15:41 |
| edleafe | cdent: so extra_specs is all we got? | 15:41 |
| cdent | I vaguely recall this being discussed in a review and that extra_specs was the way to go for now because we hadn't had a chance to kill flavors | 15:42 |
| edleafe | cdent: and do we have the code in place to convert 'resource_class' in extra_specs to an actual ironic request? | 15:42 |
| cdent | but I wouldn't want to be held to that version of the truth | 15:42 |
| *** sdague has quit IRC | 15:43 | |
| edleafe | god, extra_specs is some special kind of suck | 15:43 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova: Transform instance.trigger_crash_dump notification https://review.openstack.org/396225 | 15:43 |
| cdent | it doesn't look like it, but my understanding was that for this cycle we weren't planning to. all we were caring about was getting the inventory right, not using it (in the ironic case) | 15:44 |
| amrith | andreykurilin, looks like https://review.openstack.org/#/c/423086/ is trying to fix something related to what you are addressing; I've set that one to depend on your change; let's see how all three together do | 15:44 |
| edleafe | cdent: so what jroll needs is to be able to write the guidance for ironic operators for ocata so that they are in a good place when pike rolls out | 15:45 |
| dansmith | sfinucan: ah, we probably need to skip NotFound if we have a marker and limit!=0 | 15:45 |
| cdent | edleafe, jroll I'm sorry I don't know more. My devious plan to get people to correct me when I'm wrong or leave things out in the rp updates has been a fail | 15:45 |
| *** jerrygb has joined #openstack-nova | 15:45 | |
| dansmith | sfinucan: that's what the intent is, otherwise the marker would always have to be in cell1 | 15:45 |
| dansmith | which wouldn't be the case after we can have multiple | 15:45 |
| jroll | cdent: and my plan to make sure that's in a good place has been undermined by a million other things to do | 15:45 |
| cdent | plans suck, or something | 15:46 |
| * jroll is so excited that there's only a couple of weeks until I can mostly just write code | 15:46 | |
| andreykurilin | amrith: it should fix only tests... | 15:46 |
| *** mingyu has quit IRC | 15:46 | |
| amrith | andreykurilin, I don't even understand that test | 15:46 |
| sfinucan | jroll: And review all the code, like a good core :) | 15:46 |
| jroll | sfinucan: I'll think about it | 15:47 |
| jroll | :) | 15:47 |
| amrith | why it makes sense for a test to inspect some internal attributes of the nova client object I know not | 15:47 |
| sfinucan | heh :) | 15:47 |
| andreykurilin | amrith: heh | 15:47 |
| amrith | seems like that test would be more appropriate if it were in nova | 15:47 |
| amrith | but this is a test in trove :) | 15:47 |
| *** jerrygb__ has quit IRC | 15:47 | |
| amrith | the fix seems to correct for the fact that the variable is no longer context.user but why should trove care? | 15:47 |
| amrith | my 2c worth | 15:47 |
| cdent | jroll, edleafe: I reckon for the guideance the thing to do at this stage is establish a pattern of people setting the resource class on the ironic side so that the inventory setting, established in this patch: https://review.openstack.org/#/c/404472/ can happen | 15:48 |
| amrith | let's see if it passes tests; I'll fix the test a different way if it does and dep on your change | 15:48 |
| jroll | cdent: I agree, I just don't think we can require that for a pike upgrade, if people can't transition their flavors in ocata | 15:48 |
| cdent | Unfortunately I can't really comment on that: I don't have enough of a picture of all the pieces | 15:49 |
| *** bvanhav_ is now known as bvanhav | 15:49 | |
| edleafe | jroll: I can't imaging flavors being nuked any time soon - they're just too entrenched to kill without splattering their guts all over | 15:50 |
| edleafe | jroll: so I would recommend going the extra_specs route for the foreseeable future | 15:50 |
| edleafe | jroll: it sucks bigly, but that's all we have to work with | 15:50 |
| jroll | edleafe: if we're only tracking resources the new way (as we just said we wanted to in pike), then the existing flavors (which include cpu/ram/gb) won't work. which means we need to be able to make new flavors (with only resource classes) in ocata, and those need to work (I think?). | 15:51 |
| *** mriedem has joined #openstack-nova | 15:52 | |
| *** mfeoktistov has quit IRC | 15:53 | |
| mriedem | buenos dias | 15:53 |
| *** yamahata has joined #openstack-nova | 15:54 | |
| *** jose-phillips has joined #openstack-nova | 15:55 | |
| edleafe | mriedem: that's not how you vacation! | 15:55 |
| mriedem | it's how *i* vacation | 15:55 |
| ildikov | mriedem: hi | 15:56 |
| ildikov | mriedem: are you on vacation today? | 15:56 |
| *** hongbin has joined #openstack-nova | 15:56 | |
| mriedem | ildikov: yes | 15:56 |
| *** a-pugachev has quit IRC | 15:56 | |
| openstackgerrit | Maciej Szankin proposed openstack/nova: conf: Resolve TODOs in 'database' https://review.openstack.org/393695 | 15:57 |
| ildikov | mriedem: oh nice, then I'll not harass you to come to the meeting :) | 15:57 |
| ildikov | mriedem: BTW, would any other day at the same time work for you in general? | 15:57 |
| ildikov | mriedem: there are a few folks with collisions so I'm trying to find another slot that can work | 15:57 |
| *** _ducttape_ has joined #openstack-nova | 15:57 | |
| ildikov | mriedem: no other questions before saying Happy Vacations! :) | 15:58 |
| *** burgerk has joined #openstack-nova | 15:59 | |
| mriedem | ildikov: the time slot itself in general works for me | 16:00 |
| ildikov | mriedem: great, thanks! | 16:00 |
| *** ducttape_ has quit IRC | 16:00 | |
| *** efoley_ has joined #openstack-nova | 16:01 | |
| *** nklenke has joined #openstack-nova | 16:02 | |
| *** ratailor has joined #openstack-nova | 16:02 | |
| *** efoley has quit IRC | 16:05 | |
| *** amotoki has joined #openstack-nova | 16:06 | |
| *** Jeffrey4l_ has quit IRC | 16:06 | |
| *** Jeffrey4l_ has joined #openstack-nova | 16:06 | |
| *** voelzmo has quit IRC | 16:07 | |
| cdent | edleafe: since you committed to rebasing the nested resource providers, shall I do the customer resource classes? https://review.openstack.org/#/q/topic:bp/custom-resource-classes+status:open | 16:09 |
| edleafe | cdent: I was gonna do them both, but if you're bored... | 16:09 |
| cdent | edleafe: go for it, definitely not bored | 16:10 |
| *** gouthamr has quit IRC | 16:12 | |
| BobBall | Is there a way to set per-driver defaults for config values? For example, ram_allocation_ratio defaults to 1.5 - but this does not make sense for XenServer since the driver doesn't enable over-commit. | 16:12 |
| *** gouthamr has joined #openstack-nova | 16:12 | |
| mriedem | gibi: http://logs.openstack.org/45/403745/11/gate/gate-nova-tox-db-functional-ubuntu-xenial/76b1646/console.html#_2017-01-23_15_02_47_382251 | 16:13 |
| *** markus_z has quit IRC | 16:13 | |
| mriedem | BobBall: nope | 16:13 |
| sfinucan | BobBall: Not that I'm aware off, I'm afraid. That would probably be something for your deployment tool | 16:13 |
| sfinucan | *of | 16:13 |
| BobBall | Well, poohsticks. | 16:13 |
| mriedem | BobBall: i'd expect different virt drivers to be configured on different compute hosts | 16:13 |
| *** pcaruana has quit IRC | 16:13 | |
| BobBall | Of course - admins can set ram_allocation_ratio in the config file, I was just trying to reduce their work by making the defaults make sense :) | 16:14 |
| mriedem | BobBall: well...you could possibly do something fancy on startup if you know which virt driver is used and if the user has explicitly set the config rather than just picking up the default, | 16:15 |
| mriedem | i.e. change the default in code based on which driver is used at runtime | 16:15 |
| mriedem | but i don't think there is anything in oslo.config to tell you if the option was loaded from config or defauls | 16:16 |
| jroll | mriedem: BobBall: couldn't you use set_default? http://docs.openstack.org/developer/oslo.config/configopts.html#oslo_config.cfg.ConfigOpts.set_default | 16:17 |
| *** mvk has quit IRC | 16:18 | |
| *** jose-phillips has quit IRC | 16:18 | |
| mriedem | jroll: yeah maybe when loading up the compute driver | 16:20 |
| bauzas | f****, we have a shit ton of tests that are having 'fake-mini' as hypervisor_hostname | 16:20 |
| bauzas | :( | 16:20 |
| *** tbachman has quit IRC | 16:20 | |
| *** tbachman_ has joined #openstack-nova | 16:20 | |
| bauzas | cdent: mriedem: dansmith: I think we have a very important problem with our unique constraint about the RP name being unique :/ | 16:21 |
| *** moshele has joined #openstack-nova | 16:21 | |
| jroll | mriedem: oh, that config is in the scheduler process, isn't it, maybe not then | 16:21 |
| *** nklenke has quit IRC | 16:21 | |
| cdent | bauzas: do tell? | 16:21 |
| erlon | andreykurilin: hey andrey, I have found out that service_catalog is not part of novaclient anymore | 16:22 |
| erlon | andreykurilin: how suppose a client to use it now? | 16:22 |
| erlon | andreykurilin: do you have any good example? | 16:22 |
| bauzas | cdent: https://review.openstack.org/#/c/424110/ tries to avoid hitting -1 from functional tests because https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/api_models.py#L294-L295 | 16:22 |
| erlon | andreykurilin: this removal breaks Cinder migration and some other features | 16:23 |
| BobBall | jroll: I thought it had to be on the compute since the scheduler could be managing different computes with different ratios? | 16:23 |
| *** moshele has quit IRC | 16:23 | |
| BobBall | jroll: "This configuration specifies ratio for RamFilter which can be set | 16:23 |
| BobBall | per compute node" | 16:23 |
| bauzas | cdent: when you run 2 'compute' services in a functional test, it then runs the FakeDriver but with the same hypervisor_hostname so we're trampled by the placement engine giving us a 409 | 16:23 |
| bauzas | when creating the RP for the 2nd compute | 16:23 |
| BobBall | (sorry - thought I could delete the linebreak after paste) | 16:24 |
| jroll | BobBall: not sure, I'm not a nova+virt expert :) | 16:24 |
| andreykurilin | erlon: yes. service_catalog was a custom implementation for thing that should be done by keystoneclient. service_catalog is now available in keystone session object | 16:24 |
| cdent | bauzas: so the problem is not so much that the constraint exists, but that existing fixtures weren't prepared for it? | 16:24 |
| bauzas | cdent: zactly | 16:24 |
| erlon | andreykurilin: hmm, ok, thanks | 16:24 |
| bauzas | cdent: I'm not saying it's due to the UC, just that the problem is popping up now because all our computes have 'fake-mini' | 16:24 |
| BobBall | jroll: OK - my reading is that I think that setting the default per-compute will work as the scheduler reads from the host_state rather than using a default value from the config file. | 16:25 |
| bauzas | cdent: I fixed that in https://review.openstack.org/#/c/424110/ but that blows up, probably because it makes a shit ton of assumptions | 16:25 |
| jroll | BobBall: ah, then yeah, conf.set_default in the driver's initialization would probably work out | 16:25 |
| jroll | I guess it's easily tested | 16:25 |
| BobBall | I'm already hacking it up to test :) | 16:25 |
| cdent | bauzas: your use of the phrase "very important" made it sound like it would be hard to fix and might impact actually using the system, but it sounds more like it is something we need to address (and may be hard to address) but once addressed, not a problem? | 16:25 |
| bauzas | cdent: it's a show-stopper for merging the scheduler upcall | 16:26 |
| bauzas | tbc | 16:26 |
| edleafe | bauzas: thinking about real-world deployments: are we requiring operators to give every compute node a unique name? | 16:26 |
| bauzas | edleafe: well, I think we made this hypothesis a while ago because you can get_by_name a RP | 16:27 |
| bauzas | now, it's part of the world | 16:27 |
| bauzas | and we can't easily go against that way | 16:27 |
| *** markus_z has joined #openstack-nova | 16:27 | |
| edleafe | bauzas: ok, so how likely is it that a deployment with thousands of compute nodes has named them all uniquely? | 16:28 |
| *** jdillaman has quit IRC | 16:28 | |
| bauzas | edleafe: well, that's certainly a good point, but you're advocating of removing that UC, then ? | 16:28 |
| bauzas | sounds a big deal to me | 16:28 |
| edleafe | bauzas: not advocating - more like thinking aloud | 16:29 |
| edleafe | bauzas: as someone who hasn't ever managed a large deployment | 16:29 |
| bauzas | edleafe: AFAIR, libvirt is taking the hostname as the hypervisor_hostname | 16:29 |
| bauzas | edleafe: and that goes the same for other drivers | 16:29 |
| edleafe | bauzas: causing tests to fail is bad. Causing deployments to fail is... worse | 16:30 |
| bauzas | edleafe: given it's based on the service hostname, that looks not difficult to me to have different hostnames :) | 16:30 |
| *** nkrinner is now known as nkrinner_afk | 16:30 | |
| sfinucan | DinaBelova: Quick question - if I run something like 'tox -e py27 nova.tests.unit.scheduler.filters.test_numa_topology_filters', is it expected that the os_profiler test be run? | 16:31 |
| openstackgerrit | Eric Fried proposed openstack/nova: Introduce base PowerVM driver (disk drv) https://review.openstack.org/409401 | 16:32 |
| edleafe | bauzas: ok, that's reassuring | 16:32 |
| *** david-lyle has joined #openstack-nova | 16:32 | |
| cdent | bauzas: so sounds like the first thing to do is fix the fixtures and the functional tests? Do you need/want help with that? | 16:33 |
| bauzas | cdent: http://logs.openstack.org/10/424110/1/check/gate-nova-tox-db-functional-ubuntu-xenial/e2972f1/testr_results.html.gz | 16:33 |
| mriedem | so rp.name is optional | 16:33 |
| bauzas | cdent: giving you a sense of what I call "a shit ton of assumptions" | 16:33 |
| mriedem | is it the resource tracker that's setting it to hypervisor_hostname? | 16:33 |
| bauzas | mriedem: it is indeed | 16:34 |
| mriedem | so how about we name it hypervisor_hostname + nodename? | 16:34 |
| mriedem | surely those would be unique in the real world | 16:34 |
| mriedem | for our fixtures and tests, we probably have those copied all over the place, but we could just fix that | 16:34 |
| mriedem | with uuids or something | 16:34 |
| bauzas | mriedem: hypervisor_hostname == nodename but I get your idea | 16:34 |
| bauzas | mriedem: you mean service hostname + hypervisor_hostname | 16:35 |
| bauzas | lemme try this | 16:35 |
| openstackgerrit | Dan Smith proposed openstack/nova: Multicell support for instance listing https://review.openstack.org/396775 | 16:35 |
| openstackgerrit | Dan Smith proposed openstack/nova: Make api_samples tests use simple cell environment https://review.openstack.org/410473 | 16:35 |
| openstackgerrit | Dan Smith proposed openstack/nova: WIP Move instance creation to conductor https://review.openstack.org/319379 | 16:35 |
| dansmith | sfinucan: thanks for catching that, fixed here ^ | 16:35 |
| BobBall | jroll: What do you know - it looks like it works! Many thanks | 16:36 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: hyper-v: Adds support for VMs with nested virtualization https://review.openstack.org/398509 | 16:36 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Adds CPUFeaturesField to ImageMetaProps https://review.openstack.org/420960 | 16:36 |
| *** nklenke has joined #openstack-nova | 16:36 | |
| cdent | mriedem, bauzas: wait, are we saying that hypervisor hostname is not unique? is it not an FQDN? | 16:36 |
| bauzas | cdent: not with the FakeDriver at least | 16:36 |
| cdent | bauzas: the fake driver is _fake_ | 16:37 |
| cdent | and can be changed | 16:37 |
| cdent | so isn't really a true issue | 16:37 |
| bauzas | cdent: sure, what I did | 16:37 |
| cdent | I'm trying to figure out if there is a real issue | 16:37 |
| bauzas | cdent: I changed that to being defaulted to the service hostname | 16:37 |
| bauzas | cdent: but kaboom | 16:37 |
| cdent | hypervisor hostname is unique, job done | 16:37 |
| bauzas | because in lots of places, we litterally say hypervisor_hostname == 'fake-mini' | 16:37 |
| *** dtp has joined #openstack-nova | 16:37 | |
| cdent | that's because in a test context service hostname is not unique, right? | 16:37 |
| *** efoley__ has joined #openstack-nova | 16:38 | |
| jroll | BobBall: \o/ | 16:38 |
| cdent | so we just fix that, right? | 16:38 |
| *** bvanhav has quit IRC | 16:38 | |
| bauzas | cdent: that's because we stupidely get the value from the first CONF.host value which is set to 'fake-mini' (because of the ConfFixture) | 16:38 |
| bauzas | so we wrote around 500 tests saying "cool, let's just assume it's fake-mini whatever the compute node name is" | 16:39 |
| *** sacharya has joined #openstack-nova | 16:39 | |
| bauzas | honestly, mriedem's proposal seems the less boring approach | 16:39 |
| cdent | that means changing the names of existing resource providers out there in a deployed world? | 16:40 |
| mriedem | bauzas: i also agree with cdent, | 16:40 |
| mriedem | in that if the hostname is just fqdn, and we have a bunch of dumb tests, we just fix the tests that need fixing, | 16:40 |
| mriedem | i doubt it's 500 that we need to fix 'right now' | 16:41 |
| *** efoley_ has quit IRC | 16:41 | |
| bauzas | mriedem: sure, we can do that, I leave others fix that then | 16:41 |
| bauzas | https://review.openstack.org/#/c/424110/1 is just red tomato | 16:41 |
| *** hshiina has joined #openstack-nova | 16:42 | |
| cdent | bauzas: If it helps, I can do that. | 16:42 |
| cdent | As it has to be done, right? | 16:42 |
| *** hoonetorg has joined #openstack-nova | 16:42 | |
| *** ratailor has quit IRC | 16:43 | |
| bauzas | cdent: if we agree on that being a necessary prereq for merging the scheduler change, I'm fine | 16:43 |
| openstackgerrit | Bob Ball proposed openstack/nova: XenAPI: fix memory usage miscalculation https://review.openstack.org/394857 | 16:43 |
| bauzas | cdent: yup, lemme show you why it's important | 16:43 |
| cdent | bauzas, fine with me doing it, or fine you'll take care of it? | 16:43 |
| cdent | I know why it is important | 16:43 |
| bauzas | cdent: http://paste.openstack.org/show/595821/ L62 | 16:44 |
| *** hoonetorg has quit IRC | 16:45 | |
| mriedem | bauzas: so let's just focus on fixing the tests that need fixing right now to move you along | 16:45 |
| bauzas | okay | 16:45 |
| *** mingyu has joined #openstack-nova | 16:46 | |
| cdent | bauzas: If there's some task or chunk of a task that you can split off to me, let me know. | 16:47 |
| bauzas | mriedem: cdent: so we all agree with the fact that the FakeDriver should stop reporting the same hypervisor hostname? | 16:47 |
| mriedem | sure | 16:47 |
| mriedem | i mean yeah probably | 16:47 |
| mriedem | it depends on context | 16:47 |
| cdent | At least in the context of the scheduler related tests | 16:47 |
| *** hoonetorg has joined #openstack-nova | 16:47 | |
| mriedem | jinx | 16:47 |
| cdent | buy me a coke | 16:47 |
| bauzas | mriedem: cdent: we could compare with other drivers | 16:48 |
| cdent | Perhaps a UniqueHostnameFakeDriver subclass could help? | 16:48 |
| mriedem | yeah that's not a bad idea | 16:48 |
| bauzas | cdent: interesting idea | 16:48 |
| bauzas | that would prevent us to fix the world | 16:48 |
| mriedem | then just override the config with that in setUp for the tests that care | 16:48 |
| bauzas | mriedem: cdent: I like that idea, I can spin a patch easily then | 16:49 |
| mriedem | well, you could TODO it in ocata, | 16:49 |
| mriedem | TODO(bauzas): this is temporary until we can fix all of the tests | 16:49 |
| bauzas | mriedem: agreed | 16:49 |
| cdent | \o/ | 16:49 |
| bauzas | mriedem: I also need to verify the other drives | 16:49 |
| cdent | we have just experienced successful collaboration | 16:49 |
| gabor_antal | I'd like to notice, that we also have this hostname problem when we want to test live migration notification (e.g. https://review.openstack.org/#/c/402124/) | 16:50 |
| cdent | achievement unlocked! | 16:50 |
| bauzas | I'm pretty sure for libvirt that the nodename is equal to the service name | 16:50 |
| bauzas | for ironic, it's easy : it's the ironic node uuid | 16:50 |
| bauzas | but I leave the question open for other drivers | 16:50 |
| *** mingyu has quit IRC | 16:52 | |
| openstackgerrit | Bob Ball proposed openstack/nova: XenAPI: fix memory usage miscalculation https://review.openstack.org/394857 | 16:53 |
| *** david-lyle_ has joined #openstack-nova | 16:57 | |
| *** a-pugachev has joined #openstack-nova | 16:58 | |
| andreykurilin | mriedem: it looks like "token" auth is compitely broken in novaclient. It doesn't work for CLI where keystoneauth creates session object itself(I do not know how many releases it was broken) and in python API (my patch broke it). I'll try to fix is asap and we need to cut new release | 16:58 |
| *** efoley_ has joined #openstack-nova | 16:58 | |
| *** _ducttape_ has quit IRC | 16:58 | |
| *** ducttape_ has joined #openstack-nova | 17:01 | |
| *** baoli has quit IRC | 17:01 | |
| *** andreas_s has quit IRC | 17:01 | |
| *** efoley__ has quit IRC | 17:02 | |
| *** david-lyle_ has quit IRC | 17:03 | |
| mriedem | andreykurilin: https://review.openstack.org/#/c/419441/ right? | 17:03 |
| openstackgerrit | STEW TY proposed openstack/nova: Transform instance.unrescue notifications https://review.openstack.org/388275 | 17:03 |
| andreykurilin | mriedem: 1) it contains a bug. It fixes only python api | 17:04 |
| andreykurilin | I'll re-submit it to pass gates | 17:04 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Pass relevant parameters to Token based authentication https://review.openstack.org/419441 | 17:06 |
| openstackgerrit | Dan Smith proposed openstack/nova: Only warn about hostmappings during ocata upgrade https://review.openstack.org/424243 | 17:07 |
| *** jose-phillips has joined #openstack-nova | 17:07 | |
| dims | mriedem : notice the gate-tempest-dsvm-py35-ubuntu-xenial-nv in check queue? (example https://review.openstack.org/#/c/323250/) | 17:08 |
| *** karimb has quit IRC | 17:09 | |
| mdbooth | bauzas: When doing a migration with an explicit destination, would it make sense to do make a scheduler call anyway restricted to just that destination to see if it succeeds? | 17:09 |
| *** bvanhav has joined #openstack-nova | 17:09 | |
| mdbooth | For all I know we already do this somewhere, just thinking... | 17:10 |
| bauzas | mdbooth: for a live migration, we do that already | 17:10 |
| bauzas | mdbooth: for cold ones, there is a BP from takashi | 17:10 |
| mdbooth | I'm looking here: https://review.openstack.org/#/c/244489/58/nova/conductor/tasks/live_migrate.py | 17:10 |
| *** mlavalle has joined #openstack-nova | 17:10 | |
| mdbooth | explicit destination doesn't call _find_destination() there | 17:10 |
| mdbooth | But perhaps it's called elsewhere. | 17:11 |
| mdbooth | Well, not _find_destination, but some other scheduler call. | 17:11 |
| mriedem | dims: notice what? | 17:11 |
| mriedem | dims: if it's non-voting i'm not noticing it right now | 17:12 |
| *** markus_z has quit IRC | 17:14 | |
| bauzas | mriedem: https://github.com/openstack/nova/blob/c9eb9530314d047f5013941ebcfd5ef0192a9dc3/nova/compute/api.py#L3773 | 17:14 |
| *** tesseract has quit IRC | 17:15 | |
| dims | mriedem : ack. it's green that's all :) | 17:16 |
| mriedem | dims: oh you were looking for some back patting, i see | 17:16 |
| mriedem | #success dims got gate-tempest-dsvm-py35-ubuntu-xenial-nv green | 17:16 |
| openstackstatus | mriedem: Added success to Success page | 17:16 |
| dims | LOL, just wanted to be sure i won't get yelled at if you notice it later in the week (when it goes RED) :) | 17:17 |
| mriedem | i won't notice | 17:18 |
| *** a-pugachev has quit IRC | 17:18 | |
| mriedem | dims: i'm on vacation so only water temperature and some other key things are high priority for me right now | 17:18 |
| dims | fair enough | 17:18 |
| *** baoli has joined #openstack-nova | 17:18 | |
| *** nicolasbock has quit IRC | 17:19 | |
| mriedem | i don't have to officially care about py35 until march | 17:19 |
| mriedem | :) | 17:19 |
| dims | ack mriedem : have fun! | 17:19 |
| *** nic has joined #openstack-nova | 17:19 | |
| openstackgerrit | Dan Smith proposed openstack/nova: WIP Move instance creation to conductor https://review.openstack.org/319379 | 17:26 |
| *** mvk has joined #openstack-nova | 17:26 | |
| *** stevelle has joined #openstack-nova | 17:27 | |
| *** nicolasbock has joined #openstack-nova | 17:27 | |
| jaosorior | gibi: you around? | 17:27 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: scheduler: Don't modify RequestSpec.numa_topology https://review.openstack.org/419479 | 17:28 |
| *** mat128 is now known as mat128|afk | 17:28 | |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: [CLI] Fix token auth type https://review.openstack.org/424256 | 17:30 |
| *** sudipto has quit IRC | 17:30 | |
| *** sudipto_ has quit IRC | 17:30 | |
| *** sudipto_ has joined #openstack-nova | 17:31 | |
| *** sudipto has joined #openstack-nova | 17:31 | |
| *** sudipto has quit IRC | 17:31 | |
| *** sudipto_ has quit IRC | 17:31 | |
| andreykurilin | mriedem: have a nice vacation! I'll try to ping some other cores to merge fixes when it will be ready and propose a release | 17:31 |
| *** derekh has quit IRC | 17:33 | |
| mriedem | ack | 17:33 |
| mriedem | andreykurilin: thanks for being on top of it | 17:33 |
| sfinucan | dansmith: Added tests to https://review.openstack.org/419479. Looking at the cells limit patch again now | 17:33 |
| *** cjvolzka has quit IRC | 17:34 | |
| *** Swami has joined #openstack-nova | 17:37 | |
| *** sacharya_ has joined #openstack-nova | 17:37 | |
| *** tbachman_ is now known as tbachman | 17:37 | |
| *** sacharya has quit IRC | 17:38 | |
| *** nicolasbock has quit IRC | 17:40 | |
| *** Apoorva has joined #openstack-nova | 17:40 | |
| *** cjvolzka has joined #openstack-nova | 17:43 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova: objects: add new field cpuset_reserved in NUMACell https://review.openstack.org/400874 | 17:45 |
| mdbooth | Can anybody tell me what the 'limits' in the host object returned by scheduler_client.select_destinations() means? | 17:45 |
| *** marst has joined #openstack-nova | 17:45 | |
| mdbooth | bauzas: ^^^ ? | 17:45 |
| bauzas | mdbooth: it's what the claim uses for verifying the limit | 17:45 |
| mdbooth | What's its scope? | 17:46 |
| bauzas | mdbooth: for example, see the CoreFilter, it's telling what the max amount of the resouce is given the allocation ratio | 17:46 |
| mdbooth | i.e. can it race with the clam? | 17:46 |
| mdbooth | Ok, so it's essentially a constant? It doesn't vary as resources are used. | 17:47 |
| mdbooth | I'm looking at a patch where conductor fetches this limits value and passes it to compute, which then uses it to create a claim | 17:48 |
| *** cjvolzka has quit IRC | 17:48 | |
| mdbooth | IIUC, what you describe means there's no race there, as the value can't change in the presence of other placements. | 17:49 |
| mdbooth | I assumed that was the case because in claim: free = limit - used | 17:50 |
| mdbooth | where used is dynamic | 17:50 |
| dansmith | sfinucan: re-reviewed, thanks | 17:50 |
| andreykurilin | dims: I do not what had happened, but rally-nova-py3 jobs is ok now:) | 17:52 |
| mlavalle | mriedem: is it still the plan in Ocata to have the scheduler use the data in resource providers and inventories for instances placement in host? | 17:53 |
| andreykurilin | *I do not know | 17:53 |
| edleafe | mlavalle: yes. The scheduler will get a pared-down list of hosts from placement that match the basic resources in the request | 17:54 |
| edleafe | mlavalle: it will then use its other filters to select, as it currently does | 17:54 |
| *** browne has joined #openstack-nova | 17:54 | |
| *** nicolasbock has joined #openstack-nova | 17:55 | |
| mriedem | mlavalle: yes | 17:55 |
| openstackgerrit | John Garbutt proposed openstack/nova: WIP Support cinder v3 API https://review.openstack.org/420201 | 17:56 |
| *** david-lyle_ has joined #openstack-nova | 17:56 | |
| *** david-lyle_ has quit IRC | 17:56 | |
| *** Jeffrey4l__ has joined #openstack-nova | 17:57 | |
| *** david-lyle_ has joined #openstack-nova | 17:57 | |
| mlavalle | edleafe, mriedem: so, given that the patches that I wrote for resource providers and routed networks are merging, i can remove the first bullet here, right: http://docs.openstack.org/newton/networking-guide/config-routed-networks.html#limitations? | 17:58 |
| *** eharney has quit IRC | 17:58 | |
| edleafe | mlavalle: no | 17:58 |
| *** Jeffrey4l_ has quit IRC | 17:59 | |
| edleafe | mlavalle: I guess I should have specified what I meant by "basic" :) | 17:59 |
| edleafe | mlavalle: just disk, RAM, and VCPU for Ocata | 17:59 |
| mriedem | mlavalle: what ed said. this is also going to be a pre-req for scheduling with routed networks https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/prep-for-network-aware-scheduling-ocata | 18:00 |
| mriedem | mlavalle: and it's looking like at this point that series won't be happening in ocata | 18:00 |
| *** stevelle has left #openstack-nova | 18:01 | |
| *** ralonsoh has quit IRC | 18:01 | |
| mlavalle | edleafe, mriedem: so IPV4_ADDRESS inventories are postponed to Pike? | 18:01 |
| mlavalle | as far as being used by the scheduler | 18:02 |
| edleafe | mlavalle: They can be reported and tracked in Ocata. The scheduler won't be able to do anything with them, though | 18:02 |
| mlavalle | edleafe, mriedem: thanks for the update! | 18:02 |
| edleafe | mlavalle: IOW, the placement service will be able to handle it. The scheduler won't be using that information, yet' | 18:03 |
| mriedem | mlavalle: yeah you definitely want them reporting in as soon as possible | 18:03 |
| mriedem | so that when the scheduler can handle it, the data is already there | 18:03 |
| dims | andreykurilin : we must have fixed some stuff in nova while we got the dsvm+tempest working :) | 18:04 |
| *** kfarr has joined #openstack-nova | 18:04 | |
| dims | andreykurilin : thanks for the good news :) | 18:04 |
| mlavalle | mriedem, edleafe: ok, on our side the functionality is complete. the last patchset remaining has already 2 +2's, so it should merge today. I even update the documentation | 18:04 |
| andreykurilin | dims: there is only one failed scenario. will look at it more close a bit later | 18:05 |
| *** kbaegis has joined #openstack-nova | 18:05 | |
| dims | ack andreykurilin : thanks | 18:05 |
| mlavalle | mriedem, edleafe: I will focus next on creating a tempest test to exercise it in the gate, so we all get feedback | 18:05 |
| *** david-lyle has quit IRC | 18:05 | |
| kbaegis | Hi all. I'm having some issues with devstack and nova. From horizon I'm getting a ton of 400s, but from nova/openstack cli everything works fine | 18:05 |
| *** david-lyle_ is now known as david-lyle | 18:05 | |
| diana_clarke | kbaegis: You might be hitting this. https://bugs.launchpad.net/python-novaclient/+bug/1658148 | 18:06 |
| openstack | Launchpad bug 1654183 in python-novaclient "duplicate for #1658148 Token based authentication in Client class does not work" [High,In progress] - Assigned to Andrey Kurilin (andreykurilin) | 18:06 |
| *** bvanhav has quit IRC | 18:07 | |
| *** cjvolzka has joined #openstack-nova | 18:09 | |
| *** cjvolzka has quit IRC | 18:09 | |
| kbaegis | Thanks guys | 18:10 |
| *** armax has joined #openstack-nova | 18:10 | |
| *** owalsh is now known as owalsh-afk | 18:10 | |
| *** efoley__ has joined #openstack-nova | 18:11 | |
| *** lpetrut has quit IRC | 18:12 | |
| *** jbryce has joined #openstack-nova | 18:14 | |
| *** efoley_ has quit IRC | 18:14 | |
| *** eharney has joined #openstack-nova | 18:14 | |
| jbryce | is there a practical upper limit to how many cells a nova deployment might have? this is for a nova cells related question in the user survey | 18:16 |
| openstackgerrit | Dan Smith proposed openstack/nova: Multicell support for instance listing https://review.openstack.org/396775 | 18:17 |
| openstackgerrit | Dan Smith proposed openstack/nova: Make api_samples tests use simple cell environment https://review.openstack.org/410473 | 18:17 |
| openstackgerrit | Dan Smith proposed openstack/nova: WIP Move instance creation to conductor https://review.openstack.org/319379 | 18:17 |
| dansmith | jbryce: assume you're talking about cellsv1? | 18:17 |
| *** amotoki has quit IRC | 18:18 | |
| *** kbaegis has left #openstack-nova | 18:18 | |
| jbryce | dansmith: i’d be curious about both v1 and v2 as this round of updates will probably live into the time when we start getting v2 deployments cataloged | 18:19 |
| dansmith | jbryce: well, as of this moment cellsv2 can't have multiple cells | 18:19 |
| dansmith | jbryce: for cellsv1 you'd probably need to ask someone that actually runs it to see how many they have and/or if they've seen any such practical limits | 18:19 |
| dansmith | jbryce: there is no hard-coded limit or anything like that though | 18:20 |
| jbryce | dansmith: thanks | 18:21 |
| *** owalsh-afk has quit IRC | 18:21 | |
| *** sambetts is now known as sambetts|afk | 18:21 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova: hyper-v: Adds support for VMs with nested virtualization https://review.openstack.org/398509 | 18:22 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Adds CPUFeaturesField to ImageMetaProps https://review.openstack.org/420960 | 18:22 |
| *** mnestratov has quit IRC | 18:22 | |
| *** bvanhav has joined #openstack-nova | 18:25 | |
| *** cjvolzka has joined #openstack-nova | 18:27 | |
| *** jpena is now known as jpena|off | 18:31 | |
| *** dharinic is now known as dharinic|lunch | 18:35 | |
| *** mjura has quit IRC | 18:38 | |
| *** efoley__ has quit IRC | 18:43 | |
| *** tbachman has quit IRC | 18:45 | |
| *** catintheroof has joined #openstack-nova | 18:45 | |
| *** owalsh-afk has joined #openstack-nova | 18:45 | |
| *** catinthe_ has quit IRC | 18:47 | |
| *** beagles is now known as beagles-brb | 18:47 | |
| *** tbachman has joined #openstack-nova | 18:48 | |
| *** mingyu has joined #openstack-nova | 18:49 | |
| *** yamahata has quit IRC | 18:51 | |
| *** mingyu has quit IRC | 18:53 | |
| *** mriedem has quit IRC | 18:55 | |
| *** tbachman_ has joined #openstack-nova | 18:56 | |
| *** tbachman has quit IRC | 18:56 | |
| *** tbachman_ is now known as tbachman | 18:56 | |
| *** sree has joined #openstack-nova | 18:57 | |
| *** kukacz has quit IRC | 19:03 | |
| *** catinthe_ has joined #openstack-nova | 19:03 | |
| *** sree has quit IRC | 19:03 | |
| amrith | hi andreykurilin ... any luck? | 19:05 |
| *** catintheroof has quit IRC | 19:06 | |
| *** gszasz has quit IRC | 19:07 | |
| *** hshiina has quit IRC | 19:11 | |
| *** amoralej is now known as amoralej|off | 19:13 | |
| *** voelzmo has joined #openstack-nova | 19:15 | |
| *** Oku_OS is now known as Oku_OS-away | 19:16 | |
| *** amotoki has joined #openstack-nova | 19:18 | |
| *** gouthamr has quit IRC | 19:20 | |
| *** enki_anunnaki has joined #openstack-nova | 19:21 | |
| *** Sukhdev_ has joined #openstack-nova | 19:21 | |
| *** hshiina has joined #openstack-nova | 19:22 | |
| oomichi | slaweq_: thanks, now LGTM | 19:22 |
| *** sdague has joined #openstack-nova | 19:22 | |
| *** knikolla has quit IRC | 19:23 | |
| *** beagles-brb is now known as beaglkes | 19:23 | |
| *** knikolla has joined #openstack-nova | 19:23 | |
| *** knikolla has quit IRC | 19:23 | |
| *** beaglkes is now known as beagles | 19:24 | |
| *** knikolla has joined #openstack-nova | 19:24 | |
| *** knikolla has quit IRC | 19:24 | |
| *** knikolla has joined #openstack-nova | 19:25 | |
| *** knikolla has quit IRC | 19:25 | |
| *** knikolla has joined #openstack-nova | 19:25 | |
| *** knikolla has quit IRC | 19:25 | |
| *** Guest17268 has quit IRC | 19:26 | |
| openstackgerrit | Ken'ichi Ohmichi proposed openstack/nova: Deny invalid flavor extraspec for cpu_policy https://review.openstack.org/406377 | 19:27 |
| oomichi | gmann: thanks, applied your comments | 19:27 |
| *** yamahata has joined #openstack-nova | 19:29 | |
| openstackgerrit | Chris Dent proposed openstack/nova: Avoid redundant call to update_resource_stats from RT https://review.openstack.org/424305 | 19:35 |
| cdent | bauzas, edleafe ^ is a potential minor performance fix in the resource tracker | 19:36 |
| cdent | not sure how much it matters but seemed low hanging fruit in the bug list: https://bugs.launchpad.net/nova/+bug/1658629 | 19:37 |
| openstack | Launchpad bug 1658629 in OpenStack Compute (nova) "Resource tracker causes update of compute_nodes table every minute" [Medium,In progress] - Assigned to Chris Dent (cdent) | 19:37 |
| cdent | and with that, good night | 19:37 |
| *** cdent has quit IRC | 19:37 | |
| openstackgerrit | Stephen Gordon proposed openstack/nova: libvirt: avoid generating script with empty path https://review.openstack.org/411936 | 19:42 |
| *** jdurgin has joined #openstack-nova | 19:46 | |
| openstackgerrit | STEW TY proposed openstack/nova: Transform instance.unrescue notifications https://review.openstack.org/388275 | 19:46 |
| *** ociuhandu has quit IRC | 19:47 | |
| *** dharinic|lunch is now known as dharinic | 19:49 | |
| *** tbachman has quit IRC | 19:49 | |
| *** jaosorior has quit IRC | 19:50 | |
| *** voelzmo has quit IRC | 19:51 | |
| *** hshiina has quit IRC | 19:59 | |
| *** abalutoiu_ has quit IRC | 19:59 | |
| *** kaisers has joined #openstack-nova | 20:00 | |
| *** sdague has quit IRC | 20:00 | |
| *** raj_singh_ has joined #openstack-nova | 20:02 | |
| *** nic has quit IRC | 20:05 | |
| slaweq | oomichi: thx for +2 :) | 20:06 |
| *** jose-phillips has quit IRC | 20:10 | |
| *** satyar has quit IRC | 20:13 | |
| *** slaweq__ has joined #openstack-nova | 20:13 | |
| openstackgerrit | STEW TY proposed openstack/nova: Transform instance.rescue notifications https://review.openstack.org/385644 | 20:15 |
| *** mnestratov has joined #openstack-nova | 20:16 | |
| *** satyar has joined #openstack-nova | 20:19 | |
| *** tbachman has joined #openstack-nova | 20:20 | |
| *** liverpooler has quit IRC | 20:22 | |
| *** rfolco has quit IRC | 20:27 | |
| *** tbachman has quit IRC | 20:29 | |
| *** _ducttape_ has joined #openstack-nova | 20:30 | |
| *** pradk- has joined #openstack-nova | 20:31 | |
| *** pradk has quit IRC | 20:31 | |
| *** bvanhav has quit IRC | 20:32 | |
| *** stvnoyes has quit IRC | 20:33 | |
| *** ducttape_ has quit IRC | 20:33 | |
| *** stvnoyes has joined #openstack-nova | 20:33 | |
| *** raj_singh_ has quit IRC | 20:35 | |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: Implement new attach/detach Cinder flow https://review.openstack.org/330285 | 20:41 |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: Remove check_attach https://review.openstack.org/335358 | 20:41 |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: Implement new attach/detach Cinder flow https://review.openstack.org/330285 | 20:42 |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: Remove check_attach https://review.openstack.org/335358 | 20:42 |
| *** moshele has joined #openstack-nova | 20:43 | |
| *** abalutoiu_ has joined #openstack-nova | 20:48 | |
| *** nklenke has quit IRC | 20:50 | |
| *** mingyu has joined #openstack-nova | 20:50 | |
| amrith | andreykurilin, please let me know if I can help. | 20:52 |
| *** mingyu has quit IRC | 20:54 | |
| *** slaweq__ has quit IRC | 20:56 | |
| *** kaisers has quit IRC | 20:57 | |
| *** pradk- is now known as pradk | 20:58 | |
| *** hoangcx has joined #openstack-nova | 21:01 | |
| *** ociuhandu has joined #openstack-nova | 21:02 | |
| *** oanson has quit IRC | 21:10 | |
| *** dimtruck is now known as zz_dimtruck | 21:11 | |
| *** jerrygb has quit IRC | 21:12 | |
| *** satyar has quit IRC | 21:15 | |
| andreykurilin | amrith: I need just several minutes for latest version of patches. they should help | 21:17 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Pass relevant parameters to Token based authentication https://review.openstack.org/419441 | 21:18 |
| *** _ducttape_ has quit IRC | 21:19 | |
| *** ducttape_ has joined #openstack-nova | 21:20 | |
| *** ducttape_ has joined #openstack-nova | 21:20 | |
| *** tbachman has joined #openstack-nova | 21:22 | |
| *** ijw has joined #openstack-nova | 21:23 | |
| moshele | jaypipes: hi, can you review this simple fix https://review.openstack.org/#/c/423608/ | 21:26 |
| *** kaisers has joined #openstack-nova | 21:26 | |
| *** jose-phillips has joined #openstack-nova | 21:27 | |
| *** oanson has joined #openstack-nova | 21:30 | |
| *** smatzek_ has quit IRC | 21:30 | |
| *** jerrygb has joined #openstack-nova | 21:31 | |
| *** zz_dimtruck is now known as dimtruck | 21:32 | |
| *** jerrygb_ has joined #openstack-nova | 21:32 | |
| *** toure is now known as toure|biab | 21:32 | |
| *** Jeffrey4l_ has joined #openstack-nova | 21:34 | |
| *** thorst_ has quit IRC | 21:35 | |
| *** Jeffrey4l__ has quit IRC | 21:35 | |
| *** thorst_ has joined #openstack-nova | 21:35 | |
| *** jerrygb has quit IRC | 21:36 | |
| *** cleong has quit IRC | 21:39 | |
| *** jose-phillips has quit IRC | 21:39 | |
| *** slaweq has quit IRC | 21:39 | |
| *** slaweq has joined #openstack-nova | 21:40 | |
| *** thorst_ has quit IRC | 21:40 | |
| *** jose-phillips has joined #openstack-nova | 21:42 | |
| *** markvoelker has quit IRC | 21:47 | |
| *** moshele has quit IRC | 21:47 | |
| *** anush has quit IRC | 21:48 | |
| *** kfarr has quit IRC | 21:48 | |
| *** mnestratov has quit IRC | 21:49 | |
| openstackgerrit | Ed Leafe proposed openstack/nova: placement: adds REST API for nested providers https://review.openstack.org/384807 | 21:51 |
| openstackgerrit | Ed Leafe proposed openstack/nova: placement: allow filter providers in tree https://review.openstack.org/377215 | 21:51 |
| openstackgerrit | Ed Leafe proposed openstack/nova: pci: add uuid field to PciDevice object https://review.openstack.org/414768 | 21:51 |
| openstackgerrit | Ed Leafe proposed openstack/nova: placement: integrate ProviderTree to report client https://review.openstack.org/415921 | 21:51 |
| openstackgerrit | Ed Leafe proposed openstack/nova: placement: SRIOV PF devices as child providers https://review.openstack.org/414769 | 21:51 |
| openstackgerrit | Ed Leafe proposed openstack/nova: placement: adds ProviderTree for nested resources https://review.openstack.org/415920 | 21:51 |
| openstackgerrit | Ed Leafe proposed openstack/nova: placement: update client to set parent provider https://review.openstack.org/385693 | 21:51 |
| openstackgerrit | Ed Leafe proposed openstack/nova: placement: add nested resource providers https://review.openstack.org/377138 | 21:51 |
| *** mingyu has joined #openstack-nova | 21:51 | |
| *** slaweq has quit IRC | 21:52 | |
| *** slaweq has joined #openstack-nova | 21:54 | |
| *** mingyu has quit IRC | 21:56 | |
| *** ducttape_ has quit IRC | 21:56 | |
| *** oanson has quit IRC | 21:56 | |
| *** oanson has joined #openstack-nova | 21:57 | |
| *** smatzek_ has joined #openstack-nova | 21:58 | |
| *** thorst_ has joined #openstack-nova | 21:58 | |
| openstackgerrit | STEW TY proposed openstack/nova: Transform instance.rescue notifications https://review.openstack.org/385644 | 21:59 |
| *** kaisers has quit IRC | 22:02 | |
| *** thorst_ has quit IRC | 22:03 | |
| *** harlowja has quit IRC | 22:03 | |
| *** eharney has quit IRC | 22:03 | |
| *** baoli has quit IRC | 22:04 | |
| *** kukacz has joined #openstack-nova | 22:05 | |
| *** jdurgin has quit IRC | 22:05 | |
| *** hoangcx has quit IRC | 22:05 | |
| *** nic has joined #openstack-nova | 22:06 | |
| *** burgerk has quit IRC | 22:10 | |
| amrith | thx andreykurilin | 22:12 |
| *** jheroux has quit IRC | 22:12 | |
| *** browne has quit IRC | 22:12 | |
| *** thorst_ has joined #openstack-nova | 22:13 | |
| amrith | andreykurilin, this one (412497) isn't looking promising | 22:13 |
| *** a-pugachev has joined #openstack-nova | 22:18 | |
| *** thorst_ has quit IRC | 22:22 | |
| *** jdurgin has joined #openstack-nova | 22:23 | |
| *** oanson has quit IRC | 22:24 | |
| *** oanson has joined #openstack-nova | 22:25 | |
| *** slaweq has quit IRC | 22:26 | |
| *** slaweq has joined #openstack-nova | 22:27 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add sort_key white list for server list/detail https://review.openstack.org/415142 | 22:30 |
| *** kaisers has joined #openstack-nova | 22:32 | |
| *** esberglu has quit IRC | 22:33 | |
| *** esberglu has joined #openstack-nova | 22:33 | |
| *** esberglu_ has joined #openstack-nova | 22:35 | |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: Implement new attach/detach Cinder flow https://review.openstack.org/330285 | 22:37 |
| *** esberglu has quit IRC | 22:37 | |
| *** esberglu_ has quit IRC | 22:39 | |
| *** smatzek_ has quit IRC | 22:41 | |
| openstackgerrit | Ed Leafe proposed openstack/nova: placement: minor refactor _allocate_for_instance() https://review.openstack.org/405705 | 22:43 |
| openstackgerrit | Ed Leafe proposed openstack/nova: placement: RT now adds proper Ironic inventory https://review.openstack.org/404472 | 22:43 |
| openstackgerrit | Ed Leafe proposed openstack/nova: placement: report client handle InventoryInUse https://review.openstack.org/404990 | 22:43 |
| *** owalsh-afk has quit IRC | 22:46 | |
| *** owalsh-afk has joined #openstack-nova | 22:46 | |
| *** browne has joined #openstack-nova | 22:48 | |
| *** mtanino has quit IRC | 22:49 | |
| *** marst has quit IRC | 22:51 | |
| *** tblakes has quit IRC | 22:51 | |
| *** mingyu has joined #openstack-nova | 22:52 | |
| *** Swami_ has joined #openstack-nova | 22:56 | |
| *** mingyu has quit IRC | 22:56 | |
| *** kylek3h has quit IRC | 22:58 | |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: WIP: Scheduler calling the Placement API https://review.openstack.org/417961 | 22:58 |
| *** jamesdenton has quit IRC | 23:00 | |
| *** Swami has quit IRC | 23:00 | |
| *** Swami_ has quit IRC | 23:01 | |
| *** cjvolzka has quit IRC | 23:04 | |
| *** mdrabe has quit IRC | 23:05 | |
| *** esberglu has joined #openstack-nova | 23:12 | |
| *** esberglu has quit IRC | 23:12 | |
| andreykurilin | amrith: based on novaclient's functional tests, it should work. I do not sure that depends on tag actually works for novaclient's library. Can you recheck it locally? | 23:18 |
| *** alexpilotti has joined #openstack-nova | 23:19 | |
| *** catinthe_ has quit IRC | 23:23 | |
| *** xyang1 has quit IRC | 23:25 | |
| andreykurilin | amrith: I try to play more with trove code tomorrow, but everything should be ok now | 23:30 |
| *** edleafe- has joined #openstack-nova | 23:33 | |
| *** harlowja has joined #openstack-nova | 23:34 | |
| *** harlowja has quit IRC | 23:35 | |
| *** jose-phillips has quit IRC | 23:35 | |
| *** edleafe has quit IRC | 23:36 | |
| *** jose-phillips has joined #openstack-nova | 23:37 | |
| *** tbachman has quit IRC | 23:38 | |
| *** dave-mccowan has quit IRC | 23:38 | |
| openstackgerrit | Anusha Unnam proposed openstack/nova: Add keypairs field to InstancePayload https://review.openstack.org/419730 | 23:39 |
| *** mtanino has joined #openstack-nova | 23:40 | |
| *** mriedem has joined #openstack-nova | 23:40 | |
| *** Swami has joined #openstack-nova | 23:41 | |
| *** takashin has joined #openstack-nova | 23:42 | |
| mriedem | dansmith: looks like grenade isn't happy with https://review.openstack.org/#/c/424252/ | 23:42 |
| dansmith | mriedem: isn't that the dep thing? | 23:43 |
| * dansmith checks | 23:43 | |
| dansmith | mriedem: yeah, that's the dep thing.. I've been waiting all day to get a run of my patch with this | 23:44 |
| dansmith | it just merged and I rechecked mine -- just rechecked this one too | 23:44 |
| *** dimtruck is now known as zz_dimtruck | 23:44 | |
| mriedem | dansmith: i'm not sure what dep thing you're talking about | 23:45 |
| mriedem | you changed the d-g patch to not call discover_hosts for grenade | 23:45 |
| dansmith | mriedem: https://review.openstack.org/#/c/424323/ | 23:46 |
| *** harlowja has joined #openstack-nova | 23:48 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/os-vif: Updated from global requirements https://review.openstack.org/395347 | 23:50 |
| mriedem | dansmith: ok i wasn't aware of that | 23:51 |
| *** tbachman has joined #openstack-nova | 23:53 | |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: WIP: Scheduler calling the Placement API https://review.openstack.org/417961 | 23:56 |
| bauzas | Yippee ki-yay mother ^ | 23:57 |
| bauzas | hoping Jenkins to be green after that one | 23:57 |
| bauzas | now, calling it a day | 23:57 |
| *** edmondsw has quit IRC | 23:57 | |
| *** edmondsw has joined #openstack-nova | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!