resolved conflicts

openconfig · Oct 22, 2024 · f52f297 · f52f297
1 parent dcbc957
commit f52f297
Show file tree

Hide file tree

Showing 5 changed files with 1,059 additions and 1,028 deletions.
diff --git a/feature/system/tests/system_base_test/g_protocol_test.go b/feature/system/tests/system_base_test/g_protocol_test.go
@@ -21,6 +21,7 @@ import (
 	"testing"
 	"time"
 
+	"github.com/openconfig/featureprofiles/internal/deviations"
 	"github.com/openconfig/ondatra"
 	"github.com/openconfig/ondatra/binding/introspect"
 	"google.golang.org/grpc"
@@ -58,8 +59,14 @@ func TestGNMIClient(t *testing.T) {
 	dut := ondatra.DUT(t, "dut")
 	conn := dialConn(t, dut, introspect.GNMI, 9339)
 	c := gpb.NewGNMIClient(conn)
-	if _, err := c.Get(context.Background(), &gpb.GetRequest{Encoding: gpb.Encoding_JSON_IETF, Path: []*gpb.Path{{Elem: []*gpb.PathElem{}}}}); err != nil {
-		t.Fatalf("gnmi.Get failed: %v", err)
+	if deviations.GnmiGetRequiresConfigType(dut) {
+		if _, err := c.Get(context.Background(), &gpb.GetRequest{Encoding: gpb.Encoding_JSON_IETF, Path: []*gpb.Path{{Elem: []*gpb.PathElem{}}}, Type: gpb.GetRequest_CONFIG}); err != nil {
+			t.Fatalf("gnmi.Get failed: %v", err)
+		}
+	} else {
+		if _, err := c.Get(context.Background(), &gpb.GetRequest{Encoding: gpb.Encoding_JSON_IETF, Path: []*gpb.Path{{Elem: []*gpb.PathElem{}}}}); err != nil {
+			t.Fatalf("gnmi.Get failed: %v", err)
+		}
 	}
 }
 

diff --git a/feature/system/tests/system_base_test/metadata.textproto b/feature/system/tests/system_base_test/metadata.textproto
@@ -11,5 +11,6 @@ platform_exceptions:  {
   }
   deviations:  {
     cli_takes_precedence_over_oc:  true
+    gnmi_get_requires_config_type: true
   }
 }
diff --git a/internal/deviations/deviations.go b/internal/deviations/deviations.go
@@ -1181,3 +1181,7 @@ func ChassisGetRPCUnsupported(dut *ondatra.DUTDevice) bool {
 func PowerDisableEnableLeafRefValidation(dut *ondatra.DUTDevice) bool {
 	return lookupDUTDeviations(dut).GetPowerDisableEnableLeafRefValidation()
 }
+
+func GnmiGetRequiresConfigType(dut *ondatra.DUTDevice) bool {
+	return lookupDUTDeviations(dut).GetGnmiGetRequiresConfigType()
+}
diff --git a/proto/metadata.proto b/proto/metadata.proto
@@ -628,6 +628,10 @@ message Metadata {
     // Leaf-ref validation for list keys which is enforced for Cisco and hence deviation
     // b/373581140
     bool power_disable_enable_leaf_ref_validation = 226;
+    // gnmi_get_requires_config_type is set to true for devices that require type to be specified as config
+    // in default it will be false
+    // Juniper: b/361547597
+    bool gnmi_get_requires_config_type = 227;
     // Reserved field numbers and identifiers.
     reserved 84, 9, 28, 20, 90, 97, 55, 89, 19, 36, 35, 40, 173;
   }