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.

Sign up for GitHub

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

fix: remove squash on configtls.TLSClientSetting for AWS components #5454

Merged
merged 11 commits into from
Oct 1, 2021
Merged

fix: remove squash on configtls.TLSClientSetting for AWS components #5454

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
1290529
fix: remove squash on configtls.TLSClientSetting
Sep 24, 2021
683d24f
fix: update yamls and readmes to reflect de-squashed tls
Sep 25, 2021
3a24b83
Merge branch 'main' of github.com:open-o11y/opentelemetry-collector-c…
Sep 27, 2021
505d7a8
fix: solve merge conflict
Sep 27, 2021
f1b1fab
fix: remove unnecessary file changes
Sep 28, 2021
110f887
Merge branch 'main' of github.com:open-o11y/opentelemetry-collector-c…
Sep 28, 2021
b6859c4
chore: add CHANGELOG
Sep 28, 2021
49812c9
fix: add relevant readme files
Sep 29, 2021
ec60275
Merge branch 'main' of github.com:open-o11y/opentelemetry-collector-c…
Sep 29, 2021
5933413
Merge branch 'main' of github.com:open-o11y/opentelemetry-collector-c…
Oct 1, 2021
6536535
fix: includes aws components only
Oct 1, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions CHANGELOG.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@

## 🛑 Breaking changes 🛑

- Remove squash on configtls.TLSClientSetting for AWS components (#5454)
- Move `k8sprocessor` to `k8sattributesprocessor`.
- Rename `k8s_tagger` configuration `k8sattributes`.
- filelog receiver: use empty value for `SeverityText` field instead of `"Undefined"` (#5423)
Expand Down
5 changes: 3 additions & 2 deletions extension/awsxrayproxy/README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -17,8 +17,9 @@ extensions:
awsxrayproxy:
endpoint: 0.0.0.0:2000
proxy_address: ""
insecure: false
server_name_override: ""
tls:
insecure: false
server_name_override: ""
region: ""
role_arn: ""
aws_endpoint: ""
Expand Down
5 changes: 3 additions & 2 deletions extension/awsxrayproxy/testdata/config.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,8 +3,9 @@ extensions:
awsxrayproxy/1:
endpoint: "0.0.0.0:1234"
proxy_address: "https://proxy.proxy.com"
insecure: true
server_name_override: "something"
tls:
insecure: true
server_name_override: "something"
region: "us-west-1"
role_arn: "arn:aws:iam::123456789012:role/awesome_role"
aws_endpoint: "https://another.aws.endpoint.com"
Expand Down
2 changes: 1 addition & 1 deletion internal/aws/proxy/cfg.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@ type Config struct {

// TLSSetting struct exposes TLS client configuration when forwarding
// calls to the AWS X-Ray backend.
TLSSetting configtls.TLSClientSetting `mapstructure:",squash"`
TLSSetting configtls.TLSClientSetting `mapstructure:"tls,omitempty"`

// Region is the AWS region the local TCP server forwards requests to.
Region string `mapstructure:"region"`
Expand Down
5 changes: 3 additions & 2 deletions receiver/awsxrayreceiver/README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -18,8 +18,9 @@ receivers:
proxy_server:
endpoint: 0.0.0.0:2000
proxy_address: ""
insecure: false
server_name_override: ""
tls:
insecure: false
server_name_override: ""
region: ""
role_arn: ""
aws_endpoint: ""
Expand Down
5 changes: 3 additions & 2 deletions receiver/awsxrayreceiver/testdata/config.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,8 +12,9 @@ receivers:
proxy_server:
endpoint: "0.0.0.0:1234"
proxy_address: "https://proxy.proxy.com"
insecure: true
server_name_override: "something"
tls:
insecure: true
server_name_override: "something"
region: "us-west-1"
role_arn: "arn:aws:iam::123456789012:role/awesome_role"
aws_endpoint: "https://another.aws.endpoint.com"
Expand Down