| *** erlon has joined #openstack-nova | 00:00 | |
| *** tjgresha has quit IRC | 00:04 | |
| *** tjgresha has joined #openstack-nova | 00:05 | |
| *** wolverineav has quit IRC | 00:16 | |
| *** wolverineav has joined #openstack-nova | 00:16 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add testing guide for down cells https://review.openstack.org/650167 | 00:26 |
|---|---|---|
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add testing guide for down cells https://review.openstack.org/650167 | 00:28 |
| *** samueldmq has quit IRC | 00:29 | |
| *** lbragstad has quit IRC | 00:31 | |
| *** mriedem has quit IRC | 00:33 | |
| *** dakshina-ilangov has joined #openstack-nova | 00:34 | |
| *** erlon has quit IRC | 00:35 | |
| *** zhubx has quit IRC | 00:41 | |
| *** zhubx has joined #openstack-nova | 00:41 | |
| *** tetsuro has joined #openstack-nova | 00:44 | |
| *** mlavalle has quit IRC | 00:45 | |
| *** whoami-rajat has joined #openstack-nova | 00:48 | |
| *** brinzhang has joined #openstack-nova | 00:58 | |
| openstackgerrit | Merged openstack/nova stable/stein: Remove CellMappingPayload database_connection and transport_url fields https://review.openstack.org/650043 | 01:01 |
| openstackgerrit | Merged openstack/nova stable/stein: Libvirt: gracefully handle non-nic VFs https://review.openstack.org/649630 | 01:01 |
| *** takashin has joined #openstack-nova | 01:01 | |
| openstackgerrit | Merged openstack/nova master: Delete allocations even if _confirm_resize raises https://review.openstack.org/647566 | 01:01 |
| openstackgerrit | Merged openstack/nova master: api-ref: add more details to confirmResize troubleshooting https://review.openstack.org/647741 | 01:01 |
| 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:06 |
| *** brinzhang has quit IRC | 01:07 | |
| ileixe | efried: So I was expecting the blueprint, but I wonder why the community decided to give up the blueprint. | 01:07 |
| *** tiendc has joined #openstack-nova | 01:10 | |
| 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:12 |
| melwitt | it looks like there isn't a new etherpad yet though | 01:14 |
| 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:14 |
| 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:16 |
| melwitt | ileixe: this is the thread I was thinking about http://lists.openstack.org/pipermail/openstack-discuss/2018-November/000280.html | 01:21 |
| *** wolverineav has quit IRC | 01:28 | |
| *** BjoernT has joined #openstack-nova | 01:31 | |
| *** ttsiouts has joined #openstack-nova | 01:32 | |
| openstackgerrit | melanie witt proposed openstack/nova master: Fix SynchronousThreadPoolExecutorFixture mock spec https://review.openstack.org/650171 | 01:34 |
| *** wolverineav has joined #openstack-nova | 01:38 | |
| *** wolverineav has quit IRC | 01:39 | |
| *** bbowen has joined #openstack-nova | 01:41 | |
| *** lbragstad has joined #openstack-nova | 01:43 | |
| ileixe | @melwitt: Many thanks. I was thirsty about the background of decision making. | 01:45 |
| *** BjoernT_ has joined #openstack-nova | 01:45 | |
| *** BjoernT has quit IRC | 01:46 | |
| 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:46 |
| ileixe | @melwitt: Unfortunately, I do not participate. | 01:47 |
| ileixe | Thanks anyway :) | 01:48 |
| 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:48 |
| *** BjoernT_ has quit IRC | 01:49 | |
| ileixe | @melwitt: Cool. See you then. | 01:49 |
| melwitt | :) | 01:49 |
| openstackgerrit | melanie witt proposed openstack/nova master: Use futurist.GreenThreadPoolExecutor in scatter_gather_cells https://review.openstack.org/650172 | 01:53 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Fix cleaning up console tokens https://review.openstack.org/637716 | 01:53 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in virt/test_block_device.py https://review.openstack.org/566153 | 01:56 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in virt/test_block_device.py https://review.openstack.org/566153 | 01:56 |
| *** ttsiouts has quit IRC | 02:05 | |
| *** markvoelker has joined #openstack-nova | 02:18 | |
| *** brinzhang has joined #openstack-nova | 02:31 | |
| openstackgerrit | Tushar Patil proposed openstack/nova-specs master: Support filtering of allocation_candidates by forbidden aggregates https://review.openstack.org/609960 | 02:33 |
| *** cfriesen has quit IRC | 02:34 | |
| *** brinzhang has quit IRC | 02:37 | |
| *** brinzhang has joined #openstack-nova | 02:45 | |
| *** brinzhang has quit IRC | 02:50 | |
| *** markvoelker has quit IRC | 02:52 | |
| *** itlinux has joined #openstack-nova | 02:53 | |
| *** dakshina-ilangov has quit IRC | 02:53 | |
| *** brinzhang has joined #openstack-nova | 02:56 | |
| *** brinzh has joined #openstack-nova | 02:57 | |
| *** brinzhang has quit IRC | 03:00 | |
| *** hongbin has joined #openstack-nova | 03:02 | |
| *** psachin has joined #openstack-nova | 03:04 | |
| openstackgerrit | melanie witt proposed openstack/nova master: Fix SynchronousThreadPoolExecutorFixture mock spec https://review.openstack.org/650171 | 03:17 |
| openstackgerrit | melanie witt proposed openstack/nova master: Use futurist.GreenThreadPoolExecutor in scatter_gather_cells https://review.openstack.org/650172 | 03:17 |
| *** brinzh has quit IRC | 03:26 | |
| *** brinzh has joined #openstack-nova | 03:26 | |
| *** udesale has joined #openstack-nova | 03:42 | |
| *** mmethot has quit IRC | 03:42 | |
| *** mmethot has joined #openstack-nova | 03:42 | |
| *** markvoelker has joined #openstack-nova | 03:49 | |
| *** itlinux has quit IRC | 03:54 | |
| *** lbragstad has quit IRC | 03:54 | |
| *** lbragstad has joined #openstack-nova | 03:55 | |
| *** johnthetubaguy has quit IRC | 03:57 | |
| *** nicolasbock has quit IRC | 04:02 | |
| *** ttsiouts has joined #openstack-nova | 04:03 | |
| *** jaosorior has joined #openstack-nova | 04:10 | |
| *** hongbin has quit IRC | 04:18 | |
| *** gaoyan has joined #openstack-nova | 04:19 | |
| *** markvoelker has quit IRC | 04:22 | |
| *** dpawlik has joined #openstack-nova | 04:25 | |
| openstackgerrit | Seyeong Kim proposed openstack/nova stable/rocky: prevent common kwargs from glance client failure https://review.openstack.org/650064 | 04:31 |
| openstackgerrit | Seyeong Kim proposed openstack/nova stable/rocky: Share snapshot image membership with instance owner https://review.openstack.org/643853 | 04:31 |
| *** spsurya has joined #openstack-nova | 04:32 | |
| *** dpawlik has quit IRC | 04:36 | |
| *** ttsiouts has quit IRC | 04:36 | |
| *** ttsiouts has joined #openstack-nova | 04:46 | |
| *** bhagyashris_ has joined #openstack-nova | 04:50 | |
| openstackgerrit | melanie witt proposed openstack/nova master: Use futurist.GreenThreadPoolExecutor in scatter_gather_cells https://review.openstack.org/650172 | 04:57 |
| *** hoonetorg has quit IRC | 05:05 | |
| *** pcaruana has joined #openstack-nova | 05:08 | |
| *** igordc has quit IRC | 05:19 | |
| *** markvoelker has joined #openstack-nova | 05:19 | |
| *** tiendc has quit IRC | 05:21 | |
| *** hoonetorg has joined #openstack-nova | 05:21 | |
| openstackgerrit | Tushar Patil proposed openstack/nova-specs master: Allow compute nodes to use DISK_GB from shared storage RP https://review.openstack.org/650188 | 05:34 |
| *** ratailor has joined #openstack-nova | 05:37 | |
| *** lbragstad has quit IRC | 05:46 | |
| *** sidx64 has joined #openstack-nova | 05:52 | |
| *** markvoelker has quit IRC | 05:52 | |
| *** wolverineav has joined #openstack-nova | 05:56 | |
| *** Luzi has joined #openstack-nova | 05:58 | |
| *** wolverineav has quit IRC | 06:00 | |
| *** cfriesen has joined #openstack-nova | 06:09 | |
| *** ttsiouts has quit IRC | 06:13 | |
| *** bbowen has quit IRC | 06:22 | |
| *** dpawlik has joined #openstack-nova | 06:24 | |
| *** dansmith has quit IRC | 06:34 | |
| *** dansmith has joined #openstack-nova | 06:34 | |
| *** Vek has quit IRC | 06:34 | |
| *** Vek has joined #openstack-nova | 06:34 | |
| *** cfriesen has quit IRC | 06:35 | |
| *** brinzhang has joined #openstack-nova | 06:40 | |
| *** slaweq has joined #openstack-nova | 06:42 | |
| *** brinzh has quit IRC | 06:42 | |
| *** brinzh has joined #openstack-nova | 06:44 | |
| *** brinzhang has quit IRC | 06:48 | |
| openstackgerrit | Stephen Finucane proposed openstack/python-novaclient master: Revert "Fix crashing console-log" https://review.openstack.org/650235 | 06:54 |
| *** psachin has quit IRC | 06:54 | |
| *** gaoyan has quit IRC | 06:55 | |
| *** gaoyan has joined #openstack-nova | 06:56 | |
| *** tesseract has joined #openstack-nova | 06:59 | |
| *** tjgresha has quit IRC | 06:59 | |
| *** tjgresha has joined #openstack-nova | 06:59 | |
| *** rpittau|afk is now known as rpittau | 07:01 | |
| *** mdbooth_ has quit IRC | 07:02 | |
| *** ttsiouts has joined #openstack-nova | 07:04 | |
| *** psachin has joined #openstack-nova | 07:05 | |
| *** gouthamr has quit IRC | 07:11 | |
| *** awalende has joined #openstack-nova | 07:11 | |
| *** sidx64 has quit IRC | 07:12 | |
| *** gaoyan has quit IRC | 07:15 | |
| *** sidx64 has joined #openstack-nova | 07:16 | |
| *** tosky has joined #openstack-nova | 07:16 | |
| *** ratailor has quit IRC | 07:18 | |
| *** ratailor has joined #openstack-nova | 07:18 | |
| *** zhubx has quit IRC | 07:20 | |
| *** zhubx has joined #openstack-nova | 07:20 | |
| *** gaoyan has joined #openstack-nova | 07:20 | |
| *** gouthamr has joined #openstack-nova | 07:20 | |
| openstackgerrit | Tushar Patil proposed openstack/nova-specs master: Support filtering of allocation_candidates by forbidden aggregates https://review.openstack.org/609960 | 07:24 |
| *** sidx64 has quit IRC | 07:31 | |
| *** tssurya has joined #openstack-nova | 07:33 | |
| *** phasespace has joined #openstack-nova | 07:35 | |
| *** ccamacho has joined #openstack-nova | 07:35 | |
| *** sidx64 has joined #openstack-nova | 07:42 | |
| *** luksky has joined #openstack-nova | 07:45 | |
| *** rcernin has quit IRC | 07:48 | |
| *** markvoelker has joined #openstack-nova | 07:49 | |
| kashyap | efried: Oops, I'll add them to the Etherpad. | 07:57 |
| kashyap | bauzas: or anyone -- Morning, do you have a URL to the review-day Etherpad? | 07:58 |
| *** ralonsoh has joined #openstack-nova | 07:59 | |
| tssurya | kashyap: https://etherpad.openstack.org/p/nova-spec-review-day | 07:59 |
| *** takashin has left #openstack-nova | 08:00 | |
| kashyap | Thanks. | 08:05 |
| *** tkajinam has quit IRC | 08:07 | |
| openstackgerrit | Theodoros Tsioutsias proposed openstack/nova master: Add instance hard delete https://review.openstack.org/570202 | 08:14 |
| openstackgerrit | Theodoros Tsioutsias proposed openstack/nova master: Add requested_networks to RequestSpec https://review.openstack.org/570201 | 08:14 |
| openstackgerrit | Theodoros Tsioutsias proposed openstack/nova master: Enable rebuild for instances in cell0 https://review.openstack.org/570203 | 08:14 |
| openstackgerrit | Theodoros Tsioutsias proposed openstack/nova master: Introduce the PENDING instance state https://review.openstack.org/566473 | 08:14 |
| openstackgerrit | Theodoros Tsioutsias proposed openstack/nova master: Allow rebuild for instances in PENDING state https://review.openstack.org/637585 | 08:14 |
| *** Luzi has quit IRC | 08:16 | |
| *** slaweq has quit IRC | 08:18 | |
| openstackgerrit | Merged openstack/os-vif stable/stein: Replace openstack.org git:// URLs with https:// https://review.openstack.org/646869 | 08:19 |
| *** markvoelker has quit IRC | 08:23 | |
| *** bbowen has joined #openstack-nova | 08:25 | |
| openstackgerrit | Bhagyashri Shewale proposed openstack/nova master: Ignore root_gb for BFV in simple tenant usage API https://review.openstack.org/612626 | 08:27 |
| *** slaweq has joined #openstack-nova | 08:27 | |
| *** imacdonn has quit IRC | 08:31 | |
| *** slaweq has quit IRC | 08:32 | |
| *** imacdonn has joined #openstack-nova | 08:33 | |
| *** sidx64 has quit IRC | 08:34 | |
| *** slaweq has joined #openstack-nova | 08:35 | |
| *** sagarw has joined #openstack-nova | 08:39 | |
| *** psachin has quit IRC | 08:40 | |
| *** derekh has joined #openstack-nova | 08:41 | |
| *** dtantsur|afk is now known as dtantsur | 08:41 | |
| *** psachin has joined #openstack-nova | 08:42 | |
| *** sidx64 has joined #openstack-nova | 08:43 | |
| *** mdbooth_ has joined #openstack-nova | 08:45 | |
| *** mvkr has joined #openstack-nova | 08:52 | |
| *** tbachman has quit IRC | 09:03 | |
| *** davidsha has joined #openstack-nova | 09:07 | |
| *** markvoelker has joined #openstack-nova | 09:20 | |
| *** sidx64 has quit IRC | 09:22 | |
| *** tetsuro has quit IRC | 09:29 | |
| *** bhagyashris_ has quit IRC | 09:36 | |
| *** brinzh has quit IRC | 09:45 | |
| *** brinzh has joined #openstack-nova | 09:46 | |
| openstackgerrit | Michael Still proposed openstack/nova master: Remove fake_libvirt_utils from connection tests. https://review.openstack.org/642557 | 09:53 |
| openstackgerrit | Michael Still proposed openstack/nova master: Remove fake_libvirt_utils from snapshot tests. https://review.openstack.org/642558 | 09:53 |
| openstackgerrit | Michael Still proposed openstack/nova master: Remove fake_libvirt_utils from virt driver tests. https://review.openstack.org/643894 | 09:53 |
| openstackgerrit | Michael Still proposed openstack/nova master: Remove fake_libvirt_utils from libvirt imagebackend tests. https://review.openstack.org/643895 | 09:53 |
| openstackgerrit | Michael Still proposed openstack/nova master: Remove remaining vestiges of fake_libvirt_utils from unit tests. https://review.openstack.org/643896 | 09:53 |
| openstackgerrit | Michael Still proposed openstack/nova master: Remove fake_libvirt_utils users in functional testing. https://review.openstack.org/644793 | 09:53 |
| *** markvoelker has quit IRC | 09:54 | |
| *** sidx64 has joined #openstack-nova | 09:55 | |
| *** tbachman has joined #openstack-nova | 09:55 | |
| *** brinzh has quit IRC | 09:57 | |
| *** sidx64_ has joined #openstack-nova | 10:11 | |
| *** sidx64 has quit IRC | 10:11 | |
| *** ccamacho has quit IRC | 10:15 | |
| *** cdent has joined #openstack-nova | 10:17 | |
| *** phasespace has quit IRC | 10:31 | |
| *** tbachman has quit IRC | 10:45 | |
| *** cdent has quit IRC | 10:45 | |
| *** markvoelker has joined #openstack-nova | 10:50 | |
| *** ttsiouts has quit IRC | 10:54 | |
| *** ttsiouts has joined #openstack-nova | 10:55 | |
| *** ttsiouts has quit IRC | 10:59 | |
| *** ttsiouts has joined #openstack-nova | 11:01 | |
| *** erlon has joined #openstack-nova | 11:04 | |
| *** ccamacho has joined #openstack-nova | 11:04 | |
| *** ttsiouts has quit IRC | 11:11 | |
| *** zhubx has quit IRC | 11:11 | |
| *** ttsiouts has joined #openstack-nova | 11:12 | |
| *** zhubx has joined #openstack-nova | 11:12 | |
| *** cdent has joined #openstack-nova | 11:13 | |
| *** ttsiouts has quit IRC | 11:16 | |
| *** ttsiouts has joined #openstack-nova | 11:21 | |
| *** luksky has quit IRC | 11:24 | |
| *** markvoelker has quit IRC | 11:24 | |
| *** ratailor has quit IRC | 11:31 | |
| *** udesale has quit IRC | 11:32 | |
| *** udesale has joined #openstack-nova | 11:32 | |
| *** tetsuro has joined #openstack-nova | 11:45 | |
| *** nicolasbock has joined #openstack-nova | 11:45 | |
| *** tbachman has joined #openstack-nova | 12:06 | |
| 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:17 |
| *** sagarw has quit IRC | 12:17 | |
| *** ivve has quit IRC | 12:20 | |
| 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:21 |
| *** phasespace has joined #openstack-nova | 12:22 | |
| jaypipes | sean-k-mooney: ack | 12:22 |
| 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 | 12:23 |
| *** erlon has quit IRC | 12:24 | |
| *** davidsha has quit IRC | 12:27 | |
| *** davidsha has joined #openstack-nova | 12:31 | |
| *** cdent has quit IRC | 12:37 | |
| *** whoami-rajat has quit IRC | 12:37 | |
| *** efried is now known as fried_rice | 12:41 | |
| *** tesseract-rh has joined #openstack-nova | 12:45 | |
| *** tesseract has quit IRC | 12:45 | |
| *** tetsuro has quit IRC | 12:46 | |
| *** whoami-rajat has joined #openstack-nova | 12:54 | |
| *** luksky has joined #openstack-nova | 12:59 | |
| *** awaugama has joined #openstack-nova | 13:05 | |
| *** bnemec has joined #openstack-nova | 13:05 | |
| *** cdent has joined #openstack-nova | 13:11 | |
| *** lbragstad has joined #openstack-nova | 13:18 | |
| 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:20 |
| *** zhubx has quit IRC | 13:22 | |
| *** jaosorior has quit IRC | 13:23 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: Remove dead code https://review.openstack.org/650277 | 13:23 |
| *** cfriesen has joined #openstack-nova | 13:25 | |
| *** stephenfin is now known as finucannot | 13:31 | |
| 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:31 |
| 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:32 |
| 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 |
| sean-k-mooney | some might want want a spec for the upgrade impact/docs bug im not sure | 13:37 |
| 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:38 |
| 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 |
| tssurya | sean-k-mooney: oh yea got for it , we have no more ties to cells_v1 | 13:39 |
| tssurya | go* | 13:39 |
| 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 |
| 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 |
| *** wolverineav has joined #openstack-nova | 13:40 | |
| sean-k-mooney | yes which is why nova net is still a thing | 13:40 |
| *** kaisers has quit IRC | 13:41 | |
| finucannot | sean-k-mooney: and it'll be totally separate since this series already weighs in at 34 patches | 13:41 |
| finucannot | Some of which I'll be able to fold in but still | 13:41 |
| sean-k-mooney | ya it should be seperate | 13:42 |
| *** awalende has quit IRC | 13:43 | |
| *** awalende has joined #openstack-nova | 13:44 | |
| *** wolverineav has quit IRC | 13:45 | |
| *** mriedem has joined #openstack-nova | 13:46 | |
| *** eharney has joined #openstack-nova | 13:46 | |
| *** johanssone has joined #openstack-nova | 13:49 | |
| *** awalende has quit IRC | 13:49 | |
| *** johanssone has quit IRC | 13:49 | |
| *** johanssone has joined #openstack-nova | 13:49 | |
| *** kaisers has joined #openstack-nova | 13:57 | |
| *** mlavalle has joined #openstack-nova | 13:58 | |
| *** gaoyan has quit IRC | 13:59 | |
| *** sidx64_ has quit IRC | 13:59 | |
| *** gaoyan has joined #openstack-nova | 14:02 | |
| *** bnemec is now known as beekneemech | 14:04 | |
| *** gaoyan has quit IRC | 14:12 | |
| fried_rice | kaisers: looking... | 14:16 |
| fried_rice | finucannot: Specless BP for cellsv1 removal, hells yeah. | 14:17 |
| *** dpawlik has quit IRC | 14:20 | |
| *** eharney_ has joined #openstack-nova | 14:22 | |
| *** ttsiouts has quit IRC | 14:22 | |
| dansmith | I think we need a real spec for cellsv1 removal | 14:23 |
| *** ttsiouts has joined #openstack-nova | 14:23 | |
| melwitt | finucannot: +1 on removing cells v1 and nova-net separately | 14:23 |
| *** ttsiouts has quit IRC | 14:24 | |
| *** amodi_ has joined #openstack-nova | 14:24 | |
| *** amodi_ is now known as amodi | 14:24 | |
| *** ttsiouts has joined #openstack-nova | 14:24 | |
| *** eharney has quit IRC | 14:25 | |
| *** eharney_ is now known as eharney | 14:27 | |
| mriedem | we have had specless bps in the past for nova-net removal | 14:28 |
| fried_rice | kaisers: +A, but a couple of questions inline | 14:28 |
| mriedem | https://blueprints.launchpad.net/nova/+spec/remove-nova-network | 14:29 |
| fried_rice | dansmith: what would the spec say? | 14:29 |
| dansmith | fried_rice: I was totally joking, but you people are slow and the moment was ruined | 14:29 |
| mriedem | note nothing happened in stein https://blueprints.launchpad.net/nova/+spec/remove-nova-network-stein | 14:29 |
| fried_rice | oy | 14:29 |
| mriedem | also note it's not as easy as just deleting some code | 14:30 |
| 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:30 |
| finucannot | mriedem: Oh, I know | 14:31 |
| finucannot | that's the first patch in my series | 14:31 |
| finucannot | Sorry, I meant for cells | 14:31 |
| finucannot | same thing for nova-net tho | 14:32 |
| 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 |
| 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:32 |
| mriedem | kashyap: no | 14:33 |
| kashyap | Okido; figured as much | 14:33 |
| finucannot | mriedem: I figured the important bit was that _somebody_ did it | 14:34 |
| fried_rice | kashyap: yes, hold | 14:34 |
| finucannot | Plus, nothing stops those people review the series and provide their insight that way | 14:34 |
| mriedem | finucannot: at the very least i think there needs to be a plan | 14:35 |
| fried_rice | kashyap: http://paste.openstack.org/show/748945/ | 14:35 |
| mriedem | finucannot: like, do you know about the nova-net and cells v1 dependency for startup? | 14:35 |
| 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 |
| 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 |
| mriedem | and if so, how are you planning on removing the stuff based on deps? | 14:35 |
| finucannot | mriedem: that nova-net won't start unless cells v1 is enabled? | 14:36 |
| kashyap | fried_rice: Ah, thanks. /me clicks | 14:36 |
| mriedem | right - so going to remove nova-net first? which is harder, or break that dependency and remove cells v1 first | 14:36 |
| kashyap | fried_rice: Yeah, I saw your tox.ini patch fly-by for the 'specs/backlog' thingie | 14:36 |
| dansmith | tssurya: you're done with v1, but are you done with n-net? | 14:36 |
| dansmith | last I heard you weren't | 14:36 |
| fried_rice | kashyap: That one shouldn't affect this at all. | 14:37 |
| finucannot | I just switched that 'if not cells.enabled' check to 'if True:' so nova-net can never start now. It leaves reams of useless code but that's a trivial cleanup exercise now | 14:37 |
| mriedem | dansmith: belmiro said in the ML that we could drop nova-net now too | 14:37 |
| kashyap | fried_rice: Ah, this is a personal hack; nice | 14:37 |
| dansmith | ah okay | 14:37 |
| fried_rice | kashyap: But there was a flurry a few months ago where we changed the names of venvs and stuff. Might not have been in the specs repo though. | 14:37 |
| * kashyap nods | 14:37 | |
| fried_rice | kashyap: Yeah, looks like it boils down to the sphinx-build line with the right args, while sitting in the venv. | 14:37 |
| mriedem | finucannot: if you're looking for something that needs to be done first but avoid breaking runtime code, fix the functional tests to not rely on nova-net by default https://github.com/openstack/nova/blob/master/nova/tests/functional/integrated_helpers.py#L82 | 14:38 |
| openstackgerrit | Lee Yarwood proposed openstack/nova stable/queens: Delete instance_id_mappings record in instance_destroy https://review.openstack.org/650309 | 14:38 |
| mriedem | https://github.com/openstack/nova/blob/master/nova/tests/functional/integrated_helpers.py#L87 | 14:38 |
| mriedem | and get REQUIRES_LOCKING to default to False (it's True because by default we startup nova-network for functional tests) | 14:38 |
| *** ccamacho has quit IRC | 14:39 | |
| *** jistr is now known as jistr|call | 14:39 | |
| finucannot | mriedem: Sure, I can take a look at that. It's probably going to come up as I walk through this series fixing tests I've broken along the way | 14:39 |
| finucannot | so. much. coupling. | 14:39 |
| mriedem | finucannot: can you link me? | 14:40 |
| *** ccamacho has joined #openstack-nova | 14:40 | |
| finucannot | mriedem: to? | 14:40 |
| mriedem | you said you had a patch series | 14:40 |
| mriedem | but i don't see it in gerrit | 14:40 |
| fried_rice | finucannot: better upload it before your computer crashes and you lose it. | 14:41 |
| finucannot | mriedem: Nah, local only for now. It's failing from about half way in and I don't want to punish the gate (35 patches) | 14:41 |
| *** ccamacho has quit IRC | 14:41 | |
| *** ccamacho has joined #openstack-nova | 14:42 | |
| mriedem | jesus | 14:42 |
| finucannot | mriedem: https://github.com/stephenfin/nova/commits/cells-v1-removal | 14:42 |
| mriedem | i really wish you would have brought this up before starting a huge local patch series | 14:42 |
| finucannot | It's only a days work and I'll be folding a lot of that into other patches. If I end up dumping a lot of it, well, I learned stuff about the API | 14:43 |
| finucannot | That's nowhere near final, btw | 14:43 |
| kashyap | fried_rice: Thanks, noted. | 14:47 |
| mriedem | ok so glancing through this you're killing of cells v1 separate from nova-net | 14:47 |
| mriedem | starting with the api and cli entry points | 14:47 |
| mriedem | and then working down | 14:47 |
| finucannot | correct | 14:48 |
| mriedem | if you can get what you have here cleaned up and passing, then i say push that up and let people start to look at it | 14:48 |
| finucannot | Stop people using it so I can slowly break stuff without worrying about screwing up someones deployment | 14:48 |
| finucannot | mriedem: ack | 14:48 |
| *** itlinux has joined #openstack-nova | 14:54 | |
| mriedem | fried_rice: do we have a runways etherpad setup for stein yet and/or is anyone queueing for runways in stein? | 14:54 |
| *** tbachman has quit IRC | 14:55 | |
| *** jistr|call is now known as jistr | 14:57 | |
| melwitt | easy review to fix the SynchronousThreadPoolExecutorFixture https://review.openstack.org/650171 | 14:58 |
| *** ttsiouts has quit IRC | 15:00 | |
| *** ttsiouts has joined #openstack-nova | 15:00 | |
| *** ccamacho has quit IRC | 15:01 | |
| mriedem | looking | 15:01 |
| *** ttsiouts has quit IRC | 15:01 | |
| *** ttsiouts has joined #openstack-nova | 15:02 | |
| mriedem | melwitt: https://docs.python.org/3/library/unittest.mock.html#the-mock-class says | 15:02 |
| mriedem | "spec: This can be either a list of strings or an existing object (a class or instance)" | 15:02 |
| mriedem | maybe mock.Mock(spec='futurist.Future') was wrong b/c it wasn't a list? | 15:03 |
| melwitt | oh, hm. I wonder why it doesn't work then. I've seen other examples in our code where a string doesn't work and attributes are not found | 15:03 |
| melwitt | maybe. I can try that | 15:03 |
| mriedem | i'm assuming you hit this while trying to de-eventlet some nova code or something? | 15:04 |
| melwitt | haha yes | 15:04 |
| mriedem | yeah it would be cool if we could use more of the native concurrency stuff... | 15:04 |
| melwitt | yeah, I've got a patch going that I'm about to update again stacked on top of the fixture fix | 15:05 |
| mriedem | looks like spec=[..] doesn't help | 15:07 |
| *** whoami-rajat has quit IRC | 15:07 | |
| *** tbachman has joined #openstack-nova | 15:08 | |
| fried_rice | mriedem: I haven't set anything up yet. I guess we have approved bps, so that should happen. I'll get on it. | 15:09 |
| fried_rice | melwitt, mriedem: doesn't spec=futurist.Future work? | 15:10 |
| mriedem | that's what she did | 15:10 |
| melwitt | it does | 15:10 |
| mriedem | and it does | 15:10 |
| fried_rice | okay. | 15:10 |
| * fried_rice crawls back into hole | 15:10 | |
| melwitt | it's just the docs say you can use list of strings but it doesn't work when tried | 15:11 |
| melwitt | so I dunno what's up with that | 15:11 |
| mriedem | i see several cases of this as well | 15:11 |
| mriedem | guest = mock.Mock(spec='nova.virt.libvirt.guest.Guest') | 15:11 |
| mriedem | ntrn = mock.Mock(spec='neutronclient.v2_0.client.Client') | 15:11 |
| melwitt | yeah, I have seen it throughout our code. and have forgotten every time to clean them all up to prevent more appearing | 15:12 |
| melwitt | *help prevent | 15:12 |
| openstackgerrit | melanie witt proposed openstack/nova master: Use futurist.GreenThreadPoolExecutor in scatter_gather_cells https://review.openstack.org/650172 | 15:12 |
| *** wolverineav has joined #openstack-nova | 15:15 | |
| fried_rice | claudiub might know more. | 15:16 |
| *** ccamacho has joined #openstack-nova | 15:18 | |
| fried_rice | melwitt, mriedem: IIUC, the list of strings is supposed to be a list of attributes the object is allowed to have. | 15:18 |
| fried_rice | not, like, a list of classes. | 15:18 |
| melwitt | haha, that would explain it | 15:18 |
| fried_rice | so spec='bar' will actually make sure your mock has attributes 'b', 'a', and 'r' :) | 15:18 |
| *** Sundar has joined #openstack-nova | 15:18 | |
| kashyap | fried_rice: (Thanks for the review here; just noticed: https://review.openstack.org/#/c/639091/ -- Rework 'EBUSY' (SIGKILL) error handling code path) | 15:18 |
| *** tssurya has quit IRC | 15:19 | |
| melwitt | I'm reminded of the scene in zoolander when they try to get into the computer | 15:19 |
| fried_rice | kashyap: Sure, fwiw :) | 15:19 |
| *** wolverineav has quit IRC | 15:20 | |
| kashyap | Yeah, it is one of those fixes that lowers the chance of "death-by-a-thousand-bugs" (which are the kinds of bugs I like to attack) | 15:20 |
| kashyap | Instead of boiling ponds and lakes | 15:20 |
| mriedem | looks like neither the xenproject ci (libvirt+xen) nor the xenapi ci have been running for a long time | 15:20 |
| mriedem | finucannot: are you going to email them and copy the ML on ^? | 15:20 |
| *** Sundar has quit IRC | 15:20 | |
| finucannot | mriedem: Yeah, I can do that | 15:20 |
| *** Sundar has joined #openstack-nova | 15:21 | |
| fried_rice | melwitt, mriedem: Actually, with spec='foo', 'foo' is treated as an instance of str, so the mock gets all the attributes associated with string (index, replace, find, etc.) | 15:21 |
| melwitt | ohhh man | 15:22 |
| melwitt | haha | 15:22 |
| fried_rice | what that *should* mean is that if you have a mock like m = mock.Mock(spec='nova.virt.libvirt.guest.Guest') and you try to reference e.g. m.uuid, it *should* blow up. | 15:22 |
| *** gyee has joined #openstack-nova | 15:24 | |
| fried_rice | melwitt: we should write a hacking rule | 15:24 |
| melwitt | yeah, we should actually | 15:24 |
| *** dklyle has joined #openstack-nova | 15:24 | |
| fried_rice | melwitt: Lemme throw one together right quick... | 15:24 |
| fried_rice | ...but it means someone's gonna have to go through and scrub all of 'em. | 15:24 |
| melwitt | I can do that. I had meant to do it in the past but kept forgetting | 15:25 |
| *** ttsiouts has quit IRC | 15:25 | |
| *** ttsiouts has joined #openstack-nova | 15:26 | |
| finucannot | mriedem: Done, I think | 15:26 |
| *** tbachman has quit IRC | 15:26 | |
| fried_rice | melwitt: Looks like there's only nine. | 15:26 |
| fried_rice | stand by | 15:26 |
| fried_rice | mriedem, melwitt: wanna merge this one real quick first so we don't have to rebase/renumber? https://review.openstack.org/#/c/649190/ | 15:28 |
| finucannot | mriedem: Related. Could you give your take on this when possible? https://review.openstack.org/#/c/649974/ (ignore the failing pep8 test - I'll fix that shortly) | 15:29 |
| mriedem | finucannot: i'm pretty sure there is already a patch related to that | 15:29 |
| mriedem | which fried_rice is aware of | 15:30 |
| mriedem | the xenapi + openssl warning one | 15:30 |
| *** ttsiouts has quit IRC | 15:30 | |
| fried_rice | finucannot, mriedem: this one https://review.openstack.org/#/c/635533/ ? | 15:30 |
| mriedem | yeah | 15:31 |
| mriedem | maybe those aren't related i guess | 15:32 |
| melwitt | fried_rice: looking | 15:32 |
| *** tbachman has joined #openstack-nova | 15:33 | |
| finucannot | fried_rice, mriedem: Yeah, that looks like the same thing and is probably a better solution (though the tests are still rubbish, IMO) | 15:34 |
| finucannot | well, similar thing. Solution for that should cover my issue | 15:34 |
| * finucannot keeps tab open to fix on Monday | 15:34 | |
| jaypipes | cdent: your expertise needed on https://review.openstack.org/#/c/650172/3/nova/context.py please if you have a mo. | 15:35 |
| * cdent goes lookie | 15:35 | |
| jaypipes | thankie | 15:35 |
| openstackgerrit | Stephen Finucane proposed openstack/python-novaclient stable/stein: Revert "Fix crashing console-log" https://review.openstack.org/650363 | 15:35 |
| openstackgerrit | Stephen Finucane proposed openstack/python-novaclient stable/rocky: Revert "Fix crashing console-log" https://review.openstack.org/650364 | 15:36 |
| sean-k-mooney | finucannot: mriedem this patch right https://review.openstack.org/#/c/635533/ | 15:36 |
| finucannot | sean-k-mooney: yup | 15:38 |
| sean-k-mooney | so it was not clear to me if https://bugs.launchpad.net/nova/+bug/1771506 is a release blocking bug for cannoical or not. i twould seam that xen would be broken due to this assueming that ubuntu/debian now shiped openssl 1.1.1 | 15:42 |
| openstack | Launchpad bug 1771506 in OpenStack Compute (nova) "Unit test failure with OpenSSL 1.1.1" [Low,In progress] - Assigned to Corey Bryant (corey.bryant) | 15:42 |
| openstackgerrit | sean mooney proposed openstack/nova master: SR-IOV Live migration indirect port support https://review.openstack.org/620115 | 15:43 |
| sean-k-mooney | its a bit late but i think we should proably land https://review.openstack.org/#/c/635533/ or something similar soon and then figure out how to use the python libs for train | 15:45 |
| sean-k-mooney | the final RC went out last night so this would not be in the GA release anyway right but i think it can be applied ot stable/* once we are happy with it | 15:46 |
| finucannot | jaypipes: Thoughts on https://review.openstack.org/555081 this cycle? | 15:48 |
| melwitt | cdent: thanks for the comment on the review. I had also asked (in the same stack of replies) if there's a way to detect whether we're running under wsgi or not? sean-k-mooney had suggested as a follow on patch, if we could choose between the executor type (eventlet vs not) based on wsgi/not it would be nice | 15:49 |
| finucannot | jaypipes: I recall you were pretty fatigued with it last cycle. I'd be happy to pick it up if you're not keen to try again with it | 15:49 |
| jaypipes | finucannot: go for it. | 15:50 |
| finucannot | (y) | 15:50 |
| jaypipes | finucannot: I'm pretty fatigued with most things OpenStack at this point. | 15:50 |
| cdent | melwitt: hmmm. good question. I would think the wsgi `environ` will probably be able to tell you something, or sys.argv[0]? | 15:50 |
| sean-k-mooney | melwitt: you can ask eventlet if things have been mokeypatch but we shoudl be able to provide a clearner way | 15:50 |
| jaypipes | finucannot: and cdent's sighs and comments on that spec are spot on, fwiw. | 15:51 |
| finucannot | agreed | 15:51 |
| jaypipes | hardware-defined software strikes yet again. | 15:51 |
| melwitt | cdent: ok, I will search in that direction. thanks | 15:51 |
| cdent | melwitt: in environ there may be a way to get the name of the server, and the eventlet-driven server may have a unique name thus maybe a branch on that. goign the other way is harder since there are so many choices | 15:52 |
| sean-k-mooney | melwitt: well what i was thinking is we proably can create the executor up front once and hide the decision form that function | 15:52 |
| sean-k-mooney | melwitt: so in the case where we monkeypatch we can use teh greenlet on and where we dont we can use soemthing else | 15:53 |
| melwitt | sean-k-mooney: doesn't the upfront thing have to choose too? I don't really understand what you mean | 15:54 |
| cdent | there are two different entry points | 15:54 |
| melwitt | oh, ok | 15:54 |
| sean-k-mooney | yes that ^ and its cleaner to detect once then in all places that could need this in the future | 15:54 |
| cdent | I can't remember the exact details, but the wsgi app has some unique code in advance of being-ready-to-serve | 15:55 |
| melwitt | I see | 15:55 |
| sean-k-mooney | assuming there are other asyc things we might want to do in the api like this | 15:55 |
| mriedem | dansmith: you know how we restrict the request spec to the cell the instance is in before calling the scheduler during cold/live migrate and unshelve? | 16:00 |
| mriedem | i do'nt see us doing that for evacuate, but seems we should... yeah? | 16:01 |
| mriedem | you can't evacuate across cells | 16:01 |
| *** dtantsur is now known as dtantsur|afk | 16:01 | |
| dansmith | probably | 16:02 |
| dansmith | I thought it was done in a place that was likely to catch any instance that already exists in the db | 16:03 |
| mriedem | we explicitly lookup and set the cell on the Destination object in unshelve, cold and live migrate methods before calling the scheduler, | 16:03 |
| *** rpittau is now known as rpittau|afk | 16:03 | |
| *** udesale has quit IRC | 16:03 | |
| mriedem | so it's not generic-ified anywhere as far as i know | 16:03 |
| dansmith | hmm, okay it's been a long time since I looked, but okay | 16:04 |
| dansmith | but yeah, if we're not we should for evac | 16:04 |
| mriedem | cern must not do a lot of evacs or they are somehow otherwise restricting the server to a given cell with aggregates or something | 16:04 |
| openstackgerrit | melanie witt proposed openstack/nova master: Use futurist.GreenThreadPoolExecutor in scatter_gather_cells https://review.openstack.org/650172 | 16:05 |
| *** luksky has quit IRC | 16:05 | |
| dansmith | they have tenant aggregates, | 16:05 |
| dansmith | which probably fixes it for them, | 16:05 |
| dansmith | s/,$// | 16:05 |
| mriedem | yar, could probably easily do something with a 2-cell func test that has 1 compute in each and evacuate the server, see what blows pu | 16:06 |
| mriedem | *up | 16:06 |
| *** jangutter has quit IRC | 16:06 | |
| openstackgerrit | Eric Fried proposed openstack/nova master: Hacking N363: Don't use spec[_set]='string' https://review.openstack.org/650370 | 16:08 |
| fried_rice | mriedem, melwitt: ^ (will fail until violations are repaired) | 16:09 |
| melwitt | fried_rice: should I roll the fixes into your patch or stack a patch under yours? | 16:11 |
| openstackgerrit | Kashyap Chamarthy proposed openstack/nova-specs master: Add "CPU selection with hypervisor consideration" spec https://review.openstack.org/645814 | 16:14 |
| mriedem | dansmith: sanity check on my logic here would be nice https://bugs.launchpad.net/nova/+bug/1823369 | 16:14 |
| openstack | Launchpad bug 1823369 in OpenStack Compute (nova) "_nova_check_type scheduler hint could be accidentally persisted during a rebuild with image change" [Medium,New] | 16:14 |
| openstackgerrit | sean mooney proposed openstack/nova master: libvirt: auto detach/attach sriov ports on migration https://review.openstack.org/629589 | 16:15 |
| sean-k-mooney | ignore ^ i forgot to run pep8 | 16:15 |
| fried_rice | melwitt: Either way. If you stack under, you get to +2 :P | 16:18 |
| melwitt | heh, yup | 16:18 |
| mriedem | https://bugs.launchpad.net/nova/+bug/1823370 for the evac thing | 16:19 |
| openstack | Launchpad bug 1823370 in OpenStack Compute (nova) stein "Evacuations are not restricted to the source cell during scheduling" [Undecided,New] | 16:19 |
| * mriedem opens bugs faster than he can fix them | 16:21 | |
| openstackgerrit | sean mooney proposed openstack/nova master: libvirt: auto detach/attach sriov ports on migration https://review.openstack.org/629589 | 16:21 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Use RequestSpec.requested_destination when rebuilding with new image https://review.openstack.org/650376 | 16:29 |
| sean-k-mooney | jaypipes: finucannot sriov patches are updated let me know if i need to do anything else with them :) | 16:31 |
| *** wolverineav has joined #openstack-nova | 16:33 | |
| *** tesseract-rh has quit IRC | 16:33 | |
| *** davidsha has quit IRC | 16:33 | |
| *** ivve has joined #openstack-nova | 16:34 | |
| sean-k-mooney | hum looks like the ibm and other are not happy with the depend on actully since its already merged ill quickly fix that | 16:34 |
| openstackgerrit | sean mooney proposed openstack/nova master: SR-IOV Live migration indirect port support https://review.openstack.org/620115 | 16:36 |
| *** wolverineav has quit IRC | 16:39 | |
| *** tbachman has quit IRC | 16:40 | |
| *** wolverineav has joined #openstack-nova | 16:42 | |
| openstackgerrit | sean mooney proposed openstack/nova master: libvirt: auto detach/attach sriov ports on migration https://review.openstack.org/629589 | 16:44 |
| *** luksky has joined #openstack-nova | 16:47 | |
| *** ivve has quit IRC | 16:48 | |
| openstackgerrit | Corey Bryant proposed openstack/nova master: xenapi/agent: Change openssl error handling https://review.openstack.org/635533 | 16:49 |
| *** beekneemech has quit IRC | 16:54 | |
| *** dklyle has quit IRC | 16:56 | |
| *** dklyle has joined #openstack-nova | 16:56 | |
| *** eharney has quit IRC | 16:59 | |
| *** derekh has quit IRC | 17:06 | |
| dansmith | mriedem: ouch, I guess that one is on me | 17:14 |
| *** wolverineav has quit IRC | 17:17 | |
| *** wolverineav has joined #openstack-nova | 17:19 | |
| *** wolverineav has quit IRC | 17:24 | |
| *** itlinux has quit IRC | 17:25 | |
| *** igordc has joined #openstack-nova | 17:28 | |
| *** tbachman has joined #openstack-nova | 17:28 | |
| *** psachin has quit IRC | 17:29 | |
| *** manjeets has joined #openstack-nova | 17:32 | |
| *** whoami-rajat has joined #openstack-nova | 17:37 | |
| openstackgerrit | Merged openstack/nova master: Added mount fstype based validation of Quobyte mounts https://review.openstack.org/522245 | 17:41 |
| *** wolverineav has joined #openstack-nova | 17:45 | |
| *** kmalloc is now known as needscoffee | 17:46 | |
| *** TheJulia is now known as needssleep | 17:47 | |
| sean-k-mooney | jaypipes: i can look at moving those methods to the RT by the way. i agree it would be nicer to have them there and not spread the locking/resouce tracking code | 17:50 |
| sean-k-mooney | im going to grab soemthing to eat and ill either do it a little later or monday assumin adrianc doesnt get around to it before i do | 17:51 |
| *** hongbin has joined #openstack-nova | 17:53 | |
| mriedem | dansmith: i have to recreate it first, but i also approved the change i'm sure | 17:53 |
| mriedem | if laura ever asks, just put RequestSpec on my tombstone | 17:54 |
| dansmith | mriedem: roger that | 17:54 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add testing guide for down cells https://review.openstack.org/650167 | 17:58 |
| *** Sundar has quit IRC | 17:58 | |
| *** itlinux has joined #openstack-nova | 18:08 | |
| *** ociuhandu has joined #openstack-nova | 18:13 | |
| KH-Jared | heh, I see now I'm going to need a good bit of patience for reviews. Didn't realize the average wait time and how close to stein release it was. I think I can wait now | 18:15 |
| artom | KH-Jared, yeah, reviewer bandwidth, especially core, is a long-standing problem in Nova. We all know about, and there's no real solution. You just have to find the right balance between "politely asking" and "outright pestering" | 18:18 |
| artom | Doing reviews yourself is also an excellent way to get your own patch reviewed | 18:18 |
| KH-Jared | that's fair too | 18:19 |
| *** sidx64 has joined #openstack-nova | 18:21 | |
| melwitt | you can also link the patch in here | 18:22 |
| KH-Jared | that'd be the obvious way, I guess, https://review.openstack.org/#/c/649405/ | 18:22 |
| melwitt | cool, thanks | 18:24 |
| KH-Jared | ty for looking at it | 18:24 |
| KH-Jared | (if you are, just realized the assumption there) | 18:25 |
| *** eharney has joined #openstack-nova | 18:25 | |
| *** sidx64_ has joined #openstack-nova | 18:26 | |
| *** sidx64 has quit IRC | 18:27 | |
| melwitt | haha, yes, I'm looking at it | 18:27 |
| *** ivve has joined #openstack-nova | 18:32 | |
| openstackgerrit | Merged openstack/nova master: Hacking N362: Don't abbrev/alias privsep import https://review.openstack.org/649190 | 18:39 |
| *** itlinux has quit IRC | 18:42 | |
| *** wolverineav has quit IRC | 18:42 | |
| *** ralonsoh has quit IRC | 18:45 | |
| *** tbachman has quit IRC | 18:45 | |
| artom | KH-Jared, well you've done it now | 18:49 |
| artom | How to get artom to review a thing: catch him when he's procrastinating | 18:49 |
| KH-Jared | rofl | 18:49 |
| *** bbowen_ has joined #openstack-nova | 18:54 | |
| *** bbowen has quit IRC | 18:56 | |
| *** tbachman has joined #openstack-nova | 18:58 | |
| *** mdbooth has joined #openstack-nova | 19:02 | |
| *** mdbooth_ has quit IRC | 19:04 | |
| *** wolverineav has joined #openstack-nova | 19:05 | |
| *** bbowen_ has quit IRC | 19:05 | |
| cdent | my eyes read 'he's procrastinating' as 'hydrocasting' and I thought, "hmmm, new sport?" | 19:09 |
| *** wolverineav has quit IRC | 19:10 | |
| jaypipes | sean-k-mooney: sure, no worries. | 19:12 |
| *** itlinux has joined #openstack-nova | 19:13 | |
| artom | cdent, huh, it's actually a thing: https://en.wiktionary.org/wiki/hydrocast#English | 19:15 |
| cdent | artom: yeah, I landed there too | 19:15 |
| cdent | I was afraid to google at first because usually when you google for random word it ends up disturbing | 19:16 |
| artom | *snerk* | 19:16 |
| * artom wonders what the Urban Dictionary definition of hydrocast would be | 19:16 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add functional recreate test for bug 1823370 https://review.openstack.org/650424 | 19:18 |
| openstack | bug 1823370 in OpenStack Compute (nova) rocky "Evacuations are not restricted to the source cell during scheduling" [Medium,Confirmed] https://launchpad.net/bugs/1823370 | 19:18 |
| *** wolverineav has joined #openstack-nova | 19:38 | |
| *** wolverineav has quit IRC | 19:42 | |
| *** igordc has quit IRC | 19:44 | |
| *** igordc has joined #openstack-nova | 19:46 | |
| *** slaweq has quit IRC | 19:51 | |
| fried_rice | melwitt | 19:51 |
| fried_rice | blueprint approved ✔ | 19:51 |
| fried_rice | blueprint self-contained ✔ | 19:51 |
| fried_rice | melwitt: blueprint ready for review? | 19:51 |
| melwitt | yes | 19:51 |
| fried_rice | melwitt: Ready for quick iteration? | 19:51 |
| melwitt | yes | 19:51 |
| fried_rice | melwitt: Are you a good person? | 19:51 |
| melwitt | I think so | 19:52 |
| fried_rice | dansmith: Would you please update the channel topic to point to https://etherpad.openstack.org/p/nova-runways-train ? | 19:52 |
| dansmith | yar | 19:52 |
| *** ChanServ sets mode: +o dansmith | 19:52 | |
| *** wolverineav has joined #openstack-nova | 19:53 | |
| *** dansmith changes topic to "Current runways: https://etherpad.openstack.org/p/nova-runways-train -- This channel is for Nova development. For support of Nova deployments, please use #openstack." | 19:53 | |
| dansmith | did I botch the rc potential topic last time? | 19:53 |
| dansmith | mah bad if sorry | 19:53 |
| *** ChanServ sets mode: -o dansmith | 19:53 | |
| dansmith | *so | 19:53 |
| dansmith | must be beer o clock | 19:53 |
| melwitt | the link was right but it still said runways. but I think everyone knew what it meant | 19:54 |
| fried_rice | thx | 19:54 |
| dansmith | that's what I meant yeah | 19:54 |
| dansmith | prolly did a link swap | 19:54 |
| fried_rice | melwitt: You are cleared for takeoff. Good luck. | 19:54 |
| melwitt | thanks | 19:55 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add functional recreate test for bug 1823370 https://review.openstack.org/650424 | 19:56 |
| openstack | bug 1823370 in OpenStack Compute (nova) rocky "Evacuations are not restricted to the source cell during scheduling" [Medium,Confirmed] https://launchpad.net/bugs/1823370 | 19:56 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Use RequestSpec.requested_destination when rebuilding with new image https://review.openstack.org/650376 | 19:56 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Restrict RequestSpec to cell when evacuating https://review.openstack.org/650429 | 19:56 |
| fried_rice | mriedem, jaypipes, dansmith: would y'all mind flushing (at least the bottom two of) these nova-specs meta-patches? https://review.openstack.org/#/c/581914/ | 19:59 |
| mriedem | i am in a deep dark place atm | 20:00 |
| *** wolverineav has quit IRC | 20:01 | |
| *** wolverineav has joined #openstack-nova | 20:02 | |
| dansmith | I'm trying to get out of here .. is there a rush? | 20:03 |
| jaypipes | fried_rice: depends where you want me to flush it. | 20:03 |
| fried_rice | dansmith: no rush is kind of the problem. Go forth and beer thyself. It'll wait. | 20:04 |
| fried_rice | jaypipes: The blueprint counter has been open and effectively unchanged since last summer. It's not a spec, just tooling, as is the one above it. Just for convenience so those using the tools (melwitt, me) don't have to keep a local fork to do it. | 20:05 |
| *** wolverineav has quit IRC | 20:07 | |
| jaypipes | ack. will review shortly. | 20:07 |
| mriedem | done | 20:09 |
| mriedem | on the bottom on | 20:09 |
| mriedem | *one | 20:09 |
| fried_rice | thanks | 20:12 |
| *** wolverineav has joined #openstack-nova | 20:12 | |
| *** slaweq has joined #openstack-nova | 20:18 | |
| openstackgerrit | Merged openstack/nova-specs master: Add a script for counting blueprints https://review.openstack.org/581914 | 20:18 |
| *** tbachman has quit IRC | 20:20 | |
| *** slaweq has quit IRC | 20:23 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova stable/stein: Delete allocations even if _confirm_resize raises https://review.openstack.org/650437 | 20:28 |
| *** bbowen_ has joined #openstack-nova | 20:35 | |
| *** bbowen_ is now known as bbowen | 20:36 | |
| *** wolverineav has quit IRC | 20:51 | |
| *** wolverineav has joined #openstack-nova | 20:52 | |
| *** wolverineav has quit IRC | 20:52 | |
| *** wolverineav has joined #openstack-nova | 20:53 | |
| mriedem | hmm, so what happens if i (1) create a server on zone A and then (2) force live migrate it to zone B - i know the request spec doesn't get updated to point at zone B, so what happens then if i live migrate the server again? will the scheduler pick a host in zone A again? | 20:57 |
| mnaser | mriedem: in my experience, it will pick zone A again | 21:05 |
| *** sidx64_ has quit IRC | 21:06 | |
| *** tbachman has joined #openstack-nova | 21:06 | |
| mriedem | mnaser: that's what i'd expect | 21:06 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Document restrictions for moving servers between availability zones https://review.openstack.org/650444 | 21:06 |
| mriedem | and if zone A is gone, scheduling will just fail until you've manually fixed the request spec in the db | 21:07 |
| mriedem | cdent: ^ there is that docs patch | 21:07 |
| cdent | huzzah! | 21:07 |
| mnaser | mriedem: request spec or the instance row? | 21:07 |
| * cdent enqueues | 21:07 | |
| mriedem | mnaser: request spec | 21:07 |
| mriedem | b/c that's what goes to the scheduler | 21:07 |
| mriedem | the instance.availability_zone is really just a cache value | 21:07 |
| mnaser | okay, cool, good to know. | 21:08 |
| *** awaugama has quit IRC | 21:09 | |
| *** wolverineav has quit IRC | 21:12 | |
| *** tbachman has quit IRC | 21:14 | |
| *** bbowen_ has joined #openstack-nova | 21:22 | |
| *** bbowen has quit IRC | 21:23 | |
| openstackgerrit | Merged openstack/nova stable/queens: Delete instance_id_mappings record in instance_destroy https://review.openstack.org/650309 | 21:23 |
| *** wolverineav has joined #openstack-nova | 21:25 | |
| *** wolverineav has quit IRC | 21:26 | |
| *** wolverineav has joined #openstack-nova | 21:27 | |
| *** bbowen_ has quit IRC | 21:44 | |
| *** bbowen_ has joined #openstack-nova | 21:48 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Document restrictions for moving servers between availability zones https://review.openstack.org/650444 | 21:50 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Mention [cinder]/cross_az_attach in the AZ docs https://review.openstack.org/650456 | 21:50 |
| *** wolverineav has quit IRC | 21:51 | |
| *** wolverineav has joined #openstack-nova | 21:52 | |
| *** bbowen_ has quit IRC | 21:53 | |
| *** cdent has quit IRC | 21:55 | |
| *** wolverin_ has joined #openstack-nova | 21:56 | |
| *** mmethot_ has joined #openstack-nova | 21:57 | |
| *** bbowen_ has joined #openstack-nova | 21:57 | |
| *** jonher_ has joined #openstack-nova | 21:58 | |
| *** s1061123 has joined #openstack-nova | 21:58 | |
| *** manjeets_ has joined #openstack-nova | 21:58 | |
| *** owalsh_ has joined #openstack-nova | 21:58 | |
| *** gary_perkins_ has joined #openstack-nova | 21:59 | |
| *** eandersson_ has joined #openstack-nova | 22:00 | |
| *** dansmith_ has joined #openstack-nova | 22:00 | |
| *** ileixe has quit IRC | 22:01 | |
| *** edmondsw has quit IRC | 22:01 | |
| *** penick has quit IRC | 22:01 | |
| *** cfriesen has quit IRC | 22:01 | |
| *** dansmith has quit IRC | 22:01 | |
| *** mgariepy has quit IRC | 22:01 | |
| *** tristanC has quit IRC | 22:01 | |
| *** owalsh has quit IRC | 22:01 | |
| *** burt has quit IRC | 22:01 | |
| *** zigo has quit IRC | 22:01 | |
| *** kukacz has quit IRC | 22:01 | |
| *** NostawRm has quit IRC | 22:01 | |
| *** NewBruce has quit IRC | 22:01 | |
| *** frickler has quit IRC | 22:01 | |
| *** brtknr has quit IRC | 22:01 | |
| *** aspiers has quit IRC | 22:01 | |
| *** kukacz_ has joined #openstack-nova | 22:01 | |
| *** wolverineav has quit IRC | 22:01 | |
| *** amorin has quit IRC | 22:01 | |
| *** s1061123_ has quit IRC | 22:01 | |
| *** manjeets has quit IRC | 22:01 | |
| *** mvkr has quit IRC | 22:01 | |
| *** imacdonn has quit IRC | 22:01 | |
| *** mmethot has quit IRC | 22:01 | |
| *** jonher has quit IRC | 22:01 | |
| *** eandersson has quit IRC | 22:01 | |
| *** gary_perkins has quit IRC | 22:01 | |
| *** jonher_ is now known as jonher | 22:01 | |
| *** tristanC_ has joined #openstack-nova | 22:01 | |
| *** frickler has joined #openstack-nova | 22:01 | |
| *** mriedem has left #openstack-nova | 22:01 | |
| *** mriedem has joined #openstack-nova | 22:01 | |
| *** amorin has joined #openstack-nova | 22:01 | |
| *** mriedem has left #openstack-nova | 22:01 | |
| *** edmondsw_ has joined #openstack-nova | 22:02 | |
| *** mvkr has joined #openstack-nova | 22:02 | |
| *** mriedem has joined #openstack-nova | 22:02 | |
| *** burt has joined #openstack-nova | 22:02 | |
| *** dansmith_ has quit IRC | 22:02 | |
| *** lbragstad has quit IRC | 22:02 | |
| *** lbragstad_ has joined #openstack-nova | 22:02 | |
| *** dansmith has joined #openstack-nova | 22:02 | |
| *** imacdonn has joined #openstack-nova | 22:03 | |
| *** luksky has quit IRC | 22:08 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova stable/queens: Define irrelevant-files for tempest-full-py3 job https://review.openstack.org/650460 | 22:10 |
| *** wolverin_ has quit IRC | 22:16 | |
| *** aspiers has joined #openstack-nova | 22:22 | |
| mriedem | fried_rice: in case i forget, once stable/stein opens up post-GA we need to do a stable branch review flush to get pike cleaned out for a final release before it goes into extended maintenance | 22:26 |
| fried_rice | mriedem: ack | 22:26 |
| mriedem | i'll add to the meeting agenda to remind my future self | 22:27 |
| *** whoami-rajat has quit IRC | 22:27 | |
| fried_rice | ++ | 22:27 |
| *** openstackgerrit has quit IRC | 22:28 | |
| *** mriedem has quit IRC | 22:33 | |
| *** mriedem has joined #openstack-nova | 22:35 | |
| fried_rice | jaypipes: Long ago in a galaxy far far away you wrote a spec about representing (anti) affinity with some kind of "distance" measurement...? | 22:39 |
| *** wolverineav has joined #openstack-nova | 22:39 | |
| *** bbowen_ has quit IRC | 22:42 | |
| *** openstackgerrit has joined #openstack-nova | 22:56 | |
| openstackgerrit | Merged openstack/python-novaclient master: Revert "Fix crashing console-log" https://review.openstack.org/650235 | 22:56 |
| *** pcaruana has quit IRC | 22:57 | |
| *** hongbin has quit IRC | 23:09 | |
| *** tosky has quit IRC | 23:11 | |
| mriedem | fried_rice: i don't know that it resulted in a spec | 23:16 |
| mriedem | forum discussion in boston | 23:16 |
| fried_rice | Was it just an etherpad? | 23:16 |
| fried_rice | I know I saw things written somewhere. | 23:16 |
| fried_rice | cool, if it wasn't a spec, I don't have to link it. | 23:16 |
| mriedem | fried_rice: https://etherpad.openstack.org/p/BOS-forum-compute-instance-volume-affinity-hpc | 23:16 |
| *** N3l1x has quit IRC | 23:18 | |
| mriedem | there is an old spec from jaybird at the bottom | 23:19 |
| *** tbachman has joined #openstack-nova | 23:21 | |
| fried_rice | okay, that looks similar but not familiar. Thanks. | 23:21 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Don't report 'exiting' when mapping cells https://review.openstack.org/649340 | 23:23 |
| *** gerrykopec has quit IRC | 23:32 | |
| *** needscoffee is now known as kmalloc | 23:32 | |
| *** wolverineav has quit IRC | 23:33 | |
| *** sapd1_x has joined #openstack-nova | 23:40 | |
| *** mlavalle has quit IRC | 23:41 | |
| *** mlavalle has joined #openstack-nova | 23:46 | |
| *** gerrykopec_ has joined #openstack-nova | 23:52 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!