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

Add support for etcd logs #7431

Merged
merged 6 commits into from
Aug 27, 2020
Merged

Add support for etcd logs #7431

merged 6 commits into from
Aug 27, 2020

Conversation

nicholas-devlin
Copy link
Contributor

@nicholas-devlin nicholas-devlin commented Aug 25, 2020

What does this PR do?

updates readme and conf.yaml to let users know how to set up logs. Updates testing env to forward logs.

Motivation

https://datadoghq.atlassian.net/browse/AI-739

Additional Notes

https://github.com/DataDog/web-ui/pull/19132
https://github.com/DataDog/logs-backend/pull/11873

Review checklist (to be filled by reviewers)

  • Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
  • PR title must be written as a CHANGELOG entry (see why)
  • Files changes must correspond to the primary purpose of the PR as described in the title (small unrelated changes should have their own PR)
  • PR must have changelog/ and integration/ labels attached

run `ddev env start etcd py38-3.3.10 -e DD_LOGS_ENABLED=true -e DD_LOGS_CONFIG_CONTAINER_COLLECT_ALL=true`
@nicholas-devlin nicholas-devlin requested review from a team as code owners August 25, 2020 22:16
@ofek ofek deleted the devlin/etcd-logs branch August 26, 2020 00:28
@nicholas-devlin nicholas-devlin restored the devlin/etcd-logs branch August 26, 2020 13:39
@hithwen
Copy link
Contributor

hithwen commented Aug 26, 2020

style is failing

@nicholas-devlin
Copy link
Contributor Author

fixed the styling.

etcd/README.md Outdated Show resolved Hide resolved
etcd/README.md Outdated Show resolved Hide resolved
ofek
ofek previously approved these changes Aug 26, 2020
Copy link
Contributor

@ruthnaebeck ruthnaebeck left a comment

Choose a reason for hiding this comment

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

docs review

etcd/README.md Outdated Show resolved Hide resolved
Co-authored-by: ruthnaebeck <[email protected]>
Copy link
Contributor

@ruthnaebeck ruthnaebeck left a comment

Choose a reason for hiding this comment

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

👍 for docs

@hithwen hithwen merged commit 9cbc5de into master Aug 27, 2020
@hithwen hithwen deleted the devlin/etcd-logs branch August 27, 2020 15:34
github-actions bot pushed a commit that referenced this pull request Aug 27, 2020
* add support for etcd logs
* set up test environment to forward logs
run `ddev env start etcd py38-3.3.10 -e DD_LOGS_ENABLED=true -e DD_LOGS_CONFIG_CONTAINER_COLLECT_ALL=true`

Co-authored-by: Ofek Lev <[email protected]>
Co-authored-by: ruthnaebeck <[email protected]> 9cbc5de
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants