Skip to content
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

Cherry pick: #2101 #2120

Closed
wants to merge 2 commits into from
Closed

Conversation

johnugeorge
Copy link
Member

Cherry pick #2101 to release branch

elenzio9 and others added 2 commits February 15, 2023 19:41
…ubeflow#2101)

* backend: Update error message when no logs could be found

* Update the message the backend sends to not just expose that logs are
  not there because 'retain' might not be set, but also because the
  cluster was scaled down.

Signed-off-by: Elena Zioga <elena@arrikto.com>

* frontend: Add LOGS tab in Trial details page

In this commit:

* Create a distinct LOGS tab, which displays the trial's logs in the
  Trial details page.
* Don't show the backend's error popup for logs, but show the message
  error in the admonition.

Signed-off-by: Elena Zioga <elena@arrikto.com>

---------

Signed-off-by: Elena Zioga <elena@arrikto.com>
Signed-off-by: Yuki Iwai <yuki.iwai.tz@gmail.com>
@google-oss-prow
Copy link

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: johnugeorge

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@johnugeorge
Copy link
Member Author

Closed in favour of #2117

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants