Friday, 2018-07-13

*** shaohe_feng has quit IRC00:02
*** shaohe_feng has joined #openstack-nova00:02
*** takashin has joined #openstack-nova00:12
*** shaohe_feng has quit IRC00:12
*** stakeda has joined #openstack-nova00:12
*** shaohe_feng has joined #openstack-nova00:13
*** medberry has joined #openstack-nova00:18
*** BrinZhang has joined #openstack-nova00:22
*** shaohe_feng has quit IRC00:22
*** shaohe_feng has joined #openstack-nova00:25
*** david-lyle has joined #openstack-nova00:28
*** dklyle has quit IRC00:29
openstackgerritMerged openstack/nova master: Update process doc to be more generic about point of contact  https://review.openstack.org/58231200:30
*** agrebennikov has quit IRC00:31
*** agrebennikov_ has quit IRC00:31
*** shaohe_feng has quit IRC00:32
*** shaohe_feng has joined #openstack-nova00:33
*** phuongnh has joined #openstack-nova00:34
*** hshiina has joined #openstack-nova00:42
*** shaohe_feng has quit IRC00:43
openstackgerritMerged openstack/nova master: Handle rebuild of instances with image traits  https://review.openstack.org/56949800:43
*** hemna_ has quit IRC00:44
*** shaohe_feng has joined #openstack-nova00:44
*** gyee has quit IRC00:49
*** liuyulong has joined #openstack-nova00:51
*** shaohe_feng has quit IRC00:53
*** david-lyle has quit IRC00:54
*** shaohe_feng has joined #openstack-nova00:54
*** dklyle has joined #openstack-nova00:54
*** hemna_ has joined #openstack-nova00:58
*** david-lyle has joined #openstack-nova01:00
*** dklyle has quit IRC01:02
*** shaohe_feng has quit IRC01:03
openstackgerritVu Cong Tuan proposed openstack/nova-specs master: Switch to stestr  https://review.openstack.org/58128401:04
*** harlowja has quit IRC01:05
*** shaohe_feng has joined #openstack-nova01:08
*** yedongcan has joined #openstack-nova01:10
*** abhishekk has joined #openstack-nova01:12
*** shaohe_feng has quit IRC01:13
*** shaohe_feng has joined #openstack-nova01:15
*** yassine_ has quit IRC01:17
mriedemhttps://review.openstack.org/569498 merged so we can mark that bp complete and move it out of the runway slot01:17
*** jichen has joined #openstack-nova01:18
melwittmriedem: ack, thanks01:18
*** tiendc has joined #openstack-nova01:19
openstackgerritVu Cong Tuan proposed openstack/python-novaclient master: Switch to stestr  https://review.openstack.org/58209001:19
*** yassine_ has joined #openstack-nova01:21
melwittmriedem: I put next in line into a runway and made the end date July 26 FF (instead of July 27)01:21
mriedemmelwitt: comment on https://review.openstack.org/#/c/582312/ if you wanted to make that more actionable01:21
melwittthanks01:22
*** hongbin has joined #openstack-nova01:22
*** shaohe_feng has quit IRC01:24
*** shaohe_feng has joined #openstack-nova01:24
*** liuyulong_ has joined #openstack-nova01:26
*** liuyulong has quit IRC01:27
*** mrsoul has joined #openstack-nova01:28
*** mrsoul`` has joined #openstack-nova01:28
*** dillaman has quit IRC01:28
*** blkart has quit IRC01:29
*** blkart has joined #openstack-nova01:30
*** mschuppert has quit IRC01:31
*** mrsoul` has quit IRC01:31
*** liuyulong__ has joined #openstack-nova01:32
*** shaohe_feng has quit IRC01:34
openstackgerritmelanie witt proposed openstack/nova master: Add functional regression test for bug 1778305  https://review.openstack.org/58240701:34
openstackbug 1778305 in OpenStack Compute (nova) "Nova may erronously look up service version of a deleted service, when hostname have been reused" [Undecided,In progress] https://launchpad.net/bugs/1778305 - Assigned to melanie witt (melwitt)01:34
openstackgerritmelanie witt proposed openstack/nova master: Don't generate service UUID for deleted services  https://review.openstack.org/58240801:34
openstackgerritmelanie witt proposed openstack/nova master: Add functional regression test for bug 1764556  https://review.openstack.org/58243501:34
openstackbug 1764556 in OpenStack Compute (nova) ""nova list" fails with exception.ServiceNotFound if service is deleted and has no UUID" [Medium,In progress] https://launchpad.net/bugs/176455601:34
*** liuyulong_ has quit IRC01:35
*** dklyle has joined #openstack-nova01:36
mriedemKevin_Zheng: i see you didn't change anything about the rpc api version handling in https://review.openstack.org/#/c/568542/ - i'll see if i can talk with dansmith about that tomorrow morning so we can sort that out (he's on vacation next week)01:36
mriedemKevin_Zheng: are you going to rebase https://review.openstack.org/#/c/573136/ back onto the top of that series?01:37
*** david-lyle has quit IRC01:37
*** shaohe_feng has joined #openstack-nova01:37
Kevin_Zhengmriedem, yeah, I was planning to collect more ideas on version checks01:38
*** dklyle has quit IRC01:38
*** dklyle has joined #openstack-nova01:38
*** yamahata has quit IRC01:38
Kevin_Zhengmriedem, I did all the updates to the API patch, but somehow the branch got messed up when I upload it, so I'm going to do it again today01:39
mriedemi guess dansmith is saying the version checks would be redundant in the compute api and rpc api,01:39
*** liuyulong_ has joined #openstack-nova01:39
mriedembecause if you can't send 5.1, that's all the check you need01:39
Kevin_Zhengyeah, they are suggesting do it in API only01:39
Kevin_ZhengI mean Alex and Gmann01:39
mriedemdan is saying doing it all in rpcapi01:40
mriedemb/c that's what has the can_send_version check01:40
mriedemand knows the upgrade_levels pin01:40
mriedemhttps://docs.openstack.org/nova/latest/configuration/config.html#upgrade-levels01:40
mriedemthe API could tell us that the compute is running version 33 but the rpc api might be pinned to rocky,01:40
mriedemat which point can_send_version would still fail the 5.1 check01:40
mriedemalso good lord that's a wall of text ^01:41
Kevin_Zhenghmmm this doc helps01:41
gmannmriedem: what is case where node has compute 33 and RPC API not pinned ?01:41
* gmann this is for learning more 01:41
gmannmriedem: Kevin_Zheng the only reason of checking in api we thought is early revert back the request. check in rpc is needed as you and dansmith mentioned01:42
mriedemyou can fail fast and return an error to the user in either place01:43
mriedemsince the check happens before the cast to compute01:43
*** liuyulong__ has quit IRC01:43
mriedemi'm trying to sort out if we can *move* the !running check from api to rpcapi01:43
mriedemotherwise it's redundant01:43
mriedemthis https://review.openstack.org/#/c/568542/17/nova/compute/api.py@442701:44
gmannmriedem: you mean from rpc to api ?01:44
mriedemapi to rpc01:44
gmannagree to have check of !running in single place, either one01:44
*** shaohe_feng has quit IRC01:44
gmannhumm01:44
Kevin_ZhengHmm, that is for microversion01:44
mriedemnova.compute.api.API can't do the rpc version check01:44
mriedemit doesn't have the can_send_version logic01:44
mriedemthat's in nova.compute.rpcapi01:45
Kevin_ZhengI mean, we should check for microversion first01:45
mriedemright that's what is confusing me01:45
mriedembecause we need to build in the microversion logic as well01:45
Kevin_Zhengyeah01:45
mriedembut couldn't we just pass support_abort_in_queue to rpcapi?01:46
mriedemand do the logic there?01:46
mriedemthen your acceptable status values are based on (1) the microversion and (2) the rpcapi version01:46
*** shaohe_feng has joined #openstack-nova01:46
mriedemallowed_statuses = ['running']01:46
mriedemif support_abort_in_queue:01:46
Kevin_Zhengand can_send_version?01:47
mriedemyeah,01:47
mriedemif support_abort_in_queue and can_send_version(5.1):01:47
mriedem    allowed_statuses.extend(['queued', 'preparing'])01:47
gmannmicroversion check is on APOI controller side not enough ?01:47
Kevin_ZhengThen should I merge patch 2 and 4?01:47
mriedemif migration.status not in allowed_statuses:01:47
gmannAPI01:47
mriedemraise exception.InvalidMigrationState(01:47
mriedemgmann: no because if the compute is queens,01:48
mriedemit doesn't know how to cancel a migration in queued or preparing status01:48
gmannthat what compute version or rpc version check will handle right01:49
mriedemKevin_Zheng: i don't think you merge them01:49
mriedemKevin_Zheng: i think patch 2 has the rpcapi version bump b/c of the signature change01:49
mriedemand then in the same change that adds the microversion, you can move the status checking from api into rpcapi01:50
mriedembased and pass in the support_abort_in_queue flag01:50
Kevin_Zhengmove it in patch 4?01:50
mriedem*pass in the support_abort_in_queue01:50
gmannthis one will be passed all way till rpc right? - https://review.openstack.org/#/c/573136/6/nova/api/openstack/compute/server_migrations.py@14901:51
openstackgerritChen proposed openstack/nova master: fix cellv2 delete_host  https://review.openstack.org/58217101:51
mriedemgmann: that's what i'm suggesting,01:52
mriedemi'll leave a comment on patch 2 and then i need to call it a day01:52
gmannohk. i got confuse when you mentioned microversion checks on compute.apy.py01:52
mriedemand i'll talk to dansmith about my idea in the morning01:52
Kevin_ZhengOk, good night01:52
openstackgerritJulia Kreger proposed openstack/nova master: ironic: provide facilities to gracefully navigate versions  https://review.openstack.org/58241601:53
TheJuliamriedem: looks like I'll need to tune some job timeouts with that rescue test :\  Something for tomorrow since we seem to have a job that now wants to time out independently as well :\01:54
*** shaohe_feng has quit IRC01:54
*** shaohe_feng has joined #openstack-nova01:55
openstackgerritJulia Kreger proposed openstack/nova master: virt/ironic: Implement rescue and unrescue  https://review.openstack.org/41648701:56
TheJuliamriedem: parent update rebase ^01:56
* TheJulia goes to bed01:56
mriedemKevin_Zheng: gmann: dansmith: left my idea in https://review.openstack.org/#/c/568542/01:57
Kevin_ZhengThanks I think it will work, lets collect more ideas.01:58
*** liuyulong__ has joined #openstack-nova01:58
mriedemTheJulia: thanks, lgtm01:58
TheJuliamriedem: no, thank you! :)01:59
*** markvoelker has joined #openstack-nova02:00
*** liuyulong_ has quit IRC02:01
gmannmriedem: thanks. ll check02:02
*** mriedem has quit IRC02:04
*** shaohe_feng has quit IRC02:05
*** shaohe_feng has joined #openstack-nova02:06
*** gongysh has joined #openstack-nova02:12
*** shaohe_feng has quit IRC02:15
*** shaohe_feng has joined #openstack-nova02:16
*** liuyulong_ has joined #openstack-nova02:19
openstackgerritTakashi NATSUME proposed openstack/nova master: Transform instance.live_migration_force_complete notification  https://review.openstack.org/48056102:19
*** liuyulong__ has quit IRC02:23
*** lei-zh has joined #openstack-nova02:25
*** shaohe_feng has quit IRC02:25
*** shaohe_feng has joined #openstack-nova02:26
*** Bhujay has joined #openstack-nova02:28
*** medberry has quit IRC02:28
*** liuyulong_ is now known as liuyulong02:31
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Microversion 2.64 - Use new format policy in server group  https://review.openstack.org/56753402:32
*** markvoelker has quit IRC02:34
*** agrebennikov_ has joined #openstack-nova02:35
*** agrebennikov has joined #openstack-nova02:35
*** yassine_ has quit IRC02:35
*** shaohe_feng has quit IRC02:35
*** yedongcan1 has joined #openstack-nova02:36
*** lei-zh1 has joined #openstack-nova02:36
*** lei-zh has quit IRC02:36
*** shaohe_feng has joined #openstack-nova02:37
*** yedongcan has quit IRC02:38
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Microversion 2.64 - Use new format policy in server group  https://review.openstack.org/56753402:44
openstackgerritZhenyu Zheng proposed openstack/nova master: Use ThreadPoolExecutor for max_concurrent_live_migrations  https://review.openstack.org/56350502:45
openstackgerritMerged openstack/nova master: Follow up for Ie49d605c66062d2548241d7e04f5a2a6b98c011e  https://review.openstack.org/58229202:45
openstackgerritMerged openstack/nova master: Add policy field to ServerGroup notification object  https://review.openstack.org/56340102:45
*** shaohe_feng has quit IRC02:46
*** shaohe_feng has joined #openstack-nova02:48
*** yassine_ has joined #openstack-nova02:50
*** shaohe_feng has quit IRC02:56
*** shaohe_feng has joined #openstack-nova02:57
openstackgerritMerged openstack/nova master: Change the ServerGroupAntiAffinityFilter to adapt to new policy  https://review.openstack.org/57116602:59
*** ykarel|away has joined #openstack-nova02:59
*** yamamoto has joined #openstack-nova03:03
*** shaohe_feng has quit IRC03:06
*** shaohe_feng has joined #openstack-nova03:07
*** shaohe_feng has quit IRC03:16
*** shaohe_feng has joined #openstack-nova03:17
*** mdbooth_ has joined #openstack-nova03:21
*** mdbooth has quit IRC03:23
*** shaohe_feng has quit IRC03:27
*** shaohe_feng has joined #openstack-nova03:27
*** TxGirlGeek has joined #openstack-nova03:28
*** boris_42_ has quit IRC03:28
*** markvoelker has joined #openstack-nova03:31
openstackgerritEric Fried proposed openstack/nova master: WIP: Compute: Handle reshaped provider trees  https://review.openstack.org/57623603:32
*** mdnadeem has joined #openstack-nova03:32
*** gongysh has quit IRC03:34
*** udesale has joined #openstack-nova03:35
*** shaohe_feng has quit IRC03:37
*** shaohe_feng has joined #openstack-nova03:38
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Fix all invalid obj_make_compatible test case  https://review.openstack.org/57424003:44
*** shaohe_feng has quit IRC03:47
*** shaohe_feng has joined #openstack-nova03:49
*** ykarel|away has quit IRC03:50
*** yedongcan has joined #openstack-nova03:53
*** yedongcan1 has quit IRC03:53
*** shaohe_feng has quit IRC03:57
*** shaohe_feng has joined #openstack-nova03:58
*** spsurya_ has joined #openstack-nova03:58
*** lei-zh1 has quit IRC03:58
*** abhishekk has quit IRC04:00
*** links has joined #openstack-nova04:02
openstackgerritYikun Jiang (Kero) proposed openstack/python-novaclient master: Microversion 2.64 - Use new format policy in server group  https://review.openstack.org/57826104:03
*** markvoelker has quit IRC04:04
*** shaohe_feng has quit IRC04:08
*** shaohe_feng has joined #openstack-nova04:08
*** gongysh has joined #openstack-nova04:15
*** shaohe_feng has quit IRC04:18
*** shaohe_feng has joined #openstack-nova04:19
*** hongbin has quit IRC04:21
*** janki has joined #openstack-nova04:23
*** harlowja has joined #openstack-nova04:25
*** ykarel has joined #openstack-nova04:28
*** shaohe_feng has quit IRC04:28
*** shaohe_feng has joined #openstack-nova04:30
openstackgerritHironori Shiina proposed openstack/nova master: ironic: Log an error when API version is not available  https://review.openstack.org/58244704:35
*** shaohe_feng has quit IRC04:38
*** shaohe_feng has joined #openstack-nova04:39
*** harlowja has quit IRC04:44
*** myrat has joined #openstack-nova04:48
*** shaohe_feng has quit IRC04:49
*** shaohe_feng has joined #openstack-nova04:50
*** flwang1 has quit IRC04:58
*** shaohe_feng has quit IRC04:59
*** shaohe_feng has joined #openstack-nova05:01
*** markvoelker has joined #openstack-nova05:01
*** TxGirlGeek has quit IRC05:03
*** shaohe_feng has quit IRC05:09
*** shaohe_feng has joined #openstack-nova05:10
*** lei-zh has joined #openstack-nova05:14
*** shaohe_feng has quit IRC05:19
*** shaohe_feng has joined #openstack-nova05:20
*** abhishekk has joined #openstack-nova05:20
*** agrebennikov has quit IRC05:26
*** agrebennikov_ has quit IRC05:26
*** tetsuro has joined #openstack-nova05:28
*** shaohe_feng has quit IRC05:30
*** shaohe_feng has joined #openstack-nova05:30
*** markvoelker has quit IRC05:33
*** shaohe_feng has quit IRC05:40
*** yamamoto_ has joined #openstack-nova05:40
*** shaohe_feng has joined #openstack-nova05:41
*** ratailor has joined #openstack-nova05:42
*** yamamoto has quit IRC05:43
*** shaohe_feng has quit IRC05:50
*** shaohe_feng has joined #openstack-nova05:51
*** mrsoul is now known as mschuppert05:55
*** shaohe_feng has quit IRC06:00
*** shaohe_feng has joined #openstack-nova06:01
*** shaohe_feng has quit IRC06:11
*** shaohe_feng has joined #openstack-nova06:12
*** ccamacho has joined #openstack-nova06:12
*** ccamacho has quit IRC06:12
*** ccamacho has joined #openstack-nova06:13
*** shaohe_feng has quit IRC06:21
*** shaohe_feng has joined #openstack-nova06:22
*** armaan has joined #openstack-nova06:23
*** dr_gogeta86 has joined #openstack-nova06:24
*** dr_gogeta86 has quit IRC06:24
*** dr_gogeta86 has joined #openstack-nova06:24
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Fix all invalid obj_make_compatible test case  https://review.openstack.org/57424006:26
*** tiendc has quit IRC06:28
*** tiendc has joined #openstack-nova06:28
*** phuongnh has quit IRC06:28
*** Bhujay has quit IRC06:28
*** Bhujay has joined #openstack-nova06:29
*** phuongnh has joined #openstack-nova06:29
*** tssurya has joined #openstack-nova06:29
*** markvoelker has joined #openstack-nova06:31
*** shaohe_feng has quit IRC06:31
*** armaan has quit IRC06:31
*** shaohe_feng has joined #openstack-nova06:31
*** jichen has quit IRC06:32
tssurya /set weechat.look.prefix_join "▬▬▶"06:33
*** andreas_s has joined #openstack-nova06:34
*** adreznec has quit IRC06:37
*** jichen has joined #openstack-nova06:39
tssuryaoops, sorry ^^06:40
openstackgerritzhaolihui proposed openstack/python-novaclient master: Fix flavor keyerror when nova boot vm  https://review.openstack.org/58214706:40
*** shaohe_feng has quit IRC06:41
*** shaohe_feng has joined #openstack-nova06:42
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Address nits in server group policy series  https://review.openstack.org/58161606:43
openstackgerritHironori Shiina proposed openstack/nova master: ironic: Log an error when API version is not available  https://review.openstack.org/58244706:45
openstackgerritZhenyu Zheng proposed openstack/nova master: Compute: add support to abort queued live migration  https://review.openstack.org/56854206:45
*** tetsuro has quit IRC06:48
*** armaan has joined #openstack-nova06:48
*** tetsuro has joined #openstack-nova06:49
*** lennyb has quit IRC06:49
*** logan- has quit IRC06:50
*** logan- has joined #openstack-nova06:50
*** ispp has joined #openstack-nova06:50
*** shaohe_feng has quit IRC06:52
*** cfriesen has quit IRC06:52
*** lennyb has joined #openstack-nova06:52
*** Bhujay has quit IRC06:52
*** shaohe_feng has joined #openstack-nova06:53
*** armaan has quit IRC06:56
*** Bhujay has joined #openstack-nova06:56
*** armaan has joined #openstack-nova06:57
*** tetsuro has quit IRC06:57
*** armaan has quit IRC06:57
openstackgerritOpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata  https://review.openstack.org/57801906:58
*** tssurya has quit IRC06:58
openstackgerritZhenyu Zheng proposed openstack/nova master: Fix ServerMigrationSampleJsonTestsV2_24 to use its own sample file  https://review.openstack.org/58156206:59
*** tetsuro has joined #openstack-nova07:00
*** tssurya has joined #openstack-nova07:00
*** adreznec has joined #openstack-nova07:00
*** mdnadeem has quit IRC07:01
*** mdnadeem has joined #openstack-nova07:02
*** shaohe_feng has quit IRC07:02
*** shaohe_feng has joined #openstack-nova07:03
*** markvoelker has quit IRC07:04
*** mdnadeem_ has joined #openstack-nova07:04
*** mdnadeem has quit IRC07:08
*** yamamoto_ has quit IRC07:10
*** yamamoto has joined #openstack-nova07:11
*** tesseract has joined #openstack-nova07:12
*** shaohe_feng has quit IRC07:12
*** shaohe_feng has joined #openstack-nova07:13
*** myrat has quit IRC07:15
*** masber has joined #openstack-nova07:17
*** peereb has joined #openstack-nova07:18
*** moshele has joined #openstack-nova07:19
*** shaohe_feng has quit IRC07:22
*** shaohe_feng has joined #openstack-nova07:23
*** amoralej|off is now known as amoralej07:26
*** ispp has quit IRC07:28
*** tiendc_ has joined #openstack-nova07:28
*** Bhujay has quit IRC07:29
*** Bhujay has joined #openstack-nova07:29
*** tiendc has quit IRC07:30
*** andreas_s has quit IRC07:30
*** andreas_s has joined #openstack-nova07:31
*** phuongnh has quit IRC07:33
*** shaohe_feng has quit IRC07:33
*** phuongnh has joined #openstack-nova07:33
*** shaohe_feng has joined #openstack-nova07:33
*** andreas_s has quit IRC07:34
*** andreas_s has joined #openstack-nova07:35
openstackgerritZhenyu Zheng proposed openstack/nova master: Compute: add support to abort queued live migration  https://review.openstack.org/56854207:41
*** shaohe_feng has quit IRC07:43
*** shaohe_feng has joined #openstack-nova07:47
*** ykarel is now known as ykarel|lunch07:47
*** Bhujay has quit IRC07:47
*** youzp has joined #openstack-nova07:47
*** Bhujay has joined #openstack-nova07:48
*** hshiina has quit IRC07:51
*** alexchadin has joined #openstack-nova07:52
openstackgerritZhenyu Zheng proposed openstack/nova master: Fix ServerMigrationSampleJsonTestsV2_24 to use its own sample file  https://review.openstack.org/58156207:53
*** shaohe_feng has quit IRC07:53
*** mgoddard has joined #openstack-nova07:54
*** jangutter has quit IRC07:56
*** ispp has joined #openstack-nova07:56
*** shaohe_feng has joined #openstack-nova07:56
*** jangutter has joined #openstack-nova07:58
*** ttsiouts has joined #openstack-nova08:00
*** markvoelker has joined #openstack-nova08:01
*** mgoddard has quit IRC08:03
*** shaohe_feng has quit IRC08:03
*** andreas_s has quit IRC08:04
*** shaohe_feng has joined #openstack-nova08:05
*** andreas_s has joined #openstack-nova08:10
*** andreas__ has joined #openstack-nova08:12
*** shaohe_feng has quit IRC08:14
*** andreas_s has quit IRC08:14
*** andreas_s has joined #openstack-nova08:15
*** shaohe_feng has joined #openstack-nova08:15
*** andreas_s has quit IRC08:15
*** andreas__ has quit IRC08:16
*** tetsuro has quit IRC08:18
*** stakeda has quit IRC08:19
*** andreas_s has joined #openstack-nova08:20
*** mago_ has joined #openstack-nova08:21
*** shaohe_feng has quit IRC08:24
*** Miouge has joined #openstack-nova08:25
*** shaohe_feng has joined #openstack-nova08:25
*** alexpilotti has quit IRC08:26
*** alexpilotti has joined #openstack-nova08:26
MiougeAs an admin, can I add a VM to a server group after it’s creation?08:26
*** vabada has quit IRC08:27
MiougeI don’t find it in the API, so is the only option to go an edit the DB and re-schedule the instances?08:27
*** vabada has joined #openstack-nova08:27
*** vabada has quit IRC08:29
*** vabada has joined #openstack-nova08:30
*** takashin has left #openstack-nova08:30
*** ykarel|lunch is now known as ykarel08:32
*** moshele has quit IRC08:33
*** youzp has quit IRC08:34
*** shaohe_feng has quit IRC08:34
*** markvoelker has quit IRC08:34
*** gibi is now known as giblet_off08:35
giblet_offI'm taking the day off08:35
*** tetsuro has joined #openstack-nova08:35
*** phuongnh has quit IRC08:36
*** phuongnh has joined #openstack-nova08:37
*** shaohe_feng has joined #openstack-nova08:37
*** moshele has joined #openstack-nova08:41
*** alexchadin has quit IRC08:43
*** shaohe_feng has quit IRC08:44
*** tetsuro has quit IRC08:45
*** tetsuro has joined #openstack-nova08:45
*** shaohe_feng has joined #openstack-nova08:46
*** rcernin has quit IRC08:46
*** sridharg has joined #openstack-nova08:50
gmanngiblet_off: take rest do not be always too hard working :)08:50
*** tetsuro has quit IRC08:51
*** GonZo2000 has joined #openstack-nova08:52
*** shaohe_feng has quit IRC08:55
*** armaan has joined #openstack-nova08:55
*** shaohe_feng has joined #openstack-nova08:56
*** GonZo2000 has quit IRC08:56
*** liuyulong_ has joined #openstack-nova08:57
*** alexchadin has joined #openstack-nova08:57
*** alexchadin has quit IRC08:57
*** alexchadin has joined #openstack-nova08:58
*** liuyulong has quit IRC08:58
*** ttsiouts has quit IRC08:59
*** huanhongda has joined #openstack-nova09:00
*** psachin has joined #openstack-nova09:01
*** trungnv has quit IRC09:01
*** tetsuro has joined #openstack-nova09:01
*** sahid has joined #openstack-nova09:02
*** alexchadin has quit IRC09:02
openstackgerrittianhui proposed openstack/nova master: Change default video model from cirrus to vga  https://review.openstack.org/58247109:03
*** neha_alhat has quit IRC09:04
*** shaohe_feng has quit IRC09:05
*** shaohe_feng has joined #openstack-nova09:06
*** armaan has quit IRC09:08
*** armaan has joined #openstack-nova09:09
*** slaweq has joined #openstack-nova09:10
*** eumel8 has joined #openstack-nova09:11
*** armaan has quit IRC09:13
*** liuyulong__ has joined #openstack-nova09:14
*** ttsiouts has joined #openstack-nova09:14
eumel8Hello Nova Cores! Maybe it's possible to import Zanata translations: https://review.openstack.org/#/c/578019/09:14
*** armaan has joined #openstack-nova09:15
*** andreas_s has quit IRC09:15
*** shaohe_feng has quit IRC09:15
*** andreas_s has joined #openstack-nova09:16
*** andreas_s has quit IRC09:16
*** liuyulong_ has quit IRC09:17
*** shaohe_feng has joined #openstack-nova09:17
*** liuyulong__ has quit IRC09:17
*** liuyulong__ has joined #openstack-nova09:18
*** mgoddard has joined #openstack-nova09:19
*** andreas_s has joined #openstack-nova09:21
*** andreas_s_ has joined #openstack-nova09:22
*** andreas_s_ has quit IRC09:23
*** shaohe_feng has quit IRC09:25
*** mgoddard has quit IRC09:26
*** andreas_s has quit IRC09:26
*** udesale has quit IRC09:27
*** udesale has joined #openstack-nova09:27
*** shaohe_feng has joined #openstack-nova09:27
*** markvoelker has joined #openstack-nova09:31
*** andreas_s has joined #openstack-nova09:34
*** alexchadin has joined #openstack-nova09:35
*** alexchadin has quit IRC09:35
*** shaohe_feng has quit IRC09:36
*** alexchadin has joined #openstack-nova09:36
*** shaohe_feng has joined #openstack-nova09:38
*** andreas_s has quit IRC09:38
*** andreas_s has joined #openstack-nova09:39
*** tetsuro has quit IRC09:39
*** alexchadin has quit IRC09:40
*** abhishekk has quit IRC09:42
*** gongysh has quit IRC09:43
*** gongysh has joined #openstack-nova09:43
openstackgerritChen proposed openstack/nova master: Clean up group members in db when delete  https://review.openstack.org/58140309:43
*** shaohe_feng has quit IRC09:46
*** shaohe_feng has joined #openstack-nova09:47
*** gongysh has quit IRC09:47
*** Bhujay has quit IRC09:50
*** Bhujay has joined #openstack-nova09:51
*** andreas_s has quit IRC09:52
*** shaohe_feng has quit IRC09:56
openstackgerrithuanhongda proposed openstack/nova-specs master: Return "deleted" and "estimated_reclaim" time of instance for admin  https://review.openstack.org/58163809:57
*** shaohe_feng has joined #openstack-nova09:57
*** rmart04 has joined #openstack-nova09:58
*** takedakn has joined #openstack-nova10:00
*** GonZo2000 has joined #openstack-nova10:01
*** GonZo2000 has quit IRC10:01
*** GonZo2000 has joined #openstack-nova10:01
*** unclok has quit IRC10:02
*** unclok has joined #openstack-nova10:03
*** markvoelker has quit IRC10:04
*** tetsuro has joined #openstack-nova10:04
*** alexchadin has joined #openstack-nova10:04
*** armaan has quit IRC10:04
*** wwriverr_ has joined #openstack-nova10:04
*** alexchadin has quit IRC10:05
*** alexchadin has joined #openstack-nova10:05
*** shaohe_feng has quit IRC10:06
openstackgerritChen proposed openstack/nova master: fix cellv2 delete_host  https://review.openstack.org/58217110:07
*** ttsiouts has quit IRC10:08
*** wwriverrat has quit IRC10:08
*** shaohe_feng has joined #openstack-nova10:09
*** ispp has quit IRC10:10
*** alexchadin has quit IRC10:10
*** alexchadin has joined #openstack-nova10:11
*** takedakn has quit IRC10:12
*** lei-zh has quit IRC10:13
*** armaan has joined #openstack-nova10:14
*** alexchadin has quit IRC10:17
*** shaohe_feng has quit IRC10:17
*** shaohe_feng has joined #openstack-nova10:18
*** shaohe_feng has quit IRC10:27
*** shaohe_feng has joined #openstack-nova10:28
*** slaweq has quit IRC10:31
*** jichen has quit IRC10:31
*** yamamoto has quit IRC10:33
*** tiendc_ has quit IRC10:37
*** shaohe_feng has quit IRC10:37
*** shaohe_feng has joined #openstack-nova10:38
*** moshele has quit IRC10:39
*** annp has quit IRC10:40
*** BrinZhang has quit IRC10:41
*** huanhongda has quit IRC10:46
stephenfineumel8: Sure, done10:47
*** sapd has quit IRC10:47
*** shaohe_feng has quit IRC10:47
*** shaohe_feng has joined #openstack-nova10:48
*** andreas_s has joined #openstack-nova10:49
*** phuongnh has quit IRC10:50
*** janki has quit IRC10:52
*** jchhatbar has joined #openstack-nova10:52
*** maciejjozefczyk has quit IRC10:52
*** yedongcan has left #openstack-nova10:55
*** yassine_ has quit IRC10:55
*** yassine has joined #openstack-nova10:56
*** yassine is now known as Guest4736910:56
*** mgoddard has joined #openstack-nova10:57
*** ispp has joined #openstack-nova10:58
*** shaohe_feng has quit IRC10:58
*** shaohe_feng has joined #openstack-nova10:59
*** andreas_s has quit IRC11:01
*** markvoelker has joined #openstack-nova11:01
*** andreas_s has joined #openstack-nova11:02
*** andreas_s has quit IRC11:03
*** liuyulong__ has quit IRC11:04
*** andreas_s has joined #openstack-nova11:07
*** ttsiouts has joined #openstack-nova11:08
*** shaohe_feng has quit IRC11:08
*** shaohe_feng has joined #openstack-nova11:09
*** lyan has joined #openstack-nova11:10
*** lyan is now known as Guest8784111:10
*** alexchadin has joined #openstack-nova11:11
*** janki has joined #openstack-nova11:12
*** jchhatbar has quit IRC11:12
*** alexchadin has quit IRC11:15
openstackgerritMerged openstack/nova master: Imported Translations from Zanata  https://review.openstack.org/57801911:16
*** shaohe_feng has quit IRC11:18
*** shaohe_feng has joined #openstack-nova11:19
*** mgoddard has quit IRC11:21
*** udesale has quit IRC11:22
*** yamamoto has joined #openstack-nova11:25
*** andreas_s has quit IRC11:26
*** armaan has quit IRC11:27
*** yamamoto has quit IRC11:28
*** shaohe_feng has quit IRC11:28
*** andreas_s has joined #openstack-nova11:31
*** shaohe_feng has joined #openstack-nova11:32
*** edmondsw has joined #openstack-nova11:33
*** yamamoto has joined #openstack-nova11:33
*** markvoelker has quit IRC11:34
*** shaohe_feng has quit IRC11:39
*** shaohe_feng has joined #openstack-nova11:39
*** tbachman has joined #openstack-nova11:42
*** dpawlik has quit IRC11:44
*** andreas_s has quit IRC11:45
*** vivsoni has quit IRC11:47
*** shaohe_feng has quit IRC11:49
*** andreas_s has joined #openstack-nova11:49
*** alexchadin has joined #openstack-nova11:50
*** shaohe_feng has joined #openstack-nova11:51
*** claudiub has joined #openstack-nova11:53
*** alexchadin has quit IRC11:53
*** janki has quit IRC11:57
*** yamamoto has quit IRC11:57
*** andreas_s has quit IRC11:58
*** shaohe_feng has quit IRC11:59
*** dillaman has joined #openstack-nova12:00
*** shaohe_feng has joined #openstack-nova12:01
*** brault has joined #openstack-nova12:03
*** brault has quit IRC12:03
*** andreas_s has joined #openstack-nova12:04
*** brault has joined #openstack-nova12:04
*** alexchadin has joined #openstack-nova12:04
*** markvoelker has joined #openstack-nova12:07
*** shaohe_feng has quit IRC12:09
eumel8thx stephenfin12:10
*** ratailor has quit IRC12:10
*** andreas_s has quit IRC12:12
*** andreas_s has joined #openstack-nova12:12
*** alexchadin has quit IRC12:12
*** shaohe_feng has joined #openstack-nova12:14
*** pooja_jadhav has quit IRC12:14
*** jamesdenton has quit IRC12:15
*** brault has quit IRC12:16
*** brault has joined #openstack-nova12:17
*** Tahvok has quit IRC12:18
*** shaohe_feng has quit IRC12:20
*** alexchadin has joined #openstack-nova12:20
*** shaohe_feng has joined #openstack-nova12:21
*** shaohe_feng has quit IRC12:30
*** medberry has joined #openstack-nova12:31
*** shaohe_feng has joined #openstack-nova12:33
*** jaypipes is now known as leakypipes12:34
*** yamamoto has joined #openstack-nova12:35
*** mriedem has joined #openstack-nova12:40
*** shaohe_feng has quit IRC12:40
*** shaohe_feng has joined #openstack-nova12:41
*** leakypipes is now known as smokypipes12:42
*** hoonetorg has quit IRC12:43
*** amoralej is now known as amoralej|lunch12:44
*** tetsuro has quit IRC12:45
*** tetsuro has joined #openstack-nova12:45
*** tetsuro has quit IRC12:45
*** yamamoto has quit IRC12:49
*** shaohe_feng has quit IRC12:50
*** shaohe_feng has joined #openstack-nova12:52
*** ttsiouts has quit IRC12:54
mriedemsahid: i replied in https://review.openstack.org/#/c/515423/ - are you aware of certain vif types that will return None for port binding profile and/or vif_details?12:54
mriedemb/c otherwise those should be coming back as dicts from the neutron api12:54
*** Bhujay has quit IRC12:55
*** damien_r has quit IRC12:55
*** Bhujay has joined #openstack-nova12:55
*** damien_r has joined #openstack-nova12:56
*** dpawlik has joined #openstack-nova12:56
*** smokypipes is now known as leakypipes12:58
*** TheJulia is now known as needssleep12:58
*** links has quit IRC12:59
*** Bhujay has quit IRC13:00
*** Bhujay has joined #openstack-nova13:00
*** dpawlik has quit IRC13:00
*** shaohe_feng has quit IRC13:01
*** shaohe_feng has joined #openstack-nova13:01
*** hoonetorg has joined #openstack-nova13:01
*** armaan has joined #openstack-nova13:04
*** alexchadin has quit IRC13:08
*** shaohe_feng has quit IRC13:11
openstackgerritSurya Seetharaman proposed openstack/nova master: Online migration tool for populating queued-for-delete  https://review.openstack.org/58253613:12
*** eharney has joined #openstack-nova13:12
*** shaohe_feng has joined #openstack-nova13:13
*** sridharg has quit IRC13:13
*** jichen has joined #openstack-nova13:14
*** armaan has quit IRC13:16
*** medberry has quit IRC13:17
*** bnemec has joined #openstack-nova13:18
*** mlavalle has joined #openstack-nova13:19
*** mdnadeem_ has quit IRC13:21
*** shaohe_feng has quit IRC13:21
*** edleafe is now known as figleaf13:21
*** yamamoto has joined #openstack-nova13:22
*** shaohe_feng has joined #openstack-nova13:22
*** armaan has joined #openstack-nova13:23
*** armaan has quit IRC13:25
*** armaan has joined #openstack-nova13:26
*** yamamoto has quit IRC13:27
*** brault has quit IRC13:28
openstackgerritMerged openstack/nova master: Adapt _validate_instance_group_policy to new policy model  https://review.openstack.org/57146513:30
*** shaohe_feng has quit IRC13:31
*** cfriesen has joined #openstack-nova13:32
*** shaohe_feng has joined #openstack-nova13:33
*** armaan has quit IRC13:33
mriedemoh most illustrious, handsome and just dansmith, i draw your attention to https://review.openstack.org/#/c/568542/19/nova/compute/rpcapi.py@693 for discussion once i'm back from delivering my child to the quarry hill day camp13:36
*** armaan has joined #openstack-nova13:36
mriedemtl;dr i think we should drop the 5.1 rpc api bump since the interface doesn't change, and just check the compute service version in the api iff new microversion and status is queued/preparing13:37
dansmithoye that's a lot of reading for a friday13:37
dansmithI said that was one option yeah13:37
dansmithI'll read for your reasoning13:37
*** awaugama has joined #openstack-nova13:38
*** med_ has quit IRC13:38
*** armaan has quit IRC13:39
*** sahid has quit IRC13:40
*** armaan has joined #openstack-nova13:40
*** armaan has quit IRC13:40
*** shaohe_feng has quit IRC13:42
*** psachin has quit IRC13:42
*** peereb has quit IRC13:44
*** shaohe_feng has joined #openstack-nova13:44
*** mgoddard has joined #openstack-nova13:48
*** shaohe_feng has quit IRC13:52
*** shaohe_feng has joined #openstack-nova13:53
*** yamamoto has joined #openstack-nova13:58
mriedemyup 3am rambling13:59
melwittefried, leakypipes, stephenfin: reminder that non-client library release freeze (os-traits and os-vif) is next thurs July 19 so if there's anything you want to make the release, get them in by the end of next thurs13:59
mriedemyikun: ^ you should start working on the advanced policy changes to novaclient14:00
*** ttsiouts has joined #openstack-nova14:00
*** artom has quit IRC14:01
*** shaohe_feng has quit IRC14:02
melwittnovaclient release freeze is same as r-3 July 2614:03
*** shaohe_feng has joined #openstack-nova14:03
*** rmart04 has quit IRC14:04
*** ykarel is now known as ykarel|away14:05
*** jichen has quit IRC14:09
*** efried is now known as fried_rice14:11
*** Shilpa has quit IRC14:12
*** shaohe_feng has quit IRC14:12
*** shaohe_feng has joined #openstack-nova14:15
*** ispp has quit IRC14:19
*** ispp has joined #openstack-nova14:19
*** antosh has joined #openstack-nova14:22
*** shaohe_feng has quit IRC14:23
*** shaohe_feng has joined #openstack-nova14:24
*** yamamoto has quit IRC14:25
*** eharney has quit IRC14:28
mriedemdansmith: thanks; i think i'll clean this change up and then rebase the api change on top with the service version check i'm talking about, then Kevin_Zheng can flesh out the rest14:29
*** bnemec has quit IRC14:29
melwittdansmith: I wanted to get your thoughts on this approach for a bug fix before you are out next week https://review.openstack.org/58240814:30
dansmithack14:30
*** artom has joined #openstack-nova14:30
*** tidwellr has joined #openstack-nova14:31
mriedemmelwitt: btw, i've had a functional test up for one of those forever https://review.openstack.org/#/c/562041/14:32
melwittmriedem: oh, sorry, I didn't know14:32
dansmithmelwitt: we probably don't want to filter on deleted for the join, but I would think we should sort by deleted and pick the first service in the list, which would be the non-deleted one, if there is one14:32
dansmithwould that solve the bug without this patch?14:33
*** shaohe_feng has quit IRC14:33
melwittdansmith: I think it would because it would also avoid generating a UUID for a deleted service14:33
dansmithso that might be a better approach14:34
melwittService.save()ing a deleted service will always raise ServiceNotFound, for example14:34
*** shaohe_feng has joined #openstack-nova14:34
dansmithbecause of _check_minimum_version() yeah?14:35
*** hongbin has joined #openstack-nova14:35
melwittno, ServiceNotFound will raise if version checks out14:35
melwittif version is too old, we raise ServiceTooOld first before we get to the db.service_update call14:35
dansmithaight, well, anyway14:36
melwittto be clear, _check_minimum_version raises ServiceTooOld if the deleted service has too old a version, db.service_update raises ServiceNotFound for the deleted service14:36
dansmithyeah14:36
melwittdansmith: thanks, I can update to the approach you suggested14:36
dansmithI'm no opposed to what you have here, but it sounds like it's more the symptom than the cause14:36
dansmith*not14:36
melwittI suppose the potential problem with the idea of sorting by deleted is if somehow an instance does not have a non-deleted service... but I'm not sure if/how that could happen. it could definitely happen before we started blocking service delete for compute services that are still hosting instances14:39
*** ykarel|away is now known as ykarel14:39
dansmithwell, that's why I'm saying the thing you're doing here is also not bad,14:39
dansmithalthough we might should put the check in save() itself,14:40
dansmithsince any save() on a deleted service will fail, this is just one scenario14:40
dansmithpoint being, if the join can return multiple services for the instance, potentially in insert-order, then getting the first one from the list if we _don't_ sort will likely be wrong14:40
dansmithI almost feel like we've had a bug about that at some point actually14:41
*** bnemec has joined #openstack-nova14:41
melwittI considered that, but what should we do if trying to save a deleted service other than raise?14:41
dansmithsay what? :)14:41
dansmithbut you said save() will never work on deleted right?14:41
melwittoh, we don't get only the first one, we process all of them (there's a Instance.services field) and something else takes the first one (something in the API I guess?)14:42
dansmithor I guess it will if you have a proper context14:42
melwittwhen you said "put the check in save() itself" I was asking what we would do after we checked self.deleted there14:42
melwittor did you mean the min version check, just skip it if deleted14:42
*** lbragstad is now known as elbragstad14:43
*** ttsiouts has quit IRC14:43
dansmithI was going to say, if deleted, don't freak out about the version check failure in save9),14:43
dansmithbut that won't fix your problem during load14:43
*** shaohe_feng has quit IRC14:43
melwittah, ok14:43
*** shaohe_feng has joined #openstack-nova14:44
melwittI'm realizing I was wrong when I said sorting would help because it processes all services that came out of the join. my bad14:44
dansmithso why are we getting the deleted one14:45
dansmith?14:45
melwittbecause it has the same 'host' as a non-deleted one14:45
dansmithright, which means we chose the wrong one yeah?14:45
melwittif the operator has ever deleted a service and replaced it with another with the same host name14:45
*** traskat has quit IRC14:46
melwittthe field on Instance is 'services' so we get all of them and something in the API must be picking one of them14:46
melwittso we get a _from_db_object call per service returned from the join14:46
mriedemthe lazy load on instance.services is generic right?14:47
dansmithmelwitt: okay so we process all of them and try to generate uuids for all the previously-deleted ones?14:47
melwittwhat do you mean by generic?14:47
mriedemin that we don't do a targeted query to get services by instance uuid and filter out deleted ones14:48
melwittdansmith: yeah process all of them and try to generate uuids for all that do not have a uuid (including the previously deleted ones)14:48
mriedemwe call instance_get_by_uuid and join on the services table14:48
dansmithI see14:48
melwittmriedem: oh, for the lazy load I'm not sure. the scenario of the bug is the API is adding the 'services' expected attr before getting instances for the list14:49
mriedemand this is the backref14:49
mriedem    instance = orm.relationship(14:49
mriedem        "Instance",14:49
mriedem        backref='services',14:49
mriedem        primaryjoin='and_(Service.host == Instance.host,'14:49
mriedem                    'Service.binary == "nova-compute",'14:49
mriedem                    'Instance.deleted == 0)',14:49
mriedem        foreign_keys=host,14:49
mriedem    )14:49
mriedemumm, so why would we load deleted services?14:49
mriedemoh so not a lazy-load ok14:49
melwittbecause that's not filtering on Service.deleted?14:50
*** ttsiouts has joined #openstack-nova14:50
mriedemrigh14:51
mriedemquery_prefix = query_prefix.options(joinedload(column))14:51
melwittwhy would the lazy load filter out deleted services?14:52
*** amoralej|lunch is now known as amoralej14:52
mriedemmaybe it doesn't, i thought it would use the backref14:53
*** shaohe_feng has quit IRC14:53
*** shaohe_feng has joined #openstack-nova14:54
melwittthe backref says Instance.deleted == 0, not Service.deleted, so I thought it wouldn't filter out deleted service? maybe I'm missing something else14:54
mriedemoh right,14:55
*** felipemonteiro has joined #openstack-nova14:55
mriedemglossed right over that14:55
mriedemanyway, ignore me now14:55
*** brault has joined #openstack-nova14:55
*** andreas_s has quit IRC14:58
*** TxGirlGeek has joined #openstack-nova14:59
*** shaohe_feng has quit IRC15:04
*** TxGirlGeek has quit IRC15:04
*** armaan has joined #openstack-nova15:07
*** medberry has joined #openstack-nova15:08
*** shaohe_feng has joined #openstack-nova15:08
*** brault has quit IRC15:09
*** brault has joined #openstack-nova15:10
*** rpittau has quit IRC15:12
*** shaohe_feng has quit IRC15:14
*** brault has quit IRC15:14
*** bnemec has quit IRC15:15
*** shaohe_feng has joined #openstack-nova15:15
*** mgoddard has quit IRC15:16
*** eharney has joined #openstack-nova15:16
openstackgerritChen proposed openstack/nova master: fix cellv2 delete_host  https://review.openstack.org/58217115:18
*** EmilienM is now known as EvilienM15:18
*** bnemec has joined #openstack-nova15:19
*** wwriverr_ has quit IRC15:21
*** tesseract has quit IRC15:21
*** wwriverrat has joined #openstack-nova15:21
*** ThomasWhite has joined #openstack-nova15:22
*** ykarel is now known as ykarel|away15:23
mriedemtssurya: i'm reminded that we can get a HostMappingNotFound when deleting a compute service https://review.openstack.org/#/c/554920/12/nova/api/openstack/compute/services.py@243 - we were just talking about that this week (or late last week), but i can't remember if a bug was reported for it?15:24
*** shaohe_feng has quit IRC15:24
*** yamahata has joined #openstack-nova15:25
mriedemespecially since we lookup the service to delete by id, not by host (so we don't check for a host mapping when doing the service lookup)15:25
*** shaohe_feng has joined #openstack-nova15:25
*** yamamoto has joined #openstack-nova15:26
mriedemi just opened a bug anyway https://bugs.launchpad.net/nova/+bug/178162515:29
openstackLaunchpad bug 1781625 in OpenStack Compute (nova) "Deleting compute service can fail with 500 error if HostMappingNotFound" [High,Triaged] - Assigned to Matt Riedemann (mriedem)15:29
*** ttsiouts has quit IRC15:30
*** yamamoto has quit IRC15:31
*** ttsiouts has joined #openstack-nova15:33
*** jmlowe has quit IRC15:34
*** sapd has joined #openstack-nova15:34
*** shaohe_feng has quit IRC15:34
*** shaohe_feng has joined #openstack-nova15:36
tssuryamriedem: sorry was out, yeah I had opened one actually, https://bugs.launchpad.net/nova/+bug/178072715:39
openstackLaunchpad bug 1780727 in OpenStack Compute (nova) "Handle HostMappingNotFound when deleting a service" [Undecided,Confirmed] - Assigned to Surya Seetharaman (tssurya)15:39
mriedemack15:39
openstackgerritMatt Riedemann proposed openstack/nova master: Heal allocations with incomplete consumer information  https://review.openstack.org/57448815:39
openstackgerritMatt Riedemann proposed openstack/nova master: Refactor _heal_instances_in_cell  https://review.openstack.org/57789615:39
openstackgerritMatt Riedemann proposed openstack/nova master: Use consumer generation in _heal_allocations_for_instance  https://review.openstack.org/57790515:39
*** Bhujay has quit IRC15:40
mriedemtssurya: do you plan on putting up a patch for that soon? if not, i can do that quickly today15:41
*** ispp has quit IRC15:41
mriedemwe just backported the code that added the regression so i'd like to fix it soon15:41
tssuryamriedem: yea please go ahead, my day is almost over15:42
mriedemok15:42
mriedemin general,15:42
tssuryasorry for not getting around to that15:42
mriedemyou don't need to assign bugs to yourself if you're not going to immediately work on them15:42
tssuryaack15:42
*** shaohe_feng has quit IRC15:45
*** shaohe_feng has joined #openstack-nova15:45
*** slaweq has joined #openstack-nova15:49
*** abhishekk has joined #openstack-nova15:50
*** ttsiouts has quit IRC15:52
*** ykarel|away has quit IRC15:52
openstackgerritMatt Riedemann proposed openstack/nova master: Skip more rebuild tests for cells v1 job  https://review.openstack.org/58171715:53
*** tssurya has quit IRC15:53
*** READ10 has joined #openstack-nova15:53
dansmithmriedem: how far up the stack on the numa set do you want to get before we pull the plug and let the stack of stuff on the bottom drain out?15:55
*** shaohe_feng has quit IRC15:55
*** shaohe_feng has joined #openstack-nova15:56
*** felipemonteiro has quit IRC15:58
dansmithI guess the scheduler one still has a -1 on it15:58
mriedemthe "scheduler" one is really a whole mess o stuff16:00
mriedemincluding api changes and such that i'm saying should be split apart and the api change should land at the end of the series16:00
mriedemright now he's got the api changes turned on but the libvirt driver changes are coming after that, which doesn't make sense16:00
dansmithyeah, I saw he had rev'd that one and assumed it'd be ready now but looks like not16:01
mriedemi'd be ok with draining the bottom half if we had agreement on the nullable question in the request spec16:01
dansmithI was just thinking it might be good to clear that road before I go, but yeah16:01
mriedemi think the 3 options i pointed out yesterday were:16:01
mriedem1. make RequestSpec.network_metadata nullable so that create_resource_requests can return None and we just blindly set it (which is what he'd be doing as coded today),16:02
mriedem2. keep reqspec.network_metadata non-nullable and then: (a) he has to return an 'empty' NetworkMetadata if no networking or (b) leave RequestSpec.network_metadata unset16:02
mriedemi don't really like 2-a16:02
dansmithyeah I mean 2b is what I was thinking16:03
mriedemsince you can't tell if the network metadata is 'empty' because the user requested no networking, or because the network just doesn't have physnet/tunneled16:03
dansmithright16:03
mriedemi'm cool with 2-b16:03
dansmithor not 2-b?16:03
mriedemi'll pretend that didn't happen16:03
*** agrebennikov has joined #openstack-nova16:03
dansmithhaha16:04
mriedemif 2-b, we can drain the bottom half now and he'll just have to handle the 'if network_metadata is not None: reqspec.network_metadata = network_metadata" in the API16:04
mriedembut that's easy enough,16:04
*** agrebennikov_ has joined #openstack-nova16:04
mriedemand his hardware.py change are already unset-aware16:04
dansmithkinda need him to agree to that though right?16:04
mriedemstephenfin: ^ agree?16:04
* dansmith bets he's gone for the week16:04
* stephenfin looks up16:04
mriedemdansmith: do we? this is the week where you and i just take over everyone's changes16:05
dansmithheh16:05
dansmithstephenfin: it's four minutes past beer o clock, what are you still doing here?16:05
stephenfindansmith: I had beer o'clock last night and started late :(16:05
*** shaohe_feng has quit IRC16:05
*** ccamacho has quit IRC16:05
stephenfinBut we won our tag (rugby) final16:05
dansmithheh16:06
stephenfina'ight, what do you need from me, mriedem?16:06
stephenfinthe null'ness thing is still up in the air, right?16:06
mriedemstephenfin: summarized above16:06
mriedemwith dansmith and my preferred option16:06
*** shaohe_feng has joined #openstack-nova16:06
mriedemwhich also allows us to start draining the bottom half of the series16:07
* stephenfin looks _alllll_ the way up this time16:07
mriedem(11:01:38 AM) mriedem: i think the 3 options i pointed out yesterday were:16:07
mriedem(11:02:13 AM) mriedem: 1. make RequestSpec.network_metadata nullable so that create_resource_requests can return None and we just blindly set it (which is what he'd be doing as coded today),16:07
mriedem(11:02:49 AM) mriedem: 2. keep reqspec.network_metadata non-nullable and then: (a) he has to return an 'empty' NetworkMetadata if no networking or (b) leave RequestSpec.network_metadata unset16:07
mriedemwe both want 2-b16:07
*** GonZo2000 has quit IRC16:08
stephenfinI'm fine with 1 or 2-b. 2-a does feel weird16:08
mriedemok 2-b it is and we can start approving patches16:08
*** harlowja has joined #openstack-nova16:08
stephenfincool. I'll throw an eye over the last few patches again and address all those comments of yours I missed yesterday16:09
*** mschuppert has quit IRC16:10
*** GonZo2000 has joined #openstack-nova16:10
mriedemdansmith: there +W up to https://review.openstack.org/#/c/564442/16:10
*** GonZo2000 has quit IRC16:10
*** GonZo2000 has joined #openstack-nova16:10
mriedembbiab16:10
*** harlowja has quit IRC16:10
dansmithwoot16:11
*** armaan has quit IRC16:14
*** artom has quit IRC16:14
*** shaohe_feng has quit IRC16:15
*** shaohe_feng has joined #openstack-nova16:17
*** fried_rice is now known as fried_rolls16:18
*** mdbooth_ has quit IRC16:19
*** med_ has joined #openstack-nova16:24
*** med_ has quit IRC16:24
*** med_ has joined #openstack-nova16:24
*** shaohe_feng has quit IRC16:26
*** shaohe_feng has joined #openstack-nova16:27
*** yamamoto has joined #openstack-nova16:27
*** amoralej is now known as amoralej|off16:30
*** yamamoto has quit IRC16:32
stephenfindansmith: Thoughts on https://review.openstack.org/#/c/564444/14/nova/objects/fields.py@76 ?16:34
stephenfinI'd read that comment as "these are the things we don't use directly in our models" but that's not what it means. Just remove that comment?16:34
*** shaohe_feng has quit IRC16:36
dansmithstephenfin: those are aliases, because much of that file was initially all in nova,16:36
dansmithand so we did that so we could remove the top half, move it to ovo without changing all the references to things like "Set" below16:36
dansmithwell, not set, that clearly got added later16:36
dansmithas the comment says, that whole list should go away, and should have years ago16:37
dansmithyou shouldn't add to it,16:37
dansmithjust use fields.String below where you use String now16:37
stephenfinOh, so I should have used 'fields.String' directly?16:37
*** bnemec has quit IRC16:37
dansmithyes16:37
stephenfinCool. I'll fix that in this follow-up16:37
*** Guest47369 has quit IRC16:37
*** shaohe_feng has joined #openstack-nova16:38
*** jmlowe has joined #openstack-nova16:42
dansmithcommented for the credit^W^Wposterity16:43
*** bnemec has joined #openstack-nova16:45
*** Guest47369 has joined #openstack-nova16:45
*** shaohe_feng has quit IRC16:46
openstackgerritMatt Riedemann proposed openstack/nova master: Use ThreadPoolExecutor for max_concurrent_live_migrations  https://review.openstack.org/56350516:47
openstackgerritMatt Riedemann proposed openstack/nova master: Compute: add support to abort queued live migration  https://review.openstack.org/56854216:47
openstackgerritMatt Riedemann proposed openstack/nova master: Fix ServerMigrationSampleJsonTestsV2_24 to use its own sample file  https://review.openstack.org/58156216:47
*** shaohe_feng has joined #openstack-nova16:47
*** bnemec has quit IRC16:53
*** shaohe_feng has quit IRC16:56
*** artom has joined #openstack-nova16:57
*** shaohe_feng has joined #openstack-nova16:57
*** GonZo2000 has quit IRC16:59
*** mago_ has quit IRC16:59
*** imacdonn has quit IRC17:03
*** imacdonn has joined #openstack-nova17:04
*** shaohe_feng has quit IRC17:07
*** shaohe_feng has joined #openstack-nova17:08
*** burt has quit IRC17:08
*** med_ has quit IRC17:15
*** shaohe_feng has quit IRC17:17
*** shaohe_feng has joined #openstack-nova17:18
*** Guest47369 has quit IRC17:19
*** FoXMaN has quit IRC17:24
*** FoXMaN has joined #openstack-nova17:25
*** yamahata has quit IRC17:25
*** shaohe_feng has quit IRC17:27
*** yamamoto has joined #openstack-nova17:29
*** shaohe_feng has joined #openstack-nova17:29
*** Guest47369 has joined #openstack-nova17:33
*** yamamoto has quit IRC17:34
*** shaohe_feng has quit IRC17:37
*** bnemec has joined #openstack-nova17:38
*** mago_ has joined #openstack-nova17:38
openstackgerritStephen Finucane proposed openstack/nova master: conf: Add '[neutron] physnets' and related options  https://review.openstack.org/56444017:41
openstackgerritStephen Finucane proposed openstack/nova master: libvirt: Start populating NUMACell.network_metadata field  https://review.openstack.org/56444117:41
openstackgerritStephen Finucane proposed openstack/nova master: scheduler: Start utilizing RequestSpec.network_metadata  https://review.openstack.org/56445217:41
openstackgerritStephen Finucane proposed openstack/nova master: Assorted cleanups from numa-aware-vswitches series  https://review.openstack.org/58265117:41
*** shaohe_feng has joined #openstack-nova17:43
*** bnemec has quit IRC17:47
*** shaohe_feng has quit IRC17:48
*** shaohe_feng has joined #openstack-nova17:50
*** Bhujay has joined #openstack-nova17:54
*** mgoddard has joined #openstack-nova17:57
*** shaohe_feng has quit IRC17:58
*** abhishekk has quit IRC17:59
*** Bhujay has quit IRC17:59
*** shaohe_feng has joined #openstack-nova18:00
*** medberry has quit IRC18:00
*** yamahata has joined #openstack-nova18:01
*** claudiub has quit IRC18:08
*** claudiub has joined #openstack-nova18:08
*** shaohe_feng has quit IRC18:08
*** shaohe_feng has joined #openstack-nova18:09
*** claudiub|2 has joined #openstack-nova18:09
*** jistr has quit IRC18:11
*** mgoddard_ has joined #openstack-nova18:11
*** jistr has joined #openstack-nova18:11
*** claudiub has quit IRC18:13
*** mgoddard has quit IRC18:14
*** mgoddard has joined #openstack-nova18:15
*** fried_rolls is now known as fried_rice18:15
*** mgoddard_ has quit IRC18:18
*** shaohe_feng has quit IRC18:18
*** shaohe_feng has joined #openstack-nova18:19
*** mago_ has quit IRC18:24
*** tidwellr has quit IRC18:25
*** shaohe_feng has quit IRC18:29
*** shaohe_feng has joined #openstack-nova18:30
*** tssurya has joined #openstack-nova18:32
openstackgerritMatt Riedemann proposed openstack/nova master: API: add support to abort queued live migration in microversion 2.64  https://review.openstack.org/57313618:33
*** Tahvok has joined #openstack-nova18:34
*** antosh has quit IRC18:38
*** shaohe_feng has quit IRC18:39
*** shaohe_feng has joined #openstack-nova18:40
*** yamamoto has joined #openstack-nova18:41
*** yamamoto has quit IRC18:41
openstackgerritMatt Riedemann proposed openstack/nova master: Compute: add support to abort queued live migration  https://review.openstack.org/56854218:46
openstackgerritMatt Riedemann proposed openstack/nova master: Fix ServerMigrationSampleJsonTestsV2_24 to use its own sample file  https://review.openstack.org/58156218:46
openstackgerritMatt Riedemann proposed openstack/nova master: API: add support to abort queued live migration in microversion 2.64  https://review.openstack.org/57313618:46
*** mgoddard has quit IRC18:47
mriedemdansmith: i assume you're doneso18:48
dansmithsoon18:49
dansmithwassup?18:49
dansmitho18:49
mriedemlikely not a series you want to hit while you're in vacation brain18:49
*** shaohe_feng has quit IRC18:49
* dansmith nods18:49
mriedemthis is the compute service version check i added in there fwiw https://review.openstack.org/#/c/573136/8/nova/compute/api.py18:50
*** shaohe_feng has joined #openstack-nova18:50
mriedemdidn't realize that was still in a runway slot, so i guess some other intrepid core will hit those18:51
dansmithand we pass that flag from the api if the microversion was new enough?18:52
mriedemif only one were around...18:52
mriedemyeah18:52
*** zzzeek has quit IRC18:52
dansmithack yeah18:52
mriedemhttps://review.openstack.org/#/c/573136/8/nova/api/openstack/compute/server_migrations.py18:52
dansmithyeah looks okay to me at skim-level18:53
*** zzzeek has joined #openstack-nova18:57
*** tssurya has quit IRC18:59
*** shaohe_feng has quit IRC18:59
*** shaohe_feng has joined #openstack-nova19:00
*** yamamoto has joined #openstack-nova19:02
mriedemlooks like we have a new functional test race bug https://bugs.launchpad.net/nova/+bug/178164819:06
openstackLaunchpad bug 1781648 in OpenStack Compute (nova) "heal_allocations test randomly failing with "ValueError: Field `compute_node_uuid' cannot be None"" [Medium,Confirmed]19:06
openstackgerritEric Fried proposed openstack/nova master: PoC: reshaper guts with inventory two-step  https://review.openstack.org/58266319:07
*** yamamoto has quit IRC19:07
*** shaohe_feng has quit IRC19:10
*** shaohe_feng has joined #openstack-nova19:11
*** READ10 has quit IRC19:14
*** shaohe_feng has quit IRC19:20
*** shaohe_feng has joined #openstack-nova19:21
*** tssurya has joined #openstack-nova19:21
*** eharney has quit IRC19:22
*** eharney has joined #openstack-nova19:23
*** dpawlik has joined #openstack-nova19:23
openstackgerritMerged openstack/nova master: Heal allocations with incomplete consumer information  https://review.openstack.org/57448819:27
*** dpawlik has quit IRC19:28
*** tssurya_ has joined #openstack-nova19:30
*** shaohe_feng has quit IRC19:30
*** shaohe_feng has joined #openstack-nova19:31
*** eharney has quit IRC19:32
mriedemhttps://blueprints.launchpad.net/nova/+spec/ironic-rescue-mode is now in a runway slot19:36
mriedemneeds final +2s19:36
mriedempretty simple19:36
*** TxGirlGeek has joined #openstack-nova19:39
*** tssurya_ has quit IRC19:40
*** shaohe_feng has quit IRC19:40
*** shaohe_feng has joined #openstack-nova19:41
openstackgerritMatt Riedemann proposed openstack/nova master: Handle HostMappingNotFound when deleting a compute service  https://review.openstack.org/58267419:43
*** medberry has joined #openstack-nova19:46
*** shaohe_feng has quit IRC19:51
*** shaohe_feng has joined #openstack-nova19:52
openstackgerritMatt Riedemann proposed openstack/nova master: Stop using HostAPI.service_delete  https://review.openstack.org/58268020:00
*** shaohe_feng has quit IRC20:01
*** shaohe_feng has joined #openstack-nova20:01
*** yamamoto has joined #openstack-nova20:03
*** jmlowe has quit IRC20:06
*** TxGirlGeek has quit IRC20:06
*** yamamoto has quit IRC20:08
*** shaohe_feng has quit IRC20:11
*** shaohe_feng has joined #openstack-nova20:12
*** artom has quit IRC20:13
*** medberry has quit IRC20:16
*** tssurya_ has joined #openstack-nova20:18
*** shaohe_feng has quit IRC20:21
*** tssurya has quit IRC20:22
*** shaohe_feng has joined #openstack-nova20:23
*** tssurya_ has quit IRC20:23
*** awaugama has quit IRC20:23
*** tssurya has joined #openstack-nova20:23
*** boris_42_ has joined #openstack-nova20:28
*** shaohe_feng has quit IRC20:32
*** shaohe_feng has joined #openstack-nova20:35
*** mriedem has quit IRC20:40
*** shaohe_feng has quit IRC20:42
*** shaohe_feng has joined #openstack-nova20:43
*** shaohe_feng has quit IRC20:52
*** alexchadin has joined #openstack-nova20:53
*** shaohe_feng has joined #openstack-nova20:53
*** alexchadin has quit IRC20:58
*** shaohe_feng has quit IRC21:02
*** shaohe_feng has joined #openstack-nova21:03
*** yamamoto has joined #openstack-nova21:04
*** yamamoto has quit IRC21:08
*** shaohe_feng has quit IRC21:13
*** wolverineav has joined #openstack-nova21:13
*** shaohe_feng has joined #openstack-nova21:13
fried_ricemriedem, needssleep: Sent ironic-rescue-mode patches to the gate21:19
needssleepfried_rice: That is like... the best news possible for the end of a Friday!  Thanks!21:19
fried_rice:)21:19
*** shaohe_feng has quit IRC21:23
needssleepuhg, postfails in ironic's gate21:23
*** shaohe_feng has joined #openstack-nova21:24
needssleepand nova, and swift21:24
needssleepwoot21:24
*** gyee has joined #openstack-nova21:25
needssleepstatic and logs.o.o is apparently non-responsive21:25
needssleepSome chat in the infra room21:25
needssleepAnyway, I'm going to go not sit in front of a computer for a while, thanks everyone!21:26
*** d063130_ has joined #openstack-nova21:26
*** sapcc-bot has joined #openstack-nova21:26
*** tianhui_ has joined #openstack-nova21:27
*** sapcc-bot1 has quit IRC21:27
*** tianhui has quit IRC21:27
*** d063130_1 has quit IRC21:27
*** tssurya_ has joined #openstack-nova21:27
*** markvoelker_ has joined #openstack-nova21:27
*** tssurya has quit IRC21:29
*** markvoelker has quit IRC21:29
*** medberry has joined #openstack-nova21:32
*** shaohe_feng has quit IRC21:33
*** shaohe_feng has joined #openstack-nova21:33
*** shaohe_feng has quit IRC21:43
*** tidwellr has joined #openstack-nova21:44
*** shaohe_feng has joined #openstack-nova21:45
*** mchlumsky has quit IRC21:45
-openstackstatus- NOTICE: logs.openstack.org is offline, causing POST_FAILURE results from Zuul. Cause and resolution timeframe currently unknown.21:51
*** ChanServ changes topic to "logs.openstack.org is offline, causing POST_FAILURE results from Zuul. Cause and resolution timeframe currently unknown."21:51
*** shaohe_feng has quit IRC21:54
*** shaohe_feng has joined #openstack-nova21:55
*** mago_ has joined #openstack-nova21:57
*** mchlumsky has joined #openstack-nova21:58
*** itlinux_ has joined #openstack-nova22:00
*** shaohe_feng has quit IRC22:04
*** flwang1 has joined #openstack-nova22:04
*** shaohe_feng has joined #openstack-nova22:05
*** figleaf is now known as edleafe22:05
*** yamamoto has joined #openstack-nova22:05
*** mchlumsky has quit IRC22:05
*** tidwellr has quit IRC22:06
*** claudiub|2 has quit IRC22:06
*** yamamoto has quit IRC22:09
*** shaohe_feng has quit IRC22:14
*** shaohe_feng has joined #openstack-nova22:16
*** hongbin has quit IRC22:18
*** shaohe_feng has quit IRC22:24
*** shaohe_feng has joined #openstack-nova22:25
*** med_ has joined #openstack-nova22:27
*** med_ has quit IRC22:27
*** med_ has joined #openstack-nova22:27
openstackgerritEric Fried proposed openstack/nova master: perform reshaper operations in single transaction  https://review.openstack.org/58238322:28
openstackgerritEric Fried proposed openstack/nova master: WIP: [placement] Add /reshaper handler for POST  https://review.openstack.org/57692722:28
openstackgerritEric Fried proposed openstack/nova master: WIP: Compute: Handle reshaped provider trees  https://review.openstack.org/57623622:28
*** edmondsw has quit IRC22:31
*** shaohe_feng has quit IRC22:35
*** fried_rice is now known as efried22:36
*** shaohe_feng has joined #openstack-nova22:36
*** med_ has quit IRC22:39
*** shaohe_feng has quit IRC22:45
*** shaohe_feng has joined #openstack-nova22:46
*** itlinux_ has quit IRC22:49
*** shaohe_feng has quit IRC22:55
*** shaohe_feng has joined #openstack-nova22:57
*** slaweq has quit IRC23:03
*** shaohe_feng has quit IRC23:05
*** yamamoto has joined #openstack-nova23:06
*** shaohe_feng has joined #openstack-nova23:08
*** yamamoto has quit IRC23:11
*** shaohe_feng has quit IRC23:16
*** shaohe_feng has joined #openstack-nova23:16
*** agrebennikov_ has quit IRC23:22
*** agrebennikov has quit IRC23:22
*** dpawlik has joined #openstack-nova23:24
*** tssurya_ has quit IRC23:26
*** shaohe_feng has quit IRC23:26
*** edmondsw has joined #openstack-nova23:26
*** shaohe_feng has joined #openstack-nova23:27
*** alexchadin has joined #openstack-nova23:27
*** jmlowe has joined #openstack-nova23:28
*** dpawlik has quit IRC23:28
*** tidwellr has joined #openstack-nova23:31
*** leakypipes is now known as jaypipes23:31
*** edmondsw has quit IRC23:31
*** alexchadin has quit IRC23:32
*** ChanServ changes topic to "Current runways: abort-live-migration-in-queued-status / add-zvm-driver-rocky / glance-image-traits -- This channel is for Nova development. For support of Nova deployments, please use #openstack."23:35
-openstackstatus- NOTICE: logs.openstack.org is back on-line. Changes with "POST_FAILURE" job results should be rechecked.23:35
*** shaohe_feng has quit IRC23:36
*** TxGirlGeek has joined #openstack-nova23:37
*** shaohe_feng has joined #openstack-nova23:40
*** wolverineav has quit IRC23:44
*** wolverineav has joined #openstack-nova23:45
*** shaohe_feng has quit IRC23:47
*** shaohe_feng has joined #openstack-nova23:47
*** mago_ has quit IRC23:48
*** wolverineav has quit IRC23:49
*** tidwellr has quit IRC23:53
*** TxGirlGeek has quit IRC23:55
*** shaohe_feng has quit IRC23:57
*** shaohe_feng has joined #openstack-nova23:59

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