Wednesday, 2021-03-10

*** vishalmanchanda has quit IRC00:05
gmannttx: can you check this tempest release patch , much required by tempest plugins - https://review.opendev.org/c/openstack/releases/+/77908800:08
*** bdodd has joined #openstack-release00:14
*** brinzhang has joined #openstack-release00:22
*** tosky has quit IRC00:36
*** armstrong has quit IRC00:39
*** iurygregory_ has joined #openstack-release01:30
*** iurygregory has quit IRC01:38
*** ttx has quit IRC01:38
*** ttx has joined #openstack-release01:47
*** ykarel has joined #openstack-release04:16
*** vishalmanchanda has joined #openstack-release04:59
*** evrardjp has quit IRC05:33
*** evrardjp has joined #openstack-release05:33
openstackgerritAbhishek Kekane proposed openstack/releases master: Release final python-glanceclient for wallaby  https://review.opendev.org/c/openstack/releases/+/77965306:01
*** whoami-rajat has joined #openstack-release06:23
openstackgerritAbhishek Kekane proposed openstack/releases master: Release final python-glanceclient for wallaby  https://review.opendev.org/c/openstack/releases/+/77965306:25
*** e0ne has joined #openstack-release06:31
*** e0ne has quit IRC06:31
*** e0ne has joined #openstack-release07:22
*** jbadiapa has joined #openstack-release07:23
*** e0ne has quit IRC07:27
*** slaweq has quit IRC07:42
*** eolivare has joined #openstack-release07:46
*** slaweq has joined #openstack-release07:46
*** slaweq has quit IRC07:52
*** sboyron has joined #openstack-release07:53
*** slaweq has joined #openstack-release07:54
*** brinzhang_ has joined #openstack-release08:20
*** brinzhang has quit IRC08:23
*** rpittau|afk is now known as rpittau08:27
*** tosky has joined #openstack-release08:46
*** bdodd has quit IRC08:57
ttxgmann: done09:10
openstackgerritMerged openstack/releases master: Release Tempest 26.1.0  https://review.opendev.org/c/openstack/releases/+/77908809:23
*** ykarel is now known as ykarel|lunch10:00
*** e0ne has joined #openstack-release10:35
*** dtantsur|afk is now known as dtantsur11:07
*** bdodd has joined #openstack-release11:07
*** ykarel|lunch is now known as ykarel11:18
*** iurygregory_ is now known as iurygregory11:33
*** eolivare_ has joined #openstack-release12:16
*** ykarel_ has joined #openstack-release12:17
*** eolivare has quit IRC12:19
*** ykarel has quit IRC12:20
*** nicolasbock has quit IRC13:11
*** nicolasbock has joined #openstack-release13:11
*** ykarel_ is now known as ykarel13:15
*** brinzhang_ has quit IRC13:23
*** brinzhang_ has joined #openstack-release13:24
*** nweinber has joined #openstack-release13:50
openstackgerrityatin proposed openstack/releases master: Create ovb 2.0 release and cut stable/2.0 branch  https://review.opendev.org/c/openstack/releases/+/77928214:04
*** jrollen is now known as jroll14:13
gmannttx: thanks14:21
*** armstrong has joined #openstack-release14:25
armstrongHello @hberaud14:31
hberaudHey14:31
hberaudarmstrong: I seen your messages14:31
hberaudThe output LGTM14:32
hberaudlemme give you an example of email14:32
armstrongOK what about the email? I don't have the email address to send the messages14:32
hberaudarmstrong: http://lists.openstack.org/pipermail/openstack-discuss/2020-September/017147.html14:35
hberaudyou simply need to send the email at the ML => openstack-discuss@lists.openstack.org14:35
hberaudHowever you should ignore first few results14:36
armstrongok great!14:36
hberaudEspecially => release-test14:36
armstrongok14:36
hberaudThis is our testing project so we don't need to warn about it14:36
armstronghahaha I see14:36
hberaudall the rest looks legit14:37
hberaudEither you copy diablo_rojo's previous email ^ or you use the given template (https://releases.openstack.org/reference/process.html#r-5-week-milestone-3)14:38
hberaudarmstrong: don't forget to replace the placeholder with the right values (series name, date, etc...)14:39
openstackgerritMerged openstack/releases master: Update list_eol_stale_branches.sh documentation  https://review.opendev.org/c/openstack/releases/+/77867314:39
armstrong@hberaud  ok14:40
armstrongI will do it and show you before sending14:40
hberaudFeel free to use an etherpad to allow review first (if you want)14:41
armstrongok14:41
*** fnordahl has quit IRC14:42
openstackgerritMerged openstack/releases master: Release networking-ovn for stable/train  https://review.opendev.org/c/openstack/releases/+/77307014:42
openstackgerritMerged openstack/releases master: Release final python-swiftclient for Wallaby  https://review.opendev.org/c/openstack/releases/+/77897014:43
whoami-rajathberaud: hi, I've a question regarding your comment here https://review.opendev.org/c/openstack/releases/+/779199/3/deliverables/ussuri/os-brick.yaml#3415:04
whoami-rajat3.1.0 is used by victoria https://github.com/openstack/releases/blob/master/deliverables/victoria/os-brick.yaml#L1015:04
whoami-rajatso a minor version bump isn't possible15:04
hberaudwhoami-rajat: Ah yes sorry15:05
hberaudmy bad15:05
hberaudI misread victoria versions15:05
whoami-rajatnp, I've also mentioned this in commit msg15:06
whoami-rajatto avoid confusion15:06
hberaudI missed that point too15:06
hberaudsorry15:06
hberaudwhoami-rajat: thanks for the effort on details15:07
whoami-rajatnp, thanks for your review :)15:08
openstackgerritSlawek Kaplonski proposed openstack/releases master: Add Neutron Wallaby highlights  https://review.opendev.org/c/openstack/releases/+/77948715:32
openstackgerritHervé Beraud proposed openstack/releases master: Release final python-monascaclient for Wallaby  https://review.opendev.org/c/openstack/releases/+/77896615:35
openstackgerritHervé Beraud proposed openstack/releases master: Release final python-octaviaclient for Wallaby  https://review.opendev.org/c/openstack/releases/+/77896715:37
armstrong@hberaud   here is the draft: https://etherpad.opendev.org/p/armstrong15:39
openstackgerritHervé Beraud proposed openstack/releases master: Release final python-cinderclient for Wallaby  https://review.opendev.org/c/openstack/releases/+/77896115:39
hberaudack15:39
* hberaud join15:39
armstrong@hberaud  I removed the release-test only15:40
hberaudgood15:40
hberaudYou missed the rc1 date15:40
hberaudarmstrong: https://releases.openstack.org/wallaby/schedule.html#w-rc115:41
armstrongYes, I was about to ask that15:41
hberaudI fixed the missed elements15:42
armstrongOK thanks15:42
hberaudarmstrong: LGTM now15:42
armstrongsending15:43
hberaudDon't forget to tag the mail by using "[release]" in the topics15:44
hberauds/topics/topic15:44
johnsomhberaud Sorry for the confusion over the python-octaviaclient release. lol I have abandoned my proposed patch and +1 yours.15:44
hberaudarmstrong: and don't forget to add all the concerned team to the subject as did in the given example ^15:45
hberaudjohnsom: np15:45
armstrongYes, doing it15:45
hberaudexcellent15:45
hberaudthis help us to filter ML content by team15:46
openstackgerritMichael Johnson proposed openstack/releases master: Release final python-octaviaclient for Wallaby  https://review.opendev.org/c/openstack/releases/+/77896715:46
hberaudif a concerned team is present in the subject of an email then we will have a look more closely15:46
hberaudjohnsom: thanks for the update15:47
hberaudI'll have look15:47
johnsomYeah, just advanced it to the SHA from the duplicate patch15:47
hberaudack np15:47
armstrong@hberaud look at: [release][heat][ironic][requirements][swift][python-openstackclient] Cycle With Intermediary Unreleased Deliverables15:48
armstronghberaud: I don't know if the python-openstackclient should just be ==> python?15:48
hberaudarmstrong: s/python-openstackclient/OpenStackSDK/15:49
armstrongok thanks [OpenStackSDK]15:49
hberaudarmstrong: [release][heat][ironic][requirements][swift][OpenStackSDK] Cycle With Intermediary Unreleased Deliverables15:49
armstrongYea, this is what I got now15:50
armstrongthx hberaud15:50
armstrongsending now15:51
hberaudarmstrong: thanks15:54
armstrongnp15:55
armstrongare you still doing review?15:55
hberaudWhere?16:02
hberaudOn the etherpad or generally?16:02
armstrong your review day16:04
hberaudYes I do16:04
armstronghberaud: asking as a follow-up of what we discussed on Monday16:04
armstrongI wanted to join16:05
hberaudarmstrong: here is our review inbox => https://releases.openstack.org/reference/reviewer_guide.html#review-inbox16:05
hberaudarmstrong: patches are triaged by series (wallaby, victoria, etc...) and by type of patch16:06
hberaudarmstrong: for you it could be worth to pick a patch in the "Wallaby" section16:07
hberaudarmstrong: by example the deadline for those (https://review.opendev.org/q/topic:"cwi-client-final"+(status:open)) is tomorrow16:07
hberaudarmstrong: you can start by example with this one => https://review.opendev.org/c/openstack/releases/+/77896716:08
hberaudand by using our reviewer guide (https://releases.openstack.org/reference/reviewer_guide.html) you can leave comments if you think that the patch is ok or not16:09
armstronghberaud:  OK16:10
hberaudarmstrong: This kind of patch is based on our series' calendar16:11
armstronghberaud:  I see16:11
hberaudThis one correspond to our collaboration of the previous Friday16:12
armstronghberaud: OK16:12
hberaudzuul give you all the details about the asked release(s)16:13
hberaudif you jump into zuul's logs (especially `releases-tox-list-changes`) you can harvest all the needed details in one shot16:14
hberaud(somewhere in the middle of the logs)16:14
hberaudby example for this patch all the important informations are there => https://zuul.opendev.org/t/openstack/build/3db7d393173a4a07a7d9b40965265711/log/job-output.txt#723-122116:15
hberaudbetween line 723 and line 122116:15
armstrongsounds good16:17
armstronghberaud:  Looking through16:17
hberaudSo accordingly to our guide you should get all the needed info in these logs16:17
armstrongnow I get it16:18
hberaudIf you've some doubt about a specific point then you shouldn't hesitate to jump directly into the project repo and take a look there too16:19
hberaudarmstrong: did you send the email?16:19
hberaudI didn't receive it16:19
armstronghberaud:  Yes, I did and got a response from the moderator:16:21
armstronghttps://www.irccloud.com/pastebin/9FZCZlVH/16:21
hberaudOh ok16:21
hberaudthanks16:21
hberaudI think you need/should to subscribe to the ML first16:22
armstrongI just did16:23
hberaudnp :)16:23
armstronghberaud: I think fungi  will look at it soon :)16:24
fungii've approved it now16:26
armstrongfungi:  OK thanks16:27
fungimy pleasure16:27
hberaudarmstrong: I missed that the gate hadn't finished16:29
hberaudthe right logs are there => https://zuul.opendev.org/t/openstack/build/02863eeae47f4f8b881452db4b0f90a316:30
hberaudmodulo the latest patch set available16:30
hberaudemail received thanks armstrong16:31
armstrongnp16:31
armstronghberaud:  so the log file you sent earlier isn't good enough?16:32
hberaudyes16:33
hberaudit was not based on the right revision16:33
hberaudthe PTL updated the SHA16:33
hberaudso first remark, the commit message is outdated because the git log aren't the same16:34
hberaudthat's a details but if reviewers don't spend time to have a look to the logs then they will miss commits16:35
hberaudcommit list should be based on `git log --oneline --no-merges 2.2.0..e7922ac5`16:36
hberaudand not on `6dffa41`16:36
armstronghberaud:  interesting here16:37
openstackgerritMatthew Thode proposed openstack/releases master: add release for openstack requirements  https://review.opendev.org/c/openstack/releases/+/77983216:45
prometheanfirethink I did that right...16:46
armstronghberaud:  so the correct version of the file start from line 1052: https://zuul.opendev.org/t/openstack/build/02863eeae47f4f8b881452db4b0f90a3/log/job-output.txt#1052-157716:47
hberaudarmstrong: yes16:48
armstrongto leave a review I can just  add myself (ADD REVIEWER ) right?16:53
armstronghberaud:  i mean once I am done analyzing the log16:54
*** rpittau is now known as rpittau|afk17:05
*** dtantsur is now known as dtantsur|afk17:15
*** ykarel has quit IRC17:50
hberaudarmstrong: yes, you should own a gerrit/launchpad account and then you can send your ballot and send comments17:54
hberaudprometheanfire: requirements is tagless18:01
hberaudI think we only need a branching here18:01
prometheanfirehberaud: reqs branches last, we are generally tagless it's true18:02
armstronghberaud:  given that the commit message is outdated, should that  be a -1 ?18:15
*** eolivare_ has quit IRC18:36
*** tbarron has joined #openstack-release18:37
*** e0ne has quit IRC18:48
openstackgerritMaysa de Macedo Souza proposed openstack/releases master: Add Kuryr Wallaby highlights  https://review.opendev.org/c/openstack/releases/+/77984818:53
openstackgerritMaysa de Macedo Souza proposed openstack/releases master: Add Kuryr Wallaby highlights  https://review.opendev.org/c/openstack/releases/+/77984819:02
openstackgerritMaysa de Macedo Souza proposed openstack/releases master: Add Kuryr Wallaby highlights  https://review.opendev.org/c/openstack/releases/+/77984819:03
hberaudarmstrong: changes LGTM (without take account of the commit message)19:24
armstrongok good to go19:25
armstronghberaud:  any reason to mention the commit message? OR just a +119:28
hberaudarmstrong: Could be worth to notice that point19:29
armstrongok19:29
hberaudJust in case where someone isn't diligent19:30
*** brinzhang0 has joined #openstack-release19:33
*** mugsie__ has joined #openstack-release19:37
*** tosky_ has joined #openstack-release19:37
*** whoami-rajat has quit IRC19:40
*** brinzhang_ has quit IRC19:42
*** tosky has quit IRC19:42
*** mugsie has quit IRC19:42
*** irclogbot_2 has quit IRC19:44
*** irclogbot_3 has quit IRC19:44
*** mugsie__ is now known as mugsie19:47
openstackgerritMichael Johnson proposed openstack/releases master: Add Wallaby release highlights for Octavia  https://review.opendev.org/c/openstack/releases/+/77938120:13
*** tosky_ is now known as tosky20:16
*** tosky has quit IRC20:49
*** tosky has joined #openstack-release20:49
*** nweinber has quit IRC21:30
*** flaviof has quit IRC21:31
*** flaviof has joined #openstack-release21:31
*** vdrok has quit IRC21:37
*** vdrok has joined #openstack-release21:37
*** clarkb has joined #openstack-release21:39
clarkbhello openstack release team. It appears the release schedule says feature freeze ends friday. Is it still actually thursday as in old releases or do we do things through friday now?21:40
clarkbasking because we'd like to restart zuul in the near future (like this week) and were thinking friday would be a good time for that21:40
*** TheJulia has quit IRC21:45
*** TheJulia has joined #openstack-release21:45
*** johnsom has quit IRC21:46
*** johnsom has joined #openstack-release21:46
*** slaweq has quit IRC21:55
*** masayukig has quit IRC21:58
*** masayukig has joined #openstack-release21:58
*** vishalmanchanda has quit IRC22:08
*** armstrong has quit IRC22:23
*** armstrong has joined #openstack-release22:24
*** sboyron has quit IRC22:56
*** rpittau|afk has quit IRC23:37
*** rpittau|afk has joined #openstack-release23:37

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