| *** hamalq has quit IRC | 00:02 | |
| *** ikla has joined #openstack-nova | 00:07 | |
| ikla | can I use ceph for my instance and local disk for ephemeral storage? | 00:08 |
|---|---|---|
| ikla | appears to be either or | 00:09 |
| ikla | rbd or local | 00:09 |
| ikla | or can I pass a local partition in? | 00:10 |
| *** mlavalle has quit IRC | 00:10 | |
| *** martinkennelly has quit IRC | 00:13 | |
| *** jamesden_ has quit IRC | 00:17 | |
| *** jamesdenton has joined #openstack-nova | 00:18 | |
| *** ikla has quit IRC | 00:36 | |
| *** macz_ has joined #openstack-nova | 00:54 | |
| *** ikla has joined #openstack-nova | 00:55 | |
| *** macz_ has quit IRC | 00:58 | |
| *** rcernin_ has joined #openstack-nova | 00:59 | |
| *** rcernin has quit IRC | 00:59 | |
| *** xinranwang has joined #openstack-nova | 01:49 | |
| *** swp20 has joined #openstack-nova | 01:58 | |
| *** brinzhang_ is now known as brinzhang | 02:04 | |
| *** jangutter_ has joined #openstack-nova | 02:14 | |
| *** jangutter has quit IRC | 02:17 | |
| *** rcernin_ has quit IRC | 02:20 | |
| *** rcernin_ has joined #openstack-nova | 02:25 | |
| *** rcernin_ has quit IRC | 02:39 | |
| *** rcernin_ has joined #openstack-nova | 02:40 | |
| *** rcernin_ has quit IRC | 03:01 | |
| *** rcernin_ has joined #openstack-nova | 03:07 | |
| *** jamesdenton has quit IRC | 03:11 | |
| *** jamesdenton has joined #openstack-nova | 03:12 | |
| *** mkrai has joined #openstack-nova | 03:30 | |
| *** tkajinam has quit IRC | 03:51 | |
| *** tkajinam has joined #openstack-nova | 03:51 | |
| *** tkajinam has quit IRC | 03:52 | |
| *** tkajinam has joined #openstack-nova | 03:53 | |
| *** mkrai has quit IRC | 04:03 | |
| *** macz_ has joined #openstack-nova | 04:12 | |
| *** macz_ has quit IRC | 04:16 | |
| *** whoami-rajat has joined #openstack-nova | 04:17 | |
| *** ratailor has joined #openstack-nova | 04:23 | |
| *** xek has quit IRC | 04:33 | |
| *** xinranwang has quit IRC | 04:38 | |
| *** ratailor_ has joined #openstack-nova | 04:44 | |
| *** ratailor has quit IRC | 04:47 | |
| *** mkrai has joined #openstack-nova | 05:01 | |
| *** ratailor__ has joined #openstack-nova | 05:10 | |
| *** ratailor_ has quit IRC | 05:13 | |
| *** Luzi has joined #openstack-nova | 05:31 | |
| *** links has joined #openstack-nova | 05:47 | |
| *** macz_ has joined #openstack-nova | 05:56 | |
| *** macz_ has quit IRC | 06:01 | |
| *** tkajinam has quit IRC | 06:02 | |
| *** tkajinam has joined #openstack-nova | 06:03 | |
| *** tkajinam has quit IRC | 06:03 | |
| *** tkajinam has joined #openstack-nova | 06:03 | |
| *** jangutter has joined #openstack-nova | 06:05 | |
| *** jangutter_ has quit IRC | 06:08 | |
| *** lpetrut has joined #openstack-nova | 06:30 | |
| *** jamesdenton has quit IRC | 06:42 | |
| *** jamesden_ has joined #openstack-nova | 06:42 | |
| *** happyhemant has joined #openstack-nova | 06:48 | |
| *** viks____ has joined #openstack-nova | 06:55 | |
| gibi_away | sean-k-mooney, artom: about requester_id in InstancePciRequest, this is the past info I have about deciding not to put the flavorid into the requester_id https://review.opendev.org/c/openstack/nova/+/625310/25/nova/pci/request.py#220 | 06:58 |
| *** gibi_away is now known as gibi | 06:58 | |
| *** luksky has joined #openstack-nova | 07:01 | |
| *** rcernin_ has quit IRC | 07:03 | |
| *** dklyle has quit IRC | 07:05 | |
| *** lemko has joined #openstack-nova | 07:08 | |
| *** sorin-mihai has quit IRC | 07:22 | |
| *** sorin-mihai has joined #openstack-nova | 07:22 | |
| *** sapd1 has joined #openstack-nova | 07:23 | |
| *** CeeMac has quit IRC | 07:24 | |
| gibi | bauzas: hi! I've propose and RC2 patch and but a -W on it until next week https://review.opendev.org/c/openstack/releases/+/784201 | 07:27 |
| *** ratailor has joined #openstack-nova | 07:35 | |
| *** ratailor__ has quit IRC | 07:36 | |
| *** xek has joined #openstack-nova | 07:42 | |
| *** jamesden_ has quit IRC | 07:45 | |
| *** jamesdenton has joined #openstack-nova | 07:45 | |
| *** tosky has joined #openstack-nova | 07:45 | |
| *** ociuhandu has joined #openstack-nova | 07:56 | |
| *** MrClayPole has quit IRC | 08:01 | |
| *** lucasagomes has joined #openstack-nova | 08:02 | |
| *** MrClayPole has joined #openstack-nova | 08:03 | |
| *** mkrai has quit IRC | 08:11 | |
| *** ociuhandu has quit IRC | 08:14 | |
| *** ociuhandu has joined #openstack-nova | 08:15 | |
| *** ociuhandu has quit IRC | 08:21 | |
| *** derekh has joined #openstack-nova | 08:23 | |
| gibi | lyarwood: I put a heavy -1 to https://review.opendev.org/c/openstack/nova/+/784130 | 08:28 |
| sean-k-mooney | gibi: we could encode if its the flavorid/port id in the string | 08:28 |
| sean-k-mooney | i do not belive its defined as a uuid | 08:28 |
| sean-k-mooney | so we could do flavor/<uuid> and port/<uuid> if we needed too | 08:29 |
| *** martinkennelly has joined #openstack-nova | 08:29 | |
| sean-k-mooney | normally i would say just add a requester_type | 08:29 |
| *** ociuhandu has joined #openstack-nova | 08:29 | |
| sean-k-mooney | but that would be a db change | 08:29 |
| gibi | sean-k-mooney: it wasn't filled not becasue we don't know what to put there, we did not put flavor there as nobody uses that information | 08:29 |
| sean-k-mooney | ah yes | 08:30 |
| sean-k-mooney | that makes sense i guess | 08:30 |
| gibi | sean-k-mooney: and there later some source information is also added to the ovo in I292a0e2d840bbf657ba6d0932f9a3decbcb2778f | 08:30 |
| gibi | https://review.opendev.org/c/openstack/nova/+/659101/4/nova/objects/instance_pci_requests.py | 08:30 |
| sean-k-mooney | yes i remember that | 08:31 |
| gibi | I have no issue if somebody starts adding flavorid tehre | 08:31 |
| gibi | there | 08:31 |
| sean-k-mooney | ya i think we can use store the uuid directly | 08:31 |
| gibi | yepp | 08:31 |
| sean-k-mooney | and use that source property to figure it out if we need too | 08:31 |
| gibi | and we can add a fallback to the ovo that if the requester_id is None then it looks up the flavor stored in the instance | 08:32 |
| sean-k-mooney | have you seen artoms change by the way https://review.opendev.org/c/openstack/nova/+/784168 | 08:33 |
| sean-k-mooney | i still need to properly review it but at first glance this seams like a workable solution going forward | 08:33 |
| sean-k-mooney | im still debating if we want to add somethign to populate the requeter id for exsiting recoreds | 08:34 |
| *** ociuhandu has quit IRC | 08:34 | |
| sean-k-mooney | possibly as a nova manage command or something on init_host as a seperate patch | 08:34 |
| *** sapd1 has quit IRC | 08:35 | |
| sean-k-mooney | what i was thinkng was loop over all the claimed devices without requester_id set, then lookup the port info form the network info cache and try and use the pci slot to corralte the pci devices to the ports | 08:36 |
| gibi | sean-k-mooney: haven't looked that review yet | 08:39 |
| sean-k-mooney | its still WIP so not quite ready for review yet but the core chagne is there and the previous patch add a functional repoducer which passes after the change | 08:42 |
| *** ociuhandu has joined #openstack-nova | 08:44 | |
| *** dtantsur|afk is now known as dtantsur | 08:44 | |
| lyarwood | gibi: replied, not sure I agree tbh, seems like a trivial requirement to codify to avoid operators borking things | 08:45 |
| lyarwood | gibi: we had an example downstream of evacuations from 2019 being cleaned up recently in 2021 that spawned this btw | 08:45 |
| lyarwood | gibi: somehow in the iterim the instance(s) had been migrated back to the original host | 08:46 |
| lyarwood | gibi: I assume with manual operator intervention to cleanup instance dirs etc | 08:46 |
| lyarwood | gibi: thankfully no data was lost but I'm just trying to avoid a future situation where we don't get as lucky | 08:46 |
| lyarwood | gibi: and again it's specifically for the force down, evacuate and force up usecase. Outside of that I agree that a periodic or something would be useful | 08:47 |
| gibi | lyarwood: this is a slippery slope as then when the next bug cames that after force_down + evac the VM volume became corrupted due to two VMs writing it then somebody can say that we should not evacuate a VM if we cannot pervent VM duplication | 08:47 |
| gibi | as soon as the admin used force_down we should stop caring | 08:47 |
| lyarwood | gibi: this isn't trying to resolve any issues during the initial force down | 08:48 |
| lyarwood | gibi: *only* when the operator forces the service back up | 08:48 |
| lyarwood | gibi: and we are given the chance to check the state of the migrations | 08:48 |
| sean-k-mooney | so the alternitive here would be to make the action for what happens when we discover vms in this state configureable | 08:48 |
| lyarwood | gibi: anything that happens before the force up is out of scope for this | 08:48 |
| sean-k-mooney | like the running_delete_instance_action | 08:48 |
| sean-k-mooney | our current behavior is hardcoded reap the instance basically | 08:49 |
| lyarwood | yeah as I said yesterday sean-k-mooney you would force up the compute service first before moving them | 08:49 |
| lyarwood | moving them back* | 08:49 |
| lyarwood | so checking during force up makes more sense tbh | 08:49 |
| *** ociuhandu has quit IRC | 08:49 | |
| *** ociuhandu has joined #openstack-nova | 08:50 | |
| gibi | lyarwood: so the goal is to check if the system is in consistent state before we allow forecing the compute up | 08:50 |
| *** ociuhandu has quit IRC | 08:50 | |
| sean-k-mooney | ya im not really against that check but i do kind fo agree with gibi that once they do force down then they are assuming resposiblity to some degree | 08:50 |
| sean-k-mooney | gibi: yes | 08:50 |
| lyarwood | gibi: correct, evacuation migration records in done with a source host pointing at the localhost meaning a service restart is required | 08:51 |
| *** ociuhandu has joined #openstack-nova | 08:51 | |
| lyarwood | sean-k-mooney: yup agree but docs have been proven time and time again not to be enough to avoid PEBKAC | 08:51 |
| sean-k-mooney | lyarwood: would an alternitive be to do an rpc on force up? | 08:51 |
| sean-k-mooney | basically to trigger what woudl be done on service restart | 08:52 |
| lyarwood | sean-k-mooney: yeah we could do that moving forward and backport this check? | 08:52 |
| sean-k-mooney | well i dont know if that was a good idea or not | 08:52 |
| lyarwood | sean-k-mooney: and/or have a periodic that triggers this | 08:52 |
| sean-k-mooney | just tinking things through | 08:53 |
| gibi | sean-k-mooney: while you could trigger the evac cleanup during force up we could think through what else needs to be done. Like clean up local deleted instances. | 08:53 |
| lyarwood | we have a periodic for that right | 08:53 |
| sean-k-mooney | local deleted yes | 08:53 |
| sean-k-mooney | i tought we did for evacuated instance but stephenfin did not find one when he quickly search on the triage call | 08:54 |
| sean-k-mooney | it might exist already but proably not | 08:54 |
| gibi | lyarwood: yepp we have _cleanup_running_deleted_instances | 08:54 |
| gibi | sean-k-mooney: there is no periodic for evac cleanup I've checked | 08:54 |
| gibi | we have _error_out_instances_whose_build_was_interrupted during init_host as well | 08:56 |
| gibi | that also someting that is only done in init_host | 08:56 |
| gibi | as might need to be done after forece up | 08:56 |
| sean-k-mooney | on a related note there was also https://review.opendev.org/c/openstack/nova/+/627765 before | 08:57 |
| *** jangutter_ has joined #openstack-nova | 08:57 | |
| sean-k-mooney | in this case looking at orpahned instance which are deleted instance that have been purged from the database before that periodic runs | 08:58 |
| gibi | anyhow we are divergin. Lee only wants a backportable thing now | 08:58 |
| lyarwood | Lee always wants a backportable thing ;) | 08:58 |
| sean-k-mooney | yes to block more latent vms in thsi state | 08:59 |
| lyarwood | but yeah I think this API change is backportable, a periodic could also be but adding a RPC call from the API during force up wouldn't be | 08:59 |
| sean-k-mooney | lyarwood: i assume we woudl return a 409 | 08:59 |
| lyarwood | yup | 08:59 |
| lyarwood | that's how I've written it at the moment | 09:00 |
| sean-k-mooney | proably with that is its a new return code | 09:00 |
| gibi | I still don't like trying to build in safety into a workflow that we marked unsafe at the beginning. But if you frame the whole thing in a way that before nova assumes responsibility again about this compute node after a force_down we do some consistency check, then I can be convinced to remove my -1 | 09:00 |
| sean-k-mooney | which is normally a microverison bump | 09:00 |
| *** jangutter has quit IRC | 09:00 | |
| lyarwood | yeah but AFAIK we don't require microversion bumps for new error codes | 09:00 |
| sean-k-mooney | i think we do | 09:00 |
| lyarwood | I've asked gmann about this before | 09:00 |
| sean-k-mooney | it think that only if we are converting from a 500 | 09:00 |
| sean-k-mooney | to the flow chart | 09:01 |
| *** ociuhandu has quit IRC | 09:01 | |
| lyarwood | sean-k-mooney: I was just looking for that, can you share the link? | 09:02 |
| sean-k-mooney | https://docs.openstack.org/nova/latest/contributor/microversions.html | 09:02 |
| lyarwood | ta | 09:02 |
| sean-k-mooney | so yes we need a microversion if you use 409 | 09:02 |
| sean-k-mooney | so 400? | 09:02 |
| gibi | I think we need to keep it 400 for a backportable fix | 09:02 |
| gibi | current clients are not prepared for 409 | 09:02 |
| gibi | so it is breaking them | 09:02 |
| sean-k-mooney | yep | 09:03 |
| lyarwood | kk understood, with a TODO to move to 409 with a new microversion in Xena? | 09:04 |
| sean-k-mooney | am sure we could | 09:05 |
| gibi | hm | 09:05 |
| sean-k-mooney | as long as we have a good error message | 09:05 |
| gibi | we have have this as well | 09:05 |
| sean-k-mooney | i dont know if its needed | 09:05 |
| gibi | An obvious regression bug in an admin-only API where the bug can still be fixed upstream on active stable branches. Admin-only APIs are less of a concern for interoperability and generally a regression in behavior can be dealt with as a bug fix when the documentation clearly shows the API behavior was unexpectedly regressed. See 3 for an example. Intentional behavior changes to an admin-only | 09:05 |
| gibi | API do require a microversion, like the 2.53 microversion for example. | 09:05 |
| gibi | nvm, this bug is not a regression | 09:05 |
| gibi | it is a new behavior | 09:06 |
| sean-k-mooney | correct and the other exemption https://docs.openstack.org/nova/latest/contributor/microversions.html#id3 | 09:06 |
| sean-k-mooney | also does not apply to 409 | 09:06 |
| sean-k-mooney | also this is not a 500 | 09:06 |
| gibi | lyarwood: yeah, let's keep a todo for Xena | 09:07 |
| *** mkrai has joined #openstack-nova | 09:07 | |
| sean-k-mooney | lyarwood: you coudl just write both patches now. technially master is now xena | 09:07 |
| sean-k-mooney | although we might want to hold off api microverion bumps until the release is actully done | 09:08 |
| lyarwood | yup I'll get it posted later today before the break (./me is off until Tuesday after today). | 09:08 |
| gibi | yepp, master is not fully open to Xena yet | 09:08 |
| lyarwood | thanks both :) | 09:08 |
| sean-k-mooney | ya ill be on pto till tuesday too | 09:09 |
| gibi | I guess it is true for most of us | 09:11 |
| sean-k-mooney | lyarwood: by the way partly while im off and partly early next week i plann to dismantel my home cloud and reinstall it and do some hardware tweeks | 09:12 |
| sean-k-mooney | do you need any data form your vms | 09:12 |
| lyarwood | sean-k-mooney: ack, nope I don't nuke away | 09:12 |
| sean-k-mooney | cool ill be backing up a few thign but one of the change ill be doing is swaping my current cinder lvm sotrage for ceph so ill be easing most of the stroage | 09:13 |
| sean-k-mooney | also doing an os reinstall moving form cenots 8 to stream wroked for a while but now im getting some repo conflictis so doing a reinstall to fix that | 09:14 |
| lyarwood | huh I had assumed this was all running on Ubuntu tbh | 09:14 |
| lyarwood | but cool | 09:14 |
| *** ociuhandu has joined #openstack-nova | 09:14 | |
| sean-k-mooney | hehe well it will be soon | 09:14 |
| sean-k-mooney | stephenfin: convicned me to try centos 8 for it then it lifecycle changed | 09:15 |
| stephenfin | don't dare try to shift the blame to me - this is all on you :P | 09:15 |
| sean-k-mooney | it is | 09:15 |
| stephenfin | your fault for listening to me ;) | 09:16 |
| sean-k-mooney | i also listend to you about tryign fedora on my laptop | 09:16 |
| sean-k-mooney | i should have know better | 09:16 |
| stephenfin | ha! | 09:16 |
| stephenfin | fair | 09:16 |
| sean-k-mooney | but ya am the version of container d that is ship in stream is not happy with docker | 09:16 |
| kashyap | It's not all that bad ;-) | 09:16 |
| * stephenfin has been bitten by Fedora bugs twice in the past month, but is sticking with it | 09:16 | |
| sean-k-mooney | so i had to pin the packages | 09:16 |
| stephenfin | mainly for multiple Python versions without external repos | 09:17 |
| kashyap | Also, a gentle reminder: many virt and kernel bugs get first fixed in Fedora | 09:17 |
| sean-k-mooney | i have hit a few other things like that that im hoping to avoid | 09:17 |
| sean-k-mooney | kashyap: yep but i very realy hit those | 09:17 |
| stephenfin | kashyap: You'd swear you worked for Red Hat or something 0:) | 09:17 |
| kashyap | stephenfin: LOL, it's not about Red Hat, really :) | 09:17 |
| sean-k-mooney | kashyap: what i have hit is selinux being unhappy with me on fedora alot | 09:17 |
| kashyap | stephenfin: I'm speaking with my upstream hat, really :) | 09:17 |
| lyarwood | sean-k-mooney: there's a command and a shiny website for that problem | 09:18 |
| lyarwood | makes someone cry however so you might not want to try it | 09:18 |
| sean-k-mooney | set it to permissive mode | 09:18 |
| kashyap | stephenfin: Also, I closely w/ the virt upstreams and downstreams to the point that I feel part of those teams too :) | 09:19 |
| lyarwood | sean-k-mooney: https://stopdisablingselinux.com/ | 09:19 |
| kashyap | sean-k-mooney: It's not about bugs, per se. Even new virt features first land in Fedora, BTW :) | 09:19 |
| kashyap | And kernel, of course | 09:19 |
| lyarwood | but yeah selinux can be a royal PITA | 09:19 |
| sean-k-mooney | lyarwood: well i have tried using the command to updat the policy that you get prometed with when there is a failure by the way | 09:20 |
| kashyap | lyarwood: Go tell that to Red Hat It's part of the value prop ;-) | 09:20 |
| sean-k-mooney | kashyap: sure but i dont want bleeding edge for my infra i was new but stable which is why i normally go with debiab/ubuntu distros with the mainline kernel | 09:21 |
| lyarwood | it can add value and still be a PITA to use ;) | 09:21 |
| sean-k-mooney | centos with some tweeks i coudl live with | 09:21 |
| sean-k-mooney | fedora not so much for anything i want to not have to maintain too much | 09:21 |
| kashyap | lyarwood: Heh, sure | 09:22 |
| kashyap | lyarwood: It took me and RHT virt team several weeks to debug a crazy VMs+container interaction bug. | 09:22 |
| sean-k-mooney | kashyap: lyarwood by the way the main thing that anowyed me about selinux recently is it does not allow you to use iso form your home directoy to create vms in virt-manager | 09:23 |
| kashyap | lyarwood: Here's my summary notes, for your "bedtime SELinux reading": https://kashyapc.fedorapeople.org/SELinux_libvirt_and_QEMU_in_a_container.html | 09:23 |
| * kashyap ducks | 09:23 | |
| sean-k-mooney | i rememebr you working on that ya | 09:23 |
| lyarwood | sean-k-mooney: yeah I've updated policykit to launch domains as my normal user without sudo but I still copy disks into /var/lib/libvirt/images etc | 09:24 |
| sean-k-mooney | to be fair apparmor is not nessisarly any eaiser to debug or work with so i think any security system is going to be complex by nature of all the context they need to work and that you need to understand them | 09:25 |
| sean-k-mooney | its those kind of things that are unfreidly to new comers and make distos feel unpolished however | 09:27 |
| sean-k-mooney | i kind of get why you might not want libvirt or qemu being able to read/write your home directory | 09:27 |
| sean-k-mooney | but since i can add it as a location in virt manger and select the image i would expect it to work or give me an error when i try to add the storage pool location | 09:28 |
| lyarwood | remind me again, where are the irc chat logs for this channel? | 09:29 |
| sean-k-mooney | ill get it one sec | 09:29 |
| sean-k-mooney | http://eavesdrop.openstack.org/irclogs/%23openstack-nova/ | 09:29 |
| lyarwood | ta | 09:29 |
| *** luksky has quit IRC | 09:41 | |
| *** luksky has joined #openstack-nova | 09:42 | |
| sean-k-mooney | gibi: shall i prepare the patch to move the implemtned specs to the correct directory? have the blueprints been updated in launchpad? | 09:44 |
| sean-k-mooney | im mostly done i need to fix a few files names too wehre the file name did not match the blueprint name | 09:55 |
| *** sapd1 has joined #openstack-nova | 09:56 | |
| *** k_mouza has joined #openstack-nova | 09:56 | |
| *** elod is now known as elod_afk | 10:01 | |
| *** zzzeek has quit IRC | 10:03 | |
| *** zzzeek has joined #openstack-nova | 10:05 | |
| sean-k-mooney | lyarwood: mind if i create a second blueprint for the linux part of the ephemeral sotrage encyption | 10:06 |
| sean-k-mooney | lyarwood: the spec repo tooling assume the file name of the spec matches the repo | 10:06 |
| sean-k-mooney | *blueprint | 10:06 |
| sean-k-mooney | but we have 2 specs for the saem bluepinrt | 10:06 |
| sean-k-mooney | i can also just move or in this case not move that one by hand | 10:06 |
| sean-k-mooney | which ever you prefer | 10:07 |
| lyarwood | sean-k-mooney: yeah I don't mind having a second blueprint for the libvirt part | 10:07 |
| lyarwood | sean-k-mooney: keeps the bean counters happy :) | 10:08 |
| sean-k-mooney | ok ill creeated and make one depend on the the other | 10:08 |
| sean-k-mooney | * create it | 10:08 |
| lyarwood | cool TIL you can do that with blueprints | 10:09 |
| sean-k-mooney | yep we rarely do and i have not don it in a very long time so trying to rememebr how | 10:10 |
| gibi | sean-k-mooney: thanks for preparing that patch I forgot | 10:11 |
| sean-k-mooney | no worries i did it last time i think and i have done it at least one other time so i dont mind the script does most of the work | 10:12 |
| sean-k-mooney | just fixing the warnign form the inccorect file names now | 10:12 |
| gibi | thanks | 10:14 |
| sean-k-mooney | lyarwood: there https://blueprints.launchpad.net/nova/+spec/ephemeral-encryption-libvirt you can see the depency tree at the bottom | 10:19 |
| sean-k-mooney | we dont ofter use that feature but when doing cross project stuff we proably shoudl use it more | 10:20 |
| *** swp20 has quit IRC | 10:23 | |
| *** ociuhandu has quit IRC | 10:39 | |
| openstackgerrit | sean mooney proposed openstack/nova-specs master: Move the implemented specs for the wallaby release https://review.opendev.org/c/openstack/nova-specs/+/784369 | 10:39 |
| sean-k-mooney | gibi: so assuming the blueprint status is correcct that should be correct | 10:41 |
| sean-k-mooney | looking at the list we had a fairly good completion ratio | 10:41 |
| gibi | yepp we had ~70% | 10:49 |
| *** sapd1 has quit IRC | 10:53 | |
| *** k_mouza has quit IRC | 11:04 | |
| *** ociuhandu has joined #openstack-nova | 11:09 | |
| *** mkrai has quit IRC | 11:10 | |
| *** rcernin_ has joined #openstack-nova | 11:11 | |
| *** k_mouza has joined #openstack-nova | 11:12 | |
| *** sapd1 has joined #openstack-nova | 11:20 | |
| artom | Wait, how the hell did Zuul not yet run on a patch I proposed last evening? | 11:21 |
| gibi | artom: there was some outage | 11:21 |
| sean-k-mooney | zuul restart | 11:21 |
| gibi | do a recheck to queue it | 11:22 |
| sean-k-mooney | all pending jobs got lost so just recheck | 11:22 |
| artom | Ah | 11:22 |
| sean-k-mooney | it was mention on th emailing list but i recall was " something something ooo ...." | 11:22 |
| sean-k-mooney | then i stopped caring | 11:23 |
| *** ociuhandu has quit IRC | 11:23 | |
| artom | Lulz | 11:26 |
| *** k_mouza has quit IRC | 11:28 | |
| *** jamesdenton has quit IRC | 11:31 | |
| *** jamesden_ has joined #openstack-nova | 11:32 | |
| openstackgerrit | Merged openstack/nova-specs master: Move the implemented specs for the wallaby release https://review.opendev.org/c/openstack/nova-specs/+/784369 | 11:35 |
| *** rcernin_ has quit IRC | 11:48 | |
| *** ociuhandu has joined #openstack-nova | 11:52 | |
| openstackgerrit | Merged openstack/nova master: docs: Rewrite 'nova-manage db' docs to use proper directives https://review.opendev.org/c/openstack/nova/+/677509 | 11:55 |
| openstackgerrit | Merged openstack/nova master: docs: Rewrite 'nova-manage api_db', 'cell_v2' docs https://review.opendev.org/c/openstack/nova/+/726192 | 11:56 |
| openstackgerrit | Merged openstack/nova master: docs: Rewrite 'nova-manage placement' docs https://review.opendev.org/c/openstack/nova/+/726193 | 11:57 |
| *** ociuhandu has quit IRC | 11:57 | |
| openstackgerrit | Merged openstack/nova master: docs: Rewrite 'nova-manage libvirt' docs https://review.opendev.org/c/openstack/nova/+/781504 | 11:58 |
| *** k_mouza has joined #openstack-nova | 12:00 | |
| *** ociuhandu has joined #openstack-nova | 12:08 | |
| *** mkrai has joined #openstack-nova | 12:09 | |
| *** ociuhandu has quit IRC | 12:13 | |
| *** jraju__ has joined #openstack-nova | 12:17 | |
| *** links has quit IRC | 12:17 | |
| *** mkrai has quit IRC | 12:18 | |
| *** ociuhandu has joined #openstack-nova | 12:24 | |
| *** bhagyashris has quit IRC | 12:28 | |
| *** ociuhandu has quit IRC | 12:29 | |
| *** bhagyashris has joined #openstack-nova | 12:29 | |
| *** sapd1 has quit IRC | 12:31 | |
| gibi | cores: my today's doctor's appointment got pushed back to 17:50 CEST which means I cannot chair today's nova meeting. I updated the agenda on the wiki, and I might be able to follow the meeting. Who will volunteer to chair the meeting? | 12:38 |
| *** jangutter_ is now known as jangutter | 12:38 | |
| *** ociuhandu has joined #openstack-nova | 12:40 | |
| *** ratailor has quit IRC | 12:45 | |
| *** ociuhandu has quit IRC | 12:47 | |
| gibi | lyarwood, bauzas, stephenfin, ^^ any volunteers :) | 12:55 |
| bauzas | gibi: i can do | 12:55 |
| gibi | bauzas: thanks, it is sold to you then :) | 12:55 |
| bauzas | 18:00 CEST, right? | 12:55 |
| gibi | 18:00 CEST yes | 12:55 |
| bauzas | kk | 12:55 |
| *** stand has joined #openstack-nova | 12:55 | |
| bauzas | gibi: any agenda already ? | 12:56 |
| gibi | bauzas: the agenda is up to date on the wiki https://wiki.openstack.org/wiki/Meetings/Nova#Agenda_for_next_meeting | 12:56 |
| *** ociuhandu has joined #openstack-nova | 13:00 | |
| bauzas | gibi: all good, thanks | 13:02 |
| bauzas | gibi: fwiw, as schools are closing next week, I'll take 2 days off on next Thursday and Friday | 13:02 |
| gibi | bauzas: ack, | 13:03 |
| bauzas | (and Monday is a bank holiday) | 13:03 |
| bauzas | then, children will be on vacations for 2 weeks and then back to schools (hopefully) | 13:03 |
| *** ociuhandu has quit IRC | 13:04 | |
| *** ociuhandu has joined #openstack-nova | 13:05 | |
| *** jamesden_ has quit IRC | 13:07 | |
| *** jamesdenton has joined #openstack-nova | 13:07 | |
| stephenfin | gibi: If you've a chance, could you take a look at https://review.opendev.org/c/openstack/nova/+/726195/ and the other few patches in that series today? Would be nice to close that out | 13:10 |
| gibi | stephenfin: will try | 13:14 |
| stephenfin | good enough for me. Thanks | 13:14 |
| *** jamesdenton has quit IRC | 13:19 | |
| *** jamesden_ has joined #openstack-nova | 13:20 | |
| *** Luzi has quit IRC | 13:25 | |
| *** derekh has quit IRC | 13:33 | |
| *** derekh has joined #openstack-nova | 13:34 | |
| *** dklyle has joined #openstack-nova | 13:36 | |
| *** derekh has quit IRC | 13:36 | |
| *** derekh has joined #openstack-nova | 13:36 | |
| yonglihe | gibi, alex, xinren repropose the spec, hope you have time slot for that. btw, code alse rebased and addressed comments: Repropose smartnic support spec https://review.opendev.org/c/openstack/nova-specs/+/783632 | 13:46 |
| gibi | yonglihe: ack | 13:46 |
| yonglihe | thanks, i gonna run, have a good day. | 13:47 |
| gmann | sean-k-mooney: lyarwood gibi basically if new return error code is new to that API (not from existing one) then we need microversion bump. and these are general return error code for all APis 400, 403, 404 and 415 | 13:49 |
| gmann | sean-k-mooney: lyarwood gibi which APIs is this? I think most of server APIs have 409 exist return code | 13:51 |
| gibi | os-services/{id}/force_doewn PUT | 13:55 |
| gmann | yeah 409 is not there https://github.com/openstack/nova/blob/50fdbc752a9ca9c31488140ef2997ed59d861a41/nova/api/openstack/compute/services.py#L358 | 13:59 |
| gibi | yepp | 14:02 |
| gibi | I accept 400 as a compromise for backportability | 14:02 |
| gmann | +1 | 14:03 |
| sean-k-mooney | gmann: yep that is basically what we discusssd on irc this morning 400 for backport reasons. 409 is more correct but would require version bump so we cant use it | 14:17 |
| sean-k-mooney | at least not yet | 14:17 |
| sean-k-mooney | we can make it use 409 in a followup patch if we wish too | 14:17 |
| *** sapd1 has joined #openstack-nova | 14:20 | |
| gmann | but we would not be able to backport 409 if do in follow up due to microvesion bump. | 14:23 |
| sean-k-mooney | correct | 14:27 |
| sean-k-mooney | so we were suggeing using 400 and backporting that | 14:27 |
| sean-k-mooney | and leavign a todo to make it a 409 at some other point in the future | 14:27 |
| sean-k-mooney | we coudl bundel it with some of the ohter todos we have | 14:28 |
| sean-k-mooney | we have one for consitent return codes for unimplemnted feature for example | 14:28 |
| gmann | yeah. make sense | 14:28 |
| *** elod_afk is now known as elod | 14:29 | |
| *** lbragstad has quit IRC | 14:36 | |
| *** jangutter_ has joined #openstack-nova | 14:40 | |
| *** jangutte_ has joined #openstack-nova | 14:41 | |
| *** jangutter has quit IRC | 14:42 | |
| *** k-s-dean has joined #openstack-nova | 14:45 | |
| *** jangutter_ has quit IRC | 14:45 | |
| *** lpetrut has quit IRC | 14:45 | |
| *** belmoreira has joined #openstack-nova | 14:58 | |
| *** macz_ has joined #openstack-nova | 15:02 | |
| *** tkajinam has quit IRC | 15:02 | |
| *** macz_ has quit IRC | 15:03 | |
| *** macz_ has joined #openstack-nova | 15:04 | |
| *** dtantsur is now known as dtantsur|afk | 15:06 | |
| artom | Is Zuul... OK? | 15:19 |
| artom | https://downforeveryoneorjustme.com/zuul.opendev.org ah :( | 15:19 |
| melwitt | see #opendev for details, they've been debugging issues since yesterday | 15:20 |
| *** belmoreira has quit IRC | 15:22 | |
| *** belmoreira has joined #openstack-nova | 15:28 | |
| *** mlavalle has joined #openstack-nova | 15:33 | |
| *** derekh has quit IRC | 15:45 | |
| *** whoami-rajat has quit IRC | 15:47 | |
| bauzas | folks, gentle reminder, nova meeting in 9 mins on #openstack-meeting-3 | 15:51 |
| *** lucasagomes has quit IRC | 15:59 | |
| *** ociuhandu has quit IRC | 16:00 | |
| bauzas | meeting started on #openstack-meeting-3 | 16:00 |
| *** jraju__ has quit IRC | 16:08 | |
| *** k_mouza has quit IRC | 16:24 | |
| *** ociuhandu has joined #openstack-nova | 16:27 | |
| *** happyhemant has quit IRC | 16:27 | |
| *** ociuhandu has quit IRC | 16:33 | |
| *** jamesden_ has quit IRC | 16:33 | |
| *** jamesdenton has joined #openstack-nova | 16:34 | |
| *** hamalq has joined #openstack-nova | 16:40 | |
| *** k-s-dean has quit IRC | 16:41 | |
| * stephenfin knocks off; back on the 12th o/ | 16:57 | |
| gibi | stephenfin: enjoy o/ | 17:01 |
| *** bbowen_ has quit IRC | 17:02 | |
| *** sorin-mihai has quit IRC | 17:06 | |
| *** k_mouza has joined #openstack-nova | 17:09 | |
| *** jangutter has joined #openstack-nova | 17:09 | |
| *** jangutte_ has quit IRC | 17:12 | |
| *** k_mouza has quit IRC | 17:13 | |
| *** belmoreira has quit IRC | 17:39 | |
| *** viks____ has quit IRC | 18:07 | |
| sean-k-mooney | artom: by the way you proably want https://zuul.openstack.org/ | 18:18 |
| sean-k-mooney | https://zuul.opendev.org/tenants has all then tenants | 18:19 |
| sean-k-mooney | but zuul.openstack.org will show you the openstack ci dashboard | 18:19 |
| artom | sean-k-mooney, I was just copy-pasting from the logs url | 18:34 |
| openstackgerrit | Lee Yarwood proposed openstack/nova master: Add regression test for bug #1922053 https://review.opendev.org/c/openstack/nova/+/784129 | 18:41 |
| openstack | bug 1922053 in OpenStack Compute (nova) "Operators can force up compute services with `done` evacuation migration records still active against the host" [Medium,In progress] https://launchpad.net/bugs/1922053 - Assigned to Lee Yarwood (lyarwood) | 18:41 |
| openstackgerrit | Lee Yarwood proposed openstack/nova master: api: Reject requests to force up computes when `done` evacuation records exist https://review.opendev.org/c/openstack/nova/+/784130 | 18:41 |
| *** jamesdenton has quit IRC | 18:48 | |
| *** jamesden_ has joined #openstack-nova | 18:48 | |
| *** jangutter_ has joined #openstack-nova | 19:32 | |
| *** jangutter has quit IRC | 19:35 | |
| *** jamesden_ is now known as jamesdenton | 19:46 | |
| *** slaweq_ has joined #openstack-nova | 19:51 | |
| *** slaweq has quit IRC | 19:52 | |
| *** CeeMac has joined #openstack-nova | 19:54 | |
| *** ociuhandu has joined #openstack-nova | 20:20 | |
| *** k_mouza has joined #openstack-nova | 20:25 | |
| openstackgerrit | Artom Lifshitz proposed openstack/nova master: Neutron fixture: don't clobber profile and vif_details if empty https://review.opendev.org/c/openstack/nova/+/783385 | 20:27 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova master: Test SRIOV port move operations with PCI conflicts https://review.opendev.org/c/openstack/nova/+/783084 | 20:27 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova master: Update SRIOV port pci_slot when unshelving https://review.opendev.org/c/openstack/nova/+/784168 | 20:27 |
| sean-k-mooney | :) | 20:28 |
| sean-k-mooney | no more WIP | 20:28 |
| artom | Yeah, rewrite the ugly [0] logic, and added a unit test | 20:28 |
| artom | And a release note | 20:28 |
| artom | And the bug reference | 20:28 |
| artom | And my axe! | 20:28 |
| sean-k-mooney | ah yes your using next() | 20:29 |
| *** k_mouza has quit IRC | 20:29 | |
| sean-k-mooney | you technially have extra () | 20:29 |
| sean-k-mooney | that you dont need | 20:29 |
| sean-k-mooney | equest = next((r for r in instance.pci_requests.requests | 20:30 |
| sean-k-mooney | if r.requester_id == port['id'])) | 20:30 |
| artom | No, that's needed | 20:30 |
| artom | Wait, is it? | 20:30 |
| artom | Maybe it isn't... | 20:30 |
| artom | What you pass to next() has to be an interator | 20:30 |
| sean-k-mooney | you are creating a touple | 20:30 |
| artom | *iterator, not a list/tuple | 20:31 |
| sean-k-mooney | but you can pass the generator expression driectly to next | 20:31 |
| sean-k-mooney | iterator or generator | 20:31 |
| sean-k-mooney | "r for r in instance.pci_requests.requests" is a generator expression | 20:31 |
| artom | Oh yeah, works without the extra ( ) | 20:32 |
| artom | TIL | 20:32 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova master: Update SRIOV port pci_slot when unshelving https://review.opendev.org/c/openstack/nova/+/784168 | 20:32 |
| sean-k-mooney | with the extra () its doing a touple comprehention and then it taking the first element | 20:32 |
| sean-k-mooney | without it its iterating the generator untile the first value is yeilded | 20:32 |
| sean-k-mooney | so its faster too since it will early exit | 20:33 |
| sean-k-mooney | artom: thats a python 3 thing by the way | 20:35 |
| artom | Oh | 20:35 |
| artom | Hrmm | 20:35 |
| artom | Well, no, it's fine, Train is Python3 | 20:35 |
| sean-k-mooney | its easy to add them back when you get to train | 20:35 |
| sean-k-mooney | its both | 20:35 |
| artom | Ah, it's OSP16 that's py3-only | 20:35 |
| artom | Because RHEL8 | 20:35 |
| sean-k-mooney | ussuri was the first one that is pyton 3 only | 20:36 |
| sean-k-mooney | yes | 20:36 |
| sean-k-mooney | which is proably the only time rhel has done someting first :) | 20:36 |
| artom | *snerk* | 20:37 |
| *** ociuhandu has quit IRC | 20:38 | |
| openstackgerrit | sean mooney proposed openstack/nova stable/train: add functional regression test for bug #1888395 https://review.opendev.org/c/openstack/nova/+/759533 | 20:40 |
| openstack | bug 1888395 in OpenStack Compute (nova) train "live migration of a vm using the single port binding work flow is broken in train as a result of the introduction of sriov live migration" [High,In progress] https://launchpad.net/bugs/1888395 - Assigned to Billy Olsen (billy-olsen) | 20:40 |
| openstackgerrit | sean mooney proposed openstack/nova stable/train: Set migrate_data.vifs only when using multiple port bindings https://review.opendev.org/c/openstack/nova/+/759153 | 20:40 |
| *** luksky has quit IRC | 20:51 | |
| *** luksky has joined #openstack-nova | 20:51 | |
| *** tbachman has quit IRC | 21:03 | |
| *** tbachman has joined #openstack-nova | 21:11 | |
| *** ociuhandu has joined #openstack-nova | 21:38 | |
| *** lemko9 has joined #openstack-nova | 21:42 | |
| *** ociuhandu has quit IRC | 21:42 | |
| *** lemko has quit IRC | 21:42 | |
| *** lemko9 is now known as lemko | 21:42 | |
| *** osmanlicilegi has quit IRC | 21:44 | |
| *** jamesdenton has quit IRC | 22:10 | |
| *** jamesden_ has joined #openstack-nova | 22:11 | |
| *** eharney has quit IRC | 22:54 | |
| *** tkajinam has joined #openstack-nova | 22:57 | |
| *** tkajinam has quit IRC | 22:57 | |
| *** tkajinam has joined #openstack-nova | 22:58 | |
| *** amodi has quit IRC | 23:14 | |
| *** jamesden_ has quit IRC | 23:24 | |
| *** jamesdenton has joined #openstack-nova | 23:24 | |
| *** luksky has quit IRC | 23:30 | |
| *** jangutter_ has quit IRC | 23:38 | |
| *** tosky has quit IRC | 23:40 | |
| *** martinkennelly has quit IRC | 23:41 | |
| *** martinkennelly has joined #openstack-nova | 23:42 | |
| *** macz_ has quit IRC | 23:50 | |
| *** macz_ has joined #openstack-nova | 23:56 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!