-
Notifications
You must be signed in to change notification settings - Fork 619
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 SumoLogic logging driver #992
Conversation
Thank you! I was able to test this and get it working with a task definition like this: {
"family": "sumologic-test",
"containerDefinitions": [
{
"name": "hello",
"image": "ubuntu:latest",
"cpu": 100,
"memory": 512,
"essential": true,
"entryPoint": ["sh", "-c"],
"command": ["echo hello sumologic!"],
"logConfiguration" : {
"logDriver": "sumologic",
"options": {
"sumo-url":"<REDACTED>"
}
}
}
]
} Note that I did need to install the plugin and specify the |
Hello, It is giving me "Run tasks failed: Reasons: ["ATTRIBUTE"] " I added this to /etc/ecs/ecs.config and restart |
there should be no space between |
Hello, I solved the problem with However I see in the ECS agent log: but I do see This error went away with But now the task cannot restart: |
Hi @rh-lphan, If you're still running into trouble with the SumoLogic log driver, please feel free to open a new issue here on github with the relevant logs and configuration or engage AWS support directly. |
Summary
Add SumoLogic Docker Logging Driver support in docker client
Implementation details
Add
sumologic
in logging_drivers.goAdd 1.29 as minimum support API version
Testing
make release
)go build -out amazon-ecs-agent.exe ./agent
)make test
) passgo test -timeout=25s ./agent/...
) passmake run-integ-tests
) pass.\scripts\run-integ-tests.ps1
) passmake run-functional-tests
) pass.\scripts\run-functional-tests.ps1
) passNew tests cover the changes:
Description for the changelog
Licensing
This contribution is under the terms of the Apache 2.0 License: