Skip to content

Improve the process for importing Grafana dashboard examples#39

Open
mtlljm wants to merge 1 commit into
mainfrom
pr-9141-2
Open

Improve the process for importing Grafana dashboard examples#39
mtlljm wants to merge 1 commit into
mainfrom
pr-9141-2

Conversation

@mtlljm
Copy link
Copy Markdown
Contributor

@mtlljm mtlljm commented May 12, 2026

Add prerequisites
Add help on how to find the namespace
Remove section about external discovery service
Add extra steps on how to complete the import process

@mtlljm mtlljm requested review from chrismwsas, katjam-sas, keeper7kaSAS and mtlstr-sas and removed request for katjam-sas May 12, 2026 11:43
@mtlljm mtlljm force-pushed the pr-9141-2 branch 2 times, most recently from 9b1cc12 to 6bad436 Compare May 13, 2026 15:19
Copy link
Copy Markdown
Contributor

@megwu26 megwu26 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

adding some proposed edits

Comment thread EndToEndExamples/DASHBOARD.md Outdated
# Importing the Grafana Dashboard

Before you import the dashboard, you need to update the `grafana.json` file for your environment:
Before you import the dashboard, you must update `grafana.json` for your environment because the ESP Server connection URL is different in each cluster.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

move this lead in sentence to below the prerequisites

Comment thread EndToEndExamples/DASHBOARD.md
Comment thread EndToEndExamples/DASHBOARD.md Outdated
Before you import the dashboard, you must update `grafana.json` for your environment because the ESP Server connection URL is different in each cluster.

Prerequisites:
- You must have the SAS Event Stream Processing and Grafana services running in the same cluster.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Instead of beginning these sentences with "You must have" just say "The SAS Event Stream Processing and Grafana services are running in the same cluster." but should it be "in" or "on" here?

Comment thread EndToEndExamples/DASHBOARD.md Outdated

Prerequisites:
- You must have the SAS Event Stream Processing and Grafana services running in the same cluster.
- You must have the SAS Event Stream Processing Data Source Plug-in for Grafana installed and configured, see [SAS Event Stream Processing Data Source Plug-in for Grafana](https://github.com/sassoftware/grafana-esp-plugin/blob/main/README.md#add-the-sas-event-stream-processing-data-source).
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Split up this sentence. The second sentence should begin with, "For more information, see link"

Comment thread EndToEndExamples/DASHBOARD.md Outdated
3. From the drop-down list, select **Import**.
4. Click **Upload dashboard JSON file** and select your updated `grafana.json` file.
5. Click **Import**.
5. Enter a name for the dashboard.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This sentence should be, "In the Name field, enter a name for the dashboard."

Comment thread EndToEndExamples/DASHBOARD.md Outdated
4. Click **Upload dashboard JSON file** and select your updated `grafana.json` file.
5. Click **Import**.
5. Enter a name for the dashboard.
6. Select a folder for the dashboard.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This sentence should be, "From the Folder drop-down list, select a folder for the dashboard."

Comment thread EndToEndExamples/DASHBOARD.md Outdated
5. Enter a name for the dashboard.
6. Select a folder for the dashboard.
7. If required, change the Unique identifier (UID) by clicking the **Change uid** button and giving the dashboard a unique identifier.
8. Select the datasource that you configured with the SAS Event Stream Processing Studio application.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This sentence should be, "From the SAS Event Stream Processing Datasource drop-down list, select the datasource that you configured with the SAS Event Stream Processing Studio application."

Add prerequisites
Add help on how to find the namespace
Remove section about external discovery service
Add extra steps on how to complete the import process

Signed-off-by: mtlljm <liam.mcgregor@sas.com>
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