Wednesday, 2019-05-15

*** mvkr has quit IRC00:00
*** hamzy has joined #openstack-nova00:00
*** gyee has quit IRC00:18
*** ganso has quit IRC00:21
*** betherly has joined #openstack-nova00:23
*** ganso has joined #openstack-nova00:24
*** brinzhang has joined #openstack-nova00:26
*** betherly has quit IRC00:27
*** BjoernT has joined #openstack-nova00:28
*** dklyle has joined #openstack-nova00:30
*** BjoernT has quit IRC00:35
*** BjoernT has joined #openstack-nova00:35
*** mriedem has quit IRC00:41
*** ricolin has joined #openstack-nova00:48
*** BjoernT has quit IRC00:50
*** bhagyashris has joined #openstack-nova00:58
*** BjoernT has joined #openstack-nova00:58
*** JamesBenson has joined #openstack-nova00:59
*** BjoernT has quit IRC01:02
*** JamesBenson has quit IRC01:05
*** rcernin has quit IRC01:16
*** rcernin has joined #openstack-nova01:17
*** rcernin has quit IRC01:18
*** rcernin has joined #openstack-nova01:19
*** BjoernT has joined #openstack-nova01:28
*** ttsiouts has joined #openstack-nova01:29
*** whoami-rajat has joined #openstack-nova01:34
*** betherly has joined #openstack-nova01:34
*** macza has joined #openstack-nova01:35
*** macza has quit IRC01:39
*** betherly has quit IRC01:40
*** nicolasbock has quit IRC01:42
*** boxiang has joined #openstack-nova01:48
*** tetsuro has joined #openstack-nova01:49
openstackgerritBoxiang Zhu proposed openstack/nova master: Fix failure to boot instances with qcow2 format images  https://review.opendev.org/64027101:54
*** ttsiouts has quit IRC02:02
openstackgerritMerged openstack/nova master: Add zvm driver image type capabilities  https://review.opendev.org/65573202:05
*** cfriesen has quit IRC02:05
*** betherly has joined #openstack-nova02:06
*** betherly has quit IRC02:10
openstackgerritBrin Zhang proposed openstack/nova master: Cap sphinx for py2 to match global requirements  https://review.opendev.org/65920102:26
openstackgerritBrin Zhang proposed openstack/python-novaclient master: Cap sphinx for py2 to match global requirements  https://review.opendev.org/65920202:30
openstackgerritMerged openstack/nova master: Disable limit if affinity(anti)/same(different)host is requested  https://review.opendev.org/65811002:35
*** tetsuro_ has joined #openstack-nova02:37
*** betherly has joined #openstack-nova02:37
*** yedongcan has joined #openstack-nova02:37
*** hongbin has joined #openstack-nova02:39
*** tetsuro has quit IRC02:39
*** betherly has quit IRC02:41
*** tetsuro has joined #openstack-nova02:42
openstackgerritBrin Zhang proposed openstack/nova-specs master: Cap sphinx for py2 to match global requirements  https://review.opendev.org/65920502:44
*** tetsuro_ has quit IRC02:46
*** bbowen has quit IRC02:46
openstackgerritTetsuro Nakamura proposed openstack/nova stable/stein: Skip _exclude_nested_providers() if not nested  https://review.opendev.org/65920602:46
openstackgerritTetsuro Nakamura proposed openstack/nova stable/rocky: Skip _exclude_nested_providers() if not nested  https://review.opendev.org/65920702:51
*** tkajinam has quit IRC02:55
*** tkajinam has joined #openstack-nova02:56
*** betherly has joined #openstack-nova02:57
openstackgerritBoxiang Zhu proposed openstack/nova master: Add host and hypervisor_hostname flag to create server  https://review.opendev.org/64552002:58
*** betherly has quit IRC03:02
*** bbowen has joined #openstack-nova03:02
*** BjoernT has quit IRC03:15
*** hongbin has quit IRC03:18
*** betherly has joined #openstack-nova03:28
*** betherly has quit IRC03:33
*** cfriesen has joined #openstack-nova03:38
openstackgerritBrin Zhang proposed openstack/python-novaclient master: Cap sphinx for py2 to match global requirements  https://review.opendev.org/65920203:43
*** tetsuro has quit IRC03:50
*** tetsuro has joined #openstack-nova03:52
*** hemna has joined #openstack-nova03:53
*** psachin has joined #openstack-nova03:56
*** udesale has joined #openstack-nova03:58
*** betherly has joined #openstack-nova03:59
*** ttsiouts has joined #openstack-nova03:59
*** cfriesen has quit IRC04:01
*** betherly has quit IRC04:04
*** ttsiouts has quit IRC04:31
*** hemna has quit IRC04:36
*** tetsuro has quit IRC04:37
*** mvkr has joined #openstack-nova04:37
*** abhishekk has joined #openstack-nova04:42
*** janki has joined #openstack-nova04:43
*** Luzi has joined #openstack-nova04:46
*** betherly has joined #openstack-nova04:51
*** brinzh has joined #openstack-nova04:54
*** betherly has quit IRC04:56
*** brinzhang has quit IRC04:58
*** vincentchong1 has joined #openstack-nova05:24
*** alex_xu has quit IRC05:26
*** ratailor has joined #openstack-nova05:27
*** betherly has joined #openstack-nova05:32
*** betherly has quit IRC05:37
*** spsurya has joined #openstack-nova05:49
*** betherly has joined #openstack-nova05:53
*** psachin has quit IRC05:54
*** psachin has joined #openstack-nova05:57
*** betherly has quit IRC05:58
*** awalende has joined #openstack-nova06:04
*** awalende has quit IRC06:08
*** threestrands has joined #openstack-nova06:12
*** dpawlik has joined #openstack-nova06:23
*** lpetrut has joined #openstack-nova06:23
*** kaisers has joined #openstack-nova06:24
*** vincentchong1 has quit IRC06:24
*** vincentchong1 has joined #openstack-nova06:25
*** ttsiouts has joined #openstack-nova06:29
*** jaosorior has quit IRC06:42
*** mvkr has quit IRC06:43
*** luksky has joined #openstack-nova06:43
*** maciejjozefczyk has joined #openstack-nova06:48
*** jaosorior has joined #openstack-nova06:51
*** vincentchong1 has quit IRC06:52
*** awalende has joined #openstack-nova07:01
*** ttsiouts has quit IRC07:02
*** yonglihe has joined #openstack-nova07:04
*** awalende_ has joined #openstack-nova07:04
*** awalende has quit IRC07:05
*** abhishekk has quit IRC07:05
*** alex_xu has joined #openstack-nova07:05
openstackgerritBrin Zhang proposed openstack/python-novaclient master: Cap sphinx for py2 to match global requirements  https://review.opendev.org/65920207:06
*** rpittau|afk is now known as rpittau07:07
*** tssurya has joined #openstack-nova07:07
*** aarents has joined #openstack-nova07:08
*** tesseract has joined #openstack-nova07:13
*** awalende_ has quit IRC07:17
*** awalende has joined #openstack-nova07:17
*** helenafm has joined #openstack-nova07:18
*** awalende_ has joined #openstack-nova07:20
*** awalende has quit IRC07:22
*** awalende_ has quit IRC07:22
*** zhongjun2_ has joined #openstack-nova07:23
*** awalende has joined #openstack-nova07:25
*** rcernin has quit IRC07:25
*** zigo has quit IRC07:28
*** ttsiouts has joined #openstack-nova07:30
*** imacdonn has quit IRC07:47
*** imacdonn has joined #openstack-nova07:51
*** boxiang has quit IRC07:51
*** ivve has joined #openstack-nova07:54
sean-k-mooneybauzas: have you seen http://logs.openstack.org/97/652197/17/check/grenade-py3/7b3cac4/logs/grenade.sh.txt.gz#_2019-05-15_02_34_39_701 recently08:08
*** tkajinam has quit IRC08:08
sean-k-mooneyim not sure if our grenade job is broken after the change to os-dev08:08
sean-k-mooney*opendev08:08
*** awalende has quit IRC08:11
*** awalende has joined #openstack-nova08:11
*** awalende has quit IRC08:16
*** lennyb has joined #openstack-nova08:16
*** threestrands has quit IRC08:18
sean-k-mooneyya... 337 hits in the last 12 hours http://logstash.openstack.org/#/dashboard/file/logstash.json?query=message:%5C%22Could%20not%20install%20packages%20due%20to%20an%20EnvironmentError:%20HTTPSConnectionPool(host%3D'git.openstack.org',%20port%3D443)%5C%2208:19
bauzassean-k-mooney: nope, not yet08:23
*** awalende has joined #openstack-nova08:24
bauzassean-k-mooney: there is a redirect for https://git.openstack.org/cgit/openstack/requirements/plain/upper-constraints.txt08:25
sean-k-mooneyhttp://logstash.openstack.org/#/dashboard/file/logstash.json?query=message:%5C%22Could%20not%20install%20packages%20due%20to%20an%20EnvironmentError:%20HTTPSConnectionPool(host%3D'git.openstack.org',%20port%3D443)%5C%2208:25
bauzassean-k-mooney: so we shouldn't get the exception08:25
sean-k-mooneybauzas: we should not but there have been 337 filures since in the last week08:26
sean-k-mooneyso i think its not always working08:26
sean-k-mooneyi just mention this on the infra channel08:26
bauzasok08:26
sean-k-mooneyi might create an elastic recheck query and file a bug but im not sure where to file it08:26
bauzasgood question08:27
bauzaswe used to create bugs in Nova even if they were related to infra08:27
bauzasand we marked them as critical08:27
bauzaswe also had a tag for them08:27
sean-k-mooneyok i can do that. its affecting neutron jobs too but the grenade jobs seam to be the onse teh hit it most08:28
bauzassean-k-mooney: https://bugs.launchpad.net/nova/+bugs?field.tag=gate-failure08:28
sean-k-mooneyah cool ill file one and add the gate-failure tag08:29
*** ttsiouts has quit IRC08:34
*** dtantsur|afk is now known as dtantsur08:35
*** ralonsoh has joined #openstack-nova08:46
openstackgerritSurya Seetharaman proposed openstack/nova stable/stein: Disable limit if affinity(anti)/same(different)host is requested  https://review.opendev.org/65923908:48
sean-k-mooneybauzas: should i set it to critical by the way or just high. i have hit it twice in a row but i dont think its failing in all builds its just annoying08:49
bauzassean-k-mooney: on 1:1, sorry08:49
sean-k-mooneyno worries08:49
*** pcaruana has joined #openstack-nova08:51
*** derekh has joined #openstack-nova08:57
openstackgerritAlex Xu proposed openstack/nova master: Remove the request_destination check in the scheduler  https://review.opendev.org/65924308:58
alex_xu^ efried, is it something we can removed?08:59
*** luksky has quit IRC09:01
bhagyashrisefried, bauzas:  Hi, I just want to discuss about    http://logs.openstack.org/81/555081/28/check/openstack-tox-docs/6314f69/html/specs/train/approved/cpu-resources.html#configuration-options   regarding the reserved_host_cpu calculation for PCPU resource,09:02
bhagyashrisf I have total 16 cpus and 4 are assigned to cpu_dedicated_set and 2 are assign to cpu_shared_set then in that case reserved_cpus for PCPU resource class will be (16- (4+2)) = 10 and for VCPU resource class it will be 0 .09:03
bhagyashrisThen in that case if I request instance which requires 1 PCPU then  placement returns the 0 allocation_candiacted () (reason is it fails at _capacity_check_clause())09:03
bhagyashrisPlease correct me if I am wrong while calculating the reserved host cpu.09:03
kashyapsean-k-mooney: dansmith: Yesterday you said: just like how you don't want Nova to not land instances based on "fan speed, or cpu temperature, or firmware patch levels..." you also don't want to stop launching instances on Compute hosts with vulnerable hypervisors.09:04
openstackgerritSurya Seetharaman proposed openstack/nova stable/rocky: Disable limit if affinity(anti)/same(different)host is requested  https://review.opendev.org/65924609:04
kashyapsean-k-mooney: dansmith: Fan speed & CPU temp are really different, as they don't really affect the guest.09:05
*** jangutter has quit IRC09:06
bhagyashrissean-k-mooney: ^^09:07
openstackgerritSurya Seetharaman proposed openstack/nova stable/queens: Disable limit if affinity(anti)/same(different)host is requested  https://review.opendev.org/65924709:08
*** brinzhang has joined #openstack-nova09:08
*** brinzh has quit IRC09:10
sean-k-mooneybhagyashris: no09:23
sean-k-mooneyif you have 4 cpus in cpu_dedicated_set then that capasity of the inventory will b 4 and the resrved value will be 009:24
*** mdbooth_ has joined #openstack-nova09:25
sean-k-mooneysimilarly for vcpus the capasity will be defiened by the total number of cores defiened in the cpu_shared_set09:25
sean-k-mooneybhagyashris: so the reserved value will effectivly always be 0 unless you set it to something else via the api09:25
bhagyashrissean-k-mooney: How? but What formula says (set(all_cpus) - (set(dedicated) - set(shared)))09:26
sean-k-mooneythat is not waht the current version fo the spec say if i rememeber correctly09:27
bhagyashrisSo in my case the total cpu count on host is 16 and I have define dedicated set as 4 cpus and shared as 2 cpus09:27
bhagyashrisOk then in my case what will be the all_cpus value09:27
sean-k-mooneythen the inventory will be PCUP total 4 reserved 009:27
sean-k-mooneyno09:27
sean-k-mooneywe will only be reporting the consumable cpus to placement09:28
sean-k-mooneynot all the cpus on the plathform09:28
bhagyashrisok09:28
*** mdbooth has quit IRC09:28
sean-k-mooneylet me just double check the spec to ensure that is correct but i belive that is what we settled on09:29
bhagyashrisok check and let me help to understand09:29
bhagyashrisSorry help me to understand09:30
sean-k-mooneylooks like that has not been updated yet https://review.opendev.org/#/c/555081/28/specs/train/approved/cpu-resources.rst@20109:30
sean-k-mooneyill leave a comment09:30
sean-k-mooneystephenfin: ^ is the above correct09:30
bhagyashrisI am still confuse what will be the all_cpus value09:31
sean-k-mooneythere wont be one09:32
bhagyashrisif you check the Upgrade section there they have mentioned the formula09:32
sean-k-mooneyat least i dont think there shoudl be09:32
bhagyashrisTo calculate the reserved hot09:32
bhagyashrisHost cpu*09:32
sean-k-mooneythat is not present in the current verion as far as i can see09:33
*** mrch_ has joined #openstack-nova09:33
bhagyashrishttp://logs.openstack.org/81/555081/28/check/openstack-tox-docs/6314f69/html/specs/train/approved/cpu-resources.html#configuration-options09:34
bhagyashrisHere they have mentioned09:34
bhagyashrissean-k-mooney: ^^09:35
*** zigo has joined #openstack-nova09:36
sean-k-mooneyya i think that is in correct. this was one of the part of the spec that i really disliked but we still need to get agreement on that09:36
bhagyashrisYeah and there I am still unclear what will be the all_cpus value09:36
sean-k-mooneyi am suggstion there should not be one09:37
sean-k-mooneyi think it is incorrect to track cpus that cant be consumed in placement09:37
sean-k-mooneyit give an incorrect assement of your capastity09:37
bhagyashrisyeah09:39
bhagyashrisThen how will calculate the reserved value09:41
sean-k-mooneywe dont we always set it to 0 when creting the rp09:41
bhagyashrisOkay mean for both the PCPU and VCPU resource class inventory it will be 009:42
sean-k-mooneyyes09:42
bhagyashrisBut didn't get why this is added in spec09:43
sean-k-mooneythat is not what the spec says but i think we shoudl updated it to say that09:43
bhagyashrisAs I am implementing the spec so it's till bit confusing to me09:43
*** luksky has joined #openstack-nova09:43
sean-k-mooneywell we have changed the design a bit form last cycle09:43
sean-k-mooneythis was one of the point that there was still some disagrement on09:44
bhagyashrisOhh ok.09:45
bhagyashrisstephenfin: Hi, do you give me some suggestion and opinion about it09:46
sean-k-mooneyi think the UX of what was prosped in the spec is incrroect and i fully expect to get bug report because of it which is why i wanted to change it09:46
bhagyashrisRegarding the reserved_host_cpu calculation09:46
bhagyashrissean-k-mooney: Yeah , still waiting for final decision how will be the reserved value for cpus09:47
stephenfinsean-k-mooney: Yeah, that's a mistake that should be updated09:47
stephenfinreserved = 0 by default since we already have a way to "reserve" CPUs (by not including them in either mask)09:48
sean-k-mooneystephenfin: are we going to merge the spec as is and fix it in a follow up patch09:48
stephenfinsean-k-mooney: Yeah, I'd prefer that, personally09:48
stephenfinI'll draft the follow-up now09:49
sean-k-mooneyso whould i to finaly get it merged09:49
stephenfin'zactly09:49
stephenfinbhagyashris: Does that make sense/09:49
stephenfin*?09:49
bhagyashrissean-k-mooney, stephenfin: yes , so for both the VCPU and PCPU resource class inventory the reserved value will be 0 and will deprecate the configuration option anyways right?09:50
sean-k-mooneyyes we are deprecating resrved_host_cpus and sured really raise an error if its used with the new cpu_dedicated_set also set in the config09:53
bhagyashrissean-k-mooney, stephenfin: okay09:54
bhagyashrissean-k-mooney, stephenfin: Thank you for your inputs :)09:54
openstackgerritMatthew Booth proposed openstack/nova master: Fix retry of instance_update_and_get_original  https://review.opendev.org/65884509:56
*** bhagyashris has quit IRC09:56
*** dpawlik has quit IRC09:58
*** dpawlik has joined #openstack-nova10:03
*** faizy98 has joined #openstack-nova10:07
NewBrucehowdy all - as i mentioned last night, ive been digging into a few other issues in the migration process - hitting a wall, thinking about a bug report but figure its best to get a second opinion before doing so10:07
NewBrucelive migrations with volumes and volume-snapshots are failing - two errors10:08
NewBruce1. 'serial' is deprecated, please use the corresponding option of '-device’ instead10:08
NewBruce2. Could not open backing file: Could not open '/var/lib/nova/mnt/… : Permission denied:10:08
NewBrucein the first, an error due to deprecation is odd, but also according to libvirt docs, serial is only deprecated for type block and we have type ‘file’ in the domain xml10:09
*** tesseract has quit IRC10:12
*** tesseract has joined #openstack-nova10:12
openstackgerritMerged openstack/os-vif master: Fix Kuryr-Kubernetes job name  https://review.opendev.org/65649210:16
openstackgerritHamdy Khader proposed openstack/nova master: [WIP] OVS DPDK port representors support  https://review.opendev.org/65878510:27
*** zigo has quit IRC10:30
*** ttsiouts has joined #openstack-nova10:31
*** yedongcan has left #openstack-nova10:34
openstackgerritHamdy Khader proposed openstack/os-vif master: [WIP] OVS DPDK port representors support  https://review.opendev.org/65878610:35
*** zigo has joined #openstack-nova10:42
*** tbachman has quit IRC10:53
*** ttsiouts has quit IRC11:04
*** udesale has quit IRC11:08
*** udesale has joined #openstack-nova11:09
*** jangutter has joined #openstack-nova11:14
*** awalende has quit IRC11:50
*** awalende has joined #openstack-nova11:51
*** awalende_ has joined #openstack-nova11:54
*** nicolasbock has joined #openstack-nova11:54
*** awalende has quit IRC11:55
*** dpawlik has quit IRC11:59
*** panda is now known as panda|launch12:01
*** janki has quit IRC12:04
*** tbachman has joined #openstack-nova12:06
*** tbachman has quit IRC12:07
*** tbachman has joined #openstack-nova12:07
*** spsurya has quit IRC12:08
aarentsHi there, can some one check relevance of this please https://review.opendev.org/#/c/659054/ ?12:11
sean-k-mooneymdbooth_: ^ seams to be something you coudl weigh in on12:12
sean-k-mooneyaarents: the image cache code is rather arcane12:14
mdbooth_sean-k-mooney: It is stretched beyond all reasonable limits.12:14
*** mdbooth_ is now known as mdbooth12:14
sean-k-mooneyaarents: hardcoding size for the disk is not a good approch12:15
sean-k-mooneyoh that is the test code...12:15
aarents:)12:15
sean-k-mooneyso the change is just to pass the size parmater to the copy from host12:16
aarentsyep that's all12:16
*** cdent has joined #openstack-nova12:16
aarentsif not the qemu img disk have the size of backing file12:17
sean-k-mooneyso size in this case is the flavor root disk12:17
aarentsyes it is12:17
sean-k-mooneywell we are passing it like 10 lines above so it seams sane12:18
*** dpawlik has joined #openstack-nova12:21
*** dpawlik has quit IRC12:21
sean-k-mooneybut honestly i dont really know the full implication but at first glance it looks like an apporpiate fix for the bug you reported12:21
*** mriedem has joined #openstack-nova12:23
mdboothCould somebody hit https://review.opendev.org/#/c/658845/ for me, please? It has 1.5 +2s. It's the db retry bug.12:24
aarentssean-k-mooney: ok thks, it fixs the issue in my environment12:24
mdboothmriedem: I merged the comment you asked for you at the request of stephenfin, btw.12:24
* mdbooth was going to leave it in a follow-up.12:25
sean-k-mooneyaarents: it also passed ci so it dose not break the happy path although i dont think we have a function or tempest test that validates teh behavior12:25
mriedemmdbooth: yeah looking12:25
mdboothmriedem: Thanks12:26
sean-k-mooneymdbooth: oh this is the modifyicion of input parmaters thing you sent to the mailing list12:26
mdboothsean-k-mooney: Yeah.12:26
mriedemsean-k-mooney: the sriov live migration blueprint isn't closed yet b/c surely there are docs updates to be made, right?12:27
mriedemat least this https://docs.openstack.org/neutron/latest/admin/config-sriov.html#known-limitations12:27
mriedemmelwitt: are you ready for https://blueprints.launchpad.net/nova/+spec/count-quota-usage-from-placement to go back into a runway slot?12:27
sean-k-mooney that in neutreon but yes i should go update that.12:28
mriedemmdbooth: coreycb: btw, starlingx is saying since we merged the eventlet thing on stable/stein that they are broken ow12:28
mriedem*now12:28
mriedemso that's fun12:28
mriedemhttps://bugs.launchpad.net/nova/+bug/182906212:28
openstackLaunchpad bug 1829062 in StarlingX "nova placement api non-responsive due to eventlet error" [Critical,New] - Assigned to Bruce Jones (brucej)12:28
sean-k-mooneymriedem: dpending on how its run there were issues with some rabbitmq timeouts or somthing right12:29
mriedemgranted if starlingx moves to extracted placement in stein they won't have to deal with eventlet12:29
mdboothmriedem: Yeah, we're seeing similar downstream. However, I'm pretty sure we didn't.12:29
mdboothmriedem: Honestly what I really want to do is prioritise un-eventleting the multi-cell instance list code, and let them switch eventlet off.12:29
*** dpawlik has joined #openstack-nova12:30
mriedemwhich is mel's series isn't it?12:30
sean-k-mooneyya althoeugh we have an open question of how to we kill long running task that will timeout due to a down cell12:30
sean-k-mooneymriedem: yes12:30
mdboothmriedem: Let me clarify 'we didn't', btw: the eventlet band-aid thing didn't break it, it was already broken.12:31
mdboothI believe them that it's broken, though.12:31
mdboothmriedem: Yes. We discussed it at PTG and the feeling was that we'd address it later. However, I think that was when we believe we'd applied an effective band-aid already.12:32
sean-k-mooneyif we dont have a path forward with melwitt's serise we might want to consider reverting the scater gatehr stuff temporally and dropping eventlets12:32
*** tbachman has quit IRC12:32
mdboothsean-k-mooney: Also worthy of discussion.12:32
* mdbooth would definitely like to bump the priority of that series.12:34
*** _erlon_ has joined #openstack-nova12:35
sean-k-mooneyso looks like https://review.opendev.org/#/c/657750/1 is the last patch in that series and it passing ci but i think we are currently stuck on the open question in  https://review.opendev.org/#/c/650172/1412:36
sean-k-mooneyis the startx issue seperate for teh amqp issue reported on the mailing list12:37
sean-k-mooneyhttp://lists.openstack.org/pipermail/openstack-discuss/2019-May/005822.html12:38
*** awalende_ has quit IRC12:40
* mdbooth added a comment to that review.12:42
mdboothThanks for the reminder, sean-k-mooney12:42
*** sridharg has joined #openstack-nova12:43
*** janki has joined #openstack-nova12:44
*** awalende has joined #openstack-nova12:47
*** awalende has quit IRC12:51
*** awalende has joined #openstack-nova12:53
mriedemtssurya: we need a novaclient release before osc can use the locked reason stuff https://review.opendev.org/#/c/659124/12:54
mriedemi'll propose a release13:01
*** ttsiouts has joined #openstack-nova13:01
*** panda|launch is now known as panda13:05
openstackgerritMatt Riedemann proposed openstack/python-novaclient master: Cap sphinx for py2 to match global requirements  https://review.opendev.org/65920213:06
*** sapd1_x has quit IRC13:07
*** BjoernT has joined #openstack-nova13:13
*** BjoernT has quit IRC13:14
*** artom has quit IRC13:14
openstackgerritMatt Riedemann proposed openstack/python-novaclient master: Cap sphinx for py2 and drop keyring dependency  https://review.opendev.org/65920213:14
*** jchhatbar has joined #openstack-nova13:15
*** janki has quit IRC13:17
*** lbragstad has joined #openstack-nova13:18
*** tbachman has joined #openstack-nova13:18
mriedemthe requirements-check job is happy with ^ now if someone wants to approve so we can get that into the novaclient release i'm pushing13:19
*** sapd1_x has joined #openstack-nova13:20
*** BjoernT has joined #openstack-nova13:21
mdboothmriedem: Thanks.13:22
mdboothIncidentally, MySQL's repeatable read isolation level (which we use), is seriously obtuse when trying to handle concurrency. If I hadn't had to handle it when we were working on this code previously I would never have believe anybody would have intentionally created this behaviour. However, MySQL goes to great lengths to be broken.13:25
mdboothThey apparently see value in it.13:25
mriedemjob security13:28
dansmithkashyap: I'm entirely against (a) nova automatic scheduling based on perceived threats from vulnerabilities and (b) putting traits in placement about which vulnerabilities the host has, has been patched for, or has enabled/disabled. Hopefully that clears it up.13:29
*** brinzhang has quit IRC13:30
kashyapSure, I don't care about these things much at all; I wanted to put in a missing thing in traits and move on.13:30
kashyapdansmith: Also: you write "perceived threats" -- they're not.13:31
*** Luzi has quit IRC13:31
kashyapYou can _choose_ to take a policy stance, as "this is out of scope".  That's perfectly fine :-)13:31
sean-k-mooneythe add the pcid and 1ghugepage tratis and drop the other form the patch13:31
*** lpetrut has quit IRC13:32
dansmithkashyap: if I am running a single-tenant cloud full of workloads I control, then the threats are entirely perceived13:32
*** lpetrut has joined #openstack-nova13:32
kashyapdansmith: Fair, that's one valid example.  That's why I only said: should we consider "warn".13:33
*** awalende has quit IRC13:33
kashyapI also noted a few more valid cases where people willingly want to run on vulnerable hosts in my second reply on the 'openstack-security' thread.13:33
dansmithwe most certainly should not put traits in placement for the purposes of warning, but I also do not agree that nova should be doing the warning13:33
*** awalende has joined #openstack-nova13:34
coreycbmriedem: i wonder if they could work around it by running with python313:34
*** ttsiouts has quit IRC13:35
kashyapdansmith: Yeah, I agre with the former (not put them in traits).  The latter is a 'policy' decision -- so I'm fine with whatever the consensus of the community is.13:35
*** awalende_ has joined #openstack-nova13:37
*** awalende has quit IRC13:38
*** ratailor has quit IRC13:39
amorinhey all, can someone take a look at that: https://review.opendev.org/#/c/658362/13:39
*** awalende_ has quit IRC13:42
mdboothcoreycb: Are you talking about the eventlet thing?13:45
coreycbmdbooth: ye13:45
coreycbyes13:45
mdboothcoreycb: Unfortunately not. We're hitting this in OSP15, which is py3 only.13:46
sean-k-mooneycoreycb: if you dont use uswign and run the nova-api via the console script it shoudl not have this issue13:46
sean-k-mooney*uwsgi13:46
mdboothHowever, IIRC there's another project also hitting it?13:46
coreycbmdbooth: ok. i just saw py2.7 in the traceback and that was my thought. sean-k-mooney: ok13:46
mdboothsean-k-mooney: Do you happen to remember which one it was?13:46
sean-k-mooneymonasca i think13:47
mdboothIt's a library thing, anyway. It's nothing specific to what Nova's doing.13:47
mdboothIt's just that eventlet + wsgi is bad.13:47
sean-k-mooneyas i said the workaound currently is to not run the nova api under wsgi and fall back to using the python nova-api command and fronting that with a load blancer like ha proxy13:49
* mdbooth would prefer to expedite melwitt's patches and remove the eventlet requirement asap.13:50
* sean-k-mooney likes mdbooth ideas and want to subsribe to his newsletter13:51
mdboothI mentioned in the review that I don't think the lack of ability to cancel a thread is a showstopper in practice, and certainly not as big an issue as eventlet + wsgi.13:51
sean-k-mooneyah it was mistral13:51
kashyapsean-k-mooney: What does "monasca" even do?  Okay, let me go look up13:51
mdboothsean-k-mooney: That sounds better.13:52
sean-k-mooneymonsaca is a montioring service13:52
kashyapAh, thx.13:52
sean-k-mooneybut i just rememebre it began with m just fount th lin in the mailing list post13:52
mriedemamorin: you should have a test13:52
sean-k-mooneybut ya long story short as mdbooth said eventlet+wsgi is just a bad idea in general13:53
amorinmriedem: ok13:53
*** mlavalle has joined #openstack-nova13:59
*** awalende has joined #openstack-nova14:04
*** awalende has quit IRC14:08
efriedalex_xu: looking...14:12
tssuryamriedem: oh cool thansk :)14:21
aarentsmdbooth: thanks for reviewing https://review.opendev.org/#/c/659054/, you catch the test logic, I will add a comment in test as suggested14:23
*** artom has joined #openstack-nova14:27
openstackgerritStephen Finucane proposed openstack/nova-specs master: Follow up for I9200c2b18369da8b9194a8ccbbf6f1baf6a9613c  https://review.opendev.org/65930014:31
stephenfinbauzas: Time to merge https://review.opendev.org/#/c/555081/ ?14:31
stephenfinbauzas: I've posted a follow-up to address some of the outstanding comments and issues14:32
bauzascool then, I'll review those :)14:32
bauzasbut yeah I feel we agreed on saying cool cool at the PTG14:32
*** dpawlik has quit IRC14:32
*** jchhatbar has quit IRC14:34
*** lpetrut has quit IRC14:37
efriedalex_xu: How does that play with https://review.opendev.org/#/c/645520/ ?14:41
*** luksky has quit IRC14:41
*** dpawlik has joined #openstack-nova14:45
*** dpawlik has quit IRC14:49
*** maciejjozefczyk has quit IRC15:09
openstackgerritAlexandre arents proposed openstack/nova master: Fix live-migration when glance image deleted  https://review.opendev.org/65905415:10
*** ociuhandu has joined #openstack-nova15:13
*** ivve has quit IRC15:15
*** liuyulong has quit IRC15:21
openstackgerritArnaud Morin proposed openstack/nova master: Always Set dhcp_server in network_info  https://review.opendev.org/65836215:22
openstackgerritMerged openstack/nova-specs master: Standardize CPU resource tracking  https://review.opendev.org/55508115:24
*** panda is now known as panda|ruck15:26
*** cdent has quit IRC15:28
*** cdent has joined #openstack-nova15:29
*** dpawlik has joined #openstack-nova15:30
openstackgerritMatthew Booth proposed openstack/nova stable/stein: Fix retry of instance_update_and_get_original  https://review.opendev.org/65931715:30
openstackgerritMatthew Booth proposed openstack/nova stable/rocky: Fix retry of instance_update_and_get_original  https://review.opendev.org/65931815:31
*** ttsiouts has joined #openstack-nova15:32
*** READ10 has joined #openstack-nova15:32
openstackgerritMatthew Booth proposed openstack/nova stable/queens: Fix retry of instance_update_and_get_original  https://review.opendev.org/65932015:33
openstackgerritArnaud Morin proposed openstack/nova master: Always Set dhcp_server in network_info  https://review.opendev.org/65836215:33
*** helenafm has quit IRC15:33
*** dpawlik has quit IRC15:34
*** cdent has quit IRC15:35
*** tesseract has quit IRC15:35
amorinhey mriedem and all15:37
amorinI added some unit tests there:15:37
amorinhttps://review.opendev.org/#/c/65836215:37
*** macza has joined #openstack-nova15:37
*** davidsha has joined #openstack-nova15:38
*** cfriesen has joined #openstack-nova15:39
*** cdent has joined #openstack-nova15:41
*** BjoernT has quit IRC15:44
*** dpawlik has joined #openstack-nova15:46
*** BjoernT has joined #openstack-nova15:46
*** sridharg has quit IRC15:46
*** ttsiouts has quit IRC15:50
mriedemack it's in the tab queue15:50
*** dpawlik has quit IRC15:50
*** ccamacho has quit IRC15:56
*** imacdonn has quit IRC16:01
*** wwriverrat has joined #openstack-nova16:06
*** rpittau is now known as rpittau|afk16:06
*** imacdonn has joined #openstack-nova16:13
*** xek has quit IRC16:14
*** tesseract has joined #openstack-nova16:24
melwittmriedem: yes, should be ready. I just looked through the latest comments again and replied. and it looks like it hasn't fallen into merge conflict yet16:29
*** dklyle has quit IRC16:31
mriedemok16:33
mriedeminto the runway slot vat of acid it will be thrown16:33
*** BjoernT has quit IRC16:33
melwitthaha, thanks16:34
*** dklyle has joined #openstack-nova16:37
*** BjoernT has joined #openstack-nova16:38
*** gyee has joined #openstack-nova16:38
openstackgerritMatt Riedemann proposed openstack/nova stable/rocky: Add functional confirm_migration_error test  https://review.opendev.org/65883416:39
openstackgerritMatt Riedemann proposed openstack/nova stable/rocky: Delete allocations even if _confirm_resize raises (part 2)  https://review.opendev.org/65933816:39
mriedemganso: ^16:39
*** hemna has joined #openstack-nova16:42
gansomriedem: thanks! will rebase on top16:44
gansomriedem: oh you already rebased16:45
gansomriedem: thanks!16:45
*** bbowen has quit IRC16:45
imacdonnmriedem: pls add https://review.opendev.org/#/c/653065/ to your tab queue for revisit? I have a feeling that the issue with the functional tests has something to do with sqlalchemy sessions, or something, but may be off on the wrong track16:45
*** bbowen has joined #openstack-nova16:45
imacdonn(if there's a sqlalchemy guru available for a consult, that might work too)16:45
weshaykashyap ping.. looking to add the qemu-agent rpm to nova,  I think that would be on a container and not bm.. not sure also wanted to verify w/ you which container that rpm should be installed on16:45
weshaykashyap context is: nova.exception.QemuGuestAgentNotEnabled: QEMU guest agent is not enabled16:46
weshayhttps://bugs.launchpad.net/tripleo/+bug/182431516:46
openstackLaunchpad bug 1824315 in tripleo "periodic fedora28 standalone job failing at test_volume_boot_pattern" [Critical,In progress] - Assigned to Quique Llorente (quiquell)16:46
weshaymelwitt maybe you would know? ^16:47
melwittweshay: the qemu guest agent is something that would have to be installed on the guest, which is currently not done as part of any CI that I'm aware of16:50
weshaymelwitt ok.. some folks were saying the container but the bm makes more sense to me16:50
melwittthat is, if you create a nova instance, you'd then have to install the qemu guest agent on that guest after it comes up16:50
melwittbut I'm wondering why it should be needed16:52
*** mvkr has joined #openstack-nova16:52
dansmithif they're trying to do something that requires it, like quiesce disks (I think)16:52
melwittweshay: that error message is just saying it's going to skip quiescing the instance bc the guest agent isn't present, but it shouldn't fail the job because of that I don't think16:53
weshaymelwitt it does not fail consistently, maybe 1/10 times16:53
weshaymaybe this is a red herring16:53
dansmithif it's failing on the guest agent really not being there 1/10 times, I'd suspect the guest isn't booting fully 1/10 times or something like that16:54
melwittweshay: ok. I don't think this is the root cause of the failure because AFAICT it's just skipping that. it should say that every time I think16:54
melwittdansmith: I don't think any CI installs the guest agent, at least I was not aware of any16:55
*** mvkr has quit IRC16:55
dansmithmelwitt: if they are using a fedora guest...16:55
openstackgerritMerged openstack/python-novaclient master: Cap sphinx for py2 and drop keyring dependency  https://review.opendev.org/65920216:55
melwittdansmith: fedora guest has it by default?16:55
weshaydansmith ya.. we this in fedora deployments.. but w/ cirros guest16:56
dansmithmelwitt: no idea, I'm just saying, different from a cirros guest16:56
dansmithokay16:56
dansmiththe bug shows a big stack trace.. melwitt you think we're logging that trace just while checking the skip?16:56
melwittoh. yeah, anecdotally I've not had the qemu guest agent come with any cloud image, like centos for example I've used16:56
melwittdansmith: I think so, opening up the code to confirm16:56
dansmithokay we should not be logging a trace for a check thing, amirite?16:57
*** ociuhandu has quit IRC16:57
*** arxcruz has joined #openstack-nova16:57
melwittshould not, no. I see a LOG.info for it but no re-raise, so not yet sure what's going on16:58
dansmiththat's why I'm wondering if we're actually leaking the exception out and not just skipping16:58
melwittyeah, let's see, the trace is happening in the driver, so that must be where16:58
melwittthere's no re-raise in compute/api, I meant16:59
*** ociuhandu has joined #openstack-nova16:59
*** dtantsur is now known as dtantsur|afk17:01
*** davidsha has left #openstack-nova17:01
*** derekh has quit IRC17:02
cdentefried: is ocata nova still an active concern? there's a logging buglet in the reasource tracker + report client that can be pretty misleading but the issue goes away with pike (or maybe queens)17:06
*** ricolin has quit IRC17:07
cdent(goes away with pike, just checked)17:09
*** luksky has joined #openstack-nova17:10
*** BjoernT_ has joined #openstack-nova17:11
*** BjoernT has quit IRC17:12
*** psachin has quit IRC17:15
melwittweshay: on your latest comment on LP, the log shows the code continuing on to do the volume snapshot http://logs.openstack.org/84/655684/2/check/tripleo-ci-fedora-28-standalone/eda3002/logs/undercloud/var/log/containers/nova/nova-api.log.txt.gz#_2019-05-13_16_03_50_120 so I think that trace is unrelated17:16
melwittweshay: I don't yet find how that trace is getting logged (it shouldn't be logged), so I'm still looking for how that's happening17:17
weshayk k.. thanks.. so melwitt we're saying it's a red herring and the problem is elsewhere right?17:17
weshayconfirming that I guess17:17
melwittweshay: right. the logging of the trace is bad and I'll have to find how that's occurring to remove it, but AFAICS the code is continuing on after skipping the quiesce and shouldn't affect the pass/fail of the CI17:18
melwitt"Creating snapshot from volume"17:19
weshayk17:19
weshayarxcruz ^17:20
melwittI'll look at the rest of those logs to see if I can tell where it's failing17:20
*** udesale has quit IRC17:21
melwittweshay: looking at the test results of that run, the test_volume_boot_pattern passed but test_network_basic_ops failed due to a ssh timeout http://logs.openstack.org/84/655684/2/check/tripleo-ci-fedora-28-standalone/eda3002/logs/tempest.html.gz17:22
weshayaye17:22
weshayya.. probably not same thing17:23
weshaypaste happy17:23
weshayI guess17:23
melwittdefinitely not in this particular run. that run ^ failed because of a ssh timeout and test_volume_boot_pattern passed with the log trace17:24
tssuryamriedem: trivial doc change: https://review.opendev.org/#/c/659067/17:24
*** dklyle has quit IRC17:29
*** Vek has joined #openstack-nova17:31
openstackgerritMerged openstack/os-resource-classes master: Replace git.openstack.org URLs with opendev.org URLs  https://review.opendev.org/65851917:32
*** maciejjozefczyk has joined #openstack-nova17:33
*** ttsiouts has joined #openstack-nova17:35
*** maciejjozefczyk has quit IRC17:39
*** dpawlik has joined #openstack-nova17:47
*** tssurya has quit IRC17:47
mriedemcdent: ocata is in extended maintenance meaning we can fix things if we want, but if it's not a big deal then meh17:47
* cdent has been wrong many times about what various folk think is or is not a big deal17:48
*** dklyle has joined #openstack-nova17:49
mriedemi thought VIO was off ocata (and pike for that matter)?17:50
cdentthere are lots of supported customers back on ocata, and a few that are -2 or -4 releases from that....17:51
*** dpawlik has quit IRC17:51
mriedemjust tell those banks to upgrade17:51
mriedemdo you know if there was a specific fix in pike that fixes it? or was it some big refactor we wouldn't want to backport?17:52
mriedemif it's something dumb easy and backports cleanly than whatever17:52
cdentthere was a refactor: the introduction of _ensure_resource_provider17:53
mriedemwell then that's probably not something we'll want to backport to ocata at this point17:53
cdentquite17:53
*** dklyle has quit IRC17:56
*** tesseract has quit IRC18:01
*** READ10 has quit IRC18:02
*** dpawlik has joined #openstack-nova18:02
*** dpawlik has quit IRC18:07
*** BjoernT_ has quit IRC18:11
openstackgerritMerged openstack/nova master: [Trivial doc change] Admin can overwrite the locked_reason of an owner  https://review.opendev.org/65906718:16
*** dklyle has joined #openstack-nova18:20
*** BjoernT has joined #openstack-nova18:20
*** tbachman has quit IRC18:27
*** ttsiouts has quit IRC18:27
*** BjoernT has quit IRC18:29
*** luksky has quit IRC18:32
*** ab-a has quit IRC18:40
*** ttsiouts has joined #openstack-nova18:43
*** bbowen has quit IRC18:44
*** ttsiouts has quit IRC18:48
*** BjoernT has joined #openstack-nova18:52
*** BjoernT_ has joined #openstack-nova18:58
*** BjoernT has quit IRC18:59
*** dklyle has quit IRC19:01
*** cdent has quit IRC19:02
*** tssurya has joined #openstack-nova19:02
*** BjoernT_ has quit IRC19:03
*** bbowen has joined #openstack-nova19:06
melwittmriedem: wondering if you might know anything about this... is it known or expected that the traceback will become part of the printed exception object when that exception occurred on the other side of an rpc call? example http://logs.openstack.org/72/650172/14/check/tempest-full-py3/61914af/controller/logs/screen-n-api.txt.gz#_May_08_08_52_36_10623519:08
melwittin nova/compute/api.py, we're catching the exception and logging the exception object, which I thought should normally only contain the short message and not the traceback. but the traceback seems to also be part of that19:09
melwittI looked at exception_wrapper (from our @wrap_exception decorator) and expected_exceptions in oslo.messaging and didn't see how they could be adding the traceback. I wanted to ask you in case you've seen this before and already know what's going on19:10
mriedemi want to say it's a thing in oslo.log if there is an exception in scope19:18
mriedemwhich is what this is, it's logging the exception within an except block19:19
melwitthm, I think I found where it's added in oslo.messaging https://github.com/openstack/oslo.messaging/blob/40c25c2bde6d2f5a756e7169060b7ce389caf174/oslo_messaging/_drivers/common.py#L21219:19
mriedemremember this? https://github.com/openstack/oslo.log/commit/a63438cb5b12e2a6973fb0e1b0b9ebce2f8116b6#diff-e121e929b5792521de104c2db0d14d9419:20
mriedemoh, so,19:20
melwittif I'm understanding this code right, when oslo.messaging deserializes an exception, it tacks on the traceback to the normal message19:20
mriedemif it's an rpc call, which this is, we need to use expected_exceptions on the compute manager method19:20
mriedemare we doing that?19:20
melwittwe are19:20
melwitthttps://github.com/openstack/nova/blob/master/nova/compute/manager.py#L851319:21
melwittoh hah, I forgot about that commit. reading..19:21
mriedemso my guess is it's related to that magic stuff oslo.log does when you're logging from within an except block19:22
mriedemeven if you're not passing exc_info=True19:22
*** BjoernT has joined #openstack-nova19:24
melwittthanks, this gives me more to look into19:25
mriedemhttps://github.com/openstack/oslo.log/blob/master/oslo_log/formatters.py#L120 should handle that i'd think, but idk19:25
mriedemlight up the doug signal19:25
*** ivve has joined #openstack-nova19:25
melwittyeah... I don't think this has to do with the error summary (I'm having trouble remembering what that is, even with seeing the old commit). but rather that if we catch a remote exception, the 'message' attribute of the object is going to contain the traceback, so if we don't want to log a traceback, we can't be doing "LOG.info('stuff %s', exp)"19:30
melwittthis log line is explicitly doing that19:31
*** BjoernT has quit IRC19:35
efriedcdent: pike just went em, so I'm not sure how much attention we are (or should be) paying to ocata at this point. Probably only security vulnerabilities, kind of thing.19:39
*** BjoernT has joined #openstack-nova19:39
*** dpawlik has joined #openstack-nova20:03
*** dpawlik has quit IRC20:08
*** BjoernT has quit IRC20:12
*** BjoernT has joined #openstack-nova20:12
*** pcaruana has quit IRC20:14
*** dpawlik has joined #openstack-nova20:21
*** whoami-rajat has quit IRC20:24
*** dpawlik has quit IRC20:25
*** hemna has quit IRC20:35
*** dklyle has joined #openstack-nova20:39
mriedemi'm still fixing shit back in pike so yeah it's still active-ish20:45
*** ttsiouts has joined #openstack-nova20:45
dansmithis it easier to land things in pike than master?20:46
dansmithbecause based on master these days, I'm guessing so20:46
artomWhat's the company cores policy on stable stuff?20:47
artomSo for instance, would Mel be allowed to +W https://review.opendev.org/#/c/657125/ ?20:48
*** tbachman has joined #openstack-nova20:50
*** hongbin has joined #openstack-nova20:54
openstackgerritmelanie witt proposed openstack/nova master: Stop logging traceback when skipping quiesce  https://review.opendev.org/65937421:04
*** ralonsoh has quit IRC21:07
*** panda|ruck has quit IRC21:10
melwittmriedem: now I know what format_message() is for ^ and also I found this, from 6 years ago https://github.com/openstack/nova/commit/86031940b313532606449c400bca64707be1797a21:12
*** BjoernT has quit IRC21:13
*** BjoernT_ has joined #openstack-nova21:13
*** panda has joined #openstack-nova21:14
*** dklyle has quit IRC21:14
*** ttsiouts has quit IRC21:16
mriedemoh geez21:16
melwittyeah xD21:16
mriedemif we should always be using format_message(), shouldn't we override __str__ or __repr__ or something for NovaExceptions?21:16
melwittI _think_ that the exception type for remote exceptions is actually generated dynamically by oslo.messaging,21:17
melwitt*_Remote exceptions, and it overrides __str__ and __unicode__ to have the traceback in it21:18
mriedemif it's not a NovaException then we can't call format_message on it can we?21:19
melwittyeah, maybe it's that NovaException is the base class it uses21:19
mriedemer i guess you're talking about this https://github.com/openstack/oslo.messaging/blob/40c25c2/oslo_messaging/_drivers/common.py#L24721:20
melwittyes21:20
mriedemf what a mess21:21
melwittbut you're right that it must be deriving somehow from NovaException else format_message wouldn't work21:21
mriedemso NovaException override __str__ won't work b/c we aren't using a NovaException?21:21
mriedempython black magic21:21
melwittI think probably not, based on this https://github.com/openstack/oslo.messaging/blob/40c25c2/oslo_messaging/_drivers/common.py#L24421:22
melwittI was also looking at these tests which seem to know all about this issue :P https://github.com/openstack/nova/blob/master/nova/tests/unit/test_exception.py#L20321:23
melwittFakeNovaException_Remote is presumably simulating what oslo.messaging does21:23
melwittI'm not seeing though in oslo.messaging code how it's subclassing the original exception type though21:25
mriedemi think that's https://github.com/openstack/oslo.messaging/blob/40c25c2/oslo_messaging/_drivers/common.py#L24321:25
mriedemhttps://docs.python.org/3.6/library/functions.html#type21:26
melwittoh, I see yeah21:26
mriedemthe first arg is the new type _Remote, 2nd is the parent class which is the NovaException21:26
melwittthe second argument is the base classes21:26
mriedempython21:26
mriedemblack21:26
mriedemmagic21:26
melwittright21:26
melwittyeah, so that's fun21:26
mriedemNotImplementedError doesn't have format_message()...21:28
mriedemi know what we can do!21:28
mriedemmonkey patch oslo.messaging,21:28
mriedemextend their dynamic type,21:28
mriedemand override with our own __str__ that does format_message!21:28
melwittbrilliant!21:28
mriedemand somehow do it with eventlet21:28
*** dklyle has joined #openstack-nova21:29
openstackgerritHamdy Khader proposed openstack/nova master: [WIP] OVS DPDK port representors support  https://review.opendev.org/65878521:31
*** BjoernT_ has quit IRC21:46
openstackgerritmelanie witt proposed openstack/nova master: Stop logging traceback when skipping quiesce  https://review.opendev.org/65937421:50
melwittmriedem: just did a fugly update ^ better ideas welcome21:53
openstackgerritMerged openstack/nova master: Add image type request filter  https://review.opendev.org/65641321:53
*** dklyle has quit IRC22:00
*** takashin has joined #openstack-nova22:00
*** xek has joined #openstack-nova22:07
mriedemreplied22:10
*** artom has quit IRC22:11
*** mlavalle has quit IRC22:17
melwittthanks22:18
openstackgerritmelanie witt proposed openstack/nova master: Stop logging traceback when skipping quiesce  https://review.opendev.org/65937422:18
openstackgerritMerged openstack/nova master: Enable image type query support in nova-next  https://review.opendev.org/65690322:18
openstackgerritMerged openstack/nova master: Add docs for image type support request filter  https://review.opendev.org/65702522:18
openstackgerritMerged openstack/nova stable/queens: Fix nova-grenade-live-migration run book for opendev migration  https://review.opendev.org/65580022:19
*** dpawlik has joined #openstack-nova22:21
*** dpawlik has quit IRC22:26
*** ttsiouts has joined #openstack-nova22:32
mriedemdansmith: it is done https://blueprints.launchpad.net/nova/+spec/request-filter-image-types22:36
* dansmith drops to his knees22:36
*** dpawlik has joined #openstack-nova22:37
*** dpawlik has quit IRC22:42
*** mriedem has quit IRC22:51
*** macza has quit IRC22:59
*** mdbooth_ has joined #openstack-nova23:01
*** mdbooth has quit IRC23:04
*** tkajinam has joined #openstack-nova23:05
*** ttsiouts has quit IRC23:06
*** rcernin has joined #openstack-nova23:14
*** panda has quit IRC23:30
*** panda has joined #openstack-nova23:31
*** tssurya has quit IRC23:32
*** brinzhang has joined #openstack-nova23:41
*** d34dh0r53 has joined #openstack-nova23:43
*** sapd1_x has quit IRC23:55
*** BjoernT has joined #openstack-nova23:57
*** d34dh0r53 has quit IRC23:58

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