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.

Sign up for GitHub

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

Jump to bottom

update logs browser #20

Open
wants to merge 2 commits into
base: otel
Choose a base branch
from
Open

update logs browser #20

wants to merge 2 commits into from

Conversation

dotstar-6
Copy link
Contributor

  • update logs browsing
  • enable loki recording for statistics extraction

@gquintard
Copy link
Collaborator

alright, I need more context on this, we'll need to talk about it

my initial reaction is that it's a great change, even though I don't fully grasp its breadth, however I feel like it a BIG step away from a minimal demonstrator, and I'd rather have it live somewhere else, or at least have it disabled by default

@dotstar-6
Copy link
Contributor Author

I concur the additions are maybe too wide to be simple. On the other hand, other monitoring components are already in and showcasing a sample usage, I think, is also beneficial.

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

@gquintard gquintard gquintard left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@dotstar-6 @gquintard