-
Notifications
You must be signed in to change notification settings - Fork 1k
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
Display Data Node Logs #19352
Display Data Node Logs #19352
Conversation
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.
Backend changes lgtm.
Tested frontend with two running data nodes. Logs accessible from dropdown and button and displayed correctly.
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.
Hi, overall it looks good ! I added some suggestion inline.
<Button active={logsType === 'stderr'} onClick={() => setLogsType('stderr')}>STDERR</Button> | ||
</ButtonGroup> | ||
<pre> | ||
{logs[logsType] ? ( |
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.
I think we should extract this section to DataNodeLogsView
and use the hook in there so that the Actions component is cleaner.
<Modal.Title>{dataNode.hostname} logs</Modal.Title> | ||
</Modal.Header> | ||
<Modal.Body> | ||
<ButtonGroup> |
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.
We should also use the SegmentedControl
here.
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!
* DataNodeLogs initial setup * changelog file * Allow proxy access to datanode logs by default * Fix repsonse content type of not allowed proxy requests * show data node logs * close logs dialog on backdrop click * get the logs until one clicks "Show logs" * added review changes --------- Co-authored-by: Tomas Dvorak <tomas.dvorak@graylog.com>
* DataNodeLogs initial setup * changelog file * Allow proxy access to datanode logs by default * Fix repsonse content type of not allowed proxy requests * show data node logs * close logs dialog on backdrop click * get the logs until one clicks "Show logs" * added review changes --------- Co-authored-by: Tomas Dvorak <tomas.dvorak@graylog.com>
* DataNodeLogs initial setup * changelog file * Allow proxy access to datanode logs by default * Fix repsonse content type of not allowed proxy requests * show data node logs * close logs dialog on backdrop click * get the logs until one clicks "Show logs" * added review changes --------- Co-authored-by: Tomas Dvorak <tomas.dvorak@graylog.com>
Motivation and Context
fixes #17730
Types of changes
Checklist: