| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-04-05 | |||
| 01:06:00 | ileixe | efried: Thanks for the answer. Yes I already saw the dansmith 'oschown' repository and actually we have an third API handle the request (It's very much looks like his solution). I felt hard to maintain the custom API since there's too much exceptional cases that I could not imagine. | |
| 01:07:28 | ileixe | efried: So I was expecting the blueprint, but I wonder why the community decided to give up the blueprint. | |
| 01:12:47 | melwitt | ileixe: some of the folks in the "transfer ownership" forum session didn't like the 'oschown' idea as much and preferred something more like API plugins. but if you are in support of 'oschown', you are encouraged to tell your opinion in the session or add your opinion on the etherpad for the session with your IRC nick | |
| 01:14:46 | melwitt | it looks like there isn't a new etherpad yet though | |
| 01:14:56 | ileixe | melwitt: Thanks for the procedure. It seems to not be abandonded completely yet then. I'm with plugin like API plugins, so I willing to make comments. | |
| 01:16:30 | melwitt | ileixe: I don't think a solid decision has been made yet, discussion hasn't reached a way forward yet. sounds good. I'm trying to find other threads about it, one sec | |
| 01:21:23 | melwitt | ileixe: this is the thread I was thinking about http://lists.openstack.org/pipermail/openstack-discuss/2018-November/000280.html | |
| 01:34:11 | openstackgerrit | melanie witt proposed openstack/nova master: Fix SynchronousThreadPoolExecutorFixture mock spec https://review.openstack.org/650171 | |
| 01:45:05 | ileixe | @melwitt: Many thanks. I was thirsty about the background of decision making. | |
| 01:46:48 | melwitt | ileixe: np. are you going to be at the summit? if so, try to make it to the session https://www.openstack.org/summit/denver-2019/summit-schedule/events/23675/change-ownership-of-resources-followup | |
| 01:47:51 | ileixe | @melwitt: Unfortunately, I do not participate. | |
| 01:48:06 | ileixe | Thanks anyway :) | |
| 01:48:37 | melwitt | ileixe: ok. I'll ping you when the etherpad for the session appears, so you can write your thoughts on it and it will be part of the discussion | |
| 01:49:26 | ileixe | @melwitt: Cool. See you then. | |
| 01:49:36 | melwitt | :) | |
| 01:53:37 | openstackgerrit | melanie witt proposed openstack/nova master: Use futurist.GreenThreadPoolExecutor in scatter_gather_cells https://review.openstack.org/650172 | |
| 01:53:56 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Fix cleaning up console tokens https://review.openstack.org/637716 | |
| 01:56:20 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in virt/test_block_device.py https://review.openstack.org/566153 | |
| 01:56:25 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in virt/test_block_device.py https://review.openstack.org/566153 | |
| 02:33:49 | openstackgerrit | Tushar Patil proposed openstack/nova-specs master: Support filtering of allocation_candidates by forbidden aggregates https://review.openstack.org/609960 | |
| 03:17:17 | openstackgerrit | melanie witt proposed openstack/nova master: Fix SynchronousThreadPoolExecutorFixture mock spec https://review.openstack.org/650171 | |
| 03:17:17 | openstackgerrit | melanie witt proposed openstack/nova master: Use futurist.GreenThreadPoolExecutor in scatter_gather_cells https://review.openstack.org/650172 | |
| 04:31:51 | openstackgerrit | Seyeong Kim proposed openstack/nova stable/rocky: prevent common kwargs from glance client failure https://review.openstack.org/650064 | |
| 04:31:52 | openstackgerrit | Seyeong Kim proposed openstack/nova stable/rocky: Share snapshot image membership with instance owner https://review.openstack.org/643853 | |
| 04:57:35 | openstackgerrit | melanie witt proposed openstack/nova master: Use futurist.GreenThreadPoolExecutor in scatter_gather_cells https://review.openstack.org/650172 | |
| 05:34:59 | openstackgerrit | Tushar Patil proposed openstack/nova-specs master: Allow compute nodes to use DISK_GB from shared storage RP https://review.openstack.org/650188 | |
| 06:54:19 | openstackgerrit | Stephen Finucane proposed openstack/python-novaclient master: Revert "Fix crashing console-log" https://review.openstack.org/650235 | |
| 07:24:13 | openstackgerrit | Tushar Patil proposed openstack/nova-specs master: Support filtering of allocation_candidates by forbidden aggregates https://review.openstack.org/609960 | |
| 07:57:10 | kashyap | efried: Oops, I'll add them to the Etherpad. | |
| 07:58:05 | kashyap | bauzas: or anyone -- Morning, do you have a URL to the review-day Etherpad? | |
| 07:59:17 | tssurya | kashyap: https://etherpad.openstack.org/p/nova-spec-review-day | |
| 08:05:23 | kashyap | Thanks. | |
| 08:14:10 | openstackgerrit | Theodoros Tsioutsias proposed openstack/nova master: Add instance hard delete https://review.openstack.org/570202 | |
| 08:14:10 | openstackgerrit | Theodoros Tsioutsias proposed openstack/nova master: Add requested_networks to RequestSpec https://review.openstack.org/570201 | |
| 08:14:11 | openstackgerrit | Theodoros Tsioutsias proposed openstack/nova master: Enable rebuild for instances in cell0 https://review.openstack.org/570203 | |
| 08:14:39 | openstackgerrit | Theodoros Tsioutsias proposed openstack/nova master: Introduce the PENDING instance state https://review.openstack.org/566473 | |
| 08:14:40 | openstackgerrit | Theodoros Tsioutsias proposed openstack/nova master: Allow rebuild for instances in PENDING state https://review.openstack.org/637585 | |
| 08:19:33 | openstackgerrit | Merged openstack/os-vif stable/stein: Replace openstack.org git:// URLs with https:// https://review.openstack.org/646869 | |
| 08:27:07 | openstackgerrit | Bhagyashri Shewale proposed openstack/nova master: Ignore root_gb for BFV in simple tenant usage API https://review.openstack.org/612626 | |
| 09:53:43 | openstackgerrit | Michael Still proposed openstack/nova master: Remove fake_libvirt_utils from connection tests. https://review.openstack.org/642557 | |
| 09:53:43 | openstackgerrit | Michael Still proposed openstack/nova master: Remove fake_libvirt_utils from snapshot tests. https://review.openstack.org/642558 | |
| 09:53:43 | openstackgerrit | Michael Still proposed openstack/nova master: Remove fake_libvirt_utils from virt driver tests. https://review.openstack.org/643894 | |
| 09:53:43 | openstackgerrit | Michael Still proposed openstack/nova master: Remove fake_libvirt_utils from libvirt imagebackend tests. https://review.openstack.org/643895 | |
| 09:53:43 | openstackgerrit | Michael Still proposed openstack/nova master: Remove remaining vestiges of fake_libvirt_utils from unit tests. https://review.openstack.org/643896 | |
| 09:53:43 | openstackgerrit | Michael Still proposed openstack/nova master: Remove fake_libvirt_utils users in functional testing. https://review.openstack.org/644793 | |
| 12:17:09 | jaypipes | sean-k-mooney: see my request on https://review.openstack.org/#/c/629589/ please. If you can knock that out quick, I can quick-approve since stephenfin was +2 on it already. | |
| 12:21:38 | sean-k-mooney | jaypipes: thank i will check it in a sec. currently removing utils.lock i do actully think its useful but im happy to update it too | |
| 12:22:34 | jaypipes | sean-k-mooney: ack | |
| 12:23:50 | sean-k-mooney | and ya the change you asked for in https://review.openstack.org/#/c/629589/21 are all pretty straigt forward so ill do those before the end of the day | |
| 13:20:56 | kaisers | fried_rice: good lunch? :) Should you later on have the capacity, would you perhaps take a look at https://review.openstack.org/#/c/522245/ (missing a second +2)? I's a second long hung fix... | |
| 13:23:37 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Remove dead code https://review.openstack.org/650277 | |
| 13:31:45 | openstackgerrit | Kashyap Chamarthy proposed openstack/nova stable/queens: conf: libvirt: Make `/dev/urandom` the default for 'rng_dev_path' https://review.openstack.org/650279 | |
| 13:32:43 | finucannot | fried_rice: I think I've finished removing cells v1 in a local series (running through/fixing tests atm). I want to track the effort somehow though. Is a specless BP suitable for that kind of stuff? | |
| 13:37:16 | sean-k-mooney | well there is no api change so i dont think i spec is techncially required its definetly not a bug so a blueprint seams best | |
| 13:37:49 | sean-k-mooney | some might want want a spec for the upgrade impact/docs bug im not sure | |
| 13:38:42 | sean-k-mooney | its been deprecated for many releases so it wont be a shock to anywon but we will want to get cerns input on it before we actully fully drop it | |
| 13:39:24 | sean-k-mooney | finucannot: by the way does that removal also include nova networks or is that the next thing on your list :) | |
| 13:39:28 | tssurya | sean-k-mooney: oh yea got for it , we have no more ties to cells_v1 | |
| 13:39:33 | tssurya | go* | |
| 13:40:26 | sean-k-mooney | that is good to hear. we proably should still wait until ptg to so we can comunicate it to other operators befoer merging the deletions | |
| 13:40:30 | finucannot | sean-k-mooney: Nah, cells v1 only for now. I was actually trying to strip out nova-network but cells v1 relied on it | |
| 13:40:52 | sean-k-mooney | yes which is why nova net is still a thing | |
| 13:41:35 | finucannot | sean-k-mooney: and it'll be totally separate since this series already weighs in at 34 patches | |
| 13:41:44 | finucannot | Some of which I'll be able to fold in but still | |
| 13:42:26 | sean-k-mooney | ya it should be seperate | |
| 14:16:59 | fried_rice | kaisers: looking... | |
| 14:17:12 | fried_rice | finucannot: Specless BP for cellsv1 removal, hells yeah. | |
| 14:23:14 | dansmith | I think we need a real spec for cellsv1 removal | |
| 14:23:40 | melwitt | finucannot: +1 on removing cells v1 and nova-net separately | |
| 14:28:46 | mriedem | we have had specless bps in the past for nova-net removal | |
| 14:28:58 | fried_rice | kaisers: +A, but a couple of questions inline | |
| 14:29:20 | mriedem | https://blueprints.launchpad.net/nova/+spec/remove-nova-network | |
| 14:29:24 | fried_rice | dansmith: what would the spec say? | |
| 14:29:42 | dansmith | fried_rice: I was totally joking, but you people are slow and the moment was ruined | |
| 14:29:54 | mriedem | note nothing happened in stein https://blueprints.launchpad.net/nova/+spec/remove-nova-network-stein | |
| 14:29:58 | fried_rice | oy | |
| 14:30:40 | mriedem | also note it's not as easy as just deleting some code | |
| 14:30:54 | mriedem | there are still APIs that are nova-net specific that need to be changed to 410s https://etherpad.openstack.org/p/nova-network-removal-rocky | |
| 14:31:42 | finucannot | mriedem: Oh, I know | |
| 14:31:47 | finucannot | that's the first patch in my series | |
| 14:31:57 | finucannot | Sorry, I meant for cells | |
| 14:32:03 | finucannot | same thing for nova-net tho | |
| 14:32:35 | mriedem | i also tend to think the people that are familiar with this code should be doing the removal - i didn't realize you'd already started a removal series | |
| 14:32:54 | kashyap | Hey folks, is there some shortcut to build just _one_ releases's specs? E.g. `tox -e docs train` or something like that | |
| 14:33:10 | mriedem | kashyap: no | |
| 14:33:29 | kashyap | Okido; figured as much | |
| 14:34:13 | finucannot | mriedem: I figured the important bit was that _somebody_ did it | |
| 14:34:25 | fried_rice | kashyap: yes, hold | |
| 14:34:26 | finucannot | Plus, nothing stops those people review the series and provide their insight that way | |
| 14:35:14 | mriedem | finucannot: at the very least i think there needs to be a plan | |
| 14:35:17 | fried_rice | kashyap: http://paste.openstack.org/show/748945/ | |
| 14:35:46 | mriedem | finucannot: like, do you know about the nova-net and cells v1 dependency for startup? | |
| 14:35:52 | fried_rice | kashyap: Haven't actually used it in a while, it's possible some of the path assumptions will need to be tweaked since we've been dorking with tox.ini lately. | |
| 14:35:53 | finucannot | mriedem: That's fair. PTG is only a few weeks away. I drafted this in a day so I'm happy to let it sit in Gerrit til then | |
| 14:35:56 | mriedem | and if so, how are you planning on removing the stuff based on deps? | |
| 14:36:06 | finucannot | mriedem: that nova-net won't start unless cells v1 is enabled? | |
| 14:36:11 | kashyap | fried_rice: Ah, thanks. /me clicks | |
| 14:36:31 | mriedem | right - so going to remove nova-net first? which is harder, or break that dependency and remove cells v1 first | |
| 14:36:46 | kashyap | fried_rice: Yeah, I saw your tox.ini patch fly-by for the 'specs/backlog' thingie | |
| 14:36:53 | dansmith | tssurya: you're done with v1, but are you done with n-net? | |