Skip to content

Commit

Permalink
Bump controllergen to v0.14
Browse files Browse the repository at this point in the history
Signed-off-by: Carlos Eduardo Arango Gutierrez <[email protected]>
  • Loading branch information
ArangoGutierrez committed Feb 15, 2024
1 parent fb86a83 commit 5e006c7
Show file tree
Hide file tree
Showing 5 changed files with 31 additions and 13 deletions.
8 changes: 6 additions & 2 deletions .github/workflows/go.yml
Original file line number Diff line number Diff line change
Expand Up @@ -5,9 +5,13 @@ name: Go

on:
push:
branches: [ "main" ]
branches:
- main
- release-*
pull_request:
branches: [ "main" ]
branches:
- main
- release-*

jobs:

Expand Down
2 changes: 1 addition & 1 deletion Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,7 @@ generate: controller-gen
CONTROLLER_GEN = $(PROJECT_DIR)/bin/controller-gen
.PHONY: controller-gen
controller-gen: ## Download controller-gen locally if necessary.
@GOBIN=$(PROJECT_DIR)/bin GO111MODULE=on $(GO_CMD) install sigs.k8s.io/controller-tools/cmd/controller-gen@v0.12.1
@GOBIN=$(PROJECT_DIR)/bin GO111MODULE=on $(GO_CMD) install sigs.k8s.io/controller-tools/cmd/controller-gen@v0.14.0

# Generate manifests e.g. CRD, RBAC etc.
.PHONY: manifests
Expand Down
2 changes: 1 addition & 1 deletion api/holodeck/v1alpha1/types.go
Original file line number Diff line number Diff line change
Expand Up @@ -189,7 +189,7 @@ type ExtraPortMapping struct {
HostPort int `json:"hostPort"`
}

type NVContainerToolKit struct {
type NVContainerToolkit struct {
Install bool `json:"install"`
// If not set the latest stable version will be used
// +optional
Expand Down
28 changes: 21 additions & 7 deletions api/holodeck/v1alpha1/zz_generated.deepcopy.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 2 additions & 2 deletions pkg/provisioner/dependency.go
Original file line number Diff line number Diff line change
Expand Up @@ -62,7 +62,7 @@ func buildDependencyGraph(env v1alpha1.Environment) ([]ProvisionFunc, error) {
}

// if container toolkit is enabled then add container toolkit and nvdriver to kubeadm
if env.Spec.NVContainerToolKit.Install {
if env.Spec.NVContainerToolkit.Install {
graph["kubeadm"] = append(graph["kubeadm"], "containerToolkit")
graph["kubeadm"] = append(graph["kubeadm"], "nvdriver")

Expand Down Expand Up @@ -107,7 +107,7 @@ func buildDependencyGraph(env v1alpha1.Environment) ([]ProvisionFunc, error) {
}

// If no kubernetes is requested, we move to container-toolkit
if env.Spec.NVContainerToolKit.Install {
if env.Spec.NVContainerToolkit.Install {
for _, f := range graph["containerToolkit"] {
ordered = append(ordered, functions[f])
}
Expand Down

0 comments on commit 5e006c7

Please sign in to comment.