Skip to content
This repository has been archived by the owner on Oct 9, 2023. It is now read-only.

Ray log links #620

Merged
merged 4 commits into from
Sep 26, 2023
Merged

Ray log links #620

merged 4 commits into from
Sep 26, 2023

Conversation

EngHabu
Copy link
Contributor

@EngHabu EngHabu commented Sep 23, 2023

TL;DR

Update flyteplugins to pickup ray log links

Type

  • Bug Fix
  • Feature
  • Plugin

Are all requirements met?

  • Code completed
  • Smoke tested
  • Unit tests added
  • Code documentation added
  • Any pending items have an associated Issue

Tracking Issue

fixes flyteorg/flyte#4063

Signed-off-by: Haytham Abuelfutuh <[email protected]>
Signed-off-by: Haytham Abuelfutuh <[email protected]>
Signed-off-by: Haytham Abuelfutuh <[email protected]>
Signed-off-by: Haytham Abuelfutuh <[email protected]>
@codecov
Copy link

codecov bot commented Sep 23, 2023

Codecov Report

Merging #620 (e079209) into master (51eda24) will increase coverage by 0.44%.
The diff coverage is n/a.

❗ Current head e079209 differs from pull request most recent head 5c591fc. Consider uploading reports for the commit 5c591fc to get more accurate results

Additional details and impacted files

@EngHabu EngHabu requested review from pingsutw and removed request for hamersaw and kumare3 September 23, 2023 13:33
@EngHabu EngHabu enabled auto-merge (squash) September 23, 2023 13:33
@EngHabu EngHabu requested a review from hamersaw September 26, 2023 00:27
@EngHabu EngHabu merged commit 2aca906 into master Sep 26, 2023
@EngHabu EngHabu deleted the ray-log-links branch September 26, 2023 00:45
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Core feature] Add Ray Cluster Dashboard URL to the UI
2 participants