| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2024-09-03 | |||
| 15:14:48 | jlejeune | something like that ? | |
| 15:18:03 | dansmith | sure, but that will fail pep8 | |
| 15:18:37 | jlejeune | my bad | |
| 15:18:42 | dansmith | but looks okay to me otherwise | |
| 15:26:07 | opendevreview | Julien LE JEUNE proposed openstack/nova master: Skip snapshot test when missing qemu-img https://review.opendev.org/c/openstack/nova/+/925208 | |
| 15:33:14 | dansmith | sean-k-mooney: I think you commented about using backslash continuations recently on another patch, not sure why you didn't on this one, but I also can't find out guideline about that | |
| 15:33:55 | sean-k-mooney | ya i personally hate them but we are not as concistent about not doing it as i would like | |
| 15:33:55 | dansmith | I know we have them in some places and I'm not sure I care about our previous hard and fast rule about it, other than just wanting to be consistent | |
| 15:34:15 | sean-k-mooney | and im presonally geting very tired of style related review comments | |
| 15:34:21 | dansmith | ah right here, second bullet: https://docs.openstack.org/hacking/latest/user/hacking.html | |
| 15:34:48 | sean-k-mooney | yep its documetned but not enforced by tools | |
| 15:34:51 | dansmith | jlejeune: ^ | |
| 15:34:52 | dansmith | right | |
| 16:02:13 | opendevreview | Julien LE JEUNE proposed openstack/nova master: Skip snapshot test when missing qemu-img https://review.opendev.org/c/openstack/nova/+/925208 | |
| 16:05:50 | erlon | Are we having the nova meeting today? | |
| 16:06:01 | gibi | erlon: no, it is cancelled | |
| 16:06:10 | gibi | bauzas is off today | |
| 16:06:17 | erlon | ah, ok, missed the announcement | |
| 16:06:51 | sean-k-mooney | sylvain is at the opensatck asia summit event in korea currently | |
| 16:07:19 | sean-k-mooney | erlon: did you have a pressing topic ot bring up? | |
| 16:07:25 | gibi | yeah that is the more precise term instead of being off :) | |
| 16:07:36 | erlon | I was to bring the shared group to attention again | |
| 16:07:55 | erlon | we have addressed the last reviews suggestions | |
| 16:07:58 | sean-k-mooney | shared security groups or somethign else? | |
| 16:08:15 | erlon | shared security groups | |
| 16:08:20 | erlon | https://review.opendev.org/c/openstack/nova/+/811521 | |
| 16:08:30 | erlon | from this series: https://review.opendev.org/q/topic:sg-shared-filter | |
| 16:08:34 | sean-k-mooney | ah well for that i think we will liikely jsut reapprove that for next cycle | |
| 16:08:59 | sean-k-mooney | so once RC 1 is creeated in 2 weeks master will reopen for new work and it can proceed | |
| 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 | |