kelvin/warewulf
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

Merge pull request #669 from mslacken/FixContainerShow

Browse Source 
container show will only show valid containers
This commit is contained in:
Jonathon Anderson
2023-02-02 20:54:55 -07:00
committed by GitHub
parent 6e553992db b5725513b3
commit 963793d96e
1 changed files with 5 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
internal/pkg/api/container/container.go
View File

@@ -342,12 +342,15 @@ func ContainerShow(csp *wwapiv1.ContainerShowParameter) (response *wwapiv1.Conta
containerName := csp.ContainerName
if !container.ValidName(containerName) {
err = fmt.Errorf("%s is not a valid container", containerName)
err = fmt.Errorf("%s is not a valid container name", containerName)
return
}
rootFsDir := container.RootFsDir(containerName)
if !util.IsDir(rootFsDir) {
err = fmt.Errorf("%s is not a valid container", containerName)
return
}
kernelVersion := container.KernelVersion(containerName)
nodeDB, err := node.New()