Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-02-20
09:34:51 lyarwood mdbooth: https://review.openstack.org/#/c/543569/ - would you mind taking a look at this change for https://bugs.launchpad.net/nova/+bug/1739593
09:34:52 openstack Launchpad bug 1739593 in OpenStack Security Advisory "Swapping encrypted volumes can lead to data loss and a possible compute host DOS attack (CVE-2017-18191)" [Undecided,Incomplete]
09:39:25 openstackgerrit Merged openstack/nova master: Test websocketproxy with TLS in the nova-next job https://review.openstack.org/513160
09:58:06 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove unused LOG variables https://review.openstack.org/546097
09:59:16 kashyap Hi, can someone point to PTG Nova agenda etherpad?
09:59:31 cdent kashyap: https://etherpad.openstack.org/p/nova-ptg-rocky
09:59:39 kashyap cdent: Thank you
10:01:27 bauzas meh, LP timeout :(
10:01:33 bauzas hello folks, btw.
10:01:42 openstackgerrit Chris Dent proposed openstack/nova master: WIP: Use nova.db.api directly https://review.openstack.org/543262
10:01:42 openstackgerrit Chris Dent proposed openstack/nova master: Move db MAX constants to own file https://review.openstack.org/543469
10:03:08 cdent brb
10:19:01 openstackgerrit Surya Seetharaman proposed openstack/nova master: Extending delete_cell --force to delete instance_mappings https://review.openstack.org/540073
10:34:49 bauzas mdbooth: others, I need to disappear, but I'm tempted to set https://bugs.launchpad.net/nova/+bug/1740025 as Wontfix
10:34:50 openstack Launchpad bug 1740025 in OpenStack Compute (nova) "LVM Sparse Volumes broken" [Undecided,New]
10:34:51 bauzas agreed ?
10:35:34 mdbooth bauzas: Reading
10:36:22 mdbooth bauzas: Why wontfix?
10:36:55 mdbooth I recall there being severe problems with sparse lvm
10:37:03 mdbooth Not the specifics, though
10:38:53 bauzas mdbooth: because it looks like a feature
10:39:21 bauzas mdbooth: tbc, you can do thin provisioning with Cinder
10:39:23 kashyap I don't know, would you want the reporter to do the whole spec dance?
10:39:36 bauzas but here, the reporter only uses raw LVM
10:39:37 kashyap It's okay to flesh out the details on the bug, without hurrying to close it
10:39:47 mdbooth bauzas: It reads like a bug to me.
10:39:51 kashyap Yeah, to me too.
10:40:05 bauzas so, in case you create LVM volumes having X size
10:40:08 bauzas then, meh
10:40:15 mdbooth Sparse LVM is a thing, and if we were using it correctly, I would expect the scenario described to work.
10:40:39 bauzas mdbooth: why can't the reporter create LVs that are more than 64MB ?
10:41:01 bauzas that's something I don't understand
10:41:02 mdbooth bauzas: I'm guessing because we didn't create the sparse lv correctly.
10:41:20 mdbooth bauzas: As I said, though, I'm pretty sure I recall severe problems in our implementation of sparse lvm.
10:41:20 lyarwood yeah that's not up to the user, n-cpu is creating the lv's
10:41:29 bauzas ah ok
10:41:34 mdbooth severe == How could it ever possibly have worked
10:41:41 bauzas so when nova creates the lv, it asks for a size
10:41:48 bauzas that's not the image
10:41:48 bauzas meh
10:42:04 mdbooth bauzas: It's like creating a sparse file, except lvm
10:42:12 bauzas and then tries to thin provision by itself, while cinder can do that far better
10:42:31 mdbooth It creates a sparse block device, which only allocated blocks from the lvm block pool when they're used
10:42:35 mdbooth Or it's supposed to, anyway
10:42:36 bauzas looks to me we barely support it
10:42:44 mdbooth bauzas: Right.
10:42:54 bauzas yeah I understand the purpose of sparse LVs
10:42:59 mdbooth bauzas: I'm not convinced this has ever worked. However, I'm pretty sure that it was intended to work.
10:43:05 bauzas exactly
10:43:08 bauzas hence the Wontfix
10:43:14 bauzas we never tested that, right?
10:43:25 mdbooth Meh, still sounds like a bug to me.
10:43:29 bauzas some obscure blueprint that implemented in 2012 to make people happy
10:43:37 bauzas without having decent testing
10:43:48 bauzas call it tech debt
10:43:49 mdbooth If we mark this wontfix, we should officially deprecate the feature instead.
10:43:54 bauzas that's right
10:44:08 bauzas I need to go to the gym, but I'll consider all the options before replying to the bug
10:44:21 mdbooth bauzas: KK. Enjoy!
10:44:29 bauzas mdbooth: yeah, we can have volume-backed instances by cinder, right?
10:44:37 mdbooth bauzas: And don't forget I'll be Wattbiking in Dublin next week if you want to join me :)
10:44:41 bauzas mdbooth: if so, you can do thin provisioning with cinder
10:44:48 kashyap mdbooth: What _is_ Wattbiking? /me looks up
10:45:00 lyarwood bauzas: yeah but not using local disks as PVs as this is doing
10:45:03 kashyap Ah, indoor bike trainers
10:45:06 mdbooth bauzas: Yes, I'm sure this is much better handled by cinder.
10:45:23 bauzas mdbooth: this is what I'm doing https://vimeo.com/8178409
10:47:29 kashyap Bail out whom?
11:04:34 mdbooth kashyap: See the second definition of bail out in that list
11:04:43 kashyap I saw that...
11:04:47 kashyap The parachute thing, right?
11:04:59 kashyap No
11:05:02 kashyap "to abandon a harmful or difficult situation;"
11:05:03 kashyap :P
11:05:10 kashyap mdbooth: I doubt bauzas mean't that. Haha
11:05:32 kashyap (I'd guess, he'd be surprised to learn that.)
11:05:50 mdbooth Not used literally, it seems appropriate in context
11:05:51 kashyap s/mean't/meant/
11:06:47 kashyap Yeah, true.
11:33:08 openstackgerrit Marcin Juszkiewicz proposed openstack/nova master: Allow to configure amount of PCIe ports https://review.openstack.org/545034
13:30:08 openstackgerrit Bence Romsics proposed openstack/osc-placement master: RP list: member_of and resources parameters (v1.3, v1.4) https://review.openstack.org/511183
13:30:08 openstackgerrit Bence Romsics proposed openstack/osc-placement master: RP delete inventories (v1.5) https://review.openstack.org/514642
13:30:09 openstackgerrit Bence Romsics proposed openstack/osc-placement master: CLI for traits (v1.6) https://review.openstack.org/514643
13:30:09 openstackgerrit Bence Romsics proposed openstack/osc-placement master: Resource class set (v1.7) https://review.openstack.org/514644
13:30:10 openstackgerrit Bence Romsics proposed openstack/osc-placement master: Usages per project and user (v1.8, v1.9) https://review.openstack.org/514646
13:30:10 openstackgerrit Bence Romsics proposed openstack/osc-placement master: CLI allocation candidates (v1.10) https://review.openstack.org/514647
13:30:11 openstackgerrit Bence Romsics proposed openstack/osc-placement master: New dict format of allocations (v1.11, v1.12) https://review.openstack.org/542819
13:46:02 openstackgerrit Elod Illes proposed openstack/nova master: Functional test: cold migrate to compute down https://review.openstack.org/496280
13:46:21 openstackgerrit Elod Illes proposed openstack/nova master: Functional test: evacuate with no compute https://review.openstack.org/498482
14:10:51 openstackgerrit Matt Riedemann proposed openstack/nova stable/queens: Add the ability to get absolute limits from Cinder https://review.openstack.org/546157
14:10:52 openstackgerrit Matt Riedemann proposed openstack/nova stable/queens: Check quota before creating volume snapshots https://review.openstack.org/546158
14:12:08 mriedem claudiub: can you get these backports? https://review.openstack.org/#/q/topic:bug/1746032+status:open
14:44:49 jaypipes is it Friday yet?
14:45:40 dansmith no
14:45:57 dansmith I can tell because I'm not in a big metal tube with sick people and crying children
14:46:27 efried ^
14:46:47 efried Anybody got advice for what a guy can do with a 7h layover in NYC?
14:47:09 efried Noting that both the guy in question and his spouse pretty much loathe NYC
14:47:25 jaypipes efried: which airport?
14:48:57 efried jaypipes: NYC

Earlier   Later