Skip to content

Add custom docker image info to nvidia plugin#9124

Merged
helenosheaa merged 2 commits intomasterfrom
add-to-nvidia-readme
Apr 14, 2021
Merged

Add custom docker image info to nvidia plugin#9124
helenosheaa merged 2 commits intomasterfrom
add-to-nvidia-readme

Conversation

@helenosheaa
Copy link
Copy Markdown
Member

resolves #8531
Add context on how to use nvidia plugin with a custom nvidia/cuda docker image

@telegraf-tiger telegraf-tiger bot added the feat Improvement on an existing feature such as adding a new setting/mode to an existing plugin label Apr 13, 2021
Copy link
Copy Markdown
Contributor

@telegraf-tiger telegraf-tiger bot left a comment

Choose a reason for hiding this comment

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

🤝 ✅ CLA has been signed. Thank you!

@helenosheaa helenosheaa requested a review from ssoroka April 13, 2021 19:24
@ssoroka
Copy link
Copy Markdown
Contributor

ssoroka commented Apr 13, 2021

I'm not sure that this is the only method, it's probably possible to use volume mount binding to inject the required binary without rebuilding the docker container.

@helenosheaa
Copy link
Copy Markdown
Member Author

@ssoroka Ok I've added about mount binding as an alternative option in there also with a link to the relevant docker page.

Copy link
Copy Markdown
Contributor

@telegraf-tiger telegraf-tiger bot left a comment

Choose a reason for hiding this comment

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

Copy link
Copy Markdown
Contributor

@ssoroka ssoroka left a comment

Choose a reason for hiding this comment

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

this is probably correct. bwahaha

@BlueskyFR
Copy link
Copy Markdown

May I make my own PR instead of this one with more detailed information?
I would like to add a Dockerfile example instead of a simple note at the end.

@helenosheaa
Copy link
Copy Markdown
Member Author

@BlueskyFR if you have a docker example you can add to the plugin folder we'd be happy to review a PR for it. I think it's also worth keeping this change to the README.

@helenosheaa helenosheaa merged commit 9169080 into master Apr 14, 2021
@helenosheaa helenosheaa deleted the add-to-nvidia-readme branch April 14, 2021 13:48
arstercz pushed a commit to arstercz/telegraf that referenced this pull request Aug 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feat Improvement on an existing feature such as adding a new setting/mode to an existing plugin

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add a new Telegraf:cuda Docker image to run the Nvidia-smi input plugin

4 participants