Skip to content

Commit

Permalink
Remove gpu support (#821)
Browse files Browse the repository at this point in the history
* Remove gpu support

* Remove gpu from contrib
  • Loading branch information
jknipper committed Sep 5, 2023
1 parent a9473f3 commit 36551b8
Show file tree
Hide file tree
Showing 8 changed files with 0 additions and 364 deletions.
63 changes: 0 additions & 63 deletions contrib/nvidia-gpu/device-plugin-damonset.yaml

This file was deleted.

62 changes: 0 additions & 62 deletions contrib/nvidia-gpu/device-plugin.yaml

This file was deleted.

52 changes: 0 additions & 52 deletions contrib/nvidia-gpu/driver-daemonset.yaml

This file was deleted.

7 changes: 0 additions & 7 deletions pkg/controller/ground/bootstrap.go
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,6 @@ import (
"github.com/sapcc/kubernikus/pkg/controller/ground/bootstrap/ccm"
"github.com/sapcc/kubernikus/pkg/controller/ground/bootstrap/csi"
"github.com/sapcc/kubernikus/pkg/controller/ground/bootstrap/dns"
"github.com/sapcc/kubernikus/pkg/controller/ground/bootstrap/gpu"
"github.com/sapcc/kubernikus/pkg/controller/ground/bootstrap/network"
"github.com/sapcc/kubernikus/pkg/util"
"github.com/sapcc/kubernikus/pkg/version"
Expand Down Expand Up @@ -81,12 +80,6 @@ func SeedKluster(clients config.Clients, factories config.Factories, images vers
}
}

if ok, _ := util.KlusterVersionConstraint(kluster, ">= 1.10"); ok {
if err := gpu.SeedGPUSupport(kubernetes); err != nil {
return errors.Wrap(err, "seed GPU support")
}
}

if ok, _ := util.KlusterVersionConstraint(kluster, ">= 1.20"); ok {
dynamicKubernetes, err := clients.Satellites.DynamicClientFor(kluster)
if err != nil {
Expand Down
37 changes: 0 additions & 37 deletions pkg/controller/ground/bootstrap/gpu/gpu.go

This file was deleted.

129 changes: 0 additions & 129 deletions pkg/controller/ground/bootstrap/gpu/manifest.go

This file was deleted.

6 changes: 0 additions & 6 deletions pkg/templates/ignition.go
Original file line number Diff line number Diff line change
Expand Up @@ -108,12 +108,6 @@ func (i *ignition) GenerateNode(kluster *kubernikusv1.Kluster, pool *models.Node
isFlatcar := true
if pool != nil {
nodeLabels = append(nodeLabels, "ccloud.sap.com/nodepool="+pool.Name)
if strings.HasPrefix(pool.Flavor, "zg") {
nodeLabels = append(nodeLabels, "gpu=nvidia-tesla-v100")
}
if strings.HasPrefix(pool.Flavor, "zg") {
nodeTaints = append(nodeTaints, "nvidia.com/gpu=present:NoSchedule")
}
for _, userTaint := range pool.Taints {
nodeTaints = append(nodeTaints, userTaint)
}
Expand Down
8 changes: 0 additions & 8 deletions pkg/templates/ignition_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -121,12 +121,4 @@ func TestNodeLabels(t *testing.T) {
//Ensure we rendered the expected template
assert.Contains(t, string(data), fmt.Sprintf("--node-labels=ccloud.sap.com/nodepool=%s", pool.Name))
}

gpuPool := &models.NodePool{Name: "some-name", Flavor: "zghuh"}
data, err = Ignition.GenerateNode(kluster, gpuPool, "test", "abc123", &testKlusterSecret, false, imageRegistry, log.NewNopLogger())
if assert.NoError(t, err, "Failed to generate node") {
//Ensure we rendered the expected template
assert.Contains(t, string(data), fmt.Sprintf("--node-labels=ccloud.sap.com/nodepool=%s,gpu=nvidia-tesla-v100", pool.Name))
assert.Contains(t, string(data), fmt.Sprintf("--register-with-taints=nvidia.com/gpu=present:NoSchedule"))
}
}

0 comments on commit 36551b8

Please sign in to comment.