| opendevreview | melanie witt proposed openstack/nova master: TPM: add RequestContext checks to functional tests https://review.opendev.org/c/openstack/nova/+/966499 | 01:32 |
|---|---|---|
| opendevreview | melanie witt proposed openstack/nova master: TPM: support instances with `user` secret security https://review.opendev.org/c/openstack/nova/+/942502 | 01:32 |
| opendevreview | melanie witt proposed openstack/nova master: TPM: support instances with `host` secret security https://review.opendev.org/c/openstack/nova/+/941795 | 01:32 |
| opendevreview | melanie witt proposed openstack/nova master: Add vtpm_secret_(uuid|value) to LibvirtLiveMigrateData https://review.opendev.org/c/openstack/nova/+/952628 | 01:32 |
| opendevreview | melanie witt proposed openstack/nova master: TPM: bump service version and require it for live migration https://review.opendev.org/c/openstack/nova/+/962051 | 01:32 |
| opendevreview | melanie witt proposed openstack/nova master: TPM: support live migration of `host` secret security https://review.opendev.org/c/openstack/nova/+/941483 | 01:32 |
| opendevreview | melanie witt proposed openstack/nova master: TPM: support instances with `deployment` secret security https://review.opendev.org/c/openstack/nova/+/942021 | 01:32 |
| opendevreview | melanie witt proposed openstack/nova master: TPM: support live migration of `deployment` secret security https://review.opendev.org/c/openstack/nova/+/925771 | 01:32 |
| opendevreview | melanie witt proposed openstack/nova master: TPM: test live migration between hosts with different security https://review.opendev.org/c/openstack/nova/+/952629 | 01:32 |
| opendevreview | melanie witt proposed openstack/nova master: TPM: add late check for supported TPM secret security https://review.opendev.org/c/openstack/nova/+/956975 | 01:32 |
| opendevreview | melanie witt proposed openstack/nova master: TPM: opt-in to new TPM secret security via resize https://review.opendev.org/c/openstack/nova/+/962052 | 01:32 |
| opendevreview | melanie witt proposed openstack/nova master: TPM: add documentation and reno for live migration https://review.opendev.org/c/openstack/nova/+/962889 | 01:32 |
| opendevreview | melanie witt proposed openstack/nova master: DNM vtpm tempest https://review.opendev.org/c/openstack/nova/+/957477 | 01:32 |
| *** mhen_ is now known as mhen | 02:37 | |
| melwitt | bauzas: thanks for the reviews :) I have updated the bottom two patches you commented on | 02:46 |
| opendevreview | Ghanshyam proposed openstack/nova-specs master: Add spec for Graceful shutodwn of nova services https://review.opendev.org/c/openstack/nova-specs/+/967469 | 02:54 |
| opendevreview | Ghanshyam proposed openstack/nova master: PoC: Graceful shutodwn of nova services https://review.opendev.org/c/openstack/nova/+/967261 | 03:27 |
| opendevreview | Ghanshyam proposed openstack/nova-specs master: Add spec for Graceful shutodwn of nova services https://review.opendev.org/c/openstack/nova-specs/+/967469 | 03:44 |
| aravindh8murugesan | :sean-k-mooney I'm passing an nVidia GPU to few of my VMs in passthrough mode. I have noticed for VM with GPU passthrough, openstack nova is not honoring the memory over commitment ratio? Is this expected. Do I need 1:1 memory for passthrough GPU devices? | 04:23 |
| aravindh8murugesan | Maybe I should word it little clearly, nova does honor the overcommitment and scheduled the VM. So for example if my host has 128GB ram and provisioned VMs come out to about 140gb of RAM allocation, trying to boot up all the VMs at once, forces many of these vm to error state. | 04:25 |
| LarsErikP | tobias-urdin: thanks. So, I think I read out of the short discussion that followed, that someone is now thinking about merging it? :P | 07:43 |
| nicolairuckel | sean-k-mooney, I talked to some colleagues who also do operations. They said that it would probably be better if a patch for cold migration would be merged in the same cycle as the other operations that are currently in the NVRAM patch and they were afraid that this won't be possible if those are separate patches. They didn't see any issue with shelving in a separate patch. What do you think? | 08:08 |
| gibi | sean-k-mooney: re vmware ci. No I don't see it triggering on oslo.vmware patches https://review.opendev.org/c/openstack/oslo.vmware/+/967418 | 08:22 |
| gibi | but I can put up nova DNM with depends on to see if that helps | 08:22 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: DNM:Test with oslo.vmware eventlet removal patches https://review.opendev.org/c/openstack/nova/+/967487 | 08:27 |
| gibi | lets see ^^ | 08:27 |
| *** elodilles_pto is now known as elodilles | 08:51 | |
| opendevreview | Rajesh Tailor proposed openstack/nova master: Fix memory_details units in diagnostics API to match documentation https://review.opendev.org/c/openstack/nova/+/967505 | 11:16 |
| opendevreview | sean mooney proposed openstack/nova-specs master: Repropose and update cyborg vGPU (mdev) support https://review.opendev.org/c/openstack/nova-specs/+/967515 | 11:49 |
| nelljerram | Hi all. Could someone take a look at https://review.opendev.org/c/openstack/nova/+/960284? It already has one +2 from Sean but would appreciate a second core review. | 12:55 |
| gibi | nelljerram: have you managed to test this patch with calico succesfully? | 13:02 |
| gibi | nelljerram: the patch looks good to me and it is low risk for us upstream as it only touches a very specific code path. So if it works for calico then I'm fine approving it | 13:03 |
| nelljerram | gibi: Many thanks; I have tested that it works for Calico. | 13:05 |
| gibi | cool. approved | 13:06 |
| gibi | nelljerram: thanks for fixing it | 13:07 |
| nelljerram | gibi: To add a bit more detail, we have a pretty extensive system test - unfortunately closed source at the moment - and it completely fails without this patch, but passes with it. | 13:07 |
| nelljerram | gibi: when tested against Caracal or later, I mean | 13:07 |
| gibi | nelljerram: OK. If you can eventually open up that test env then we would appreciate a 3rd party CI with Calico :) | 13:08 |
| gibi | nelljerram: feel free to ping me with the backports when you propose them | 13:08 |
| nelljerram | gibi: Thanks so much. I would indeed like to open up the testing, and will work towards also connecting that to OpenStack CI. But I expect that will take a little while, so please don't hold your breath for it! | 13:10 |
| nelljerram | gibi: Do I wait until the change is merged to master, before proposing the backports? | 13:11 |
| gibi | I think you can start backporting with `git cherry-pick -x ` It is unlikely that the hash of the commit will change now. But if it does then you just respin the backport to point to the final hash. | 13:13 |
| gibi | nelljerram: no worries I'm not holding breath. :) But I'm happy to see that there is motivated folks trying to open it up | 13:13 |
| sean-k-mooney | gibi: nelljerram the funny thing is we are likely going to end up using managed=no | 13:24 |
| sean-k-mooney | for ovs as well in the future or at least with ovn | 13:24 |
| sean-k-mooney | but before we do that we need to change os-vif to support creating the tap | 13:24 |
| sean-k-mooney | also we chagne form interface=bridge to interface=ethernet precincely because it did not delete the tap | 13:25 |
| sean-k-mooney | so thet libvirt behavior change regressed that again | 13:25 |
| sean-k-mooney | well not quite we didi dit for the ovs prot defintion | 13:25 |
| sean-k-mooney | so im remmeber that wrong but its a similr effect | 13:25 |
| sean-k-mooney | it cause our pre-plugging of the neutron port in pre-livemigrate to be less effective | 13:26 |
| sean-k-mooney | gibi: oh i ment patches to nova | 13:27 |
| sean-k-mooney | gibi: for the vmware changes | 13:27 |
| sean-k-mooney | gibi: do you also need to update oslo.vmware for the eventlet removal | 13:27 |
| sean-k-mooney | libs in general are not ment ot use eventlet directly but i guess they were for some api requests? | 13:28 |
| nelljerram | sean-k-mooney: interesting, thanks. I guess that will also be strong motivation for hooking up some Calico CI. | 13:32 |
| sean-k-mooney | we likely wont be modifying the vif_type=tap code paths but we will be setting the xml atibute form more places | 13:33 |
| sean-k-mooney | we basiclly need neutron to tell us to do it in the vif_details and as i said extened os-vif to be able to create the tap device | 13:34 |
| nelljerram | sean-k-mooney: fwiw I am also currently working on pre-livemigrate for Calico. Currently we ignore that and just flip the port over when migration completes - which is not very seamless :-) | 13:34 |
| sean-k-mooney | im not actully sure how to do that exactly but apprently its a thing we can do | 13:34 |
| sean-k-mooney | nelljerram: ah cool good to know | 13:35 |
| sean-k-mooney | for what its worht we have several jobs that do live migrationin the ci today on nova | 13:35 |
| sean-k-mooney | we can lilkely just add calico to one of them | 13:35 |
| sean-k-mooney | similar to how the nova-alt-config job use ml2/ovs with iptables instead of ovn | 13:35 |
| gibi | sean-k-mooney: oslo_vmware had some home made looping calls, a bit of greenthread.sleep and an eventlet.timeout for image download. So yeah if we want to keep support the vmware virt driver in native threading mode we need changes in the lib | 13:36 |
| sean-k-mooney | we just need to choose one of the jobs, enable the devstack plugin and that will give us some coverage | 13:36 |
| sean-k-mooney | gibi: ack, want might be a strong word for it but if its tested then sure | 13:37 |
| sean-k-mooney | gibi: we dont have visablity into the job defition but i assume depend on does nto work for there third party ci | 13:37 |
| sean-k-mooney | i.e. you cant triger it form nova? | 13:37 |
| nelljerram | thanks, I'll take a look at those. I've found it surprisingly hard to write an effective test for fully seamless live migration. I mean I already have a test that tests continuity of an SSH connection to a guest, across a live migration, and it passes, even though our live plumbing migration is really not properly seamless yet. | 13:38 |
| sean-k-mooney | nelljerram: so we actuly kind of have one | 13:39 |
| sean-k-mooney | nelljerram: there is a neutron test for live migration connectiviy | 13:39 |
| sean-k-mooney | however because our ci provider are quite slow/overloaded | 13:40 |
| sean-k-mooney | we have to set the downtime high in the jobs for stablity | 13:40 |
| gibi | sean-k-mooney: I tried here but it did not triggered (yet) There is also a spec about changing the device attach APIs async. Currently owned by Dan but expected to be taken over by Johannes (outside RH) https://review.opendev.org/c/openstack/nova-specs/+/958900 that I personally care about due to eventlet removal relatedness. If Johannes take it then I would like to get this on the list of | 13:40 |
| gibi | priorities. If Johannes does not take it the I would like to ask if RH can pick it up for G. | 13:40 |
| gibi | ahh sorry | 13:40 |
| gibi | wrong copy paste buffer | 13:40 |
| sean-k-mooney | nelljerram: https://github.com/openstack/tempest/blob/8ccefd4bea48d6b38f7e16c50f97931b3fc2a5e2/tempest/scenario/manager.py#L1062 | 13:40 |
| sean-k-mooney | there are live migration test that use that funciton to verify the conenctivy durign the migration | 13:41 |
| nelljerram | sean-k-mooney: thanks! | 13:41 |
| gibi | sean-k-mooney: so I tried to trigger it from here but no luck yet https://review.opendev.org/c/openstack/nova/+/967487 | 13:41 |
| gibi | I will discuss with fwiesel | 13:41 |
| gibi | when he is arond | 13:41 |
| sean-k-mooney | ack | 13:42 |
| sean-k-mooney | gibi: ya no rush either way i just said i woudl sugget that incase you had not tried it yet | 13:42 |
| gibi | ack | 13:42 |
| sean-k-mooney | if the job is not using zuul or even if it is but is not usign requreed project and devstack it woudl not pull in the patch | 13:43 |
| sean-k-mooney | most job devs dont include libs in required prodject to test with the released verion outside the lib repo so my guess is this wont actully work unless they do some tweaks to test it for us | 13:44 |
| gibi | yeah it is likely wont work | 13:45 |
| *** mikal4 is now known as mikal | 14:49 | |
| opendevreview | Merged openstack/nova master: Add managed='no' flag to libvirt XML definition for VIF type TAP https://review.opendev.org/c/openstack/nova/+/960284 | 14:57 |
| opendevreview | Nell Jerram proposed openstack/nova stable/2025.2: Add managed='no' flag to libvirt XML definition for VIF type TAP https://review.opendev.org/c/openstack/nova/+/967567 | 15:08 |
| opendevreview | Nell Jerram proposed openstack/nova stable/2025.1: Add managed='no' flag to libvirt XML definition for VIF type TAP https://review.opendev.org/c/openstack/nova/+/967568 | 15:09 |
| opendevreview | Nell Jerram proposed openstack/nova stable/2024.2: Add managed='no' flag to libvirt XML definition for VIF type TAP https://review.opendev.org/c/openstack/nova/+/967569 | 15:09 |
| nelljerram | gibi: sean-k-mooney: fyi I've failed the cherry-picks for https://review.opendev.org/c/openstack/nova/+/960284 now; hope I did it right | 15:14 |
| sean-k-mooney | nelljerram: almost | 15:15 |
| sean-k-mooney | so you are cherrypicking the older patches dreictly form master | 15:15 |
| sean-k-mooney | you shoudl cherry pick them form the newer stable branch instead | 15:15 |
| nelljerram | (just spotted that I wrote "failed" when I meant "filed"...) | 15:16 |
| sean-k-mooney | so 2025.1 should be cherry picked form 2025.2 and it shoudl have 1 "cherry picked from commit" lien per branch | 15:16 |
| nelljerram | so it should be master -> 2025.2, then 2025.2 -> 2025.1, then 2025.1 -> 2024.2 and so on? | 15:16 |
| sean-k-mooney | yep | 15:17 |
| gibi | nelljerram: also a small nit, please change the topic of the gerrit reviews to have the same topic as the master patch, that help discovering the reviews | 15:17 |
| sean-k-mooney | that helps too ya | 15:18 |
| sean-k-mooney | the master patch shoudl have had "bug/2033681" as the topic | 15:18 |
| sean-k-mooney | but its too late to change it from https://review.opendev.org/q/topic:%22libvirt-tap-managed-no%22 | 15:18 |
| sean-k-mooney | so "libvirt-tap-managed-no" is fine for all the bafckports | 15:19 |
| nelljerram | right, will change them all to "libvirt-tap-managed-no" | 15:19 |
| opendevreview | Nell Jerram proposed openstack/nova stable/2025.1: Add managed='no' flag to libvirt XML definition for VIF type TAP https://review.opendev.org/c/openstack/nova/+/967568 | 15:27 |
| opendevreview | Nell Jerram proposed openstack/nova stable/2024.2: Add managed='no' flag to libvirt XML definition for VIF type TAP https://review.opendev.org/c/openstack/nova/+/967569 | 15:28 |
| nelljerram | gibi: sean-k-mooney: hopefully all correct now? | 15:30 |
| sean-k-mooney | the commit message are not quiter right | 15:32 |
| sean-k-mooney | they still only have one cherry picked form line | 15:32 |
| nelljerram | OK, I guess that's because the message wasn't recreated when I redid the picks in order. Let me try to fix that manually... | 15:35 |
| sean-k-mooney | you need to use -x or -X to add the line each time rather hten just a cherry pick without flags | 15:35 |
| sean-k-mooney | normally i use git-review to do this | 15:35 |
| nelljerram | I did it using the web UI at review.opendev.org - is that not recommended? (the vertical dots menu on the top right, then "Cherry pick") | 15:36 |
| sean-k-mooney | the web ui wont actully add the cherry-picked form lines properly unless you do it form a merged commit | 15:37 |
| nelljerram | ok, let me try this again using git-review... | 15:37 |
| sean-k-mooney | you can just edit the commit message in the ui | 15:38 |
| sean-k-mooney | to add the correct lines | 15:38 |
| nelljerram | OK, I'll try that first, presumably just need to discover the commit IDs... | 15:38 |
| sean-k-mooney | so the reason we do it this way is if there is a merge conflict along the way you fix it only in the correct only in the banch that it happend in hten cherry pick the fixed version to older ones | 15:39 |
| sean-k-mooney | the commit is aviabel in the ui beside where you slect the patch set just under where it say Comments | 15:39 |
| opendevreview | Nell Jerram proposed openstack/nova stable/2025.1: Add managed='no' flag to libvirt XML definition for VIF type TAP https://review.opendev.org/c/openstack/nova/+/967568 | 15:41 |
| opendevreview | Nell Jerram proposed openstack/nova stable/2024.2: Add managed='no' flag to libvirt XML definition for VIF type TAP https://review.opendev.org/c/openstack/nova/+/967569 | 15:44 |
| nelljerram | Ah I see. The commit ID is also available from the gitea link, but the one you pointed to is better. | 15:49 |
| sean-k-mooney | if you use the cli (with or without git review) you can pass a flag to add it automaticly | 15:53 |
| sean-k-mooney | but ya you can do it form the ui too | 15:53 |
| sean-k-mooney | when i do it form the ui i wait for the previous version to merge ot avoid having to update it | 15:53 |
| sean-k-mooney | i approved the first backport ill look at the ohter once its merged | 15:53 |
| nelljerram | many thanks! | 15:53 |
| opendevreview | Merged openstack/osc-placement master: Replace remaining py39 target https://review.opendev.org/c/openstack/osc-placement/+/963809 | 16:13 |
| opendevreview | Ghanshyam proposed openstack/nova-specs master: Add spec for Graceful shutodwn of nova services https://review.opendev.org/c/openstack/nova-specs/+/967469 | 16:36 |
| opendevreview | Ghanshyam proposed openstack/nova-specs master: Propose Step 1 of Graceful shutodwn spec https://review.opendev.org/c/openstack/nova-specs/+/967585 | 17:21 |
| Uggla | gmaan, question, server_shares policies are PROJECT_MEMBER or PROJECT_READER. Is it normal. Should it be PROJECT_MEMBER_OR_ADMIN or PROJECT_READER_OR_ADMIN respectively ? | 17:23 |
| jrosser | AWS host key signing | 17:24 |
| jrosser | argh | 17:24 |
| sean-k-mooney | Uggla: so project reader allsow anyoen wotih project member to access it | 17:25 |
| gmaan | Uggla: *_OR ADMIN should be there as we do want admin to access those | 17:25 |
| sean-k-mooney | so really the question is shoudl ti be PROJECT_READER or shoudl it be PROJECT_READER_OR_ADMIN | 17:25 |
| gmaan | yeah admin will not able to access PROJECT_READER as project id check is there so it should be PROJECT_READER_OR_ADMIN | 17:26 |
| gmaan | PROJECT_MEMBER can access PROJECT_READER as sean-k-mooney mentioned | 17:26 |
| sean-k-mooney | basicaly there is role inheritnace the member role grants the reader role and the admin role grant member | 17:26 |
| sean-k-mooney | so when we write the policy we specify the minium rule that is required | 17:27 |
| sean-k-mooney | the _OR_ADMIN version | 17:27 |
| gmaan | yeah | 17:27 |
| sean-k-mooney | allows admins to bypass the project membership check | 17:27 |
| sean-k-mooney | i..e its for the --all-tenants type commands | 17:27 |
| gmaan | sean-k-mooney: Uggla to avoid using those without OR_ADMIN, i think we should remove these alias and have a note about it there https://github.com/openstack/nova/blob/master/nova/policies/base.py#L41-L43 | 17:28 |
| gmaan | I kept them but forgot to add notes about that we need to support admin to access those project scoped role base permission | 17:29 |
| Uggla | gmaan, so as I suspected, it needs to be corrected on the server_shares, I will open a patch for it. ok ? | 17:30 |
| gmaan | Uggla: yeah | 17:30 |
| Uggla | gmaan 👍 | 17:31 |
| dansmith | melwitt: did you see bauzas' comment on the user patch about coverage? | 17:31 |
| dansmith | I didn't go validate the concern (yet) but thought maybe you missed it | 17:31 |
| melwitt | dansmith: yes, I added coverage. unless I messed up a rebase or upload. I'll check it | 17:32 |
| dansmith | oh, I missed that there was a PS after that comment.. you didn't mark it done so I thought it was fresh | 17:32 |
| melwitt | ah crap | 17:33 |
| dansmith | I guess I'm also not sure the delta between those PSes is what he was asking about | 17:34 |
| melwitt | dansmith: oh really? I thought he was pointing out that use of the image property for tpm_model was not covered? | 17:37 |
| dansmith | I don't know because I don't really know what he's asking.. he said "check the property" and you didn't add any new checks... and the new path excludes the model property | 17:38 |
| dansmith | like you already had a test that passed the tpm_model property | 17:39 |
| dansmith | ohh, as the image prop and not the flavor I guess | 17:39 |
| melwitt | yeah | 17:39 |
| melwitt | I didn't know what else it could mean but I'm open to maybe I misunderstood something :) | 17:40 |
| dansmith | I'm sure you're right I just didn't make sense of it initially | 17:40 |
| opendevreview | Merged openstack/nova master: api: Add response body schemas for snapshots APIs https://review.opendev.org/c/openstack/nova/+/952349 | 17:54 |
| opendevreview | Merged openstack/nova master: api: Add response body schemas for volume attachments APIs https://review.opendev.org/c/openstack/nova/+/952350 | 17:54 |
| opendevreview | Merged openstack/nova master: api: Add response body schemas for floating IP APIs https://review.opendev.org/c/openstack/nova/+/952972 | 18:08 |
| opendevreview | Merged openstack/placement master: Make sure [cors] allowed_origin accepts a list value https://review.opendev.org/c/openstack/placement/+/966275 | 18:08 |
| dansmith | melwitt: would it be possible to move the deployment mode patch down to just above host before we start into all the live migration related bits? | 18:39 |
| dansmith | or is there some dependency there I'm not seeing? | 18:39 |
| melwitt | dansmith: I don't think there is a dependency, so I think it can be moved. I'll try it and let you know if I run into anything that could prevent it | 18:47 |
| dansmith | okay, seems like it would be a nice organization to get all three of the modes landed and then review the migration stuff with all that sorted in our (my) minds | 18:48 |
| dansmith | if it's not hard.. | 18:48 |
| melwitt | yeah, makes sense | 18:50 |
| opendevreview | Stephen Finucane proposed openstack/placement master: setup: Remove pbr's wsgi_scripts https://review.opendev.org/c/openstack/placement/+/919582 | 19:00 |
| opendevreview | Stephen Finucane proposed openstack/placement master: Correct default oslo.conf file https://review.opendev.org/c/openstack/placement/+/967600 | 19:00 |
| opendevreview | Stephen Finucane proposed openstack/placement master: Migrate setup configuration to pyproject.toml https://review.opendev.org/c/openstack/placement/+/967601 | 19:00 |
| opendevreview | Stephen Finucane proposed openstack/placement master: pre-commit: Bump versions https://review.opendev.org/c/openstack/placement/+/967602 | 19:00 |
| opendevreview | Merged openstack/nova stable/2025.2: Add managed='no' flag to libvirt XML definition for VIF type TAP https://review.opendev.org/c/openstack/nova/+/967567 | 19:22 |
| opendevreview | melanie witt proposed openstack/nova master: TPM: support instances with `deployment` secret security https://review.opendev.org/c/openstack/nova/+/942021 | 19:27 |
| opendevreview | melanie witt proposed openstack/nova master: Add vtpm_secret_(uuid|value) to LibvirtLiveMigrateData https://review.opendev.org/c/openstack/nova/+/952628 | 19:27 |
| opendevreview | melanie witt proposed openstack/nova master: TPM: bump service version and require it for live migration https://review.opendev.org/c/openstack/nova/+/962051 | 19:27 |
| opendevreview | melanie witt proposed openstack/nova master: TPM: support live migration of `host` secret security https://review.opendev.org/c/openstack/nova/+/941483 | 19:27 |
| opendevreview | melanie witt proposed openstack/nova master: TPM: support live migration of `deployment` secret security https://review.opendev.org/c/openstack/nova/+/925771 | 19:27 |
| opendevreview | melanie witt proposed openstack/nova master: TPM: test live migration between hosts with different security https://review.opendev.org/c/openstack/nova/+/952629 | 19:27 |
| opendevreview | melanie witt proposed openstack/nova master: TPM: add late check for supported TPM secret security https://review.opendev.org/c/openstack/nova/+/956975 | 19:27 |
| opendevreview | melanie witt proposed openstack/nova master: TPM: opt-in to new TPM secret security via resize https://review.opendev.org/c/openstack/nova/+/962052 | 19:27 |
| opendevreview | melanie witt proposed openstack/nova master: TPM: add documentation and reno for live migration https://review.opendev.org/c/openstack/nova/+/962889 | 19:27 |
| opendevreview | melanie witt proposed openstack/nova master: DNM vtpm tempest https://review.opendev.org/c/openstack/nova/+/957477 | 19:28 |
| dansmith | melwitt: was it that easy? | 19:31 |
| melwitt | dansmith: yes, seems so | 19:33 |
| opendevreview | Lajos Katona proposed openstack/nova master: Use SDK for Neutron floating IPs https://review.opendev.org/c/openstack/nova/+/962604 | 19:33 |
| dansmith | damn I suck at negotiation.. I should have asked for more :P | 19:34 |
| melwitt | hah | 19:34 |
| opendevreview | Merged openstack/placement master: Replace remaining py39 target https://review.opendev.org/c/openstack/placement/+/963808 | 19:44 |
| opendevreview | Merged openstack/placement master: Remove tags from README https://review.opendev.org/c/openstack/placement/+/941792 | 19:44 |
| opendevreview | Merged openstack/placement master: Remove installation guide for openSUSE/SLES https://review.opendev.org/c/openstack/placement/+/949864 | 19:44 |
| opendevreview | Ghanshyam proposed openstack/nova-specs master: Add backlog spec for Graceful shutodwn of nova services https://review.opendev.org/c/openstack/nova-specs/+/967469 | 21:04 |
| opendevreview | Ghanshyam proposed openstack/nova-specs master: Propose Step 1 of Graceful shutodwn spec https://review.opendev.org/c/openstack/nova-specs/+/967585 | 21:11 |
| opendevreview | Ghanshyam proposed openstack/nova master: PoC: Graceful shutodwn of nova services https://review.opendev.org/c/openstack/nova/+/967261 | 21:23 |
| gmaan | dansmith: gibi: graceful shutdown spec is ready for review (a backlog spec to propose complete things and another change to implement the Step1 proposal for this cycle). | 21:26 |
| gmaan | I uploaded my PoC results, PoC and results links are in spec also. https://review.opendev.org/q/topic:%22bp/nova-services-graceful-shutdown%22 | 21:26 |
| gmaan | whenever you get time, no hurry | 21:26 |
| opendevreview | Ghanshyam proposed openstack/nova master: PoC: Graceful shutodwn of nova services https://review.opendev.org/c/openstack/nova/+/967261 | 21:54 |
| opendevreview | Ghanshyam proposed openstack/nova master: PoC: Graceful shutodwn of nova services https://review.opendev.org/c/openstack/nova/+/967261 | 23:59 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!