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

docs: add documentation for CDI #4799

Merged
merged 1 commit into from
Jan 19, 2024
Merged

docs: add documentation for CDI #4799

merged 1 commit into from
Jan 19, 2024

Conversation

dvdksn
Copy link
Contributor

@dvdksn dvdksn commented Jan 18, 2024

Closes #4525

- What I did

  • Add section about cdi-spec-dirs daemon configuration
  • Add subsection about cdi in --device section for docker run
  • Update docker info output example

- How I did it

- How to verify it

- Description for the changelog

- A picture of a cute animal (not mandatory but encouraged)
🧸

@dvdksn dvdksn force-pushed the docs-cdi branch 3 times, most recently from 1009d8a to e5f5dc1 Compare January 18, 2024 12:30
@codecov-commenter
Copy link

codecov-commenter commented Jan 18, 2024

Codecov Report

Merging #4799 (852d198) into master (e758fe5) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #4799   +/-   ##
=======================================
  Coverage   59.63%   59.63%           
=======================================
  Files         287      287           
  Lines       24777    24777           
=======================================
  Hits        14775    14775           
  Misses       9114     9114           
  Partials      888      888           

- Add section about cdi-spec-dirs daemon configuration
- Add subsection about cdi in --device section for docker run
- Update `docker info` output example

Signed-off-by: David Karlsson <[email protected]>
Copy link
Contributor

@sam-thibault sam-thibault left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@thaJeztah thaJeztah left a comment

Choose a reason for hiding this comment

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

LGTM

@thaJeztah thaJeztah merged commit ca95bad into docker:master Jan 19, 2024
77 checks passed
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.

4 participants