| tonyb | There's a glance_store constraints bump on stable/2303.1 that's failing | 03:29 |
|---|---|---|
| tonyb | can someone take a look at: https://zuul.opendev.org/t/openstack/build/e485baf66457414fa8b79420e50441ad/log/job-output.txt#30523 | 03:29 |
| tonyb | and give me a pointer on what's going on? There are lots of libvirt and db errors | 03:30 |
| melwitt | sean-k-mooney: fyi I'm working on func tests for rescue atm ... and also handling to not go into ERROR state in the case of an encryption secret not being found in barbican | 05:44 |
| sean-k-mooney[m] | ack, i work up early and since im awake i might as well review code :) | 05:46 |
| sean-k-mooney[m] | its totally not because that will make me fall a asleep :P | 05:46 |
| sean-k-mooney[m] | i havent got that far in the seriese but ill keep that in mind | 05:46 |
| melwitt | sean-k-mooney[m]: haha :) ok, just wanted to mention that in case you did get that far and being like where's the func tests | 05:47 |
| tkajinam | I'm just curious but wonder why rng device specs can be configured only by flavor extra specs and can't by image properties. | 07:07 |
| Uggla | bauzas, gibi I have pushed the virtio serie yesterday evening. I still need to fix some comments and do extensive tests with devstack but it will be glad to have a review on the new asynchronous stuff. | 07:09 |
| opendevreview | Takashi Kajinami proposed openstack/nova-specs master: libvirt: Stateless firmware support https://review.opendev.org/c/openstack/nova-specs/+/908297 | 07:36 |
| opendevreview | Takashi Kajinami proposed openstack/os-traits master: Add a new trait for stateless firmware support https://review.opendev.org/c/openstack/os-traits/+/908885 | 07:40 |
| opendevreview | Takashi Kajinami proposed openstack/os-traits master: IUpdate spec reference links https://review.opendev.org/c/openstack/os-traits/+/908886 | 07:48 |
| opendevreview | Takashi Kajinami proposed openstack/os-traits master: Update spec reference links https://review.opendev.org/c/openstack/os-traits/+/908886 | 07:48 |
| opendevreview | Takashi Kajinami proposed openstack/os-traits master: Add a new trait for stateless firmware support https://review.opendev.org/c/openstack/os-traits/+/908885 | 07:50 |
| opendevreview | Takashi Kajinami proposed openstack/os-traits master: Add a new trait for stateless firmware support https://review.opendev.org/c/openstack/os-traits/+/908887 | 07:54 |
| opendevreview | Takashi Kajinami proposed openstack/os-traits master: Add a new trait for stateless firmware support https://review.opendev.org/c/openstack/os-traits/+/908885 | 07:55 |
| opendevreview | Takashi Kajinami proposed openstack/os-traits master: Add a new trait for stateless firmware support https://review.opendev.org/c/openstack/os-traits/+/908885 | 08:13 |
| opendevreview | Takashi Kajinami proposed openstack/nova master: Report availability of stateless firmware support https://review.opendev.org/c/openstack/nova/+/908888 | 08:14 |
| opendevreview | Takashi Kajinami proposed openstack/nova master: Report availability of stateless firmware support https://review.opendev.org/c/openstack/nova/+/908888 | 08:16 |
| sean-k-mooney[m] | melwitt: i made it though the whole seriese. overall im +2 on most of it +1 on most of the rest and -1 on one patch | 08:57 |
| sean-k-mooney[m] | in general i think we could proably merge the seriese as is safely but there is i think one bug in the resize api validation | 08:58 |
| sean-k-mooney[m] | in terms of funcitonal tests i think we can still improve them but i could also see use doing that as follow up patches even after code freeze | 08:59 |
| sean-k-mooney[m] | im going to pause my reivew of this seriese for today and ill come back to it tomrrow | 08:59 |
| melwitt | sean-k-mooney[m]: ok, great, thanks! I'll go through and fix up things based on your comments later today | 09:02 |
| opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Launch instances with stateless firmware https://review.opendev.org/c/openstack/nova/+/908890 | 09:33 |
| *** tosky_ is now known as tosky | 09:48 | |
| opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Launch instances with stateless firmware https://review.opendev.org/c/openstack/nova/+/908890 | 10:12 |
| opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Detect vtpm support by libvirt https://review.opendev.org/c/openstack/nova/+/908508 | 10:23 |
| sean-k-mooney | melwitt: turns out there is a file size limit in headgedoc | 14:32 |
| sean-k-mooney | which i now hit. | 14:32 |
| sean-k-mooney | i quickly tested snapshot and booting a new vm form that snapshot to test booting form an encypted disk image | 14:32 |
| sean-k-mooney | both worked | 14:32 |
| sean-k-mooney | im going to create a new note and continue testing but i started working at 5am my timea and its now 14:30 ish so im undesincedif im going to keep workign today and just not work friday or finish shortly | 14:34 |
| opendevreview | Merged openstack/nova master: Augment the LibvirtLiveMigrateData object https://review.opendev.org/c/openstack/nova/+/904175 | 15:14 |
| gibi | Uggla: left feedback in the manial series | 15:15 |
| Uggla | gibi thanks I'm gonna have a look. | 15:17 |
| wncslln | hello nova o/, I have a environment with kolla and I have been a problem with nova and rabbitmq. In one of the tasks, it tries to ensure that rabbitmq user exists, but crashes and we not could reconfigure nova service :( | 15:42 |
| wncslln | anyone can help? | 15:42 |
| wncslln | i mispoken. I have been facing a problem* | 15:44 |
| bauzas | #startmeeting nova | 16:00 |
| opendevmeet | Meeting started Tue Feb 13 16:00:15 2024 UTC and is due to finish in 60 minutes. The chair is bauzas. Information about MeetBot at http://wiki.debian.org/MeetBot. | 16:00 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 16:00 |
| opendevmeet | The meeting name has been set to 'nova' | 16:00 |
| bauzas | hey folks | 16:00 |
| auniyal | o/ | 16:00 |
| dansmith | o/ | 16:00 |
| bauzas | sorry, those days I'm a bit off from the channel | 16:00 |
| bauzas | #link https://wiki.openstack.org/wiki/Meetings/Nova#Agenda_for_next_meeting | 16:01 |
| elodilles | o/ | 16:01 |
| ratailor | o/ | 16:01 |
| bauzas | let's start | 16:01 |
| bauzas | people will arrive | 16:01 |
| bauzas | #topic Bugs (stuck/critical) | 16:01 |
| bauzas | #info No Critical bug | 16:01 |
| bauzas | #link https://bugs.launchpad.net/nova/+bugs?search=Search&field.status=New 59 new untriaged bugs (+3 since the last meeting) | 16:01 |
| bauzas | #info Add yourself in the team bug roster if you want to help https://etherpad.opendev.org/p/nova-bug-triage-roster | 16:01 |
| bauzas | #info bug baton is bauzas | 16:01 |
| gibi | o/ | 16:01 |
| bauzas | (I forgot again to look at the bugs :( ) | 16:01 |
| bauzas | any bugs you would want to discuss ? | 16:02 |
| bauzas | looks not, moving on | 16:02 |
| bauzas | #topic Gate status | 16:02 |
| fwiesel | \o | 16:02 |
| bauzas | #link https://bugs.launchpad.net/nova/+bugs?field.tag=gate-failure Nova gate bugs | 16:02 |
| bauzas | #link https://etherpad.opendev.org/p/nova-ci-failures-minimal | 16:02 |
| bauzas | #link https://zuul.openstack.org/builds?project=openstack%2Fnova&project=openstack%2Fplacement&pipeline=periodic-weekly Nova&Placement periodic jobs status | 16:02 |
| bauzas | #info Please look at the gate failures and file a bug report with the gate-failure tag. | 16:03 |
| bauzas | so, we have some issue with the c9s job | 16:03 |
| bauzas | but we know why :) | 16:03 |
| bauzas | (some libvirt regression) | 16:03 |
| bauzas | in the next week, the new libvirt release (that fixes the issue) would be in the c9s | 16:04 |
| bauzas | (hopefully) | 16:05 |
| bauzas | any CI failures you would want to discuss ? | 16:05 |
| bauzas | looks not | 16:07 |
| bauzas | moving on | 16:08 |
| bauzas | #topic Release Planning | 16:08 |
| bauzas | #link https://releases.openstack.org/caracal/schedule.html#nova | 16:08 |
| bauzas | #info Caracal-3 (and feature freeze) milestone in 2 weeks | 16:08 |
| bauzas | tick-tock | 16:08 |
| bauzas | 2 weeks and a half, tbh | 16:08 |
| bauzas | for me, given I was working on my own series for testing it, I'll eventually review by tomorrow | 16:09 |
| bauzas | review *other* features, I mean | 16:09 |
| bauzas | I was wanting to do this today, but I had another problem that I fixed | 16:09 |
| bauzas | so, please look at your gerrit emails tomorrow :)= | 16:10 |
| bauzas | #topic Review priorities | 16:10 |
| bauzas | #link https://etherpad.opendev.org/p/nova-caracal-status | 16:10 |
| bauzas | again, everything is there | 16:10 |
| bauzas | nothing to say more about it | 16:10 |
| bauzas | #topic Stable Branches | 16:10 |
| bauzas | elodilles: heya | 16:10 |
| elodilles | o/ | 16:11 |
| elodilles | #info stable/ussuri transitioned to End of Life | 16:11 |
| elodilles | this i forgot to mention last week ^^^ | 16:11 |
| bauzas | \o/ | 16:11 |
| elodilles | #info unmaintained/yoga is open for patches | 16:11 |
| elodilles | though gate might be still problematic, but at least generated patch has merged \o/ | 16:11 |
| elodilles | #info stable gates don't seem blocked, though deleted stable/yoga can cause problems (e.g. on grenade jobs) | 16:12 |
| elodilles | this might be important for 2023.1 (skip level grenade) and zed (grenade) for the coming weeks ^^^ | 16:12 |
| elodilles | so probably the best is to follow things here: | 16:13 |
| elodilles | #info stable branch status / gate failures tracking etherpad: https://etherpad.opendev.org/p/nova-stable-branch-ci | 16:13 |
| elodilles | and maybe one more thing to mention: | 16:13 |
| elodilles | now that we have unmaintained/yoga | 16:13 |
| elodilles | so far only openstack-unmaintained-core have rights for nova's unmaintained/ branch | 16:14 |
| elodilles | there are two option here | 16:14 |
| elodilles | 1) add people to this group | 16:14 |
| elodilles | 2) create a nova-unmaintained-core group | 16:14 |
| dansmith | the goal is that most projects go with #1 | 16:15 |
| elodilles | and populate that one with interested people ^^^ | 16:15 |
| bauzas | I have no opinion | 16:15 |
| elodilles | dansmith: yepp, true, though some project already started to create their own group | 16:15 |
| dansmith | meaning "nova doesn't really maintain the unmaintained stuff, but community members that are interested in old versions do" | 16:15 |
| dansmith | elodilles: right, but see the discussion in -tc right now.. that seems to be a misundersanding | 16:16 |
| elodilles | anyway, this is just a heads up to the team to think about this | 16:16 |
| bauzas | but in general, people wanting to "maintain" a project also wants to maintain cinder and neutron | 16:16 |
| gibi | I don't want to maintain ussuri :) | 16:16 |
| elodilles | dansmith: ACK | 16:16 |
| dansmith | we're hoping/expecting most projects to effectively let those dry up and only get maintained if there are people around to care | 16:16 |
| dansmith | bauzas: right, the idea is someone maintaining ussuri need to care about nova and neutron in that release, so let them | 16:16 |
| elodilles | ACK, then we can keep things as it is then :) | 16:16 |
| bauzas | tbh, after thinking a bit, I wouldn't want to have a specific group told 'nova-' something | 16:17 |
| dansmith | that's certainly my preference | 16:17 |
| elodilles | (note, i am member of openstack-unmaintained-core, so either way, i have rights to mess around there) | 16:17 |
| bauzas | so no about 2) if the name is 'nova-unmaintained-group' | 16:17 |
| dansmith | the idea is that the nova project gets to stop worrying about those old releases | 16:17 |
| dansmith | if we create a nova- group, we kinda have to keep caring about it | 16:17 |
| bauzas | yeah, that's why I don't want that group to be named 'nova' | 16:17 |
| bauzas | dansmith: that's my point | 16:17 |
| dansmith | yup | 16:17 |
| elodilles | ACK | 16:18 |
| elodilles | OK, that's it from my side then about stable :X | 16:18 |
| bauzas | anyway, the unmaintained branch is no longer supported by nova | 16:18 |
| bauzas | so, people can create groups like they want | 16:19 |
| bauzas | but again, I'm fine till they don't name those groups like the projects | 16:19 |
| bauzas | unmaintained-compute-specialized-group meh to it | 16:20 |
| elodilles | no need for a group at all, there is openstack-unmaintained-core group already | 16:20 |
| bauzas | then I'm cool | 16:20 |
| dansmith | and that's the preference :) | 16:21 |
| elodilles | +1 | 16:21 |
| dansmith | the per-project group override is for people who don't want the new plan, basically | 16:21 |
| bauzas | anyway, unmaintained projects are actually now forks | 16:21 |
| dansmith | I think this horse is dead | 16:21 |
| bauzas | unmaintained branches | 16:21 |
| bauzas | * | 16:21 |
| bauzas | people can fork as much as they want provided they don't name those branches "nova-something" | 16:22 |
| bauzas | anyway, I'm done | 16:23 |
| bauzas | moving on ? | 16:23 |
| elodilles | +1 | 16:23 |
| bauzas | #topic vmwareapi 3rd-party CI efforts Highlights | 16:23 |
| fwiesel | #info Fixes to CI for various branches. Missing is still versions prior zed (requiring Ubuntu 20.04) | 16:23 |
| bauzas | fwiesel: heya | 16:23 |
| bauzas | yoga is now unmaintained :D | 16:23 |
| fwiesel | So, I only tested before on master, which didn't translate well to other branches. | 16:23 |
| bauzas | so IMHO you shouldn't really care on maintaining 3rd party jobs for Yoga and older branches | 16:24 |
| fwiesel | Problem is xena. It is also Ubuntu 20.04. | 16:24 |
| fwiesel | Ah, x<z ... | 16:24 |
| fwiesel | Great, that was my implicit question. So we are fine just with zed and later? | 16:25 |
| elodilles | (and Zed will move to Unmaintained in ~3 months) | 16:25 |
| bauzas | fwiesel: that actually depends on what you want to test for yourselves | 16:26 |
| bauzas | but if you don't really want to test Xena for your own sake, fwiw, the nova project is fine with not testing vmwareapi on that branch | 16:26 |
| fwiesel | Well, we want to move away from Xena ourselves. My main reason for testing older versions was trying to bisect the bug I was mentioning. | 16:26 |
| fwiesel | #info Debugging local root disk (Raised bug: https://bugs.launchpad.net/nova/+bug/2053027) | 16:26 |
| fwiesel | That took most of my time. It works for us on our heavily patched xena, I hoped to establish a base-line. | 16:27 |
| fwiesel | At least from what I can gather, it is a rather strange one, and probably is too much for the summary in this meeting. | 16:28 |
| fwiesel | I am almost of a mind to rip out the code for image upload for the one used in cinder (incidentally the one in oslo.vmware). Any strong feelings on this one? | 16:28 |
| bauzas | fwiesel: from your report, it sounds to me this is a glance issue | 16:28 |
| fwiesel | Well, cinder works. | 16:28 |
| fwiesel | With the same glance :) | 16:29 |
| bauzas | so that's a client issue | 16:29 |
| fwiesel | It all runs in the same venv, so the code for all the libraries are the same. | 16:29 |
| fwiesel | The debug output for the request for nova and cinder against glance is almost the same. Cinder also passes on the X-Service-Token to glance, while nova doesn't. | 16:31 |
| dansmith | not sure how it's a glance thing | 16:31 |
| fwiesel | Not that I believe that to be the error, just for completness. | 16:31 |
| bauzas | dansmith: right my bad, it's a client thing | 16:32 |
| bauzas | but the OSError sounds a permission issue | 16:32 |
| fwiesel | Either way, probably not something that can be solved in a couple of minutes in this meeting. I presume | 16:32 |
| fwiesel | You get an "OSError" because the client (i.e. nova) closes the connections, so glance cannot write to the socket anymore. | 16:32 |
| dansmith | fwiesel: not sure that makes sense either :) | 16:33 |
| fwiesel | Yeah, either way. I would suggest to leave the discussion on the details of the bug for after the meeting. | 16:34 |
| dansmith | I'll have to look more at the logs after the meeting, there's not really enough meat in the bug to really see I think | 16:34 |
| bauzas | I don't really know the workflow that's implied by this oslo.vmware call | 16:34 |
| bauzas | oh found it | 16:35 |
| bauzas | http://openstack-ci-logs.global.cloud.sap/openstack/nova/35af4b345d997b63f999a090e236d91b78ea4304/n-cpu-1.service.log | 16:35 |
| bauzas | that's when getting the image from glance | 16:35 |
| sean-k-mooney | right so that should be via http via the glance api | 16:36 |
| sean-k-mooney | even if its on NFS as a stoage backend | 16:36 |
| bauzas | I don't actually indeed see why we need to call oslo.vmware | 16:37 |
| sean-k-mooney | well for the volume creation we would just ask cindeer to create the voluem from the image right | 16:37 |
| fwiesel | That happens for the boot from volume case. And that works. But we also have the boot from "local" disk, and then nova needs to pull the image. | 16:38 |
| sean-k-mooney | we would only need to call into oslo.vmware when asking vmware to create the instance using the cidner volume | 16:38 |
| bauzas | yeah their problem is with the standard boot from image to disk | 16:38 |
| sean-k-mooney | ok but we dont have an agent runing on the esxi host | 16:38 |
| sean-k-mooney | or vshper node | 16:38 |
| sean-k-mooney | we are asking vsphare to download it form glance right | 16:39 |
| bauzas | apparently they do some caching on the downloaded image | 16:39 |
| bauzas | so this is indeed not a glance communication problem | 16:39 |
| dansmith | can we discuss after the meeting? | 16:39 |
| bauzas | from what I can understand from the stacktrace, the image is downloaded but then you call oslo.vmware to cache that image | 16:39 |
| bauzas | dansmith: good point | 16:39 |
| bauzas | the next topic should be empty | 16:40 |
| bauzas | fwiesel: are you done with your points ? we'll continue troubleshooting right after the meeting | 16:40 |
| bauzas | apparently so | 16:40 |
| fwiesel | I am done. Over to you | 16:40 |
| bauzas | #topic Open discussion | 16:40 |
| bauzas | . | 16:40 |
| bauzas | nothing in the agenda | 16:41 |
| bauzas | anything anyone ? | 16:41 |
| sean-k-mooney | did you want to chat about the sate of the vgpu seriese | 16:41 |
| bauzas | sean-k-mooney: well, I eventually was able to live-migrate | 16:41 |
| sean-k-mooney | or wait till next week when you have done some more testing | 16:41 |
| bauzas | I know now the reason why we need some very large downtime option | 16:41 |
| sean-k-mooney | do wehttps://docs.openstack.org/nova/latest/configuration/config.html#libvirt.live_migration_downtime | 16:42 |
| sean-k-mooney | is that enough to configure it | 16:42 |
| bauzas | but again, I think I'm done, I'm currently working on providing a asciinema stuff | 16:42 |
| bauzas | so people will see it | 16:42 |
| bauzas | sean-k-mooney: correct, that and the two other options | 16:43 |
| sean-k-mooney | ack | 16:43 |
| sean-k-mooney | i just wanted to confirm if the existign config options and the code you have for review is enough | 16:43 |
| sean-k-mooney | assuming yes we can proceed on gerrit | 16:44 |
| bauzas | yeah, so I'll modify https://review.opendev.org/c/openstack/nova/+/904258/13/doc/source/admin/virtual-gpu.rst | 16:44 |
| bauzas | to explain that people will need to use a large max downtimez | 16:44 |
| sean-k-mooney | ack | 16:44 |
| bauzas | anyway, I'm done now | 16:45 |
| bauzas | any other things ? | 16:45 |
| sean-k-mooney | nope just wanted to confim that before i review the rest of your seies | 16:45 |
| bauzas | sean-k-mooney: no worries | 16:45 |
| bauzas | so, thanks all | 16:45 |
| fwiesel | Thanks a lot. | 16:46 |
| bauzas | #endmeeting | 16:46 |
| opendevmeet | Meeting ended Tue Feb 13 16:46:08 2024 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:46 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/nova/2024/nova.2024-02-13-16.00.html | 16:46 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/nova/2024/nova.2024-02-13-16.00.txt | 16:46 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/nova/2024/nova.2024-02-13-16.00.log.html | 16:46 |
| sean-k-mooney | fwiesel: so it looks like the vmware direver has some image caching code in spawn https://github.com/openstack/nova/blob/master/nova/virt/vmwareapi/vmops.py#L738 is it the _fetch_image_if_missing function that is failing in hte local storage case? | 16:51 |
| sean-k-mooney | fwiesel: there are some other error stoo but i noticed "RROR nova.db.main.api [None req-4114694a-df48-45f8-b7ff-32706629cf90 None None] No DB access allowed in nova-compute: File "/opt/stack/data/venv/lib/python3.10/site-packages/eventlet/greenthread.py", line 264," | 16:54 |
| sean-k-mooney | that sames to go away however | 16:54 |
| sean-k-mooney | i added the backtrace to the bug https://bugs.launchpad.net/nova/+bug/2053027/comments/2 | 16:54 |
| fwiesel | sean-k-mooney: Yes that is the failing one. The invalid argument is a bit misleading, and comes from a zero byte file size. | 16:59 |
| opendevreview | Merged openstack/nova stable/2023.1: testing: Reset affinity support global variables https://review.opendev.org/c/openstack/nova/+/908839 | 17:00 |
| sean-k-mooney | fwiesel: how is your glanced configured is it using cinder for storage | 17:01 |
| gmann | dansmith: can you re-check this https://review.opendev.org/c/openstack/nova/+/906629 | 17:03 |
| gmann | dansmith: sean-k-mooney and rest of the patch in remove hyperv series | 17:03 |
| sean-k-mooney | fwiesel: glance_store.default_store = file so ya thats odd it should be pulling the image form the rest api then | 17:04 |
| dansmith | gmann: sorry I didn't realize I hadn't circled back on that one after you fixed the qemu thing | 17:04 |
| sean-k-mooney | gmann: sure | 17:04 |
| gmann | thanks | 17:05 |
| fwiesel | sean-k-mooney: Yes, it is pulling it via the rest api from a local file. And the fun part is, cinder does the same and it works. | 17:05 |
| dansmith | sean-k-mooney: maybe read my comment here: https://review.opendev.org/c/openstack/nova/+/906636/3/nova/tests/unit/objects/test_objects.py#b1105 | 17:05 |
| sean-k-mooney | dansmith: ya i saw that. i was personally fine with leave everythign after the virt driver remove for a later release | 17:06 |
| sean-k-mooney | dansmith: so with https://review.opendev.org/c/openstack/nova/+/894466/3 merged we have removed the mvp | 17:07 |
| dansmith | yeah | 17:07 |
| sean-k-mooney | so if you want to keep HyperVLiveMigrateData that ok | 17:07 |
| dansmith | although I'd say the doc patch should be considered mandatory | 17:07 |
| sean-k-mooney | we can do the RDP console changes if we want even with out the object change | 17:08 |
| sean-k-mooney | oh yes but that has +2w no? | 17:08 |
| sean-k-mooney | ya it has so that just need to make it way through ci | 17:08 |
| dansmith | sean-k-mooney: it does now, I'm just saying if you're talking about mvp, we should include not having docs about drivers that aren't in tree :) | 17:08 |
| sean-k-mooney | :) yes your right | 17:09 |
| gmann | but if we keep field and remove HyperVLiveMigrateData maybe that is fine? | 17:11 |
| gmann | dansmith: forgot to reply there but I checked and that is what we did for xenapi driver removal | 17:11 |
| sean-k-mooney | gmann: having the opbject has little to no maintance overhead | 17:12 |
| dansmith | gmann: the xenbus object is still there.. what object are you talking about? | 17:13 |
| dansmith | maybe the xen live migratedata? | 17:13 |
| gmann | yeah live migration one was removed | 17:14 |
| gmann | https://review.opendev.org/c/openstack/nova/+/749307 | 17:14 |
| gmann | this is what I checked | 17:14 |
| dansmith | ack, I didn't review that :P | 17:15 |
| gmann | k, we can keep that with a note to remove in later cycle after 1 or 2. | 17:16 |
| dansmith | we can remove the xenbus module when we do :) | 17:16 |
| opendevreview | Merged openstack/nova master: Remove HyperV: cleanup doc/code ref https://review.opendev.org/c/openstack/nova/+/906629 | 17:38 |
| sean-k-mooney | dansmith: by the way if you have time to start revieing melwitt serise i think most of the patches are in good shape | 17:58 |
| sean-k-mooney | https://review.opendev.org/q/topic:%22specs/yoga/approved/ephemeral-encryption-libvirt%22+status:open+project:openstack/nova | 17:58 |
| dansmith | yeah it's kinda half on my list, but I'm not sure I'll really be able to dive into it in detail | 17:58 |
| melwitt | sean-k-mooney: ack, thanks for testing that | 18:13 |
| bauzas | sean-k-mooney, dansmith : ok, I'm eventually done with testing my series, I'll provide the asciinema gif tomorrow | 18:44 |
| dansmith | bauzas: updates needed to the set or good as is? | 18:44 |
| sean-k-mooney | i started looking at the serise a littel to day but likely wont have time to review it till tomorrow | 18:52 |
| sean-k-mooney | going though the vgpu serise is on my todo list for tomorrow before i loop back to melainies series | 18:52 |
| bauzas | dansmith: nope, no need to change, we just need to set a large number for downtime but you will see it tomorrow ;-) | 19:05 |
| dansmith | cool | 19:06 |
| bauzas | the reason that qemu doesn't know about the right GPU memory, it just thinks that it needs to migrate 4GB | 19:06 |
| dansmith | ack | 19:07 |
| opendevreview | Ghanshyam proposed openstack/nova master: HyperV: Add todo to remove HyperVLiveMigrateData object https://review.opendev.org/c/openstack/nova/+/906636 | 19:27 |
| opendevreview | Ghanshyam proposed openstack/nova master: HyperV: Remove RDP console connection information API https://review.opendev.org/c/openstack/nova/+/906991 | 19:28 |
| opendevreview | Ghanshyam proposed openstack/nova master: HyperV: Remove RDP console API https://review.opendev.org/c/openstack/nova/+/906809 | 19:28 |
| opendevreview | Ghanshyam proposed openstack/nova master: HyperV: Remove extra specs of HyperV driver https://review.opendev.org/c/openstack/nova/+/906992 | 19:28 |
| gmann | dansmith: sean-k-mooney ^^ keeping hyperv object and rebased API change, please check | 19:28 |
| dansmith | gmann: already done | 19:29 |
| gmann | thanks, so fast :) | 19:29 |
| sean-k-mooney | im just leaveing but ill do that then drop for today | 19:29 |
| gmann | sean-k-mooney: thanks, and sorry for making it late for you | 19:30 |
| sean-k-mooney | its normally not laste but i got up very early today | 19:30 |
| sean-k-mooney | so no worries | 19:30 |
| sean-k-mooney | gmann: ok im +2 on all of those they look ok assuming they also pass ci | 19:37 |
| opendevreview | Merged openstack/nova stable/2023.1: tests: Use GreenThreadPoolExecutor.shutdown(wait=True) https://review.opendev.org/c/openstack/nova/+/908806 | 19:47 |
| spatel | I am getting strange nova-novncproxy error and not sure what is wrong. Everything was working but after restart nova-novncproxy it started throwing error - https://paste.opendev.org/show/baZjIpTcibpksgajegav/ | 20:13 |
| opendevreview | Ghanshyam proposed openstack/nova master: HyperV: Remove RDP console API https://review.opendev.org/c/openstack/nova/+/906809 | 20:25 |
| opendevreview | Ghanshyam proposed openstack/nova master: HyperV: Remove extra specs of HyperV driver https://review.opendev.org/c/openstack/nova/+/906992 | 20:25 |
| gmann | dansmith: ^^ updated it to keep RPC method | 23:04 |
| dansmith | gmann: ack thanks | 23:33 |
| *** dmitriis is now known as Guest2683 | 23:44 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!