Terminal code mappings for RouteTable (#69)

Issue #, if available: [1362](aws-controllers-k8s/community#1362)

Description of changes:

- Removed `InvalidVpcId.NotFound`, `InvalidVpcId.Malformed` and `InvalidParameterValue` terminal codes for **Route Table** resource.
- Removed Integration test that exists for `InvalidVpcId.NotFound` terminal code.

I tried adding integration test for the `InvalidParameterValue` by passing an invalid parameter, but it leaves a dangling **VPC** resource which is incorrect, tested locally by using patch and found dangling resource being created. Considering these scenarios, removed the above mentioned terminal codes for **Route Table** resource.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Author: LikithaVemulapalli

apis/v1alpha1/ack-generate-metadata.yaml

@@ -1,13 +1,13 @@
 ack_generate_info:
-  build_date: "2022-08-11T14:16:06Z"
+  build_date: "2022-08-15T17:09:48Z"
   build_hash: 87477ae8ca8ac6ddb8c565bbd910cc7e30f55ed0
-  go_version: go1.18.1
+  go_version: go1.18.3
   version: v0.19.3
 api_directory_checksum: ff86d89efc3212fed4eb14bdc83af54601428a83
 api_version: v1alpha1
 aws_sdk_go_version: v1.42.0
 generator_config_info:
-  file_checksum: a3238ca676f6a461ed229d04c6bcb8f962f9435b
+  file_checksum: 8b8371fa96d088f1656b21e76951932d4daa48fb
   original_file_name: generator.yaml
 last_modification:
   reason: API generation

apis/v1alpha1/generator.yaml

@@ -290,11 +290,6 @@ resources:
       sdk_create_post_build_request:
         template_path: hooks/nat_gateway/sdk_create_post_build_request.go.tpl
   RouteTable:
-    exceptions:
-      terminal_codes:
-        - InvalidVpcID.Malformed
-        - InvalidVpcID.NotFound
-        - InvalidParameterValue
     fields:
       # RouteStatuses as Route to ensure
       # fields set server-side (active, origin)

generator.yaml

@@ -290,11 +290,6 @@ resources:
       sdk_create_post_build_request:
         template_path: hooks/nat_gateway/sdk_create_post_build_request.go.tpl
   RouteTable:
-    exceptions:
-      terminal_codes:
-        - InvalidVpcID.Malformed
-        - InvalidVpcID.NotFound
-        - InvalidParameterValue
     fields:
       # RouteStatuses as Route to ensure
       # fields set server-side (active, origin)

pkg/resource/route_table/sdk.go

@@ -83,39 +83,6 @@ def test_create_delete(self, ec2_client):
         # Check Route Table no longer exists in AWS
         ec2_validator.assert_route_table(resource_id, exists=False)
 
-
-    def test_terminal_condition(self):
-        test_resource_values = REPLACEMENT_VALUES.copy()
-        resource_name = random_suffix_name("route-table-fail", 24)
-        test_resource_values["ROUTE_TABLE_NAME"] = resource_name
-        test_resource_values["VPC_ID"] = "InvalidVpcId"
-
-        # Load RouteTable CR
-        resource_data = load_ec2_resource(
-            "route_table",
-            additional_replacements=test_resource_values,
-        )
-        logging.debug(resource_data)
-
-        # Create k8s resource
-        ref = k8s.CustomResourceReference(
-            CRD_GROUP, CRD_VERSION, RESOURCE_PLURAL,
-            resource_name, namespace="default",
-        )
-        k8s.create_custom_resource(ref, resource_data)
-        cr = k8s.wait_resource_consumed_by_controller(ref)
-
-        assert cr is not None
-        assert k8s.get_resource_exists(ref)
-
-        expected_msg = "InvalidVpcID.NotFound: The vpc ID 'InvalidVpcId' does not exist"
-        terminal_condition = k8s.get_resource_condition(ref, "ACK.Terminal")
-        # Example condition message:
-        #   InvalidVpcID.NotFound: The vpc ID 'InvalidVpcId' does not exist
-        #   status code: 400, request id: 5801fc80-67cf-465f-8b83-5e02d517d554
-        # This check only verifies the error message; the request hash is irrelevant and therefore can be ignored.
-        assert expected_msg in terminal_condition['message']
-
     def test_crud_route(self, ec2_client):
         test_resource_values = REPLACEMENT_VALUES.copy()
         resource_name = random_suffix_name("route-table-test", 24)