[CIAPP] Document support for logs in CI Visibility junit XML upload #13680
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
1796c2d to
2762174
Compare
2762174 to
034719b
Compare
cg14823
reviewed
Apr 13, 2022
| @@ -118,6 +116,11 @@ This is the full list of options available when using the `datadog-ci junit uplo | |||
| **Example**: `team:backend`<br/> | |||
| **Note**: Tags specified using `--tags` and with the `DD_TAGS` environment variable are merged. If the same key appears in both `--tags` and `DD_TAGS`, the value in the environment variable `DD_TAGS` takes precedence. | |||
|
|
|||
| `--logs` **(beta)** | |||
| : Enable forwarding content from the XML reports as [Logs][7]. The content inside `<system-out>`, `<system-err>`, `<failure>` will be loaded. Logs from elements inside a `<testcase>` are automatically connected to test.<br/> | |||
There was a problem hiding this comment.
Suggested change
| : Enable forwarding content from the XML reports as [Logs][7]. The content inside `<system-out>`, `<system-err>`, `<failure>` will be loaded. Logs from elements inside a `<testcase>` are automatically connected to test.<br/> | |
| : Enable forwarding content from the XML reports as [Logs][7]. The content inside `<system-out>`, `<system-err>`, `<failure>` will be sent to the Logs system. Logs from elements inside a `<testcase>` are automatically linked to the test.<br/> |
There was a problem hiding this comment.
I used the word connected because of https://docs.datadoghq.com/tracing/connect_logs_and_traces/ but good suggestions for the others I'll make the changes
fermayo
approved these changes
Apr 19, 2022
fermayo
reviewed
Apr 19, 2022
| `--logs` **(beta)** | ||
| : Enable forwarding content from the XML reports as [Logs][7]. The content inside `<system-out>`, `<system-err>`, `<failure>` is collected as logs. Logs from elements inside a `<testcase>` are automatically connected to the test.<br/> | ||
| **Default**: `false`<br/> | ||
| **Note**: Logs are billed separately from CI Visibility. |
Co-authored-by: Fernando Mayo <fernando@datadog.com>
d29f2a1 to
5a6b463
Compare
jtappa
suggested changes
Apr 19, 2022
jtappa
approved these changes
Apr 19, 2022
|
When this is ready for merge, please remove the github label and ping us in the documentation channel in slack! :) Thanks! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What does this PR do?
Document a new feature when uploading test report in XML to CI Visibility. A new CLI flag and environment variable enables this additional feature where the logs will be forwarded to the logs product and connected to the tests in CI Visibility.
This was added in DataDog/datadog-ci#510
Preview
https://docs-staging.datadoghq.com/adrian.lopezcalvo%2Fci-visibility-junitxml-logs-support
Reviewer checklist