Thursday, 2023-12-07

dpawlikmelwitt ack. Let me know if you would like to increase the retention time 07:14
opendevreviewAmit Uniyal proposed openstack/nova-specs master: Enforce console session timeout  https://review.opendev.org/c/openstack/nova-specs/+/89855307:25
*** elodilles_pto is now known as elodilles07:34
opendevreviewVasyl Saienko proposed openstack/nova master: Fix returning empty availability zone  https://review.opendev.org/c/openstack/nova/+/90287510:41
sfinucansean-k-mooney: Thanks for starting reviewing that ironic SDK migration series. Given it's a priority for the ironic folks and also relatively trivial, I wonder if I could get another "review sponsor" for the series? CC: gibi melwitt dansmith bauzas gmann10:58
*** sfinucan is now known as stephenfin10:58
stephenfinIt's been long enough since I wrote them that I could review myself at this point, but I'd rather not if I can avoid it10:59
sean-k-mooney[m]ya they are failrly simple sofar. since you rebased them yesterday it would be good to land them before they start to bit rot11:07
sean-k-mooney[m]ok done. im +2 all the way11:34
opendevreviewStephen Finucane proposed openstack/nova master: tests: Use clearer syntax for empty generator  https://review.opendev.org/c/openstack/nova/+/90307611:51
stephenfinsean-k-mooney[m]: "remove dark magic" patch11:51
stephenfin^11:51
sean-k-mooney[m]thanks11:52
opendevreviewPranali Deore proposed openstack/nova master: Support glance's new location API  https://review.opendev.org/c/openstack/nova/+/89103612:09
opendevreviewPranali Deore proposed openstack/nova master: DNM: Test glance new location api  https://review.opendev.org/c/openstack/nova/+/89120712:10
*** tobias-urdin9 is now known as tobias-urdin13:03
sean-k-mooneyhi can i get some eyes on https://review.opendev.org/c/openstack/devstack/+/890693 please.13:36
sean-k-mooneyoh that was for qa..13:37
sean-k-mooneytempest-integrated compute often seamm to get pretty close to its 2 hour timeout but there is also a lot of variance13:40
sean-k-mooneyhttps://zuul.opendev.org/t/openstack/builds?job_name=tempest-integrated-compute&project=openstack%2Fnova13:40
gibistephenfin: nice series, and now it is approved all the way up :)14:10
stephenfingibi: 👏 thank you!14:10
bauzasif people want to see my face live (after all, this is not like I see most of you everyday on gmeet) : https://www.youtube.com/live/thidlQGX29M?feature=shared15:09
bauzastl;dr: PTG recap OIL episode15:10
melwittgmann: if you get a moment https://review.opendev.org/c/openstack/nova/+/90280916:15
melwittsean-k-mooney: I have noticed TIMED_OUT too for job running just a little over 2 hours16:16
sean-k-mooneyyep16:16
sean-k-mooneyi noticed that too16:16
sean-k-mooneythats why i mentioned my zswap changes for devstack16:17
melwittI said "too" first, I saw your message in the scrollback :)16:17
sean-k-mooneysince that was ment to speed up the jobs16:17
melwittah, cool, I'll check that out16:17
sean-k-mooneyit does not seam to be that common16:17
stephenfinis it expected that adding a port will add the default security group to an instance if not already present?16:57
stephenfinI'm guessing yes assuming a port is using the default security group16:58
bauzassean-k-mooney: I had zero time today to discuss with you about my mdev live-migration spec and the ping-pong game17:04
bauzaslong story short, the ping-pong game is made when the conductor calls check_dest() https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L8501-L857217:09
bauzasif we really want to do a lockstep here by once, then we move the type check later in the post claim method, and we just return the current mdev and their types from the source in check_can_live_migrate_source() and we test and claim them in post_claim_migrate_data()17:12
bauzasI'm OK with this change17:12
*** tobias-urdin34 is now known as tobias-urdin17:27
opendevreviewMerged openstack/nova master: Use SDK for add/remove instance info from node  https://review.opendev.org/c/openstack/nova/+/65969117:30
opendevreviewMerged openstack/nova master: Use SDK for getting network metadata from node  https://review.opendev.org/c/openstack/nova/+/67021317:33
opendevreviewMerged openstack/nova master: Use SDK for volume_target.*  https://review.opendev.org/c/openstack/nova/+/86779017:35
opendevreviewMerged openstack/nova master: Use SDK for node.set_provision_state  https://review.opendev.org/c/openstack/nova/+/86779117:35
opendevreviewMerged openstack/nova master: Use SDK for node.set_power_state  https://review.opendev.org/c/openstack/nova/+/86779217:35
* bauzas drops for the day with rest.17:41
sean-k-mooneystephenfin: that is ill docuemtned buyt it depend on when the port is added18:02
gmannmelwitt: done.  i opened it but somehow i forgot to review it yesterday18:49
melwittthanks gmann !18:51
opendevreviewMerged openstack/nova master: Remove redundant setting of DEFAULT_IMAGE_NAME(_FILE)  https://review.opendev.org/c/openstack/nova/+/90273421:08
opendevreviewMerged openstack/nova master: Set UEC image vars for jobs not defined in Nova  https://review.opendev.org/c/openstack/nova/+/90280921:08
opendevreviewMerged openstack/nova master: Use SDK for node.vif*  https://review.opendev.org/c/openstack/nova/+/86779322:28

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!