| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2024-12-05 | |||
| 15:01:25 | opendevreview | Serhii Ivanov proposed openstack/nova-specs master: Proposes `Add Graceful Nova Compute Shutdown` https://review.opendev.org/c/openstack/nova-specs/+/937185 | |
| 15:27:08 | opendevreview | Serhii Ivanov proposed openstack/nova-specs master: Proposes `Add Graceful Nova Compute Shutdown` https://review.opendev.org/c/openstack/nova-specs/+/937185 | |
| 15:39:39 | opendevreview | Serhii Ivanov proposed openstack/nova-specs master: Proposes `Add Graceful Nova Compute Shutdown` https://review.opendev.org/c/openstack/nova-specs/+/937185 | |
| 16:06:49 | s3rj1k | Hi all, please take a look at proposed spec above ^, thanks | |
| 16:15:41 | melwitt | stephenfin: sorry to bug you (also thank you for the +2 🙂) but I uploaded a new PS for the tpool.Proxy patch to reference a bug number filed at https://review.opendev.org/c/openstack/nova/+/932669 if you could pls look again at your convenience | |
| 16:16:16 | stephenfin | done | |
| 16:16:36 | melwitt | thank you! | |
| 16:29:46 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for instance actions https://review.opendev.org/c/openstack/nova/+/937048 | |
| 17:30:31 | opendevreview | Kaloyan Kotlarski proposed openstack/nova master: libvirt: Fetch the available vCPUs from the respective cgroup https://review.opendev.org/c/openstack/nova/+/927474 | |
| 18:33:50 | opendevreview | Merged openstack/os-traits master: Update python classifier as per the 2025.1 cycle testing runtime https://review.opendev.org/c/openstack/os-traits/+/935582 | |
| 18:36:59 | opendevreview | Merged openstack/os-resource-classes master: Update python classifier as per the 2025.1 cycle testing runtime https://review.opendev.org/c/openstack/os-resource-classes/+/935578 | |
| 20:58:21 | opendevreview | Merged openstack/nova master: Check shares support (compute manager) https://review.opendev.org/c/openstack/nova/+/885751 | |
| 23:22:12 | opendevreview | Merged openstack/nova master: api: Add response body schemas for versions APIs https://review.opendev.org/c/openstack/nova/+/924587 | |
| #openstack-nova - 2024-12-06 | |||
| 02:54:07 | opendevreview | melanie witt proposed openstack/nova master: libvirt: Add conf option for live migration monitor interval https://review.opendev.org/c/openstack/nova/+/937223 | |
| 03:13:17 | opendevreview | Takashi Kajinami proposed openstack/nova master: Replace deprecated FormatChecker.cls_checks https://review.opendev.org/c/openstack/nova/+/935733 | |
| 03:15:20 | opendevreview | Takashi Kajinami proposed openstack/placement master: Replace deprecated FormatChecker.cls_checks https://review.opendev.org/c/openstack/placement/+/935734 | |
| 07:59:18 | opendevreview | Kaloyan Kotlarski proposed openstack/nova master: libvirt: Fetch the available vCPUs from the respective cgroup https://review.opendev.org/c/openstack/nova/+/927474 | |
| 10:52:05 | amorin | hey all, am I wrong saying that the availability-zone param set on boot is immutable? E.G. if I forgot to set it, I will end-up without any AZ pinned, nova will schedule my instance on a specific az, but I have no way to pin that later without delete/recreate? | |
| 10:55:01 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for hypervisors APIs (1/3) https://review.opendev.org/c/openstack/nova/+/937245 | |
| 10:55:01 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for hypervisors APIs (2/3) https://review.opendev.org/c/openstack/nova/+/937246 | |
| 10:55:02 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for hypervisors APIs (3/3) https://review.opendev.org/c/openstack/nova/+/937247 | |
| 11:35:24 | opendevreview | Takashi Kajinami proposed openstack/nova master: Replace deprecated FormatChecker.cls_checks https://review.opendev.org/c/openstack/nova/+/935733 | |
| 13:56:46 | opendevreview | Ildiko Vancsa proposed openstack/nova master: Update contributor process documentation https://review.opendev.org/c/openstack/nova/+/936759 | |
| 14:18:58 | opendevreview | Ildiko Vancsa proposed openstack/nova master: Update contributor process documentation https://review.opendev.org/c/openstack/nova/+/936759 | |
| 14:36:43 | ildikov | sean-k-mooney: I saw your comment in the contributor doc patch about updating the README.rst file in the root of the nova repo. I'm a little confused in terms of having both the index.rst and contributing.rst with similar information and most places in the different READMEs point to contributing.rst. | |
| 14:38:04 | ildikov | I updated the README as you suggested, but it is inconsistent with other references now. Should it still point to contributing.rst and have a direct link to process.rst in addition? | |
| 14:46:39 | opendevreview | Ildiko Vancsa proposed openstack/nova master: Update contributor process documentation https://review.opendev.org/c/openstack/nova/+/936759 | |
| 14:48:59 | ildikov | sean-k-mooney: gmann has a patch that adds a new link to the tracking etherpad in contributing.rst. I think that would be a better place to update links and references, and make sure that the changes are consistent throughout the Nova repo: https://review.opendev.org/c/openstack/nova/+/936776/1/doc/source/contributor/contributing.rst | |
| 14:49:05 | ildikov | What do you all think? | |
| 14:49:54 | sean-k-mooney | yep we can do that too | |
| 14:50:19 | sean-k-mooney | but part of the problemis its quite hard to find the process doc in the contibutor docs | |
| 14:50:28 | sean-k-mooney | there is a lot of text too read through first | |
| 14:50:47 | sean-k-mooney | so that is why i wated to add riectrly links to the process doc in the readme | |
| 14:51:23 | sean-k-mooney | *direct links | |
| 14:51:44 | sean-k-mooney | i dont want to link directly to the etherpad there | |
| 14:53:04 | sean-k-mooney | ildikov: developer will generally look at the git repo before anythign else so that why i wante to have this clear in the readme | |
| 14:53:47 | sean-k-mooney | we can do that as a sperate chagne if you want but it took me some time to find the prcoess doc when i wanted to find the links for the comment | |
| 14:56:30 | ildikov | sean-k-mooney: I don't question the value to add links to the process doc more directly, I'm personally good with that direction | |
| 14:56:51 | sean-k-mooney | ok ill just submit a patch for that later myself | |
| 14:56:56 | ildikov | based on pervious discussions, it doesn't seem like there's an overall agreement on what should be linked from where | |
| 14:57:22 | ildikov | and I would like to avoid holding up landing the process documentation, because people don't agree on where to put what link :) | |
| 14:57:45 | sean-k-mooney | my main concern is peopel still wont find the procss doc | |
| 14:58:16 | sean-k-mooney | the feed back we got was there was very littel docusmation for contibutors and as you have seen that is not the case | |
| 14:58:29 | sean-k-mooney | so +1 for makeign the exitsing docs correct | |
| 14:58:36 | ildikov | I can add the process doc link back into my patch, and leave the decision to another path whether or not to update the link to contributing.rst vs contributor/index.rst | |
| 14:58:37 | sean-k-mooney | but we need to also fix the discoveray aspect | |
| 14:58:53 | ildikov | would that be a good middle ground? | |
| 14:59:07 | sean-k-mooney | sure | |
| 14:59:12 | ildikov | ok, cool | |
| 14:59:18 | sean-k-mooney | findig that process link was actully what took time | |
| 14:59:29 | ildikov | and yes, I totally agree on the discoverability aspect | |
| 14:59:30 | sean-k-mooney | so that the part i find hard to discover | |
| 14:59:39 | ildikov | +1 | |
| 15:01:32 | opendevreview | Ildiko Vancsa proposed openstack/nova master: Update contributor process documentation https://review.opendev.org/c/openstack/nova/+/936759 | |
| 15:02:35 | ildikov | sean-k-mooney: done ^^ | |
| 15:03:08 | ildikov | I also consolidated the template part in process.rst, I missed your comment on that earlier, apologies for the delay on that. | |
| 15:03:41 | sean-k-mooney | no worries | |
| 15:04:08 | sean-k-mooney | im ok with the current version | |
| 15:04:21 | ildikov | awesome, thank you for checking! | |
| 15:04:25 | sean-k-mooney | i was ok without consolidating them by the way | |
| 15:04:51 | ildikov | ah ok, I wasn't sure | |
| 15:05:14 | ildikov | with the comment from bauzas, I think it's better consolidated | |
| 15:05:27 | sean-k-mooney | i wanted the template but not pushed on which is more readable | |
| 15:05:48 | sean-k-mooney | so if sylvain has a prefence thats fine by me | |
| 15:13:31 | ildikov | sounds good | |
| 15:13:32 | ildikov | I'll keep an eye out for his review and see if there are any more updates he would like to have | |
| 15:16:44 | opendevreview | Ildiko Vancsa proposed openstack/nova master: Update contributor process documentation https://review.opendev.org/c/openstack/nova/+/936759 | |
| 15:17:35 | ildikov | it looks like I got the formatting wrong in rst, haven't done rst in a while so I'm a bit rusty :) | |
| 15:23:34 | opendevreview | Balazs Gibizer proposed openstack/placement master: Add a global limit on the number of allocation candidates https://review.opendev.org/c/openstack/placement/+/936658 | |
| 15:23:35 | opendevreview | Balazs Gibizer proposed openstack/placement master: Add round-robin candidate generation strategy https://review.opendev.org/c/openstack/placement/+/936832 | |
| 15:24:29 | opendevreview | Balazs Gibizer proposed openstack/placement master: Add a global limit on the number of allocation candidates https://review.opendev.org/c/openstack/placement/+/936658 | |
| 15:24:29 | opendevreview | Balazs Gibizer proposed openstack/placement master: Add round-robin candidate generation strategy https://review.opendev.org/c/openstack/placement/+/936832 | |
| 15:30:29 | opendevreview | Ildiko Vancsa proposed openstack/nova master: Update contributor process documentation https://review.opendev.org/c/openstack/nova/+/936759 | |
| 15:35:19 | gibi | melwitt: thanks for volunteering of looking at the placement a_c explosion fix. The first patch of the series in https://review.opendev.org/q/topic:%22bug/2070257%22 is ready for review. The second is also already working but I still intended to clean it up so that is WIP but any feedback is highly appreciated | |
| 15:37:09 | gibi | melwitt: I'm working on getting timing and memory numbers for comparision of before / after the fix and I will add that to the commit message as soon as I can convince my test node to build not just 4 but 8 devstack VMs | |
| 16:20:09 | opendevreview | Merged openstack/nova master: api: Add response body schemas for assisted volume snapshots APIs https://review.opendev.org/c/openstack/nova/+/924588 | |
| 17:39:55 | opendevreview | Balazs Gibizer proposed openstack/placement master: Add round-robin candidate generation strategy https://review.opendev.org/c/openstack/placement/+/936832 | |
| 17:39:56 | opendevreview | Balazs Gibizer proposed openstack/placement master: Factor out allocation candidate generation strategy https://review.opendev.org/c/openstack/placement/+/937273 | |
| 17:39:56 | opendevreview | Balazs Gibizer proposed openstack/placement master: DNM: test with breadth-first in tempest https://review.opendev.org/c/openstack/placement/+/937274 | |
| 17:41:31 | opendevreview | Balazs Gibizer proposed openstack/nova master: DNM: test with placement ac breadth-first https://review.opendev.org/c/openstack/nova/+/937275 | |
| 18:00:08 | gmann | ildikov: current version lgtm, thanks for updates | |
| 18:24:47 | ildikov | gmann: great, thank you for checking! | |
| 18:38:43 | opendevreview | Merged openstack/nova master: Replace deprecated FormatChecker.cls_checks https://review.opendev.org/c/openstack/nova/+/935733 | |
| 18:38:54 | opendevreview | Merged openstack/nova master: Attach Manila shares via virtiofs (API) https://review.opendev.org/c/openstack/nova/+/836830 | |
| 18:39:02 | opendevreview | Merged openstack/nova master: Add helper methods to attach/detach shares https://review.opendev.org/c/openstack/nova/+/885753 | |
| 21:05:44 | opendevreview | Artom Lifshitz proposed openstack/nova-specs master: vTPM live migration https://review.opendev.org/c/openstack/nova-specs/+/936775 | |
| 22:29:15 | opendevreview | Merged openstack/nova master: Add instance.share_attach notification https://review.opendev.org/c/openstack/nova/+/850501 | |
| #openstack-nova - 2024-12-07 | |||
| 00:11:00 | opendevreview | Merged openstack/nova master: api: Add response body schemas for port interface APIs https://review.opendev.org/c/openstack/nova/+/924589 | |
| 00:11:07 | opendevreview | Merged openstack/nova master: api: Add response body schemas for availability zone APIs https://review.opendev.org/c/openstack/nova/+/924590 | |
| #openstack-nova - 2024-12-08 | |||
| 04:07:36 | opendevreview | Merged openstack/nova master: Add instance.share_detach notification https://review.opendev.org/c/openstack/nova/+/851028 | |
| #openstack-nova - 2024-12-09 | |||
| 00:38:55 | opendevreview | Merged openstack/nova master: Add shares to InstancePayload https://review.opendev.org/c/openstack/nova/+/851029 | |
| 12:15:57 | opendevreview | Stephen Finucane proposed openstack/nova master: api-ref: Update note on flavor disabled property https://review.opendev.org/c/openstack/nova/+/937356 | |
| 12:17:36 | opendevreview | benlei proposed openstack/nova master: Avoid duplication instances when nova compute servcie restart https://review.opendev.org/c/openstack/nova/+/937357 | |
| 12:21:59 | opendevreview | benlei proposed openstack/nova master: Avoid duplication instances when nova compute servcie restart https://review.opendev.org/c/openstack/nova/+/937357 | |
| 12:27:12 | opendevreview | Balazs Gibizer proposed openstack/nova master: DNM: test with placement ac breadth-first https://review.opendev.org/c/openstack/nova/+/937275 | |
| 13:32:33 | opendevreview | Kaloyan Kotlarski proposed openstack/nova master: libvirt: Fetch the available vCPUs from the respective cgroup https://review.opendev.org/c/openstack/nova/+/927474 | |
| 13:40:39 | opendevreview | Balazs Gibizer proposed openstack/placement master: Add a global limit on the number of allocation candidates https://review.opendev.org/c/openstack/placement/+/936658 | |
| 13:40:39 | opendevreview | Balazs Gibizer proposed openstack/placement master: Factor out allocation candidate generation strategy https://review.opendev.org/c/openstack/placement/+/937273 | |
| 13:40:40 | opendevreview | Balazs Gibizer proposed openstack/placement master: Add round-robin candidate generation strategy https://review.opendev.org/c/openstack/placement/+/936832 | |
| 13:40:40 | opendevreview | Balazs Gibizer proposed openstack/placement master: DNM: test with breadth-first in tempest https://review.opendev.org/c/openstack/placement/+/937274 | |