Monday, 2017-11-20

*** yassine has joined #openstack-nova00:04
*** yassine is now known as Guest6534500:05
*** yingjun has quit IRC00:05
*** salv-orlando has quit IRC00:14
*** yingjun has joined #openstack-nova00:21
*** Apoorva has joined #openstack-nova00:24
*** hongbin has quit IRC00:31
*** lyan has joined #openstack-nova00:37
*** itlinux has quit IRC00:37
*** claudiub|2 has quit IRC00:38
*** Apoorva has quit IRC00:45
*** Apoorva has joined #openstack-nova00:46
*** Apoorva_ has joined #openstack-nova00:49
*** Apoorva has quit IRC00:52
*** hieulq has joined #openstack-nova00:54
openstackgerritTakashi NATSUME proposed openstack/nova master: [cellv2] Improve getting BDMs in multiple cells  https://review.openstack.org/52140000:56
*** itlinux has joined #openstack-nova00:56
*** phuongnh has joined #openstack-nova01:01
*** hieulq has quit IRC01:02
*** hieulq has joined #openstack-nova01:04
*** itlinux has quit IRC01:06
*** Anticimex has quit IRC01:14
*** salv-orlando has joined #openstack-nova01:15
*** Anticimex has joined #openstack-nova01:15
*** larainema has joined #openstack-nova01:17
openstackgerritMaciej Kucia proposed openstack/nova-specs master: Non-unique network names in Servers IPs API response  https://review.openstack.org/52139201:18
*** Apoorva_ has quit IRC01:24
*** gbarros has quit IRC01:25
*** itlinux has joined #openstack-nova01:30
*** Daisy_ has joined #openstack-nova01:33
*** moshele has joined #openstack-nova01:34
*** hongbin has joined #openstack-nova01:34
*** moshele has quit IRC01:39
*** lbragstad has joined #openstack-nova01:45
*** salv-orlando has quit IRC01:49
*** gcb has joined #openstack-nova01:52
*** fragatin_ has quit IRC01:53
*** moshele has joined #openstack-nova02:04
*** armax has joined #openstack-nova02:07
*** gongysh has joined #openstack-nova02:16
openstackgerritZhenyu Zheng proposed openstack/nova master: Add instance action record for attach/detach interface  https://review.openstack.org/51219002:27
*** gbarros has joined #openstack-nova02:29
*** armax has quit IRC02:42
*** salv-orlando has joined #openstack-nova02:46
*** annp has joined #openstack-nova03:01
*** fragatina has joined #openstack-nova03:02
openstackgerritJianghua Wang proposed openstack/nova master: XenAPI: create vGPU for instance  https://review.openstack.org/51689903:05
openstackgerritZhenyu Zheng proposed openstack/nova master: Add instance action record for attach/detach interface  https://review.openstack.org/51219003:16
*** liusheng has joined #openstack-nova03:17
*** moshele has quit IRC03:18
*** udesale has joined #openstack-nova03:23
*** snasir has quit IRC03:28
*** snasir has joined #openstack-nova03:28
*** akekane has joined #openstack-nova03:29
*** akekane is now known as abhishekk03:30
*** sree has joined #openstack-nova03:33
*** dave-mccowan has quit IRC03:39
openstackgerritJianghua Wang proposed openstack/nova master: vgpu: add enabled white list  https://review.openstack.org/51258003:41
openstackgerritJianghua Wang proposed openstack/nova master: XenAPI: get vGPU stats from hypervisor  https://review.openstack.org/51296503:41
openstackgerritJianghua Wang proposed openstack/nova master: XenAPI: provide vGPU inventory in compute node  https://review.openstack.org/51621703:41
openstackgerritJianghua Wang proposed openstack/nova master: XenAPI: create vGPU for instance  https://review.openstack.org/51689903:41
*** mdnadeem has joined #openstack-nova03:47
*** armax has joined #openstack-nova03:48
*** rcernin has quit IRC03:50
*** rcernin has joined #openstack-nova03:52
*** hongbin has quit IRC03:57
*** psachin has joined #openstack-nova04:03
*** ankit has joined #openstack-nova04:08
openstackgerritZhenyu Zheng proposed openstack/nova master: Add instance action record for attach/detach interface  https://review.openstack.org/51219004:09
*** fragatin_ has joined #openstack-nova04:11
*** fragatina has quit IRC04:11
*** links has joined #openstack-nova04:13
*** yamamoto has joined #openstack-nova04:21
*** yangyapeng has joined #openstack-nova04:26
*** gmann is now known as gmann_lunch04:35
*** ellioth has quit IRC04:38
*** ankit has quit IRC04:39
*** fragatina has joined #openstack-nova04:56
*** owalsh_ has joined #openstack-nova04:57
*** fragatin_ has quit IRC04:58
*** owalsh has quit IRC05:01
*** janki has joined #openstack-nova05:03
*** tetsuro has joined #openstack-nova05:07
*** trinaths has joined #openstack-nova05:13
*** gongysh has quit IRC05:15
*** suresh12 has joined #openstack-nova05:31
*** eharney has quit IRC05:35
*** rcernin has quit IRC05:38
*** diga has joined #openstack-nova05:41
*** ratailor has joined #openstack-nova05:44
*** damien_r has joined #openstack-nova05:48
*** moshele has joined #openstack-nova05:51
*** gbarros has quit IRC05:55
jianghuaw_gibi, will you help to review this patch which we mentioned in last Thursday? https://review.openstack.org/#/c/512580/05:57
*** gongysh has joined #openstack-nova06:02
*** threestrands has quit IRC06:03
*** lyan has quit IRC06:13
openstackgerritYikun Jiang proposed openstack/nova master: Add pagination and Changes-since filter support for os-migrations.  https://review.openstack.org/33040606:14
*** eharney has joined #openstack-nova06:19
*** gouthamr has quit IRC06:20
*** janki is now known as janki|afk06:22
*** lpetrut has joined #openstack-nova06:27
*** rodolof has joined #openstack-nova06:27
moshelestephenfin: hi06:30
*** suresh12 has quit IRC06:32
*** tianhui has joined #openstack-nova06:35
openstackgerritZhenyu Zheng proposed openstack/nova master: Add instance action record for attach/detach interface  https://review.openstack.org/51219006:36
*** youzp has joined #openstack-nova06:37
*** salv-orlando has quit IRC06:37
*** salv-orlando has joined #openstack-nova06:38
*** karthiks has joined #openstack-nova06:38
*** hoonetorg has quit IRC06:41
openstackgerritZhenyu Zheng proposed openstack/nova master: nova-manage db archive_deleted_rows is not multi-cell aware  https://review.openstack.org/50748606:44
openstackgerritZhenyu Zheng proposed openstack/nova master: Clean up exception caught in _validate_and_build_base_options  https://review.openstack.org/46325306:45
*** yamamoto has quit IRC06:47
*** hoonetorg has joined #openstack-nova06:54
*** gouthamr has joined #openstack-nova06:54
*** tbachman has quit IRC06:54
*** yamamoto has joined #openstack-nova06:56
*** tbachman has joined #openstack-nova06:59
*** xinliang has quit IRC07:06
*** lpetrut has quit IRC07:08
*** lpetrut has joined #openstack-nova07:08
*** gongysh has quit IRC07:11
*** lpetrut has quit IRC07:14
*** itlinux has quit IRC07:15
*** gongysh has joined #openstack-nova07:16
*** xinliang has joined #openstack-nova07:18
*** youzp has quit IRC07:21
*** youzp has joined #openstack-nova07:22
*** lpetrut has joined #openstack-nova07:22
*** lajoskatona has joined #openstack-nova07:25
*** gcb has quit IRC07:25
*** gcb has joined #openstack-nova07:26
openstackgerritTakashi NATSUME proposed openstack/nova stable/pike: Add 'delete_host' command in 'nova-manage cell_v2'  https://review.openstack.org/51371707:26
*** rodolof has quit IRC07:27
openstackgerritTakashi NATSUME proposed openstack/nova stable/pike: Add 'delete_host' command in 'nova-manage cell_v2'  https://review.openstack.org/51371707:27
*** rodolof has joined #openstack-nova07:27
*** andreas_s has joined #openstack-nova07:27
*** janki|afk is now known as janki07:35
*** nsingh has quit IRC07:35
*** nsingh has joined #openstack-nova07:35
*** abhishekk is now known as abhishekk|lunch07:37
*** belmoreira has joined #openstack-nova07:38
*** sahid has joined #openstack-nova07:43
*** TuanLA has joined #openstack-nova07:44
*** AlexeyAbashkin has joined #openstack-nova07:47
*** ankit_ has joined #openstack-nova07:49
*** youzp_ has joined #openstack-nova07:49
*** youzp_ has quit IRC07:50
*** youzp has quit IRC07:50
*** youzp has joined #openstack-nova07:50
*** lpetrut has quit IRC07:53
*** Ankit__ has joined #openstack-nova07:53
*** youzp_ has joined #openstack-nova08:03
*** pcaruana has joined #openstack-nova08:04
*** niraj_singh has joined #openstack-nova08:05
*** nsingh has quit IRC08:05
*** youzp has quit IRC08:08
*** josecastroleon has joined #openstack-nova08:08
*** youzp_ has quit IRC08:10
*** alexchadin has joined #openstack-nova08:11
openstackgerritZhenyu Zheng proposed openstack/nova master: Add instance action record for attach/detach/swap volumes  https://review.openstack.org/51720508:12
*** sshwarts has joined #openstack-nova08:13
*** alexchadin has quit IRC08:16
*** damien_r has quit IRC08:16
*** alexchadin has joined #openstack-nova08:17
openstackgerritChris Dent proposed openstack/nova master: Include project_id and user_id in AllocationList.get_all_by_consumer_id  https://review.openstack.org/51242008:18
openstackgerritChris Dent proposed openstack/nova master: [placement] Symmetric GET and PUT /allocations/{consumer_uuid}  https://review.openstack.org/51062608:18
openstackgerritChris Dent proposed openstack/nova master: [placement] POST /allocations to set allocations for >1 consumers  https://review.openstack.org/50007308:18
*** alexchadin has quit IRC08:19
*** danpawlik has quit IRC08:19
*** alexchadin has joined #openstack-nova08:19
*** Ankit__ has quit IRC08:21
*** gouthamr has quit IRC08:25
*** ircuser-1 has joined #openstack-nova08:28
*** abhishekk|lunch is now known as abhishekk08:39
*** salv-orlando has quit IRC08:42
*** gmann_lunch is now known as gmann08:42
*** salv-orlando has joined #openstack-nova08:43
openstackgerritGhanshyam Mann proposed openstack/nova master: Implement query param schema for sec group APIs  https://review.openstack.org/52135308:44
*** damien_r has joined #openstack-nova08:46
*** salv-orlando has quit IRC08:47
*** jpena|off is now known as jpena08:48
*** Daisy_ has joined #openstack-nova08:49
*** claudiub|2 has joined #openstack-nova08:53
*** rmart04 has joined #openstack-nova08:53
*** yamamoto has quit IRC08:54
moshelestephenfin: hi08:56
*** belmoreira has quit IRC08:58
*** salv-orlando has joined #openstack-nova08:58
*** owalsh_ is now known as owalsh08:58
*** spectr has joined #openstack-nova09:07
*** spectr has quit IRC09:07
*** tetsuro has quit IRC09:09
*** spectr has joined #openstack-nova09:11
*** spectr has quit IRC09:12
*** diga has quit IRC09:12
*** hoonetorg has quit IRC09:13
*** slaweq has joined #openstack-nova09:14
*** ragiman has joined #openstack-nova09:15
*** danpawlik has joined #openstack-nova09:18
*** yamamoto has joined #openstack-nova09:20
*** Daisy_ has quit IRC09:22
*** Daisy_ has joined #openstack-nova09:22
*** trinaths has quit IRC09:25
*** lucas-afk is now known as lucasagomes09:25
*** derekh has joined #openstack-nova09:26
*** Daisy_ has quit IRC09:26
*** Daisy_ has joined #openstack-nova09:27
*** qsyqian has joined #openstack-nova09:34
*** yingjun has quit IRC09:35
*** Daisy_ has quit IRC09:35
*** yamamoto has quit IRC09:36
*** gmann is now known as gmann_afk09:37
*** yamamoto has joined #openstack-nova09:37
*** sahid has quit IRC09:42
yankcrimewebdev09:42
*** sahid has joined #openstack-nova09:42
*** yamamoto has quit IRC09:45
*** sahid has quit IRC09:45
*** sahid has joined #openstack-nova09:46
*** slaweq has quit IRC09:48
*** lpetrut has joined #openstack-nova09:50
*** belmoreira has joined #openstack-nova09:52
openstackgerritChris Dent proposed openstack/nova master: [placement] Symmetric GET and PUT /allocations/{consumer_uuid}  https://review.openstack.org/51062609:53
openstackgerritChris Dent proposed openstack/nova master: [placement] POST /allocations to set allocations for >1 consumers  https://review.openstack.org/50007309:53
*** slaweq has joined #openstack-nova09:54
openstackgerritjichenjc proposed openstack/nova master: WIP: remove reserve_quota_delta  https://review.openstack.org/52146909:54
openstackgerritjichenjc proposed openstack/nova master: WIP: Remove quota reserve/commit/rollback  https://review.openstack.org/52147009:54
*** Daisy_ has joined #openstack-nova09:54
*** lpetrut has quit IRC09:56
*** Daisy_ has quit IRC09:59
*** annp has quit IRC10:03
*** cdent has joined #openstack-nova10:08
openstackgerritTakashi NATSUME proposed openstack/nova master: [Placement] Fix parameter order in placement API ref  https://review.openstack.org/51623310:09
openstackgerritTakashi NATSUME proposed openstack/nova master: [placement] Fix parameter order in placement API ref  https://review.openstack.org/51623310:10
*** andreas_s has quit IRC10:10
*** andreas_s has joined #openstack-nova10:11
*** trinaths has joined #openstack-nova10:12
*** lpetrut has joined #openstack-nova10:13
*** slaweq has quit IRC10:14
*** purplerbot has joined #openstack-nova10:15
*** slaweq has joined #openstack-nova10:15
*** andreas_s has quit IRC10:15
*** yamamoto has joined #openstack-nova10:17
openstackgerritZhenyu Zheng proposed openstack/nova master: Add instance action record for attach/detach/swap volumes  https://review.openstack.org/51720510:17
*** slaweq has quit IRC10:20
*** andreas_s has joined #openstack-nova10:21
*** tianhui has quit IRC10:21
*** TuanLA has quit IRC10:23
*** slaweq has joined #openstack-nova10:24
*** diga has joined #openstack-nova10:25
*** dtantsur|afk is now known as dtantsur10:29
*** jaosorior has joined #openstack-nova10:30
*** sree has quit IRC10:32
*** gongysh has quit IRC10:33
*** sree has joined #openstack-nova10:33
openstackgerritTakashi NATSUME proposed openstack/nova master: [placement] Fix format in placement API ref  https://review.openstack.org/52148310:33
*** andreas_s has quit IRC10:34
*** sree has quit IRC10:37
*** andreas_s has joined #openstack-nova10:39
*** andreas_s has quit IRC10:43
*** andreas_s has joined #openstack-nova10:43
*** ratailor has quit IRC10:51
*** abhishekk has quit IRC10:53
openstackgerritMaciej Kucia proposed openstack/nova-specs master: Non-unique network names in Servers IPs API response  https://review.openstack.org/52139210:54
*** jpena is now known as jpena|off10:57
*** jpena|off is now known as jpena10:58
*** Guest65345 has quit IRC10:59
*** kmalloc has joined #openstack-nova11:01
*** szaher has quit IRC11:02
*** gszasz has joined #openstack-nova11:05
*** diga has quit IRC11:09
openstackgerritYikun Jiang proposed openstack/nova master: [WIP] Add pagination and changes-since for instance-actions  https://review.openstack.org/32632611:11
*** moshele has quit IRC11:13
openstackgerritYikun Jiang proposed openstack/nova master: Add pagination and changes-since for instance-actions  https://review.openstack.org/32632611:13
*** andreas_s has quit IRC11:17
*** andreas_s has joined #openstack-nova11:18
*** dtantsur is now known as dtantsur|bbl11:21
*** belmoreira has quit IRC11:21
*** phuongnh has quit IRC11:22
*** andreas_s has quit IRC11:22
*** sean-k-mooney has quit IRC11:22
*** gcb has quit IRC11:24
*** alexchadin has quit IRC11:28
openstackgerritTakashi NATSUME proposed openstack/nova master: [placemant] Add 'CUSTOM_' prefix description in API ref  https://review.openstack.org/52149511:31
openstackgerritTakashi NATSUME proposed openstack/nova master: [placement] Add 'CUSTOM_' prefix description in API ref  https://review.openstack.org/52149511:32
*** openstackgerrit has quit IRC11:32
*** tojuvone has quit IRC11:35
*** trinaths has left #openstack-nova11:36
*** andreas_s has joined #openstack-nova11:37
*** slaweq has quit IRC11:39
*** andreas_s has quit IRC11:45
*** yingjun has joined #openstack-nova11:46
*** andreas_s has joined #openstack-nova11:46
*** slaweq has joined #openstack-nova11:49
*** szaher has joined #openstack-nova11:51
*** andreas_s has quit IRC11:51
*** qsyqian has quit IRC11:52
*** gongysh has joined #openstack-nova11:52
*** slaweq has quit IRC11:53
*** ratailor has joined #openstack-nova11:55
*** udesale has quit IRC11:57
*** moshele has joined #openstack-nova12:00
*** andreas_s has joined #openstack-nova12:02
*** belmoreira has joined #openstack-nova12:06
efried_zzzcdent Greetings!  How was your vacation time Down Under?12:08
*** lucasagomes is now known as lucas-hungry12:08
*** raorn_ is now known as raorn12:08
*** openstackgerrit has joined #openstack-nova12:08
openstackgerritTakashi NATSUME proposed openstack/nova master: [placement] Add aggregate link note in API ref  https://review.openstack.org/52150212:08
cdenthola efried_zzz : too short but otherwise very nice, I can link to some pics if you like12:09
*** efried_zzz is now known as efried12:09
efriedcdent Sure!12:09
cdenthttps://t.co/BWtRih7zPU <- expands to a google pics link12:09
efriedcdent Wow, looks like the weather just sucked.12:11
cdentyeah, was dreadful12:12
cdentmid 20s and sunny, all the time. couldn’t be worse.12:12
efriedcdent I know how to get to the building docs - but I haven't been able to find the placement API ref from there.12:14
efriedcdent The links on the main page point to the real published docs.12:14
cdentif you click on the right test, it goes straight to the placement docs: I named the test in my response, but can’t remember now12:16
cdentor do you mean links somewhere else?12:16
efriedcdent ooohhh, crap, that's right, placement docs are a different build job.12:17
efriedI didn't get that far down - I just hit the first sphinx doc build job and was like, where tf is the placement ref??12:18
*** yassine has joined #openstack-nova12:18
*** yassine is now known as Guest1891712:18
cdentIt’s all part of this glorious future where placement is separate12:19
stephenfinmoshele: Hey12:20
stephenfinWhat's up?12:20
*** Eran_Kuris has quit IRC12:20
moshelestephenfin: regarding the revert https://review.openstack.org/#/c/518833/12:22
stephenfinYup?12:23
moshelestephenfin: so I have a fix https://review.openstack.org/#/c/519066/12:23
stephenfinmoshele: Excellent. I'll take a look at that today :)12:23
moshelestephenfin: but I didn't find the guy to say if it not breaking heat12:23
efriedcdent Am I missing the part of the series where GET /allocation_candidates returns the new format expected by PUT /allocations ?12:23
stephenfinmoshele: Hmm, that probably wouldn't be a bad idea. ramishra was the nick. I wonder if he's on #heat?12:24
* stephenfin has been gone for the last week and is literally back online for an hour. Catching up12:25
moshelestephenfin: I was looking for him in openstack-heat :)12:25
stephenfinAhh, yeah. Look at the welcome message ;)12:25
moshelestephenfin: feel free to review the patch I should solve the problem12:27
mosheles/I/ti/12:27
stephenfinWill do (y)12:27
*** sdague has joined #openstack-nova12:28
*** janki has quit IRC12:29
*** udesale has joined #openstack-nova12:29
cdentefried: line 286ish https://review.openstack.org/#/c/510626/14/nova/api/openstack/placement/handlers/allocation.py12:29
cdentoh sorry, that’s the put you want to know the get12:30
cdentit’s in the _allocations_dict method12:30
cdentGET was already nearly correct, it just needed the project_id and user_id added12:31
cdentGET being different was why PUT was changed to be more like GET12:31
*** links has quit IRC12:31
efriedcdent I'm talking about GET /allocation*_candidates*12:31
cdentah, sorry, names are hard on a monday12:32
efriedcdent The latest changes to the spec were that the resource request bit of the GET /allocation_candidates response was going to be changed to match what PUT /allocations expects so that you can still just turn it around and PUT it without looking at it (and converting it)12:33
*** gongysh has quit IRC12:33
*** sree has joined #openstack-nova12:34
*** Eran_Kuris has joined #openstack-nova12:34
cdentoh, yeah, I think that’s just not done yet, as that was decided right before I started travelling12:34
cdentso I’ll stick a -W on there somewhere12:34
openstackgerritAndrey Kurilin proposed openstack/python-novaclient master: [functional] Remove duplication of boot helper  https://review.openstack.org/52107112:35
openstackgerritAndrey Kurilin proposed openstack/python-novaclient master: [ci] Use pseudo-random names for new resources  https://review.openstack.org/47977712:35
efriedcdent Okay, cool.  Didn't want to embarrass my self by +1ing that.  Dodged a bullet, phew!12:35
cdentMy memory is starting to surface: I updated the spec and volunteered you or jay to do the work, but then you and jay fell in the refactoring hole related to resources etc and so now it is back to me.12:36
cdentOr something12:36
efriedcdent Yes, that's the long and short of it.  Though refactoring isn't the *only* thing that happened - see update email 41.75 :)12:37
cdentthat’s the “etc” part12:37
cdentspeaking of: thanks for doing those. you can carry on doing it if you’re feeling keen, or I’m happy to take it back if not12:37
*** sree has quit IRC12:38
*** yamamoto has quit IRC12:39
efriedcdent I'd be fine either way.  I've been happily slinging code for virt drivers getting hands on ProviderTree, but that stuff (as well as granular resource request work) is high atop other series work, so not urgent.  Whereas symmetric stuff is pretty close to done.12:40
efriedcdent So if you're buried under other stuff, I can take it on for the sake of expediency.  But if you've got the bandwidth to finish it, you should.12:40
cdentthe “those” I meant was the weekly updates12:41
efriedHah.  Oh.12:41
efriedYou're welcome - and yes, please take them back.12:41
cdentroger that. and I’ll take back the symmetric stuff12:41
efriedcdent I made sure to leave you #42 :)12:41
cdentso kind12:41
efriedcdent And speaking of taking stuff over - you feel like running the scheduler meeting today?12:42
efriededleafe- is out12:42
*** andreas_s has quit IRC12:44
cdentoh is he now12:44
cdentyeah, I can do it12:44
*** andreas_s has joined #openstack-nova12:44
efriedcdent Cool12:46
efriedcdent Being an American in the UK, are you going to be out later this week or no?12:46
cdentjust to check my calendar it is scheduled for a 1.25 hours from now?12:46
cdentno12:46
cdentUsually I take holiday back to the US to see family, but with sydney just now I couldn’t stand the idea of yet more international travel12:47
efriedcdent Okay.  Yes, scheduler meeting is 1.25 hours from now if my UTC clock is sane.  (DST is stupid)12:47
efriedcdent FYI I'll be out W-F12:48
* cdent makes plans to change _everything_ on thursday12:48
*** andreas_s has quit IRC12:49
*** jpena is now known as jpena|lunch12:50
*** takedakn has joined #openstack-nova12:50
*** sree has joined #openstack-nova12:54
*** sree_ has joined #openstack-nova12:55
*** sree_ is now known as Guest7269612:56
*** andreas_s has joined #openstack-nova12:56
*** josecastroleon has quit IRC12:59
*** sree has quit IRC12:59
*** salv-orlando has quit IRC12:59
*** gcb has joined #openstack-nova12:59
*** salv-orlando has joined #openstack-nova13:00
*** andreas_s has quit IRC13:00
*** slaweq has joined #openstack-nova13:01
*** yamamoto has joined #openstack-nova13:01
*** salv-orlando has quit IRC13:04
*** andreas_s has joined #openstack-nova13:06
*** ijw has joined #openstack-nova13:07
*** andreas_s has quit IRC13:11
*** alexchadin has joined #openstack-nova13:13
*** yamamoto has quit IRC13:13
*** ijw_ has joined #openstack-nova13:14
*** lucas-hungry is now known as lucasagomes13:15
*** josecastroleon has joined #openstack-nova13:16
*** links has joined #openstack-nova13:17
*** ijw has quit IRC13:17
*** andreas_s has joined #openstack-nova13:18
*** fragatina has quit IRC13:20
*** andreas_s has quit IRC13:22
*** andreas_s has joined #openstack-nova13:22
*** liverpooler has joined #openstack-nova13:22
*** ijw has joined #openstack-nova13:23
*** andreas_s has quit IRC13:23
*** markvoelker has quit IRC13:23
*** markvoelker has joined #openstack-nova13:24
*** andreas_s has joined #openstack-nova13:24
*** ijw_ has quit IRC13:26
niraj_singhefried: i was going through ( https://review.openstack.org/#/c/512329/ ) this patch. A new exception is cretaed i.e. "ServiceUserTokenNoAuth". But i didn't find where this exception is handled.13:27
*** andreas_s has quit IRC13:27
*** andreas_s has joined #openstack-nova13:27
*** Eran_Kuris has quit IRC13:28
*** damien_r1 has joined #openstack-nova13:28
*** damien_r has quit IRC13:29
*** andreas_s has quit IRC13:29
*** maciejjo1 has joined #openstack-nova13:29
*** dpawlik has joined #openstack-nova13:29
*** andreas_s has joined #openstack-nova13:29
*** andreas_s has quit IRC13:30
*** andreas_s has joined #openstack-nova13:30
*** danpawlik has quit IRC13:30
*** maciejjozefczyk has quit IRC13:30
*** takedakn has quit IRC13:31
openstackgerritTakashi NATSUME proposed openstack/nova stable/pike: Add 'delete_host' command in 'nova-manage cell_v2'  https://review.openstack.org/51371713:36
*** dave-mccowan has joined #openstack-nova13:36
efriedniraj_singh One sec.13:36
efriedniraj_singh Okay - what do you mean "handled"?13:37
efriedniraj_singh The motivation behind this patch was to make an exception that was easier for the user to understand.13:38
efriedniraj_singh So it's not actually supposed to be handled anywhere - it bubbles up to the user.13:38
efriedniraj_singh Sorry, by which I mean, it ends up in the logs where the operator can see it.13:40
*** slaweq has quit IRC13:41
*** ijw has quit IRC13:41
*** udesale has quit IRC13:42
*** Eran_Kuris has joined #openstack-nova13:43
*** Daisy_ has joined #openstack-nova13:43
*** maciejjo1 is now known as maciejjozefczyk13:44
*** slaweq has joined #openstack-nova13:45
*** dave-mcc_ has joined #openstack-nova13:46
stephenfinmoshele: Done13:46
stephenfinEven though I suggested it, I'm not sure whether we should log an exception message without raising it. I'll let someone else comment on this13:47
*** dave-mccowan has quit IRC13:48
*** Daisy_ has quit IRC13:48
niraj_singhefried: I thought it should handle any where. ok thanks for information.13:48
*** Daisy_ has joined #openstack-nova13:50
*** vladikr has joined #openstack-nova13:51
openstackgerritpriyaduggirala proposed openstack/nova master: Rename parameters in call() of nova/image/glance.py  https://review.openstack.org/50853313:54
*** jpena|lunch is now known as jpena13:54
*** sean-k-mooney has joined #openstack-nova13:55
*** Daisy_ has quit IRC13:55
*** josecastroleon has quit IRC13:56
efriedScheduler meeting in three minutes in #openstack-meeting-alt13:57
cdentwhoa13:57
efriedDid I just read your mind?13:57
cdentI was, like, literally, typing, literally, like, the same, like thing13:58
efriedGreat minds, and all that.13:58
*** Guest72696 has quit IRC13:58
cdentwe are merely expressions of the platonic nova scheduler dev13:58
*** lyan has joined #openstack-nova13:58
openstackgerritTakashi NATSUME proposed openstack/nova master: [placement] Add 'Location' parameters in API ref  https://review.openstack.org/52154113:59
*** sree has joined #openstack-nova13:59
*** yangyapeng has quit IRC13:59
*** salv-orlando has joined #openstack-nova13:59
efriedI had an almost-mother-in-law when I was a youngster who would always know the second half of pithy aphorisms, which was slightly annoying.  In this case, "Great minds think alike" apparently concludes, "And fools seldom differ"14:00
*** eharney has quit IRC14:00
*** rodolof has quit IRC14:01
stephenfinsean-k-mooney: I'm +2 on this if you want to take a second look https://review.openstack.org/#/c/493082/14:02
*** sree has quit IRC14:03
stephenfinefried: nested resource providers are on my review radar for this week14:03
stephenfin(RE: the review request you sent at some point last week :))14:03
efriedstephenfin That's great to hear.  Can you join the sched meeting in -alt ?14:04
stephenfinNow?14:04
efriednow14:04
* stephenfin wishes we had a CalDav server set up so I wouldn't have to mess around with iCal files14:05
cdent+114:05
*** Eran_Kuris has quit IRC14:06
*** pchavva has joined #openstack-nova14:08
*** slaweq has quit IRC14:09
sean-k-mooneystephenfin: just taking a look now14:13
sean-k-mooneyah the printable mixin14:13
*** mriedem has joined #openstack-nova14:14
*** josecastroleon has joined #openstack-nova14:14
stephenfinbauzas: This one's another easy +W, IMO https://review.openstack.org/#/c/466051/14:14
*** yamamoto has joined #openstack-nova14:14
bauzasstephenfin: ack14:15
*** liverpoo1er has joined #openstack-nova14:16
*** liverpoo1er has quit IRC14:16
stephenfincdent: Seen this? https://review.openstack.org/#/c/511936/ Are we expecting to have a python-placementclient library?14:16
*** andreas_s has quit IRC14:17
cdentthere’s going to be an osc plugin, but no plans for a specific client14:17
*** shvepsy_ has joined #openstack-nova14:17
cdentin large part because it’s “just json”14:17
*** andreas_s has joined #openstack-nova14:18
cdentstephenfin: https://review.openstack.org/#/q/topic:bp/placement-osc-plugin+(status:open+OR+status:merged)14:18
*** bkopilov has quit IRC14:19
*** shvepsy has quit IRC14:19
efriedcdent I would be in favor of a client lib that performs useful API calls, like SchedulerReportClient does, and parlays "just json" into useful python objects, like ProviderTree.14:19
*** Eran_Kuris has joined #openstack-nova14:19
*** yamamoto has quit IRC14:20
cdentI was really hoping we’d keep ProviderTree hidden in the bowels of nova and not make that a thing that people commonly want to use...14:20
*** yamamoto has joined #openstack-nova14:20
cdentBut if that’s not the case, then sure, I guess. The deal simply was that until multiple implementations with commonanilities showed up, why bother?14:20
efriedIt doesn't have to be ProviderTree itself, but in that vein.14:21
openstackgerritMerged openstack/os-vif master: Add VersionedObjectPrintable mixin  https://review.openstack.org/49308214:21
efriedcdent Oh, I can answer that one: so that we don't have multiple implementations with commonalities show up, which we then have to reconcile and consolidate.14:21
cdentsigh14:22
*** bkopilov has joined #openstack-nova14:22
cdentreconciliation and consilidation is a _good_ thing. it allows for compare and contrast amongst a variety of thoughts14:22
cdentrather than imposing a single vision14:22
efriedBut then you wind up with things you can't get rid of because momentum.14:23
cdentThe way nova chooses to do things internal to the report client isn’t necessarily right for the rest of the world and vice versa14:23
cdentand if you can’t just use requests for the API when we may as well give up and go home because we’ve failed14:23
* cdent is currently idealized world mode and will get over it soon14:24
efriedcdent This is interesting in light of some of the work I did last week.  Let's chat after the meeting.14:24
*** yangyapeng has joined #openstack-nova14:24
* cdent has to leave after the meeting...14:24
cdent(or at least by 3)14:25
efriedcdent Okay, well, if you get a chance, have a look at this: https://review.openstack.org/#/c/521098/4/nova/scheduler/client/report.py14:25
*** toure has joined #openstack-nova14:25
efriedA non-trivial method to collate all the information about a group of providers.  Call it ProviderTree or whatever, IMO it's conceivably a useful thing to be able to do for any client.14:26
cdentIn idealized world mode, the fact that we need a provider tree (or whatever) is a reason to go home14:27
cdentso now isn’t the best time for me to think about that...14:27
cdentif we make it a habit that anything beyond a compute-node/hypervisor needs to know that stuff all the puppies will cry14:27
*** yangyapeng has quit IRC14:29
sean-k-mooneystephenfin: can you take a look at this https://review.openstack.org/#/c/520788/114:30
*** andreas_s has quit IRC14:31
*** gouthamr has joined #openstack-nova14:32
openstackgerritAndrey Kurilin proposed openstack/python-novaclient master: [ci] Use pseudo-random names for new resources  https://review.openstack.org/47977714:32
*** josecastroleon has quit IRC14:32
*** tssurya has quit IRC14:32
*** alexchadin has quit IRC14:32
stephenfinsean-k-mooney: I agree with AJaeger there: Sphinx doesn't require you to set the version number and it wouldn't make semantic sense for release notes anyway14:33
sean-k-mooneystephenfin: well i was just about to respond mail thread14:33
stephenfinsean-k-mooney: pbr also overrides the project name and version if built with 'build_sphinx'. I'm not sure if we're using that here but it's probably worth mentioning14:34
stephenfinCool. I'll hold off on merging so14:34
sean-k-mooneystephenfin: i why are release nots not considerd to be versiond. they are written from the perspective of the release in which they are added14:34
*** belmoreira has quit IRC14:34
sean-k-mooneyi do agree that ideally we should not need to install the projects14:34
stephenfinsean-k-mooney: Because of how they're distributed. At the moment, we don't distribute them at all14:35
sean-k-mooneybut i think we have bigger issue if i cant look at just the ocata release nots without seeing the pike release nots if i am upgrading to ocata14:35
stephenfinWe "distribute" them only online, via docs.o.o (or is it release.o.o?)14:36
*** READ10 has joined #openstack-nova14:36
*** andreas_s has joined #openstack-nova14:39
*** jamesdenton has joined #openstack-nova14:39
stephenfinsean-k-mooney: As for locally, distribution is via an sdist but you can't currently build release notes from this because you don't have the metadata necessary14:39
stephenfin(because it's not a git repo)14:39
stephenfinreno can export this metadata to a file, but we haven't done this historically. This is changing in the next MAJOR version of pbr, thankfully https://review.openstack.org/#/c/475034/14:40
stephenfinsean-k-mooney: Also, RE: " i think we have bigger issue if i cant look at just the ocata release nots without seeing the pike release nots if i am upgrading to ocata"14:40
stephenfinIsn't this what you want? https://docs.openstack.org/releasenotes/nova/ocata.html14:40
sean-k-mooneystephenfin: yes :) it is. the commit message as written however implies that there is no versioning of release notes at all.14:42
stephenfinsean-k-mooney: Ahh, no no14:42
*** yangyapeng has joined #openstack-nova14:42
stephenfinWhat he's saying is that there'll be no version number provided to Sphinx, because it's no necessary14:43
sean-k-mooneythat fine then ill remove my -1  i was fine with the change as long as we could still view it per release14:43
*** burt has joined #openstack-nova14:43
stephenfinActually, sorry, what he _really_ meant was that version notes are versioned like everything else. E.g. we just have '/releasenotes/{project}' instead of '{series}/releasenotes/{project}'14:44
stephenfinEither way, you have what you want :)14:44
*** mdnadeem has quit IRC14:44
stephenfinbauzas: albeit...?14:45
*** csatari_ has joined #openstack-nova14:45
sean-k-mooneystephenfin: see this is why i ping you on docs changes so i can learn somthing new :)14:45
bauzasstephenfin: oh dammit14:46
bauzasalbeit = although14:46
*** yangyapeng has quit IRC14:46
bauzasI thought it was something like "all be it" which sounds to me like "ok, fine, accepted."14:47
stephenfinsean-k-mooney: Anytime, buddy :D14:47
*** andreas_s has quit IRC14:48
*** tssurya has joined #openstack-nova14:48
stephenfinbauzas: Not quite, heh. It's pronounced al-bait (like, bait a fish)14:48
*** andreas_s has joined #openstack-nova14:48
* bauzas shouts indistinctively14:48
dansmithstephenfin: here it's pronounced "all be it"14:49
stephenfindansmith: Please, haven't you ruined our language enough?14:50
dansmithstephenfin: yeah, sorry about that14:50
sean-k-mooney:)14:50
*** dtantsur|bbl is now known as dtantsur14:51
openstackgerritElod Illes proposed openstack/nova master: Functional test: cold migrate to compute down  https://review.openstack.org/49628014:52
*** andreas_s has quit IRC14:53
*** andreas_s has joined #openstack-nova14:53
dansmithmriedem: I responded to your comments on that patch14:53
dansmithmriedem: I have an errand to run in about 30 minutes and then can push up another rev14:54
*** bkopilov has quit IRC14:54
*** eharney has joined #openstack-nova14:55
mriedemdansmith: ack14:55
mriedemefried: if you want an easy docs improvement, we should put something in the top of the placement api-ref about the microversion header to use for placement https://developer.openstack.org/api-ref/placement/#placement-api14:56
*** csatari_ has quit IRC14:56
efriedbauzas French has the elegant 'soit'.  If that's what you're going for, in English, "So be it" is probably closest.14:56
bauzasefried: ah you're right14:57
openstackgerritMerged openstack/os-vif master: Remove setting of version/release from releasenotes  https://review.openstack.org/52078814:57
efriedmriedem Noted, I'll look into that.14:57
efriedbauzas If you want something more imperative, that Star Trek fans will appreciate, use "Make it so" (optionally addressing your audience as "number one")14:58
*** gbarros has joined #openstack-nova14:59
dansmithheh14:59
*** lokioden has quit IRC15:00
bauzasefried: I personnally use "dont acte", in case it rings you a bell15:00
*** udesale has joined #openstack-nova15:00
*** csatari___ has joined #openstack-nova15:00
*** yamahata has joined #openstack-nova15:01
*** sree has joined #openstack-nova15:01
*** links has quit IRC15:01
efriedbauzas TIL.  Celui-çi https://fr.wiktionary.org/wiki/dont_acte ?15:02
*** takashin has left #openstack-nova15:03
mriedemstephenfin: for the PciPassthroughFilter,15:04
*** yangyapeng has joined #openstack-nova15:04
mriedemdo the pci requests only come through flavor extra specs?15:04
mriedemor can they also come through image metadata?15:04
*** csatari___ has quit IRC15:04
bauzasefried: yup15:05
bauzasacknowledged sounds the most appropriate15:05
bauzasbut it sounds less harsh in English than it sounds in French15:05
bauzasanyway15:05
bauzaswe should open a new chan15:05
efried #openstack-linguistic-nerdiness15:06
*** dave-mcc_ is now known as dave-mccowan15:06
bauzasefried: I was more thinking of a general #openstack-schmooze15:07
*** mlavalle has joined #openstack-nova15:07
efriedbauzas Ironically, if you say "dont acte", it sounds like you're saying "don't act" in English, which is almost the opposite.15:07
*** awaugama has joined #openstack-nova15:07
*** sree has quit IRC15:08
*** gcb has quit IRC15:08
*** yangyapeng has quit IRC15:08
*** sree has joined #openstack-nova15:08
efried(By the way, you should not google "don't act")15:09
bauzasefried: "eventually" is the most interesting false friend I know of15:10
*** hongbin has joined #openstack-nova15:11
bauzasefried: 'éventuellement' means "maybe" in our language15:11
efriedbauzas How is that different from "peut-être"?15:12
bauzasthat's a synonym15:12
efriedbauzas Google translate doesn't agree, which I'm sure leads to lots of humorous exchanges: https://translate.google.com/#fr/en/%C3%A9ventuellement15:13
*** links has joined #openstack-nova15:13
efriedbauzas In Spanish/Porguguese, "indroducir" means "insert", not "introduce".  So people wind up saying, "Allow me to insert myself".15:14
*** jaypipes has joined #openstack-nova15:15
stephenfinmriedem: iirc, it's only for extra specs15:15
* stephenfin checks15:15
efriedstephenfin At least according to the docs, there's no affordance for PCI requests via image metadata.  I'm like 80% sure.15:15
*** aloga has quit IRC15:15
*** aloga has joined #openstack-nova15:16
stephenfinmriedem, efried: https://github.com/openstack/nova/blob/6535abdaaa73d92ebe9101626cc3f5b11117a194/nova/compute/api.py#L825-L83415:16
sean-k-mooneyyou can request pci device using the alias in the flavor our via the vnic type on a neutorn port in the case of sriov nics15:17
efriedstephenfin sean-k-mooney Yup, that all gels with what I believed to be true.15:17
efried(phew)15:17
*** udesale has quit IRC15:18
openstackgerritMerged openstack/nova master: Fix docstring for GET /os-migrations and related DB API  https://review.openstack.org/52020015:18
openstackgerritMerged openstack/nova master: api-ref: make a note about os:scheduler_hints being a top-level key  https://review.openstack.org/51971715:18
*** moshele has quit IRC15:18
sean-k-mooneyefried: i do belive that some peole wante to be able to restrict the type of sriov device that could be passthough via the image metadata at some point but i dont think that ever went anywhere15:19
efriedGood to know.15:19
sean-k-mooneyefried: e.g. i only have intel nic driver so when i state that in image metatdata via vendor ids and product ids and then request an sriov nic in neutron and nova would make sure it iwas an intel one15:20
efriedsean-k-mooney But that's now done via the whitelist, nah?15:20
*** marst has joined #openstack-nova15:20
sean-k-mooneyefried: no they are two different things15:20
sean-k-mooneythe whitelist is the operators way to say this is what i will allow to be passed through . in the neutron sriov case it will be resticted to that list, but this was a way to allow tenants to say i only budeled x drives in my image so limit to those vedors15:22
efriedsean-k-mooney That makes sense.  But you say that never got implemented?15:22
openstackgerritAndreas Jaeger proposed openstack/nova master: DNM - test updated releasenotes job  https://review.openstack.org/52067715:23
sean-k-mooneycorrect there was also a counter proposal for it to be supported as an atribute on the neutron port when you set vnic type direct via the port binding profile but neither were impleneted in the end15:23
cfriesenmriedem: what's your plan for https://review.openstack.org/#/c/521200/  vs https://review.openstack.org/#/c/520660 ?   Are you looking to break it up into a number of smaller commits?15:24
*** yangyapeng has joined #openstack-nova15:24
mriedemcfriesen: do you mean because of this? https://review.openstack.org/#/c/520660/5/nova/compute/api.py@295415:26
*** links has quit IRC15:27
mriedemcfriesen: as i see it, https://review.openstack.org/#/c/520660/ is really fixing 2 things, (1) rebuilding a volume-backed server changes the instance.image_ref from '' to $image_id, and (2) rebuilding a volume-backed server with a different image from what's in the root volume doesn't fail in the api15:27
mriedemthinking about backports, i think backporting the fix for (1) is an easier case than (2)15:28
cfriesenmriedem: whoops, I meant https://review.openstack.org/#/c/521391....which seems to be the same as 52066015:28
mriedemit's not the same15:28
*** yangyapeng has quit IRC15:28
mriedemhttps://review.openstack.org/#/c/521391/ fixes the bug where any volume-backed server, regardless of the image being different or not, sends the instance through the scheduler15:29
mriedemfor rebuidl15:29
mriedem*rebuild15:29
mriedemw/o introducing the change that changing the image makes the api fail15:29
mriedemit's a subtle difference15:30
cfriesenmriedem: right...so are you basically proposing breaking up 520660 into a number of commits?15:30
mriedemi already broke 520660 into 2 changes, but 520660 could also be broken into 2 more changes if we wanted15:31
mriedemwhere the final change in the series is the new behavior to fail fast if the image is changing on a volume-backed instance15:31
efriedThere goes mriedem, breaking stuff again.15:31
alex_xuefried: which bug you want me help to debug? I see there is a mention in the scheduler meeting15:32
efriedalex_xu The crossing-aggregates thing15:32
alex_xuefried: ah, i see, there are some testcase you/gibi added15:34
efriedalex_xu Yes, all part of the pile :)15:34
cfriesenmriedem: the thing that's confusing me is that 521391 is still included in 52066015:34
alex_xuefried: I will try, but I'm in the travel from tomorrow.15:34
cfriesenoh, wait, I see....you're just adding the else clause15:35
efriedalex_xu No worries.  It's probably something jaypipes should be looking at too/instead.15:35
*** artom has joined #openstack-nova15:36
jaypipeswait, I thought the scheduler meeting was in 25 minutes? :( daylight savings and all that...15:36
efriedjaypipes Wrong direction.  It was 1.5h ago.15:37
efriedDST is stupid.15:37
jaypipesoh ffs... sorry :(15:37
efriedjaypipes http://eavesdrop.openstack.org/meetings/nova_scheduler/2017/nova_scheduler.2017-11-20-14.00.log.html15:37
jaypipesdanke15:37
openstackgerritElod Illes proposed openstack/nova master: Deduplicate keypair notification samples  https://review.openstack.org/52156215:38
efriedjaypipes Some interesting discussion at the end...15:38
openstackgerritEric Fried proposed openstack/nova master: placement: Document microversion header in api-ref  https://review.openstack.org/52156315:40
efriedmriedem Like-a-that? ^15:40
*** yingjun has quit IRC15:40
alex_xuefried: yea15:40
mriedemefried: no15:41
mriedemin https://developer.openstack.org/api-ref/placement/#placement-api15:41
efriedmriedem As in, the section above the section titled "Versions" ?15:42
mriedemyes15:43
mriedemcommented inline15:43
*** artom has quit IRC15:43
*** artom has joined #openstack-nova15:44
efriedmriedem Ack15:44
maciejjozefczykjaypipes: Could you please take a look on https://review.openstack.org/#/c/520024/15:47
*** artom has quit IRC15:47
maciejjozefczykjaypipes: :) ?15:47
*** edmondsw has joined #openstack-nova15:47
jaypipesmaciejjozefczyk: you see the func test failure in that? is it related to your changes?15:48
openstackgerritayoung proposed openstack/nova master: Admin API Policy contingent on is_admin_project  https://review.openstack.org/38414815:48
maciejjozefczykjaypipes: yes it fails and my questions are about how to fix it15:49
jaypipesmaciejjozefczyk: k, gimme a sec to finish reading the scheduler meeting notes and I'll look into it.15:49
maciejjozefczykjaypipes: thx15:49
jaypipesefried: "<efried> I see clients finding it useful to be able to do one call and get a single blob with all that information collated." <-- that's kinda like the purpose of GET /allocation_candidates in a way.15:50
mriedemjaypipes: stephenfin: i'm +2 on the placement osc plugin patch for rp inventories https://review.openstack.org/#/c/457533/ - it has some warts which i'll clean up in a follow up patch, but would be good to keep this ball rolling15:50
*** mvk has quit IRC15:51
efriedjaypipes I can see that.  The difference being the input parameters: GET /allocation_candidates wants me to tell it what resources/traits I need.  Versus saying "here's a RP - give me all the information about that guy".15:51
jaypipesefried: yes, understood.15:52
efriedjaypipes Which is basically what that new SRC method get_provider_tree is doing.  But on the client side, via 2N+1 calls (ish)15:52
jaypipesefried: though one might make the argument that the code in https://review.openstack.org/#/c/521098/4/nova/scheduler/client/report.py only needs to be run once and the client is the appropriate place to put that logic and caching.15:53
efriedjaypipes Only once... per what?15:53
jaypipesefried: per instantiation of the report client.15:53
*** bnemec-pto is now known as bnemec15:54
efriedjaypipes And then the report client has to keep the local copy of ProviderTree up to date whenever Something Changes?15:54
jaypipesefried: err, that's already exactly what it does...15:54
jaypipesself._provider_tree.15:54
*** sean-k-mooney has quit IRC15:55
efriedjaypipes Hum, which I was *not* planning to pass into the virt driver.15:55
jaypipesefried: I was hoping you'd pass a clone()/deepcopy of self._provider_tree.15:55
*** sean-k-mooney has joined #openstack-nova15:55
efriedjaypipes Maybe - need to look into that further.  But if I do, self._provider_tree is gonna need to be populated with more stuff than it is today, I think.15:56
jaypipesefried: and do the diff against self._provider_tree when update_inventory() runs and the resulting ProviderTree is sent to reportclient for set_inventory_for_tree() (or whatever that method is named)15:56
efriedAlso, self._provider_tree very well may be a superset of what the virt driver cares about.  But I suppose that's okay.15:56
efriedjaypipes I think keeping-local-copy-current may be doable when the compute node owns all the resources.  But sharing providers could (and will) be updated from outside.  So we'll have to resync those periodically.15:57
*** ratailor has quit IRC15:57
jaypipesefried: sure, but that's a relatively rare operation and the compute worker won't be updating inventory records against shared providers (it will delete allocations against shared providers, of course, when terminations happen)15:58
*** hoonetorg has joined #openstack-nova15:58
efriedjaypipes Actually, I do see virt updating shared provider records.15:59
jaypipesefried: maybe for powervm15:59
*** itlinux has joined #openstack-nova15:59
efriedjaypipes As a specific example, yes, our shared storage provider (confusingly called SSP) is managed by the same hypervisor.16:00
*** lajoskatona has quit IRC16:00
efriedIt's just that any of the hypervisors attached to the pool are allowed to manage it.16:00
jaypipesefried: that's fine. that's what the generation is for.16:00
*** yamamoto has quit IRC16:00
efriedokay.16:00
*** jmlowe has joined #openstack-nova16:01
sean-k-mooneydoes anyone know how to check if a compute node is running a specific version of nova from the conductor. is there an rpc call i can use from the conductor to get the version?16:02
openstackgerritEric Fried proposed openstack/nova master: placement: Document microversion header in api-ref  https://review.openstack.org/52156316:02
*** andrea___ has joined #openstack-nova16:02
efriedmriedem ^ That looks a lot more like the compute one now.16:02
jaypipessean-k-mooney: yeah. nova.objects.Service.get_minimum_version() or something like that. one sec, grabbing link16:03
jaypipessean-k-mooney: https://github.com/openstack/nova/blob/master/nova/objects/service.py#L40516:03
*** chyka has joined #openstack-nova16:04
*** andreas_s has quit IRC16:05
*** andrea___ has quit IRC16:06
sean-k-mooneyjaypipes: thanks looking at it now.16:06
*** yamamoto has joined #openstack-nova16:06
*** andreas_s has joined #openstack-nova16:06
sean-k-mooneyjaypipes: that looks like it returns the minimum version supported by the could rather then for a specific host is that correct16:08
*** rmart04 has quit IRC16:08
jaypipessean-k-mooney: yes, correct.16:09
*** suresh12 has joined #openstack-nova16:09
jaypipessean-k-mooney: if you want a specific compute host's version, use Service.get_by_host_and_binary().version16:10
jaypipessean-k-mooney: sorry, figured you just wanted to find out if the deployment had been upgraded completely or not...16:10
sean-k-mooneyjaypipes: yes that the latter is exactly what i'm looking for16:10
jaypipescool16:10
*** yamamoto has quit IRC16:11
sean-k-mooneyits for the livemigration with multiple port bindings. during a rolling upgrade we want to allow the new flow if both the source and destination are already upgraded.16:11
*** snasir has quit IRC16:12
*** andreas_s has quit IRC16:13
*** andreas_s has joined #openstack-nova16:14
jaypipessean-k-mooney: gotcha.16:14
openstackgerritEric Fried proposed openstack/nova master: placement: Document request headers in api-ref  https://review.openstack.org/52156316:14
jaypipessean-k-mooney: might want to run your idea by dansmith but sounds fine to me.16:14
*** chyka has quit IRC16:14
*** mlavalle has quit IRC16:15
*** mlavalle has joined #openstack-nova16:15
*** chyka has joined #openstack-nova16:15
*** mlavalle has quit IRC16:16
*** andreas_s has quit IRC16:18
*** sree has quit IRC16:22
*** josecastroleon has joined #openstack-nova16:23
*** andreas_s has joined #openstack-nova16:24
*** AlexeyAbashkin has quit IRC16:25
*** Sandy619 has joined #openstack-nova16:25
sean-k-mooneyhow often is the nova-specs repo mirrored to github?16:26
*** josecastroleon has quit IRC16:28
sean-k-mooneyit looks like it has not been synced since september16:28
*** Sandy619 has quit IRC16:30
*** Sandy619 has joined #openstack-nova16:31
*** tssurya has quit IRC16:31
openstackgerritMatt Riedemann proposed openstack/osc-placement master: Address comments from original inventory patch  https://review.openstack.org/52157816:32
*** andreas_s has quit IRC16:33
*** andreas_s has joined #openstack-nova16:34
*** mlavalle has joined #openstack-nova16:34
mriedemsean-k-mooney: looks like an infra problem16:35
*** pcaruana has quit IRC16:35
mriedemhttps://specs.openstack.org/openstack/nova-specs/specs/queens/index.html16:35
mriedemif you're looking for the spec16:35
*** josecastroleon has joined #openstack-nova16:36
*** moshele has joined #openstack-nova16:36
*** yamamoto has joined #openstack-nova16:36
*** salv-orlando has quit IRC16:38
*** andreas_s has quit IRC16:38
*** salv-orlando has joined #openstack-nova16:39
sean-k-mooneymriedem: ya just asked over on the infra channel. apparently the repo is corrupt in some way and github currently does not like it. the git.openstack.org version works fine16:39
*** moshele has quit IRC16:41
*** yamamoto has quit IRC16:41
*** psachin has quit IRC16:43
*** jpena is now known as jpena|brb16:43
*** salv-orlando has quit IRC16:43
*** Apoorva has joined #openstack-nova16:44
*** andreas_s has joined #openstack-nova16:45
*** artom has joined #openstack-nova16:45
*** tssurya has joined #openstack-nova16:46
openstackgerritEric Fried proposed openstack/nova master: placement: Document request headers in api-ref  https://review.openstack.org/52156316:47
cdentefried: I just left a potentially relevant link/comment on that ^16:50
efriedcdent looking...16:50
*** Apoorva has quit IRC16:51
*** itlinux has quit IRC16:53
efriedcdent Good deal.  I'll let mriedem decide whether that's reason to keep the words about the Accept header.16:53
*** Apoorva has joined #openstack-nova16:53
cdent16:53
*** andreas_s has quit IRC16:54
*** itlinux has joined #openstack-nova16:54
*** andreas_s has joined #openstack-nova16:56
*** andreas_s has quit IRC16:57
*** andreas_s has joined #openstack-nova16:57
*** armax has quit IRC16:59
*** ratailor has joined #openstack-nova17:00
*** Sandy619 has quit IRC17:03
*** yamamoto has joined #openstack-nova17:05
*** yamamoto has quit IRC17:05
cdentmriedem: your comment on https://review.openstack.org/#/c/518223/1//COMMIT_MSG doesn’t make sense (to me). That paragraph I’m saying “there’s two changes here one fixes a 500, but another changes the content type of existing 400-499 responses so not sure if there is a violation or not"17:05
*** ragiman has quit IRC17:06
mriedemok, i didn't read too deep into that paragraph17:06
*** andreas_s has quit IRC17:06
dansmithmriedem: see my comment here? https://review.openstack.org/#/c/521186/3/nova/tests/functional/test_servers.py I think a custom hint is probably the best way forward here, so we don't depend on that filter being enabled17:07
dansmithmriedem: we can also use that to make the "is policy" more straight forward, and if we do it customly, we'll be able to enforce host and node equivalence17:08
mriedemdidn't see the comment until now, but i was thinking about a custom hint over the weekend too, but forgot about it17:09
mriedembut then we don't need deployers to change anything for the fix17:09
dansmithright17:09
mriedemso would you add a new filter too?17:09
dansmithno,17:10
dansmithI'd just do it in host manager like force_hosts works17:10
mriedemok17:10
mriedemthen tallyho17:10
dansmiththis is a real fun one17:10
*** Guest18917 has quit IRC17:11
openstackgerritSylvain Bauza proposed openstack/nova master: libvirt: Provide VGPU inventory for a single GPU type  https://review.openstack.org/51476317:11
*** Apoorva has quit IRC17:15
*** purplerbot has quit IRC17:15
*** purplerbot has joined #openstack-nova17:15
*** hemna_ has quit IRC17:17
*** hemna_ has joined #openstack-nova17:17
*** lucasagomes is now known as lucas-afk17:23
*** lpetrut has quit IRC17:23
*** yamahata has quit IRC17:24
*** jpena|brb is now known as jpena17:26
openstackgerritAndrey Kurilin proposed openstack/python-novaclient master: [ci] Use pseudo-random names for new resources  https://review.openstack.org/47977717:36
*** armax has joined #openstack-nova17:36
*** ratailor has quit IRC17:36
*** suresh12 has quit IRC17:37
*** edmondsw has quit IRC17:37
*** edmondsw has joined #openstack-nova17:38
*** bkopilov has joined #openstack-nova17:39
*** fragatina has joined #openstack-nova17:39
*** salv-orlando has joined #openstack-nova17:39
*** edmondsw has quit IRC17:42
*** moshele has joined #openstack-nova17:42
*** yangyapeng has joined #openstack-nova17:43
openstackgerritEric Fried proposed openstack/nova master: placement: Traits ops on ProviderTree  https://review.openstack.org/52160517:44
*** Apoorva has joined #openstack-nova17:44
efriedjaypipes Reckon we're needing this ^ soon, though I couldn't say what bp to put it under.17:44
*** efried is now known as efried_rollin17:44
jaypipesefried_rollin: k17:44
*** salv-orlando has quit IRC17:45
*** Apoorva has quit IRC17:45
openstackgerritMatt Riedemann proposed openstack/nova master: Add instance action record for attach/detach interface  https://review.openstack.org/51219017:45
*** yangyapeng has quit IRC17:50
*** Apoorva has joined #openstack-nova17:50
*** Apoorva_ has joined #openstack-nova17:59
*** pcaruana has joined #openstack-nova18:00
*** derekh has quit IRC18:01
*** Apoorva has quit IRC18:02
*** yamahata has joined #openstack-nova18:03
*** rodolof has joined #openstack-nova18:03
*** slaweq has joined #openstack-nova18:05
*** yamamoto has joined #openstack-nova18:05
*** slaweq_ has joined #openstack-nova18:06
*** sshwarts has quit IRC18:06
*** slaweq has quit IRC18:10
*** yamamoto has quit IRC18:11
*** yangyapeng has joined #openstack-nova18:12
*** moshele has quit IRC18:14
*** yangyapeng has quit IRC18:17
*** slaweq_ has quit IRC18:19
*** jpena is now known as jpena|off18:20
*** dtantsur is now known as dtantsur|afk18:27
*** lpetrut has joined #openstack-nova18:29
*** suresh12 has joined #openstack-nova18:30
*** yangyapeng has joined #openstack-nova18:31
*** yangyapeng has quit IRC18:34
*** Apoorva_ has quit IRC18:35
*** Apoorva has joined #openstack-nova18:36
*** salv-orlando has joined #openstack-nova18:40
*** salv-orlando has quit IRC18:42
*** salv-orlando has joined #openstack-nova18:42
*** imacdonn has joined #openstack-nova18:45
*** liverpooler has quit IRC18:45
*** lpetrut has quit IRC18:51
*** rmcall_ has joined #openstack-nova18:52
*** peter-hamilton has joined #openstack-nova18:54
*** lpetrut has joined #openstack-nova18:55
*** Daisy_ has joined #openstack-nova18:56
*** yangyapeng has joined #openstack-nova18:57
*** rodolof has quit IRC18:57
*** rmcall_ has quit IRC18:58
*** Daisy_ has quit IRC19:00
*** lpetrut has quit IRC19:01
*** yangyapeng has quit IRC19:01
*** lpetrut has joined #openstack-nova19:01
*** itlinux has quit IRC19:03
*** Guest18917 has joined #openstack-nova19:08
*** andreas_s has joined #openstack-nova19:13
*** yangyapeng has joined #openstack-nova19:15
openstackgerritChris Dent proposed openstack/nova master: [placement] Object changes to support last-modified headers  https://review.openstack.org/52163919:16
openstackgerritChris Dent proposed openstack/nova master: [placement] Add cache headers to placement api requests  https://review.openstack.org/52164019:16
*** andreas_s has quit IRC19:18
cdentdansmith: that ^ is the last-modified stuff done all in one patch/microversion19:19
*** yangyapeng has quit IRC19:20
openstackgerritMatt Riedemann proposed openstack/nova master: Add migration_get_by_uuid in db api.  https://review.openstack.org/51142119:28
openstackgerritMatt Riedemann proposed openstack/nova master: Add migration db and object pagination support.  https://review.openstack.org/51490419:28
openstackgerritMatt Riedemann proposed openstack/nova master: Implement query param schema for migration index  https://review.openstack.org/51864419:28
openstackgerritMatt Riedemann proposed openstack/nova master: Extract SortContext/SortWrapper of instance_list into utils  https://review.openstack.org/51887119:28
openstackgerritMatt Riedemann proposed openstack/nova master: Add cross cell sort support for get_migrations  https://review.openstack.org/51727319:28
openstackgerritMatt Riedemann proposed openstack/nova master: Add pagination and Changes-since filter support for os-migrations.  https://review.openstack.org/33040619:28
*** itlinux has joined #openstack-nova19:31
cfriesendoes the current master branch scheduler handle working as a hybrid scheduler for baremetal as well as VM compute nodes?  I'm mostly wondering about things like the NUMATopologyFilter.19:36
*** yangyapeng has joined #openstack-nova19:36
mriedemdon't think so19:37
mriedemb/c there are different host managers19:37
*** jmlowe has quit IRC19:37
mriedemHostManager vs IronicHostManager19:38
mriedemalthough,19:38
mriedemwith resource classes in pike, i'm not sure how much you need the IronicHostManager anymore19:38
cfriesenmriedem: yeah...IronicHostManager mostly calls the superclass functions19:38
*** toure is now known as toure_biab19:39
mriedemand use_baremetal_filters and baremetal_enabled_filters are deprecated19:39
mriedemso, IronicHostManager should probably be deprecated19:39
cfriesenmriedem: we're making it work on Pike and need to modify some of the filters.  was just wondering if this was something that I should propose for Rocky as a spec19:39
mriedemsince the related config options for the filters are also deprecated19:39
cfriesenor if it's just bugfix material19:39
mriedemspec for what?19:40
mriedemsame scheduler serving vm and bm?19:40
cfriesenyes19:40
mriedemi think you get that with custom resource classes19:40
cfriesenbut they don't handle the fancy filters, right?19:40
cfriesenso if you enable numatopologyfilter it breaks baremetal19:40
mriedemi don't know how the fancy filters handle ironic nodes19:40
dansmithI didn't think you needed ironic host manager even before placement19:40
mriedemi bet johnthetubaguy would love to talk about this19:41
dansmithlike, originally you did but I thought aside from a couple details you were good now19:41
mriedemwell, difference is in the filters that are used19:41
mriedemIronicHostManager used the Exact* filters19:41
mriedemand all of those are deprecated since pike19:41
*** yangyapeng has quit IRC19:41
dansmithright, but if you didn't need those filters, then you could use the same host manager as virt19:42
cfriesendansmith: the docs still say to enable it.  there are some differences in how states are reported19:42
cfriesenit looks like IronicHostManager is supposed to work with VM nodes19:42
mriedemso this19:42
mriedemdef _locked_consume_from_request(self, spec_obj):19:42
mriedem        """Consume nodes entire resources regardless of instance request."""19:43
mriedem        self.free_ram_mb = 019:43
mriedem        self.free_disk_mb = 019:43
mriedem        self.vcpus_used = self.vcpus_total19:43
openstackgerritMatt Riedemann proposed openstack/nova master: Merge flavor extensions controller code  https://review.openstack.org/51610419:43
cfriesenbut some of the filters don't19:43
dansmithI don't know all the details, but before placement came and kindof distracted us I thought we were about good to remove that19:43
mriedemthe ironic driver will consume all memory_mb/disk_gb/vcpu inventory on a node in placement19:43
dansmiththe docs are mostly aimed at just ironic deployments though where there's no reason not to enable the dedicated thing right?19:43
mriedemso that IronicHostState is irrelevant19:43
mriedem*IronicNodeState19:44
cfriesenyeah...there's not a lot of info on how to set up hybrid.  I did talk with Julia at the summit about some of this.19:44
mriedemsdague: easy api extension merge-a-roo https://review.openstack.org/#/c/516104/19:45
*** gbarros has quit IRC19:45
cfriesencurrently we modified NUMATopologyFilter.host_passes() to just return True for ironic nodes...wondering if this is something to push upstream19:45
mriedemcan ironic report numa capabilities? from what i gather, lots of people rely on the CapabilitiesFilter for ironic19:46
cfriesenno idea. :)19:46
mriedem*ComputeCapabilitiesFilter19:46
*** gbarros has joined #openstack-nova19:47
*** gbarros has quit IRC19:48
*** Sukhdev has joined #openstack-nova19:48
*** gbarros has joined #openstack-nova19:49
openstackgerritMatt Riedemann proposed openstack/nova master: Fix some incorrect option references for scheduler filters  https://review.openstack.org/52164519:51
*** yangyapeng has joined #openstack-nova19:57
openstackgerritAndrey Kurilin proposed openstack/python-novaclient master: [ci] Use pseudo-random names for new resources  https://review.openstack.org/47977719:59
*** cdent has quit IRC20:01
openstackgerritDan Smith proposed openstack/nova master: Alternate fix for validating image on rebuild  https://review.openstack.org/52118620:01
*** yangyapeng has quit IRC20:01
*** claudiub|2 has quit IRC20:14
*** artom has quit IRC20:14
openstackgerritMatt Riedemann proposed openstack/nova master: Deprecate the IronicHostManager  https://review.openstack.org/52164820:14
*** tssurya_ has joined #openstack-nova20:16
*** yangyapeng has joined #openstack-nova20:17
*** yangyapeng has quit IRC20:22
*** salv-orlando has quit IRC20:24
*** salv-orlando has joined #openstack-nova20:25
*** suresh12 has quit IRC20:26
mriedemsean-k-mooney: i'm confused, is this a nova bp for just work that is all done in os-vif? https://blueprints.launchpad.net/nova/+spec/vif-port-profile20:28
*** salv-orlando has quit IRC20:28
*** edmondsw has joined #openstack-nova20:29
*** edmondsw has quit IRC20:33
*** liverpooler has joined #openstack-nova20:33
*** salv-orlando has joined #openstack-nova20:36
*** pcaruana has quit IRC20:39
*** yangyapeng has joined #openstack-nova20:41
*** efried_rollin is now known as efried20:43
*** awaugama has quit IRC20:44
*** yangyapeng has quit IRC20:45
*** sahid has quit IRC20:50
*** rodolof has joined #openstack-nova20:50
*** felipemonteiro has joined #openstack-nova20:57
*** Apoorva_ has joined #openstack-nova21:02
*** krtaylor has quit IRC21:03
efriedjaypipes What is the proper way to talk about a RP's association with an aggregate?  Can I say the RP is *in* the aggregate?21:03
jaypipesefried: yes21:03
efriedk21:03
efriedjaypipes So here's what I'm thinking.21:03
efriedjaypipes Today the report client keeps aggregate associations outside of ProviderTree.  I believe we need to move what is currently SchedulerReportClient._provider_aggregate_map into ProviderTree.  Specifically, _Provider has a set() of aggregate UUIDs.21:04
*** READ10 has quit IRC21:05
efriedjaypipes Because the virt driver will ultimately be in charge of assigning aggregate associations, so we'll want the same suite of methods on ProviderTree to ask if a given RP is "in" an aggregate, add (or replace) aggregate associations; etc.21:05
efriedjaypipes Same thing for traits, which is where that change set of a couple hours ago was coming from - to be followed up by SRC methods to populate/query it.21:06
*** Apoorva has quit IRC21:06
jaypipesefried: not sure the virt driver will be responsible for associated an aggregate to a provider -- unless the virt driver is also creating those aggregates. Does PowerVM do that?21:11
jaypipesefried: for traits, yeah, totes agree.21:11
efriedjaypipes In the only real world case we have right now - shared storage pool - yes.  I can't think how else the aggregate would get created.21:12
jaypipesefried: it was originally intended to be created by an admin or an external agent (for example, Cinder...)21:12
*** erlon has joined #openstack-nova21:12
*** AlexeyAbashkin has joined #openstack-nova21:12
efriedjaypipes Yeah, I suppose I can see that.  In our case, we use SSP for ephemeral, so it would have to be more like the admin.  It would just be another step they have to do when they're setting up the host.21:14
jaypipesefried: understood.21:14
efriedjaypipes BTW, is there any rule against me using the same UUID for an aggregate and a RP that's in it?21:14
efriedI mean, I'm sure you'll say that's not a good idea, but is anything gonna stop me?21:14
jaypipesefried: not a good idea, but nothing will prevent you from doing that.21:15
efriedk21:15
*** krtaylor has joined #openstack-nova21:16
*** AlexeyAbashkin has quit IRC21:17
efriedjaypipes Okay, so one more thing: If we're moving aggregates (and adding traits) to ProviderTree, we could keep track of the last-refreshed time in either place.  I'm assuming we'd like to keep doing that part at the SRC rather than in ProviderTree?21:18
*** dave-mccowan has quit IRC21:18
efriedsince he's the one doing the refreshing.21:18
*** suresh12 has joined #openstack-nova21:19
*** claudiub|2 has joined #openstack-nova21:20
openstackgerritMatt Riedemann proposed openstack/nova master: Remove the deprecated TrustedFilter and related options  https://review.openstack.org/52165921:21
jaypipesefried: yes21:26
efriedight21:26
openstackgerritDan Smith proposed openstack/nova master: Add regression test for rebuild with new image doubling allocations  https://review.openstack.org/52115321:30
openstackgerritDan Smith proposed openstack/nova master: Fix doubling allocations on rebuild  https://review.openstack.org/52166221:30
openstackgerritMatt Riedemann proposed openstack/nova master: Use the RequestSpec when getting scheduler_hints in compute  https://review.openstack.org/51546121:38
openstackgerritMatt Riedemann proposed openstack/nova master: Pass RequestSpec to ConductorTaskAPI.build_instances  https://review.openstack.org/51549521:38
*** Sukhdev has quit IRC21:41
*** tssurya_ has quit IRC21:44
*** felipemonteiro has quit IRC21:44
*** eharney has quit IRC21:45
*** linkmark has joined #openstack-nova21:45
openstackgerritDan Smith proposed openstack/nova master: Fix doubling allocations on rebuild  https://review.openstack.org/52166221:47
*** rcernin has joined #openstack-nova21:48
*** tssurya_ has joined #openstack-nova21:49
*** gbarros has quit IRC21:57
efriedjaypipes Confirm: We weren't actually using the SRC aggregate map for anything yet?21:57
*** tssurya_ has quit IRC21:57
*** tssurya_ has joined #openstack-nova21:59
*** gbarros has joined #openstack-nova22:00
*** itlinux has quit IRC22:01
*** gbarros_ has joined #openstack-nova22:02
*** itlinux has joined #openstack-nova22:04
*** gbarros has quit IRC22:04
*** Apoorva_ has quit IRC22:06
*** tssurya_ has quit IRC22:07
*** hangyang has joined #openstack-nova22:08
*** pchavva has quit IRC22:10
mriedemefried: what happened here? https://review.openstack.org/#/q/status:abandoned+topic:bp/add-trait-support-in-allocation-candidates22:12
mriedemis that all superseded by jay's other series?22:12
efriedmriedem Most of it, yes.22:13
efriedLet me look for outliers...22:13
*** rodolof has quit IRC22:13
efriedmriedem Yeah; alex_xu started off doing the refactor one way; then jaypipes did it a different way and jaypipes "prevailed" :)22:14
mriedemok, the new series isn't tracked in launchpad as far as i can tell so i don't know what's going on there22:14
*** mvk has joined #openstack-nova22:14
efriedmriedem TBH, add-trait-support-in-allocation-candidates may be superseded by granular-resource-requests, depending on timing.22:17
*** edmondsw has joined #openstack-nova22:17
hangyangHi there, I have some questions for time sync in vm and its KVM host. I have some RHEL7 instances in Ocata openstack cluster. The VMs are not running ntpd and after suspend/resume, I notice there is a time drift between the VM and its KVM host and the VM will never catch up with host. But for stop/start, the vm can keep time in sync with its host. Any idea about how to solve the suspend issue? Should I run ntpd in the guest22:17
efriedmriedem See my comments in https://review.openstack.org/#/c/479776/22:17
efried(which is the last open one tracked by that bp)22:17
*** gbarros has joined #openstack-nova22:19
*** gbarros_ has quit IRC22:21
*** rtjure has quit IRC22:21
*** edmondsw has quit IRC22:22
*** rtjure has joined #openstack-nova22:23
efriedmriedem Since you seem to be looking, the rest of the ksa patches are gonna be based on https://review.openstack.org/#/c/490057/22:23
efried...which I believe you once referred to as "scary".22:23
*** flwang has joined #openstack-nova22:23
mriedemyes22:23
*** yangyapeng has joined #openstack-nova22:23
mriedemi'm going through all of our queens blueprints and tracking latest status in https://etherpad.openstack.org/p/nova-queens-blueprint-status22:24
mriedemif you must know22:24
mriedemonly about 35 more to go22:24
flwangmriedem: sorry for chiming in22:24
flwangmriedem: question about the poverVM driver, does it support Power 720 series?22:24
*** liverpooler has quit IRC22:24
efriedflwang I *think* so.  Note that we have a dedicated #openstack-powervm channel - though it's not going to help you much today as everyone else is on vacation already.  But let me see if I can find out that answer for sure...22:26
flwangefried: BIG thanks from an ex-IBMer22:27
*** yangyapeng has quit IRC22:28
tonybdansmith: What are the ds* flavors in devstack used for?22:29
efriedflwang The answer is: nope, P8 and up.22:29
tonybflwang: I have a 720 in my garage, and yeah no KVM, PowerVM or even wemu(TCG) love there :(22:29
flwangreally? that's a pitty22:30
flwangbut I found this  https://www-01.ibm.com/events/wwe/grp/grp020.nsf/vLookupPDFs/Demo%20days%20LoP%20-%20Cloud%20on%20Power%20-%2018%20Mag%202015/$file/Demo%20days%20LoP%20-%20Cloud%20on%20Power%20-%2018%20Mag%202015.pdf22:31
flwangsee page 1822:31
efriedflwang Was just about to say: *PowerVC* supports 720.22:32
efriedflwang It runs through HMC, not community code, but the interface is seamlessly identical I believe.22:32
flwangefried: I see. so that only means PowerVC can via HMC manage 720, right?22:33
jaypipesefried, mriedem: off for the night. will review all the things in the morning.22:33
efriedflwang Correct22:34
efriedjaypipes Have a good evening.22:34
efriedjeez, it got late22:34
flwangefried: where can i double check if there is a KVM/PowerVM support for Power 720?22:34
tonybefried: Yeah but not helpful for people lik me that don't have an HMC or PowerVC, still I only use mine for testing yaboot which is nearly dead on 64-bit anyway ... longlive grub!22:34
*** jaypipes has quit IRC22:34
*** hangyang has quit IRC22:35
efriedflwang Let me PM you an email address to get that conversation started...22:35
flwangefried: thank you so much22:35
*** hangyang has joined #openstack-nova22:36
* tonyb wonders how hard it'd be to write an ironic diver for (old) PowerVM #tangent22:37
tonybthat'd be kinda cool from a geeky POV22:37
mriedemtonyb: i'm sure mrodden and lbragstad would like to help with that22:38
tonybmriedem: \o/22:38
*** Apoorva has joined #openstack-nova22:39
tonybthe more I think about it the more fun I think that'd be22:39
*** Apoorva has quit IRC22:40
efriedtonyb I'll review your code :)22:40
*** Apoorva has joined #openstack-nova22:40
*** gszasz has quit IRC22:41
*** lyan has quit IRC22:42
*** yangyapeng has joined #openstack-nova22:44
tonybefried: :)22:48
*** Sukhdev has joined #openstack-nova22:49
*** yangyapeng has quit IRC22:49
*** itlinux has quit IRC22:49
*** itlinux has joined #openstack-nova22:50
*** lbragstad has quit IRC22:51
*** lbragstad has joined #openstack-nova23:04
openstackgerritIldiko Vancsa proposed openstack/nova master: Implement new attach Cinder flow  https://review.openstack.org/33028523:04
openstackgerritIldiko Vancsa proposed openstack/nova master: libvirt: Allow multiple volume attachments  https://review.openstack.org/26758723:04
*** sdague has quit IRC23:09
ildikovmriedem: so quickly, did you think of something like this for the Nova side check: https://review.openstack.org/#/c/330285/148..149/nova/compute/api.py ?23:11
mriedemildikov: close23:14
mriedembut -123:14
mriedemmuwahahahaha23:14
mriedemwe should only do that for the new flow23:14
mriedemcinder will check it otherwise during reserve()23:14
mriedemwhich is why we removed check_attach in the first place23:15
ildikovmriedem: ok, so do a microversion check before the check calls, right?23:15
ildikovyeah, I know, I just wasn't sure whether this is the right direction and didn't want to overcomplicate this before asking for your opinion23:15
mriedemwell, presumably we know if we're doing the new flow or not based on the attachment stuff we do23:16
ildikovas I expected something, 'no, that's insane, you should've done this and that instead', lol23:16
mriedemlike for swap_volume we check if the old bdm has an attachment_id i think23:16
mriedemfor attach_volume it's probably trickier, because i think we try to call attachment_create and if that fails we fallback to calling reserve()23:16
ildikovyeah, but for the regular attach the first time we check is _check_attach_and_reserve_volume23:16
ildikovyep, so by that time we created a new BDM, etc23:17
mriedemyeah, so we probably have to do this before attachment_create() for the attach_volume case23:17
ildikovhence I would do this check before that23:17
mriedembut for swap_volume i think we can not do this if we're old flow23:17
mriedemand we know that23:17
mriedemyou also have to handle that bdm method raising a VolumeBDMNotFound error23:17
ildikovsure, I'll check that part23:17
ildikovah, right, forgot that23:18
mriedemotherwise yeah i think this is what i was thinking of23:18
ildikovok, cool, I'll fix up the nits then23:18
ildikovthanks!23:18
mriedemyw23:19
mriedemKevin_Zheng: have you started any of the neutron code for this yet? https://bugs.launchpad.net/neutron/+bug/171860523:20
openstackLaunchpad bug 1718605 in neutron "[RFE] Support regex matching when filtering port with IP address" [Wishlist,Triaged] - Assigned to Zhenyu Zheng (zhengzhenyu)23:20
Kevin_Zhengmriedem I’ve wrote some poc code23:21
*** yamamoto has joined #openstack-nova23:21
Kevin_ZhengDidn’t push to gerrit yet23:21
*** marst has quit IRC23:21
mriedemok, just checking23:22
*** mlavalle has quit IRC23:24
clarkbmriedem: hey, so one of the reasons logstash is so behind right now is we keep trying to push million line libvirtd log files through it. How huge of a loss would it be if we stopped indexing those files?23:25
mriedemclarkb: i didn't think we indexed those, so not a loss i'd say23:26
clarkbmostly trying to get it back to manageable then add things back in while we can address potential problems with it23:26
clarkbmriedem: ok cool I'm going to go ahead and drop indexing for them then23:26
mriedemif anything, i've needed the qemu guest logs indexed at times for things like live migration failures23:26
mriedemthat reminds me that i need to cleanup our e-r queries23:27
clarkbmriedem: https://review.openstack.org/521681 fyi23:29
mriedem+123:30
*** lpetrut has quit IRC23:32
*** gszasz has joined #openstack-nova23:34
*** mriedem is now known as mriedem_away23:36
*** linkmark has quit IRC23:38
*** Sukhdev has quit IRC23:40
*** gfhellma has joined #openstack-nova23:49
*** hangyang has quit IRC23:54
dansmithtonyb: when we removed the default flavors from the db migrations we added those in devstack setup so you could depend on some flavors in a default devstack environment23:55
dansmithtonyb: I think we re-added the original ones to devstack after that when people complained (IIRC23:55
*** yingjun has joined #openstack-nova23:56
openstackgerritEric Fried proposed openstack/nova master: Traits ops on ProviderTree  https://review.openstack.org/52160523:58
openstackgerritEric Fried proposed openstack/nova master: Move aggregates from report client to ProviderTree  https://review.openstack.org/52168523:58
openstackgerritEric Fried proposed openstack/nova master: Track provider traits in report client  https://review.openstack.org/52168623:58
*** armax has quit IRC23:59

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