Tuesday, 2017-11-21

*** edmondsw has joined #openstack-nova00:05
*** salv-orlando has quit IRC00:09
*** salv-orlando has joined #openstack-nova00:09
*** edmondsw has quit IRC00:10
*** rtjure has quit IRC00:11
*** AlexeyAbashkin has joined #openstack-nova00:13
*** salv-orlando has quit IRC00:14
*** hongbin has quit IRC00:14
*** AlexeyAbashkin has quit IRC00:17
*** Sukhdev has joined #openstack-nova00:19
*** rtjure has joined #openstack-nova00:19
*** claudiub|2 has quit IRC00:25
*** itlinux has quit IRC00:26
openstackgerritVipin Balachandran proposed openstack/nova master: VMware: Support volumes backed by VStorageObject  https://review.openstack.org/52169000:33
*** amodi has joined #openstack-nova00:35
*** felipemonteiro has joined #openstack-nova00:40
*** yamamoto has quit IRC00:42
*** Sandy619 has joined #openstack-nova00:43
*** armax has joined #openstack-nova00:44
*** Sandy619 has quit IRC00:45
*** trinaths has joined #openstack-nova00:46
*** edmondsw has joined #openstack-nova00:49
*** edmondsw has quit IRC00:52
*** edmondsw has joined #openstack-nova00:52
*** gfhellma has quit IRC00:58
*** phuongnh has joined #openstack-nova01:04
*** suresh12 has quit IRC01:06
*** Guest18917 has quit IRC01:11
*** salv-orlando has joined #openstack-nova01:11
*** yassine has joined #openstack-nova01:11
*** yassine has quit IRC01:11
*** yassine has joined #openstack-nova01:12
*** AlexeyAbashkin has joined #openstack-nova01:12
*** yassine is now known as Guest2544201:12
*** rtjure has quit IRC01:13
*** trungnv has joined #openstack-nova01:13
*** rtjure has joined #openstack-nova01:14
*** AlexeyAbashkin has quit IRC01:16
*** suresh12 has joined #openstack-nova01:18
*** edmondsw has quit IRC01:20
*** edmondsw has joined #openstack-nova01:20
*** Daisy_ has joined #openstack-nova01:22
*** Daisy_ has quit IRC01:24
*** Daisy_ has joined #openstack-nova01:24
*** edmondsw has quit IRC01:25
*** salv-orlando has quit IRC01:26
*** takashin has joined #openstack-nova01:26
*** masayukig has quit IRC01:28
tonybdansmith: Thanks.01:28
*** Daisy_ has quit IRC01:29
*** gcb has joined #openstack-nova01:29
*** masayukig has joined #openstack-nova01:29
*** tianhui has joined #openstack-nova01:30
*** yangyapeng has joined #openstack-nova01:31
*** dave-mccowan has joined #openstack-nova01:31
*** Daisy_ has joined #openstack-nova01:32
*** yangyapeng has quit IRC01:35
*** trinaths has left #openstack-nova01:40
*** yamamoto has joined #openstack-nova01:42
*** gouthamr has quit IRC01:48
*** yangyapeng has joined #openstack-nova01:49
*** yamamoto has quit IRC01:49
*** yamamoto has joined #openstack-nova01:51
*** vladikr has quit IRC01:52
*** cheneydc has joined #openstack-nova01:52
*** yangyapeng has quit IRC01:54
openstackgerritTakashi NATSUME proposed openstack/nova master: api-ref: Fix an example in "Delete Assisted Volume Snapshot"  https://review.openstack.org/51412802:00
openstackgerritTakashi NATSUME proposed openstack/nova master: api-ref: Add a description of 'key_name' in rebuild  https://review.openstack.org/51999302:00
openstackgerritTakashi NATSUME proposed openstack/nova master: Update document related to host aggregate  https://review.openstack.org/51449902:01
*** Apoorva has quit IRC02:09
*** Apoorva has joined #openstack-nova02:10
*** Apoorva has quit IRC02:11
*** Apoorva has joined #openstack-nova02:11
*** gongysh has joined #openstack-nova02:13
*** yangyapeng has joined #openstack-nova02:15
*** chyka has quit IRC02:19
*** yangyapeng has quit IRC02:19
*** salv-orlando has joined #openstack-nova02:22
*** trungnv has quit IRC02:23
*** gmann_afk is now known as gmann02:25
*** moshele has joined #openstack-nova02:26
*** salv-orlando has quit IRC02:27
*** chyka has joined #openstack-nova02:27
openstackgerritMathieu Gagné proposed openstack/nova master: Regenerate and pass configdrive when rebuild Ironic nodes  https://review.openstack.org/50308802:29
openstackgerritMerged openstack/nova master: Include project_id and user_id in AllocationList.get_all_by_consumer_id  https://review.openstack.org/51242002:29
*** yamahata has quit IRC02:29
*** chyka has quit IRC02:32
*** trungnv has joined #openstack-nova02:34
*** yangyapeng has joined #openstack-nova02:34
*** gouthamr has joined #openstack-nova02:34
dimsmriedem_away : you need to send a translation for your team meeting email :)02:34
*** suresh12 has quit IRC02:38
*** itlinux has joined #openstack-nova02:38
*** yangyapeng has quit IRC02:38
gmannalex_xu: can you re review this -https://review.openstack.org/#/c/516104/902:45
gmannyikun: i replied on this, can you check - https://review.openstack.org/#/c/515413/02:48
*** moshele has quit IRC02:49
*** hongbin has joined #openstack-nova02:52
*** Sukhdev has quit IRC02:53
*** edmondsw has joined #openstack-nova02:55
*** vladikr has joined #openstack-nova02:57
*** felipemonteiro has quit IRC02:57
*** erlon has quit IRC02:58
*** moshele has joined #openstack-nova02:59
*** yangyapeng has joined #openstack-nova02:59
*** edmondsw has quit IRC03:00
Kevin_Zhenggmann: yikun is on his trip to bug smash, maybe not available for the next 6 hours03:03
*** yangyapeng has quit IRC03:04
gmannKevin_Zheng: ohk, thanks for info.03:04
*** tianhui_ has joined #openstack-nova03:07
*** suresh12 has joined #openstack-nova03:07
*** gongysh has quit IRC03:09
*** tianhui has quit IRC03:09
*** dave-mccowan has quit IRC03:10
*** suresh12 has quit IRC03:11
*** yangyapeng has joined #openstack-nova03:18
*** links has joined #openstack-nova03:19
*** tianhui_ is now known as tianhui03:19
*** harlowja has quit IRC03:22
*** yangyapeng has quit IRC03:22
*** salv-orlando has joined #openstack-nova03:23
*** akekane has joined #openstack-nova03:26
*** akekane is now known as abhishekk03:26
*** salv-orlando has quit IRC03:28
*** Daisy_ has quit IRC03:29
*** Daisy_ has joined #openstack-nova03:29
*** Daisy_ has quit IRC03:29
*** Daisy_ has joined #openstack-nova03:30
*** takashin has quit IRC03:30
*** Apoorva has quit IRC03:31
*** takashin has joined #openstack-nova03:31
*** zerda3 has quit IRC03:33
*** Daisy_ has quit IRC03:34
*** vladikr has quit IRC03:35
*** vladikr has joined #openstack-nova03:35
*** gcb has quit IRC03:42
*** gszasz has quit IRC03:53
*** itlinux has quit IRC03:58
*** harlowja has joined #openstack-nova04:01
*** yangyapeng has joined #openstack-nova04:03
*** gouthamr has quit IRC04:05
*** yangyapeng has quit IRC04:08
*** moshele has quit IRC04:09
*** mdnadeem has joined #openstack-nova04:10
*** psachin has joined #openstack-nova04:11
*** yamamoto has quit IRC04:11
*** vladikr has quit IRC04:11
*** itlinux has joined #openstack-nova04:12
*** hongbin has quit IRC04:15
*** hongbin has joined #openstack-nova04:15
*** chyka has joined #openstack-nova04:16
*** Daisy_ has joined #openstack-nova04:17
*** chyka has quit IRC04:20
*** Daisy_ has quit IRC04:22
*** diga has joined #openstack-nova04:24
*** yangyapeng has joined #openstack-nova04:24
*** Daisy_ has joined #openstack-nova04:24
*** salv-orlando has joined #openstack-nova04:24
*** Sukhdev has joined #openstack-nova04:25
*** TuanLA has joined #openstack-nova04:27
*** Daisy_ has quit IRC04:28
*** yangyapeng has quit IRC04:28
*** salv-orlando has quit IRC04:29
*** Daisy_ has joined #openstack-nova04:30
*** sridharg has joined #openstack-nova04:32
*** Daisy_ has quit IRC04:35
openstackgerritMerged openstack/nova master: api-ref: Add a description of 'key_name' in rebuild  https://review.openstack.org/51999304:35
*** tetsuro has joined #openstack-nova04:42
*** Daisy_ has joined #openstack-nova04:43
*** edmondsw has joined #openstack-nova04:44
*** itlinux has quit IRC04:44
*** gbarros has quit IRC04:46
*** gcb has joined #openstack-nova04:46
*** itlinux has joined #openstack-nova04:47
*** Daisy_ has quit IRC04:47
*** hongbin has quit IRC04:48
*** edmondsw has quit IRC04:48
*** yamamoto has joined #openstack-nova04:48
*** yangyapeng has joined #openstack-nova04:48
*** hshiina has joined #openstack-nova04:50
*** diga has quit IRC04:52
*** yangyapeng has quit IRC04:52
*** lbragstad has quit IRC04:58
*** Daisy_ has joined #openstack-nova04:59
*** ratailor has joined #openstack-nova04:59
*** markvoelker has quit IRC05:00
*** diga has joined #openstack-nova05:05
*** claudiub|2 has joined #openstack-nova05:08
*** yamahata has joined #openstack-nova05:10
*** janki has joined #openstack-nova05:13
*** trinaths has joined #openstack-nova05:15
*** ijw has joined #openstack-nova05:17
*** suresh12 has joined #openstack-nova05:20
*** salv-orlando has joined #openstack-nova05:25
*** sree has joined #openstack-nova05:27
*** amodi has quit IRC05:28
*** salv-orlando has quit IRC05:29
*** edmondsw has joined #openstack-nova05:29
*** itlinux has quit IRC05:31
openstackgerritMerged openstack/nova master: api-ref: Fix an example in "Delete Assisted Volume Snapshot"  https://review.openstack.org/51412805:31
*** udesale has joined #openstack-nova05:32
*** yamamoto has quit IRC05:32
*** edmondsw has quit IRC05:33
*** yamamoto has joined #openstack-nova05:34
*** yangyapeng has joined #openstack-nova05:37
*** chyka has joined #openstack-nova05:39
*** pcaruana has joined #openstack-nova05:39
*** yamamoto_ has joined #openstack-nova05:40
openstackgerritTakashi NATSUME proposed openstack/nova master: [placement] Add 'Location' parameters in API ref  https://review.openstack.org/52154105:42
openstackgerritTakashi NATSUME proposed openstack/nova master: [placement] Add 'Location' parameters in API ref  https://review.openstack.org/52154105:42
*** yamamoto has quit IRC05:43
*** chyka has quit IRC05:43
*** edmondsw has joined #openstack-nova05:48
openstackgerritjichenjc proposed openstack/nova master: WIP: remove reserve_quota_delta  https://review.openstack.org/52146905:48
openstackgerritjichenjc proposed openstack/nova master: WIP: Remove quota reserve/commit/rollback  https://review.openstack.org/52147005:48
*** harlowja has quit IRC05:52
*** edmondsw has quit IRC05:52
openstackgerritjichenjc proposed openstack/nova master: Add description for resize down support  https://review.openstack.org/51657205:53
*** lajoskatona has joined #openstack-nova05:53
openstackgerritjichenjc proposed openstack/nova master: mv generate_glance_url to get_image_endpoint_url  https://review.openstack.org/51140005:54
openstackgerritNaichuan Sun proposed openstack/nova master: xenapi: Add parse function to get gpu group  uuid and vgpu type uuid  https://review.openstack.org/52171705:54
*** yamamoto_ has quit IRC05:56
*** rodolof has joined #openstack-nova05:57
openstackgerritNaichuan Sun proposed openstack/nova master: xenapi: Add parse function to get gpu group  uuid and vgpu type uuid  https://review.openstack.org/52171705:58
*** sree has quit IRC06:00
*** hshiina2 has joined #openstack-nova06:00
*** markvoelker has joined #openstack-nova06:00
openstackgerritNaichuan Sun proposed openstack/nova master: xenapi: Add parse function to get gpu group  uuid and vgpu type uuid  https://review.openstack.org/52171706:00
*** chyka has joined #openstack-nova06:00
openstackgerritNaichuan Sun proposed openstack/nova master: xenapi: Add parse function to get gpu group  uuid and vgpu type uuid  https://review.openstack.org/52171706:02
*** hshiina has quit IRC06:03
*** sree has joined #openstack-nova06:04
*** chyka has quit IRC06:04
openstackgerritjichenjc proposed openstack/nova master: conf: remove deprecated 'project_id_regex' opt  https://review.openstack.org/50105706:06
*** Sukhdev has quit IRC06:06
*** sree has quit IRC06:12
*** pcaruana has quit IRC06:13
*** armax has quit IRC06:14
*** chyka has joined #openstack-nova06:16
*** gongysh has joined #openstack-nova06:16
*** chyka has quit IRC06:20
*** dtruong_ has joined #openstack-nova06:22
*** mdnadeem has quit IRC06:22
*** cheneydc_ has joined #openstack-nova06:22
*** dtruong has quit IRC06:23
*** cheneydc has quit IRC06:23
*** damien_r1 has quit IRC06:23
*** janki has quit IRC06:23
*** damien_r has joined #openstack-nova06:23
*** janki has joined #openstack-nova06:23
*** rodolof has quit IRC06:23
*** annp has joined #openstack-nova06:23
*** rodolof has joined #openstack-nova06:23
*** salv-orlando has joined #openstack-nova06:24
openstackgerritTakashi NATSUME proposed openstack/nova master: Fix 500 error while passing 4-byte unicode data  https://review.openstack.org/40751406:24
openstackgerritTakashi NATSUME proposed openstack/nova master: Fix missing marker functions  https://review.openstack.org/51457906:24
*** edmondsw has joined #openstack-nova06:29
*** suresh12 has quit IRC06:31
*** hemna_ has quit IRC06:33
*** suresh12 has joined #openstack-nova06:33
*** edmondsw has quit IRC06:33
*** suresh12 has quit IRC06:38
*** mdnadeem has joined #openstack-nova06:38
*** gongysh has quit IRC06:40
*** moshele has joined #openstack-nova06:47
*** peter-hamilton has quit IRC06:47
*** edmondsw has joined #openstack-nova06:48
*** trinaths has left #openstack-nova06:52
*** edmondsw has quit IRC06:53
*** moshele has quit IRC06:54
*** trungnv has quit IRC06:57
*** mdnadeem has quit IRC06:57
*** trinaths has joined #openstack-nova06:58
*** Daisy_ has quit IRC06:59
*** rcernin has quit IRC06:59
*** Daisy_ has joined #openstack-nova06:59
*** dtantsur|afk is now known as dtantsur07:01
*** Daisy__ has joined #openstack-nova07:03
*** hshiina3 has joined #openstack-nova07:04
*** Daisy__ has quit IRC07:04
openstackgerritZhenyu Zheng proposed openstack/nova master: Add instance action record for attach/detach/swap volumes  https://review.openstack.org/51720507:04
*** Daisy__ has joined #openstack-nova07:04
*** Daisy_ has quit IRC07:04
*** josecastroleon has quit IRC07:04
*** josecastroleon1 has joined #openstack-nova07:04
*** hshiina2 has quit IRC07:05
*** edmondsw has joined #openstack-nova07:08
*** yamamoto has joined #openstack-nova07:10
*** mdnadeem has joined #openstack-nova07:10
*** yamamoto_ has joined #openstack-nova07:10
*** spectr has joined #openstack-nova07:11
*** spectr has quit IRC07:12
*** edmondsw has quit IRC07:12
*** yamamoto has quit IRC07:14
*** Daisy__ has quit IRC07:14
*** andreas_s has joined #openstack-nova07:19
*** Daisy_ has joined #openstack-nova07:22
*** trungnv has joined #openstack-nova07:23
*** edmondsw has joined #openstack-nova07:23
*** pcaruana has joined #openstack-nova07:30
*** chyka has joined #openstack-nova07:30
*** takashin has left #openstack-nova07:30
*** lpetrut has joined #openstack-nova07:32
*** udesale has quit IRC07:32
*** edmondsw has quit IRC07:33
*** sahid has joined #openstack-nova07:35
*** chyka has quit IRC07:35
*** ijw has quit IRC07:37
*** josecastroleon1 has quit IRC07:40
*** trungnv has quit IRC07:41
*** edmondsw has joined #openstack-nova07:48
*** trungnv has joined #openstack-nova07:52
*** edmondsw has quit IRC07:52
*** ralonsoh has joined #openstack-nova07:56
*** slaweq has joined #openstack-nova07:59
openstackgerritNaichuan Sun proposed openstack/nova master: vgpu: add enabled white list  https://review.openstack.org/51258008:00
openstackgerritNaichuan Sun proposed openstack/nova master: XenAPI: get vGPU stats from hypervisor  https://review.openstack.org/51296508:00
openstackgerritNaichuan Sun proposed openstack/nova master: XenAPI: provide vGPU inventory in compute node  https://review.openstack.org/51621708:00
openstackgerritNaichuan Sun proposed openstack/nova master: XenAPI: create vGPU for instance  https://review.openstack.org/51689908:00
openstackgerritNaichuan Sun proposed openstack/nova master: xenapi(N-R-P): Add API to support vgpu resource provider create  https://review.openstack.org/52031308:01
*** moshele has joined #openstack-nova08:02
*** edand has joined #openstack-nova08:03
*** priteau has joined #openstack-nova08:03
*** karthiks has quit IRC08:06
*** Daisy_ has quit IRC08:06
*** edmondsw has joined #openstack-nova08:08
*** gcb has quit IRC08:12
*** josecastroleon has joined #openstack-nova08:12
*** edmondsw has quit IRC08:13
*** gcb has joined #openstack-nova08:13
*** Daisy_ has joined #openstack-nova08:14
*** rmart04 has joined #openstack-nova08:16
*** salv-orlando has quit IRC08:18
*** slaweq has quit IRC08:18
*** hoonetorg has quit IRC08:18
*** salv-orlando has joined #openstack-nova08:18
*** gcb has quit IRC08:19
*** gcb has joined #openstack-nova08:19
*** udesale has joined #openstack-nova08:19
*** slaweq has joined #openstack-nova08:20
*** Daisy_ has quit IRC08:21
openstackgerritNaichuan Sun proposed openstack/nova master: xenapi: Add parse function to get gpu group  uuid and vgpu type uuid  https://review.openstack.org/52171708:21
*** salv-orlando has quit IRC08:22
*** tesseract has joined #openstack-nova08:23
*** karthiks has joined #openstack-nova08:23
*** slaweq has quit IRC08:24
openstackgerritRodolfo Alonso Hernandez proposed openstack/nova master: Format NIC features using os-traits definitions  https://review.openstack.org/46605108:26
*** edmondsw has joined #openstack-nova08:28
*** yamamoto_ has quit IRC08:29
*** salv-orlando has joined #openstack-nova08:31
openstackgerritAndreas Jaeger proposed openstack/nova master: DNM - test updated releasenotes and sphinx job  https://review.openstack.org/52067708:32
openstackgerritAndreas Jaeger proposed openstack/nova master: Update bindep.txt for doc builds  https://review.openstack.org/52175508:32
*** karthiks has quit IRC08:33
*** edmondsw has quit IRC08:33
*** karthiks has joined #openstack-nova08:33
*** hoonetorg has joined #openstack-nova08:35
*** cheneydc_ has quit IRC08:36
*** lpetrut has quit IRC08:36
*** cheneydc_ has joined #openstack-nova08:37
*** Daisy_ has joined #openstack-nova08:37
*** ijw has joined #openstack-nova08:38
*** slaweq has joined #openstack-nova08:38
*** lpetrut has joined #openstack-nova08:40
*** cheneydc_ has quit IRC08:40
*** cheneydc_ has joined #openstack-nova08:40
openstackgerritZhenyu Zheng proposed openstack/nova master: Add instance action record for attach/detach/swap volumes  https://review.openstack.org/51720508:41
*** ragiman has joined #openstack-nova08:42
*** jpena|off is now known as jpena08:43
*** ijw has quit IRC08:45
*** slaweq has quit IRC08:47
openstackgerritRodolfo Alonso Hernandez proposed openstack/nova master: Change 'InstancePCIRequest' spec field  https://review.openstack.org/44925708:47
openstackgerritRodolfo Alonso Hernandez proposed openstack/nova master: Add Neutron port capabilities to devspec in request  https://review.openstack.org/45177708:47
openstackgerritRodolfo Alonso Hernandez proposed openstack/nova master: Format NIC features using os-traits definitions  https://review.openstack.org/46605108:47
*** slaweq has joined #openstack-nova08:47
*** aloga has quit IRC08:48
*** aloga has joined #openstack-nova08:48
*** edmondsw has joined #openstack-nova08:49
openstackgerritVan Hung Pham proposed openstack/nova master: Remove translation of log messages  https://review.openstack.org/46699508:53
*** edmondsw has quit IRC08:53
fricklerthe config ref for ocata shows options that are only available in pike and later, I'm assuming that this is because they are autogenerated using uncapped oslo.log reqs, how can that be fixed? https://bugs.launchpad.net/nova/+bug/173351508:57
openstackLaunchpad bug 1733515 in OpenStack Compute (nova) "use_journal option not available in ocata" [Undecided,New]08:57
*** yamamoto has joined #openstack-nova08:58
*** lpetrut has quit IRC08:59
*** Sandy619 has joined #openstack-nova09:05
*** Daisy_ has quit IRC09:05
openstackgerritChris Dent proposed openstack/nova master: [placement] Object changes to support last-modified headers  https://review.openstack.org/52163909:05
openstackgerritChris Dent proposed openstack/nova master: [placement] Add cache headers to placement api requests  https://review.openstack.org/52164009:05
*** Daisy_ has joined #openstack-nova09:09
*** gszasz has joined #openstack-nova09:10
*** Daisy_ has quit IRC09:10
*** Daisy_ has joined #openstack-nova09:10
*** andreas_s has quit IRC09:13
openstackgerritxulei proposed openstack/nova master: Delete out-of-date record for resource provider  https://review.openstack.org/52176409:13
*** andreas_s has joined #openstack-nova09:13
*** trungnv has quit IRC09:14
*** slaweq has quit IRC09:14
*** abhishekk has quit IRC09:16
*** diga has quit IRC09:17
*** akekane has joined #openstack-nova09:17
*** andreas_s has quit IRC09:18
*** sree has joined #openstack-nova09:19
*** dtantsur_ has joined #openstack-nova09:21
*** dtantsur has quit IRC09:22
*** dtantsur_ is now known as dtantsur09:22
*** sree has quit IRC09:24
*** andreas_s has joined #openstack-nova09:24
*** andreas_s has quit IRC09:29
*** Sandy619 has quit IRC09:30
*** mvk has quit IRC09:30
*** cdent has joined #openstack-nova09:30
*** mvk has joined #openstack-nova09:30
openstackgerritJianghua Wang proposed openstack/nova master: vgpu: add enabled white list  https://review.openstack.org/51258009:32
openstackgerritJianghua Wang proposed openstack/nova master: XenAPI: get vGPU stats from hypervisor  https://review.openstack.org/51296509:32
openstackgerritJianghua Wang proposed openstack/nova master: XenAPI: provide vGPU inventory in compute node  https://review.openstack.org/51621709:32
openstackgerritJianghua Wang proposed openstack/nova master: XenAPI: create vGPU for instance  https://review.openstack.org/51689909:32
openstackgerritJianghua Wang proposed openstack/nova master: XenAPI: restrict to only enable one vGPU type  https://review.openstack.org/52176909:32
*** lucas-afk is now known as lucasagomes09:32
openstackgerritMerged openstack/osc-placement master: CLI for inventories  https://review.openstack.org/45753309:33
*** yingjun has quit IRC09:34
*** Daisy_ has quit IRC09:35
*** andreas_s has joined #openstack-nova09:36
*** slaweq has joined #openstack-nova09:39
*** slaweq has quit IRC09:39
*** slaweq has joined #openstack-nova09:40
*** andreas_s has quit IRC09:40
*** derekh has joined #openstack-nova09:41
*** lpetrut has joined #openstack-nova09:42
*** andreas_s has joined #openstack-nova09:44
*** yamahata has quit IRC09:46
*** andreas_s has quit IRC09:48
*** andreas_s has joined #openstack-nova09:49
*** andreas_s has quit IRC09:51
*** andreas_s has joined #openstack-nova09:52
*** trungnv has joined #openstack-nova09:56
*** yamamoto has quit IRC09:57
*** yamamoto has joined #openstack-nova10:00
*** cheneydc_ has quit IRC10:00
*** salv-orlando has quit IRC10:00
*** salv-orlando has joined #openstack-nova10:01
*** rcernin has joined #openstack-nova10:01
*** yamamoto has quit IRC10:02
*** yamamoto has joined #openstack-nova10:02
*** janki has quit IRC10:02
*** janki has joined #openstack-nova10:02
*** TuanLA has quit IRC10:05
*** salv-orlando has quit IRC10:05
*** slaweq has quit IRC10:09
*** andreas_s has quit IRC10:11
*** andreas_s has joined #openstack-nova10:12
*** slaweq has joined #openstack-nova10:13
*** slaweq has quit IRC10:14
*** slaweq has joined #openstack-nova10:15
*** andreas_s has quit IRC10:16
openstackgerritZhenyu Zheng proposed openstack/nova master: Add instance action record for attach/detach/swap volumes  https://review.openstack.org/51720510:18
*** gszasz has quit IRC10:21
*** andreas_s has joined #openstack-nova10:23
*** annp has quit IRC10:28
*** jhesketh has quit IRC10:28
*** andreas_s has quit IRC10:28
*** ratailor has quit IRC10:30
*** jhesketh has joined #openstack-nova10:30
*** gmann is now known as gmann_afk10:34
*** ralonsoh has quit IRC10:34
*** ralonsoh has joined #openstack-nova10:34
openstackgerritMoshe Levi proposed openstack/nova master: Don't overwrite binding-profile  https://review.openstack.org/51906610:38
*** erlon has joined #openstack-nova10:40
*** yamamoto has quit IRC10:41
*** yamamoto has joined #openstack-nova10:41
*** ijw has joined #openstack-nova10:43
*** tester352 has joined #openstack-nova10:47
*** ijw has quit IRC10:47
*** ratailor has joined #openstack-nova10:47
*** edmondsw has joined #openstack-nova10:48
moshelestephenfin: hi https://review.openstack.org/#/c/519066/ is ready for review again10:49
tester352Hello! Can anyone help to resolve following issue: https://pastebin.com/raw/9GBXCkas  ? In short - after failed attempt to create instance due to volume quota limit I get "Error: Unable to retrieve attachment information." in horizon.10:52
*** edmondsw has quit IRC10:52
tester352could be a bug because of failed instance creation there should not remain remnants of instance10:53
*** ratailor_ has joined #openstack-nova10:53
*** akekane has quit IRC10:53
*** andreas_s has joined #openstack-nova10:54
*** ratailor has quit IRC10:56
*** ralonsoh has quit IRC10:56
*** chyka has joined #openstack-nova10:57
*** ralonsoh has joined #openstack-nova10:57
openstackgerritMerged openstack/nova master: VMware: serial console log (completed)  https://review.openstack.org/45063610:57
*** trinaths has quit IRC11:00
*** salv-orlando has joined #openstack-nova11:01
*** chyka has quit IRC11:01
*** slaweq has quit IRC11:02
tetsurois alex or jay or efried here?11:02
openstackgerritStephen Finucane proposed openstack/nova master: Fix the usage of instance.snapshot notification sample  https://review.openstack.org/52057911:03
*** slaweq has joined #openstack-nova11:04
*** salv-orlando has quit IRC11:06
*** gszasz has joined #openstack-nova11:06
openstackgerritNaichuan Sun proposed openstack/nova master: (WIP)xenapi(N-R-P): Add API to support compute node resource provider update and create  https://review.openstack.org/52104111:07
*** edmondsw has joined #openstack-nova11:08
*** slaweq has quit IRC11:08
openstackgerritNaichuan Sun proposed openstack/nova master: (WIP)xenapi(N-R-P): Add API to support compute node resource provider update and create  https://review.openstack.org/52104111:09
*** edand has quit IRC11:11
*** edmondsw has quit IRC11:12
*** andreas_s has quit IRC11:13
openstackgerritChris Dent proposed openstack/nova master: [placement] Symmetric GET and PUT /allocations/{consumer_uuid}  https://review.openstack.org/51062611:14
openstackgerritChris Dent proposed openstack/nova master: [placement] POST /allocations to set allocations for >1 consumers  https://review.openstack.org/50007311:14
openstackgerritTakashi NATSUME proposed openstack/nova master: Fix missing marker functions  https://review.openstack.org/51457911:15
*** andreas_s has joined #openstack-nova11:18
*** AlexeyAbashkin has joined #openstack-nova11:24
*** mvk has quit IRC11:27
*** edmondsw has joined #openstack-nova11:28
*** andreas_s has quit IRC11:29
openstackgerritjichenjc proposed openstack/nova master: WIP: Remove quota reserve/commit/rollback  https://review.openstack.org/52147011:29
*** andreas_s has joined #openstack-nova11:30
*** udesale has quit IRC11:30
*** edmondsw has quit IRC11:32
*** andreas_s has quit IRC11:39
*** andreas_s has joined #openstack-nova11:39
*** slaweq has joined #openstack-nova11:40
*** edand has joined #openstack-nova11:42
*** slaweq has quit IRC11:42
*** slaweq has joined #openstack-nova11:43
*** diga has joined #openstack-nova11:44
*** ijw has joined #openstack-nova11:44
*** slaweq_ has joined #openstack-nova11:47
*** slaweq has quit IRC11:47
*** edmondsw has joined #openstack-nova11:48
*** ijw has quit IRC11:49
*** tbachman has quit IRC11:50
*** vladikr has joined #openstack-nova11:50
*** slaweq_ has quit IRC11:51
*** slaweq has joined #openstack-nova11:52
*** edmondsw has quit IRC11:52
*** andreas_s has quit IRC11:53
*** slaweq_ has joined #openstack-nova11:53
*** chyka has joined #openstack-nova11:54
*** slaweq has quit IRC11:56
*** tbachman has joined #openstack-nova11:57
*** mvk has joined #openstack-nova11:57
*** slaweq_ has quit IRC11:58
*** chyka has quit IRC11:58
*** andreas_s has joined #openstack-nova11:58
mdboothAnybody know why I'm getting logged out of gerrit every 5 minutes?12:00
*** tbachman has quit IRC12:01
cdentmdbooth: because gerrit is trying to do you a favor?12:01
*** salv-orlando has joined #openstack-nova12:01
mdboothcdent: Heh. It would be the first time ever, though :)12:02
openstackgerritjichenjc proposed openstack/nova master: Remove command nova-manage quota refresh  https://review.openstack.org/52182912:05
*** andreas_s has quit IRC12:08
*** sdague has joined #openstack-nova12:08
*** vladikr has quit IRC12:13
*** slaweq has joined #openstack-nova12:14
*** andreas_s has joined #openstack-nova12:15
*** pooja_jadhav has quit IRC12:18
openstackgerritjichenjc proposed openstack/nova master: remove nova-manage account and nova-manage project  https://review.openstack.org/52183312:18
*** gcb has quit IRC12:18
*** slaweq has quit IRC12:19
openstackgerritjichenjc proposed openstack/nova master: Remove command nova-manage quota refresh  https://review.openstack.org/52182912:19
*** vivsoni__ has joined #openstack-nova12:20
*** vivsoni_ has quit IRC12:20
openstackgerritMerged openstack/nova master: [placement] Fix a wrong redirection in placement doc  https://review.openstack.org/51716312:23
openstackgerritjichenjc proposed openstack/nova master: remove nova-manage shell command  https://review.openstack.org/52183512:24
*** phuongnh has quit IRC12:25
openstackgerritMerged openstack/nova master: Fix format in live-migration-usage.rst  https://review.openstack.org/51641612:26
openstackgerritMerged openstack/nova master: Using --option ARGUMENT  https://review.openstack.org/51623812:26
openstackgerritMerged openstack/nova master: Document the real behavior of notify_on_state_change  https://review.openstack.org/51626412:27
openstackgerritMerged openstack/nova master: Add a note about versioned notification samples being per-release  https://review.openstack.org/52019312:27
*** chyka has joined #openstack-nova12:27
*** slaweq has joined #openstack-nova12:29
*** chyka has quit IRC12:32
openstackgerritjichenjc proposed openstack/nova master: remove 'nova-manage host' and 'nova-manage agent'  https://review.openstack.org/52183712:33
*** slaweq_ has joined #openstack-nova12:33
*** slaweq has quit IRC12:34
*** pooja_jadhav has joined #openstack-nova12:34
*** pooja-jadhav has joined #openstack-nova12:35
*** pooja-jadhav has quit IRC12:35
*** bhagyashris has quit IRC12:36
*** niraj_singh has quit IRC12:37
*** slaweq_ has quit IRC12:38
openstackgerritEric Fried proposed openstack/nova master: Get auth from context for glance endpoint  https://review.openstack.org/49005712:38
efriedtetsuro Sorry, just saw your ping.  What's up?12:39
efrieddims Definitely an American-centric email, that.  It being Thanksgiving, people eat turkey ("cousin of a yard bird") and watch football (for some reason mriedem_away is a Vikings fan).12:44
*** yangyapeng has quit IRC12:44
tetsurocool12:45
*** yangyapeng has joined #openstack-nova12:45
*** bhagyashris has joined #openstack-nova12:45
*** ijw has joined #openstack-nova12:45
tetsuroi was debugging and thought i solve it, but seems that it was my misunderstanding.12:46
tetsuroForgive me for messing up here... https://review.openstack.org/#/c/519081/12:46
*** moshele has quit IRC12:48
*** edmondsw has joined #openstack-nova12:48
*** lucasagomes is now known as lucas-hungry12:49
*** yangyapeng has quit IRC12:49
tetsuroThe day after tomorrow is a thanksgiving day in Japan.12:49
*** ijw has quit IRC12:50
efriedtetsuro I don't see any harm done on https://review.openstack.org/#/c/519081/ -- what's the summary of this issue?12:51
tetsuroI just thought I had solved it and wondered where to throw a patch so was about to ask you on which patch I should make it.12:52
tetsuroBut now I am still testing, so please forget my ping.12:53
efriedtetsuro Okay, no worries.  Appreciate your efforts.12:53
tetsuronow i should get back home from my office.12:54
efriedtetsuro Looking at that sort, I can't imagine why it's there or what it thinks it's doing.12:54
tetsuroit is sorting not to do inner join at first, I think. Anyway, thanks for answering !12:55
tetsuronot to do outer join at first.12:55
*** vladikr has joined #openstack-nova12:55
tetsuroI gotta go. see you tomorrow.12:57
*** tetsuro has quit IRC12:57
*** chyka has joined #openstack-nova12:58
*** diga has quit IRC12:58
*** jpena is now known as jpena|lunch12:58
*** tianhui has quit IRC13:01
*** chyka has quit IRC13:03
*** efried has quit IRC13:04
*** tbachman has joined #openstack-nova13:10
*** dave-mccowan has joined #openstack-nova13:11
*** tianhui has joined #openstack-nova13:11
*** yangyapeng has joined #openstack-nova13:16
*** dave-mcc_ has joined #openstack-nova13:16
*** pchavva has joined #openstack-nova13:17
*** dave-mccowan has quit IRC13:17
*** efried has joined #openstack-nova13:17
*** tbachman has quit IRC13:18
*** belmoreira has joined #openstack-nova13:18
*** tbachman has joined #openstack-nova13:19
sean-k-mooneymriedem_away: at the ptg we discussed the fact that rodolfo had created a bluprint in os-vif for this but we dont use them there so you suggested just tracking this in nova since os-vif is a nova deliverable.13:21
*** moshele has joined #openstack-nova13:21
sean-k-mooneymriedem_away: so yes https://blueprints.launchpad.net/nova/+spec/vif-port-profile is a nova blueprint for work that is predominately done in os-vif13:22
*** sree has joined #openstack-nova13:25
*** mdnadeem has quit IRC13:26
*** moshele has quit IRC13:27
*** rcernin has quit IRC13:30
*** ratailor_ is now known as ratailor13:31
*** mhenkel has quit IRC13:32
*** suresh12 has joined #openstack-nova13:34
*** mhenkel has joined #openstack-nova13:38
*** suresh12 has quit IRC13:39
*** rodolof has quit IRC13:40
*** ijw has joined #openstack-nova13:40
*** udesale has joined #openstack-nova13:45
*** ijw has quit IRC13:45
*** jaypipes has joined #openstack-nova13:46
*** rmart04 has quit IRC13:48
efriedstephenfin gibi bauzas Anyone up for a couple of easy doc reviews?  https://review.openstack.org/521216  https://review.openstack.org/52156313:49
*** mhenkel has quit IRC13:49
stephenfinefried: Sure thing13:49
efriedTa verramuch13:49
*** jpena|lunch is now known as jpena13:50
*** lucas-hungry is now known as lucasagomes13:51
*** salv-orlando has quit IRC13:51
*** salv-orlando has joined #openstack-nova13:52
*** slaweq has joined #openstack-nova13:52
*** slaweq has quit IRC13:53
*** slaweq has joined #openstack-nova13:53
*** jianghuaw has joined #openstack-nova13:55
*** salv-orlando has quit IRC13:55
*** salv-orlando has joined #openstack-nova13:56
*** mlavalle has joined #openstack-nova13:56
*** gszasz has quit IRC13:59
*** gszasz has joined #openstack-nova14:02
stephenfinsean-k-mooney, ralonsoh: Say, can you get 82599ES (Niantic) NICs in embedded form?14:02
stephenfine.g. as part of the board, rather than a discrete board14:02
*** sahid has quit IRC14:04
*** mgariepy has quit IRC14:05
jianghuawgibi, hi.14:05
*** mkoderer_ has joined #openstack-nova14:06
*** tpatzig_ has joined #openstack-nova14:06
*** dgonzalez_ has joined #openstack-nova14:06
*** david_1 has joined #openstack-nova14:06
ralonsohstephenfin: I'll ask this question to drivers team. But I don't think so: https://ark.intel.com/products/codename/32659/Niantic14:08
*** dgonzalez_ has quit IRC14:08
*** tpatzig_ has quit IRC14:08
*** mkoderer_ has quit IRC14:08
*** david_1 has quit IRC14:08
*** sahid has joined #openstack-nova14:09
*** tbachman has quit IRC14:12
*** READ10 has joined #openstack-nova14:14
gibiefried: hi! I'm still going through my mail backlog, fit those patches are still open when I finish that I can check these14:14
gibijianghuaw: hi!14:14
efriedgibi stephenfin already pushed them.  Thanks!14:14
*** lyan has joined #openstack-nova14:14
*** mgariepy has joined #openstack-nova14:14
openstackgerritStephen Finucane proposed openstack/nova master: Change 'InstancePCIRequest' spec field  https://review.openstack.org/44925714:15
openstackgerritStephen Finucane proposed openstack/nova master: Add Neutron port capabilities to devspec in request  https://review.openstack.org/45177714:15
openstackgerritStephen Finucane proposed openstack/nova master: Format NIC features using os-traits definitions  https://review.openstack.org/46605114:15
jianghuawgibi, are you going to help reviewing this patch? https://review.openstack.org/#/c/512580/14:15
stephenfinbauzas: Want to take a second look at https://review.openstack.org/#/c/466051/14:15
jianghuawgibi, we've got +2 and serveral +1s on it. But need another core help to review it.14:16
jianghuawWe mentioned this patches in last week.14:16
gibijianghuaw: I will try but14:18
*** mhenkel has joined #openstack-nova14:18
gibijianghuaw: I have a long backlog for today14:18
*** toure_biab is now known as toure14:19
jianghuawgibi, I see. It's appreciated if you can put it in the todo queue.14:19
mriedem_awaysean-k-mooney: ack thanks14:19
jaypipesI can't read "TripleO IPSEC Integration" without thinking "Triple-sec integration" and then thinking about Cointreau.14:20
*** mriedem_away is now known as mriedem14:20
*** ratailor has quit IRC14:21
*** takedakn has joined #openstack-nova14:21
*** gbarros has joined #openstack-nova14:23
*** lbragstad has joined #openstack-nova14:24
stephenfinralonsoh: Thought so, but I wasn't sure. Thanks :)14:24
*** andreas_s has quit IRC14:25
stephenfinefried, jaypipes: Am I correct in saying the Memory/DiskFilter is going away/gone thanks to placement? If so, what about the weighers?14:25
*** andreas_s has joined #openstack-nova14:25
efriedstephenfin Weighers won't be affected by placement.14:25
efriedstephenfin Because placement doesn't do any weighing; everything's binary.14:26
jaypipesstephenfin: yes. and what efried said.14:26
mriedemjianghuaw: on https://review.openstack.org/#/c/512580/ - was there a specific reason to use MultiStrOpt over ListOpt?14:26
*** mdnadeem has joined #openstack-nova14:26
efriedstephenfin As for those filters going away... I would expect the answer is yes.14:26
efried...eventually.14:26
stephenfinefried: That's what I was thinking. So does this mean something like this wouldn't be affected by placement? https://review.openstack.org/#/c/379525/14:26
mriedemstephenfin: the core/ram/disk filters are no longer part of the default list of enabled filters b/c of placement14:26
bauzasstephenfin: ralonsoh: so the problem is that we don't have a os-traits tag, right?14:26
stephenfin(not saying it's something we want, but placement is not the reason not to want it)14:26
*** suresh12 has joined #openstack-nova14:26
efriedstephenfin Agree.14:27
stephenfinbauzas: Correct. We need to wait for the next release14:27
bauzasstephenfin: ralonsoh: honestly, we could just create a new release14:27
bauzasmriedem: ^14:27
stephenfinAlso, I wouldn't want to block that patch while we wait for it14:27
*** clayton has quit IRC14:27
jaypipesstephenfin: eventually the *inputs* to the weighers will be created from the returned results of the GET /allocation_candidates call's provider_summaries response, instead of a constructing from the ComputeNode object/cell DB table.14:27
stephenfinThat's also an option :D14:27
bauzasmriedem: context is https://review.openstack.org/#/c/466051/12..15/nova/virt/libvirt/config.py14:27
openstackgerritTakashi NATSUME proposed openstack/nova master: Enable cold migration with target host(1/2)  https://review.openstack.org/40895514:27
stephenfinjaypipes: But filters would still be a nova (vs placement thing), right?14:28
bauzasmriedem: I could just provide a new release for os-traits, fine with you ?14:28
stephenfinSorry - weighters14:28
mriedembauzas: sure14:28
ralonsohbauzas: you are right, os-traits is not updated. But the patch can work as is14:28
jaypipesstephenfin: weighers will continue to be in nova-scheduler, yes14:28
mriedembauzas: you'll be working this week if the gate explodes and the rest of us are on vacation?14:28
bauzasstephenfin: ralonsoh: I'll just provide a new release change for os-traits14:28
* stephenfin goes to unabandon that patch14:28
*** felipemonteiro has joined #openstack-nova14:28
bauzasmriedem: sure, I can14:28
jianghuawmriedem, both should work. But I think bauzas suggested MultiStrOpt when reviewing the spec.14:29
stephenfinjaypipes, efried: If either of you want to review that once I do, I'd appreciate it https://review.openstack.org/#/c/379525/14:29
jaypipesstephenfin: k14:29
*** andreas_s has quit IRC14:30
*** felipemonteiro_ has joined #openstack-nova14:30
stephenfinCheers :) Seems I'm on the hook for nested RP reviews this week too, you'll be happy to hear14:30
mriedemstephenfin: https://review.openstack.org/#/c/379525/ isn't happening for queens14:30
efriedstephenfin This is for a 3-year-old blueprint that's not approved for Q ?14:30
stephenfin /unhappy14:30
bauzasjianghuaw: I'd love to see you reviewing my inventory patch for the VGPU in libvirt14:30
*** clayton has joined #openstack-nova14:30
stephenfinAhh, hold on so. I didn't realize there was a BP attached14:30
jianghuawbauzas, sure. I will. Actually I've opened that patch.14:30
stephenfinIn that case, I'll reopen and target for Rocky14:31
*** suresh12 has quit IRC14:31
efriedSGTM14:31
bauzasjianghuaw: I have multiple problems with libvirt mdevs, but that change would be okay14:31
*** markvoelker has quit IRC14:31
mriedembauzas: any specific reason to use MultiStrOpt over ListOpt here? https://review.openstack.org/#/c/512580/14:31
*** slaweq has quit IRC14:31
*** markvoelker has joined #openstack-nova14:31
stephenfinjianghuaw, bauzas, mriedem: MultiStrOpt is deprecated, afaik14:31
stephenfinOr will be soon. Don't use it14:31
bauzasmriedem: I don't remember me commenting that14:32
*** gszasz has quit IRC14:32
bauzaslemme verify14:32
mriedemstephenfin: it doesn't say that it is https://github.com/openstack/oslo.config/blob/master/oslo_config/cfg.py#L157114:32
bauzashem, no, I haven't said that https://review.openstack.org/#/c/512580/1/nova/conf/vgpu.py14:32
stephenfinmriedem: Hmm, odd. I was sure gcb talked about that at the PTG14:33
mriedemi'm asking in -oslo14:33
*** edand has quit IRC14:33
bauzasmriedem: stephenfin: yeah, don't see it in https://docs.openstack.org/oslo.config/latest/reference/opts.html#oslo_config.cfg.MultiStrOpt14:33
bauzasif that's deprecated, then please change that14:33
*** felipemonteiro has quit IRC14:34
stephenfinI'm pretty sure gcb did talk about it, so maybe it's on his list. In any case, I wouldn't use it. It's very confusing14:34
bauzashonestly, I don't see the difference14:34
bauzasbut meh14:34
bauzasI trust you14:34
bauzasif you say ListOpt should be used, then -1 the change14:34
stephenfinEvery other option is specified once except that special one :)14:34
bauzasAFAIK, you're an oslo core :p14:34
stephenfinHeh, I hate -1ing for that kind of stuff, but ok14:35
openstackgerritSurya Seetharaman proposed openstack/nova master: cleanup mapping/reqspec after archive instance  https://review.openstack.org/51503414:35
*** moshele has joined #openstack-nova14:35
jianghuawbauzas, stephenfin, mriedem: Got it. So will proceed to change it from MultiStrOpt to ListOpts. Thanks.14:36
bauzas++14:36
bauzasI'll need to rebase my change, but meh14:36
stephenfin++ as commented, those things are hard to change after the fact14:36
bauzasjianghuaw: just ping me when it's uploaded so I can rebase my series14:36
stephenfinbauzas: I'm a pbr core - not oslo (yet ;))14:36
jianghuawbauzas, Sure.14:36
bauzasstephenfin: WAT?14:37
bauzas:p14:37
stephenfinProbably for the better 🙈14:37
*** hshiina3 has quit IRC14:37
bauzasralonsoh: FWIW, working on a new release for os-traits14:37
ralonsohbauzas: thank you very much14:38
*** larainema has quit IRC14:39
*** artom has joined #openstack-nova14:39
*** tbachman has joined #openstack-nova14:41
*** ijw has joined #openstack-nova14:41
*** amodi has joined #openstack-nova14:43
*** gszasz has joined #openstack-nova14:44
openstackgerritJianghua Wang proposed openstack/nova master: vgpu: add enabled white list  https://review.openstack.org/51258014:46
openstackgerritJianghua Wang proposed openstack/nova master: XenAPI: get vGPU stats from hypervisor  https://review.openstack.org/51296514:46
openstackgerritJianghua Wang proposed openstack/nova master: XenAPI: provide vGPU inventory in compute node  https://review.openstack.org/51621714:46
openstackgerritJianghua Wang proposed openstack/nova master: XenAPI: create vGPU for instance  https://review.openstack.org/51689914:46
openstackgerritJianghua Wang proposed openstack/nova master: XenAPI: restrict to only enable one vGPU type  https://review.openstack.org/52176914:46
*** ijw has quit IRC14:46
jianghuawbauzas, ^14:46
*** edand has joined #openstack-nova14:47
*** links has quit IRC14:47
openstackgerritStephen Finucane proposed openstack/nova master: Add CPUWeigher  https://review.openstack.org/37952514:48
bauzasjianghuaw: roger, will rebase14:48
jianghuawbauzas, cool.14:49
moshelestephenfin: hi can you review https://review.openstack.org/#/c/519066/ ?14:49
stephenfinmoshele: Big backlog today, but it's on the list :)14:49
moshelestephenfin: so tomorrow :)14:50
stephenfinDepends on how quickly I work through the backlog, heh14:50
moshelestephenfin: :)14:50
openstackgerritBalazs Gibizer proposed openstack/nova stable/pike: Document the real behavior of notify_on_state_change  https://review.openstack.org/52187014:51
openstackgerritDan Smith proposed openstack/nova master: Alternate fix for validating image on rebuild  https://review.openstack.org/52118614:51
openstackgerritDan Smith proposed openstack/nova master: Add regression test for rebuild with new image doubling allocations  https://review.openstack.org/52115314:51
openstackgerritDan Smith proposed openstack/nova master: Fix doubling allocations on rebuild  https://review.openstack.org/52166214:51
dansmithmriedem: you wanna write the reno for ^? I'm struggling to formulate english sentences at the moment14:52
bauzasmriedem: ralonsoh: stephenfin: https://review.openstack.org/#/c/521869/14:52
bauzasmriedem: need your lovely +114:52
mriedemdansmith: sure14:53
stephenfinbauzas: Do we want to merge the requirements updates first14:53
stephenfin?14:53
stephenfinhttps://review.openstack.org/#/q/project:openstack/os-traits+status:open14:53
stephenfinThe rest are WIP14:53
*** slaweq has joined #openstack-nova14:53
bauzasstephenfin: I looked at those14:53
openstackgerritBalazs Gibizer proposed openstack/nova stable/ocata: Document the real behavior of notify_on_state_change  https://review.openstack.org/52187114:53
bauzasstephenfin: and honestly, I don't see why we need to block a new release because of those14:54
stephenfinSweet. Then you have your +114:54
bauzasstephenfin: most of cores will be on vacations by the next days14:54
*** READ10 has quit IRC14:54
bauzasso if we wanna merge a new release and modify g-r, then it needs to be soon14:54
*** weshay is now known as weshay_pto14:55
bauzasmriedem: FWIW, I'm not sure we need to bump g-r given nova reqs.txt is os-traits>=0.3.2 # Apache-2.014:56
bauzasyou okay with that ?14:56
bauzasor do you think it would be better to just amend g-r ?14:56
mriedemif we're doing a release for new code in os-traits to use in nova, then we need to bump the minimum required version in g-r14:56
*** ijw has joined #openstack-nova14:57
stephenfinbauzas: What he said. If someone's on 0.3.2, it would break14:57
bauzask14:57
bauzasI'll provide a new g-r then14:57
stephenfinWhile we're here, these two should probably go in at some point too (or be abandoned) https://review.openstack.org/#/q/project:openstack/os-traits+status:open+branch:stable/pike14:58
* stephenfin has no idea who the stable cores for os-traits are? Same as nova?14:58
*** awaugama has joined #openstack-nova15:00
bauzasthat's correct15:00
openstackgerritEric Fried proposed openstack/nova master: WIP: placement: GET /RPs with aggs, inventories, traits  https://review.openstack.org/52187515:00
mriedembauzas: -115:00
efriedjaypipes cdent ^ Along the lines of what we discussed briefly yesterday.15:00
bauzasmriedem: oh man, good point15:01
bauzasmriedem: It's just a long day that I haven't provided a new release15:01
bauzasmeh15:01
jaypipesefried: k. it's gonna be low priority though :)15:01
mriedempsh15:01
mriedemexcuses excuses15:01
cdentefried: noted15:01
bauzasmriedem: no excuses, I'm a d-bag, that's it15:02
stephenfinbauzas: Well we didn't want to say anything, but...15:02
* stephenfin also forgot about semantic versioning15:02
efriedcdent jaypipes Would I be correct in assuming that we get a significant performance improvement by reducing the number of placement API calls, assuming everything else (amount of db work, etc.) remains the same?15:02
*** mvk has quit IRC15:03
bauzasthere, fixed15:03
jaypipesefried: for the nova-computes running Ironic virt driver, yeah.15:03
*** yamamoto has quit IRC15:03
*** eharney has joined #openstack-nova15:03
jaypipesefried: for the KVM ones, probably won't notice much of a difference.15:03
cdentefried: performance _where_ ?15:03
openstackgerritBalazs Gibizer proposed openstack/nova master: Transform missing delete notifications  https://review.openstack.org/41029715:04
efriedjaypipes cdent  I'm talking about just placement calls.  The above change set allows us to reduce 3N+1 calls down to one, to get the same data.  (N is the number of providers in the tree.)15:04
openstackgerritBalazs Gibizer proposed openstack/nova master: Send soft_delete from context manager  https://review.openstack.org/47645915:04
efriedjaypipes cdent The data as a whole is identical, the work being done on the placement side is identical, etc. - just being done in one API call instead of many.15:05
cdentefried: you’re limiting the opportunities for the placement service to parallize across multiple instances of placement itself, which may or may not be a concern. I tend to prefer lots of smaller calls on many servers15:05
jaypipesefried: understood. and what I'm saying is that N is not a large number for KVM. For Ironic (and maybe powervm) N can be a large number.15:05
openstackgerritMerged openstack/os-traits master: Update the documentation urls  https://review.openstack.org/50112215:05
*** sree_ has joined #openstack-nova15:05
*** sree_ is now known as Guest130215:06
efriedjaypipes Ah, I understand.  Nevertheless, even for a one-node tree, it would be taking 4 calls down to 1.15:06
jaypipesefried: understood. :)15:06
efriedjaypipes I guess I'm asking if the overhead of the API calls is significant.15:06
*** janki has quit IRC15:07
cdentefried: if it is possible to measure instead of speculate that would be great, but my speculation would be that http overhead is not the limiting factor in the big picture in these situations15:07
*** udesale has quit IRC15:07
cdentcertainly as a micro optimization, at webscale™, http request overhead matters15:08
*** armax has joined #openstack-nova15:08
efriedcdent Okay, interesting.  Well, if for no other reason than it makes the client code easier...15:08
*** sree has quit IRC15:08
cdentbut only if it cannot be ameliorated by parallelization15:08
*** ijw has quit IRC15:09
cdentmy gut reaction is that your prematurely optimizing, and that your efforts would be better served by trying to break stuff15:09
cdentso that when we fix stuff, we’re fixing the right stuff15:09
cdentbut I’m sure telling grandma how to suck eggs15:09
cdentefried: Are you familiar with that phrase? I didn’t know it when I first heard it and it totally confused me.15:10
efriedcdent No, but google helped me out.15:10
*** mdnadeem has quit IRC15:10
efriedcdent In the context of the SchedulerReportClient, parallelization would entail some nontrivial rework, likely including spawing extra threads, which doesn't seem likely.15:10
cdentno, I mean parallization at the global level, not the per “query” level15:11
cdentwe are more concerned with the performance of placement being good and predictable for everyone, at large15:11
cdentand given N is almost always quite small...15:12
*** tianhui has quit IRC15:12
openstackgerritIldiko Vancsa proposed openstack/nova master: Implement new attach Cinder flow  https://review.openstack.org/33028515:13
openstackgerritIldiko Vancsa proposed openstack/nova master: libvirt: Allow multiple volume attachments  https://review.openstack.org/26758715:13
efriedcdent Though even at its smallest, we would be reducing the number of calls by 4x15:13
cdentI hear you, but as we have no evidence, yet, that it matters, it’s early optimization, right?15:13
efriedcdent If I was doing it just for performance, yeah.  But I have *hard* evidence that it's gonna make my coding easier :)15:14
cdentwhile making the server more complex15:14
*** takedakn has quit IRC15:14
efriedcdent See this series: https://review.openstack.org/#/c/521605/15:15
*** moshele has quit IRC15:15
* cdent nods15:15
efriedcdent But also https://review.openstack.org/#/c/521098/4/nova/scheduler/client/report.py as we discussed yesterday.15:16
efriedAnyway, I did happen to notice a dearth of functional tests in test_report_client, so maybe I will do as you suggest and go back to breaking things.15:17
cdentefried: I’m not against your idea at all, I just don’t think it is urgent in the face of the usual constraints about review bandwidth, getting other stuff merged in this cycle, etc etc. I’d love it if we operated on a “this is a good idea, let’s just do it” mode but we’re not quite there.15:18
*** andreas_s has joined #openstack-nova15:19
*** mvk has joined #openstack-nova15:19
efriedcdent I'm with you.  Though TBH I'm not sure the amount of code will be more one way or the other.  We either collate on the server side or the client side.15:20
cdentI’m biased to doing collation on the client side because I want the server to be as simple as possible, and will consistently resist adding new stuff, until overwhelmed and then blithely acquiese15:21
*** Guest1302 has quit IRC15:24
*** fnordahl has quit IRC15:24
*** sree has joined #openstack-nova15:24
openstackgerritMerged openstack/nova master: placement: Document request headers in api-ref  https://review.openstack.org/52156315:25
openstackgerritMerged openstack/nova master: placement: Document `in:` prefix for ?member_of=  https://review.openstack.org/52121615:26
artomdansmith, mriedem, is it worth it to "argue" about which filters should be policy at this point?15:26
dansmithartom: if you see something glaringly wrong, point it out15:26
artomI see there are many comments to that effect already, and I haven't read all of them yet15:26
artomdansmith, I wouldn't say glaringly. To me it's "does policy == things can change through a rebuild"?15:27
artomOr rather, is that what we want?15:27
artomOr are we going with the "real" semantic definition of policy, even it it means running some filters needlessly?15:27
*** andreas_s has quit IRC15:27
*** esberglu has joined #openstack-nova15:27
efriedcdent Are you aware of a bug complaining that GET / of the placement API should be unauthenticated?15:28
*** lajoskatona has quit IRC15:28
dansmithartom: well, as we discussed on that review, it's hard to draw a neat box around that15:28
efriedcdent AFAIK, we want every service to permit version discovery (which is what's at that URI) for free.  <== mordred True?15:28
*** sree has quit IRC15:29
artomdansmith, yeah, which is why I'm asking here before leaving a bunch of somments15:29
dansmithartom: I want it to be "affects _where_ a thing should land" vs. "affects whether a host is "allowed" or "can" land" or something15:29
artom*comments15:29
dansmithartom: but clearly for the purposes of this backport, it's mostly just "does this break rebuild"15:29
artomFor instance, the AZ filter... the AZ can't change with a rebuild, can it? Do we need to run it?15:29
dansmithartom: and I'm not running it on rebuild right?15:30
* dansmith checks15:30
dansmithoh I am15:30
dansmiththat might be a mistake.. I was thinking that certain aggregate lines could have changed such that the operator wants to block the rebuild,15:30
dansmithbut az is probably not right15:30
mriedemheh, maybe the variable should just be, RUN_ON_REBUILD15:31
dansmithartom: I was thinking https://review.openstack.org/#/c/521186/5/nova/scheduler/filters/aggregate_instance_extra_specs.py so I might have confused the az one15:31
mordredefried: yes - we want all the things to be able to do version discovery - lemme read more scrollback though15:31
artomThe aggregate stuff as well...15:31
efriedmordred Just the comment two up from that one15:31
artomI grok that aggregate metadata can be changed by an operator15:31
efriedmordred Today placement's GET / requires auth15:31
artomBut nothing about the instance can change to affect its aggregate placement, I think?15:32
dansmithartom: so we have a test that makes sure that rebuild fails if the operator has manually migrated one of your anti-affinity brothers to your same host.. I figure that means that late violation of such a policy should make rebuild fail15:32
mriedemartom: the image can change on rebuild15:32
mriedemso the image meta can change the host aggregates you can be in15:32
mriedemwhich was the original bug15:32
artommriedem, ah, does that specify aggregate stuff? OK, that's ignorance on my part then15:32
* artom goes looking for a comprehensive list of image properties15:33
mriedemefried: didn't you have a devstack patch at one point that depended on https://review.openstack.org/#/c/490057/ and removed the [glance] auth options from nova.conf to see it passing?15:33
efriedmriedem one sec...15:34
mriedemartom: good luck15:34
mriedemoh wait, nevermind, we codify image meta15:34
mriedemnova.objects.image_meta15:34
dansmithyeah it's in there15:34
mriedemwas thinking extra specs15:34
artomThose are flavour though... (right?)15:35
efriedmriedem https://review.openstack.org/#/c/490031/  (It was sdague's)15:35
mordredefried: yes. please make to all version discovery unauthenticated15:35
dansmithhowever, I think the point is that you can put extra_specs on the aggregate that looks at image properties to keep things in place15:35
artomAnd flavour can't change with a rebuild...15:35
mriedemartom: correct,15:35
mriedemflavor can only change on resize15:35
mriedemand we run through the scheduler on resize already15:35
dansmithartom: that's the point of the original bug, you might have landed on a linux host, rebuild to a windows image, and blow a license for the operator15:35
sdagueefried: those test results current? if so we can get that moved in15:37
efriedsdague They're ~6w old.  I'll recheck.15:37
sdagueefried: cool15:37
artomdansmith, going back to anti-affinity... we never checked the scheduler previously, so if during an instance's lifetime an anti-affinity brother migrated to the same host, the rebuild would not fail15:39
artomWhereas now it would15:39
mriedemefried: i'm a bit confused in that we don't even fill in the auth options for [glance] in nova.conf in our dsvm ci runs today,15:40
dansmithartom: we already have a test for it15:40
mriedemdoes nova really not talk to glance anywhere where it needs a token that doesn't come from the user context?15:40
sdaguemriedem: correct15:40
artomdansmith, ah right, I was reading scrollback backwards15:40
mriedemsame for cinder15:40
sdaguemriedem: yes15:40
efriedwhat he said ^15:40
artomThis is a minefield:S15:41
dansmithartom: it's in functional.test_server_group somewhere15:41
sdagueneutron runs mixed mode, some times user token, some times system15:41
*** slaweq has quit IRC15:41
sdagueand ironic is system level, because nova is it's multi tenancy15:41
mriedemthe other thing i remembered i need to look at for this is the xenapi plugins for glance15:41
efriedsdague Neutron some of each?  Hum, I may have screwed up https://review.openstack.org/50989215:41
sdagueefried: yeh, there is a background network info cache refresh iirc15:42
efriedsdague Actually, I didn't change how the auth plugin is retrieved for the neutron patch.  So whatever it was doing before, it's still doing now.15:43
sdagueand, there are times we "escalate" privs because neutron privs for users aren't always sufficient to do server builds15:43
sdagueefried: cool15:43
sdagueyeh, the neutron one is messier15:43
efriedsdague This business - unchanged: https://review.openstack.org/#/c/509892/6/nova/network/neutronv2/api.py@15015:44
mriedemsdague: do you remember any of the xenapi glance plugin whackiness? https://github.com/openstack/os-xenapi/blob/master/os_xenapi/dom0/etc/xapi.d/plugins/glance.py15:44
mriedemi can't tell if any of that would be affected by this15:44
*** gouthamr has joined #openstack-nova15:44
sdagueI do, I can look15:45
*** sree has joined #openstack-nova15:46
sdagueoh, ug, that's in a different tree now, one sec15:46
mriedemyeah as far as i can tell,15:47
mriedemhttps://github.com/openstack/os-xenapi/blob/master/os_xenapi/dom0/etc/xapi.d/plugins/glance.py#L63015:47
efriedThat sucker is hardcoding glance URIs.  And I can't even see how it's doing authentication.15:47
mriedemit's not getting a token,15:47
mriedemso extra_headers would have to include the token15:47
sdagueright15:47
mriedemjianghuaw: still around?15:48
sdaguehttps://github.com/openstack/nova/blob/25afc31b202d66c294a8ac84895e2876cdd6681b/nova/virt/xenapi/image/glance.py#L4015:49
sdagueI think that's where it all comes together15:49
jianghuawmriedem, yes.15:49
sdaguehttps://github.com/openstack/nova/blob/25afc31b202d66c294a8ac84895e2876cdd6681b/nova/image/glance.py#L99-L106 yeh15:50
*** sree has quit IRC15:50
mriedemok, so xenapi just gets the glance identity headers and passes those through to the plugin for upload/download15:50
sdagueyep15:50
sdagueso it's the same basic model, it just does custom serialization15:51
mriedemok15:51
mriedemjianghuaw: nevermind, we were just discussing https://review.openstack.org/#/c/490057/ and whether or not it would affect the xenapi glance plugin in os-xenapi15:51
mriedembut it looks like the user token is passed through15:51
jianghuawmriedem, ok:-)15:51
mriedemxenserver ci did fail on that patch15:52
mriedembut it looks like due to devstack setup failing15:52
mriedemhttp://dd6b71949550285df7dc-dda4e480e005aaa13ec303551d2d8155.r49.cf1.rackcdn.com/57/490057/25/check/dsvm-tempest-neutron-network/c802d48/logs/devstacklog.txt.gz15:52
mriedemTypeError: __init__() got an unexpected keyword argument 'retry_on_request'15:52
efriedmordred Can you point me to a doc somewhere that says discovery URIs should be auth-less?15:53
*** mvk has quit IRC15:54
mriedemweird that TypeError was fixed a few months ago Ic98f8cfeb6f78a7e066613787e7cbcea02fcec5c15:55
mriedemso why is the xenserver ci failing on that now?15:55
mriedemSkip the change: openstack/nova:master:refs/changes/57/490057/2515:56
mriedemwtf15:56
mriedemhttps://review.openstack.org/#/c/490057/ probably just hasn't been rebased in forever15:59
*** jistr is now known as jistr|mtg15:59
*** sree has joined #openstack-nova16:00
*** liuyulong has joined #openstack-nova16:00
mordredefried: https://review.openstack.org/#/c/459710/17/guidelines/discoverability.rst - line 9016:00
*** marst has joined #openstack-nova16:00
mordredefried: it hasn't landed yet because I still need to do another pass16:01
mriedemefried: there https://review.openstack.org/#/c/490057/16:01
jianghuawmriedem, yeah. That patch may need a rebased. I met one similar failure some days ago.16:02
*** yangyapeng has quit IRC16:02
mordredefried: but, tl;dr - it's fundamental metadata about a service and there is nothing about it that is different per user or secret in any way, so protecting it behind auth makes things harder for no benefit16:02
*** yangyapeng has joined #openstack-nova16:03
*** links has joined #openstack-nova16:03
openstackgerritStephen Finucane proposed openstack/nova master: console: introduce framework for RFB authentication  https://review.openstack.org/34539716:03
openstackgerritStephen Finucane proposed openstack/nova master: console: introduce the VeNCrypt RFB authentication scheme  https://review.openstack.org/34539816:03
openstackgerritStephen Finucane proposed openstack/nova master: console: provide an RFB security proxy implementation  https://review.openstack.org/34539916:03
openstackgerritStephen Finucane proposed openstack/nova master: doc: Document TLS security setup for noVNC proxy  https://review.openstack.org/50054416:03
*** yamamoto has joined #openstack-nova16:03
*** andreas_s has joined #openstack-nova16:04
*** sree has quit IRC16:04
*** READ10 has joined #openstack-nova16:04
*** slaweq has joined #openstack-nova16:05
jianghuawjaypipes, bauzas: may you help to check this patch? https://review.openstack.org/#/c/518294/ I think we got some discussion on it before. But do need your review. Eric has +1 on it.16:05
jaypipesjianghuaw: yes, will get to that this afternoon.16:06
*** janki has joined #openstack-nova16:06
jianghuawjaypipes, thanks:-)16:06
efriedmordred Thanks16:06
jaypipesefried: do you have any local changes on that refactor series? cuz I want to fix up the one thing alex_xu wants to see w.r.t. trait validation.16:07
*** mvk has joined #openstack-nova16:07
efriedjaypipes Nope, it's all you.16:07
*** gaoyan has joined #openstack-nova16:07
jaypipesroger16:07
efriedjaypipes You saw my comment there too?16:07
jaypipesyup16:07
jaypipesefried: will try to accommodate both of you16:08
efried:)16:08
*** yangyapeng has quit IRC16:08
*** andreas_s has quit IRC16:09
jaypipesefried: shall I rebase all the way to https://review.openstack.org/#/c/521189/ ?16:09
efriedjaypipes Yes please.16:09
jaypipesyou got it.16:09
*** yamamoto has quit IRC16:09
*** sree has joined #openstack-nova16:10
*** itlinux has joined #openstack-nova16:10
*** links has quit IRC16:12
*** gaoyan has quit IRC16:13
jianghuawefried, jaypipes: After switching to update_provider_tree(), the implementation in driver should cover the things done in _normalize_inventory_from_cn_obj(). right?16:14
efriedjianghuaw That's my position, yes.  Others have yet to confirm.16:14
jaypipesjianghuaw: yep, exactly correct.16:14
*** sree has quit IRC16:14
jianghuawefried, jaypipes thanks. XenAPI will go with that way. Cheers:-)16:15
efriedjianghuaw Note that it's gonna be a while before update_provider_tree is ready...16:15
efriedjianghuaw Lots of code underneath it that jaypipes and I are working on.16:16
*** jistr|mtg is now known as jistr16:16
*** moshele has joined #openstack-nova16:16
*** toure is now known as toure_biab16:16
openstackgerritEric Fried proposed openstack/nova master: WIP: placement: GET /RPs with aggs, inventories, traits  https://review.openstack.org/52187516:17
jianghuawefried, I see. the change will depends your patches and Jay's patches. Just also try to prepare thing as much as possible for XenAPI.16:18
efriedjianghuaw Yup, that's a good idea.  Just be prepared to be patient (and flexible - the API very well might change).16:18
openstackgerritEric Fried proposed openstack/nova master: Get auth from context for glance endpoint  https://review.openstack.org/49005716:19
jianghuawefried, sure. Thanks for the heads-up.16:19
openstackgerritStephen Finucane proposed openstack/os-vif master: Migrate from 'ip' commands to 'pyroute2'  https://review.openstack.org/48438616:21
*** sree has joined #openstack-nova16:21
*** fnordahl has joined #openstack-nova16:22
*** andreas_s has joined #openstack-nova16:23
moshelestephenfin: respond to you comment in https://review.openstack.org/#/c/519066/16:24
*** edmondsw has quit IRC16:24
*** edmondsw has joined #openstack-nova16:25
stephenfinmoshele: replied16:25
*** sree has quit IRC16:25
*** jose-phillips has joined #openstack-nova16:26
stephenfinralonsoh: Rebased your 'pyroute2' patch for os-vif. If you can move that to a lib dir, I think that should keep both sean-k-mooney and sahid happy16:26
ralonsohstephenfin: I'll do this, thanks16:27
*** AlexeyAbashkin has quit IRC16:28
*** sree has joined #openstack-nova16:28
moshelestephenfin: so you want message like https://github.com/openstack/nova/blob/master/nova/network/neutronv2/api.py#L517 ?16:28
stephenfinmoshele: Exactly. Something to give a little more context (and make me more comfortable with ignoring the exception)16:29
*** edmondsw has quit IRC16:29
*** josecastroleon has quit IRC16:29
moshelestephenfin: undestood16:29
*** psachin has quit IRC16:29
openstackgerritEric Fried proposed openstack/nova master: Get auth from context for glance endpoint  https://review.openstack.org/49005716:30
efriedmriedem ^16:30
openstackgerritMerged openstack/nova master: Deduplicate keypair notification samples  https://review.openstack.org/52156216:31
openstackgerritEric Fried proposed openstack/nova master: Use ksa adapter for neutron client  https://review.openstack.org/50989216:31
openstackgerritEric Fried proposed openstack/nova master: Use ksa adapter for keystone conf & requests  https://review.openstack.org/50769316:31
*** andreas_s has quit IRC16:31
openstackgerritEric Fried proposed openstack/nova master: Use ksa adapter for cinder client  https://review.openstack.org/50834516:32
*** liuyulong has quit IRC16:32
*** niraj_singh has joined #openstack-nova16:32
*** sree has quit IRC16:32
niraj_singhi am working on service token for cinder nova interaction.16:32
niraj_singhCinder backend is NFS and Nova backend is LVM.16:32
niraj_singhConf option for nova and cinder: http://paste.openstack.org/show/626972/16:32
niraj_singhi am creating volume snapshot that is attached to an instance.16:32
niraj_singhwhile creating snapshot i am getting error at compute side.16:32
niraj_singhPlease find the log :http://paste.openstack.org/show/626966/16:32
niraj_singhAny help?16:32
*** hemna_ has joined #openstack-nova16:32
bauzasralonsoh: https://pypi.python.org/pypi/os-traits/0.4.0 is there16:33
bauzasralonsoh: could you please provide a new patchset that would use it and just depend on https://review.openstack.org/#/c/521883/ ?16:34
ralonsohbauzas: i'll submit another patch for https://review.openstack.org/#/c/466051/16:34
bauzasralonsoh: cool thanks16:34
*** damien_r1 has joined #openstack-nova16:34
*** niraj_singh has quit IRC16:35
*** toure_biab is now known as toure16:36
*** damien_r has quit IRC16:36
efriedsdague I think this was one you and I talked about: https://review.openstack.org/#/c/512329/16:37
*** belmoreira has quit IRC16:39
*** sree has joined #openstack-nova16:42
*** fragatina has quit IRC16:43
gibijianghuaw: I see I missed the party in https://review.openstack.org/#/c/512580 and now it is on the gate. Sorry for not having the bandwidth to check it earlier16:43
*** fragatina has joined #openstack-nova16:44
*** salv-orlando has quit IRC16:45
*** salv-orlando has joined #openstack-nova16:46
*** fragatina has quit IRC16:46
*** fragatina has joined #openstack-nova16:46
*** sree has quit IRC16:47
openstackgerritMoshe Levi proposed openstack/nova master: Don't overwrite binding-profile  https://review.openstack.org/51906616:48
moshelestephenfin: done ^16:49
*** salv-orlando has quit IRC16:50
*** gbarros has quit IRC16:50
*** fragatina has quit IRC16:50
*** moshele has quit IRC16:51
*** fragatina has joined #openstack-nova16:51
*** gbarros has joined #openstack-nova16:51
openstackgerritJay Pipes proposed openstack/nova master: required traits for no sharing providers  https://review.openstack.org/51702716:52
openstackgerritJay Pipes proposed openstack/nova master: Add ProviderSummary.resource_class_names @property  https://review.openstack.org/51764616:52
*** chyka has joined #openstack-nova16:52
openstackgerritJay Pipes proposed openstack/nova master: handle traits with sharing providers  https://review.openstack.org/51711916:52
openstackgerritJay Pipes proposed openstack/nova master: Test alloc_cands with non overlapping sharing RPs  https://review.openstack.org/51938016:52
openstackgerritJay Pipes proposed openstack/nova master: Test alloc_cands with one RP shared between two RPs  https://review.openstack.org/51961716:52
openstackgerritJay Pipes proposed openstack/nova master: Test allocation candidates: multiple aggregates  https://review.openstack.org/51863316:52
openstackgerritJay Pipes proposed openstack/nova master: Test helper: validate provider summaries  https://review.openstack.org/51898216:52
openstackgerritJay Pipes proposed openstack/nova master: Fix accumulated nits in refactor series  https://review.openstack.org/52118916:52
jaypipesefried: et voila. ^16:52
efriedjaypipes ack16:52
sean-k-mooneyjaypipes: did you just push shared RP support or just traits to model shared and non shared RPs?16:54
*** andreas_s has joined #openstack-nova16:54
*** dtantsur is now known as dtantsur|afk16:55
*** ragiman has quit IRC16:55
*** fragatina has quit IRC16:55
sean-k-mooneystephenfin: regarding embedded niantics i believe Intel sells embedded mezzanine form factor cards to dell/hp and there is also a version that is integrated into the motherboard too.16:57
efriedsean-k-mooney We're kinda working shared in as we go - at least to the point of maintaining existing functionality.  Though we won't officially "support" it in Q, I don't imagine.16:57
*** chyka_ has joined #openstack-nova16:58
*** openstackstatus has quit IRC16:58
*** shaner has quit IRC16:58
*** chyka has quit IRC16:58
stephenfindansmith: Could you take a look at this? https://review.openstack.org/#/c/396184/ Think it's something I should backport after16:58
*** andreas_s has quit IRC16:59
dansmithstephenfin: oh gawd16:59
dansmithstephenfin: yeah, will look when I get off this call16:59
stephenfin(y)16:59
sean-k-mooneyefried: ah cool.16:59
*** shaner has joined #openstack-nova16:59
*** openstackstatus has joined #openstack-nova17:00
*** ChanServ sets mode: +v openstackstatus17:00
*** slaweq has quit IRC17:01
*** salv-orlando has joined #openstack-nova17:02
openstackgerritMatt Riedemann proposed openstack/nova master: Fix NoneType error when [service_user] is misconfigured  https://review.openstack.org/52194717:03
mriedemefried: you'll like this ^ let's not fail with NoneType errors during re-auth17:03
mriedemniraj_singh had a real issue and then left17:04
*** ratailor has joined #openstack-nova17:04
efriedmriedem  https://review.openstack.org/#/c/512329/17:04
sean-k-mooneystephenfin: so looking at that you infer the request for pinning by checking "self.cpu_pinning is not None" what dose self.cpu_pinning contain? will that check work if i explcitly set the policy to shared in the flavour17:05
stephenfinsean-k-mooney: We _used_ to do that, then we added a field to actually store the policy17:05
efriedmriedem And yeah, I just sent him an email.17:05
stephenfinWith recent object versions, we check the policy field. However, that patch ensure the older object versions continue to work17:06
efriedmriedem I bet he didn't realize the conf split that happened a couple months ago, and has his stuff in the wrong conf file.17:06
*** yamamoto has joined #openstack-nova17:06
*** jianghuaw has quit IRC17:06
*** sridharg has quit IRC17:06
mriedemefried: replied on yours17:06
mriedemi don't like raising a random exception here17:06
efriedmriedem Having seen yours, I suspected that would be the case.17:07
mriedemwell,17:07
mriedemit punishes the user for the operator screwing up17:07
efriedmriedem I would rather fail early.17:07
efriedmriedem Because otherwise they probably get the original bug17:07
mriedemplus, it could leak 500s out of the api17:07
mriedemthe original bug is the user token times out and you can't re-auth, which is no different from not using the service user stuff17:08
*** artom_ has joined #openstack-nova17:08
mriedemwith your change, the operator screws up and the api user is punished, plus we probably get 500s for this now17:08
mriedembecause i'm sure there is REST API code calling glance/cinder/neutron client stuff and not handling the error you're raising17:09
efriedmriedem The service will fail very early, likely before the user even got his hands on it.17:09
*** yangyapeng has joined #openstack-nova17:09
*** tesseract has quit IRC17:09
mriedemyou're assuming operators are doing full api test coverage, including all of the proxy apis which are going to hit tis code17:09
*** josecastroleon has joined #openstack-nova17:10
openstackgerritStephen Finucane proposed openstack/nova master: test: Store the OutputStreamCapture fixture  https://review.openstack.org/51514617:10
openstackgerritStephen Finucane proposed openstack/nova master: nova-status: Migrate to cliff  https://review.openstack.org/51514717:10
openstackgerritStephen Finucane proposed openstack/nova master: trivial: Rename 'policy_check' -> 'policy'  https://review.openstack.org/51514817:10
*** ratailor has quit IRC17:10
openstackgerritStephen Finucane proposed openstack/nova master: nova-policy: Migrate to cliff  https://review.openstack.org/51514917:10
efriedmriedem It only has to be hit once from one API.  Like if they create a flavor or an image or whatever.17:10
efriedmriedem Which admins tend to do early on before they hand their cloud off to users, true?17:10
efriedmriedem So I guess we don't cover the case where they've got an existing cloud and they just decide to enable this thing.17:10
mriedemi also wouldn't backport yours17:11
*** yamamoto has quit IRC17:12
mriedembecause if i'm on stable and pick that up, and all of a sudden i start getting 500s, i'd be annoyed17:12
*** artom has quit IRC17:13
*** yangyapeng has quit IRC17:14
efriedmriedem Is backporting a consideration?  The service user thing is experimental, right?17:14
mriedemi don't consider it experimental17:14
mriedemit's been in since ocata and we run with it enabled in our nova-next job17:14
mriedemso yes i was going to backport17:14
mriedemi've also marketed this feature during 2 summit project update talks17:15
efriedmriedem https://github.com/openstack/nova/blob/master/nova/conf/service_token.py#L4317:15
efriedSo I guess we oughtta take that line out ^17:15
jaypipessean-k-mooney: that series touches both traits and shared resource providers17:15
*** lucasagomes is now known as lucas-afk17:15
jaypipessean-k-mooney: it's a combo of me, efried and gibi.17:15
mriedemefried: yeah i'd be fine with that17:16
mriedemi'd leave it disabled by default17:16
*** purplerbot has quit IRC17:18
*** purplerbot has joined #openstack-nova17:18
*** edand has quit IRC17:19
openstackgerritEric Fried proposed openstack/nova master: Service token is not experimental  https://review.openstack.org/52195517:21
efriedmriedem ^17:21
*** janki has quit IRC17:22
mriedemwanna remove the 'this is disabled by default' line?17:23
mriedemthen i'm +217:23
*** Apoorva has joined #openstack-nova17:23
dansmithartom_: good comments, thanks for those17:23
dansmithmriedem: you might want to look at those before you push up a rev and see if you have opinions17:24
openstackgerritStephen Finucane proposed openstack/nova master: conf: Remove deprecated 'multi_instance_display_name_template' opt  https://review.openstack.org/49961217:24
openstackgerritStephen Finucane proposed openstack/nova master: Simplify instance name generation  https://review.openstack.org/51657317:24
artom_dansmith, yey, my brain works!17:24
sean-k-mooneystephenfin: sorry had to step away for a minute. well what i was wondering is wether "is not none" is enough e.g. can self.cpu_pinning contain "shared"17:27
mriedemefried: +2 on https://review.openstack.org/#/c/490057 - thanks for the quick updates17:29
stephenfinsean-k-mooney: cpu_pinning contains a dict of host to guest CPU mappings17:29
efriedmriedem Thanks.17:30
stephenfinYou're thinking of 'cpu_policy', which would contain 'shared' or 'dedicated'17:30
* stephenfin is catching a lift and has to run himself17:30
openstackgerritMerged openstack/nova master: Merge flavor extensions controller code  https://review.openstack.org/51610417:30
*** fragatina has joined #openstack-nova17:32
sean-k-mooneystephenfin: yes i just noticed  self.cpu_pinning is a proxy field for cpu_pinning_raw whic  is a dict of integers presumable the vCPU to pCPU mappings17:33
efriedmriedem I dup'd my bug (https://bugs.launchpad.net/nova/+bug/1724689) to yours and abandoned my patch.17:34
openstackLaunchpad bug 1733642 in OpenStack Compute (nova) "duplicate for #1724689 AttributeError: 'NoneType' object has no attribute 'get_token'" [Medium,In progress] - Assigned to Matt Riedemann (mriedem)17:34
mriedemefried: ok, i was going to dupe my bug against yours but ok :)17:34
artom_I feel like naming it "policy" is confusing us17:34
artom_As I writing an internal email about it, I came up with "qualitative" (what we're calling policy) vs "quantitative" (ie, resources)17:35
artom_Does that work better? (Or at all?)17:35
*** artom_ is now known as artom17:36
openstackgerritEric Fried proposed openstack/nova master: Service token is not experimental  https://review.openstack.org/52195517:36
efriedmriedem That's done ^17:36
mriedem+217:37
efriedsdague Service token-y stuff while you're in the mood: https://review.openstack.org/#/c/521947/  https://review.openstack.org/52195517:38
dansmithartom_: I don't really think that has much more meaning to me, but I'm obviously biased17:40
dansmithartom_: I'll let mriedem make the call on what the name should be17:40
artomUgh, it'll be some obscure 80s rock band17:40
mriedemi agree it's confusing that _GroupAntiAffinityFilter is a policy filter but _GroupAffinityFilter isn't17:41
dansmithfine by me :P17:41
sdagueefried: looking17:41
mriedemi mean, we could just call the damn thing RUN_FOR_REBUILD but that doesn't help a ton with the reasoning behind which filters should be run during rebuild or not17:41
dansmithand I expect we need to re-use much of the logic for some other things where we need to check with the scheduler17:42
mriedemexample?17:42
dansmithalthough I guess resize does need to check the resourcey things17:42
dansmithI dunno17:42
dansmithfeels far too targeted to just say it's for resize17:42
mriedemi think rebuild is just the odd duck17:42
dansmithwell, if that's really the case, then maybe we should just be specific until we have a counterexample17:43
mriedemresize runs through the scheduler because it's going to do a claim on the chosen host17:43
dansmithyeah17:43
mriedemlive migration doesn't do a claim17:43
mriedembut live migration does allocate on the dest host17:43
mriedemusing the same flavor17:43
sdagueefried: so, on https://review.openstack.org/#/c/490057/27/nova/api/openstack/compute/servers.py, while it's fine to pass context around, did you look into just pulling it from thread local storage?17:43
sdagueefried: https://github.com/openstack/oslo.context/blob/18aa6ec496e54b2403c2cc65234ef6447021fdee/oslo_context/context.py#L490-L49517:44
efriedsdague I didn't.17:44
efriedsdague tbh, mordred did the code-side work on that patch.  I just did the test, and kept it up to date.17:44
sdagueok, the long pass through of some of these variables gets a little spidery. It's probably fine here, but something to think about in the future17:44
mriedemthat's why i worried about something doing context.get_admin_context() and that eventually going through here17:45
mriedembut apparently we don't do that with images17:45
efriedsdague Yeah, that would have made things a lot easier.  Is that the same thing?17:45
*** pcaruana has quit IRC17:48
sdaguemriedem: yeh, I could see that17:49
sdagueit's one of those things where context was written originally to not have to pass it around17:49
sdaguebut that got forgotten at some point17:49
*** ralonsoh has quit IRC17:49
sdagueso, meh17:49
sdagueefried: +A on that patch, I'll look at the rest of the conversions on top once I get some lunch17:50
efriedsdague Thank you sir.17:50
efriedsdague (mriedem) Note that the cinder one might still be busted.  But now that this one is going through, I'll get back to looking at that.17:50
mriedemefried: it does remind me that someone had a patch to add some code for nova to auth with a service user to do things with cinder like we do for neutron17:52
mriedemi think in one of the periodic tasks17:52
efriedmriedem Yeah, I couldn't remember if that was niraj_singh, but I don't think so.17:52
*** claudiub|2 has quit IRC17:52
mriedemefried: https://review.openstack.org/#/c/384799/17:52
efriedmriedem Oh, actually I was thinking of someone who was trying to use service auth from *within* cinder.17:53
*** READ10 has quit IRC17:53
*** sree has joined #openstack-nova17:53
artomdansmith, mriedem, I think we need to get stephenfin to look at the PCI and NUMA bits when he comes back tomorrow17:53
dansmithokay I'm off tomorrow and mriedem is too I think17:53
mriedempci is extra specs only17:53
mriedemi asked stephen about that yesterday17:54
mriedemnuma is definitely a resource thing17:54
mriedemwe do claims on numa limits in the resource tracker17:54
artommriedem, can an admin change them though?17:54
*** yamahata has joined #openstack-nova17:54
mriedeman admin could change the pci passthrough settings in flavor extra specs, sure17:54
artomFor NUMA the cpu and thread policy can change, I dunno if that can give rise to the situation where the host can no longer accept them17:54
mriedemchanging extra specs on an existing flavor would be kind of a bad move i think17:55
*** damien_r1 has quit IRC17:55
mriedembut we allow it17:55
dansmiththe flavor embedded in the instance will have the pci stuff then, which means we won't see the changed extra_specs from the top-level flavor in the rebuild17:55
dansmiththe cpu and thread policy change in the image meta might be a reason to do the rebuild though,17:56
dansmithso I dunno what to say about that17:56
*** sree has quit IRC17:57
artomdansmith, oh right, that's true, flavours stay with instances now17:57
*** hongbin has joined #openstack-nova17:58
artomSo anything involving flavor extra specs isn't policy then - ie, no need to check with it on rebuild, it hasn't changed for the actual instance17:58
dansmithwell, no17:58
*** derekh has quit IRC17:58
dansmithflavors stay with the instance, flavours are not a thing in nova :P17:58
artomWhat do you mean not a thing?17:58
artomOh17:59
dansmithartom: yeah, that's probably a good guideline, although some flavor extra_specs get overridden by image meta in some weird ways I think17:59
artomUgh.18:00
mriedemNUMATopologyFilter gets values from flavors first, and if not set, checks the image meta18:00
artomJust set everything to policy and wait for the bug reports then >:(18:00
mriedemso flavor extra specs take precedence there18:00
dansmithartom: that's the way it is now, and the bug we're fixing :P18:00
mriedemoh wait nvm that18:00
mriedemre the question about the az filter18:01
mriedemhttps://review.openstack.org/#/c/446446/18:01
mriedem"Unfortunately, as an operator18:02
mriedemcan modify any metadata, it is possible to change the value of the18:02
mriedemavailability_zone key for a specific aggregate and accordingly change what the18:02
mriedemuser can see when looking at AZs"18:02
*** jistr is now known as jistr|off|trng18:04
openstackgerritMerged openstack/nova master: vgpu: add enabled white list  https://review.openstack.org/51258018:05
*** moshele has joined #openstack-nova18:07
*** yamamoto has joined #openstack-nova18:08
*** mvk has quit IRC18:09
*** david-lyle has quit IRC18:09
*** david-lyle has joined #openstack-nova18:09
mriedemwe persist the flavor with the request spec yes?18:10
dansmithflavor id you mean?18:10
mriedemthe full flavor18:10
mriedemlooks like yes18:11
dansmithhrm, I guess we do18:11
mriedemwhich means, re the TrustedFilter question,18:11
mriedemif the admin changes the flavor extra specs for the trusted filter after you've created an instance, when you rebuild or migrate that instance, the filter is going to work on the original flavor used to create the instance18:11
mriedemnot the modified one18:11
mriedem*i think*18:12
artomThat's what I understood when dansmith said that flavoUrs stay with the instance, yeah18:12
*** yamamoto has quit IRC18:12
*** _pewp_ has quit IRC18:14
*** sean-k-mooney has quit IRC18:16
mriedemmy favorite burstlet flavour is tripe18:17
mriedemartom: dansmith: ok, some replies inline https://review.openstack.org/#/c/521186/18:17
*** _pewp_ has joined #openstack-nova18:18
mriedemdansmith: btw, i changed _GroupAntiAffinityFilter to False, ran functional tests and nothing failed18:18
dansmithmriedem: it'll be non-deterministic18:19
mriedemgha18:19
mriedem*gah18:19
dansmithbecause if it selects the other host by chance it'll be happy18:19
*** Swami has joined #openstack-nova18:19
dansmithit was oddly repeatable for me.. two tests validating the same thing (for two microversions) one would always pass the other would always fail18:19
dansmithtook me a while to figure out why18:19
dansmithI was able to shake it up with different levels of concurrency and running the tests in isolation18:20
mriedemtest_rebuild_with_soft_anti_affinity ?18:20
*** jpena is now known as jpena|off18:20
mriedemor test_rebuild_with_anti_affinity18:20
dansmithsoft I think18:20
*** moshele has quit IRC18:21
mriedemthat one involves a weigher so yeah might be that one18:21
dansmithI can try to repro if it's important18:21
mriedemwill run it a few times18:21
openstackgerritEric Fried proposed openstack/nova master: Use ksa adapter for cinder client  https://review.openstack.org/50834518:21
mriedemi am dangerously close to needing some deterministic lunch18:22
dansmithmriedem: http://logs.openstack.org/86/521186/3/check/openstack-tox-functional/dc8ab81/testr_results.html.gz18:23
dansmiththat was the failure I tried to repro locally that led me down the path I think18:23
*** felipemonteiro__ has joined #openstack-nova18:23
*** slaweq has joined #openstack-nova18:24
*** felipemonteiro_ has quit IRC18:26
*** ijw has joined #openstack-nova18:31
*** ijw has quit IRC18:35
*** READ10 has joined #openstack-nova18:36
*** READ10 has quit IRC18:36
*** AlexeyAbashkin has joined #openstack-nova18:40
*** AlexeyAbashkin has quit IRC18:44
*** imacdonn has quit IRC18:45
*** imacdonn has joined #openstack-nova18:45
dansmithartom_: mriedem well, now I can't get it to fail18:48
*** pchavva has quit IRC18:49
mriedem:/18:49
dansmithoh, wait18:49
mriedemwas that with same_host or something?18:49
dansmithI bet it's because I hit this before I was doing the force_hosts18:49
mriedemah18:49
dansmithit was when I was going to do the check in conductor manager I think18:50
mriedemso we can probably change that one then18:50
mriedemwhich would help with the confusion18:50
dansmithyeah I guess so18:50
mriedemdansmith: artom: in general, what are your thoughts/feelings/hopes/dreams about just calling the hint _is_rebuild and the filter attribute RUN_FOR_REBUILD? then each filter decision is about that scope/use case rather than what policy means18:51
dansmithI dunno, it feels too specific, but I can't come up with another reason to do this really18:52
mriedemright, like you said, we could change it later if we have a need to generify it18:52
dansmithyeah18:53
mriedemwe could also maybe remove the variable at some point later with rpc version changes, like passing an actual is_rebuild parameter to select_destinations or something18:53
dansmithyeah18:53
mriedemi'm not sure that would buy us much, except maybe more straight-forward18:53
dansmithideally it'd be operation="rebuild" to give us some room for the future18:53
dansmithbut yeah18:53
mriedemsure18:53
mriedemoperation="mind crime"18:54
dansmithheh18:54
mriedem80s queensryche reference for artom18:54
*** mvk has joined #openstack-nova18:54
*** itlinux has quit IRC18:57
*** slaweq has quit IRC18:58
artomI was -5 for half the 80s, though I do enjoy some of their music now :)18:58
artomRenaming it to RUN_FOR_REBUILD would definitely make the scope clearer18:58
*** slaweq has joined #openstack-nova18:58
artomI agree it's not pretty18:58
*** Sukhdev has joined #openstack-nova18:59
artomThough I'm not as repulsed by the api <-> scheduler coupling - the scheduler knowing about rebuilds isn't as far fetched as it sounds18:59
artomI think ideally I'd like more methods in the filters, like I said in my last comment18:59
artomSo host_passes for "can this host run this instance now?", host_acceptable for "can this host accept this instance, resource considerations aside", and maybe other ones for cases we haven't thought of19:00
artomBut that doesn't look very backportable19:00
*** jeblair is now known as thecount19:01
*** thecount is now known as jeblair19:01
*** slaweq has quit IRC19:03
*** Gorian has quit IRC19:03
*** itlinux has joined #openstack-nova19:03
*** Sukhdev has quit IRC19:04
*** gbarros has quit IRC19:04
openstackgerritMerged openstack/nova master: cleanup mapping/reqspec after archive instance  https://review.openstack.org/51503419:04
*** gbarros has joined #openstack-nova19:05
mriedemmgagne: a couple of small changes to make here and then i think we're good https://review.openstack.org/#/c/50308819:05
mriedemartom: the scheduler is definitely aware of moves for placement allocations, even if it's not super obvious19:06
mgagnemriedem: will work on it today. I think committed the Ironic release note by mistake, will reword as suggested19:06
mriedemmgagne: yeah heh, that was my comment :)19:07
mgagnedidn't read yet :P, will do19:07
artommriedem, yeah, it decides which instances go to which hosts, to operations on instances that have to do with hosts aren't outside its domain :)19:07
artom*so operations19:08
*** Gorian has joined #openstack-nova19:08
artomIf we started telling it about keypair updates I'd be more concerned ;)19:08
mriedemartom: i mean this specifically https://github.com/openstack/nova/blob/master/nova/scheduler/client/report.py#L103519:08
*** ijw has joined #openstack-nova19:08
*** yamamoto has joined #openstack-nova19:09
*** hoonetorg has quit IRC19:09
*** hoonetorg has joined #openstack-nova19:10
*** AlexeyAbashkin has joined #openstack-nova19:11
*** ijw has quit IRC19:12
*** yamamoto has quit IRC19:13
*** gbarros has quit IRC19:13
*** gbarros has joined #openstack-nova19:14
efriedcdent Still around?19:16
*** AlexeyAbashkin has quit IRC19:16
cdentyessir19:16
mgagnemriedem: so should I refactor _generate_configdrive or not?19:16
efriedcdent Is there a way in gabbi for me to say "This must *not* be in the response" ?19:16
cdentin a body or header?19:17
efriedbody19:17
efriedI know about checking the length of a chunk of the response; will fall back on that if nothing else is available.19:18
cdentthere are a couple of additional options:19:18
cdentif the surrounding chunk of json is small and you know all of it, you can make the value of the jsonpath be the entire chunk, instead of just a scalar value19:19
cdentor you might be able to do something with regular expressions19:19
cdent(the value side of the jsonpath statement can be a regex)19:19
cdentefried: if you got a pointer to an almost there example I can probably say something more concrete19:20
*** marst has quit IRC19:20
efriedcdent Like here: https://github.com/openstack/nova/blob/master/nova/tests/functional/api/openstack/placement/gabbits/resource-provider.yaml#L10519:21
efriedcdent I want to make sure there's no "traits" key19:21
efriedetc.19:21
* cdent thinks19:21
efriedresponse_forbidden_json_paths:\n\t$.resource_providers[0].traits19:22
cdentheh19:22
cdentis your intent to validate the entire response? If so, you might consider:19:23
cdentresponse_json_paths:\n$: @<my-big-response.json19:23
mriedemmgagne: no, i think what you have is ok19:24
efriedcdent Right; though the docs kinda recommended against that.19:24
mriedemthe error handling being different makes a refactor less worthwhile19:24
openstackgerritEric Young proposed openstack/nova master: Implement ScaleIO image backend  https://review.openstack.org/40744019:24
mgagnemriedem: got it19:24
cdentefried: they also recommend not validating entire responses19:24
cdent(in one test)19:24
efriedcdent Specifically in this case I'd like to ensure I don't return the extra fields when I don't request them.  (Related to the traits/aggregates/inventories thing from this morning)19:25
efriedcdent (Yes, I realize I haven't abandoned this patch, but I'm taking it as an opportunity to learn gabbi.)19:25
cdent:)19:26
cdentanother option would be to write your own CustomHandler that actually implements response_forbidden_json_paths19:26
* cdent looks at the jsonpath_rw docs19:27
efriedcdent That would live in nova, or that would be contributed back to gabbi?19:27
efriedoh, jsonpath_rw, that's a separate thing.  /me looks too...19:28
*** slaweq has joined #openstack-nova19:28
*** gszasz has quit IRC19:29
*** pchavva has joined #openstack-nova19:29
cdentI was thinking live in nova initially, and then merged to gabbi if it felt sufficiently useful/nice19:30
*** tianhui has joined #openstack-nova19:30
openstackgerritEric Berglund proposed openstack/nova master: WIP(5): PowerVM driver: ovs vif  https://review.openstack.org/42251219:31
cdentefried: this might be a case where it would be better to have a unit test of your serializer19:32
efriedcdent Also.19:33
dansmithmriedem: you have some changes against the set locally, right? will you make the hint change and the other things we discussed as part of that?19:34
mriedemdansmith: i don't have anything local19:34
*** slaweq has quit IRC19:34
dansmithoh okay I thought you were working on the reno and flipping some of the flags19:35
efriedcdent Thanks for the help.  A CustomHandler might be more than I'm willing to bite off at the moment, but I'll pester you if I decide to have a crack at it.19:35
dansmithI'll go ahead and do that other stuff then19:35
mriedemdansmith: that got sidetracked by the review comments and waiting to hear back from some people19:35
dansmithokay19:35
*** tianhui has quit IRC19:37
openstackgerritVipin Balachandran proposed openstack/nova master: VMware: Support volumes backed by VStorageObject  https://review.openstack.org/52169019:37
*** NightKhaos has quit IRC19:39
*** ijw has joined #openstack-nova19:41
*** NightKhaos has joined #openstack-nova19:44
openstackgerritChris Dent proposed openstack/nova master: [placement] Symmetric GET and PUT /allocations/{consumer_uuid}  https://review.openstack.org/51062619:44
openstackgerritChris Dent proposed openstack/nova master: [placement] POST /allocations to set allocations for >1 consumers  https://review.openstack.org/50007319:44
*** ijw has quit IRC19:47
*** josecastroleon has quit IRC19:48
*** ijw has joined #openstack-nova19:48
efriedcdent Sanity check me here: POST /resource_providers/{uuid}/inventories is not documented?19:49
cdentyeah, it was sort deprecated19:50
cdentthere’s a review somewhere adding it back in, that’s under discussion19:51
efriedcdent Oh, okay.  So I won't hurry to document it.19:51
openstackgerritMerged openstack/python-novaclient master: Microversion 2.54 - Enable reset keypair while rebuild  https://review.openstack.org/51957219:51
cdentefried: here’s that review https://review.openstack.org/#/c/511342/19:53
*** ijw has quit IRC19:55
*** suresh12 has joined #openstack-nova20:00
*** cdent has quit IRC20:00
*** slaweq has joined #openstack-nova20:01
openstackgerritAndreas Jaeger proposed openstack/nova master: Update bindep.txt for doc builds  https://review.openstack.org/52175520:02
*** tbachman has quit IRC20:02
openstackgerritMerged openstack/nova master: Get auth from context for glance endpoint  https://review.openstack.org/49005720:04
*** suresh12 has quit IRC20:05
*** slaweq has quit IRC20:06
*** pchavva has quit IRC20:07
openstackgerritAndreas Jaeger proposed openstack/nova stable/pike: Update bindep.txt for doc builds  https://review.openstack.org/52199920:09
*** tbachman has joined #openstack-nova20:09
*** jose-phillips has quit IRC20:09
openstackgerritAndreas Jaeger proposed openstack/nova stable/ocata: Update bindep.txt for doc builds  https://review.openstack.org/52200020:09
openstackgerritAndreas Jaeger proposed openstack/nova stable/newton: Update bindep.txt for doc builds  https://review.openstack.org/52200120:09
*** yamamoto has joined #openstack-nova20:10
*** AlexeyAbashkin has joined #openstack-nova20:11
*** jose-phillips has joined #openstack-nova20:12
openstackgerritHongbin Lu proposed openstack/nova master: Skip authentication on version discovery URI  https://review.openstack.org/52200220:12
*** yamamoto has quit IRC20:14
*** AlexeyAbashkin has quit IRC20:15
*** salv-orlando has quit IRC20:15
*** salv-orlando has joined #openstack-nova20:16
openstackgerritHongbin Lu proposed openstack/nova master: placement: skip authentication on root URI  https://review.openstack.org/52200220:16
*** salv-orlando has quit IRC20:20
*** ijw has joined #openstack-nova20:22
*** tbachman has quit IRC20:22
openstackgerritDan Smith proposed openstack/nova master: Alternate fix for validating image on rebuild  https://review.openstack.org/52118620:23
openstackgerritDan Smith proposed openstack/nova master: Add regression test for rebuild with new image doubling allocations  https://review.openstack.org/52115320:23
openstackgerritDan Smith proposed openstack/nova master: Fix doubling allocations on rebuild  https://review.openstack.org/52166220:23
dansmithmriedem: artom: ^20:24
openstackgerritMathieu Gagné proposed openstack/nova master: Regenerate and pass configdrive when rebuild Ironic nodes  https://review.openstack.org/50308820:28
artomdansmith, looking20:29
dansmithI think I got everything but we talked about lots of things20:29
*** sahid has quit IRC20:29
openstackgerritAndreas Jaeger proposed openstack/nova master: DNM - test updated releasenotes and sphinx job  https://review.openstack.org/52067720:30
*** slaweq has joined #openstack-nova20:35
mriedemdansmith: comments inline20:37
mriedemmostly just nits20:37
mriedemi can update next week if you're tired of this20:37
*** salv-orlando has joined #openstack-nova20:37
*** vladikr has quit IRC20:38
*** vladikr has joined #openstack-nova20:38
*** suresh12 has joined #openstack-nova20:38
*** suresh12 has quit IRC20:40
*** slaweq has quit IRC20:40
*** ijw_ has joined #openstack-nova20:41
dansmithoh I'm tired of it for sure but:20:41
openstackgerritDan Smith proposed openstack/nova master: Alternate fix for validating image on rebuild  https://review.openstack.org/52118620:41
openstackgerritDan Smith proposed openstack/nova master: Add regression test for rebuild with new image doubling allocations  https://review.openstack.org/52115320:41
openstackgerritDan Smith proposed openstack/nova master: Fix doubling allocations on rebuild  https://review.openstack.org/52166220:41
dansmithapparently I broke the top patch in the process here20:41
* dansmith re-runs tests20:41
*** sbezverk has quit IRC20:42
mriedemdid you drop the filter scheduler claim part?20:43
*** ijw has quit IRC20:44
mriedemoh nvm i'm looking at the wrong patch20:45
mriedemdansmith: yeah this needs updating https://review.openstack.org/#/c/521662/5/nova/scheduler/filter_scheduler.py20:45
mriedem+1 on the bottom change though20:47
artomHah, I think you fixed some of what I was about to post before I had a chance to post it20:49
* artom starts over with 720:49
dansmithmriedem: yep, that's it20:50
openstackgerritDan Smith proposed openstack/nova master: Fix doubling allocations on rebuild  https://review.openstack.org/52166220:51
*** ijw_ has quit IRC20:52
openstackgerritMerged openstack/nova master: Service token is not experimental  https://review.openstack.org/52195520:52
*** priteau has quit IRC20:54
*** swebster has joined #openstack-nova20:55
*** moshele has joined #openstack-nova20:59
mriedemok barring renos and such i think that series is good20:59
*** suresh12 has joined #openstack-nova21:00
*** Guest25442 has quit IRC21:01
artomdansmith, two nits from me. And I need to look at the tests eventually21:02
*** moshele has quit IRC21:03
*** ijw has joined #openstack-nova21:04
dansmithartom: mriedem is going to write the reno, so I'll let him update those things if he agrees21:04
dansmithI'm going to be fading out here in a bit21:04
artomSounds good21:05
mriedemthe numa one is probably a change, i forgot that was using image props21:08
mriedembut yeah next week at this point21:08
mriedemi'm out of here in less than an hour21:08
*** ijw has quit IRC21:08
artomEnjoy flightless birds and excessive consumption :)21:08
mriedemi don't actually care for turkey all that much21:08
artomMaybe I meant emus, for all you know21:08
mriedemgross21:09
artomOr kiwis21:09
mriedemi am no longer in australia so none of those animals apply21:09
melwittwe ate some emu while in australia. it was good21:09
*** salv-orlando has quit IRC21:11
*** yamamoto has joined #openstack-nova21:11
*** sbezverk has joined #openstack-nova21:11
*** slaweq has joined #openstack-nova21:11
*** salv-orlando has joined #openstack-nova21:11
*** yamamoto has quit IRC21:15
mriedemlet me go tell laura21:16
*** salv-orlando has quit IRC21:16
*** slaweq has quit IRC21:16
*** marst has joined #openstack-nova21:19
*** lpetrut has quit IRC21:21
*** sree has joined #openstack-nova21:21
*** vishwanathj has joined #openstack-nova21:23
*** sree has quit IRC21:26
*** tbachman has joined #openstack-nova21:26
*** tssurya_ has joined #openstack-nova21:31
*** tssurya_ has quit IRC21:32
*** eharney has quit IRC21:37
*** salv-orlando has joined #openstack-nova21:41
*** sree has joined #openstack-nova21:47
*** dave-mcc_ has quit IRC21:47
*** threestrands has joined #openstack-nova21:48
*** rcernin has joined #openstack-nova21:50
*** sree has quit IRC21:52
*** tbachman has quit IRC21:53
openstackgerritHongbin Lu proposed openstack/nova master: placement: skip authentication on root URI  https://review.openstack.org/52200221:54
*** awaugama has quit IRC21:56
*** ijw has joined #openstack-nova21:56
*** mvk has quit IRC21:59
*** ijw has quit IRC22:00
*** ijw has joined #openstack-nova22:00
*** Apoorva_ has joined #openstack-nova22:02
*** mriedem has quit IRC22:05
*** Apoorva has quit IRC22:05
openstackgerritMatt Riedemann proposed openstack/nova master: WIP: Deprecate file injection  https://review.openstack.org/52202722:05
*** ijw_ has joined #openstack-nova22:07
openstackgerritEric Fried proposed openstack/nova master: placement: GET /RPs with aggs, inventories, traits  https://review.openstack.org/52187522:09
*** sree has joined #openstack-nova22:09
efriedjaypipes FYI, added gabbits to ^22:09
jaypipesack22:10
efriedjaypipes Now I'm a gabbi expert.22:10
jaypipes:)22:10
*** ijw has quit IRC22:10
efriedjaypipes And I will go away for a long weekend and eat turkey and forget all of it.22:10
*** yangyapeng has joined #openstack-nova22:10
jaypipesheh, understood :)22:10
efriedjaypipes Anything you want my eyes on before I take off today?22:11
efriedjaypipes FYI I'll be out tomorrow through the weekend, back Monday.22:11
*** yamamoto has joined #openstack-nova22:12
*** sree has quit IRC22:14
*** yangyapeng has quit IRC22:14
jaypipesefried: enjoy your time off and the holiday22:15
jaypipesefried: and no, I'm good :)22:15
*** mvk has joined #openstack-nova22:15
efriedjaypipes You too (assuming you're having some)22:15
jaypipesthanks though, you've been super over the last month22:15
efriedMost welcome.22:15
jaypipesefried: yes, definitely doing thanksgiving din-din :)22:15
*** markvoelker has quit IRC22:16
*** yamamoto has quit IRC22:17
cfriesenanyone know if nova is susceptible to https://bugzilla.redhat.com/show_bug.cgi?id=1439078 or are our compatibility checks stringent enough?22:21
openstackbugzilla.redhat.com bug 1439078 in qemu-kvm-rhev "After migration,VM crash in dst host with "qemu-kvm: error: failed to set MSR 0x38f to 0x7000000ff"" [Unspecified,Closed: notabug] - Assigned to dgilbert22:21
jaypipescfriesen: lol, I read that as "assigned to dilbert"22:21
cfriesenbasically live migration failed when going from non-HT host to HT host with the same CPU22:21
cfriesenbecause the number of PMU registers changed22:21
jaypipescfriesen: it vaguely rings a bell. I'd ping sahid or kashyap in the morning.22:22
cfriesensounds like a plan22:22
*** amodi has quit IRC22:25
*** markvoelker has joined #openstack-nova22:27
*** marst has quit IRC22:30
*** slaweq has joined #openstack-nova22:31
*** slaweq has quit IRC22:32
*** slaweq has joined #openstack-nova22:32
openstackgerritEric Fried proposed openstack/nova master: Use ksa adapter for cinder client  https://review.openstack.org/50834522:34
*** claudiub|2 has joined #openstack-nova22:35
*** tbachman has joined #openstack-nova22:36
*** burt has quit IRC22:37
openstackgerritMathieu Gagné proposed openstack/nova master: Regenerate and pass configdrive when rebuild Ironic nodes  https://review.openstack.org/50308822:37
*** suresh12 has quit IRC22:37
efriedHasta la Monday, y'all22:38
* efried waves22:38
*** efried is now known as fried_turkey22:38
*** suresh12 has joined #openstack-nova22:42
*** lyan has quit IRC22:45
*** ijw_ has quit IRC22:51
*** eharney has joined #openstack-nova22:52
*** sree has joined #openstack-nova22:58
*** sree has quit IRC23:01
*** itlinux has quit IRC23:05
*** bnemec has quit IRC23:09
*** jaypipes has quit IRC23:11
*** bnemec has joined #openstack-nova23:12
*** AlexeyAbashkin has joined #openstack-nova23:12
*** felipemonteiro__ has quit IRC23:13
*** esberglu has quit IRC23:13
*** yamamoto has joined #openstack-nova23:14
*** AlexeyAbashkin has quit IRC23:16
*** yamamoto has quit IRC23:18
*** shvepsy_ has quit IRC23:22
*** slaweq has quit IRC23:25
*** Sandy619 has joined #openstack-nova23:28
*** gmann_afk is now known as gmann23:28
*** sdague has quit IRC23:29
*** Sandy619 has quit IRC23:30
*** takashin has joined #openstack-nova23:35
gmannjianghuaw_:23:56
gmannjianghuaw_: opps just ignore23:56
*** salv-orlando has quit IRC23:59
*** salv-orlando has joined #openstack-nova23:59

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!