| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2024-02-19 | |||
| 17:43:12 | sean-k-mooney | JayF: you can set teh shard key in the nova.conf trivially like this https://github.com/openstack/nova/blob/master/.zuul.yaml#L459-L485 | |
| 17:43:57 | sean-k-mooney | im not sure how you go about tagging the relevent fake ironics with a shard key | |
| 17:44:46 | sean-k-mooney | if really needed to hack thing quickly i would use a local.sh file created by a pre playbook | |
| 17:45:10 | sean-k-mooney | like this https://review.opendev.org/c/openstack/nova/+/679656/13/playbooks/nfv/nfv.yaml | |
| 17:45:24 | sean-k-mooney | but i assume you have a nicer way to do that in teh ironic plugin | |
| 17:45:37 | sean-k-mooney | ideally via a local.conf parmater | |
| 17:46:40 | sean-k-mooney | JayF: if your not familar with /opt/stack/devstack/local.sh if it exist it run right before devstack finishes so if a job creates on eit will be run after devstack has otherwise run fully but before any tests are run | |
| 17:47:31 | sean-k-mooney | so its a way to hack in funcitonaltiy quickly via a zuul job that hsould eventually be in a devstack plugin | |
| 17:50:27 | JayF | honestly that's as nifty for local dev setup as it is for a CI trick, ty | |
| 17:50:58 | sean-k-mooney | yes so the reason ci jobs should not use it hevialy | |
| 17:51:05 | sean-k-mooney | is because its ment to be reserved for local dev | |
| 17:51:12 | sean-k-mooney | but they can use it | |
| 17:51:28 | sean-k-mooney | plugins just should not require it to have stuff to work | |
| 17:51:46 | sean-k-mooney | since its for the user not the "framework/plugins" to use | |
| 17:52:32 | sean-k-mooney | so form a ci point of view its perfectly valid to use | |
| 17:52:46 | sean-k-mooney | but if its complex it proably shoudl be a feature in the plugin/devstack instead | |
| 17:54:00 | JayF | I mean, yeah, it's kinda obvious it'd be hacky to land a CI job using that | |
| 17:56:58 | sean-k-mooney | JayF: i was epecting to see something in enroll_nodes https://github.com/openstack/ironic/blob/245505dc04da6e401e1810812809fef3f3757f04/devstack/lib/ironic#L2431 | |
| 17:57:09 | sean-k-mooney | to set the shard key on the node | |
| 17:57:12 | sean-k-mooney | did i miss that | |
| 17:57:49 | JayF | shard key is optional; you only need it when setting things up | |
| 17:58:07 | JayF | and it's unused upstream until the nova change lands | |
| 17:58:20 | JayF | so there's nothing to miss (yet) | |
| 17:58:29 | sean-k-mooney | ah ok so the devstack pluging change has not happened yet | |
| 17:58:47 | sean-k-mooney | i was epectign to see a new Settign added to the plugin | |
| 17:59:03 | sean-k-mooney | and hten have it append the shared key into the node options if its defined | |
| 17:59:17 | JayF | sean-k-mooney: https://opendev.org/openstack/ironic-tempest-plugin/commit/f2b9c74c9cb8459a979d9002aae3c1a41737c77a is the testing that exists r/n in tempest, no scenario tests yet | |
| 17:59:23 | JayF | this conversation is me kicking off that work for this week | |
| 17:59:54 | sean-k-mooney | right but that patch is in the tempest plugin not the devstack plugin | |
| 18:00:24 | JayF | yep; that's what I'm saying, nothing scenario is done, I have the old patch from last cycle which is basically useless now and never landed (but did what you're lookign for) | |
| 18:00:33 | sean-k-mooney | ah ok | |
| 18:01:29 | sean-k-mooney | well its not useless in that it allows you to deploy locally with ironic fake nodes via devstack | |
| 18:01:36 | sean-k-mooney | with shards | |
| 18:01:54 | JayF | Yeah, but it's useless in terms of testing coverage | |
| 18:02:07 | JayF | because it was passing even when the client wasn't even passing ?shard=blah to the Ironic PAI | |
| 18:02:11 | JayF | that's what I mean by useless :D | |
| 18:18:39 | sean-k-mooney | it depends on what you were trying to test. but i agree it alone was not suffienct | |
| 18:20:27 | sean-k-mooney | dansmith: melwitt nova-lvm passed with https://review.opendev.org/c/openstack/nova/+/909469 by the way https://zuul.opendev.org/t/openstack/build/5069b706178e4430ba48bdd52234b065 so do we want to just merge that for now | |
| 18:21:15 | sean-k-mooney | memory_tracker low_point: 358852 | |
| 18:21:51 | sean-k-mooney | SwapFree: 5804540 kB | |
| 18:22:06 | sean-k-mooney | so we are not ussing most of the addtional swap | |
| 18:22:37 | dansmith | I'm cool with it yeah :) | |
| 18:22:52 | sean-k-mooney | but that does not mean it was not needed and just not clolelcted i nthe memory tracker | |
| 18:23:11 | dansmith | um, parse error | |
| 18:23:26 | dansmith | but yeah, I'm in favor of 8G swap for any of these that are on the border | |
| 18:23:47 | sean-k-mooney | i mean the memory tracker log runs perodically adn the extra swap could have been needed breifly in between runs | |
| 18:26:32 | melwitt | sean-k-mooney: noice, +W | |
| 20:43:53 | opendevreview | Merged openstack/nova stable/2023.1: libvirt: stop enabling hyperv feature reenlightenment https://review.opendev.org/c/openstack/nova/+/909229 | |
| 21:11:31 | opendevreview | Merged openstack/nova master: bump nova-lvm to use 8G of swap from 4G https://review.opendev.org/c/openstack/nova/+/909469 | |
| #openstack-nova - 2024-02-20 | |||
| 06:26:12 | opendevreview | Merged openstack/nova stable/2023.1: libvirt: Stop unconditionally enabling evmcs https://review.opendev.org/c/openstack/nova/+/909228 | |
| 07:01:12 | opendevreview | Merged openstack/nova master: HyperV: Remove RDP console API https://review.opendev.org/c/openstack/nova/+/906809 | |
| 08:56:23 | opendevreview | Fabian Wiesel proposed openstack/nova master: Do not close returned image-chunk iterator https://review.opendev.org/c/openstack/nova/+/909474 | |
| 09:05:25 | opendevreview | Fabian Wiesel proposed openstack/nova master: Do not close returned image-chunk iterator https://review.opendev.org/c/openstack/nova/+/909474 | |
| 11:51:14 | opendevreview | Mohammed Naser proposed openstack/nova stable/zed: libvirt: Stop unconditionally enabling evmcs https://review.opendev.org/c/openstack/nova/+/909541 | |
| 11:52:03 | opendevreview | Mohammed Naser proposed openstack/nova stable/zed: libvirt: stop enabling hyperv feature reenlightenment https://review.opendev.org/c/openstack/nova/+/909098 | |
| 12:04:01 | opendevreview | Fabian Wiesel proposed openstack/nova master: Do not close returned image-chunk iterator https://review.opendev.org/c/openstack/nova/+/909474 | |
| 12:05:43 | opendevreview | Fabian Wiesel proposed openstack/nova master: Do not close returned image-chunk iterator https://review.opendev.org/c/openstack/nova/+/909474 | |
| 12:53:53 | frickler | if someone would want to look into nova testing failures when upgrading jsonschema, that would be great https://review.opendev.org/c/openstack/requirements/+/909452 | |
| 13:00:31 | sean-k-mooney | looks like there is only one failing test | |
| 13:02:24 | sean-k-mooney | this might be an issue in placment rather then nova | |
| 13:03:31 | sean-k-mooney | perhaps not | |
| 13:03:45 | sean-k-mooney | i dont really have time to look at this now but its hopefuly a smal fix | |
| 13:04:15 | sean-k-mooney | frickler: i assume ye are tryign to do this bump before the non clint lib freeze on thrusday | |
| 14:10:45 | frickler | sean-k-mooney: well actually iiuc reqs freeze is next week, but yes, IMO it would be nice if we could be up to date for that. hoping cinder + ironic can also fix their issues | |
| 14:11:34 | sean-k-mooney | frickler: feature freeze is next week but the non client lib freeze is this week so would that not also apply to tird party deps like jsonscema | |
| 14:11:39 | sean-k-mooney | its not a clint lib | |
| 14:16:13 | frickler | sean-k-mooney: well this week is final release for non-client libs, they still have until next week to make it into reqs then | |
| 14:16:33 | sean-k-mooney | oh ok i guess that makes sense | |
| 14:16:33 | frickler | and external libs aren't released by us, so they can get bumped until next week, too | |
| 14:17:20 | sean-k-mooney | ack i was epecting the non clint lib freeze to affect external deps since we migh tnot have time to bump them before ff | |
| 14:17:49 | sean-k-mooney | but i guess we will see if we can adjust it | |
| 14:18:13 | sean-k-mooney | my concern is really is this a behvaior change in the placment api when we do the bump | |
| 14:18:23 | sean-k-mooney | or just a change in the test fixture | |
| 14:19:07 | frickler | yeah, maybe placement my change how validation is done. and I would accept if that is not to be a last minute change | |
| 14:19:17 | frickler | s/my/might need to/ | |
| 14:19:44 | sean-k-mooney | well that would violate teh microverion contract so we would have to restore the old behavior assuming that it was valid before | |
| 14:20:09 | sean-k-mooney | as i said eairler i have not had time to look at this some im hoping someone else will look at this | |
| 15:35:10 | mnaser | what is the nova-next job exactly? | |
| 15:35:33 | mnaser | both nova-next and nova-grenade-multinode failed on changes i was backporting | |
| 15:40:47 | sean-k-mooney | mnaser: its ment to be the job we test new defaults in | |
| 15:41:24 | sean-k-mooney | we sometimes use it to test special configs that we dont want on by default too | |
| 15:41:37 | sean-k-mooney | like right now it tests usign q35 manchine type | |
| 15:41:59 | sean-k-mooney | that probaly should become our default eventually but until it is we test it in nova-next | |
| 15:43:02 | opendevreview | melanie witt proposed openstack/nova master: libvirt: Configure and teardown ephemeral encryption secrets https://review.opendev.org/c/openstack/nova/+/826754 | |
| 15:43:03 | opendevreview | melanie witt proposed openstack/nova master: imagebackend: Add support to libvirt_info for LUKS based encryption https://review.opendev.org/c/openstack/nova/+/826755 | |
| 15:43:03 | opendevreview | melanie witt proposed openstack/nova master: Add encryption support to convert_image https://review.opendev.org/c/openstack/nova/+/870934 | |
| 15:43:04 | opendevreview | melanie witt proposed openstack/nova master: Add hw_ephemeral_encryption_secret_uuid image property https://review.opendev.org/c/openstack/nova/+/870935 | |
| 15:43:04 | opendevreview | melanie witt proposed openstack/nova master: libvirt: make |
|
| 15:43:05 | opendevreview | melanie witt proposed openstack/nova master: Support create with ephemeral encryption for qcow2 https://review.opendev.org/c/openstack/nova/+/870932 | |
| 15:43:05 | opendevreview | melanie witt proposed openstack/nova master: Support (resize|cold migration) with ephemeral encryption for qcow2 https://review.opendev.org/c/openstack/nova/+/870933 | |
| 15:43:06 | opendevreview | melanie witt proposed openstack/nova master: Support live migration with ephemeral encryption for qcow2 https://review.opendev.org/c/openstack/nova/+/905512 | |
| 15:43:06 | opendevreview | melanie witt proposed openstack/nova master: Support rebuild with ephemeral encryption for qcow2 https://review.opendev.org/c/openstack/nova/+/870939 | |
| 15:43:08 | opendevreview | melanie witt proposed openstack/nova master: Support rescue with ephemeral encryption for qcow2 https://review.opendev.org/c/openstack/nova/+/873675 | |
| 15:43:08 | opendevreview | melanie witt proposed openstack/nova master: Add encryption support to qemu-img rebase https://review.opendev.org/c/openstack/nova/+/870936 | |
| 15:43:10 | opendevreview | melanie witt proposed openstack/nova master: Support snapshot with ephemeral encryption for qcow2 https://review.opendev.org/c/openstack/nova/+/870937 | |
| 15:43:10 | opendevreview | melanie witt proposed openstack/nova master: Add backing_encryption_secret_uuid to BlockDeviceMapping https://review.opendev.org/c/openstack/nova/+/907960 | |
| 15:43:12 | opendevreview | melanie witt proposed openstack/nova master: WIP Support encrypted backing files for qcow2 https://review.opendev.org/c/openstack/nova/+/907961 | |
| 15:43:12 | opendevreview | melanie witt proposed openstack/nova master: libvirt: Introduce support for raw with LUKS https://review.opendev.org/c/openstack/nova/+/884313 | |
| 15:43:14 | opendevreview | melanie witt proposed openstack/nova master: libvirt: Introduce support for rbd with LUKS https://review.opendev.org/c/openstack/nova/+/889912 | |
| 15:43:14 | opendevreview | melanie witt proposed openstack/nova master: Support cross cell resize with ephemeral encryption for qcow2 https://review.opendev.org/c/openstack/nova/+/909595 | |