Thursday, 2017-12-14

openstackgerritDan Smith proposed openstack/nova master: Don't persist could-be-stale InstanceGroup fields in RequestSpec  https://review.openstack.org/52779900:00
*** kumarmn has joined #openstack-nova00:03
*** klindgren_ has quit IRC00:05
openstackgerritmelanie witt proposed openstack/nova master: Convert websocketproxy to use db for token validation  https://review.openstack.org/33399000:15
*** felipemonteiro has joined #openstack-nova00:18
*** felipemonteiro_ has joined #openstack-nova00:19
*** hiro-kobayashi has joined #openstack-nova00:21
*** itlinux has quit IRC00:22
*** felipemonteiro has quit IRC00:23
*** felipemonteiro_ has quit IRC00:26
*** fragatina has quit IRC00:28
*** kumarmn has quit IRC00:28
*** fragatina has joined #openstack-nova00:28
openstackgerritMerged openstack/nova master: Update Instance action's updated_at when action event updated.  https://review.openstack.org/50747300:29
*** fragatin_ has joined #openstack-nova00:33
*** armax has joined #openstack-nova00:33
*** fragatina has quit IRC00:33
*** fragatin_ has quit IRC00:34
*** Swami has quit IRC00:35
*** prateek_ has quit IRC00:37
*** prateek_ has joined #openstack-nova00:38
*** gouthamr has joined #openstack-nova00:42
*** harlowja has quit IRC00:45
*** sree has joined #openstack-nova00:47
*** fragatina has joined #openstack-nova00:49
*** chyka has quit IRC00:51
*** sree has quit IRC00:52
*** AlexeyAbashkin has joined #openstack-nova00:53
*** fragatina has quit IRC00:54
*** AlexeyAbashkin has quit IRC00:57
*** kumarmn has joined #openstack-nova00:58
openstackgerritMerged openstack/nova master: Add instance action db and obj pagination support.  https://review.openstack.org/52642200:59
*** kumarmn has quit IRC01:01
*** kumarmn has joined #openstack-nova01:01
*** mriedem has quit IRC01:06
*** phuongnh has joined #openstack-nova01:09
*** mriedem has joined #openstack-nova01:10
*** yangyapeng has joined #openstack-nova01:12
*** psachin has joined #openstack-nova01:14
openstackgerritMatt Riedemann proposed openstack/nova master: Pass mountpoint to volume attachment_update  https://review.openstack.org/52746801:16
openstackgerritMatt Riedemann proposed openstack/nova master: Update and complete volume attachments during resize  https://review.openstack.org/52722801:16
openstackgerritMatt Riedemann proposed openstack/nova master: Pass bdms to versioned notifications during finish_revert_resize  https://review.openstack.org/52744601:16
openstackgerritMatt Riedemann proposed openstack/nova master: Pass mountpoint to volume attachment_create with connector  https://review.openstack.org/52747901:16
openstackgerritMatt Riedemann proposed openstack/nova master: Remove unnecessary connector stash in attachment_update  https://review.openstack.org/52748401:16
openstackgerritMerged openstack/nova master: Raise MarkerNotFound if BuildRequestList.get_by_filters doesn't find marker  https://review.openstack.org/52756401:17
*** Apoorva has quit IRC01:18
*** Sukhdev has quit IRC01:19
*** kumarmn has quit IRC01:20
*** kumarmn has joined #openstack-nova01:21
*** kumarmn has quit IRC01:22
*** kumarmn has joined #openstack-nova01:22
*** liverpooler has joined #openstack-nova01:23
*** pramodrj07 has quit IRC01:28
*** kumarmn has quit IRC01:36
*** kumarmn has joined #openstack-nova01:36
*** kumarmn has quit IRC01:36
*** kumarmn has joined #openstack-nova01:37
*** esberglu has joined #openstack-nova01:37
*** namnh has joined #openstack-nova01:37
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Add pagination and changes-since for instance-actions  https://review.openstack.org/32632601:41
*** esberglu has quit IRC01:41
*** andreas_s has joined #openstack-nova01:45
yikun^ rebase 2.57 --> 2.5801:47
*** andreas_s has quit IRC01:49
*** amodi has quit IRC01:57
openstackgerritTakashi NATSUME proposed openstack/nova master: Fix missing marker functions  https://review.openstack.org/51457901:59
*** kukacz has quit IRC02:00
*** litao__ has joined #openstack-nova02:01
*** kukacz has joined #openstack-nova02:01
mriedemyikun: on it02:02
*** Apoorva has joined #openstack-nova02:03
yikunmriedem, :), thanks02:03
mriedemyikun: you dropped something from the rest api history doc02:05
*** kumarmn has quit IRC02:07
yikunmr02:07
yikunmriedem, oops, my bad, I will address soon02:07
*** Apoorva has quit IRC02:09
mriedemyikun: i can get it from gerrit02:09
*** itlinux has joined #openstack-nova02:09
openstackgerritMatt Riedemann proposed openstack/nova master: Add pagination and changes-since for instance-actions  https://review.openstack.org/32632602:10
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Add pagination and changes-since for instance-actions  https://review.openstack.org/32632602:10
*** Tom-Tom has joined #openstack-nova02:13
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Add pagination and changes-since for instance-actions  https://review.openstack.org/32632602:14
*** kumarmn has joined #openstack-nova02:15
yikunmriedem, another typo fix, so, I update again...now I think it's a final one, ╮(╯▽╰)╭02:19
Kevin_Zhenghttps://github.com/openstack/nova/blob/master/nova/api/openstack/compute/schemas/servers.py#L391 are we ever going to set this to False?02:20
*** kumarmn has quit IRC02:21
*** threestrands_ has joined #openstack-nova02:22
*** threestrands_ has quit IRC02:22
*** threestrands_ has joined #openstack-nova02:22
*** kumarmn has joined #openstack-nova02:22
*** threestrands has quit IRC02:22
*** threestrands_ has quit IRC02:22
*** threestrands_ has joined #openstack-nova02:23
*** threestrands_ has quit IRC02:23
*** threestrands_ has joined #openstack-nova02:23
openstackgerritMatt Riedemann proposed openstack/nova master: Retry _trait_sync on deadlock  https://review.openstack.org/52783602:24
*** kumarmn has quit IRC02:26
*** sree has joined #openstack-nova02:27
mriedemKevin_Zheng: we can, but it has to be on a microversion02:28
Kevin_ZhengYeah, thats what I thought02:28
gmanni feel if we do then we can do all together for query things of all APIs02:29
*** hoangcx has quit IRC02:29
*** trungnv has quit IRC02:29
*** moshele has joined #openstack-nova02:29
*** trungnv has joined #openstack-nova02:29
*** salv-orl_ has joined #openstack-nova02:29
*** hoangcx has joined #openstack-nova02:29
mriedemyikun: got it, thanks02:30
Kevin_ZhengYeah, so maybe we should add them to all APIs first :)02:30
Kevin_Zhengstill alot missing?02:30
mriedemKevin_Zheng: there is a blacklist of filters that we don't allow02:30
Kevin_Zhengyeah, I know02:31
Kevin_ZhengJust curious :)02:31
mriedemhttps://github.com/openstack/nova/blob/master/nova/api/openstack/compute/schemas/servers.py#L29502:31
mriedembecause those will 50002:31
*** threestrands_ has quit IRC02:32
*** Tom-Tom has quit IRC02:32
*** liverpooler has quit IRC02:32
*** psachin has quit IRC02:32
*** takashin has quit IRC02:32
*** amotoki has quit IRC02:32
*** openstackstatus has quit IRC02:32
*** salv-orlando has quit IRC02:32
*** bkopilov has quit IRC02:32
*** s1061123 has quit IRC02:32
*** purplerbot has quit IRC02:32
*** jose-phillips has quit IRC02:32
*** pooja_jadhav has quit IRC02:32
*** ChanServ has quit IRC02:32
*** rmk has quit IRC02:32
*** sree has quit IRC02:32
mriedemand this https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/servers.py#L18702:33
mriedemif you try filtering on private properties in the data model02:33
*** ChanServ has joined #openstack-nova02:34
*** barjavel.freenode.net sets mode: +o ChanServ02:34
*** vladikr has quit IRC02:34
*** vladikr has joined #openstack-nova02:35
*** threestrands_ has joined #openstack-nova02:36
*** Tom-Tom has joined #openstack-nova02:36
*** liverpooler has joined #openstack-nova02:36
*** psachin has joined #openstack-nova02:36
*** takashin has joined #openstack-nova02:36
*** amotoki has joined #openstack-nova02:36
*** openstackstatus has joined #openstack-nova02:36
*** bkopilov has joined #openstack-nova02:36
*** s1061123 has joined #openstack-nova02:36
*** purplerbot has joined #openstack-nova02:36
*** jose-phillips has joined #openstack-nova02:36
*** pooja_jadhav has joined #openstack-nova02:36
*** barjavel.freenode.net sets mode: +v openstackstatus02:36
*** rmk has joined #openstack-nova02:36
*** _ix has joined #openstack-nova02:37
mriedemyikun: looks like http://logs.openstack.org/26/326326/48/check/openstack-tox-functional/496db26/testr_results.html.gz is failing02:38
mriedemi'm guessing it's using an older server create sample which has personality files in it?02:39
Kevin_Zhenghttps://github.com/openstack/nova/blob/master/nova/api/openstack/compute/servers.py#L18602:39
Kevin_Zheng^ JOINED_TABLE_QUERY_PARAMS_SERVERS also return 40002:39
Kevin_Zhengnot 50002:39
gmannmriedem: is this looks fine now (server whitelist doc) - https://review.openstack.org/#/c/527042/302:39
mriedemKevin_Zheng: right, i mean,02:39
mriedembefore that blacklist, those would be a 50002:39
Kevin_ZhengAh ok02:40
*** zhurong has joined #openstack-nova02:41
*** gbarros has joined #openstack-nova02:43
*** liverpooler has quit IRC02:48
*** Daisy_ has joined #openstack-nova02:49
*** lei-zh has joined #openstack-nova02:50
*** Daisy_ has quit IRC02:50
openstackgerritTakashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in delete  https://review.openstack.org/52523102:55
*** TuanLA has joined #openstack-nova02:55
*** sree has joined #openstack-nova02:58
*** sree has quit IRC03:03
*** lei-zh has quit IRC03:05
*** lei-zh has joined #openstack-nova03:06
*** Tom-Tom has quit IRC03:14
yikunmriedem, a bit confusing, we set version=2.58 (https://review.openstack.org/#/c/326326/48/nova/tests/functional/api_sample_tests/test_instance_actions.py@137), why call 2.1 API?03:15
mriedemyikun: it's calling with 2.5803:16
mriedembut using a 2.1 sample i think03:16
Kevin_Zhengyeah03:16
mriedemand the 2.1 sample has 'personality' files in it,03:16
mriedemwhich aren't supported past 2.5703:16
Kevin_Zheng_post_server() reads old samples by default03:17
mriedemi think it's sending this https://github.com/openstack/nova/blob/master/doc/api_samples/servers/server-create-req.json#L1303:17
mriedemyikun: you need it to send this request https://github.com/openstack/nova/blob/master/doc/api_samples/servers/v2.57/server-create-req.json03:17
mriedemis there a parameter in _post_server() to override the sample to use?03:17
*** gouthamr has quit IRC03:17
mriedemi think it might not work if you override it though if you don't have a server-create-req.json sample in the 2.57 directory03:18
mriedemyou'd have to try it03:18
mriedemotherwise, there is an easier way03:19
mriedemdon't use a sample to create the server03:19
mriedemjust use self.api.post_server(...)03:19
mriedemhttps://github.com/openstack/nova/blob/master/nova/tests/functional/api/client.py#L27103:19
openstackgerritGhanshyam Mann proposed openstack/nova master: Add server filters whitelist in server api-ref  https://review.openstack.org/52704203:19
Kevin_Zhengjust set use_common_server_api_samples to false03:19
*** bkopilov has quit IRC03:20
mriedemyou can build the request using https://github.com/openstack/nova/blob/master/nova/tests/functional/integrated_helpers.py#L14003:20
openstackgerritGhanshyam Mann proposed openstack/nova master: Add server filters whitelist in server api-ref  https://review.openstack.org/52704203:20
*** amodi has joined #openstack-nova03:20
mriedemi think use_common_server_api_samples=False might make it look in the 2.57 directory for the sample03:20
mriedemnot sure03:20
*** armax has quit IRC03:21
Kevin_Zhengyeah, have to add a new sample file03:21
*** markvoelker_ has quit IRC03:24
*** penick has quit IRC03:25
*** esberglu has joined #openstack-nova03:26
gmannmriedem: yikun : if use_common_server_api_samples=False,  it will use 'cls.sample_dir + /microversion' to find sample files. which is all set in https://review.openstack.org/#/c/326326/48/nova/tests/functional/api_sample_tests/test_instance_actions.py@13703:27
gmannbut you need to put server req and res sample in that dir03:27
openstackgerritDai Dang Van proposed openstack/nova master: WIP: libvirt: Allow multiple volume attachments  https://review.openstack.org/26758703:27
yikunyes, use_common_server_api_samples=False look in the dir 2.58. :), and not found if we don't add it03:27
gmanncurrent logic does not fallback to previous version directory, it has to be present in all directory you want to tests03:27
mriedemyeah that's why i'm saying,03:27
mriedemdon't use a sample,03:27
mriedemjust create the server directly03:28
mriedemhttps://github.com/openstack/nova/blob/master/nova/tests/functional/api/client.py#L27103:28
mriedemhttps://github.com/openstack/nova/blob/master/nova/tests/functional/integrated_helpers.py#L14003:28
mriedemthe test only cares about the actions, not how the server is created03:28
gmannyea, +103:28
yikunyea, got it03:28
mriedemcan i get a hell yeah?!03:28
* mriedem is getting loopy after a long day03:28
Kevin_Zhengeveryday is a long day for mriedem03:29
*** esberglu has quit IRC03:31
openstackgerritMatthew Oliver proposed openstack/nova master: Update the obs placement-api port  https://review.openstack.org/52784603:31
*** Apoorva has joined #openstack-nova03:31
*** annp has joined #openstack-nova03:31
*** gongysh has joined #openstack-nova03:34
*** zhurong has quit IRC03:34
*** andreas_s has joined #openstack-nova03:43
*** lbragstad has quit IRC03:44
*** sridharg has joined #openstack-nova03:46
*** andreas_s has quit IRC03:48
*** gongysh has quit IRC03:49
*** sree has joined #openstack-nova03:49
*** sree has quit IRC03:53
*** moshele has quit IRC03:55
*** armax has joined #openstack-nova03:55
*** moshele has joined #openstack-nova03:57
openstackgerritTakashi NATSUME proposed openstack/nova master: List/show all server migration types (1/2)  https://review.openstack.org/43060804:02
*** udesale has joined #openstack-nova04:03
*** owalsh_ has joined #openstack-nova04:04
*** owalsh has quit IRC04:07
*** kumarmn has joined #openstack-nova04:08
*** lei-zh has quit IRC04:11
*** armax has quit IRC04:12
*** armax has joined #openstack-nova04:13
*** armax has quit IRC04:13
*** kumarmn has quit IRC04:13
*** armax has joined #openstack-nova04:14
*** armax has quit IRC04:14
*** armax has joined #openstack-nova04:14
*** moshele has quit IRC04:14
*** armax has quit IRC04:15
*** diga has joined #openstack-nova04:17
*** karthiks has quit IRC04:17
*** karthiks has joined #openstack-nova04:17
*** yamamoto has joined #openstack-nova04:18
*** vladikr has quit IRC04:22
*** _ix has quit IRC04:24
*** vladikr has joined #openstack-nova04:25
*** sree has joined #openstack-nova04:27
*** kumarmn has joined #openstack-nova04:30
*** sree has quit IRC04:31
*** TuanLA has quit IRC04:36
*** hieulq has quit IRC04:36
*** hieulq has joined #openstack-nova04:36
*** TuanLA has joined #openstack-nova04:36
*** hoangcx has quit IRC04:40
*** namnh has quit IRC04:40
*** phuongnh has quit IRC04:40
*** annp has quit IRC04:40
*** trungnv has quit IRC04:40
*** phuongnh has joined #openstack-nova04:40
*** hoangcx has joined #openstack-nova04:40
*** trungnv has joined #openstack-nova04:40
*** annp has joined #openstack-nova04:40
*** namnh has joined #openstack-nova04:40
*** itlinux has quit IRC04:45
*** sree has joined #openstack-nova04:47
*** bkopilov has joined #openstack-nova04:48
*** mriedem has quit IRC04:49
*** ratailor has joined #openstack-nova04:50
*** sree_ has joined #openstack-nova04:51
*** sree_ is now known as Guest9752804:51
*** kumarmn has quit IRC04:53
*** kumarmn has joined #openstack-nova04:53
*** sree has quit IRC04:55
*** gbarros has quit IRC05:00
*** lei-zh has joined #openstack-nova05:04
*** adisky__ has joined #openstack-nova05:18
*** s1061123 has quit IRC05:20
*** kumarmn has quit IRC05:21
*** kumarmn has joined #openstack-nova05:21
*** kumarmn has quit IRC05:23
*** kumarmn has joined #openstack-nova05:24
*** markvoelker has joined #openstack-nova05:25
*** s1061123 has joined #openstack-nova05:25
*** kumarmn has quit IRC05:29
openstackgerritzhufl proposed openstack/nova master: Fix some typos in nova doc  https://review.openstack.org/52785605:31
*** links has joined #openstack-nova05:35
*** sapd has quit IRC05:38
*** sapd_ has joined #openstack-nova05:38
*** Tom-Tom has joined #openstack-nova05:44
*** Tom-Tom_ has joined #openstack-nova05:47
*** eandersson has quit IRC05:47
*** janki has joined #openstack-nova05:48
*** Tom-Tom has quit IRC05:48
*** penick has joined #openstack-nova05:48
*** dtruong_ has quit IRC05:49
*** penick_ has joined #openstack-nova05:53
*** penick has quit IRC05:54
*** udesale__ has joined #openstack-nova05:54
*** amodi has quit IRC05:55
*** udesale has quit IRC05:56
*** udesale has joined #openstack-nova05:57
*** markvoelker has quit IRC05:58
*** udesale__ has quit IRC05:59
*** ratailor has quit IRC06:01
*** ratailor has joined #openstack-nova06:03
*** klindgren has joined #openstack-nova06:07
*** hiro-kobayashi has quit IRC06:08
*** amodi has joined #openstack-nova06:12
*** links has quit IRC06:13
*** links has joined #openstack-nova06:21
*** links has quit IRC06:21
*** lpetrut has joined #openstack-nova06:24
*** edand has joined #openstack-nova06:26
*** esberglu has joined #openstack-nova06:26
*** esberglu has quit IRC06:31
*** Apoorva has quit IRC06:31
*** karthiks has quit IRC06:33
*** hoangcx has quit IRC06:41
*** namnh has quit IRC06:41
*** trungnv has quit IRC06:41
*** annp has quit IRC06:41
*** phuongnh has quit IRC06:41
*** namnh has joined #openstack-nova06:41
*** trungnv has joined #openstack-nova06:41
*** annp has joined #openstack-nova06:41
*** hoangcx has joined #openstack-nova06:41
*** phuongnh has joined #openstack-nova06:42
*** andreas_s has joined #openstack-nova06:42
*** karthiks has joined #openstack-nova06:46
*** andreas_s has quit IRC06:47
*** openstackgerrit has quit IRC06:47
*** TuanLA_ has joined #openstack-nova06:48
*** TuanLA has quit IRC06:50
*** threestrands_ has quit IRC06:53
*** openstackgerrit has joined #openstack-nova06:55
openstackgerritZhenyu Zheng proposed openstack/nova master: Updated common create server sample request because of microversion 2.57  https://review.openstack.org/52787806:55
*** markvoelker has joined #openstack-nova06:55
*** karthiks has quit IRC06:55
openstackgerritZhenyu Zheng proposed openstack/nova master: Updated common create server sample request because of microversion 2.57  https://review.openstack.org/52787806:57
*** damien_r has joined #openstack-nova07:00
*** huanxie has quit IRC07:02
*** huanxie has joined #openstack-nova07:02
*** amodi has quit IRC07:04
*** esberglu has joined #openstack-nova07:04
*** sapd__ has joined #openstack-nova07:07
*** sapd_ has quit IRC07:07
*** Tom-Tom_ has quit IRC07:07
*** josecastroleon has joined #openstack-nova07:07
*** esberglu has quit IRC07:08
*** karthiks has joined #openstack-nova07:08
openstackgerritYa Lian Pan proposed openstack/nova master: z/VM Driver: Spawn and destroy function of z/VM driver  https://review.openstack.org/52765807:10
*** hoangcx has quit IRC07:14
*** annp has quit IRC07:14
*** namnh has quit IRC07:14
*** phuongnh has quit IRC07:14
*** trungnv has quit IRC07:14
*** trungnv has joined #openstack-nova07:14
*** annp has joined #openstack-nova07:14
*** namnh has joined #openstack-nova07:14
*** phuongnh has joined #openstack-nova07:14
*** hoangcx has joined #openstack-nova07:15
*** amodi has joined #openstack-nova07:15
*** hieulq has quit IRC07:23
*** damien_r has quit IRC07:27
*** Alex_Staf has joined #openstack-nova07:27
*** lpetrut has quit IRC07:28
*** chyka has joined #openstack-nova07:28
*** markvoelker has quit IRC07:30
*** karthiks has quit IRC07:31
*** felipemonteiro_ has joined #openstack-nova07:31
openstackgerritOpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata  https://review.openstack.org/52479507:32
*** chyka has quit IRC07:33
*** bkopilov has quit IRC07:37
*** moshele has joined #openstack-nova07:38
*** diga_ has joined #openstack-nova07:38
*** diga has quit IRC07:39
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Add pagination and changes-since for instance-actions  https://review.openstack.org/32632607:42
*** klindgren has quit IRC07:43
openstackgerritYa Lian Pan proposed openstack/nova master: z/VM Driver: Spawn and destroy function of z/VM driver  https://review.openstack.org/52765807:44
*** vladikr has quit IRC07:48
*** sshwarts has joined #openstack-nova07:48
*** vladikr has joined #openstack-nova07:48
*** karthiks has joined #openstack-nova07:49
*** bkopilov has joined #openstack-nova07:53
openstackgerritTakashi NATSUME proposed openstack/nova master: List/show all server migration types (2/2)  https://review.openstack.org/45948307:59
openstackgerritChason Chan proposed openstack/nova master: Fix the bug report link of API Guide  https://review.openstack.org/52766008:01
*** felipemonteiro__ has joined #openstack-nova08:03
*** sahid has joined #openstack-nova08:04
*** felipemonteiro_ has quit IRC08:06
*** AlexeyAbashkin has joined #openstack-nova08:09
*** alexchadin has joined #openstack-nova08:13
*** janki has quit IRC08:15
*** rcernin has quit IRC08:15
*** sapcc-bot has quit IRC08:18
*** penick_ has quit IRC08:18
*** sapcc-bot has joined #openstack-nova08:18
*** david_1 has joined #openstack-nova08:19
*** tpatzig_ has joined #openstack-nova08:19
*** mkoderer_ has joined #openstack-nova08:19
*** dgonzalez_ has joined #openstack-nova08:19
*** carthaca_ has joined #openstack-nova08:19
*** amodi has quit IRC08:19
*** amodi has joined #openstack-nova08:20
*** sapcc-bot has quit IRC08:21
*** carthaca_ has quit IRC08:21
*** david_1 has quit IRC08:21
*** tpatzig_ has quit IRC08:21
*** mkoderer_ has quit IRC08:21
*** dgonzalez_ has quit IRC08:21
*** tpatzig_ has joined #openstack-nova08:21
*** sapcc-bot has joined #openstack-nova08:21
*** dgonzalez_ has joined #openstack-nova08:21
*** carthaca_ has joined #openstack-nova08:21
*** tpatzig_1 has joined #openstack-nova08:21
*** mkoderer_ has joined #openstack-nova08:21
*** david_2 has joined #openstack-nova08:21
*** dgonzalez_1 has joined #openstack-nova08:21
*** carthaca_1 has joined #openstack-nova08:21
*** david_1 has joined #openstack-nova08:21
*** mkoderer_1 has joined #openstack-nova08:22
*** dgonzalez_ has quit IRC08:23
*** carthaca_ has quit IRC08:23
*** tpatzig_1 has quit IRC08:23
*** david_1 has quit IRC08:23
*** mkoderer_1 has quit IRC08:23
*** TuanLA__ has joined #openstack-nova08:26
*** markvoelker has joined #openstack-nova08:26
*** tssurya has joined #openstack-nova08:27
*** tesseract has joined #openstack-nova08:28
*** TuanLA_ has quit IRC08:28
*** mdnadeem has joined #openstack-nova08:36
*** hoonetorg has quit IRC08:37
*** zzzeek has quit IRC08:43
*** zzzeek has joined #openstack-nova08:45
*** jpena|off is now known as jpena08:46
*** ralonsoh has joined #openstack-nova08:47
*** links has joined #openstack-nova08:52
*** obre has quit IRC08:54
*** hoonetorg has joined #openstack-nova08:54
*** obre has joined #openstack-nova08:56
*** links has quit IRC08:57
*** Alex_Staf has quit IRC08:59
*** markvoelker has quit IRC09:00
openstackgerritTakashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in delete (2)  https://review.openstack.org/52626309:00
openstackgerritTakashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in delete (3)  https://review.openstack.org/52655709:00
*** ociuhandu has joined #openstack-nova09:08
*** alexchadin has quit IRC09:09
*** alexchadin has joined #openstack-nova09:09
*** links has joined #openstack-nova09:10
*** Alex_Staf has joined #openstack-nova09:16
*** felipemonteiro__ has quit IRC09:16
*** bkopilov has quit IRC09:17
openstackgerritTakashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in post  https://review.openstack.org/52682309:17
*** ociuhandu has quit IRC09:18
*** ociuhandu has joined #openstack-nova09:22
*** ociuhandu has quit IRC09:22
*** rcernin has joined #openstack-nova09:23
*** edand_ has joined #openstack-nova09:27
*** owalsh_ is now known as owalsh09:28
*** yangyapeng has quit IRC09:29
openstackgerritDai Dang Van proposed openstack/nova master: WIP: Allow multi-attach in compute api  https://review.openstack.org/27104709:30
*** yangyapeng has joined #openstack-nova09:30
*** edand has quit IRC09:31
*** takashin has left #openstack-nova09:33
*** lucas-afk is now known as lucasagomes09:33
openstackgerritYa Lian Pan proposed openstack/nova master: z/VM Driver: Spawn and destroy function of z/VM driver  https://review.openstack.org/52765809:36
*** janki has joined #openstack-nova09:36
*** mkoderer_1 has joined #openstack-nova09:37
*** tpatzig_1 has joined #openstack-nova09:37
*** david_ has joined #openstack-nova09:37
*** dgonzalez_ has joined #openstack-nova09:37
*** david_ has quit IRC09:39
*** dgonzalez_ has quit IRC09:39
*** tpatzig_1 has quit IRC09:39
*** mkoderer_1 has quit IRC09:39
*** lei-zh has quit IRC09:41
*** derekh has joined #openstack-nova09:42
*** yangyapeng has quit IRC09:49
*** ygl has joined #openstack-nova09:49
*** yangyapeng has joined #openstack-nova09:49
*** ygl has left #openstack-nova09:52
*** edand__ has joined #openstack-nova09:55
*** markvoelker has joined #openstack-nova09:57
*** edand_ has quit IRC09:59
*** lpetrut has joined #openstack-nova09:59
*** sdague has joined #openstack-nova09:59
*** gszasz has joined #openstack-nova10:00
*** yangyapeng has quit IRC10:00
openstackgerritMatthew Booth proposed openstack/nova master: Improve error message on invalid BDM fields  https://review.openstack.org/52791510:01
openstackgerritMatthew Booth proposed openstack/nova master: DriverBlockDevice: make subclasses inherit _proxy_as_attr  https://review.openstack.org/52416710:02
openstackgerritMatthew Booth proposed openstack/nova master: Give volume DriverBlockDevice classes a common prefix  https://review.openstack.org/52634610:02
openstackgerritMatthew Booth proposed openstack/nova master: Add DriverLocalImageBlockDevice  https://review.openstack.org/52634710:02
openstackgerritMatthew Booth proposed openstack/nova master: Use real block_device_info data in test_blockinfo  https://review.openstack.org/52791610:02
*** esberglu has joined #openstack-nova10:04
*** TuanLA__ has quit IRC10:05
*** annp has quit IRC10:07
*** esberglu has quit IRC10:08
*** bkopilov has joined #openstack-nova10:09
*** namnh has quit IRC10:09
openstackgerritBalazs Gibizer proposed openstack/nova master: Fix possible TypeError in VIF.fixed_ips  https://review.openstack.org/52792010:16
*** mounika has joined #openstack-nova10:16
*** josecastroleon has quit IRC10:18
openstackgerritBalazs Gibizer proposed openstack/nova master: handle traits with sharing providers  https://review.openstack.org/51711910:21
openstackgerritBalazs Gibizer proposed openstack/nova master: Test alloc_cands with non overlapping sharing RPs  https://review.openstack.org/51938010:21
openstackgerritBalazs Gibizer proposed openstack/nova master: Test alloc_cands with one RP shared between two RPs  https://review.openstack.org/51961710:21
openstackgerritBalazs Gibizer proposed openstack/nova master: Test allocation candidates: multiple aggregates  https://review.openstack.org/51863310:21
openstackgerritBalazs Gibizer proposed openstack/nova master: Test helper: validate provider summaries  https://review.openstack.org/51898210:21
openstackgerritBalazs Gibizer proposed openstack/nova master: Fix accumulated nits in refactor series  https://review.openstack.org/52118910:21
openstackgerritBalazs Gibizer proposed openstack/nova master: Add aggregates check in allocation candidates  https://review.openstack.org/52240710:21
openstackgerritBalazs Gibizer proposed openstack/nova master: Fix missing rps in allocation candidates  https://review.openstack.org/52240910:21
openstackgerritBalazs Gibizer proposed openstack/nova master: placement: func tests for multiple shared RPs  https://review.openstack.org/49873710:21
*** sambetts|afk is now known as sambetts10:21
gibijaypipes: ^^ rebased the series as master already has the fix for the frequently failing functional test that prevented these to be merged10:23
*** kumarmn has joined #openstack-nova10:25
*** dtantsur|afk is now known as dtantsur10:25
*** edand__ has quit IRC10:25
*** edand__ has joined #openstack-nova10:25
openstackgerritrahul bardia proposed openstack/python-novaclient master: nova limits ERROR (Exception): Field names must be unique This happens when 'Others' field is occuring more than once which is not handled in code causing multiple columns having same name. Which throws above exception and does not pretty prints absolute   https://review.openstack.org/52792210:26
*** yangyapeng has joined #openstack-nova10:28
*** kumarmn has quit IRC10:29
*** markvoelker has quit IRC10:29
*** damien_r has joined #openstack-nova10:30
*** alexchadin has quit IRC10:32
*** alexchadin has joined #openstack-nova10:33
*** amodi has quit IRC10:34
*** amodi has joined #openstack-nova10:35
*** sapd__ has quit IRC10:37
*** yangyapeng has quit IRC10:37
*** tosky has joined #openstack-nova10:37
*** andreas_s has joined #openstack-nova10:38
*** yangyapeng has joined #openstack-nova10:40
*** udesale has quit IRC10:41
*** udesale has joined #openstack-nova10:42
*** karthiks has quit IRC10:42
*** links has quit IRC10:43
*** andreas_s has quit IRC10:43
*** sapd__ has joined #openstack-nova10:51
*** tssurya has quit IRC10:55
*** alexchadin has quit IRC10:55
*** Alex_Staf has quit IRC10:55
*** Alex_Staf has joined #openstack-nova10:55
*** alexchadin has joined #openstack-nova10:55
*** links has joined #openstack-nova10:57
*** zzzeek has quit IRC11:05
*** trozet has quit IRC11:07
*** _pewp_ has quit IRC11:08
*** trozet has joined #openstack-nova11:08
*** _pewp_ has joined #openstack-nova11:09
*** AlexeyAbashkin has quit IRC11:10
*** AlexeyAbashkin has joined #openstack-nova11:11
kashyapstephenfin: Hi there, when you get a moment, you know why is Zuul complaning here - https://review.openstack.org/#/c/527691/11:11
*** zzzeek has joined #openstack-nova11:12
*** yamamoto has quit IRC11:12
*** syjulian has joined #openstack-nova11:13
*** alexchadin has quit IRC11:18
stephenfinkashyap: Looking11:21
*** gszasz has quit IRC11:22
stephenfinkashyap: That job's been failing a lot lately, but I don't know why. It's unrelated to this, in any case11:22
stephenfinWe should probably investigate that, actually11:23
stephenfinbauzas: Ta for the review :)11:24
kashyapstephenfin: Yeah, noted; right now I need to run an errand.11:25
*** markvoelker has joined #openstack-nova11:26
openstackgerritMerged openstack/nova stable/ocata: Add regression test for rebuilding a volume-backed server  https://review.openstack.org/52566411:34
*** linkmark has joined #openstack-nova11:40
openstackgerritBalazs Gibizer proposed openstack/nova master: Add regression test for bug 1735407  https://review.openstack.org/52609511:41
openstackbug 1735407 in OpenStack Compute (nova) "[Nova] Evacuation doesn't respect anti-affinity rules" [Medium,In progress] https://launchpad.net/bugs/1735407 - Assigned to Balazs Gibizer (balazs-gibizer)11:41
openstackgerritBalazs Gibizer proposed openstack/nova master: Add late server group policy check to rebuild  https://review.openstack.org/52524211:41
*** lucasagomes is now known as lucas-hungry11:42
*** diga_ has quit IRC11:43
*** janki has quit IRC11:46
*** alexchadin has joined #openstack-nova11:49
*** amodi has quit IRC11:49
openstackgerritBalazs Gibizer proposed openstack/nova master: Transform missing delete notifications  https://review.openstack.org/41029711:51
openstackgerritBalazs Gibizer proposed openstack/nova master: Send soft_delete from context manager  https://review.openstack.org/47645911:51
*** yamamoto has joined #openstack-nova11:53
*** yamamoto has quit IRC11:54
*** yamamoto has joined #openstack-nova11:55
*** jafeha has quit IRC11:55
*** jafeha has joined #openstack-nova11:58
*** sapcc-bot has quit IRC11:58
*** tpatzig_ has quit IRC11:58
*** carthaca_1 has quit IRC11:58
*** mkoderer_ has quit IRC11:58
*** david_2 has quit IRC11:58
*** dgonzalez_1 has quit IRC11:58
*** mkoderer_ has joined #openstack-nova11:59
*** carthaca_ has joined #openstack-nova11:59
*** tpatzig_ has joined #openstack-nova11:59
*** dgonzalez_ has joined #openstack-nova11:59
*** david_3 has joined #openstack-nova11:59
*** sapcc-bot has joined #openstack-nova11:59
*** mkoderer_1 has joined #openstack-nova11:59
*** tpatzig_1 has joined #openstack-nova11:59
*** david_1 has joined #openstack-nova11:59
*** carthaca_1 has joined #openstack-nova11:59
*** dgonzalez_1 has joined #openstack-nova11:59
*** markvoelker has quit IRC12:00
*** cdent has joined #openstack-nova12:00
*** tpatzig_1 has quit IRC12:01
*** david_1 has quit IRC12:01
*** mkoderer_1 has quit IRC12:01
*** carthaca_1 has quit IRC12:01
*** dgonzalez_1 has quit IRC12:01
*** jpena is now known as jpena|lunch12:01
gibikashyap, stephenfin: the test_live_migration_actions functional test failure has already been fixed on master so you just have to rebase12:01
*** ratailor has quit IRC12:02
stephenfingibi: Aha. What patch fixed it, out of curiosity?12:02
gibikashyap, stephenfin: this was the fix https://review.openstack.org/#/c/527440/12:02
stephenfinjinx :)_12:02
*** amodi has joined #openstack-nova12:02
stephenfinOh, nice12:02
gibistephenfin: there is one more fix in the pipe that fixes another failure mode of that test here https://review.openstack.org/#/c/527444/12:03
openstackgerritBalazs Gibizer proposed openstack/nova master: Extract instance allocation removal code  https://review.openstack.org/51304112:04
*** gszasz has joined #openstack-nova12:04
*** sahid has quit IRC12:09
*** david_3 has quit IRC12:11
*** sapcc-bot has quit IRC12:11
*** mkoderer_ has quit IRC12:11
*** carthaca_ has quit IRC12:11
*** tpatzig_ has quit IRC12:11
*** dgonzalez_ has quit IRC12:11
*** david_4 has joined #openstack-nova12:11
*** tpatzig_ has joined #openstack-nova12:11
*** dgonzalez_ has joined #openstack-nova12:11
*** mkoderer_ has joined #openstack-nova12:11
*** sapcc-bot has joined #openstack-nova12:11
*** carthaca_ has joined #openstack-nova12:11
*** tpatzig_1 has joined #openstack-nova12:11
*** dgonzalez_1 has joined #openstack-nova12:11
*** david_1 has joined #openstack-nova12:11
*** mkoderer_1 has joined #openstack-nova12:11
*** sapd__ has quit IRC12:11
*** carthaca_1 has joined #openstack-nova12:11
*** sapd__ has joined #openstack-nova12:12
*** tpatzig_1 has quit IRC12:13
*** carthaca_1 has quit IRC12:13
*** mkoderer_1 has quit IRC12:13
*** david_1 has quit IRC12:13
*** dgonzalez_1 has quit IRC12:13
*** smatzek has joined #openstack-nova12:15
*** tssurya has joined #openstack-nova12:18
*** yamamoto has quit IRC12:19
*** gbarros has joined #openstack-nova12:20
*** dave-mccowan has joined #openstack-nova12:20
*** udesale has quit IRC12:23
*** dave-mcc_ has joined #openstack-nova12:24
*** tpatzig_ has quit IRC12:25
*** dave-mccowan has quit IRC12:25
*** dgonzalez_1 has joined #openstack-nova12:25
*** david_1 has joined #openstack-nova12:25
*** carthaca_1 has joined #openstack-nova12:25
*** tpatzig_ has joined #openstack-nova12:25
*** mkoderer_1 has joined #openstack-nova12:25
*** kumarmn has joined #openstack-nova12:25
*** tpatzig_1 has joined #openstack-nova12:25
*** tpatzig_ has quit IRC12:27
*** alexchadin has quit IRC12:27
*** carthaca_1 has quit IRC12:27
*** david_1 has quit IRC12:27
*** dgonzalez_1 has quit IRC12:27
*** mkoderer_1 has quit IRC12:27
*** alexchadin has joined #openstack-nova12:28
*** kumarmn has quit IRC12:30
*** BryanS68 has joined #openstack-nova12:30
openstackgerritChris Dent proposed openstack/nova master: [placement] Enable limiting GET /allocation_candidates  https://review.openstack.org/51352612:35
*** Guest97528 has quit IRC12:38
*** sree has joined #openstack-nova12:38
openstackgerritChason Chan proposed openstack/nova master: Fix the bug report link of API Guide  https://review.openstack.org/52766012:40
openstackgerritStephen Finucane proposed openstack/nova master: Add PCI NUMA policies  https://review.openstack.org/52747212:43
openstackgerritStephen Finucane proposed openstack/nova master: trivial: Modify signature of _filter_non_requested_pfs  https://review.openstack.org/52747312:43
*** psachin has quit IRC12:43
efried_cya_janstephenfin gibi Thanks for looking after those patches!12:44
*** Trident has joined #openstack-nova12:45
cdentefried_cya_jan: since you're here (naughty!) I should report that I've not had a chance to do anything with your stuff: I've got a nasty cold so mostly done nothing for the past couple of days. But when I surface, I will.12:45
efried_cya_jancdent I somehow caught your cold, which is a sucky way to start a vacation.  It looks like stephenfin has been addressing review comments and he & gibi have been rebasing stuff, so I think you're in the clear... for now :)12:46
cdentinternet germs are the worst12:46
gibiefried_cya_jan: o/12:46
*** ttsiouts has quit IRC12:47
openstackgerritRaghad Qutteneh proposed openstack/nova master: Allow force-delete to delete instance in task_state block_device_mapping  https://review.openstack.org/52795112:48
*** catintheroof has joined #openstack-nova12:49
*** mounika has quit IRC12:49
*** lucas-hungry is now known as lucasagomes12:50
*** chyka has joined #openstack-nova12:52
gibiefried_cya_jan: if you are here I have a question. Do I understand correctly that the GET allocation_candidate API does not handle nested RPs yet?12:53
*** tetsuro has quit IRC12:53
efried_cya_jangibi That's correct.  That's a major piece we need before we can slot in the granular resource requests work.12:53
gibiefried_cya_jan: OK, cool12:54
efried_cya_jangibi: jaypipes is on the hook for that.12:54
gibiOK12:55
*** damien_r has quit IRC12:55
*** markvoelker has joined #openstack-nova12:57
*** chyka has quit IRC12:57
*** jpena|lunch is now known as jpena13:00
*** mvenesio has joined #openstack-nova13:01
*** phuongnh has quit IRC13:03
*** yamamoto has joined #openstack-nova13:07
*** bkopilov has quit IRC13:07
*** damien_r has joined #openstack-nova13:08
*** kaisers has quit IRC13:09
*** artom has joined #openstack-nova13:10
*** kaisers has joined #openstack-nova13:15
*** claudiub has joined #openstack-nova13:18
*** mingyu has joined #openstack-nova13:19
*** damien_r has quit IRC13:20
*** damien_r has joined #openstack-nova13:21
*** liverpooler has joined #openstack-nova13:23
*** esberglu has joined #openstack-nova13:23
*** ralonsoh has quit IRC13:25
*** markvoelker has quit IRC13:28
*** markvoelker has joined #openstack-nova13:28
*** esberglu has quit IRC13:28
*** ralonsoh has joined #openstack-nova13:30
*** liverpooler has quit IRC13:35
*** liverpooler has joined #openstack-nova13:35
*** edmondsw has joined #openstack-nova13:37
*** abhishekk has joined #openstack-nova13:37
*** BryanS68 has quit IRC13:37
*** yamamoto has quit IRC13:41
*** cleong has joined #openstack-nova13:41
kashyapgibi: Thanks for the (non-null) pointer :-)13:48
*** yamamoto has joined #openstack-nova13:50
*** BryanS68 has joined #openstack-nova13:51
*** mriedem has joined #openstack-nova13:51
mriedemoh crap today is thursday huh13:52
mriedemgibi: can you run the meeting?13:52
gibimriedem: sure13:53
*** takashin has joined #openstack-nova13:53
mriedemthanks. i'll update the agenda quick.13:54
gibimriedem: thanks, that helps13:54
openstackgerritChris Dent proposed openstack/nova master: [placement] Enable limiting GET /allocation_candidates  https://review.openstack.org/51352613:54
*** pchavva has joined #openstack-nova13:54
*** yamamoto has quit IRC13:54
*** lyan has joined #openstack-nova13:57
mriedemdone https://wiki.openstack.org/wiki/Meetings/Nova#Agenda_for_next_meeting13:57
gibicool13:58
*** _ix has joined #openstack-nova13:58
*** rcernin has quit IRC13:59
gibinova meeting is about to start on @openstack-meeting14:00
cdentmriedem: just wanted to say: thanks for trying to keep it real in the cycle length thread.14:00
mriedemnp14:00
cdentI'm trying to gather my thoughts, but not having much success14:01
mriedemi think i will have to move on today from that thread14:02
cdentthat would be a bummer, you are uniquely positioned to, uh, keep it real14:02
*** _ix has quit IRC14:03
mriedemi don't have the energy to fight that battle, and it sounds like the majority of people (ops, users?) seem to like the idea. i think it will turn out to be more than what they bargained for (some things they'll lose, some things they won't get out of it)14:04
*** dosaboy has quit IRC14:04
mriedemwe'll do what we need to do for nova, if that means intermediate releases just to give us a deadline to work toward even if no one ever consumes it14:05
cdenti definitely think that if it happens hard interediate deadlines will be _very_ important14:06
edleafeso if you don't meet a deadline, would that mean waiting a year? Or what?14:07
* cdent hopes not14:08
edleafejust wondering how "dead" such a deadline would be14:09
cdentas you know, I think all deadlines are made up and thus fungible, but they are useful encouragement tools?14:10
*** dosaboy has joined #openstack-nova14:10
edleafeIMO, deadlines work as sticks, not carrots.14:11
cdentmaybe call them boundaries?14:12
*** ttsiouts has joined #openstack-nova14:13
mriedemjaypipes: i put this up for that dbdeadlock issue https://review.openstack.org/#/c/527836/ - not sure that's best, but it's a thing14:13
mriedemedleafe: i think we'd do intermediate releases so we could have more than 1 freeze in a year14:13
mriedemso either 1 or 2 intermediate releases (6 month or 4 month between them), and do our normal spec review, spec freeze, code churn, feature freeze, stability minute, then open up for new specs again14:14
mriedemi don't think we can freeze out specs for an entire year,14:14
mriedemand i don't think we can leave spec reviews open for an entire year and focus on actually getting things done14:14
*** yamamoto has joined #openstack-nova14:14
edleafemriedem: so would they be Rocky I, Rocky II, ...14:14
mriedemrocky IV is where we fight the mirantis guys14:15
mriedemus guys,14:15
mriedemand yous guys14:15
mriedemhaving said that, i don't know what we'd do for upgrade testing with intermediate releases14:16
mriedemdo you do queens->rocky.1, queens->rocky.2, or queens->rocky.1->rocky.2?14:16
mriedemtoday it would be the former because of how the CI system works on branches14:17
mriedemi assume we would use minor version bumps for intermediate releases14:17
edleafeyuck14:17
cdentblargh14:17
mriedemso rocky 1 would be 17.1.014:17
*** erlon has joined #openstack-nova14:17
mriedemif rocky GA was 18.0.014:17
mriedembut,14:17
mriedemthat gets tricky if we actually need to backport something to stable/pike which would normally be a minor update14:18
mriedemlike a schema migration14:18
mriedemlikely a question i'll need to ask about in "the thread"14:19
*** yamamoto has quit IRC14:19
*** kumarmn has joined #openstack-nova14:19
stephenfinefried_cya_jan: See your comments now. I'll address them shortly14:20
stephenfinFwiw, I'll keep trying to do the minimum possible amount to keep everyone happy and get it in (ignored most of my own comments/preferences for this reason)14:20
openstackgerritLee Yarwood proposed openstack/nova master: libvirt: Refactor encryptor attach and detach calls  https://review.openstack.org/46024314:20
openstackgerritLee Yarwood proposed openstack/nova master: libvirt: Introduce disk encryption config classes  https://review.openstack.org/46400814:20
openstackgerritLee Yarwood proposed openstack/nova master: WIP libvirt: Use QEMU's native LUKS support  https://review.openstack.org/52395814:20
*** kumarmn has quit IRC14:24
openstackgerritLee Yarwood proposed openstack/nova master: WIP libvirt: Use QEMU's native LUKS support  https://review.openstack.org/52395814:24
*** kumarmn has joined #openstack-nova14:24
*** gouthamr has joined #openstack-nova14:25
openstackgerritMerged openstack/nova master: [placement] add name to resource provider create error  https://review.openstack.org/52671014:26
openstackgerritMerged openstack/nova master: [placement] Add 'Location' parameters in API ref  https://review.openstack.org/52154114:26
openstackgerritMerged openstack/nova master: doc: link in some Sydney summit content  https://review.openstack.org/52658714:26
dansmithbauzas: need you to look at https://review.openstack.org/#/c/527799/214:27
*** mdnadeem has quit IRC14:28
*** kumarmn has quit IRC14:29
takashinmriedem: The compute API microversion 2.57 was added in "Deprecate file injection" https://review.openstack.org/#/c/522027/ .14:30
takashinmriedem: But I cannot find a patch for microversion 2.57 in python-novaclient. Would you submit the patch?14:30
*** smatzek has quit IRC14:30
takashinmriedem: Are you around?14:31
jaypipesmriedem: ok, looking at it now.14:33
*** mlavalle has joined #openstack-nova14:34
*** baoli has joined #openstack-nova14:35
mriedemtakashin: i plan on working on that novaclient patch today14:37
mriedemtakashin: didn't get the time yesterday - too much other stuff going on14:37
takashinmriedem: okay. Thanks.14:38
*** yangyapeng has quit IRC14:38
*** awaugama has joined #openstack-nova14:39
*** mdnadeem has joined #openstack-nova14:41
*** sapd__ has quit IRC14:41
*** andreas_s has joined #openstack-nova14:41
*** yikun_ has joined #openstack-nova14:42
jrollmriedem: we should talk about releases/upgrades at PTG if this goes through, ironic has some experience with that14:43
*** sapcc-bot has quit IRC14:45
*** mkoderer_ has quit IRC14:45
*** carthaca_ has quit IRC14:45
*** dgonzalez_ has quit IRC14:45
*** tpatzig_1 has quit IRC14:45
*** david_4 has quit IRC14:45
*** dgonzalez_ has joined #openstack-nova14:45
*** mkoderer_ has joined #openstack-nova14:45
*** mkoderer_1 has joined #openstack-nova14:45
*** tpatzig_ has joined #openstack-nova14:45
*** carthaca_ has joined #openstack-nova14:45
*** sapcc-bot has joined #openstack-nova14:45
*** dgonzalez_1 has joined #openstack-nova14:45
*** david_5 has joined #openstack-nova14:45
*** tpatzig_2 has joined #openstack-nova14:45
*** alexchadin has quit IRC14:46
*** andreas_s has quit IRC14:46
cdentjroll: are you back on the scene?14:46
cdentas in: gonna be in dublin?14:46
mriedemjroll: yeah i was going to ask in the thread to get the experience of other teams already doing it14:46
jrollcdent: waiting for approval but hopefully yes14:47
jrollmriedem: ++14:47
*** david_5 has quit IRC14:47
*** carthaca_ has quit IRC14:47
*** mkoderer_1 has quit IRC14:47
*** dgonzalez_1 has quit IRC14:47
*** sapcc-bot has quit IRC14:47
*** tpatzig_ has quit IRC14:47
*** tpatzig_2 has quit IRC14:47
*** mkoderer_ has quit IRC14:47
*** dgonzalez_ has quit IRC14:47
*** carthaca_ has joined #openstack-nova14:48
*** tpatzig_ has joined #openstack-nova14:48
*** 7ITABX779 has joined #openstack-nova14:48
*** mkoderer_ has joined #openstack-nova14:48
*** sapcc-bot has joined #openstack-nova14:48
*** 18WAABEXO has joined #openstack-nova14:48
*** dgonzalez_2 has joined #openstack-nova14:48
*** tpatzig_1 has joined #openstack-nova14:48
*** tpatzig_3 has joined #openstack-nova14:48
*** mkoderer_1 has joined #openstack-nova14:48
*** david_6 has joined #openstack-nova14:48
*** mkoderer_2 has joined #openstack-nova14:48
mriedemi didn't want to ask jroll that question14:49
*** tpatzig_ has quit IRC14:49
*** mkoderer_ has quit IRC14:49
*** 18WAABEXO has quit IRC14:49
mriedemsomething something "don't ask a question you don't already know the answer to"14:50
*** lbragstad has joined #openstack-nova14:50
jrollyou don't wanna know anyway14:50
*** xyang1 has joined #openstack-nova14:50
*** tpatzig_ has joined #openstack-nova14:50
*** dgonzalez_ has joined #openstack-nova14:50
*** mkoderer_ has joined #openstack-nova14:50
*** mdnadeem has quit IRC14:50
*** burt has joined #openstack-nova14:50
*** mdnadeem has joined #openstack-nova14:51
*** mkoderer_ has quit IRC14:52
*** tpatzig_ has quit IRC14:52
*** dgonzalez_ has quit IRC14:52
*** abhishekk has quit IRC14:53
*** tidwellr has joined #openstack-nova14:54
*** esberglu has joined #openstack-nova14:56
*** yangyapeng has joined #openstack-nova14:56
*** smatzek has joined #openstack-nova14:58
*** smatzek has quit IRC14:59
*** jmlowe_ has joined #openstack-nova14:59
*** smatzek has joined #openstack-nova14:59
*** smatzek has quit IRC14:59
*** smatzek has joined #openstack-nova15:00
*** jmlowe has quit IRC15:00
stephenfinmdbooth: So if I'm understanding this correctly, this fixes and issue we'd only see when using microversion < 2.25? https://review.openstack.org/#/c/524681/15:01
*** marst has joined #openstack-nova15:01
*** takashin has left #openstack-nova15:01
*** yangyapeng has quit IRC15:02
*** jmlowe has joined #openstack-nova15:04
*** jmlowe_ has quit IRC15:06
*** rodolof has joined #openstack-nova15:09
cdentzounds, an email thread has drawn dansmith, this is a banner day15:09
dansmithcdent: I really really wanted to ignore this one even more than usual15:09
cdentdansmith: I'm glad you didn't.15:10
*** litao__ has quit IRC15:10
*** felipemonteiro__ has joined #openstack-nova15:13
*** yangyapeng has joined #openstack-nova15:14
openstackgerritTakashi NATSUME proposed openstack/nova master: [placement] Separate API schemas (usage)  https://review.openstack.org/52060315:15
openstackgerritJianghua Wang proposed openstack/nova master: XenAPI: create vGPU for instance  https://review.openstack.org/51689915:16
openstackgerritTakashi NATSUME proposed openstack/nova master: [placement] Separate API schemas (trait)  https://review.openstack.org/52060515:16
openstackgerritTakashi NATSUME proposed openstack/nova master: [placement] Separate API schemas (aggregate)  https://review.openstack.org/52060815:16
*** r-daneel has joined #openstack-nova15:16
dansmithapparently I'm just replying to edleafe's emails today15:16
openstackgerritTakashi NATSUME proposed openstack/nova master: [placement] Add functional tests for resource class API  https://review.openstack.org/52450615:16
mriedemsdague: bauzas: can you take a look at these ocata backports? https://review.openstack.org/#/q/topic:bug/1732947+status:open+branch:stable/ocata - holding up newton backports for the same changes which is holding up newton eol15:16
openstackgerritTakashi NATSUME proposed openstack/nova master: [placement] Separate API schemas (inventory)  https://review.openstack.org/52061315:17
bauzasmriedem: roger, captain15:17
mriedembauzas: sdague: and this ocata one https://review.openstack.org/#/c/526426/15:17
mriedemsame story15:17
*** felipemonteiro__ has quit IRC15:17
mriedemtonyb: i sent an email to the dev list about what i think needs to happen to get nova to newton-eol15:17
*** yangyapeng has quit IRC15:18
jianghuaw_hi bauzas, need your help to look at this patch: https://review.openstack.org/#/c/516899/15:19
mriedemtonyb: http://lists.openstack.org/pipermail/openstack-dev/2017-December/125592.html15:19
*** itlinux has joined #openstack-nova15:19
*** itlinux has quit IRC15:22
*** amotoki has quit IRC15:22
lyarwoodmriedem: did we also want to fix https://review.openstack.org/#/c/526426/1 in newton?15:24
*** felipemonteiro__ has joined #openstack-nova15:25
*** tosky has left #openstack-nova15:25
lyarwoodmriedem: ah nvm, you listed the stable/newton version in that query, ignore me15:25
*** Alex_Staf has quit IRC15:26
*** felipemonteiro_ has joined #openstack-nova15:26
*** links has quit IRC15:26
* edleafe is happy to be the dansmith email lure15:28
cdentshiny pretty edleafe15:29
*** Sanatics has joined #openstack-nova15:29
*** felipemonteiro__ has quit IRC15:30
*** moshele has quit IRC15:30
mriedemi did think about the impact to the deprecation policy this morning when i got up, glad dan mentioned that15:32
mriedemwe will be supporting turds for longer, and hamstrung by said turds potentially15:32
mriedemi'm glad we said rocky is the target to drop nova-net and cellsv115:32
cdentthe bits with "support both ways of doing this thing" have to stay around longer too, which is icky15:33
cdentbut then again, apparently the style is something the ff-upgrade people don't like?15:34
*** yangyapeng has joined #openstack-nova15:36
dansmithmriedem: so my goal on that reqspec fix is to try to converse with bauzas about it today to make sure he's okay with it and/or there's not something huge we're missing on it15:39
mriedemack15:39
dansmithmriedem: and then hand over the patch and backports to artom, who unfortunately is the recipient of many such turdish patches of mine right before I leave on vacay :P15:39
artomSeriously, I'm like dansmith's crap scapegaot15:40
artomCrapgoat15:40
mriedemdansmith: i'm personally not comfortable with that one going to newton right before we eol15:40
dansmithartom: as you will henceforth be known.15:40
mriedemi think the column alter is fine15:40
mriedemfor newton15:40
*** yangyapeng has quit IRC15:40
mriedemi'm worried too much about reqspec/group side effects15:40
dansmithmriedem: well, I'd rather just not backport any of it to newton myself,15:40
dansmithmriedem: and don't really want to backport the migration at all, but that's fine15:41
dansmithwe will, of course, but newton needs to go at some point15:41
artomI agree that the migration might have more side effects than the load by UUID patch15:41
mriedemi know but i love it so15:41
artomEr, less15:41
dansmithand this is something that has been this way since the release and which we've only _just_ heard about15:41
artomSo if we're going to backport something, it might actually be safer to do just the migration15:41
mriedemyeah, because people are just now upgraded15:41
mriedem*upgrading15:41
mriedemwell, people with big ass server groups apparently :)15:42
artomAlso good point - there's like a 1-2 year lag between upstream and what's actually running15:42
dansmithwell, there are definitely people already on newton and beyond, but I understand that reasoning15:42
dansmithmriedem: fwiw, there were only four such instances in a database of tens of thousands that were affected here15:43
dansmithmriedem: and it seemed like it was a very specific case, which might have been testing and/or a script run amok15:43
dansmithjust for the data point15:43
mriedemok15:43
dansmiththe fact that we're crystallizing known-bad server groups in the database at creation time is almost more concerning to me than the overrun case15:44
*** edand__ has quit IRC15:44
artomKnown-bad?15:44
dansmithartom: yeah, so we're capturing the server group (and specifically its members) at the time in which we create/write the reqspec for an instance15:46
mriedemi'm fully on board with fixing that15:46
dansmithso lets say you do this: nova boot --group foo; nova boot --group foo15:46
artomAh, and some of those can get deleted, so we're left with a group with non existing members15:46
dansmiththe first one has an InstanceGroup with .members=[justme]15:46
dansmiththe second one is complete15:46
dansmiththere is no reason we should be persisting that and then ever using it ever, but we are15:47
dansmithartom: either stale members, or incomplete, depending on when you snapshotted it15:47
*** armax has joined #openstack-nova15:47
artomI'm trying to think if anyone could be relying of this behaviour for whatever reason15:47
artomThis is used in stuff like the group (anti)affinity filters, right?15:48
cfriesendansmith: mriedem: for what it's worth, for reliability you have to update the server group members before rescheduling anyways, otherwise you're working off stale data15:48
gibimriedem: could you check back to https://review.openstack.org/#/c/527444/ I start seeing the related bug appearing more and more on the gate15:48
dansmithcfriesen: right15:48
mriedemgibi: yeah15:49
gibimriedem: thanks15:49
* mriedem is watching the -tc chatter15:50
bauzasdansmith: hola, what's the point about the request spec ?15:51
artomSo are we coming to the consensus that we fix the stale group membership in master, but to play it safe only backport the db migration?15:51
dansmithbauzas: we're persisting InstanceGroup in RequestSpec with stale members for some reason.. why is that?15:51
artomThough I suppose it's conceivable the db migration might not fix stuff either, if mediumtext is too small still15:51
dansmithbauzas: this came because someone with a very large number of members in that group (at the time of that snapshot) can overrun a TEXT field15:52
dansmithartom: mediumtext is very large I think15:52
dansmithlike 16mb or something15:52
artomSeriously?15:52
dansmithyup15:52
artomWait, so if we migrate a DB with like, thousands of rows, we're adding how much space requirement?15:52
artomAnd how long will such a thing take?15:52
dansmithartom: hence me not wanting to do it unnecessarily :)15:53
dansmithwe rewrite that whole table15:53
bauzasdansmith: mmmmm15:53
bauzasdansmith: AFAIK, we lazy-load the members group15:53
bauzasbut I could be wrong15:53
cfriesenpostgres has unlimited text fields.15:53
dansmithbauzas: no, we write them all into requestspec15:53
artomYeha, so my SQL knowledge is fuzzy, but if we increase it to 16 MB we're essentially pre-allocating 16 MB for every row, correct?15:53
dansmithbauzas: and explode the database15:53
dansmithartom: yeah15:54
dansmithartom: which is why I'd rather fix the problem, because 64kb should be enough I think15:54
bauzasseriously? :(15:54
dansmithbauzas: https://review.openstack.org/#/c/527799/315:54
bauzaslooking15:54
*** yangyapeng has joined #openstack-nova15:56
cfriesendansmith: I'm seeing docs saying that text colums are stored separately from the row, so would it really preallocate 16MB for each row?15:56
*** Sanatics has quit IRC15:56
cfriesen(for mysql)15:56
dansmithcfriesen: oh? I didn't think that's how it worked15:57
*** vivsoni__ has joined #openstack-nova15:57
*** Alex_Staf has joined #openstack-nova15:57
cfriesenI'm *not* a mysql guy, so do your own research15:57
artomWhere's Monty when you need him15:58
dansmithit doesn't matter that much anyway, as I think fixing the problem will fix the problem15:58
dansmithartom: jaypipes knows a thing or two :)15:58
dansmithit only adds fuel to the discussion though, we need to fix the real problem15:59
artomI think we're just afraid fixing the problem might create another one15:59
artomLike sending rebuild to the scheduler :)15:59
cfriesendansmith: agreed...I don't see any reason to persist the group members at time of booting15:59
dansmithartom: well, that's why I want bauzas to look at it closely, but also, I think my fix is pretty straightforward16:00
artomdansmith, yeah16:00
*** catintheroof has quit IRC16:00
cfriesenartom: you need to check the group members right before scheduling.16:00
dansmithit didn't even require functional test change16:00
*** catintheroof has joined #openstack-nova16:00
*** yangyapeng has quit IRC16:01
artomI suppose there's no way for a request_spec to end up with an empty group in code that expects something there16:01
mriedemartom: umm16:01
mriedemit's request spec and groups and scheduler stuff,16:01
artomdansmith, you know objects better than I do, other than _from_db_object we don't generate it in any other way?16:01
mriedemthere is totally a way to fuck this up16:01
mriedemw/o knowing it16:01
artommriedem, oh I agree16:02
artomWhich is why I'm thinking about it :)16:02
bauzasmriedem: dansmith: FWIW, just trying to look at earlier code16:02
dansmithmriedem: nobody asserted otherwise I don't think :)16:02
openstackgerritStephen Finucane proposed openstack/nova master: SchedulerReportClient._get_providers_in_aggregates  https://review.openstack.org/52109716:02
openstackgerritStephen Finucane proposed openstack/nova master: Traits ops on ProviderTree  https://review.openstack.org/52160516:02
openstackgerritStephen Finucane proposed openstack/nova master: Aggregate ops on ProviderTree  https://review.openstack.org/52621616:02
openstackgerritStephen Finucane proposed openstack/nova master: Move aggregates from report client to ProviderTree  https://review.openstack.org/52168516:02
openstackgerritStephen Finucane proposed openstack/nova master: Track provider traits in report client  https://review.openstack.org/52168616:02
openstackgerritStephen Finucane proposed openstack/nova master: Track associated sharing RPs in report client  https://review.openstack.org/52653916:02
openstackgerritStephen Finucane proposed openstack/nova master: Raise on API errors getting aggregates/traits  https://review.openstack.org/52654016:02
openstackgerritStephen Finucane proposed openstack/nova master: ProviderTree.populate_from_iterable  https://review.openstack.org/52075616:02
openstackgerritStephen Finucane proposed openstack/nova master: Track tree-associated providers in report client  https://review.openstack.org/52654116:02
openstackgerritStephen Finucane proposed openstack/nova master: WIP: Scheduler[Report]Client.get_provider_tree  https://review.openstack.org/52109816:02
openstackgerritStephen Finucane proposed openstack/nova master: WIP: ComputeDriver.update_provider_tree()  https://review.openstack.org/52118716:02
openstackgerritStephen Finucane proposed openstack/nova master: WIP: Use update_provider_tree from resource tracker  https://review.openstack.org/52024616:02
*** tidwellr has quit IRC16:02
mriedemdansmith: maybe i misinterpreted "i think it basically boils down to (1) foundatoin doesn't want to pay or plan 2 ptgs, (2) people don't want to upgrade, or feel the pressure to upgrade, every 6 months and (3) people that are now going to multiple events (kubecon+) don't want to feel stretched"16:02
mriedemoops16:02
mriedem"artom: I suppose there's no way for a request_spec to end up with an empty group in code that expects something there"16:02
artommriedem, yeah, "suppose" being the key word there ;)16:03
artomSo, what if we move the loading by uuid into instance group?16:03
artomThat way there's definitely no way to end up with an empty one16:03
dansmithmriedem: I think that just meant group and members is not always present, can be null, nothing is depending on that being set to something specific that we might mess up16:03
dansmithwhich is true I think16:03
*** alee has joined #openstack-nova16:03
bauzasdansmith: oh shit, 'members' is not lazy-loadable, 'hosts' is16:03
artomdansmith, yep, you understood that16:03
bauzashence my confusion16:04
*** alee has left #openstack-nova16:04
bauzasdansmith: and since we persist the group object in the DB, yeah I understand the issue16:04
* bauzas facepalms16:04
dansmithartom: well, not sure what that matters, since you didn't snip it out before we wrote it into reqspec anyway16:04
dansmithbut yeah16:04
dansmither16:04
dansmiththat was for bauzas:16:04
* dansmith is watching too many convos16:04
openstackgerritStephen Finucane proposed openstack/nova master: Raise on API errors getting aggregates/traits  https://review.openstack.org/52654016:05
openstackgerritStephen Finucane proposed openstack/nova master: ProviderTree.populate_from_iterable  https://review.openstack.org/52075616:05
openstackgerritStephen Finucane proposed openstack/nova master: Track tree-associated providers in report client  https://review.openstack.org/52654116:05
openstackgerritStephen Finucane proposed openstack/nova master: WIP: Scheduler[Report]Client.get_provider_tree  https://review.openstack.org/52109816:05
openstackgerritStephen Finucane proposed openstack/nova master: WIP: ComputeDriver.update_provider_tree()  https://review.openstack.org/52118716:05
openstackgerritStephen Finucane proposed openstack/nova master: WIP: Use update_provider_tree from resource tracker  https://review.openstack.org/52024616:05
*** catintheroof has quit IRC16:05
artomdansmith, so what if we move the loading by uuid down to instance_group? That way we make sure we can never be in a situation with empty members in code that expects them...16:06
artomRight?16:06
dansmithartom: meaning make it lazy-loadable?16:06
artomdansmith, would it make it lazy-loadable? Not necessarily...16:07
artomActually16:07
artomYeah it would16:07
*** jdillaman has joined #openstack-nova16:07
dansmithartom: so that won't work if we end up lazy-loading on the compute, because it'll be in a cell and can't get that info later16:08
dansmithartom: we need to pre-load it16:08
dansmithwhich is kinda the point of reqspec: to pass along everything the stuff downstream needs in order to move the instance16:08
*** klindgren has joined #openstack-nova16:08
artomOh right, cells16:09
cfriesendansmith: would your proposed change to update the instance_group when reading the request spec from DB cover the live-migration case as well?  We had a local bugfix to cover that case as part of various affinity/antiaffinity race fixes that haven't made it upstream.16:10
dansmithcfriesen: if I know what you're asking, then it should yeah16:10
dansmithcfriesen: when we load the reqspec in the api before kicking off the live migration, it will have a fresh and accurate set of group members passed down to anything that gets the reqspec in the process16:11
*** gbarros has quit IRC16:11
cfriesendansmith: I'm looking at _find_destination() in live_migrate.py.  It calls request_spec = self._get_request_spec_for_select_destinations()16:12
cfriesendansmith: okay, so looks like that request spec is normally pulled from the DB in compute.api.API.live_migrate(), so we should be good.16:15
dansmithcfriesen: it kinda has to be since it's in the api db16:16
*** yangyapeng has joined #openstack-nova16:17
bauzasdansmith: +2 on the fix16:17
artomSemi-related thought16:18
artomWill the code actually fix existing cases?16:18
dansmithartom: it will, because:16:19
bauzasartom: you mean upstream bugs :p16:19
artomAh, yeah, it sets members to None when saving16:19
dansmithartom: if you are under the limit now, then you are good.. if you're over, this will make you fit now16:19
artombauzas, no, I meant our downstream bug16:19
bauzasexactly16:19
dansmithartom: and, the api will immediately stop reading the stale group after this fix, for reqspecs with stale data from before the fix16:19
bauzasartom: I do understand but we're upstream :)16:19
* bauzas is schizophrenic16:19
artombauzas, well, yes, but it's still relevant to talk about whether we're actually fixing the existing problem, or just preventing it in the future16:20
bauzasit'll pull fresh data indeed16:20
dansmiththat's why I did it that way16:20
jaypipescfriesen, artom: TEXT/MEDIUMTEXT/VARCHAR fields are only pre-allocated at full size when the field is used in a GROUP BY or ORDER BY expression. Otherwise, only a pointer to the data in the data file is used during the projection phase of the SELECT expression (when the rows are actually sent to the client)16:20
dansmithbecause we have to account for all the broken data in the db now16:20
jaypipescfriesen, artom: this is for MySQL, not PostgreSQL16:21
bauzasdansmith: and you did it the right way, thanks :)16:21
dansmithjaypipes: we're talking about allocation on disk, not memory16:21
*** yangyapeng has quit IRC16:21
*** itlinux has joined #openstack-nova16:21
dansmithjaypipes: like whether 1 byte in a MEDIUMTEXT takes up 16mb on disk16:21
*** klindgren has quit IRC16:21
jaypipesdansmith: there's almost no difference between a TEXT, MEDIUMTEXT or VARCHAR(8000) w.r.t. on-disk storage requirements or layout.16:22
jaypipesdansmith: for InnoDB that is.16:22
bauzasdansmith: FWIW, I fixed the problem way in the past for hosts https://github.com/openstack/nova/commit/c48c1098cbcd7a9c7980d5fbe4668c38b16767f516:22
dansmithjaypipes: okay16:22
bauzasdansmith: but I didn't thought it would be a huge problem for members, given not a lot of operators use a long list of them16:22
jaypipesdansmith: no, for all those field types, InnoDB stores a small amount of data in the data page itself and makes room (when needed) in separate extents by providing a pointer to that extent/page from the main data page.16:23
jaypipesdansmith: the only thing that can be performance-wise a bad thing is if the size of those fields changes often (which isn't the case for these fields in Nova's DB)16:23
artomSo for MySQL/InnoDB a migration to mediumtext would actually make sense...16:23
jaypipesartom: for InnoDB, yes. for NDB, no...16:23
jaypipesartom: but that's a different problem entirely :)16:24
artomDo we support NDB?16:24
jaypipesartom: what is the current column type for this field you're talking about?16:24
bauzasmriedem: jaypipes: https://review.openstack.org/#/c/527799/ looks legit to me16:24
dansmithjaypipes: and what is the cost of resizing to medium from text?16:24
jaypipesdansmith: virtually zero.16:25
artomjaypipes, dansmith answered that for me16:25
dansmithjaypipes: awesome, then we should do that too16:25
dansmithartom, mriedem ^16:25
dansmithstill need the real fix, but sounds like a backport is too good to pass up16:25
dansmithof the migration I mean16:25
artomdansmith, that assumes we only support InnoDB16:25
jaypipesdansmith: I don't recommend mediumtext anyway. just set it to TEXT16:25
dansmithjaypipes: text isn't big enough16:25
artomWhich... is it the case?16:25
artomSurely some people are running Galera16:26
dansmithartom: I think everyone would be running innodb in prod, AFAIK16:26
jaypipesdansmith: TEXT > MEDIUMTEXT16:26
dansmithjaypipes: um I don't think so16:26
jaypipessorry, yeah...16:26
dansmithhttps://stackoverflow.com/questions/13932750/tinytext-text-mediumtext-and-longtext-maximum-storage-sizes16:26
jaypipeswas thinking LONGTEXT16:26
bauzasyeah LONGTEXT no ?16:27
artomOh, Galera uses InnoDB16:27
bauzasTEXT to LONGTEXT I mean16:27
jaypipesdansmith, artom: what is the specific field we are talking about here?16:28
mriedemdansmith: jaypipes: cool - thanks16:28
jaypipesdansmith: is this metadata value?16:28
* mriedem is in 4 places at once16:28
*** kumarmn has joined #openstack-nova16:28
dansmithjaypipes: request_specs.spec16:28
jaypipeshmm.16:28
dansmithjaypipes: it's a text we serialize the object into, and because we store instance_group.members, which can be huge,16:28
artomErr, instance_group surely?16:28
dansmithwe can generate more than 64kb of data16:28
*** itlinux has quit IRC16:28
dansmithartom: instance_group is in the spec16:28
bauzaswe serialize all at a whole16:29
artomAh, doh16:29
jaypipesdansmith: k. then I'd recommend changing it to MEDIUMTEXT from TEXT.16:29
dansmithjaypipes: that's what we're doing :)16:29
jaypipesdansmith: ok, sorry for interrupting then :)16:29
dansmithjaypipes: waaaaayyyy ahead of you bro16:29
dansmithjaypipes: and when I say "we" I mean "artom"16:29
jaypipesk16:29
artomI, wha?16:30
jaypipesyet another reason why storing denormalized data in a TEXT-y field is a bad idea...16:30
artomI've barely had breakfast and it's 11:30, I'm not ahead of anything16:30
openstackgerritMerged openstack/nova master: Fix the bug report link of API Guide  https://review.openstack.org/52766016:30
*** gyee has joined #openstack-nova16:30
dansmithjaypipes: well, it has its place, IMHO, and the complexity required to store a reqspec normalized is not worth it since it's a snapshot16:32
bauzasI remember we had a thought on that with alaski16:32
dansmithreqspec is an archive snapshot really, which is fine for this, we just need to not snapshot an infinite amount of stale data into it ;)16:33
bauzasand we said it was better to just store the whole object so we would totally decouple what we store from what we expose16:33
bauzasdansmith: yeah, I still agree on it tbc16:33
jaypipesdansmith: +W'd that patch16:34
bauzaswe just expose the instance UUID for querying reasons that's it16:34
dansmithjaypipes: thanks16:34
bauzasdansmith: is there another patch related to that about a DB schema modification ?16:34
dansmithjaypipes: and thanks for being my fleshy mysql manual :P16:34
dansmithbauzas: young artom will be presenting it forthwith16:34
bauzascool16:35
jaypipesdansmith: sounds kinky.16:35
*** hoonetorg has quit IRC16:35
bauzasoh man, I wish I could "OK jaypipes" on my phone16:35
* dansmith makes bedroom eyes at jaypipes 16:35
jaypipeslol16:35
artomjaypipes, you got off easy, I'm his young turdgoat16:35
cdent"fleshy mysql manual" is way too visual16:35
jaypipesartom: yes, I read that. very flattering indeed. ;)16:35
dansmithcdent: you're welcome16:35
cdentclears my head nicely, thank you dansmith16:35
*** Apoorva has joined #openstack-nova16:36
*** Alex_Staf has quit IRC16:37
artomdansmith, so I'll handle the backports - who's proposing the DB migration?16:37
*** yangyapeng has joined #openstack-nova16:37
dansmithartom: you are16:38
artomOr are we satisfied that the code alone is enough?16:38
dansmithI mean, I can do the migration if you really don't want to, but.. figured you'd want all the glory and praise that will come with it16:38
artomIf I want glory and praise I'll become a murderous dictator16:39
mriedemartom: there is a db migratoin for extending the size of build_requests.instance16:39
mriedemartom: just copy ^16:39
mriedemsuper easy16:39
mriedemexact same issue16:39
dansmithartom: an important point on the resume of a murderous dictator is "did a schema migration once"16:39
*** moshele has joined #openstack-nova16:39
artomFor now I just want to revel turdgoatery16:39
mriedemffs you should probably give me co-author now because i even told you16:39
artomWhat's the magic word?16:40
mriedemtrump16:40
artomDo we support pgsql as well?16:41
*** yangyapeng has quit IRC16:41
artomI remember there was a whole thing about dropping it from the gate a while back16:42
bauzasI'm a 20% time person this week, funny16:42
*** andreas_s has joined #openstack-nova16:42
*** chyka has joined #openstack-nova16:43
dansmithartom: we don't anymore16:43
dansmithartom: and if you could figure out some way to break pgsql in this process, I'd sure appreciate it16:43
dansmithif for no other reason than just to cause pain for that smug cfriesen guy16:43
bauzasartom: we are not gating pgsql, hence not supporting it16:44
bauzasI don't know if other projects still gate pg, but we very made it clear that pg is not supported anyway in nova16:44
*** yamahata has joined #openstack-nova16:44
*** yamamoto has joined #openstack-nova16:46
*** andreas_s has quit IRC16:47
*** _ix has joined #openstack-nova16:47
*** hoonetorg has joined #openstack-nova16:48
*** tbachman has quit IRC16:49
*** slaweq has joined #openstack-nova16:51
*** slaweq has left #openstack-nova16:51
*** edand__ has joined #openstack-nova16:51
*** tssurya has quit IRC16:52
*** kumarmn_ has joined #openstack-nova16:53
*** yamamoto has quit IRC16:54
*** kumarmn has quit IRC16:56
*** xnox has joined #openstack-nova16:57
xnoxHello!16:57
xnoxis there nova quota for CPU cores.... per-arch/per-machinetype?16:57
*** yangyapeng has joined #openstack-nova16:58
*** derekh has quit IRC16:58
openstackgerritStephen Finucane proposed openstack/nova master: doc: Document PCI NUMA affinity policy  https://review.openstack.org/52801116:58
*** sshwarts has quit IRC16:59
*** klindgren has joined #openstack-nova16:59
jaypipesxnox: no. quota is for VCPU, that's it.17:00
openstackgerritArtom Lifshitz proposed openstack/nova master: Make request_spec.spec MediumText  https://review.openstack.org/52801217:00
artommriedem, ^^ there you go17:00
mriedemartom: dansmith: yes we support postgres17:01
dansmithmriedem: we don't test it right?17:01
*** tbachman has joined #openstack-nova17:01
mriedemwe don't actively gate on it, we have an experimental job that runs it,17:01
xnoxjaypipes, ack, thanks.17:01
dansmithoh I didn't realize we still had that17:01
mriedembut i consider support == if we break it and there is a patch, we don't block it17:01
dansmithmeh17:01
mriedemffs we don't gate on ceph either17:01
*** klindgren has quit IRC17:01
dansmithI call that not supporting it17:01
mriedemso i guess we don't support ceph17:01
*** xyang1 has quit IRC17:01
stephenfinbauzas: Added the docs promised for the PCI NUMA policy https://review.openstack.org/52801117:02
stephenfinGoing to add user docs shortly17:02
stephenfinThough I really should do the real-time ones first. I've them only in blog form right now17:02
artomAnd I totally forgot tests, didn't I/17:02
*** yangyapeng has quit IRC17:03
artomAnd a lot of other stuff, actually:(17:03
*** penick has joined #openstack-nova17:04
bauzasstephenfin: well, the biggest concern I had was indeed https://review.openstack.org/#/c/527472/3/nova/virt/hardware.py@119817:05
bauzasI know we already do this17:05
bauzasbut looks like an operator pain17:05
bauzashonestly, I need to look at the spec's comments17:06
*** edmondsw has quit IRC17:06
stephenfinIf we fix that though, isn't it something we should fix in one fell swoop?17:06
stephenfini.e. all potential conflicts should throw that the same kind of warning17:06
stephenfin*throw the17:06
*** edmondsw has joined #openstack-nova17:06
stephenfinYeah, spec discussion would probably be best. /me goes to find relevant comments17:06
*** tidwellr has joined #openstack-nova17:07
bauzasstephenfin: I just found those https://review.openstack.org/#/c/361140/6..7/specs/ocata/approved/share-pci-between-numa-nodes.rst17:10
*** fragatin_ has joined #openstack-nova17:10
*** edmondsw_ has joined #openstack-nova17:10
bauzasstephenfin: oops https://review.openstack.org/#/c/361140/6/specs/ocata/approved/share-pci-between-numa-nodes.rst@6917:10
bauzashonestly, we're late in the cycle so your concern is valid17:10
bauzasbut I do feel it's a PITA for operators17:11
stephenfinbauzas: That's the one. There's also this discussion on the need for both flavor extra spec and image metadata https://review.openstack.org/#/c/361140/19/specs/pike/approved/share-pci-between-numa-nodes.rst@5917:11
*** edmondsw has quit IRC17:11
stephenfin)note that I mixed up the use of flavors extra specs vs image metadata - it's *operators* that would set the former and not user)17:11
stephenfinbauzas: Would a TODO to resolve that in a future effort be reasonable? I'm pretty sure I can revive the spec I talked about17:12
bauzasstephenfin: well, PS19 convo is strictier than the one I found17:12
openstackgerritClaudiu Belu proposed openstack/nova master: tests: fixes mock autospec usage  https://review.openstack.org/44750517:12
bauzasstephenfin: it's more about whether we need to use image metadata17:12
stephenfinIndeed. Different argument but in a similar area17:12
bauzashere, I still think it's understandable to do such things because users need a way to ask for a policy17:12
stephenfinHere's the spec I was talking about https://review.openstack.org/#/c/382382/17:12
stephenfinWe can use that determine what we want to do going forward17:13
bauzasstephenfin: documenting the flaws of using flavor metadata for such things is indeed a necessary thing to do17:13
bauzasI'd recommend operators to *not* use flavor metadata if they don't know their users17:14
*** lpetrut has quit IRC17:14
*** edmondsw_ has quit IRC17:14
stephenfinI'd also like to use something like JSONSchema (though maybe not that) to validate flavour extra specs and determine what things can't be used together17:14
stephenfin...with exceptions for custom flavour properties17:15
bauzasin my mind, flavors take precedence over images17:15
bauzasbecause they are admin-driven17:15
bauzasso, your call, man17:15
stephenfinbauzas: Aye, that's how I see them too17:15
stephenfinBut I don't want a user to think "I've got a strictly-pinned NUMA device" when they actually don't17:16
stephenfincos the operator set the policy to 'preferred'17:16
bauzasright17:16
stephenfinfor resource usage reasons or whatever17:16
bauzasbut users won't know anyway17:16
stephenfinThey won't?17:17
bauzashow could they ?17:17
openstackgerritMerged openstack/nova master: Fix disk size during live migration with disk over-commit  https://review.openstack.org/52468117:17
openstackgerritMerged openstack/nova stable/ocata: Only query BDMs once in API during rebuild  https://review.openstack.org/52566517:17
bauzasdo we expose the topology to the users ? hell no, of course17:17
* stephenfin assumed Forbidden-class exceptions would get bubbled up via the API17:17
bauzasah, that17:17
*** alex_xu has quit IRC17:17
bauzaswell, two cases, and that requires looking at code17:18
*** alex_xu has joined #openstack-nova17:18
bauzas#1 either we fail and we reschedule to another host17:18
bauzaswhich could eventually fail too17:18
bauzas#2 or we make it full stop17:18
bauzasand then we error out17:18
bauzaseither way, the user experience isn't terrible17:18
stephenfinOh, actually, #1 would actually happen17:18
*** yangyapeng has joined #openstack-nova17:18
stephenfinbecause the NUMATopologyFilter (eventually) calls those functions17:19
bauzasisn't that called by the compute service?17:19
stephenfinThat too, yes17:19
stephenfinIt's this call17:19
stephenfinhttps://github.com/openstack/nova/blob/master/nova/scheduler/filters/numa_topology_filter.py#L94-L9817:20
bauzasman, I got 3 days of legal and accouting training, but I don't know what hurts more : that or discussing PCI scheduling17:20
stephenfinHahaha17:20
mriedemjaypipes: is an alter statement on a column type change idempotent? https://review.openstack.org/#/c/528012/1/nova/db/sqlalchemy/api_migrations/migrate_repo/versions/051_request_spec_spec_mediumtext.py17:21
mriedemif it's already that type?17:21
stephenfinDon't worry - jaypipes and co are going to save us with nested-rps17:21
stephenfinAnd if they don't - guaranteed job security :P17:21
bauzashaving a job security doesn't mean brain sanity17:21
mriedemgd 11:30 already17:22
bauzas11:22 technically17:22
mriedemnot in minnesota17:22
*** edmondsw has joined #openstack-nova17:22
*** AlexeyAbashkin has quit IRC17:23
bauzasMinnesota is that better from other states that it saves 8 mins for every worker ?17:23
stephenfinbauzas: But yeah, that call ends up here https://github.com/openstack/nova/blob/master/nova/virt/hardware.py#L143617:23
mriedembauzas: it is; that's why our taxes are high17:23
bauzasmriedem: I'm sure we beat your state17:24
*** yangyapeng has quit IRC17:24
stephenfinwhich then calls in here, where we _eventually_ raise the exception https://github.com/openstack/nova/blob/master/nova/virt/hardware.py#L151317:24
mriedembauzas: sure, but,17:24
mriedemwe don't get 50% of the month off for PTO :P17:24
bauzasjeez17:25
bauzastouché17:25
*** edmondsw has quit IRC17:25
bauzasstephenfin: so, give me a sec to rephrase that17:25
bauzasstephenfin: you mean that the policy check will be done primarly by the scheduler then ?17:26
bauzasI need to rereview then the implications17:26
stephenfinIt'll be done twice - once by the scheduler and once when we're booting17:26
stephenfinBecause races are possible17:26
bauzasyeah, but totally not at the same time :)17:26
stephenfinIndeed :)17:26
*** baoli has quit IRC17:27
stephenfinSo, per above, what a user will see is a failure to find suitable hosts17:27
bauzasso, say we raise an exception, all of that stuff if called by the filter, right?17:27
bauzass/if/is17:27
stephenfinAll of what stuff?17:27
bauzasstephenfin: https://review.openstack.org/#/c/527472/4/nova/virt/hardware.py@1197 is called eventually by the filter ?17:28
stephenfinYes17:28
stephenfinDitto for https://review.openstack.org/#/c/527472/4/nova/virt/hardware.py@118417:29
*** klindgren has joined #openstack-nova17:29
*** damien_r has quit IRC17:29
bauzasstephenfin: so that worst than a reschedule17:29
bauzasthat's17:29
*** klindgren has quit IRC17:29
bauzasstephenfin: if the filter raises an exception, the whole scheduling process fails17:30
bauzasI need to test that, but I'm pretty it goes into ERROR/scheduling17:30
bauzasoh man, fat fingers17:30
*** fragatin_ has quit IRC17:31
vivsoni__mriedem: Hi17:31
stephenfinbauzas: Ack. I'm going to test it now myself with CPU policies17:32
bauzasstephenfin: I need to bail out, family duty17:32
bauzas++17:32
stephenfinA-ok. I'm gone tomorrow but I'll be around Monday if you've questions17:33
*** mdnadeem has quit IRC17:33
mriedemvivsoni__: hi17:33
bauzasstephenfin: drop me a note about your findings and if that's not a full scheduling break, then +2 on the last PS17:34
stephenfinbauzas: Spot on. Enjoy family time :)17:34
bauzasif that's a full scheduling stop then I'm pretty -1 on the existing17:34
bauzasor make the filter resilient to errors17:34
vivsoni__mriedem: Need your input in resolving tempest Test case "tempest.api.compute.volumes.test_attach_volume.AttachVolumeShelveTestJSON.test_detach_volume_shelved_or_offload_server"17:35
*** lucasagomes is now known as lucas-afk17:35
vivsoni__hpe ci is failing because of that17:35
*** tidwellr has quit IRC17:36
*** Apoorva has quit IRC17:36
openstackgerritAde Lee proposed openstack/nova master: Add flag to make glance image verification optional  https://review.openstack.org/52801717:37
*** dtruong has joined #openstack-nova17:37
*** mdbooth has quit IRC17:37
*** ansiwen has quit IRC17:37
*** baoli has joined #openstack-nova17:38
vivsoni__mriedem: In this TC 'connector', which is passed to cinder vendor driver's 'initialize_connection' & 'terminate_connection' is empty17:39
*** edmondsw has joined #openstack-nova17:40
*** mdbooth has joined #openstack-nova17:40
*** ansiwen has joined #openstack-nova17:41
vivsoni__basically the driver's 'def initialize_connection' is not called, when 'attach_volume' is called https://github.com/openstack/tempest/blob/master/tempest/api/compute/volumes/test_attach_volume.py#L25217:43
*** felipemonteiro_ has quit IRC17:43
vivsoni__its failing here https://github.com/openstack/tempest/blob/master/tempest/api/compute/volumes/test_attach_volume.py#L25317:43
*** klindgren has joined #openstack-nova17:44
*** logan- has quit IRC17:44
*** logan- has joined #openstack-nova17:45
mriedemvivsoni__: that's because nova doesn't call initialize_connection anymore17:46
mriedemit calls attachment_update17:46
vivsoni__this detach_volume(), internally calls to 'def terminate_connection()' of cinder driver's and failed when tries to get connector['host']17:47
mriedemvivsoni__: this would be easier if you could point me at CI job logs17:48
mriedemwith an actual failure17:48
mriedemi am woefully behind on my todo list for today17:48
openstackgerritMerged openstack/nova stable/ocata: Get original image_id from volume for volume-backed instance rebuild  https://review.openstack.org/52566617:49
*** sree has quit IRC17:49
*** artom has quit IRC17:50
vivsoni__mriedem: request you to please have a look at http://54.201.44.218/38/527338/2/check/3par-fc-driver-master-client-pip-c8k01-dsvm/c156eb1/17:50
*** moshele has quit IRC17:50
*** artom has joined #openstack-nova17:51
*** jmlowe has quit IRC17:51
*** sree has joined #openstack-nova17:52
*** edmondsw has quit IRC17:53
mriedemvivsoni__: ok so it's attaching a volume to a shelved offloaded server,17:53
mriedemso there is no host connector because there is no compute host for that server,17:53
mriedemand then the test detaches the volume while the server is still shelved offloaded17:53
mriedemand that's where it breaks right?17:53
*** tesseract has quit IRC17:53
vivsoni__mriedem: yes17:53
mriedemok that's right here v17:54
mriedemhttps://github.com/openstack/nova/blob/master/nova/compute/api.py#L199417:54
mriedemthat's the new call to cinder in this case17:54
*** edand__ has quit IRC17:54
vivsoni__basically volume is not attach and tries to detaches and failed17:54
*** edmondsw has joined #openstack-nova17:54
openstackgerritMerged openstack/nova stable/newton: Add regression test for rebuilding a volume-backed server  https://review.openstack.org/52654217:54
mriedemvivsoni__: i'm not sure why this would be any different from a nova perspective - when attaching a volume to a shelved offloaded server, not never calls os-initialize_connection17:55
mriedem*nova never calls17:55
mriedemvivsoni__: so the explosion is likely on the cinder side17:55
*** xyang1 has joined #openstack-nova17:55
mriedemi.e. attachment_delete in cinder is assuming there will be a connector, and there isn't17:55
*** READ10 has joined #openstack-nova17:56
*** catintheroof has joined #openstack-nova17:57
mriedemvivsoni__: where is the error in the c-vol logs? http://54.201.44.218/38/527338/2/check/3par-fc-driver-master-client-pip-c8k01-dsvm/c156eb1/logs/screen-c-vol.txt.gz17:57
mriedemah here http://paste.openstack.org/show/628980/17:58
*** Alex_Staf has joined #openstack-nova17:58
mriedemvivsoni__: open a cinder bug17:58
smcginnisvivsoni__: That's a bug in the driver. It should handle that.17:59
*** andreas_s has joined #openstack-nova17:59
mriedemit's confusing how that ever worked17:59
*** xyang1 has quit IRC17:59
melwittlooks like nova.tests.functional.notification_sample_tests.test_instance.TestInstanceNotificationSampleWithMultipleComputeOldAttachFlow is failing intermittently in the gate18:00
mriedemsmcginnis: vivsoni_: oh i see18:00
mriedemhttps://github.com/openstack/nova/blob/master/nova/compute/api.py#L200418:00
mriedemin the old flow, if we didn't have a connector, we just didn't call terminate_connection18:00
mriedemmelwitt: ok18:00
*** dtantsur is now known as dtantsur|afk18:01
mriedemsmcginnis: which means other drivers could blow up18:02
mriedemsmcginnis: i'm thinking the volume manager's attachment_delete shouldn't call the driver if there is no connector18:02
*** sree has quit IRC18:02
*** klindgren has quit IRC18:03
*** Apoorva has joined #openstack-nova18:03
vivsoni__mriedem, smcginnis: many 3dr paty CI has skipped those TCs18:03
*** Apoorva has quit IRC18:03
mriedemok so i'll push a fix quick18:03
*** Swami has joined #openstack-nova18:03
*** Apoorva has joined #openstack-nova18:03
mriedemvivsoni__: do you have a bug number?18:03
vivsoni__mriedem: No... not sure whether there is a bug raised18:04
*** andreas_s has quit IRC18:04
mriedemvivsoni__: ok, can you open a cinder bug then?18:04
vivsoni__sure18:04
*** yamamoto has joined #openstack-nova18:05
vivsoni__please give a min.. will raised and let you know in a min or two18:05
smcginnisWe had discussed this in Cinder several months back. If there are any drivers that are not handling not having a connector, a bug should be filed against those drivers.18:05
smcginnisBut most (maybe some) should be handling it correctly.18:05
*** penick has quit IRC18:06
*** yamahata has quit IRC18:06
*** sree has joined #openstack-nova18:07
melwittI notice a lot of tracing of ComputeHostNotFound in gate n-cpu logs too18:07
*** penick has joined #openstack-nova18:08
*** klindgren has joined #openstack-nova18:09
mriedemmelwitt: that's been around for a long time, on startup18:09
mriedemsince ocata18:09
mriedemhappens when n-cpu starts before the host is discovered i think18:09
*** klindgren has quit IRC18:10
melwittoh. I thought it used to be a warning but not a backtrace18:10
openstackgerritArtom Lifshitz proposed openstack/nova master: Make request_spec.spec MediumText  https://review.openstack.org/52801218:10
mriedemit's a warning + traceback18:11
mriedembecause of an oslo.log bug18:11
mriedemsee my failed attempts https://review.openstack.org/#/c/517102/18:11
*** sree has quit IRC18:11
* melwitt examines18:13
* mriedem hopes some geddy lee can turn this day around18:13
*** klindgren has joined #openstack-nova18:13
melwittlooking at the oslo.log patch and trying to figure out, is the fix obvious or not -_-18:14
*** edmondsw has quit IRC18:14
*** baoli has quit IRC18:14
vivsoni__mriedem: https://bugs.launchpad.net/cinder/+bug/173825418:15
openstackLaunchpad bug 1738254 in Cinder "TEMPEST test case failed "tempest.api.compute.volumes.test_attach_volume.AttachVolumeShelveTestJSON.test_detach_volume_shelved_or_offload_server"" [Undecided,New]18:15
vivsoni__mriedem: i have raised a cinder bug against that tempest failure18:15
*** jpena is now known as jpena|off18:16
*** baoli has joined #openstack-nova18:16
openstackgerritStephen Finucane proposed openstack/nova master: Add PCI NUMA policies  https://review.openstack.org/52747218:17
openstackgerritStephen Finucane proposed openstack/nova master: trivial: Modify signature of _filter_non_requested_pfs  https://review.openstack.org/52747318:17
openstackgerritStephen Finucane proposed openstack/nova master: doc: Document PCI NUMA affinity policy  https://review.openstack.org/52801118:17
melwittI think I can propose something18:17
stephenfinbauzas: For when you're back online, that image-flavor conflict exception gets bubbled all the way back to the user's terminal via a HTTP 400. I think we're good18:18
*** mingyu has quit IRC18:20
*** mingyu has joined #openstack-nova18:21
*** edmondsw has joined #openstack-nova18:21
*** huanxie has quit IRC18:23
*** sambetts is now known as sambetts|afk18:23
*** gouthamr has quit IRC18:24
*** mingyu has quit IRC18:25
*** edmondsw has quit IRC18:25
*** itlinux has joined #openstack-nova18:26
*** edmondsw has joined #openstack-nova18:27
*** klindgren has quit IRC18:28
*** jafeha__ has joined #openstack-nova18:29
*** mingyu has joined #openstack-nova18:29
stephenfinmelwitt: I'm not about tomorrow but if you have a chance to review [1] before Monday, I'd be eternally grateful [1] https://review.openstack.org/#/q/topic:bp/websocket-proxy-to-host-security+status:open18:29
*** harlowja has joined #openstack-nova18:29
stephenfinmelwitt: I think jianghuaw_'s comments are valid but something that can be addressed in a follow-up (I don't know how much work 3.8 -> 3.3 conversion would be)18:30
* stephenfin heads home18:30
melwittstephenfin: I've been looking at it and it looks overall good to me, but I saw recently someone -1ed it saying it doesn't work with xen? okay18:30
*** jafeha has quit IRC18:31
*** edmondsw has quit IRC18:32
*** stvnoyes has joined #openstack-nova18:32
*** edmondsw has joined #openstack-nova18:33
*** mingyu has quit IRC18:34
*** imacdonn_ has quit IRC18:35
*** imacdonn_ has joined #openstack-nova18:35
*** mingyu has joined #openstack-nova18:37
*** gouthamr has joined #openstack-nova18:37
*** edmondsw has quit IRC18:37
*** tidwellr has joined #openstack-nova18:38
*** edmondsw has joined #openstack-nova18:39
*** edmondsw has quit IRC18:43
*** penick has quit IRC18:44
*** edmondsw has joined #openstack-nova18:45
*** penick has joined #openstack-nova18:45
*** clayton has quit IRC18:46
pprokop118:47
*** clayton has joined #openstack-nova18:47
*** AlexeyAbashkin has joined #openstack-nova18:47
*** Alex_Staf has quit IRC18:49
*** AlexeyAbashkin has quit IRC18:52
melwittmriedem: let's see how this goes https://review.openstack.org/#/c/52803618:55
mriedemcool18:56
*** felipemonteiro_ has joined #openstack-nova18:56
*** andreas_s has joined #openstack-nova18:57
*** itlinux has quit IRC18:57
*** felipemonteiro__ has joined #openstack-nova18:58
*** gszasz has quit IRC19:00
*** mvk has quit IRC19:00
*** mvk has joined #openstack-nova19:01
*** felipemonteiro_ has quit IRC19:02
*** andreas_s has quit IRC19:02
*** ralonsoh has quit IRC19:03
cfriesenwhen dealing with quotas, why do we distinguish between "in use" and "reserved"?19:05
*** Guest26528 has quit IRC19:06
*** catintheroof has quit IRC19:06
melwittbecause the old system had an initial "reserve" step that happened before switching it from "reserved" -> "in use"19:06
melwittreserve() reserved it and commit() changed it from reserved to in use19:07
openstackgerritEd Leafe proposed openstack/nova master: Make conductor pass and use host_lists  https://review.openstack.org/51135819:08
openstackgerritEd Leafe proposed openstack/nova master: Change compute RPC to use alternates for resize  https://review.openstack.org/52643619:08
cfriesenmelwitt: thanks...so in pike would we ever see a nonzero reserved value?19:09
melwittcfriesen: right, no more reserve step so reserved always zero19:11
*** adisky__ has quit IRC19:11
*** jmlowe has joined #openstack-nova19:12
*** edmondsw has quit IRC19:12
cfriesenmelwitt: great, thanks19:12
*** edmondsw has joined #openstack-nova19:13
*** vivsoni__ has quit IRC19:13
*** itlinux has joined #openstack-nova19:15
*** openstack has joined #openstack-nova20:32
*** ChanServ sets mode: +o openstack20:32
*** yangyapeng has quit IRC20:34
*** edmondsw has quit IRC20:36
*** pchavva has quit IRC20:38
*** lyan has joined #openstack-nova20:38
*** lpetrut has joined #openstack-nova20:38
*** gbarros has quit IRC20:38
openstackgerritmelanie witt proposed openstack/nova master: live-mig: keep disk device address same  https://review.openstack.org/51802220:45
*** jmlowe has joined #openstack-nova20:45
*** AlexeyAbashkin has joined #openstack-nova20:46
*** gouthamr has quit IRC20:47
*** liverpooler has quit IRC20:47
*** yangyapeng has joined #openstack-nova20:50
*** smatzek has quit IRC20:51
*** AlexeyAbashkin has quit IRC20:51
*** cleong has quit IRC20:52
*** awaugama has quit IRC20:52
*** yangyapeng has quit IRC20:55
*** pchavva has joined #openstack-nova20:57
*** jafeha__ has quit IRC20:58
*** yamamoto has joined #openstack-nova21:07
*** sree has joined #openstack-nova21:07
*** SamYaple has quit IRC21:07
*** SamYaple has joined #openstack-nova21:08
*** pchavva has quit IRC21:08
*** yamahata has joined #openstack-nova21:09
*** openstack has joined #openstack-nova21:11
*** ChanServ sets mode: +o openstack21:11
*** sree has quit IRC21:14
*** yamamoto has quit IRC21:15
*** yangyapeng has quit IRC21:15
*** tidwellr_ has joined #openstack-nova21:22
*** tidwellr has quit IRC21:22
*** SamYaple has quit IRC21:22
*** rodolof has quit IRC21:22
*** rodolof has joined #openstack-nova21:22
*** SamYaple has joined #openstack-nova21:23
mriedemcan someone hit this? https://review.openstack.org/#/c/527878/ - unblocks the 2.58 api change and is needed for the common server create api samples after 2.57 where 'personality' is not an allowed request parameter21:27
*** sree has joined #openstack-nova21:27
openstackgerritChris Dent proposed openstack/nova master: WIP: Implement set_and_clear_allocations in report client  https://review.openstack.org/52808921:27
cdentdansmith. mriedem that ^ is a start on POST to /allocations for migration stuff. It's just a spike at this point but I figured I'd push it up for early "is this what we were thinking style review". Also I needed to do something besides talk.21:28
mriedemheh, cool. thanks.21:28
mriedemthat was on my list as well21:28
dansmithsweet21:30
*** tidwellr_ has quit IRC21:30
dansmithI was literally lying in bed last night thinking I needed to ask if you were interested in doing that21:30
dansmithand then forgot21:30
cdentit appears to "work". the test_servers functional tests pass, and I assume it exercised by that21:30
*** yangyapeng has joined #openstack-nova21:30
tonybmriedem: Thanks for the update.  I'll try again to look at thos backports.21:31
cdentactually, dansmith, I was standing in your room last night, watching you sleep21:31
*** threestrands has joined #openstack-nova21:31
* dansmith checks his magazine21:31
cdentheh21:31
*** sree has quit IRC21:31
openstackgerritMerged openstack/nova master: Fix some typos in nova doc  https://review.openstack.org/52785621:31
* mriedem disables zkvm ci again21:32
*** READ10 has quit IRC21:34
*** yangyapeng has quit IRC21:35
mriedemthe live migrate task calls this same code in the migrate module right?21:35
mriedemyup21:36
mriedemok21:36
mriedemlooks like a sane start to me21:36
mriedemnot sure we need to handle the 1.13 not being there but it doesn't really hurt either21:36
cdentmriedem: yeah, I responded on that within. I couldn't remember whether we had nailed the 1.14 requirment as for realz or not21:37
mriedemit is21:37
mriedemcreate resource provider doesn't fallback if it's not there21:37
mriedemhttps://review.openstack.org/#/c/526505/21:37
cdentthen I suppose may as well remove that fallback21:37
cdentso does that mean we can remove all the fallback code in report client?21:38
cdentas there's some others too21:38
*** andreas_s has joined #openstack-nova21:40
mriedemmaybe21:40
mriedemi think we can remove the safe_connect decorator21:40
mriedemthat was for placement being optional in newton21:41
openstackgerritMatt Riedemann proposed openstack/nova stable/pike: libvirt: properly decode error message from qemu guest agent  https://review.openstack.org/51772721:42
*** gouthamr has joined #openstack-nova21:43
*** andreas_s has quit IRC21:44
cdentit does other things21:45
cdentincluding dealing with the fact that placement is not currently up, but might be in a little while21:45
cdentwhich helps make it resilient across reboots21:45
cdentor during installs where things get out of order21:45
*** gouthamr_ has joined #openstack-nova21:46
*** erlon has joined #openstack-nova21:46
mriedemoh i suppose21:46
*** tidwellr has joined #openstack-nova21:47
openstackgerritMerged openstack/nova master: XenAPI: Don't use nicira-iface-id for XenServer VIF  https://review.openstack.org/45164121:47
openstackgerritMerged openstack/nova master: Retry _trait_sync on deadlock  https://review.openstack.org/52783621:47
*** penick has quit IRC21:48
*** gouthamr has quit IRC21:48
*** yamamoto has joined #openstack-nova21:51
openstackgerritMatt Riedemann proposed openstack/nova stable/pike: Retry _trait_sync on deadlock  https://review.openstack.org/52809421:51
*** yangyapeng has joined #openstack-nova21:52
*** yangyapeng has quit IRC21:57
*** gbarros has joined #openstack-nova21:58
*** rcernin has joined #openstack-nova21:58
openstackgerritJackie Truong proposed openstack/nova master: Add trusted_certs object  https://review.openstack.org/48940822:02
openstackgerritJackie Truong proposed openstack/nova master: Add trusted_image_certificates to REST API  https://review.openstack.org/48620422:02
*** rodolof has quit IRC22:03
*** eandersson has joined #openstack-nova22:03
*** penick has joined #openstack-nova22:04
mriedemwho wants to delete some stuff?22:04
openstackgerritMatt Riedemann proposed openstack/nova master: Delete the TypeAffinityFilter  https://review.openstack.org/52809922:04
mriedemjogo: i believe this was your baby ^22:04
*** gouthamr_ is now known as gouthamr22:04
*** mvenesio has quit IRC22:05
mriedemi suppose i should run pep822:07
openstackgerritMatt Riedemann proposed openstack/nova master: Delete the TypeAffinityFilter  https://review.openstack.org/52809922:10
*** cdent has quit IRC22:12
cfriesenmriedem: has anyone ever talked about not overwriting the UEFI variables on every boot?22:14
cfriesenmriedem: I don't see any related nova bug, but we've got a customer saying that instances should preserve UEFI variables over stop/start, for example.22:15
mriedemcfriesen: no one has ever talked about uefi period22:15
openstackgerritmelanie witt proposed openstack/nova master: Enable test_volume_backed_live_migration in tempest  https://review.openstack.org/52810422:15
cfriesenmriedem: heh22:15
*** Apoorva has quit IRC22:17
*** ChanServ has quit IRC22:17
*** yangyapeng has joined #openstack-nova22:19
openstackgerritJackie Truong proposed openstack/python-novaclient master: Microversion 2.58 - Add trusted_image_certificates  https://review.openstack.org/50039622:20
*** kumarmn_ has quit IRC22:20
*** yangyapeng has quit IRC22:24
*** ChanServ has joined #openstack-nova22:24
*** barjavel.freenode.net sets mode: +o ChanServ22:24
*** artom has quit IRC22:26
openstackgerritMerged openstack/nova stable/ocata: Fix 'force' parameter in os-quota-sets PUT schema  https://review.openstack.org/52642622:26
*** gbarros has quit IRC22:27
*** gouthamr has quit IRC22:28
*** ChanServ has quit IRC22:28
*** moshele has quit IRC22:30
*** esberglu has quit IRC22:31
*** ChanServ has joined #openstack-nova22:31
*** barjavel.freenode.net sets mode: +o ChanServ22:31
*** esberglu_ has joined #openstack-nova22:34
*** esbergl__ has joined #openstack-nova22:36
*** yangyapeng has joined #openstack-nova22:37
*** kumarmn has joined #openstack-nova22:39
*** esberglu_ has quit IRC22:39
mriedemcfriesen: are you going to be in dublin?22:40
mriedemdid you see my query about that old bp of yours about specific numa scheduling?22:40
*** esbergl__ has quit IRC22:40
*** kumarmn has quit IRC22:41
*** kumarmn has joined #openstack-nova22:41
openstackgerritJackie Truong proposed openstack/nova master: Add trusted_certs object  https://review.openstack.org/48940822:41
*** tidwellr has quit IRC22:41
*** yangyapeng has quit IRC22:42
*** sree has joined #openstack-nova22:48
*** esberglu has joined #openstack-nova22:49
*** felipemonteiro_ has joined #openstack-nova22:50
*** BryanS68 has quit IRC22:51
openstackgerritMatt Riedemann proposed openstack/nova master: Fix the formatting for 2.56 in the compute REST API history doc  https://review.openstack.org/52811422:52
*** sree has quit IRC22:53
*** felipemonteiro__ has quit IRC22:53
*** esberglu has quit IRC22:54
*** felipemonteiro_ has quit IRC22:56
*** baoli has quit IRC22:57
*** yangyapeng has joined #openstack-nova22:59
*** lyan has quit IRC23:01
*** yangyapeng has quit IRC23:04
*** sree has joined #openstack-nova23:07
*** threestrands_ has joined #openstack-nova23:10
*** sree has quit IRC23:12
*** threestrands has quit IRC23:13
*** felipemonteiro_ has joined #openstack-nova23:14
*** masuberu has quit IRC23:15
*** Sukhdev has joined #openstack-nova23:16
*** weshay is now known as weshay_pto23:17
*** yangyapeng has joined #openstack-nova23:19
*** _ix_ has joined #openstack-nova23:21
*** lpetrut has quit IRC23:21
*** baoli has joined #openstack-nova23:23
*** _ix has quit IRC23:24
*** yangyapeng has quit IRC23:24
*** kumarmn has quit IRC23:24
*** kumarmn has joined #openstack-nova23:25
*** moshele has joined #openstack-nova23:27
*** baoli has quit IRC23:28
*** kumarmn has quit IRC23:30
*** harlowja has quit IRC23:32
*** felipemonteiro_ has quit IRC23:35
openstackgerritMatt Riedemann proposed openstack/python-novaclient master: WIP: Add support for the 2.57 microversion  https://review.openstack.org/52812823:35
*** dave-mcc_ has quit IRC23:37
*** Apoorva has joined #openstack-nova23:38
*** harlowja has joined #openstack-nova23:39
*** yangyapeng has joined #openstack-nova23:40
*** harlowja has quit IRC23:40
*** harlowja has joined #openstack-nova23:42
*** andreas_s has joined #openstack-nova23:44
*** yangyapeng has quit IRC23:44
*** tetsuro has joined #openstack-nova23:47
*** andreas_s has quit IRC23:48
*** takashin has joined #openstack-nova23:49
*** harlowja has quit IRC23:50
*** lennyb has quit IRC23:51
openstackgerritMerged openstack/nova stable/newton: Only query BDMs once in API during rebuild  https://review.openstack.org/52654323:54
*** erlon has quit IRC23:55
jogomriedem: doesn't sound very backwards compatible to delete it  ... but goodbye filter23:57

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