Skip to content

Fix R configuration quotes bug preventing package installation on Linux VMs#4658

Merged
marrobi merged 4 commits into
mainfrom
copilot/fix-4657
Nov 25, 2025
Merged

Fix R configuration quotes bug preventing package installation on Linux VMs#4658
marrobi merged 4 commits into
mainfrom
copilot/fix-4657

Conversation

Copilot AI commented Aug 28, 2025

Copy link
Copy Markdown
Contributor

The R configuration in Linux VMs had malformed quotes that prevented R and RStudio from properly installing packages. The issue was in line 160 of templates/workspace_services/guacamole/user_resources/guacamole-azure-linuxvm/terraform/vm_config.sh where extra quotes caused syntax errors in the generated R configuration.

Problem:
When users tried to run install.packages("tidyverse") or any package installation in R/RStudio on Linux VMs, it would fail because the R configuration had malformed quotes around the Nexus proxy URL.

Root Cause:
The shell script contained extra quotes in the R configuration:

r["Nexus"] <- """${NEXUS_PROXY_URL}"/repository/r-proxy/"

This generated invalid R syntax with mismatched quotes.

Solution:

  • Removed the extra quotes to match the working Windows VM configuration pattern:
r["Nexus"] <- "${NEXUS_PROXY_URL}/repository/r-proxy/"
  • Updated bundle version from 1.3.2 to 1.3.3 following semantic versioning guidelines for bug fixes

Verification:

  • Script passes shellcheck validation
  • Generated R configuration produces valid syntax
  • Configuration now matches the working Windows equivalent
  • Simulated output correctly configures the Nexus repository URL

This ensures R can properly connect to the Nexus proxy for package installation without syntax errors.

Fixes #4657.


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

…ux VMs

Co-authored-by: marrobi <17089773+marrobi@users.noreply.github.com>
Copilot AI changed the title [WIP] R is not correctly configured for the proxy on Linux Fix R configuration quotes bug preventing package installation on Linux VMs Aug 28, 2025
Copilot AI requested a review from marrobi August 28, 2025 16:58

@marrobi marrobi left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@copilot the bundle version also needs updating.

Co-authored-by: marrobi <17089773+marrobi@users.noreply.github.com>

Copilot AI commented Aug 29, 2025

Copy link
Copy Markdown
Contributor Author

@copilot the bundle version also needs updating.

Updated the bundle version from 1.3.2 to 1.3.3 and added the version bump to CHANGELOG.md. Commit: 3280a69

Copilot AI requested a review from marrobi August 29, 2025 08:17
@marrobi

marrobi commented Sep 11, 2025

Copy link
Copy Markdown
Member

@TonyWildish are you able to test this fix? Thanks.

@marrobi marrobi marked this pull request as ready for review September 11, 2025 10:57
@TonyWildish-BH

Copy link
Copy Markdown
Contributor

@marrobi, it's on my TODO list, but I'm firefighting other issues at the moment. It'll be several days before I can get to this.

@marrobi marrobi requested a review from a team as a code owner November 25, 2025 17:22
@marrobi marrobi enabled auto-merge (squash) November 25, 2025 17:22
@github-actions

Copy link
Copy Markdown

Unit Test Results

0 tests   0 ✅  0s ⏱️
0 suites  0 💤
0 files    0 ❌

Results for commit 144a6fc.

@JC-wk JC-wk left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@marrobi

marrobi commented Nov 25, 2025

Copy link
Copy Markdown
Member

/test-force-approve 144a6fc

Note tested.

@github-actions

Copy link
Copy Markdown

🤖 pr-bot 🤖

✅ Marking tests as complete (for commit 144a6fc)

(in response to this comment from @marrobi)

@marrobi marrobi merged commit 59ccd52 into main Nov 25, 2025
15 checks passed
@marrobi marrobi deleted the copilot/fix-4657 branch November 25, 2025 17:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

R is not correctly configured for the proxy on Linux

5 participants