Friday, 2024-06-28

*** bauzas_ is now known as bauzas05:18
*** bauzas_ is now known as bauzas05:33
*** bauzas_ is now known as bauzas08:57
*** whoami-rajat_ is now known as whoami-rajat11:25
opendevreviewAmy Marrich proposed openstack/governance master: Update to include docs and miscellaneuos repos for AC status  https://review.opendev.org/c/openstack/governance/+/91502116:47
opendevreviewAmy Marrich proposed openstack/governance master: Update to include docs and miscellaneuos repos for AC status  https://review.opendev.org/c/openstack/governance/+/91502116:48
*** bauzas_ is now known as bauzas17:17
spotz[m]Hey all https://review.opendev.org/c/openstack/governance/+/915021 is failing because it can't find tc-repos, I did a rebase thinking that would enable it to find it as that merged after this patch was put up, Any ideas?20:33
gouthamrspotz[m]: https://review.opendev.org/c/openstack/governance/+/918488/ hasn't merged yet20:33
spotz[m]gouthamr: duh!:)20:34
gouthamrspotz[m]: may be you can review and workflow it20:34
spotz[m]reviewed but no +w in that repo20:34
gouthamrohh20:35
gouthamrwoah i didn't realize this is a special superpower with the tc-chair :p thought you all were just being nice to me, letting me merge things  20:35
gouthamrspotz[m]: done; i'll post a "recheck" your change after that patch merges20:37
JayFI tossed an extra CR+1/RC+1 on 918488 as well, even if you ninja'd me on the merge ;) 20:38
gouthamr:D 20:38
spotz[m]Yeah I've gone to do a few reviews on things that popped in one of my channels to find them already merged:)20:39
opendevreviewMerged openstack/governance master: Show TC and SIG repos under reference  https://review.opendev.org/c/openstack/governance/+/91848820:46
*** bauzas_ is now known as bauzas20:55
gmannanyone would like to do 2nd review and +w on this https://review.opendev.org/c/openstack/project-team-guide/+/92197821:38
gouthamrspotz[m]: gmann: i'll be okay +2/+W'ing this change given what fungi's already added: "Clearly lots of edits will be needed to reshape this into documentation (and large chunks can probably be simply deleted), but as you say that can happen in follow-up changes and that way we get a clear delta."22:33
gouthamrgmann: did you plan on improving this? i would suggest deleting the whole history and motivation sections and make this an official "Release Cadence" doc  - (history is available in the resolution that you've linked)22:35
gouthamrwdyt?22:35
gouthamrgmann: if you prefer someone else does this, we can seek volunteers in the next weekly call22:37
fungiif memory serves (i don't have the change in front of me), the commit message also implied that was the plan22:39
gouthamrfungi: ah my bad22:39
spotz[m]That's fine there were some other parts I was like hrm doesn't read quite right but that one really bothered me for some reason:)22:41
gouthamryep; lots to edit 22:41
fungiit looked to me like the plan was 1. verbatim copy the resolution into the guide, 2. iterate on the text to make it fit into its new home22:42
fungibut at least as soon as we have the new url, we can change other references like the link on the releases site22:43
gouthamroh; if that;s the motivation, i'd fix the url to satisfy my nitpicking brain --- we're publishing facts, i'd drop "adjustment"22:44
* gouthamr has work-flowed this.. hasn't let his desire for "perfection" be an impediment to progress22:46
opendevreviewMerged openstack/project-team-guide master: Move release cadence adjustment resolution to p-t-g  https://review.opendev.org/c/openstack/project-team-guide/+/92197822:51

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