| *** mhen_ is now known as mhen | 01:20 | |
| *** clarkb is now known as Guest24593 | 07:31 | |
| zigo | sean-k-mooney: Is Zed affected by OSSN-0094 ? That's in Debian Bookworm (ie: oldstable), which is why I wonder. So far, I did the work for caracal, dalmatian and epoxy, as there was patches available. | 08:56 |
|---|---|---|
| zigo | It'd be nice ot have patches for zed, antelope and bobcat btw. | 08:58 |
| sean-k-mooney | zigo: well those are unmaintended so as a rule the core team do not patch any bugs on them but we have antelop downstream and i think it was a clean cherry pick | 11:10 |
| sean-k-mooney | zigo: but yes this has basicly been a thing forever. gibi did some monumental heavy liftign to backprot to wallabyt and train inertnally with a lot fo conflict in the tests. | 11:11 |
| sean-k-mooney | but you might be able to just apply it back to zed | 11:11 |
| sean-k-mooney | zigo: the risk profile fo this is relitivly low which is why we didn not push for a CVE ro similar | 11:12 |
| sean-k-mooney | but i woudl encurage packages to patch it where ever they deem resonable | 11:13 |
| zigo | If it's clean apply for Zed to Bobcat, then I'll attempt it. | 11:19 |
| zigo | It doesn't apply on antelope, though it's only one hunk. | 11:21 |
| sean-k-mooney | i suspect if that is resovled for antelope it would not need more change in zed. downstream we only have branches for the release we productise | 11:23 |
| sean-k-mooney | so we jumped form 2023.1 to antelope then to wallaby then to train | 11:23 |
| sean-k-mooney | and witht he other backprot we have those patches are not alwasy useful upstrem | 11:23 |
| sean-k-mooney | since they generally wont apply clenaly | 11:23 |
| sean-k-mooney | our "antelope" branch as bits of epoxy in it alsthoru we really try not to do feature backport mroe then one-two upstream releases as it gets harder as time goes by to backprot bug fixes if we diverge too much | 11:25 |
| zigo | Looks ok, I could backport to Zed and Antelope for at least Nova. | 11:26 |
| zigo | Will do Watcher next. | 11:27 |
| sean-k-mooney | watcher i think will be simpler because it did not have much code developement for quite a while | 11:27 |
| zigo | Thanks for all. :P | 11:28 |
| sean-k-mooney | by the way if you want to propose those to the unmainted branches you can but i have no +2 right there but elodilles may be interested in them | 11:29 |
| zigo | I confirm it's super easy to backport! :) | 11:40 |
| elodilles | zigo: please cherry pick the backport to unmaintained/2023.1 and zed branches and add me as reviewer and i'll review them | 11:40 |
| elodilles | s/backport/bugfix | 11:40 |
| elodilles | sean-k-mooney: thanks, i'm interested, indeed :) | 12:03 |
| tkajinam | Could someone from the core review this remaining python 3.9 removal patch ? https://review.opendev.org/c/openstack/placement/+/953367 | 12:33 |
| tkajinam | and also this tiny one to get rid of a deprecation warning https://review.opendev.org/c/openstack/placement/+/953759 | 12:33 |
| tkajinam | I'd also appreciate any feedback about sev-es series https://review.opendev.org/q/topic:%22bp/amd-sev-es-libvirt-support%22 ! | 12:34 |
| sean-k-mooney | tkajinam: done ill look at sev later if i have time or monday | 12:40 |
| tkajinam | sean-k-mooney, thanks ! | 12:56 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Fix validators for hw:cpu_max_* extra specs https://review.opendev.org/c/openstack/nova/+/957732 | 13:13 |
| zigo | elodilles: https://review.opendev.org/c/openstack/nova/+/958200 https://review.opendev.org/c/openstack/nova/+/958202 | 13:49 |
| zigo | Will you also review backports for Watcher? | 13:49 |
| elodilles | zigo: thanks! sure, i can do review watcher backports as well | 14:22 |
| zigo | Ok, working on it then. | 14:22 |
| zigo | elodilles: I did this one for Antelope: https://review.opendev.org/c/openstack/watcher/+/958207 though there's no unmaintained/zed branch, how can I do it then? | 14:29 |
| opendevreview | Merged openstack/placement master: sqlalchemy: Use built-in declarative https://review.opendev.org/c/openstack/placement/+/953759 | 14:46 |
| elodilles | zigo: thanks for the nova backports, i've found some nits in it, otherwise looks good! see my comments! | 14:54 |
| elodilles | zigo: as watcher is only open for backports till antelope, we cannot test it and post it on zed (and older branches), so antilope is fine there | 14:55 |
| elodilles | zigo: i'll review the watcher patch as well, thanks for that, too! | 14:59 |
| zigo | Fixed according to your suggestions. | 15:00 |
| *** Guest24593 is now known as clarkb | 15:05 | |
| elodilles | zigo: thanks! though i think you forgot to git push it o:) | 15:15 |
| elodilles | zigo: on the other hand i see that tempest jobs are failing as we use pinned tempest (branchless repository) there, so we probably have to exclude some tests in .zuul.yaml to make the tempest based jobs functional on unmaintained/2023.1 | 15:17 |
| priteau | sean-k-mooney: Hello. Could you please review these backports like you did for 2025.1? Thanks! https://review.opendev.org/c/openstack/nova/+/945098 and https://review.opendev.org/c/openstack/nova/+/945099 | 15:25 |
| priteau | Same request for elodilles :) | 15:25 |
| sean-k-mooney | elodilles: there are exactly 3 that will fail | 15:27 |
| sean-k-mooney | elodilles: so you would need to update the regex or backprot and release a new tempest verion to support the unmainted versions | 15:28 |
| sean-k-mooney | elodilles: i think modifying the regex is proably mroe correct | 15:28 |
| elodilles | priteau: ACK, will review in a minute | 15:31 |
| elodilles | sean-k-mooney: backport on tempest is not possible, so yes, the regex is the way-forward i think (see my comment: https://review.opendev.org/c/openstack/nova/+/958200/2#message-f75c8b173bbd03f257e96a5291b4db82f9b3d4c8 ) | 15:32 |
| sean-k-mooney | the proble is that cinder and other project likely also have those tests runnning | 15:34 |
| sean-k-mooney | so the regex change will impact far more then nova | 15:34 |
| elodilles | sean-k-mooney: oh i see :S then probably we have to set the exclude regex in job definition directly, so that will fix more jobs in multiple repositories, though i guess we won't fix everything in one single place :/ | 15:46 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Address issues with images APIs https://review.opendev.org/c/openstack/nova/+/956102 | 16:01 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Separate volume, snapshot and volume attachments https://review.opendev.org/c/openstack/nova/+/952347 | 16:01 |
| opendevreview | Stephen Finucane proposed openstack/nova master: tests: Use valid UUIDs for cinder resources https://review.opendev.org/c/openstack/nova/+/952935 | 16:01 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Only apply "soft" additionalProperties validation to requests https://review.opendev.org/c/openstack/nova/+/952936 | 16:01 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for volumes APIs https://review.opendev.org/c/openstack/nova/+/952348 | 16:01 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for snapshots APIs https://review.opendev.org/c/openstack/nova/+/952349 | 16:01 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for volume attachments APIs https://review.opendev.org/c/openstack/nova/+/952350 | 16:01 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for floating IP APIs https://review.opendev.org/c/openstack/nova/+/952972 | 16:01 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for security group APIs https://review.opendev.org/c/openstack/nova/+/952973 | 16:01 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for services APIs https://review.opendev.org/c/openstack/nova/+/953196 | 16:01 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for server usage audit log APIs https://review.opendev.org/c/openstack/nova/+/953209 | 16:01 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for tenant network APIs https://review.opendev.org/c/openstack/nova/+/956088 | 16:01 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for simple tenant usage APIs https://review.opendev.org/c/openstack/nova/+/956096 | 16:01 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for limits API https://review.opendev.org/c/openstack/nova/+/956139 | 16:01 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for quota class sets API https://review.opendev.org/c/openstack/nova/+/956140 | 16:01 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for quota sets API https://review.opendev.org/c/openstack/nova/+/956141 | 16:01 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for migrations API https://review.opendev.org/c/openstack/nova/+/956142 | 16:01 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for server migrations API https://review.opendev.org/c/openstack/nova/+/956143 | 16:01 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for server tags API https://review.opendev.org/c/openstack/nova/+/956144 | 16:01 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for server metadata APIs https://review.opendev.org/c/openstack/nova/+/956145 | 16:01 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Simplify servers views (1/3) https://review.opendev.org/c/openstack/nova/+/956231 | 16:01 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Simplify servers views (2/3) https://review.opendev.org/c/openstack/nova/+/956232 | 16:01 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Simplify servers views (3/3) https://review.opendev.org/c/openstack/nova/+/956233 | 16:01 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for servers APIs (1/6) https://review.opendev.org/c/openstack/nova/+/956234 | 16:01 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for servers APIs (2/6) https://review.opendev.org/c/openstack/nova/+/956236 | 16:01 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for servers APIs (3/6) https://review.opendev.org/c/openstack/nova/+/956237 | 16:01 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for servers APIs (4/6) https://review.opendev.org/c/openstack/nova/+/956238 | 16:01 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for servers APIs (5/6) https://review.opendev.org/c/openstack/nova/+/956239 | 16:01 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for servers APIs (6/6) https://review.opendev.org/c/openstack/nova/+/956240 | 16:01 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for server shares APIs https://review.opendev.org/c/openstack/nova/+/956266 | 16:01 |
| opendevreview | Stephen Finucane proposed openstack/nova master: tests: Invert validation check https://review.opendev.org/c/openstack/nova/+/956241 | 16:01 |
| stephenfin | sean-k-mooney: gmaan: I'd appreciate reviews on https://review.opendev.org/c/openstack/nova/+/952347 when you can get to it. That's been the cause of a few rebases now, due to the number of lines it touches (it's just code shuffling but git doesn't know that) | 16:02 |
| stephenfin | and from sean-k-mooney on the one below it (gmaan is already +2 there) | 16:03 |
| gmaan | stephenfin: +2 | 16:18 |
| stephenfin | ty | 16:18 |
| opendevreview | Stephen Finucane proposed openstack/nova stable/2025.1: api: Fix validators for hw:cpu_max_* extra specs https://review.opendev.org/c/openstack/nova/+/957739 | 16:21 |
| opendevreview | Stephen Finucane proposed openstack/nova stable/2024.2: api: Fix validators for hw:cpu_max_* extra specs https://review.opendev.org/c/openstack/nova/+/957744 | 16:21 |
| opendevreview | Stephen Finucane proposed openstack/nova stable/2024.1: api: Fix validators for hw:cpu_max_* extra specs https://review.opendev.org/c/openstack/nova/+/957745 | 16:22 |
| opendevreview | Max proposed openstack/nova master: performance: reduce calls to libvirt / add caching https://review.opendev.org/c/openstack/nova/+/922855 | 16:24 |
| opendevreview | Masahito Muroi proposed openstack/nova master: Add image_response_extra_properties config https://review.opendev.org/c/openstack/nova/+/958230 | 17:24 |
| masahito | hi, nova team. We have already used the image properties fields to see our own custom image properties which not listed in the image meta object. if nova keeps showing the properties, it's super helpful. please review the patch if it's not a wrong direction. | 17:30 |
| melwitt | sean-k-mooney: do you know if host domain capabilities like TPM are ever expected to be available in CI? I tried to create some regular tempest TPM tests and it failed bc there was tpm=no in host domain caps | 18:26 |
| sean-k-mooney | you mean upstream | 18:27 |
| sean-k-mooney | it shoudl work | 18:27 |
| sean-k-mooney | vtpm does not depend on havign a tpm on the host | 18:27 |
| sean-k-mooney | youjust need ot have the swtpm package isntalled | 18:27 |
| sean-k-mooney | and ubuntu 24.04 is new enoguht to meet all our requiremetns | 18:28 |
| sean-k-mooney | melwitt: do you have an exampel failure | 18:28 |
| melwitt | uhm yeah, sec | 18:28 |
| melwitt | sean-k-mooney: this, just trying to run only the nova-live-migration job https://review.opendev.org/c/openstack/nova/+/957477 | 18:29 |
| melwitt | ok I must need some additional setup or such | 18:29 |
| sean-k-mooney | is https://docs.openstack.org/nova/latest/configuration/config.html#libvirt.swtpm_enabled set to true | 18:33 |
| sean-k-mooney | ok it is https://fdc1d05a9a337a8993b4-089607d394060d72ce519e30966a0033.ssl.cf2.rackcdn.com/openstack/4eb4b144eadc47b2848afc0e4ac315c8/controller/logs/etc/nova/nova-cpu_conf.txt | 18:34 |
| melwitt | yes but I don't have the swtpm_user and swtpm_group set, not sure if that matters | 18:34 |
| melwitt | I noticed in whitebox it sets those | 18:34 |
| sean-k-mooney | it shouldnt by default | 18:34 |
| sean-k-mooney | hum is it passign there? | 18:34 |
| sean-k-mooney | the default should work | 18:35 |
| sean-k-mooney | i dont see the swtpm package installed | 18:36 |
| sean-k-mooney | https://fdc1d05a9a337a8993b4-089607d394060d72ce519e30966a0033.ssl.cf2.rackcdn.com/openstack/4eb4b144eadc47b2848afc0e4ac315c8/controller/logs/dpkg-l.txt | 18:36 |
| sean-k-mooney | i wonder if its a suggested or recomemned package | 18:36 |
| melwitt | yeah when you said that earlier I was thinking I need to add something to install it probablah | 18:36 |
| sean-k-mooney | 25-08-16 18:29:02.173 | alsa-ucm-conf alsa-topology-conf libxml2-utils swtpm swtpm-tools mdevctl | 18:36 |
| sean-k-mooney | so it in recommended | 18:37 |
| melwitt | I shall try it. thanks | 18:37 |
| sean-k-mooney | https://paste.opendev.org/show/bcoAzqGjo8UbLR3JWy1i/ | 18:37 |
| melwitt | right next to our friend mdevctl | 18:38 |
| sean-k-mooney | https://github.com/openstack/devstack/blob/master/lib/nova_plugins/functions-libvirt#L72 | 18:39 |
| sean-k-mooney | you can add it there | 18:39 |
| sean-k-mooney | proably with mdevctl while yoru at it for good messure | 18:39 |
| melwitt | cool thanks :) | 18:40 |
| sean-k-mooney | i woudl sepcficly add `swtpm swtpm-tools mdevctl` | 18:41 |
| sean-k-mooney | i think the names are the same for centos too but i woudl just do it for ubuntu for now | 18:42 |
| sean-k-mooney | if you submit a patch feel free to add me and ill take a look on monday | 18:42 |
| sean-k-mooney | https://repology.org/project/swtpm/versions | 18:43 |
| sean-k-mooney | swtpm is the same for waht its worth | 18:43 |
| sean-k-mooney | im just not sure about the tools package | 18:43 |
| sean-k-mooney | https://repology.org/project/mdevctl/versions | 18:43 |
| sean-k-mooney | mdevctl also has the same name everywhere we care about | 18:44 |
| sean-k-mooney | https://rpmfind.net/linux/rpm2html/search.php?query=swtpm-tools | 18:45 |
| sean-k-mooney | swtpm-tools looks portable too | 18:45 |
| sean-k-mooney | so you could add all 3 in both branches and just see if the devstack josb pass | 18:46 |
| sean-k-mooney | it probly will | 18:46 |
| melwitt | ok sweet thanks | 18:48 |
| *** mtreinish_ is now known as mtreinish | 20:37 | |
| *** logan_ is now known as Guest24660 | 20:52 | |
| *** dviroel_ is now known as dviroel | 20:55 | |
| *** bryanfraschetti_ is now known as bryanfraschetti | 20:55 | |
| *** abongale_ is now known as abongale | 20:55 | |
| *** jayaanand__ is now known as jayaanand_ | 20:55 | |
| *** jungleboyj_ is now known as jungleboyj | 20:55 | |
| *** msaravan__ is now known as msaravan | 20:55 | |
| *** rpittau_ is now known as rpittau | 20:55 | |
| *** tonyb_ is now known as tonyb | 20:55 | |
| *** gmaan_ is now known as gmaan | 20:55 | |
| *** clarkb is now known as Guest24679 | 20:58 | |
| opendevreview | Merged openstack/nova master: api: Address issues with images APIs https://review.opendev.org/c/openstack/nova/+/956102 | 21:14 |
| *** Guest24679 is now known as clarkb | 21:44 | |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!