bugfix: no need to validate if placement mode is set for vineyard runtime (#5186)

TrafalgarZZZ · web-flow · commit b75c75bea33d · 2025-07-25T07:59:19.000Z
Signed-off-by: trafalgarzzz &lt;trafalgarz@outlook.com&gt;
diff --git a/pkg/ddc/vineyard/validate.go b/pkg/ddc/vineyard/validate.go
@@ -17,7 +17,7 @@ limitations under the License.
 package vineyard
 
 import (
-	"github.com/fluid-cloudnative/fluid/pkg/ddc/base"
+	fluiderrs "github.com/fluid-cloudnative/fluid/pkg/errors"
 	cruntime "github.com/fluid-cloudnative/fluid/pkg/runtime"
 )
 
@@ -29,9 +29,8 @@ func (e *VineyardEngine) Validate(ctx cruntime.ReconcileRequestContext) (err err
 		return err
 	}
 
-	err = base.ValidateRuntimeInfo(runtimeInfo)
-	if err != nil {
-		return err
+	if len(runtimeInfo.GetOwnerDatasetUID()) == 0 {
+		return fluiderrs.NewTemporaryValidationFailed("OwnerDatasetUID is not set in runtime info, this is usually a temporary state, retrying")
 	}
 
 	// TODO: impl validation logic for VineyardEngine

Original file line number	Diff line number	Diff line change
`@@ -17,7 +17,7 @@ limitations under the License.`
`17`	`17`	`package vineyard`
`18`	`18`
`19`	`19`	`import (`
`20`		`- "github.com/fluid-cloudnative/fluid/pkg/ddc/base"`
	`20`	`+ fluiderrs "github.com/fluid-cloudnative/fluid/pkg/errors"`
`21`	`21`	`cruntime "github.com/fluid-cloudnative/fluid/pkg/runtime"`
`22`	`22`	`)`
`23`	`23`
`@@ -29,9 +29,8 @@ func (e *VineyardEngine) Validate(ctx cruntime.ReconcileRequestContext) (err err`
`29`	`29`	`return err`
`30`	`30`	`}`
`31`	`31`
`32`		`- err = base.ValidateRuntimeInfo(runtimeInfo)`
`33`		`- if err != nil {`
`34`		`- return err`
	`32`	`+ if len(runtimeInfo.GetOwnerDatasetUID()) == 0 {`
	`33`	`+ return fluiderrs.NewTemporaryValidationFailed("OwnerDatasetUID is not set in runtime info, this is usually a temporary state, retrying")`
`35`	`34`	`}`
`36`	`35`
`37`	`36`	`// TODO: impl validation logic for VineyardEngine`