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

fix(ui): repair stale database list in Log Viewer #5683

Merged
merged 3 commits into from
Mar 5, 2021

Conversation

sranka
Copy link
Contributor

@sranka sranka commented Mar 3, 2021

Closes #5544

Briefly describe your proposed changes:

  • list of namespaces (database + retention policy pair) is fetched whever Log Viewer page is opened
    • it is also verified that the current namespace exists, the default namespace is selected otherwise
  • allow clearing local storage with ?clearLocalStorage query, such as `http://localhost:8080/logs?clearLocalStorage
    What was the problem?
  • list of databases was not refreshed from the server, local storage was used
    What was the solution?
  • fetch the list on open of Log Viewer

@sranka sranka changed the title 5544/logs databases stale feat(ui): repair stale database list in Log Viewer Mar 3, 2021
@sranka sranka changed the title feat(ui): repair stale database list in Log Viewer fix(ui): repair stale database list in Log Viewer Mar 4, 2021
@sranka sranka requested a review from ivankudibal March 4, 2021 08:55
@sranka sranka marked this pull request as ready for review March 4, 2021 08:55
@sranka sranka force-pushed the 5544/logs_databases_stale branch from c4e6c32 to e951eae Compare March 5, 2021 04:35
@sranka sranka merged commit fd68352 into master Mar 5, 2021
@sranka sranka deleted the 5544/logs_databases_stale branch March 5, 2021 05:18
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.

Log viewer doesn't show database which has syslog measurment.
2 participants