Wednesday, 2022-03-02

vishalmanchanda#startmeeting horizon15:00
opendevmeetMeeting started Wed Mar  2 15:00:17 2022 UTC and is due to finish in 60 minutes.  The chair is vishalmanchanda. Information about MeetBot at http://wiki.debian.org/MeetBot.15:00
opendevmeetUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.15:00
opendevmeetThe meeting name has been set to 'horizon'15:00
rdopierao/15:00
vishalmanchandaHello everyone15:00
e0nehi15:00
vishalmanchandae0ne: hi, I hope you are safe.15:00
noonedeadpunk+1 to ^15:00
vishalmanchandae0ne: good to see you hear.15:01
vishalmanchandahere*15:01
e0neI'm still safe and alive!15:01
vishalmanchandae0ne: Do you still in Ukraine.15:01
e0neyes15:02
e0neI'm still in Kharkiv15:02
e0nenoonedeadpunk: where are you now?15:02
noonedeadpunkoh my....15:02
noonedeadpunkIt's almost destroyed :(15:02
e0nenoonedeadpunk: not really15:02
* noonedeadpunk left it 3 weeks ago for leisure trip15:03
vishalmanchandaI'll pray all these War thing end soon.15:03
vishalmanchandaok let's start the meeting.15:04
noonedeadpunkstay safe and keep care!15:04
vishalmanchandaWe not have much to discuss for this week as I can see our weekly agenda etherpad.15:05
vishalmanchandahttps://etherpad.opendev.org/p/horizon-release-priorities (at Line 32)15:05
vishalmanchandaI was on PTO this week, joined today.15:06
vishalmanchanda#topic Notices15:06
vishalmanchandaThis week is R3 week15:06
vishalmanchandaSorry R4 week https://releases.openstack.org/yoga/schedule.html15:07
vishalmanchandaMost of the open bp patches is merged for Yoga cycle.15:08
vishalmanchandaOnly this one left https://review.opendev.org/c/openstack/horizon/+/81757015:08
vishalmanchandaWhile reviewing  amotoki  raised a concern on this patch https://review.opendev.org/c/openstack/horizon/+/817570/6..11/horizon/middleware/simultaneous_sessions.py#b5215:09
* ganso sneaks in late15:11
vishalmanchandaI can see owner of the patch is agree to fix above comment in a follow-up patch.15:11
vishalmanchandaSo I am fine with merging this patch, please let me know if anyone see any other issue with this patch.15:12
vishalmanchandahttps://review.opendev.org/c/openstack/horizon/+/817570/15:12
amotokiI am okay with it. I just read a reply from the author.15:12
vishalmanchandaamotoki: ack.15:12
amotokiI updated the patch not to add a new setting to HORIZON_CONFIG.15:13
vishalmanchandawill review the latest P.S. after this meeting.15:13
vishalmanchandaamotoki: thanks:)15:13
amotokiit now uses settings.SIMULATANEOUS_SESSIONS directly.15:13
amotokiso we don't need to set the default everywhre :)15:14
vishalmanchandamoving to next announcement15:14
vishalmanchandaJust a reminder for adding topics for Z cycle PTG15:14
vishalmanchanda#link https://etherpad.opendev.org/p/horizon-zed-ptg15:14
vishalmanchandaDon't forget to Signup for PTG to get all PTG related updates.15:15
vishalmanchandathat's all announcement from my side for this week.15:16
vishalmanchandaIf anyone else have any announcement to make, please go ahead15:16
vishalmanchandamoving to next topic15:17
vishalmanchanda#topic On-Demand Agenda15:17
vishalmanchandathere is a request for patch review.15:18
vishalmanchandahttps://review.opendev.org/c/openstack/horizon/+/829662/15:18
gansoo/15:18
gansoI added that item to the agenda. Considering there is a +2 and several +1, mostly to raise awareness15:19
vishalmanchandaamotoki: I can see your comments on P.S. 415:19
vishalmanchandaganso: thanks15:19
vishalmanchandaamotoki: Can we squash https://review.opendev.org/c/openstack/horizon/+/830630 with https://review.opendev.org/c/openstack/horizon/+/829662/15:21
amotokivishalmanchanda: flavor_get API is not a costy call, so I am not worried so much15:21
vishalmanchandaafter https://review.opendev.org/c/openstack/horizon/+/830630 merged in master branch15:22
amotokivishalmanchanda: I am okay with either way but the master patch has not landed yet.15:22
amotokivishalmanchanda: I think we can squash it.15:22
vishalmanchandaamotoki: ok great.15:22
vishalmanchandaamotoki: will review https://review.opendev.org/c/openstack/horizon/+/830630 soon15:23
gansook so we will propose new backports that squash both commits and discard the existing ones. Correct?15:23
amotokiganso: the plan is to update the current backport to include the follow-up commit porposed in the mater.15:24
vishalmanchanda++15:24
amotokiganso: I can take care of updating it once it lands.15:24
gansoamotoki: thanks!15:24
vishalmanchandaDoes anyone have any other topic to discuss?15:25
amotokiI have review requests on https://review.opendev.org/q/topic:django40-warnings15:25
amotokiespecially (2) and (3) ones.15:25
e0neamotoki: will review them15:25
amotokithe warnings are too noisy when checking test results.15:26
vishalmanchandae0ne: thanks..15:26
amotokie0ne: thanks15:26
e0nenp15:26
vishalmanchandaok if nothing else to discuss we can end this meeting.15:27
vishalmanchandaThanks everyone for joining.15:27
vishalmanchandaSee you next week.15:27
amotokio/15:27
vishalmanchanda#endmeeting15:28
opendevmeetMeeting ended Wed Mar  2 15:28:03 2022 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)15:28
opendevmeetMinutes:        https://meetings.opendev.org/meetings/horizon/2022/horizon.2022-03-02-15.00.html15:28
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/horizon/2022/horizon.2022-03-02-15.00.txt15:28
opendevmeetLog:            https://meetings.opendev.org/meetings/horizon/2022/horizon.2022-03-02-15.00.log.html15:28
e0nesee you!=15:28
vishalmanchandae0ne: Stay safe and TC.15:28

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