refactor(test): allow linux integ test to run in github action (#8386)

* integ * remove cred * fix multiple failures * nit * nit2 * nit3 * nit4 * nit o * use test account * direct * retry * nit * nit * fix multiple stuff * move get cred after init * test * use maven version * use original function * nit * check ruby issue * ruby * ruby 3.4 * maven * nit * enable java 21 * finalize sync test * test build integ test * split container/no container tests * fix * fix container * use -k correctly * k * k2 * kk * split container/non-container integ test * revert test * make sure in buildcmd, all container test has container in its name * fix test issues * disk space * tweak disk usage * rest * nit * n * n * use secret * secr * n * run all test * m * schedule * test * fix remote test event failure * address feedback * Update test_remote_test_event.py

Author: roger-zhangg

.github/workflows/integration-tests.yml

@@ -173,7 +173,7 @@ def do_cli(
  event = lambda_test_event["json"]
  LOG.debug("Remote event contents: %s", event)
  metadata = lambda_test_event["metadata"]
- if "invocationType" in metadata:
+ if "invocationType" in metadata and metadata["invocationType"] is not None:
  parameter.setdefault("InvocationType", metadata["invocationType"])
  elif test_event_name:
  LOG.info("Note: remote event is only supported for AWS Lambda Function resource.")

samcli/commands/remote/invoke/cli.py

@@ -12,7 +12,7 @@
 MIN_ARN_PARTS = 5 # Minimum parts for a valid ARN structure
 
 LAMBDA_FUNCTION_NAME_PATTERN = (
- r"^(arn:[^:]+:lambda:[^:]*:\d{12}:function:|\d{12}:function:)?[a-zA-Z0-9-_\.]+(:[\w$-]+)?$"
+ r"^(arn:[^:]+:lambda:[^:]*:\d{12}:function:|\d{12}:function:)?[a-zA-Z0-9-_\.\[\]]+(:[\w$-]+)?$"
 )
 
 

samcli/lib/utils/name_utils.py

@@ -5,6 +5,7 @@
 
 import json
 import os
+import sys
 
 import boto3
 from boto3.session import Session
@@ -16,7 +17,8 @@
 
 
 def main():
- env_vars = get_testing_credentials()
+ skip_role_deletion = len(sys.argv) > 1 and sys.argv[1] == "skip_role_deletion"
+ env_vars = get_testing_credentials(skip_role_deletion)
  # Assume testing account credential in order to access managed test resource stack
  test_session = Session(
  aws_access_key_id=env_vars["accessKeyID"],
@@ -37,7 +39,7 @@ def get_managed_test_resource_outputs(session: Session):
  return outputs_dict
 
 
-def get_testing_credentials():
+def get_testing_credentials(skip_role_deletion=False):
  lambda_arn = os.environ["CREDENTIAL_DISTRIBUTION_LAMBDA_ARN"]
  # Max attempts to 0 so that boto3 will not invoke multiple times
  lambda_client = boto3.client(
@@ -49,7 +51,14 @@ def get_testing_credentials():
  ),
  region_name="us-west-2",
  )
- response = lambda_client.invoke(FunctionName=lambda_arn)
+
+ # Prepare payload if skip_role_deletion is True
+ if skip_role_deletion:
+ payload_data = json.dumps({"skip_role_deletion": True})
+ response = lambda_client.invoke(FunctionName=lambda_arn, Payload=payload_data)
+ else:
+ response = lambda_client.invoke(FunctionName=lambda_arn)
+
  payload = json.loads(response["Payload"].read())
  if response.get("FunctionError"):
  raise ValueError(f"Failed to get credential. {payload['errorType']}")

tests/get_testing_resources.py

@@ -7,6 +7,7 @@
 import time
 import logging
 import json
+from datetime import datetime, timezone
 from typing import Optional
 from unittest import TestCase
 
@@ -36,6 +37,27 @@
 LOG = logging.getLogger(__name__)
 
 
+def show_container_in_test_name(testcase_func, param_num, param):
+ """
+ Generates a custom name for parameterized test cases.
+ Adds '_in_container' suffix when any parameter contains 'container' in its string representation.
+ """
+ # Get the base test name
+ base_name = f"{testcase_func.__name__}_{param_num}"
+
+ # Check if any parameter contains "container" in its string representation
+ for arg in param.args:
+ if isinstance(arg, str) and "container" in arg.lower():
+ base_name += "_in_container"
+ break
+ elif arg is True: # Also check for boolean True which might indicate use_container
+ # Check if this might be a use_container parameter by position
+ # This is a fallback for cases where True is used instead of "use_container"
+ continue
+
+ return base_name
+
+
 class BuildIntegBase(TestCase):
  template: Optional[str] = "template.yaml"
 
@@ -543,6 +565,12 @@ def _test_with_go(self, runtime, code_uri, mode, relative_path, architecture=Non
  newenv["GOPROXY"] = "direct"
  newenv["GOPATH"] = str(self.working_dir)
 
+ # Build with musl target to avoid glibc compatibility issues
+ # This ensures the binary works in the Lambda execution environment
+ newenv["GOOS"] = "linux"
+ newenv["GOARCH"] = "arm64" if architecture == ARM64 else "amd64"
+ newenv["CGO_ENABLED"] = "0"
+
  run_command(cmdlist, cwd=self.working_dir, env=newenv)
 
  self._verify_built_artifact(
@@ -638,7 +666,7 @@ def _test_with_building_java(
  osutils.convert_to_unix_line_ending(os.path.join(self.test_data_path, self.USING_GRADLEW_PATH, "gradlew"))
  # Use shorter timeout in GitHub Actions to fail faster;
  # Putting 1800 because Windows Canary Instances takes longer
- timeout = 90 if os.environ.get("GITHUB_ACTIONS") else 1800
+ timeout = 1800 if os.environ.get("BY_CANARY") else 180
  run_command(cmdlist, cwd=self.working_dir, timeout=timeout)
 
  self._verify_built_artifact(