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: update telemetry overview and Woopra plug-in docs to be devworkspace-specific #2268

Merged
merged 5 commits into from
Apr 5, 2022

Conversation

dkwon17
Copy link
Contributor

@dkwon17 dkwon17 commented Mar 30, 2022

What does this pull request change?

Updates the telemetry overview and Woopra telemetry plug-in docs to be devworkspace-specific.

What issues does this pull request fix or reference?

eclipse-che/che#21264
eclipse-che/che#21283

Specify the version of the product this pull request applies to

Devworkspace telemetry is supported for I believe Che version 7.27.x and above.

Pull Request checklist

The author and the reviewers validate the content of this pull request with the following checklist, in addition to the automated tests.

  • Any procedure:
    • Successfully tested.
  • Any page or link rename:
  • Builds on Eclipse Che hosted by Red Hat.
  • the Validate language on files added or modified step reports no vale warnings.

@dkwon17 dkwon17 requested review from rkratky and themr0c as code owners March 30, 2022 21:24
@github-actions
Copy link

Click here to review and test in web IDE: Contribute

@dkwon17 dkwon17 changed the title telemetry-devworkspace-update docs: update telemetry overview and Woopra plug-in docs to be devworkspace-specific Mar 30, 2022
@dkwon17 dkwon17 force-pushed the telemetry-devworkspace-update branch from 3402f6f to e8e0ec2 Compare March 30, 2022 21:30
This plug-in is used by link:https://workspaces.openshift.com[Eclipse Che hosted by Red Hat], but any {prod} deployment can take advantage of this plug-in.
There are no dependencies other than a valid Woopra domain and Segment Write key.
The link:https://raw.githubusercontent.com/che-incubator/che-workspace-telemetry-woopra-plugin/master/meta.yaml[plug-in's meta.yaml] file has 5 environment variables that can be passed to the plug-in:
The plug-in's link:https://raw.githubusercontent.com/che-incubator/devworkspace-telemetry-woopra-plugin/main/plugin.yaml[plugin.yaml] devfile v2 file has 4 environment variables that can be passed to the plug-in:
Copy link
Contributor Author

Choose a reason for hiding this comment

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

The build fails because this file: https://raw.githubusercontent.com/che-incubator/devworkspace-telemetry-woopra-plugin/main/plugin.yaml

does not exist yet.

It will however, when this PR che-incubator/devworkspace-telemetry-woopra-plugin#9 is merged.

Copy link
Member

Choose a reason for hiding this comment

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

@dkwon17 dkwon17 force-pushed the telemetry-devworkspace-update branch from e8e0ec2 to 3870ea1 Compare March 30, 2022 22:12
@dkwon17 dkwon17 force-pushed the telemetry-devworkspace-update branch from 3870ea1 to a0c6805 Compare March 30, 2022 22:22
@themr0c themr0c added this to the 7.44 milestone Apr 1, 2022
Copy link
Contributor

@max-cx max-cx left a comment

Choose a reason for hiding this comment

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

LGTM

dkwon17 and others added 2 commits April 1, 2022 10:27
@ibuziuk ibuziuk merged commit b5d4c98 into eclipse-che:master Apr 5, 2022
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.

5 participants