| *** hongbin has quit IRC | 00:04 | |
| *** patriciadomin has quit IRC | 00:06 | |
| *** egarbade has quit IRC | 00:08 | |
| *** patriciadomin has joined #openstack-nova | 00:09 | |
| *** egarbade has joined #openstack-nova | 00:09 | |
| *** yassine has joined #openstack-nova | 00:16 | |
| *** yassine is now known as Guest4496 | 00:16 | |
| *** mdrabe has quit IRC | 00:35 | |
| *** mdrabe has joined #openstack-nova | 00:38 | |
| *** zhurong has joined #openstack-nova | 00:39 | |
| *** slaweq has joined #openstack-nova | 00:42 | |
| *** mdrabe has quit IRC | 00:45 | |
| *** slaweq has quit IRC | 00:47 | |
| *** tovin07 has joined #openstack-nova | 00:48 | |
| *** mdrabe has joined #openstack-nova | 00:48 | |
| *** hoangcx has joined #openstack-nova | 00:50 | |
| *** patriciadomin has quit IRC | 00:51 | |
| *** patriciadomin has joined #openstack-nova | 00:52 | |
| *** salv-orlando has joined #openstack-nova | 00:57 | |
| *** zhaochao has joined #openstack-nova | 00:59 | |
| *** jichen has joined #openstack-nova | 00:59 | |
| *** namnh has joined #openstack-nova | 01:00 | |
| *** ekhugen has quit IRC | 01:00 | |
| *** ekhugen has joined #openstack-nova | 01:01 | |
| *** mdbooth has joined #openstack-nova | 01:02 | |
| *** ansiwen has joined #openstack-nova | 01:02 | |
| *** salv-orlando has quit IRC | 01:03 | |
| *** patriciadomin has quit IRC | 01:03 | |
| *** tiendc has joined #openstack-nova | 01:04 | |
| *** patriciadomin has joined #openstack-nova | 01:04 | |
| *** gcb has joined #openstack-nova | 01:05 | |
| *** ekhugen has quit IRC | 01:06 | |
| *** ekhugen has joined #openstack-nova | 01:06 | |
| *** mdrabe has quit IRC | 01:07 | |
| *** egarbade has quit IRC | 01:08 | |
| *** egarbade has joined #openstack-nova | 01:08 | |
| *** patriciadomin has quit IRC | 01:09 | |
| *** jichen_ has joined #openstack-nova | 01:09 | |
| *** patriciadomin has joined #openstack-nova | 01:09 | |
| *** jichen__ has joined #openstack-nova | 01:10 | |
| *** mdrabe has joined #openstack-nova | 01:10 | |
| *** jichen has quit IRC | 01:10 | |
| *** jichen__ is now known as jichen | 01:10 | |
| *** egarbade has quit IRC | 01:13 | |
| *** egarbade- has joined #openstack-nova | 01:13 | |
| *** jichen_ has quit IRC | 01:14 | |
| *** Guest4496 has quit IRC | 01:15 | |
| *** phuongnh has joined #openstack-nova | 01:16 | |
| *** aloga has joined #openstack-nova | 01:16 | |
| *** Guest4496 has joined #openstack-nova | 01:16 | |
| *** takedakn has joined #openstack-nova | 01:17 | |
| *** patriciadomin has quit IRC | 01:17 | |
| *** patriciadomin has joined #openstack-nova | 01:18 | |
| *** gcb has quit IRC | 01:18 | |
| *** egarbade- has quit IRC | 01:19 | |
| *** egarbade has joined #openstack-nova | 01:20 | |
| *** mdrabe has quit IRC | 01:22 | |
| *** yangyapeng has quit IRC | 01:25 | |
| *** yangyapeng has joined #openstack-nova | 01:26 | |
| *** mdrabe has joined #openstack-nova | 01:26 | |
| *** yangyapeng has quit IRC | 01:31 | |
| *** yangyapeng has joined #openstack-nova | 01:32 | |
| *** yangyapeng has quit IRC | 01:36 | |
| *** acormier has joined #openstack-nova | 01:48 | |
| openstackgerrit | Leopard Ma proposed openstack/nova master: fix paused instance not support shelve https://review.openstack.org/541660 | 01:52 |
|---|---|---|
| *** patriciadomin has quit IRC | 01:56 | |
| *** patriciadomin has joined #openstack-nova | 01:56 | |
| *** zhurong has quit IRC | 01:57 | |
| openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova-specs master: Add host info to instance action events https://review.openstack.org/543277 | 02:01 |
| *** takedakn has quit IRC | 02:01 | |
| *** yangyapeng has joined #openstack-nova | 02:02 | |
| *** acormier has quit IRC | 02:03 | |
| *** mdrabe_ has joined #openstack-nova | 02:03 | |
| *** acormier has joined #openstack-nova | 02:03 | |
| openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova-specs master: Add host info to instance action events https://review.openstack.org/543277 | 02:04 |
| *** mdrabe has quit IRC | 02:05 | |
| *** yangyapeng has quit IRC | 02:05 | |
| *** yangyapeng has joined #openstack-nova | 02:06 | |
| *** suresh12 has joined #openstack-nova | 02:06 | |
| *** ekhugen has quit IRC | 02:08 | |
| *** ekhugen has joined #openstack-nova | 02:09 | |
| *** Tom-Tom has joined #openstack-nova | 02:14 | |
| *** takedakn has joined #openstack-nova | 02:19 | |
| *** suresh12 has quit IRC | 02:23 | |
| *** hongbin has joined #openstack-nova | 02:25 | |
| *** ratailor has joined #openstack-nova | 02:30 | |
| *** gcb has joined #openstack-nova | 02:30 | |
| *** takedakn has quit IRC | 02:37 | |
| *** ekhugen- has joined #openstack-nova | 02:41 | |
| *** ekhugen has quit IRC | 02:41 | |
| *** Tom-Tom has quit IRC | 02:42 | |
| *** Tom-Tom has joined #openstack-nova | 02:52 | |
| *** Tom-Tom has quit IRC | 02:55 | |
| *** Tom-Tom has joined #openstack-nova | 03:02 | |
| *** Tom-Tom has quit IRC | 03:04 | |
| *** edmondsw has joined #openstack-nova | 03:07 | |
| *** gcb has quit IRC | 03:07 | |
| *** itlinux has quit IRC | 03:10 | |
| *** patriciadomin has quit IRC | 03:11 | |
| *** ratailor has quit IRC | 03:11 | |
| *** edmondsw has quit IRC | 03:11 | |
| *** patriciadomin has joined #openstack-nova | 03:11 | |
| *** gcb has joined #openstack-nova | 03:20 | |
| *** egarbade has quit IRC | 03:22 | |
| *** egarbade has joined #openstack-nova | 03:23 | |
| *** ekhugen- has quit IRC | 03:23 | |
| *** patriciadomin has quit IRC | 03:24 | |
| *** mdrabe_ has quit IRC | 03:24 | |
| *** mdrabe has joined #openstack-nova | 03:25 | |
| *** patriciadomin has joined #openstack-nova | 03:25 | |
| *** acormier_ has joined #openstack-nova | 03:29 | |
| *** ekhugen_alt has joined #openstack-nova | 03:29 | |
| *** mdrabe has quit IRC | 03:32 | |
| *** acormier has quit IRC | 03:33 | |
| *** pooja_jadhav has joined #openstack-nova | 03:33 | |
| *** mdrabe has joined #openstack-nova | 03:34 | |
| *** acormier_ has quit IRC | 03:42 | |
| *** mdrabe has quit IRC | 03:43 | |
| *** links has joined #openstack-nova | 03:44 | |
| *** takedakn has joined #openstack-nova | 03:47 | |
| *** patriciadomin has quit IRC | 03:48 | |
| *** patriciadomin has joined #openstack-nova | 03:50 | |
| *** Guest4496 has quit IRC | 03:51 | |
| *** mdrabe has joined #openstack-nova | 03:51 | |
| *** yamamoto has joined #openstack-nova | 03:54 | |
| *** yikun_jiang has quit IRC | 03:55 | |
| *** yikun_jiang has joined #openstack-nova | 03:56 | |
| *** flwang1 has quit IRC | 03:59 | |
| *** itlinux has joined #openstack-nova | 04:01 | |
| *** ekhugen_alt has quit IRC | 04:02 | |
| *** ekhugen_alt has joined #openstack-nova | 04:03 | |
| *** mdrabe has quit IRC | 04:05 | |
| *** patriciadomin has quit IRC | 04:05 | |
| *** patriciadomin has joined #openstack-nova | 04:05 | |
| *** Guest4496 has joined #openstack-nova | 04:06 | |
| *** mdrabe has joined #openstack-nova | 04:07 | |
| *** dave-mccowan has quit IRC | 04:10 | |
| *** dave-mccowan has joined #openstack-nova | 04:11 | |
| *** udesale has joined #openstack-nova | 04:13 | |
| *** dave-mccowan has quit IRC | 04:18 | |
| *** jmlowe has joined #openstack-nova | 04:20 | |
| *** psachin has joined #openstack-nova | 04:21 | |
| *** jmlowe_ has quit IRC | 04:22 | |
| *** itlinux has quit IRC | 04:24 | |
| *** egarbade has quit IRC | 04:28 | |
| *** bhagyashris has joined #openstack-nova | 04:29 | |
| *** psachin has quit IRC | 04:30 | |
| *** egarbade has joined #openstack-nova | 04:30 | |
| *** abhishekk has joined #openstack-nova | 04:30 | |
| *** itlinux has joined #openstack-nova | 04:30 | |
| *** psachin has joined #openstack-nova | 04:32 | |
| *** mdrabe has quit IRC | 04:32 | |
| *** nicolasbock has quit IRC | 04:33 | |
| *** mdrabe has joined #openstack-nova | 04:34 | |
| *** egarbade has quit IRC | 04:34 | |
| *** egarbade has joined #openstack-nova | 04:35 | |
| *** xinliang has joined #openstack-nova | 04:36 | |
| *** lpetrut has joined #openstack-nova | 04:39 | |
| *** janki has joined #openstack-nova | 04:41 | |
| *** patriciadomin has quit IRC | 04:42 | |
| *** patriciadomin has joined #openstack-nova | 04:43 | |
| *** mdrabe has quit IRC | 04:44 | |
| *** slaweq has joined #openstack-nova | 04:45 | |
| *** slaweq has quit IRC | 04:49 | |
| *** mdrabe has joined #openstack-nova | 04:50 | |
| *** hongbin has quit IRC | 04:56 | |
| *** suresh12 has joined #openstack-nova | 04:59 | |
| *** brad[] has quit IRC | 05:03 | |
| *** brad[] has joined #openstack-nova | 05:05 | |
| *** lpetrut has quit IRC | 05:07 | |
| openstackgerrit | Dinesh Bhor proposed openstack/nova-specs master: Strict isolation of group of hosts for images https://review.openstack.org/381912 | 05:09 |
| *** ratailor has joined #openstack-nova | 05:11 | |
| *** lpetrut has joined #openstack-nova | 05:13 | |
| *** lpetrut has quit IRC | 05:15 | |
| *** lpetrut has joined #openstack-nova | 05:15 | |
| *** mdrabe_ has joined #openstack-nova | 05:17 | |
| *** mdrabe has quit IRC | 05:17 | |
| *** ekhugen_alt has quit IRC | 05:18 | |
| *** ekhugen- has joined #openstack-nova | 05:18 | |
| *** patriciadomin has quit IRC | 05:19 | |
| *** patriciadomin has joined #openstack-nova | 05:20 | |
| *** mdrabe_ has quit IRC | 05:22 | |
| *** mdrabe has joined #openstack-nova | 05:22 | |
| *** moshele has joined #openstack-nova | 05:24 | |
| *** egarbade has quit IRC | 05:25 | |
| *** ekhugen- has quit IRC | 05:25 | |
| *** egarbade has joined #openstack-nova | 05:25 | |
| *** ekhugen_alt has joined #openstack-nova | 05:26 | |
| *** moshele has quit IRC | 05:29 | |
| *** lpetrut has quit IRC | 05:31 | |
| *** fragatina has joined #openstack-nova | 05:38 | |
| *** acormier has joined #openstack-nova | 05:43 | |
| openstackgerrit | Huang Rui proposed openstack/nova master: z/VM driver: add snapshot function https://review.openstack.org/534240 | 05:44 |
| openstackgerrit | Huang Rui proposed openstack/nova master: add power actions https://review.openstack.org/543340 | 05:44 |
| openstackgerrit | Huang Rui proposed openstack/nova master: z/VM Driver: add power actions https://review.openstack.org/543340 | 05:45 |
| *** kholkina has joined #openstack-nova | 05:45 | |
| *** fragatina has quit IRC | 05:45 | |
| *** patriciadomin has quit IRC | 05:46 | |
| *** acormier has quit IRC | 05:47 | |
| *** patriciadomin has joined #openstack-nova | 05:47 | |
| *** sridharg has joined #openstack-nova | 05:49 | |
| *** egarbade- has joined #openstack-nova | 05:52 | |
| *** egarbade has quit IRC | 05:52 | |
| *** mdrabe has quit IRC | 05:58 | |
| *** yikun_jiang has quit IRC | 05:59 | |
| *** patriciadomin has quit IRC | 05:59 | |
| *** yikun_jiang has joined #openstack-nova | 05:59 | |
| *** mdrabe has joined #openstack-nova | 06:00 | |
| *** patriciadomin has joined #openstack-nova | 06:00 | |
| *** salv-orlando has joined #openstack-nova | 06:03 | |
| *** yamamoto has quit IRC | 06:05 | |
| openstackgerrit | Huang Rui proposed openstack/nova master: z/VM Driver: add get console output https://review.openstack.org/543344 | 06:06 |
| openstackgerrit | Rajesh Tailor proposed openstack/nova stable/pike: Allow force-delete even if task_state is not None https://review.openstack.org/543345 | 06:06 |
| *** links has quit IRC | 06:07 | |
| *** itlinux has quit IRC | 06:10 | |
| *** moshele has joined #openstack-nova | 06:14 | |
| *** yamamoto has joined #openstack-nova | 06:15 | |
| *** ircuser-1 has quit IRC | 06:32 | |
| *** lajoskatona has joined #openstack-nova | 06:34 | |
| openstackgerrit | Ameed Ashour proposed openstack/osc-placement master: tox.ini settings for global constraints are out of date https://review.openstack.org/543348 | 06:38 |
| *** suresh12 has quit IRC | 06:39 | |
| *** gcb has quit IRC | 06:41 | |
| openstackgerrit | Debo Zhang proposed openstack/nova master: Replace Chinese quotes to English quotes https://review.openstack.org/543349 | 06:46 |
| *** links has joined #openstack-nova | 06:51 | |
| *** gcb has joined #openstack-nova | 06:56 | |
| *** janki has quit IRC | 06:57 | |
| *** janki has joined #openstack-nova | 06:57 | |
| *** moshele has quit IRC | 06:57 | |
| *** moshele has joined #openstack-nova | 06:59 | |
| *** gcb has quit IRC | 07:02 | |
| *** yamamoto has quit IRC | 07:20 | |
| *** stvnoyes has left #openstack-nova | 07:23 | |
| *** sahid has joined #openstack-nova | 07:27 | |
| *** maciejjozefczyk has quit IRC | 07:28 | |
| *** lpetrut has joined #openstack-nova | 07:29 | |
| *** Tom-Tom has joined #openstack-nova | 07:33 | |
| *** slaweq has joined #openstack-nova | 07:45 | |
| *** maciejjozefczyk has joined #openstack-nova | 07:50 | |
| *** AlexeyAbashkin has joined #openstack-nova | 07:51 | |
| *** ragiman has joined #openstack-nova | 07:52 | |
| openstackgerrit | jichenjc proposed openstack/nova master: WIP: add zvm document into https://review.openstack.org/532720 | 07:59 |
| openstackgerrit | jichenjc proposed openstack/nova master: WIP: Add zvm admin doc https://review.openstack.org/533125 | 07:59 |
| openstackgerrit | jichenjc proposed openstack/nova master: WIP: Add zvm CI information https://review.openstack.org/533512 | 07:59 |
| vivsoni_ | Hi Team, | 08:02 |
| vivsoni_ | import uuid | 08:02 |
| vivsoni_ | from oslo_serialization import base64 | 08:02 |
| vivsoni_ | >>> base64.encode_as_text(uuid.UUID('ccdec515-4a96-4927-b05c-fecbed1a5515').bytes) | 08:02 |
| vivsoni_ | u'zN7FFUqWSSewXP7L7RpVFQ==' | 08:02 |
| vivsoni_ | how to convert from string "zN7FFUqWSSewXP7L7RpVFQ==" to 'ccdec515-4a96-4927-b05c-fecbed1a5515' | 08:02 |
| vivsoni_ | i tried | 08:03 |
| vivsoni_ | base64.decode_as_bytes(u'zN7FFUqWSSewXP7L7RpVFQ==') | 08:03 |
| vivsoni_ | it gives "\xcc\xde\xc5\x15J\x96I'\xb0\\\xfe\xcb\xed\x1aU\x15" | 08:03 |
| vivsoni_ | any suggestion to get 'ccdec515-4a96-4927-b05c-fecbed1a5515' from u'zN7FFUqWSSewXP7L7RpVFQ== | 08:04 |
| *** slaweq_ has joined #openstack-nova | 08:06 | |
| *** pcaruana has joined #openstack-nova | 08:07 | |
| *** ccamacho has joined #openstack-nova | 08:08 | |
| *** slaweq_ has quit IRC | 08:11 | |
| *** ralonsoh has joined #openstack-nova | 08:17 | |
| *** tesseract has joined #openstack-nova | 08:18 | |
| *** yamamoto has joined #openstack-nova | 08:21 | |
| *** salv-orlando has quit IRC | 08:27 | |
| *** yamamoto has quit IRC | 08:29 | |
| *** Tom-Tom has quit IRC | 08:30 | |
| *** Tom-Tom has joined #openstack-nova | 08:31 | |
| *** dtantsur|afk is now known as dtantsur | 08:33 | |
| *** amoralej|off is now known as amoralej | 08:38 | |
| *** rcernin has quit IRC | 08:39 | |
| *** yamahata has quit IRC | 08:39 | |
| ameeda | vivsoni_: did you try base64.decode_as_text(u'zN7FFUqWSSewXP7L7RpVFQ==') ? | 08:40 |
| vivsoni_ | ameeda: yes | 08:41 |
| vivsoni_ | it gives UnicodeDecodeError: 'utf8' codec can't decode byte 0xcc in position 0: invalid continuation byte | 08:41 |
| openstackgerrit | Huang Rui proposed openstack/nova master: z/VM Driver: Initial change set of z/VM driver https://review.openstack.org/523387 | 08:41 |
| openstackgerrit | Huang Rui proposed openstack/nova master: z/VM Driver: Spawn and destroy function of z/VM driver https://review.openstack.org/527658 | 08:41 |
| openstackgerrit | Huang Rui proposed openstack/nova master: z/VM Driver: add snapshot function https://review.openstack.org/534240 | 08:41 |
| openstackgerrit | Huang Rui proposed openstack/nova master: z/VM Driver: add power actions https://review.openstack.org/543340 | 08:41 |
| openstackgerrit | Huang Rui proposed openstack/nova master: z/VM Driver: add get console output https://review.openstack.org/543344 | 08:41 |
| vivsoni_ | ameeda: this base64.decode_as_bytes(u'zN7FFUqWSSewXP7L7RpVFQ==') | 08:42 |
| vivsoni_ | gives a closer results | 08:42 |
| *** bauwser is now known as bauzas | 08:43 | |
| bauzas | good morning stackers | 08:43 |
| ameeda | morning :) | 08:43 |
| ameeda | vivsoni_: I am not sure, I will check that soon | 08:43 |
| *** damien_r has joined #openstack-nova | 08:45 | |
| *** jpena|off is now known as jpena | 08:45 | |
| *** flwang1 has joined #openstack-nova | 08:54 | |
| *** lpetrut has quit IRC | 08:54 | |
| *** danpawlik has joined #openstack-nova | 08:54 | |
| vivsoni_ | ameeda: Thanks :) | 08:54 |
| *** sshwarts has joined #openstack-nova | 08:54 | |
| *** lpetrut has joined #openstack-nova | 09:00 | |
| *** claudiub has joined #openstack-nova | 09:02 | |
| *** abhishekk is now known as abhishekk|afk | 09:04 | |
| *** abhishekk|afk is now known as abhishekk | 09:11 | |
| *** ttsiouts has joined #openstack-nova | 09:12 | |
| gryf | vivsoni_, you can get your uuid by creating uuid object, like uuid.UUID(bytes=base64.decode_as_bytes(u'zN7FFUqWSSewXP7L7RpVFQ==')) | 09:12 |
| *** alexchadin has joined #openstack-nova | 09:16 | |
| vivsoni_ | gryf: this worked | 09:16 |
| vivsoni_ | :) | 09:16 |
| vivsoni_ | gryf: how you proceed to solve this ? | 09:18 |
| vivsoni_ | gryf: please let me know | 09:19 |
| gryf | vivsoni_, I just looked at documentation for uuid object ;) | 09:19 |
| *** brad[] has quit IRC | 09:20 | |
| *** Shilpa has joined #openstack-nova | 09:26 | |
| *** ispp has quit IRC | 09:26 | |
| *** ispp has joined #openstack-nova | 09:27 | |
| *** salv-orlando has joined #openstack-nova | 09:28 | |
| *** Tom-Tom_ has joined #openstack-nova | 09:32 | |
| *** brad[] has joined #openstack-nova | 09:32 | |
| vivsoni_ | gryf: Ok... Thanks :) | 09:32 |
| *** salv-orlando has quit IRC | 09:33 | |
| *** flwang1 has quit IRC | 09:35 | |
| *** Tom-Tom has quit IRC | 09:36 | |
| *** yamamoto has joined #openstack-nova | 09:43 | |
| *** yamamoto has quit IRC | 09:44 | |
| *** Tom-Tom_ has quit IRC | 09:46 | |
| *** Tom-Tom has joined #openstack-nova | 09:47 | |
| *** test222 has quit IRC | 09:47 | |
| *** test222 has joined #openstack-nova | 09:48 | |
| *** phuongnh has quit IRC | 09:48 | |
| *** Tom-Tom has quit IRC | 09:51 | |
| *** jichen has quit IRC | 10:01 | |
| *** alexchadin has quit IRC | 10:03 | |
| *** slaweq_ has joined #openstack-nova | 10:07 | |
| *** namnh has quit IRC | 10:08 | |
| *** tovin07 has quit IRC | 10:10 | |
| *** alexchadin has joined #openstack-nova | 10:12 | |
| *** slaweq_ has quit IRC | 10:12 | |
| *** tiendc has quit IRC | 10:17 | |
| *** derekh has joined #openstack-nova | 10:17 | |
| hrw | morning | 10:19 |
| *** wxy has quit IRC | 10:26 | |
| *** wxy has joined #openstack-nova | 10:27 | |
| *** Tom-Tom has joined #openstack-nova | 10:29 | |
| *** salv-orlando has joined #openstack-nova | 10:29 | |
| ameeda | morning | 10:31 |
| ameeda | hello, how can I get python2-mistralclient version ? | 10:31 |
| openstackgerrit | Chris Dent proposed openstack/nova master: WIP: Use nova.db.api directly https://review.openstack.org/543262 | 10:32 |
| *** cdent has joined #openstack-nova | 10:32 | |
| *** salv-orlando has quit IRC | 10:34 | |
| *** tssurya has joined #openstack-nova | 10:34 | |
| *** lucas-afk is now known as lucasagomes | 10:44 | |
| *** salv-orlando has joined #openstack-nova | 10:44 | |
| *** yamamoto has joined #openstack-nova | 10:45 | |
| *** elmaciej has joined #openstack-nova | 10:48 | |
| *** yamamoto has quit IRC | 10:51 | |
| *** udesale_ has joined #openstack-nova | 10:58 | |
| *** udesale has quit IRC | 11:01 | |
| *** Guest4496 has quit IRC | 11:01 | |
| *** sambetts|afk is now known as sambetts | 11:06 | |
| cdent | bauzas, giblet: if you're around this is a fairly trivial related-bug fix and followup to some earlier stuff that merged: https://review.openstack.org/#/c/526691/ | 11:06 |
| *** ralonsoh_ has joined #openstack-nova | 11:06 | |
| *** udesale_ has quit IRC | 11:07 | |
| *** ralonsoh has quit IRC | 11:10 | |
| *** jaosorior has joined #openstack-nova | 11:10 | |
| *** abhishekk has quit IRC | 11:12 | |
| cdent | bauzas, giblet: this one is also fairly trivial: https://review.openstack.org/#/c/536784/ | 11:15 |
| *** coreycb has quit IRC | 11:19 | |
| openstackgerrit | Chris Dent proposed openstack/nova master: Avoid inventory DELETE API (no conflict detection) https://review.openstack.org/539712 | 11:20 |
| *** coreycb has joined #openstack-nova | 11:20 | |
| cdent | efried: fixed the merge conflict on ^ | 11:20 |
| *** yamamoto has joined #openstack-nova | 11:20 | |
| *** mdnadeem has joined #openstack-nova | 11:20 | |
| *** giblet is now known as gibi | 11:24 | |
| gibi | cdent: will look at them soon | 11:25 |
| *** catmando has quit IRC | 11:25 | |
| bauzas | cdent: ack | 11:25 |
| cdent | gibi: awesome thanks. I'm going through all the stuff from last week's rp update, looking for easy wins... | 11:25 |
| *** catmando has joined #openstack-nova | 11:26 | |
| cdent | thanks bauzas | 11:30 |
| *** yamamoto has quit IRC | 11:32 | |
| *** tbachman has quit IRC | 11:35 | |
| *** sdague has joined #openstack-nova | 11:36 | |
| gibi | cdent: just on more small clarification request in https://review.openstack.org/#/c/526691/2/nova/api/openstack/placement/deploy.py@97 | 11:37 |
| hrw | bauzas: can you help get https://review.openstack.org/#/c/541728/ (aarch64 status matrix) merged? mriedem already gave +2 so you can finish merge process :D | 11:38 |
| *** bhagyashris has quit IRC | 11:38 | |
| *** finucannot is now known as stephenfin | 11:40 | |
| cdent | gibi: yeah, good point. will fix | 11:40 |
| openstackgerrit | Chris Dent proposed openstack/nova master: [placement] annotate loadapp as public interface https://review.openstack.org/526691 | 11:43 |
| cdent | bauzas, gibi: ^ please and thank you | 11:43 |
| *** alexchadin has quit IRC | 11:45 | |
| *** tbachman has joined #openstack-nova | 11:47 | |
| *** tbachman has quit IRC | 11:49 | |
| *** bhagyashris has joined #openstack-nova | 11:50 | |
| *** nicolasbock has joined #openstack-nova | 11:53 | |
| *** ratailor has quit IRC | 11:59 | |
| *** yamamoto has joined #openstack-nova | 12:02 | |
| *** slaweq_ has joined #openstack-nova | 12:08 | |
| *** dave-mccowan has joined #openstack-nova | 12:08 | |
| *** slaweq_ has quit IRC | 12:13 | |
| *** yangyapeng has quit IRC | 12:14 | |
| gibi | cdent: thanks, +2d | 12:26 |
| *** jpena is now known as jpena|lunch | 12:26 | |
| *** acormier has joined #openstack-nova | 12:27 | |
| *** jmlowe has quit IRC | 12:27 | |
| AJaeger | novaclient team, please review https://review.openstack.org/543115 , https://review.openstack.org/#/c/543114, https://review.openstack.org/#/c/541370/ - the first two are needed to fix building, the third is needed for Zuul v3 | 12:27 |
| *** acormier has quit IRC | 12:28 | |
| *** acormier has joined #openstack-nova | 12:28 | |
| *** yangyapeng has joined #openstack-nova | 12:30 | |
| *** acormier has quit IRC | 12:35 | |
| *** yangyapeng has quit IRC | 12:35 | |
| *** wxy has quit IRC | 12:36 | |
| ameeda | gibi: could you please check https://review.openstack.org/#/c/526900/, it doesn't has any activity from awhile | 12:38 |
| *** ralonsoh__ has joined #openstack-nova | 12:46 | |
| *** yangyapeng has joined #openstack-nova | 12:47 | |
| *** ralonsoh_ has quit IRC | 12:50 | |
| *** r-daneel has joined #openstack-nova | 12:56 | |
| *** takedakn has quit IRC | 13:04 | |
| *** takedakn has joined #openstack-nova | 13:05 | |
| *** takedakn has quit IRC | 13:05 | |
| bauzas | hrw: https://review.openstack.org/#/c/541728/ will be merged in master, which is now for Rocky | 13:08 |
| *** tbachman has joined #openstack-nova | 13:08 | |
| bauzas | hrw: let's discuss with mriedem if we should cherry-pick to Queens | 13:08 |
| *** mdnadeem has quit IRC | 13:09 | |
| *** weshay|ruck is now known as weshay | 13:10 | |
| *** yassine has joined #openstack-nova | 13:11 | |
| *** yassine is now known as Guest58801 | 13:11 | |
| *** edmondsw has joined #openstack-nova | 13:15 | |
| *** edmondsw has quit IRC | 13:15 | |
| *** edmondsw_ has joined #openstack-nova | 13:16 | |
| *** jpena|lunch is now known as jpena | 13:26 | |
| *** ccamacho has quit IRC | 13:31 | |
| *** pchavva has joined #openstack-nova | 13:31 | |
| *** mdnadeem has joined #openstack-nova | 13:36 | |
| *** edmondsw_ is now known as edmondsw | 13:37 | |
| *** ispp has quit IRC | 13:40 | |
| hrw | bauzas: thanks | 13:41 |
| hrw | bauzas: I think that status-matrix part can be merged | 13:42 |
| *** acormier has joined #openstack-nova | 13:45 | |
| *** slaweq_ has joined #openstack-nova | 13:46 | |
| *** vladikr has joined #openstack-nova | 13:47 | |
| efried | cdent: Thank you sir. | 13:49 |
| *** ccamacho has joined #openstack-nova | 13:49 | |
| efried | cdent: Was it because of global_id? | 13:49 |
| *** acormier has quit IRC | 13:49 | |
| cdent | efried: I _think_ so. The method that you had deleted was different from when you had deleted if before, so I just deleted it again | 13:50 |
| efried | right | 13:50 |
| *** slaweq_ has quit IRC | 13:51 | |
| *** dtantsur is now known as dtantsur|brb | 13:52 | |
| efried | leakypipes: Yet another +2-after-rebase please: https://review.openstack.org/#/c/539712/ | 13:54 |
| *** nicolasbock has quit IRC | 13:54 | |
| efried | cdent: here's that spec on resource class affinity for NUMA: https://review.openstack.org/543062 | 13:55 |
| *** takashin has joined #openstack-nova | 13:55 | |
| leakypipes | efried: done | 13:56 |
| efried | leakypipes: Thanks | 13:56 |
| cdent | efried: yeah, already noted it because of the spying you suggested i might do | 13:56 |
| * leakypipes goes back to reading efried's resource class affinity spec | 13:56 | |
| efried | cdent: Turns out the "aggregate distance" business isn't actually suited for NUMA. I'll still send out a link to the eavesdrop/etherpad, but I'm going to kinda push it out of main memory to keep room for other stuff. | 13:57 |
| efried | cdent: eh, spying? | 13:57 |
| *** acormier has joined #openstack-nova | 13:57 | |
| cdent | efried: I haven't digested it yet. glanced over the weekend and was all "ugh, not another table" and decided to come back later | 13:58 |
| *** liverpooler has joined #openstack-nova | 13:58 | |
| cdent | efried: spying: p!spy in https://anticdent.org/purple-irc-bot.html | 13:58 |
| efried | cdent: Another table - you're talking about agg distance, not rc affinity, right? | 13:58 |
| cdent | efried: right, I haven't looked at rc affinity yet, but its in the queue | 13:59 |
| efried | cdent: Ah :) | 13:59 |
| efried | okay, cool. | 13:59 |
| * cdent fixes typo in that blog post | 14:00 | |
| edleafe | Scheduler subteam meeting running now in #openstack-meeting-alt | 14:00 |
| *** jmlowe has joined #openstack-nova | 14:00 | |
| mriedem | bauzas: i don't think https://review.openstack.org/#/c/541728/ needs to go to queens | 14:01 |
| *** leakypipes is now known as jaypipes | 14:01 | |
| bauzas | sure, np | 14:01 |
| *** acormier has quit IRC | 14:02 | |
| efried | p!spy | 14:02 |
| *** jmlowe has quit IRC | 14:05 | |
| *** kholkina has quit IRC | 14:08 | |
| *** amoralej is now known as amoralej|lunch | 14:08 | |
| *** ralonsoh__ is now known as ralonsoh | 14:11 | |
| *** rajinir has quit IRC | 14:12 | |
| *** rajinir has joined #openstack-nova | 14:12 | |
| *** ispp has joined #openstack-nova | 14:13 | |
| *** ekhugen- has joined #openstack-nova | 14:15 | |
| *** patriciadomin_ has joined #openstack-nova | 14:16 | |
| *** egarbade has joined #openstack-nova | 14:16 | |
| *** mdrabe_ has joined #openstack-nova | 14:17 | |
| openstackgerrit | Chris Dent proposed openstack/nova master: Move db MAX constants to own file https://review.openstack.org/543469 | 14:17 |
| *** ekhugen_alt has quit IRC | 14:17 | |
| *** ispp has quit IRC | 14:18 | |
| *** ispp has joined #openstack-nova | 14:18 | |
| *** egarbade- has quit IRC | 14:18 | |
| *** patriciadomin has quit IRC | 14:19 | |
| *** mdrabe has quit IRC | 14:20 | |
| *** udesale has joined #openstack-nova | 14:20 | |
| AJaeger | mriedem, who's reviewing python-novaclient nowadays? I have some reviews: review https://review.openstack.org/543115 , https://review.openstack.org/#/c/543114, https://review.openstack.org/#/c/541370/ - the first two are needed to fix building, the third is needed for Zuul v3. | 14:21 |
| * AJaeger is pinging here to point out how those are related | 14:21 | |
| efried | AJaeger: I hear takashin might be a good candidate :) | 14:21 |
| *** mdrabe_ is now known as mdrabe | 14:23 | |
| *** yamamoto has quit IRC | 14:24 | |
| *** eharney has joined #openstack-nova | 14:24 | |
| *** udesale_ has joined #openstack-nova | 14:24 | |
| *** udesale has quit IRC | 14:25 | |
| AJaeger | takashin: ^ please ping me in case of questions. Without the first two changes, the stable branches are broken. | 14:25 |
| *** psachin has quit IRC | 14:26 | |
| *** pchavva has quit IRC | 14:27 | |
| *** lajoskatona has quit IRC | 14:29 | |
| mriedem | AJaeger: i can look after the scheduler meeting | 14:30 |
| AJaeger | mriedem: thanks | 14:30 |
| *** slaweq has quit IRC | 14:31 | |
| *** tssurya has quit IRC | 14:32 | |
| *** slaweq has joined #openstack-nova | 14:32 | |
| takashin | AJaeger: I'm not a core reviewer for python-novaclient project... | 14:33 |
| *** ttsiouts_ has joined #openstack-nova | 14:34 | |
| openstackgerrit | Merged openstack/nova master: Provide basic data for AArch64 support matrix/functionality https://review.openstack.org/541728 | 14:36 |
| *** slaweq has quit IRC | 14:36 | |
| *** yamahata has joined #openstack-nova | 14:37 | |
| *** david-lyle has quit IRC | 14:38 | |
| *** jobewan has joined #openstack-nova | 14:38 | |
| *** edmondsw has quit IRC | 14:38 | |
| *** janki has quit IRC | 14:39 | |
| *** lbragstad has joined #openstack-nova | 14:39 | |
| hrw | mriedem, bauzas: thanks for helping with getting this stuff merged | 14:42 |
| *** yamahata has quit IRC | 14:42 | |
| *** pchavva has joined #openstack-nova | 14:42 | |
| *** yamahata has joined #openstack-nova | 14:42 | |
| gibi | mriedem, bauzas: when I mentioned that a volume used in BFV case might not have any image associated to it I thought about someting like this: http://paste.openstack.org/show/670002/ | 14:44 |
| gibi | mriedem, bauzas: In this case nova cannot do anyithing to have the image reference, as there was never an image associated to the volume | 14:44 |
| *** r-daneel has quit IRC | 14:45 | |
| gibi | mriedem, bauzas: I don't think we can enforce to have image_id in the request spec | 14:46 |
| mriedem | gibi: that vm is worthless though right? | 14:46 |
| gibi | mriedem: yeah, I oversimplified it, but I could attach my-vol to a vm, populate it, detach it, then boot from it | 14:46 |
| *** tssurya has joined #openstack-nova | 14:47 | |
| mriedem | sure, i wasn't saying we should 'require' an id in the requestspec.image field, | 14:47 |
| mriedem | just that in certain cases (presumably the more normal cases), it's available via the volume_image_metadata | 14:48 |
| *** lucasagomes is now known as lucas-hungry | 14:48 | |
| mriedem | and furthermore, we can use that information later if a cell is down | 14:48 |
| mriedem | and we use the request spec to represent the instance in the GET /servers/{id}/detail response | 14:48 |
| mriedem | *if we use | 14:48 |
| gibi | mriedem, bauzas: OK, it is clearer now. I had the feeling that we want to make image_id mandatory in the request spec | 14:49 |
| mriedem | jroll: added the healthcheck middleware to both https://etherpad.openstack.org/p/self-healing-queens-ptg and https://etherpad.openstack.org/p/api-sig-ptg-rocky | 14:50 |
| mriedem | we probably need a few more SIGs for this | 14:50 |
| bauzas | mriedem: heh for "more SIGs" | 14:51 |
| bauzas | that can become a very discussed and argued thing | 14:51 |
| openstackgerrit | Chris Dent proposed openstack/nova master: WIP: Move resource provider objects into placement hierarchy https://review.openstack.org/540049 | 14:51 |
| openstackgerrit | Chris Dent proposed openstack/nova master: Optional separate database for placement API https://review.openstack.org/362766 | 14:51 |
| openstackgerrit | Chris Dent proposed openstack/nova master: WIP: Isolate placement database config https://review.openstack.org/541435 | 14:51 |
| jroll | mriedem: what have I done | 14:51 |
| jroll | this really wasn't something I intended to tilt that hard at | 14:51 |
| bauzas | gibi: mriedem: for the image.id being mandatory, I think I understand the point | 14:51 |
| *** edmondsw has joined #openstack-nova | 14:52 | |
| *** acormier has joined #openstack-nova | 14:52 | |
| bauzas | gibi: mriedem: the problem is that AFAICS we assert in a lot of places about having an instance with an image | 14:52 |
| mriedem | jroll: i just wanted to see it socialized more before it's adopted across a few projects, | 14:52 |
| mriedem | because i'm projecting it on the community wide goals process | 14:52 |
| bauzas | so we can 'fix' that, but that won't solve the design issue that we accepted in the past a different paradigm | 14:52 |
| jroll | mriedem: also, I wasn't aware of the "new" thing until today, so adding the new thing you linked wasn't my intention at all | 14:52 |
| mriedem | jroll: so your ironic patch and graham's spec aren't the same thing? | 14:53 |
| jroll | mriedem: the new thing sounds like it'll take a year just to get through committee and have code written | 14:53 |
| jroll | correct | 14:53 |
| bauzas | jroll: from a very practical standpoint, do you feel blocked ? | 14:53 |
| jroll | mine is the "old" one | 14:53 |
| mriedem | bauzas: jim just wants to upstream it, but meh | 14:53 |
| mriedem | it sounds like | 14:53 |
| jroll | bauzas: no, what matt said | 14:53 |
| mriedem | it's middleware | 14:53 |
| bauzas | jroll: because I don't really see *why*it needs to be something | 14:53 |
| jroll | bauzas: because it's what we use for every paste-enabled service, and so trying to standardize the rest there | 14:54 |
| * jroll isn't terribly opinionated on this being the thing that is used | 14:54 | |
| bauzas | jroll: well, you wanna standardize your own monitoring model :) | 14:54 |
| jroll | bauzas: s/your own/oslo's/ | 14:55 |
| gibi | bauzas: on the image_id, theoretically we can forbid booting from volume in nova where the volume doesn't have the image_id in the metadata, but I think that would be restrict some use cases that is possible today | 14:55 |
| cdent | jroll: I'd say go ahead with the old thing. the new thing needs to be discussed but is a whole different kettle | 14:55 |
| jroll | I'm not forcing it on anyone ¯\_(ツ)_/¯ | 14:55 |
| bauzas | I'm not sure operators are sold on using middleware healthchecks for knowing whether my app is up | 14:55 |
| cdent | jroll: I reckon the way it is done in ironic is just fine for placement | 14:55 |
| bauzas | gibi: If I'm sure about one thing, it's how operators can find solutions with things we don't officially support | 14:56 |
| mriedem | gibi: i wasn't suggesting making that break now, | 14:56 |
| cdent | it doesn't preclude some different future | 14:56 |
| mriedem | gibi: was just wondering if we have the information from the volume meta, if we should store it in the request spec | 14:56 |
| bauzas | mriedem: at the request_spec creation time, then ? | 14:56 |
| mriedem | yes | 14:56 |
| bauzas | well | 14:56 |
| mriedem | the confusion will be, | 14:56 |
| *** amodi has joined #openstack-nova | 14:56 | |
| bauzas | if you rebuild ? | 14:57 |
| mriedem | instance.image_ref == None == volume-backed, | 14:57 |
| gibi | copying image_id over from the volume meta to the request_spec is fine by me | 14:57 |
| bauzas | also | 14:57 |
| *** amoralej|lunch is now known as amoralej | 14:57 | |
| mriedem | but requestspec.image.id == None....? | 14:57 |
| *** yamamoto has joined #openstack-nova | 14:57 | |
| mriedem | bauzas: if you rebuild a volume-backed instance with a new image, we fail today | 14:57 |
| jroll | cdent: thanks | 14:58 |
| *** simondodsley has quit IRC | 14:58 | |
| mriedem | bauzas: https://github.com/openstack/nova/blob/master/nova/compute/api.py#L2979 | 14:58 |
| *** simondodsley has joined #openstack-nova | 14:58 | |
| bauzas | mriedem: okay, gtk | 14:59 |
| *** esberglu has joined #openstack-nova | 14:59 | |
| *** mchlumsky has joined #openstack-nova | 15:01 | |
| takashin | mriedem: Would you check my reply to you comment in https://review.openstack.org/#/c/513721/ ? | 15:01 |
| mriedem | takashin: done | 15:04 |
| takashin | mriedem: Thanks. | 15:05 |
| *** acormier has quit IRC | 15:05 | |
| *** acormier has joined #openstack-nova | 15:05 | |
| *** damien_r has quit IRC | 15:06 | |
| *** takashin has left #openstack-nova | 15:06 | |
| *** tbachman has quit IRC | 15:07 | |
| mriedem | bauzas: https://bugs.launchpad.net/nova/+bug/1748858 | 15:08 |
| openstack | Launchpad bug 1748858 in OpenStack Compute (nova) "allocation_ratio set by aggregate metadata don't work in NUMATopologyFilter" [Undecided,New] - Assigned to yangjie (yang.jie) | 15:09 |
| mriedem | sounds familiar | 15:09 |
| bauzas | mriedem: sec, thinking hard of https://review.openstack.org/#/c/543263/2/nova/scheduler/filters/isolated_hosts_filter.py | 15:09 |
| bauzas | the change isn't trivial | 15:09 |
| *** auggy has quit IRC | 15:09 | |
| mriedem | bauzas: really? | 15:10 |
| *** auggy has joined #openstack-nova | 15:10 | |
| mriedem | bauzas: that's basically what it did before it was regressed here https://review.openstack.org/#/c/202678/35/nova/scheduler/filters/isolated_hosts_filter.py | 15:10 |
| bauzas | mriedem: yeah, my guts feel some problem that I still don't see | 15:10 |
| mriedem | maybe you need to go to the bathroom | 15:10 |
| bauzas | probably | 15:11 |
| *** tssurya has quit IRC | 15:11 | |
| bauzas | but I'm looking at https://review.openstack.org/#/c/543263/2/nova/scheduler/filters/isolated_hosts_filter.py@43 | 15:11 |
| gibi | bauzas: shall we pull the patch from the gate as I +Wd it recently | 15:11 |
| mriedem | if you look at how it worked before https://review.openstack.org/#/c/202678/35/nova/scheduler/filters/isolated_hosts_filter.py | 15:11 |
| bauzas | gibi: I'm not saying we *have* a problem | 15:11 |
| mriedem | the image id was always something it handled as optional | 15:11 |
| mriedem | using dict.get() | 15:11 |
| bauzas | the thing is | 15:11 |
| mriedem | changing to request spec in that patch made it required | 15:12 |
| bauzas | if we say we don't have an image | 15:12 |
| bauzas | then the image isn't isolated, right | 15:12 |
| mriedem | bauzas: that's the point of my email, | 15:12 |
| bauzas | in that case, we should accept host1 | 15:12 |
| mriedem | which is, this never did anything for volume-backed instances, | 15:12 |
| mriedem | but that's not the point of my fix right now | 15:12 |
| mriedem | i'm restoring this filter to how it previously worked, before mitaka | 15:12 |
| bauzas | then the doc is invalid | 15:13 |
| bauzas | at least the docstring I mean | 15:13 |
| bauzas | and I'd like to fix that | 15:13 |
| mriedem | so clean that up in a follow up? | 15:13 |
| bauzas | well now that gibi pulled the trigger, /me shrufs | 15:13 |
| cdent | Is there a general opinion about the merit of conditional imports in nova code? I'd like to only import rpc and sqlalchemy_api here https://github.com/openstack/nova/blob/master/nova/config.py#L54-L58 if those conditionals are true. The other option (which might make better long term sense) is to make a different parse_args | 15:13 |
| bauzas | shrugs* | 15:13 |
| bauzas | mriedem: the thing is, I thought you wanna backport ? | 15:14 |
| bauzas | moar fun then | 15:14 |
| mriedem | bauzas: i do | 15:14 |
| bauzas | so, we should backport both your change *and* the doc change | 15:14 |
| mriedem | i don't think the docstring is something that bars us from backporting this fix | 15:14 |
| bauzas | looks sad | 15:14 |
| bauzas | I know | 15:14 |
| *** tbachman has joined #openstack-nova | 15:14 | |
| mriedem | bauzas: if you want to call out the volume-backed scenario, then push a patch on top of this https://review.openstack.org/#/c/543264/ | 15:15 |
| bauzas | I just would have appreciated if we could have amended the upstream filters documentation *and* the docstring in the sme change | 15:15 |
| mriedem | i didn't say anything about volume-backed behavior in https://review.openstack.org/#/c/543264/ because, as i said in the ML, i'm not sure if that behavior is intentional or not | 15:15 |
| mriedem | since the filter predates my involvement in nova | 15:15 |
| bauzas | mriedem: I don't really care of the BFV case | 15:15 |
| bauzas | the thing is, should we assume that an instance having no image is behaving identical as if we were having an instance with a non-isolated image ? | 15:16 |
| bauzas | my guts feel yues | 15:16 |
| bauzas | hence my trouble | 15:16 |
| mriedem | fwiw essex https://github.com/openstack/nova/commit/972fc7d80fae386cf4c72b7891f6601d6f7fb00b#diff-b4fa3e44aeb7757f04d296529b8492e5 | 15:16 |
| *** sahid has quit IRC | 15:16 | |
| mriedem | bauzas: well, that's the point of my ML thread | 15:16 |
| *** moshele has quit IRC | 15:17 | |
| bauzas | again, I know | 15:17 |
| mriedem | and why i also sent it to the ops ML | 15:17 |
| bauzas | but people pulled the trigger before I had time to both review and reply | 15:17 |
| bauzas | that's fine, it's my bad | 15:17 |
| mriedem | i wanted to find out (1) does anyone use this filter (2) if so, do they know about this bfv thing, and (3) if so, do they care? | 15:17 |
| bauzas | I need to be quicker | 15:17 |
| gibi | I feel there is two different change on that is now on the gate to restore the regressed behavior and one that is on the ML to find out the what is the intended behavior for BFV | 15:19 |
| bauzas | mriedem: like I said in the meeting, if people use that filter, I'd be surprised | 15:19 |
| *** tbachman has quit IRC | 15:19 | |
| bauzas | gibi: after thinking hard, I think we can leave the patch merge | 15:19 |
| *** lucas-hungry is now known as lucasagomes | 15:19 | |
| bauzas | mriedem: I'd be surprised because it's conf-driven | 15:19 |
| bauzas | which operators hate | 15:19 |
| edleafe | cdent: you'd need to import rpc for line 44 :) | 15:20 |
| bauzas | while ImageAggragateExtraSpecsMyStuffFilter does that programatically with aggregates :) | 15:20 |
| cdent | edleafe: would isolate that too | 15:20 |
| cdent | edleafe: but just going to write another one, as there's redundancy happening in that stuff that placement doesn't care about | 15:20 |
| cdent | edleafe: still curious about the general question (even though I'm not going to do it) | 15:21 |
| cdent | some people hate them | 15:21 |
| edleafe | cdent: yeah, just funnin' ya | 15:21 |
| edleafe | FWIW, I tend to not use them, except when performance is affected. | 15:22 |
| *** ccamacho has quit IRC | 15:22 | |
| *** sergek_ has quit IRC | 15:22 | |
| *** sergek_ has joined #openstack-nova | 15:23 | |
| mriedem | melwitt: looks like functional test failures in https://review.openstack.org/#/c/340614/ are real | 15:23 |
| *** belmoreira has joined #openstack-nova | 15:23 | |
| *** jmlowe has joined #openstack-nova | 15:23 | |
| * stephenfin is having _great_ fun with OVS-DPDK today :/ | 15:24 | |
| * alex_xu gets into holiday mode, won't be active in two weeks | 15:25 | |
| cdent | stephenfin: wait, DPDK isn't solving _all_ your problems. unpossible. | 15:26 |
| mriedem | alex_xu: have fun, eat lots of dumplings | 15:26 |
| bauzas | stephenfin: DPDK like Don't Play Donkey Kong ? | 15:26 |
| alex_xu | mriedem: thanks :) | 15:27 |
| *** tssurya has joined #openstack-nova | 15:27 | |
| bauzas | mriedem: what does the follow-up button on the Gerrit UI if I'm clicking it on your top change ? | 15:27 |
| mriedem | bauzas: no idea | 15:27 |
| bauzas | mriedem: I'd like to just amend the doc to explain about BFV instances | 15:27 |
| mriedem | https://stackoverflow.com/questions/45621912/what-is-the-gerrit-follow-up-button-doing | 15:28 |
| stephenfin | cdent: It's not even solving one of them yet. I probably have it configured incorrectly...again | 15:28 |
| *** jmlowe has quit IRC | 15:28 | |
| bauzas | mriedem: and the fact that if you have an image that is isolated, but then you create a volume from that image and then boot from that volume, Nova won't guarantee your host will be excluded | 15:28 |
| *** sahid has joined #openstack-nova | 15:28 | |
| stephenfin | bauzas: Don't be dissing the Kong now, ya hear ;) | 15:28 |
| mriedem | bauzas: well, it will depending on the other config options | 15:29 |
| bauzas | mriedem: exactly | 15:29 |
| bauzas | mriedem: but consider the case where I have an image that is restricted to host1 | 15:29 |
| mriedem | i'd say, if you care, use AggregateImagePropertiesIsolation | 15:29 |
| mriedem | actually, | 15:30 |
| mriedem | AggregateImagePropertiesIsolation likely doesn't work for volume-backed instances properly either | 15:30 |
| bauzas | mriedem: if I'm making a volume from that image and then booting from it, then the instance will be exclused from that host if strict_host_blah=True | 15:30 |
| *** jmlowe has joined #openstack-nova | 15:30 | |
| mriedem | for a volume-backed instance, the only image properties we store in the request spec are the ones from the volume_image_metadata https://github.com/openstack/nova/blob/master/nova/utils.py#L1033 | 15:31 |
| mriedem | and i don't know if that actually stores the properties from the backing image, and even if it does, they could be stale | 15:31 |
| *** fmccrthy has quit IRC | 15:31 | |
| * bauzas is despressed | 15:32 | |
| *** fmccrthy has joined #openstack-nova | 15:32 | |
| mriedem | so yeah, any filter that relies on image properties is likely semi-broken with boot from volume | 15:32 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova master: for doc clarity https://review.openstack.org/543494 | 15:32 |
| openstackgerrit | Chris Dent proposed openstack/nova master: Isolate config parse_args for placement https://review.openstack.org/543495 | 15:33 |
| *** mlavalle has joined #openstack-nova | 15:34 | |
| AJaeger | mriedem, the novaclient reviews I mentioned earlier are: review https://review.openstack.org/543115 , https://review.openstack.org/#/c/543114, https://review.openstack.org/#/c/541370/ - the first two are needed to fix the jobs, the third is needed for Zuul v3. | 15:35 |
| *** felipemonteiro_ has joined #openstack-nova | 15:37 | |
| openstackgerrit | Sylvain Bauza proposed openstack/nova master: doc: BFV instances and IsolatedHostsFilter https://review.openstack.org/543494 | 15:37 |
| bauzas | mriedem: just made a first try for doc ^ | 15:38 |
| *** tidwellr has joined #openstack-nova | 15:38 | |
| *** felipemonteiro__ has joined #openstack-nova | 15:38 | |
| *** lpetrut has quit IRC | 15:39 | |
| *** links has quit IRC | 15:39 | |
| *** READ10 has joined #openstack-nova | 15:40 | |
| AJaeger | thanks, mriedem | 15:40 |
| bauzas | stephenfin: sahid: do you have any idea why the NUMATopologyFilter needs to set some limits for CPU and RAM ? https://github.com/openstack/nova/blob/master/nova/scheduler/filters/numa_topology_filter.py#L92-L93 | 15:41 |
| bauzas | context is https://bugs.launchpad.net/nova/+bug/1748858 | 15:41 |
| openstack | Launchpad bug 1748858 in OpenStack Compute (nova) "allocation_ratio set by aggregate metadata don't work in NUMATopologyFilter" [Undecided,New] - Assigned to yangjie (yang.jie) | 15:41 |
| bauzas | stephenfin: sahid: the NUMATopologyFilter should only care in my opinion to enforce the numa limits, not any other resource usage | 15:42 |
| stephenfin | bauzas: Not off the top of my head, no. That whole 'NUMATopologyLimits' object is a weird one I've yet to grasp properly. Hopefully sahid knows more | 15:42 |
| *** felipemonteiro_ has quit IRC | 15:42 | |
| * bauzas looking at https://github.com/openstack/nova/commit/0b0f1515acd43ec3b39f2f748161890c80a8a939 | 15:43 | |
| *** david-lyle has joined #openstack-nova | 15:46 | |
| *** slaweq has joined #openstack-nova | 15:47 | |
| *** tbachman has joined #openstack-nova | 15:48 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Fix typo in NUMATopologyFilter docs https://review.openstack.org/543505 | 15:49 |
| *** zhaochao has quit IRC | 15:50 | |
| *** slaweq has quit IRC | 15:51 | |
| *** udesale has joined #openstack-nova | 15:53 | |
| *** udesale_ has quit IRC | 15:53 | |
| *** tidwellr has quit IRC | 15:54 | |
| mriedem | bauzas: fwiw, the image properties are stored on the volume's volume_image_metadata dict: http://paste.openstack.org/show/670139/ | 15:56 |
| mriedem | i add foo=bar to the image i used to create that bootable volume | 15:56 |
| mriedem | so AggregateImagePropertiesIsolation would only be as good as how accurate that metadata is | 15:56 |
| bauzas | mriedem: are you persuing me to look up the volume if BFV ? | 15:56 |
| mriedem | since nova doesn't re-fetch the image to get the latest meta | 15:57 |
| mriedem | persuing you how? | 15:57 |
| bauzas | the fact is that filters don't lookup the volume image metadata, right? | 15:57 |
| bauzas | they just assume every instance has an image | 15:57 |
| bauzas | well, they rather assume every RequestSpec has an image | 15:58 |
| mriedem | the filter assumes the request spec image has the details from when the api created the request spec | 15:58 |
| mriedem | yes that | 15:58 |
| *** sahid has quit IRC | 15:58 | |
| mriedem | and the request spec for a volume-backed instance will have the image meta stored in the volume from the time the volume was created | 15:58 |
| *** sahid has joined #openstack-nova | 15:58 | |
| bauzas | mriedem: if we create a fake image based on that, right? | 15:59 |
| mriedem | but my point is, i could create a volume with some image, then change properties on that image, and then create a server from the volume and the image meta in the volume would be stale | 15:59 |
| bauzas | I agre | 15:59 |
| bauzas | I got your point | 15:59 |
| bauzas | if we want to support BFV instances correctly, we would necessarly have to lazy-load things on the fly if we need to access the "image" | 15:59 |
| *** wolsen has quit IRC | 15:59 | |
| bauzas | hence the "are you persuing me to lookup the volume" | 16:00 |
| *** wolsen has joined #openstack-nova | 16:00 | |
| mriedem | no i'm not suggesting that, | 16:00 |
| mriedem | i was saying, if we cared, we could get the image_id from the volume meta, and lookup the image at server create time to get the current image meta from glance | 16:01 |
| mriedem | rather than the possibly stale image meta from the volume | 16:01 |
| mriedem | but, i'm not suggesting we do that either | 16:01 |
| mriedem | i guess we should just assume that the image meta with the volume is like a snapshot | 16:01 |
| mriedem | if the image meta is changed after the volume is created, meh | 16:01 |
| mriedem | if it was changed in such a substantial way that it would impact nova scheduling for the instance, then it should probably be a different image | 16:02 |
| bauzas | that looks like the best, indeed | 16:03 |
| *** pcaruana has quit IRC | 16:04 | |
| *** slaweq has joined #openstack-nova | 16:06 | |
| *** Swanson has quit IRC | 16:07 | |
| bauzas | mriedem: https://bugs.launchpad.net/nova/+bug/1748858 | 16:07 |
| openstack | Launchpad bug 1748858 in OpenStack Compute (nova) "allocation_ratio set by aggregate metadata don't work in NUMATopologyFilter" [Undecided,Won't fix] - Assigned to yangjie (yang.jie) | 16:07 |
| *** tidwellr has joined #openstack-nova | 16:07 | |
| bauzas | mriedem: we can clarify the doc, but I don't feel we should open a can of worms about aggregate ratios | 16:07 |
| *** itlinux has joined #openstack-nova | 16:08 | |
| *** krtaylor has quit IRC | 16:10 | |
| *** itlinux has quit IRC | 16:10 | |
| openstackgerrit | Eric Berglund proposed openstack/nova master: WIP: PowerVM Driver: Snapshot https://review.openstack.org/543023 | 16:10 |
| *** moshele has joined #openstack-nova | 16:12 | |
| mriedem | bauzas: i don't either | 16:12 |
| openstackgerrit | Merged openstack/nova-specs master: Fix the names on some specs https://review.openstack.org/542504 | 16:13 |
| *** suresh12 has joined #openstack-nova | 16:13 | |
| openstackgerrit | Merged openstack/nova-specs master: Move queens implemented specs https://review.openstack.org/542505 | 16:15 |
| *** damien_r has joined #openstack-nova | 16:16 | |
| *** suresh12 has quit IRC | 16:17 | |
| openstackgerrit | Eric Berglund proposed openstack/nova master: Use dict.get() when accessing capabilities dict https://review.openstack.org/539715 | 16:18 |
| *** tbachman has quit IRC | 16:20 | |
| *** itlinux has joined #openstack-nova | 16:22 | |
| *** itlinux has quit IRC | 16:23 | |
| *** dtantsur|brb is now known as dtantsur | 16:24 | |
| *** tbachman has joined #openstack-nova | 16:25 | |
| *** r-daneel has joined #openstack-nova | 16:32 | |
| *** felipemonteiro__ has quit IRC | 16:32 | |
| *** felipemonteiro_ has joined #openstack-nova | 16:32 | |
| *** damien_r has quit IRC | 16:33 | |
| *** jamespage has quit IRC | 16:34 | |
| *** damien_r has joined #openstack-nova | 16:34 | |
| *** jamespage has joined #openstack-nova | 16:35 | |
| *** itlinux has joined #openstack-nova | 16:35 | |
| *** imacdonn has joined #openstack-nova | 16:37 | |
| *** ragiman has quit IRC | 16:37 | |
| openstackgerrit | Bence Romsics proposed openstack/osc-placement master: PUT dict format to /allocations (v1.11, v1.12) https://review.openstack.org/542819 | 16:38 |
| openstackgerrit | sahid proposed openstack/nova master: libvirt: disconnect volume from host during detach https://review.openstack.org/515008 | 16:40 |
| *** suresh12 has joined #openstack-nova | 16:41 | |
| *** suresh12 has quit IRC | 16:41 | |
| *** suresh12 has joined #openstack-nova | 16:41 | |
| *** moshele has quit IRC | 16:43 | |
| *** felipemonteiro_ has quit IRC | 16:45 | |
| openstackgerrit | sahid proposed openstack/nova master: hardware: only take into account small pages https://review.openstack.org/532168 | 16:46 |
| *** links has joined #openstack-nova | 16:47 | |
| *** damien_r has quit IRC | 16:47 | |
| *** elmaciej has quit IRC | 16:47 | |
| *** lpetrut has joined #openstack-nova | 16:48 | |
| *** damien_r has joined #openstack-nova | 16:48 | |
| *** moshele has joined #openstack-nova | 16:51 | |
| *** hongbin has joined #openstack-nova | 16:51 | |
| *** acormier has quit IRC | 16:52 | |
| *** acormier has joined #openstack-nova | 16:52 | |
| *** links has quit IRC | 16:53 | |
| *** links has joined #openstack-nova | 16:54 | |
| gibi | mriedem: Would it be OK with you cancelling the next 3 notification subteam meeting? This week I don't feel we have things to talk about and next week I will already travelling to Dublin and then there will be PTG week. | 16:55 |
| *** moshele has quit IRC | 16:55 | |
| mriedem | gibi: works for me | 16:56 |
| gibi | mriedem: OK, cool | 16:56 |
| *** damien_r has quit IRC | 16:58 | |
| *** mgoddard_ has joined #openstack-nova | 17:00 | |
| *** damien_r has joined #openstack-nova | 17:01 | |
| *** mdnadeem has quit IRC | 17:02 | |
| mgariepy | is there a way to set a flavor to set the kvm=off option or i really need to pass the key from glance image ? | 17:04 |
| mgariepy | referring to : https://review.openstack.org/#/c/459753 | 17:05 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Move the nova-next job in-tree and update it https://review.openstack.org/541474 | 17:06 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Test websocketproxy with TLS in the nova-next job https://review.openstack.org/513160 | 17:06 |
| mriedem | mgariepy: looking at the code https://review.openstack.org/#/c/459753/18/nova/virt/libvirt/driver.py@4184 it only checks the image meta, not flavor extra specs | 17:06 |
| mgariepy | thanks mriedem i'll add the key to an image would it be a lot of work to add that to flavor extra specs ? | 17:07 |
| mriedem | probably not a lot of work, no, but generally i think we prefer image meta since those are standardized and flavor extra specs are not | 17:08 |
| *** fragatina has joined #openstack-nova | 17:08 | |
| mriedem | i guess the flavor extra spec could maybe make sense in this case if you're using a GPU trait on the flavor or something | 17:09 |
| *** sahid has quit IRC | 17:09 | |
| *** fragatin_ has joined #openstack-nova | 17:09 | |
| mriedem | https://docs.openstack.org/nova/latest/admin/virtual-gpu.html#configure-a-flavor-controller | 17:09 |
| *** suresh12 has quit IRC | 17:09 | |
| mriedem | this is getting dangerously into bauzas territory though | 17:10 |
| *** suresh12 has joined #openstack-nova | 17:10 | |
| mgariepy | i define my flavor to have the gpus via pci_passthrough:alias would have liked to have the hidden kvm stuff on that flavor as well. | 17:12 |
| mgariepy | but as long as it works i guess i'll use the workaround. | 17:12 |
| *** felipemonteiro_ has joined #openstack-nova | 17:13 | |
| openstackgerrit | Lee Yarwood proposed openstack/nova stable/pike: Do not attempt volume swap when guest is stopped/suspended https://review.openstack.org/533733 | 17:13 |
| *** fragatina has quit IRC | 17:13 | |
| *** mdnadeem has joined #openstack-nova | 17:15 | |
| mriedem | mgariepy: ok, i guess it's unclear to me if that's only an issue for windows guests as the commit message implies | 17:15 |
| mgariepy | i need that for linux guest as well. | 17:15 |
| mriedem | in that case it makes sense to me to be tied to the image | 17:15 |
| mriedem | oh | 17:15 |
| mgariepy | nvidia driver don't like kvm guest. | 17:15 |
| *** AJaeger has left #openstack-nova | 17:16 | |
| mriedem | i'm just amazed the thing is documented https://docs.openstack.org/python-glanceclient/latest/cli/property-keys.html | 17:16 |
| jaypipes | efried: where is that Numbered Request Group Affinity spec? | 17:16 |
| *** udesale has quit IRC | 17:17 | |
| efried | jaypipes: It's not a spec... yet. It's a subsection under Alternatives. | 17:17 |
| efried | jaypipes: I'm considering writing a spec for it, though. It would be largely the same as the one you're looking at. | 17:17 |
| *** Guest58801 has quit IRC | 17:17 | |
| *** belmoreira has quit IRC | 17:19 | |
| *** ttsiouts_ has quit IRC | 17:21 | |
| *** mvk_ has quit IRC | 17:22 | |
| *** ircuser-1 has joined #openstack-nova | 17:24 | |
| *** acormier_ has joined #openstack-nova | 17:27 | |
| *** acormier has quit IRC | 17:27 | |
| *** damien_r has quit IRC | 17:28 | |
| *** felipemonteiro_ has quit IRC | 17:28 | |
| *** suresh12 has quit IRC | 17:29 | |
| *** damien_r has joined #openstack-nova | 17:31 | |
| *** moshele has joined #openstack-nova | 17:32 | |
| *** damien_r has quit IRC | 17:34 | |
| *** damien_r has joined #openstack-nova | 17:34 | |
| *** mdnadeem has quit IRC | 17:35 | |
| *** suresh12_ has joined #openstack-nova | 17:35 | |
| *** moshele has quit IRC | 17:35 | |
| *** damien_r has quit IRC | 17:36 | |
| *** AlexeyAbashkin has quit IRC | 17:36 | |
| *** sridharg has quit IRC | 17:40 | |
| *** damien_r has joined #openstack-nova | 17:43 | |
| *** bkopilov has quit IRC | 17:43 | |
| *** yamamoto has quit IRC | 17:49 | |
| *** damien_r has quit IRC | 17:49 | |
| *** damien_r has joined #openstack-nova | 17:53 | |
| *** yamamoto has joined #openstack-nova | 17:54 | |
| *** david-lyle has quit IRC | 17:55 | |
| *** damien_r has quit IRC | 17:55 | |
| openstackgerrit | Merged openstack/nova-specs master: Granular Resource Request Syntax (Rocky) https://review.openstack.org/540179 | 17:57 |
| edleafe | jaypipes: Did we ever answer the question about whether we need "distance" to be continuous or binary? IOW, do we need "at least 2 away" or "closer than 5", or do we just need "is/isn't in the same grouping"? | 17:58 |
| *** damien_r has joined #openstack-nova | 17:58 | |
| edleafe | jaypipes: you mentioned some HA use cases that I'm not familiar with | 17:58 |
| *** mgoddard_ has quit IRC | 17:58 | |
| *** vivsoni__ has joined #openstack-nova | 17:59 | |
| *** yamamoto has quit IRC | 18:00 | |
| *** derekh has quit IRC | 18:00 | |
| *** damien_r has quit IRC | 18:00 | |
| *** Eran_Kuris_ has joined #openstack-nova | 18:02 | |
| *** ralonsoh has quit IRC | 18:07 | |
| *** dklyle has joined #openstack-nova | 18:07 | |
| *** Eran_Kuris_ has quit IRC | 18:07 | |
| openstackgerrit | Merged openstack/nova master: Add regression test for BFV+IsolatedHostsFilter failure https://review.openstack.org/543257 | 18:08 |
| *** Tom-Tom has quit IRC | 18:08 | |
| openstackgerrit | Merged openstack/nova master: Handle volume-backed instances in IsolatedHostsFilter https://review.openstack.org/543263 | 18:08 |
| openstackgerrit | Merged openstack/nova master: Fix docs for IsolatedHostsFilter https://review.openstack.org/543264 | 18:08 |
| *** Tom-Tom has joined #openstack-nova | 18:09 | |
| *** yamamoto has joined #openstack-nova | 18:11 | |
| *** yamamoto has quit IRC | 18:11 | |
| jaypipes | edleafe: well, the HA use cases are more about anti-affinity... i.e. "make sure the three nodes of my cluster are running in separate failure domains (or groups, or racks, or whatever)" | 18:13 |
| *** Tom-Tom has quit IRC | 18:13 | |
| *** harlowja has joined #openstack-nova | 18:14 | |
| edleafe | jaypipes: thanks. Are there any cases that you know of that don't boil down to yes/no? Like, are X and Y in the same "thing" (rack, row, etc.) or aren't they | 18:14 |
| openstackgerrit | Lee Yarwood proposed openstack/nova stable/pike: libvirt: Block swap_volume attempts with encrypted volumes https://review.openstack.org/543569 | 18:15 |
| jaypipes | edleafe: yeah. the whole "soft affinity" is what those use cases are about. | 18:17 |
| jaypipes | edleafe: in "soft affinity", it's a preference vs. yes/no | 18:17 |
| *** jpena is now known as jpena|off | 18:18 | |
| jaypipes | edleafe: i.e. "place these things *as near to each other* as possible" or "place these related things as far away as possible" | 18:18 |
| *** bkopilov has joined #openstack-nova | 18:18 | |
| jaypipes | edleafe: in the soft affinity cases, the distance being relative and not yes/no enables that sorting of potential destinations | 18:18 |
| edleafe | jaypipes: how widespread is that need? IOW, 20% of use cases? 1%? | 18:19 |
| jaypipes | edleafe: the soft affinity thing? | 18:20 |
| edleafe | jaypipes: yeah. It seems like overkill to build all our affinity/anti stuff to handle these types of things | 18:20 |
| jaypipes | edleafe: BTW, the "soft" stuff is also interpreted as "if the strict condition doesn't hold, then just get me the thing that closest matches my condition" -- i.e. sorting.. | 18:21 |
| edleafe | jaypipes: when the vast majority just want yes/no | 18:21 |
| jaypipes | edleafe: it may indeed be overkill. but it's what we currently have. | 18:21 |
| edleafe | jaypipes: where do we have it? | 18:21 |
| jaypipes | edleafe: the soft affinity "filters" are implemented as weighers. | 18:22 |
| openstackgerrit | Matthew Edmonds proposed openstack/nova master: improve support matrix notes https://review.openstack.org/540534 | 18:22 |
| jaypipes | edleafe: https://github.com/openstack/nova/blob/master/nova/scheduler/weights/affinity.py | 18:22 |
| *** tbachman has quit IRC | 18:22 | |
| *** lpetrut has quit IRC | 18:23 | |
| *** lpetrut has joined #openstack-nova | 18:23 | |
| edleafe | jaypipes: that doesn't use any sort of "distance" concept. It just favors those things that meet (anti)affinity | 18:24 |
| jaypipes | edleafe: correct. | 18:24 |
| jaypipes | edleafe: the distance thing is something we need in order to understand the difference between various failure domain sizes. it's also necessary for informing placement about the different "levels" of affinity. right now, Nova only knows one level (the compute *host* (note: *not* the compute *node*). In other words, Ironic can't use the existing affinity filters/weights in Nova. | 18:26 |
| jaypipes | edleafe: this is partly to address that issue: https://review.openstack.org/#/c/529135/ | 18:26 |
| *** lpetrut has quit IRC | 18:27 | |
| *** lpetrut has joined #openstack-nova | 18:27 | |
| edleafe | jaypipes: yeah, I like that | 18:27 |
| *** r-daneel has quit IRC | 18:27 | |
| openstackgerrit | Eric Berglund proposed openstack/nova master: Use correct arguments in task inits https://review.openstack.org/543571 | 18:28 |
| *** lpetrut has quit IRC | 18:28 | |
| *** lpetrut_ has joined #openstack-nova | 18:28 | |
| *** lpetrut has joined #openstack-nova | 18:28 | |
| jaypipes | edleafe: of course, that doesn't touch placement... | 18:28 |
| edleafe | jaypipes: understood | 18:29 |
| *** lpetrut has quit IRC | 18:29 | |
| *** lpetrut_ has quit IRC | 18:29 | |
| *** lpetrut has joined #openstack-nova | 18:29 | |
| *** dklyle has quit IRC | 18:30 | |
| edleafe | jaypipes: my concern is that adding the new tables and relations you proposed on Friday would not be necessary for the vast majority of (anti)affinity cases. We could do it with placement aggs | 18:30 |
| *** lpetrut has quit IRC | 18:30 | |
| *** lpetrut has joined #openstack-nova | 18:30 | |
| openstackgerrit | Lee Yarwood proposed openstack/nova master: Enable test_volume_backed_live_migration in tempest https://review.openstack.org/528104 | 18:31 |
| mriedem | edleafe: we don't really know what the vast majority of anything is | 18:31 |
| mriedem | usage wise | 18:31 |
| openstackgerrit | Merged openstack/python-novaclient stable/pike: Avoid tox_install.sh for constraints support https://review.openstack.org/543115 | 18:31 |
| edleafe | mriedem: and what we do know may change | 18:32 |
| mriedem | i want a minimum of 3 DB servers, put on different hosts if you can, but don't fail if you can't | 18:32 |
| openstackgerrit | Merged openstack/python-novaclient stable/pike: Zuul: Remove project name https://review.openstack.org/541370 | 18:32 |
| edleafe | s/may/will | 18:32 |
| openstackgerrit | Merged openstack/python-novaclient stable/ocata: Avoid tox_install.sh for constraints support https://review.openstack.org/543114 | 18:32 |
| mriedem | we're going to be proposing a spec for something that builds on the soft affinity stuff | 18:32 |
| mriedem | to put a limit on the softness | 18:32 |
| edleafe | mriedem: sure. The complicating part is I want three servers, at least two racks away from each other | 18:32 |
| edleafe | not just on different racks | 18:33 |
| mriedem | yeah the distance concept is complicated | 18:33 |
| cdent | what does "two rack away" even mean? | 18:35 |
| *** lpetrut has quit IRC | 18:35 | |
| *** r-daneel has joined #openstack-nova | 18:35 | |
| *** tidwellr has quit IRC | 18:35 | |
| *** lpetrut has joined #openstack-nova | 18:35 | |
| *** sshwarts has quit IRC | 18:36 | |
| edleafe | cdent: it would depend on the operator mapping out their DC, and assigning values to each "location" | 18:36 |
| dansmith | cdent: separated by power and TOR switch | 18:36 |
| *** ttsiouts_ has joined #openstack-nova | 18:36 | |
| dansmith | cdent: so two adjacent racks might share a power domain, but not TOR switch, | 18:37 |
| cdent | dansmith: I understand why people want what you just described in terms of "power" and "TOR" but don't see how that is a "distance of 2" | 18:37 |
| dansmith | so two away would mean not adjacent at the next level up in separation | 18:37 |
| cdent | so "2" is something the operator is defining? | 18:37 |
| dansmith | cdent: it's a simpler way to accomplish the same thing | 18:37 |
| dansmith | yeah | 18:37 |
| *** tidwellr has joined #openstack-nova | 18:38 | |
| dansmith | I think we said we'd try to work up some guidelines | 18:38 |
| dansmith | that they could follow to try to have some sort of a uniform idea about said things, | 18:38 |
| edleafe | dansmith: couldn't it be simplified to "not in the same power domain"? | 18:38 |
| dansmith | but at the end of the day, it'll come down to your layout | 18:38 |
| *** suresh12_ has quit IRC | 18:39 | |
| dansmith | edleafe: sure, but then you start providing the "not power domain and not network domain and not cooling domain", and distance is a simpler way to describe relative closeness of two thngs | 18:39 |
| *** lpetrut has quit IRC | 18:39 | |
| edleafe | dansmith: I'm not following. Are those domains nested? | 18:40 |
| dansmith | I believe that was jaypipes' original intent, to provide a way to use a simple metric for such things instead of requiring the user to know about and upload a full map | 18:40 |
| dansmith | edleafe: in my example yeah | 18:40 |
| *** lpetrut has joined #openstack-nova | 18:40 | |
| edleafe | dansmith: ok, so that could be simplified to "not in the outermost grouping I care about", right? | 18:41 |
| openstackgerrit | Mark Goddard proposed openstack/nova master: Don't JSON encode instance_info.traits for ironic https://review.openstack.org/543574 | 18:41 |
| dansmith | edleafe: well, that's not the outermost group | 18:42 |
| *** tidwellr has quit IRC | 18:42 | |
| dansmith | because you'd still have potentially room, building, site, region, etc | 18:42 |
| edleafe | dansmith: of course | 18:42 |
| *** suresh12 has joined #openstack-nova | 18:42 | |
| dansmith | we could define the way people have to nest things (i.e. TOR is always smaller scope than power) but that won't fly of course | 18:43 |
| edmondsw | mriedem I just added the queens-rc-potential tag to https://bugs.launchpad.net/nova/+bug/1748950 | 18:43 |
| openstack | Launchpad bug 1748950 in OpenStack Compute (nova) "PowerVM driver passing bad kwargs into Task init method" [Undecided,In progress] - Assigned to Eric Berglund (esberglu) | 18:43 |
| edleafe | dansmith: you would define these groupings by what you care about as far as affinity is concerned | 18:43 |
| mriedem | edmondsw: how did this not fail CI? | 18:43 |
| dansmith | yeah, we could totally add a way to define the groupings, and a complex way to say which things you do and don't care about | 18:43 |
| dansmith | all I'm saying is that distance as a simple metric to abstract that is pretty straightforward | 18:43 |
| edleafe | dansmith: or we could add a simple way to say that | 18:44 |
| edmondsw | mriedem it only happens if you have non-ascii characters in the instance | 18:44 |
| mriedem | edmondsw: oh i see, unicode | 18:44 |
| edmondsw | mriedem it actually did cause a log error in CI which is how we noticed it, on update_server_name | 18:44 |
| edmondsw | but didn't fail the test | 18:44 |
| *** jmlowe has quit IRC | 18:44 | |
| edmondsw | was just on cleanup | 18:44 |
| mriedem | is there no way to have a test for this fix? | 18:45 |
| *** links has quit IRC | 18:45 | |
| edmondsw | mriedem yeah, good point... I'd asked esberglu to look at test but then had a brain fart when reviewing I guess | 18:46 |
| mriedem | pew | 18:47 |
| *** suresh12 has quit IRC | 18:47 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova-specs master: Re-propose use Neutron's new port binding API https://review.openstack.org/543578 | 18:48 |
| *** itlinux has quit IRC | 18:49 | |
| mriedem | efried: want to re-propose this for rocky? https://review.openstack.org/#/c/505209/ | 18:50 |
| *** itlinux has joined #openstack-nova | 18:51 | |
| openstackgerrit | Dan Smith proposed openstack/nova master: Drop compute RPC 4.x compatibility https://review.openstack.org/543580 | 18:51 |
| jaypipes | dansmith, mriedem: https://etherpad.openstack.org/p/going-the-distance has ideas about the distance concept... | 18:52 |
| mriedem | nice pad name | 18:52 |
| *** itlinux has quit IRC | 18:53 | |
| mriedem | dansmith: you want to -W that rpc change until we release so someone doesn't accidentally merge it? | 18:53 |
| dansmith | mriedem: rc1 is tagged so we're good no? | 18:53 |
| *** david-lyle has joined #openstack-nova | 18:54 | |
| mriedem | i don't know if we'll have something that requires a backport before GA | 18:54 |
| dansmith | I'm going to argue against that really hard, | 18:54 |
| openstackgerrit | Merged openstack/nova master: doc: mention that --on-shared-storage is not needed with nova evacuate https://review.openstack.org/542512 | 18:54 |
| dansmith | but also, it won't invalidate this | 18:54 |
| mriedem | granted backporting an rpc change for rc2 would be.. | 18:54 |
| dansmith | but no crazy rush so that's fine | 18:54 |
| openstackgerrit | Merged openstack/nova master: Update reno for stable/queens https://review.openstack.org/542658 | 18:55 |
| *** itlinux has joined #openstack-nova | 18:55 | |
| esberglu | edmondsw: What's the best way to test this? I can assert that the __init__ methods are being passed the right args | 18:59 |
| edmondsw | esberglu autospec? | 18:59 |
| *** ttsiouts_ has quit IRC | 18:59 | |
| *** lucasagomes is now known as lucas-pto | 19:03 | |
| *** ttsiouts_ has joined #openstack-nova | 19:03 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova-specs master: Add z/VM Driver in Rocky https://review.openstack.org/536186 | 19:04 |
| *** jmlowe has joined #openstack-nova | 19:04 | |
| *** jmlowe has quit IRC | 19:06 | |
| openstackgerrit | Merged openstack/nova-specs master: Modify grammatical errors https://review.openstack.org/541765 | 19:06 |
| *** jmlowe has joined #openstack-nova | 19:06 | |
| *** jmlowe has quit IRC | 19:08 | |
| *** jmlowe has joined #openstack-nova | 19:09 | |
| *** tidwellr has joined #openstack-nova | 19:11 | |
| *** yamamoto has joined #openstack-nova | 19:12 | |
| *** gyee has joined #openstack-nova | 19:12 | |
| *** sambetts is now known as sambetts|afk | 19:13 | |
| *** eharney has quit IRC | 19:13 | |
| *** jmlowe has quit IRC | 19:15 | |
| *** moshele has joined #openstack-nova | 19:15 | |
| mriedem | melwitt: need to re-propose https://review.openstack.org/#/c/500975/ for rocky | 19:15 |
| *** suresh12 has joined #openstack-nova | 19:16 | |
| *** tbachman has joined #openstack-nova | 19:17 | |
| *** tbachman has quit IRC | 19:18 | |
| *** tesseract has quit IRC | 19:18 | |
| openstackgerrit | Mark Goddard proposed openstack/nova master: Unplug all VIFs from ironic nodes during tear down https://review.openstack.org/537626 | 19:19 |
| *** yamamoto has quit IRC | 19:19 | |
| *** tbachman has joined #openstack-nova | 19:21 | |
| *** suresh12 has quit IRC | 19:21 | |
| *** jmlowe has joined #openstack-nova | 19:22 | |
| *** felipemonteiro has joined #openstack-nova | 19:23 | |
| mriedem | jianghuaw: remember to bring this bp up during the nova meeting at some point (after the holiday at this point) https://blueprints.launchpad.net/nova/+spec/xenapi-image-handler-option-improvement | 19:25 |
| mgoddard | mriedem: I've just seen your ML email about backports. Would https://bugs.launchpad.net/nova/+bug/1748942 qualify? The fix is simple, and limited to the ironic virt driver | 19:26 |
| openstack | Launchpad bug 1748942 in OpenStack Compute (nova) "Ironic virt driver should not JSON encode traits in instance_info" [Undecided,In progress] - Assigned to Mark Goddard (mgoddard) | 19:26 |
| mgoddard | mriedem: fix is this: https://review.openstack.org/#/c/543574/1 | 19:26 |
| *** amoralej is now known as amoralej|off | 19:27 | |
| *** vivsoni__ has quit IRC | 19:27 | |
| mriedem | mgoddard: so traits-based scheduling with ironic+nova doesn't work without this fix? | 19:27 |
| *** yamahata has quit IRC | 19:28 | |
| mriedem | or what is instance_info.traits used for? | 19:28 |
| mriedem | i thought that was something to do with the capabilities filter in nova | 19:28 |
| *** suresh12 has joined #openstack-nova | 19:28 | |
| mriedem | which we're superseding with placement + traits | 19:28 |
| mgoddard | mriedem: the scheduling works, but any flavor-requested traits won't be pushed to the ironic node's instance_info. These are not yet used by ironic, but will be used in future for some capabilities-like things | 19:29 |
| mriedem | mgoddard: if they aren't used in ironic in queens, then it's not something worth holding up rc2 IMO | 19:29 |
| mgoddard | or rather, they are pushed incorrectly | 19:29 |
| mriedem | i assume you're asking about this for rc2 | 19:30 |
| mriedem | rc2 changes at this point == stop ship | 19:30 |
| mriedem | once we release queens 17.0.0 we can backport regular old bug fixes to stable/queens as usual | 19:30 |
| *** moshele has quit IRC | 19:30 | |
| mgoddard | mriedem: I suspect you're right - it isn't sufficiently important to block rc2 | 19:31 |
| mriedem | does ironic have a tempest plugin? | 19:31 |
| mriedem | ooo look it does https://github.com/openstack/ironic-tempest-plugin | 19:31 |
| mgoddard | mriedem: it does | 19:31 |
| mriedem | can we work on something in there that actually uses real live traits to make sure the plumbing is all correct between the two projects? | 19:32 |
| *** suresh12 has quit IRC | 19:33 | |
| mgoddard | mriedem: that sounds sensible | 19:33 |
| mgoddard | mriedem: I'll look into it | 19:34 |
| mriedem | thanks | 19:34 |
| *** Tom-Tom has joined #openstack-nova | 19:34 | |
| *** Tom-Tom has quit IRC | 19:34 | |
| mriedem | this amount of coupling between projects means we really need some higher level integration testing | 19:34 |
| mriedem | than unit tests | 19:34 |
| mgoddard | there are tempest tests covering the ironic traits API, but you're right in needing to test the nova-ironic interface | 19:36 |
| *** jmlowe has quit IRC | 19:38 | |
| *** felipemonteiro has quit IRC | 19:39 | |
| *** felipemonteiro_ has joined #openstack-nova | 19:39 | |
| *** suresh12 has joined #openstack-nova | 19:39 | |
| *** jmlowe has joined #openstack-nova | 19:40 | |
| *** lbragstad_ has joined #openstack-nova | 19:41 | |
| *** moshele has joined #openstack-nova | 19:43 | |
| *** lbragstad_ has quit IRC | 19:43 | |
| *** suresh12 has quit IRC | 19:44 | |
| *** suresh12 has joined #openstack-nova | 19:44 | |
| *** dtantsur is now known as dtantsur|afk | 19:44 | |
| openstackgerrit | Aditya Vaja proposed openstack/nova master: remove IVS plug/unplug as they're moved to separate plugin https://review.openstack.org/534371 | 19:46 |
| *** moshele has quit IRC | 19:46 | |
| *** pramodrj07 has joined #openstack-nova | 19:49 | |
| efried | mriedem: I wouldn't mind, but shouldn't edleafe or jaypipes repropose the NRP spec? | 19:52 |
| mriedem | efried: ed did it in queens, | 19:55 |
| mriedem | i think anyone familiar with the changes can do it | 19:55 |
| mriedem | which i think you are | 19:55 |
| efried | mriedem: Roger that. jaypipes edleafe Any objections? | 19:56 |
| efried | ... or local WIP already? | 19:57 |
| edleafe | efried: not from me | 19:58 |
| *** READ10 has quit IRC | 20:01 | |
| *** READ10 has joined #openstack-nova | 20:03 | |
| *** eharney has joined #openstack-nova | 20:03 | |
| *** yamahata has joined #openstack-nova | 20:05 | |
| openstackgerrit | Lee Yarwood proposed openstack/nova master: rbd: flatten images when creating/unshelving an instance https://review.openstack.org/457886 | 20:06 |
| mriedem | dansmith: we likely want 1 or 2 blueprints for rocky for cellsv1 and nova-network removal...figured you'd like to create those | 20:08 |
| *** Swami has joined #openstack-nova | 20:08 | |
| mriedem | the nova-net removal is going to be a pita b/c of our functional and unit tests that rely on stubs and crap | 20:08 |
| *** ttsiout__ has joined #openstack-nova | 20:10 | |
| jaypipes | efried: nope, go for it. | 20:11 |
| efried | ack | 20:11 |
| *** ttsiouts_ has quit IRC | 20:11 | |
| efried | jaypipes, edleafe: Though some of these assumptions (e.g. about NUMA) have... "evolved". | 20:12 |
| efried | ...or are under discussion. | 20:12 |
| *** moshele has joined #openstack-nova | 20:13 | |
| edleafe | efried: yeah, that's why we need to re-propose: to reflect today's situation | 20:14 |
| *** moshele has quit IRC | 20:15 | |
| mriedem | did somebody say "the situation"? | 20:15 |
| efried | edleafe: Considering how much of the work is done, and how we've refined/detailed what remains, it's pretty much gonna be a brand new spec. | 20:15 |
| mriedem | http://pmcdeadline2.files.wordpress.com/2014/03/mike-the-situation__140331172717.jpg | 20:15 |
| efried | Which means no fast approval. But hopefully closer fidelity to reality. | 20:16 |
| edleafe | efried: isn't that always the case? :) | 20:16 |
| edleafe | mriedem: ah, your laptop's background image | 20:16 |
| mriedem | i use it as motivation | 20:16 |
| mriedem | to be jerseylicious | 20:16 |
| openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: Add regression test for BFV+IsolatedHostsFilter failure https://review.openstack.org/543602 | 20:17 |
| openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: Handle volume-backed instances in IsolatedHostsFilter https://review.openstack.org/543603 | 20:17 |
| openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: Fix docs for IsolatedHostsFilter https://review.openstack.org/543604 | 20:17 |
| mriedem | efried: how much of the change to the spec is implementation details vs just high level design? | 20:18 |
| efried | mriedem: Let me finish my initial reread and get back to you on that. But at a glance, the main thrust of the spec is implementing root_provider_uuid and parent_provider_uuid, which is done. What remains, and what we should probably give a little more detail about, is the GET /allocation_candidates changes. | 20:19 |
| *** esberglu has quit IRC | 20:19 | |
| mriedem | then for the done stuff, say it's done in queens and update the not-yet-done-stuff parts | 20:20 |
| efried | mriedem: That latter part isn't even mentioned in this spec. | 20:21 |
| efried | But yeah. I'll do the needful. | 20:22 |
| efried | There will be controversy over NUMA, though. | 20:22 |
| mriedem | efried: well, if the NRP spec is all about stuff that's already done, then there probably isn't a need to re-propose it, | 20:22 |
| mriedem | and we create a new bp/spec for the things that are pending and move all of those changes under that new bp? | 20:22 |
| mriedem | but i deferred the NRP bp from queens b/c there were remaining things to use it | 20:23 |
| efried | mriedem: Finished reading now, and yeah, that's kinda what I was thinking. | 20:23 |
| *** alexchadin has joined #openstack-nova | 20:23 | |
| efried | mriedem: I think that (deferring/remaining) was a reflection of "scope creep". | 20:24 |
| mriedem | https://specs.openstack.org/openstack/nova-specs/specs/queens/approved/nested-resource-providers.html#work-items | 20:24 |
| mriedem | the last 2 bullets in there aren't done are they? | 20:24 |
| efried | mriedem: Nor will they be. | 20:24 |
| efried | Cause that's no longer the plan. | 20:24 |
| mriedem | oy | 20:24 |
| mriedem | ok, so the nrp spec became the api stuff and foundational plumbing, | 20:24 |
| mriedem | and support for modeling numa and sriov are separate new thing snow | 20:25 |
| mriedem | yeah? | 20:25 |
| efried | That's part of the story, yes. | 20:25 |
| mriedem | if that's the case, then maybe we should amend the queens spec, and mark the bp as done for queens, even though we don't have numa/sriov modeling with it | 20:25 |
| efried | If the update_provider_tree stuff had gotten finished in Q, it would have been tucked under the umbrella of the nrp blueprint (most of it was merged under that topic) even though it was never mentioned or even hinted at in the nrp spec. | 20:26 |
| edleafe | jaypipes: cdent: efried: dansmith: mriedem: wrote up something quick on my ideas on affinity: https://blog.leafe.com/modeling-affinity/ | 20:26 |
| edleafe | comments welcome! | 20:26 |
| mriedem | efried: that's an implementation detail though isn't it? | 20:26 |
| *** suresh12 has quit IRC | 20:26 | |
| cdent | thanks edleafe enqueued for tomorrow, currently writing my own | 20:27 |
| efried | mriedem: Which, update_provider_tree? I don't think there's any possible interpretation by which that stack of work could be considered just "implementation detail". | 20:27 |
| *** moshele has joined #openstack-nova | 20:27 | |
| mriedem | efried: ok, i guess i need to read your provider tree spec then | 20:28 |
| mriedem | which i thought you said was less of a spec and more just docs of the thing that merged in queens? | 20:28 |
| efried | ...which needed to be specified in some form, yeah. | 20:29 |
| efried | And which didn't quite get finished in queens, and is a big enough piece of work to warrant a bp and spec of its own (at least if it wasn't going to be described in the nrp spec) | 20:29 |
| *** esberglu has joined #openstack-nova | 20:30 | |
| *** READ10 has quit IRC | 20:30 | |
| mriedem | ok, i'll get to new business some other day; was just trying to go through easy re-approvals | 20:31 |
| mriedem | i will mentally note that this isn't an easy re-approval | 20:32 |
| efried | mriedem: Anyway, to tie that back together, I believe update_provider_tree changes the NUMA plan as described around https://review.openstack.org/#/c/505209/7/specs/queens/approved/nested-resource-providers.rst@136 | 20:32 |
| *** moshele has quit IRC | 20:33 | |
| efried | mriedem: Because we're no longer going to be basing anything on get_available_resource. Rather, we'll ask virt to report NUMA topology through update_provider_tree (in some standardized format TBD? Swhat jaypipes and I are debating in https://review.openstack.org/543062) and then deal with the pinning/affinitizing from there. | 20:34 |
| *** alexchadin has quit IRC | 20:34 | |
| efried | Suspect there will be much PTGing at the PTG about this. | 20:34 |
| mriedem | ok - thinking again about just amending the queens spec, making a note that the spec is just focusing on groundwork stuff, might give examples of use cases, but details for supporting other use cases (sriov/numa) are going to be found in later specs | 20:35 |
| efried | mriedem: Do you want me to propose that edit? | 20:35 |
| mriedem | up to you | 20:35 |
| efried | Do we do that? | 20:35 |
| mriedem | sometimes if things changed quite a bit | 20:35 |
| mriedem | the thing i wouldn't want to say is, | 20:35 |
| mriedem | the nrp spec is marked 'implemented' for queens and it says 'we'll model sriov with this in queens' and then we didn't | 20:36 |
| mriedem | if we aren't going to re-propose the nrp spec for rocky because it's basically done and the rest is some other new spec, then we should say it was just done in queens | 20:36 |
| mriedem | and amend accordingly | 20:36 |
| efried | mriedem: Okay, well, the spec does say the NUMA/SR-IOV stuff won't be done in Q. | 20:36 |
| efried | But it also asserts pretty forcefully what that will look like when it does get implemented. | 20:37 |
| *** suresh12 has joined #openstack-nova | 20:37 | |
| mriedem | the goal in queens was sriov | 20:37 |
| mriedem | https://specs.openstack.org/openstack/nova-specs/specs/queens/approved/nested-resource-providers.html#work-items | 20:37 |
| mriedem | "Please note that not all of this spec is expected to be implemented in a single release cycle. At the Queens PTG we agreed that fully suppporting NUMA will probably have to be deferred to the next release." just came out of the ptg saying we wouldn't try to get numa done | 20:38 |
| *** lpetrut has quit IRC | 20:38 | |
| mriedem | which implies sriov would be supported in queens | 20:38 |
| *** tssurya has quit IRC | 20:39 | |
| efried | which is kind of a wow, because we're not even close to supporting SR-IOV. It's more than just NRP that needs to be wrapped up to make that happen. | 20:47 |
| efried | mriedem: Softening those statements jeeest a leetle bit would make them more or less true: "...the ability to *model* NUMA and SR-IOV..." | 20:48 |
| efried | You can model it. You just can't *do* anything with that model. | 20:48 |
| efried | And you can't actually model it without writing more code on the nova side; but the placement side will handle it. | 20:49 |
| efried | Except for the GET /allocation_candidates thing, which is required if it's going to be usable at all. | 20:49 |
| mriedem | https://specs.openstack.org/openstack/nova-specs/specs/queens/approved/nested-resource-providers.html#rest-api-impact | 20:49 |
| efried | So yeah, it's a mess. | 20:49 |
| mriedem | "The filter parameter root={uuid} will not be added to GET /allocation_candidates, as this call is for a specific use case for the Nova scheduler, and there is no use case for it." | 20:49 |
| efried | Still true. | 20:50 |
| mriedem | i think the nrp spec is mostly just about the modeling and placement api bits, | 20:50 |
| mriedem | i think it's fine to say nova's usage of it for certain use cases (sriov/numa) are separate efforts that will build on this later | 20:50 |
| efried | But GET /allocation_candidates - the algorithm on the back end - needs to be updated to understand and handle nested providers. Today it doesn't. If you have a tree model, you get no candidates. | 20:50 |
| mriedem | sure, that's a new work effort | 20:51 |
| efried | That's something that's not mentioned at all, but needs to be, and needs to be completed in Rocky ASAP. | 20:51 |
| *** itlinux has quit IRC | 20:51 | |
| mriedem | so skimming over the spec again, it seems like if one just removed (or somehow re-worded) the parts about the scheduler reporting client in nova, it would explain what was completed in queens | 20:53 |
| mriedem | agree? | 20:53 |
| mriedem | add root_provider_uuid and parent_provider_uuid to GET /resource_providers, fin | 20:53 |
| mriedem | .. note:: Future work will be needed client-side to model things like SR-IOV and NUMA for nova to leverage the Placement API for nested resource providers. | 20:54 |
| efried | and ?tree= to the latter (written as ?root= in the spec, was renamed during impl) | 20:54 |
| mriedem | you mean ?in_tree? | 20:54 |
| mriedem | bazinga! | 20:55 |
| mriedem | yeah so amend the ?in_tree thingy (we still haven't defined how that filter works i don't think) | 20:55 |
| mriedem | remove/reword the parts about the scheduler client - say those are done separately later | 20:55 |
| efried | fusque | 20:55 |
| mriedem | and i think we say the rest is done in queens | 20:55 |
| mriedem | with caveats like the GET /allocation_candidates thing | 20:56 |
| efried | Oh, we nailed down in_tree. It's implemented. | 20:56 |
| mriedem | i know it's in the api | 20:56 |
| mriedem | but the semantics of how that works is not documented anywhere as far as i know | 20:56 |
| mriedem | i remember bringing this up in a review or the ML or something at one point | 20:56 |
| mriedem | https://review.openstack.org/#/c/520663/9/nova/tests/unit/scheduler/client/test_report.py@1400 | 20:57 |
| efried | https://developer.openstack.org/api-ref/placement/#list-resource-providers | 20:57 |
| mriedem | yeah i know about that, all it says is "A UUID of a resource provider. The returned resource providers will be in the same “provider tree” as the specified provider." | 20:57 |
| jaypipes | what's unclear about that? | 20:58 |
| mriedem | based on your reply to https://review.openstack.org/#/c/520663/9/nova/tests/unit/scheduler/client/test_report.py@1400 it means, in_tree with *any* node in the tree returns the entire tree | 20:58 |
| jaypipes | ah I see | 20:58 |
| mriedem | in my example, B and D are in the same tree because they share a common ancestor | 20:59 |
| *** suresh12 has quit IRC | 20:59 | |
| mriedem | but i wasn't sure about that | 20:59 |
| mriedem | i mean on first reading of the api-ref | 20:59 |
| efried | It doesn't say "subtree" or "below". I mean, we could add words to explain it more fully, including an example like the one in the comment. | 21:00 |
| mriedem | http://lists.openstack.org/pipermail/openstack-dev/2017-December/125453.html | 21:00 |
| mriedem | yeah that's what i asked for in ^ | 21:00 |
| jaypipes | sorry, must have missed that | 21:01 |
| mriedem | it's fine, it was 2 weeks before xmas | 21:01 |
| efried | Think I was already out by then. Yeah, that's the excuse I'll use. | 21:01 |
| mriedem | anyway, i think we're maybe in agreement about the spec? | 21:02 |
| mriedem | before the tangent | 21:02 |
| *** ChanServ has quit IRC | 21:02 | |
| efried | mriedem: I'll take a crack at it. And a new spec for R. | 21:02 |
| mriedem | thanks; i know it's not fun | 21:03 |
| efried | mriedem: I'm doing it all for that shiny nickel you promised me back in the Fall. | 21:04 |
| mriedem | i promised a nickel | 21:04 |
| mriedem | ? | 21:04 |
| efried | oh, totally. I have it in writing. | 21:04 |
| mriedem | https://www.reddit.com/r/TheSimpsons/comments/1saamn/and_remember_a_shiny_new_donkey_for_whoever/ | 21:04 |
| mriedem | if i did, it's because i ripped off the simpsons | 21:04 |
| mriedem | ooo https://www.youtube.com/watch?v=0gGsKxMCM9o | 21:05 |
| efried | mriedem: https://review.openstack.org/#/c/506370/ | 21:07 |
| efried | note topic | 21:08 |
| efried | I still need to find the eavesdrop where you offered it up, tho. | 21:08 |
| *** felipemonteiro__ has joined #openstack-nova | 21:08 | |
| mriedem | http://eavesdrop.openstack.org/irclogs/%23openstack-nova/%23openstack-nova.2017-09-21.log.html#t2017-09-21T16:39:19 | 21:08 |
| efried | bwaHA http://eavesdrop.openstack.org/irclogs/%23openstack-nova/%23openstack-nova.2017-09-21.log.html#t2017-09-21T16:39:19 | 21:08 |
| efried | yeah, what you said. | 21:08 |
| mriedem | i win | 21:09 |
| efried | Happy to concede that victory. I still get my nickel. | 21:09 |
| mriedem | alright, fair enough | 21:09 |
| * mriedem makes a note about this gd nickel for dublin | 21:09 | |
| *** slaweq has quit IRC | 21:09 | |
| efried | Do they have Euro nickels? | 21:09 |
| efried | We've merged some Rocky specs at this point. Is the publish job broken again, or just slow? http://specs.openstack.org/openstack/nova-specs/specs/rocky/ | 21:10 |
| *** liverpooler has quit IRC | 21:11 | |
| *** ChanServ has joined #openstack-nova | 21:11 | |
| *** barjavel.freenode.net sets mode: +o ChanServ | 21:11 | |
| mriedem | don't know | 21:11 |
| *** felipemonteiro_ has quit IRC | 21:12 | |
| efried | I don't see 540179 in the post queue. So... | 21:12 |
| * efried heads over to -infra... | 21:12 | |
| mriedem | i've got some 5 kronors from the czech republic.. | 21:13 |
| *** jaypipes has quit IRC | 21:16 | |
| *** r-daneel has quit IRC | 21:17 | |
| *** r-daneel has joined #openstack-nova | 21:17 | |
| mriedem | *koruna? | 21:17 |
| openstackgerrit | Merged openstack/nova master: Remove unused tempest-dsvm-lxc-rc https://review.openstack.org/539244 | 21:18 |
| openstackgerrit | Merged openstack/nova master: [placement] annotate loadapp as public interface https://review.openstack.org/526691 | 21:19 |
| efried | mriedem: FYI that publish job is queued - filter by 32b00cc (which is apparently the merge commit hash for the spec patch). | 21:22 |
| *** felipemonteiro__ has quit IRC | 21:23 | |
| *** felipemonteiro__ has joined #openstack-nova | 21:23 | |
| *** jgriffith has quit IRC | 21:26 | |
| *** mvk_ has joined #openstack-nova | 21:33 | |
| efried | mriedem: Just confirming: we don't branch nova-specs? So I don't need to propose this edit "back" to queens? | 21:34 |
| *** suresh12 has joined #openstack-nova | 21:34 | |
| efried | ...other than by virtue of it being in the .../queens/... subdirectory | 21:34 |
| *** oomichi has joined #openstack-nova | 21:35 | |
| mriedem | efried: correct | 21:35 |
| efried | ack | 21:35 |
| mriedem | much like the american constitution, it's a living document | 21:36 |
| *** edmondsw has quit IRC | 21:39 | |
| *** felipemonteiro__ has quit IRC | 21:39 | |
| *** slaweq has joined #openstack-nova | 21:41 | |
| openstackgerrit | Chris Dent proposed openstack/nova master: WIP: Move resource provider objects into placement hierarchy https://review.openstack.org/540049 | 21:48 |
| openstackgerrit | Chris Dent proposed openstack/nova master: Optional separate database for placement API https://review.openstack.org/362766 | 21:48 |
| openstackgerrit | Chris Dent proposed openstack/nova master: WIP: Isolate placement database config https://review.openstack.org/541435 | 21:48 |
| openstackgerrit | Chris Dent proposed openstack/nova master: Isolate config parse_args for placement https://review.openstack.org/543495 | 21:48 |
| openstackgerrit | Matt Riedemann proposed openstack/nova-specs master: Re-propose use Neutron's new port binding API https://review.openstack.org/543578 | 21:53 |
| *** ttsiout__ has quit IRC | 21:53 | |
| *** ttsiouts_ has joined #openstack-nova | 21:54 | |
| *** kaisers_ has joined #openstack-nova | 21:55 | |
| *** burt has quit IRC | 21:57 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova stable/ocata: Add regression test for BFV+IsolatedHostsFilter failure https://review.openstack.org/543634 | 21:58 |
| openstackgerrit | Matt Riedemann proposed openstack/nova stable/ocata: Handle volume-backed instances in IsolatedHostsFilter https://review.openstack.org/543635 | 21:58 |
| openstackgerrit | Matt Riedemann proposed openstack/nova stable/ocata: Fix docs for IsolatedHostsFilter https://review.openstack.org/543636 | 21:58 |
| *** ttsiouts_ has quit IRC | 21:58 | |
| *** felipemonteiro__ has joined #openstack-nova | 21:59 | |
| mriedem | i think archive_deleted_rows might be broken | 22:06 |
| dansmith | mriedem: I think I might be volunteering to work on archive and purge this cycle | 22:07 |
| mriedem | there is something weird going on here http://logs.openstack.org/10/538510/7/check/legacy-tempest-dsvm-neutron-nova-next-full/b808474/job-output.txt.gz#_2018-02-09_20_15_11_712319 | 22:07 |
| *** Sukhdev has joined #openstack-nova | 22:07 | |
| mriedem | although maybe it's just my patch | 22:07 |
| mriedem | because this is fine http://logs.openstack.org/63/543263/2/check/legacy-tempest-dsvm-neutron-nova-next-full/1c23f03/logs/devstack-gate-post_test_hook.txt.gz | 22:08 |
| mriedem | i bet it's the set -e | 22:08 |
| dansmith | if you set -e in the post hook script then yeah | 22:09 |
| mriedem | but i'm not sure why it'd be blowing up when archive_deleted_rows runs | 22:09 |
| mriedem | since it doesn't look like that is failing | 22:10 |
| dansmith | it returns 1 until you don't need to run it again right? | 22:10 |
| mriedem | it runs the loop up to 30 times as long as the CLI returns 1 | 22:11 |
| mriedem | until it returns 0 for all done | 22:11 |
| dansmith | right | 22:12 |
| dansmith | but set -e will break that right? | 22:12 |
| *** threestrands has joined #openstack-nova | 22:12 | |
| mriedem | yeah i forgot how strict that was | 22:12 |
| mriedem | need to capture the $? differently | 22:12 |
| mriedem | or, just move the set -e to later | 22:12 |
| dansmith | well, unset -e during that loop or put it somewhere else or run it in a subshell | 22:12 |
| dansmith | yeah | 22:12 |
| mriedem | yeah i had this in a separate file earlier but couldn't figure out why it was failng, now it's pretty obvious | 22:13 |
| *** jmlowe has quit IRC | 22:14 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Check for leaked server resource allocations in post_test_hook https://review.openstack.org/538510 | 22:16 |
| *** pchavva has quit IRC | 22:21 | |
| *** amodi has quit IRC | 22:22 | |
| openstackgerrit | Jackie Truong proposed openstack/nova master: Add trusted_certs to instance_extra https://review.openstack.org/537897 | 22:24 |
| openstackgerrit | Jackie Truong proposed openstack/nova master: Add trusted_certs object https://review.openstack.org/489408 | 22:24 |
| openstackgerrit | Jackie Truong proposed openstack/nova master: Implement certificate_utils https://review.openstack.org/479949 | 22:24 |
| openstackgerrit | Jackie Truong proposed openstack/nova master: Add trusted_image_certificates to REST API https://review.openstack.org/486204 | 22:24 |
| *** felipemonteiro__ has quit IRC | 22:26 | |
| *** felipemonteiro__ has joined #openstack-nova | 22:26 | |
| *** masber has quit IRC | 22:29 | |
| *** slaweq has quit IRC | 22:30 | |
| *** rcernin has joined #openstack-nova | 22:31 | |
| openstackgerrit | Merged openstack/nova-specs master: Add z/VM Driver in Rocky https://review.openstack.org/536186 | 22:34 |
| openstackgerrit | Merged openstack/os-traits stable/pike: Update .gitreview for stable/pike https://review.openstack.org/488667 | 22:35 |
| openstackgerrit | Merged openstack/os-traits stable/pike: Update UPPER_CONSTRAINTS_FILE for stable/pike https://review.openstack.org/488668 | 22:35 |
| *** acormier_ has quit IRC | 22:36 | |
| *** acormier has joined #openstack-nova | 22:36 | |
| *** acormier has quit IRC | 22:40 | |
| *** acormier has joined #openstack-nova | 22:41 | |
| *** tidwellr has quit IRC | 22:46 | |
| *** acormier has quit IRC | 22:46 | |
| *** kaisers_ has quit IRC | 22:55 | |
| *** felipemonteiro_ has joined #openstack-nova | 22:57 | |
| *** felipemonteiro__ has quit IRC | 23:01 | |
| *** slaweq has joined #openstack-nova | 23:02 | |
| claudiub | mriedem: hello. just an fyi. regarding the undeleted placement allocations claims I've had, apparently there's a reliable way to replicate the issue. Basically, I'm doing a nova resize on the same host to an invalid flavor (smaller disk in my case). The original allocations are removed, and in place 2 sets of allocations for the new flavor are created. | 23:04 |
| claudiub | but the 2 sets of allocations are never deleted, even if the instance's task state got reverted and is now in Running, normal state. | 23:04 |
| claudiub | and even if I destroy the instance, only one set of allocations get removed. | 23:05 |
| *** slaweq has quit IRC | 23:07 | |
| *** Sukhdev has quit IRC | 23:20 | |
| *** AlexeyAbashkin has joined #openstack-nova | 23:21 | |
| *** mlavalle has quit IRC | 23:25 | |
| *** AlexeyAbashkin has quit IRC | 23:26 | |
| *** mlavalle has joined #openstack-nova | 23:27 | |
| *** mlavalle has quit IRC | 23:29 | |
| *** abalutoiu__ is now known as abalutoiu | 23:29 | |
| *** abalutoiu has quit IRC | 23:30 | |
| *** abalutoiu has joined #openstack-nova | 23:30 | |
| *** Sukhdev has joined #openstack-nova | 23:32 | |
| *** claudiub has quit IRC | 23:34 | |
| *** Shilpa has quit IRC | 23:40 | |
| *** bhagyashris has quit IRC | 23:40 | |
| *** bhagyashris has joined #openstack-nova | 23:41 | |
| *** Shilpa has joined #openstack-nova | 23:41 | |
| *** cdent has quit IRC | 23:43 | |
| *** acormier has joined #openstack-nova | 23:48 | |
| *** acormier has quit IRC | 23:52 | |
| *** takashin has joined #openstack-nova | 23:53 | |
| *** felipemonteiro_ has quit IRC | 23:53 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!
