| opendevreview | melanie witt proposed openstack/nova master: libvirt: Add conf option for live migration monitor interval https://review.opendev.org/c/openstack/nova/+/937223 | 02:54 |
|---|---|---|
| opendevreview | Takashi Kajinami proposed openstack/nova master: Replace deprecated FormatChecker.cls_checks https://review.opendev.org/c/openstack/nova/+/935733 | 03:13 |
| opendevreview | Takashi Kajinami proposed openstack/placement master: Replace deprecated FormatChecker.cls_checks https://review.opendev.org/c/openstack/placement/+/935734 | 03:15 |
| *** elodilles_pto is now known as elodilles | 07:23 | |
| opendevreview | Kaloyan Kotlarski proposed openstack/nova master: libvirt: Fetch the available vCPUs from the respective cgroup https://review.opendev.org/c/openstack/nova/+/927474 | 07:59 |
| 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:52 |
| 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 |
| 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 |
| 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 | 10:55 |
| opendevreview | Takashi Kajinami proposed openstack/nova master: Replace deprecated FormatChecker.cls_checks https://review.opendev.org/c/openstack/nova/+/935733 | 11:35 |
| opendevreview | Ildiko Vancsa proposed openstack/nova master: Update contributor process documentation https://review.opendev.org/c/openstack/nova/+/936759 | 13:56 |
| opendevreview | Ildiko Vancsa proposed openstack/nova master: Update contributor process documentation https://review.opendev.org/c/openstack/nova/+/936759 | 14:18 |
| 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:36 |
| 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:38 |
| opendevreview | Ildiko Vancsa proposed openstack/nova master: Update contributor process documentation https://review.opendev.org/c/openstack/nova/+/936759 | 14:46 |
| 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:48 |
| ildikov | What do you all think? | 14:49 |
| sean-k-mooney | yep we can do that too | 14:49 |
| sean-k-mooney | but part of the problemis its quite hard to find the process doc in the contibutor docs | 14:50 |
| sean-k-mooney | there is a lot of text too read through first | 14:50 |
| sean-k-mooney | so that is why i wated to add riectrly links to the process doc in the readme | 14:50 |
| sean-k-mooney | *direct links | 14:51 |
| sean-k-mooney | i dont want to link directly to the etherpad there | 14:51 |
| 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 |
| 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:53 |
| 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 |
| sean-k-mooney | ok ill just submit a patch for that later myself | 14:56 |
| ildikov | based on pervious discussions, it doesn't seem like there's an overall agreement on what should be linked from where | 14:56 |
| 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 |
| sean-k-mooney | my main concern is peopel still wont find the procss doc | 14:57 |
| 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 |
| sean-k-mooney | so +1 for makeign the exitsing docs correct | 14:58 |
| 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 |
| sean-k-mooney | but we need to also fix the discoveray aspect | 14:58 |
| ildikov | would that be a good middle ground? | 14:58 |
| sean-k-mooney | sure | 14:59 |
| ildikov | ok, cool | 14:59 |
| sean-k-mooney | findig that process link was actully what took time | 14:59 |
| ildikov | and yes, I totally agree on the discoverability aspect | 14:59 |
| sean-k-mooney | so that the part i find hard to discover | 14:59 |
| ildikov | +1 | 14:59 |
| opendevreview | Ildiko Vancsa proposed openstack/nova master: Update contributor process documentation https://review.opendev.org/c/openstack/nova/+/936759 | 15:01 |
| ildikov | sean-k-mooney: done ^^ | 15:02 |
| -opendevstatus- NOTICE: Gerrit on review.opendev.org is being upgraded to version 3.10 and will be offline. We have allocated an hour for the outage window lasting until 1700 UTC. | 15:03 | |
| 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 |
| sean-k-mooney | no worries | 15:03 |
| sean-k-mooney | im ok with the current version | 15:04 |
| ildikov | awesome, thank you for checking! | 15:04 |
| sean-k-mooney | i was ok without consolidating them by the way | 15:04 |
| ildikov | ah ok, I wasn't sure | 15:04 |
| ildikov | with the comment from bauzas, I think it's better consolidated | 15:05 |
| sean-k-mooney | i wanted the template but not pushed on which is more readable | 15:05 |
| sean-k-mooney | so if sylvain has a prefence thats fine by me | 15:05 |
| -opendevstatus- NOTICE: Gerrit on review.opendev.org is being upgraded to version 3.10 and will be offline starting at 1600 UTC. We have allocated an hour for the outage window lasting until 1700 UTC. | 15:06 | |
| ildikov | sounds good | 15:13 |
| ildikov | I'll keep an eye out for his review and see if there are any more updates he would like to have | 15:13 |
| opendevreview | Ildiko Vancsa proposed openstack/nova master: Update contributor process documentation https://review.opendev.org/c/openstack/nova/+/936759 | 15:16 |
| 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:17 |
| 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 |
| opendevreview | Balazs Gibizer proposed openstack/placement master: Add round-robin candidate generation strategy https://review.opendev.org/c/openstack/placement/+/936832 | 15:23 |
| 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 |
| opendevreview | Balazs Gibizer proposed openstack/placement master: Add round-robin candidate generation strategy https://review.opendev.org/c/openstack/placement/+/936832 | 15:24 |
| opendevreview | Ildiko Vancsa proposed openstack/nova master: Update contributor process documentation https://review.opendev.org/c/openstack/nova/+/936759 | 15:30 |
| 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:35 |
| 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 | 15:37 |
| -opendevstatus- NOTICE: Gerrit on review.opendev.org is being upgraded to version 3.10 and will be offline momentarily. We have allocated an hour for the outage window lasting until 1700 UTC. | 16:01 | |
| opendevreview | Merged openstack/nova master: api: Add response body schemas for assisted volume snapshots APIs https://review.opendev.org/c/openstack/nova/+/924588 | 16:20 |
| opendevreview | Balazs Gibizer proposed openstack/placement master: Add round-robin candidate generation strategy https://review.opendev.org/c/openstack/placement/+/936832 | 17:39 |
| opendevreview | Balazs Gibizer proposed openstack/placement master: Factor out allocation candidate generation strategy https://review.opendev.org/c/openstack/placement/+/937273 | 17:39 |
| opendevreview | Balazs Gibizer proposed openstack/placement master: DNM: test with breadth-first in tempest https://review.opendev.org/c/openstack/placement/+/937274 | 17:39 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: DNM: test with placement ac breadth-first https://review.opendev.org/c/openstack/nova/+/937275 | 17:41 |
| gmann | ildikov: current version lgtm, thanks for updates | 18:00 |
| ildikov | gmann: great, thank you for checking! | 18:24 |
| opendevreview | Merged openstack/nova master: Replace deprecated FormatChecker.cls_checks https://review.opendev.org/c/openstack/nova/+/935733 | 18:38 |
| opendevreview | Merged openstack/nova master: Attach Manila shares via virtiofs (API) https://review.opendev.org/c/openstack/nova/+/836830 | 18:38 |
| opendevreview | Merged openstack/nova master: Add helper methods to attach/detach shares https://review.opendev.org/c/openstack/nova/+/885753 | 18:39 |
| opendevreview | Artom Lifshitz proposed openstack/nova-specs master: vTPM live migration https://review.opendev.org/c/openstack/nova-specs/+/936775 | 21:05 |
| opendevreview | Merged openstack/nova master: Add instance.share_attach notification https://review.opendev.org/c/openstack/nova/+/850501 | 22:29 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!