Tuesday, 2018-01-02

*** jmlowe_ has quit IRC00:00
*** jmlowe has joined #openstack-nova00:00
*** nore_rabel has quit IRC00:02
*** tanee has quit IRC00:04
*** nore_rabel has joined #openstack-nova00:10
*** tanee has joined #openstack-nova00:11
*** nore_rabel has quit IRC00:16
*** markvoelker has joined #openstack-nova00:27
*** markvoelker has quit IRC00:31
*** ljjjustin has joined #openstack-nova00:33
*** zhurong has joined #openstack-nova00:43
*** hoangcx has joined #openstack-nova01:02
*** namnh has joined #openstack-nova01:03
*** phuongnh has joined #openstack-nova01:07
*** yangyapeng has joined #openstack-nova01:10
*** heyongli has joined #openstack-nova01:17
*** heyongli has quit IRC01:27
*** heyongli has joined #openstack-nova01:27
*** markvoelker has joined #openstack-nova01:28
*** xinliang has joined #openstack-nova01:29
*** markvoelker has quit IRC01:32
openstackgerritEli Qiao proposed openstack/nova master: Api-guide: Add Block Device Mapping  https://review.openstack.org/52208401:34
*** heyongli has quit IRC01:37
*** heyongli has joined #openstack-nova01:37
*** inara has quit IRC01:45
*** heyongli has quit IRC01:47
*** heyongli has joined #openstack-nova01:47
*** inara has joined #openstack-nova01:47
*** Tom-Tom has joined #openstack-nova01:52
*** archit has joined #openstack-nova01:55
*** archit_ has joined #openstack-nova01:55
*** archit is now known as amodi01:55
*** heyongli has quit IRC01:57
*** heyongli has joined #openstack-nova01:58
*** amodi has quit IRC02:05
*** gcb has joined #openstack-nova02:06
*** heyongli has quit IRC02:08
*** heyongli has joined #openstack-nova02:08
*** Tom-Tom has quit IRC02:13
*** zhenguo has joined #openstack-nova02:14
*** heyongli has quit IRC02:18
*** heyongli has joined #openstack-nova02:18
*** yangyapeng has quit IRC02:22
*** yangyapeng has joined #openstack-nova02:23
*** heyongli has quit IRC02:28
*** heyongli has joined #openstack-nova02:28
*** artom_ has quit IRC02:29
*** artom_ has joined #openstack-nova02:29
*** artom_ has joined #openstack-nova02:30
*** heyongli has quit IRC02:38
*** heyongli has joined #openstack-nova02:39
*** trungnv has joined #openstack-nova02:43
*** edmondsw has joined #openstack-nova02:45
*** heyongli has quit IRC02:49
*** heyongli has joined #openstack-nova02:49
*** yangyape_ has joined #openstack-nova02:55
*** yangyapeng has quit IRC02:55
*** heyongli has quit IRC02:59
*** heyongli has joined #openstack-nova02:59
*** edmondsw has quit IRC03:03
*** edmondsw has joined #openstack-nova03:05
*** edmondsw has quit IRC03:06
*** tbachman has joined #openstack-nova03:07
*** heyongli has quit IRC03:09
*** heyongli has joined #openstack-nova03:09
*** bkopilov has quit IRC03:10
*** suresh12 has joined #openstack-nova03:10
*** annp has joined #openstack-nova03:14
*** heyongli has quit IRC03:19
*** heyongli has joined #openstack-nova03:20
*** yamamoto has quit IRC03:27
*** markvoelker has joined #openstack-nova03:29
*** heyongli has quit IRC03:30
*** heyongli has joined #openstack-nova03:30
*** links has joined #openstack-nova03:34
*** markvoelker has quit IRC03:34
*** heyongli has quit IRC03:40
*** heyongli has joined #openstack-nova03:40
*** yangyapeng has joined #openstack-nova03:41
*** yangyape_ has quit IRC03:41
*** suresh12 has quit IRC03:48
*** heyongli has quit IRC03:50
*** heyongli has joined #openstack-nova03:50
*** dave-mccowan has quit IRC03:51
*** abhishekk has joined #openstack-nova03:53
*** tbachman has quit IRC03:57
*** heyongli has quit IRC04:00
*** heyongli has joined #openstack-nova04:01
*** yamamoto has joined #openstack-nova04:01
*** pooja_jadhav has joined #openstack-nova04:03
*** heyongli has quit IRC04:11
*** heyongli has joined #openstack-nova04:11
*** vivsoni has quit IRC04:11
*** jaianshu has joined #openstack-nova04:12
*** yamamoto has quit IRC04:17
*** heyongli has quit IRC04:21
*** heyongli has joined #openstack-nova04:21
*** vivsoni has joined #openstack-nova04:23
*** markvoelker has joined #openstack-nova04:30
*** heyongli has quit IRC04:31
*** bkopilov has joined #openstack-nova04:31
*** heyongli has joined #openstack-nova04:31
*** markvoelker has quit IRC04:35
*** edmondsw has joined #openstack-nova04:35
*** yamamoto has joined #openstack-nova04:36
*** nicolasbock has quit IRC04:38
*** edmondsw has quit IRC04:40
*** yamamoto has quit IRC04:41
*** heyongli has quit IRC04:41
*** heyongli has joined #openstack-nova04:42
*** udesale has joined #openstack-nova04:43
*** suresh12 has joined #openstack-nova04:44
*** bhagyashris has joined #openstack-nova04:45
*** heyongli has quit IRC04:52
*** heyongli has joined #openstack-nova04:52
*** heyongli has quit IRC05:02
*** heyongli has joined #openstack-nova05:02
*** heyongli has quit IRC05:12
*** heyongli has joined #openstack-nova05:12
*** sridharg has joined #openstack-nova05:13
*** heyongli has quit IRC05:22
*** heyongli has joined #openstack-nova05:23
*** zhurong has quit IRC05:23
*** karthiks has joined #openstack-nova05:24
*** psachin has joined #openstack-nova05:32
*** heyongli has quit IRC05:33
*** heyongli has joined #openstack-nova05:33
*** heyongli has quit IRC05:43
*** heyongli has joined #openstack-nova05:43
*** yamamoto has joined #openstack-nova05:44
*** rui has joined #openstack-nova05:48
*** yamamoto has quit IRC05:50
*** hongbin has joined #openstack-nova05:52
*** heyongli has quit IRC05:53
*** heyongli has joined #openstack-nova05:53
*** heyongli has quit IRC06:03
*** heyongli has joined #openstack-nova06:04
*** heyongli has quit IRC06:14
*** heyongli has joined #openstack-nova06:14
*** jaianshu_ has joined #openstack-nova06:16
*** jaianshu has quit IRC06:20
*** tuanla____ has joined #openstack-nova06:22
*** edmondsw has joined #openstack-nova06:23
*** heyongli has quit IRC06:24
*** heyongli has joined #openstack-nova06:24
*** edmondsw has quit IRC06:28
*** suresh12 has quit IRC06:29
*** ratailor has joined #openstack-nova06:30
*** heyongli has quit IRC06:34
*** heyongli has joined #openstack-nova06:34
*** Tom-Tom has joined #openstack-nova06:37
*** Tom-Tom has quit IRC06:44
*** heyongli has quit IRC06:44
*** heyongli has joined #openstack-nova06:45
openstackgerritAmeed Ashour proposed openstack/nova master: change instance_system_metadata column type  https://review.openstack.org/52690006:54
*** heyongli has quit IRC06:55
*** heyongli has joined #openstack-nova06:55
*** ameeda has joined #openstack-nova06:56
*** ameeda has quit IRC06:56
*** ameeda has joined #openstack-nova06:57
*** gmann__ has joined #openstack-nova07:00
gmann__alex_xu: ping07:00
gmann__alex_xu: regarding this - https://review.openstack.org/#/c/526258/3/nova/policies/security_groups.py07:01
gmann__alex_xu: in server response we show only security_groups name not details like you mentioned the case of keypair.07:01
gmann__alex_xu i got your point of leaking information of sec group in server show if sec group things are hidden via policy on os-security-group api07:03
gmann__alex_xu: i am considering the same case for keypair also.07:03
*** heyongli has quit IRC07:05
gmann__but with open question like - is name (sec group, key pair) something we should hide in server response  if sec group or keypair specific APIs are not permit in policy ?07:05
*** heyongli has joined #openstack-nova07:05
gmann__or we consider that is not harmful to show only name in server response without policy control .07:05
openstackgerritJianle He proposed openstack/nova-specs master: Back up volume-backed-instance  https://review.openstack.org/53021407:06
*** jaosorior has joined #openstack-nova07:07
alex_xugmann__: ah, right, we only have the sec_group name in the server API07:10
gmann__yea07:11
alex_xugmann__: If the admin doesn't allow the user to access the keypairs API, I guess the admin doesn't want to know the detail about the keypairs07:12
alex_xusame as sec_group07:12
gmann__yea07:12
*** rui has quit IRC07:13
alex_xugmann__: so I think we are good07:13
gmann__alex_xu: ok, thanks. please check the deprecation msg if looks good or we need more/short info.07:14
alex_xugmann__: I think they are good07:14
gmann__k07:14
*** heyongli has quit IRC07:15
*** heyongli has joined #openstack-nova07:15
openstackgerritOpenStack Proposal Bot proposed openstack/python-novaclient master: Updated from global requirements  https://review.openstack.org/53069707:15
gmann__alex_xu: thanks for review :)07:17
openstackgerritOpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata  https://review.openstack.org/52479507:18
alex_xugmann__: np07:20
*** gmann__ is now known as gmann07:23
*** pcaruana has joined #openstack-nova07:24
openstackgerritAmeed Ashour proposed openstack/nova master: change instance_system_metadata column type  https://review.openstack.org/52690007:24
openstackgerritChen Hanxiao proposed openstack/nova master: manager: more detailed info of unsupported compute driver  https://review.openstack.org/53070107:24
*** pcaruana has quit IRC07:25
*** heyongli has quit IRC07:25
*** pcaruana has joined #openstack-nova07:26
*** heyongli has joined #openstack-nova07:26
*** pcaruana has quit IRC07:29
*** pcaruana has joined #openstack-nova07:29
*** markvoelker has joined #openstack-nova07:32
*** heyongli has quit IRC07:36
*** heyongli has joined #openstack-nova07:36
openstackgerritAmeed Ashour proposed openstack/nova master: change instance_system_metadata column type  https://review.openstack.org/52690007:45
*** Tom-Tom has joined #openstack-nova07:46
*** heyongli has quit IRC07:46
*** heyongli has joined #openstack-nova07:46
*** larainema has quit IRC07:53
*** hongbin has quit IRC07:54
*** Tom-Tom has quit IRC07:56
*** heyongli has quit IRC07:56
*** heyongli has joined #openstack-nova07:56
*** jaosorior has quit IRC07:58
*** jaosorior has joined #openstack-nova08:00
*** jafeha has joined #openstack-nova08:04
*** heyongli has quit IRC08:06
*** heyongli has joined #openstack-nova08:07
*** edmondsw has joined #openstack-nova08:12
*** Tom-Tom has joined #openstack-nova08:14
*** edmondsw has quit IRC08:16
*** ragiman has joined #openstack-nova08:16
*** heyongli has quit IRC08:17
*** heyongli has joined #openstack-nova08:17
*** tesseract has joined #openstack-nova08:20
*** Tom-Tom has quit IRC08:24
*** heyongli has quit IRC08:27
*** heyongli has joined #openstack-nova08:27
*** pooja_jadhav has quit IRC08:30
*** pooja_jadhav has joined #openstack-nova08:32
*** heyongli has quit IRC08:37
*** heyongli has joined #openstack-nova08:37
*** danpawlik has joined #openstack-nova08:39
*** heyongli has quit IRC08:47
*** heyongli has joined #openstack-nova08:48
*** cdent has joined #openstack-nova08:49
*** moshele has joined #openstack-nova08:49
*** purplerbot has joined #openstack-nova08:50
*** mdnadeem has joined #openstack-nova08:51
*** purplerbot has quit IRC08:54
*** purplerbot has joined #openstack-nova08:55
*** heyongli has quit IRC08:58
*** heyongli has joined #openstack-nova08:58
*** ljjjustin has quit IRC08:59
openstackgerritIldiko Vancsa proposed openstack/nova master: WIP: libvirt: Allow multiple volume attachments  https://review.openstack.org/26758709:02
openstackgerritIldiko Vancsa proposed openstack/nova master: WIP: Allow multi-attach in compute api  https://review.openstack.org/27104709:02
*** ragiman has quit IRC09:02
*** yassine has joined #openstack-nova09:06
*** yassine is now known as Guest1072609:06
*** heyongli has quit IRC09:08
*** heyongli has joined #openstack-nova09:08
*** Tom-Tom has joined #openstack-nova09:14
*** ragiman has joined #openstack-nova09:16
*** linpopilan has joined #openstack-nova09:18
*** heyongli has quit IRC09:18
*** heyongli has joined #openstack-nova09:18
*** mvk has joined #openstack-nova09:18
*** heyongli has quit IRC09:28
*** heyongli has joined #openstack-nova09:29
*** lyarwood has joined #openstack-nova09:33
*** lucas-afk is now known as lucasagomes09:33
*** heyongli has quit IRC09:39
*** heyongli has joined #openstack-nova09:39
openstackgerritIldiko Vancsa proposed openstack/nova master: [libvirt] Allow multiple volume attachments  https://review.openstack.org/26758709:48
openstackgerritIldiko Vancsa proposed openstack/nova master: [api] Allow multi-attach in compute api  https://review.openstack.org/27104709:48
*** heyongli has quit IRC09:49
*** heyongli has joined #openstack-nova09:49
*** ragiman has quit IRC09:51
*** yikun_jiang has quit IRC09:52
*** yikun_jiang has joined #openstack-nova09:52
*** derekh has joined #openstack-nova09:55
*** heyongli has quit IRC09:59
*** heyongli has joined #openstack-nova09:59
*** edmondsw has joined #openstack-nova10:00
*** ragiman has joined #openstack-nova10:03
*** edmondsw has quit IRC10:04
*** sambetts_ is now known as sambetts10:07
openstackgerritLee Yarwood proposed openstack/nova master: libvirt: Refactor encryptor attach and detach calls  https://review.openstack.org/46024310:09
openstackgerritLee Yarwood proposed openstack/nova master: libvirt: Introduce disk encryption config classes  https://review.openstack.org/46400810:09
*** heyongli has quit IRC10:09
*** heyongli has joined #openstack-nova10:10
*** Eran_Kuris has quit IRC10:11
*** cdent has quit IRC10:12
*** phuongnh has quit IRC10:13
*** namnh has quit IRC10:14
*** jaianshu__ has joined #openstack-nova10:15
*** jaianshu_ has quit IRC10:18
*** Eran_Kuris has joined #openstack-nova10:19
*** gmann has quit IRC10:19
*** heyongli has quit IRC10:20
*** heyongli has joined #openstack-nova10:20
*** liusheng has quit IRC10:23
*** liusheng has joined #openstack-nova10:23
*** heyongli has quit IRC10:30
*** heyongli has joined #openstack-nova10:30
*** markvoelker has quit IRC10:32
openstackgerritRajesh Tailor proposed openstack/nova master: Host addition host-aggregate should be case-sensitive  https://review.openstack.org/49833410:35
*** heyongli has quit IRC10:40
*** heyongli has joined #openstack-nova10:40
*** cdent has joined #openstack-nova10:43
*** heyongli has quit IRC10:50
*** heyongli has joined #openstack-nova10:51
*** udesale has quit IRC10:56
*** Tom-Tom has quit IRC10:59
*** Guest10726 has quit IRC11:00
*** heyongli has quit IRC11:01
*** heyongli has joined #openstack-nova11:01
*** heyongli has quit IRC11:11
*** heyongli has joined #openstack-nova11:11
*** kuzko has joined #openstack-nova11:14
*** abhishekk has quit IRC11:16
*** heyongli has quit IRC11:21
*** heyongli has joined #openstack-nova11:21
*** ratailor has quit IRC11:21
*** bkopilov has quit IRC11:23
*** heyongli has quit IRC11:31
*** heyongli has joined #openstack-nova11:32
*** markvoelker has joined #openstack-nova11:32
*** markvoelker has quit IRC11:37
*** mdnadeem has quit IRC11:40
*** heyongli has quit IRC11:42
*** heyongli has joined #openstack-nova11:42
*** heyongli has quit IRC11:52
*** heyongli has joined #openstack-nova11:52
*** nicolasbock has joined #openstack-nova11:58
*** Tom-Tom has joined #openstack-nova11:59
*** annp has quit IRC12:01
openstackgerritStephen Finucane proposed openstack/nova master: Address nits in I46d483f9de6776db1b025f925890624e5e682ada  https://review.openstack.org/53046212:01
*** heyongli has quit IRC12:02
*** heyongli has joined #openstack-nova12:02
*** Tom-Tom has quit IRC12:03
*** mdnadeem has joined #openstack-nova12:04
*** tbachman has joined #openstack-nova12:11
*** tbachman has quit IRC12:11
*** heyongli has quit IRC12:12
*** heyongli has joined #openstack-nova12:13
*** links has quit IRC12:13
ameedaHi12:19
*** edmondsw has joined #openstack-nova12:19
ameedacan you please review my code here https://review.openstack.org/#/c/528069/ ?12:19
*** edmondsw has quit IRC12:20
bhagyashrisianw_pto: Hi, are you around?12:21
*** moshele has quit IRC12:22
*** heyongli has quit IRC12:23
*** heyongli has joined #openstack-nova12:23
*** Tom-Tom has joined #openstack-nova12:30
*** lucasagomes is now known as lucas-hugry12:30
*** heyongli has quit IRC12:33
*** moshele has joined #openstack-nova12:33
*** heyongli has joined #openstack-nova12:33
*** markvoelker has joined #openstack-nova12:34
*** dave-mccowan has joined #openstack-nova12:35
*** Tom-Tom has quit IRC12:35
*** moshele has quit IRC12:36
*** jaypipes has joined #openstack-nova12:37
*** moshele has joined #openstack-nova12:37
*** markvoelker has quit IRC12:38
*** tuanla____ has quit IRC12:40
*** smatzek has joined #openstack-nova12:41
*** heyongli has quit IRC12:43
*** heyongli has joined #openstack-nova12:43
ameedacan you please review my code here https://review.openstack.org/#/c/528069/ ?12:53
*** heyongli has quit IRC12:53
*** heyongli has joined #openstack-nova12:54
openstackgerritTakashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in delete (2)  https://review.openstack.org/52626312:54
openstackgerritTakashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in delete (3)  https://review.openstack.org/52655712:54
openstackgerritTakashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in post  https://review.openstack.org/52682312:55
openstackgerritTakashi NATSUME proposed openstack/nova master: api-ref: Parameter verification for servers.inc  https://review.openstack.org/52820112:55
openstackgerritTakashi NATSUME proposed openstack/nova master: api-ref: Example verification for servers.inc  https://review.openstack.org/52952012:55
openstackgerritTakashi NATSUME proposed openstack/nova master: api-ref: Fix parameter order in rebuild  https://review.openstack.org/52971812:56
openstackgerritTakashi NATSUME proposed openstack/python-novaclient master: Add CLI to show instance usage audit logs  https://review.openstack.org/40941312:57
openstackgerritTakashi NATSUME proposed openstack/nova master: List/show all server migration types (1/2)  https://review.openstack.org/43060812:57
*** larainema has joined #openstack-nova12:57
openstackgerritTakashi NATSUME proposed openstack/nova master: List/show all server migration types (2/2)  https://review.openstack.org/45948312:58
*** tbachman has joined #openstack-nova12:58
openstackgerritTakashi NATSUME proposed openstack/nova master: [cellv2] Improve getting BDMs in multiple cells  https://review.openstack.org/52140012:58
openstackgerritTakashi NATSUME proposed openstack/nova master: Fix 500 error while passing 4-byte unicode data  https://review.openstack.org/40751412:58
openstackgerritTakashi NATSUME proposed openstack/python-novaclient master: Microversion 2.59 - List/Show all server migration types  https://review.openstack.org/43083912:59
openstackgerritTakashi NATSUME proposed openstack/nova master: [placement] Add functional tests for resource class API  https://review.openstack.org/52450612:59
openstackgerritTakashi NATSUME proposed openstack/nova master: [placement] Add functional tests for traits API  https://review.openstack.org/52409412:59
*** tbachman_ has joined #openstack-nova13:00
*** liverpooler has joined #openstack-nova13:00
openstackgerritTakashi NATSUME proposed openstack/nova master: Add a warning in 'nova-manage cell_v2 delete_cell'  https://review.openstack.org/51377113:00
openstackgerritTakashi NATSUME proposed openstack/nova master: Fix the order of target host checks  https://review.openstack.org/52622513:00
openstackgerritTakashi NATSUME proposed openstack/nova master: [placement] Separate API schemas (resource_provider)  https://review.openstack.org/52862913:00
openstackgerritTakashi NATSUME proposed openstack/nova master: Fix missing marker functions  https://review.openstack.org/51457913:01
openstackgerritTakashi NATSUME proposed openstack/nova master: [placement] Move body examples to an isolated directory  https://review.openstack.org/52999813:01
*** tbachman has quit IRC13:02
*** tbachman_ is now known as tbachman13:02
*** moshele has quit IRC13:03
*** heyongli has quit IRC13:04
*** heyongli has joined #openstack-nova13:04
openstackgerritAmeed Ashour proposed openstack/nova master: change instance_system_metadata column type  https://review.openstack.org/52690013:04
*** moshele has joined #openstack-nova13:05
*** yikun_jiang has quit IRC13:13
*** yikun_jiang has joined #openstack-nova13:13
*** heyongli has quit IRC13:14
*** heyongli has joined #openstack-nova13:14
*** gmann has joined #openstack-nova13:18
*** dave-mccowan has quit IRC13:21
*** heyongli has quit IRC13:24
*** heyongli has joined #openstack-nova13:24
openstackgerritKashyap Chamarthy proposed openstack/nova-specs master: Add UEFI Secure Boot support for QEMU/KVM guests, using OVMF  https://review.openstack.org/50672013:26
*** markvoelker has joined #openstack-nova13:28
*** yassine has joined #openstack-nova13:28
*** yassine has quit IRC13:28
kashyapstephenfin: When you get a moment, in the above spec, you can see all those lines greyed out due to indentation13:29
*** yassine has joined #openstack-nova13:29
kashyapstephenfin: But if you read it, you see my intention for why I indented it that way.13:29
*** yassine is now known as Guest7410813:29
*** Tom-Tom has joined #openstack-nova13:30
openstackgerritMerged openstack/python-novaclient master: boot: error out if no images match the property from --image-with  https://review.openstack.org/53042613:31
stephenfinkashyap: Cool. Just waiting for the build to complete13:32
kashyapstephenfin: No problem; maybe it's 6 spaces vs 8 spaces indentation.13:33
*** jaianshu__ has quit IRC13:33
kashyapI saw a half-finished sentence, fixing that, and indentation13:33
*** cleong has joined #openstack-nova13:34
*** heyongli has quit IRC13:34
jaypipesstephenfin, kashyap: morning fellers.13:34
stephenfino/13:34
kashyapMorning, and happy new year.13:34
jaypipeshappy 201813:34
*** heyongli has joined #openstack-nova13:35
jaypipesfuck 2017.13:35
stephenfinLikewise13:35
stephenfinon both counts13:35
*** udesale has joined #openstack-nova13:36
openstackgerritKashyap Chamarthy proposed openstack/nova-specs master: Add UEFI Secure Boot support for QEMU/KVM guests, using OVMF  https://review.openstack.org/50672013:36
kashyapYeah, fugree with stephenfin13:37
*** stvnoyes has joined #openstack-nova13:37
kashyapjaypipes: When you get a moment, I'd appreciate your view on the above spec, it's a long (and a bit dry read, tried to make it as readable as I can).  My eyes need some fresh perspective to be able to rework it further13:37
kashyapjaypipes: More specifically, I'm looking for comments in the 'Work Items' section13:38
jaypipeskashyap: yep, saw that in my email this morning... will look later today when I'm properly caffeinated.13:38
kashyapWhere you see a couple of: "TODO / Need Opinions" sections13:38
*** lucas-hugry is now known as lucasagomes13:38
kashyapjaypipes: Sure, caffine, very important.  Or else, it might irrirate the delicate neurons.13:38
kashyapThanks!13:38
* kashyap now goes to have a bite13:39
jaypipesnp :)13:39
*** edmondsw has joined #openstack-nova13:39
openstackgerritStephen Finucane proposed openstack/nova master: Add an online migration for PciDevice.uuid  https://review.openstack.org/53048713:40
*** moshele has quit IRC13:40
*** moshele has joined #openstack-nova13:41
*** edmondsw has quit IRC13:44
*** heyongli has quit IRC13:45
*** heyongli has joined #openstack-nova13:45
*** efried has joined #openstack-nova13:46
efriedGood morning nova13:47
*** eharney has joined #openstack-nova13:47
cdentwelcome back efried. apologies but I utterly failed to do much with regard to the stuff I said I would do while you were away, sickness and other stuff got in the way. I believe stephenfin manage to keep some of your stuff up to date, though13:48
efriedcdent No worries, hope you're feeling better.  I saw some rebasing and minor updates by stephenfin and others, but actually haven't logged in at all since last Sunday - so I have some catching up to do.13:50
cdenta) good on you for staying away, b) any urgency is false, c) still coughing, but not in bed13:50
efriedWow, sounds like a doozy13:51
efriedAny idea who's still going to be out this week?13:51
jaypipesefried: welcome back! :)13:52
efriedjaypipes Thanks!13:52
*** ygl has joined #openstack-nova13:52
openstackgerritStephen Finucane proposed openstack/nova stable/pike: Fix disk size during live migration with disk over-commit  https://review.openstack.org/53074313:53
openstackgerritStephen Finucane proposed openstack/nova stable/ocata: Fix disk size during live migration with disk over-commit  https://review.openstack.org/53074413:53
*** ygl has left #openstack-nova13:54
efriedLovely, 17 of my open patches are in merge conflict.13:54
jaypipesefried, cdent: I'm going to grab some breakfast here and will be back in around an hour, but I'd love to have a google hangout with you guys (and edleafe/alex_xu and anyone else placement-focused) when I return.13:54
efriedjaypipes Sounds good to me.13:55
*** heyongli has quit IRC13:55
cdentjaypipes: that sounds like a great idea, but I'm in public coffee land, so a bit later in the day would be better for me13:55
*** heyongli has joined #openstack-nova13:55
jaypipescdent: gotcha, np13:55
*** mriedem has joined #openstack-nova13:56
*** mlavalle has joined #openstack-nova13:56
*** liverpooler has quit IRC13:58
*** lyan has joined #openstack-nova14:01
*** heyongli has quit IRC14:05
*** heyongli has joined #openstack-nova14:05
*** yikun_jiang has quit IRC14:12
*** yikun has joined #openstack-nova14:13
*** edmondsw has joined #openstack-nova14:14
*** heyongli has quit IRC14:15
*** heyongli has joined #openstack-nova14:16
*** peter-hamilton has joined #openstack-nova14:21
stephenfinkashyap: I assume you fixed the problem with https://review.openstack.org/#/c/506720/ ? I can't see anything wrong with it now14:23
kashyapstephenfin: Is it the "Upgrade Impact" thing?14:24
kashyap_Just_ looking at the error14:24
kashyaphttp://logs.openstack.org/20/506720/4/check/openstack-tox-pep8/1f31318/job-output.txt.gz14:24
kashyap2018-01-02 13:45:19.289008 | ubuntu-xenial | AssertionError: While checking 'specs/rocky/approved/ovmf-with-secure-boot-and-smm.rst':14:24
kashyap2018-01-02 13:45:19.289047 | ubuntu-xenial |   Section 'Proposed change' is missing subsections: [u'Upgrade impact']14:24
*** jobewan has joined #openstack-nova14:24
stephenfinOh, the pep8 job. Yeah, I guess so14:24
stephenfinBut the grey blocks you were on about earlier: I can't see anything wrong in that regard14:25
kashyapHmm14:25
kashyapI'll just leave it at that for now.14:25
*** heyongli has quit IRC14:26
*** heyongli has joined #openstack-nova14:26
*** abhishekk has joined #openstack-nova14:27
jrolljaypipes: I may or may not be here (have an appt in an hour), but would like a ping for that hangout if it happens14:29
*** moshele has quit IRC14:29
*** nore_rabel has joined #openstack-nova14:29
*** abhishekk has quit IRC14:32
openstackgerritKashyap Chamarthy proposed openstack/nova-specs master: Add UEFI Secure Boot support for QEMU/KVM guests, using OVMF  https://review.openstack.org/50672014:32
*** moshele has joined #openstack-nova14:33
alex_xujaypipes: yea, i still wake up \o/14:35
*** heyongli has quit IRC14:36
cdentjroll, jaypipes, alex_xu if it needs to happen sooner than later so as to include more folk, I'm happy to listen in and speak rarely14:36
openstackgerritIldiko Vancsa proposed openstack/nova master: Add support for getting volume details with a specified microversion  https://review.openstack.org/52965614:36
openstackgerritIldiko Vancsa proposed openstack/nova master: Use volume shared_targets to lock during attach/detach  https://review.openstack.org/52969514:36
*** heyongli has joined #openstack-nova14:36
alex_xucdent: jaypipes, no, just pick the time good for you guys, i'm really close to sleep time :)14:36
*** psachin has quit IRC14:37
*** tbachman has quit IRC14:37
*** esberglu has joined #openstack-nova14:37
*** mdrabe_ is now known as mdrabe14:38
jrollcdent: you're more involved than me, I'm still trying to catch back up on things. I should be back by 1600ish anyway.14:38
*** bkopilov has joined #openstack-nova14:39
*** hongbin has joined #openstack-nova14:41
*** trozet has joined #openstack-nova14:45
*** READ10 has joined #openstack-nova14:45
*** heyongli has quit IRC14:46
*** heyongli has joined #openstack-nova14:46
*** pchavva has joined #openstack-nova14:52
*** heyongli has quit IRC14:56
*** heyongli has joined #openstack-nova14:57
*** efried has quit IRC15:02
*** dave-mccowan has joined #openstack-nova15:02
*** jobewan has quit IRC15:02
*** heyongli has quit IRC15:07
*** heyongli has joined #openstack-nova15:07
*** Mozart_ has joined #openstack-nova15:11
*** Apoorva has joined #openstack-nova15:11
*** moshele has quit IRC15:13
*** efried has joined #openstack-nova15:13
*** amodi has joined #openstack-nova15:14
*** moshele has joined #openstack-nova15:15
jaypipesalex_xu: ok, no prob. will fill you in tomorrow if we have a hangout.15:15
*** heyongli has quit IRC15:17
*** heyongli has joined #openstack-nova15:17
*** jobewan has joined #openstack-nova15:21
openstackgerritMatt Riedemann proposed openstack/python-novaclient master: Add support for the 2.57 microversion  https://review.openstack.org/52812815:24
openstackgerritMatt Riedemann proposed openstack/python-novaclient master: Remove incorrect legacy QuotaSet.id property  https://review.openstack.org/53075715:24
jaypipeskashyap: tox -epep8, sir. ;)15:26
jaypipeswell, good morning mriedem.15:26
mriedemhi15:26
*** heyongli has quit IRC15:27
kashyapjaypipes: Yeah, I normally run - `tox -epep8 -- -HEAD`; seems it got commented out in my shell :P15:27
*** heyongli has joined #openstack-nova15:27
mriedemtox -e fast815:28
edleafejaypipes: just saw your message. I'm currently consumed with my getting set up in my Austin office, after being forced by IBM to commute here instead of, you know, being productive15:32
jaypipesedleafe: :(15:32
jaypipesedleafe: that sux.15:32
*** alee is now known as alee_afk15:33
*** tidwellr has joined #openstack-nova15:34
openstackgerritMatt Riedemann proposed openstack/python-novaclient master: Microversion 2.58 - Instance actions list pagination  https://review.openstack.org/52860115:34
edleafejaypipes: this will also mean that we will need someone to run the scheduler subteam meetings, as I'll be on the road at that hour15:35
jaypipesedleafe: ack.15:35
jaypipesedleafe: or you could tell your management chain to stuff it and let you be productive.15:35
*** mozart89 has joined #openstack-nova15:37
*** heyongli has quit IRC15:37
*** gmann has quit IRC15:37
*** jaianshu has joined #openstack-nova15:37
*** Mozart_ has quit IRC15:37
*** heyongli has joined #openstack-nova15:38
*** eharney has quit IRC15:38
edleafejaypipes: I'd love to (and sort of have already), but it's dictated by someone clueless several levels above me.15:42
*** dhellmann has joined #openstack-nova15:42
edleafejaypipes: I've been trying to land a new gig, but OpenStack isn't the hiring hotness it was a few short years ago15:43
dhellmannhey folks, is this patch to add whereto for testing your doc redirects of any interest? or should I drop it? https://review.openstack.org/51638515:43
* cdent nominates efried for scheduler subteam meeting runner?15:45
cdent(i'm excluding myself because that plus the friday report would kill me)15:45
stephenfindhellmann: sdague would normally be your man but he's not around. Maybe mriedem15:45
*** Guest74108 has quit IRC15:45
*** Tom-Tom has quit IRC15:46
*** bnemec-xmas is now known as bnemec15:46
dhellmannstephenfin : yeah, I added sdague to the review since he's the one that threw the fit that drove us to add support for this in the first place.15:46
efriedcdent I wouldn't necessarily have a problem doing that; but IMO we should move the meeting to a time where edleafe can at least attend, if not continue to run it.15:46
cdentalso true15:46
*** liverpooler has joined #openstack-nova15:46
cdentI think part of the problem is that 4 hours of edleafe's day is being eaten by stupidity15:47
cdentso scheduling around him may be challenging15:47
edleafecdent: nailed it15:47
* cdent hates being right15:47
mdboothIs the gate unhappy today?15:47
*** david-lyle has joined #openstack-nova15:47
jaypipesedleafe: there's always k8s.15:47
*** heyongli has quit IRC15:48
*** crushil has joined #openstack-nova15:48
mdboothjaypipes: Thanks for all those reviews on my patch series, btw.15:48
*** heyongli has joined #openstack-nova15:48
jaypipesmdbooth: have they merged yet?15:48
mdbooth2 landed, currently trying to corral 3 more.15:48
edleafejaypipes: heh, I spent a lot of my holiday time improving my golang skills15:48
jaypipesmdbooth: k15:48
jaypipesedleafe: a good idea.15:48
*** baoli has joined #openstack-nova15:49
mdboothedleafe: Thoughts on golang?15:49
edleafemdbooth: it's a perfectly fine language. The main issue isn't syntax, of course, but learning the proper way of doing things. 15 years of Pythonic thinking is a bit difficult to overcome. :)15:50
mriedemmdbooth: might have a problem here https://review.openstack.org/#/c/525599/15:50
*** eharney has joined #openstack-nova15:51
*** baoli has quit IRC15:51
mdboothmriedem: I'll regret pointing this out, but that's in the gate right now.15:51
mriedemmdbooth: put a fix on top15:51
mdboothmriedem: Thanks :)15:51
mriedemshould be easy to adjust your test as well to use a deleted bdm15:51
mdboothmriedem: Ah, actually I think that was intentional because otherwise we can't but a not null constraint on the column subsequently.15:53
mriedemso we can load a deleted record and update it?15:54
mriedemyou should at least have a wrinkle in your test showing this is ok15:54
mriedemalso, another reason we don't have constraints on these things at first15:54
mdboothmriedem: That is an excellent point. That should at least be tested.15:54
mozart89Hello. Is it necessary to use Numa Topology ?15:54
mdboothAnd I wouldn't be surprised to find it's broken.15:54
mriedemmozart89: no15:54
kashyapOops, the 'i' in "Performance impact" _have_ to be capital; or else pep8 fails.  /me corrected it out of OCD, and realized the template itself was like htat15:55
mdboothThere's no constraint on it right now. The goal is to put a constraint on it in the first release where we positively assert that this migration has already run.15:55
mdboothi.e. if it fails, it's the operator's fault.15:56
mriedemi was thinking of the unique constraint https://review.openstack.org/#/c/242602/28/nova/db/sqlalchemy/migrate_repo/versions/374_bdm_uuid.py15:56
mriedembut i see uuid is nullable=True15:56
mriedemwhich is the constraint you're talking about15:56
mriedemthe BlockDeviceMapping.uuid is not nullable though15:57
mriedemwhich is probably good enough15:57
*** heyongli has quit IRC15:58
mdboothmriedem: Ah, actually this is still ok because it never loads the object. This is purely a db operation. I'll still add a test for it, if for no other reason than to make it obvious it's intentional.15:58
*** heyongli has joined #openstack-nova15:58
*** imacdonn has joined #openstack-nova15:58
mozart89mriedem: I'm not using any NUMA Topology ( CPU pinning and e.t.c). But when i'm triggering "live migration" it stops with message "Require both a host and instance NUMA topology to fit instance on host".15:59
*** gouthamr has joined #openstack-nova16:00
mriedemmozart89: that's just a debug message isn't it?16:00
*** armax has joined #openstack-nova16:00
mriedemmozart89: that's just a debug message that is output when the NUMATopologyFilter runs16:00
mriedemand is likely not your issue16:01
*** mozart89 has quit IRC16:03
*** mozart89 has joined #openstack-nova16:04
kashyapjaypipes: Is it true---to fix this (http://paste.openstack.org/show/636270/)---I have to explicitly change the order pf py27/py34 in tox.ini?16:04
mriedemkashyap: no16:05
mriedemrm -rf .testrepository16:05
kashyapYep, just did that.16:05
stephenfinmriedem, kashyap: That isn't necessary on master anymore, I thought16:05
jaypipeswhy is there a py27/py35 env for a specs repo?16:05
stephenfinSince the switch to stestr16:05
mozart89mriedem: ok, thank you for direction :)16:06
stephenfinOh, this is nova-specs. nvm me16:06
*** jobewan has quit IRC16:06
kashyapstephenfin: Yeah, it seems to be PEBKAC16:06
mriedemthere is no py27/py35 targets for nova-specs16:06
mriedemRequire both a host and instance NUMA topology16:07
mriedemoops16:07
*** heyongli has quit IRC16:08
*** crushil has quit IRC16:08
*** heyongli has joined #openstack-nova16:08
*** kmalloc has joined #openstack-nova16:09
*** jobewan has joined #openstack-nova16:10
*** vishwanathj has quit IRC16:12
kashyapmriedem: When you get a sec, is this also my env prob - http://paste.openstack.org/show/636286/16:17
*** alee_afk is now known as alee16:18
stephenfinkashyap: Stick 'basepython=python2' in the requisite tox target16:18
*** heyongli has quit IRC16:18
stephenfinI'd say your system Python is python3.x16:18
mriedemyeah ^16:18
kashyapstephenfin: I wanted to do that, but I felt: "wait a moment, it should work with existin master"16:18
kashyapYeah, it is16:18
*** heyongli has joined #openstack-nova16:19
stephenfinThe gate/Ubuntu uses Python 2 so we'd never see it there16:19
stephenfinSo it needs basepython or the requisite changes to be Python3 compatible. Latter is better long term, naturally (one year closer to 2020)16:20
*** crushil has joined #openstack-nova16:20
openstackgerritStephen Finucane proposed openstack/nova master: Address nits in I46d483f9de6776db1b025f925890624e5e682ada  https://review.openstack.org/53046216:21
stephenfingibi_away, jaypipes: So that ^ should _definitely_ work now, if you fancy sticking it on your review queue (it's cleanup for the PCI policy thing)16:22
*** cdent has quit IRC16:22
kashyapstephenfin: That's it; thanks.16:23
mriedemkashyap: push a patch so others don't hit it16:24
kashyap`tox -epep8 -- -HEAD` succeeds;  ( mriedem - `tox -e fast8` was a joke, right16:24
kashyapmriedem: Okido16:24
mriedemkashyap: tox -e fast8 is a nova repo thing16:24
mriedemnot nova-specs16:24
kashyapAh, okay.16:24
kashyapmriedem: Oops on, actually "that's not it".  Fast switching between repos, I ran it on Nova upstream16:25
kashyapstephenfin: That's my tox.ini - http://paste.openstack.org/show/636299/16:26
kashyapThat should more than do, right?16:26
kashyap(Again, this is for nova-specs)16:26
* kashyap does a couple more tests16:27
*** heyongli has quit IRC16:29
*** heyongli has joined #openstack-nova16:29
*** tidwellr has quit IRC16:30
stephenfinkashyap: Should do, yup16:30
stephenfinYou can drop line 2116:30
jaypipesstephenfin: done16:30
stephenfinkashyap: Also line 3-5, but that's another change16:31
mriedemildikov: are you going to rebase https://review.openstack.org/#/c/267587/ on top of https://review.openstack.org/#/c/529695/ ?16:31
stephenfinjaypipes: Thank you, sir16:31
kashyapstephenfin: Yep16:31
ildikovmriedem: I added it in 'Depends-On'16:31
ildikovmriedem: I can make a chain if you would prefer that16:32
ildikov?16:32
stephenfinmdbooth: Given the similarities to your own BDM patch, wanna take a shot at https://review.openstack.org/#/c/530487/ this week, once you're feeling better?16:33
mriedemildikov: the chain is better when it's in the same repo16:34
mriedemIMO16:34
ildikovmriedem: ok, will make it a chain16:35
*** crushil has quit IRC16:36
mriedemstephenfin: is this part of the pci policy bp i just marked as complete? https://review.openstack.org/#/c/530487/16:37
mriedemhttps://blueprints.launchpad.net/nova/+spec/share-pci-between-numa-nodes16:37
*** sridharg has quit IRC16:37
stephenfinmriedem: Nope, nothing to do with that. Just a nice-to-have that was talked about here before Xmas16:37
mriedemok i seem to remember that16:37
mriedempcidevice uuid was added for nested RPs?16:38
stephenfinYup16:38
mriedemok16:38
*** heyongli has quit IRC16:39
*** heyongli has joined #openstack-nova16:39
kashyapstephenfin: Hmm, bizarre, despite that .tox.ini file you saw, it is still going to 3.6:16:39
kashyap[...] nova-specs/.tox/docs/lib/python3.6/site-packages/yasfb/formatter.py", line 451, in format_rss2_file16:40
kashyap    handle.write(string.encode('utf-8'))16:40
kashyapTypeError: write() argument must be str, not byte16:40
kashyapIs there any other way I can force py2.7?16:40
clarkbkashyap: you may need to run with tox -r for it to rebuild the tox env16:40
clarkbif it already ran and built an older env with different python version it won't rebuilt it unless you pass -r16:40
kashyapclarkb: I just nuked the .tox and re-ran, to precisely avoid that16:41
kashyapBut let be double-check for sure16:41
*** pcaruana has quit IRC16:41
*** moshele has quit IRC16:43
*** Apoorva has quit IRC16:44
openstackgerritIldiko Vancsa proposed openstack/nova master: [libvirt] Allow multiple volume attachments  https://review.openstack.org/26758716:45
openstackgerritIldiko Vancsa proposed openstack/nova master: [api] Allow multi-attach in compute api  https://review.openstack.org/27104716:45
*** tidwellr has joined #openstack-nova16:46
ildikovmriedem: you were right as always :) done ^^16:46
*** ragiman has quit IRC16:46
*** chyka has joined #openstack-nova16:48
*** heyongli has quit IRC16:49
*** heyongli has joined #openstack-nova16:49
kashyapclarkb: Thanks; that was it; after re-building, this succeeds `tox -e py27`16:50
*** crushil has joined #openstack-nova16:52
openstackgerritKashyap Chamarthy proposed openstack/nova-specs master: Add UEFI Secure Boot support for QEMU/KVM guests, using OVMF  https://review.openstack.org/50672016:52
*** chyka has quit IRC16:53
*** eandersson has joined #openstack-nova16:53
*** cdent has joined #openstack-nova16:54
*** udesale has quit IRC16:55
cdentjaypipes, edleafe, efried, jroll: I'm home now and mostly free.16:55
openstackgerritMerged openstack/python-novaclient master: Updated from global requirements  https://review.openstack.org/53069716:57
*** tidwellr has quit IRC16:59
*** heyongli has quit IRC16:59
*** heyongli has joined #openstack-nova17:00
*** tidwellr has joined #openstack-nova17:00
openstackgerritMatthew Booth proposed openstack/nova master: Add test for assignment of uuid to a deleted BDM  https://review.openstack.org/53077417:00
*** jaianshu has quit IRC17:01
*** gyee has joined #openstack-nova17:01
*** jaianshu has joined #openstack-nova17:01
*** jafeha__ has joined #openstack-nova17:03
*** jafeha has quit IRC17:04
*** tidwellr has quit IRC17:04
mdboothmriedem: ^^^17:04
mriedemack17:04
*** crushil has quit IRC17:08
*** heyongli has quit IRC17:10
*** heyongli has joined #openstack-nova17:10
mdboothstephenfin: That pci uuid patch screams refactor with library code to me. You?17:10
mdboothI mean, it's a whole bunch of cut/paste.17:11
stephenfinmdbooth: It's getting there, but I could only find the two occurrences17:11
stephenfinOne more and I'd have dragged it out17:11
mdboothOk, if it's only 2.17:11
mdboothStill borderline, though.17:11
* mdbooth sacrifices his favourite goat to Zuul17:12
mdboothI just need legacy-tempest-dsvm-py35 to complete for the top patch, then the top 2 will merge.17:13
mdboothIt's taking a suspiciously long time, though :(17:13
mdboothPerhaps my goat didn't find favour.17:14
* mdbooth adds some libations17:14
openstackgerritKashyap Chamarthy proposed openstack/nova-specs master: tox.ini: Add Python 2.7 as 'basepython' for 'pep8' target  https://review.openstack.org/53077617:18
*** heyongli has quit IRC17:20
openstackgerritStephen Finucane proposed openstack/nova master: Handle deprecation of inspect.getargspec  https://review.openstack.org/53077717:20
*** heyongli has joined #openstack-nova17:20
*** karthiks has quit IRC17:22
*** lucasagomes is now known as lucas-afk17:25
*** fragatina has joined #openstack-nova17:29
*** heyongli has quit IRC17:30
*** heyongli has joined #openstack-nova17:30
*** fragatin_ has joined #openstack-nova17:31
jaypipeskashyap: reviewed.17:31
kashyapjaypipes: Thanks; /me goes to look17:33
kashyapjaypipes: Damn, thought I fixed all the "brain coughs" :P  Thanks for catching17:34
*** fragatina has quit IRC17:34
*** Apoorva has joined #openstack-nova17:39
*** lyan has quit IRC17:40
*** heyongli has quit IRC17:40
*** heyongli has joined #openstack-nova17:41
kashyapAh, I think you were writing that comment ("seems like you started one sentence but then switched to another") as I already addressed it :-)17:42
*** mdnadeem has quit IRC17:42
* kashyap bbiab17:42
cburgessjaypipes Apparently nic loves pugs  too... http://submedia.net17:44
jaypipescburgess: lol17:46
cburgessApparently he has had that for years and I just now learned of it.17:46
cdentreminds me: that pug bath plug is the bomb diggity. Except it is a bit disconcerting when he spins around and stares at naked entuberated me with his beady little put eyes17:47
cdents/put/pug/17:47
jaypipeslmao17:47
cdentdoes this work (SFW): https://scontent.flhr2-1.fna.fbcdn.net/v/t35.0-12/26236425_10154900649336370_718762856_o.jpg?oh=e3e9ccca81ae034887843abe17781943&oe=5A4DB6DD17:49
*** mvk has quit IRC17:49
*** heyongli has quit IRC17:51
*** heyongli has joined #openstack-nova17:51
*** catintheroof has joined #openstack-nova17:51
*** lyan has joined #openstack-nova17:55
*** derekh has quit IRC17:59
*** tbachman has joined #openstack-nova18:00
*** heyongli has quit IRC18:01
*** heyongli has joined #openstack-nova18:01
jrollamazing18:02
mdboothWhat will Zuul do when it times out a job? Restart or fail?18:06
jrollmdbooth: fails18:06
* mdbooth sighs18:07
openstackgerritMerged openstack/python-novaclient master: Remove incorrect legacy QuotaSet.id property  https://review.openstack.org/53075718:08
*** jaianshu has quit IRC18:10
*** heyongli has quit IRC18:11
*** heyongli has joined #openstack-nova18:11
*** harlowja has joined #openstack-nova18:12
mdboothWell, that's been stalled for so long I'm almost entirely certain it's going to fail, and when it fails the 2 behind it will also fail.18:14
mdboothSo I'm just gonna do my rebase anyway.18:14
openstackgerritMatthew Booth proposed openstack/nova master: Add an online migration for BDM.uuid  https://review.openstack.org/52559918:14
openstackgerritMatthew Booth proposed openstack/nova master: DriverBlockDevice: make subclasses inherit _proxy_as_attr  https://review.openstack.org/52416718:14
openstackgerritMatthew Booth proposed openstack/nova master: Expose BDM uuid to drivers  https://review.openstack.org/52903718:14
openstackgerritMatthew Booth proposed openstack/nova master: Give volume DriverBlockDevice classes a common prefix  https://review.openstack.org/52634618:14
* mdbooth watches the world burn18:14
openstackgerritMatthew Booth proposed openstack/nova master: Add DriverLocalImageBlockDevice  https://review.openstack.org/52634718:14
openstackgerritMatthew Booth proposed openstack/nova master: Rename block_device_info_get_root  https://review.openstack.org/52902818:14
openstackgerritMatthew Booth proposed openstack/nova master: Add local_root to block_device_info  https://review.openstack.org/52902918:14
openstackgerritMatthew Booth proposed openstack/nova master: Expose driver_block_device fields as attributes  https://review.openstack.org/52836218:14
openstackgerritMatthew Booth proposed openstack/nova master: Pass DriverBlockDevice to driver.attach_volume  https://review.openstack.org/52836318:14
openstackgerritMatthew Booth proposed openstack/nova master: Use real block_device_info data in libvirt tests  https://review.openstack.org/52791618:14
openstackgerritMatthew Booth proposed openstack/nova master: Fix libvirt volume tests passing invalid disk_info  https://review.openstack.org/52932818:15
openstackgerritMatthew Booth proposed openstack/nova master: Pass disk_info dict to libvirt_info  https://review.openstack.org/52932918:15
openstackgerritMatthew Booth proposed openstack/nova master: Local disk serial numbers for the libvirt driver  https://review.openstack.org/52938018:15
openstackgerritMatthew Booth proposed openstack/nova master: Expose volume host type and path independent of libvirt config  https://review.openstack.org/53078618:15
openstackgerritMatthew Booth proposed openstack/nova master: Don't generate fake disk_info in swap_volume  https://review.openstack.org/53078718:15
*** sambetts is now known as sambetts|afk18:19
mdboothstephenfin: Any chance you could stick the +A back on https://review.openstack.org/#/c/525599/ https://review.openstack.org/#/c/524167/ and https://review.openstack.org/#/c/529037/ ?18:19
mdboothstephenfin: Rebase18:19
mdboothThey just failed in the gate, and I needed to rebase the series anyway to pull in some other recently landed patches.18:20
mdboothI think I fixed the problem with swap_volume.18:20
*** heyongli has quit IRC18:21
*** heyongli has joined #openstack-nova18:22
*** fragatin_ has quit IRC18:31
*** heyongli has quit IRC18:32
*** heyongli has joined #openstack-nova18:32
*** hemna_ has joined #openstack-nova18:35
*** heyongli has quit IRC18:42
*** heyongli has joined #openstack-nova18:42
*** jafeha__ has quit IRC18:45
*** heyongli has quit IRC18:52
*** heyongli has joined #openstack-nova18:52
openstackgerritMatt Riedemann proposed openstack/nova master: Only call numa_fit_instance_to_host if necessary  https://review.openstack.org/53079218:54
*** tesseract has quit IRC18:59
*** heyongli has quit IRC19:02
*** heyongli has joined #openstack-nova19:03
*** priteau has joined #openstack-nova19:08
*** priteau has quit IRC19:11
*** priteau has joined #openstack-nova19:12
*** heyongli has quit IRC19:13
*** heyongli has joined #openstack-nova19:13
*** priteau has quit IRC19:13
*** priteau has joined #openstack-nova19:14
*** tbachman has quit IRC19:17
*** yikun has quit IRC19:21
*** yikun has joined #openstack-nova19:22
*** heyongli has quit IRC19:23
*** heyongli has joined #openstack-nova19:23
*** heyongli has quit IRC19:33
*** heyongli has joined #openstack-nova19:33
*** moshele has joined #openstack-nova19:34
*** suresh12 has joined #openstack-nova19:38
*** heyongli has quit IRC19:43
*** heyongli has joined #openstack-nova19:44
*** heyongli has quit IRC19:54
*** moshele has quit IRC19:54
*** heyongli has joined #openstack-nova19:54
*** READ10 has quit IRC19:57
efriedmikal Does/will the privsep work make the use_rootwrap_daemon conf option obsolete?19:59
openstackgerritJay Pipes proposed openstack/nova-specs master: Support aggregate affinity filters/weighers  https://review.openstack.org/52913520:00
*** suresh12 has quit IRC20:00
efriedmriedem Do you know that answer?20:01
mriedemefried: yes i think it will20:01
efriedk, thx20:02
mriedemsince there would be no need for rootwrap20:02
efriedright20:02
efriedesberglu ^20:02
dansmithexcept for starting privsep20:02
efriedoh20:02
dansmithI can't remember how we settled on that.. it might be hard-coded in privsep stuff now or something20:02
mriedemi thought we did a hard-coded thing to get around grenade20:03
efriedBasically, trying to figure out if my driver needs that opt set to True if the only root-y stuff it does is via privsep now.20:03
mriedemwhich was the great schizm of 201520:03
*** heyongli has quit IRC20:04
* cdent needs a schism counter20:04
*** heyongli has joined #openstack-nova20:04
mriedemhttps://github.com/openstack/oslo.rootwrap/commit/37c2a041d33f0fdce7ce2832398c1f60f3ee870320:04
mriedemis what i'm thinking of20:04
mriedemwhich is different from what dansmith is talking about, i think20:05
*** tbachman has joined #openstack-nova20:05
*** bkopilov has quit IRC20:06
esbergluefried: I will push a run through without use_rootwrap_daemon and confirm20:09
*** purplerbot has quit IRC20:09
*** purplerbot has joined #openstack-nova20:09
efriedesberglu Sounds like a plan.  Could consider splitting that bit out of the change; I'd be +2 on the other parts.20:09
*** larainema has quit IRC20:10
*** heyongli has quit IRC20:14
*** heyongli has joined #openstack-nova20:14
openstackgerritMatt Riedemann proposed openstack/nova stable/pike: Do not set allocation.id in AllocationList.create_all()  https://review.openstack.org/53079420:20
*** fragatina has joined #openstack-nova20:24
*** heyongli has quit IRC20:24
*** heyongli has joined #openstack-nova20:25
*** bjolo has joined #openstack-nova20:25
*** suresh12 has joined #openstack-nova20:27
*** peter-hamilton has quit IRC20:30
*** suresh12 has quit IRC20:33
*** klindgren_ has joined #openstack-nova20:34
*** suresh12 has joined #openstack-nova20:34
*** heyongli has quit IRC20:35
*** heyongli has joined #openstack-nova20:35
*** yamahata has quit IRC20:35
*** klindgren has quit IRC20:36
*** heyongli has quit IRC20:45
*** heyongli has joined #openstack-nova20:45
*** catintheroof has quit IRC20:46
*** catintheroof has joined #openstack-nova20:46
openstackgerritEric Fried proposed openstack/nova master: Move aggregates from report client to ProviderTree  https://review.openstack.org/52168520:49
openstackgerritEric Fried proposed openstack/nova master: Track provider traits in report client  https://review.openstack.org/52168620:49
openstackgerritEric Fried proposed openstack/nova master: Track associated sharing RPs in report client  https://review.openstack.org/52653920:49
openstackgerritEric Fried proposed openstack/nova master: Raise on API errors getting aggregates/traits  https://review.openstack.org/52654020:49
openstackgerritEric Fried proposed openstack/nova master: ProviderTree.populate_from_iterable  https://review.openstack.org/52075620:49
openstackgerritEric Fried proposed openstack/nova master: Track tree-associated providers in report client  https://review.openstack.org/52654120:49
openstackgerritEric Fried proposed openstack/nova master: WIP: Scheduler[Report]Client.get_provider_tree  https://review.openstack.org/52109820:49
openstackgerritEric Fried proposed openstack/nova master: WIP: ComputeDriver.update_provider_tree()  https://review.openstack.org/52118720:49
openstackgerritEric Fried proposed openstack/nova master: WIP: Use update_provider_tree from resource tracker  https://review.openstack.org/52024620:49
efriedRebase only (some manual) ^^20:49
*** armax has quit IRC20:49
*** catintheroof has quit IRC20:51
*** smatzek has quit IRC20:54
*** smatzek has joined #openstack-nova20:55
*** smatzek has quit IRC20:55
*** heyongli has quit IRC20:55
*** smatzek has joined #openstack-nova20:55
*** heyongli has joined #openstack-nova20:55
*** jafeha has joined #openstack-nova20:56
*** suresh12 has quit IRC20:58
openstackgerritJackie Truong proposed openstack/python-novaclient master: Microversion 2.59 - Add trusted_image_certificates  https://review.openstack.org/50039620:59
*** fragatina has quit IRC21:00
*** smatzek has quit IRC21:00
*** heyongli has quit IRC21:05
*** heyongli has joined #openstack-nova21:06
*** suresh12 has joined #openstack-nova21:07
*** liverpooler has quit IRC21:09
openstackgerritJay Pipes proposed openstack/nova-specs master: Support aggregate affinity filters/weighers  https://review.openstack.org/52913521:10
*** pchavva has quit IRC21:10
*** slaweq_ has joined #openstack-nova21:13
*** slaweq_ has quit IRC21:14
*** heyongli has quit IRC21:16
*** heyongli has joined #openstack-nova21:16
*** cleong has quit IRC21:18
*** armax has joined #openstack-nova21:19
*** tidwellr has joined #openstack-nova21:22
*** jose-phillips has joined #openstack-nova21:24
*** markmcclain has quit IRC21:25
*** heyongli has quit IRC21:26
mnaserwhat are the implications of using cpu_allocation_ratio on compute with the placement api?21:26
*** heyongli has joined #openstack-nova21:26
mnaseri'm testing around cpu_allocation_ratio=2.0 on a compute node but i'm not sure if its properly reporting that number to the placement as im getting no valid host21:27
*** markmcclain has joined #openstack-nova21:28
jaypipesmnaser: easy enough to check... in the API database (the one placement uses, do this: SELECT * FROM inventories WHERE allocation_ratio > 1.0 AND resource_class = 0;21:28
mnaserjaypipes: gah thah was an obvious one21:29
jaypipesmnaser: :)21:29
mnaserand i was trying to think how i was going to query the placement api21:29
mnaserand making curl calls21:29
mnaser:p21:29
jaypipesmnaser: you could do that, too, but wouldn't be as fun ;)21:29
jaypipesmnaser: FYI, VCPU == resource_class "0"21:29
mnaserjaypipes: true21:29
jaypipesMEMORY_MB == 121:29
jaypipesDISK_GB == 221:29
mnaserok the allocation ratio is indeed there21:30
mnaseri guess ill have to run the scheduler in verbose21:30
*** fragatina has joined #openstack-nova21:31
mnaseri could swear scheduling failures were reported as warnings in the scheduler21:31
mriedem"and i was trying to think how i was going to query the placement api" - osc-placement plugin has some placement CLI now21:31
mnaseroh really21:31
mriedempretty minimal at this point21:32
*** fragatina has quit IRC21:32
mriedemsupports working on resource providers and inventories records21:32
jaypipesmnaser: no, but in debug/verbose mode, you will see some log messages like "Filter XXX: Started with 10 hosts, finished with 6 hosts" or something like that.21:32
*** fragatina has joined #openstack-nova21:32
* mnaser is lazy about restarting 3 schedulers so restarting 1 and rebuilding instances till i hit the one running in verbose21:32
dansmithmnaser: like a (statistically-consistent) boss21:33
jaypipesmnaser: however it's just as good to look in the placement-api logs for the requests to GET /allocation_candidates and then I can give you the SQL to run that represents that particular request.21:33
mnaser3 schedulers, 1 running verbose, 3 build attempts and i am 0-321:33
mnaseroh duh because verbose=True is a default now and I should be debug=True21:34
* mnaser hides21:34
jaypipesheh21:34
*** nore_rabel has quit IRC21:34
mnaserum21:34
mnaser"Got no allocation candidates from the Placement API. This may be a temporary occurrence as compute nodes start up and begin reporting inventory to the Placement service."21:35
mnaseri might have a bit more on my hands than i expected21:35
mnasershouldn't that be WARN instead of DEBUG?21:35
jaypipesmnaser: there was a deliberate use of DEBUG for most/all messages in the scheduler code paths due to concern about performance.21:36
cdentmnaser: that's what I was going to suggest: first make sure you're getting any allocations candidates, you can contrust queries direct to /allocation_candidates: https://developer.openstack.org/api-ref/placement/#allocation-candidates21:36
*** heyongli has quit IRC21:36
cdents/contrust/construct/21:36
mnaserjaypipes: i see21:36
*** heyongli has joined #openstack-nova21:36
jaypipesmnaser: what was the GET /allocation_candidates HTTP request you see in your logs?\21:36
mnasercdent: ok, i'll try to do that, might be a bit difficult considering the volume of traffic at the placement api21:37
jaypipesplacement-api logs, that is.21:37
mnaserbecause i'm pretty sure this is some upgrade-leftover21:37
mnaserlet me check21:37
mnaser"GET /allocation_candidates?resources=MEMORY_MB%3A65536%2CVCPU%3A64" status: 200 len: 53 microversion: 1.1021:38
cdentmnaser: you can also make similar queries to /resource_providers to confirm what you think should be there is in fact there (basically a shorter version of what an /allocation_candidates query might report. you want 64 vcpus?21:39
mnaserso right now i have an empty 32 physical core machine and im trying to launch 64 cores on it (with allocation ratio set to 2) -- im aware of performance implications but this is just to test out oversubscription21:39
jaypipesahhhhhhhhh21:40
jaypipesmnaser: so, you will notice that max_unit is == 32 for your VCPU inventories.21:41
*** eharney has quit IRC21:41
mnaserfor that host correct21:41
jaypipesmnaser: this is to prevent someone from attempting to launch an instance that consumes more VCPU than the physical number of CPUs on the host.21:41
jaypipesmnaser: allocation_ratio does not affect this constraint.21:41
mnaserso, no single VM can have above max_unit, but the total vcpus is limited to max_unit * allocation_ratio ?21:42
jaypipesmnaser: allocation_ratio only allows over-subscription of resources when min_unit <= $REQUESTED_AMOUNT <= max_unit21:42
jaypipesmnaser: that is correct.21:42
mnaserso everything was probably working and my lazy (launch one big instance to test overcommit) was at fault :D21:43
jaypipesheh, yeah :)21:43
cdentIs bauzas back yet? He'd be relishing this moment (even though I still think we made the right decision)21:43
mnaserlet me double check this by launching a few instances21:43
mnaserand indeed, that was it21:44
*** rcernin has joined #openstack-nova21:45
mnaser2x 32c was ok21:45
*** priteau has quit IRC21:45
*** hongbin has quit IRC21:45
mnaserthanks jaypipes and everyone else for the input21:46
*** heyongli has quit IRC21:46
*** dave-mccowan has quit IRC21:47
*** heyongli has joined #openstack-nova21:47
cdentmnaser, while you're hear can you clarify something for me. above you said "might be a bit difficult considering the volume of traffic at the placement api". What did you mean by that? How is/was the traffic at the api impacting your ability to make requests of it, in this case?21:47
cdentsigh: s/hear/here/21:47
mnasercdent: oh, i was doing some testing on a host aggregate on our public cloud, so the volume of traffic hitting the placement api is pretty significant21:48
mnaserfrom compute nodes reporting inventory to VMs getting booted its pretty busy and gets churned a lot21:48
cdentOkay, my suggestion was, effectively, to construst the query that the scheduler was doing, yourself, so you could isolate the "no candidates returned" problem, without having to boot anything21:49
mnasercdent: yeah, i have to admit the placement api is still one of the things that i'm not as comfortable with as the rest of nova.. but this little experience added a bunch of stuff to my 'toolkit'21:50
cdentI'm the opposite :)21:50
*** jaypipes has quit IRC21:51
mnaser:D21:52
*** mvk has joined #openstack-nova21:56
*** heyongli has quit IRC21:57
*** heyongli has joined #openstack-nova21:57
*** suresh12 has quit IRC22:05
*** suresh12 has joined #openstack-nova22:06
*** heyongli has quit IRC22:07
*** heyongli has joined #openstack-nova22:07
*** heyongli has quit IRC22:17
*** heyongli has joined #openstack-nova22:17
openstackgerritRodolfo Alonso Hernandez proposed openstack/nova master: Change 'InstancePCIRequest' spec field  https://review.openstack.org/44925722:24
openstackgerritRodolfo Alonso Hernandez proposed openstack/nova master: Add Neutron port capabilities to devspec in request  https://review.openstack.org/45177722:24
openstackgerritRodolfo Alonso Hernandez proposed openstack/nova master: Format NIC features using os-traits definitions  https://review.openstack.org/46605122:24
*** mozart89 has quit IRC22:26
*** heyongli has quit IRC22:27
*** heyongli has joined #openstack-nova22:28
openstackgerritRodolfo Alonso Hernandez proposed openstack/nova master: Read Neutron port 'binding_profile' during boot  https://review.openstack.org/50748122:28
*** lyan has quit IRC22:34
*** tidwellr has quit IRC22:35
eanderssonIs services versions supposed to be upgraded when you upgrade the version of Openstack?22:36
*** jobewan has quit IRC22:37
*** heyongli has quit IRC22:38
*** heyongli has joined #openstack-nova22:38
*** suresh12 has quit IRC22:43
*** suresh12 has joined #openstack-nova22:43
*** suresh12 has quit IRC22:43
*** suresh12 has joined #openstack-nova22:43
*** suresh12 has quit IRC22:43
*** suresh12 has joined #openstack-nova22:44
*** suresh12 has quit IRC22:44
*** fragatina has quit IRC22:44
*** catintheroof has joined #openstack-nova22:46
*** heyongli has quit IRC22:48
*** heyongli has joined #openstack-nova22:48
*** heyongli has quit IRC22:58
*** heyongli has joined #openstack-nova22:58
*** jobewan has joined #openstack-nova23:07
*** heyongli has quit IRC23:08
*** heyongli has joined #openstack-nova23:09
*** yamahata has joined #openstack-nova23:18
*** heyongli has quit IRC23:19
*** edmondsw has quit IRC23:19
*** heyongli has joined #openstack-nova23:19
*** edmondsw has joined #openstack-nova23:19
*** edmondsw has quit IRC23:24
*** heyongli has quit IRC23:29
*** heyongli has joined #openstack-nova23:29
*** tidwellr has joined #openstack-nova23:30
*** amodi has quit IRC23:32
openstackgerritEric Berglund proposed openstack/nova master: PowerVM driver: ovs vif  https://review.openstack.org/42251223:35
*** jobewan has quit IRC23:37
*** tidwellr has quit IRC23:39
*** heyongli has quit IRC23:39
*** heyongli has joined #openstack-nova23:39
openstackgerritMatt Riedemann proposed openstack/nova master: Workaround CellDatabase fixture limitation when creating computes  https://review.openstack.org/52426423:45
openstackgerritMatt Riedemann proposed openstack/nova master: Add multi-cell negative test for cold migration with target host  https://review.openstack.org/52402723:45
*** amodi has joined #openstack-nova23:47
*** heyongli has quit IRC23:49
*** heyongli has joined #openstack-nova23:50

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