-
Notifications
You must be signed in to change notification settings - Fork 612
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
adding check to validate default config file path for Windows #2866
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.
LGTM!
won't this break every customer currently using |
I dont think we ask customers to use /etc/ecs_container_agent/config.json directly anywhere in the docs. We advertise the env variable and ask customers to set this env variable if they want to use a custom config.json |
e5874c9
Summary
The PR adds a check to validate the default agent config file path on Windows.
Currently, the default config file path on Windows is
"C:/etc/ecs_container_agent/config.json"
. We're adding a check to validate whether the admin has created the config file or not. Note that this check is not done if the envECS_AGENT_CONFIG_FILE_PATH
is set, since it overrides the default config file.Licensing
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.