Skip to content
This repository has been archived by the owner on Aug 30, 2019. It is now read-only.

config: use shared config for loading Agent 5 paths #528

Merged
merged 7 commits into from
Nov 26, 2018
Merged

Conversation

gbbr
Copy link
Contributor

@gbbr gbbr commented Nov 21, 2018

Uses shared config to load legacy .conf file and removes all loading logic and dependencies. Repo size with vendor folder is now at 500MB. The size problem will be solved in the 6.9.0 release when we will merge into datadog-agent.

Binary size now 21M (compared to master 20M).

⚠️ Do not view diff as a whole (exclude final commit which adds vendor folder). Click here to view diff without last commit.

@gbbr gbbr added this to the 6.8.0 milestone Nov 21, 2018
@gbbr gbbr force-pushed the gbbr/guild-ini branch 5 times, most recently from deab6b8 to 19befbc Compare November 22, 2018 11:30
@gbbr gbbr force-pushed the gbbr/guild-ini branch 3 times, most recently from 20b4858 to e5ac28b Compare November 22, 2018 11:49
config/agent.go Outdated Show resolved Hide resolved
config/merge_yaml.go Outdated Show resolved Hide resolved
@gbbr
Copy link
Contributor Author

gbbr commented Nov 26, 2018

Thanks for the review! I will clean the confusing stuff up in the next PR.

@gbbr gbbr merged commit 825db8f into master Nov 26, 2018
@gbbr gbbr deleted the gbbr/guild-ini branch November 26, 2018 12:53

[trace.sampler]
extra_sample_rate: 0.5
max_traces_per_second: 5
max_events_per_second: 50

[trace.ignore]
resource: "/health,/500"
resource: "/health","/500"

Choose a reason for hiding this comment

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

@gbbr this is a breaking change that we introduced in a minor release.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants