-
Notifications
You must be signed in to change notification settings - Fork 3
Add docs about system env variables. #106
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
base: main
Are you sure you want to change the base?
Conversation
platform/variables.mdx
Outdated
|
|
||
| ## System Variables | ||
|
|
||
| Checkly provides a set of built-in system variables that you can use in your checks. These variables provide information about the check execution environment, and can be especially useful in playwright checks to distinguish between Checkly executions and local runs. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| Checkly provides a set of built-in system variables that you can use in your checks. These variables provide information about the check execution environment, and can be especially useful in playwright checks to distinguish between Checkly executions and local runs. | |
| Checkly provides a set of built-in system variables that you can use in your checks. These variables provide information about the check execution environment and can be especially useful in Playwright Check Suites to distinguish between Checkly executions and local runs. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's maybe also link to Playwright Check Suites here.
platform/variables.mdx
Outdated
| ## System Variables | ||
|
|
||
| Checkly provides a set of built-in system variables that you can use in your checks. These variables provide information about the check execution environment, and can be especially useful in playwright checks to distinguish between Checkly executions and local runs. | ||
|
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just to confirm: are all these variables 👇 set in all environments (browser checks, API checks, multistep checks)? Of course, I've got no clue, but as far as I understand, this change was about the Playwright Check Suites environment?
platform/variables.mdx
Outdated
|
|
||
| Checkly provides a set of built-in system variables that you can use in your checks. These variables provide information about the check execution environment, and can be especially useful in playwright checks to distinguish between Checkly executions and local runs. | ||
|
|
||
| * **CHECKLY**: We set this to "1" for all check runs executed by Checkly. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| * **CHECKLY**: We set this to "1" for all check runs executed by Checkly. | |
| * **CHECKLY**: Set to `1` for all check runs executed by Checkly. |
platform/variables.mdx
Outdated
| Checkly provides a set of built-in system variables that you can use in your checks. These variables provide information about the check execution environment, and can be especially useful in playwright checks to distinguish between Checkly executions and local runs. | ||
|
|
||
| * **CHECKLY**: We set this to "1" for all check runs executed by Checkly. | ||
| * **CHECKLY_RUN_SOURCE**: This variable indicates the source or reason why the check was run. You can use this to skip parts of your test suite in certain scenarios. Possible values include: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| * **CHECKLY_RUN_SOURCE**: This variable indicates the source or reason why the check was run. You can use this to skip parts of your test suite in certain scenarios. Possible values include: | |
| * **CHECKLY_RUN_SOURCE**: Indicates the manual or scheduled check run trigger. Use this variable to skip parts of your test suite in certain scenarios or adjust the check run configuration. Possible values include: |
platform/variables.mdx
Outdated
|
|
||
| * **CHECKLY**: We set this to "1" for all check runs executed by Checkly. | ||
| * **CHECKLY_RUN_SOURCE**: This variable indicates the source or reason why the check was run. You can use this to skip parts of your test suite in certain scenarios. Possible values include: | ||
| * `CLI_DEPLOY`: When checks are deployed using `npx checkly deploy` their first run is scheduled with this type. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nit: for all the following, I'd value consistency for all entries. E.g.
-> The check was triggered by/as ... (or any different wording).
platform/variables.mdx
Outdated
| * `TRIGGER_API`: The check was triggered via the API. | ||
| * `TRIGGER_NO_RECORD`: CLI Trigger `npx checkly trigger`. | ||
| * `TRIGGER_RECORD`: CLI Trigger `npx checkly trigger --record`. | ||
| * **CI**: We set this to "1" for the following check runs: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| * **CI**: We set this to "1" for the following check runs: | |
| * **CI**: Set to `1` for the following check runs: |
We and "personal" words can often be omitted in tech writing. :)
platform/variables.mdx
Outdated
| * `TRIGGER_RECORD`: CLI Trigger `npx checkly trigger --record`. | ||
| * **CI**: We set this to "1" for the following check runs: | ||
| * CLI runs via `npx checkly test` or `npx checkly trigger`. | ||
| * Check runs that are triggered by [deployments](/integrations/ci-cd/github/deployments). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looking at the infra PR, seems like the OTEL variables are missing?
stefanjudis
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@hwartig Thank you for considering the docs. I've left some niggles. :)
Co-authored-by: Stefan Judis <[email protected]>
8f4a09a to
d8c8ec4
Compare
detect/synthetic-monitoring/playwright-checks/environment-variables.mdx
Outdated
Show resolved
Hide resolved
| "group": "Incidents", | ||
| "pages": [ | ||
| "incidents/create-an-incident", | ||
| "api-reference/incidents/create-an-incident", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nice! 👏
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've left one last tiny comment that might be misleading and I'm unsure what should be written there because I'm unfamiliar with the details.
Huge kudos for fixing broken things on the way! 💯
Feel free to move forward once the tiny thing is resolved. :)
Affected Components