Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-01-03
21:09:17 mriedem sure
21:21:23 openstackgerrit Matt Riedemann proposed openstack/nova master: Fix circular import in nova.privsep.utils https://review.openstack.org/628302
21:43:29 openstackgerrit Matt Riedemann proposed openstack/nova master: Run nova-lvm job on nova/privsep/* changes https://review.openstack.org/628306
21:46:04 mriedem ade_lee: ^
21:47:27 cfriesen I'm looking for input on https://review.openstack.org/#/c/618542...for the "quota:" stuff that is defined in https://docs.openstack.org/nova/latest/user/flavors.html but might only apply to one or two hypervisors, does it make sense to enforce the rules even if the instance might end up on a hypervisor that ignores those extra-specs?
21:47:47 sean-k-mooney1 i was going to mention https://review.openstack.org/#/c/591607/ for open discussion. melwitt: looks like the ordering was corrected
21:48:59 sean-k-mooney1 ah the parent patch has a -1 while they work on functional tests
21:49:08 sean-k-mooney1 nevermind
21:50:01 melwitt sean-k-mooney1: ah, ok
21:51:18 melwitt I saw some discussion in gerrit
21:52:27 ade_lee mriedem, thanks!
21:57:57 dansmith mriedem: there, I replied
21:58:34 mriedem <3
22:20:28 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (3) https://review.openstack.org/574104
22:20:43 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (4) https://review.openstack.org/574106
22:20:52 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (5) https://review.openstack.org/574110
22:21:36 openstackgerrit Matt Riedemann proposed openstack/nova master: Fix circular import in nova.privsep.utils https://review.openstack.org/628302
22:21:36 openstackgerrit Matt Riedemann proposed openstack/nova master: Run nova-lvm job on nova/privsep/* changes https://review.openstack.org/628306
23:26:21 openstackgerrit Matt Riedemann proposed openstack/nova-specs master: Spec for cross-cell resize https://review.openstack.org/616037
23:26:36 mriedem round trois
23:28:50 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (6) https://review.openstack.org/574113
23:29:01 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (7) https://review.openstack.org/574974
23:29:19 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (8) https://review.openstack.org/575311
23:48:29 openstackgerrit Merged openstack/nova-specs master: Remove doc about metadata from 'share-pci-between-numa-nodes' https://review.openstack.org/619807
#openstack-nova - 2019-01-04
00:06:28 openstackgerrit Merged openstack/nova-specs master: Remove the duplicate words https://review.openstack.org/617615
01:24:09 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (9) https://review.openstack.org/575581
01:24:17 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (10) https://review.openstack.org/576017
01:24:38 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (11) https://review.openstack.org/576018
02:24:05 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (12) https://review.openstack.org/576019
02:24:24 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (13) https://review.openstack.org/576020
02:24:51 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (14) https://review.openstack.org/576027
03:38:31 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (15) https://review.openstack.org/576031
03:38:46 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (16) https://review.openstack.org/576299
03:38:59 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (17) https://review.openstack.org/576344
04:10:15 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (18) https://review.openstack.org/576673
04:10:23 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (19) https://review.openstack.org/576676
04:11:19 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (20) https://review.openstack.org/576689
04:11:50 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (21) https://review.openstack.org/576709
04:12:18 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (22) https://review.openstack.org/576712
06:46:16 openstackgerrit weizj proposed openstack/python-novaclient master: Update hacking version https://review.openstack.org/627726
06:47:23 openstackgerrit Yikun Jiang proposed openstack/nova master: WIP: Per aggregate scheduling weight https://review.openstack.org/628163
08:41:29 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Allow run metadata api per cell https://review.openstack.org/624612
09:14:30 openstackgerrit Yikun Jiang proposed openstack/nova master: Per aggregate scheduling weight https://review.openstack.org/628163
09:42:06 openstackgerrit Yikun Jiang proposed openstack/nova master: Per aggregate scheduling weight https://review.openstack.org/628163
09:45:26 openstackgerrit Brin Zhang proposed openstack/nova-specs master: Specifying az when restore shelved server https://review.openstack.org/624689
10:16:00 openstackgerrit melanie witt proposed openstack/nova-specs master: Propose counting quota usage from placement and API database https://review.openstack.org/509042
12:58:20 openstackgerrit Rui Zang proposed openstack/nova-specs master: support virtual persistent memory https://review.openstack.org/601596
13:03:31 openstackgerrit Rui Zang proposed openstack/nova-specs master: support virtual persistent memory https://review.openstack.org/601596
13:57:50 openstackgerrit Chris Dent proposed openstack/os-resource-classes master: Don't use upper-constraints when installing package https://review.openstack.org/628442
14:42:41 mriedem https://review.openstack.org/#/c/628302/ fixes the lvm job
16:01:28 mriedem johnthetubaguy: i could use review from you on the mega-beast cross-cell resize spec https://review.openstack.org/#/c/616037/ - got a +2 from dan, and it's a high level plan based on some prototyping, but clearly is complicated enough that a lot of stuff is just going to have to be dealt with during implementation
16:02:34 mriedem tl;dr orchestrate from conductor across the two cells, try to maintain similar style flows as normal resize with minimal disruption to the end user experience
16:43:01 openstackgerrit weizj proposed openstack/os-resource-classes master: Update hacking version to latest https://review.openstack.org/628599
16:43:04 openstackgerrit weizj proposed openstack/os-vif master: Update hacking version to latest https://review.openstack.org/628603
16:43:05 openstackgerrit weizj proposed openstack/os-traits master: Update hacking version to latest https://review.openstack.org/628604
17:28:08 mriedem gibi_off: i'm out next week but we should maybe have a google hangout call the week of the 14th to try and level set with some others involved like jaypipes, dansmith, melwitt etc to sort out the path forward with the bw scheduling stuff - a 30 minute call would probably be more productive than the ML thread at this point
17:29:14 mriedem if the series doesn't get reversed in order to move the api/control plane changes to the end of the stack, then maybe putting an in-code feature flag placeholder for the time being would allow us to sort out what we need to do re discoverability etc
18:04:25 mriedem melwitt: +2 on the counting quotas from placement spec
18:04:30 mriedem thanks for the detailed updates
18:16:51 jaypipes mriedem: I'm fine to do a hangout next week.
18:17:00 jaypipes or the week after
18:17:12 mriedem week after is going to be all hangouts all the time
18:28:07 melwitt mriedem: hangout on the 14th works for me
18:29:25 melwitt mriedem: and thanks for reviewing counting quotas from placement before you head out next week. I appreciate it
18:31:28 cdent melwitt, mriedem for the placement aggressive checkpoint meeting, can we do that not on the monday? any other day/time should be fine for me
18:32:49 melwitt cdent: I think so, I don't think we were thinking the 14th specifically, just some time during the week of the 14th
18:33:06 cdent cool, thanks
18:51:02 mriedem yeah i don't really want meetings on the first day i'm back :)
18:52:33 openstackgerrit JiaJunsu proposed openstack/nova master: Remove args(os=False) in monkey_patch https://review.openstack.org/568999
18:57:56 melwitt oh, yeah, I misread "week of the 14th" as "on the 14th" in your earlier sentence today. during the week of the 14th works for me
19:07:28 mriedem johnthetubaguy: i'm +2 on the 'rebuild from cell0' spec despite some cleanup/clarification that needs to be made in the spec https://review.openstack.org/#/c/554218/ - but that could be done in a follow up, otherwise it seems pretty straight-forward
19:08:39 mriedem raised a good question in there about what happens if you've enabled soft delete and instance fails scheduling, is put in cell0 and then the user deletes it, i'm not sure we have a functional test for that
19:08:54 mriedem i *think* the api does the local delete on it, but that code is pretty gorpy so i could be missing something
19:15:54 mriedem actually i'm just going to approve that spec, john was +2 at PS7 but that needed a pep8 fix, otherwise there were no changes
19:19:20 mriedem melwitt: https://review.openstack.org/#/c/554218/ touches on stuff you're familiar with as well so would probably be good to just review that to make sure you know what's being proposed
19:20:47 melwitt mriedem: ack
19:21:15 mriedem would be a good step toward preemptible instance support if we can get that done in stein
19:23:17 openstackgerrit Merged openstack/nova-specs master: Enable rebuild for instances in cell0 https://review.openstack.org/554218
19:25:48 melwitt yeah, and seems like the present situation is a regression/change in behavior from pre-cells v2, not being able to rebuild an ERROR instance if it was never scheduled. or was that also the case in the past?
19:26:34 mriedem that was also the case in the past,
19:26:36 dansmith I don't think rebuild of a failed schedule ever worked
19:26:42 dansmith because rebuild is on the host you're on
19:26:46 mriedem right
19:26:59 mriedem and the decorator on the api method requires must_have_launched=True
19:27:04 mriedem which is only set if you actually built once
19:27:46 melwitt oh, I see, yeah
19:27:51 melwitt that makes sense
19:53:09 mriedem welp, we still have gate failures and e-r isn't reporting on patches for a reason https://review.openstack.org/#/c/628669/
20:10:21 melwitt I've been wondering why it wasn't commenting, so glad you found it
21:17:38 openstackgerrit Matt Riedemann proposed openstack/nova-specs master: Per-instance serial number (spec) https://review.openstack.org/612531
21:56:53 openstackgerrit Matt Riedemann proposed openstack/nova master: Remove args(os=False) in monkey_patch https://review.openstack.org/568999
22:05:47 mriedem dansmith: i meant to follow up with you about the statement in the cells v2 layout docs about not running the nova-api-metadata script in a multi-cell deployment https://review.openstack.org/#/c/624612/5/doc/source/user/cellsv2-layout.rst - i think that was meant if running globally because the nova-api-metadata script uses the indirection API?
22:05:59 mriedem we do run that service separately in our gate jobs though
22:06:23 mriedem anyway, Kevin split that layout doc into global vs local notes and moved that 'must not run' remark under the global section
22:17:52 dansmith mriedem: yeah, the standalone is per-cell, so if you arrange for it to have the right config and instances only hit it from within the cell, then it's fine, it just doesn't so the switching
22:17:56 dansmith (IIRC(
22:23:26 mriedem ok yeah so i think his doc change is correct there
22:23:30 mriedem except the thing i pointd out
22:24:44 mriedem jmlowe: your users use shelve/unshelve right? https://review.openstack.org/#/c/624689/
22:24:57 mriedem how much would you hate or want to have the ability to specify an AZ on unshelve?

Earlier   Later