Skip to content

Add tool to blend logs, taking timestamps into consideration#326

Open
stefanv wants to merge 1 commit into
cesium-ml:mainfrom
stefanv:log-merge-tool
Open

Add tool to blend logs, taking timestamps into consideration#326
stefanv wants to merge 1 commit into
cesium-ml:mainfrom
stefanv:log-merge-tool

Conversation

@stefanv
Copy link
Copy Markdown
Contributor

@stefanv stefanv commented Jun 15, 2022

@mcoughlin Here is an alternative approach to skyportal/skyportal#3089 to address skyportal/skyportal#3084

This does not currently support the nginx log, but we can add that.

@mcoughlin
Copy link
Copy Markdown
Collaborator

@stefanv Oh wow that's nifty. Maybe it could optionally return the log string if called as a function for potential parsing and display of the health events on a dedicated page like I was thinking about in that other PR?

@stefanv
Copy link
Copy Markdown
Contributor Author

stefanv commented Jun 15, 2022

Is there a specific reason we need to expose the logs via the web?

@mcoughlin
Copy link
Copy Markdown
Collaborator

@stefanv i am thinking it would be very nice to at a glance (in an admin only page) to be able to see the logging rather than needing to be logged in and pulling down logs and such. It's mostly about convenience of rapid assessment.

@stefanv
Copy link
Copy Markdown
Contributor Author

stefanv commented Jun 16, 2022

Okay, maybe we should start with a tool and then talk about expanding scope. I'm a bit nervous, from a security perspective, to have log files served up. Perhaps it's fine, but then we may also want to consider whether we don't just want to use Google or GrayLog or similar.

@mcoughlin
Copy link
Copy Markdown
Collaborator

@stefanv sounds good. I am cool with your tool for sure.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants