Thursday, 2019-03-28

*** wolverineav has quit IRC00:02
*** luksky has quit IRC00:06
*** hongbin has joined #openstack-nova00:18
*** betherly has joined #openstack-nova00:28
*** gbarros has quit IRC00:32
*** betherly has quit IRC00:33
*** lbragstad has joined #openstack-nova00:45
*** lbragstad has quit IRC00:57
*** tiendc has joined #openstack-nova01:00
*** betherly has joined #openstack-nova01:01
*** ileixe has joined #openstack-nova01:03
*** dsuper7 has quit IRC01:05
*** betherly has quit IRC01:05
openstackgerritTakashi NATSUME proposed openstack/nova master: doc: Fix openstack CLI command  https://review.openstack.org/64829901:06
*** awalende has joined #openstack-nova01:13
*** gyee has quit IRC01:16
*** awalende has quit IRC01:18
openstackgerritmelanie witt proposed openstack/nova master: Add get_counts() to InstanceMappingList  https://review.openstack.org/63807201:18
openstackgerritmelanie witt proposed openstack/nova master: Count instances from mappings and cores/ram from placement  https://review.openstack.org/63807301:18
openstackgerritmelanie witt proposed openstack/nova master: Use instance mappings to count server group members  https://review.openstack.org/63832401:18
*** betherly has joined #openstack-nova01:33
*** betherly has quit IRC01:37
*** gouthamr has quit IRC02:10
*** jhesketh has joined #openstack-nova02:12
*** gouthamr has joined #openstack-nova02:12
*** betherly has joined #openstack-nova02:15
*** _alastor_ has joined #openstack-nova02:19
*** betherly has quit IRC02:20
*** _alastor_ has quit IRC02:23
openstackgerritTakashi NATSUME proposed openstack/nova master: Add minimum value in max_concurrent_live_migrations  https://review.openstack.org/64830202:33
*** wolverineav has joined #openstack-nova02:37
*** wolverineav has quit IRC02:42
*** psachin has joined #openstack-nova02:52
*** whoami-rajat has joined #openstack-nova02:58
*** betherly has joined #openstack-nova03:08
*** betherly has quit IRC03:13
openstackgerritMerged openstack/nova stable/pike: Document unset/reset wrinkle for *_allocation_ratio options  https://review.openstack.org/64729203:19
*** hongbin has quit IRC03:28
openstackgerritMerged openstack/nova stable/queens: Don't persist zero allocation ratios in ResourceTracker  https://review.openstack.org/61327103:33
openstackgerritMerged openstack/nova stable/queens: Update resources once in update_available_resource  https://review.openstack.org/61229403:33
*** ivve has joined #openstack-nova04:01
*** betherly has joined #openstack-nova04:02
*** udesale has joined #openstack-nova04:05
*** betherly has quit IRC04:07
*** erlon has joined #openstack-nova04:11
*** cfriesen has quit IRC04:20
*** ivve has quit IRC04:24
*** sridharg has joined #openstack-nova04:28
*** ileixe has quit IRC04:29
*** erlon has quit IRC04:36
*** janki has joined #openstack-nova04:41
*** rcernin has quit IRC04:47
*** betherly has joined #openstack-nova04:55
*** zhubx007 has quit IRC04:57
*** ileixe has joined #openstack-nova04:57
*** betherly has quit IRC04:59
*** rcernin has joined #openstack-nova05:02
*** c6 has joined #openstack-nova05:02
*** markvoelker has joined #openstack-nova05:04
*** zhubx has joined #openstack-nova05:05
*** whoami-rajat has quit IRC05:07
*** jaypipes has quit IRC05:09
*** jaypipes has joined #openstack-nova05:09
*** c6 has left #openstack-nova05:14
*** tetsuro has quit IRC05:35
*** betherly has joined #openstack-nova05:48
*** betherly has quit IRC05:52
*** igordc has quit IRC05:57
*** rcernin has quit IRC06:03
*** slaweq has quit IRC06:07
*** sidx64 has joined #openstack-nova06:22
*** ccamacho has quit IRC06:23
*** wolverineav has joined #openstack-nova06:24
*** whoami-rajat has joined #openstack-nova06:31
*** wolverineav has quit IRC06:36
*** wolverineav has joined #openstack-nova06:37
*** betherly has joined #openstack-nova06:40
*** sidx64 has quit IRC06:40
*** sidx64 has joined #openstack-nova06:42
*** betherly has quit IRC06:45
*** rcernin has joined #openstack-nova06:47
*** rcernin has quit IRC06:47
*** rcernin has joined #openstack-nova06:48
*** ratailor has joined #openstack-nova06:50
*** sidx64 has quit IRC06:56
*** ileixe has quit IRC07:02
*** phasespace has quit IRC07:02
*** ileixe has joined #openstack-nova07:03
*** sidx64 has joined #openstack-nova07:06
*** slaweq has joined #openstack-nova07:14
*** pcaruana has joined #openstack-nova07:20
openstackgerritMerged openstack/nova master: Exec systemd-run without --user flag in Quobyte driver  https://review.openstack.org/55419507:28
*** amorin has quit IRC07:28
*** amorin has joined #openstack-nova07:28
*** artom has quit IRC07:29
*** sridharg has quit IRC07:31
*** sridharg has joined #openstack-nova07:31
*** sidx64 has quit IRC07:33
*** rpittau|afk is now known as rpittau07:41
*** sidx64 has joined #openstack-nova07:42
*** tetsuro has joined #openstack-nova07:46
openstackgerritMerged openstack/nova master: Update instance.availability_zone on revertResize  https://review.openstack.org/64315507:46
*** phasespace has joined #openstack-nova07:46
*** rcernin has quit IRC07:48
*** ttsiouts has joined #openstack-nova07:50
*** sridharg has quit IRC07:50
*** gryf has quit IRC07:59
*** wolverineav has quit IRC08:01
*** wolverineav has joined #openstack-nova08:02
*** tssurya has joined #openstack-nova08:03
*** sridharg has joined #openstack-nova08:04
*** tosky has joined #openstack-nova08:04
*** rcernin has joined #openstack-nova08:05
*** ccamacho has joined #openstack-nova08:05
*** betherly has joined #openstack-nova08:06
*** wolverineav has quit IRC08:08
*** tkajinam has quit IRC08:09
openstackgerritYongli He proposed openstack/nova master: Clean up orphan instances  https://review.openstack.org/62776508:09
*** wolverineav has joined #openstack-nova08:11
*** betherly has quit IRC08:11
*** luksky has joined #openstack-nova08:11
*** tesseract has joined #openstack-nova08:12
*** mrch_ has joined #openstack-nova08:12
*** awalende has joined #openstack-nova08:16
*** _alastor_ has joined #openstack-nova08:22
*** wolverineav has quit IRC08:24
*** ttsiouts has quit IRC08:26
*** wolverineav has joined #openstack-nova08:26
*** ttsiouts has joined #openstack-nova08:26
*** _alastor_ has quit IRC08:26
*** helenafm has joined #openstack-nova08:28
*** sidx64 has quit IRC08:29
*** takashin has left #openstack-nova08:30
*** ttsiouts has quit IRC08:30
*** sidx64 has joined #openstack-nova08:41
*** wolverineav has quit IRC08:42
*** wolverineav has joined #openstack-nova08:43
*** wolverineav has quit IRC08:52
*** betherly has joined #openstack-nova08:59
*** ttsiouts has joined #openstack-nova09:01
*** betherly has quit IRC09:04
tssuryaalex_xu, dansmith: I get the backwards compatibility point and all here: https://review.openstack.org/#/c/638629/6/specs/train/approved/add-locked-reason.rst@106 , but introducing them as individual keys means more keys which would have the value as NULL most of the time.09:06
*** ralonsoh has joined #openstack-nova09:06
*** oanson has quit IRC09:10
*** mdbooth has joined #openstack-nova09:13
*** ileixe has quit IRC09:18
*** betherly has joined #openstack-nova09:23
*** zhubx has quit IRC09:24
*** zhubx has joined #openstack-nova09:24
*** sidx64 has quit IRC09:27
*** betherly has quit IRC09:28
*** ileixe has joined #openstack-nova09:29
*** sidx64 has joined #openstack-nova09:33
*** sidx64 has quit IRC09:34
*** derekh has joined #openstack-nova09:37
*** luksky has quit IRC09:37
*** oanson has joined #openstack-nova09:41
*** sidx64 has joined #openstack-nova09:43
sean-k-mooneyo/09:45
sean-k-mooneyany nova cores about that care to review a neutron spec im proposing? i will be writing a nova spec for the nova/os-vif side but if its a total non starter then ill move on to more important things09:47
sean-k-mooneygibi: thanks for taking a look already09:47
sean-k-mooneyhttps://review.openstack.org/#/c/64517309:47
gibisean-k-mooney: I think the goal to fix the problems you listed in that spec is a good goal. For me it is a bit hard to imagine how this will impact nova, especially the dependency between nova and neutron backends09:49
sean-k-mooneyby the way im aware we are trying to finalise RC2 so i know people are busy09:49
gibisean-k-mooney: I have to go back and read you answers still09:49
sean-k-mooneygibi: ya i think i need to sit down and write the nova centric spec to illistrate that impact so that nova people dont need to look at the neutron aspect as much09:50
sean-k-mooneyi have left a comment that tried to summerise it but im not sure how clear it was09:51
sean-k-mooneythe nova/os-vif impact is summerised here https://review.openstack.org/#/c/645173/2/specs/train/port-binding-extended-information.rst@13009:52
sean-k-mooneybut anyway if other nova cores are interested in nova/neutron stuff early feedback on "is this somethign we want to fix" would be good as i dont want to speed lots of time on the nova spec if its a total non starter09:54
*** lpetrut has joined #openstack-nova09:54
*** priteau has joined #openstack-nova09:55
*** dtantsur|afk is now known as dtantsur09:59
*** cdent has joined #openstack-nova10:01
*** sidx64 has quit IRC10:02
*** maciejjozefczyk has quit IRC10:06
*** tobias-urdin has quit IRC10:06
sean-k-mooneygibi: so looking at you https://blueprints.launchpad.net/nova/+spec/enhance-support-for-ports-having-resource-request i have some long standing wish list items for network aware schduling that might be able to tie into that10:06
sean-k-mooneyif we look back at https://etherpad.openstack.org/p/placement-nova-neutron-queens-ptg i listed some of them when i was making the case for completeing nested resouce providers10:07
gibisean-k-mooney: let's list your needs part of the item in https://etherpad.openstack.org/p/nova-ptg-train10:08
gibiphysnet aware placement looks interesting10:08
*** luksky has joined #openstack-nova10:09
gibivswitch vif capacity also seems doable now10:09
sean-k-mooneywell see that is the thing there are not execatly needs but they are more techdebt removal that happens to also imporve sechduling10:09
sean-k-mooneygibi: ya i think you have put most of the infrastuture in place already as part of your bandwith work to adress some of those wishlist items10:10
sean-k-mooneynuma affinity of vswitches we have done in a none placement way already too10:11
*** tetsuro has quit IRC10:11
gibiI think it worth to talk about these on the PTG. I can definitely give review support for some of this items while I work on the move operation bp10:11
gibisean-k-mooney: do we want to re-do numa aware vswitches based on placement?10:12
sean-k-mooneymaybe at somepoint but i have been wonderign if we should punt numa in placement to U10:12
sean-k-mooneyto try and get other lower hanging furit feature workign first10:13
sean-k-mooneylong term i still hope we can model numa in placement but it makes everything more complicated10:13
sean-k-mooneyfor better or worse the numa topology filter and numa resouce tracker work well enough that i think the benifit of track pci devices or cpus in placement without numa outways the beniftis of numa in placment10:15
gibisean-k-mooney: I think, either we need to put big focus on the numa-in-placement feature or focus on the small improvements that already possible, but both cannot be done in the same cycle10:16
gibisean-k-mooney: I can get behind both effort as at the end of the they I would like to see both done10:17
gibijust dont want to overcommit10:17
sean-k-mooneygibi: ya that was the same conclution that i came to when talking to some of the redhat folks internally10:17
sean-k-mooneybut we also didnt know what the comunity wanted10:17
sean-k-mooneyi think this is already listed as a ptg topic but i hope we will as you said decied to either do numa or a set of other small things but not both10:18
sean-k-mooneyin this cycle10:18
sean-k-mooneygibi: related to your move operation work i wanted to also highlight this patch form bauzas https://review.openstack.org/#/c/589085/10:19
sean-k-mooneygibi: one of the issue the move operation will have is the fact that resize/migrate does not have acess to allocation candiaties. bauzas started to fix it and stopped for his vGPU work10:20
sean-k-mooneyim hoping if we fix this generically we can acclearate move operation for both vGPU and bandwith based schduleing10:21
sean-k-mooneyi know there are other gaps but i think there will be a lot fo synergey between those features10:22
gibisean-k-mooney: thanks for the link. I need to sit down and plan the support for move operations. There I will use what bauzas discovered already10:22
bauzasgibi: sean-k-mooney: yeah, I was planning to update the change later10:22
*** melwitt has quit IRC10:26
*** melwitt has joined #openstack-nova10:32
*** hoonetorg has quit IRC10:39
*** zhubx has quit IRC10:41
*** zhubx has joined #openstack-nova10:42
*** sidx64 has joined #openstack-nova10:42
gibieasy +A (efried already +2d) https://review.openstack.org/#/c/638847/10:42
*** sidx64 has quit IRC10:52
*** hoonetorg has joined #openstack-nova10:52
*** maciejjozefczyk has joined #openstack-nova10:53
*** sidx64 has joined #openstack-nova10:54
*** tiendc has quit IRC10:54
*** sapd1_x has joined #openstack-nova10:55
*** ttsiouts has quit IRC10:57
*** zhubx has quit IRC10:57
*** ttsiouts has joined #openstack-nova10:57
*** zhubx has joined #openstack-nova10:57
*** _alastor_ has joined #openstack-nova10:58
*** mdbooth has quit IRC10:59
*** udesale has quit IRC11:01
*** ttsiouts has quit IRC11:02
*** _alastor_ has quit IRC11:02
bauzasgibi: done11:05
*** mvkr has joined #openstack-nova11:09
*** sidx64 has quit IRC11:11
*** sidx64 has joined #openstack-nova11:16
*** janki has quit IRC11:18
*** cdent has quit IRC11:21
openstackgerritSilvan Kaiser proposed openstack/nova master: Adds systemd detection result caching in Quobyte driver  https://review.openstack.org/64809311:23
openstackgerritzhaixiaojun proposed openstack/nova master: Remove flavor id and name validation code because of jsonschema validation.The jsonschema validation was added with Ieba96718264ad2ddfba63b65425f7e5bbb8606a9.  https://review.openstack.org/63815011:24
openstackgerritSilvan Kaiser proposed openstack/nova master: WIP: Adds systemd detection result caching in Quobyte driver  https://review.openstack.org/64809311:25
*** sidx64 has quit IRC11:25
*** cdent has joined #openstack-nova11:27
openstackgerritRodolfo Alonso Hernandez proposed openstack/os-vif master: Remove IP proxy methods  https://review.openstack.org/64311511:27
*** mdbooth has joined #openstack-nova11:33
*** mdbooth has quit IRC11:34
*** mdbooth has joined #openstack-nova11:43
*** janki has joined #openstack-nova11:43
*** sidx64 has joined #openstack-nova11:43
openstackgerritSilvan Kaiser proposed openstack/nova master: WIP: Adds systemd detection result caching in Quobyte driver  https://review.openstack.org/64809311:50
*** weshay_pto is now known as weshay11:59
*** artom has joined #openstack-nova12:02
gibibauzas: thanks12:02
*** artom has quit IRC12:02
*** artom has joined #openstack-nova12:03
*** sidx64 has quit IRC12:06
*** ttsiouts has joined #openstack-nova12:08
*** sidx64 has joined #openstack-nova12:09
*** whoami-rajat has quit IRC12:10
*** markvoelker has quit IRC12:20
*** whoami-rajat has joined #openstack-nova12:21
*** ratailor has quit IRC12:22
*** lbragstad has joined #openstack-nova12:30
*** gryf has joined #openstack-nova12:32
*** sidx64 has quit IRC12:35
mdboothstephenfin: Left a comment on https://review.openstack.org/#/c/647831/12:40
mdboothstephenfin: That second line is belt and braces, which normally I'd be in favour of.12:41
*** sidx64 has joined #openstack-nova12:45
*** erlon has joined #openstack-nova12:46
* cdent suddenly feels insecurely dressed12:46
*** sidx64 has quit IRC12:46
*** mchlumsky has joined #openstack-nova12:47
mdboothcdent: lol12:47
*** lbragstad has quit IRC12:49
*** wolverineav has joined #openstack-nova12:49
*** udesale has joined #openstack-nova12:50
*** sidx64 has joined #openstack-nova12:51
*** wolverineav has quit IRC12:54
dansmithtssurya: right, but what does that matter? vs. literally every client having to change if you change the type...12:57
*** ileixe has quit IRC13:00
*** lbragstad has joined #openstack-nova13:02
*** Luzi has joined #openstack-nova13:03
*** erlon has quit IRC13:06
openstackgerritMerged openstack/nova master: doc: Fix openstack CLI command  https://review.openstack.org/64829913:07
openstackgerritMerged openstack/nova master: s,git://github.com/,https://git.openstack.org/,  https://review.openstack.org/64747213:07
openstackgerritMerged openstack/nova master: Speed up test_report  https://review.openstack.org/64376013:07
*** lbragstad has quit IRC13:07
*** ttsiouts has quit IRC13:13
*** ttsiouts has joined #openstack-nova13:13
*** lbragstad has joined #openstack-nova13:17
*** gbarros has joined #openstack-nova13:17
*** sidx64 has quit IRC13:17
*** jhesketh has quit IRC13:18
*** ttsiouts has quit IRC13:18
*** munimeha1 has joined #openstack-nova13:20
*** derekh has quit IRC13:24
*** lpetrut has quit IRC13:25
*** itlinux has quit IRC13:26
*** erlon has joined #openstack-nova13:30
*** imacdonn has quit IRC13:33
openstackgerritBalazs Gibizer proposed openstack/nova master: nova-manage: heal port allocations  https://review.openstack.org/63795513:35
openstackgerritBalazs Gibizer proposed openstack/nova master: cache neutron ports in heal allocation  https://review.openstack.org/63820713:35
tssuryadansmith: fine with me then :) there was also an opinion earlier on that the boolean "locked" then becomes redundant as its just determined based on locked_by being NULL or not13:37
tssuryabut I personally don't have a strong opinion on the way its revealed13:37
*** ttsiouts has joined #openstack-nova13:40
*** udesale has quit IRC13:41
*** udesale has joined #openstack-nova13:41
openstackgerritMerged openstack/nova master: Add retry_on_deadlock to migration_update DB API  https://review.openstack.org/63307513:48
openstackgerritMerged openstack/nova master: Change the TODO to NOTE about instance multi-create  https://review.openstack.org/63884713:48
*** ttsiouts has quit IRC13:50
*** ttsiouts has joined #openstack-nova13:51
*** gbarros has quit IRC13:53
openstackgerritAbhishek Kekane proposed openstack/nova-specs master: Support multiple backend of Glance  https://review.openstack.org/64121013:55
*** ttsiouts has quit IRC13:55
*** awaugama has joined #openstack-nova13:56
openstackgerritMatt Riedemann proposed openstack/nova stable/stein: Add functional recreate test for bug 1819963  https://review.openstack.org/64840113:58
openstackbug 1819963 in OpenStack Compute (nova) rocky "Reverting a resize does not update the instance.availability_zone value to the source az" [Medium,Confirmed] https://launchpad.net/bugs/181996313:58
openstackgerritMatt Riedemann proposed openstack/nova stable/stein: Update instance.availability_zone on revertResize  https://review.openstack.org/64840213:58
*** dpawlik has quit IRC13:58
gibiefried: how do you feel about the nova team meetings in the EU timezone? If you need help running it then let me know and I can do it.14:01
*** ttsiouts has joined #openstack-nova14:01
efriedgibi: I assumed I would be running all of them at this point, but I wouldn't say no if you're itching to help :)14:02
gibiefried: it was an old reflex last week as we split this work with melwitt. But I see that you are in a better TZ to run both14:02
*** derekh has joined #openstack-nova14:04
efriedgibi: Yes, 8 or 9am for me depending on DST.14:05
gibiefried: I go forward with the assumption that you will run the EU meetings but you can always ask for help14:05
efriedsounds good, thanks.14:05
gibiefried: a similar question regrading nova onboarding in Denver. Do you need help?14:05
efriedgibi: absolutely!14:05
efriedHere's where I will eagerly take all the help I can get.14:06
*** mlavalle has joined #openstack-nova14:06
efriedI'm never sure what form onboarding should take14:06
efriedthe audience always seems so diverse in terms of background/experience14:06
efriedI was toying with the idea of doing it as a live demo of developing a bug fix.14:07
efriedSo like, look at the bug, write and upload a test case patch, write and upload the fix patch14:07
efriedwhich of course is your wheelhouse completely.14:08
gibiefried: we migth be able to do that, but it needs to be a canned bug / bugfix to make sure we can really fix it14:08
gibiin the given time14:09
efriedgibi: Yeah, I thought maybe prepare by uploading a patch that reverts a known bug fix (and its test case) and then basically redo the work on top of it.14:09
efriedSo it would be a three-patch series, but we only talk about two of the patches.14:09
efriedand abandon all three at the end.14:09
gibiefried: good idea14:09
efriedneed to pick a bug that's pretty nova-y; otherwise it really just turns into a demo of "best practices for fixing a bug in the gerrit workflow"14:10
efriedfocus attention on the flow of the test case and the nature of the fix.14:10
gibipci? bdm? live migration? :)14:11
gibithe heavy nova stuff14:11
efriedsomething with a test case that's not like hundreds of lines long.14:11
*** mvkr has quit IRC14:12
*** gbarros has joined #openstack-nova14:12
* gibi looking at the git log of the functional test directory of nova14:12
gibiefried: this might be too heavy on test side https://review.openstack.org/#/c/624444/5/nova/tests/functional/libvirt/test_pci_sriov_servers.py14:14
gibiefried: maybe this https://review.openstack.org/#/c/640302/14:15
gibismall fix, on the API with API sample test14:15
efriedgibi: Well, in that pci/sriov one the delta is bigger than I would like, but if we used it for the demo, it would be just L243-8814:16
efriedthat second one doesn't really have a clear [test case] and [fix]14:17
gibiefried: it is api sample so it is well automataed14:17
*** ttsiouts has quit IRC14:17
gibibut I hear you14:18
*** ttsiouts has joined #openstack-nova14:18
cdentdo we still have people who actively understand and like the api sample tests?14:18
efriedI'm looking for commits whose title includes the word 'recreate'14:19
openstackgerritSilvan Kaiser proposed openstack/nova master: WIP: Adds systemd detection result caching in Quobyte driver  https://review.openstack.org/64809314:19
*** Zara has left #openstack-nova14:19
*** Zara has joined #openstack-nova14:20
gibicdent: at least I would not be affraid to dig into it if needed14:20
gibiefried: what about https://review.openstack.org/#/c/632904/3/nova/tests/functional/test_servers.py14:20
openstackgerritMatt Riedemann proposed openstack/nova stable/rocky: Add functional recreate test for bug 1819963  https://review.openstack.org/64840914:21
openstackbug 1819963 in OpenStack Compute (nova) rocky "Reverting a resize does not update the instance.availability_zone value to the source az" [Medium,Confirmed] https://launchpad.net/bugs/181996314:21
openstackgerritMatt Riedemann proposed openstack/nova stable/rocky: Update instance.availability_zone on revertResize  https://review.openstack.org/64841014:21
openstackgerritSilvan Kaiser proposed openstack/nova master: Adds systemd detection result caching in Quobyte driver  https://review.openstack.org/64809314:21
efriedgibi: heh, what about --^14:21
gibiefried: could be. I feel https://review.openstack.org/#/c/632904/3/nova/tests/functional/test_servers.py is more self contained14:22
*** ttsiouts has quit IRC14:22
efriedgibi: I was being lazy and looking for one where the recreate was separate from the fix, but it would be easy enough to split out.14:23
efriedagain with https://review.openstack.org/#/c/648409/1/nova/tests/functional/test_availability_zones.py we wouldn't have to include the refactor part in the demo, just L145-7314:23
gibiefried: OK. you convinced me14:24
kaisersefried: FYI, added the requested tests/nit-corrections to  https://review.openstack.org/648093 , pls check if that fits your view14:24
efriedkaisers: ack14:25
openstackgerritTakashi NATSUME proposed openstack/nova stable/stein: doc: Fix openstack CLI command  https://review.openstack.org/64841214:26
*** awalende has quit IRC14:27
*** awalende has joined #openstack-nova14:27
openstackgerritMatt Riedemann proposed openstack/nova stable/queens: Add functional recreate test for bug 1819963  https://review.openstack.org/64841414:28
openstackbug 1819963 in OpenStack Compute (nova) rocky "Reverting a resize does not update the instance.availability_zone value to the source az" [Medium,In progress] https://launchpad.net/bugs/1819963 - Assigned to Matt Riedemann (mriedem)14:28
openstackgerritMatt Riedemann proposed openstack/nova stable/queens: Update instance.availability_zone on revertResize  https://review.openstack.org/64841514:28
*** ttsiouts has joined #openstack-nova14:29
*** whoami-rajat has quit IRC14:30
Zarahi! I have a question; aiui when disk space is overcommitted, disk_available_least shows a negative value (eg: with `openstack hypervisor stats show`). Why is a negative value displayed instead of '0'? It seems like the value could be infinitely low, so I'm wondering how the information would then be used?14:31
openstackgerritMatt Riedemann proposed openstack/nova stable/queens: Update instance.availability_zone on revertResize  https://review.openstack.org/64841514:31
*** awalende has quit IRC14:32
efriedZara: Just spitballing here, but I assume it gives you an indication of how overcommitted you are.14:32
sean-k-mooneyZara: it means that the total amount of worse case diskspace for the current instance on the host exceed the availble diskspace14:32
sean-k-mooneyefried: yes it does14:32
*** maciejjozefczyk has quit IRC14:32
sean-k-mooneyZara: i found this really good github gist that explains it https://gist.github.com/JCallicoat/43505cab0535057ca4fb14:33
efriedarguably this could be "fixed" to floor out at zero, but that would require a new microversion, so it probably hasn't been considered worth the hassle since you could do that on the client side if you don't care about the "extra information".14:33
sean-k-mooneywe porably shoudl have somthing similar in our docs14:33
sean-k-mooneyefried: i dont think this is a bug14:33
sean-k-mooneyit was by design14:33
openstackgerritRodolfo Alonso Hernandez proposed openstack/os-vif master: Remove IP proxy methods  https://review.openstack.org/64311514:34
efriedRight, right, it's a feature, not a bug.14:34
sean-k-mooneyefried: that said this was origianly used by the disk filters i nthe scheduler but we do that in placmenet now14:34
sean-k-mooneyorperhaps it was the disk wheigher14:35
Zarathanks for the gist (yeah, that says nova-scheduler).14:35
*** phasespace has quit IRC14:36
sean-k-mooneyhum the weigher actully used free_disk_mb14:36
ZaraAn official docs note somewhere would be good; I encountered it in the context of someone thinking it was a bug.14:37
*** yan0s has joined #openstack-nova14:37
*** mvkr has joined #openstack-nova14:37
openstackgerritMatt Riedemann proposed openstack/nova stable/pike: Add functional recreate test for bug 1819963  https://review.openstack.org/64842114:39
openstackbug 1819963 in OpenStack Compute (nova) rocky "Reverting a resize does not update the instance.availability_zone value to the source az" [Medium,In progress] https://launchpad.net/bugs/1819963 - Assigned to Matt Riedemann (mriedem)14:39
openstackgerritMatt Riedemann proposed openstack/nova stable/pike: Update instance.availability_zone on revertResize  https://review.openstack.org/64842214:39
*** hongbin has joined #openstack-nova14:39
sean-k-mooneylookin futher we do https://github.com/openstack/nova/blob/9db1c6061d10617b40dbc1d0d64faed79503cb81/nova/scheduler/host_manager.py#L213 so we set free_gb in the context of the host state object to the smalles to free_gb or lesast_gb14:39
sean-k-mooneyso yes the disk weighter will prefer host that are less over commited14:40
sean-k-mooneyhttps://github.com/openstack/nova/blob/master/nova/scheduler/weights/disk.py14:40
sean-k-mooneywhich is why we have the negitive value14:40
openstackgerritTakashi NATSUME proposed openstack/nova stable/rocky: doc: Fix openstack CLI command  https://review.openstack.org/64842514:41
Zarasean-k-mooney: ah, I see, thanks14:43
efriedZara: It would be great if you wanted to find the appropriate spot in the docs and propose a patch to enhance it :)14:44
Zaraefried: I'll see what I can do. :)14:46
openstackgerritMatt Riedemann proposed openstack/nova stable/stein: Add retry_on_deadlock to migration_update DB API  https://review.openstack.org/64842814:46
*** itlinux has joined #openstack-nova14:47
efriedZara: Thanks!14:56
*** cfriesen has joined #openstack-nova14:58
*** edleafe_ has joined #openstack-nova14:59
*** _alastor_ has joined #openstack-nova15:00
*** gbarros has quit IRC15:04
*** hamzy has quit IRC15:10
*** edleafe_ has quit IRC15:17
*** maciejjozefczyk has joined #openstack-nova15:21
*** erlon has quit IRC15:23
Zara(aw, I found a nice place to add it... but the section is in the ocata docs; I haven't yet found the correlate of those docs for more recent releases (https://docs.openstack.org/ocata/config-reference/compute/schedulers.html )15:23
sean-k-mooneyZara: we generate our config docmuemnt from code now15:24
amodiwhat was the spam about? :?15:25
*** Luzi has quit IRC15:25
Zaraah okay, guess I'll need to add a docstring somewhere15:25
sean-k-mooneyZara: https://github.com/openstack/nova/blob/master/nova/conf/scheduler.py15:25
Zarasean-k-mooney: thanks15:25
sean-k-mooneywe do have ohter schduler docs somewhere actully15:26
sean-k-mooneyZara: i think this is what you want https://github.com/openstack/nova/blob/master/doc/source/admin/configuration/schedulers.rst15:27
*** jhesketh has joined #openstack-nova15:28
*** jhesketh has quit IRC15:34
Zaraseank-k-mooney: ahaaa, yes that looks like the same doc15:34
Zarathanks15:34
*** hamzy has joined #openstack-nova15:35
*** wolverineav has joined #openstack-nova15:38
*** wolverineav has quit IRC15:42
*** bbowen has joined #openstack-nova15:51
openstackgerritDan Smith proposed openstack/os-traits master: Add COMPUTE_IMAGE_TYPE_* traits  https://review.openstack.org/64814715:54
*** sapd1_x has quit IRC15:55
*** ricolin has joined #openstack-nova15:58
*** udesale has quit IRC16:01
ricolincdent, jaypipes, zhipeng do you think you guys can help with http://lists.openstack.org/pipermail/openstack-discuss/2019-March/004246.html for Resource Management SIG?16:02
cdentricolin: there hasn't been any resource mgt sig activity, well, ever16:02
cdentjust hope16:02
cdentzhipeng may have some thoughts, but though we wanted to we never really got it off the ground16:02
*** erlon has joined #openstack-nova16:03
ricolincdent, is there any plan for mgt sig in Summit or PTG this time?16:03
cdentnot that I'm aware of16:03
cdentbut again, zhipeng may have more info16:04
*** mrch_ has quit IRC16:05
*** janki has quit IRC16:07
*** gyee has joined #openstack-nova16:12
*** yan0s has quit IRC16:15
*** wolverineav has joined #openstack-nova16:18
openstackgerritZara proposed openstack/nova master: Explain why disk_available_least can be negative  https://review.openstack.org/64845416:19
ricolincdent, thanks, will try to reach out with zhipeng16:21
Zara(my patch may be a bit brief, but I guess it's a start)16:22
openstackgerritMerged openstack/os-traits master: Add COMPUTE_IMAGE_TYPE_* traits  https://review.openstack.org/64814716:22
*** ttsiouts has quit IRC16:24
*** tssurya has quit IRC16:27
*** gbarros has joined #openstack-nova16:31
*** whoami-rajat has joined #openstack-nova16:32
*** wolverineav has quit IRC16:37
*** helenafm has quit IRC16:40
openstackgerritEric Fried proposed openstack/os-traits master: Add contextual comment about image type traits  https://review.openstack.org/64846616:46
*** smcginnis is now known as smcginnis_pto16:48
*** gbarros has quit IRC16:49
*** luksky has quit IRC16:49
*** sidx64 has joined #openstack-nova16:55
*** erlon has quit IRC16:57
*** awaugama_ has joined #openstack-nova16:59
*** awaugama has quit IRC17:00
*** psachin has quit IRC17:05
efriedstephenfin: Feel free to carry my +2 forward if kaisers respins17:06
stephenfinefried: I might just do it myself if that suits you?17:06
efriedsure17:06
stephenfinmriedem-like17:06
openstackgerritStephen Finucane proposed openstack/nova master: Adds systemd detection result caching in Quobyte driver  https://review.openstack.org/64809317:08
openstackgerritEric Fried proposed openstack/nova master: WIP: Poison sleep() in tests  https://review.openstack.org/64376317:08
cdentsean-k-mooney: whatever happened about your lower-constraints fixes? I'm trying to decide if my version still matters https://review.openstack.org/#/c/622972/ . If not I'd like to forget about it.17:09
stephenfindone-zo17:09
*** tesseract has quit IRC17:09
*** sidx64_ has joined #openstack-nova17:10
*** igordc has joined #openstack-nova17:11
*** sidx64 has quit IRC17:12
sean-k-mooneycdent: i got a little side tracked. it was ment to be the thing i worked on today but that did not happen :(17:12
sean-k-mooneycdent: ill pull down yours and test it locally and see if that still works.17:13
sean-k-mooneycdent: i could be wrong but i assume if i rebased yours it would have teh same rabbitmq issues17:14
cdentsean-k-mooney:  I can't remember the issue you're talking about17:16
sean-k-mooneycdent: when using lower constraint in my version a bunch of the unit tests end up using the amqp oslo messageing driver and actully try to connect to rabbitmq17:17
openstackgerritMerged openstack/os-traits master: Add contextual comment about image type traits  https://review.openstack.org/64846617:18
openstackgerritBalazs Gibizer proposed openstack/nova master: nova-manage: heal port allocations  https://review.openstack.org/63795517:19
openstackgerritBalazs Gibizer proposed openstack/nova master: cache neutron ports in heal allocation  https://review.openstack.org/63820717:19
sean-k-mooneycdent: you bumped oslo.messaging to 7.0.0 in your and i didnt bump it in min so i think that is the reason17:20
sean-k-mooneyyour version seams to be working but you have more version change then i have. just waiting for it too finish locally.17:23
*** ccamacho has quit IRC17:23
openstackgerritZara proposed openstack/nova master: Explain why disk_available_least can be negative  https://review.openstack.org/64845417:23
*** rpittau is now known as rpittau|afk17:24
sean-k-mooneyZara: thanks it was only a nit so you didnt have to respin but that does look fien to me17:24
Zaraheh, easy enough to edit inline :)17:25
*** ricolin has quit IRC17:26
*** mvkr has quit IRC17:31
*** ricolin has joined #openstack-nova17:38
openstackgerritGhanshyam Mann proposed openstack/nova master: WIP: Add new default roles in os-services API policies  https://review.openstack.org/64848017:42
*** awaugama_ has quit IRC17:43
openstackgerritGhanshyam Mann proposed openstack/nova master: WIP:Introduce scope_types in os-services, lock policies  https://review.openstack.org/64545217:43
*** awaugama_ has joined #openstack-nova17:43
openstackgerritGhanshyam Mann proposed openstack/nova master: WIP: Add new default roles in os-services API policies  https://review.openstack.org/64848017:43
*** wolverineav has joined #openstack-nova17:44
*** wolverineav has quit IRC17:44
*** wolverineav has joined #openstack-nova17:46
*** ricolin has quit IRC17:46
*** itlinux has quit IRC17:46
*** itlinux has joined #openstack-nova17:46
*** wolverineav has quit IRC17:51
*** wolverineav has joined #openstack-nova17:52
*** wolverineav has quit IRC17:52
*** wolverineav has joined #openstack-nova17:53
*** wolverineav has quit IRC17:53
*** wolverineav has joined #openstack-nova17:53
*** Sundar has joined #openstack-nova17:57
*** awaugama_ has quit IRC17:59
*** wolverineav has quit IRC18:00
*** sridharg has quit IRC18:00
*** wolverineav has joined #openstack-nova18:01
*** wolverineav has quit IRC18:01
*** wolverineav has joined #openstack-nova18:01
*** derekh has quit IRC18:02
*** dtantsur is now known as dtantsur|afk18:08
*** sridharg has joined #openstack-nova18:17
*** mrch_ has joined #openstack-nova18:25
*** ralonsoh has quit IRC18:31
*** MasterofJOKers has quit IRC18:52
*** MasterofJOKers has joined #openstack-nova18:52
openstackgerritMerged openstack/nova master: Temporarily mutate migration object in finish_revert_resize  https://review.openstack.org/64113719:02
*** wolverineav has quit IRC19:07
*** wolverineav has joined #openstack-nova19:08
*** wolverineav has quit IRC19:13
openstackgerritMerged openstack/nova master: De-cruftify the finish_resize methods  https://review.openstack.org/63534919:15
*** luksky has joined #openstack-nova19:26
*** priteau has quit IRC19:27
openstackgerritEric Fried proposed openstack/nova master: Error out migration when confirm_resize fails  https://review.openstack.org/64754619:34
openstackgerritEric Fried proposed openstack/nova master: Delete allocations even if _confirm_resize raises  https://review.openstack.org/64756619:34
openstackgerritEric Fried proposed openstack/nova master: api-ref: add more details to confirmResize troubleshooting  https://review.openstack.org/64774119:34
*** rcernin has quit IRC19:36
*** sridharg has quit IRC19:40
*** wolverineav has joined #openstack-nova19:48
*** wolverineav has quit IRC19:53
*** jaypipes_ has joined #openstack-nova19:54
*** jaypipes has quit IRC19:54
*** mchlumsky has quit IRC19:58
*** wolverineav has joined #openstack-nova20:00
*** zhubx has quit IRC20:01
*** zhubx has joined #openstack-nova20:02
*** bbowen has quit IRC20:05
*** pcaruana has quit IRC20:08
*** Vek has quit IRC20:12
*** efried has quit IRC20:22
*** efried has joined #openstack-nova20:26
*** cdent has quit IRC20:27
openstackgerritMerged openstack/nova master: Explain why disk_available_least can be negative  https://review.openstack.org/64845420:28
*** wolverineav has quit IRC20:32
*** wolverineav has joined #openstack-nova20:33
*** eharney has quit IRC20:37
*** wolverineav has quit IRC20:37
*** efried has quit IRC20:44
*** wolverineav has joined #openstack-nova20:50
*** takashin has joined #openstack-nova20:54
*** hamzy has quit IRC20:54
*** efried has joined #openstack-nova20:58
*** itlinux has quit IRC20:59
openstackgerritmelanie witt proposed openstack/nova master: libvirt: set device address tag only if setting disk unit  https://review.openstack.org/61197420:59
*** openstackgerrit has quit IRC21:07
*** efried has quit IRC21:09
*** openstackgerrit has joined #openstack-nova21:20
openstackgerritGhanshyam Mann proposed openstack/nova-specs master: Spec for API policy updates  https://review.openstack.org/54785021:20
*** efried has joined #openstack-nova21:23
gmannone quick update is about API policy improvement proposal'21:23
gmann#link https://review.openstack.org/#/c/547850/21:24
*** wolverineav has quit IRC21:24
gmannthis is spec ^^ and added the PTG item discussion also (hope we can schedule the sessions to avoid conflict QA/TC related time)21:24
gmannthat's all from my side21:25
*** wolverineav has joined #openstack-nova21:25
gmannops this was for meeting21:25
gmannmelwitt:  i have updated spec with PoC https://review.openstack.org/#/c/547850/21:26
gmannand the flow of patches it will looks like with complete series21:27
*** wolverineav has quit IRC21:30
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in virt/test_block_device.py  https://review.openstack.org/56615321:35
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove deprecated 'default_flavor' config option  https://review.openstack.org/64547621:35
*** wolverineav has joined #openstack-nova21:40
*** rcernin has joined #openstack-nova21:44
openstackgerritEric Fried proposed openstack/nova master: Adds systemd detection result caching in Quobyte driver  https://review.openstack.org/64809321:45
*** rcernin has quit IRC21:55
*** sidx64_ has quit IRC22:01
*** rcernin has joined #openstack-nova22:04
*** wolverineav has quit IRC22:08
*** wolverineav has joined #openstack-nova22:10
*** tbachman has joined #openstack-nova22:12
*** wolverineav has quit IRC22:14
*** munimeha1 has quit IRC22:20
*** igordc has quit IRC22:22
*** awalende has joined #openstack-nova22:28
gmannjohnthetubaguy: lbragstad updated the policy spec with PoC examples - https://review.openstack.org/#/c/547850/22:29
*** awalende has quit IRC22:33
*** wolverineav has joined #openstack-nova22:40
openstackgerritEric Fried proposed openstack/nova master: Poison sleep() in tests  https://review.openstack.org/64376322:44
efriedman, that sucked22:44
*** wolverineav has quit IRC22:45
*** erlon has joined #openstack-nova22:51
*** whoami-rajat has quit IRC22:52
*** jding1_ has joined #openstack-nova22:54
*** wolverineav has joined #openstack-nova22:56
*** jackding has quit IRC22:57
*** tkajinam has joined #openstack-nova22:59
*** wolverineav has quit IRC22:59
*** wolverineav has joined #openstack-nova23:00
*** wolverineav has quit IRC23:01
*** wolverineav has joined #openstack-nova23:01
openstackgerritMerged openstack/python-novaclient stable/queens: Fix up userdata argument to rebuild.  https://review.openstack.org/61309023:02
openstackgerritMerged openstack/python-novaclient stable/queens: Follow up "Fix up userdata argument to rebuild"  https://review.openstack.org/61309123:02
*** hongbin has quit IRC23:02
*** luksky has quit IRC23:05
*** _alastor_ has quit IRC23:07
*** wolverineav has quit IRC23:07
*** _alastor_ has joined #openstack-nova23:07
*** wolverineav has joined #openstack-nova23:08
*** wolverineav has quit IRC23:09
*** wolverineav has joined #openstack-nova23:09
*** Sundar has quit IRC23:15
*** slaweq has quit IRC23:17
*** takashin has quit IRC23:20
*** gbarros has joined #openstack-nova23:21
*** gbarros has quit IRC23:43
*** gyee has quit IRC23:57
*** jhesketh has joined #openstack-nova23:58

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