Skip to content

Add debug-level log option for ctk-installer#1551

Merged
elezar merged 2 commits intoNVIDIA:mainfrom
jactor-sue:feat-debug-level-log
Feb 13, 2026
Merged

Add debug-level log option for ctk-installer#1551
elezar merged 2 commits intoNVIDIA:mainfrom
jactor-sue:feat-debug-level-log

Conversation

@jactor-sue
Copy link
Contributor

Enable debug-level log for nvidia-ctk-installer cmd by --debug or -d.

@copy-pr-bot
Copy link

copy-pr-bot bot commented Dec 19, 2025

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.


var availableRuntimes = map[string]struct{}{"docker": {}, "crio": {}, "containerd": {}}
var defaultLowLevelRuntimes = []string{"runc", "crun"}
var (
Copy link
Member

Choose a reason for hiding this comment

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

If you're going to make these changes please make them in a separate commit.

Copy link
Contributor Author

@jactor-sue jactor-sue Jan 26, 2026

Choose a reason for hiding this comment

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

Nothing changed here, auto-formated by the gofumpt formatter.

Copy link
Member

@elezar elezar left a comment

Choose a reason for hiding this comment

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

Thanks for the contribution.

I have made a proposal for using log-verbosity instead as well as some other changes that better align with some things that I've been considering recently.

Please feel free to squash / clean up as required.

@elezar elezar force-pushed the feat-debug-level-log branch from 5d80b24 to 6f4666c Compare February 2, 2026 12:51
@coveralls
Copy link

coveralls commented Feb 2, 2026

Pull Request Test Coverage Report for Build 21825020489

Details

  • 2 of 5 (40.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.002%) to 39.545%

Changes Missing Coverage Covered Lines Changed/Added Lines %
cmd/nvidia-ctk-installer/main.go 2 5 40.0%
Totals Coverage Status
Change from base Build 21761866307: -0.002%
Covered Lines: 5725
Relevant Lines: 14477

💛 - Coveralls

@elezar
Copy link
Member

elezar commented Feb 5, 2026

/ok-to-test 2cc32fd

@elezar elezar added this to the v1.19.0 milestone Feb 5, 2026
jactor-sue and others added 2 commits February 9, 2026 13:25
This change adds a --debug flag to the nvidia-ctk-installer
to set the log level for INSTALLATION to debug. This does not
change the log level of the installed toolkit.

Signed-off-by: Jactor Sue <jactor_sue@live.com>
Co-authored-by: Evan Lezar <elezar@nvidia.com>
Signed-off-by: Evan Lezar <elezar@nvidia.com>
@elezar elezar force-pushed the feat-debug-level-log branch from 2cc32fd to 1e37ca3 Compare February 9, 2026 12:26
@elezar elezar merged commit c5232e1 into NVIDIA:main Feb 13, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants