-
Notifications
You must be signed in to change notification settings - Fork 121
Logging file #3294
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
Logging file #3294
Conversation
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.
Infra-wise, looks good
Co-authored-by: Eric Arellano <[email protected]>
Co-authored-by: Kyle Ross <[email protected]>
## Fields by log type | ||
|
||
For information about fields included in every platform log, see [Fields that are included in platform logs.](https://cloud.ibm.com/docs/logs-router?topic=logs-router-about-platform-logs#about-platform-logs-2) | ||
|
||
The following fields are included in the log record. | ||
|
||
| Field | Type | Description | | ||
|-----------------|----------|--------------------------------------------------------------------------| | ||
| logSourceCRN | Required | Defines the account and flow log instance where the log is published. | | ||
| saveServiceCopy | Required | Defines whether IBM saves a copy of the record for operational purposes. | | ||
| message | Required | Description of the log that is generated. | |
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.
This is accurate in the sense the records include these log lines, and include the generic platform level log fields. They also include a couple other fields that are not listed, and we're working to adjust them to make sure they include information that we choose to expose. I don't think it is a problem to publish this as is for what it is worth as the other fields are not relevant to a user either way.
It still needs a home.