resolve coment

Author: AlinsRan

api/adc/types.go

@@ -128,9 +128,9 @@ type Route struct {
 }
 
 type Timeout struct {
-	Connect int64 `json:"connect"`
-	Read    int64 `json:"read"`
-	Send    int64 `json:"send"`
+	Connect float64 `json:"connect"`
+	Read    float64 `json:"read"`
+	Send    float64 `json:"send"`
 }
 
 type StreamRoute struct {

internal/controller/policies.go

@@ -60,17 +60,17 @@ func ProcessBackendTrafficPolicy(c client.Client,
 				condition := NewPolicyCondition(policy.Generation, true, "Policy has been accepted")
 				if sectionName != nil && !portNameExist[string(*sectionName)] {
 					condition = NewPolicyCondition(policy.Generation, false, fmt.Sprintf("SectionName %s not found in Service %s/%s", *sectionName, service.Namespace, service.Name))
-					goto record_status
+					if ok := SetAncestors(&policy.Status, tctx.ParentRefs, condition); ok {
+						updated = true
+					}
 				}
 				if p, ok := conflicts[key.String()]; ok && (p.Name == policy.Name && p.Namespace == policy.Namespace) {
 					condition = NewPolicyConflictCondition(policy.Generation, fmt.Sprintf("Unable to target Service %s/%s, because it conflicts with another BackendTrafficPolicy", service.Namespace, service.Name))
-					goto record_status
+					if ok := SetAncestors(&policy.Status, tctx.ParentRefs, condition); ok {
+						updated = true
+					}
 				}
 				conflicts[key.String()] = policy
-			record_status:
-				if ok := SetAncestors(&policy.Status, tctx.ParentRefs, condition); ok {
-					updated = true
-				}
 			}
 			if _, ok := tctx.BackendTrafficPolicies[types.NamespacedName{
 				Name:      policy.Name,
@@ -108,6 +108,10 @@ func SetAncestors(status *v1alpha1.PolicyStatus, parentRefs []gatewayv1.ParentRe
 func SetAncestorStatus(status *v1alpha1.PolicyStatus, ancestorStatus gatewayv1alpha2.PolicyAncestorStatus) bool {
 	for _, c := range status.Ancestors {
 		if c.AncestorRef == ancestorStatus.AncestorRef {
+			if len(c.Conditions) == 0 || len(ancestorStatus.Conditions) == 0 {
+				c.Conditions = ancestorStatus.Conditions
+				return true
+			}
 			if c.Conditions[0].ObservedGeneration < ancestorStatus.Conditions[0].ObservedGeneration {
 				c.Conditions = ancestorStatus.Conditions
 				return true

internal/provider/adc/translator/policies.go

@@ -41,9 +41,9 @@ func (t *Translator) attachBackendTrafficPolicyToUpstream(policy *v1alpha1.Backe
 	}
 	if policy.Spec.Timeout != nil {
 		upstream.Timeout = &adctypes.Timeout{
-			Connect: policy.Spec.Timeout.Connect.Duration.Milliseconds(),
-			Read:    policy.Spec.Timeout.Read.Duration.Milliseconds(),
-			Send:    policy.Spec.Timeout.Send.Duration.Milliseconds(),
+			Connect: policy.Spec.Timeout.Connect.Duration.Seconds(),
+			Read:    policy.Spec.Timeout.Read.Duration.Seconds(),
+			Send:    policy.Spec.Timeout.Send.Duration.Seconds(),
 		}
 	}
 	if policy.Spec.LoadBalancer != nil {