| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-04-05 | |||
| 10:02:26 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add new default roles in server migration policies https://review.opendev.org/717590 | |
| 10:08:38 | openstackgerrit | Takashi Natsume proposed openstack/nova-specs master: Adds spec infrastructure for Victoria https://review.opendev.org/710023 | |
| 10:19:34 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Pass the actual target in server migration policy https://review.opendev.org/717591 | |
| 10:22:24 | openstackgerrit | Guo Jingyu proposed openstack/nova master: WIP: nova supports password encrypted VNC https://review.opendev.org/622336 | |
| 12:36:17 | openstackgerrit | norman shen proposed openstack/nova master: After connection info in pre_live_migration https://review.opendev.org/717597 | |
| 12:40:50 | openstackgerrit | norman shen proposed openstack/nova master: Update connection info in pre_live_migration https://review.opendev.org/717597 | |
| 18:29:12 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add new default roles in server migration policies https://review.opendev.org/717590 | |
| 18:32:16 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add new default roles in server migration policies https://review.opendev.org/717590 | |
| 18:32:39 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Pass the actual target in server migration policy https://review.opendev.org/717591 | |
| 18:38:23 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add new default roles in server topology policies https://review.opendev.org/717585 | |
| 20:06:47 | openstackgerrit | Ivaylo Mitev proposed openstack/nova master: Implementing graceful shutdown. https://review.opendev.org/666245 | |
| 20:08:35 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing flavor_manage policies https://review.opendev.org/714814 | |
| 22:53:09 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing flavor_manage policies https://review.opendev.org/714814 | |
| 23:02:37 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing server migrations policies https://review.opendev.org/717531 | |
| 23:03:04 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in server migration https://review.opendev.org/717588 | |
| 23:03:17 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add new default roles in server migration policies https://review.opendev.org/717590 | |
| 23:03:30 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Pass the actual target in server migration policy https://review.opendev.org/717591 | |
| #openstack-nova - 2020-04-06 | |||
| 02:37:37 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in os-flavor-manage https://review.opendev.org/714818 | |
| 02:37:50 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add new default roles in os-flavor_manage policies https://review.opendev.org/714819 | |
| 02:38:05 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Pass the actual target in os-flavor-manage policy https://review.opendev.org/714822 | |
| 03:14:08 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add new default roles in server topology policies https://review.opendev.org/717585 | |
| 05:39:04 | openstackgerrit | melanie witt proposed openstack/nova master: Reset the cell cache for database access in Service https://review.opendev.org/717662 | |
| 07:47:38 | bauzas | gibi: stephenfin: good morning, I worked on finishing the implementation for the vGPU multiple types https://review.opendev.org/#/c/715490/ | |
| 07:47:58 | bauzas | I will provide a new change for having a functional test | |
| 07:54:18 | gibi | bauzas: ack, I will try to look at it today | |
| 07:54:25 | gibi | and good morning | |
| 07:54:26 | gibi | :) | |
| 08:11:05 | bauzas | alex_xu: gtk | |
| 08:11:38 | bauzas | the only problem is that some of us are not having temperature issues if they have COVID-19 | |
| 08:11:55 | alex_xu | bauzas: yea | |
| 08:12:39 | alex_xu | bauzas: another fun is the phone tracked here, if you are out of the city, you will be notice to ioslate for 14days | |
| 08:13:28 | bauzas | alex_xu: you're living in Peking, right? | |
| 08:13:35 | alex_xu | bauzas: the joke is if you use vpn, then you are being notice for isolation :) | |
| 08:13:39 | alex_xu | bauzas: yes | |
| 08:13:47 | bauzas | ack okay | |
| 08:15:16 | alex_xu | so Matthew is right in shanghai summit, we are totally tracked :) | |
| 08:31:05 | bauzas | alex_xu: well, we have some kind of discussion in France about being tracked or not | |
| 08:31:31 | bauzas | the problem is that a lot of folks around me are just walking outside while they shouldn't | |
| 08:31:52 | bauzas | and we're now locked down since 3 weeks, and for maybe 3 or 4 weeks again | |
| 08:32:16 | alex_xu | bauzas: yea, i'm pretty sure you will feel safe on the stree. but yes...the big brother is watching you. but anyway, we discuss the same issue privately :) | |
| 08:33:16 | alex_xu | bauzas: pretty sure you need another 3 weeks...from the experience we have... | |
| 08:34:55 | alex_xu | bauzas: ^ the work above the above one, I'm talking about the another side of tracking :) | |
| 08:35:03 | alex_xu | s/work/word/ | |
| 08:35:09 | bauzas | yeah In understand you | |
| 08:36:26 | alex_xu | bauzas: we have people on the street now, since people already lock in the room since Feb... just shopping mall is pretty empty, but the park is full of people... | |
| 09:19:17 | openstackgerrit | Marcin Juszkiewicz proposed openstack/nova master: libvirt: check for AMD SEV only on x86-64 https://review.opendev.org/714425 | |
| 09:30:23 | bauzas | stephenfin: FWIW, /me starts to review https://review.opendev.org/#/c/704643/ | |
| 09:30:40 | stephenfin | Ta. vGPU is on my list for today :) | |
| 09:43:52 | johnthetubaguy | ah, I was just looking at that too, maybe bad timing | |
| 09:46:00 | bauzas | stephenfin: johnthetubaguy: I'll provide comments and a -1 | |
| 09:46:19 | bauzas | just downloading the patch for verifying somethin | |
| 09:51:39 | johnthetubaguy | stephenfin: is it worth being more specific on what happens when you don't set an extra spec? or describing dependencies, like things that imply numa aware placed VMs, etc | |
| 09:52:10 | gibi | cores, here is a quick functional test stabilization patch needing a second core https://review.opendev.org/#/c/717070 | |
| 09:52:31 | stephenfin | You mean in the descriptions for the extra specs themselves? | |
| 09:52:32 | bauzas | gibi: on it, I'm mostly done | |
| 09:52:44 | johnthetubaguy | stephenfin: somewhere like that | |
| 09:52:50 | gibi | bauzas: thanks | |
| 09:53:01 | johnthetubaguy | stephenfin: comparing it to configuration, basically | |
| 09:53:21 | stephenfin | Yeah, sure. I probably need to better flesh out the descriptions for much of them, including things like what virt driver supports what extra spec | |
| 09:53:46 | stephenfin | In a follow-up though, preferably? I'd like to focus more on making sure every possible extra spec is listed there first | |
| 09:53:54 | johnthetubaguy | stephenfin: I thought you did fairly well on the virt support, I guess there are a bunch of libvirt only in there that are not spelled out | |
| 09:54:11 | johnthetubaguy | stephenfin: yeah, I am OK with that, just wondering about your plans really | |
| 09:54:41 | johnthetubaguy | stephenfin: there are things like PCI alias, when you check the format and not the content, which is typical of json schema checks, just wondering about your thoughts there | |
| 09:54:47 | stephenfin | Yeah, I plan to massively improve the documentation and do things like add cross-referencing | |
| 09:55:19 | johnthetubaguy | to be clear, I would be fine just checking the keys and not the value, as a step forward, so this is a step ahead of that | |
| 09:55:49 | stephenfin | Ah, so for more detailed things like that, I was planning to leave it to the virt driver for now | |
| 09:56:22 | stephenfin | ditto for things like "you need to specify 'hw:cpu_policy' to use 'hw:cpu_thread_policy'" | |
| 09:56:57 | stephenfin | I wanted to do that via the validator initially but it was way too much /o\ | |
| 09:57:12 | johnthetubaguy | I guess I see this as our global API abstraction, that has been implemented by limited drivers... but yeah, its totally a next step | |
| 10:05:05 | bauzas | stephenfin: I don't see anything in the spec about API interop consistency with updates on, say, https://review.opendev.org/#/c/704643/21/nova/api/validation/extra_specs/capabilities.py | |
| 10:05:28 | bauzas | like, I want 'baz' to be accepted in Victoria | |
| 10:05:45 | bauzas | but calling a Ussuri API will tell you 'sorry but no' | |
| 10:05:55 | bauzas | stephenfin: do you plan to address this later on ? | |
| 10:06:13 | johnthetubaguy | its not turned on yet right, haven't set the microversion where it starts | |
| 10:08:08 | bauzas | johnthetubaguy: indeed, see my comments https://review.opendev.org/#/c/704643/21 | |
| 10:08:36 | stephenfin | bauzas: There's no easy answer for that. I think that's the main reason we had to make the policy configurable. We didn't want to require a microversion to add a new extra spec so people needed a escape lever | |
| 10:09:22 | bauzas | stephenfin: okay, I just feel my biggest concern is that if we avoid to whitelist some key for some filter, then people will see a behavioural change | |
| 10:09:23 | bauzas | right? | |
| 10:09:26 | stephenfin | bauzas: tbh though, it's the exact same issue we see with image metadata today and configuring flavor extra specs, unlike configuring image metadata, is admin-only | |
| 10:10:13 | bauzas | alas. people can use 2.85 microversion if they want to disable it | |
| 10:10:14 | stephenfin | bauzas: Correct, but we have signalled that change with a microversion and we've provided a way to "escape" validation | |
| 10:10:29 | bauzas | stephenfin: yeah I understood it | |
| 10:10:32 | stephenfin | or pass '?validation=permissive', iirc | |
| 10:10:50 | bauzas | stephenfin: I'm just afraid that we could merge something that would opt-in | |
| 10:10:59 | bauzas | close to RC1 | |
| 10:11:09 | bauzas | stephenfin: but we can go like it is now | |
| 10:11:20 | bauzas | at least if we have ways to address issues easilty | |
| 10:11:42 | bauzas | because reviewing all the key and value regexes for every filter isn't an easy thing | |
| 10:11:47 | bauzas | but I don't want to hold this one | |
| 10:12:22 | bauzas | so, my take is : if we mess things up, we all assume that we're about to provide bugfixes that will touch the API behaviour and should be backportable | |
| 10:12:31 | bauzas | if we all agree on this, I'm thumbs up | |
| 10:12:36 | bauzas | gibi: stephenfin: johnthetubaguy: ^ | |
| 10:12:55 | bauzas | or we go permissive *by default* | |
| 10:13:05 | johnthetubaguy | hang on... I am missing something | |
| 10:13:06 | bauzas | (which is not the default case IIUC) | |
| 10:13:22 | johnthetubaguy | don't we currently do zero validation by default right now? | |
| 10:13:33 | gibi | bauzas: fixing bugs in the extra_spec validation will not need a new microversion in my view. | |
| 10:13:40 | johnthetubaguy | including after the proposed validation | |
| 10:13:58 | bauzas | gibi: me too, but I'm just saying that we will change the API behaviour silently | |
| 10:14:05 | johnthetubaguy | hang about | |