kubernetes
/
kubectl
mirror of https://github.com/kubernetes/kubectl.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #109525 from jonyhy96/feat-wait 

feat: make jsonpath wait logics consistent with condition

Kubernetes-commit: 526650fc137b2e68267182b1b5116aa566be51e5
This commit is contained in:
Kubernetes Publisher 2022-10-30 09:12:43 -07:00
parent 8e2918ab8b 1906bda541
commit f0a2aee8eb
2 changed files with 5 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
pkg/cmd/wait/wait.go
View File

@ -229,7 +229,7 @@ func conditionFuncFor(condition string, errOut io.Writer) (ConditionFunc, error)
// newJSONPathParser will create a new JSONPath parser based on the jsonPathExpression // newJSONPathParser will create a new JSONPath parser based on the jsonPathExpression
func newJSONPathParser(jsonPathExpression string) (*jsonpath.JSONPath, error) { func newJSONPathParser(jsonPathExpression string) (*jsonpath.JSONPath, error) {
j := jsonpath.New("wait") j := jsonpath.New("wait").AllowMissingKeys(true)
if jsonPathExpression == "" { if jsonPathExpression == "" {
return nil, errors.New("jsonpath expression cannot be empty") return nil, errors.New("jsonpath expression cannot be empty")
} }
@ -624,6 +624,9 @@ func (j JSONPathWait) checkCondition(obj *unstructured.Unstructured) (bool, erro
if err != nil { if err != nil {
return false, err return false, err
} }
if len(parseResults) == 0 || len(parseResults[0]) == 0 {
return false, nil
}
if err := verifyParsedJSONPath(parseResults); err != nil { if err := verifyParsedJSONPath(parseResults); err != nil {
return false, err return false, err
} }
@ -637,9 +640,6 @@ func (j JSONPathWait) checkCondition(obj *unstructured.Unstructured) (bool, erro
// verifyParsedJSONPath verifies the JSON received from the API server is valid. // verifyParsedJSONPath verifies the JSON received from the API server is valid.
// It will only accept a single JSON // It will only accept a single JSON
func verifyParsedJSONPath(results [][]reflect.Value) error { func verifyParsedJSONPath(results [][]reflect.Value) error {
if len(results) == 0 {
return errors.New("given jsonpath expression does not match any value")
}
if len(results) > 1 { if len(results) > 1 {
return errors.New("given jsonpath expression matches more than one list") return errors.New("given jsonpath expression matches more than one list")
} }

2
pkg/cmd/wait/wait_test.go
View File

@ -1043,7 +1043,7 @@ func TestWaitForDifferentJSONPathExpression(t *testing.T) {
jsonPathExp: "{.foo.bar}", jsonPathExp: "{.foo.bar}",
jsonPathCond: "baz", jsonPathCond: "baz",
expectedErr: "foo is not found", expectedErr: "timed out waiting for the condition on theresource/foo-b6699dcfb-rnv7t",
}, },
{ {
name: "compare boolean JSONPath entry", name: "compare boolean JSONPath entry",