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

envoy config specs #7157

Merged
merged 6 commits into from
Jul 21, 2020
Merged

envoy config specs #7157

merged 6 commits into from
Jul 21, 2020

Conversation

hithwen
Copy link
Contributor

@hithwen hithwen commented Jul 20, 2020

Depends on #7156

@hithwen hithwen force-pushed the julia/envoy-config-specs branch from 434657e to 0915e57 Compare July 20, 2020 14:40
## Set path if type is file.
## Set channel_path if type is windows_event.
## source - required - Attribute that defines which Integration sent the logs.
## service - required - The name of the service that generates the log.
Copy link
Member

Choose a reason for hiding this comment

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

Any way to remove that comment as service is not required anymore?

## This overrides the `proxy` setting in `init_config`.
##
## Set HTTP or HTTPS proxies. Use the `no_proxy` list
## Set HTTP or HTTPS proxies for all instances. Use the `no_proxy` list
Copy link
Member

Choose a reason for hiding this comment

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

This is incorrect, it just sets the proxy to the current instance. But this should be fixed in a separate PR.

@hithwen hithwen merged commit 9adfa42 into master Jul 21, 2020
@hithwen hithwen deleted the julia/envoy-config-specs branch July 21, 2020 10:32
github-actions bot pushed a commit that referenced this pull request Jul 21, 2020
* Add config specs
* Bump base package requirement 9adfa42
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.

2 participants