Skip to content

Update observability.md #51567

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

MichalPitr
Copy link

@MichalPitr MichalPitr commented Mar 20, 2025

Fixed a code snippet to correctly use the exported head node env var defined on the preceding line.

Why are these changes needed?

N/A

Related issue number

N/A

Checks

Docs fix, no need for testing.

Fixed env var to point to head node that hosts the dashboard endpoint.

Signed-off-by: michal pitr <[email protected]>
@MichalPitr MichalPitr requested review from pcmoritz, kevin85421 and a team as code owners March 20, 2025 21:21
@jcotant1 jcotant1 added the core Issues that should be addressed in Ray Core label Mar 21, 2025
@kevin85421 kevin85421 added the go add ONLY when ready to merge, run all tests label Mar 31, 2025
@kevin85421
Copy link
Member

cc @jjyao @edoakes would you mind merging this PR?

@kevin85421 kevin85421 self-assigned this Mar 31, 2025
@edoakes edoakes enabled auto-merge (squash) March 31, 2025 18:18
@hainesmichaelc hainesmichaelc added the community-contribution Contributed by the community label Apr 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
community-contribution Contributed by the community core Issues that should be addressed in Ray Core go add ONLY when ready to merge, run all tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants