| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2024-09-03 | |||
| 16:10:28 | erlon | ok, sounds good | |
| 16:10:53 | erlon | is there a way to flag it like that so the team can approve it next? | |
| 16:10:58 | erlon | I | |
| 16:12:04 | sean-k-mooney | put it in the team adjenta for the next meeting and you can link to this irc converstiaon if you like | |
| 16:12:37 | sean-k-mooney | so add it to open disucsion here https://wiki.openstack.org/wiki/Meetings/Nova#Weekly_Nova_team_meeting | |
| 16:13:54 | sean-k-mooney | erlon: skiming the updated version it looks ok but i will need to review it properly later. it looks like you have simplifed it a little since the last version which is nice to see | |
| 16:14:21 | haleyb | sean-k-mooney: i tried to address all the comments | |
| 16:14:51 | sean-k-mooney | haleyb: sorry you are the most recent uploader :) yes i think you have | |
| 16:15:22 | sean-k-mooney | i basically am getting a headache right now which is why im holding of reviewing it proeprly | |
| 16:15:22 | haleyb | i just wanted to move that along, np | |
| 16:15:38 | haleyb | hope you feel better | |
| 16:16:13 | sean-k-mooney | i think its just a lack of drinking water so im goign to go remiday that | |
| 17:22:14 | opendevreview | Merged openstack/nova stable/2024.1: libvirt: call get_capabilities() with all CPUs online https://review.opendev.org/c/openstack/nova/+/927556 | |
| 17:41:26 | opendevreview | Merged openstack/nova stable/2024.1: Fix port group network metadata generation https://review.opendev.org/c/openstack/nova/+/923689 | |
| 17:41:35 | opendevreview | Merged openstack/nova master: Skip snapshot test when missing qemu-img https://review.opendev.org/c/openstack/nova/+/925208 | |
| 19:07:12 | gmann | sean-k-mooney: if you are around https://review.opendev.org/c/openstack/placement/+/926935 | |
| 19:27:21 | sean-k-mooney | +2w | |
| 19:41:04 | gmann | thanks | |
| 20:54:12 | opendevreview | Brian Haley proposed openstack/nova master: Support creating servers with RBAC SGs https://review.opendev.org/c/openstack/nova/+/811521 | |
| 21:34:53 | opendevreview | Merged openstack/placement master: Bump oslo.policy version to enable new RBAC by default https://review.opendev.org/c/openstack/placement/+/926935 | |
| 22:07:57 | opendevreview | Merged openstack/nova master: Remove default override for RBAC config options https://review.opendev.org/c/openstack/nova/+/925519 | |
| #openstack-nova - 2024-09-04 | |||
| 00:55:38 | opendevreview | Merged openstack/nova master: [ironic] Followup fixes around flavor ids https://review.opendev.org/c/openstack/nova/+/927764 | |
| 02:27:29 | melwitt | sean-k-mooney: re: your review comment about being a freeze period.. we are frozen for features but I didn't think we are frozen on merging bug fixes. at least that isn't how it has worked in the past | |
| 02:28:22 | melwitt | if something has changed, I missed it | |
| 07:04:35 | opendevreview | Julien LE JEUNE proposed openstack/nova stable/2024.1: Skip snapshot test when missing qemu-img https://review.opendev.org/c/openstack/nova/+/927896 | |
| 07:07:13 | opendevreview | Julien LE JEUNE proposed openstack/nova stable/2023.2: Skip snapshot test when missing qemu-img https://review.opendev.org/c/openstack/nova/+/927897 | |
| 07:08:13 | opendevreview | Julien LE JEUNE proposed openstack/nova stable/2023.1: Skip snapshot test when missing qemu-img https://review.opendev.org/c/openstack/nova/+/927898 | |
| 11:10:53 | gibi | bauzas: if you are around, would https://review.opendev.org/c/openstack/placement/+/919569 allowed to land after Feature Freeze even if it has a feature release notes attached? I similar patch for nova landed before FF. | |
| 13:43:58 | opendevreview | sean mooney proposed openstack/nova master: Remove nova debugger funcitonality https://review.opendev.org/c/openstack/nova/+/922496 | |
| 13:43:59 | opendevreview | sean mooney proposed openstack/nova master: split monkey_patching form import https://review.opendev.org/c/openstack/nova/+/922425 | |
| 13:43:59 | opendevreview | sean mooney proposed openstack/nova master: refactor cmd dir for non eventlet console_scripts https://review.opendev.org/c/openstack/nova/+/904424 | |
| 13:44:00 | opendevreview | sean mooney proposed openstack/nova master: Add IO Thread Pool Executor https://review.opendev.org/c/openstack/nova/+/922497 | |
| 13:44:00 | opendevreview | sean mooney proposed openstack/nova master: remove eventlet.tpool https://review.opendev.org/c/openstack/nova/+/905287 | |
| 13:44:01 | opendevreview | sean mooney proposed openstack/nova master: use futureist for scatter_gather https://review.opendev.org/c/openstack/nova/+/905284 | |
| 13:44:01 | opendevreview | sean mooney proposed openstack/nova master: use futerist for RBD calls https://review.opendev.org/c/openstack/nova/+/917962 | |
| 13:44:30 | sean-k-mooney | gibi: that is mostly just a rebase but i pull the debugger removal to the front and added the release note | |
| 13:53:08 | jlejeune | sean-k-mooney: dansmith: gibi: hello, can I have quick review on my cherry-picks please ? | |
| 13:53:11 | jlejeune | https://review.opendev.org/c/openstack/nova/+/927896 | |
| 13:53:11 | jlejeune | https://review.opendev.org/c/openstack/nova/+/927898 | |
| 13:53:14 | jlejeune | https://review.opendev.org/c/openstack/nova/+/927897 | |
| 13:53:57 | dansmith | um, did we backport the ami changes? | |
| 13:54:58 | dansmith | huh, I guess so | |
| 13:56:37 | dansmith | sean-k-mooney: these are not functional changes, which means there's no chance of introducing a regression if we approve these out of order.. any opposition to just approving them all and let the dust settle? | |
| 13:57:10 | sean-k-mooney | you mean approving across all stabel branches | |
| 13:57:36 | sean-k-mooney | it will fail the backport check in gate requiring a recheck | |
| 13:58:01 | sean-k-mooney | unless we workaround it by addding StableOnly | |
| 13:58:21 | sean-k-mooney | or whatever the commit message thing we need to add upstream is | |
| 13:58:58 | vicent | Good morning! I am installing openstack-ansible 29.0.2 on metal. I noticed the nova.conf template doesn't include the rootwrap_config field. | |
| 13:59:00 | sean-k-mooney | dansmith: so no no objection but mechanically it wont work right now | |
| 14:00:14 | dansmith | sean-k-mooney: oh right forgot | |
| 14:00:31 | vicent | I am using nova image_type: lvm. | |
| 14:01:33 | vicent | Because of that nova gives me error: "2024-09-04 17:00:41.313 73654 WARNING oslo.privsep.daemon [-] privsep log: sudo: nova-rootwrap: command not found". Isn't that a bug? | |
| 14:06:52 | jlejeune | dansmith: yes we backported AMI removal commit on all stable branchs | |
| 14:11:07 | sean-k-mooney | ill review those back ports and premetivly +2 but hold +w on the older ones | |
| 14:11:13 | sean-k-mooney | then we can just +w as they merge | |
| 14:11:41 | vicent | ok, adding the rootwrap_config field on nova.conf didn't work. Any idea why nova-rootwrap is not found? It is there on the venv... $ ls /openstack/venvs/nova-29.0.2/bin/nova-rootwrap | |
| 14:11:45 | vicent | /openstack/venvs/nova-29.0.2/bin/nova-rootwrap | |
| 14:29:03 | vicent | I am basically trying this: | |
| 14:29:06 | vicent | https://www.cloudnull.io/2017/12/nova-lvm-an-iop-love-story/ | |
| 14:33:36 | sean-k-mooney | so nova does not use rootwap realy itslef its used by os-brick | |
| 14:33:55 | sean-k-mooney | and its used for spawnign privsep | |
| 14:39:33 | vicent | https://github.com/openstack/nova/blob/master/nova/virt/libvirt/storage/lvm.py#L92 this is where nova calls privsep | |
| 14:48:55 | vicent | If privsev is not used in nova, then this is a problem: "2024-09-04 17:47:01.170 113107 INFO oslo.privsep.daemon [None req-f50b55b0-5b93-4a92-9c95-7d03b61d04b6 - - - - - -] Running privsep helper: ['sudo', 'nova-rootwrap', '/etc/nova/rootwrap.conf', 'privsep-helper', '--config-file', '/etc/nova/nova.conf', '--privsep_context', 'nova.privsep.sys_admin_pctxt', '--privsep_sock_path', | |
| 14:49:01 | vicent | '/tmp/tmpdshabbj0/privsep.sock']" | |
| 15:18:24 | opendevreview | Merged openstack/nova stable/2024.1: Skip snapshot test when missing qemu-img https://review.opendev.org/c/openstack/nova/+/927896 | |
| 15:39:32 | mnaser | dansmith: are you able to merge https://review.opendev.org/c/openstack/os-test-images/+/926942 by any chance? we use github mirrors for running jobs and since the job to mirror that repo was missing, its empty and breaking ci | |
| 15:39:55 | mnaser | https://review.opendev.org/c/openstack/project-config/+/927877 merged now so it will mirror moving forward, but it needs a change merged to push it up | |
| 15:40:49 | dansmith | mnaser: um, maybe because it's marked as usable=false, but I don't understand why you need this specific image | |
| 15:41:29 | mnaser | dansmith: i dont need it at all, but i need _anything_ to merge to make it successfully mirror to github - https://github.com/openstack/os-test-images | |
| 15:41:46 | mnaser | if you dont want to merge that im happy to make a small noop change that we can push through so it mirrors successfully | |
| 15:41:56 | dansmith | I think I'd rather merge something trivial instead of this just because it could cause everything to fail | |
| 15:42:04 | dansmith | yeah, push something up and I'll +W it | |
| 15:42:32 | mnaser | ok thank you | |
| 15:43:41 | mnaser | dansmith: https://review.opendev.org/c/openstack/os-test-images/+/928001 | |
| 15:44:16 | dansmith | done | |
| 16:00:31 | melwitt | sean-k-mooney: hey, I saw your comment on https://review.opendev.org/c/openstack/nova/+/924319 about being in a freeze period. I thought we are frozen for features but not frozen for bug fixes? | |
| 16:11:33 | dansmith | melwitt: no bug reference was the point I think | |
| 16:11:43 | dansmith | looks like a feature without it as it doesn't even say "fix.." or anything | |
| 16:13:31 | melwitt | dansmith: ah, ok. that makes more sense, thanks | |
| 16:19:12 | sean-k-mooney | ya i wanted a bug link for backporting mainly | |
| 16:19:40 | sean-k-mooney | until RC1 we can merge bug afeter RC1 they shoudl only be regression bugs but we will have mater at that point | |
| 16:19:54 | sean-k-mooney | *master at that point reopend for dev | |
| 16:20:08 | dansmith | yeah so between m3 and rc1 being stricter about requiring a bug ref makes sense ot me | |
| 16:20:31 | melwitt | sean-k-mooney: gotcha. thanks | |
| 16:20:41 | sean-k-mooney | that and i asked for a minor release note becasue if operator have doen this already | |
| 16:21:00 | sean-k-mooney | they need to be aware that the changes they made did not propagate to the requst spec | |
| 16:21:17 | melwitt | I saw that but misinterpreted the "freeze" part of your sentence | |
| 16:21:36 | sean-k-mooney | well i wanted to also check with other how we felt about merging it | |
| 16:21:46 | sean-k-mooney | but i saw this as a very low risk | |
| 16:22:29 | melwitt | sure | |
| 16:25:56 | mnaser | https://github.com/openstack/os-test-images -- thanks, i got what i need | |
| 16:26:33 | dansmith | mnaser: cool | |
| 16:36:35 | opendevreview | Lajos Katona proposed openstack/nova master: WIP Use SDK for Neutron https://review.opendev.org/c/openstack/nova/+/928022 | |
| 16:37:26 | sean-k-mooney | lajoskatona: best of luck. you are the third person to start that | |
| 16:38:32 | sean-k-mooney | lajoskatona: with that said we would happliy move to the sdk if you can make it work properly | |
| 16:39:23 | sean-k-mooney | lajoskatona: there are some subtleies in that we need to use both admin and non admin clinets and we need to includ a service token when making request ot nutron too | |
| 16:41:16 | sean-k-mooney | lajoskatona: gibi and Uggla have had expirnce with using service token and the sdk recently with the manila serise so we likely ahve some refernc matiral for that | |
| 16:41:52 | sean-k-mooney | our previous usage of sdk was for ironic and placment both of which were admin only clients | |
| 16:42:20 | sean-k-mooney | so neutron and manila will be the first that need to have a mix of admin and non admin token usage | |
| 17:02:37 | lajoskatona | sean-k-mooney: :-) | |