)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"426bd481b6004b689552fb96563daad1f54fdcf8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"86b1342a_95e4d562","updated":"2024-04-30 05:24:46.000000000","message":"Hi James,\n\nDo you intend to update this change?","commit_id":"55ba00ab681dc544082f5eae930cd29356b4db82"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"c66d3f4109dd5086be68d763e94c86512cb10912","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"c2c0680c_271bef6c","updated":"2023-12-14 00:16:54.000000000","message":"I think we need to follow the repo retirement complete process here which move them to legacy file and do rest of other infra changes etc if needed https://docs.openstack.org/project-team-guide/repository.html#retiring-a-repository","commit_id":"55ba00ab681dc544082f5eae930cd29356b4db82"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"ecc906c16d62f6485d7dbae8307a62b586a4df8a","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"3d243f2b_83d796e5","updated":"2023-12-13 23:53:27.000000000","message":"I\u0027m not familiar with charms in general; but will this pose a problem for operators who might be consuming them directly from git? Does that even make sense for charms?","commit_id":"55ba00ab681dc544082f5eae930cd29356b4db82"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"1e59fb038891c6c3c6c4e067241ad1c43b323cd5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"207c4c06_5ec6d8ca","updated":"2023-12-20 09:06:43.000000000","message":"please add links to the reviews for the next steps once you have them, but I don\u0027t think that those should block this patch","commit_id":"55ba00ab681dc544082f5eae930cd29356b4db82"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"39e3c82771b15b2c7a8f9c4e97818c774d4c86e9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"8d62a850_e0106234","in_reply_to":"207c4c06_5ec6d8ca","updated":"2023-12-20 19:56:18.000000000","message":"There is one part pending in this step. We need to update the legacy file also for the retired repo. Details here - https://docs.openstack.org/project-team-guide/repository.html#step-3-retire-repository-from-the-governance-repository","commit_id":"55ba00ab681dc544082f5eae930cd29356b4db82"},{"author":{"_account_id":935,"name":"James Page","email":"ringo.page@gmail.com","username":"james-page"},"change_message_id":"f7d75c13ad2b5dd5e127f6a8bee8ed419dfcf0df","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"f6546aab_16d79331","in_reply_to":"3d243f2b_83d796e5","updated":"2023-12-14 08:49:21.000000000","message":"Operators would typically consume the built and published charms in the downstream charmhub - even if they are directly using the single charm repos, its fairly trivial to switch to the mono repo as the source for charms.","commit_id":"55ba00ab681dc544082f5eae930cd29356b4db82"},{"author":{"_account_id":935,"name":"James Page","email":"ringo.page@gmail.com","username":"james-page"},"change_message_id":"c8f90ed153ce3c0fa3c31337a08a00ee21124fba","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"7cdd8b0d_be766664","in_reply_to":"72e3413a_253db63a","updated":"2024-04-30 13:46:09.000000000","message":"Done","commit_id":"55ba00ab681dc544082f5eae930cd29356b4db82"},{"author":{"_account_id":935,"name":"James Page","email":"ringo.page@gmail.com","username":"james-page"},"change_message_id":"1d3ecc6ee8909bff3d5b50218036dd0c8c0b4191","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"6d0f025b_f5fffcee","in_reply_to":"86b1342a_95e4d562","updated":"2024-04-30 09:47:34.000000000","message":"I do yes!","commit_id":"55ba00ab681dc544082f5eae930cd29356b4db82"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"664e8106d8546d040ab3ee196fb919f33d55ef74","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"72e3413a_253db63a","in_reply_to":"8d62a850_e0106234","updated":"2023-12-21 09:42:35.000000000","message":"Oops, I missed that, ack.","commit_id":"55ba00ab681dc544082f5eae930cd29356b4db82"},{"author":{"_account_id":935,"name":"James Page","email":"ringo.page@gmail.com","username":"james-page"},"change_message_id":"f7d75c13ad2b5dd5e127f6a8bee8ed419dfcf0df","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"9a68596f_3498c8fe","in_reply_to":"c2c0680c_271bef6c","updated":"2023-12-14 08:49:21.000000000","message":"Yep - we\u0027ve just complete step 2 (https://review.opendev.org/q/topic:%22sunbeam-repo-retirements%22) and I\u0027m endeavouring to get the final step raised for review today.","commit_id":"55ba00ab681dc544082f5eae930cd29356b4db82"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"c29ee80c3c99749867daa731980446dcdfcdd125","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"50d69b38_e5a27ee8","updated":"2024-04-30 17:13:45.000000000","message":"-1 for changing the validation script otherwise lgtm","commit_id":"08fe64c98d94173b26a128a03ce66404f1988327"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"ebe62644d3aa6c75044ed57bfe1b99d733a11b32","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"5ddc6978_e2b97776","updated":"2024-09-10 22:42:46.000000000","message":"actually; +1 only on the retirement; the script could still need a minor fix","commit_id":"331144012f45f114bba73fe4834baff2ed43ecc3"},{"author":{"_account_id":935,"name":"James Page","email":"ringo.page@gmail.com","username":"james-page"},"change_message_id":"358941c7dcb6a37880a03858193d0e39b3387bed","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"61807b82_b2da75b8","updated":"2024-08-15 08:48:32.000000000","message":"recheck","commit_id":"331144012f45f114bba73fe4834baff2ed43ecc3"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"34c951c14fa45c8e1105ffa32a1ddff9d5649ae1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"91447928_afbbca11","updated":"2024-08-15 06:18:40.000000000","message":"recheck not sure if all dependencies have been merged, but it seems there has been some progress at least","commit_id":"331144012f45f114bba73fe4834baff2ed43ecc3"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"93901ba3bad2fc30076f431613e5182a5ccebc97","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"c06a104c_e474d758","updated":"2024-09-11 16:32:59.000000000","message":"I am in favor of retirement but not in favor of allowing non master brnach name in tooling. \n\nIf you add these charms branch in Ignore-repo list and no other modification in script then i am ok.","commit_id":"c2abd8086335fb842d6dcae433b72f6fd6fea043"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"51abb39b2d75be95310b2be25108dad51dc7cbcb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"7303e288_df00b34b","updated":"2024-09-29 16:46:52.000000000","message":"This has a sufficient number of votes and a good amount of soak time; lets get this in.. \n\nThe discussion here veered off into a tangential aspect regarding the name of the development branch in the repositories being retired.. \n\nWe\u0027ll be discussing development branch naming at the upcoming PTG; In general, the community has expressed support for inclusive language; although no action has been taken by the TC with regard to the name of the development branch so far - we have maintained status quo where most OpenStack repositories have a \"master\" branch. \n\nIt is possible to specify a name for a development branch with OpenDev\u0027s tooling; which allowed charm repositories (amongst others) to retain a branch name called \"main\" (certainly a better accepted name for a development branch). Some tools such as codesearch.opendev.org are yet to catch up to this custom naming. If/when the TC decides a course of action; we\u0027ll try to round up the issues and address them.","commit_id":"c2abd8086335fb842d6dcae433b72f6fd6fea043"}],"tools/validate-legacy.py":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"c29ee80c3c99749867daa731980446dcdfcdd125","unresolved":true,"context_lines":[{"line_number":81,"context_line":"                    pass"},{"line_number":82,"context_line":""},{"line_number":83,"context_line":"            for file in file_names:"},{"line_number":84,"context_line":"                if file not in (\u0027.gitreview\u0027, \u0027.zuul.yaml\u0027, \u0027README.rst\u0027):"},{"line_number":85,"context_line":"                    msg \u003d \u0027{} is not properly retired, files: {}.\u0027.format("},{"line_number":86,"context_line":"                        repo, file_names)"},{"line_number":87,"context_line":"                    print(msg)"}],"source_content_type":"text/x-python","patch_set":4,"id":"68c03943_b1304506","line":84,"range":{"start_line":84,"start_character":46,"end_line":84,"end_character":58},"updated":"2024-04-30 17:13:45.000000000","message":"I do not think we need to keep zuul file for retired repo. Only two file .gitreview and README.rst should exist.","commit_id":"08fe64c98d94173b26a128a03ce66404f1988327"},{"author":{"_account_id":12898,"name":"Tony Breeds","email":"tony@bakeyournoodle.com","username":"tonyb"},"change_message_id":"e24470bd98161f60099af9c9db09cc8b029c5962","unresolved":true,"context_lines":[{"line_number":81,"context_line":"                    pass"},{"line_number":82,"context_line":""},{"line_number":83,"context_line":"            for file in file_names:"},{"line_number":84,"context_line":"                if file not in (\u0027.gitreview\u0027, \u0027.zuul.yaml\u0027, \u0027README.rst\u0027):"},{"line_number":85,"context_line":"                    msg \u003d \u0027{} is not properly retired, files: {}.\u0027.format("},{"line_number":86,"context_line":"                        repo, file_names)"},{"line_number":87,"context_line":"                    print(msg)"}],"source_content_type":"text/x-python","patch_set":4,"id":"ffcfc2da_a9b91b88","line":84,"range":{"start_line":84,"start_character":46,"end_line":84,"end_character":58},"in_reply_to":"5eb0200b_9f8a32f3","updated":"2024-05-03 20:21:02.000000000","message":"@James, FWIW, when you remove .zuul.yaml you\u0027ll also need to update the README to point to `HEAD^2` rather than `HEAD^1`  As you\u0027ll now have 2 changes to get back to the code.","commit_id":"08fe64c98d94173b26a128a03ce66404f1988327"},{"author":{"_account_id":935,"name":"James Page","email":"ringo.page@gmail.com","username":"james-page"},"change_message_id":"39cc0a94975754a21929896bcaabd0ebf3dcb79c","unresolved":true,"context_lines":[{"line_number":81,"context_line":"                    pass"},{"line_number":82,"context_line":""},{"line_number":83,"context_line":"            for file in file_names:"},{"line_number":84,"context_line":"                if file not in (\u0027.gitreview\u0027, \u0027.zuul.yaml\u0027, \u0027README.rst\u0027):"},{"line_number":85,"context_line":"                    msg \u003d \u0027{} is not properly retired, files: {}.\u0027.format("},{"line_number":86,"context_line":"                        repo, file_names)"},{"line_number":87,"context_line":"                    print(msg)"}],"source_content_type":"text/x-python","patch_set":4,"id":"ec5a8f1b_5b2046b3","line":84,"range":{"start_line":84,"start_character":46,"end_line":84,"end_character":58},"in_reply_to":"68c03943_b1304506","updated":"2024-05-01 06:13:59.000000000","message":"I\u0027ve raised a project-config review to enable noop jobs for all of the impacted reviews, rather than leaving the ,zuul.yaml in the github repo for each - once that has been reviewed and landed I\u0027ll do another round of updates to remove the .zuul.yaml at which point I\u0027ll refresh this review to deal with the master/main branch differences only.","commit_id":"08fe64c98d94173b26a128a03ce66404f1988327"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"4fa2bf62384f231e6ccdad9efc3aed7015dbe3de","unresolved":true,"context_lines":[{"line_number":81,"context_line":"                    pass"},{"line_number":82,"context_line":""},{"line_number":83,"context_line":"            for file in file_names:"},{"line_number":84,"context_line":"                if file not in (\u0027.gitreview\u0027, \u0027.zuul.yaml\u0027, \u0027README.rst\u0027):"},{"line_number":85,"context_line":"                    msg \u003d \u0027{} is not properly retired, files: {}.\u0027.format("},{"line_number":86,"context_line":"                        repo, file_names)"},{"line_number":87,"context_line":"                    print(msg)"}],"source_content_type":"text/x-python","patch_set":4,"id":"5eb0200b_9f8a32f3","line":84,"range":{"start_line":84,"start_character":46,"end_line":84,"end_character":58},"in_reply_to":"ec5a8f1b_5b2046b3","updated":"2024-05-01 14:56:57.000000000","message":"Thanks. yeah that is the first step in repo retirement, once you do that you can remove the repo content as next step and then this governance change will pass without this script change\n\n- https://docs.openstack.org/project-team-guide/repository.html#step-1-end-project-gating","commit_id":"08fe64c98d94173b26a128a03ce66404f1988327"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"f98821cc411e3cf3bf4c687e5ab183462724279a","unresolved":true,"context_lines":[{"line_number":38,"context_line":"    \u0027openstack/python-magnetodbclient\u0027,"},{"line_number":39,"context_line":"    \u0027openstack/refstack\u0027,"},{"line_number":40,"context_line":"    \u0027openstack/salt\u0027,"},{"line_number":41,"context_line":"    \u0027openstack/security-doc\u0027]"},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"parser \u003d argparse.ArgumentParser()"},{"line_number":44,"context_line":"parser.add_argument("}],"source_content_type":"text/x-python","patch_set":5,"id":"3e9d95ce_7b3504e2","line":41,"range":{"start_line":41,"start_character":27,"end_line":41,"end_character":29},"updated":"2024-08-15 17:36:16.000000000","message":"as charms are already have main branch and not master so it is possible to solve it there so my suggestion is to add these charms repo in IGNORED_REPOS list here but not allow \u0027main\u0027 branch handling in this script","commit_id":"331144012f45f114bba73fe4834baff2ed43ecc3"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"dfc296b4f13f8f580f92f2cfb6c4b5228e6506ad","unresolved":true,"context_lines":[{"line_number":38,"context_line":"    \u0027openstack/python-magnetodbclient\u0027,"},{"line_number":39,"context_line":"    \u0027openstack/refstack\u0027,"},{"line_number":40,"context_line":"    \u0027openstack/salt\u0027,"},{"line_number":41,"context_line":"    \u0027openstack/security-doc\u0027]"},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"parser \u003d argparse.ArgumentParser()"},{"line_number":44,"context_line":"parser.add_argument("}],"source_content_type":"text/x-python","patch_set":5,"id":"f9f64af8_b858f706","line":41,"range":{"start_line":41,"start_character":27,"end_line":41,"end_character":29},"in_reply_to":"3e9d95ce_7b3504e2","updated":"2024-09-10 22:36:54.000000000","message":"These ignored repos were where we\u0027ve left content behind though based on the comment above.. we could make another set of ignored repos, or improve the comment.. \n\n\nbut for future proofing, i think we can let the script handle \"master\" and \"main\" branches as done below instead.","commit_id":"331144012f45f114bba73fe4834baff2ed43ecc3"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"93901ba3bad2fc30076f431613e5182a5ccebc97","unresolved":true,"context_lines":[{"line_number":38,"context_line":"    \u0027openstack/python-magnetodbclient\u0027,"},{"line_number":39,"context_line":"    \u0027openstack/refstack\u0027,"},{"line_number":40,"context_line":"    \u0027openstack/salt\u0027,"},{"line_number":41,"context_line":"    \u0027openstack/security-doc\u0027]"},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"parser \u003d argparse.ArgumentParser()"},{"line_number":44,"context_line":"parser.add_argument("}],"source_content_type":"text/x-python","patch_set":5,"id":"c5c7df08_8a841b51","line":41,"range":{"start_line":41,"start_character":27,"end_line":41,"end_character":29},"in_reply_to":"f9f64af8_b858f706","updated":"2024-09-11 16:32:59.000000000","message":"Sure, we can have another list, but I am not in favor of extending tooling to \u0027main\u0027 or any other name of branches. Do we have a policy somewhere where OpenStack allows branch names to be anything other than the master? Branch names are mainly defined in the opendev, I think, and if we mix naming in OpenStack, it will be difficult to handle in tooling.","commit_id":"331144012f45f114bba73fe4834baff2ed43ecc3"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"f98821cc411e3cf3bf4c687e5ab183462724279a","unresolved":true,"context_lines":[{"line_number":105,"context_line":"                print(msg)"},{"line_number":106,"context_line":"                continue"},{"line_number":107,"context_line":""},{"line_number":108,"context_line":"            for branch in (\u0027master\u0027, \u0027main\u0027):"},{"line_number":109,"context_line":"                try:"},{"line_number":110,"context_line":"                    url \u003d FILES_URL.format(repo, branch)"},{"line_number":111,"context_line":"                    files \u003d requests.get(url).json()"}],"source_content_type":"text/x-python","patch_set":5,"id":"17fe6fe7_c095919b","line":108,"range":{"start_line":108,"start_character":37,"end_line":108,"end_character":45},"updated":"2024-08-15 17:36:16.000000000","message":"I am not sure we need to handle the different branch name other than master as that is what openstack namespace should have.\n\nI saw all the retired charms repo has the main branch instead of master. Any specific reason on that? We should not allow the mixture of master and main until we decide to go for \u0027master\u0027 to \u0027main\u0027 transition but that is a big and separate discussion.\n\nAs these are retired repo, I am less concern but until then we did not notice these \u0027main\u0027 branch in OpenStack namespace which is another things to solve.","commit_id":"331144012f45f114bba73fe4834baff2ed43ecc3"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"dfc296b4f13f8f580f92f2cfb6c4b5228e6506ad","unresolved":true,"context_lines":[{"line_number":105,"context_line":"                print(msg)"},{"line_number":106,"context_line":"                continue"},{"line_number":107,"context_line":""},{"line_number":108,"context_line":"            for branch in (\u0027master\u0027, \u0027main\u0027):"},{"line_number":109,"context_line":"                try:"},{"line_number":110,"context_line":"                    url \u003d FILES_URL.format(repo, branch)"},{"line_number":111,"context_line":"                    files \u003d requests.get(url).json()"}],"source_content_type":"text/x-python","patch_set":5,"id":"6904b0e8_ea85f556","line":108,"range":{"start_line":108,"start_character":37,"end_line":108,"end_character":45},"in_reply_to":"17fe6fe7_c095919b","updated":"2024-09-10 22:36:54.000000000","message":"sunbeam-charms - the new monorepo has a `main` instead of a `master` branch; the project team guide process [1] or the Infra guide [2] don\u0027t mention \"main\" as a substitute to \"master\"... \n\nThis \"inconsistency\" needs to be acknowledged rather than fixed in my opinion. \"master\" has negative connotations; so I don\u0027t know if any of the OpenStack processes are enforcing projects to name their development branch as \"master\". \n\nGerrit import process for new projects allows a \"default-branch\"; implicitly conveying that the main/default/development branch can be called anything. \n\n@amy@demarco.com: Do you have any thoughts to share here? I don\u0027t recall the decision we took many years ago when we visited this as part of addressing problematic language conventions. \n\n[1] https://docs.openstack.org/project-team-guide/stable-branches.html#processes\n[2] https://docs.opendev.org/opendev/infra-manual/latest/drivers.html#branches","commit_id":"331144012f45f114bba73fe4834baff2ed43ecc3"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"0fd6c9ef42b80743d9f2ec4454e2e385235e08f5","unresolved":true,"context_lines":[{"line_number":105,"context_line":"                print(msg)"},{"line_number":106,"context_line":"                continue"},{"line_number":107,"context_line":""},{"line_number":108,"context_line":"            for branch in (\u0027master\u0027, \u0027main\u0027):"},{"line_number":109,"context_line":"                try:"},{"line_number":110,"context_line":"                    url \u003d FILES_URL.format(repo, branch)"},{"line_number":111,"context_line":"                    files \u003d requests.get(url).json()"}],"source_content_type":"text/x-python","patch_set":5,"id":"5d0ab926_b1e599e2","line":108,"range":{"start_line":108,"start_character":37,"end_line":108,"end_character":45},"in_reply_to":"2e3a2b1d_d83e6895","updated":"2024-09-11 16:51:29.000000000","message":"I am extremely -1 to forcing any OpenStack team to use controversial language in their branch name.","commit_id":"331144012f45f114bba73fe4834baff2ed43ecc3"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"51abb39b2d75be95310b2be25108dad51dc7cbcb","unresolved":true,"context_lines":[{"line_number":105,"context_line":"                print(msg)"},{"line_number":106,"context_line":"                continue"},{"line_number":107,"context_line":""},{"line_number":108,"context_line":"            for branch in (\u0027master\u0027, \u0027main\u0027):"},{"line_number":109,"context_line":"                try:"},{"line_number":110,"context_line":"                    url \u003d FILES_URL.format(repo, branch)"},{"line_number":111,"context_line":"                    files \u003d requests.get(url).json()"}],"source_content_type":"text/x-python","patch_set":5,"id":"2df3ec31_2aa8a15b","line":108,"range":{"start_line":108,"start_character":37,"end_line":108,"end_character":45},"in_reply_to":"5960f29f_17afc06f","updated":"2024-09-29 16:46:52.000000000","message":"We should rehash this discussion at the Oct 2024 PTG: https://etherpad.opendev.org/p/oct2024-ptg-os-tc\n\nMarking this conversation \"Unresolved\".","commit_id":"331144012f45f114bba73fe4834baff2ed43ecc3"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"64706468aee24fe899e8338a55a29158735b72b2","unresolved":true,"context_lines":[{"line_number":105,"context_line":"                print(msg)"},{"line_number":106,"context_line":"                continue"},{"line_number":107,"context_line":""},{"line_number":108,"context_line":"            for branch in (\u0027master\u0027, \u0027main\u0027):"},{"line_number":109,"context_line":"                try:"},{"line_number":110,"context_line":"                    url \u003d FILES_URL.format(repo, branch)"},{"line_number":111,"context_line":"                    files \u003d requests.get(url).json()"}],"source_content_type":"text/x-python","patch_set":5,"id":"5960f29f_17afc06f","line":108,"range":{"start_line":108,"start_character":37,"end_line":108,"end_character":45},"in_reply_to":"5d0ab926_b1e599e2","updated":"2024-09-11 17:43:41.000000000","message":"I was chatting with @amy@demarco.com since my search-fu is failing me. The only reference I found around branch name standardization is here: https://wiki.openstack.org/wiki/Diversity/Inclusivity\n\nFrom what I recall, we did discuss it at length a few years ago (~2021) and we took the stance that:\n\n1) We wouldn\u0027t rename the development branch of OpenStack repositories en-masse\n2) Projects can have any name set as the \"default\"/\"development\" branch - project-config and the opendev systems allow this\n3) We were hoping the community would collectively actively discourage non-inclusive language\n\nWe probably have a formal policy somewhere. I agree that this is getting side-loaded to the TC; I really don\u0027t mind us chasing a TC resolution for this so we can make a formal recommendation to teams. In my preference, we can allow project teams to determine what the default branch should be named.  \n\nIt\u0027s been a few years since we\u0027ve seen how the experience has gone in adjacent communities like Kubernetes (not completed yet) [1][2], Ceph, Gnome, ... so I don\u0027t know if we want to restart a community-wide effort to affect existing repositories. I think us clarifying our stance on inclusive language via a policy is important. \n\n\n\n[1] https://www.kubernetes.dev/resources/rename/ \n[2] https://github.com/kubernetes/enhancements/issues/2853","commit_id":"331144012f45f114bba73fe4834baff2ed43ecc3"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"689b77f82a9b97da384ce249be46623010bab5a5","unresolved":true,"context_lines":[{"line_number":105,"context_line":"                print(msg)"},{"line_number":106,"context_line":"                continue"},{"line_number":107,"context_line":""},{"line_number":108,"context_line":"            for branch in (\u0027master\u0027, \u0027main\u0027):"},{"line_number":109,"context_line":"                try:"},{"line_number":110,"context_line":"                    url \u003d FILES_URL.format(repo, branch)"},{"line_number":111,"context_line":"                    files \u003d requests.get(url).json()"}],"source_content_type":"text/x-python","patch_set":5,"id":"9e6b411c_f70182b5","line":108,"range":{"start_line":108,"start_character":37,"end_line":108,"end_character":45},"in_reply_to":"5d0ab926_b1e599e2","updated":"2024-09-11 17:27:40.000000000","message":"then what you suggest 1. change \u0027master\u0027 name to something else in every branch 2. Make inconsistency among OpeNStack repo with some using \u0027master\u0027 and some other name (currently it is \u0027main\u0027 and it can be anything in future)","commit_id":"331144012f45f114bba73fe4834baff2ed43ecc3"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"93901ba3bad2fc30076f431613e5182a5ccebc97","unresolved":true,"context_lines":[{"line_number":105,"context_line":"                print(msg)"},{"line_number":106,"context_line":"                continue"},{"line_number":107,"context_line":""},{"line_number":108,"context_line":"            for branch in (\u0027master\u0027, \u0027main\u0027):"},{"line_number":109,"context_line":"                try:"},{"line_number":110,"context_line":"                    url \u003d FILES_URL.format(repo, branch)"},{"line_number":111,"context_line":"                    files \u003d requests.get(url).json()"}],"source_content_type":"text/x-python","patch_set":5,"id":"2e3a2b1d_d83e6895","line":108,"range":{"start_line":108,"start_character":37,"end_line":108,"end_character":45},"in_reply_to":"6904b0e8_ea85f556","updated":"2024-09-11 16:32:59.000000000","message":"That is separate discussion on whether we should use master or other name but things is what OpenStack use currently if we change then we need to change it in entire system. We discussed that during inclusive lang effort time and it was a huge change to entire community and tooling so dropped the idea to change it?\n\nAlso, I am not sure if sunbeam-charms having a \u0027main\u0027 branch is the right direction too.","commit_id":"331144012f45f114bba73fe4834baff2ed43ecc3"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"e9fb95c48b8697b8e6375dad77be3a7fc35f6a14","unresolved":true,"context_lines":[{"line_number":105,"context_line":"                print(msg)"},{"line_number":106,"context_line":"                continue"},{"line_number":107,"context_line":""},{"line_number":108,"context_line":"            for branch in (\u0027master\u0027, \u0027main\u0027):"},{"line_number":109,"context_line":"                try:"},{"line_number":110,"context_line":"                    url \u003d FILES_URL.format(repo, branch)"},{"line_number":111,"context_line":"                    files \u003d requests.get(url).json()"}],"source_content_type":"text/x-python","patch_set":5,"id":"358de512_8b0955a0","line":108,"range":{"start_line":108,"start_character":37,"end_line":108,"end_character":45},"in_reply_to":"9e6b411c_f70182b5","updated":"2024-09-11 17:38:38.000000000","message":"I am not in favour of introducing any inconsistency. That is the only concern I have. I am ok with using either name, but it must be the same among all OpenStack projects; otherwise, you can imagine how difficult it will be to handle it in tooling, and job configurations. For example, as sunbeam-charms has \u0027main\u0027 later cinder can change the master to \u0027main\u0027 or \u0027current\u0027 then it become difficult to handle. And if we allow one project to do this then we will not be able to stop other projects to do the same.\n\nAllowing a mix of names in development branches creates a lot of confusion and work in all the tooling and job configurations.","commit_id":"331144012f45f114bba73fe4834baff2ed43ecc3"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"dfc296b4f13f8f580f92f2cfb6c4b5228e6506ad","unresolved":true,"context_lines":[{"line_number":115,"context_line":"                    # Try with next default branch!"},{"line_number":116,"context_line":"                    pass"},{"line_number":117,"context_line":""},{"line_number":118,"context_line":"            for file in file_names:"},{"line_number":119,"context_line":"                if file not in (\u0027.gitreview\u0027, \u0027README.rst\u0027):"},{"line_number":120,"context_line":"                    msg \u003d \u0027{} is not properly retired, files: {}.\u0027.format("},{"line_number":121,"context_line":"                        repo, file_names)"}],"source_content_type":"text/x-python","patch_set":5,"id":"19c72199_24505c45","line":118,"range":{"start_line":118,"start_character":24,"end_line":118,"end_character":34},"updated":"2024-09-10 22:36:54.000000000","message":"\"file_names\" is declared within try..except above; there\u0027s a chance that a repo that has neither \"master\" or \"main\" branch and fail here with a `NameError`","commit_id":"331144012f45f114bba73fe4834baff2ed43ecc3"}]}