Skip to content

Conversation

@dependabot
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Apr 14, 2024

Bumps github.com/NVIDIA/go-nvml from 0.12.0-3 to 0.12.0-4.

Commits
  • 1fa43fd Replace Library interface with Extensions interface
  • d72aa68 Add a mock DGXA100 server that implements parts of the NVML interface
  • 9a1711a Add mocks for all interfaces under pkg/nvml/mock
  • 6f20742 Add ability to generate interfaces and package methods from library
  • b34e706 Ensure that exported structs reference interface types not private types
  • f01937e Fix API call for GetVgpuCompatibility
  • 3d3e863 Convert all uppercase local variables to lowercase
  • 80b4064 Convert GpmSample into an interface instead of a concrete type
  • 630d5a6 Fix API call for GpmSampleAlloc
  • fbdfe74 Convert ComputeInstance into an interface instead of a concrete type
  • Additional commits viewable in compare view

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot show <dependency name> ignore conditions will show all of the ignore conditions of the specified dependency
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

@dependabot dependabot bot added the dependencies Issue/PR Pull about a dependency file label Apr 14, 2024
dependabot bot and others added 4 commits April 18, 2024 14:53
Bumps [github.com/NVIDIA/go-nvml](https://github.com/NVIDIA/go-nvml) from 0.12.0-3 to 0.12.0-4.
- [Commits](NVIDIA/go-nvml@v0.12.0-3...v0.12.0-4)

---
updated-dependencies:
- dependency-name: github.com/NVIDIA/go-nvml
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Signed-off-by: Evan Lezar <elezar@nvidia.com>
Signed-off-by: Evan Lezar <elezar@nvidia.com>
Signed-off-by: Evan Lezar <elezar@nvidia.com>
@elezar elezar force-pushed the dependabot/go_modules/main/github.com/NVIDIA/go-nvml-0.12.0-4 branch from e9444cb to d5f6e6f Compare April 18, 2024 12:53
@elezar
Copy link
Member

elezar commented Apr 18, 2024

@klueska I have updated this to also use the go-nvml interfaces instead of those from go-nvlib. Please take a look.

@tariq1890
Copy link
Contributor

@elezar Wouldn't the right thing to do here be "updating the go-nvlib interface methods to comply with the changes in go-nvml"?

@elezar
Copy link
Member

elezar commented Apr 19, 2024

@elezar Wouldn't the right thing to do here be "updating the go-nvlib interface methods to comply with the changes in go-nvml"?

We didn't update the go-nvlib interface methods from the perspective of this caller. We changed the imports from go-nvlib/pkg/nvml to go-nvml/pkg/nvml -- with the interfaces now existing there.

@elezar elezar merged commit 0aed9a1 into main Apr 19, 2024
@elezar elezar deleted the dependabot/go_modules/main/github.com/NVIDIA/go-nvml-0.12.0-4 branch April 19, 2024 09:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Issue/PR Pull about a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants