Monday, 2018-02-12

*** hongbin has quit IRC00:04
*** patriciadomin has quit IRC00:06
*** egarbade has quit IRC00:08
*** patriciadomin has joined #openstack-nova00:09
*** egarbade has joined #openstack-nova00:09
*** yassine has joined #openstack-nova00:16
*** yassine is now known as Guest449600:16
*** mdrabe has quit IRC00:35
*** mdrabe has joined #openstack-nova00:38
*** zhurong has joined #openstack-nova00:39
*** slaweq has joined #openstack-nova00:42
*** mdrabe has quit IRC00:45
*** slaweq has quit IRC00:47
*** tovin07 has joined #openstack-nova00:48
*** mdrabe has joined #openstack-nova00:48
*** hoangcx has joined #openstack-nova00:50
*** patriciadomin has quit IRC00:51
*** patriciadomin has joined #openstack-nova00:52
*** salv-orlando has joined #openstack-nova00:57
*** zhaochao has joined #openstack-nova00:59
*** jichen has joined #openstack-nova00:59
*** namnh has joined #openstack-nova01:00
*** ekhugen has quit IRC01:00
*** ekhugen has joined #openstack-nova01:01
*** mdbooth has joined #openstack-nova01:02
*** ansiwen has joined #openstack-nova01:02
*** salv-orlando has quit IRC01:03
*** patriciadomin has quit IRC01:03
*** tiendc has joined #openstack-nova01:04
*** patriciadomin has joined #openstack-nova01:04
*** gcb has joined #openstack-nova01:05
*** ekhugen has quit IRC01:06
*** ekhugen has joined #openstack-nova01:06
*** mdrabe has quit IRC01:07
*** egarbade has quit IRC01:08
*** egarbade has joined #openstack-nova01:08
*** patriciadomin has quit IRC01:09
*** jichen_ has joined #openstack-nova01:09
*** patriciadomin has joined #openstack-nova01:09
*** jichen__ has joined #openstack-nova01:10
*** mdrabe has joined #openstack-nova01:10
*** jichen has quit IRC01:10
*** jichen__ is now known as jichen01:10
*** egarbade has quit IRC01:13
*** egarbade- has joined #openstack-nova01:13
*** jichen_ has quit IRC01:14
*** Guest4496 has quit IRC01:15
*** phuongnh has joined #openstack-nova01:16
*** aloga has joined #openstack-nova01:16
*** Guest4496 has joined #openstack-nova01:16
*** takedakn has joined #openstack-nova01:17
*** patriciadomin has quit IRC01:17
*** patriciadomin has joined #openstack-nova01:18
*** gcb has quit IRC01:18
*** egarbade- has quit IRC01:19
*** egarbade has joined #openstack-nova01:20
*** mdrabe has quit IRC01:22
*** yangyapeng has quit IRC01:25
*** yangyapeng has joined #openstack-nova01:26
*** mdrabe has joined #openstack-nova01:26
*** yangyapeng has quit IRC01:31
*** yangyapeng has joined #openstack-nova01:32
*** yangyapeng has quit IRC01:36
*** acormier has joined #openstack-nova01:48
openstackgerritLeopard Ma proposed openstack/nova master: fix paused instance not support shelve  https://review.openstack.org/54166001:52
*** patriciadomin has quit IRC01:56
*** patriciadomin has joined #openstack-nova01:56
*** zhurong has quit IRC01:57
openstackgerritYikun Jiang (Kero) proposed openstack/nova-specs master: Add host info to instance action events  https://review.openstack.org/54327702:01
*** takedakn has quit IRC02:01
*** yangyapeng has joined #openstack-nova02:02
*** acormier has quit IRC02:03
*** mdrabe_ has joined #openstack-nova02:03
*** acormier has joined #openstack-nova02:03
openstackgerritYikun Jiang (Kero) proposed openstack/nova-specs master: Add host info to instance action events  https://review.openstack.org/54327702:04
*** mdrabe has quit IRC02:05
*** yangyapeng has quit IRC02:05
*** yangyapeng has joined #openstack-nova02:06
*** suresh12 has joined #openstack-nova02:06
*** ekhugen has quit IRC02:08
*** ekhugen has joined #openstack-nova02:09
*** Tom-Tom has joined #openstack-nova02:14
*** takedakn has joined #openstack-nova02:19
*** suresh12 has quit IRC02:23
*** hongbin has joined #openstack-nova02:25
*** ratailor has joined #openstack-nova02:30
*** gcb has joined #openstack-nova02:30
*** takedakn has quit IRC02:37
*** ekhugen- has joined #openstack-nova02:41
*** ekhugen has quit IRC02:41
*** Tom-Tom has quit IRC02:42
*** Tom-Tom has joined #openstack-nova02:52
*** Tom-Tom has quit IRC02:55
*** Tom-Tom has joined #openstack-nova03:02
*** Tom-Tom has quit IRC03:04
*** edmondsw has joined #openstack-nova03:07
*** gcb has quit IRC03:07
*** itlinux has quit IRC03:10
*** patriciadomin has quit IRC03:11
*** ratailor has quit IRC03:11
*** edmondsw has quit IRC03:11
*** patriciadomin has joined #openstack-nova03:11
*** gcb has joined #openstack-nova03:20
*** egarbade has quit IRC03:22
*** egarbade has joined #openstack-nova03:23
*** ekhugen- has quit IRC03:23
*** patriciadomin has quit IRC03:24
*** mdrabe_ has quit IRC03:24
*** mdrabe has joined #openstack-nova03:25
*** patriciadomin has joined #openstack-nova03:25
*** acormier_ has joined #openstack-nova03:29
*** ekhugen_alt has joined #openstack-nova03:29
*** mdrabe has quit IRC03:32
*** acormier has quit IRC03:33
*** pooja_jadhav has joined #openstack-nova03:33
*** mdrabe has joined #openstack-nova03:34
*** acormier_ has quit IRC03:42
*** mdrabe has quit IRC03:43
*** links has joined #openstack-nova03:44
*** takedakn has joined #openstack-nova03:47
*** patriciadomin has quit IRC03:48
*** patriciadomin has joined #openstack-nova03:50
*** Guest4496 has quit IRC03:51
*** mdrabe has joined #openstack-nova03:51
*** yamamoto has joined #openstack-nova03:54
*** yikun_jiang has quit IRC03:55
*** yikun_jiang has joined #openstack-nova03:56
*** flwang1 has quit IRC03:59
*** itlinux has joined #openstack-nova04:01
*** ekhugen_alt has quit IRC04:02
*** ekhugen_alt has joined #openstack-nova04:03
*** mdrabe has quit IRC04:05
*** patriciadomin has quit IRC04:05
*** patriciadomin has joined #openstack-nova04:05
*** Guest4496 has joined #openstack-nova04:06
*** mdrabe has joined #openstack-nova04:07
*** dave-mccowan has quit IRC04:10
*** dave-mccowan has joined #openstack-nova04:11
*** udesale has joined #openstack-nova04:13
*** dave-mccowan has quit IRC04:18
*** jmlowe has joined #openstack-nova04:20
*** psachin has joined #openstack-nova04:21
*** jmlowe_ has quit IRC04:22
*** itlinux has quit IRC04:24
*** egarbade has quit IRC04:28
*** bhagyashris has joined #openstack-nova04:29
*** psachin has quit IRC04:30
*** egarbade has joined #openstack-nova04:30
*** abhishekk has joined #openstack-nova04:30
*** itlinux has joined #openstack-nova04:30
*** psachin has joined #openstack-nova04:32
*** mdrabe has quit IRC04:32
*** nicolasbock has quit IRC04:33
*** mdrabe has joined #openstack-nova04:34
*** egarbade has quit IRC04:34
*** egarbade has joined #openstack-nova04:35
*** xinliang has joined #openstack-nova04:36
*** lpetrut has joined #openstack-nova04:39
*** janki has joined #openstack-nova04:41
*** patriciadomin has quit IRC04:42
*** patriciadomin has joined #openstack-nova04:43
*** mdrabe has quit IRC04:44
*** slaweq has joined #openstack-nova04:45
*** slaweq has quit IRC04:49
*** mdrabe has joined #openstack-nova04:50
*** hongbin has quit IRC04:56
*** suresh12 has joined #openstack-nova04:59
*** brad[] has quit IRC05:03
*** brad[] has joined #openstack-nova05:05
*** lpetrut has quit IRC05:07
openstackgerritDinesh Bhor proposed openstack/nova-specs master: Strict isolation of group of hosts for images  https://review.openstack.org/38191205:09
*** ratailor has joined #openstack-nova05:11
*** lpetrut has joined #openstack-nova05:13
*** lpetrut has quit IRC05:15
*** lpetrut has joined #openstack-nova05:15
*** mdrabe_ has joined #openstack-nova05:17
*** mdrabe has quit IRC05:17
*** ekhugen_alt has quit IRC05:18
*** ekhugen- has joined #openstack-nova05:18
*** patriciadomin has quit IRC05:19
*** patriciadomin has joined #openstack-nova05:20
*** mdrabe_ has quit IRC05:22
*** mdrabe has joined #openstack-nova05:22
*** moshele has joined #openstack-nova05:24
*** egarbade has quit IRC05:25
*** ekhugen- has quit IRC05:25
*** egarbade has joined #openstack-nova05:25
*** ekhugen_alt has joined #openstack-nova05:26
*** moshele has quit IRC05:29
*** lpetrut has quit IRC05:31
*** fragatina has joined #openstack-nova05:38
*** acormier has joined #openstack-nova05:43
openstackgerritHuang Rui proposed openstack/nova master: z/VM driver: add snapshot function  https://review.openstack.org/53424005:44
openstackgerritHuang Rui proposed openstack/nova master: add power actions  https://review.openstack.org/54334005:44
openstackgerritHuang Rui proposed openstack/nova master: z/VM Driver: add power actions  https://review.openstack.org/54334005:45
*** kholkina has joined #openstack-nova05:45
*** fragatina has quit IRC05:45
*** patriciadomin has quit IRC05:46
*** acormier has quit IRC05:47
*** patriciadomin has joined #openstack-nova05:47
*** sridharg has joined #openstack-nova05:49
*** egarbade- has joined #openstack-nova05:52
*** egarbade has quit IRC05:52
*** mdrabe has quit IRC05:58
*** yikun_jiang has quit IRC05:59
*** patriciadomin has quit IRC05:59
*** yikun_jiang has joined #openstack-nova05:59
*** mdrabe has joined #openstack-nova06:00
*** patriciadomin has joined #openstack-nova06:00
*** salv-orlando has joined #openstack-nova06:03
*** yamamoto has quit IRC06:05
openstackgerritHuang Rui proposed openstack/nova master: z/VM Driver: add get console output  https://review.openstack.org/54334406:06
openstackgerritRajesh Tailor proposed openstack/nova stable/pike: Allow force-delete even if task_state is not None  https://review.openstack.org/54334506:06
*** links has quit IRC06:07
*** itlinux has quit IRC06:10
*** moshele has joined #openstack-nova06:14
*** yamamoto has joined #openstack-nova06:15
*** ircuser-1 has quit IRC06:32
*** lajoskatona has joined #openstack-nova06:34
openstackgerritAmeed Ashour proposed openstack/osc-placement master: tox.ini settings for global constraints are out of date  https://review.openstack.org/54334806:38
*** suresh12 has quit IRC06:39
*** gcb has quit IRC06:41
openstackgerritDebo Zhang proposed openstack/nova master: Replace Chinese quotes to English quotes  https://review.openstack.org/54334906:46
*** links has joined #openstack-nova06:51
*** gcb has joined #openstack-nova06:56
*** janki has quit IRC06:57
*** janki has joined #openstack-nova06:57
*** moshele has quit IRC06:57
*** moshele has joined #openstack-nova06:59
*** gcb has quit IRC07:02
*** yamamoto has quit IRC07:20
*** stvnoyes has left #openstack-nova07:23
*** sahid has joined #openstack-nova07:27
*** maciejjozefczyk has quit IRC07:28
*** lpetrut has joined #openstack-nova07:29
*** Tom-Tom has joined #openstack-nova07:33
*** slaweq has joined #openstack-nova07:45
*** maciejjozefczyk has joined #openstack-nova07:50
*** AlexeyAbashkin has joined #openstack-nova07:51
*** ragiman has joined #openstack-nova07:52
openstackgerritjichenjc proposed openstack/nova master: WIP: add zvm document into  https://review.openstack.org/53272007:59
openstackgerritjichenjc proposed openstack/nova master: WIP: Add zvm admin doc  https://review.openstack.org/53312507:59
openstackgerritjichenjc proposed openstack/nova master: WIP: Add zvm CI information  https://review.openstack.org/53351207:59
vivsoni_Hi Team,08:02
vivsoni_import uuid08:02
vivsoni_from oslo_serialization import base6408: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 tried08: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-nova08:06
*** pcaruana has joined #openstack-nova08:07
*** ccamacho has joined #openstack-nova08:08
*** slaweq_ has quit IRC08:11
*** ralonsoh has joined #openstack-nova08:17
*** tesseract has joined #openstack-nova08:18
*** yamamoto has joined #openstack-nova08:21
*** salv-orlando has quit IRC08:27
*** yamamoto has quit IRC08:29
*** Tom-Tom has quit IRC08:30
*** Tom-Tom has joined #openstack-nova08:31
*** dtantsur|afk is now known as dtantsur08:33
*** amoralej|off is now known as amoralej08:38
*** rcernin has quit IRC08:39
*** yamahata has quit IRC08:39
ameedavivsoni_: did you try base64.decode_as_text(u'zN7FFUqWSSewXP7L7RpVFQ==') ?08:40
vivsoni_ameeda: yes08:41
vivsoni_it gives UnicodeDecodeError: 'utf8' codec can't decode byte 0xcc in position 0: invalid continuation byte08:41
openstackgerritHuang Rui proposed openstack/nova master: z/VM Driver: Initial change set of z/VM driver  https://review.openstack.org/52338708:41
openstackgerritHuang Rui proposed openstack/nova master: z/VM Driver: Spawn and destroy function of z/VM driver  https://review.openstack.org/52765808:41
openstackgerritHuang Rui proposed openstack/nova master: z/VM Driver: add snapshot function  https://review.openstack.org/53424008:41
openstackgerritHuang Rui proposed openstack/nova master: z/VM Driver: add power actions  https://review.openstack.org/54334008:41
openstackgerritHuang Rui proposed openstack/nova master: z/VM Driver: add get console output  https://review.openstack.org/54334408:41
vivsoni_ameeda: this base64.decode_as_bytes(u'zN7FFUqWSSewXP7L7RpVFQ==')08:42
vivsoni_gives a closer results08:42
*** bauwser is now known as bauzas08:43
bauzasgood morning stackers08:43
ameedamorning :)08:43
ameedavivsoni_: I am not sure, I will check that soon08:43
*** damien_r has joined #openstack-nova08:45
*** jpena|off is now known as jpena08:45
*** flwang1 has joined #openstack-nova08:54
*** lpetrut has quit IRC08:54
*** danpawlik has joined #openstack-nova08:54
vivsoni_ameeda: Thanks :)08:54
*** sshwarts has joined #openstack-nova08:54
*** lpetrut has joined #openstack-nova09:00
*** claudiub has joined #openstack-nova09:02
*** abhishekk is now known as abhishekk|afk09:04
*** abhishekk|afk is now known as abhishekk09:11
*** ttsiouts has joined #openstack-nova09:12
gryfvivsoni_, 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-nova09:16
vivsoni_gryf: this worked09:16
vivsoni_:)09:16
vivsoni_gryf: how you proceed to solve this ?09:18
vivsoni_gryf: please let me know09:19
gryfvivsoni_, I just looked at documentation for uuid object ;)09:19
*** brad[] has quit IRC09:20
*** Shilpa has joined #openstack-nova09:26
*** ispp has quit IRC09:26
*** ispp has joined #openstack-nova09:27
*** salv-orlando has joined #openstack-nova09:28
*** Tom-Tom_ has joined #openstack-nova09:32
*** brad[] has joined #openstack-nova09:32
vivsoni_gryf: Ok... Thanks :)09:32
*** salv-orlando has quit IRC09:33
*** flwang1 has quit IRC09:35
*** Tom-Tom has quit IRC09:36
*** yamamoto has joined #openstack-nova09:43
*** yamamoto has quit IRC09:44
*** Tom-Tom_ has quit IRC09:46
*** Tom-Tom has joined #openstack-nova09:47
*** test222 has quit IRC09:47
*** test222 has joined #openstack-nova09:48
*** phuongnh has quit IRC09:48
*** Tom-Tom has quit IRC09:51
*** jichen has quit IRC10:01
*** alexchadin has quit IRC10:03
*** slaweq_ has joined #openstack-nova10:07
*** namnh has quit IRC10:08
*** tovin07 has quit IRC10:10
*** alexchadin has joined #openstack-nova10:12
*** slaweq_ has quit IRC10:12
*** tiendc has quit IRC10:17
*** derekh has joined #openstack-nova10:17
hrwmorning10:19
*** wxy has quit IRC10:26
*** wxy has joined #openstack-nova10:27
*** Tom-Tom has joined #openstack-nova10:29
*** salv-orlando has joined #openstack-nova10:29
ameedamorning10:31
ameedahello, how can I get python2-mistralclient version ?10:31
openstackgerritChris Dent proposed openstack/nova master: WIP: Use nova.db.api directly  https://review.openstack.org/54326210:32
*** cdent has joined #openstack-nova10:32
*** salv-orlando has quit IRC10:34
*** tssurya has joined #openstack-nova10:34
*** lucas-afk is now known as lucasagomes10:44
*** salv-orlando has joined #openstack-nova10:44
*** yamamoto has joined #openstack-nova10:45
*** elmaciej has joined #openstack-nova10:48
*** yamamoto has quit IRC10:51
*** udesale_ has joined #openstack-nova10:58
*** udesale has quit IRC11:01
*** Guest4496 has quit IRC11:01
*** sambetts|afk is now known as sambetts11:06
cdentbauzas, 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-nova11:06
*** udesale_ has quit IRC11:07
*** ralonsoh has quit IRC11:10
*** jaosorior has joined #openstack-nova11:10
*** abhishekk has quit IRC11:12
cdentbauzas, giblet: this one is also fairly trivial: https://review.openstack.org/#/c/536784/11:15
*** coreycb has quit IRC11:19
openstackgerritChris Dent proposed openstack/nova master: Avoid inventory DELETE API (no conflict detection)  https://review.openstack.org/53971211:20
*** coreycb has joined #openstack-nova11:20
cdentefried: fixed the merge conflict on ^11:20
*** yamamoto has joined #openstack-nova11:20
*** mdnadeem has joined #openstack-nova11:20
*** giblet is now known as gibi11:24
gibicdent: will look at them soon11:25
*** catmando has quit IRC11:25
bauzascdent: ack11:25
cdentgibi: 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-nova11:26
cdentthanks bauzas11:30
*** yamamoto has quit IRC11:32
*** tbachman has quit IRC11:35
*** sdague has joined #openstack-nova11:36
gibicdent: just on more small clarification request in https://review.openstack.org/#/c/526691/2/nova/api/openstack/placement/deploy.py@9711:37
hrwbauzas: can you help get https://review.openstack.org/#/c/541728/ (aarch64 status matrix) merged? mriedem already gave +2 so you can finish merge process :D11:38
*** bhagyashris has quit IRC11:38
*** finucannot is now known as stephenfin11:40
cdentgibi: yeah, good point. will fix11:40
openstackgerritChris Dent proposed openstack/nova master: [placement] annotate loadapp as public interface  https://review.openstack.org/52669111:43
cdentbauzas, gibi: ^ please and thank you11:43
*** alexchadin has quit IRC11:45
*** tbachman has joined #openstack-nova11:47
*** tbachman has quit IRC11:49
*** bhagyashris has joined #openstack-nova11:50
*** nicolasbock has joined #openstack-nova11:53
*** ratailor has quit IRC11:59
*** yamamoto has joined #openstack-nova12:02
*** slaweq_ has joined #openstack-nova12:08
*** dave-mccowan has joined #openstack-nova12:08
*** slaweq_ has quit IRC12:13
*** yangyapeng has quit IRC12:14
gibicdent: thanks, +2d12:26
*** jpena is now known as jpena|lunch12:26
*** acormier has joined #openstack-nova12:27
*** jmlowe has quit IRC12:27
AJaegernovaclient 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 v312:27
*** acormier has quit IRC12:28
*** acormier has joined #openstack-nova12:28
*** yangyapeng has joined #openstack-nova12:30
*** acormier has quit IRC12:35
*** yangyapeng has quit IRC12:35
*** wxy has quit IRC12:36
ameedagibi: could you please check https://review.openstack.org/#/c/526900/, it doesn't has any activity from awhile12:38
*** ralonsoh__ has joined #openstack-nova12:46
*** yangyapeng has joined #openstack-nova12:47
*** ralonsoh_ has quit IRC12:50
*** r-daneel has joined #openstack-nova12:56
*** takedakn has quit IRC13:04
*** takedakn has joined #openstack-nova13:05
*** takedakn has quit IRC13:05
bauzashrw: https://review.openstack.org/#/c/541728/ will be merged in master, which is now for Rocky13:08
*** tbachman has joined #openstack-nova13:08
bauzashrw: let's discuss with mriedem if we should cherry-pick to Queens13:08
*** mdnadeem has quit IRC13:09
*** weshay|ruck is now known as weshay13:10
*** yassine has joined #openstack-nova13:11
*** yassine is now known as Guest5880113:11
*** edmondsw has joined #openstack-nova13:15
*** edmondsw has quit IRC13:15
*** edmondsw_ has joined #openstack-nova13:16
*** jpena|lunch is now known as jpena13:26
*** ccamacho has quit IRC13:31
*** pchavva has joined #openstack-nova13:31
*** mdnadeem has joined #openstack-nova13:36
*** edmondsw_ is now known as edmondsw13:37
*** ispp has quit IRC13:40
hrwbauzas: thanks13:41
hrwbauzas: I think that status-matrix part can be merged13:42
*** acormier has joined #openstack-nova13:45
*** slaweq_ has joined #openstack-nova13:46
*** vladikr has joined #openstack-nova13:47
efriedcdent: Thank you sir.13:49
*** ccamacho has joined #openstack-nova13:49
efriedcdent: Was it because of global_id?13:49
*** acormier has quit IRC13:49
cdentefried: I _think_ so. The method that you had deleted was different from when you had deleted if before, so I just deleted it again13:50
efriedright13:50
*** slaweq_ has quit IRC13:51
*** dtantsur is now known as dtantsur|brb13:52
efriedleakypipes: Yet another +2-after-rebase please: https://review.openstack.org/#/c/539712/13:54
*** nicolasbock has quit IRC13:54
efriedcdent: here's that spec on resource class affinity for NUMA: https://review.openstack.org/54306213:55
*** takashin has joined #openstack-nova13:55
leakypipesefried: done13:56
efriedleakypipes: Thanks13:56
cdentefried: yeah, already noted it because of the spying you suggested i might do13:56
* leakypipes goes back to reading efried's resource class affinity spec13:56
efriedcdent: 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
efriedcdent: eh, spying?13:57
*** acormier has joined #openstack-nova13:57
cdentefried: I haven't digested it yet. glanced over the weekend and was all "ugh, not another table" and decided to come back later13:58
*** liverpooler has joined #openstack-nova13:58
cdentefried: spying: p!spy in https://anticdent.org/purple-irc-bot.html13:58
efriedcdent: Another table - you're talking about agg distance, not rc affinity, right?13:58
cdentefried: right, I haven't looked at rc affinity yet, but its in the queue13:59
efriedcdent: Ah :)13:59
efriedokay, cool.13:59
* cdent fixes typo in that blog post14:00
edleafeScheduler subteam meeting running now in #openstack-meeting-alt14:00
*** jmlowe has joined #openstack-nova14:00
mriedembauzas: i don't think https://review.openstack.org/#/c/541728/ needs to go to queens14:01
*** leakypipes is now known as jaypipes14:01
bauzassure, np14:01
*** acormier has quit IRC14:02
efriedp!spy14:02
*** jmlowe has quit IRC14:05
*** kholkina has quit IRC14:08
*** amoralej is now known as amoralej|lunch14:08
*** ralonsoh__ is now known as ralonsoh14:11
*** rajinir has quit IRC14:12
*** rajinir has joined #openstack-nova14:12
*** ispp has joined #openstack-nova14:13
*** ekhugen- has joined #openstack-nova14:15
*** patriciadomin_ has joined #openstack-nova14:16
*** egarbade has joined #openstack-nova14:16
*** mdrabe_ has joined #openstack-nova14:17
openstackgerritChris Dent proposed openstack/nova master: Move db MAX constants to own file  https://review.openstack.org/54346914:17
*** ekhugen_alt has quit IRC14:17
*** ispp has quit IRC14:18
*** ispp has joined #openstack-nova14:18
*** egarbade- has quit IRC14:18
*** patriciadomin has quit IRC14:19
*** mdrabe has quit IRC14:20
*** udesale has joined #openstack-nova14:20
AJaegermriedem, 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 related14:21
efriedAJaeger: I hear takashin might be a good candidate :)14:21
*** mdrabe_ is now known as mdrabe14:23
*** yamamoto has quit IRC14:24
*** eharney has joined #openstack-nova14:24
*** udesale_ has joined #openstack-nova14:24
*** udesale has quit IRC14:25
AJaegertakashin: ^ please ping me in case of questions. Without the first two changes, the stable branches are broken.14:25
*** psachin has quit IRC14:26
*** pchavva has quit IRC14:27
*** lajoskatona has quit IRC14:29
mriedemAJaeger: i can look after the scheduler meeting14:30
AJaegermriedem: thanks14:30
*** slaweq has quit IRC14:31
*** tssurya has quit IRC14:32
*** slaweq has joined #openstack-nova14:32
takashinAJaeger: I'm not a core reviewer for python-novaclient project...14:33
*** ttsiouts_ has joined #openstack-nova14:34
openstackgerritMerged openstack/nova master: Provide basic data for AArch64 support matrix/functionality  https://review.openstack.org/54172814:36
*** slaweq has quit IRC14:36
*** yamahata has joined #openstack-nova14:37
*** david-lyle has quit IRC14:38
*** jobewan has joined #openstack-nova14:38
*** edmondsw has quit IRC14:38
*** janki has quit IRC14:39
*** lbragstad has joined #openstack-nova14:39
hrwmriedem, bauzas: thanks for helping with getting this stuff merged14:42
*** yamahata has quit IRC14:42
*** pchavva has joined #openstack-nova14:42
*** yamahata has joined #openstack-nova14:42
gibimriedem, 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
gibimriedem, bauzas: In this case nova cannot do anyithing to have the image reference, as there was never an image associated to the volume14:44
*** r-daneel has quit IRC14:45
gibimriedem, bauzas: I don't think we can enforce to have image_id in the request spec14:46
mriedemgibi: that vm is worthless though right?14:46
gibimriedem: yeah, I oversimplified it, but I could attach my-vol to a vm, populate it, detach it, then boot from it14:46
*** tssurya has joined #openstack-nova14:47
mriedemsure, i wasn't saying we should 'require' an id in the requestspec.image field,14:47
mriedemjust that in certain cases (presumably the more normal cases), it's available via the volume_image_metadata14:48
*** lucasagomes is now known as lucas-hungry14:48
mriedemand furthermore, we can use that information later if a cell is down14:48
mriedemand we use the request spec to represent the instance in the GET /servers/{id}/detail response14:48
mriedem*if we use14:48
gibimriedem, bauzas: OK, it is clearer now. I had the feeling that we want to make image_id mandatory in the request spec14:49
mriedemjroll: added the healthcheck middleware to both https://etherpad.openstack.org/p/self-healing-queens-ptg and https://etherpad.openstack.org/p/api-sig-ptg-rocky14:50
mriedemwe probably need a few more SIGs for this14:50
bauzasmriedem: heh for "more SIGs"14:51
bauzasthat can become a very discussed and argued thing14:51
openstackgerritChris Dent proposed openstack/nova master: WIP: Move resource provider objects into placement hierarchy  https://review.openstack.org/54004914:51
openstackgerritChris Dent proposed openstack/nova master: Optional separate database for placement API  https://review.openstack.org/36276614:51
openstackgerritChris Dent proposed openstack/nova master: WIP: Isolate placement database config  https://review.openstack.org/54143514:51
jrollmriedem: what have I done14:51
jrollthis really wasn't something I intended to tilt that hard at14:51
bauzasgibi: mriedem: for the image.id being mandatory, I think I understand the point14:51
*** edmondsw has joined #openstack-nova14:52
*** acormier has joined #openstack-nova14:52
bauzasgibi: mriedem: the problem is that AFAICS we assert in a lot of places about having an instance with an image14:52
mriedemjroll: i just wanted to see it socialized more before it's adopted across a few projects,14:52
mriedembecause i'm projecting it on the community wide goals process14:52
bauzasso we can 'fix' that, but that won't solve the design issue that we accepted in the past a different paradigm14:52
jrollmriedem: also, I wasn't aware of the "new" thing until today, so adding the new thing you linked wasn't my intention at all14:52
mriedemjroll: so your ironic patch and graham's spec aren't the same thing?14:53
jrollmriedem: the new thing sounds like it'll take a year just to get through committee and have code written14:53
jrollcorrect14:53
bauzasjroll: from a very practical standpoint, do you feel blocked ?14:53
jrollmine is the "old" one14:53
mriedembauzas: jim just wants to upstream it, but meh14:53
mriedemit sounds like14:53
jrollbauzas: no, what matt said14:53
mriedemit's middleware14:53
bauzasjroll: because I don't really see *why*it needs to be something14:53
jrollbauzas: because it's what we use for every paste-enabled service, and so trying to standardize the rest there14:54
* jroll isn't terribly opinionated on this being the thing that is used14:54
bauzasjroll: well, you wanna standardize your own monitoring model :)14:54
jrollbauzas: s/your own/oslo's/14:55
gibibauzas: 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 today14:55
cdentjroll: I'd say go ahead with the old thing. the new thing needs to be discussed but is a whole different kettle14:55
jrollI'm not forcing it on anyone ¯\_(ツ)_/¯14:55
bauzasI'm not sure operators are sold on using middleware healthchecks for knowing whether my app is up14:55
cdentjroll: I reckon the way it is done in ironic is just fine for placement14:55
bauzasgibi: If I'm sure about one thing, it's how operators can find solutions with things we don't officially support14:56
mriedemgibi: i wasn't suggesting making that break now,14:56
cdentit doesn't preclude some different future14:56
mriedemgibi: was just wondering if we have the information from the volume meta, if we should store it in the request spec14:56
bauzasmriedem: at the request_spec creation time, then ?14:56
mriedemyes14:56
bauzaswell14:56
mriedemthe confusion will be,14:56
*** amodi has joined #openstack-nova14:56
bauzasif you rebuild ?14:57
mriedeminstance.image_ref == None == volume-backed,14:57
gibicopying image_id over from the volume meta to the request_spec is fine by me14:57
bauzasalso14:57
*** amoralej|lunch is now known as amoralej14:57
mriedembut requestspec.image.id == None....?14:57
*** yamamoto has joined #openstack-nova14:57
mriedembauzas: if you rebuild a volume-backed instance with a new image, we fail today14:57
jrollcdent: thanks14:58
*** simondodsley has quit IRC14:58
mriedembauzas: https://github.com/openstack/nova/blob/master/nova/compute/api.py#L297914:58
*** simondodsley has joined #openstack-nova14:58
bauzasmriedem: okay, gtk14:59
*** esberglu has joined #openstack-nova14:59
*** mchlumsky has joined #openstack-nova15:01
takashinmriedem: Would you check my reply to you comment in https://review.openstack.org/#/c/513721/ ?15:01
mriedemtakashin: done15:04
takashinmriedem: Thanks.15:05
*** acormier has quit IRC15:05
*** acormier has joined #openstack-nova15:05
*** damien_r has quit IRC15:06
*** takashin has left #openstack-nova15:06
*** tbachman has quit IRC15:07
mriedembauzas: https://bugs.launchpad.net/nova/+bug/174885815:08
openstackLaunchpad 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
mriedemsounds familiar15:09
bauzasmriedem: sec, thinking hard of https://review.openstack.org/#/c/543263/2/nova/scheduler/filters/isolated_hosts_filter.py15:09
bauzasthe change isn't trivial15:09
*** auggy has quit IRC15:09
mriedembauzas: really?15:10
*** auggy has joined #openstack-nova15:10
mriedembauzas: that's basically what it did before it was regressed here https://review.openstack.org/#/c/202678/35/nova/scheduler/filters/isolated_hosts_filter.py15:10
bauzasmriedem: yeah, my guts feel some problem that I still don't see15:10
mriedemmaybe you need to go to the bathroom15:10
bauzasprobably15:11
*** tssurya has quit IRC15:11
bauzasbut I'm looking at https://review.openstack.org/#/c/543263/2/nova/scheduler/filters/isolated_hosts_filter.py@4315:11
gibibauzas: shall we pull the patch from the gate as I +Wd it recently15:11
mriedemif you look at how it worked before https://review.openstack.org/#/c/202678/35/nova/scheduler/filters/isolated_hosts_filter.py15:11
bauzasgibi: I'm not saying we *have* a problem15:11
mriedemthe image id was always something it handled as optional15:11
mriedemusing dict.get()15:11
bauzasthe thing is15:11
mriedemchanging to request spec in that patch made it required15:12
bauzasif we say we don't have an image15:12
bauzasthen the image isn't isolated, right15:12
mriedembauzas: that's the point of my email,15:12
bauzasin that case, we should accept host115:12
mriedemwhich is, this never did anything for volume-backed instances,15:12
mriedembut that's not the point of my fix right now15:12
mriedemi'm restoring this filter to how it previously worked, before mitaka15:12
bauzasthen the doc is invalid15:13
bauzasat least the docstring I mean15:13
bauzasand I'd like to fix that15:13
mriedemso clean that up in a follow up?15:13
bauzaswell now that gibi pulled the trigger, /me shrufs15:13
cdentIs 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_args15:13
bauzasshrugs*15:13
bauzasmriedem: the thing is, I thought you wanna backport ?15:14
bauzasmoar fun then15:14
mriedembauzas: i do15:14
bauzasso, we should backport both your change *and* the doc change15:14
mriedemi don't think the docstring is something that bars us from backporting this fix15:14
bauzaslooks sad15:14
bauzasI know15:14
*** tbachman has joined #openstack-nova15:14
mriedembauzas: 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
bauzasI just would have appreciated if we could have amended the upstream filters documentation *and* the docstring in the sme change15:15
mriedemi 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 not15:15
mriedemsince the filter predates my involvement in nova15:15
bauzasmriedem: I don't really care of the BFV case15:15
bauzasthe 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
bauzasmy guts feel yues15:16
bauzashence my trouble15:16
mriedemfwiw essex https://github.com/openstack/nova/commit/972fc7d80fae386cf4c72b7891f6601d6f7fb00b#diff-b4fa3e44aeb7757f04d296529b8492e515:16
*** sahid has quit IRC15:16
mriedembauzas: well, that's the point of my ML thread15:16
*** moshele has quit IRC15:17
bauzasagain, I know15:17
mriedemand why i also sent it to the ops ML15:17
bauzasbut people pulled the trigger before I had time to both review and reply15:17
bauzasthat's fine, it's my bad15:17
mriedemi 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
bauzasI need to be quicker15:17
gibiI 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 BFV15:19
bauzasmriedem: like I said in the meeting, if people use that filter, I'd be surprised15:19
*** tbachman has quit IRC15:19
bauzasgibi: after thinking hard, I think we can leave the patch merge15:19
*** lucas-hungry is now known as lucasagomes15:19
bauzasmriedem: I'd be surprised because it's conf-driven15:19
bauzaswhich operators hate15:19
edleafecdent: you'd need to import rpc for line 44 :)15:20
bauzaswhile ImageAggragateExtraSpecsMyStuffFilter does that programatically with aggregates :)15:20
cdentedleafe: would isolate that too15:20
cdentedleafe: but just going to write another one, as there's redundancy happening in that stuff that placement doesn't care about15:20
cdentedleafe: still curious about the general question (even though I'm not going to do it)15:21
cdentsome people hate them15:21
edleafecdent: yeah, just funnin' ya15:21
edleafeFWIW, I tend to not use them, except when performance is affected.15:22
*** ccamacho has quit IRC15:22
*** sergek_ has quit IRC15:22
*** sergek_ has joined #openstack-nova15:23
mriedemmelwitt: looks like functional test failures in https://review.openstack.org/#/c/340614/ are real15:23
*** belmoreira has joined #openstack-nova15:23
*** jmlowe has joined #openstack-nova15:23
* stephenfin is having _great_ fun with OVS-DPDK today :/15:24
* alex_xu gets into holiday mode, won't be active in two weeks15:25
cdentstephenfin: wait, DPDK isn't solving _all_ your problems. unpossible.15:26
mriedemalex_xu: have fun, eat lots of dumplings15:26
bauzasstephenfin: DPDK like Don't Play Donkey Kong ?15:26
alex_xumriedem: thanks :)15:27
*** tssurya has joined #openstack-nova15:27
bauzasmriedem: what does the follow-up button on the Gerrit UI if I'm clicking it on your top change ?15:27
mriedembauzas: no idea15:27
bauzasmriedem: I'd like to just amend the doc to explain about BFV instances15:27
mriedemhttps://stackoverflow.com/questions/45621912/what-is-the-gerrit-follow-up-button-doing15:28
stephenfincdent: It's not even solving one of them yet. I probably have it configured incorrectly...again15:28
*** jmlowe has quit IRC15:28
bauzasmriedem: 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 excluded15:28
*** sahid has joined #openstack-nova15:28
stephenfinbauzas: Don't be dissing the Kong now, ya hear ;)15:28
mriedembauzas: well, it will depending on the other config options15:29
bauzasmriedem: exactly15:29
bauzasmriedem: but consider the case where I have an image that is restricted to host115:29
mriedemi'd say, if you care, use AggregateImagePropertiesIsolation15:29
mriedemactually,15:30
mriedemAggregateImagePropertiesIsolation likely doesn't work for volume-backed instances properly either15:30
bauzasmriedem: 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=True15:30
*** jmlowe has joined #openstack-nova15:30
mriedemfor 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#L103315:31
mriedemand i don't know if that actually stores the properties from the backing image, and even if it does, they could be stale15:31
*** fmccrthy has quit IRC15:31
* bauzas is despressed15:32
*** fmccrthy has joined #openstack-nova15:32
mriedemso yeah, any filter that relies on image properties is likely semi-broken with boot from volume15:32
openstackgerritSylvain Bauza proposed openstack/nova master: for doc clarity  https://review.openstack.org/54349415:32
openstackgerritChris Dent proposed openstack/nova master: Isolate config parse_args for placement  https://review.openstack.org/54349515:33
*** mlavalle has joined #openstack-nova15:34
AJaegermriedem, 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-nova15:37
openstackgerritSylvain Bauza proposed openstack/nova master: doc: BFV instances and IsolatedHostsFilter  https://review.openstack.org/54349415:37
bauzasmriedem: just made a first try for doc ^15:38
*** tidwellr has joined #openstack-nova15:38
*** felipemonteiro__ has joined #openstack-nova15:38
*** lpetrut has quit IRC15:39
*** links has quit IRC15:39
*** READ10 has joined #openstack-nova15:40
AJaegerthanks, mriedem15:40
bauzasstephenfin: 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-L9315:41
bauzascontext is https://bugs.launchpad.net/nova/+bug/174885815:41
openstackLaunchpad 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
bauzasstephenfin: sahid: the NUMATopologyFilter should only care in my opinion to enforce the numa limits, not any other resource usage15:42
stephenfinbauzas: 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 more15:42
*** felipemonteiro_ has quit IRC15:42
* bauzas looking at https://github.com/openstack/nova/commit/0b0f1515acd43ec3b39f2f748161890c80a8a93915:43
*** david-lyle has joined #openstack-nova15:46
*** slaweq has joined #openstack-nova15:47
*** tbachman has joined #openstack-nova15:48
openstackgerritMatt Riedemann proposed openstack/nova master: Fix typo in NUMATopologyFilter docs  https://review.openstack.org/54350515:49
*** zhaochao has quit IRC15:50
*** slaweq has quit IRC15:51
*** udesale has joined #openstack-nova15:53
*** udesale_ has quit IRC15:53
*** tidwellr has quit IRC15:54
mriedembauzas: fwiw, the image properties are stored on the volume's volume_image_metadata dict: http://paste.openstack.org/show/670139/15:56
mriedemi add foo=bar to the image i used to create that bootable volume15:56
mriedemso AggregateImagePropertiesIsolation would only be as good as how accurate that metadata is15:56
bauzasmriedem: are you persuing me to look up the volume if BFV ?15:56
mriedemsince nova doesn't re-fetch the image to get the latest meta15:57
mriedempersuing you how?15:57
bauzasthe fact is that filters don't lookup the volume image metadata, right?15:57
bauzasthey just assume every instance has an image15:57
bauzaswell, they rather assume every RequestSpec has an image15:58
mriedemthe filter assumes the request spec image has the details from when the api created the request spec15:58
mriedemyes that15:58
*** sahid has quit IRC15:58
mriedemand the request spec for a volume-backed instance will have the image meta stored in the volume from the time the volume was created15:58
*** sahid has joined #openstack-nova15:58
bauzasmriedem: if we create a fake image based on that, right?15:59
mriedembut 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 stale15:59
bauzasI agre15:59
bauzasI got your point15:59
bauzasif 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 IRC15:59
bauzashence the "are you persuing me to lookup the volume"16:00
*** wolsen has joined #openstack-nova16:00
mriedemno i'm not suggesting that,16:00
mriedemi 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 glance16:01
mriedemrather than the possibly stale image meta from the volume16:01
mriedembut, i'm not suggesting we do that either16:01
mriedemi guess we should just assume that the image meta with the volume is like a snapshot16:01
mriedemif the image meta is changed after the volume is created, meh16:01
mriedemif it was changed in such a substantial way that it would impact nova scheduling for the instance, then it should probably be a different image16:02
bauzasthat looks like the best, indeed16:03
*** pcaruana has quit IRC16:04
*** slaweq has joined #openstack-nova16:06
*** Swanson has quit IRC16:07
bauzasmriedem: https://bugs.launchpad.net/nova/+bug/174885816:07
openstackLaunchpad 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-nova16:07
bauzasmriedem: we can clarify the doc, but I don't feel we should open a can of worms about aggregate ratios16:07
*** itlinux has joined #openstack-nova16:08
*** krtaylor has quit IRC16:10
*** itlinux has quit IRC16:10
openstackgerritEric Berglund proposed openstack/nova master: WIP: PowerVM Driver: Snapshot  https://review.openstack.org/54302316:10
*** moshele has joined #openstack-nova16:12
mriedembauzas: i don't either16:12
openstackgerritMerged openstack/nova-specs master: Fix the names on some specs  https://review.openstack.org/54250416:13
*** suresh12 has joined #openstack-nova16:13
openstackgerritMerged openstack/nova-specs master: Move queens implemented specs  https://review.openstack.org/54250516:15
*** damien_r has joined #openstack-nova16:16
*** suresh12 has quit IRC16:17
openstackgerritEric Berglund proposed openstack/nova master: Use dict.get() when accessing capabilities dict  https://review.openstack.org/53971516:18
*** tbachman has quit IRC16:20
*** itlinux has joined #openstack-nova16:22
*** itlinux has quit IRC16:23
*** dtantsur|brb is now known as dtantsur16:24
*** tbachman has joined #openstack-nova16:25
*** r-daneel has joined #openstack-nova16:32
*** felipemonteiro__ has quit IRC16:32
*** felipemonteiro_ has joined #openstack-nova16:32
*** damien_r has quit IRC16:33
*** jamespage has quit IRC16:34
*** damien_r has joined #openstack-nova16:34
*** jamespage has joined #openstack-nova16:35
*** itlinux has joined #openstack-nova16:35
*** imacdonn has joined #openstack-nova16:37
*** ragiman has quit IRC16:37
openstackgerritBence Romsics proposed openstack/osc-placement master: PUT dict format to /allocations (v1.11, v1.12)  https://review.openstack.org/54281916:38
openstackgerritsahid proposed openstack/nova master: libvirt: disconnect volume from host during detach  https://review.openstack.org/51500816:40
*** suresh12 has joined #openstack-nova16:41
*** suresh12 has quit IRC16:41
*** suresh12 has joined #openstack-nova16:41
*** moshele has quit IRC16:43
*** felipemonteiro_ has quit IRC16:45
openstackgerritsahid proposed openstack/nova master: hardware: only take into account small pages  https://review.openstack.org/53216816:46
*** links has joined #openstack-nova16:47
*** damien_r has quit IRC16:47
*** elmaciej has quit IRC16:47
*** lpetrut has joined #openstack-nova16:48
*** damien_r has joined #openstack-nova16:48
*** moshele has joined #openstack-nova16:51
*** hongbin has joined #openstack-nova16:51
*** acormier has quit IRC16:52
*** acormier has joined #openstack-nova16:52
*** links has quit IRC16:53
*** links has joined #openstack-nova16:54
gibimriedem: 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 IRC16:55
mriedemgibi: works for me16:56
gibimriedem: OK, cool16:56
*** damien_r has quit IRC16:58
*** mgoddard_ has joined #openstack-nova17:00
*** damien_r has joined #openstack-nova17:01
*** mdnadeem has quit IRC17:02
mgariepyis 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
mgariepyreferring to : https://review.openstack.org/#/c/45975317:05
openstackgerritMatt Riedemann proposed openstack/nova master: Move the nova-next job in-tree and update it  https://review.openstack.org/54147417:06
openstackgerritMatt Riedemann proposed openstack/nova master: Test websocketproxy with TLS in the nova-next job  https://review.openstack.org/51316017:06
mriedemmgariepy: 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 specs17:06
mgariepythanks 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
mriedemprobably not a lot of work, no, but generally i think we prefer image meta since those are standardized and flavor extra specs are not17:08
*** fragatina has joined #openstack-nova17:08
mriedemi guess the flavor extra spec could maybe make sense in this case if you're using a GPU trait on the flavor or something17:09
*** sahid has quit IRC17:09
*** fragatin_ has joined #openstack-nova17:09
mriedemhttps://docs.openstack.org/nova/latest/admin/virtual-gpu.html#configure-a-flavor-controller17:09
*** suresh12 has quit IRC17:09
mriedemthis is getting dangerously into bauzas territory though17:10
*** suresh12 has joined #openstack-nova17:10
mgariepyi 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
mgariepybut as long as it works i guess i'll use the workaround.17:12
*** felipemonteiro_ has joined #openstack-nova17:13
openstackgerritLee Yarwood proposed openstack/nova stable/pike: Do not attempt volume swap when guest is stopped/suspended  https://review.openstack.org/53373317:13
*** fragatina has quit IRC17:13
*** mdnadeem has joined #openstack-nova17:15
mriedemmgariepy: ok, i guess it's unclear to me if that's only an issue for windows guests as the commit message implies17:15
mgariepyi need that for linux guest as well.17:15
mriedemin that case it makes sense to me to be tied to the image17:15
mriedemoh17:15
mgariepynvidia driver don't like kvm guest.17:15
*** AJaeger has left #openstack-nova17:16
mriedemi'm just amazed the thing is documented https://docs.openstack.org/python-glanceclient/latest/cli/property-keys.html17:16
jaypipesefried: where is that Numbered Request Group Affinity spec?17:16
*** udesale has quit IRC17:17
efriedjaypipes: It's not a spec... yet.  It's a subsection under Alternatives.17:17
efriedjaypipes: 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 IRC17:17
*** belmoreira has quit IRC17:19
*** ttsiouts_ has quit IRC17:21
*** mvk_ has quit IRC17:22
*** ircuser-1 has joined #openstack-nova17:24
*** acormier_ has joined #openstack-nova17:27
*** acormier has quit IRC17:27
*** damien_r has quit IRC17:28
*** felipemonteiro_ has quit IRC17:28
*** suresh12 has quit IRC17:29
*** damien_r has joined #openstack-nova17:31
*** moshele has joined #openstack-nova17:32
*** damien_r has quit IRC17:34
*** damien_r has joined #openstack-nova17:34
*** mdnadeem has quit IRC17:35
*** suresh12_ has joined #openstack-nova17:35
*** moshele has quit IRC17:35
*** damien_r has quit IRC17:36
*** AlexeyAbashkin has quit IRC17:36
*** sridharg has quit IRC17:40
*** damien_r has joined #openstack-nova17:43
*** bkopilov has quit IRC17:43
*** yamamoto has quit IRC17:49
*** damien_r has quit IRC17:49
*** damien_r has joined #openstack-nova17:53
*** yamamoto has joined #openstack-nova17:54
*** david-lyle has quit IRC17:55
*** damien_r has quit IRC17:55
openstackgerritMerged openstack/nova-specs master: Granular Resource Request Syntax (Rocky)  https://review.openstack.org/54017917:57
edleafejaypipes: 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-nova17:58
edleafejaypipes: you mentioned some HA use cases that I'm not familiar with17:58
*** mgoddard_ has quit IRC17:58
*** vivsoni__ has joined #openstack-nova17:59
*** yamamoto has quit IRC18:00
*** derekh has quit IRC18:00
*** damien_r has quit IRC18:00
*** Eran_Kuris_ has joined #openstack-nova18:02
*** ralonsoh has quit IRC18:07
*** dklyle has joined #openstack-nova18:07
*** Eran_Kuris_ has quit IRC18:07
openstackgerritMerged openstack/nova master: Add regression test for BFV+IsolatedHostsFilter failure  https://review.openstack.org/54325718:08
*** Tom-Tom has quit IRC18:08
openstackgerritMerged openstack/nova master: Handle volume-backed instances in IsolatedHostsFilter  https://review.openstack.org/54326318:08
openstackgerritMerged openstack/nova master: Fix docs for IsolatedHostsFilter  https://review.openstack.org/54326418:08
*** Tom-Tom has joined #openstack-nova18:09
*** yamamoto has joined #openstack-nova18:11
*** yamamoto has quit IRC18:11
jaypipesedleafe: 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 IRC18:13
*** harlowja has joined #openstack-nova18:14
edleafejaypipes: 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 they18:14
openstackgerritLee Yarwood proposed openstack/nova stable/pike: libvirt: Block swap_volume attempts with encrypted volumes  https://review.openstack.org/54356918:15
jaypipesedleafe: yeah. the whole "soft affinity" is what those use cases are about.18:17
jaypipesedleafe: in "soft affinity", it's a preference vs. yes/no18:17
*** jpena is now known as jpena|off18:18
jaypipesedleafe: 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-nova18:18
jaypipesedleafe: in the soft affinity cases, the distance being relative and not yes/no enables that sorting of potential destinations18:18
edleafejaypipes: how widespread is that need? IOW, 20% of use cases? 1%?18:19
jaypipesedleafe: the soft affinity thing?18:20
edleafejaypipes: yeah. It seems like overkill to build all our affinity/anti stuff to handle these types of things18:20
jaypipesedleafe: 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
edleafejaypipes: when the vast majority just want yes/no18:21
jaypipesedleafe: it may indeed be overkill. but it's what we currently have.18:21
edleafejaypipes: where do we have it?18:21
jaypipesedleafe: the soft affinity "filters" are implemented as weighers.18:22
openstackgerritMatthew Edmonds proposed openstack/nova master: improve support matrix notes  https://review.openstack.org/54053418:22
jaypipesedleafe: https://github.com/openstack/nova/blob/master/nova/scheduler/weights/affinity.py18:22
*** tbachman has quit IRC18:22
*** lpetrut has quit IRC18:23
*** lpetrut has joined #openstack-nova18:23
edleafejaypipes: that doesn't use any sort of "distance" concept. It just favors those things that meet (anti)affinity18:24
jaypipesedleafe: correct.18:24
jaypipesedleafe: 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
jaypipesedleafe: this is partly to address that issue: https://review.openstack.org/#/c/529135/18:26
*** lpetrut has quit IRC18:27
*** lpetrut has joined #openstack-nova18:27
edleafejaypipes: yeah, I like that18:27
*** r-daneel has quit IRC18:27
openstackgerritEric Berglund proposed openstack/nova master: Use correct arguments in task inits  https://review.openstack.org/54357118:28
*** lpetrut has quit IRC18:28
*** lpetrut_ has joined #openstack-nova18:28
*** lpetrut has joined #openstack-nova18:28
jaypipesedleafe: of course, that doesn't touch placement...18:28
edleafejaypipes: understood18:29
*** lpetrut has quit IRC18:29
*** lpetrut_ has quit IRC18:29
*** lpetrut has joined #openstack-nova18:29
*** dklyle has quit IRC18:30
edleafejaypipes: 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 aggs18:30
*** lpetrut has quit IRC18:30
*** lpetrut has joined #openstack-nova18:30
openstackgerritLee Yarwood proposed openstack/nova master: Enable test_volume_backed_live_migration in tempest  https://review.openstack.org/52810418:31
mriedemedleafe: we don't really know what the vast majority of anything is18:31
mriedemusage wise18:31
openstackgerritMerged openstack/python-novaclient stable/pike: Avoid tox_install.sh for constraints support  https://review.openstack.org/54311518:31
edleafemriedem: and what we do know may change18:32
mriedemi want a minimum of 3 DB servers, put on different hosts if you can, but don't fail if you can't18:32
openstackgerritMerged openstack/python-novaclient stable/pike: Zuul: Remove project name  https://review.openstack.org/54137018:32
edleafes/may/will18:32
openstackgerritMerged openstack/python-novaclient stable/ocata: Avoid tox_install.sh for constraints support  https://review.openstack.org/54311418:32
mriedemwe're going to be proposing a spec for something that builds on the soft affinity stuff18:32
mriedemto put a limit on the softness18:32
edleafemriedem: sure. The complicating part is I want three servers, at least two racks away from each other18:32
edleafenot just on different racks18:33
mriedemyeah the distance concept is complicated18:33
cdentwhat does "two rack away" even mean?18:35
*** lpetrut has quit IRC18:35
*** r-daneel has joined #openstack-nova18:35
*** tidwellr has quit IRC18:35
*** lpetrut has joined #openstack-nova18:35
*** sshwarts has quit IRC18:36
edleafecdent: it would depend on the operator mapping out their DC, and assigning values to each "location"18:36
dansmithcdent: separated by power and TOR switch18:36
*** ttsiouts_ has joined #openstack-nova18:36
dansmithcdent: so two adjacent racks might share a power domain, but not TOR switch,18:37
cdentdansmith: 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
dansmithso two away would mean not adjacent at the next level up in separation18:37
cdentso "2" is something the operator is defining?18:37
dansmithcdent: it's a simpler way to accomplish the same thing18:37
dansmithyeah18:37
*** tidwellr has joined #openstack-nova18:38
dansmithI think we said we'd try to work up some guidelines18:38
dansmiththat they could follow to try to have some sort of a uniform idea about said things,18:38
edleafedansmith: couldn't it be simplified to "not in the same power domain"?18:38
dansmithbut at the end of the day, it'll come down to your layout18:38
*** suresh12_ has quit IRC18:39
dansmithedleafe: 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 thngs18:39
*** lpetrut has quit IRC18:39
edleafedansmith: I'm not following. Are those domains nested?18:40
dansmithI 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 map18:40
dansmithedleafe: in my example yeah18:40
*** lpetrut has joined #openstack-nova18:40
edleafedansmith: ok, so that could be simplified to "not in the outermost grouping I care about", right?18:41
openstackgerritMark Goddard proposed openstack/nova master: Don't JSON encode instance_info.traits for ironic  https://review.openstack.org/54357418:41
dansmithedleafe: well, that's not the outermost group18:42
*** tidwellr has quit IRC18:42
dansmithbecause you'd still have potentially room, building, site, region, etc18:42
edleafedansmith: of course18:42
*** suresh12 has joined #openstack-nova18:42
dansmithwe could define the way people have to nest things (i.e. TOR is always smaller scope than power) but that won't fly of course18:43
edmondswmriedem I just added the queens-rc-potential tag to https://bugs.launchpad.net/nova/+bug/174895018:43
openstackLaunchpad 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
edleafedansmith: you would define these groupings by what you care about as far as affinity is concerned18:43
mriedemedmondsw: how did this not fail CI?18:43
dansmithyeah, we could totally add a way to define the groupings, and a complex way to say which things you do and don't care about18:43
dansmithall I'm saying is that distance as a simple metric  to abstract that is pretty straightforward18:43
edleafedansmith: or we could add a simple way to say that18:44
edmondswmriedem it only happens if you have non-ascii characters in the instance18:44
mriedemedmondsw: oh i see, unicode18:44
edmondswmriedem it actually did cause a log error in CI which is how we noticed it, on update_server_name18:44
edmondswbut didn't fail the test18:44
*** jmlowe has quit IRC18:44
edmondswwas just on cleanup18:44
mriedemis there no way to have a test for this fix?18:45
*** links has quit IRC18:45
edmondswmriedem yeah, good point... I'd asked esberglu to look at test but then had a brain fart when reviewing I guess18:46
mriedempew18:47
*** suresh12 has quit IRC18:47
openstackgerritMatt Riedemann proposed openstack/nova-specs master: Re-propose use Neutron's new port binding API  https://review.openstack.org/54357818:48
*** itlinux has quit IRC18:49
mriedemefried: want to re-propose this for rocky? https://review.openstack.org/#/c/505209/18:50
*** itlinux has joined #openstack-nova18:51
openstackgerritDan Smith proposed openstack/nova master: Drop compute RPC 4.x compatibility  https://review.openstack.org/54358018:51
jaypipesdansmith, mriedem: https://etherpad.openstack.org/p/going-the-distance has ideas about the distance concept...18:52
mriedemnice pad name18:52
*** itlinux has quit IRC18:53
mriedemdansmith: you want to -W that rpc change until we release so someone doesn't accidentally merge it?18:53
dansmithmriedem: rc1 is tagged so we're good no?18:53
*** david-lyle has joined #openstack-nova18:54
mriedemi don't know if we'll have something that requires a backport before GA18:54
dansmithI'm going to argue against that really hard,18:54
openstackgerritMerged openstack/nova master: doc: mention that --on-shared-storage is not needed with nova evacuate  https://review.openstack.org/54251218:54
dansmithbut also, it won't invalidate this18:54
mriedemgranted backporting an rpc change for rc2 would be..18:54
dansmithbut no crazy rush so that's fine18:54
openstackgerritMerged openstack/nova master: Update reno for stable/queens  https://review.openstack.org/54265818:55
*** itlinux has joined #openstack-nova18:55
esbergluedmondsw: What's the best way to test this? I can assert that the __init__ methods are being passed the right args18:59
edmondswesberglu autospec?18:59
*** ttsiouts_ has quit IRC18:59
*** lucasagomes is now known as lucas-pto19:03
*** ttsiouts_ has joined #openstack-nova19:03
openstackgerritMatt Riedemann proposed openstack/nova-specs master: Add z/VM Driver in Rocky  https://review.openstack.org/53618619:04
*** jmlowe has joined #openstack-nova19:04
*** jmlowe has quit IRC19:06
openstackgerritMerged openstack/nova-specs master: Modify grammatical errors  https://review.openstack.org/54176519:06
*** jmlowe has joined #openstack-nova19:06
*** jmlowe has quit IRC19:08
*** jmlowe has joined #openstack-nova19:09
*** tidwellr has joined #openstack-nova19:11
*** yamamoto has joined #openstack-nova19:12
*** gyee has joined #openstack-nova19:12
*** sambetts is now known as sambetts|afk19:13
*** eharney has quit IRC19:13
*** jmlowe has quit IRC19:15
*** moshele has joined #openstack-nova19:15
mriedemmelwitt: need to re-propose https://review.openstack.org/#/c/500975/ for rocky19:15
*** suresh12 has joined #openstack-nova19:16
*** tbachman has joined #openstack-nova19:17
*** tbachman has quit IRC19:18
*** tesseract has quit IRC19:18
openstackgerritMark Goddard proposed openstack/nova master: Unplug all VIFs from ironic nodes during tear down  https://review.openstack.org/53762619:19
*** yamamoto has quit IRC19:19
*** tbachman has joined #openstack-nova19:21
*** suresh12 has quit IRC19:21
*** jmlowe has joined #openstack-nova19:22
*** felipemonteiro has joined #openstack-nova19:23
mriedemjianghuaw: 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-improvement19:25
mgoddardmriedem: 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 driver19:26
openstackLaunchpad 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
mgoddardmriedem: fix is this: https://review.openstack.org/#/c/543574/119:26
*** amoralej is now known as amoralej|off19:27
*** vivsoni__ has quit IRC19:27
mriedemmgoddard: so traits-based scheduling with ironic+nova doesn't work without this fix?19:27
*** yamahata has quit IRC19:28
mriedemor what is instance_info.traits used for?19:28
mriedemi thought that was something to do with the capabilities filter in nova19:28
*** suresh12 has joined #openstack-nova19:28
mriedemwhich we're superseding with placement + traits19:28
mgoddardmriedem: 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 things19:29
mriedemmgoddard: if they aren't used in ironic in queens, then it's not something worth holding up rc2 IMO19:29
mgoddardor rather, they are pushed incorrectly19:29
mriedemi assume you're asking about this for rc219:30
mriedemrc2 changes at this point == stop ship19:30
mriedemonce we release queens 17.0.0 we can backport regular old bug fixes to stable/queens as usual19:30
*** moshele has quit IRC19:30
mgoddardmriedem: I suspect you're right - it isn't sufficiently important to block rc219:31
mriedemdoes ironic have a tempest plugin?19:31
mriedemooo look it does https://github.com/openstack/ironic-tempest-plugin19:31
mgoddardmriedem: it does19:31
mriedemcan 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 IRC19:33
mgoddardmriedem: that sounds sensible19:33
mgoddardmriedem: I'll look into it19:34
mriedemthanks19:34
*** Tom-Tom has joined #openstack-nova19:34
*** Tom-Tom has quit IRC19:34
mriedemthis amount of coupling between projects means we really need some higher level integration testing19:34
mriedemthan unit tests19:34
mgoddardthere are tempest tests covering the ironic traits API, but you're right in needing to test the nova-ironic interface19:36
*** jmlowe has quit IRC19:38
*** felipemonteiro has quit IRC19:39
*** felipemonteiro_ has joined #openstack-nova19:39
*** suresh12 has joined #openstack-nova19:39
*** jmlowe has joined #openstack-nova19:40
*** lbragstad_ has joined #openstack-nova19:41
*** moshele has joined #openstack-nova19:43
*** lbragstad_ has quit IRC19:43
*** suresh12 has quit IRC19:44
*** suresh12 has joined #openstack-nova19:44
*** dtantsur is now known as dtantsur|afk19:44
openstackgerritAditya Vaja proposed openstack/nova master: remove IVS plug/unplug as they're moved to separate plugin  https://review.openstack.org/53437119:46
*** moshele has quit IRC19:46
*** pramodrj07 has joined #openstack-nova19:49
efriedmriedem: I wouldn't mind, but shouldn't edleafe or jaypipes repropose the NRP spec?19:52
mriedemefried: ed did it in queens,19:55
mriedemi think anyone familiar with the changes can do it19:55
mriedemwhich i think you are19:55
efriedmriedem: Roger that.  jaypipes edleafe Any objections?19:56
efried... or local WIP already?19:57
edleafeefried: not from me19:58
*** READ10 has quit IRC20:01
*** READ10 has joined #openstack-nova20:03
*** eharney has joined #openstack-nova20:03
*** yamahata has joined #openstack-nova20:05
openstackgerritLee Yarwood proposed openstack/nova master: rbd: flatten images when creating/unshelving an instance  https://review.openstack.org/45788620:06
mriedemdansmith: we likely want 1 or 2 blueprints for rocky for cellsv1 and nova-network removal...figured you'd like to create those20:08
*** Swami has joined #openstack-nova20:08
mriedemthe nova-net removal is going to be a pita b/c of our functional and unit tests that rely on stubs and crap20:08
*** ttsiout__ has joined #openstack-nova20:10
jaypipesefried: nope, go for it.20:11
efriedack20:11
*** ttsiouts_ has quit IRC20:11
efriedjaypipes, 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-nova20:13
edleafeefried: yeah, that's why we need to re-propose: to reflect today's situation20:14
*** moshele has quit IRC20:15
mriedemdid somebody say "the situation"?20:15
efriededleafe: 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
mriedemhttp://pmcdeadline2.files.wordpress.com/2014/03/mike-the-situation__140331172717.jpg20:15
efriedWhich means no fast approval.  But hopefully closer fidelity to reality.20:16
edleafeefried: isn't that always the case? :)20:16
edleafemriedem: ah, your laptop's background image20:16
mriedemi use it as motivation20:16
mriedemto be jerseylicious20:16
openstackgerritMatt Riedemann proposed openstack/nova stable/pike: Add regression test for BFV+IsolatedHostsFilter failure  https://review.openstack.org/54360220:17
openstackgerritMatt Riedemann proposed openstack/nova stable/pike: Handle volume-backed instances in IsolatedHostsFilter  https://review.openstack.org/54360320:17
openstackgerritMatt Riedemann proposed openstack/nova stable/pike: Fix docs for IsolatedHostsFilter  https://review.openstack.org/54360420:17
mriedemefried: how much of the change to the spec is implementation details vs just high level design?20:18
efriedmriedem: 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 IRC20:19
mriedemthen for the done stuff, say it's done in queens and update the not-yet-done-stuff parts20:20
efriedmriedem: That latter part isn't even mentioned in this spec.20:21
efriedBut yeah.  I'll do the needful.20:22
efriedThere will be controversy over NUMA, though.20:22
mriedemefried: 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
mriedemand we create a new bp/spec for the things that are pending and move all of those changes under that new bp?20:22
mriedembut i deferred the NRP bp from queens b/c there were remaining things to use it20:23
efriedmriedem: Finished reading now, and yeah, that's kinda what I was thinking.20:23
*** alexchadin has joined #openstack-nova20:23
efriedmriedem: I think that (deferring/remaining) was a reflection of "scope creep".20:24
mriedemhttps://specs.openstack.org/openstack/nova-specs/specs/queens/approved/nested-resource-providers.html#work-items20:24
mriedemthe last 2 bullets in there aren't done are they?20:24
efriedmriedem: Nor will they be.20:24
efriedCause that's no longer the plan.20:24
mriedemoy20:24
mriedemok, so the nrp spec became the api stuff and foundational plumbing,20:24
mriedemand support for modeling numa and sriov are separate new thing snow20:25
mriedemyeah?20:25
efriedThat's part of the story, yes.20:25
mriedemif 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 it20:25
efriedIf 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
edleafejaypipes: cdent: efried: dansmith: mriedem: wrote up something quick on my ideas on affinity: https://blog.leafe.com/modeling-affinity/20:26
edleafecomments welcome!20:26
mriedemefried: that's an implementation detail though isn't it?20:26
*** suresh12 has quit IRC20:26
cdentthanks edleafe enqueued for tomorrow, currently writing my own20:27
efriedmriedem: 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-nova20:27
mriedemefried: ok, i guess i need to read your provider tree spec then20:28
mriedemwhich 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
efriedAnd 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-nova20:30
*** READ10 has quit IRC20:30
mriedemok, i'll get to new business some other day; was just trying to go through easy re-approvals20:31
mriedemi will mentally note that this isn't an easy re-approval20:32
efriedmriedem: 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@13620:32
*** moshele has quit IRC20:33
efriedmriedem: 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 IRC20:34
efriedSuspect there will be much PTGing at the PTG about this.20:34
mriedemok - 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 specs20:35
efriedmriedem: Do you want me to propose that edit?20:35
mriedemup to you20:35
efriedDo we do that?20:35
mriedemsometimes if things changed quite a bit20:35
mriedemthe thing i wouldn't want to say is,20:35
mriedemthe nrp spec is marked 'implemented' for queens and it says 'we'll model sriov with this in queens' and then we didn't20:36
mriedemif 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 queens20:36
mriedemand amend accordingly20:36
efriedmriedem: Okay, well, the spec does say the NUMA/SR-IOV stuff won't be done in Q.20:36
efriedBut it also asserts pretty forcefully what that will look like when it does get implemented.20:37
*** suresh12 has joined #openstack-nova20:37
mriedemthe goal in queens was sriov20:37
mriedemhttps://specs.openstack.org/openstack/nova-specs/specs/queens/approved/nested-resource-providers.html#work-items20: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 done20:38
*** lpetrut has quit IRC20:38
mriedemwhich implies sriov would be supported in queens20:38
*** tssurya has quit IRC20:39
efriedwhich 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
efriedmriedem: Softening those statements jeeest a leetle bit would make them more or less true: "...the ability to *model* NUMA and SR-IOV..."20:48
efriedYou can model it.  You just can't *do* anything with that model.20:48
efriedAnd you can't actually model it without writing more code on the nova side; but the placement side will handle it.20:49
efriedExcept for the GET /allocation_candidates thing, which is required if it's going to be usable at all.20:49
mriedemhttps://specs.openstack.org/openstack/nova-specs/specs/queens/approved/nested-resource-providers.html#rest-api-impact20:49
efriedSo 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
efriedStill true.20:50
mriedemi think the nrp spec is mostly just about the modeling and placement api bits,20:50
mriedemi 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 later20:50
efriedBut 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
mriedemsure, that's a new work effort20:51
efriedThat'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 IRC20:51
mriedemso 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 queens20:53
mriedemagree?20:53
mriedemadd root_provider_uuid and parent_provider_uuid to GET /resource_providers, fin20: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
efriedand ?tree= to the latter (written as ?root= in the spec, was renamed during impl)20:54
mriedemyou mean ?in_tree?20:54
mriedembazinga!20:55
mriedemyeah so amend the ?in_tree thingy (we still haven't defined how that filter works i don't think)20:55
mriedemremove/reword the parts about the scheduler client - say those are done separately later20:55
efriedfusque20:55
mriedemand i think we say the rest is done in queens20:55
mriedemwith caveats like the GET /allocation_candidates thing20:56
efriedOh, we nailed down in_tree.  It's implemented.20:56
mriedemi know it's in the api20:56
mriedembut the semantics of how that works is not documented anywhere as far as i know20:56
mriedemi remember bringing this up in a review or the ML or something at one point20:56
mriedemhttps://review.openstack.org/#/c/520663/9/nova/tests/unit/scheduler/client/test_report.py@140020:57
efriedhttps://developer.openstack.org/api-ref/placement/#list-resource-providers20:57
mriedemyeah 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
jaypipeswhat's unclear about that?20:58
mriedembased 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 tree20:58
jaypipesah I see20:58
mriedemin my example, B and D are in the same tree because they share a common ancestor20:59
*** suresh12 has quit IRC20:59
mriedembut i wasn't sure about that20:59
mriedemi mean on first reading of the api-ref20:59
efriedIt 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
mriedemhttp://lists.openstack.org/pipermail/openstack-dev/2017-December/125453.html21:00
mriedemyeah that's what i asked for in ^21:00
jaypipessorry, must have missed that21:01
mriedemit's fine, it was 2 weeks before xmas21:01
efriedThink I was already out by then.  Yeah, that's the excuse I'll use.21:01
mriedemanyway, i think we're maybe in agreement about the spec?21:02
mriedembefore the tangent21:02
*** ChanServ has quit IRC21:02
efriedmriedem: I'll take a crack at it.  And a new spec for R.21:02
mriedemthanks; i know it's not fun21:03
efriedmriedem: I'm doing it all for that shiny nickel you promised me back in the Fall.21:04
mriedemi promised a nickel21:04
mriedem?21:04
efriedoh, totally.  I have it in writing.21:04
mriedemhttps://www.reddit.com/r/TheSimpsons/comments/1saamn/and_remember_a_shiny_new_donkey_for_whoever/21:04
mriedemif i did, it's because i ripped off the simpsons21:04
mriedemooo https://www.youtube.com/watch?v=0gGsKxMCM9o21:05
efriedmriedem: https://review.openstack.org/#/c/506370/21:07
efriednote topic21:08
efriedI still need to find the eavesdrop where you offered it up, tho.21:08
*** felipemonteiro__ has joined #openstack-nova21:08
mriedemhttp://eavesdrop.openstack.org/irclogs/%23openstack-nova/%23openstack-nova.2017-09-21.log.html#t2017-09-21T16:39:1921:08
efriedbwaHA http://eavesdrop.openstack.org/irclogs/%23openstack-nova/%23openstack-nova.2017-09-21.log.html#t2017-09-21T16:39:1921:08
efriedyeah, what you said.21:08
mriedemi win21:09
efriedHappy to concede that victory.  I still get my nickel.21:09
mriedemalright, fair enough21:09
* mriedem makes a note about this gd nickel for dublin21:09
*** slaweq has quit IRC21:09
efriedDo they have Euro nickels?21:09
efriedWe'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 IRC21:11
*** ChanServ has joined #openstack-nova21:11
*** barjavel.freenode.net sets mode: +o ChanServ21:11
mriedemdon't know21:11
*** felipemonteiro_ has quit IRC21:12
efriedI don't see 540179 in the post queue.  So...21:12
* efried heads over to -infra...21:12
mriedemi've got some 5 kronors from the czech republic..21:13
*** jaypipes has quit IRC21:16
*** r-daneel has quit IRC21:17
*** r-daneel has joined #openstack-nova21:17
mriedem*koruna?21:17
openstackgerritMerged openstack/nova master: Remove unused tempest-dsvm-lxc-rc  https://review.openstack.org/53924421:18
openstackgerritMerged openstack/nova master: [placement] annotate loadapp as public interface  https://review.openstack.org/52669121:19
efriedmriedem: 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 IRC21:23
*** felipemonteiro__ has joined #openstack-nova21:23
*** jgriffith has quit IRC21:26
*** mvk_ has joined #openstack-nova21:33
efriedmriedem: 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-nova21:34
efried...other than by virtue of it being in the .../queens/... subdirectory21:34
*** oomichi has joined #openstack-nova21:35
mriedemefried: correct21:35
efriedack21:35
mriedemmuch like the american constitution, it's a living document21:36
*** edmondsw has quit IRC21:39
*** felipemonteiro__ has quit IRC21:39
*** slaweq has joined #openstack-nova21:41
openstackgerritChris Dent proposed openstack/nova master: WIP: Move resource provider objects into placement hierarchy  https://review.openstack.org/54004921:48
openstackgerritChris Dent proposed openstack/nova master: Optional separate database for placement API  https://review.openstack.org/36276621:48
openstackgerritChris Dent proposed openstack/nova master: WIP: Isolate placement database config  https://review.openstack.org/54143521:48
openstackgerritChris Dent proposed openstack/nova master: Isolate config parse_args for placement  https://review.openstack.org/54349521:48
openstackgerritMatt Riedemann proposed openstack/nova-specs master: Re-propose use Neutron's new port binding API  https://review.openstack.org/54357821:53
*** ttsiout__ has quit IRC21:53
*** ttsiouts_ has joined #openstack-nova21:54
*** kaisers_ has joined #openstack-nova21:55
*** burt has quit IRC21:57
openstackgerritMatt Riedemann proposed openstack/nova stable/ocata: Add regression test for BFV+IsolatedHostsFilter failure  https://review.openstack.org/54363421:58
openstackgerritMatt Riedemann proposed openstack/nova stable/ocata: Handle volume-backed instances in IsolatedHostsFilter  https://review.openstack.org/54363521:58
openstackgerritMatt Riedemann proposed openstack/nova stable/ocata: Fix docs for IsolatedHostsFilter  https://review.openstack.org/54363621:58
*** ttsiouts_ has quit IRC21:58
*** felipemonteiro__ has joined #openstack-nova21:59
mriedemi think archive_deleted_rows might be broken22:06
dansmithmriedem: I think I might be volunteering to work on archive and purge this cycle22:07
mriedemthere 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_71231922:07
*** Sukhdev has joined #openstack-nova22:07
mriedemalthough maybe it's just my patch22:07
mriedembecause 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.gz22:08
mriedemi bet it's the set -e22:08
dansmithif you set -e in the post hook script then yeah22:09
mriedembut i'm not sure why it'd be blowing up when archive_deleted_rows runs22:09
mriedemsince it doesn't look like that is failing22:10
dansmithit returns 1 until you don't need to run it again right?22:10
mriedemit runs the loop up to 30 times as long as the CLI returns 122:11
mriedemuntil it returns 0 for all done22:11
dansmithright22:12
dansmithbut set -e will break that right?22:12
*** threestrands has joined #openstack-nova22:12
mriedemyeah i forgot how strict that was22:12
mriedemneed to capture the $? differently22:12
mriedemor, just move the set -e to later22:12
dansmithwell, unset -e during that loop or put it somewhere else or run it in a subshell22:12
dansmithyeah22:12
mriedemyeah i had this in a separate file earlier but couldn't figure out why it was failng, now it's pretty obvious22:13
*** jmlowe has quit IRC22:14
openstackgerritMatt Riedemann proposed openstack/nova master: Check for leaked server resource allocations in post_test_hook  https://review.openstack.org/53851022:16
*** pchavva has quit IRC22:21
*** amodi has quit IRC22:22
openstackgerritJackie Truong proposed openstack/nova master: Add trusted_certs to instance_extra  https://review.openstack.org/53789722:24
openstackgerritJackie Truong proposed openstack/nova master: Add trusted_certs object  https://review.openstack.org/48940822:24
openstackgerritJackie Truong proposed openstack/nova master: Implement certificate_utils  https://review.openstack.org/47994922:24
openstackgerritJackie Truong proposed openstack/nova master: Add trusted_image_certificates to REST API  https://review.openstack.org/48620422:24
*** felipemonteiro__ has quit IRC22:26
*** felipemonteiro__ has joined #openstack-nova22:26
*** masber has quit IRC22:29
*** slaweq has quit IRC22:30
*** rcernin has joined #openstack-nova22:31
openstackgerritMerged openstack/nova-specs master: Add z/VM Driver in Rocky  https://review.openstack.org/53618622:34
openstackgerritMerged openstack/os-traits stable/pike: Update .gitreview for stable/pike  https://review.openstack.org/48866722:35
openstackgerritMerged openstack/os-traits stable/pike: Update UPPER_CONSTRAINTS_FILE for stable/pike  https://review.openstack.org/48866822:35
*** acormier_ has quit IRC22:36
*** acormier has joined #openstack-nova22:36
*** acormier has quit IRC22:40
*** acormier has joined #openstack-nova22:41
*** tidwellr has quit IRC22:46
*** acormier has quit IRC22:46
*** kaisers_ has quit IRC22:55
*** felipemonteiro_ has joined #openstack-nova22:57
*** felipemonteiro__ has quit IRC23:01
*** slaweq has joined #openstack-nova23:02
claudiubmriedem: 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
claudiubbut 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
claudiuband even if I destroy the instance, only one set of allocations get removed.23:05
*** slaweq has quit IRC23:07
*** Sukhdev has quit IRC23:20
*** AlexeyAbashkin has joined #openstack-nova23:21
*** mlavalle has quit IRC23:25
*** AlexeyAbashkin has quit IRC23:26
*** mlavalle has joined #openstack-nova23:27
*** mlavalle has quit IRC23:29
*** abalutoiu__ is now known as abalutoiu23:29
*** abalutoiu has quit IRC23:30
*** abalutoiu has joined #openstack-nova23:30
*** Sukhdev has joined #openstack-nova23:32
*** claudiub has quit IRC23:34
*** Shilpa has quit IRC23:40
*** bhagyashris has quit IRC23:40
*** bhagyashris has joined #openstack-nova23:41
*** Shilpa has joined #openstack-nova23:41
*** cdent has quit IRC23:43
*** acormier has joined #openstack-nova23:48
*** acormier has quit IRC23:52
*** takashin has joined #openstack-nova23:53
*** felipemonteiro_ has quit IRC23:53

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