Friday, 2018-09-28

*** vabada has quit IRC00:01
*** vabada has joined #openstack-nova00:02
*** tbachman has quit IRC00:17
*** tbachman has joined #openstack-nova00:19
*** erlon has quit IRC00:19
*** brinzhang has joined #openstack-nova00:25
*** hamzy has joined #openstack-nova00:25
*** erlon has joined #openstack-nova00:36
*** takashin has joined #openstack-nova00:42
openstackgerritBryan Song proposed openstack/nova master: Creation image for volume-backend instance should use volume size in image property 'min_disk'  https://review.openstack.org/60559600:44
*** mlavalle has quit IRC00:44
*** gyee has quit IRC00:51
*** mhen has quit IRC01:10
*** mhen has joined #openstack-nova01:12
*** erlon has quit IRC01:16
*** Dinesh_Bhor has joined #openstack-nova01:20
*** dpawlik has joined #openstack-nova01:21
*** dpawlik has quit IRC01:26
*** Dinesh_Bhor has quit IRC01:39
*** itlinux has joined #openstack-nova01:44
*** itlinux has quit IRC01:44
*** Dinesh_Bhor has joined #openstack-nova01:48
*** zzzeek has quit IRC01:48
*** zzzeek has joined #openstack-nova01:49
openstackgerritJack Ding proposed openstack/nova master: Add HPET timer support for x86 guests  https://review.openstack.org/60590201:53
*** mrsoul has quit IRC01:55
*** brinzh has joined #openstack-nova01:56
*** brinzhang has quit IRC01:59
*** tbachman has quit IRC02:05
*** jamesdenton has joined #openstack-nova02:07
openstackgerrittianhui proposed openstack/nova master: Update doc: launch-instance-from-volume  https://review.openstack.org/60564002:10
openstackgerrittianhui proposed openstack/nova master: Update doc  https://review.openstack.org/60564002:16
*** hshiina has joined #openstack-nova02:17
*** hongbin has joined #openstack-nova02:20
*** stakeda has joined #openstack-nova02:26
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove unnecessary redirect  https://review.openstack.org/60591102:36
*** tetsuro has joined #openstack-nova02:45
*** imacdonn has quit IRC02:51
*** imacdonn has joined #openstack-nova02:51
*** tbachman has joined #openstack-nova02:54
*** med_ has quit IRC02:57
*** tbachman has quit IRC02:59
*** cfriesen has quit IRC03:02
*** tbachman has joined #openstack-nova03:05
*** dpawlik has joined #openstack-nova03:22
openstackgerritTao Li proposed openstack/python-novaclient master: Deprecate the unused instance-name  https://review.openstack.org/60252003:25
*** dpawlik has quit IRC03:27
openstackgerritBrin Zhang proposed openstack/nova master: Verify the volume type is supported in compute API  https://review.openstack.org/60557303:29
*** psachin has joined #openstack-nova03:29
*** hongbin has quit IRC03:38
*** Dinesh_Bhor has quit IRC03:54
*** Nel1x has quit IRC04:07
*** Dinesh_Bhor has joined #openstack-nova04:24
*** jaypipes has quit IRC04:27
*** swamireddy has quit IRC04:44
*** toabctl has quit IRC04:48
*** Dinesh_Bhor has quit IRC04:58
*** Dinesh_Bhor has joined #openstack-nova05:02
openstackgerritMerged openstack/nova master: Time how long select_destinations() takes in conductor  https://review.openstack.org/51710805:06
*** udesale has joined #openstack-nova05:09
*** itlinux has joined #openstack-nova05:12
*** dpawlik has joined #openstack-nova05:23
openstackgerritMerged openstack/nova master: Enforce case-sensitive hostnames in aggregate host add  https://review.openstack.org/60490605:26
openstackgerritMerged openstack/nova stable/queens: Skip ServerShowV254Test.test_rebuild_server in cells v1 job  https://review.openstack.org/60527005:26
*** dpawlik has quit IRC05:29
*** tbachman has quit IRC05:31
*** tetsuro has quit IRC05:35
*** Dinesh_Bhor has quit IRC05:51
*** Dinesh_Bhor has joined #openstack-nova05:56
*** pcaruana has joined #openstack-nova06:11
*** jdillaman has quit IRC06:17
openstackgerritTao Li proposed openstack/python-novaclient master: Deprecate the unused instance-name  https://review.openstack.org/60252006:21
*** dpawlik has joined #openstack-nova06:27
*** dpawlik has quit IRC06:32
*** dpawlik has joined #openstack-nova06:32
*** mrsoul has joined #openstack-nova06:38
*** jiaopengju has quit IRC06:50
*** jiaopengju has joined #openstack-nova06:52
*** Luzi has joined #openstack-nova06:53
*** icey has quit IRC06:54
*** maciejjozefczyk has joined #openstack-nova06:56
*** hshiina has quit IRC06:59
*** ralonsoh has joined #openstack-nova07:00
*** takashin has left #openstack-nova07:05
*** icey has joined #openstack-nova07:07
*** rcernin has quit IRC07:12
openstackgerritBryan Song proposed openstack/nova master: Use volume_size for 'min_disk' image property when create image from BFV instance.  https://review.openstack.org/60559607:19
openstackgerritBrin Zhang proposed openstack/nova master: Specifies the storage backend to boot instance  https://review.openstack.org/57936007:24
*** helenafm has joined #openstack-nova07:25
openstackgerritliuming proposed openstack/nova master: Deletes evacuated instance files when source host is ok  https://review.openstack.org/60598707:31
*** dpawlik has quit IRC07:32
openstackgerritBrin Zhang proposed openstack/nova master: Add volume_type field to BlockDeviceMapping object  https://review.openstack.org/60468707:32
*** dpawlik has joined #openstack-nova07:34
*** dpawlik has quit IRC07:34
*** dpawlik has joined #openstack-nova07:34
openstackgerritMerged openstack/nova stable/queens: Skip ServerShowV247Test.test_update_rebuild_list_server in nova-cells-v1 job  https://review.openstack.org/60527107:35
openstackgerritMerged openstack/nova stable/queens: Skip more rebuild tests for cells v1 job  https://review.openstack.org/60541607:36
*** dpawlik has quit IRC07:36
*** markvoelker has quit IRC07:36
*** markvoelker has joined #openstack-nova07:37
*** gibi is now known as giblet07:38
giblethappy friday nva07:38
gibletnova07:38
*** jdillaman has joined #openstack-nova07:41
*** markvoelker has quit IRC07:42
*** hoangcx has quit IRC07:43
*** jpena|off is now known as jpena07:43
openstackgerritBrin Zhang proposed openstack/nova master: Verify the volume type is supported in compute API  https://review.openstack.org/60557307:53
*** ratailor has joined #openstack-nova07:56
*** alexchadin has joined #openstack-nova07:57
*** hoangcx has joined #openstack-nova07:57
*** alexchadin has quit IRC08:01
*** bauzas is now known as PapaOurs08:03
PapaOursgood morning nova08:03
openstackgerritBrin Zhang proposed openstack/nova master: Specifies the storage backend to boot instance  https://review.openstack.org/57936008:05
*** alexchadin has joined #openstack-nova08:11
*** dpawlik has joined #openstack-nova08:12
*** mgoddard has joined #openstack-nova08:16
*** alexchadin has quit IRC08:16
*** alexchadin has joined #openstack-nova08:18
openstackgerritMerged openstack/nova master: Null out instance.availability_zone on shelve offload  https://review.openstack.org/59908708:24
*** a-pugachev has joined #openstack-nova08:24
*** alexchadin has quit IRC08:25
*** stephenfin is now known as finucannot08:27
openstackgerritBalazs Gibizer proposed openstack/nova master: Run ServerMovingTests with nested resources  https://review.openstack.org/60408408:28
openstackgerritBalazs Gibizer proposed openstack/nova master: Ignore forcing of live migration for nested instance  https://review.openstack.org/60578508:28
*** alexchadin has joined #openstack-nova08:28
*** ttsiouts has joined #openstack-nova08:32
*** derekh has joined #openstack-nova08:37
*** markvoelker has joined #openstack-nova08:37
*** ratailor has quit IRC08:40
*** ratailor has joined #openstack-nova08:41
openstackgerritBrin Zhang proposed openstack/nova master: Verify the volume type is supported in compute API  https://review.openstack.org/60557308:49
*** vivsoni_ has quit IRC08:56
*** markvoelker has quit IRC08:59
*** alexchadin has quit IRC09:10
*** ttsiouts has quit IRC09:15
*** ttsiouts has joined #openstack-nova09:18
openstackgerritNaichuan Sun proposed openstack/nova master: os-xenapi(n-rp): add traits for vgpu n-rp  https://review.openstack.org/60426909:20
*** alexchadin has joined #openstack-nova09:25
*** rmulugu has joined #openstack-nova09:28
*** toabctl has joined #openstack-nova09:42
*** tssurya has joined #openstack-nova09:46
*** Dinesh_Bhor has quit IRC09:47
*** Dinesh_Bhor has joined #openstack-nova09:54
*** markvoelker has joined #openstack-nova09:57
*** cdent has joined #openstack-nova09:58
*** scroll is now known as hfjvjffju09:59
*** alexchadin has quit IRC10:05
openstackgerritMatthew Booth proposed openstack/nova master: Fix a race evacuating instances in an anti-affinity group  https://review.openstack.org/60543610:15
*** mdbooth has joined #openstack-nova10:15
*** markvoelker has quit IRC10:18
*** alexchadin has joined #openstack-nova10:22
*** hoangcx has quit IRC10:23
*** alexchadin has quit IRC10:29
*** alexchadin has joined #openstack-nova10:39
*** stakeda has quit IRC10:43
openstackgerritRadoslav Gerganov proposed openstack/nova master: VMware: Live migration of instances  https://review.openstack.org/27011610:49
openstackgerritGhanshyam Mann proposed openstack/nova master: Merge used_limits extension response into limit view builder  https://review.openstack.org/60603110:52
*** dtantsur|afk is now known as dtantsur10:57
*** med_ has joined #openstack-nova11:02
*** erlon has joined #openstack-nova11:03
openstackgerritBrin Zhang proposed openstack/nova master: Add compute version 36 to support ``volume_type``  https://review.openstack.org/57936011:05
*** erlon has quit IRC11:05
*** erlon has joined #openstack-nova11:06
*** jpena is now known as jpena|lunch11:07
*** erlon has quit IRC11:07
*** erlon has joined #openstack-nova11:08
*** alexchadin has quit IRC11:08
*** ttsiouts has quit IRC11:09
openstackgerritBrin Zhang proposed openstack/nova master: Add compute version 36 to support ``volume_type``  https://review.openstack.org/57936011:10
*** ratailor has quit IRC11:17
openstackgerritBrin Zhang proposed openstack/nova master: Add compute version 36 to support ``volume_type``  https://review.openstack.org/57936011:20
*** rmulugu has quit IRC11:25
openstackgerritChen proposed openstack/nova master: doc: update availability zone doc  https://review.openstack.org/60530611:28
*** udesale has quit IRC11:32
*** dpawlik has quit IRC11:37
*** cdent has quit IRC11:37
*** panda|off is now known as panda11:42
brinzhHello, If multiple patches have dependencies, how do I update them synchronously?11:48
brinzhFor example: I updated 2stpatch, how to synchronize 2st's latest patch in 3st patch?11:48
*** mrsoul has quit IRC11:54
*** mgoddard has quit IRC11:54
*** Bhujay has joined #openstack-nova11:58
*** mgoddard has joined #openstack-nova12:00
*** jpena|lunch is now known as jpena12:05
openstackgerritBalazs Gibizer proposed openstack/nova master: Consider nested allocations during allocation cleanup  https://review.openstack.org/60605012:15
*** ttsiouts has joined #openstack-nova12:21
openstackgerritBalazs Gibizer proposed openstack/nova master: Consider nested allocations during allocation cleanup  https://review.openstack.org/60605512:27
openstackgerritBalazs Gibizer proposed openstack/nova master: Consider nested allocations during allocation cleanup  https://review.openstack.org/60605012:28
*** mriedem has joined #openstack-nova12:29
openstackgerritBrin Zhang proposed openstack/nova master: Add microversion 2.67 to support ``volume_type``  https://review.openstack.org/60557312:30
mriedemlyarwood: can you hit these rocky backports? https://review.openstack.org/#/c/605260/12:30
mriedemneed to keep that revert train moving12:31
*** med_ has quit IRC12:31
*** nicolasbock_ has joined #openstack-nova12:32
kashyapmriedem: He's on PTO this week, IIRC.  Swimming in beer in Germany12:34
mriedemwhat's bauzas' excuse?12:34
kashyaphttps://www.oktoberfest.de/en/12:34
kashyapmriedem: It's France.  They work in between holidays :D12:35
kashyapFrankly, Belgium is no slouch either!  I'll admit, for the last 3 years I didn't even use half my PTO.12:35
mriedemjohnthetubaguy: if you're around can you hit those rocky backports? starting at https://review.openstack.org/#/c/605260/12:36
kashyapmriedem: BTW, thanks for this detailed commit message and the fix! -- https://review.openstack.org/#/c/567630/ ("check image type before removing snapshots in _cleanup_resize")12:37
johnthetubaguymriedem: will take a look12:37
mriedemkashyap: sure, that's from awhile back12:37
* kashyap had to handle a downstream backport to Newton. So was catching up with the context for it12:37
mriedemoh12:37
kashyapmriedem: Yeah, I know.  I just appreciate the references, and "thinking out loud".12:38
kashyapIf you have an Amazon wishlist (not too expensive!) or some such, do post a link :D12:38
mriedemi'll just take a root bier in DE12:40
kashyapmriedem: Ah, you'll be in Berlin12:41
kashyapNever heard of root bier.  /me looks up12:41
mriedemhttps://en.wikipedia.org/wiki/Root_beer12:41
kashyapAh, it's a soft drink.12:42
kashyap(Actually the page clarifies it can be a 'hard' drink, too.)12:42
johnthetubaguymriedem: seems that commit got into queens and pike too, did we get the revert up for those yet?12:43
*** artom has quit IRC12:44
mriedemjohnthetubaguy: yup12:44
mriedemhence the train12:44
johnthetubaguyah, right12:45
*** s10 has joined #openstack-nova12:45
kashyapUnrelated: If anyone here has any complaints / wishlist with libvirt & QEMU, feel free to type here (or https://etherpad.openstack.org/p/nova-libvirt-qemu).  FWIW, next month I'll be in Edinburgh (KVM Forum / Open Source Summit), at the gathering of KVM / QEMU / libvirt folks.12:47
openstackgerritMatt Riedemann proposed openstack/nova stable/ocata: unquiesce instance after quiesce failure  https://review.openstack.org/60588412:49
openstackgerritBrin Zhang proposed openstack/nova master: Add microversion 2.67 to support ``volume_type``  https://review.openstack.org/60557312:52
openstackgerritBrin Zhang proposed openstack/nova master: Add compute version 36 to support ``volume_type``  https://review.openstack.org/57936012:58
*** brinzh has quit IRC13:00
*** jaosorior has quit IRC13:00
PapaOursmriedem: I'm here y' know13:05
*** dpawlik has joined #openstack-nova13:05
*** cdent has joined #openstack-nova13:06
* PapaOurs just wonder if I should somehow use again bauwser13:07
PapaOursI mean, even if mriedem can't find my friday nick, who else ?13:08
PapaOursif even*13:08
*** dpawlik has quit IRC13:08
*** dpawlik has joined #openstack-nova13:08
mriedemi can't keep track13:11
*** psachin has quit IRC13:13
mriedemjohnthetubaguy: PapaOurs: while you're here, in case you haven't seen the thread, we're due to EM the ocata branch,13:13
mriedemso i'm trying to flush through what ocata backports we have so we can do a final release before we tag the branch as EM,13:13
mriedemwhich means we don't do releases of it after that point13:13
PapaOursgotcha13:13
mriedemthis will mean needing to flush the stuff in rocky/queens/pike first, since some of the ocata backports are also open on those other branches13:14
PapaOursI've seen a couple of changes in terms of relmgmt but I missed this one13:14
mriedemi'll work on making an etherpad today and send something to the dev list, nova-specific, so we can burn that down next week13:14
PapaOursmriedem: good idea13:14
johnthetubaguymriedem: so that is end maintenance rather than extend maintenance?13:14
mriedemjohnthetubaguy: it just means we're allowed to let ocata rot at that point if we want13:15
PapaOursjohnthetubaguy: it's basically the signal that we won't EOL but we won't also release13:15
mriedemit's not in active maintenance13:15
johnthetubaguyah, got it13:15
mriedemhttps://docs.openstack.org/project-team-guide/stable-branches.html#extended-maintenance13:15
PapaOursmriedem: do we have some deadline for Ocata being EM ?13:15
PapaOursmriedem: trying to find the email you mention13:15
* PapaOurs gives one advice : avoid Gmail by any cost13:16
*** zul has quit IRC13:16
mriedemhttps://releases.openstack.org/13:16
mriedemExtended Maintenance estimated 2018-08-2713:17
PapaOursoh fun13:17
mriedemlike most things for stable, it's a guideline for the timeline, but we're always slow to get around to eol13:17
PapaOursI see, so the sooner is the better13:17
mriedemwell, the sooner we EM, the sooner we can stop caring as much about ocata13:17
mriedemnot that most people probably care that much about it right now13:17
PapaOurssome people care, but they're paid to care :)13:18
mriedemlike jroll?13:18
mriedemjroll: does oath even pick up fixes from upstream stable?13:18
PapaOurslike above my shoulders13:18
jrollmriedem: yes, we build our packages from latest upstream branch + patches13:19
PapaOursjroll: I guess you have downstream branches too ?13:20
PapaOursso, EM isn't really a problem for you, right?13:20
jrollPapaOurs: we keep a repo of patches in `git format-patch` format that we apply, yes13:20
jrollI mean13:20
jrollwe can handle it ourselves if ocata disappears13:21
PapaOursthat's my point13:21
jrollbut we'd prefer to push any bugfixes back to share with others13:21
PapaOursor you can continue to feed the branch, if you will, that's the intent of EM13:21
*** yikun has quit IRC13:21
PapaOursit's just that it becomes out of the nova-stable team control13:21
jrollright13:22
jrollPapaOurs: I can't tell if you're trying to answer a question or what?13:22
jroll:)13:22
mdboothmriedem: Just looking at your comment on https://review.openstack.org/#/c/604400/. Did that job even get as far as running the evacuate test?13:22
mdboothLooks like it had tempest failures, no?13:22
mriedemmdbooth: it did for the local storage/lvm phase13:23
mdboothAh...13:23
PapaOursjroll: no, no, my point is that I don't see a reason why we should continue to have Ocata maintained by us upstream13:23
mriedemthen ceph + live migration failed b/c i don't think we bring one of the computes back up after the evacuate run in the first phase and wait for it to be ready before switching to ceph13:23
PapaOursjroll: so, even if people care of Ocata for various reasons, EM will still be there13:23
mdboothmriedem: K. I'll go look harder.13:23
jrollPapaOurs: yeah, I don't think anyone is disagreeing there13:24
*** efried is now known as fried_rice13:25
mdboothecho '2. NFS testing is skipped due to setup failures with Ubuntu 16.04'13:25
mdbooth#echo '2. test with NFS for root + ephemeral disks'13:25
*** derekh has quit IRC13:26
mdboothThere's my nfs testing :)13:26
*** derekh has joined #openstack-nova13:26
*** tbachman has joined #openstack-nova13:28
*** artom has joined #openstack-nova13:29
*** Luzi has quit IRC13:30
*** david-lyle has joined #openstack-nova13:34
* mriedem heads to the kinderbus13:36
*** dklyle has quit IRC13:37
*** _ix has joined #openstack-nova13:41
*** cdent has quit IRC13:41
*** mlavalle has joined #openstack-nova13:42
_ixGood morning, folks. I'm trying to validate my openstack installation with tempest, and although my machines are instantiating more-or-less correctly, metadata fails to be verified: `http://169.254.169.254/latest/meta-data/public-ipv4 is empty`13:43
*** zzzeek has quit IRC13:43
_ixOn existing, functional machines, trying to access that same endpoint is indeed empty. Is there something I'm missing here?13:43
*** artom has quit IRC13:43
*** artom has joined #openstack-nova13:44
*** zzzeek has joined #openstack-nova13:45
*** a-pugachev has quit IRC13:47
*** zzzeek has quit IRC13:48
openstackgerritGhanshyam Mann proposed openstack/nova master: Add hide server address tests in test_serversV21.py  https://review.openstack.org/60543513:48
*** zzzeek has joined #openstack-nova13:49
*** artom has quit IRC13:52
mdboothdansmith: lazy load of a field from a deleted object? https://review.openstack.org/#/c/604400/513:53
mriedemmdbooth: that's been fixed13:54
mriedemwell, something related to that has been fixed13:54
mriedemhttps://github.com/openstack/nova/blob/master/nova/objects/instance.py#L90013:54
mriedemhttps://github.com/openstack/nova/commit/6ba8a35825a7ec839b2d0aab7559351d573130ab13:55
mriedembut that's generic attributes, which flavor is not13:55
mriedemhttps://github.com/openstack/nova/blob/master/nova/objects/instance.py#L95813:55
*** Nel1x has joined #openstack-nova13:56
mriedemah yup http://logs.openstack.org/00/604400/5/check/nova-live-migration/6aa7a4b/logs/subnode-2/screen-n-cpu.txt.gz#_Sep_26_23_53_06_47500513:56
*** burt has joined #openstack-nova13:57
*** s10 has quit IRC13:57
mriedemmdbooth: good call13:57
mriedemdo we have a bug for that?13:57
*** awaugama has joined #openstack-nova13:57
mdboothmriedem: Haven't checked. I wrote that comment up in a panic as I'm about to be late for a meeting ;)13:58
*** s10 has joined #openstack-nova13:59
mriedemi'll look14:00
openstackgerritMatt Riedemann proposed openstack/nova stable/rocky: Null out instance.availability_zone on shelve offload  https://review.openstack.org/60608614:00
*** artom has joined #openstack-nova14:01
melwitt.14:03
*** cdent has joined #openstack-nova14:05
openstackgerritMerged openstack/nova master: Send soft_delete from context manager  https://review.openstack.org/47645914:06
mriedemhttps://bugs.launchpad.net/nova/+bug/179499614:07
openstackLaunchpad bug 1794996 in OpenStack Compute (nova) "_destroy_evacuated_instances fails and kills n-cpu startup if lazy-loading flavor on a deleted instance" [High,Triaged]14:07
*** ShilpaSD has quit IRC14:17
*** bnemec is now known as beekneemech14:20
jungleboyjmelwitt:  You around?14:20
melwittjungleboyj: yes14:20
jungleboyjmelwitt:  Morning!14:20
jungleboyjQuetion for you.14:20
melwittgmorning14:20
jungleboyjSo, talking to a customer asking about being able to replicate instances between multiple data centers.14:21
*** jaypipes has joined #openstack-nova14:21
jungleboyjSo, for DR, where if one DC goes down the other can just pick up.14:21
jungleboyjmelwitt:  Nova doesn't currently have anything like that.  Right?14:21
mriedemjungleboyj: no14:22
mriedemthe root volume attach/detach blueprint is semi related though14:22
mriedemhttps://review.openstack.org/#/c/600628/14:22
mriedemdetach root volume from DC1, attach new root volume from DC214:23
melwittjungleboyj: you mean like some sort of instance HA? we don't have that. downstream, we have some kind of scripts that do something to handle the use case but I'm not too familiar with them14:23
jungleboyjmriedem:  Ok.14:23
mriedemjungleboyj: kingbird is supposed to be related to this i think https://wiki.openstack.org/wiki/Kingbird in some way14:23
jungleboyjmelwitt:  Ah, so there are ways to make it happen but nothing that is community developed?14:24
jungleboyjOr supported by Nova?14:24
mriedemand freezer? https://docs.openstack.org/freezer/latest/14:24
mriedemnova does not natively orchestrate DR14:24
mriedemmasakari deals with instance HA14:24
mriedemfreezer and kingbird are about backups/DR/replication i think14:25
jungleboyjSo that is what all those other projects are about.  :-)14:25
*** edmondsw_ has joined #openstack-nova14:26
jungleboyjSo, it looks like the answer is that there are projects out there working on this kind of problem.14:27
melwittjungleboyj: yeah, I mean, we've talked about this before and DR/HA would be best orchestrated outside of nova by another service/entity. and I wasn't familiar with the projects mriedem has mentioned ^14:28
*** edleafe has quit IRC14:29
jungleboyjmelwitt:  We both learned something.  :-)14:29
*** edmondsw has quit IRC14:29
*** edmondsw_ is now known as edmondsw14:29
*** edleafe has joined #openstack-nova14:30
melwittyep :)14:30
jungleboyjmriedem:  Thank you very much.  Very helpful.14:35
mdboothmriedem: That bug iss an immediate practical win for your new test :)14:36
*** s10 has quit IRC14:37
*** s10 has joined #openstack-nova14:39
*** artom has quit IRC14:39
mdboothmriedem: Side note, I'm not a fan of using temporary_mutation on a context object, because context objects are shared between threads in various places. That could produce some incredibly weird behaviour would would be almost impossible to debug.14:41
mdboothBetter to use a copy14:42
mdboothOutside of tests it's only ever used to set read_deleted on a context object, so probably best to add a method to context which returns a copy with read deleted set14:43
mdbooth-> no possibility of weird threading issues14:44
mriedemmdbooth: i've got a recreate regression functional test and it's an easy fix on top,14:46
mriedemjust need to clean up these patches14:46
mriedemand yes it's amazing what you find when you actually have integration testing for features... :)14:46
*** munimeha1 has joined #openstack-nova14:54
*** Bhujay has quit IRC14:58
*** macza has joined #openstack-nova14:58
*** pooja_jadhav has quit IRC14:59
openstackgerritMatt Riedemann proposed openstack/nova master: Add functional regression test for bug 1794996  https://review.openstack.org/60610615:00
openstackbug 1794996 in OpenStack Compute (nova) rocky "_destroy_evacuated_instances fails and kills n-cpu startup if lazy-loading flavor on a deleted instance" [High,Triaged] https://launchpad.net/bugs/179499615:00
*** cfriesen has joined #openstack-nova15:05
*** jaypipes is now known as leakypipes15:06
*** artom has joined #openstack-nova15:09
openstackgerritBalazs Gibizer proposed openstack/nova master: Ignore forcing of evacuation for nested instance  https://review.openstack.org/60611115:12
gibletmriedem: FYI I applied the same ignore-the-force-flag tactic for the force evacuation with nested allocation case ^^15:12
*** _ix has quit IRC15:14
*** mgoddard has quit IRC15:16
*** cdent has quit IRC15:18
openstackgerritBalazs Gibizer proposed openstack/nova master: Run negative server moving tests with nested RPs  https://review.openstack.org/60412515:22
openstackgerritElod Illes proposed openstack/nova master: Reject networks with QoS policy  https://review.openstack.org/57007915:36
*** helenafm has quit IRC15:37
*** lbragstad is now known as elbragstad15:38
*** s10 has quit IRC15:41
*** burt has quit IRC15:41
*** burt has joined #openstack-nova15:43
*** mgoddard has joined #openstack-nova15:47
openstackgerritMatt Riedemann proposed openstack/nova master: Add functional regression test for bug 1794996  https://review.openstack.org/60610615:48
openstackbug 1794996 in OpenStack Compute (nova) rocky "_destroy_evacuated_instances fails and kills n-cpu startup if lazy-loading flavor on a deleted instance" [High,Triaged] https://launchpad.net/bugs/179499615:48
openstackgerritMatt Riedemann proposed openstack/nova master: Fix InstanceNotFound during _destroy_evacuated_instances  https://review.openstack.org/60612215:48
openstackgerritMatt Riedemann proposed openstack/nova master: Add functional regression test for bug 1794996  https://review.openstack.org/60610615:53
openstackbug 1794996 in OpenStack Compute (nova) "_destroy_evacuated_instances fails and kills n-cpu startup if lazy-loading flavor on a deleted instance" [High,In progress] https://launchpad.net/bugs/1794996 - Assigned to Matt Riedemann (mriedem)15:53
openstackgerritMatt Riedemann proposed openstack/nova master: Fix InstanceNotFound during _destroy_evacuated_instances  https://review.openstack.org/60612215:53
openstackgerritMatt Riedemann proposed openstack/nova master: Run evacuate tests with local/lvm and shared/rbd storage  https://review.openstack.org/60440015:53
openstackgerritMatt Riedemann proposed openstack/nova master: DNM: Test against mdbooth's evacuate patch  https://review.openstack.org/60442315:53
mriedemmelwitt: please to be approving backport to queens https://review.openstack.org/#/c/602409/15:55
*** ttsiouts has quit IRC15:55
mriedemoh and the change below it for testing15:56
*** cdent has joined #openstack-nova15:58
mriedemocata backport https://review.openstack.org/#/c/570653/ is ready to go16:06
mriedemand https://review.openstack.org/#/c/599885/116:08
cfriesenwhen adding a new flavor extra-spec, what's the appropriate place to document it?  embedded docs, reno, both?16:08
mriedemand https://review.openstack.org/#/c/583422/16:09
mriedemcfriesen: https://docs.openstack.org/nova/latest/user/flavors.html#extra-specs16:09
mriedemand reno16:09
cfriesenthanks16:09
openstackgerritMatt Riedemann proposed openstack/nova stable/ocata: Fix host validity check for live-migration  https://review.openstack.org/59061116:10
mriedemstable core should just fast approve this https://review.openstack.org/#/q/I9435fe84c8dc8a6f35ffb195ddbf71ca34edb242+status:open16:12
mriedemit avoids us running the ironic job on docs-only changes in stable16:12
*** fried_rice is now known as fried_rolls16:15
*** manjeets has quit IRC16:17
mdboothcfriesen: Are you in the US, btw?16:17
*** dpawlik has quit IRC16:18
mdboothcfriesen: The evacuate anti-affinity bug again: https://review.openstack.org/#/c/605436/ now with support for max_server_per_host16:18
mdboothThat actually made it a bit simpler :)16:18
*** manjeets has joined #openstack-nova16:18
mdboothI feel like we're squashing quite a lot of evacuate bugs16:19
mdboothcfriesen: Apologies, Canada. Of course I knew that...16:20
mdboothI had it in my head you were in the UK for some bizarre reason.16:20
mriedemocata-em call to arms email sent16:21
*** mriedem is now known as mriedem_lunch16:21
cfriesenmdbooth: no worries, we have the same queen. :)16:22
mdboothIncidentally, as a bonus the above patch will also correctly handle multiple builds from the same anti-affinity group sent to a single compute. Previously they could all be rejected, now it will always allow max_server_per_host to succeed16:22
mdboothThat's builds in addition to rebuilds16:23
*** gyee has joined #openstack-nova16:24
mdboothIs there some documentation somewhere of the query language used by logstash?16:31
mdboothThis was what got me last time I tried to use this.16:31
cfriesenmdbooth: do we still need the nested contexts given we're basically doing refcounting?  I mean, it's an elegant way to ensure we call builders.remove(instance.uuid) but we could presumably also do that other ways.  Just wondering if it'd end up being less code churn.16:31
mdboothcfriesen: I'd strongly prefer to keep the nested contexts16:32
mdboothOtherwise you'll end up having disjoint code, which is a maintenance nightmare16:32
mdboothand we have enough of it already16:33
mdboothcfriesen: Consider all the error cleanup edge cases you'd have to consider16:34
cfriesenfair enough...just thought I'd mention it16:34
mdboothIncidentally, it also makes sense to do this test *before* taking a claim, as it's really scheduler take 216:35
cfriesenarguably both the claim and the affinity validation are both scheduler take 2.16:37
cfriesenbut sure, it'd be less wasted work to check affinity first16:38
mdboothafaict the syntax documented here is wrong: https://docs.openstack.org/infra/system-config/logstash.html#query-logstash16:41
mdboothI can't make any @message query return anything16:42
mdbooth@fields.loglevel:"ERROR" returns nothing16:43
mdbooth...which is nice. We have no errors :)16:43
*** munimeha1 has quit IRC16:43
mdboothSimilarly striking out here: https://www.elastic.co/guide/en/kibana/3.0/queries.html16:45
*** dtantsur is now known as dtantsur|afk16:45
mdboothAh, ha! Drop the @ seems to work16:48
*** priteau has joined #openstack-nova16:54
*** derekh has quit IRC16:55
*** jpena is now known as jpena|off16:57
*** Nel1x has quit IRC17:04
*** psachin has joined #openstack-nova17:09
melwittmriedem_lunch: roger copy17:11
*** mdbooth has quit IRC17:15
*** mdrabe has quit IRC17:16
openstackgerritJack Ding proposed openstack/nova master: Handle missing marker during online data migration  https://review.openstack.org/60516417:19
*** macza has quit IRC17:24
*** macza has joined #openstack-nova17:24
*** ejat has quit IRC17:29
*** macza has quit IRC17:29
*** tssurya has quit IRC17:31
openstackgerritStephen Finucane proposed openstack/nova master: doc: Rewrite the console doc  https://review.openstack.org/60614817:37
finucannotmelwitt, mriedem_lunch: You two might be interested in that ^17:38
* finucannot heads home17:38
melwittack17:38
*** sapd1_ has joined #openstack-nova17:51
*** mriedem_lunch has quit IRC17:52
*** mriedem has joined #openstack-nova17:54
mriedemeasy fix needs a +W https://review.openstack.org/#/c/605748/17:55
*** manjeets has quit IRC17:56
*** david-lyle has quit IRC17:56
*** karimull has quit IRC17:56
*** alex_xu has quit IRC17:56
*** TheJulia is now known as needssleep18:02
*** macza has joined #openstack-nova18:07
openstackgerritMatt Riedemann proposed openstack/nova stable/queens: Null out instance.availability_zone on shelve offload  https://review.openstack.org/60615518:12
*** macza has quit IRC18:12
imacdonnmriedem: Fix merged in os-brick that will hopefully address https://bugs.launchpad.net/nova/+bug/1732199 - thing is, I'm not sure where zuul gets os-brick from - it appears to be using /usr/local/lib/python2.7/dist-packages/os_brick, so not sure if it needs a release18:15
openstackLaunchpad bug 1732199 in OpenStack Compute (nova) "test_extend_attached_volume fails with Unexpected compute_extend_volume result 'Error'" [Medium,Confirmed]18:15
*** med_ has joined #openstack-nova18:15
mriedemimacdonn: yes nova can't use it until it's released and upper-constraints has that new version18:20
mriedemimacdonn: so probably work with jungleboyj to see when he's ready to do a new os-brick release18:21
jungleboyj:-)18:21
jungleboyjimacdonn:  I can look at doing that.18:22
jungleboyjIt will cost you a beer though.  ;-)18:22
imacdonnmriedem jungleboyj: OK. We might want to let this soak a little bit ... I think Gorka was a bit apprehensive about the fix18:22
mriedemwfm,18:22
jungleboyjimacdonn:  Ok, want to put it on next week's meeting agenda and we can see how things look then?18:22
mriedemit's been a latent issue since queens18:23
mriedemnote that nothing will be using this in CI until it's released and in upper-constraints18:23
mriedemso i'm not sure how you're going to soak it in18:23
mriedemwell, except os-brick CI itself18:23
jungleboyjmriedem:  Oh, good point.18:23
imacdonnyeah, I was just pondering that too18:24
imacdonnI just I'd at least want to give Gorka a chance to object before a release is made18:24
imacdonnI guess*18:24
jungleboyjI can propose a release and tag Gorka to review.18:25
*** manjeets has joined #openstack-nova18:25
imacdonnOK, that works18:25
jungleboyjLets do that.18:25
openstackgerritMatt Riedemann proposed openstack/nova stable/pike: Null out instance.availability_zone on shelve offload  https://review.openstack.org/60616118:25
*** macza has joined #openstack-nova18:30
*** david-lyle has joined #openstack-nova18:41
*** david-lyle is now known as dklyle18:42
*** mriedem has quit IRC18:44
*** mgoddard has quit IRC18:47
*** med_ has quit IRC19:00
*** elbragstad has quit IRC19:10
*** elbragstad has joined #openstack-nova19:10
*** artom has quit IRC19:22
*** tssurya has joined #openstack-nova19:22
*** sapd1_ has quit IRC19:24
*** artom has joined #openstack-nova19:30
*** ralonsoh has quit IRC19:35
melwittzzzeek: hey, are you around?19:38
zzzeekmelwitt: heya19:38
melwittzzzeek: hey, I'm looking at how we can eliminate the warnings we're getting from our code, "OsloDBDeprecationWarning: EngineFacade is deprecated; please use oslo_db.sqlalchemy.enginefacade'"19:39
melwittI suspect they're coming from calls like this https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/api.py#L13719:39
zzzeekmelwitt: probably19:40
melwittand I'm looking for the correct supported way in oslo.db to achieve the same thing19:40
zzzeekmelwitt: there might be a direct engine accessor at this point but the original idea was you wouldnt be using an "Engine" directly19:41
zzzeekmelwitt: looks like you can just drop the get_legacy_facade() part19:41
melwitthm, ok19:41
zzzeekmelwitt: there's a get_engine() now19:41
melwittzzzeek: I saw that, but it doesn't have the ability to specify use_slave, correct?19:42
zzzeekmelwitt: the "async reader" would be the "slave"19:42
zzzeekso...allow_async.reader.get_engine() ?    I guess19:43
melwittoh, I see. ok19:43
melwittI think that would do it then, thank you. looking to make an incremental progress19:43
zzzeekmelwitt: or maybe async_.reader.get_engine....sorry i havent looked at this in over a year19:44
melwittzzzeek: np, I can find it. just needed that hint19:45
zzzeekmelwitt: Ok i think "reader engine" is the slave, async or not19:47
*** mriedem has joined #openstack-nova19:50
*** mdbooth has joined #openstack-nova19:54
*** awaugama has quit IRC20:05
*** erlon has quit IRC20:07
*** openstackgerrit has quit IRC20:07
*** mdbooth has quit IRC20:10
*** psachin has quit IRC20:10
*** openstackgerrit has joined #openstack-nova20:25
openstackgerritCarlos Goncalves proposed openstack/os-vif master: Introduce network namespace aware plugging  https://review.openstack.org/54696120:25
melwittzzzeek: ok, so the old way was context_manager.get_legacy_facade().get_engine() but when I do context_manager.get_engine() it does ValueError("mode should be WRITER or READER")'20:31
zzzeekmelwitt: yes, try context_manager.reader.get_engine()20:31
melwittzzzeek: ok, it's just that these get_engine methods we have were not calling out whether they were reader or writer20:32
zzzeekmelwitt: the reader is the "slave"20:32
zzzeekyou want the "slave" right20:32
melwittzzzeek: not always, only if use_slave=True was passed20:32
zzzeekmelwitt: if you want the non-slave hten ctx.writer.get_engine()20:32
melwittzzzeek: ok. what about this one? is this one implying that it is a writer then? https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/api.py#L14020:33
zzzeekthat sohuld be the same as the "writer" yes20:34
melwittok, thank you. I didn't realize that's what they were underneath20:34
melwittah, yup I see it here now https://github.com/openstack/oslo.db/blob/dff11337f55b7a9a28d7c30658df51b7b0b443eb/oslo_db/sqlalchemy/enginefacade.py#L129020:44
openstackgerritMerged openstack/nova stable/ocata: Cleanup the caches when deleting a resource provider  https://review.openstack.org/60402120:46
openstackgerritMerged openstack/nova stable/queens: libvirt: Always escape IPv6 addresses when used in migration URI  https://review.openstack.org/60373720:46
*** artom has quit IRC20:54
*** panda has quit IRC20:57
*** panda has joined #openstack-nova20:58
*** PapaOurs is now known as bauzas21:05
openstackgerritMerged openstack/nova master: nova-manage - fix online_data_migrations counts  https://review.openstack.org/60532921:14
openstackgerritMerged openstack/nova stable/queens: cover migration cases with functional tests  https://review.openstack.org/60240821:15
*** mgoddard has joined #openstack-nova21:21
*** tssurya has quit IRC21:26
*** slaweq has quit IRC21:28
*** mriedem has quit IRC21:37
*** priteau has quit IRC21:42
*** mgoddard has quit IRC21:43
openstackgerritmelanie witt proposed openstack/nova master: Replace usage of get_legacy_facade() with get_engine()  https://review.openstack.org/60621322:10
*** artom has joined #openstack-nova22:13
*** fried_rolls is now known as efried22:20
*** panda is now known as panda|off22:26
openstackgerritmelanie witt proposed openstack/nova master: Use an alternative class 'MoxStubout'  https://review.openstack.org/57948222:38
*** macza_ has joined #openstack-nova22:40
*** macza has quit IRC22:42
*** elbragstad has quit IRC22:48
*** macza_ has quit IRC22:53
*** macza has joined #openstack-nova22:53
*** macza has quit IRC22:58
*** elbragstad has joined #openstack-nova23:01
*** elbragstad has quit IRC23:11
*** macza has joined #openstack-nova23:17
*** macza has quit IRC23:21
*** cdent has quit IRC23:22
*** gyee has quit IRC23:33
*** mdbooth has joined #openstack-nova23:36
*** mdbooth has quit IRC23:42
*** mdbooth has joined #openstack-nova23:46

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