Skip to content

Update readme for ubuntu sudo java_home behavior#1013

Merged
HarithaVattikuti merged 1 commit into
actions:mainfrom
mahabaleshwars:docs/note-sudo-java-home-ubuntu
Jun 8, 2026
Merged

Update readme for ubuntu sudo java_home behavior#1013
HarithaVattikuti merged 1 commit into
actions:mainfrom
mahabaleshwars:docs/note-sudo-java-home-ubuntu

Conversation

@mahabaleshwars

Copy link
Copy Markdown
Contributor

Description:
Adds a README note that sudo on Ubuntu runners falls back to the system JDK because JAVA_HOME/PATH aren't inherited.

Related issue:
#997

Check list:

  • Mark if documentation changes are required.
  • Mark if tests were added or updated to cover the changes.

Copilot AI review requested due to automatic review settings June 3, 2026 16:35
@mahabaleshwars mahabaleshwars requested a review from a team as a code owner June 3, 2026 16:35

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Adds a documentation note explaining why Java selected by actions/setup-java may not be used when commands are executed via sudo on Ubuntu GitHub-hosted runners, aligning the README with the behavior described in issue #997.

Changes:

  • Document that sudo on Ubuntu runners may not inherit JAVA_HOME/PATH configured by setup-java.
  • Clarify a common cause of “wrong JDK version” reports when using elevated commands.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread README.md
@HarithaVattikuti HarithaVattikuti merged commit 307d3a2 into actions:main Jun 8, 2026
3 checks passed
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.

5 participants