Skip to content

Conversation

@ni-jfitzger
Copy link
Collaborator

@ni-jfitzger ni-jfitzger commented Jan 17, 2026

  • This contribution adheres to CONTRIBUTING.md.
  • [ ] I've updated CHANGELOG.md if applicable.
  • [ ] I've added tests applicable for this pull request

What does this Pull Request accomplish?

We've upgraded the driver versions that are installed to our testing VM, nimibot. This change updates our documentation to reflect the change in driver versions that we use for testing.

We've deployed the latest released driver runtime versions for everything but NI Switch Executive, which is held back by a blocking bug. We also deployed the latest gRPC Device Server, but we don't note the version of that in our documentation.

List issues fixed by this Pull Request below, if any.

None

What testing has been done?

PR Checks

@codecov-commenter
Copy link

codecov-commenter commented Jan 17, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 90.08%. Comparing base (8b2f991) to head (5577073).

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #2158   +/-   ##
=======================================
  Coverage   90.07%   90.08%           
=======================================
  Files          71       71           
  Lines       18666    18666           
=======================================
+ Hits        16814    16815    +1     
+ Misses       1852     1851    -1     
Flag Coverage Δ
codegenunittests 84.44% <ø> (ø)
nidcpowersystemtests 94.65% <ø> (+0.04%) ⬆️
nidcpowerunittests 89.53% <ø> (ø)
nidigitalsystemtests 92.26% <ø> (ø)
nidigitalunittests 68.44% <ø> (ø)
nidmmsystemtests 92.72% <ø> (ø)
nifakeunittests 85.51% <ø> (ø)
nifgensystemtests 94.61% <ø> (ø)
nimodinstsystemtests 73.85% <ø> (ø)
nimodinstunittests 94.20% <ø> (ø)
nirfsgsystemtests 82.40% <ø> (ø)
niscopesystemtests 92.94% <ø> (ø)
niscopeunittests 43.20% <ø> (ø)
nisesystemtests 91.50% <ø> (ø)
niswitchsystemtests 82.03% <ø> (ø)
nitclksystemtests 94.87% <ø> (ø)
nitclkunittests 98.26% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
see 1 file with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8b2f991...5577073. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Collaborator

@vnktshr21 vnktshr21 left a comment

Choose a reason for hiding this comment

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

Maybe we can specify the basis of the choice of driver versions in the PR description? In this case, we are upgrading to the last released versions at the time of this change, right?

@ni-jfitzger
Copy link
Collaborator Author

Hmm ... looks like I forgot to deploy the driver runtime update, first.
We should wait on that and I'll rerun the system tests, when it's done.

@ni-jfitzger
Copy link
Collaborator Author

Maybe we can specify the basis of the choice of driver versions in the PR description? In this case, we are upgrading to the last released versions at the time of this change, right?

For everything except NI Switch Executive, yes.

@ni-jfitzger ni-jfitzger merged commit fb4aee8 into ni:master Jan 21, 2026
39 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