Thursday, 2018-02-22

melwittmnaser: that's the series that caused the regression00:02
*** yamamoto has joined #openstack-nova00:08
*** tbachman has quit IRC00:09
efrieddansmith: yt?00:09
*** tetsuro has joined #openstack-nova00:13
*** yamamoto has quit IRC00:14
*** lbragstad has quit IRC00:18
*** amodi has quit IRC00:26
openstackgerritMerged openstack/nova master: Add functional test for deleting BFV server with old attach flow  https://review.openstack.org/54631500:31
openstackgerritMerged openstack/nova master: Ensure attachment_id always exists for block device mapping  https://review.openstack.org/54639800:31
*** moshele has quit IRC00:32
*** liusheng has quit IRC00:36
mnaserwell00:37
*** liusheng has joined #openstack-nova00:37
mnaserneutron-grenade is blowing up for the queens backports00:37
efriedSomebody, anybody with sqlalchemy skills00:37
efriedor SQL skills.  edleafe yt?00:37
*** chyka has quit IRC00:37
*** liuzz has joined #openstack-nova00:42
melwittso the grenade is blowing up00:46
melwittsomething is messed up: "rsync: change_dir "/opt/stack/old/tempest/.testrepository" failed: No such file or directory (2)"00:48
*** tbachman has joined #openstack-nova00:48
melwitthttp://logs.openstack.org/00/546200/2/gate/neutron-grenade/dbcbf60/logs/grenade.sh.txt.gz#_2018-02-21_23_43_04_99000:48
mnasermelwitt: indeed i tracked it down00:50
mnasermelwitt: https://review.openstack.org/#/q/Ic1fa3a98b6bcd151c489b078028687892655a19b00:50
melwittoh, nice00:50
mnasermelwitt: got a first +2, maybe if you can track down someone to +A .. hopefully00:52
* mnaser grumbles something about branchless tempest00:53
melwittokay, lemme see00:54
*** tiendc has joined #openstack-nova01:00
*** Dave has quit IRC01:01
openstackgerritEric Berglund proposed openstack/nova master: WIP: PowerVM Driver: Network interface attach/detach  https://review.openstack.org/54681301:03
*** gyee has quit IRC01:05
*** phuongnh has joined #openstack-nova01:06
melwittdansmith: looks like you're a grenade core, if you could review these to unblock our stable/queens and stable/pike backports when you get a chance, maybe tomorrow morning if you can real quick https://review.openstack.org/#/c/546828/ https://review.openstack.org/#/c/54683101:06
*** moshele has joined #openstack-nova01:07
*** zhaochao has joined #openstack-nova01:09
*** moshele has quit IRC01:10
*** yamamoto has joined #openstack-nova01:10
*** Guest87022 has quit IRC01:14
*** claudiub has quit IRC01:15
*** yamamoto has quit IRC01:16
*** Guest87022 has joined #openstack-nova01:17
*** acormier has joined #openstack-nova01:29
*** acormier has quit IRC01:30
*** acormier has joined #openstack-nova01:30
*** yangyapeng has quit IRC01:33
*** yangyapeng has joined #openstack-nova01:33
*** slaweq has quit IRC01:34
*** slaweq has joined #openstack-nova01:35
*** esberglu has quit IRC01:35
openstackgerritEric Fried proposed openstack/nova master: WIP: rp: GET /resource_providers?required=<traits>  https://review.openstack.org/54683701:35
efriedcdent, jaypipes: FYI ^  -- I think it's mostly done.  Please do vet my sqla for potential optimizations.  But be gentle - it took me a couple hours to get it working at all.01:37
*** tianhui has joined #openstack-nova01:37
efriedmelwitt: Potential to have a Rocky bp closed before the PTG :)01:37
*** yangyapeng has quit IRC01:38
* efried is outta here. See ya in ~12h.01:38
*** slaweq has quit IRC01:39
*** Kevin_Zheng has joined #openstack-nova01:40
*** salv-orlando has joined #openstack-nova01:41
*** salv-orlando has quit IRC01:46
*** namnh has joined #openstack-nova01:50
*** owalsh has quit IRC01:56
*** lbragstad has joined #openstack-nova01:56
*** owalsh has joined #openstack-nova01:56
*** acormier has quit IRC01:58
*** gjayavelu has quit IRC01:58
*** moshele has joined #openstack-nova02:00
*** owalsh has quit IRC02:01
*** annp has joined #openstack-nova02:02
*** eharney has quit IRC02:05
*** owalsh has joined #openstack-nova02:06
*** Zames has joined #openstack-nova02:08
*** chyka has joined #openstack-nova02:10
melwittdansmith: nvm on my earlier message, mtreinish was able to fast-approve02:10
*** yamamoto has joined #openstack-nova02:12
*** yikun has joined #openstack-nova02:12
*** hongbin has joined #openstack-nova02:13
*** chyka has quit IRC02:14
*** Zames has quit IRC02:14
*** dklyle has joined #openstack-nova02:17
*** yamamoto has quit IRC02:18
*** itlinux has joined #openstack-nova02:18
*** phuongnh has quit IRC02:18
*** tetsuro_ has joined #openstack-nova02:19
*** tetsuro has quit IRC02:19
*** tetsuro_ is now known as tetsuro02:19
*** phuongnh has joined #openstack-nova02:19
*** david-lyle has quit IRC02:21
*** tylerderosagrund has joined #openstack-nova02:25
*** dave-mccowan has joined #openstack-nova02:28
*** amodi has joined #openstack-nova02:29
*** moshele has quit IRC02:29
*** tylerderosagrund has quit IRC02:30
*** mriedem has quit IRC02:32
openstackgerritMatt Riedemann proposed openstack/nova master: Drop API compat handling for old compute error cases  https://review.openstack.org/54625602:32
*** zhenguo has joined #openstack-nova02:33
*** yamahata has quit IRC02:34
*** phuongnh has quit IRC02:35
*** phuongnh has joined #openstack-nova02:36
openstackgerritMatt Riedemann proposed openstack/nova master: Handle spawning error on unshelving  https://review.openstack.org/37800902:39
*** Tom-Tom has joined #openstack-nova02:47
*** moshele has joined #openstack-nova02:50
*** moshele has quit IRC02:52
openstackgerritZhenyu Zheng proposed openstack/nova-specs master: Add extra-specs to the flavor show and detail API calls  https://review.openstack.org/54415802:55
*** dave-mccowan has quit IRC02:58
*** wolverineav has quit IRC03:00
*** andymccr has quit IRC03:00
*** andymccr has joined #openstack-nova03:03
*** lbragstad has quit IRC03:04
*** andreas_s has joined #openstack-nova03:06
*** itlinux has quit IRC03:10
*** andreas_s has quit IRC03:10
*** acormier has joined #openstack-nova03:12
*** yamamoto has joined #openstack-nova03:14
*** yamamoto has quit IRC03:20
*** mriedem has joined #openstack-nova03:23
*** mriedem has left #openstack-nova03:23
*** acormier has quit IRC03:27
openstackgerritMerged openstack/nova master: libvirt: disconnect volume from host during detach  https://review.openstack.org/51500803:28
*** fragatina has quit IRC03:32
*** amodi has quit IRC03:33
*** wolverineav has joined #openstack-nova03:33
*** itlinux has joined #openstack-nova03:37
*** yamamoto has joined #openstack-nova03:37
*** wolverineav has quit IRC03:37
*** dave-mccowan has joined #openstack-nova03:38
*** wolverineav has joined #openstack-nova03:40
*** salv-orlando has joined #openstack-nova03:43
*** wolverineav has quit IRC03:44
*** salv-orlando has quit IRC03:49
openstackgerritTakashi NATSUME proposed openstack/nova master: api-ref: Parameter verification for servers.inc  https://review.openstack.org/52820103:50
*** wolverineav has joined #openstack-nova03:51
openstackgerritTakashi NATSUME proposed openstack/nova master: api-ref: Example verification for servers.inc  https://review.openstack.org/52952003:52
openstackgerritTakashi NATSUME proposed openstack/nova master: api-ref: Fix parameter order in rebuild  https://review.openstack.org/52971803:53
*** wolverineav has quit IRC03:57
*** slaweq has joined #openstack-nova03:59
*** slaweq has quit IRC04:04
*** udesale has joined #openstack-nova04:04
*** udesale_ has joined #openstack-nova04:07
*** dave-mccowan has quit IRC04:09
*** vks1 has joined #openstack-nova04:13
*** Dinesh__Bhor has joined #openstack-nova04:14
*** wolverineav has joined #openstack-nova04:17
*** masahisa has left #openstack-nova04:21
*** wolverineav has quit IRC04:22
*** bhagyashris has joined #openstack-nova04:25
*** sree_ has joined #openstack-nova04:25
*** sree_ is now known as Guest9694704:26
*** tylerderosagrund has joined #openstack-nova04:30
*** links has joined #openstack-nova04:30
*** udesale_ has quit IRC04:31
*** udesale_ has joined #openstack-nova04:31
*** udesale has quit IRC04:32
*** psachin has joined #openstack-nova04:34
*** tylerderosagrund has quit IRC04:35
*** zhenguo has quit IRC04:42
*** salv-orlando has joined #openstack-nova04:45
*** salv-orlando has quit IRC04:49
*** pooja_jadhav has joined #openstack-nova04:50
*** Guest87022 has quit IRC04:53
*** abhishekk has joined #openstack-nova04:55
*** Guest87022 has joined #openstack-nova04:56
*** fragatin_ has joined #openstack-nova04:56
*** sridharg has joined #openstack-nova04:58
*** Zames has joined #openstack-nova05:01
*** Dinesh__Bhor has quit IRC05:01
*** hongbin has quit IRC05:02
*** Dinesh__Bhor has joined #openstack-nova05:02
*** Dinesh__Bhor has quit IRC05:04
*** Zames has quit IRC05:05
*** Zames has joined #openstack-nova05:09
*** ratailor has joined #openstack-nova05:10
*** tylerderosagrund has joined #openstack-nova05:14
*** Dinesh__Bhor has joined #openstack-nova05:14
*** tylerderosagrund has quit IRC05:19
*** Zames has quit IRC05:21
*** wolverineav has joined #openstack-nova05:21
*** slaweq has joined #openstack-nova05:25
*** wolverineav has quit IRC05:27
*** janki has joined #openstack-nova05:27
*** slaweq has quit IRC05:30
*** Dinesh__Bhor has quit IRC05:32
*** Dinesh__Bhor has joined #openstack-nova05:34
*** sidx64 has joined #openstack-nova05:40
openstackgerritTetsuro Nakamura proposed openstack/nova-specs master: Support shared/dedicated vCPUs in one instance  https://review.openstack.org/54573405:43
*** bhujay has joined #openstack-nova05:45
*** salv-orlando has joined #openstack-nova05:45
*** claudiub has joined #openstack-nova05:47
*** vks1 has quit IRC05:47
*** ttsiouts has quit IRC05:48
*** ttsiouts has joined #openstack-nova05:49
*** salv-orlando has quit IRC05:50
openstackgerritMerged openstack/python-novaclient master: Clean imports in code  https://review.openstack.org/54654305:52
*** tylerderosagrund has joined #openstack-nova05:57
*** bhujay has quit IRC06:00
*** tylerderosagrund has quit IRC06:01
*** vks1 has joined #openstack-nova06:02
*** jogo has quit IRC06:03
*** hieulq has joined #openstack-nova06:03
*** bhujay has joined #openstack-nova06:04
*** rha has quit IRC06:07
*** mdnadeem has joined #openstack-nova06:08
*** lpetrut has joined #openstack-nova06:18
*** salv-orlando has joined #openstack-nova06:19
*** slaweq has joined #openstack-nova06:27
*** acormier has joined #openstack-nova06:27
*** lajoskatona has joined #openstack-nova06:30
*** mdnadeem_ has joined #openstack-nova06:31
*** slaweq has quit IRC06:31
*** acormier has quit IRC06:31
*** hshiina2 has joined #openstack-nova06:32
*** Eran_Kuris has joined #openstack-nova06:33
*** mdnadeem has quit IRC06:33
*** hshiina has quit IRC06:34
*** hshiina2 is now known as hshiina|afk06:34
*** alexchadin has joined #openstack-nova06:41
*** salv-orlando has quit IRC06:51
*** salv-orlando has joined #openstack-nova06:51
*** mdnadeem_ has quit IRC06:52
*** kholkina_ has joined #openstack-nova06:55
openstackgerritMerged openstack/nova stable/queens: Drop extra loop which modifies Cinder volume status  https://review.openstack.org/54620006:56
*** moshele has joined #openstack-nova06:56
*** salv-orlando has quit IRC06:56
*** bhujay has quit IRC06:57
*** itlinux has quit IRC06:58
*** jogo has joined #openstack-nova07:00
*** mdnadeem has joined #openstack-nova07:01
openstackgerritblkart proposed openstack/nova-specs master: don't show description content in code-block  https://review.openstack.org/53947807:01
*** bhujay has joined #openstack-nova07:07
*** sahid has joined #openstack-nova07:10
*** lpetrut has quit IRC07:11
*** sidx64_ has joined #openstack-nova07:12
openstackgerritMerged openstack/nova stable/queens: Store block device mappings in cell0  https://review.openstack.org/54620107:12
*** bhujay has quit IRC07:13
*** Swami has quit IRC07:13
*** sidx64 has quit IRC07:13
*** sidx64 has joined #openstack-nova07:15
*** sidx64_ has quit IRC07:18
*** tylerderosagrund has joined #openstack-nova07:20
*** rcernin has quit IRC07:20
*** tylerderosagrund has quit IRC07:24
openstackgerritTakashi NATSUME proposed openstack/python-novaclient master: Fix a comment in novaclient/api_versions.py  https://review.openstack.org/54689707:30
*** andreas_s has joined #openstack-nova07:33
ameedaMorning :) , should I reverse os-xenapi version from 0.3.1 to 0.1.0 as this bug mentioned " https://bugs.launchpad.net/nova/+bug/1718606 "07:34
openstackLaunchpad bug 1718606 in OpenStack Compute (nova) "doc: Outdated XenServer document" [Low,Confirmed]07:34
*** sidx64 has quit IRC07:36
openstackgerritsahid proposed openstack/nova stable/queens: libvirt: disconnect volume from host during detach  https://review.openstack.org/54665507:36
*** pcaruana has joined #openstack-nova07:37
*** lpetrut has joined #openstack-nova07:37
*** john51 has quit IRC07:39
openstackgerritHuang Rui proposed openstack/nova master: z/VM Driver: add power actions  https://review.openstack.org/54334007:41
openstackgerritHuang Rui proposed openstack/nova master: z/VM Driver: add get console output  https://review.openstack.org/54334407:41
*** slaweq has joined #openstack-nova07:45
*** fragatin_ has quit IRC07:46
*** slaweq_ has joined #openstack-nova07:47
*** slaweq has quit IRC07:48
*** slaweq has joined #openstack-nova07:49
*** salv-orlando has joined #openstack-nova07:52
*** slaweq_ has quit IRC07:52
*** salv-orlando has quit IRC07:56
*** chyka has joined #openstack-nova07:56
*** ragiman has joined #openstack-nova07:57
*** Dinesh__Bhor has quit IRC07:57
*** slaweq_ has joined #openstack-nova08:00
*** chyka has quit IRC08:01
*** slaweq_ has quit IRC08:04
*** AlexeyAbashkin has joined #openstack-nova08:07
*** lpetrut has quit IRC08:09
openstackgerritTetsuro Nakamura proposed openstack/nova-specs master: Support shared and dedicated VMs in one host  https://review.openstack.org/54380508:11
openstackgerritTetsuro Nakamura proposed openstack/nova-specs master: Support shared and dedicated VMs in one host  https://review.openstack.org/54380508:14
*** threestrands_ has joined #openstack-nova08:15
*** threestrands_ has quit IRC08:15
*** threestrands_ has joined #openstack-nova08:15
*** hshiina|afk has quit IRC08:16
*** hshiina|afk has joined #openstack-nova08:16
*** liusheng has quit IRC08:17
*** hoonetorg has quit IRC08:18
openstackgerritMerged openstack/nova stable/queens: Update plugs Contrail methods to work with privsep  https://review.openstack.org/54677808:18
*** liusheng has joined #openstack-nova08:18
openstackgerritMerged openstack/nova stable/queens: Lazy-load instance attributes with read_deleted=yes  https://review.openstack.org/54598708:18
*** tesseract has joined #openstack-nova08:18
openstackgerritMerged openstack/nova stable/queens: Add functional tests to ensure BDM removal on delete  https://review.openstack.org/54620208:18
openstackgerritMerged openstack/nova stable/queens: Clean up ports and volumes when deleting ERROR instance  https://review.openstack.org/54620308:18
*** threestrands has quit IRC08:18
openstackgerritMerged openstack/nova stable/queens: Add functional recreate test of deleting a BFV server pre-scheduling  https://review.openstack.org/54620408:18
openstackgerritMerged openstack/nova stable/queens: Detach volumes when deleting a BFV server pre-scheduling  https://review.openstack.org/54627708:19
openstackgerritMerged openstack/nova stable/queens: Fix error handling in compute API for multiattach errors  https://review.openstack.org/54672908:19
*** hshiina2 has joined #openstack-nova08:21
*** tylerderosagrund has joined #openstack-nova08:24
*** hshiina|afk has quit IRC08:25
*** ralonsoh has joined #openstack-nova08:27
*** slaweq_ has joined #openstack-nova08:28
*** tylerderosagrund has quit IRC08:29
*** ttsiouts has quit IRC08:34
*** ccamacho has joined #openstack-nova08:34
*** hoonetorg has joined #openstack-nova08:35
*** strigazi has joined #openstack-nova08:36
openstackgerritNeha Alhat proposed openstack/nova master: Fix soft deleting vm fails after "nova resize" vm  https://review.openstack.org/54692008:36
*** ttsiouts has joined #openstack-nova08:37
*** sshwarts has joined #openstack-nova08:38
*** tbh_ has joined #openstack-nova08:40
*** sidx64 has joined #openstack-nova08:41
*** liusheng has quit IRC08:44
*** liusheng has joined #openstack-nova08:44
*** slaweq_ has quit IRC08:45
*** sapcc-bot1 has joined #openstack-nova08:47
*** sapcc-bot has quit IRC08:47
*** jpena|off is now known as jpena08:50
*** hshiina2 is now known as hshiina08:52
*** salv-orlando has joined #openstack-nova08:52
*** tssurya has joined #openstack-nova08:53
openstackgerritChen Hanxiao proposed openstack/nova master: libvirt: guest: introduce blockStats instead of domain.blockStats  https://review.openstack.org/52683308:54
*** damien_r has joined #openstack-nova08:56
*** salv-orlando has quit IRC08:57
*** salv-orlando has joined #openstack-nova08:57
bauzasgood morning Nova08:57
*** vivsoni has quit IRC08:59
*** vivsoni has joined #openstack-nova08:59
*** alexchadin has quit IRC09:03
*** moshele has quit IRC09:05
*** amoralej|off is now known as amoralej09:13
*** annp has quit IRC09:14
*** tiendc has quit IRC09:14
*** phuongnh has quit IRC09:14
*** phuongnh has joined #openstack-nova09:14
*** tiendc has joined #openstack-nova09:14
*** annp has joined #openstack-nova09:14
*** dtantsur|afk is now known as dtantsur09:17
*** hamzy_ has quit IRC09:20
*** hamzy_ has joined #openstack-nova09:20
*** tylerderosagrund has joined #openstack-nova09:25
*** takashin has left #openstack-nova09:30
*** tylerderosagrund has quit IRC09:30
openstackgerritYikun Jiang (Kero) proposed openstack/nova-specs master: [WIP] Allow Specifying Limit For Soft (Anti-)Affinity Groups  https://review.openstack.org/54692509:31
*** mgoddard_ has joined #openstack-nova09:31
*** sridharg has quit IRC09:32
*** moshele has joined #openstack-nova09:35
*** threestrands_ has quit IRC09:37
*** jaosorior has joined #openstack-nova09:39
*** Zames has joined #openstack-nova09:43
*** moshele has quit IRC09:45
*** moshele has joined #openstack-nova09:45
*** ratailor_ has joined #openstack-nova09:52
*** abhishekk has quit IRC09:53
*** ratailor has quit IRC09:55
*** danpawlik has joined #openstack-nova09:55
openstackgerritYikun Jiang (Kero) proposed openstack/nova-specs master: [WIP] Allow Specifying Limit For Soft (Anti-)Affinity Groups  https://review.openstack.org/54692509:57
*** sahid has quit IRC10:04
*** yamamoto has quit IRC10:06
*** Tom-Tom has quit IRC10:08
*** links has quit IRC10:09
*** Zames has quit IRC10:09
*** links has joined #openstack-nova10:10
*** Zames has joined #openstack-nova10:10
*** Zames has quit IRC10:12
*** lpetrut has joined #openstack-nova10:17
openstackgerritTetsuro Nakamura proposed openstack/nova-specs master: Support shared and dedicated VMs in one host  https://review.openstack.org/54380510:20
*** annp has quit IRC10:23
*** vipul has quit IRC10:28
* efried waves at bauzas10:31
bauzasefried: you're too early, go to bed10:31
bauzasor you're stuck in some airport, hopefully not traveling with our lovely national airline10:32
*** yamahata has joined #openstack-nova10:32
efriedbauzas: Trying to get ahead of jetlag a little bit.10:32
openstackgerritTetsuro Nakamura proposed openstack/nova-specs master: Support shared and dedicated VMs in one host  https://review.openstack.org/54380510:33
*** alexchadin has joined #openstack-nova10:34
*** sahid has joined #openstack-nova10:34
tetsurojaypipes, cfriesen: ^ updated the spec.10:34
bauzasefried: jetlag is just a subjective problem10:36
efriedSays bauzas, who's only one time zone off10:36
bauzasefried: heh, touché10:37
bauzasefried: but as an european, I mostly went to the US than any other region in the world for OpenStack :p10:37
efriedI grew up mostly in Europe with family back in the US, so I flew back a couple times a year and got to experience jetlag plenty as a kid.10:37
efriedSounds like you've got a good handle on it, and/or you're one of the lucky ones it doesn't affect badly.10:38
*** vipul has joined #openstack-nova10:38
efriedEasier traveling East, if I recall, as long as you have the discipline to stay up all night.10:39
efriedor maybe it's the other way around.  Sheesh, it HAS been a long time since I've been overseas.10:39
efriedsomething like 15y10:40
*** hieulq has quit IRC10:41
*** cz2 is now known as swagnemite10:41
*** Guest87022 has quit IRC10:44
bauzasefried: easier traveling West, if you're a night owl10:46
efriedbauzas: I haven't had the luxury of being a night owl in many many years.10:47
*** yassine has joined #openstack-nova10:47
*** yassine is now known as Guest9811610:47
efriedMan, I used to get my best work done between about 9pm and 3am.  But that was back in the college & startup days.10:48
efriedNow I have kids and a side business.10:48
bauzasI have kids too10:48
bauzasbut taking them at school at 4:30pm kills me10:48
bauzasso I tend to try working later in the evening10:48
*** trozet_ has joined #openstack-nova10:49
*** trozet has quit IRC10:49
*** tianhui_ has joined #openstack-nova10:50
*** swagnemite is now known as cz210:50
efriedMy wife *usually* handles the school runs on both ends.  She has multiple flexible part-time jobs, so it's fair :)10:51
efriedhm10:52
efriedbauzas: How do you feel about this:10:52
bauzasI recently invested in a Bose QC35 II, that helps a lot10:52
efriedI think when you do GET /allocation_candidates?required=BOGUS_TRAIT you get a 404 (because the trait isn't found).10:53
*** tianhui has quit IRC10:53
efriedShould that be a 400 instead?  (bad query params)10:53
*** Zames has joined #openstack-nova10:53
bauzasI'm not an API specialist10:53
efriedYeah, but you're *here*10:53
bauzasbut it's not a malformed query ?10:53
bauzasright?10:53
efriedWell, "malformed"...10:54
bauzasit's just that the resource isn't found10:54
bauzashence the 40410:54
efriedBut it's not the *resource* that's not found.  It's the *trait*.10:54
efriedI think all of our placement APIs return 200 with an empty list when you use filter criteria that yield zero results (but are otherwise valid)10:55
*** mhenkel_ has quit IRC10:55
efriedAnyway, this seems wrong to me.  I'm going to raise a bug and see how others feel.10:55
efriednever mind, I think I'm wrong.10:58
*** XueFeng_ has joined #openstack-nova10:58
openstackgerritSurya Seetharaman proposed openstack/nova-specs master: Support disabling a cell  https://review.openstack.org/54668410:59
*** ericyoung has quit IRC10:59
*** EmilienM has quit IRC10:59
*** obre_ has quit IRC10:59
*** links has quit IRC10:59
*** ingy has quit IRC10:59
*** XueFeng has quit IRC10:59
*** obre has joined #openstack-nova10:59
*** ingy1 has joined #openstack-nova10:59
*** ericyoung has joined #openstack-nova10:59
*** links has joined #openstack-nova10:59
*** EmilienM has joined #openstack-nova11:00
*** EmilienM has quit IRC11:00
*** EmilienM has joined #openstack-nova11:00
*** salv-orl_ has joined #openstack-nova11:00
*** yamahata has quit IRC11:00
openstackgerritSurya Seetharaman proposed openstack/nova-specs master: Support disabling a cell  https://review.openstack.org/54668411:01
openstackgerritEric Fried proposed openstack/nova master: rp: GET /resource_providers?required=<traits>  https://review.openstack.org/54683711:01
efriedka-blam ^11:01
bauzasefried: sorry, my computer decided on its own without asking me first to consider taking a rest11:02
bauzasso it froze11:02
efriedheh, lovely.11:02
*** mhenkel has joined #openstack-nova11:02
*** Zames has quit IRC11:03
*** salv-orlando has quit IRC11:03
efriedSo yeah, it was my mistake; GET /allocation_candidates probably returns 400 in that case.  I was missing an exception handler.11:03
bauzasefried: sweet11:03
efriedFeel free to review the above placement microversion bump with new query param :)11:03
*** namnh has quit IRC11:03
bauzasefried: sure, /me needs to look a few things11:03
efriedWho knows, if we get the right people on IRC today, we could have that blueprint completed before the PTG...11:04
*** yamamoto has joined #openstack-nova11:06
*** pcaruana has quit IRC11:07
efriedAh, dangit, forgot the reference docs.11:08
bauzasefried: well, sure, but I'm triaging the bugs now11:08
bauzas18 new bugs yesterday, now 22 :'11:08
bauzas:(11:08
efriedIt's totally not urgent.  Just be a nice tickymark to have11:08
*** alex____ has joined #openstack-nova11:09
*** alex____ has quit IRC11:09
*** fusmu has joined #openstack-nova11:10
*** ratailor_ has quit IRC11:12
*** yamamoto has quit IRC11:13
*** ratailor has joined #openstack-nova11:13
openstackgerritEric Fried proposed openstack/nova master: rp: GET /resource_providers?required=<traits>  https://review.openstack.org/54683711:14
efriedAPI ref added11:14
*** Jack_Iv has joined #openstack-nova11:19
hrwmonrign11:21
bauzasefried: interesting bug https://bugs.launchpad.net/nova/+bug/175079011:24
openstackLaunchpad bug 1750790 in OpenStack Compute (nova) "resources on target host aren't released if resize fails" [Undecided,New]11:24
bauzasefried: I wonder if we should support that11:24
bauzassome possible exception we forgot11:24
efriedbauzas: Just based on the title, it sounds like a valid bug.11:25
*** kaisers1 has joined #openstack-nova11:25
bauzasefried: the problem was that the folk forgot to provide SSH keys11:25
bauzashence the error11:26
efriedDoes the instance exist on both hosts while it's in the error state?11:26
bauzasno, because https://docs.openstack.org/nova/latest/admin/configuration/resize.html11:27
efriedOh, I see, so it didn't even get started.11:28
*** obre_ has joined #openstack-nova11:28
efriedbauzas: 2.16 - that's Pike, right?11:28
efriedYou may want to check with dansmith on this one, but I'm pretty sure in Pike we were still doing the "duplicate allocations" thing the wrong way.11:29
efriedSo this is at least a duplicate bug, and possibly a wontfix11:29
bauzasefried: 16.0.3 is the Pike 3rd stable release11:29
*** jgriffit1 has joined #openstack-nova11:29
efriedYeah, IIRC, we punted on actually trying to fix the duplicate allocations thing in pike.  I actually don't even know if we fully fixed it in queens - it depended on cdent's POST allocations thing - but I want to say we did.11:30
efriedBut don't quote me on this.  dansmith will know for sure, mriedem probably will too.11:30
*** rmk` has joined #openstack-nova11:31
efriedum, I also may be thinking of live migration - which isn't the same as resize??  I can never keep it straight.11:31
efriedGotta run for a couple hours.11:32
*** efried is now known as efried_rollin11:32
*** tiendc has quit IRC11:32
*** Faster-Fanboi_ has joined #openstack-nova11:32
*** phuongnh has quit IRC11:34
*** kaisers has quit IRC11:34
*** rmk has quit IRC11:34
*** cz2 has quit IRC11:34
*** obre has quit IRC11:34
*** jgriffith has quit IRC11:34
*** Faster-Fanboi has quit IRC11:34
*** rmk` is now known as rmk11:34
*** rmk has joined #openstack-nova11:34
*** cz2 has joined #openstack-nova11:36
*** sridharg has joined #openstack-nova11:39
*** jaosorior has quit IRC11:45
*** jaosorior has joined #openstack-nova11:45
*** Zames has joined #openstack-nova11:45
*** danpawlik has quit IRC11:45
*** Jack_Iv has quit IRC11:47
*** Zames has quit IRC11:48
*** vks1 has quit IRC11:48
*** danpawlik has joined #openstack-nova11:49
*** udesale_ has quit IRC11:50
*** tbachman has quit IRC11:51
*** jpena is now known as jpena|lunch11:57
*** danpawlik has quit IRC12:00
*** BryanS68 has joined #openstack-nova12:02
*** danpawlik has joined #openstack-nova12:05
*** Eran_Kuris has quit IRC12:06
*** takashin has joined #openstack-nova12:08
*** tetsuro has quit IRC12:09
*** yamamoto has joined #openstack-nova12:09
*** pcaruana has joined #openstack-nova12:09
*** sidx64 has quit IRC12:10
*** pchavva has joined #openstack-nova12:14
*** yamamoto has quit IRC12:14
*** aloga_ has joined #openstack-nova12:14
openstackgerritCarlos Goncalves proposed openstack/os-vif master: Introduce network namespace aware plugging  https://review.openstack.org/54696112:14
*** sidx64 has joined #openstack-nova12:15
*** aloga_ has quit IRC12:17
*** aloga_ has joined #openstack-nova12:17
*** sidx64 has quit IRC12:23
*** dave-mccowan has joined #openstack-nova12:26
*** ratailor has quit IRC12:27
*** janki has quit IRC12:27
*** yamamoto has joined #openstack-nova12:27
openstackgerritTakashi NATSUME proposed openstack/nova master: Transform servergroup.addmember notification  https://review.openstack.org/54110112:29
*** links has quit IRC12:29
*** Tom-Tom has joined #openstack-nova12:34
*** Jack_Iv has joined #openstack-nova12:34
*** Jack_Iv has quit IRC12:34
*** Zames has joined #openstack-nova12:35
*** takedakn has joined #openstack-nova12:35
*** aloga_ has quit IRC12:36
*** takedakn has quit IRC12:36
*** aloga_ has joined #openstack-nova12:38
*** chyka has joined #openstack-nova12:39
*** Zames has quit IRC12:40
*** links has joined #openstack-nova12:41
*** tylerderosagrund has joined #openstack-nova12:41
*** chyka has quit IRC12:43
*** wolsen has quit IRC12:50
*** wolsen has joined #openstack-nova12:51
*** zul has quit IRC12:51
*** yikun has quit IRC12:51
*** yikun has joined #openstack-nova12:52
*** Lingwu has quit IRC12:52
*** wxy has quit IRC12:52
*** mdurrant has quit IRC12:52
*** Lingwu has joined #openstack-nova12:52
*** wxy has joined #openstack-nova12:52
*** zhaochao has quit IRC12:53
*** josecastroleon has joined #openstack-nova12:54
*** blkart has quit IRC12:54
*** zul has joined #openstack-nova12:55
*** zhaochao has joined #openstack-nova12:55
*** blkart has joined #openstack-nova12:56
*** Zames has joined #openstack-nova12:57
*** Zames has quit IRC12:58
*** mchlumsky has quit IRC12:59
*** mrjk has quit IRC12:59
*** josecastroleon has quit IRC13:00
*** josecastroleon has joined #openstack-nova13:00
*** Zames has joined #openstack-nova13:01
*** damien_r1 has joined #openstack-nova13:02
*** damien_r has quit IRC13:03
*** Zames has quit IRC13:04
*** tylerder_ has joined #openstack-nova13:04
*** mchlumsky has joined #openstack-nova13:05
*** tylerderosagrund has quit IRC13:05
*** mrjk has joined #openstack-nova13:05
*** yikun has quit IRC13:06
*** yikun has joined #openstack-nova13:06
*** salv-orl_ has quit IRC13:10
*** salv-orlando has joined #openstack-nova13:11
*** Eran_Kuris has joined #openstack-nova13:11
*** amoralej is now known as amoralej|lunch13:13
*** betherly has quit IRC13:15
*** salv-orlando has quit IRC13:15
*** salv-orlando has joined #openstack-nova13:15
*** betherly has joined #openstack-nova13:15
*** hamzy_ is now known as hamzy13:16
*** abhishekk has joined #openstack-nova13:17
*** jpena|lunch is now known as jpena13:27
*** vivsoni has quit IRC13:28
*** vivsoni has joined #openstack-nova13:28
*** vladikr has joined #openstack-nova13:28
*** liverpooler has joined #openstack-nova13:31
*** ralonsoh_ has joined #openstack-nova13:32
*** udesale has joined #openstack-nova13:32
*** yamamoto has quit IRC13:34
*** yangyapeng has joined #openstack-nova13:34
*** efried_rollin is now known as efried13:35
*** ralonsoh has quit IRC13:35
*** yangyapeng has quit IRC13:39
*** acormier has joined #openstack-nova13:39
*** hshiina is now known as hshiina|afk13:40
*** fragatina has joined #openstack-nova13:41
*** mriedem has joined #openstack-nova13:42
mriedemmelwitt: are you running the meeting today?13:42
*** acormier has quit IRC13:42
*** yamamoto has joined #openstack-nova13:44
*** yamamoto has quit IRC13:44
*** vks1 has joined #openstack-nova13:45
mriedemany other takers?13:46
mriedemi've got a kid i have to get on the bus13:46
*** lyan has joined #openstack-nova13:46
jaypipesmriedem: when is it?13:47
mriedem12 minutes13:48
melwittmriedem: yeah, I didn't realize gibi was on vacation this week13:48
mriedemok13:48
kashyapmelwitt: Wauw, isn't it like 05:45 AM for you there?13:48
melwitttis13:48
kashyapJee13:49
melwittoh, fyi everyone I'm traveling today and tomorrow to the PTG. so I won't be around later today and tomorrow13:50
openstackgerritMerged openstack/nova stable/queens: Only pull associated *sharing* providers  https://review.openstack.org/54674013:51
jaypipesefried: you around?13:51
efriedjaypipes: Yup.13:51
sean-k-mooneymelwitt: nice :) going to do some siteseeing over the weekend?13:51
jaypipesefried: you pinged me last night about something but it's gone from my scrollback... what was it?13:51
efriedjaypipes: I desperately needed some SQL help.13:52
melwittsean-k-mooney: on saturday yes. on sunday we have an internal meeting day13:52
jaypipesefried: I don't know anything about SQL13:52
efriedjaypipes: But I slogged through it, with much cursing, and put up a patch for GET /resource_providers?required=<traits>13:52
jaypipesefried: ah, cool.13:52
* hrw prepared for a meetin. ate, coffee mug refilled ;D13:53
*** tianhui has joined #openstack-nova13:53
efriedjaypipes: We could conceivably get this blueprint closed before the PTG :)13:53
jaypipesefried: nothing wrong with that, IMHO.13:53
efriedjaypipes: https://review.openstack.org/#/c/546837/13:53
efriedYeah, I thought it would be a nice tickymark.13:53
jaypipesefried: that multiattach failure related?13:53
jaypipesdon't see how it could be, but worth asking...13:54
efriedjaypipes: No, random timeout.13:54
efriedI rechecked.13:54
jaypipesk13:54
*** mwhahaha has quit IRC13:54
jaypipesmelwitt, dansmith, mriedem, bauzas: one of the easier specs to +2/W here: https://review.openstack.org/#/c/546009/213:54
bauzasmriedem: melwitt: gibi_off told me about that, I was planning to chair it13:55
bauzasbut fine by me13:55
*** ralonsoh_ is now known as ralonsoh13:55
*** tianhui_ has quit IRC13:55
*** mwhahaha has joined #openstack-nova13:56
sean-k-mooneyjaypipes: efried: if your makeing a list of blueprints that could be closed before the ptg the nic feature based scheduling blueprint has been code complete since pike and is fully rebased for rocky. spec needs to be reapproved and patches re reviwed though13:56
*** blkart has quit IRC13:56
melwittbauzas: okay, cool. yeah, I realized I'd have to get up early to make sure you were able to run it anyway so I might as well do it. it was my bad. thanks though13:56
jaypipessean-k-mooney: link pls13:56
*** imacdonn has quit IRC13:56
*** BryanS68_ has joined #openstack-nova13:57
sean-k-mooneyjaypipes: code https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/enable-sriov-nic-features spec https://review.openstack.org/#/c/545951/13:57
sean-k-mooneyon tox faild on the spec ill go fix that now13:57
jaypipessean-k-mooney: danke13:58
sean-k-mooneydid something change in the template between rocky and queens?13:58
*** amoralej|lunch is now known as amoralej13:58
*** Kevin_Zheng has quit IRC13:58
*** tbachman has joined #openstack-nova13:58
melwittsean-k-mooney: there's one new section, Upgrade impact13:58
bauzasmelwitt: heh, no worries13:59
*** BryanS68 has quit IRC13:59
sean-k-mooneymelwitt: ah yes that is exactly the error http://logs.openstack.org/51/545951/1/check/openstack-tox-pep8/9898780/job-output.txt.gz#_2018-02-19_20_27_16_685503 ill add it and reupload13:59
mriedemjaypipes: i'll look after said busification13:59
mriedemand coffee injections14:00
*** kolesovdv has joined #openstack-nova14:00
bauzasmriedem: alrready looking ;)14:00
jaypipesmriedem: 10-414:00
*** Kevin_Zheng has joined #openstack-nova14:02
*** tbachman has quit IRC14:03
openstackgerritCarlos Goncalves proposed openstack/os-vif master: WIP: Add internal port  https://review.openstack.org/54699114:05
*** jmlowe has quit IRC14:06
*** imacdonn has joined #openstack-nova14:06
*** alexchadin has quit IRC14:08
*** tbachman has joined #openstack-nova14:09
*** lbragstad has joined #openstack-nova14:10
*** sahid has quit IRC14:10
openstackgerritTetsuro Nakamura proposed openstack/nova-specs master: Support shared and dedicated VMs in one host  https://review.openstack.org/54380514:11
*** hshiina|afk has quit IRC14:12
openstackgerritTetsuro Nakamura proposed openstack/nova-specs master: Support shared and dedicated VMs in one host  https://review.openstack.org/54380514:13
bauzasmelwitt: mriedem: I'm about to set Invalid for https://bugs.launchpad.net/nova/+bug/175103614:13
openstackLaunchpad bug 1751036 in OpenStack Compute (nova) "ConfigFileValueError when defining enabled_apis" [Undecided,New]14:13
bauzasbecause of the interop APIs14:14
mriedembauzas: i'm in a meeting atm14:14
bauzasbut do you agree ?14:14
bauzasthat I now, let's discuss later14:14
bauzasknow*14:14
*** derekh has joined #openstack-nova14:15
*** tetsuro has joined #openstack-nova14:15
*** trozet_ is now known as trozet14:19
*** kholkina_ has quit IRC14:25
*** baoli has joined #openstack-nova14:27
openstackgerritTetsuro Nakamura proposed openstack/nova-specs master: Support shared and dedicated VMs in one host  https://review.openstack.org/54380514:27
*** baoli has quit IRC14:28
*** jmlowe has joined #openstack-nova14:28
*** baoli has joined #openstack-nova14:29
*** esberglu has joined #openstack-nova14:31
*** alexchadin has joined #openstack-nova14:35
*** Guest96947 has quit IRC14:36
*** tidwellr has quit IRC14:36
*** sree_ has joined #openstack-nova14:36
*** sree_ is now known as Guest6481014:37
*** dtantsur is now known as dtantsur|brb14:38
tetsurojaypipes, cfriesen: now this spec https://review.openstack.org/543805 is ready for review again.14:38
*** dklyle has quit IRC14:39
*** Guest64810 has quit IRC14:41
jaypipestetsuro: cool, thanks. (currently in nova meeting)14:41
openstackgerritEric Fried proposed openstack/nova master: Make _get_sharing_providers more efficient  https://review.openstack.org/54700814:42
tetsuroOops14:42
efriedjaypipes, cdent: ^ the optimization that prompted the feature14:42
jaypipestetsuro: it's no prob :)14:44
*** yamamoto has joined #openstack-nova14:45
*** josecastroleon has quit IRC14:45
*** josecastroleon has joined #openstack-nova14:46
*** eharney has joined #openstack-nova14:46
*** r-daneel has joined #openstack-nova14:46
*** psachin has quit IRC14:49
*** Eran_Kuris has quit IRC14:49
*** sahid has joined #openstack-nova14:50
*** tbh_ has quit IRC14:50
*** ameeda has quit IRC14:51
*** yamamoto has quit IRC14:51
*** josecastroleon has quit IRC14:53
tetsurojaypipes: I put your proposal into alternative section. The proposal sounds really attractive to me. But it is not kindful to operators, I guess.14:53
jaypipestetsuro: the proposal to just determine the shared pin set by subtracting the dedicate pin set from vcpu_pin_set?14:54
*** abhishekk has quit IRC14:55
tetsurojaypipes: right.14:55
jaypipestetsuro: gotcha. ok, will re-review this morning :)14:56
tetsurothanks in advance!14:56
*** awaugama has joined #openstack-nova14:57
*** namnh has joined #openstack-nova14:57
jaypipestetsuro: no problemo :)14:58
*** josecastroleon has joined #openstack-nova14:58
*** takashin has left #openstack-nova15:00
*** fragatina has quit IRC15:00
bauzasmelwitt: mriedem: maybe worth discussing Rambo's spec at the PTG during the cinder/nova cross-session on Thurs ?15:01
*** tetsuro has left #openstack-nova15:01
*** wolverineav has joined #openstack-nova15:01
bauzasor we could discuss the design before that, but if after, we loose the possibility to ask Cinder for a rebuild thingy15:02
*** baoli has quit IRC15:05
*** wolverineav has quit IRC15:05
*** baoli has joined #openstack-nova15:06
melwittbauzas: that might be a good idea if we have time. since it's possible there's something they could do on their side to facilitate it15:07
bauzasmelwitt: for the moment I added L333 on https://etherpad.openstack.org/p/nova-ptg-rocky15:08
*** tidwellr has joined #openstack-nova15:08
bauzasmelwitt: because mriedem wanted to discuss about all the rebuild problems we have15:08
mriedemthere is nothing i would like to discuss more than rebuild problems15:14
*** moshele has quit IRC15:15
bauzasmriedem: melwitt: maybe we could start a discussion about rebuild on wed if time allows15:15
bauzasI know there is a huuuuuge Placement discussion that would occur most on all the afternoon, but maybe I should be tired after 3 hours ;)15:17
*** mlavalle has joined #openstack-nova15:18
*** udesale has quit IRC15:19
*** mdnadeem has quit IRC15:19
*** links has quit IRC15:20
mriedemif we talk about this rebuild + bfv thing,15:20
mriedemit should either be with the cinder people in the room,15:21
mriedemor on misc friday15:21
melwitt+115:21
mriedemb/c this is a misc thing15:21
bauzasmriedem: melwitt: FWIW, I'll need to leave the room by Friday noon15:22
bauzasfor my flight :(15:22
bauzasbut yeah, we could discuss about rebuilding volume-backed stuff with Cinder folks15:23
bauzasanyway15:23
mriedemwe'll be sure to schedule all cheese-related discussions for friday morning15:23
dansmithjaypipes: I approved that spec15:26
dansmithjaypipes: you can repay me by approving this compute rpc compat drop, which I know you want anyway: https://review.openstack.org/#/c/543580/15:26
*** dtantsur|brb is now known as dtantsur15:26
*** alexchadin has quit IRC15:26
openstackgerritEric Berglund proposed openstack/nova master: WIP: PowerVM Driver: Network interface attach/detach  https://review.openstack.org/54681315:27
*** alexchadin has joined #openstack-nova15:27
*** slaweq has quit IRC15:30
*** slaweq has joined #openstack-nova15:30
bauzasmriedem: on an interop concern, should we accept other APIs but the ones we have in-tree ? https://bugs.launchpad.net/nova/+bug/175103615:31
openstackLaunchpad bug 1751036 in OpenStack Compute (nova) "ConfigFileValueError when defining enabled_apis" [Undecided,New]15:31
bauzasI tend to say no15:31
*** alexchadin has quit IRC15:31
*** acormier has joined #openstack-nova15:32
*** slaweq has quit IRC15:35
melwittmriedem: please double check me on the rc3 tag https://review.openstack.org/#/c/547031/1/deliverables/queens/nova.yaml15:36
*** amodi has joined #openstack-nova15:36
*** alexchadin has joined #openstack-nova15:37
*** sshwarts has quit IRC15:37
mriedemefried: jaypipes: dansmith: left a comment in that spec https://review.openstack.org/#/c/546009/ - fine to amend with some details15:37
*** krtaylor has quit IRC15:38
mriedemmelwitt: done15:39
mriedemmelwitt: also that commit hash might end up being a merge commit15:39
mriedemthat's why we need the depends-on15:39
melwittmriedem: thanks. so if I keep the commit hash the same + depends-on, it will work fine?15:39
*** yamahata has joined #openstack-nova15:40
mriedemnot if we need to use a merge commit15:40
mriedemi would just add the depends-on and then WIP15:40
mriedemdon't want to get caught by the old smcginnis trigger finger15:41
melwittoh okay. I thought I had seen you propose a commit hash ahead of merge before but maybe I made that up15:41
smcginnis:)15:41
mriedemi generally do, and the job fails until i fix it up15:41
melwittthanks15:41
*** alexchadin has quit IRC15:42
openstackgerritMerged openstack/nova-specs master: Filter Resource Provider List for Traits  https://review.openstack.org/54600915:44
jaypipesmriedem: k15:44
mriedembauzas: i'm sort of surprised we don't allow ec2 in enabled_apis? i'm not sure how the ec2api code works w/o that15:45
*** josecastroleon has quit IRC15:46
*** hongbin has joined #openstack-nova15:46
bauzasmriedem: ec2api is somehow stale now AFAIK15:46
bauzasansiwen: around ?15:47
*** yamamoto has joined #openstack-nova15:47
bauzasansiwen: see mriedem's point about ec2api not being in the enabled apis15:47
bauzasif so, is it run separately?15:47
mriedemit's still maintained15:47
bauzasmriedem: back to my original concern, I don't see why Nova should support other out-of-tree APIs15:47
bauzasif people want to run their own APIs, why not running a specific WSGI server et voilà ?15:48
bauzasa new endpoint15:48
*** lajoskatona has quit IRC15:49
edleafebauzas: I've wanted to simplify the pathing to reflect that there is only one Nova API15:49
edleafebauzas: e.g., change nova/api/openstack/compute to just nova/api/compute15:49
*** david-lyle has joined #openstack-nova15:50
edleafeBut alex_xu has pointed out there are compatibilty issues15:50
edleafewith api-paste.ini15:50
bauzasedleafe: so you pointed me that enabled_apis shouldn't accept more than just nova and metadata ?15:50
mriedemwhat does the GCE API do?15:50
bauzass/pointed/agreed with ?15:50
*** yamamoto has quit IRC15:51
*** zhaochao has quit IRC15:52
bauzasmriedem: mmm, WAT ?15:52
dansmithmriedem: I guess I see that distinction as a fundamental difference between allocation_candidates and resource_providers15:52
mriedemhttps://github.com/openstack/gce-api/blob/master/gceapi/cmd/api.py#L3415:52
bauzaswe have a GCE proxy ?15:52
mriedembauzas: the gce api proxy is way old15:52
dansmithmriedem: which is kinda why we ended up adding a_c I think15:52
* bauzas is amazed15:52
edleafebauzas: yeah, we only support the openstack api, so the extra directory level in the project is not needed15:53
bauzasanyhow15:53
bauzaslet's triage the bug as Invalid15:53
bauzasand ask them to run their own WSGI service15:53
bauzas(FWIW, I know those OCCI folks since 2013, I'm not surprised they never went upstream)15:54
bauzasmriedem: back to the ec2api, ansiwen told us that it was barely maintained15:55
mriedembauzas: done15:55
bauzasso maybe it's some misunderstanding from me15:55
mriedemwell the occi nova ooi or whatever code is in github15:55
mriedemso it's available, which is fine15:55
*** blkart has joined #openstack-nova15:58
*** BryanS68_ has quit IRC15:59
*** BryanS68 has joined #openstack-nova15:59
*** tbachman has quit IRC16:00
efriedmriedem: What dansmith said.  It doesn't make sense for me to say "return me a list of resource providers where the union of all their traits is [this list]".  Because I would either get all of them or none of them.  An allocation candidate is a list of *groups* of resource providers, where each *group* must satisfy the list of traits.  s/traits/resources/ and the answer is the same.16:02
efriedI.e. the difference between rp and ac for traits is the same as the difference between rp and ac for resources, if that makes any sense :)16:02
dansmithefried: right, that's how I think of it exactly16:03
*** itlinux has joined #openstack-nova16:03
mriedemi get it fellas16:06
mriedemand it was discussed in comments in the spec,16:06
mriedembut for posterity sake,16:07
mriedemlet's doc that in the spec16:07
mriedemso a year from now we can say why16:07
efriedmriedem: Roger that, will do.16:07
dansmithmriedem: if you mean saying "because a_c is $description, we behave this way on this..." then sure16:07
dansmithI don't really want to have to fully explain a_c every time we do something like this, but...16:08
mriedemyou can explain it now in a doc so we can point people to it,16:08
mriedemor you can explain it in channel once per quarter16:08
efriedor I can win the lottery and dansmith can explain it.16:10
efriedI'll write a fup.16:10
mriedemwill there be duty free irish spring at the airport?16:12
ansiwenbauzas: mriedem , yeah, ec2api is a pure "hobby" project nowadays. Which doesn't mean it's not maintained. But there is less activity to be expected, compared to a project which is backed by a company that has commercial interests.16:12
mriedemansiwen: ack, and it has users, so maintenance is probably ok16:13
bauzascall it maintenance state then16:13
bauzasanyway16:13
ansiwenbauzas: mriedem _if_ something breaks, it could take quite a while until it is fixed.16:15
mriedemansiwen: it takes us 4 years to fix a bug16:15
mriedemso it's fine16:15
bauzasthat's exactly what a maintenance state describes16:15
bauzasit doesn't tell you how long you will fix bugs16:15
ansiwenmriedem: he he... ok, fair enough :-)16:15
bauzasit will just say "okay, we know we have bugs and we will provide bugfixes soon"16:15
mriedemsoon, or not,16:16
mriedemwhatever16:16
bauzas"but for features, don't get me wrong and don't expect a pony"16:16
mriedemunless that pony makes it run at the EDGE16:16
bauzas"unless you have your pockets full of dollars"16:17
bauzasor euros, rather16:17
mriedemright that too16:17
bauzasgiven the exchange rate16:17
* mriedem makes kick ass air guitar motions and sounds16:17
*** namnh has quit IRC16:17
mriedemjaypipes: so on your compute capabilities traits patch, i was thinking,16:18
bauzasmriedem: unrelated, I looked at that bug this EU morning https://bugs.launchpad.net/nova/+bug/175079016:18
jaypipesuhoh16:18
openstackLaunchpad bug 1750790 in OpenStack Compute (nova) "resources on target host aren't released if resize fails" [Undecided,New]16:18
mriedemit should be possible to do my nova patch that adds the custom capabilities traits, then if/when we have them standardized in os-traits, we can swap those out, should be ok...16:19
bauzasmriedem: and I wonder if that would merit a Wontfix given it's a pike change16:19
mriedemalthough, anything relying on the former would have to change..16:19
bauzass/pike change/pike bug/16:19
mriedemjaypipes: i'm trying to think through what happens when we add a new capability, if that depends on putting it in os-traits first, and it probably should16:19
mriedembecause if we have say a placement request filter adding in required traits for driver capabilities, it has to specify one or the other16:20
mriedemor maybe not...16:20
jaypipesmriedem: I wouldn't advise the "auto-create CUSTOM_COMPUTE" thing for that reason...16:20
*** tbachman has joined #openstack-nova16:20
mriedemi guess if allocation candidations would return either it'd be fine16:20
mriedem*candidates16:20
openstackgerritMathieu Gagné proposed openstack/nova master: Fix rebuild of baremetal instance when vm_state is ERROR  https://review.openstack.org/52355916:20
jaypipesmriedem: I'd instead advise having the virt drivers report specific known os-traits compute traits.16:20
mriedemyeah i agree that's the goal16:21
*** ingy1 is now known as ingy16:21
mriedemso probably worth doing that to start16:21
mriedemi.e. land your os-traits patch, release it, and then use it in my patch16:21
jaypipesmriedem: ya. soon as efried stops being such a pedant about it :P16:21
efriedoo, that's a good point jaypipes16:22
efried(not about me being a pedant, which is also a good point)16:22
jaypipes:)16:22
efriedWe should move the capabilities reporting away from the awkward dict and just have virt drivers report as traits directly.16:23
efried(long term)16:23
mriedemdon't get me started on awkward dict16:24
jaypipesoh boy, here it comes16:25
openstackgerritEric Fried proposed openstack/nova-specs master: Explain how filters differ for /r_p vs. /a_c  https://review.openstack.org/54705616:25
efriedmriedem: There ya go, ya pedant ^16:25
efrieddansmith: ^16:25
dansmithefried: disappointed you didn't shame mriedem a little more with commit message snark16:25
efriedoh, I forgot my backreference, sec...16:26
edleafeefried: yeah, -1 for lack of snark16:26
mriedemgive me a f'ing break16:26
mriedemfrom the placement people that complain constantly about not documenting decisions enough16:26
openstackgerritEric Fried proposed openstack/nova-specs master: Explain how filters differ for /r_p vs. /a_c  https://review.openstack.org/54705616:26
efriedfixed commit message ^16:26
dansmithmriedem: I kid, I kid <316:27
efriedmriedem: You're right, you're right, you're always right.16:27
mriedemdansmith: you're not in the 'we don't document stuff enough' camp16:27
efried(that chant doesn't come across quite the same in IRC as when I have to say it to my wife)16:27
dansmithmriedem: yep, just wild-west cowboy style for me.. pew pew16:27
jaypipeslol16:28
mriedemi assume those chaps are assless16:29
mriedemwhich is redundant i know16:29
efriedIt's gonna be low to mid 30s in Dublin.16:30
jaypipeswell, assless chaps really are the best place to store an awkward dict.16:30
jaypipesI'll be here all week folks.16:30
efriedPlease dansmith, for the love of all that is hol(e)y, wear something under those chaps.16:30
efriedNot a kilt, either16:30
mriedemeasy bug fix here compadres https://review.openstack.org/#/c/546268/16:32
*** tbachman has quit IRC16:33
jaypipes-216:34
mriedemhongbin: i was thinking about your simple fix here https://review.openstack.org/#/c/546357/ - we should likely have a functional test change with that too, and i think there might be an easy way to do that with an existing test16:35
efriedjaypipes, dansmith: Does GET /resource_providers?resources=... take allocations into account?16:35
jaypipesefried: yes16:35
efriedk16:35
mriedemhongbin: see https://review.openstack.org/#/c/546268/1/nova/tests/functional/test_servers.py - you can probably modify that same functional test such that before posting the rebuild action, we change the compute provider's inventory such that placement wouldn't return the provider w/o your fix16:36
hongbinmriedem: sure, will look into it16:36
jaypipesmriedem: done.16:37
mriedemhongbin: thanks for that patch btw, it was a lot simpler than i thought it would be16:37
mriedemjaypipes: thanks16:37
*** fusmu has quit IRC16:37
*** andreas_s has quit IRC16:38
*** yikun has quit IRC16:38
*** yikun has joined #openstack-nova16:38
*** tbachman has joined #openstack-nova16:39
*** ccamacho has quit IRC16:40
*** slaweq has joined #openstack-nova16:40
*** tbachman_ has joined #openstack-nova16:41
*** moshele has joined #openstack-nova16:41
*** damien_r1 has quit IRC16:42
*** david-lyle is now known as dklyle16:42
*** andreas_s_ has joined #openstack-nova16:43
*** tbachman has quit IRC16:44
*** tbachman_ is now known as tbachman16:44
openstackgerritEric Fried proposed openstack/nova master: Clarify `resources` query param for /r_p and /a_c  https://review.openstack.org/54706516:44
efriedjaypipes, mriedem, dansmith: There's that clarification you asked for ^16:44
mriedemyou missed your reference again16:44
*** tianhui has quit IRC16:45
jaypipeslolol16:45
*** tianhui has joined #openstack-nova16:45
efriedthe faah?16:45
*** slaweq has quit IRC16:45
mriedemefried: but thanks for actually following up with my complaints16:45
mriedemkarma points abound16:45
efriedHigh impact:work ratio16:45
* jaypipes envisions mriedem as the kung-fu master from Kill Bill Vol 216:45
mriedemthe guy that died via sexual asphyixiation?16:46
jaypipesrofl, no16:46
efriedoy vay, you people.16:46
mriedemoh the white haired guy16:46
jaypipesyes16:46
jaypipesthanks goodness I wasn't sipping coffee just nopw16:46
mriedemi was thinking carradine16:46
mriedemisn't that how he died?16:46
jaypipesyes, I got that.16:47
mriedemhe's dead right?16:47
jaypipesyes16:47
mriedemok, whew16:47
openstackgerritEric Fried proposed openstack/nova master: Clarify `resources` query param for /r_p and /a_c  https://review.openstack.org/54706516:47
*** andreas_s has joined #openstack-nova16:47
efriedUpdated commit msg, sheesh ^16:47
*** andreas_s_ has quit IRC16:47
*** moshele has quit IRC16:47
bauzasdansmith: https://bugs.launchpad.net/nova/+bug/1750790 should be marked as Wontfix, right?16:48
openstackLaunchpad bug 1750790 in OpenStack Compute (nova) "resources on target host aren't released if resize fails" [Undecided,New]16:48
bauzasdansmith: as it's a Pike bug, and you fixed that in queens16:48
*** yamamoto has joined #openstack-nova16:48
*** chyka has joined #openstack-nova16:48
dansmithum16:49
kashyapbauzas: BTW, once distributions upgrade to Linux 4.14 kernel, this will cease to be "Low" priority16:49
kashyapIt'll definitely bite us, and cause nuisance16:49
dansmithbauzas: I'm not sure why we wouldn't fix it in pike just because we fixed it in queens16:50
* dansmith is waiting for united wifi to load the bug16:50
efrieddansmith: bauzas may be basing that statement on something I said at 5am.  This is about duplicate allocations on source & target.16:50
bauzasdansmith: no worries, I'll leave the bug open16:50
bauzasI'm way off reviewing specs, and I need to16:51
dansmithefried: okay16:51
bauzasso I'll stop looking at bugs16:51
dansmithI expect it's fixed in queens because of migration allocations16:51
mriedembauzas: that's likely a bug that claudiub has a fix up for16:51
efriedI didn't know whether we punted fixing that in pike because too complicated, and just fixed it in Queens with cdent's POST allocations business16:51
dansmithbut we probably need a fix for pike still16:51
*** andreas_s has quit IRC16:51
mriedemdansmith: bauzas: not necessarily16:51
mriedemdepends on where the failure happens16:51
efriedyeah.  But I also didn't know whether resize ~= migration in this case16:51
mriedemresize = cold migrate16:52
efriedbut still subject to duplicate allocations mriedem?16:52
efriedi would spect so16:52
mriedembauzas: dansmith: this https://review.openstack.org/#/c/543971/16:52
dansmithmriedem: um, what? we only need a fix depending on where it happens?16:52
*** ragiman has quit IRC16:52
*** yamamoto has quit IRC16:52
mriedemefried: all move operations require an allocation on the source and dest hosts,16:52
efriedmakes sense.16:53
mriedempre-queens, we just doubled those rather than split them16:53
bauzasmriedem: excellent, tz16:53
bauzaswill close the bug as duplicate16:53
mriedemdansmith: if prep_resize fails, we cleanup the allocations16:53
mriedemif resize_instance fails, we don't, which is https://review.openstack.org/#/c/543971/16:53
mriedemthere are 10 places you can blow off your foot with resize16:53
dansmithsure, sounded like you were disagreeing with me that we need a fix16:53
*** kolesovdv has quit IRC16:54
mriedemi think we're talking about different things16:54
mriedemor different failure scenarios16:54
bauzasright16:54
bauzasanyway, I'm done with triaging bugs for that week16:54
mriedembauzas: i don't know if it's a duplicate, depends on where it fails16:55
*** tidwellr has quit IRC16:55
mriedemwhere does resize fail for missing ssh host keys?16:55
bauzasmriedem: anyhow, the bug was reported because the user forgot to provide SSH keys16:55
*** wolverineav has joined #openstack-nova16:55
*** tidwellr has joined #openstack-nova16:55
bauzaswhile we ask for that16:55
*** wolverineav has quit IRC16:55
mriedemok let's assume it's the same problem then :)16:55
mriedemif it's not, they can re-open16:55
*** wolverineav has joined #openstack-nova16:55
bauzaszactly16:55
*** dklyle has quit IRC16:56
*** vks1 has quit IRC17:01
*** david-lyle has joined #openstack-nova17:01
openstackgerritMerged openstack/nova stable/queens: Add functional test for deleting BFV server with old attach flow  https://review.openstack.org/54680317:02
openstackgerritMerged openstack/nova stable/queens: Ensure attachment_id always exists for block device mapping  https://review.openstack.org/54680417:02
*** BryanS68 has quit IRC17:03
*** BryanS68 has joined #openstack-nova17:03
mriedem+2 on claudiub's fix now https://review.openstack.org/#/c/543971/17:03
*** sridharg has quit IRC17:04
*** links has joined #openstack-nova17:04
*** sahid has quit IRC17:08
*** fragatina has joined #openstack-nova17:08
*** AlexeyAbashkin has quit IRC17:09
mriedemgdi i'm annoyed that we run several tempest jobs for docs changes17:09
mriedemand that irrelevant-files isn't overridable17:09
mriedemmdbooth: are you around?17:12
mdboothmriedem: I am!17:13
*** pchavva has quit IRC17:13
mriedemmdbooth: can you take a look at https://review.openstack.org/#/c/542646/ ? it's related to a restriction / validation you put into the api17:13
openstackgerritEric Fried proposed openstack/nova master: rp: GET /resource_providers?required=<traits>  https://review.openstack.org/54683717:15
openstackgerritEric Fried proposed openstack/nova master: Make _get_sharing_providers more efficient  https://review.openstack.org/54700817:15
efriededleafe: ^^17:15
efriedI mean ^17:15
efriedAnyone else have an IRC client that converts emoticons to graphics?17:15
mdboothmriedem: So it's my understanding that your reproducer *should* fail.17:18
efriedThat *would* be an awkward dict situation17:19
mdboothmriedem: I have always taken min_disk to mean: the stuff this image does requires at least this much space.17:19
mdboothWhich is obviously different to the image size.17:20
*** pchavva has joined #openstack-nova17:20
*** pchavva has quit IRC17:20
*** sridharg has joined #openstack-nova17:21
mriedemmdbooth: i don't think that's the original image min_disk though17:22
mriedemwe say that's the min_disk b/c of the flavor root_gb17:23
mdboothmriedem: Reading back a bit more I was just going to ask exactly that.17:23
mdboothIt sounds to me like the bug is in the setting of min_disk in the first place, then.17:23
mdboothIt should be the min_disk of the original image.17:23
mriedemso on my devstack right now, the single cirros image has min_disk=017:23
*** baoli has quit IRC17:24
mriedemi can create a 1GB volume as the boot disk for my server that has that image in it,17:24
mriedemsnapshot it, and nova says the min_disk for the snapshot is now equal to the flavor.root_gb17:24
*** baoli has joined #openstack-nova17:24
mriedem20 in my case17:24
mriedem20GB17:24
mdboothIMHO the bug is " and nova says the min_disk for the snapshot is now equal to the flavor.root_gb"17:25
mriedemlemme create a volume from the image and see what the image meta shows in the volume17:25
mriedemi agre17:25
mriedem*agree17:25
mdboothIOW, I think the bug happens before the proposed fix17:25
*** gyee has joined #openstack-nova17:25
claudiubmriedem: regarding your comment on https://review.openstack.org/#/c/543971/5/nova/compute/manager.py , you mean the docstring which is on finish_resize (L4444), or a new one explaining the allocation cleanup part?17:26
*** aloga_ has quit IRC17:26
mriedemmdbooth: ok so i created a 1GB volume from a cirros image in devstack http://paste.openstack.org/show/682285/17:26
mriedem| volume_image_metadata        | {u'container_format': u'bare', u'min_ram': u'0', u'disk_format': u'qcow2', u'image_name': u'cirros-0.3.5-x86_64-disk', u'image_id': u'9491dd49-665c-46f4-9dbf-9a94aa1c9843', u'checksum': u'f8ab98ff5e73ebab884d80c9dc9c7290', u'min_disk': u'0', u'size': u'13267968'} |17:26
mriedemthe volume image meta shows the internal image min_disk is 017:26
mriedemmdbooth: what i'm not sure about, is if nova should say the volume-backed snapshot image min_disk should be the original image min_disk, or the volume size?17:27
mriedemor, maybe we just don't care17:27
mriedemand shouldn't even put min_disk in the snapshot17:27
mriedembecause if you use that thing to create another server, nova is going to boot from volume using the volume snapshot metadata embedded in the snapshot image17:27
mriedemie block_device_mapping='[{"guest_format": null, "boot_index": 0, "delete_on_termination": false, "no_device": null, "snapshot_id": "1158debc-9922-458d-9b46-8ef688334d65",  "device_name": "/dev/vda", "disk_bus": "virtio", "image_id": null,  "source_type": "snapshot", "tag": null, "device_type": "disk",  "volume_id": null, "destination_type": "volume", "volume_size": 1}]'17:28
mriedemso nova shouldn't really care about min_disk in this case17:28
mriedemwe're not going to create an ephemeral disk for the server, we're going to create a volume17:28
*** slaweq has joined #openstack-nova17:29
mriedemhere https://github.com/openstack/nova/blob/master/nova/virt/block_device.py#L68417:29
openstackgerritMerged openstack/nova-specs master: Explain how filters differ for /r_p vs. /a_c  https://review.openstack.org/54705617:29
mriedemclaudiub: i meant on the new _finish_resize_helper method,17:29
mdboothI think min_disk should stay, and be the min_disk of the original image, because it's image metadata.17:30
mdboothSame as hw_* says the image needs certain hardware17:30
mdboothmin_disk says it needs so much disk17:30
mriedemfor the image sure,17:31
mriedemso the min_disk on the image could be 100MB, and the volume could be 1GB17:31
mriedemanyway, that also works for me i think17:31
mdboothmin_disk is about the *workload*, not how big the disk is17:31
mdboothWe already know exactly how many bits are physically required to unpack it.17:31
mdboothmin_disk says how much space is required to operate17:32
mdboothIOW we shouldn't be arbitrarily setting it where it wasn't set before.17:32
mdboothIf we're going to exclude it, it should be in the context of a generic ability to filter arbitrary image metadata when creating a snapshot.17:33
mdboothOr entirely disabling copying image metadata, more likely.17:34
mdboothBut I don't think min_disk can be treated specially by excluding it if it's set.17:35
mriedemwell we do have the non_inheritable_image_properties config option, which sucks since it's config-based API behavior17:35
mdboothYeah17:36
mriedemi guess that's just more about stuff you put into your images which shouldn't apply across snapshots, like signatures17:36
mdboothFor now, I think the solution is to find whatever's setting it where it wasn't set before and kill that.17:36
mdboothHopefully it's not cinder :/17:37
mriedemnot following you - setting it where it wasn't set before?17:37
mriedemthat's nova17:37
mriedemhttps://review.openstack.org/#/c/542646/2/nova/compute/api.py17:37
mriedemoh sec17:38
openstackgerritClaudiu Belu proposed openstack/nova master: hyper-v: autospec classes before they are instantiated  https://review.openstack.org/34221117:38
mriedemmdbooth: nova.utils.get_image_from_system_metadata17:38
*** pchavva has joined #openstack-nova17:38
mriedembut yeah i'm not exactly where where that min_disk value is coming from the flavor.root_gb17:39
mriedem*sure17:39
* mdbooth has a horrible idea he's touched that before17:39
mdboothI think there might have been some code Jenga involved.17:40
jaypipescfriesen: ok, now I'm confused... I thought "the VM's I/O threads" and "the VM's vCPU threads" were the same thing :(17:40
mriedemmdbooth: found it17:43
cfriesenjaypipes: you have emulator threads, IO threads, and vCPU threads.17:43
*** tssurya has quit IRC17:43
mriedemmdbooth: https://github.com/openstack/nova/blob/master/nova/utils.py#L98917:43
jaypipescfriesen: oh? is this something new in libvirt?17:43
mriedemmdbooth: that says if the image min_disk is not set, or 0, just store it as the flavor root disk in the instance system_metadata, which we then use later to build the image meta for the snapshot17:44
cfriesenjaypipes: no.  if you're just running pure userspace code in the guest then there's no work for the IO threads.  they're used for reading from virtual disk and such17:44
dansmithjaypipes: no :)17:44
mdboothmriedem: Yep, I think that's it.17:44
mdboothmriedem: I have a suspicion that at some point in the past it was really hard to change min_disk due to assumptions about it dotted around the code.17:45
jaypipescfriesen, dansmith: I've always just referred to the qemu-kvm processes that perform work for a guest as "I/O threads".. :(17:45
jaypipesoops.17:45
dansmithjaypipes: nay17:45
cfriesenhow does placement deal with the difference between "shared" vcpus and "dedicated" vcpus?17:45
jaypipescfriesen: it doesn't. and it won't.17:45
mdboothAlthough my grepping suggests it's no longer misused.17:46
*** lpetrut has quit IRC17:46
mriedemmdbooth: it doesn't explain why it uses the flavor root_gb https://review.openstack.org/#/c/43782/17:46
cfriesenjaypipes: so how does it deal with a host that has X pCPUs, which lets it run X dedicated vCPUs or X*cpu_allocation_ratio shared vCPUs?17:46
jaypipescfriesen: it doesn't.17:46
cfriesenokay, so do we not track cpu usage in placement?17:47
jaypipescfriesen: we do not track cpu pinning or numa topology in placement, no.17:47
openstackgerritClaudiu Belu proposed openstack/nova master: compute: Cleans up allocations after failed resize  https://review.openstack.org/54397117:47
jaypipescfriesen: all of that is done in the highly-coupled NUMA topology stuff in nova/virt/hardware.py and the NUMATopologyFilter.17:47
mriedemmdbooth: https://bugs.launchpad.net/nova/+bug/103966217:47
openstackLaunchpad bug 1039662 in OpenStack Compute (nova) "Snapshot queries original image which could be gone" [Medium,Fix released]17:47
*** slaweq has quit IRC17:48
mriedemmdbooth: so yeah, i don't have any clue here why nova decided that if min_disk isn't set, we should set it to the flavor root_gb17:48
claudiubmriedem: if you're feeling generous with the reviews, can you take a look at the autospec stuff? :D https://review.openstack.org/#/q/topic:bug/1735588+(status:open+OR+status:merged)+project:openstack/nova17:48
jaypipescfriesen: also, allocation ratio isn't considered in any CPU pinning scenarios at all.17:48
mriedemclaudiub: i'm not that generous17:48
cfriesenjaypipes: okay, so if you use up all the pCPUs running dedicated vCPUs placement will still think there are vcpus available?17:48
jaypipescfriesen: it's an entirely different system of accounting that uses the vcpu_pin_set CONF option.17:48
mdboothmriedem: Sounds like the right place to fix it to me.17:48
*** yamamoto has joined #openstack-nova17:48
jaypipescfriesen: it was an entirely different system of accounting that was added in the hardware.py module and has ZERO to do with how any other resources are tracked in Nova.17:49
claudiubmriedem: then, if I may abuse your not that generous-ness, can you take a look at the autospec stuff? :D17:49
*** jpena is now known as jpena|off17:52
cfriesenjaypipes:  in the context of the "shared/dedicated on one host", I just added some comments.  If we're going to allow both on one host then I think it would make sense to have the usage numbers ("vcpus_used" in "nova hypervisor-show" for example) accurately reflect what's on the system.17:54
*** yamamoto has quit IRC17:54
jaypipescfriesen: yeah, I saw your comments. this is definitely something to talk about next week...17:54
jaypipescfriesen: will you be there?17:55
cfriesenyep, starting monday afternoon17:55
*** mgoddard_ has quit IRC17:55
jaypipescfriesen: also, keep in mind vcpus_used has absolutely nothing to do with pCPUs.17:55
mriedemumm wtf - flavor doesn't show up in the server response? http://paste.openstack.org/show/682314/17:55
jaypipescfriesen: ne'er the two shall meet. ;)17:55
jaypipesmriedem: lol.17:56
mriedemoh i bet i know what it is,17:56
mriedemosc can't handle microversion>=2.47 responses17:56
mriedemwhere the flavor is a dict instead of a string17:56
jaypipesmriedem: an awkward dict?17:56
cfriesenjaypipes: hypervisor-show has "vcpus" and "vcpus_used".  in our case we made it such that "vcpus" is len(vcpu_pin_set).  This makes it easy to see how "full" your hypervisor is.17:57
mriedemyup http://paste.openstack.org/show/682315/17:57
jaypipescfriesen: that is an abuse.17:57
jaypipescfriesen: and completely incorrect :)17:57
mriedemcfriesen: since you delt it, maybe you'd like to fix this https://bugs.launchpad.net/python-openstackclient/+bug/175110417:58
openstackLaunchpad bug 1751104 in python-openstackclient "openstack server show with --os-compute-api-version >= 2.47 doesn't show the flavor" [Undecided,New]17:58
cfriesenooo, fun17:59
cfriesenyah, I think I can take that.18:00
*** derekh has quit IRC18:00
mriedemmdbooth: i think removing that would be wrong, and it's likely there for non-volume-backed instances anyway (like everything in nova)18:02
mriedemmdbooth: i created a 20gb flavor instance from the devstack cirros image which has min_disk=0, and the resulting snapshot has min_disk=20 to match the flavor root_gb18:02
mdboothRight. So the 20GB restriction here should be because that's how many bits are physically required, which is represented in 'size'.18:03
mdboothIMHO min_disk should still be unset.18:03
mriedemfor the volume-backed snapshot you mean?18:04
mriedemi.e. exactly what this patch is doing?18:04
mdboothYeah.18:04
*** moshele has joined #openstack-nova18:04
mdboothThat patch is arbitrarily popping min_disk out of image metadata.18:05
mdboothI don't think it should have been in there in the first place.18:05
mdboothmin_disk is analogous to min_ram here18:05
*** sridharg has quit IRC18:07
mriedemmdbooth: i don't really agree,18:09
mriedemfor a volume-backed instance, the disk is what matters (or doesn't)18:09
mriedemif i have a 20gb flavor instance and snapshot it to get a 20gb image, then the image should say that you need a flavor of a minimum of 20gb to boot an instance with that image, right?18:10
mriedembut that's not true with a volume, since the flavor.root_gb isn't used for the volume18:10
*** ralonsoh has quit IRC18:12
dansmithmriedem: min_disk should be the size of the volume in that case when you snapshot I think18:14
dansmithflavor.root_gb if ephemeral18:14
mriedemdansmith: that's what i originally asked in the patch18:14
mriedemrather than pop off min_disk,18:14
mriedemwhy not just set it to root_bdm.size18:14
mriedemgood point though18:15
dansmithI haven't been following the discussion or the patch really, but .. yeah18:15
*** slaweq has joined #openstack-nova18:15
* dansmith is landing soon18:15
mdboothdansmith: That's size18:16
mdboothsize != min_disk18:16
mdboothSize can be expanded, min_disk is what it is18:17
mriedemanother good point, i was combining those image attrs in my head18:17
cfriesendo I need to do something special for "nova get-serial-console" to support the socat console used by ironic?18:18
*** dklyle has joined #openstack-nova18:18
mriedemcheck the ironic docs18:19
mriedemhttps://docs.openstack.org/ironic/pike/admin/console.html18:19
mriedemcfriesen: man that google search was hard...18:19
* mriedem is hangry18:20
cfriesenthe "ironic node-get-console" command is working and gives a URL, but "nova get-serial-console" returns an error.18:22
*** david-lyle has quit IRC18:22
*** yamahata has quit IRC18:22
*** slaweq has quit IRC18:25
*** lpetrut has joined #openstack-nova18:27
*** chergd has joined #openstack-nova18:27
*** tesseract has quit IRC18:28
*** moshele has quit IRC18:29
*** gbarros has joined #openstack-nova18:32
melwittI'm off to the airport o/ seeya at the PTG18:32
*** openstackgerrit has quit IRC18:33
*** AlexeyAbashkin has joined #openstack-nova18:36
mriedemlyarwood: if you're around still, i'm going to be going through this pike backport series which is pretty important, starting at https://review.openstack.org/#/c/546218/18:36
*** slaweq has joined #openstack-nova18:39
*** AlexeyAbashkin has quit IRC18:40
*** claudiub has quit IRC18:41
*** slaweq has quit IRC18:43
mriedemmnaser: wondering about some missing compute api unit tests in this pike backport https://review.openstack.org/#/c/546221/418:47
*** moshele has joined #openstack-nova18:48
efriedjaypipes: Do you want me to remove "in the Traits database" everywhere or just that one spot?18:48
efriedjaypipes: E.g. in the reno18:48
jaypipesefried: everywhere. that's just implementation detail that isn't relevant.18:49
mriedemheh, i can see ops freaking out now18:49
efriedight18:49
mriedem"nova added *another* database?!"18:49
jaypipesright.18:49
mnasermriedem: thank you for reviewing the patches meticulously.  i made sure to leave the conflicts because i don't trust my git conflict resolution stuff :<18:50
mnaserill update it now18:50
*** yamamoto has joined #openstack-nova18:50
*** openstackgerrit has joined #openstack-nova18:51
openstackgerritEric Fried proposed openstack/nova master: rp: GET /resource_providers?required=<traits>  https://review.openstack.org/54683718:51
openstackgerritEric Fried proposed openstack/nova master: Make _get_sharing_providers more efficient  https://review.openstack.org/54700818:51
efriededleafe, jaypipes: All done ^18:51
*** dtantsur is now known as dtantsur|afk18:52
edleafeefried: thx18:54
efriedMy pleasure.18:54
efriedExcept for the part where I had to delete that sqla block18:54
efriedThat was pain18:55
efriedpure pain18:55
*** yamamoto has quit IRC18:56
openstackgerritMohammed Naser proposed openstack/nova stable/pike: Clean up ports and volumes when deleting ERROR instance  https://review.openstack.org/54622118:57
openstackgerritMohammed Naser proposed openstack/nova stable/pike: Add functional test for deleting BFV server with old attach flow  https://review.openstack.org/54681218:57
openstackgerritMohammed Naser proposed openstack/nova stable/pike: Ensure attachment_id always exists for block device mapping  https://review.openstack.org/54627518:57
mnasermriedem: fixed + rebased stuff on top18:57
mriedemmnaser: i see you've mastered git review -R :)18:58
mnasermriedem: yes, got to keep my +2 with a rebase :p18:58
mnasercherry picking skills still need a bit more work though18:59
*** yamamoto has joined #openstack-nova18:59
*** BryanS68 has quit IRC19:00
*** BryanS68 has joined #openstack-nova19:00
mriedemmnaser: +2s up and down19:00
*** slaweq has joined #openstack-nova19:00
mnasermriedem: nice, bauzas mentioned he would have a look at backports when he's back on thursday :P19:01
mriedembauzas: if you're still around https://review.openstack.org/#/q/topic:bug/1750666+branch:stable/pike19:01
mnaser2 seconds apart, ha19:01
*** tbachman has quit IRC19:01
jaypipesefried: +2 from me on both. nice work.19:02
efriedThanks jaypipes19:02
* efried looks around for another core19:02
* mriedem leaves19:02
jaypipeslol19:03
* efried eyeballs dansmith. Tumbleweeds drift across the dusty road.19:03
jaypipesefried: dansmith is somewhere over the Atlantic I think.19:03
* efried closes eyes to avoid seeing chaps. He aims blindly. He fires. He misses :(19:03
jaypipeshehe19:04
*** yamamoto has quit IRC19:04
gryfdon't be afraid. just increase the frequency of the shots ;)19:04
*** bkopilov has quit IRC19:05
*** jaypipes is now known as jay-afk-219:05
*** slaweq has quit IRC19:05
*** moshele has quit IRC19:09
*** tidwellr has quit IRC19:10
*** links has quit IRC19:11
*** tidwellr has joined #openstack-nova19:11
*** moshele has joined #openstack-nova19:11
*** amoralej is now known as amoralej|off19:12
*** wolverineav has quit IRC19:12
*** wolverineav has joined #openstack-nova19:13
*** wolverineav has quit IRC19:13
*** wolverineav has joined #openstack-nova19:14
*** yamamoto has joined #openstack-nova19:14
*** wolverineav has quit IRC19:16
*** tidwellr has quit IRC19:16
*** tidwellr has joined #openstack-nova19:16
*** wolverineav has joined #openstack-nova19:17
*** gjayavelu has joined #openstack-nova19:17
*** moshele has quit IRC19:18
*** liverpooler has quit IRC19:18
*** munimeha has joined #openstack-nova19:19
*** yamamoto has quit IRC19:19
*** edleafe- has joined #openstack-nova19:24
*** tbachman has joined #openstack-nova19:25
*** edleafe has quit IRC19:25
*** edleafe- is now known as edleafe19:25
*** yamamoto has joined #openstack-nova19:29
*** tssurya has joined #openstack-nova19:30
*** moshele has joined #openstack-nova19:32
*** yamamoto has quit IRC19:34
*** tylerder_ has quit IRC19:34
*** tylerderosagrund has joined #openstack-nova19:34
*** tylerderosagrund has quit IRC19:35
*** tylerderosagrund has joined #openstack-nova19:35
*** gbarros has quit IRC19:35
*** moshele has quit IRC19:39
*** yamamoto has joined #openstack-nova19:44
*** tylerderosagrund has quit IRC19:48
mriedemcfriesen: we don't support live migration with sriov ports attached right?19:49
*** tylerderosagrund has joined #openstack-nova19:49
*** yamamoto has quit IRC19:52
mriedemi guess i'm thinking about https://review.openstack.org/#/c/286744/19:52
*** BryanS68 has quit IRC19:53
*** sidx64 has joined #openstack-nova19:54
*** sidx64 has quit IRC19:55
*** mriedem has left #openstack-nova19:57
*** mriedem has quit IRC19:57
*** mriedem has joined #openstack-nova19:57
mriedemjay-afk-2: this should be a simple spec re-approval https://review.openstack.org/#/c/543578/19:57
*** slaweq has joined #openstack-nova20:01
*** yamahata has joined #openstack-nova20:03
*** BryanS68 has joined #openstack-nova20:03
*** slaweq_ has joined #openstack-nova20:03
*** slaweq has quit IRC20:06
*** slaweq_ has quit IRC20:07
*** tylerderosagrund has quit IRC20:08
*** tylerderosagrund has joined #openstack-nova20:08
*** BryanS68 has quit IRC20:10
*** gbarros has joined #openstack-nova20:10
*** BryanS68 has joined #openstack-nova20:11
*** gbarros has quit IRC20:12
*** fragatina has quit IRC20:12
*** tylerder_ has joined #openstack-nova20:16
*** tylerderosagrund has quit IRC20:18
*** slaweq has joined #openstack-nova20:22
openstackgerritMatt Riedemann proposed openstack/nova master: Add check if neutron "binding-extended" extension is available  https://review.openstack.org/52354820:24
openstackgerritMatt Riedemann proposed openstack/nova master: WIP: Add code to bind a port against a dest host during live migration  https://review.openstack.org/52360420:24
*** slaweq_ has joined #openstack-nova20:24
*** fyxim has quit IRC20:28
*** fyx has joined #openstack-nova20:28
*** slaweq_ has quit IRC20:29
dansmithmriedem: looking at the glance docs, I'm not sure I understand what you and mdbooth said earlier while I was landing.. min_disk should be the minimum size disk you can boot it on, which would be defined by the filesystem in the image20:34
mriedemhe left a comment in that patch if that helps20:35
dansmithmriedem: the glance docs say --size is deprecated20:37
dansmithhttps://docs.openstack.org/python-openstackclient/pike/cli/command-objects/image.html20:37
dansmith"Image version 1 only."20:37
mriedemah20:38
mriedemdidn't realize that20:38
dansmithwhat I don't understand from what he said, is that if you boot something that was min_disk=10 and even size=10,20:38
dansmiththen you inflate that into a 20gb flavor and snapshot it,20:38
dansmithyour min_disk is 20g now20:38
dansmithunless there's some other meaning for that that I don't understand20:38
mriedemwtf the glance v2 API says that size is in the response, and not deprecated https://developer.openstack.org/api-ref/image/v2/index.html#show-image-details20:38
dansmithsee, that says the size is the "size of the image data"20:39
dansmithwhich may be smaller than the size of the disk you expect to have20:39
dansmithif you have something like qcow220:39
mriedemok i'm asking rosmaita if size is a thing in v220:40
mriedemor what20:40
dansmithI mean, it's not overly verbose about it I guess, but..20:40
*** awaugama has quit IRC20:45
*** tidwellr_ has joined #openstack-nova20:47
*** tidwellr has quit IRC20:47
openstackgerritMerged openstack/nova master: Extending delete_cell --force to delete instance_mappings  https://review.openstack.org/54007320:47
openstackgerritMerged openstack/nova master: Functional test: evacuate with no compute  https://review.openstack.org/49848220:47
*** yamamoto has joined #openstack-nova20:48
*** jmlowe has quit IRC20:50
*** yamamoto has quit IRC20:55
*** gjayavelu has quit IRC20:56
*** vivsoni_ has joined #openstack-nova20:57
*** vivsoni has quit IRC20:57
openstackgerritMerged openstack/nova master: Clarify `resources` query param for /r_p and /a_c  https://review.openstack.org/54706520:58
dansmithefried: what did you want to have a shootout over?21:01
efrieddansmith: Not a shootout.  I was hearkening back to the chaps conversation.  I was just looking for a review on placement API feature: https://review.openstack.org/#/c/546837/21:02
dansmithefried: jebus, don't you think you have had your fair share of core time today?21:02
* dansmith turns his assless chaps towards efried 21:03
efrieddansmith: Since you *just* approved the spec earlier, I figured you would want to get the code reviewed before it all falls out of your old, old brain.21:03
dansmithefried: oh, I see, so you're really doing me a favor, I see21:03
* efried still sucks at this "insulting dansmith to get him to do stuff" thing.21:03
efrieddansmith: Exactly.21:03
efriedYou're welcome.21:04
dansmithso really I should be thanking you21:04
efried:)21:04
* dansmith wonders how this happens21:04
cfriesenmriedem: correct, no live migration with sriov ports attached.21:05
cfriesenare we wanting to go anywhere with cold migration support for LVM?21:06
*** r-daneel has quit IRC21:06
*** r-daneel has joined #openstack-nova21:06
*** mgoddard_ has joined #openstack-nova21:07
*** slaweq_ has joined #openstack-nova21:09
*** Tom-Tom has quit IRC21:09
*** Tom-Tom has joined #openstack-nova21:09
*** eharney has quit IRC21:10
mriedemefried: i think you should have alex_xu review that21:11
mriedemgiven he wrote the traits support in the allocation candidates API21:11
mriedemhe should be back to the work grind now21:11
efriedmriedem: Sure, sounds good.21:12
mriedemcfriesen: doesn't matter to me21:12
*** r-daneel_ has joined #openstack-nova21:13
dansmithmriedem: looks fairly straightforward to me21:13
*** gbarros has joined #openstack-nova21:13
*** r-daneel has quit IRC21:13
*** r-daneel_ is now known as r-daneel21:13
*** slaweq_ has quit IRC21:14
*** Tom-Tom has quit IRC21:14
dansmiththe biggest question I'd have is over the sql stuff, but jay specifically made his mark on that bit21:14
mriedemnit in the release note, but could be cleaned up on top21:17
mriedem+121:18
dansmithefried: if you wanna update that nit I'll throw my +2 on there and we can try to get alex_xu to look during his day21:22
efrieddansmith: Roger that.  Picking nit...21:23
dansmithbut it looks pretty good to me21:23
openstackgerritEric Fried proposed openstack/nova master: rp: GET /resource_providers?required=<traits>  https://review.openstack.org/54683721:24
efrieddansmith, mriedem: Hecho ^21:24
dansmithestabladajo ^21:25
efrieddansmith: Thanks.21:26
efriedI mean, you're welcome.21:26
dansmithheh21:26
* dansmith can't believe nobody is going to call him on his spanish21:27
efriedSpanish?  That's Esperanto.21:28
*** tylerder_ has quit IRC21:29
*** tbachman has quit IRC21:30
efriedOh, shit, it actually is21:30
dansmithwhat? can't be, I just made it up21:31
efriedhttps://translate.google.com/#eo/en/establadajo21:31
dansmithdamn, I must be a sevant or something21:31
efriedNote that Esperanto was also "just made up" by people.21:31
dansmith*savant21:31
dansmithhaha21:31
efriedIt didn't come about like any other normal languages.21:31
* dansmith knows nothing about it21:32
efriedWas supposed to be a simple language that anyone could learn, and therefore become the universal lingua franca of the world.21:32
efriedBut the creators effed it up21:32
efriedLike, the first thing you should do is make sure there aren't any goofy exceptions in spelling and grammar and stuff.21:33
* dansmith wikis that pedia21:33
efriedBut they totally brought that kind of garbage in from their native languages, which I think were Portuguese and English.21:33
efriedFor that reason and/or others, it never caught on.21:33
efriedThat's my understanding anyway.21:33
* efried consults the wikipedia oracle too...21:34
*** mgoddard_ has quit IRC21:35
efriedThis article was clearly written by an advocate/evangelist, not an impartial researcher.21:35
*** acormier has quit IRC21:36
*** acormier has joined #openstack-nova21:37
dansmithheh21:37
*** pcaruana has quit IRC21:37
*** acormier has quit IRC21:41
*** jay-afk-2 is now known as jaypipes21:41
*** slaweq_ has joined #openstack-nova21:46
*** dave-mcc_ has joined #openstack-nova21:48
*** tssurya has quit IRC21:49
*** dave-mccowan has quit IRC21:49
*** slaweq_ has quit IRC21:50
*** yamamoto has joined #openstack-nova21:53
efriedOkay kids, I'm out til Dublin.  See some of y'all there!21:56
*** yamamoto has quit IRC21:57
openstackgerritMatt Riedemann proposed openstack/nova master: Add vif info to migration object  https://review.openstack.org/51542322:00
openstackgerritMatt Riedemann proposed openstack/nova master: WIP stub out changes for multi port bindings  https://review.openstack.org/52253722:00
mriedemsean-k-mooney: finally got this series all lined up and rebased, and i think the conductor changes are mostly done ^22:00
mriedemlikely need to split that into virt drivers > compute manager changes > conductor at the end22:00
mriedemsince conductor turns it all on22:00
mriedemfor the new flow22:00
*** yamamoto has joined #openstack-nova22:01
openstackgerritMatt Riedemann proposed openstack/nova master: WIP stub out changes for multi port bindings  https://review.openstack.org/52253722:03
*** r-daneel_ has joined #openstack-nova22:05
dansmithvif info on migration object huh?22:05
*** r-daneel has quit IRC22:06
*** r-daneel_ is now known as r-daneel22:06
mriedemmigratedata22:06
*** danpawlik has quit IRC22:08
dansmithah22:11
*** pchavva has quit IRC22:11
dansmithsubject says migration22:11
mriedemdansmith: if you're looking for a good time https://review.openstack.org/#/q/topic:bug/1750666+status:open+branch:stable/pike22:15
mriedemall need a final +222:15
*** tbachman has joined #openstack-nova22:15
dansmithgaahd22:15
mriedemwell sylvain ran off and never reviewed those22:15
mriedemso tag you're it22:15
*** slaweq_ has joined #openstack-nova22:18
*** tidwellr_ has quit IRC22:21
*** slaweq has quit IRC22:22
*** slaweq has joined #openstack-nova22:22
*** slaweq_ has quit IRC22:23
dansmithhmm, cinder failure on the cinder-related patch...22:25
dansmithyou checked it to confirm it's unrelated?22:25
*** slaweq has quit IRC22:27
*** lyan has quit IRC22:28
openstackgerritMerged openstack/nova-specs master: Re-propose use Neutron's new port binding API  https://review.openstack.org/54357822:29
*** itlinux has quit IRC22:30
*** esberglu has quit IRC22:32
*** tbachman has quit IRC22:33
mriedemdansmith: yeah it's a cinder api consistency group snapshot test which is nowhere related to nova http://logs.openstack.org/75/546275/4/check/tempest-full/a6efea1/job-output.txt.gz#_2018-02-22_20_19_14_91958422:33
dansmithyeah, alright22:35
*** tonyb has quit IRC22:35
*** tonyb has joined #openstack-nova22:36
*** rcernin has joined #openstack-nova22:38
*** fragatina has joined #openstack-nova22:42
*** dave-mcc_ has quit IRC22:44
*** BryanS68 has quit IRC22:44
*** lpetrut has quit IRC22:50
*** vladikr has quit IRC22:54
*** mrjk has quit IRC22:54
*** mrjk has joined #openstack-nova22:55
*** mrjk has quit IRC22:55
*** mrjk has joined #openstack-nova22:56
*** hoonetorg has quit IRC22:56
*** acormier has joined #openstack-nova22:57
*** tetsuro has joined #openstack-nova22:59
*** burt has quit IRC23:00
*** acormier has quit IRC23:01
*** yamamoto has quit IRC23:02
*** jmlowe has joined #openstack-nova23:08
*** hoonetorg has joined #openstack-nova23:11
*** tbachman has joined #openstack-nova23:26
*** itlinux has joined #openstack-nova23:29
*** efried has quit IRC23:32
*** slaweq has joined #openstack-nova23:33
*** hongbin has quit IRC23:34
*** acormier has joined #openstack-nova23:38
*** acormier has quit IRC23:38
*** acormier has joined #openstack-nova23:39
*** slaweq has quit IRC23:39
openstackgerritMatthew Edmonds proposed openstack/nova master: make PowerVM capabilities explicit  https://review.openstack.org/54716923:40
*** hshiina|afk has joined #openstack-nova23:41
*** slaweq has joined #openstack-nova23:43
*** slaweq has quit IRC23:48
openstackgerritMatthew Edmonds proposed openstack/nova master: make PowerVM capabilities explicit  https://review.openstack.org/54716923:53

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