-
Notifications
You must be signed in to change notification settings - Fork 5.6k
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
feat(inputs.docker): Support swarm jobs #16292
Conversation
@zomfg could you please rebase this PR on latest master to get CircleCI going? |
e26ace2
to
23aef88
Compare
Download PR build artifacts for linux_amd64.tar.gz, darwin_arm64.tar.gz, and windows_amd64.zip. 📦 Click here to get additional PR build artifactsArtifact URLs |
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.
Thank you @zomfg! I really think this mode iffery should be a switch statement but it feels wrong to put that burden on you. ;-) If you feel the motivation to change this in a separate PR your help is welcome!
Summary
Gather info about swarm jobs like for normal swarm services and don't flood logs with errors about "unsupported replica mode"
relevant test configs
telegraf.conf
compose.yml
output sample
Checklist
Related issues
resolves #16291