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

Dashboards: added support to experimental read-write deployment mode #2780

Merged
merged 1 commit into from
Aug 22, 2022

Conversation

pracucci
Copy link
Collaborator

What this PR does

In this PR I'm adding a basic support for read-write deployment mode in the dashboards. Resources and networking dashboards are not addressed yet (will be done in a follow up PR).

Which issue(s) this PR fixes or relates to

Part of #2749

Checklist

  • Tests updated
  • Documentation added
  • CHANGELOG.md updated - the order of entries should be [CHANGE], [FEATURE], [ENHANCEMENT], [BUGFIX]

Signed-off-by: Marco Pracucci <marco@pracucci.com>
@colega
Copy link
Contributor

colega commented Aug 22, 2022

Resources and networking dashboards are not addressed yet (will be done in a follow up PR).

I can see them already updated, but I guess we'll need to tweak them as multiple components now run in the same binary?

@pracucci
Copy link
Collaborator Author

Resources and networking dashboards are not addressed yet (will be done in a follow up PR).

I can see them already updated, but I guess we'll need to tweak them as multiple components now run in the same binary?

Right. In the write path, both distributor and ingester panels will show the same process when running the read-write deployment mode. I'm still brainstorming on the best way to address it, so deferring to a follow up PR for now.

@pracucci pracucci merged commit ec587dd into main Aug 22, 2022
@pracucci pracucci deleted the add-read-write-support-to-dashboards branch August 22, 2022 07:21
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.

None yet

2 participants