Automated Code Change

h-joo · Capirca Team · commit 47d9bdb2c1a0 · 2026-07-02T02:30:36.000-07:00
PiperOrigin-RevId: 941568242
diff --git a/capirca/aclgen.py b/capirca/aclgen.py
@@ -143,7 +143,7 @@ def SkipLines(text, skip_line_func=False):
   """
   if not skip_line_func:
     return text
-  return [x for x in text if not skip_line_func(x)]
+  return [x for x in text if not skip_line_func(x)]  # pyrefly: ignore[not-callable]
 
 
 def RenderFile(base_directory: str, input_file: pathlib.Path,
diff --git a/capirca/lib/arista_tp.py b/capirca/lib/arista_tp.py
@@ -217,7 +217,7 @@ def __str__(self):
 
     term_block.append([
         TERM_INDENT,
-        "match %s %s" % (self.term.name, family_keywords["addr_fam"]), False
+        "match %s %s" % (self.term.name, family_keywords["addr_fam"]), False  # pyrefly: ignore[unsupported-operation]
     ])
 
     term_af = self.AF_MAP.get(self.term_type)
diff --git a/capirca/lib/aruba.py b/capirca/lib/aruba.py
@@ -153,7 +153,7 @@ def __str__(self):
       str_tok.append(dst_addr_token)
 
       str_tok.append(dst_port)
-      str_tok.append(self._ACTIONS.get(self.term.action[0]))
+      str_tok.append(self._ACTIONS.get(self.term.action[0]))  # pyrefly: ignore[bad-argument-type]
       ret_str.append(' '.join(t for t in str_tok if t))
 
     self.netdestinations = netdestinations
diff --git a/capirca/lib/cisco.py b/capirca/lib/cisco.py
@@ -766,7 +766,7 @@ def __init__(self, term, af=4, proto_int=True, enable_dsmo=False,
       self.text_af = 'inet'
     else:
       self.text_af = 'inet6'
-    self.ALLOWED_PROTO_STRINGS.extend([self.PROTO_MAP.get(x)
+    self.ALLOWED_PROTO_STRINGS.extend([self.PROTO_MAP.get(x)  # pyrefly: ignore[bad-argument-type]
                                        for x in self.ALWAYS_PROTO_NUM])
 
   def __str__(self):
@@ -978,7 +978,7 @@ def __str__(self):
                   ret_str.extend(
                       self._TermletToStr(
                           sequence,
-                          action,
+                          action,  # pyrefly: ignore[unbound-name]
                           self._FormatProto(proto, self.af),
                           saddr,
                           self._FormatPort(sport, proto),
diff --git a/capirca/lib/ciscoasa.py b/capirca/lib/ciscoasa.py
@@ -198,7 +198,7 @@ def __str__(self):
 
     return '\n'.join(ret_str)
 
-  def _TermletToStr(self, filter_name, action, proto, saddr, sport, daddr,
+  def _TermletToStr(self, filter_name, action, proto, saddr, sport, daddr,  # pyrefly: ignore[bad-override]
                     dport, icmp_type, option):
     """Take the various compenents and turn them into a cisco acl line.
 
diff --git a/capirca/lib/fortigate.py b/capirca/lib/fortigate.py
@@ -334,7 +334,7 @@ def add_address_to_fw_addrgrps(
 
     if address_exclude_v6:
       raise FortiGateValueError(
-          'Exclude IPv6 address is unsupported: {}'.join([f'{x.with_prefix_len}' for x in address_exclude_v6]))
+          'Exclude IPv6 address is unsupported: {}'.join([f'{x.with_prefix_len}' for x in address_exclude_v6]))  # pyrefly: ignore[missing-attribute]
 
     addr_names = []
     if address_v4 or address_exclude_v4:
diff --git a/capirca/lib/gce.py b/capirca/lib/gce.py
@@ -581,7 +581,7 @@ def _TranslatePolicy(self, pol, exp_info):
                   'Attribute count (%d) for %s exceeded the maximum (%d)' %
                   (total_attribute_count, filter_name, max_attribute_count))
             self.gce_policies.append(rules)
