Tuesday, 2019-03-12

« Monday, 2019-03-11 Index Wednesday, 2019-03-13 »
openstackgerritMerged openstack/nova master: Remove fake_libvirt_utils from the cache concurrency tests.  https://review.openstack.org/64255600:12
*** gyee has quit IRC00:16
openstackgerritKashyap Chamarthy proposed openstack/nova master: qemu: Make disk image conversion dramatically faster  https://review.openstack.org/64078100:28
openstackgerritKashyap Chamarthy proposed openstack/nova master: libvirt: Use 'writeback' QEMU cache mode for Nova instances  https://review.openstack.org/64198100:28
*** erlon has joined #openstack-nova00:41
openstackgerritKashyap Chamarthy proposed openstack/nova-specs master: WIP: Gracefully handle QEMU machine types for guests  https://review.openstack.org/63115400:44
*** marst has joined #openstack-nova00:49
*** lbragstad has quit IRC00:49
*** dave-mccowan has joined #openstack-nova00:54
*** marst has quit IRC00:54
*** erlon has quit IRC00:57
openstackgerritMatt Riedemann proposed openstack/nova stable/rocky: Handle templated cell mappings in nova-status  https://review.openstack.org/64261101:03
*** agopi has quit IRC01:04
*** whoami-rajat has joined #openstack-nova01:12
*** wolverineav has quit IRC01:18
*** wolverineav has joined #openstack-nova01:18
*** wolverineav has quit IRC01:23
*** itlinux has joined #openstack-nova01:25
*** wolverineav has joined #openstack-nova01:33
*** wolverineav has quit IRC01:33
*** mvkr has quit IRC01:36
*** mvkr has joined #openstack-nova01:38
*** erlon has joined #openstack-nova01:48
*** mvkr has quit IRC01:49
*** itlinux has quit IRC01:51
*** mvkr has joined #openstack-nova01:52
*** hongbin has joined #openstack-nova01:53
*** tetsuro has joined #openstack-nova02:10
*** openstackstatus has quit IRC02:22
*** openstack has joined #openstack-nova02:24
*** ChanServ sets mode: +o openstack02:24
*** BjoernT has joined #openstack-nova02:41
*** Dinesh_Bhor has quit IRC02:49
*** BjoernT has quit IRC02:50
*** lbragstad has joined #openstack-nova02:58
*** wolverineav has joined #openstack-nova02:59
*** agopi has joined #openstack-nova03:06
*** lbragstad has quit IRC03:07
*** BjoernT has joined #openstack-nova03:12
*** hongbin has quit IRC03:33
*** erlon has quit IRC03:35
*** dave-mccowan has quit IRC03:40
*** munimeha1 has quit IRC03:43
*** liuyulong has quit IRC03:47
*** udesale has joined #openstack-nova03:49
*** Ahzzmandius has quit IRC03:52
*** mchlumsky has quit IRC03:58
*** BjoernT has quit IRC04:03
*** mordred has quit IRC04:14
*** mordred has joined #openstack-nova04:21
*** mordred has quit IRC04:37
*** janki has joined #openstack-nova04:38
*** udesale has quit IRC04:45
*** mordred has joined #openstack-nova04:45
*** udesale has joined #openstack-nova04:46
*** abhishekk has joined #openstack-nova04:57
*** wolverineav has quit IRC05:00
*** wolverineav has joined #openstack-nova05:01
*** markvoelker has joined #openstack-nova05:06
*** wolverineav has quit IRC05:17
*** wolverineav has joined #openstack-nova05:18
*** sridharg has joined #openstack-nova05:22
*** wolverineav has quit IRC05:22
*** sapd1 has joined #openstack-nova05:24
*** wolverineav has joined #openstack-nova05:25
*** wolverineav has quit IRC05:30
*** wolverineav has joined #openstack-nova05:32
*** wolverineav has quit IRC05:40
openstackgerritTakashi NATSUME proposed openstack/nova master: api-ref: Add description for BDM volume_size  https://review.openstack.org/64125505:57
openstackgerritTakashi NATSUME proposed openstack/nova master: api-ref: Add description for BDM volume_size  https://review.openstack.org/64125505:57
*** slaweq_ has joined #openstack-nova06:00
*** tristanC has quit IRC06:02
*** wolverineav has joined #openstack-nova06:04
*** tristanC2 has joined #openstack-nova06:04
*** wolverineav has quit IRC06:04
*** slaweq has quit IRC06:07
*** kevinbenton has quit IRC06:07
*** yankcrime has quit IRC06:07
*** mgoddard has quit IRC06:07
*** HD|Laptop has quit IRC06:07
*** kevinbenton has joined #openstack-nova06:08
*** gary_perkins has quit IRC06:08
*** andreaf has quit IRC06:09
*** gary_perkins has joined #openstack-nova06:10
*** mgoddard has joined #openstack-nova06:10
*** panda|rover|off has quit IRC06:10
*** andreaf has joined #openstack-nova06:12
*** panda has joined #openstack-nova06:12
*** wolverineav has joined #openstack-nova06:14
*** dr_gogeta86 has quit IRC06:16
*** masayukig[m] has joined #openstack-nova06:19
*** wolverineav has quit IRC06:32
*** masayukig[m] is now known as masayukig[m]106:35
*** masayukig[m]1 is now known as masayukig[m]206:35
*** Luzi has joined #openstack-nova06:53
*** jlejeune has quit IRC07:00
openstackgerritYing Wang proposed openstack/nova stable/rocky: Modified the filter regulation of 'Instance Name' for filter instance  https://review.openstack.org/64265907:01
*** ivve has joined #openstack-nova07:23
*** dpawlik has joined #openstack-nova07:30
openstackgerritBoxiang Zhu proposed openstack/nova master: Remove the unsafe convert of lvm imagebackend  https://review.openstack.org/64266707:37
*** pcaruana has joined #openstack-nova07:39
*** owalsh has quit IRC07:39
*** pcaruana has quit IRC07:43
*** pcaruana has joined #openstack-nova07:43
openstackgerritYing Wang proposed openstack/nova master: It was modified to the exact filter regulation of 'Instance Name' for filter instance  https://review.openstack.org/64267507:54
*** wolverineav has joined #openstack-nova07:55
*** wolverineav has quit IRC07:59
*** rpittau|afk is now known as rpittau07:59
*** takashin has left #openstack-nova08:00
*** tkajinam has quit IRC08:07
*** tssurya has joined #openstack-nova08:08
*** awalende has joined #openstack-nova08:15
*** tesseract has joined #openstack-nova08:16
*** luksky has joined #openstack-nova08:20
*** ccamacho has joined #openstack-nova08:21
*** xek has joined #openstack-nova08:34
*** ralonsoh has joined #openstack-nova08:36
*** sapd1 has quit IRC08:47
*** ttsiouts has joined #openstack-nova08:49
openstackgerritBoxiang Zhu proposed openstack/nova master: Remove the unsafe convert of lvm imagebackend  https://review.openstack.org/64266708:51
*** dtantsur|afk is now known as dtantsur08:52
*** luksky has quit IRC08:52
*** yankcrime has joined #openstack-nova08:55
*** luksky has joined #openstack-nova08:57
*** tetsuro has quit IRC08:59
*** priteau has joined #openstack-nova09:02
*** ttsiouts has quit IRC09:04
*** ttsiouts has joined #openstack-nova09:05
*** tetsuro has joined #openstack-nova09:08
*** tetsuro has quit IRC09:10
*** snevi has joined #openstack-nova09:11
*** derekh has joined #openstack-nova09:34
*** mikal has quit IRC09:37
*** mikal has joined #openstack-nova09:37
*** mikal has quit IRC09:42
*** tosky has joined #openstack-nova09:44
*** helenafm has joined #openstack-nova09:52
openstackgerritya.wang proposed openstack/nova master: Update docs: User token times out during long-running operations  https://review.openstack.org/64241009:59
*** owalsh has joined #openstack-nova10:14
*** cdent has joined #openstack-nova10:22
*** panda is now known as panda|rover10:22
*** lpetrut has joined #openstack-nova10:33
*** tbachman has quit IRC10:40
*** liuyulong has joined #openstack-nova11:17
*** dave-mccowan has joined #openstack-nova11:23
*** ttsiouts has quit IRC11:45
*** ttsiouts has joined #openstack-nova11:45
*** owalsh has quit IRC11:50
*** ttsiouts has quit IRC11:50
*** owalsh has joined #openstack-nova11:52
*** awalende has quit IRC11:53
*** awalende has joined #openstack-nova11:56
*** owalsh has quit IRC11:56
*** mikal has joined #openstack-nova11:56
*** owalsh has joined #openstack-nova11:57
*** tbachman has joined #openstack-nova11:59
*** awalende has quit IRC12:00
*** mikal has quit IRC12:01
*** awalende has joined #openstack-nova12:03
*** owalsh has quit IRC12:06
*** owalsh has joined #openstack-nova12:07
*** ttsiouts has joined #openstack-nova12:09
*** markvoelker has quit IRC12:18
*** owalsh has quit IRC12:18
*** luksky has quit IRC12:19
*** owalsh has joined #openstack-nova12:21
*** panda|rover is now known as panda|rover|lunc12:21
*** liuyulong_ has joined #openstack-nova12:31
*** jaosorior has quit IRC12:43
*** udesale has quit IRC12:50
*** udesale has joined #openstack-nova12:51
*** mriedem has joined #openstack-nova12:53
*** janki has quit IRC12:59
*** mlavalle has joined #openstack-nova12:59
*** awalende has quit IRC12:59
*** janki has joined #openstack-nova12:59
*** owalsh has quit IRC13:01
*** owalsh has joined #openstack-nova13:02
*** awalende has joined #openstack-nova13:03
*** FlorianFa has quit IRC13:03
*** sapd1 has joined #openstack-nova13:03
*** luksky has joined #openstack-nova13:08
*** slaweq_ is now known as slaweq13:08
*** owalsh has quit IRC13:10
*** panda|rover|lunc is now known as panda|rover13:12
mriedemlyarwood: a few small things in https://review.openstack.org/#/c/551349/13:13
*** yan0s has joined #openstack-nova13:13
*** jaosorior has joined #openstack-nova13:15
*** owalsh has joined #openstack-nova13:15
*** lbragstad has joined #openstack-nova13:17
*** owalsh has quit IRC13:19
*** owalsh has joined #openstack-nova13:20
*** dave-mccowan has quit IRC13:22
*** marst has joined #openstack-nova13:26
mriedemgibi: heh found a use case for something we talked about recently https://review.openstack.org/#/c/387687/13:26
*** irclogbot_2 has quit IRC13:26
mriedemcompute node went down, admin unbinds a port to re-use it elsewhere, compute node comes back up and delete the old server and nova deletes the port13:27
mriedemor unbinds it from the server it was currently bound to13:27
*** irclogbot_2 has joined #openstack-nova13:28
gibimriedem: hm, that sounds like a bug13:31
gibimriedem: nova should check if the port is bound to the same server as nova things to bound to before unbind13:32
mriedemyeah there is an old bug for it13:34
mriedemhttps://bugs.launchpad.net/nova/+bug/163426913:34
openstackLaunchpad bug 1634269 in OpenStack Compute (nova) "Incorrect port unbind request on instance delete" [Low,In progress] - Assigned to AJAY KALAMBUR (akalambu)13:34
openstackgerritMatt Riedemann proposed openstack/nova master: Fix incorrect port unbind request in nova on instance delete  https://review.openstack.org/38768713:34
*** dave-mccowan has joined #openstack-nova13:35
gibiOK, so in the fix we simply refresh nova's view based on neutron's view and only unbind if the vif is still belongs to the instance13:37
gibithat could work13:37
*** marst has quit IRC13:38
gibiin a similar scenario with bandwidth when neutron admin forcefully unbind the port then the resource allocation view will become inaccurate13:38
gibithen when the old nova instance is deleted the whole resource allocation is deleted in placement so the inventory become correct eventually13:39
gibihm, in the bug, the instance is deleted while the compute was still down. I'm not sure when the allocation for such instance is deleted from placement13:41
gibiOK. compute.api._local_delete calls delete_allocation_for_instance() so if the admin unbinds the port _after_ the instance is local deleted then the resource view is in a correct state13:43
gibimriedem: So I don't see that this sceanrio leads to any issue with bandwidth resource13:46
mriedemthis bug predates placement by a long shot13:46
mriedemi'm not saying it does,13:46
mriedemwe have just talked about the weird corner case of unbinding ports out of band13:47
*** owalsh has quit IRC13:50
gibimriedem: yeah, the use case seems valid13:51
*** lennyb has quit IRC13:51
*** mlavalle has quit IRC13:52
openstackgerritJack Ding proposed openstack/nova master: Preserve UEFI NVRAM variable store  https://review.openstack.org/62164613:53
*** mlavalle has joined #openstack-nova13:54
mriedemmelwitt: is this really stein rc potential? https://bugs.launchpad.net/nova/+bug/1816399 - isn't that a latent issue since rocky?13:56
openstackLaunchpad bug 1816399 in OpenStack Compute (nova) "The periodic task to clean up expired console_auth tokens is invalid" [Undecided,In progress] - Assigned to Takashi NATSUME (natsume-takashi)13:56
*** owalsh has joined #openstack-nova13:56
*** awaugama has joined #openstack-nova13:56
*** openstack has joined #openstack-nova15:39
*** ChanServ sets mode: +o openstack15:39
*** jchhatbar has quit IRC15:47
*** ttsiouts has joined #openstack-nova15:48
*** sapd1 has quit IRC15:53
*** Vek has joined #openstack-nova15:58
*** sapd1 has joined #openstack-nova16:01
*** cfriesen has joined #openstack-nova16:04
*** ivve has quit IRC16:07
*** sapd1 has quit IRC16:08
openstackgerritMatt Riedemann proposed openstack/nova-specs master: Re-propose cross-cell-resize spec for Train  https://review.openstack.org/64280716:10
dansmithmriedem: regarding this: https://review.openstack.org/#/c/567701, where is the az set on the reqspec during live migration?16:12
*** ttsiouts has quit IRC16:14
*** ttsiouts has joined #openstack-nova16:14
mriedemthe request spec az is set on server create and never changes16:14
dansmithokay I'm missing something about your assertion in the commit message then16:15
dansmithyou say the reqspec.az is what the az filter looks at, and that means the instance can be moved across azs, which I assumed meant we update it so after you move it it's correct,16:16
dansmithbut if not, then if you force a move across azs, and then do something like a resize,16:16
dansmiththe instance will pop back over to the old az during scheduling, yeah? Is that what you meant?16:16
*** weshay is now known as Dwight16:18
*** Dwight is now known as DwightH16:18
*** DwightH is now known as weshay16:19
mriedemreplied inline16:19
dansmithmriedem: okay I'm more talking about the case where az *is* set16:20
dansmithagree that if it's not set, it shouldn't matter, that's the easy case16:20
mriedemi wasn't considering the force case here,16:21
mriedemthat's a separate f'ed up issue that evacuate will also suffer from16:21
mriedembut yeah, if you request server create in AZ1, admin forces live migrate or evacuate to AZ2, and then the user resizes, they'll wind up back in AZ116:21
mriedemb/c of the request spec16:21
mriedemanother example of why we removed the force parameter16:22
mriedembut it's a separate bug imo16:22
*** agopi has joined #openstack-nova16:22
dansmithmriedem: I'm more interested in the v2.1 case, since that's all that osc seems to support right now, which by my reading is kinda equivalent to force=True with a destination right?16:24
dansmithmriedem: I'm actually asking about this for another reason, not for confirmation of your patch itself16:24
mriedem"which by my reading is kinda equivalent to force=True with a destination right" yup16:26
mriedemalso,16:27
mriedemhttps://docs.openstack.org/python-openstackclient/latest/cli/command-objects/server.html#server-migrate is totally borked in this regard,16:27
dansmithyes16:27
mriedemif you're using osc to do live migrations, you're *always* forcing by default16:27
mriedemit's mega fucked16:27
*** gyee has joined #openstack-nova16:27
mriedemi've talked with dean about this and unfucking it means essentially a major version bump16:28
mriedemb/c it has to be an interface change16:28
mriedemwhat you're talking about with force not updating the request spec az is a latent bug yeah16:28
*** ccamacho has quit IRC16:29
mriedemhttps://review.openstack.org/#/c/627801/16:29
mriedemmaybe should queue that up for the ptg16:30
mriedemmordred: i added an item to the nova train ptg etherpad about finally unbreaking the osc server migrate interface, since we should shit or get off that pot16:33
mriedem~L104 https://etherpad.openstack.org/p/nova-ptg-train16:34
dansmithmriedem: on the --force removal, we just removed that flag and use the presence of the host parameter to indicate that we're serious?16:34
dansmithbecause I thought the point was to be able to provide a host and let the scheduler sanity check it, and force was "this is an emergency just do it"16:34
mriedemyeah - the provided host is a requested destination and the only one the scheduler will look at16:35
dansmithso we've lost the emergency button?16:35
mriedemif you're using the latest microversion yes,16:35
mriedembut you can still 2.1 yourself into oblivion if you want16:36
dansmithI vaguely recall these discussions of course, I'm just trying to fit my head around the current state16:36
mriedemwe also intentionally broke force for any servers that have nested allocations16:36
mriedemregardless of microversion16:36
dansmithif we start saying 2.1 is how you override things in an emergency then we really need to put the cap on "we're never bumping the minimum" :)16:37
*** udesale has quit IRC16:37
dansmithin the 2.1 case where you provide a host, we're still updating allocations though as far as I can see16:37
dansmithyou just mean nested allocs specifically?16:37
mriedemwe'll just do something dumb like adding the force parameter back16:37
mriedemforcev2!16:37
mriedemdansmith: yes if you force (2.1 where host is not None), we copy the allocations over to the forced dest compute node provider16:38
mriedembut only if they are flat allocations16:38
mordredmriedem: ok. happy to help fixing that16:38
mriedemdansmith: that's this part https://github.com/openstack/nova/blob/e608568518ed91a0cbf08f779c5adb851762d80a/nova/conductor/tasks/live_migrate.py#L11116:39
*** tssurya has quit IRC16:40
*** snevi is now known as IvensZambrano16:40
dansmithmriedem: ack eyah16:40
*** yan0s has quit IRC16:41
mriedemso it sounds like you want me to change https://review.openstack.org/#/c/567701/ to at the very least save off the current instance.availabilty_zone before setting it based on the new dest host selection and then if we hit rollback to set it back to the saved off value?16:42
mriedemwhich would only happen if i guess the rpc cast blew up?16:42
mriedemdansmith: doing it on the compute when we change instance.host is a non-starter if you care about up-calls16:43
dansmithmriedem: you can do it in the cell on the instance :)16:44
dansmithas long as we pass the az into compute so it doesn't have to look it up, which is what I was saying was likely an rpc change16:44
dansmithmriedem: I don't know what I want really.. what you're doing seems wrong, but what I did was equally wrong although slightly less concerning because of how those actions work16:45
*** wolverineav has joined #openstack-nova16:45
dansmithso I dunno, do we justify it with prior sins?16:45
*** wolverineav has quit IRC16:45
mriedemi can't backport an rpc change16:45
*** wolverineav has joined #openstack-nova16:45
mriedemwe already have several places where we know we need to pass the az down to compute to avoid up-calls https://bugs.launchpad.net/nova/+bug/178128616:46
openstackLaunchpad bug 1781286 in OpenStack Compute (nova) "CantStartEngineError in cell conductor during reschedule - get_host_availability_zone up-call" [Medium,Triaged]16:46
mriedemwhich reminds me https://review.openstack.org/#/c/581912/16:47
*** wolverineav has quit IRC16:47
dansmithhow about we just toss a grenade in this mofo and run?16:48
*** wolverineav has joined #openstack-nova16:48
mriedemwfm16:48
*** igordc has quit IRC16:48
cfriesenwhat's the current behaviour for resizing from a flavor with swap to one without swap?  Do we allow it?16:50
mriedemcfriesen: i'm fairly certain you ask this once per quarter16:51
mriedemit's time for a tattoo16:51
cfriesenas of last year it was supposed to work, but one of our testers just said it was busted16:51
*** igordc has joined #openstack-nova16:52
cfriesenI'm looking for automated tests of it16:52
mriedemhttps://bugs.launchpad.net/nova/+bug/176242316:52
openstackLaunchpad bug 1762423 in OpenStack Compute (nova) "Resize to swapless flavor still looking for swap" [Low,In progress] - Assigned to jichenjc (jichenjc)16:52
*** igordc has quit IRC16:53
cfriesenaccording to my comment at the bottom of https://bugs.launchpad.net/nova/+bug/1552777 it was working in Pike16:53
openstackLaunchpad bug 1552777 in OpenStack Compute (nova) "resizing from flavor with swap to one without swap puts instance into Error status" [Medium,Fix released] - Assigned to Kam Nasim (knasim-wrs)16:53
mriedemcfriesen: were you testing nova or starlingx?16:55
cfriesenmriedem: pretty sure it was devstack, but can't say for sure now16:55
*** igordc has joined #openstack-nova16:56
*** helenafm has quit IRC16:56
mriedemdansmith: unrelated softball https://review.openstack.org/#/c/642611/16:56
dansmith*crack*16:57
* mriedem goes to eat his feelings16:57
*** rpittau is now known as rpittau|afk16:59
*** ttsiouts has quit IRC17:01
*** ttsiouts has joined #openstack-nova17:01
openstackgerritBalazs Gibizer proposed openstack/nova master: Documentation for bandwidth support  https://review.openstack.org/64206417:05
*** tesseract has quit IRC17:05
*** ttsiouts has quit IRC17:06
*** eandersson_ has joined #openstack-nova17:07
*** priteau has quit IRC17:08
*** igordc has quit IRC17:10
*** igordc has joined #openstack-nova17:21
*** panda|rover is now known as panda|rover|off17:24
*** READ10 has joined #openstack-nova17:33
*** dtantsur is now known as dtantsur|afk17:39
mriedemefried: i'm mostly ok with https://review.openstack.org/#/c/640879/ but comments inline - the ironicclient valid_interfaces patch17:43
*** igordc has quit IRC17:43
mriedemi don't know if you can wordsmith something in as a code comment and commit message details, or if it's not worth it...17:45
*** igordc has joined #openstack-nova17:48
*** luksky has quit IRC17:50
*** READ10 has quit IRC17:50
*** ivve has joined #openstack-nova17:52
*** zhubx has quit IRC17:55
*** zhubx has joined #openstack-nova17:55
*** derekh has quit IRC17:55
openstackgerritMatt Riedemann proposed openstack/nova master: RT: replace _instance_in_resize_state with _is_trackable_migration  https://review.openstack.org/56046718:04
openstackgerritJeremy Stanley proposed openstack/nova stable/rocky: [DNM] Exercising base-test for a rocky change  https://review.openstack.org/64258218:09
mriedemstephenfin: regarding https://review.openstack.org/#/c/641653/ - if that feature *never worked* since queens, doesn't backporting this stuff now essentially mean we're backporting a feature?18:10
openstackgerritJeremy Stanley proposed openstack/nova master: [DNM] Exercising base-test for a master change  https://review.openstack.org/64258018:10
*** erlon has quit IRC18:15
openstackgerritJeremy Stanley proposed openstack/nova stable/rocky: [DNM] Exercising base-test for a master change  https://review.openstack.org/64258218:16
dansmithmriedem: hmm.. "claiming will always fail"18:17
dansmithmriedem: I'm trying to figure out what the upgrade impact of this is18:18
*** sridharg has quit IRC18:18
dansmithah, so I guess it's not that terrible upgrade-wise, since we had blob pci requests before without this field18:21
dansmithso this makes us save it, and load it from the request if it's there18:21
dansmithwhich it might not be because the request was old, or because we were running the code for this and not saving it18:21
dansmithdefinitely sounds fishy though, I'm not sure how this could have passed tests18:22
mriedem"tests"18:22
mriedemit's enabling a feature as far as i can tell18:22
dansmithI mean according to the bug it doesn't work _at_all_18:23
mriedemlooking back on the blueprint code that merged in queens, the only tests are unit tests18:24
dansmithbut I mean, I expected this was being tested on real hardware with an "it's too hard to automate that" sort of justification18:24
mriedemone would hope18:25
mriedemsomething something this is why we push for at least functional tests with stubs18:25
*** gmann is now known as gmann_afk18:26
mriedemdansmith: want to just fast approve this backport https://review.openstack.org/#/c/641401/18:27
mriedemi'm trying to flush stable queues18:27
mriedemand https://review.openstack.org/#/c/641355/18:27
dansmithkewwwwwisssshh18:28
mriedemi'm not sure what that sound is, you flushing a toilet?18:29
mriedemoh right, flush18:29
mriedemheh18:29
dansmithum, duh18:29
mriedemat first i was thinking about basketball and sinking a shot...18:29
mriedemi.e. swisssshhh18:29
mriedembut then i thought, "dan making basketball references?"18:29
dansmithhahaah, right18:29
mriedemok rocky is ready for a few more if some stable cores want to look https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:stable/rocky+label:Code-Review=218:36
*** pcaruana has quit IRC18:39
openstackgerritMerged openstack/nova master: Share snapshot image membership with instance owner  https://review.openstack.org/63076918:43
*** IvensZambrano has quit IRC18:47
openstackgerritEric Fried proposed openstack/nova master: pass endpoint interface to Ironic client  https://review.openstack.org/64087918:57
efriedmriedem: howzat ^18:57
efriedah crap, I didn't use enough digits in my change IDs.18:58
openstackgerritEric Fried proposed openstack/nova master: pass endpoint interface to Ironic client  https://review.openstack.org/64087919:00
efriedmriedem: fixed ^19:00
mriedem+W thanks19:00
efriedthanks19:01
mriedemefried: actually one question about the release note https://review.openstack.org/#/c/640879/11/releasenotes/notes/set-endpoint-interface-for-ironicclient-a0b6b8f8dedc7341.yaml19:02
efriedahcrap, I didn't even look at the reno.19:03
dansmithbusted19:03
mriedemtime for your spanking19:03
efriedconsidering this isn't my patch, I'd like to bring in my whipping boy.19:03
mriedemthe gimp?19:03
efriedIs that what he is? I've been thinking of him as "leather ninja"19:04
mriedemhey it's taco tuesday right? https://media.giphy.com/media/ToMjGpHaUPwqzAVaxoI/giphy.gif19:04
efriedThat's obscene in a way I can't begin to define19:05
dansmithoh...mahgod.19:05
*** mikal has joined #openstack-nova19:05
dansmithman mikal just barely missed that one19:05
openstackgerritEric Fried proposed openstack/nova master: pass endpoint interface to Ironic client  https://review.openstack.org/64087919:06
efriedmriedem: ^19:06
openstackgerritMatt Riedemann proposed openstack/nova master: pass endpoint interface to Ironic client  https://review.openstack.org/64087919:08
mriedemlet's all touch that patch19:08
*** wolverineav has quit IRC19:09
openstackgerritMatt Riedemann proposed openstack/nova master: Require python-ironicclient>=2.7.0  https://review.openstack.org/64286319:11
mriedemefried: right? ^19:11
efriedmriedem: in master only, yeah.19:12
efried+219:13
efriedo right, we load ironicclient dynamically, swhy it's not in requirements.txt19:14
*** mikal has quit IRC19:19
*** wolverineav has joined #openstack-nova19:29
*** ttsiouts has joined #openstack-nova19:29
*** betherly has joined #openstack-nova19:32
*** igordc has quit IRC19:32
*** igordc has joined #openstack-nova19:33
*** wolverineav has quit IRC19:34
cfriesenI assume this is a test-tool problem, not a problem with my change? http://logs.openstack.org/63/631363/22/check/nova-grenade-live-migration/1a03cbc/job-output.txt.gz#_2019-03-11_07_51_09_35960819:36
mriedemit means live migration failed19:36
mriedemu'status': u'error'19:36
*** betherly has quit IRC19:37
mriedemhttp://logs.openstack.org/63/631363/22/check/nova-grenade-live-migration/1a03cbc/logs/screen-n-cpu.txt.gz?level=TRACE#_Mar_11_07_50_39_50622019:37
mriedemcfriesen: so yeah i assume that's not your fault19:40
mriedemlibvirt randomly puked19:41
melwittmriedem: yeah, I guess you're right it's latent since rocky19:44
*** luksky has joined #openstack-nova19:48
*** ralonsoh has quit IRC19:48
*** wolverineav has joined #openstack-nova20:02
*** wolverineav has quit IRC20:06
*** wolverineav has joined #openstack-nova20:06
openstackgerritMatt Riedemann proposed openstack/nova master: Don't warn on network-vif-unplugged event during live migration  https://review.openstack.org/64287720:12
*** gibi has quit IRC20:12
*** cdent has quit IRC20:13
*** igordc has quit IRC20:13
openstackgerritMatt Riedemann proposed openstack/nova master: Don't warn on network-vif-unplugged event during live migration  https://review.openstack.org/64287720:15
*** waverider has joined #openstack-nova20:15
mriedembunch of stable/queens changes with a +2 that need approval https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:stable/queens+label:Code-Review=220:16
*** dave-mccowan has quit IRC20:24
*** agopi has quit IRC20:42
*** waverider has quit IRC20:44
*** IvensZambrano has joined #openstack-nova20:44
*** hongbin has joined #openstack-nova20:52
*** gmann_afk is now known as gmann20:54
openstackgerritmelanie witt proposed openstack/nova master: Add get_counts() to InstanceMappingList  https://review.openstack.org/63807220:57
openstackgerritmelanie witt proposed openstack/nova master: Count instances from mappings and cores/ram from placement  https://review.openstack.org/63807320:57
openstackgerritmelanie witt proposed openstack/nova master: Use instance mappings to count server group members  https://review.openstack.org/63832420:57
*** zhubx has quit IRC20:59
*** zhubx has joined #openstack-nova20:59
*** eharney has quit IRC21:03
*** mriedem has quit IRC21:05
*** IvensZambrano has quit IRC21:06
*** mriedem has joined #openstack-nova21:08
mriedemwe should probably change the channel topic from the runways etherpad link to the rc potential etherpad link21:14
*** xek has quit IRC21:16
melwittyeah, we should. dansmith ^21:16
*** ChanServ sets mode: +o dansmith21:16
melwitthttps://etherpad.openstack.org/p/nova-stein-rc-potential21:16
dansmithlink me? it's21:16
dansmiththanks21:16
* dansmith is one lazy mofo21:16
*** dansmith changes topic to "Current runways: https://etherpad.openstack.org/p/nova-stein-rc-potential -- This channel is for Nova development. For support of Nova deployments, please use #openstack."21:17
*** ChanServ sets mode: -o dansmith21:17
melwittthanks21:17
*** gibi has joined #openstack-nova21:18
*** wolverineav has quit IRC21:20
*** betherly has joined #openstack-nova21:21
*** marst has quit IRC21:25
*** betherly has quit IRC21:26
*** wolverineav has joined #openstack-nova21:26
*** igordc has joined #openstack-nova21:26
*** igordc has quit IRC21:26
*** agopi has joined #openstack-nova21:27
*** igordc has joined #openstack-nova21:27
*** wolverineav has quit IRC21:31
*** marst has joined #openstack-nova21:33
mriedemgot a bunch of pike backports that need a +W https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:stable/pike+label:Code-Review=221:33
mriedemthere are also several pike backports from me which haven't had core review, and i probably shouldn't approve my own backports21:33
mriedemmelwitt: ^21:33
melwittk, I'll go through them21:34
openstackgerritMerged openstack/nova master: Revert "Fixes race condition with privsep utime"  https://review.openstack.org/64103821:35
mriedemfeel free to proxy my backports as an implicit +221:35
melwittcopy that21:35
*** ab-a has quit IRC21:37
*** ab-a has joined #openstack-nova21:38
openstackgerritMatt Riedemann proposed openstack/nova master: Add functional test to delete a server while in VERIFY_RESIZE  https://review.openstack.org/61564421:42
mriedemmelwitt: this is another old change that you've reviewed at one point https://review.openstack.org/#/c/541420/ but without it is going to make our eventual data migration for old volume attachments take that much longer21:43
openstackgerritEric Fried proposed openstack/nova master: WIP/PoC: Bypass ironicclient for node.get  https://review.openstack.org/64289921:44
efriedTheJulia, jroll, dtantsur|afk: This should interest you ^21:45
*** wolverineav has joined #openstack-nova21:45
melwittmriedem: oh yeah, I remember that one. it fell off my radar a long time ago. I'll re-queue it21:45
openstackgerritMatt Riedemann proposed openstack/nova master: Add retry_on_deadlock to migration_update DB API  https://review.openstack.org/63307521:45
efriedTheJulia, jroll, dtantsur|afk: I decided that other thing - getting ironicclient to accept a pre-made Adapter - was going to be pretty heinous and an unnecessary step on the road to getting ironicclient out of the picture. This is a different approach.21:45
TheJuliaefried: ++++++++++++++++++=21:46
openstackgerritMatt Riedemann proposed openstack/nova master: Create volume attachment during boot from volume in compute  https://review.openstack.org/54142021:47
openstackgerritEric Fried proposed openstack/nova master: WIP/PoC: Bypass ironicclient for node.get  https://review.openstack.org/64289921:48
mriedemefried: now that you're an oslo.policy master https://review.openstack.org/#/c/624185/21:49
efriedayfkm?21:49
melwittmriedem: speaking of old patches, I have an old patch bug fix that I've been wanting review on, if you could add it to your list. the patch has been verified by an operator https://review.openstack.org/61197421:51
efriedmriedem: I'm not touching it until I see lbragstad's +121:51
*** awaugama has quit IRC21:51
mriedemmelwitt: that looks like something for lyarwood and/or mdbooth to look at first21:52
mriedemhell or even cfriesen since he reported the bug21:53
*** marst has quit IRC21:54
melwittok. I guess I forgot to ask them21:54
melwittlooks like I added them to the review but didn't bug them. I'll do that21:56
cfriesenI can probably confirm that it works, but I don't know the code21:56
melwittoh, no I didn't. you did. guh21:56
*** rcernin has joined #openstack-nova22:01
lbragstadmriedem oh - i forgot about that guy22:01
lbragstadlooked again, left comments22:01
*** betherly has joined #openstack-nova22:08
mriedemoh boy ok that's enough policy for me for one day22:08
*** hoonetorg has quit IRC22:09
*** cdent has joined #openstack-nova22:10
*** hoonetorg has joined #openstack-nova22:10
*** whoami-rajat has quit IRC22:12
*** betherly has quit IRC22:12
*** agopi has quit IRC22:23
openstackgerritSam Morrison proposed openstack/nova-specs master: add spec "flavor-classes"  https://review.openstack.org/63540822:23
*** igordc has quit IRC22:25
*** hongbin has quit IRC22:32
*** igordc has joined #openstack-nova22:43
*** tosky has quit IRC22:43
*** ivve has quit IRC22:44
*** ttsiouts has quit IRC22:45
*** ttsiouts has joined #openstack-nova22:46
*** ttsiouts has quit IRC22:50
*** mriedem has quit IRC22:51
*** tkajinam has joined #openstack-nova22:53
*** tbachman has joined #openstack-nova23:05
*** betherly has joined #openstack-nova23:10
*** cdent has quit IRC23:13
*** betherly has quit IRC23:15
*** slaweq has quit IRC23:24
*** tristanC2 is now known as tristanC23:36
*** itlinux has joined #openstack-nova23:37
*** betherly has joined #openstack-nova23:42
*** betherly has quit IRC23:46
*** BjoernT has quit IRC23:47
*** IvensZambrano has joined #openstack-nova23:53
« Monday, 2019-03-11 Index Wednesday, 2019-03-13 »

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