| opendevreview | Jay Faulkner proposed openstack/nova master: [ironic] Followup fixes around flavor ids https://review.opendev.org/c/openstack/nova/+/927764 | 00:56 |
|---|---|---|
| *** bauzas_ is now known as bauzas | 01:05 | |
| *** bauzas_ is now known as bauzas | 01:26 | |
| stan | Guys can I set a limit of how many VMs can be created simultaneously? Let's say if the user wants to create 20 VMs at a time, is there a mechanism to create 5vms in the first batch, and put the rest of the vm request in queue? | 01:39 |
| bauzas | I officially called out the FeatureFreeze | 01:45 |
| bauzas | all the 2024.2 blueprints are now either Deferred or Implemented | 01:45 |
| bauzas | numbers aren't great, tbh. | 01:45 |
| bauzas | https://blueprints.launchpad.net/nova/2024.2 | 01:46 |
| *** bauzas_ is now known as bauzas | 02:15 | |
| *** bauzas_ is now known as bauzas | 02:54 | |
| *** bauzas_ is now known as bauzas | 03:15 | |
| *** bauzas_ is now known as bauzas | 05:39 | |
| *** bauzas_ is now known as bauzas | 06:02 | |
| opendevreview | Takashi Kajinami proposed openstack/nova master: Add hw_mem_encryption_model image property https://review.opendev.org/c/openstack/nova/+/927706 | 06:33 |
| opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Launch instances with SEV-ES memory encryption https://review.opendev.org/c/openstack/nova/+/926106 | 06:33 |
| *** bauzas_ is now known as bauzas | 06:35 | |
| *** bauzas_ is now known as bauzas | 06:51 | |
| *** elodilles_pto is now known as elodilles | 07:26 | |
| opendevreview | Viktor Křivák proposed openstack/nova master: Ensure claims to placement are freed after evacuate complete Improve resource_tracker delete_allocation_for_evacuated_instance to handle non-cached compute hosts Fix test to accept this change https://review.opendev.org/c/openstack/nova/+/926292 | 07:34 |
| opendevreview | Takashi Kajinami proposed openstack/nova master: Add hw_mem_encryption_model image property https://review.opendev.org/c/openstack/nova/+/927706 | 09:05 |
| opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Launch instances with SEV-ES memory encryption https://review.opendev.org/c/openstack/nova/+/926106 | 09:05 |
| *** bauzas_ is now known as bauzas | 09:30 | |
| *** bauzas_ is now known as bauzas | 09:43 | |
| sean-k-mooney | stan: not on a per user basis or at teh api level | 10:43 |
| sean-k-mooney | stan: you can set a limit per compute node | 10:44 |
| sean-k-mooney | it defaults to 10 https://docs.openstack.org/nova/latest/configuration/config.html#DEFAULT.max_concurrent_builds | 10:44 |
| *** bauzas_ is now known as bauzas | 10:48 | |
| stan | sean-k-mooney Thanks for the information. :) | 10:59 |
| stan | I assume I can make changes to this in combination with scheduler timeout to achieve our use case. | 11:04 |
| sean-k-mooney | note really | 11:04 |
| sean-k-mooney | the compute node config only takes effect after the instance is schduled and just delays it spawning | 11:04 |
| stan | sean-k-mooney Alright. I will test this in our lab 👍 | 11:17 |
| stan | sean-k-mooney So nova timeout will effectively come into play after its schedule or after the VM starts to spawn? | 11:30 |
| sean-k-mooney | no | 11:30 |
| sean-k-mooney | there are a few tiemouts at play. there is an api request time out for the intiall http respocne, then there is the rpc timeout between the conductor schduelr adn computes | 11:31 |
| sean-k-mooney | and then there is an instance build timeout https://docs.openstack.org/nova/latest/configuration/config.html#DEFAULT.instance_build_timeout | 11:32 |
| sean-k-mooney | which defaults ot 0 meaning no limit | 11:32 |
| *** bauzas_ is now known as bauzas | 11:38 | |
| sean-k-mooney | dansmith: looks like we have another test that should be skiped without qemu-img | 12:16 |
| sean-k-mooney | https://paste.opendev.org/show/bOFlCycmdpOV1shHsHQX/ | 12:16 |
| sean-k-mooney | did we plan to merge your patch to use the oslo version? i lost track of where we were with that | 12:17 |
| jlejeune_ | sean-k-mooney: hello, indeed, that's why I've proposed: https://review.opendev.org/c/openstack/nova/+/925208 | 12:29 |
| sean-k-mooney | ah yes i tought this was familar | 12:30 |
| *** jlejeune_ is now known as jlejeune | 12:30 | |
| jlejeune | sean-k-mooney: :) | 12:30 |
| sean-k-mooney | i am usign distobox to create a clean dev env on my laptop to workaround some packaging issues so didnt have it installed | 12:31 |
| jlejeune | like on our CI | 12:31 |
| sean-k-mooney | well in the first party ci we do have qemu-img isntalled | 12:32 |
| sean-k-mooney | but in general it should not be requried | 12:32 |
| sean-k-mooney | https://review.opendev.org/c/openstack/nova/+/925208/comment/2b6ac8e7_286f92b1/ is still an open question that needs input form others | 12:33 |
| sean-k-mooney | im ok with the stub as i dont think the existin unit test should be executing that code but we still need others to weigh in | 12:34 |
| sean-k-mooney | jlejeune: to be honest i had tought we merged that already | 12:35 |
| *** bauzas_ is now known as bauzas | 12:35 | |
| jlejeune | I know I have some work to do to tka in consideration your comment but I didn't have time to work on | 12:36 |
| sean-k-mooney | thats fine we didnt have time to review it | 12:36 |
| jlejeune | ok so I don't change my code and let someone else review it based on your opinion ? | 12:46 |
| opendevreview | Takashi Kajinami proposed openstack/nova master: Replace deprecated glanceclient exception https://review.opendev.org/c/openstack/nova/+/914761 | 12:55 |
| *** bauzas_ is now known as bauzas | 13:00 | |
| sean-k-mooney | jlejeune: dan has a stonger prefernce then i do. i was ok with the reductionin code execution because i dont think a unit test should be as large as that one is and the coverage shoudl eb provided by smaller more targed test or or functional test suite | 13:04 |
| dansmith | sean-k-mooney: yes, merged, and that trace isn't in my code | 13:32 |
| sean-k-mooney | dansmith: i know its just a missing mock | 13:33 |
| dansmith | okay, I'm not sure what you're asking | 13:36 |
| sean-k-mooney | i was just letting you knwo we still have once ase where its not mocked and jlejeune remindiend me that we had dicussed this on tehre patch previously https://review.opendev.org/c/openstack/nova/+/925208 | 13:37 |
| jlejeune | sean-k-mooney: dansmith: not sure what I have to do about it now :) | 13:56 |
| dansmith | tbh, I don't hate the very first rev of this patch, and I think it's better than the current one | 13:59 |
| dansmith | I would str(e) and make sure there's a qemu-img in there so we only skip for that case | 13:59 |
| dansmith | but that leaves us all the coverage we have today on systems that can test it, | 13:59 |
| dansmith | and just skips it for limited environments | 13:59 |
| dansmith | just like our opportunistic DB testing | 13:59 |
| dansmith | jlejeune: ^ | 14:07 |
| jlejeune | dansmith: ok, you mean I had to check the str(e) of the exception to detect if it's really related to qemu-img, in my first patchset ? | 14:11 |
| dansmith | jlejeune: yeah | 14:12 |
| jlejeune | ok clear, doing, thanks | 14:12 |
| dansmith | sounded like sean-k-mooney was not okay with that approach though, but I'm not sure why.. it keeps the most coverage | 14:12 |
| dansmith | jlejeune: might want to wait for sean-k-mooney to say that's okay as well | 14:12 |
| jlejeune | ok | 14:12 |
| sean-k-mooney | dansmith: i not entirly against it but form my point of view the unit test is not a unit test today | 14:13 |
| sean-k-mooney | unit tests should not be calling code out side of the same call liek that | 14:13 |
| sean-k-mooney | if you think we have a clean way to keep the coverage and make this work that you are happy with then thats fine | 14:14 |
| dansmith | sean-k-mooney: you can argue that, sure, and I'm sure these exist since before the functional split, but unless you want jlejeune to refactor and move them out to functional, it seems like a reasonable "just fix the problem without the nuclear option of losing a bunch of coverage (regardless of the quality)" option | 14:14 |
| sean-k-mooney | its a trade off between fragile tests and test coverage | 14:15 |
| sean-k-mooney | dansmith: i dont have the review open right now but did the coverage report show a decresse in coverage? | 14:16 |
| sean-k-mooney | a hte resulsts expired | 14:16 |
| dansmith | sean-k-mooney: the only way it wouldn't is if we have all that code covered and mocked elsewhere, but the point of this test is to have some basic exercising of those things, | 14:16 |
| dansmith | so that if we were to remove the specific tests we'd at least have this | 14:17 |
| sean-k-mooney | dansmith: i dont think refactoring into the funcitonal test is a reasonable ask so i agree we shoudl just fix the exiting test for now | 14:17 |
| dansmith | sean-k-mooney: right, me either, I was just using that to make the point :) | 14:17 |
| sean-k-mooney | dansmith: ok so i disagree that that shoudl be its point | 14:17 |
| sean-k-mooney | it should be testing the hihgh level interface | 14:17 |
| sean-k-mooney | adn the lower leves shoudl be coverd elsewhere | 14:18 |
| sean-k-mooney | btu for now if your happy with yoru proposed solution then lets proceed with that | 14:18 |
| dansmith | again, I don't disagree with the "should" part.. just don't want to lose the "is currently" part | 14:18 |
| sean-k-mooney | so you want to go back to https://review.opendev.org/c/openstack/nova/+/925208/1/nova/tests/unit/virt/test_virt_drivers.py then and add a check that the excption has qemu-img in it | 14:19 |
| dansmith | if we get a process execution error, and that error mentions qemu-img in it, it's pretty obviously a very targeted case I think | 14:20 |
| sean-k-mooney | just to make sure we only skip if that why | 14:20 |
| dansmith | you could even look for "not found" or something if you want to get super targeted | 14:20 |
| dansmith | right | 14:20 |
| sean-k-mooney | if thats your prfernce then lets do that. i dont like skiping test after we tired to do soemthign but i think this is ok for now | 14:21 |
| sean-k-mooney | when swaping to the oslo version hwoever we shoudl be mocking out the image inpection usage | 14:21 |
| jlejeune | sean-k-mooney: +1 | 14:21 |
| dansmith | again, I'm not sure what this has to do with moving to the oslo version | 14:21 |
| dansmith | sean-k-mooney: you know this is *already* merged right? https://review.opendev.org/c/openstack/nova/+/925025 | 14:22 |
| sean-k-mooney | no i was not aware that merged | 14:22 |
| sean-k-mooney | the reason that relevent is while it might be ok to call | 14:24 |
| sean-k-mooney | nova.image.format_inspector.detect_file_format | 14:24 |
| sean-k-mooney | in a unit test inteded to test the driver interface | 14:25 |
| sean-k-mooney | it feel much less ok to call "oslo_utils.imageutils.format_inspector.detect_file_format" | 14:25 |
| dansmith | but we're not doing either of those here right?? | 14:25 |
| dansmith | s/??/?/ | 14:25 |
| sean-k-mooney | that what causing the qemu-image call as far as im aware | 14:25 |
| dansmith | no, it's libvirt actually creating the snapshot | 14:26 |
| dansmith | qemu-img create -f qcow2 ./tmpm1aqe2cp/25c6cab7c98f4cbba03223754b5d2e74.delta | 14:26 |
| dansmith | this is definitely not from format inspector ^ | 14:26 |
| opendevreview | Takashi Kajinami proposed openstack/nova master: Replace deprecated glanceclient exception https://review.opendev.org/c/openstack/nova/+/914761 | 14:26 |
| sean-k-mooney | oh then this is an unrelatd case wehre we alredy depended on qemu-img? | 14:26 |
| dansmith | this is all code that wasn't being tested at all before because it was being ignored since ImageMeta was missing a key and causing the test to bail super early for libvirt | 14:26 |
| sean-k-mooney | i tought this was intoduced by the cve fix | 14:27 |
| sean-k-mooney | oh right you changed it to not return NotImplemented | 14:27 |
| dansmith | sean-k-mooney: yes, I fixed a case where we were not getting the coverage we expected and now we're actually running code we weren't before, that's why I don't want to go back to not covering that | 14:27 |
| dansmith | right | 14:27 |
| jlejeune | by the removal of AMI support actually | 14:27 |
| sean-k-mooney | so its related but not directly because you fixed the missing test coverage | 14:27 |
| dansmith | exactly, the AMI thing, unrelated to anything else | 14:27 |
| sean-k-mooney | sorry | 14:27 |
| opendevreview | Takashi Kajinami proposed openstack/nova master: Replace deprecated glanceclient exception https://review.opendev.org/c/openstack/nova/+/914761 | 14:27 |
| sean-k-mooney | dansmith: with that context then yes im much more ok with your suggeed approch | 14:28 |
| dansmith | cool | 14:28 |
| dansmith | jlejeune: sorry for the runaround :) | 14:28 |
| jlejeune | no prob :) | 14:29 |
| jlejeune | let's do that ! | 14:29 |
| *** bauzas_ is now known as bauzas | 14:50 | |
| opendevreview | Julien LE JEUNE proposed openstack/nova master: Skip snapshot test when missing qemu-img https://review.opendev.org/c/openstack/nova/+/925208 | 15:12 |
| jlejeune | dansmith: sean-k-mooney: ^ | 15:14 |
| jlejeune | something like that ? | 15:14 |
| dansmith | sure, but that will fail pep8 | 15:18 |
| jlejeune | my bad | 15:18 |
| dansmith | but looks okay to me otherwise | 15:18 |
| opendevreview | Julien LE JEUNE proposed openstack/nova master: Skip snapshot test when missing qemu-img https://review.opendev.org/c/openstack/nova/+/925208 | 15:26 |
| dansmith | sean-k-mooney: I think you commented about using backslash continuations recently on another patch, not sure why you didn't on this one, but I also can't find out guideline about that | 15:33 |
| sean-k-mooney | ya i personally hate them but we are not as concistent about not doing it as i would like | 15:33 |
| dansmith | I know we have them in some places and I'm not sure I care about our previous hard and fast rule about it, other than just wanting to be consistent | 15:33 |
| sean-k-mooney | and im presonally geting very tired of style related review comments | 15:34 |
| dansmith | ah right here, second bullet: https://docs.openstack.org/hacking/latest/user/hacking.html | 15:34 |
| sean-k-mooney | yep its documetned but not enforced by tools | 15:34 |
| dansmith | jlejeune: ^ | 15:34 |
| dansmith | right | 15:34 |
| opendevreview | Julien LE JEUNE proposed openstack/nova master: Skip snapshot test when missing qemu-img https://review.opendev.org/c/openstack/nova/+/925208 | 16:02 |
| erlon | Are we having the nova meeting today? | 16:05 |
| gibi | erlon: no, it is cancelled | 16:06 |
| gibi | bauzas is off today | 16:06 |
| erlon | ah, ok, missed the announcement | 16:06 |
| sean-k-mooney | sylvain is at the opensatck asia summit event in korea currently | 16:06 |
| sean-k-mooney | erlon: did you have a pressing topic ot bring up? | 16:07 |
| gibi | yeah that is the more precise term instead of being off :) | 16:07 |
| erlon | I was to bring the shared group to attention again | 16:07 |
| erlon | we have addressed the last reviews suggestions | 16:07 |
| sean-k-mooney | shared security groups or somethign else? | 16:07 |
| erlon | shared security groups | 16:08 |
| erlon | https://review.opendev.org/c/openstack/nova/+/811521 | 16:08 |
| erlon | from this series: https://review.opendev.org/q/topic:sg-shared-filter | 16:08 |
| sean-k-mooney | ah well for that i think we will liikely jsut reapprove that for next cycle | 16:08 |
| sean-k-mooney | so once RC 1 is creeated in 2 weeks master will reopen for new work and it can proceed | 16:08 |
| *** bauzas_ is now known as bauzas | 16:09 | |
| erlon | ok, sounds good | 16:10 |
| erlon | is there a way to flag it like that so the team can approve it next? | 16:10 |
| erlon | I | 16:10 |
| sean-k-mooney | put it in the team adjenta for the next meeting and you can link to this irc converstiaon if you like | 16:12 |
| sean-k-mooney | so add it to open disucsion here https://wiki.openstack.org/wiki/Meetings/Nova#Weekly_Nova_team_meeting | 16:12 |
| sean-k-mooney | erlon: skiming the updated version it looks ok but i will need to review it properly later. it looks like you have simplifed it a little since the last version which is nice to see | 16:13 |
| haleyb | sean-k-mooney: i tried to address all the comments | 16:14 |
| sean-k-mooney | haleyb: sorry you are the most recent uploader :) yes i think you have | 16:14 |
| sean-k-mooney | i basically am getting a headache right now which is why im holding of reviewing it proeprly | 16:15 |
| haleyb | i just wanted to move that along, np | 16:15 |
| haleyb | hope you feel better | 16:15 |
| sean-k-mooney | i think its just a lack of drinking water so im goign to go remiday that | 16:16 |
| *** bauzas_ is now known as bauzas | 16:55 | |
| *** bauzas_ is now known as bauzas | 17:15 | |
| opendevreview | Merged openstack/nova stable/2024.1: libvirt: call get_capabilities() with all CPUs online https://review.opendev.org/c/openstack/nova/+/927556 | 17:22 |
| opendevreview | Merged openstack/nova stable/2024.1: Fix port group network metadata generation https://review.opendev.org/c/openstack/nova/+/923689 | 17:41 |
| opendevreview | Merged openstack/nova master: Skip snapshot test when missing qemu-img https://review.opendev.org/c/openstack/nova/+/925208 | 17:41 |
| *** bauzas_ is now known as bauzas | 18:45 | |
| *** bauzas_ is now known as bauzas | 18:57 | |
| gmann | sean-k-mooney: if you are around https://review.opendev.org/c/openstack/placement/+/926935 | 19:07 |
| sean-k-mooney | +2w | 19:27 |
| *** bauzas_ is now known as bauzas | 19:30 | |
| gmann | thanks | 19:41 |
| *** bauzas_ is now known as bauzas | 19:50 | |
| *** bauzas_ is now known as bauzas | 20:10 | |
| opendevreview | Brian Haley proposed openstack/nova master: Support creating servers with RBAC SGs https://review.opendev.org/c/openstack/nova/+/811521 | 20:54 |
| *** bauzas_ is now known as bauzas | 21:08 | |
| *** bauzas_ is now known as bauzas | 21:32 | |
| opendevreview | Merged openstack/placement master: Bump oslo.policy version to enable new RBAC by default https://review.opendev.org/c/openstack/placement/+/926935 | 21:34 |
| opendevreview | Merged openstack/nova master: Remove default override for RBAC config options https://review.opendev.org/c/openstack/nova/+/925519 | 22:07 |
| *** bauzas_ is now known as bauzas | 22:13 | |
| *** bauzas_ is now known as bauzas | 22:45 | |
| *** bauzas_ is now known as bauzas | 22:53 | |
| *** bauzas_ is now known as bauzas | 23:01 | |
| *** bauzas_ is now known as bauzas | 23:09 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!