diff --git a/hook/main.go b/hook/main.go index 82ea26038040d1ad61aea0ca93f59679161b96bd..8355ca8d461f496892455f1d05257630f98b8d74 100644 --- a/hook/main.go +++ b/hook/main.go @@ -124,13 +124,15 @@ func parseRuntimeOptions(runtimeOptions string) ([]string, error) { } const maxLength = 128 if len(runtimeOptions) > maxLength { - return nil, fmt.Errorf("invalid runtime option") + hwlog.RunLog.Errorf("length of ASCEND_RUNTIME_OPTIONS value is invalid, its length: %v", len(runtimeOptions)) + return nil, fmt.Errorf("invalid runtime option, the length exceeds 128 characters") } for _, option := range strings.Split(runtimeOptions, ",") { option = strings.TrimSpace(option) if !isRuntimeOptionValid(option) { - return nil, fmt.Errorf("invalid runtime option") + hwlog.RunLog.Errorf("value of ASCEND_RUNTIME_OPTIONS is not in valid option list, value: %v", option) + return nil, fmt.Errorf("invalid runtime option of invalid input value") } parsedOptions = append(parsedOptions, option) @@ -213,6 +215,7 @@ func getValueByKey(data []string, name string) string { for _, s := range data { p := strings.SplitN(s, "=", splitNumber) if len(p) != kvPairSize { + hwlog.RunLog.Errorf("env is not key-value mode, env: %v", s) log.Panicln("environment error") } diff --git a/runtime/main_test.go b/runtime/main_test.go index a3920a41f11859d423426d91f233e0a8407827cd..6721a4b7a2692ba0fd1f8e8a91eb94ca04c468da 100644 --- a/runtime/main_test.go +++ b/runtime/main_test.go @@ -27,7 +27,7 @@ import ( "github.com/opencontainers/runtime-spec/specs-go" "github.com/stretchr/testify/assert" - "main/dcmi" + "ascend-docker-runtime/runtime/dcmi" ) const (