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

docs: Document telemetry #4972

Open
wants to merge 11 commits into
base: main
Choose a base branch
from
Open

docs: Document telemetry #4972

wants to merge 11 commits into from

Conversation

Dan-Heath
Copy link
Contributor

@Dan-Heath Dan-Heath commented Jul 26, 2024

Added 2 new pages to expand on Boundary's telemetry capabilities:

And edited the existing index page to provide additional info and reduce redundancy:

Copy link
Collaborator

@stellarsquall stellarsquall left a comment

Choose a reason for hiding this comment

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

great work on this 👍

Dan-Heath and others added 3 commits November 12, 2024 14:32
Co-authored-by: Robin Beck <stellarsquall@users.noreply.github.com>
Co-authored-by: Robin Beck <stellarsquall@users.noreply.github.com>
@Dan-Heath Dan-Heath added the backport/website Backport changes to `stable-website` and the latest release-branch label Nov 12, 2024
Complete the following steps to configure events:

1. Open the configuration file for the controller or worker server that you want to emit events.
1. Edit the `events` stanza in the configuration file.
Copy link
Collaborator

Choose a reason for hiding this comment

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

I think this should be item 2

Copy link
Contributor Author

@Dan-Heath Dan-Heath Nov 12, 2024

Choose a reason for hiding this comment

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

Ah yeah, our Markdown standard is a little wonky. That still renders as numbers 2 and 3. It should be OK in the preview deployment:

image

Learn what data Boundary captures when telemetry is enabled and how you can view it.
---

# Telemetry data
Copy link
Collaborator

Choose a reason for hiding this comment

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

Is this going to be an internal doc?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

No, Anando wants to publish these steps for our external users so they can benefit from telemetry. So we would publish this to developer.com with our other external docs.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Got it thanks for clarifying Dan

<td>Auth method URI</td>
</tr>
<tr>
<td>Link (Includes location: org ID, project ID, and internal ID)</td>
Copy link
Collaborator

Choose a reason for hiding this comment

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

I believe the link is only available to us because we append them into the telemetry events when we are processing them in cloud boundary service. so boundary user wont see this information when they turn on the telemetry

</thead>
<tbody>
<tr>
<td rowspan="10">HCP Boundary clusters created</td>
Copy link
Collaborator

Choose a reason for hiding this comment

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

This is internal as well. only visible to us. customer won't be able to see this.

Copy link
Collaborator

@sepehrfrgh sepehrfrgh left a comment

Choose a reason for hiding this comment

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

There is some misunderstanding on the telemetry data list. In this PR list, I see, our internal telemetry events that we send to Snowflake but the one that customers are seeing is slightly different.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport/website Backport changes to `stable-website` and the latest release-branch docs/concepts docs/configuration website
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants