From eeed78169aaa14248192588bbc9406a958540a79 Mon Sep 17 00:00:00 2001
From: wenovus <wenbli@google.com>
Date: Thu, 3 Aug 2023 16:57:06 -0700
Subject: [PATCH] Now revert all model changes

---
 release/models/acl/openconfig-acl.yang | 28 ++++++++++++++------------
 1 file changed, 15 insertions(+), 13 deletions(-)

diff --git a/release/models/acl/openconfig-acl.yang b/release/models/acl/openconfig-acl.yang
index 79bca7c46..6b3977907 100644
--- a/release/models/acl/openconfig-acl.yang
+++ b/release/models/acl/openconfig-acl.yang
@@ -34,12 +34,12 @@ module openconfig-acl {
     packets should be handled. Entries have a type that indicates
     the type of match criteria, e.g., MAC layer, IPv4, IPv6, etc.";
 
-  oc-ext:openconfig-version "1.4.3";
+  oc-ext:openconfig-version "1.3.3";
 
   revision "2023-02-06" {
     description
       "Add clarifying comments on use of interface-ref.";
-    reference "1.4.3";
+    reference "1.3.3";
   }
 
   revision "2023-01-29" {
@@ -284,23 +284,25 @@ module openconfig-acl {
 
 
     leaf forwarding-action {
-      type binary;
+      type identityref {
+        base FORWARDING_ACTION;
+      }
       mandatory true;
       description
         "Specifies the forwarding action.  One forwarding action
         must be specified for each ACL entry";
     }
 
-    //leaf log-action {
-    //  type identityref {
-    //    base LOG_ACTION;
-    //  }
-    //  default LOG_NONE;
-    //  description
-    //    "Specifies the log action and destination for
-    //    matched packets.  The default is not to log the
-    //    packet.";
-    //}
+    leaf log-action {
+      type identityref {
+        base LOG_ACTION;
+      }
+      default LOG_NONE;
+      description
+        "Specifies the log action and destination for
+        matched packets.  The default is not to log the
+        packet.";
+    }
 
 
   }