-    logging.info('Total rule count of policy %s is: %d', filter_name,
+    logging.info('Total rule count of policy %s is: %d', filter_name,  # pyrefly: ignore[unbound-name]
                  total_rule_count)
     logging.info('Total attribute count of policy %s is: %d', filter_name,
                  total_attribute_count)
diff --git a/capirca/lib/gce_vpc_tf.py b/capirca/lib/gce_vpc_tf.py
@@ -599,7 +599,7 @@ def _TranslatePolicy(self, pol, exp_info):
                   (total_attribute_count, filter_name, max_attribute_count))
             self.terraform_resources['google_compute_firewall'].append(rules)
             priority_counter += len(rules)
-    logging.info('Total rule count of policy %s is: %d', filter_name,
+    logging.info('Total rule count of policy %s is: %d', filter_name,  # pyrefly: ignore[unbound-name]
                  total_rule_count)
     logging.info('Total attribute count of policy %s is: %d', filter_name,
                  total_attribute_count)
diff --git a/capirca/lib/gcp_hf.py b/capirca/lib/gcp_hf.py
@@ -298,11 +298,11 @@ def ConvertToDict(self, priority_index):
       for daddr_chunk in destination_address_chunks:
         rule = copy.deepcopy(term_dict)
         if self.api_version == 'ga':
-          rule['match'][dest_ip_range] = [
+          rule['match'][dest_ip_range] = [  # pyrefly: ignore[unsupported-operation]
               daddr.with_prefixlen for daddr in daddr_chunk
           ]
         else:
-          rule['match']['config'][dest_ip_range] = [
+          rule['match']['config'][dest_ip_range] = [  # pyrefly: ignore[bad-index]
               daddr.with_prefixlen for daddr in daddr_chunk
           ]
         rule['priority'] = priority_index
@@ -345,11 +345,11 @@ def ConvertToDict(self, priority_index):
       for saddr_chunk in source_address_chunks:
         rule = copy.deepcopy(term_dict)
         if self.api_version == 'ga':
-          rule['match'][src_ip_range] = [
+          rule['match'][src_ip_range] = [  # pyrefly: ignore[unsupported-operation]
               saddr.with_prefixlen for saddr in saddr_chunk
           ]
         else:
-          rule['match']['config'][src_ip_range] = [
+          rule['match']['config'][src_ip_range] = [  # pyrefly: ignore[bad-index]
               saddr.with_prefixlen for saddr in saddr_chunk
           ]
         rule['priority'] = priority_index
@@ -511,7 +511,7 @@ def _TranslatePolicy(self, pol, exp_info):
 
       # If compute api_version is beta, set type to FIREWALL
       if api_version == 'beta':
-        policy['type'] = 'FIREWALL'
+        policy['type'] = 'FIREWALL'  # pyrefly: ignore[unsupported-operation]
 
       # Handle mixed for each indvidual term as inet and inet6.
       # inet/inet6 are treated the same.
@@ -571,7 +571,7 @@ def _TranslatePolicy(self, pol, exp_info):
       raise ExceededCostError(
           'Policy cost (%d) for %s reached the '
           'maximum (%d)' %
-          (total_cost, policy[display_name], policies_max_cost))
+          (total_cost, policy[display_name], policies_max_cost))  # pyrefly: ignore[unbound-name]
 
     self.policies.append(policy)
 
@@ -581,7 +581,7 @@ def _TranslatePolicy(self, pol, exp_info):
 
     if total_cost > 0:
       logging.info('Policy %s quota cost: %d',
-                   policy[display_name], total_cost)
+                   policy[display_name], total_cost)  # pyrefly: ignore[unbound-name]
 
 
 def GetRuleTupleCount(dict_term: Dict[str, Any], api_version):
diff --git a/capirca/lib/juniper.py b/capirca/lib/juniper.py
@@ -269,12 +269,12 @@ def __str__(self, term_type_table=None):
         elif opt.startswith('established'):
           if self.term.protocol == ['tcp']:
             if 'tcp-established;' not in from_str:
-              from_str.append(family_keywords['tcp-est'] + ';')
+              from_str.append(family_keywords['tcp-est'] + ';')  # pyrefly: ignore[unsupported-operation]
 
         # if tcp-established specified, but more than just tcp is included
         # in the protocols, raise an error
         elif opt.startswith('tcp-established'):
-          flag = family_keywords['tcp-est'] + ';'
+          flag = family_keywords['tcp-est'] + ';'  # pyrefly: ignore[unsupported-operation]
           if self.term.protocol == ['tcp']:
             if flag not in from_str:
               from_str.append(flag)
@@ -347,7 +347,7 @@ def __str__(self, term_type_table=None):
         address = summarizer.Summarize(address)
 
       if address:
-        config.Append('%s {' % family_keywords['addr'])
+        config.Append('%s {' % family_keywords['addr'])  # pyrefly: ignore[unsupported-operation]
         for addr in address:
           for comment in self._Comment(addr):
             config.Append('%s' % comment)
@@ -373,7 +373,7 @@ def __str__(self, term_type_table=None):
         src_addr, src_addr_ex = self._MinimizePrefixes(src_addr, src_addr_ex)
 
       if src_addr:
-        config.Append('%s {' % family_keywords['saddr'])
+        config.Append('%s {' % family_keywords['saddr'])  # pyrefly: ignore[unsupported-operation]
         for addr in src_addr:
           for comment in self._Comment(addr):
             config.Append('%s' % comment)
@@ -407,7 +407,7 @@ def __str__(self, term_type_table=None):
         dst_addr, dst_addr_ex = self._MinimizePrefixes(dst_addr, dst_addr_ex)
 
       if dst_addr:
-        config.Append('%s {' % family_keywords['daddr'])
+        config.Append('%s {' % family_keywords['daddr'])  # pyrefly: ignore[unsupported-operation]
         for addr in dst_addr:
           for comment in self._Comment(addr):
             config.Append('%s' % comment)
@@ -507,7 +507,7 @@ def __str__(self, term_type_table=None):
         if 'icmpv6' in self.term.protocol:
           loc = self.term.protocol.index('icmpv6')
           self.term.protocol[loc] = 'icmp6'
-        config.Append(family_keywords['protocol'] + ' ' +
+        config.Append(family_keywords['protocol'] + ' ' +  # pyrefly: ignore[unsupported-operation]
                       self._Group(self.term.protocol))
 
       # protocol
@@ -516,7 +516,7 @@ def __str__(self, term_type_table=None):
         if 'icmpv6' in self.term.protocol_except:
           loc = self.term.protocol_except.index('icmpv6')
           self.term.protocol_except[loc] = 'icmp6'
-        config.Append(family_keywords['protocol-except'] + ' ' +
+        config.Append(family_keywords['protocol-except'] + ' ' +  # pyrefly: ignore[unsupported-operation]
                       self._Group(self.term.protocol_except))
 
       if self.term.traffic_type:
@@ -686,7 +686,7 @@ def __str__(self, term_type_table=None):
       # If there is a routing-instance defined, skip reject/accept/etc actions.
       if not self.term.routing_instance:
         for action in self.term.action:
-          config.Append(self.ACTIONS.get(action) + ';')
+          config.Append(self.ACTIONS.get(action) + ';')  # pyrefly: ignore[unsupported-operation]
 
       # DSCP SET
       if self.term.dscp_set:
diff --git a/capirca/lib/junipermsmpc.py b/capirca/lib/junipermsmpc.py
@@ -300,7 +300,7 @@ def __str__(self, term_type_table=None):
       ret_str.Append('then {')
       # ACTION
       for action in self.term.action:
-        ret_str.Append(self._ACTIONS.get(str(action)) + ';')
+        ret_str.Append(self._ACTIONS.get(str(action)) + ';')  # pyrefly: ignore[unsupported-operation]
       if self.term.logging and 'disable' not in [
           x.value for x in self.term.logging
       ]:
diff --git a/capirca/lib/junipersrx.py b/capirca/lib/junipersrx.py
@@ -196,7 +196,7 @@ def __str__(self):
 
       # VPN target can be only specified when ACTION is accept
       if str(action) == 'accept' and self.term.vpn:
-        ret_str.IndentAppend(5, self.ACTIONS.get(
+        ret_str.IndentAppend(5, self.ACTIONS.get(  # pyrefly: ignore[unsupported-operation]
             str(action)) + ' {')
         ret_str.IndentAppend(6, 'tunnel {')
         ret_str.IndentAppend(7, 'ipsec-vpn %s;' % self.term.vpn[0])
@@ -206,7 +206,7 @@ def __str__(self):
         ret_str.IndentAppend(6, '}')
         ret_str.IndentAppend(5, '}')
       else:
-        ret_str.IndentAppend(5, self.ACTIONS.get(str(action)) + ';')
+        ret_str.IndentAppend(5, self.ACTIONS.get(str(action)) + ';')  # pyrefly: ignore[unsupported-operation]
 
       # DSCP SET
       if self.term.dscp_set:
diff --git a/capirca/lib/k8s.py b/capirca/lib/k8s.py
@@ -209,7 +209,7 @@ def ConvertToDict(self):
           if peer_selector['ipBlock'].get('except') is None:
             peer_selector['ipBlock']['except'] = []
 
-          peer_selector['ipBlock']['except'].append(str(exclude))
+          peer_selector['ipBlock']['except'].append(str(exclude))  # pyrefly: ignore[missing-attribute]
         peer_selectors.append(peer_selector)
       peer_selector_key = 'from'
     else:
@@ -219,7 +219,7 @@ def ConvertToDict(self):
           if peer_selector['ipBlock'].get('except') is None:
             peer_selector['ipBlock']['except'] = []
 
-          peer_selector['ipBlock']['except'].append(str(exclude))
+          peer_selector['ipBlock']['except'].append(str(exclude))  # pyrefly: ignore[missing-attribute]
         peer_selectors.append(peer_selector)
       peer_selector_key = 'to'
 
@@ -350,7 +350,7 @@ def _TranslatePolicy(self, pol, exp_info):
           total_rule_count += 1
           self.network_policies.append(term_dict)
 
-    logging.info('Total rule count of policy %s is: %d', filter_name,
+    logging.info('Total rule count of policy %s is: %d', filter_name,  # pyrefly: ignore[unbound-name]
                  total_rule_count)
 
   def __str__(self):
diff --git a/capirca/lib/nacaddr.py b/capirca/lib/nacaddr.py
@@ -113,7 +113,7 @@ def AddComment(self, comment=''):
     else:
       self.text = comment
 
-  def supernet(self, prefixlen_diff=1):
+  def supernet(self, prefixlen_diff=1):  # pyrefly: ignore[bad-override]
     """Override ipaddress.IPv4 supernet so we can maintain comments.
 
     See ipaddress.IPv4.Supernet for complete documentation.
@@ -178,7 +178,7 @@ def __deepcopy__(self, memo):
     result.parent_token = self.parent_token
     return result
 
-  def supernet(self, prefixlen_diff=1):
+  def supernet(self, prefixlen_diff=1):  # pyrefly: ignore[bad-override]
     """Override ipaddress.IPv6Network supernet so we can maintain comments.
 
     See ipaddress.IPv6Network.Supernet for complete documentation.
diff --git a/capirca/lib/naming.py b/capirca/lib/naming.py
@@ -133,7 +133,7 @@ def __init__(self, naming_dir=None, naming_file=None, naming_type=None):
                               re.IGNORECASE | re.DOTALL)
     self.token_re = re.compile(r'(^[-_A-Z0-9]+$)', re.IGNORECASE)
     if naming_file and naming_type:
-      filename = os.path.sep.join([naming_dir, naming_file])
+      filename = os.path.sep.join([naming_dir, naming_file])  # pyrefly: ignore[no-matching-overload]
       with open(filename, 'r') as file_handle:
         self._ParseFile(file_handle, naming_type)
     elif naming_dir:
diff --git a/capirca/lib/nsxt.py b/capirca/lib/nsxt.py
@@ -122,7 +122,7 @@ def get(self):
       services = []
       for icmp_type in self.icmp_types:
         new_service = service.copy()
-        new_service['icmp_type'] = icmp_type
+        new_service['icmp_type'] = icmp_type  # pyrefly: ignore[unsupported-operation]
         services.append(new_service)
       return services
 
@@ -142,13 +142,13 @@ def format_port_range(ports: Tuple[str, str]) -> str:
 
       if self.source_ports:
         source_ports = [format_port_range(p) for p in self.source_ports]
-        service['source_ports'] = source_ports
+        service['source_ports'] = source_ports  # pyrefly: ignore[bad-assignment]
 
       if self.destination_ports:
         destination_ports = [
             format_port_range(p) for p in self.destination_ports
         ]
-        service['destination_ports'] = destination_ports
+        service['destination_ports'] = destination_ports  # pyrefly: ignore[bad-assignment]
       return [service]
     else:
       return []
@@ -384,7 +384,7 @@ def __str__(self):
       protocol = [self.PROTO_MAP.get(p) for p in self.term.protocol]
       for proto in protocol:
         if proto != 'any':
-          service = ServiceEntries(proto, self.term.source_port,
+          service = ServiceEntries(proto, self.term.source_port,  # pyrefly: ignore[bad-argument-type]
                                    self.term.destination_port, icmp_types)
           services.extend(service.get())
       rule['service_entries'] = services
diff --git a/capirca/lib/nsxv.py b/capirca/lib/nsxv.py
@@ -343,7 +343,7 @@ def __str__(self):
           services.append(self._ServiceToString(proto,
                                                 source_port,
                                                 destination_port,
-                                                icmp_types))
+                                                icmp_types))  # pyrefly: ignore[unbound-name]
       services.append('</services>')
 
     service = ''
@@ -650,4 +650,4 @@ def __str__(self):
 
       target_as_xml = xml.dom.minidom.parseString(''.join(target))
     # TODO(robankeny) utf encoding with refactoring after migration to py3
-    return target_as_xml.toprettyxml(indent='  ')
+    return target_as_xml.toprettyxml(indent='  ')  # pyrefly: ignore[unbound-name]
diff --git a/capirca/lib/openconfig.py b/capirca/lib/openconfig.py
@@ -94,7 +94,7 @@ def ConvertToDict(self):
 
     # Convert the integer to the proper openconfig schema name str, ipv4/ipv6.
     term_af = self.AF_MAP.get(self.inet_version)
-    family = self.AF_RENAME[term_af]
+    family = self.AF_RENAME[term_af]  # pyrefly: ignore[bad-index]
 
     # Action
     action = self.ACTION_MAP[self.term.action[0]]
diff --git a/capirca/lib/paloaltofw.py b/capirca/lib/paloaltofw.py
@@ -1110,7 +1110,7 @@ def __str__(self):
             log = etree.SubElement(entry, "log-end")
             log.text = "yes"
 
-    if no_addr_obj:
+    if no_addr_obj:  # pyrefly: ignore[unbound-name]
       address_book_groups_dict = {}
       address_book_keys = {}
 
diff --git a/capirca/lib/policy.py b/capirca/lib/policy.py
@@ -908,7 +908,7 @@ def __str__(self):
     if self.timeout:
       ret_str.append('  timeout: %s' % self.timeout)
     if self.vpn:
-      vpn_name, pair_policy = self.vpn
+      vpn_name, pair_policy = self.vpn  # pyrefly: ignore[not-iterable]
       if pair_policy:
         ret_str.append(
             '  vpn: name = %s, pair_policy = %s' % (vpn_name, pair_policy)
diff --git a/capirca/lib/policy_simple.py b/capirca/lib/policy_simple.py
diff --git a/capirca/lib/port.py b/capirca/lib/port.py
diff --git a/capirca/lib/sonic.py b/capirca/lib/sonic.py
