Skip to content

Commit fd94416

Browse files
authored
feat(pkg, nvidia): remove unused nvidia query func, flags for scan command (#669)
Signed-off-by: Gyuho Lee <[email protected]> --------- Signed-off-by: Gyuho Lee <[email protected]>
1 parent 945dc05 commit fd94416

File tree

9 files changed

+6
-464
lines changed

9 files changed

+6
-464
lines changed

README.md

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -72,10 +72,6 @@ You can also start with the standalone mode and later switch to the managed opti
7272
sudo gpud login --token <LEPTON_AI_TOKEN>
7373
```
7474

75-
To access the local web UI, open https://localhost:15132 in your browser.
76-
77-
If run with `gpud up`, you may disable this local web UI by setting `FLAGS="--web-enable=false"` to the `/etc/default/gpud` environment file and restart the service.
78-
7975
#### Run GPUd with Kubernetes
8076

8177
See [gpud helm chart](./charts/gpud/README.md) to deploy GPUd in your Kubernetes cluster.

cmd/gpud/command/command.go

Lines changed: 2 additions & 37 deletions
Original file line numberDiff line numberDiff line change
@@ -32,22 +32,12 @@ var (
3232

3333
retentionPeriod time.Duration
3434

35-
netcheck bool
36-
diskcheck bool
37-
kmsgCheck bool
38-
3935
enableAutoUpdate bool
4036
autoUpdateExitCode int
4137

42-
kernelModulesToCheck cli.StringSlice
43-
38+
kernelModulesToCheck cli.StringSlice
4439
dockerIgnoreConnectionErrors bool
45-
46-
ibstatCommand string
47-
48-
checkInfiniBand bool
49-
50-
deprecatedWebEnable bool
40+
ibstatCommand string
5141
)
5242

5343
const (
@@ -198,11 +188,6 @@ sudo rm /etc/systemd/system/gpud.service
198188
Destination: &retentionPeriod,
199189
Value: config.DefaultRetentionPeriod.Duration,
200190
},
201-
&cli.BoolFlag{
202-
Name: "web-enable",
203-
Usage: "(DEPRECATED) enable local web interface",
204-
Destination: &deprecatedWebEnable,
205-
},
206191
cli.StringFlag{
207192
Name: "endpoint",
208193
Usage: "endpoint for control plane",
@@ -445,26 +430,6 @@ sudo rm /etc/systemd/system/gpud.service
445430
Usage: "set the logging level [debug, info, warn, error, fatal, panic, dpanic]",
446431
Destination: &logLevel,
447432
},
448-
&cli.BoolTFlag{
449-
Name: "netcheck",
450-
Usage: "enable network connectivity checks to global edge/derp servers (default: true)",
451-
Destination: &netcheck,
452-
},
453-
&cli.BoolTFlag{
454-
Name: "diskcheck",
455-
Usage: "enable disk checks (default: true)",
456-
Destination: &diskcheck,
457-
},
458-
&cli.BoolTFlag{
459-
Name: "kmsg-check",
460-
Usage: "enable kmsg checks (default: true)",
461-
Destination: &kmsgCheck,
462-
},
463-
&cli.BoolFlag{
464-
Name: "check-ib",
465-
Usage: "enable infiniband checks (default: false)",
466-
Destination: &checkInfiniBand,
467-
},
468433

469434
// only for testing
470435
cli.StringFlag{

cmd/gpud/command/scan.go

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -19,11 +19,7 @@ func cmdScan(cliContext *cli.Context) error {
1919
log.Logger = log.CreateLogger(zapLvl, logFile)
2020

2121
opts := []scan.OpOption{
22-
scan.WithNetcheck(netcheck),
23-
scan.WithDiskcheck(diskcheck),
24-
scan.WithKMsgCheck(kmsgCheck),
2522
scan.WithIbstatCommand(ibstatCommand),
26-
scan.WithCheckInfiniband(checkInfiniBand),
2723
}
2824
if zapLvl.Level() <= zap.DebugLevel { // e.g., info, warn, error
2925
opts = append(opts, scan.WithDebug(true))

e2e/e2e_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -62,7 +62,7 @@ var _ = Describe("[GPUD E2E]", Ordered, func() {
6262
Expect(err).ToNot(HaveOccurred(), "failed to mock lspci")
6363

6464
By("start gpud scan")
65-
cmd = exec.CommandContext(gCtx, os.Getenv("GPUD_BIN"), "scan", "--kmsg-check=false")
65+
cmd = exec.CommandContext(gCtx, os.Getenv("GPUD_BIN"), "scan")
6666
b, err := cmd.CombinedOutput()
6767
Expect(err).NotTo(HaveOccurred(), fmt.Sprintf("failed to run gpud scan:\n%s", string(b)))
6868
GinkgoLogr.Info("gpud scan successfully", "output", string(b))

pkg/nvidia-query/options.go

Lines changed: 0 additions & 51 deletions
This file was deleted.

pkg/nvidia-query/options_test.go

Lines changed: 0 additions & 59 deletions
This file was deleted.

0 commit comments

Comments
 (0)