fix: BOMS-235 Create script to set up remotes for devstack

[Akanshu-2u]

Description:

Developers need to set up git remotes for forked repositories to access both edx and openedx organizations. Script was a need to be prepared to handle the remote , origin , personal and non-forked repos.

Solution:

1. Added a script that automatically configures git remotes for list of repositories.
2. Renames existing origin remote to match the primary organization (edx or openedx)
3. Adds the complementary remote (if origin is edx, adds openedx remote and vice versa)
4. Integrated with make commands: make dev.setup-remotes and automatic setup during clone operations
5. Works for all repository types: edx repos, openedx repos, personal forks, and non-forked repos

Run test file:

• Make the test file executable:
  chmod +x test_repo.sh

• Run dry run
  ./test_repo.sh

Expected dry-run output:

1. Remote setup:

# Before
origin  https://github.com/edx/ecommerce.git

# After  
edx     https://github.com/edx/ecommerce.git
openedx https://github.com/openedx/ecommerce.git

2. Error - Personal Fork:

Setting up remotes for forked repository: credentials
ERROR: Unexpected origin URL in credentials: https://github.com/personaluser/credentials.git
Expected URL to be from either edx or openedx organization

3. Skipped Repository:

Repository course-discovery is not cloned. Skipping.

4. Final summary:

Remote Setup Summary:
✓ Successfully configured remotes for 2 repositories:
  - ecommerce, edx-platform

✗ Failed to configure remotes for 1 repositories:
  - credentials

◦ Skipped 69 repositories (not cloned or not forked)

Total repositories processed: 72

JIRA Ticket:

BOMS-235

[Copilot]

Pull Request Overview

Implements a git remote setup system for devstack repositories to handle forked repositories between edx and openedx organizations. The script automatically configures appropriate remotes for developers working with forks.

• Adds remote setup functionality that renames origin to the primary organization and adds the complementary remote
• Creates a comprehensive test suite to validate the remote setup behavior across different repository types
• Integrates remote setup into the existing make clone workflow for automatic configuration

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 7 comments.

File	Description
repo.sh	Adds core remote setup logic, forked repository configuration map, and new command handler
Makefile	Integrates remote setup command and adds automatic setup to clone operations
test_repo.sh	Comprehensive test suite for validating remote setup functionality

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

[Copilot]

The command substitution $(git remote | wc -l) should be quoted to prevent word splitting. Use [ \"$(git remote | wc -l)\" -eq 2 ] instead.

Suggested change

	if git remote get-url openedx | grep -q "openedx/course-discovery" && git remote get-url edx | grep -q "edx/course-discovery" && [ $(git remote | wc -l) -eq 2 ]; then
	if git remote get-url openedx | grep -q "openedx/course-discovery" && git remote get-url edx | grep -q "edx/course-discovery" && [ "$(git remote | wc -l)" -eq 2 ]; then

[Copilot]

The command substitutions $(git remote | wc -l) should be quoted to prevent word splitting. Use [ \"$(git remote | wc -l)\" -eq 1 ] and [ \"$(git remote | wc -l)\" -eq 2 ] instead.

[Copilot]

The command substitutions $(git remote | wc -l) should be quoted to prevent word splitting. Use [ \"$(git remote | wc -l)\" -eq 1 ] and [ \"$(git remote | wc -l)\" -eq 2 ] instead.

Suggested change

	if [ $(git remote | wc -l) -eq 2 ] && git remote | grep -q "edx" && git remote | grep -q "openedx"; then
	if [ "$(git remote | wc -l)" -eq 2 ] && git remote | grep -q "edx" && git remote | grep -q "openedx"; then

[Copilot]

Iterating over two arrays separately can lead to unexpected behavior if elements contain spaces. Consider combining arrays first: combined_repos=(\"${repos[@]}\" \"${non_release_repos[@]}\") then iterate over \"${combined_repos[@]}\".

Suggested change

	for repo in "${repos[@]}" "${non_release_repos[@]}"
	do
	local combined_repos=( "${repos[@]}" "${non_release_repos[@]}" )
	for repo in "${combined_repos[@]}"

[Copilot]

The arithmetic expansion should be quoted to handle edge cases properly. Use echo \"Total repositories processed: $((${#successful_repos[@]} + ${#failed_repos[@]} + ${#skipped_repos[@]}))\" instead.

Suggested change

	echo "Total repositories processed: $((${#successful_repos[@]} + ${#failed_repos[@]} + ${#skipped_repos[@]}))"
	echo "Total repositories processed: \"$((${#successful_repos[@]} + ${#failed_repos[@]} + ${#skipped_repos[@]}))\""

[Copilot]

Pull Request Overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

[Copilot]

The edx-analytics-dashboard repository appears twice in the ssh_repos array (lines 82 and 88). This duplication should be removed to maintain consistency.

[Copilot]

The variable $name_pattern is referenced but not defined in this function scope. This pattern should be defined locally or passed as a parameter to avoid potential runtime errors.