| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2023-06-26 | |||
| 09:06:39 | bauzas | gibi: it was approved IIRC, but lemme double-check | |
| 09:06:56 | bauzas | https://blueprints.launchpad.net/nova/+spec/cold-migrate-to-host-policy correct it was | |
| 09:07:14 | gibi | bauzas: thanks | |
| 09:07:31 | bauzas | https://meetings.opendev.org/meetings/nova/2023/nova.2023-04-25-16.00.log.html#l-203 | |
| 09:07:56 | bauzas | gibi: thanks for doing this round of reviews, I know you're busy | |
| 09:08:21 | gibi | ohh sh*t now I have to do these reviews :D | |
| 09:14:14 | bauzas | gibi: nah, don't feel pressured | |
| 09:14:51 | bauzas | I'm thinking of creating an etherpad of code reviews either way :) | |
| 09:15:50 | gibi | done :D | |
| 09:15:57 | gibi | I mean with the reviews | |
| 09:23:24 | sean-k-mooney | i can review some of them while i wait for my dev env to deploy | |
| 09:26:03 | sean-k-mooney | there is one minor english issue in https://review.opendev.org/c/openstack/nova/+/881562/2/api-ref/source/servers-admin-action.inc and a slight nit later | |
| 09:26:07 | sean-k-mooney | but im +2 on it | |
| 09:26:19 | sean-k-mooney | i held +w in case you want to quickly fix it? | |
| 09:30:52 | bauzas | sean-k-mooney: sorry was working on updating some change | |
| 09:31:06 | bauzas | sean-k-mooney: sure, I can quickly respin for that english point | |
| 09:31:28 | bauzas | should be better than a FUP | |
| 09:31:28 | sean-k-mooney | ack ill fast approve once its ready | |
| 09:34:23 | opendevreview | Sylvain Bauza proposed openstack/nova master: Add a new policy for cold-migrate with host https://review.opendev.org/c/openstack/nova/+/881562 | |
| 09:34:31 | bauzas | sean-k-mooney: gibi: ^ | |
| 10:02:11 | opendevreview | sean mooney proposed openstack/nova master: Remove deprecated AZ filter. https://review.opendev.org/c/openstack/nova/+/886779 | |
| 10:11:20 | sean-k-mooney | bauzas: speaking of easy reviews can you hit this https://review.opendev.org/c/openstack/nova/+/886596 to drop fedora form teh experimental pipeline | |
| 10:11:34 | sean-k-mooney | the infra team are woking on droping the fedora images from ci | |
| 11:00:15 | opendevreview | John Garbutt proposed openstack/nova master: WIP: Limit nodes by ironic shard key https://review.opendev.org/c/openstack/nova/+/886980 | |
| 11:09:52 | opendevreview | John Garbutt proposed openstack/nova master: Deprecate ironic.peer_list https://review.opendev.org/c/openstack/nova/+/883346 | |
| 11:10:12 | opendevreview | John Garbutt proposed openstack/nova master: WIP: Limit nodes by ironic shard key https://review.opendev.org/c/openstack/nova/+/886980 | |
| 11:44:48 | opendevreview | Merged openstack/nova master: Verify a move operation for cross_az_attach=False https://review.opendev.org/c/openstack/nova/+/878948 | |
| 12:15:49 | opendevreview | Danylo Vodopianov proposed openstack/nova master: Packed virtqueue support was added. https://review.opendev.org/c/openstack/nova/+/876075 | |
| 15:07:42 | dansmith | bauzas: I would sure love to have some review on my compute object id set | |
| 15:08:05 | dansmith | I wrote it a long time ago, but I barely remember what's in it anymore, so addressing feedback becomes more and more expensive as time goes on... :/ | |
| 15:08:08 | bauzas | dansmith: aha, right, I need to look at your series | |
| 15:29:18 | bauzas | reminder : tomorrow will be our spec review day | |
| 15:29:38 | bauzas | (adding this reminder also for me :) ) | |
| 16:00:09 | opendevreview | John Garbutt proposed openstack/nova master: Limit nodes by ironic shard key https://review.opendev.org/c/openstack/nova/+/886980 | |
| 16:42:39 | opendevreview | John Garbutt proposed openstack/nova master: WIP: Add nova-manage ironic-compute-node-move https://review.opendev.org/c/openstack/nova/+/886989 | |
| 17:56:50 | gmann | bauzas: dansmith: can either of you approve this, adding py310 func job on gate pipeline too https://review.opendev.org/c/openstack/nova/+/881339 | |
| 17:58:56 | dansmith | approved, but I guess I'm not sure it's really important to run on so many python versions | |
| 17:59:09 | dansmith | it's not likely that python version problems will crop up between check and gate :) | |
| 18:00:29 | gmann | nova-tox-functional-py39 can be removed as py38 and py310 can cover it | |
| 18:01:07 | dansmith | definitely seems prudent to do that yeah | |
| 23:00:10 | opendevreview | melanie witt proposed openstack/nova-specs master: Re-propose spec for ephemeral storage encryption https://review.opendev.org/c/openstack/nova-specs/+/887011 | |
| 23:14:31 | opendevreview | melanie witt proposed openstack/nova-specs master: Re-propose spec for ephemeral encryption for libvirt https://review.opendev.org/c/openstack/nova-specs/+/887012 | |
| #openstack-nova - 2023-06-27 | |||
| 01:52:06 | opendevreview | melanie witt proposed openstack/nova-specs master: Propose tooling and docs for unified limits https://review.opendev.org/c/openstack/nova-specs/+/887014 | |
| 04:33:24 | opendevreview | Ghanshyam proposed openstack/nova-specs master: Re-propose "Policy service role spec" https://review.opendev.org/c/openstack/nova-specs/+/881880 | |
| 05:39:03 | opendevreview | Amit Uniyal proposed openstack/os-vif stable/wallaby: set default qos policy https://review.opendev.org/c/openstack/os-vif/+/886778 | |
| 05:39:04 | opendevreview | Amit Uniyal proposed openstack/os-vif stable/wallaby: Move mtu update request into ovsdb transaction https://review.opendev.org/c/openstack/os-vif/+/887017 | |
| 07:13:41 | bauzas | happy spec review day everyone | |
| 07:30:38 | opendevreview | Amit Uniyal proposed openstack/os-vif stable/wallaby: set default qos policy https://review.opendev.org/c/openstack/os-vif/+/886778 | |
| 09:07:40 | zigo | I'm currently rebuilding a new version of the Qemu package with the fix, I'll let you know if this really fixes the issue ! :) | |
| 09:07:40 | zigo | https://bugzilla.redhat.com/show_bug.cgi?id=1984401#c34 | |
| 09:07:40 | zigo | sean-k-mooney[m]: bauzas: You guys remember I reported that some of my bullseye instances wouldn't live-migrate with an issue in virtio? It appears that the Qemu package in Bullseye has the typo described here: | |
| 09:07:52 | zigo | Gosh, all this for just one char ... | |
| 09:10:17 | bauzas | ok | |
| 09:15:29 | opendevreview | Sylvain Bauza proposed openstack/nova-specs master: Re-propose spec for ephemeral storage encryption https://review.opendev.org/c/openstack/nova-specs/+/887011 | |
| 09:29:48 | opendevreview | Merged openstack/nova-specs master: Re-propose spec for ephemeral encryption for libvirt https://review.opendev.org/c/openstack/nova-specs/+/887012 | |
| 09:46:29 | opendevreview | John Garbutt proposed openstack/nova master: Add nova-manage ironic-compute-node-move https://review.opendev.org/c/openstack/nova/+/886989 | |
| 09:49:14 | sean-k-mooney1 | "vitio-balloon-device/page-poison", | |
| 09:49:16 | sean-k-mooney1 | i see | |
| 10:06:27 | opendevreview | John Garbutt proposed openstack/nova master: Deprecate ironic.peer_list https://review.opendev.org/c/openstack/nova/+/883346 | |
| 10:06:28 | opendevreview | John Garbutt proposed openstack/nova master: Add nova-manage ironic-compute-node-move https://review.opendev.org/c/openstack/nova/+/886989 | |
| 10:06:28 | opendevreview | John Garbutt proposed openstack/nova master: Limit nodes by ironic shard key https://review.opendev.org/c/openstack/nova/+/886980 | |
| 10:58:48 | opendevreview | Amit Uniyal proposed openstack/os-vif stable/wallaby: set default qos policy https://review.opendev.org/c/openstack/os-vif/+/886778 | |
| 11:55:48 | sahid | o/ qucik question guys, have we a tool to clean old deleted instances in database? some sort of housecleaning job | |
| 11:59:37 | sahid | hum looks like nova-manage is the tool that i'm looking for | |
| 12:06:22 | sean-k-mooney | yep the archive-delete-rows and prune commands | |
| 12:06:42 | sean-k-mooney | we generally recommend runing those in a corn job using --before | |
| 12:07:01 | sean-k-mooney | so arcive all delete insace ofver 30ds and pruge them after 90 days | |
| 12:07:16 | sean-k-mooney | run that at least daily in cron | |
| 12:07:43 | sean-k-mooney | with max rows set high enogugh to ensure there is no build up over time | |
| 12:08:35 | sean-k-mooney | https://docs.openstack.org/nova/latest/cli/nova-manage.html#db-archive-deleted-rows and https://docs.openstack.org/nova/latest/cli/nova-manage.html#db-purge | |
| 12:09:13 | sahid | i was actually scary to set a max rows too high, and falldown in a timeout, perhaps having the cron running several times a day with max rows small is better, no? | |
| 12:09:49 | sean-k-mooney | you should not hit any time out but it will lod the db | |
| 12:10:12 | sean-k-mooney | you can do it severlal timees a day but honestly if thise casus an issue on your cloud you have undersided your DB | |
| 12:12:35 | sahid | yes that is right, we perhaps have to care of that the first time | |
| 12:12:50 | sean-k-mooney | i wourd personlaly do something like nova-manage db archive_deleted_rows --until-complete --before $(date -I -d '-1 month') --all-cells --task-log | |
| 12:14:18 | opendevreview | Pierre Libeau proposed openstack/nova master: Use admin_client to allocate port for instance https://review.opendev.org/c/openstack/nova/+/861172 | |
| 13:06:44 | zigo | I can confirm that the one char patch that I decribed at https://bugs.debian.org/1039567 fixes the live-migration of VMs with page_poison activated. | |
| 13:06:46 | zigo | :) | |
| 13:59:07 | sean-k-mooney | zigo: congrats sorry it took so long to figure it out but good to know that is coming to repo near by soon | |
| 13:59:41 | zigo | I already upgraded the unofficial debian.net repo with my custom patched qemu, and will try to upgrade official Bullseye! :) | |
| 14:28:45 | sahid | sean-k-mooney: ack thanks ! | |
| 14:30:27 | opendevreview | Amit Uniyal proposed openstack/os-vif stable/yoga: set default qos policy https://review.opendev.org/c/openstack/os-vif/+/886710 | |
| 14:45:46 | opendevreview | Amit Uniyal proposed openstack/os-vif stable/yoga: set default qos policy https://review.opendev.org/c/openstack/os-vif/+/886710 | |
| 14:49:06 | opendevreview | Amit Uniyal proposed openstack/os-vif stable/xena: set default qos policy https://review.opendev.org/c/openstack/os-vif/+/886716 | |
| 15:01:49 | opendevreview | Amit Uniyal proposed openstack/os-vif stable/yoga: set default qos policy https://review.opendev.org/c/openstack/os-vif/+/886710 | |
| 15:06:13 | bauzas | reminder : nova meeting in 55 mins here | |
| 15:10:33 | opendevreview | Amit Uniyal proposed openstack/os-vif stable/xena: set default qos policy https://review.opendev.org/c/openstack/os-vif/+/886716 | |
| 15:17:41 | opendevreview | Amit Uniyal proposed openstack/nova master: Disconnecting volume from the compute host https://review.opendev.org/c/openstack/nova/+/877446 | |
| 15:17:41 | opendevreview | Amit Uniyal proposed openstack/nova master: Added context manager for instance lock https://review.opendev.org/c/openstack/nova/+/873648 | |
| 16:01:26 | opendevmeet | The meeting name has been set to 'nova' | |
| 16:01:26 | opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | |
| 16:01:26 | opendevmeet | Meeting started Tue Jun 27 16:01:26 2023 UTC and is due to finish in 60 minutes. The chair is bauzas. Information about MeetBot at http://wiki.debian.org/MeetBot. | |
| 16:01:26 | bauzas | #startmeeting nova | |
| 16:01:31 | bauzas | hey folks \o | |
| 16:01:33 | elodilles | o/ | |
| 16:01:34 | auniyal | o/ | |
| 16:01:39 | gibi | o\ | |
| 16:01:40 | bauzas | #link https://wiki.openstack.org/wiki/Meetings/Nova#Agenda_for_next_meeting | |
| 16:02:54 | bauzas | ok let's start while the folks are arriving :) | |
| 16:03:02 | bauzas | #topic Bugs (stuck/critical) | |