| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-03-09 | |||
| 13:42:02 | sean-k-mooney | efried: ok that was what i was hoping you would say. its what i assumed it would do but wanted to double check | |
| 13:42:06 | efried | sean-k-mooney But I could be wrong about all of that ;-) | |
| 13:42:50 | sean-k-mooney | well worst case i recerate my dev enviornment. i dont currently have anything on it that is not pushed to gerrit bar my local.conf | |
| 13:42:52 | efried | I assume resize on a live instance will do a graceful shutdown. | |
| 13:43:55 | openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: Temporarily untarget context when deleting from cell0 https://review.openstack.org/443403 | |
| 13:44:03 | mriedem | melwitt: Kevin_Zheng: ^ updated that patch | |
| 13:44:33 | sean-k-mooney | well the cloud its on is at max capasity and i have the instance shut down and some spawned another instance and now i cant start it so trying to see if i can startit with less ram instead of deleting other insances which is my next option | |
| 13:44:45 | mriedem | i'll work on a functional test for the recreate today | |
| 13:45:05 | efried | thorst adreznec esberglu Please re-+1 https://review.openstack.org/438598 - had merge conflict all the way up the chain, but this was the only one I had to rebase by hand. (gerrit carries +1s through auto rebases, it seems.) | |
| 13:45:19 | efried | sean-k-mooney If only you were running PowerVM ;-) | |
| 13:45:31 | thorst | efried: will do | |
| 13:45:32 | frickler | cdent: regarding "tenant_id in endpoint", is there a document somewhere how it should be upgraded/changed properly? | |
| 13:46:40 | efried | sean-k-mooney Isn't there a memory overcommit in the conf? | |
| 13:46:49 | cdent | frickler: yes, probably, but I'm not sure where at the moment, and buried in reviewing these docs. sdague will know (when returns from tc things) | |
| 13:47:08 | sean-k-mooney | efried: im not sure intel it would be trilled with me runing power lol :) | |
| 13:47:13 | efried | sean-k-mooney So you would at worst have to shut down other instances if you're truly out of memory on the host. | |
| 13:47:47 | mriedem | mdbooth: given your comment on the os-migrations patch and the target cell stuff, and corruption, see https://review.openstack.org/443403 | |
| 13:47:51 | sean-k-mooney | libvirt was failing to alloc the memory so ya it was shutdown because i really was out of memory on the host | |
| 13:49:08 | efried | sean-k-mooney I can never keep straight which is which, but one of shelve, pause, or suspend will save an instance in such a way that it's recoverable, but its resources are freed up. | |
| 13:50:27 | bauzas | sfinucan: heh, fun you took it :p | |
| 13:50:34 | bauzas | sfinucan: https://review.openstack.org/#/c/235930/ | |
| 13:50:43 | openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: Temporarily untarget context when deleting from cell0 https://review.openstack.org/443403 | |
| 13:50:59 | sfinucan | bauzas: After 18 months I figured you wouldn't mind :P | |
| 13:51:18 | sfinucan | also, I was trying to get DevStack in Vagrant going (again) | |
| 13:51:43 | bauzas | sfinucan: heh, you hit my 18-month acceptance deadline | |
| 13:51:49 | sean-k-mooney | sfinucan: qq. you were doing some struff with adding tags to alias correct? e.g. for pci passthrough? | |
| 13:52:14 | sfinucan | sean-k-mooney: Not me personally, but someone was iirc | |
| 13:53:24 | sean-k-mooney | sfinucan: do you know if that ever merged? | |
| 13:53:52 | sfinucan | sean-k-mooney: I don't, I'm afraid. moshele might be a better person to ask | |
| 13:54:23 | sean-k-mooney | im asking for the nic feature spec, jay was suggesting moveing the feautre request for the alias to the flavor extra spec | |
| 13:55:08 | sean-k-mooney | we could move it to the flavor or image mentadata but i was wondering if the device tags would be better but that assumes that work was completed | |
| 13:55:30 | sean-k-mooney | sfinucan: but cool thanks ill take a look and see | |
| 13:58:15 | cdent | coreycb (or anyone else): where's the right place to report an apparent bug in the the packaging (specifically the apache config) for placement api? | |
| 13:58:50 | mriedem | cdent: probably ubuntu/nova | |
| 13:58:59 | cdent | thanks mriedem | |
| 13:59:06 | mriedem | https://bugs.launchpad.net/ubuntu/+source/nova/+bugs ? | |
| 13:59:23 | coreycb | cdent, mriedem, yes that's correct | |
| 13:59:25 | mriedem | i'm not entirely sure, jamespage can probably help too | |
| 13:59:27 | mriedem | \o/ | |
| 13:59:38 | cdent | gold star for mriedem | |
| 13:59:42 | coreycb | :) | |
| 13:59:52 | coreycb | cdent, mind forwarding that along once it's open? | |
| 14:00:02 | cdent | yeah, will do, | |
| 14:00:12 | coreycb | cdent, thanks | |
| 14:14:24 | jaypipes | sean-k-mooney, sfinucan: fancy a hangout to discuss this? | |
| 14:15:02 | sfinucan | jaypipes: The NIC feature spec, you mean? | |
| 14:18:16 | jaypipes | sfinucan: ya | |
| 14:18:41 | sfinucan | jaypipes: Sure, once the meeting is over | |
| 14:18:43 | jaypipes | sfinucan: because, frankly, it's more than just the NIC features. | |
| 14:31:19 | openstackgerrit | Lee Yarwood proposed openstack/nova master: compute: Move detach logic from manager into driver BDM https://review.openstack.org/439520 | |
| 14:31:20 | openstackgerrit | Lee Yarwood proposed openstack/nova master: objects: Add attachment_id to BlockDeviceMapping https://review.openstack.org/437665 | |
| 14:31:21 | openstackgerrit | Lee Yarwood proposed openstack/nova master: compute: Only destroy BDMs after successful detach call https://review.openstack.org/440693 | |
| 14:34:25 | sean-k-mooney | jaypipes: sfinucan rodolfo will be tied up for an hour he is presenting in a meeting. but after that sure | |
| 14:34:37 | jaypipes | sean-k-mooney: coolio. | |
| 14:35:15 | sfinucan | sean-k-mooney: (y) | |
| 14:44:51 | mdbooth | cdent: Pinging you only because you raised it in the meeting, but fwiw I'm -1 on the patch which implements https://blueprints.launchpad.net/nova/+spec/lvm-thin-pool for 'spec type reasons' | |
| 14:45:04 | mdbooth | There's no spec for me to -1, though | |
| 14:45:09 | gibi | jaypipes: hi! Here is a fix to for the orphaning of the instance.flavor we talked about at the PTG. https://review.openstack.org/#/c/440347/ Could you check it please if you have time? | |
| 14:45:35 | cdent | mdbooth: yeah, I'm not up to date on any of that, was just parroting the agenda | |
| 14:45:43 | mdbooth | cdent: Figured | |
| 14:46:01 | mdbooth | mri | |
| 14:46:05 | mdbooth | Bleargh | |
| 14:46:10 | mdbooth | He's not here | |
| 14:46:20 | cdent | he has to parent | |
| 14:46:25 | asettle | johnthetubaguy: in relate to your question earlier about testing packages, there's a bit of info here: https://docs.openstack.org/contributor-guide/release/taskdetail.html | |
| 14:46:33 | asettle | Under "installation tutorial testing" | |
| 14:46:38 | mdbooth | I hate it when that happens | |
| 14:46:46 | asettle | TL;DR - locate pre-release packages for each distribution, and disseminate information about obtaining the packages for testing purposes. | |
| 14:49:32 | mdbooth | TL;DR thin pools are very similar to sparse volumes. I don't want to support both. Happy to support thin pools, but only if we deprecate sparse volumes, we have a plan to convert sparse volumes to thin pools automatically (so we don't need to support both), and it's painless for the operator. | |
| 14:49:35 | lyarwood | mdbooth: I'll go over it again, did you see my comment a few days ago about the current lvm.conf in F25 defaulting to using thinp with the current lvcreate commands for sparse volumes in Nova | |
| 14:49:50 | mdbooth | lyarwood: I did, yes. | |
| 14:50:13 | mdbooth | I wonder if there are any implications of mixing them in the same VG. | |
| 14:50:42 | mdbooth | i.e. I created these sparse LVs before that change, and these other LVs after that change, so I have a mix in the same VG. | |
| 14:51:26 | lyarwood | mdbooth: I assume that's fine as they wouldn't change the default otherwise right? | |
| 14:51:29 | mdbooth | lyarwood: Anyway, I don't object to thin pools, only supporting 2 very similar ways to do the same thing. | |
| 14:52:03 | mdbooth | lyarwood: Hard data would be nice. | |
| 14:52:08 | lyarwood | yup agreed | |
| 14:53:25 | cdent | coreycb: that bug https://bugs.launchpad.net/ubuntu/+source/nova/+bug/1671509 | |
| 14:53:25 | openstack | Launchpad bug 1671509 in nova (Ubuntu) "placement api configuration in apache configure CustomLog incorrectly" [Undecided,New] | |
| 14:54:55 | jaypipes | edleafe, cdent: your reviews on https://review.openstack.org/#/c/441544/ would be great. thx in advance! | |
| 14:55:07 | cdent | jaypipes: noted! | |
| 14:55:27 | coreycb | cdent, thanks we'll get that fixed up asap | |
| 14:55:36 | edleafe | jaypipes: queued | |
| 14:55:42 | jaypipes | danke | |
| 14:57:31 | jaypipes | mdbooth: you previously reviewed https://review.openstack.org/#/c/399679/. would be great to have a once over from you on that. | |
| 14:57:57 | mdbooth | jaypipes: Sure. | |
| 14:58:03 | jaypipes | cheers | |
| 14:58:55 | asettle | johnthetubaguy: further explanation, package managers test often with automatic setup and that might include updating config files | |
| 14:58:57 | mdbooth | jaypipes: Incidentally, I was looking for validation of this approach for handling RequestContexts for cells: https://review.openstack.org/#/c/443627/ | |
| 14:59:14 | mdbooth | Or invalidation, obviously, which is why I haven't tried to do the whole job yet. | |
| 14:59:25 | jaypipes | mdbooth: k, queued. | |
| 14:59:37 | mdbooth | jaypipes: Thanks | |
| 15:01:30 | openstackgerrit | Gábor Antal proposed openstack/nova master: Add json style checking for sample notifications https://review.openstack.org/443677 | |
| 15:03:05 | dansmith | mdbooth: I skimmed it briefly | |
| 15:03:28 | dansmith | mdbooth: I think we can go with that approach for things for object contexts, where we iterate over multiple cells | |
| 15:03:36 | dansmith | mdbooth: I don't like it for clearly single-cell-targeted operations on an instance or a host | |
| 15:04:03 | dansmith | mdbooth: because after getting all of this to work, I feel confident we'll have some leaks where we don't realize that we've now got two copies of the context | |
| 15:04:06 | mdbooth | dansmith: Ok. So it's worth adding context.for_cell() and fixing up say get_all() and its users? | |
| 15:04:12 | dansmith | mdbooth: the context _is_ gobal and by reference everywhere | |
| 15:04:21 | dansmith | mdbooth: we get it by reference from the request | |