search

cesium-ml / baselayer

Fully customizable (scientific, compute-intensive) web application template
http://cesium-ml.org/baselayer/
30 stars 18 forks source link

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

Open stefanv opened 2 years ago

stefanv commented 2 years ago

@mcoughlin Here is an alternative approach to https://github.com/skyportal/skyportal/pull/3089 to address https://github.com/skyportal/skyportal/issues/3084

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

mcoughlin commented 2 years ago

@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 commented 2 years ago

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

mcoughlin commented 2 years ago

@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 commented 2 years ago

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 commented 2 years ago

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