-
Notifications
You must be signed in to change notification settings - Fork 6.7k
Add observability for label-selectors #53423
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
Conversation
Signed-off-by: Alan Guo <aguo@anyscale.com>
@nikitavemuri @MengjinYan, this is working except for the export task events. Any ideas what's missing? @MengjinYan , ptal if this interferes with your new export work. |
Any idea why clang-format is not working? I installed it on my mac with |
I tried with a local setup and I can see the
Is it something you tested or you were looking at something else for the export task event?
No. This won't interferes with my work now. |
I never tried Normally I follow https://docs.ray.io/en/latest/ray-contribute/development.html#installing-additional-dependencies-for-development to install the additional dependencies for lint:
Probably you can try that to see if it will be helpful. |
yes, this fixed it. Thanks! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks for adding this!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry for the late review. It looks good. Thanks!
label-based selection is being added as a feature. tracker here: #51564 As part of the feature, we should allow users to observe labels of nodes and selection rules of tasks and actors. This PR adds these fields to: - state api CLI output - state api API responses - Ray Dashboard UI Nodes:   Tasks:   Actors:   --------- Signed-off-by: Alan Guo <aguo@anyscale.com> Signed-off-by: elliot-barn <elliot.barnwell@anyscale.com>
label-based selection is being added as a feature. tracker here: #51564 As part of the feature, we should allow users to observe labels of nodes and selection rules of tasks and actors. This PR adds these fields to: - state api CLI output - state api API responses - Ray Dashboard UI Nodes:   Tasks:   Actors:   --------- Signed-off-by: Alan Guo <aguo@anyscale.com> Signed-off-by: elliot-barn <elliot.barnwell@anyscale.com>
…and task detail pages (#54292) Follow-up to #53423 Missed a few places in the UI. Also updates placement group tables to use the same code preview component as the actor and tasks tables. Placement group table   Actor detail  Task detail  --------- Signed-off-by: Alan Guo <aguo@anyscale.com>
…and task detail pages (ray-project#54292) Follow-up to ray-project#53423 Missed a few places in the UI. Also updates placement group tables to use the same code preview component as the actor and tasks tables. Placement group table   Actor detail  Task detail  --------- Signed-off-by: Alan Guo <aguo@anyscale.com> Signed-off-by: ChanChan Mao <chanchanmao1130@gmail.com>
Why are these changes needed?
label-based selection is being added as a feature. tracker here: #51564
As part of the feature, we should allow users to observe labels of nodes and selection rules of tasks and actors.
This PR adds these fields to:
Nodes:


Tasks:


Actors:


Related issue number
Checks
git commit -s
) in this PR.scripts/format.sh
to lint the changes in this PR.method in Tune, I've added it in
doc/source/tune/api/
under thecorresponding
.rst
file.