| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-07-23 | |||
| 23:02:19 | opendevreview | melanie witt proposed openstack/nova master: DNM testing https://review.opendev.org/c/openstack/nova/+/802133 | |
| #openstack-nova - 2021-07-24 | |||
| 08:04:08 | gibi | gmann: fyi, I will be off next week. I'm just skimmed the discussion about the placement - oslo.policy change. | |
| 08:05:59 | gibi | gmann: I'm OK to change nova to test with master placement in funct test | |
| 08:07:18 | gibi | the placement-lib idea is pretty old and given the lack of developers on placement side I would not try to do such big change | |
| 08:08:33 | gibi | if we switch from pypi to placement master in nova func test, does that solve your dependency issue towards oslo.policy in Xena? | |
| #openstack-nova - 2021-07-25 | |||
| 00:32:12 | opendevreview | melanie witt proposed openstack/nova master: DNM testing https://review.opendev.org/c/openstack/nova/+/802133 | |
| 04:11:00 | opendevreview | Merged openstack/nova master: objects: Fix VIFMigrateData.supports_os_vif_delegation setter https://review.opendev.org/c/openstack/nova/+/797142 | |
| 10:40:22 | opendevreview | Merged openstack/nova master: api: Align availability zone info with forced host https://review.opendev.org/c/openstack/nova/+/798145 | |
| #openstack-nova - 2021-07-26 | |||
| 01:48:34 | opendevreview | chengsheng proposed openstack/nova master: libvirt: Use improved guest CPU config APIs https://review.opendev.org/c/openstack/nova/+/762330 | |
| 02:44:52 | opendevreview | Qiu Fossen proposed openstack/nova master: Allow migrating PMEM's data https://review.opendev.org/c/openstack/nova/+/802225 | |
| 06:43:42 | opendevreview | Felix Huettner proposed openstack/nova master: compute: Avoid duplicate BDMs during reserve_block_device_name https://review.opendev.org/c/openstack/nova/+/801990 | |
| 08:28:22 | 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:32:01 | 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:35:05 | 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:06 | 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:37:19 | 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:20 | 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:52:01 | lyarwood | https://zuul.opendev.org/t/openstack/builds?project=openstack%2Fnova&branch=master&pipeline=gate&result=failure - me takes a look at these failures | |
| 09:12:37 | 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:17:26 | 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:23:49 | bauzas | lyarwood: I can take a look | |
| 09:23:55 | lyarwood | many thanks | |
| 10:56:31 | sean-k-mooney | the pdf docs jobs might be broken https://zuul.opendev.org/t/openstack/build/e58c187bc90f43c8a2bb189a54c6175d/log/sphinx-build-pdf.log | |
| 10:57:12 | sean-k-mooney | LaTeX Error: File `tgtermes.sty' not found. | |
| 10:59:07 | sean-k-mooney | we might need texlive-latex-extra | |
| 11:28:58 | sean-k-mooney | ok the missing file is in tex-gyre | |
| 11:38:30 | sean-k-mooney | i think https://review.opendev.org/c/openstack/openstack-zuul-jobs/+/802296 will fix the docs jobs | |
| 14:41:47 | 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:47:07 | 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:48:04 | 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:17 | stephenfin | Not sure what we'll do about updating BDMs though | |
| 14:49:26 | kashyap | Hmm, this feels invasive to do the deprecation for these - I see that no one has done such a thing so far | |
| 14:50:10 | 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:33 | kashyap | From the data I've seen from OSP deployments, not a single floppy in guest XMLs I see. | |
| 15:09:48 | 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:25:31 | opendevreview | Alexandre arents proposed openstack/nova master: libvirt: Abort live-migration job when monitoring fails https://review.opendev.org/c/openstack/nova/+/764435 | |
| 15:58:24 | prometheanfire | do nova people mind looking at https://review.opendev.org/801759 looks like some tests need updating for the new cinderclient | |
| 16:06:41 | sean-k-mooney | prometheanfire: ah we have some unit test that import cinder.v2 | |
| 16:06:53 | sean-k-mooney | well cinderclient.v2 | |
| 16:07:59 | sean-k-mooney | this https://github.com/openstack/nova/blob/master/nova/tests/unit/volume/test_cinder.py#L19 | |
| 16:08:12 | sean-k-mooney | we can porbly grab the v3 verison of that | |
| 16:08:48 | sean-k-mooney | ya https://github.com/openstack/python-cinderclient/blob/master/cinderclient/v3/limits.py exists | |
| 16:09:17 | 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:36 | sean-k-mooney | and the usage looks compatible | |
| 16:11:05 | sean-k-mooney | prometheanfire: ill quickly swap it to v3 and see if that fixes it | |
| 16:12:32 | lyarwood | tbh I think you can drop the version entirely | |
| 16:12:58 | lyarwood | ah maybe not | |
| 16:13:03 | sean-k-mooney | i dont see a proxy import | |
| 16:13:33 | sean-k-mooney | there is noting in https://github.com/openstack/python-cinderclient/blob/stable/wallaby/cinderclient/__init__.py | |
| 16:13:37 | lyarwood | right I assumed there would be one | |
| 16:13:44 | lyarwood | something to fup in cinderclient later | |
| 16:13:58 | sean-k-mooney | s/proxy import/alias/ | |
| 16:14:21 | gmann | yeah, for now we can just replace it to v3 | |
| 16:14:42 | sean-k-mooney | yep just running that locall and ill push a patch in a second | |
| 16:15:05 | gmann | ok, I will stop then. | |
| 16:17:00 | opendevreview | sean mooney proposed openstack/nova master: use cinder v3 https://review.opendev.org/c/openstack/nova/+/802334 | |
| 16:18:00 | sean-k-mooney | prometheanfire: if you put a depends on against ^ then the other patch should pass | |
| 16:20:17 | 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 | |
| 17:10:37 | prometheanfire | sean-k-mooney: thanks | |
| 17:51:00 | opendevreview | Merged openstack/nova stable/victoria: Reject open redirection in the console proxy https://review.opendev.org/c/openstack/nova/+/791805 | |
| 17:51:07 | opendevreview | Merged openstack/nova master: Add tests for 'AggregateImagePropertiesIsolation' https://review.opendev.org/c/openstack/nova/+/754115 | |
| 17:51:16 | opendevreview | Merged openstack/nova master: Add tests for 'AggregateInstanceExtraSpecsFilter' https://review.opendev.org/c/openstack/nova/+/754116 | |
| 17:51:26 | opendevreview | Merged openstack/nova master: use cinder v3 https://review.opendev.org/c/openstack/nova/+/802334 | |
| 19:27:00 | opendevreview | melanie witt proposed openstack/nova stable/ussuri: Reject open redirection in the console proxy https://review.opendev.org/c/openstack/nova/+/791806 | |
| 19:45:19 | opendevreview | Merged openstack/nova master: libvirt: Always delegate OVS plug to os-vif https://review.opendev.org/c/openstack/nova/+/797428 | |
| #openstack-nova - 2021-07-27 | |||
| 02:00:27 | opendevreview | Takashi Kajinami proposed openstack/python-novaclient master: Use Block Storage API v3 instead of API v2 https://review.opendev.org/c/openstack/python-novaclient/+/802417 | |
| 08:40:23 | MrClayPole | Time appropriate greetings, we have a company policy for all network and compute devices have their timezone set to UTC time. The issue we have is that during the summer in the UK we have our clocks set to UTC+1. Our Windows servers are currently booting with clock 1 hour in the past. Is there a way for libvirt/nova to track our timezone while keeping Ubuntu 18.04 set to UTC time? | |
| 08:43:25 | lyarwood | MrClayPole: are you using os_type=windows for the images? | |
| 08:43:39 | lyarwood | https://bugs.launchpad.net/nova/+bug/1231254 a slightly old bug but I wonder if it still applies | |
| 08:45:02 | lyarwood | https://libvirt.org/formatdomain.html#time-keeping would be how libvirt controls this FWIW | |
| 08:46:00 | lyarwood | https://github.com/openstack/nova/blob/1c490ecd7be5722c6cfdaddfacc8d7a5539dd035/nova/virt/libvirt/driver.py#L5792-L5806 is where we switch to localtime when os_type=windows | |
| 08:46:05 | lyarwood | instead of utc | |
| 08:47:59 | lyarwood | ah right but your hosts remain on UTC so that isn't useful | |
| 08:56:05 | MrClayPole | lyarwood: We are running OpenStack rocky, as you said the os_type didn't help but it's interesting that libvirt xml supports a timezone. Can this be set from Nova? | |
| 08:57:56 | lyarwood | MrClayPole: not at the moment AFAICT, we have all of the config code just no way for a user to request it | |
| 08:59:05 | MrClayPole | I wondering if the best way forward might just be to have an exception for our compute nodes so they can bet set to "Europe/London" rather than UTC | |
| 08:59:37 | lyarwood | Yeah for your env on Rocky that would be for the best | |
| 09:00:08 | lyarwood | For the Yoga OpenStack release we could easily add an image property to control the timezone directly | |
| 09:02:27 | MrClayPole | That would defo be useful to us so we can revert the compute nodes back to UTC. | |
| 09:15:47 | stephenfin | bauzas: gate fix for novaclient here if you have 2 | |
| 09:15:48 | stephenfin | https://review.opendev.org/c/openstack/python-novaclient/+/802417 | |
| 09:16:02 | bauzas | stephenfin: we have a gate issue ? | |
| 09:16:12 | stephenfin | Yes, a very slight one due to cinderclient's removal of cinder API v2 | |
| 09:16:46 | stephenfin | Spotted after reading through Takashi Kajinami's email to openstack-discuss ("[all] Broken gate caused by Block Storage API v2 removal") | |
| 09:16:46 | bauzas | stephenfin: any bug y'know ? | |
| 09:16:54 | bauzas | ahah | |
| 09:17:23 | stephenfin | lots of related patches available also https://review.opendev.org/q/topic:%22volumev2-removal%22+(status:open%20OR%20status:merged) | |
| 09:18:09 | bauzas | I just read the email, I don't see failing jobs | |
| 09:18:49 | bauzas | hah | |
| 09:19:15 | lyarwood | stephenfin: ACK'd the novaclient change | |
| 09:19:24 | stephenfin | ty! | |
| 09:20:59 | bauzas | are we sure that the v3 client supports the same than for the v2 ? | |
| 09:21:09 | bauzas | lyarwood: ^ | |
| 09:21:20 | lyarwood | bauzas: yeah it does | |
| 09:22:03 | bauzas | ok, I was a bit afraid of just using the new version without making sure it wasn't creating problems for us | |
| 09:22:19 | bauzas | but if it doesn't change our client API, fair enough | |
| 09:22:26 | lyarwood | https://docs.openstack.org/api-ref/block-storage/api_microversion_history.html#maximum-in-mitaka | |
| 09:22:42 | lyarwood | The 3.0 Cinder API includes all v2 core APIs existing prior to the introduction of microversions. The /v3 URL is used to call 3.0 APIs. This is the initial version of the Cinder API which supports microversions. | |
| 09:26:41 | sean-k-mooney | it looks like in most cases the convertion to v3 is trivial as a result | |
| 09:26:57 | bauzas | lyarwood: thanks | |