| opendevreview | Nobuhiro MIKI proposed openstack/nova-specs master: Fix build failures caused by changes in Pillow v10.0.0 https://review.opendev.org/c/openstack/nova-specs/+/887661 | 09:44 |
|---|---|---|
| opendevreview | Amit Uniyal proposed openstack/nova master: Refactor CinderFixture https://review.opendev.org/c/openstack/nova/+/885756 | 10:28 |
| opendevreview | Amit Uniyal proposed openstack/nova master: Reproducer for dangling bdms https://review.opendev.org/c/openstack/nova/+/881457 | 10:28 |
| opendevreview | Amit Uniyal proposed openstack/nova master: Delete dangling bdms https://review.opendev.org/c/openstack/nova/+/882284 | 10:28 |
| zigo | Hi there! Considering what happened with the xsaves flag being removed from hosts with the latest kernels, I backported https://review.opendev.org/c/openstack/nova/+/774240 to Victoria in my Debian packaging, so we can remove xsaves from compute nodes where the flag is still there, and have always the same feature set in our deployment. | 10:30 |
| zigo | Moving forward, we have lots of VMs with the xsaves feature. Is there a way to trick libvirt to make VMs live-migrate to a host that doesn't have the flag? | 10:30 |
| zigo | This is an enormous headache ... | 10:31 |
| opendevreview | Amit Uniyal proposed openstack/nova master: Refactor CinderFixture https://review.opendev.org/c/openstack/nova/+/885756 | 10:41 |
| opendevreview | Amit Uniyal proposed openstack/nova master: Reproducer for dangling bdms https://review.opendev.org/c/openstack/nova/+/881457 | 10:41 |
| opendevreview | Amit Uniyal proposed openstack/nova master: Delete dangling bdms https://review.opendev.org/c/openstack/nova/+/882284 | 10:41 |
| opendevreview | Merged openstack/nova stable/victoria: Remove mentions of removed scheduler filters https://review.opendev.org/c/openstack/nova/+/858051 | 12:21 |
| opendevreview | Merged openstack/nova master: enable validations in nova-lvm https://review.opendev.org/c/openstack/nova/+/887632 | 13:00 |
| elodilles | oh, cool, this has merged ^^^ \o/ | 13:12 |
| elodilles | sean-k-mooney: do you want to backport it to stable/2023.1? | 13:13 |
| sean-k-mooney | sure i can do it now | 13:15 |
| sean-k-mooney | we need it on all non EM branchs | 13:16 |
| sean-k-mooney | where tempest is using master | 13:16 |
| opendevreview | sean mooney proposed openstack/nova stable/2023.1: enable validations in nova-lvm https://review.opendev.org/c/openstack/nova/+/887674 | 13:16 |
| elodilles | sean-k-mooney: ack, thanks! | 13:33 |
| elodilles | i also backported the devstack patch to stable/2023.1 that fixes the similar problem with nova-ceph-multistore job, but it is somehow not enough (this is the patch: https://review.opendev.org/c/openstack/devstack/+/887660 ) | 13:35 |
| bauzas | stephenfin: other sphinx SMEs, it looks like we have a CI failure on nova specs, fortunately we have a fix https://review.opendev.org/c/openstack/nova-specs/+/887661 | 13:57 |
| stephenfin | I wonder if we should just drop blockdiag? | 13:58 |
| bauzas | stephenfin: IMHO we use it for some specs | 14:01 |
| stephenfin | right, but we can just generate the SVGs once and store them | 14:01 |
| stephenfin | or PNGs or whatever they are | 14:02 |
| bauzas | stephenfin: the problem is that we have the spec freeze tomorrow | 14:03 |
| bauzas | stephenfin: so, for the moment our gate is blocked | 14:04 |
| sean-k-mooney | i woudl be fien with droping it | 14:04 |
| sean-k-mooney | and just having it ascii or svg | 14:04 |
| bauzas | ditto, but for this week, I'd prefer to first fix our CI | 14:05 |
| sean-k-mooney | yep | 14:07 |
| sean-k-mooney | fix ci but going forward just using ascii or an svg woudl be fine | 14:08 |
| bauzas | yup, fair enough, I'm okay provided we keep the sources | 14:12 |
| bauzas | sean-k-mooney: stephenfin: provided a comment on https://review.opendev.org/c/openstack/nova-specs/+/887661 | 14:14 |
| bauzas | can either of you just approve this patch and we could revert it next week ? | 14:14 |
| opendevreview | Stephen Finucane proposed openstack/nova-specs master: Remove sphinxcontrib.seqdiag https://review.opendev.org/c/openstack/nova-specs/+/887702 | 14:24 |
| stephenfin | bauzas: there you go | 14:24 |
| stephenfin | builds fine locally | 14:25 |
| bauzas | sean-k-mooney: your vote may be appreciated on https://review.opendev.org/c/openstack/nova-specs/+/887702 | 14:34 |
| bauzas | gibi: sean-k-mooney: can either of you can vote for https://review.opendev.org/c/openstack/nova-specs/+/887702 in order to unblock our specs repo ? | 15:28 |
| sean-k-mooney | sure ill do it now | 15:37 |
| sean-k-mooney | stephenfin: the content looks correct so far. there is a tiny regession in fucntionaltiy with how it interact with a client side browser extention | 15:43 |
| sean-k-mooney | but i think i cna overlook that | 15:43 |
| sean-k-mooney | basically i use "dark reader" to view everythingin darkmode even when the site does not have a dark mode. | 15:45 |
| sean-k-mooney | with the existing docs site the svg get inverted properlfy with the new content on the docs preview site they are not inverted | 15:45 |
| sean-k-mooney | i think this is because we changed from using <svg> to the <image> | 15:47 |
| sean-k-mooney | so even though we are displayign an svg its content is not aviable to the bowser as an svg so it being rendered differently | 15:49 |
| sean-k-mooney | as a result darkreader cannot modify the styling | 15:50 |
| sean-k-mooney | i approved it for now but if there was a way to embded the svg as an svg that would be nice | 15:50 |
| sean-k-mooney | https://github.com/sphinx-doc/sphinx/issues/2240 apprently its an open issue ssicne 2016 | 15:52 |
| opendevreview | Merged openstack/nova-specs master: Remove sphinxcontrib.seqdiag https://review.opendev.org/c/openstack/nova-specs/+/887702 | 15:56 |
| dvo-plv | sean-k-mooney, I have rerun zuul after your fix, all pass. You can review if you have a chance: bp https://review.opendev.org/c/openstack/nova/+/876075/16 | 16:47 |
| sean-k-mooney | dvo-plv: not today as im debugging somethign but ill try and take a look tomorrow | 16:49 |
| sean-k-mooney | quick glance https://review.opendev.org/c/openstack/nova/+/876075/16/nova/tests/functional/test_servers.py looks good | 16:50 |
| mgariepy | can this be pushed a bit ? https://review.opendev.org/c/openstack/nova/+/882914 | 18:04 |
| mgariepy | thanks sean-k-mooney | 21:05 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!