| opendevreview | chengsheng proposed openstack/nova master: libvirt: Use improved guest CPU config APIs https://review.opendev.org/c/openstack/nova/+/762330 | 01:48 |
|---|---|---|
| opendevreview | Qiu Fossen proposed openstack/nova master: Allow migrating PMEM's data https://review.opendev.org/c/openstack/nova/+/802225 | 02:44 |
| opendevreview | Felix Huettner proposed openstack/nova master: compute: Avoid duplicate BDMs during reserve_block_device_name https://review.opendev.org/c/openstack/nova/+/801990 | 06:43 |
| *** rpittau|afk is now known as rpittau | 07:55 | |
| opendevreview | Stephen Finucane proposed openstack/nova stable/wallaby: api: Align availability zone info with forced host https://review.opendev.org/c/openstack/nova/+/802236 | 08:28 |
| opendevreview | Stephen Finucane proposed openstack/nova stable/victoria: api: Align availability zone info with forced host https://review.opendev.org/c/openstack/nova/+/802237 | 08:32 |
| opendevreview | Stephen Finucane proposed openstack/nova stable/ussuri: Fix error that cannot overwrite policy rule for 'forced_host' https://review.opendev.org/c/openstack/nova/+/802238 | 08:35 |
| opendevreview | Stephen Finucane proposed openstack/nova stable/ussuri: api: Align availability zone info with forced host https://review.opendev.org/c/openstack/nova/+/802239 | 08:35 |
| opendevreview | Stephen Finucane proposed openstack/nova stable/train: Fix error that cannot overwrite policy rule for 'forced_host' https://review.opendev.org/c/openstack/nova/+/802240 | 08:37 |
| opendevreview | Stephen Finucane proposed openstack/nova stable/train: api: Align availability zone info with forced host https://review.opendev.org/c/openstack/nova/+/802241 | 08:37 |
| lyarwood | https://zuul.opendev.org/t/openstack/builds?project=openstack%2Fnova&branch=master&pipeline=gate&result=failure - me takes a look at these failures | 08:52 |
| opendevreview | Lee Yarwood proposed openstack/nova master: libvirt: Handle silent failures to extend volume within os-brick https://review.opendev.org/c/openstack/nova/+/801714 | 09:12 |
| lyarwood | bauzas: would you mind hitting this series to land the changes to the cherry-pick script on stable? https://review.opendev.org/q/Id3e4452883f6a3cf44ff58b39ded82e882e28c23 | 09:17 |
| bauzas | lyarwood: I can take a look | 09:23 |
| lyarwood | many thanks | 09:23 |
| sean-k-mooney | the pdf docs jobs might be broken https://zuul.opendev.org/t/openstack/build/e58c187bc90f43c8a2bb189a54c6175d/log/sphinx-build-pdf.log | 10:56 |
| sean-k-mooney | LaTeX Error: File `tgtermes.sty' not found. | 10:57 |
| sean-k-mooney | we might need texlive-latex-extra | 10:59 |
| sean-k-mooney | ok the missing file is in tex-gyre | 11:28 |
| sean-k-mooney | i think https://review.opendev.org/c/openstack/openstack-zuul-jobs/+/802296 will fix the docs jobs | 11:38 |
| *** gibi is now known as gibi_pto | 14:03 | |
| opendevreview | Lee Yarwood proposed openstack/nova master: WIP compute: Do not mark disabled but down services as in maintenance https://review.opendev.org/c/openstack/nova/+/802317 | 14:41 |
| kashyap | stephenfin: Am I right in recalling that to deprecate disk-bus for floopy devices also require data migrations to remove DB entries using the old values, yeah? | 14:47 |
| stephenfin | I don't think so. I think you'll need operators to do that manually via a nova-manage command (which lyarwood is working on) | 14:48 |
| stephenfin | Not sure what we'll do about updating BDMs though | 14:48 |
| kashyap | Hmm, this feels invasive to do the deprecation for these - I see that no one has done such a thing so far | 14:49 |
| kashyap | I feel tempted to just nuking it is far more effective than all these subtle deprecations and in turn testing, etc. The very few floppy users, if they're there at all, will have to adjust a little | 14:50 |
| kashyap | From the data I've seen from OSP deployments, not a single floppy in guest XMLs I see. | 14:50 |
| * kashyap checks again carefully | 14:50 | |
| melwitt | lyarwood: could you look at https://review.opendev.org/c/openstack/nova/+/791805 (and its backports eventually) if you get a chance? these backports have to merge before a security advisory can be issued | 15:09 |
| opendevreview | Alexandre arents proposed openstack/nova master: libvirt: Abort live-migration job when monitoring fails https://review.opendev.org/c/openstack/nova/+/764435 | 15:25 |
| prometheanfire | do nova people mind looking at https://review.opendev.org/801759 looks like some tests need updating for the new cinderclient | 15:58 |
| sean-k-mooney | prometheanfire: ah we have some unit test that import cinder.v2 | 16:06 |
| sean-k-mooney | well cinderclient.v2 | 16:06 |
| sean-k-mooney | this https://github.com/openstack/nova/blob/master/nova/tests/unit/volume/test_cinder.py#L19 | 16:07 |
| sean-k-mooney | we can porbly grab the v3 verison of that | 16:08 |
| sean-k-mooney | ya https://github.com/openstack/python-cinderclient/blob/master/cinderclient/v3/limits.py exists | 16:08 |
| sean-k-mooney | its only used in one place https://github.com/openstack/nova/blob/master/nova/tests/unit/volume/test_cinder.py#L892 | 16:09 |
| sean-k-mooney | and the usage looks compatible | 16:09 |
| sean-k-mooney | prometheanfire: ill quickly swap it to v3 and see if that fixes it | 16:11 |
| lyarwood | tbh I think you can drop the version entirely | 16:12 |
| * lyarwood checks | 16:12 | |
| lyarwood | ah maybe not | 16:12 |
| sean-k-mooney | i dont see a proxy import | 16:13 |
| sean-k-mooney | there is noting in https://github.com/openstack/python-cinderclient/blob/stable/wallaby/cinderclient/__init__.py | 16:13 |
| lyarwood | right I assumed there would be one | 16:13 |
| lyarwood | something to fup in cinderclient later | 16:13 |
| sean-k-mooney | s/proxy import/alias/ | 16:13 |
| gmann | yeah, for now we can just replace it to v3 | 16:14 |
| sean-k-mooney | yep just running that locall and ill push a patch in a second | 16:14 |
| gmann | ok, I will stop then. | 16:15 |
| opendevreview | sean mooney proposed openstack/nova master: use cinder v3 https://review.opendev.org/c/openstack/nova/+/802334 | 16:17 |
| sean-k-mooney | prometheanfire: if you put a depends on against ^ then the other patch should pass | 16:18 |
| sean-k-mooney | looking at codesearch https://codesearch.opendev.org/?q=cinderclient.v2&i=nope&files=&excludeFiles=&repos= that is the only usage in nova but there are other project that still use v2 | 16:20 |
| *** rpittau is now known as rpittau|afk | 16:37 | |
| prometheanfire | sean-k-mooney: thanks | 17:10 |
| opendevreview | Merged openstack/nova stable/victoria: Reject open redirection in the console proxy https://review.opendev.org/c/openstack/nova/+/791805 | 17:51 |
| opendevreview | Merged openstack/nova master: Add tests for 'AggregateImagePropertiesIsolation' https://review.opendev.org/c/openstack/nova/+/754115 | 17:51 |
| opendevreview | Merged openstack/nova master: Add tests for 'AggregateInstanceExtraSpecsFilter' https://review.opendev.org/c/openstack/nova/+/754116 | 17:51 |
| opendevreview | Merged openstack/nova master: use cinder v3 https://review.opendev.org/c/openstack/nova/+/802334 | 17:51 |
| opendevreview | melanie witt proposed openstack/nova stable/ussuri: Reject open redirection in the console proxy https://review.opendev.org/c/openstack/nova/+/791806 | 19:27 |
| opendevreview | Merged openstack/nova master: libvirt: Always delegate OVS plug to os-vif https://review.opendev.org/c/openstack/nova/+/797428 | 19:45 |
| *** ChanServ changes topic to "This channel is for Nova development. For support of Nova deployments, please use #openstack" | 22:56 | |
| *** prometheanfire is now known as Guest2352 | 22:56 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!