| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-07-26 | |||
| 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 | |
| 09:29:04 | sean-k-mooney | MrClayPole: im not sure th at nova should really provide a facility to set a timezone | |
| 09:29:35 | sean-k-mooney | at least not at the host level provbly not ant the image or flaovr level. although im more open to the image idea | |
| 09:30:20 | sean-k-mooney | if we were to suprot settign a time zone i think i would want it to be per instance hosetly but really i think that woule be better handeles withine the workload | |
| 09:30:38 | sean-k-mooney | just always set your host to utc | |
| 09:31:09 | sean-k-mooney | and in windows set the timezone appropreiately if needed | |
| 09:32:06 | CeeMac | sean-k-mooney: thats exactly the issues we have at the moment though. The host is set to utc and windows seems locked to that, so during DST the guest still runs -1 hour instead of updating. | |
| 09:32:54 | CeeMac | i must confess I'm a little perplexed as I was expecting the os_type=windows to resolve the issue, but this doesn't appear to be the case | |
| 09:35:21 | sean-k-mooney | CeeMac: you should be able to set the time in windows the clock source will by utc | |
| 09:35:35 | sean-k-mooney | but in windows itself you can change the local | |
| 09:36:09 | CeeMac | sean-k-mooney: yes, "should" appears to be the main issues here as that was my expectation, but the results don't appear to support this :( | |
| 09:36:16 | sean-k-mooney | are you saying if you go into the region settign in windows it does not work? | |
| 09:36:43 | sean-k-mooney | CeeMac: that sound like a windows bug you should report to microsoft | |
| 09:36:54 | CeeMac | our guest VMs are all set to BST (UTC+1) and the system clock is reporting UTC+0 times, so they're all an hour in the past | |
| 09:37:07 | sean-k-mooney | in a cloud enviornemt you cannot know the timezone of the tenant | |
| 09:37:17 | sean-k-mooney | so you shoudl always be abel to set it indepenently of the host | |
| 09:37:52 | CeeMac | it appears to be a known issue around how windows handles time from the bios clock, which i believe is the entire premise on why the os_type=windows patch was ported from hyper-v hypervisor to kvm hypervisor in nova | |
| 09:37:59 | sean-k-mooney | CeeMac: for what its worth i hate daylight saving time so im glad ill by on utc permently next year | |
| 09:38:06 | CeeMac | sean-k-mooney: yeah, thats the dream! | |
| 09:38:28 | CeeMac | yeah, can't say I'm a big fan either. its those pesky customers who are complaining :) | |