Merge branch 'cpack/jenkins-main' of https://github.com/aws-lumberyard-dev/o3de into cpack/jenkins-main

Assets/Editor/Editor.tip

@@ -1,34 +0,0 @@
-CryEngine tips of the day
-
-You can toggle snap to grid by pressing G.
-Ctrl+Shift+Clicking somewhere with an object selected quickly moves the object to that position when in move mode.
-Pressing M will open the material editor.
-Show and Hide helpers is bound to Shift + Space by default.
-Enable AI/Physics is bound to Ctrl + P by default.
-You can save a viewport location by pressing Ctrl + F1 through f12 and go to that position using Shift + F1 through F12.
-You can link objects together by using the link command on the top menu of the editor.
-Pressing 1 through 5 on the keyboard will cycle through brush operations such as move or scale.
-You can simply bind keyboard shortcuts to editor functions by going to Tools --> Customize Keyboard.
-Pressing H will hide the selected objects, Ctrl-H will unhide all hidden objects.
-Pressing F will freeze the selected objects, Ctrl-F will unfreeze all frozen objects.
-Pressing F3 will toggle wireframe view.
-Camera/terrain collision can be toggled using Q.
-You can restart the Editor by pressing the restart button on your PC.
-Pressing Ctrl-C with an object selected will clone that object.
-Toggle the console by pressing the tilde (~) key.
-You can dock windows by dragging them onto the blue helpers that appear when you grab a window by the titlebar.
-You can select materials by clicking on the dropper icon in the material editor and then clicking on the material you wish to select.
-You can right click on the previewer in the material editor and change the model to different shapes and background colors.
-Materials can be saved in the local level folder for re-distribution.
-Always keep your level free of errors and immidiately fix errors reported by the error report screen when you load your level.
-You must always export to engine before you can run it in pure game mode. (File --> Export to engine)
-You must re-triangulate AI before playing your level in game mode. (AI --> Generate all navigation)
-You must always re-generate surface textures after you finish painting the terrain. (File --> Regenerate surface textures)
-Press Ctrl-G or F12 to go into the Game mode, ESC to return to Editing mode.
-Quickly rebuild a level (without regenerating the ground texture) by pressing Ctrl-E.
-Hold down the third mouse button and drag to move the camera up and down.
-Missing objects are represented by a bright yellow sphere. 
-Hold Alt + Middle Mouse button to rotate around an object.
-Select multiple objects by holding Ctrl.
-You can place multiple instances of vegetation by holding Shift and clicking on the terrain.
-A number of useful commands can be found in Tools --> User commands. This can also be dragged and docked to the main window.

AutomatedTesting/Gem/PythonTests/AWS/CMakeLists.txt

@@ -21,6 +21,7 @@ if(PAL_TRAIT_BUILD_TESTS_SUPPORTED AND PAL_TRAIT_BUILD_HOST_TOOLS)
         TEST_SUITE periodic
         TEST_SERIAL
         PATH ${CMAKE_CURRENT_LIST_DIR}/${PAL_PLATFORM_NAME}/
+        TIMEOUT 3000
         RUNTIME_DEPENDENCIES
             Legacy::Editor
             AZ::AssetProcessor

AutomatedTesting/Gem/PythonTests/AWS/Windows/aws_metrics/aws_metrics_automation_test.py

@@ -18,11 +18,12 @@ import typing
 from datetime import datetime
 import ly_test_tools.log.log_monitor
 
-from assetpipeline.ap_fixtures.asset_processor_fixture import asset_processor as asset_processor
-from AWS.common.aws_utils import aws_utils
-from AWS.common.aws_credentials import aws_credentials
+# fixture imports
 from AWS.Windows.resource_mappings.resource_mappings import resource_mappings
-from AWS.Windows.cdk.cdk import cdk
+from AWS.Windows.cdk.cdk_utils import Cdk
+from AWS.common.aws_utils import AwsUtils
+from assetpipeline.ap_fixtures.asset_processor_fixture import asset_processor as asset_processor
+from AWS.common.aws_credentials import aws_credentials
 from .aws_metrics_utils import aws_metrics_utils
 
 AWS_METRICS_FEATURE_NAME = 'AWSMetrics'
@@ -32,7 +33,7 @@ logger = logging.getLogger(__name__)
 
 
 def setup(launcher: ly_test_tools.launchers.Launcher,
-          cdk: cdk,
+          cdk: Cdk,
           asset_processor: asset_processor,
           resource_mappings: resource_mappings,
           context_variable: str = '') -> typing.Tuple[ly_test_tools.log.log_monitor.LogMonitor, str, str]:
@@ -116,18 +117,18 @@ def remove_file(file_path: str) -> None:
 @pytest.mark.parametrize('region_name', ['us-west-2'])
 @pytest.mark.parametrize('assume_role_arn', ['arn:aws:iam::645075835648:role/o3de-automation-tests'])
 @pytest.mark.parametrize('session_name', ['o3de-Automation-session'])
-class TestAWSMetrics_Windows(object):
-    def test_AWSMetrics_RealTimeAnalytics_MetricsSentToCloudWatch(self,
-                                                                  level: str,
-                                                                  launcher: ly_test_tools.launchers.Launcher,
-                                                                  asset_processor: pytest.fixture,
-                                                                  workspace: pytest.fixture,
-                                                                  aws_utils: aws_utils,
-                                                                  aws_credentials: aws_credentials,
-                                                                  resource_mappings: resource_mappings,
-                                                                  cdk: cdk,
-                                                                  aws_metrics_utils: aws_metrics_utils,
-                                                                  ):
+class TestAWSMetricsWindows(object):
+    def test_realtime_analytics_metrics_sent_to_cloudwatch(self,
+                                                           level: str,
+                                                           launcher: ly_test_tools.launchers.Launcher,
+                                                           asset_processor: pytest.fixture,
+                                                           workspace: pytest.fixture,
+                                                           aws_utils: pytest.fixture,
+                                                           aws_credentials: aws_credentials,
+                                                           resource_mappings: pytest.fixture,
+                                                           cdk: pytest.fixture,
+                                                           aws_metrics_utils: aws_metrics_utils,
+                                                           ):
         """
         Tests that the submitted metrics are sent to CloudWatch for real-time analytics.
         """
@@ -148,7 +149,7 @@ class TestAWSMetrics_Windows(object):
                 'AWS/Lambda',
                 'Invocations',
                 [{'Name': 'FunctionName',
-                  'Value': f'{stack_name}-AnalyticsProcessingLambda'}],
+                  'Value': f'{stack_name}-AnalyticsProcessingLambdaName'}],
                 start_time)
             logger.info('Operational health metrics sent to CloudWatch.')
 
@@ -162,14 +163,14 @@ class TestAWSMetrics_Windows(object):
         # Stop the Kinesis Data Analytics application.
         aws_metrics_utils.stop_kinesis_data_analytics_application(analytics_application_name)
 
-    def test_AWSMetrics_UnauthorizedUser_RequestRejected(self,
-                                                         level: str,
-                                                         launcher: ly_test_tools.launchers.Launcher,
-                                                         cdk: cdk,
-                                                         aws_credentials: aws_credentials,
-                                                         asset_processor: pytest.fixture,
-                                                         resource_mappings: resource_mappings,
-                                                         workspace: pytest.fixture):
+    def test_unauthorized_user_request_rejected(self,
+                                                level: str,
+                                                launcher: ly_test_tools.launchers.Launcher,
+                                                cdk: pytest.fixture,
+                                                aws_credentials: aws_credentials,
+                                                asset_processor: pytest.fixture,
+                                                resource_mappings: pytest.fixture,
+                                                workspace: pytest.fixture):
         """
         Tests that unauthorized users cannot send metrics events to the AWS backed backend.
         """
@@ -187,14 +188,14 @@ class TestAWSMetrics_Windows(object):
             assert result, 'Metrics events are sent successfully by unauthorized user'
             logger.info('Unauthorized user is rejected to send metrics.')
 
-    def test_AWSMetrics_BatchAnalytics_MetricsDeliveredToS3(self,
+    def test_batch_analytics_metrics_delivered_to_s3(self,
                                                             level: str,
                                                             launcher: ly_test_tools.launchers.Launcher,
-                                                            cdk: cdk,
+                                                            cdk: pytest.fixture,
                                                             aws_credentials: aws_credentials,
                                                             asset_processor: pytest.fixture,
-                                                            resource_mappings: resource_mappings,
-                                                            aws_utils: aws_utils,
+                                                            resource_mappings: pytest.fixture,
+                                                            aws_utils: pytest.fixture,
                                                             aws_metrics_utils: aws_metrics_utils,
                                                             workspace: pytest.fixture):
         """
@@ -234,4 +235,3 @@ class TestAWSMetrics_Windows(object):
         time.sleep(60)
         # Empty the S3 bucket. S3 buckets can only be deleted successfully when it doesn't contain any object.
         aws_metrics_utils.empty_s3_bucket(analytics_bucket_name)
-

AutomatedTesting/Gem/PythonTests/AWS/Windows/cdk/cdk_utils.py

@@ -12,6 +12,10 @@ WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 import os
 import pytest
 import boto3
+import uuid
+import logging
+import subprocess
+import botocore
 
 import ly_test_tools.environment.process_utils as process_utils
 from typing import List
@@ -19,22 +23,71 @@ from typing import List
 BOOTSTRAP_STACK_NAME = 'CDKToolkit'
 BOOTSTRAP_STAGING_BUCKET_LOGIC_ID = 'StagingBucket'
 
+logger = logging.getLogger(__name__)
+
+
 class Cdk:
     """
     Cdk class that provides methods to run cdk application commands.
     Expects system to have NodeJS, AWS CLI and CDK installed globally and have their paths setup as env variables.
     """
 
-    def __init__(self, cdk_path: str, project: str, account_id: str,
-                 workspace: pytest.fixture, session: boto3.session.Session):
+    def __init__(self):
+        self._cdk_env = ''
+        self._stacks = []
+        self._cdk_path = os.path.dirname(os.path.realpath(__file__))
+        self._session = ''
+
+        cdk_npm_latest_version_cmd = ['npm', 'view', 'aws-cdk', 'version']
+
+        output = process_utils.check_output(
+            cdk_npm_latest_version_cmd,
+            cwd=self._cdk_path,
+            shell=True)
+        cdk_npm_latest_version = output.split()[0]
+
+        cdk_version_cmd = ['cdk', 'version']
+        output = process_utils.check_output(
+            cdk_version_cmd,
+            cwd=self._cdk_path,
+            shell=True)
+        cdk_version = output.split()[0]
+        logger.info(f'Current CDK version {cdk_version}')
+
+        if cdk_version != cdk_npm_latest_version:
+            try:
+                logger.info(f'Updating CDK to latest')
+                # uninstall and reinstall cdk in case npm has been updated.
+                output = process_utils.check_output(
+                    'npm uninstall -g aws-cdk',
+                    cwd=self._cdk_path,
+                    shell=True)
+
+                logger.info(f'Uninstall CDK output: {output}')
+
+                output = process_utils.check_output(
+                    'npm install -g aws-cdk@latest',
+                    cwd=self._cdk_path,
+                    shell=True)
+
+                logger.info(f'Install CDK output: {output}')
+            except subprocess.CalledProcessError as error:
+                logger.warning(f'Failed reinstalling latest CDK on npm'
+                               f'\nError:{error.stderr}')
+
+    def setup(self, cdk_path: str, project: str, account_id: str,
+                 workspace: pytest.fixture, session: boto3.session.Session, bootstrap_required: bool):
         """
         :param cdk_path: Path where cdk app.py is stored.
         :param project: Project name used for cdk project name env variable.
         :param account_id: AWS account id to use with cdk application.
         :param workspace: ly_test_tools workspace fixture.
+        :param workspace: bootstrap_required deploys bootstrap stack.
         """
+
         self._cdk_env = os.environ.copy()
-        self._cdk_env['O3DE_AWS_PROJECT_NAME'] = project
+        unique_id = uuid.uuid4().hex[-4:]
+        self._cdk_env['O3DE_AWS_PROJECT_NAME'] = project[:4] + unique_id if len(project) > 4 else project + unique_id
         self._cdk_env['O3DE_AWS_DEPLOY_REGION'] = session.region_name
         self._cdk_env['O3DE_AWS_DEPLOY_ACCOUNT'] = account_id
         self._cdk_env['PATH'] = f'{workspace.paths.engine_root()}\\python;' + self._cdk_env['PATH']
@@ -43,27 +96,37 @@ class Cdk:
         self._cdk_env['AWS_ACCESS_KEY_ID'] = credentials.access_key
         self._cdk_env['AWS_SECRET_ACCESS_KEY'] = credentials.secret_key
         self._cdk_env['AWS_SESSION_TOKEN'] = credentials.token
-        self._stacks = []
         self._cdk_path = cdk_path
 
+        self._session = session
+
         output = process_utils.check_output(
             'python -m pip install -r requirements.txt',
             cwd=self._cdk_path,
             env=self._cdk_env,
             shell=True)
 
+        logger.info(f'Installing cdk python dependencies: {output}')
+
+        if bootstrap_required:
+            self.bootstrap()
+
     def bootstrap(self) -> None:
         """
         Deploy the bootstrap stack.
         """
-        bootstrap_cmd = ['cdk', 'bootstrap',
-                         f'aws://{self._cdk_env["O3DE_AWS_DEPLOY_ACCOUNT"]}/{self._cdk_env["O3DE_AWS_DEPLOY_REGION"]}']
+        try:
+            bootstrap_cmd = ['cdk', 'bootstrap',
+                             f'aws://{self._cdk_env["O3DE_AWS_DEPLOY_ACCOUNT"]}/{self._cdk_env["O3DE_AWS_DEPLOY_REGION"]}']
 
-        process_utils.check_call(
-            bootstrap_cmd,
-            cwd=self._cdk_path,
-            env=self._cdk_env,
-            shell=True)
+            process_utils.check_call(
+                bootstrap_cmd,
+                cwd=self._cdk_path,
+                env=self._cdk_env,
+                shell=True)
+        except botocore.exceptions.ClientError as clientError:
+            logger.warning(f'Failed creating Bootstrap stack {BOOTSTRAP_STACK_NAME} not found. '
+                           f'\nError:{clientError["Error"]["Message"]}')
 
     def list(self) -> List[str]:
         """
@@ -131,83 +194,51 @@ class Cdk:
         """
         Destroys the cdk application.
         """
-        destroy_cdk_application_cmd = ['cdk', 'destroy', '-f']
-        process_utils.check_output(
-            destroy_cdk_application_cmd,
-            cwd=self._cdk_path,
-            env=self._cdk_env,
-            shell=True)
+
+        logger.info(f'CDK Path {self._cdk_path}')
+        destroy_cdk_application_cmd = ['cdk', 'destroy', '--all', '-f']
+
+        try:
+            process_utils.check_output(
+                destroy_cdk_application_cmd,
+                cwd=self._cdk_path,
+                env=self._cdk_env,
+                shell=True)
+
+        except subprocess.CalledProcessError as e:
+            logger.error(e.output)
+            raise e
 
         self._stacks = []
-        self._cdk_path = ''
 
-    @staticmethod
-    def remove_bootstrap_stack(aws_utils: pytest.fixture) -> None:
+    def remove_bootstrap_stack(self) -> None:
         """
         Remove the CDK bootstrap stack.
         :param aws_utils: aws_utils fixture.
         """
         # Check if the bootstrap stack exists.
-        response = aws_utils.client('cloudformation').describe_stacks(
+        response = self._session.client('cloudformation').describe_stacks(
             StackName=BOOTSTRAP_STACK_NAME
         )
         stacks = response.get('Stacks', [])
-        if not stacks:
+        if not stacks or len(stacks) is 0:
             return
 
         # Clear the bootstrap staging bucket before deleting the bootstrap stack.
-        response = aws_utils.client('cloudformation').describe_stack_resource(
+        response = self._session.client('cloudformation').describe_stack_resource(
             StackName=BOOTSTRAP_STACK_NAME,
             LogicalResourceId=BOOTSTRAP_STAGING_BUCKET_LOGIC_ID
         )
 
         staging_bucket_name = response.get('StackResourceDetail', {}).get('PhysicalResourceId', '')
         if staging_bucket_name:
-            s3 = aws_utils.resource('s3')
+            s3 = self._session.resource('s3')
             bucket = s3.Bucket(staging_bucket_name)
             for key in bucket.objects.all():
                 key.delete()
 
         # Delete the bootstrap stack.
-        aws_utils.client('cloudformation').delete_stack(
-            StackName=BOOTSTRAP_STACK_NAME
-        )
-
-
-@pytest.fixture(scope='function')
-def cdk(
-        request: pytest.fixture,
-        project: str,
-        feature_name: str,
-        workspace: pytest.fixture,
-        aws_utils: pytest.fixture,
-        bootstrap_required: bool = True,
-        destroy_stacks_on_teardown: bool = True) -> Cdk:
-    """
-    Fixture for setting up a Cdk
-    :param request: _pytest.fixtures.SubRequest class that handles getting
-        a pytest fixture from a pytest function/fixture.
-    :param project: Project name used for cdk project name env variable.
-    :param feature_name: Feature gem name to expect cdk folder in.
-    :param workspace: ly_test_tools workspace fixture.
-    :param aws_utils: aws_utils fixture.
-    :param bootstrap_required: Whether the bootstrap stack needs to be created to
-        provision resources the AWS CDK needs to perform the deployment.
-    :param destroy_stacks_on_teardown: option to control calling destroy ot the end of test.
-    :return Cdk class object.
-    """
-
-    cdk_path = f'{workspace.paths.engine_root()}/Gems/{feature_name}/cdk'
-    cdk_obj = Cdk(cdk_path, project, aws_utils.assume_account_id(), workspace, aws_utils.assume_session())
-
-    if bootstrap_required:
-        cdk_obj.bootstrap()
-
-    def teardown():
-        if destroy_stacks_on_teardown:
-            cdk_obj.destroy()
-            cdk_obj.remove_bootstrap_stack(aws_utils)
-
-    request.addfinalizer(teardown)
-
-    return cdk_obj
+        # Should not need to delete the stack if S3 bucket can be cleaned.
+        # self._session.client('cloudformation').delete_stack(
+        #     StackName=BOOTSTRAP_STACK_NAME
+        # )

AutomatedTesting/Gem/PythonTests/AWS/Windows/client_auth/test_anonymous_credentials.py

@@ -12,9 +12,10 @@ import os
 import logging
 import ly_test_tools.log.log_monitor
 
+# fixture imports
 from AWS.Windows.resource_mappings.resource_mappings import resource_mappings
-from AWS.Windows.cdk.cdk import cdk
-from AWS.common.aws_utils import aws_utils
+from AWS.Windows.cdk.cdk_utils import Cdk
+from AWS.common.aws_utils import AwsUtils
 from assetpipeline.ap_fixtures.asset_processor_fixture import asset_processor as asset_processor
 
 AWS_PROJECT_NAME = 'AWS-AutomationTest'
@@ -75,5 +76,5 @@ class TestAWSClientAuthAnonymousCredentials(object):
                 expected_lines=['(Script) - Success anonymous credentials'],
                 unexpected_lines=['(Script) - Fail anonymous credentials'],
                 halt_on_unexpected=True,
-                )
+            )
             assert result, 'Anonymous credentials fetched successfully.'

AutomatedTesting/Gem/PythonTests/AWS/Windows/client_auth/test_password_signin.py

@@ -12,9 +12,10 @@ import os
 import logging
 import ly_test_tools.log.log_monitor
 
+# fixture imports
 from AWS.Windows.resource_mappings.resource_mappings import resource_mappings
-from AWS.Windows.cdk.cdk import cdk
-from AWS.common.aws_utils import aws_utils
+from AWS.Windows.cdk.cdk_utils import Cdk
+from AWS.common.aws_utils import AwsUtils
 from assetpipeline.ap_fixtures.asset_processor_fixture import asset_processor as asset_processor
 
 AWS_PROJECT_NAME = 'AWS-AutomationTest'

AutomatedTesting/Gem/PythonTests/AWS/common/aws_utils.py

@@ -8,11 +8,12 @@ remove or modify any license notices. This file is distributed on an "AS IS" BAS
 WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 """
 import boto3
-import pytest
 import logging
 
 logger = logging.getLogger(__name__)
-logging.getLogger('boto').setLevel(logging.CRITICAL)
+logging.getLogger('boto3').setLevel(logging.WARNING)
+logging.getLogger('botocore').setLevel(logging.WARNING)
+logging.getLogger('nose').setLevel(logging.WARNING)
 
 
 class AwsUtils:
@@ -63,28 +64,3 @@ class AwsUtils:
         clears stored session
         """
         self._assume_session = None
-
-
-@pytest.fixture(scope='function')
-def aws_utils(
-        request: pytest.fixture,
-        assume_role_arn: str,
-        session_name: str,
-        region_name: str):
-    """
-    Fixture for AWS util functions
-    :param request: _pytest.fixtures.SubRequest class that handles getting
-        a pytest fixture from a pytest function/fixture.
-    :param assume_role_arn: Role used to fetch temporary aws credentials, configure service clients with obtained credentials.
-    :param session_name: Session name to set.
-    :param region_name: AWS account region to set for session.
-    :return AWSUtils class object.
-    """
-    aws_utils_obj = AwsUtils(assume_role_arn, session_name, region_name)
-
-    def teardown():
-        aws_utils_obj.destroy()
-
-    request.addfinalizer(teardown)
-
-    return aws_utils_obj

AutomatedTesting/Gem/PythonTests/AWS/conftest.py

@@ -0,0 +1,87 @@
+"""
+All or portions of this file Copyright (c) Amazon.com, Inc. or its affiliates or
+its licensors.
+For complete copyright and license terms please see the LICENSE at the root of this
+distribution (the "License"). All use of this software is governed by the License,
+or, if provided, by the license below or the license accompanying this file. Do not
+remove or modify any license notices. This file is distributed on an "AS IS" BASIS,
+WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+"""
+import pytest
+import logging
+from AWS.common.aws_utils import AwsUtils
+from AWS.Windows.cdk.cdk_utils import Cdk
+
+logger = logging.getLogger(__name__)
+
+
+@pytest.fixture(scope='function')
+def aws_utils(
+        request: pytest.fixture,
+        assume_role_arn: str,
+        session_name: str,
+        region_name: str):
+    """
+    Fixture for AWS util functions
+    :param request: _pytest.fixtures.SubRequest class that handles getting
+        a pytest fixture from a pytest function/fixture.
+    :param assume_role_arn: Role used to fetch temporary aws credentials, configure service clients with obtained credentials.
+    :param session_name: Session name to set.
+    :param region_name: AWS account region to set for session.
+    :return AWSUtils class object.
+    """
+
+    aws_utils_obj = AwsUtils(assume_role_arn, session_name, region_name)
+
+    def teardown():
+        aws_utils_obj.destroy()
+
+    request.addfinalizer(teardown)
+
+    return aws_utils_obj
+
+# Set global pytest variable for cdk to avoid recreating instance
+pytest.cdk_obj = None
+
+
+@pytest.fixture(scope='function')
+def cdk(
+        request: pytest.fixture,
+        project: str,
+        feature_name: str,
+        workspace: pytest.fixture,
+        aws_utils: pytest.fixture,
+        bootstrap_required: bool = True,
+        destroy_stacks_on_teardown: bool = True) -> Cdk:
+    """
+    Fixture for setting up a Cdk
+    :param request: _pytest.fixtures.SubRequest class that handles getting
+        a pytest fixture from a pytest function/fixture.
+    :param project: Project name used for cdk project name env variable.
+    :param feature_name: Feature gem name to expect cdk folder in.
+    :param workspace: ly_test_tools workspace fixture.
+    :param aws_utils: aws_utils fixture.
+    :param bootstrap_required: Whether the bootstrap stack needs to be created to
+        provision resources the AWS CDK needs to perform the deployment.
+    :param destroy_stacks_on_teardown: option to control calling destroy ot the end of test.
+    :return Cdk class object.
+    """
+
+    cdk_path = f'{workspace.paths.engine_root()}/Gems/{feature_name}/cdk'
+    logger.info(f'CDK Path {cdk_path}')
+
+    if pytest.cdk_obj is None:
+        pytest.cdk_obj = Cdk()
+
+    pytest.cdk_obj.setup(cdk_path, project, aws_utils.assume_account_id(), workspace, aws_utils.assume_session(),
+                             bootstrap_required)
+    def teardown():
+        if destroy_stacks_on_teardown:
+            pytest.cdk_obj.destroy()
+            # Enable after https://github.com/aws/aws-cdk/issues/986 is fixed.
+            # Until then clean the bootstrap bucket manually.
+            # cdk_obj.remove_bootstrap_stack()
+
+    request.addfinalizer(teardown)
+
+    return pytest.cdk_obj

AutomatedTesting/Gem/PythonTests/CMakeLists.txt

@@ -60,5 +60,4 @@ add_subdirectory(streaming)
 add_subdirectory(smoke)
 
 ## AWS ##
-# Enable when AWS Gems work on Linux and Android.
-# add_subdirectory(AWS)
+add_subdirectory(AWS)

AutomatedTesting/Levels/AWS/ClientAuth/ClientAuth.ly

@@ -0,0 +1,3 @@
+version https://git-lfs.github.com/spec/v1
+oid sha256:f0f4d4e0155feaa76c80a14128000a0fd9570ab76e79f4847eaef9006324a4d2
+size 9084

AutomatedTesting/Levels/AWS/ClientAuth/LevelData/Environment.xml

@@ -0,0 +1 @@
+<Environment />

AutomatedTesting/Levels/AWS/ClientAuth/LevelData/TimeOfDay.xml

@@ -0,0 +1 @@
+<TimeOfDay Time="12"/>

AutomatedTesting/Levels/AWS/ClientAuth/filelist.xml

@@ -0,0 +1,6 @@
+<download name="ClientAuth" type="Map">
+ <index src="filelist.xml" dest="filelist.xml"/>
+ <files>
+  <file src="level.pak" dest="level.pak" size="ED0" md5="dbf5115226e4b0ea38ebdc3967ba3aa9"/>
+ </files>
+</download>

AutomatedTesting/Levels/AWS/ClientAuth/level.pak

@@ -0,0 +1,3 @@
+version https://git-lfs.github.com/spec/v1
+oid sha256:da041115014f11696d5878d5c21247c17b8d694fa9674e30692259261a7223a2
+size 3792

AutomatedTesting/Levels/AWS/ClientAuth/tags.txt

@@ -0,0 +1,12 @@
+0,0,0,0,0,0
+0,0,0,0,0,0
+0,0,0,0,0,0
+0,0,0,0,0,0
+0,0,0,0,0,0
+0,0,0,0,0,0
+0,0,0,0,0,0
+0,0,0,0,0,0
+0,0,0,0,0,0
+0,0,0,0,0,0
+0,0,0,0,0,0
+0,0,0,0,0,0

AutomatedTesting/Levels/AWS/ClientAuthPasswordSignIn/ClientAuthPasswordSignIn.ly

@@ -0,0 +1,3 @@
+version https://git-lfs.github.com/spec/v1
+oid sha256:43b1a23b62fe2ffa05545ac99524f40b6fff49d6e35925b9d6138c00d8082e86
+size 9073

AutomatedTesting/Levels/AWS/ClientAuthPasswordSignIn/filelist.xml

@@ -0,0 +1,6 @@
+<download name="ClientAuthPasswordSignIn" type="Map">
+ <index src="filelist.xml" dest="filelist.xml"/>
+ <files>
+  <file src="level.pak" dest="level.pak" size="DDF" md5="ebe91ae5f1ea1ec735b6650b14f3f95b"/>
+ </files>
+</download>

AutomatedTesting/Levels/AWS/ClientAuthPasswordSignIn/level.pak

@@ -0,0 +1,3 @@
+version https://git-lfs.github.com/spec/v1
+oid sha256:a58292341785cb260dc0ccf346259e35e2817ee48fc401a21ab528f6afb97b52
+size 3551

AutomatedTesting/Levels/AWS/ClientAuthPasswordSignIn/tags.txt

@@ -0,0 +1,12 @@
+0,0,0,0,0,0
+0,0,0,0,0,0
+0,0,0,0,0,0
+0,0,0,0,0,0
+0,0,0,0,0,0
+0,0,0,0,0,0
+0,0,0,0,0,0
+0,0,0,0,0,0
+0,0,0,0,0,0
+0,0,0,0,0,0
+0,0,0,0,0,0
+0,0,0,0,0,0

AutomatedTesting/Levels/AWS/ClientAuthPasswordSignUp/ClientAuthPasswordSignUp.ly

@@ -0,0 +1,3 @@
+version https://git-lfs.github.com/spec/v1
+oid sha256:f3d5121b26608b02747e245071ccff29ac57358cb6349ec9495a7a003ac12467
+size 8942

AutomatedTesting/Levels/AWS/ClientAuthPasswordSignUp/filelist.xml

@@ -0,0 +1,6 @@
+<download name="ClientAuthPasswordSignUp" type="Map">
+ <index src="filelist.xml" dest="filelist.xml"/>
+ <files>
+  <file src="level.pak" dest="level.pak" size="DDA" md5="aa6df891d505d6d9175beee4b55626db"/>
+ </files>
+</download>

AutomatedTesting/Levels/AWS/ClientAuthPasswordSignUp/level.pak

@@ -0,0 +1,3 @@
+version https://git-lfs.github.com/spec/v1
+oid sha256:605391d415b828b100bada11d108099520c0b6a020f17588887b610475805d90
+size 3546

AutomatedTesting/Levels/AWS/ClientAuthPasswordSignUp/tags.txt

@@ -0,0 +1,12 @@
+0,0,0,0,0,0
+0,0,0,0,0,0
+0,0,0,0,0,0
+0,0,0,0,0,0
+0,0,0,0,0,0
+0,0,0,0,0,0
+0,0,0,0,0,0
+0,0,0,0,0,0
+0,0,0,0,0,0
+0,0,0,0,0,0
+0,0,0,0,0,0
+0,0,0,0,0,0

AutomatedTesting/Levels/AWS/Metrics/Metrics.ly

@@ -0,0 +1,3 @@
+version https://git-lfs.github.com/spec/v1
+oid sha256:023992998ab5a1d64b38dacd1d5e1a9dc930ff704289c0656ed6eaba6951d660
+size 9066

AutomatedTesting/Levels/AWS/Metrics/Script/Metrics.lua

@@ -0,0 +1,79 @@
+----------------------------------------------------------------------------------------------------
+--
+-- All or portions of this file Copyright (c) Amazon.com, Inc. or its affiliates or
+-- its licensors.
+--
+-- For complete copyright and license terms please see the LICENSE at the root of this
+-- distribution (the "License"). All use of this software is governed by the License,
+-- or, if provided, by the license below or the license accompanying this file. Do not
+-- remove or modify any license notices. This file is distributed on an "AS IS" BASIS,
+-- WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+--
+--
+----------------------------------------------------------------------------------------------------
+local metrics = {
+}
+
+function metrics:OnActivate()
+    self.tickTime = 0
+    self.numSubmittedMetricsEvents = 0
+
+    self.tickBusHandler = TickBus.Connect(self,self.entityId)
+    self.metricsNotificationHandler = AWSMetricsNotificationBus.Connect(self, self.entityId)
+
+    LyShineLua.ShowMouseCursor(true)
+end
+
+function metrics:OnSendMetricsSuccess(requestId)
+    Debug.Log("Metrics is sent successfully.")
+end
+
+function metrics:OnSendMetricsFailure(requestId, errorMessage)
+    Debug.Log("Failed to send metrics.")
+end
+
+function metrics:OnDeactivate()
+    AWSMetricsRequestBus.Broadcast.FlushMetrics()
+    Debug.Log("Stop generating new test events and flushed the buffered metrics.")
+
+    self.tickBusHandler:Disconnect()
+    self.metricsNotificationHandler:Disconnect()
+end
+
+function metrics:OnTick(deltaTime, timePoint)
+    self.tickTime = self.tickTime + deltaTime
+
+    if self.tickTime > 2.0 then
+        defaultAttribute = AWSMetrics_MetricsAttribute()
+        defaultAttribute:SetName("event_name")
+        defaultAttribute:SetStrValue("login")
+
+        customAttribute = AWSMetrics_MetricsAttribute()
+        customAttribute:SetName("custom_attribute")
+        customAttribute:SetStrValue("value")
+
+        attributeList = AWSMetrics_AttributesSubmissionList()
+        attributeList.attributes:push_back(defaultAttribute)
+        attributeList.attributes:push_back(customAttribute)
+
+
+        if self.numSubmittedMetricsEvents % 2 == 0 then
+            if AWSMetricsRequestBus.Broadcast.SubmitMetrics(attributeList.attributes, 0, "lua", false) then
+                Debug.Log("Submitted metrics without buffer.")
+            else
+                Debug.Log("Failed to Submit metrics without buffer.")
+            end
+        else
+            if AWSMetricsRequestBus.Broadcast.SubmitMetrics(attributeList.attributes, 0, "lua", true) then
+                Debug.Log("Submitted metrics with buffer.")
+            else
+                Debug.Log("Failed to Submit metrics with buffer.")
+            end
+        end
+        
+        self.numSubmittedMetricsEvents = self.numSubmittedMetricsEvents + 1
+        self.tickTime = 0
+    end
+end
+
+return metrics

AutomatedTesting/Levels/AWS/Metrics/filelist.xml

@@ -0,0 +1,6 @@
+<download name="Metrics" type="Map">
+ <index src="filelist.xml" dest="filelist.xml"/>
+ <files>
+  <file src="level.pak" dest="level.pak" size="E09" md5="f16fff2970a4037af5909b269ceece4b"/>
+ </files>
+</download>

AutomatedTesting/Levels/AWS/Metrics/level.pak

@@ -0,0 +1,3 @@
+version https://git-lfs.github.com/spec/v1
+oid sha256:e7c0c07b13bb64db344b94d5712e1e802e607a9dee506768b34481f4a76d8505
+size 3593

AutomatedTesting/Levels/AWS/Metrics/tags.txt

@@ -0,0 +1,12 @@
+0,0,0,0,0,0
+0,0,0,0,0,0
+0,0,0,0,0,0
+0,0,0,0,0,0
+0,0,0,0,0,0
+0,0,0,0,0,0
+0,0,0,0,0,0
+0,0,0,0,0,0
+0,0,0,0,0,0
+0,0,0,0,0,0
+0,0,0,0,0,0
+0,0,0,0,0,0

Code/Framework/AzCore/Tests/Serialization/Json/JsonSerializerConformityTests.h

@@ -1206,7 +1206,7 @@ namespace JsonSerializationTests
         if (this->m_features.m_enableInitializationTest)
         {
             auto instance = this->m_description.CreateDefaultInstance();
-            typename TypeParam::Type compare;
+            typename TypeParam::Type compare = typename TypeParam::Type{};
             if (!this->m_description.AreEqual(*instance, compare))
             {
                 auto serializer = this->m_description.CreateSerializer();

Code/Framework/AzNetworking/AzNetworking/ConnectionLayer/IConnection.h

@@ -44,6 +44,11 @@ namespace AzNetworking
 
     //! @class IConnection
     //! @brief interface class for network connections.
+    //!
+    //! IConnection provides a pure-virtual interface for all network connection types. IConnections provide access to
+    //! a ConnectionMetrics object which provides a variety of metrics on the connection itself such as data rate, RTT and
+    //! packet statistics. 
+
     class IConnection
     {
     public:

Code/Framework/AzNetworking/AzNetworking/ConnectionLayer/IConnectionListener.h

@@ -22,6 +22,12 @@ namespace AzNetworking
 {
     //! @class IConnectionListener
     //! @brief interface class for application layer dealing with connection level events.
+    //!
+    //! IConnectionListener defines an abstract interface that the user of AzNetworking is expected to implement to react and
+    //! handle all IConnection related events, including the handling of any received IPacket derived packets.  The AzNetworking
+    //! user should derive a handler class from IConnectionListener, and provide an instance of that handler to any
+    //! INetworkInterface the user instantiates.  The lifetime of the IConnectionListener must outlive the lifetime of the
+    //! INetworkInterface.
     class IConnectionListener
     {
     public:

Code/Framework/AzNetworking/AzNetworking/ConnectionLayer/IConnectionSet.h

@@ -18,6 +18,11 @@ namespace AzNetworking
 {
     //! @class IConnectionSet
     //! @brief interface class for managing a set of connections.
+    //!
+    //! IConnectionSet defines a simple interface for working with an abstract set of IConnections bound to an
+    //! INetworkInterface.  Generally users of AzNetworking will not have reason to interact directly with the IConnectionSet,
+    //! as its interface is completely wrapped by INetworkInterface.
+
     class IConnectionSet
     {
     public:

Code/Framework/AzNetworking/AzNetworking/Framework/ICompressor.h

@@ -23,10 +23,10 @@ namespace AzNetworking
     //! Collection of compression related error codes
     enum class CompressorError
     {
-        Ok,                   ///< No error, operation finished successfully
-        InsufficientBuffer,   ///< Buffer size is insufficient for the operation to complete, increase the size and try again
-        CorruptData,          ///< Malformed or hacked packet, potentially security issue
-        Uninitialized         ///< Compressor or supplied buffers are uninitialized
+        Ok,                   //!< No error, operation finished successfully
+        InsufficientBuffer,   //!< Buffer size is insufficient for the operation to complete, increase the size and try again
+        CorruptData,          //!< Malformed or hacked packet, potentially security issue
+        Uninitialized         //!< Compressor or supplied buffers are uninitialized
     };
 
     //! Unique identifier of a given compressor
@@ -34,6 +34,12 @@ namespace AzNetworking
 
     //! @class ICompressor
     //! @brief Packet data compressor interface.
+    //!
+    //! ICompressor is an abstract compression interface meant for user provided GEMs to implement (such as the [Multiplayer
+    //! Compression Gem](http://docs.o3de.org/docs/user-guide/gems/reference/multiplayer-compression)).
+    //! Compression is supported for both TCP and UDP connections.  Instantiation of a compressor is controlled by the
+    //! `net_UdpCompressor` or `net_TcpCompressor` cvar for their respective protocols.  
+
     class ICompressor
     {
     public:
@@ -87,8 +93,16 @@ namespace AzNetworking
         ) = 0;
     };
 
-    //! Abstract factory to instantiate compressors.
-    //! Used by the network interface to create a compressor
+    //! @class ICompressorFactory
+    //! @brief Abstract factory to instantiate compressors.
+    //!
+    //! ICompressorFactory is an abstract compression interface meant for user provided GEMs to implement. ICompressorFactory
+    //! implementations can be registered to classes implementing INetworking. Registered factories can then be used to create
+    //! ICompressor implementations on demand. The [Multiplayer Compression
+    //! Gem](http://docs.o3de.org/docs/user-guide/gems/reference/multiplayer-compression) is an example of an ICompressorFactory
+    //! for an LZ4 Compressor. In it, MultiplayerCompressionSystemComponent registers its ICompressorFactory with
+    //! NetworkingSystemComponent, which is an implementation of INetworking. Registered factories are keyed by their AZ Name
+    //! which is accessed through the factory's GetFactoryName method.
     class ICompressorFactory
     {
     public:

Code/Framework/AzNetworking/AzNetworking/Framework/INetworkInterface.h

@@ -22,7 +22,16 @@
 namespace AzNetworking
 {
     //! @class INetworkInterface
-    //! @brief pure virtual network interface class to abstract client/server and tcp/udp concerns from application code.
+    //! @brief Network interface class to abstract client/server and protocol concerns from application code.
+    //!
+    //! INetworkInterface provides an abstract API capable of receiving and opening IConnection objects, sending IPacket objects with optional
+    //! reliability, and determining the delivery status of packets that have been sent unreliably (delivery of reliable packets
+    //! is guaranteed as long as the associated connection remains open). INetworkInterface must be provided an
+    //! IConnectionListener instance that outlives the INetworkInterface itself. The INetworkInterface also creates and manages
+    //! the IConnectionSet, which tracks all open connections bound to the interface. INetworkInterface also provides GetMetrics
+    //! functions which can be used to fetch a struct detailing a variety of metrics relating to send and receive rates for both
+    //! packets and bytes in addition to the effect of features on those rates (such as packet size reduction due to compression.)
+    
     class INetworkInterface
     {
     public:

Code/Framework/AzNetworking/AzNetworking/Framework/INetworking.h

@@ -23,6 +23,17 @@ namespace AzNetworking
 
     //! @class INetworking
     //! @brief The interface for creating and working with network interfaces.
+    //!
+    //! INetworking is an Az::Interface<T> that provides applications access to higher level networking abstractions.
+    //! AzNetworking::INetworking can be used to instantiate new INetworkInterfaces that can be configured to operate over
+    //! either TCP or UDP, enable or disable encryption, and be assigned a trust level.
+    //! 
+    //! INetworking is also responsible for registering ICompressorFactory implementations. This allows a developer to have
+    //! access to multiple ICompressorFactory implementations by name.  The [MultiplayerCompressor
+    //! Gem](http://docs.o3de.org/docs/user-guide/gems/reference/multiplayer-compression) is an example of this using the
+    //! [LZ4](https://wikipedia.org/wiki/LZ4_%28compression_algorithm%29) algorithm.
+    //! 
+
     class INetworking
     {
     public:

Code/Framework/AzNetworking/AzNetworking/PacketLayer/IPacket.h

@@ -24,6 +24,15 @@ namespace AzNetworking
 
     //! @class IPacket
     //! @brief Base class for all packets.
+    //!
+    //! IPacket defines an abstract interface that all packets transmitted using AzNetworking must conform to. While there are
+    //! a number of core packets used internally by AzNetworking, it is fully possible for end-users to define their own custom
+    //! packets using this interface. PacketType should be distinct, and should be greater than
+    //! AzNetworking::CorePackets::MAX. The Serialize method allows the IPacket to be used by an
+    //! ISerializer to move data between hosts safely and efficiently.
+    //!
+    //! For more information on the packet format and best practices for extending the packet system, read
+    //! [Networking Packets](http://docs.o3de.org/docs/user-guide/networking/packets) on the O3DE documentation site.
     class IPacket
     {
     public:

Code/Framework/AzNetworking/AzNetworking/PacketLayer/IPacketHeader.h

@@ -28,6 +28,19 @@ namespace AzNetworking
 
     //! @class IPacketHeader
     //! @brief A packet header that lets us deduce packet type for any incoming packet.
+    //! 
+    //! IPacketHeader defines an abstract interface for a descriptor of all AzNetworking::IPacket sent through AzNetworking. The
+    //! PacketHeader is used to identify and describe the contents of a Packet so that transport logic can identify what
+    //! additional processing steps need to be taken (if any) and what type of Packet is being inspected.
+    //! 
+    //! The PacketFlags portion of the header represents the first byte of the header.  While it can be encrypted it is
+    //! otherwise not exposed to additional processing (such as an AzNetworking::ICompressor).  PacketFlags are a bitfield use to provide up
+    //! front information about the state of the packet. Currently there is only one flag to indicate if the Packet is
+    //! compressed or not.
+    //! 
+    //! The remainder of the header contains the PacketType and the PacketId. While the PacketFlags byte is exempt from most
+    //! additional forms of processing, the remainder of the header is not.
+    
     class IPacketHeader
     {
     public:

Code/Framework/AzNetworking/AzNetworking/Serialization/ISerializer.h

@@ -27,6 +27,18 @@ namespace AzNetworking
 
     //! @class ISerializer
     //! @brief Interface class for all serializers to derive from.
+    //!
+    //! ISerializer defines an abstract interface for visiting an object hierarchy and performing operations upon that hierarchy,
+    //! typically reading from or writing data to the object hierarchy for reasons of persistence or network transmission.
+    //! 
+    //! While the most common types of serializers are provided by the AzNetworking framework, users can implement custom
+    //! serializers and perform complex operations on any serializable structures. A few types native to AzNetworking, many of which
+    //! relate to packets, demonstrate this.
+    //! 
+    //! Provided serializers include NetworkInputSerializer for writing an object model into a bytestream, NetworkOutputSerializer
+    //! for writing to an object model, TrackChangesSerializer which is used to efficiently serialize objects without incurring significant
+    //! copy or comparison overhead, and HashSerializer which can be used to generate a hash of all visited data which is important for
+    //! automated desync detection. 
     class ISerializer
     {
     public:

Code/Framework/AzNetworking/AzNetworking/TcpTransport/TcpNetworkInterface.h

@@ -25,6 +25,44 @@ namespace AzNetworking
 
     //! @class TcpNetworkInterface
     //! @brief This class implements a TCP network interface.
+    //!
+    //! TcpNetworkInterface is an implementation of AzNetworking::INetworkInterface.
+    //! Unlike UDP, TCP implements a variety of transport features such as congestion
+    //! avoidance, flow control, and reliability. These features are valuable, but TCP
+    //! offers minimal configuration of them. This is why UdpNetworkInterface offers
+    //! similar features, but with greater flexibility in configuration. If your project doesn't
+    //! require the low latency of UDP, consider using TCP.
+    //! 
+    //! ## Packet structure
+    //! 
+    //! * Flags - A bitfield a receiving endpoint can quickly inspect to learn about configuration of a packet
+    //! * Header - Details the type of packet and other information related to reliability
+    //! * Payload - The actual serialized content of the packet
+    //! 
+    //! For more information, read [Networking Packets](http://docs.o3de.org/docs/user-guide/networking/packets) in the O3DE documentation.
+    //! 
+    //! ## Reliability
+    //! 
+    //! TCP packets can only be sent reliably. This is a feature of TCP itself.
+    //! 
+    //! ## Fragmentation
+    //! 
+    //! TCP implements fragmentation under the hood. Consumers of TCP packets will never
+    //! need to worry about reconstructing the contents over multiple transmissions.
+    //! 
+    //! ## Compression
+    //! 
+    //! Compression here refers to content insensitive compression using libraries like
+    //! LZ4. If enabled, the target payload is run through the compressor and replaces
+    //! the original payload if it's in fact smaller. To tell if compression is enabled
+    //! on a given packet, we operate on a bit in the packet's Flags. The Sender writes
+    //! this bit while the Receiver checks it to see if a packet needs to be
+    //! decompressed.
+    //! 
+    //! ## Encryption
+    //! 
+    //! AzNetworking uses the [OpenSSL](https://www.openssl.org/) library to implement TLS encryption. If enabled,
+    //! the O3DE network layer handles the OpenSSL handshake under the hood using provided certificates.
     class TcpNetworkInterface final
         : public INetworkInterface
     {

Code/Framework/AzNetworking/AzNetworking/UdpTransport/UdpNetworkInterface.h

@@ -27,12 +27,58 @@ namespace AzNetworking
     class IConnectionListener;
     class ICompressor;
 
-    // 20 byte IPv4 header + 8 byte UDP header
-    static const uint32_t UdpPacketHeaderSize = 20 + 8;
-    static const uint32_t DtlsPacketHeaderSize = 13; // DTLS1_RT_HEADER_LENGTH
+    static const uint32_t UdpPacketHeaderSize = 20 + 8; //!< 20 byte IPv4 header + 8 byte UDP header
+    static const uint32_t DtlsPacketHeaderSize = 13; //!< DTLS1_RT_HEADER_LENGTH
 
     //! @class UdpNetworkInterface
     //! @brief This class implements a UDP network interface.
+    //!
+    //! UdpNetworkInterface is an implementation of AzNetworking::INetworkInterface. Since UDP is a very bare bones protocol,
+    //! the Open 3D Engine implementation has to provide significantly more than its TCP counterpart (since TCP implements a
+    //! significant number of reliability features.)
+    //! 
+    //! When sent through UDP, a packet can have additional actions performed on it depending on which features are enabled and
+    //! configured. Each feature listed in this description is in the order a packet will see them on Send.
+    //! 
+    //! ### Packet structure
+    //! 
+    //! The general structure of a UDP packet is:
+    //! 
+    //! * Flags - A bitfield a receiving endpoint can quickly inspect to learn about configuration of a packet
+    //! * Header - Details the type of packet and other information related to reliability
+    //! * Payload - The actual serialized content of the packet
+    //! 
+    //! For more information, read [Networking Packets](http://docs.o3de.org/docs/user-guide/networking/packets) in the O3DE documentation.
+    //! 
+    //! ### Reliability
+    //! 
+    //! UDP packets can be sent reliably or unreliably. Reliably sent packets are registered for tracking first. This causes the
+    //! reliable packet to be resent if a timeout on the packet is reached. Once the packet is acknowledged, the packet is
+    //! unregistered.
+    //! 
+    //! ### Fragmentation
+    //! 
+    //! If the raw packet size exceeds the configured maximum transmission unit (MTU) then the packet is broken into
+    //! multiple reliable fragments to avoid fragmentation at the routing level. Fragments are always reliable so the original
+    //! packet can be reconstructed. Operations that alter the payload generally follow this step so that they can be
+    //! separately applied to the Fragments in addition to not being applied to both the original and Fragments.
+    //! 
+    //! ### Compression
+    //! 
+    //! Compression here refers to content insensitive compression using libraries like LZ4. If enabled, the target payload is
+    //! run through the compressor and replaces the original payload if it's in fact smaller. To tell if compression is enabled
+    //! on a given packet, we operate on a bit in the packet's Flags. The Sender writes this bit while the Receiver checks it to
+    //! see if a packet needs to be decompressed.
+    //! 
+    //! O3DE could potentially move from over MTU to under with compression, and the UDP interface doesn't check for this. Detecting a change
+    //! that would reduce the number of fragmented packets would require pre-emptively compressing payloads to tell if that change happened,
+    //! which could potentially lead to a lot of unnecessary calls to the compressor.
+    //! 
+    //! ### Encryption
+    //! 
+    //! AzNetworking uses the [OpenSSL](https://www.openssl.org/) library to implement Datagram Layer Transport Security (DTLS) encryption
+    //! on UDP traffic. Encryption operates as described in [O3DE Networking Encryption](http://docs.o3de.org/docs/user-guide/networking/encryption)
+    //! on the documentation website. Once both endpoints have completed their handshake, all traffic is expected to be fully encrypted.
     class UdpNetworkInterface final
         : public INetworkInterface
     {

Code/Framework/AzToolsFramework/AzToolsFramework/Prefab/Instance/Instance.cpp

@@ -326,6 +326,16 @@ namespace AzToolsFramework
             return *(m_nestedInstances[newInstanceAlias] = std::move(instance));
         }
 
+        void Instance::DetachNestedInstances(const AZStd::function<void(AZStd::unique_ptr<Instance>)>& callback)
+        {
+            for (auto&& [instanceAlias, instance] : m_nestedInstances)
+            {
+                instance->m_parent = nullptr;
+                callback(AZStd::move(instance));
+            }
+            m_nestedInstances.clear();
+        }
+
         AZStd::unique_ptr<Instance> Instance::DetachNestedInstance(const InstanceAlias& instanceAlias)
         {
             AZStd::unique_ptr<Instance> removedNestedInstance;

Code/Framework/AzToolsFramework/AzToolsFramework/Prefab/Instance/Instance.h

@@ -103,6 +103,7 @@ namespace AzToolsFramework
             Instance& AddInstance(AZStd::unique_ptr<Instance> instance);
             Instance& AddInstance(AZStd::unique_ptr<Instance> instance, InstanceAlias instanceAlias);
             AZStd::unique_ptr<Instance> DetachNestedInstance(const InstanceAlias& instanceAlias);
+            void DetachNestedInstances(const AZStd::function<void(AZStd::unique_ptr<Instance>)>& callback);
 
             /**
             * Gets the aliases for the entities in the Instance DOM.

Code/Framework/AzToolsFramework/AzToolsFramework/Prefab/PrefabLoader.cpp

@@ -151,6 +151,10 @@ namespace AzToolsFramework
                 return InvalidTemplateId;
             }
 
+            // Add or replace the Source parameter in the dom
+            PrefabDomPath sourcePath = PrefabDomPath((AZStd::string("/") + PrefabDomUtils::SourceName).c_str());
+            sourcePath.Set(readPrefabFileResult.GetValue(), relativePath.Native().c_str());
+
             // Create new Template with the Prefab DOM.
             TemplateId newTemplateId = m_prefabSystemComponentInterface->AddTemplate(relativePath, readPrefabFileResult.TakeValue());
             if (newTemplateId == InvalidTemplateId)

Code/Framework/AzToolsFramework/AzToolsFramework/Prefab/PrefabPublicHandler.cpp

@@ -1211,25 +1211,23 @@ namespace AzToolsFramework
 
                     const auto instanceTemplateId = instancePtr->GetTemplateId();
                     auto parentContainerEntityId = parentInstance.GetContainerEntityId();
-                    instancePtr->GetNestedInstances(
-                        [&](AZStd::unique_ptr<Instance>& nestedInstancePtr)
+
+                    instancePtr->DetachNestedInstances(
+                        [&](AZStd::unique_ptr<Instance> detachedNestedInstance)
                     {
-                        //get previous link patch
-                        auto linkRef = m_prefabSystemComponentInterface->FindLink(nestedInstancePtr->GetLinkId());
-                        PrefabDomValueReference linkPatches = linkRef->get().GetLinkPatches();
-                        AZ_Assert(
-                            linkPatches.has_value(), "Unable to get patches on link with id '%llu' during prefab creation.",
-                            nestedInstancePtr->GetLinkId());
+                        PrefabDom& nestedInstanceTemplateDom =
+                            m_prefabSystemComponentInterface->FindTemplateDom(detachedNestedInstance->GetTemplateId());
 
-                        PrefabDom linkPatchesCopy;
-                        linkPatchesCopy.CopyFrom(linkPatches->get(), linkPatchesCopy.GetAllocator());
-
-                        RemoveLink(nestedInstancePtr, instanceTemplateId, undoBatch.GetUndoBatch());
-
-                        UpdateLinkPatchesWithNewEntityAliases(linkPatchesCopy, oldEntityAliases, parentInstance);
+                        Instance& nestedInstanceUnderNewParent = parentInstance.AddInstance(AZStd::move(detachedNestedInstance));
                         
-                        CreateLink(*nestedInstancePtr, parentTemplateId, undoBatch.GetUndoBatch(),
-                            AZStd::move(linkPatchesCopy), true);
+                        PrefabDom nestedInstanceDomUnderNewParent;
+                        m_instanceToTemplateInterface->GenerateDomForInstance(
+                            nestedInstanceDomUnderNewParent, nestedInstanceUnderNewParent);
+                        PrefabDom reparentPatch;
+                        m_instanceToTemplateInterface->GeneratePatch(
+                            reparentPatch, nestedInstanceTemplateDom, nestedInstanceDomUnderNewParent);
+                        
+                        CreateLink(nestedInstanceUnderNewParent, parentTemplateId, undoBatch.GetUndoBatch(), AZStd::move(reparentPatch), true);
                     });
                 }
 

Code/Framework/AzToolsFramework/AzToolsFramework/Prefab/Template/Template.cpp

@@ -207,6 +207,10 @@ namespace AzToolsFramework
                 instanceValue->CopyFrom(linkDom, m_prefabDom.GetAllocator());
             }
 
+            // Remove Source parameter from the dom. It will be added on file load, and should not be stored to disk.
+            PrefabDomPath sourcePath = PrefabDomPath((AZStd::string("/") + PrefabDomUtils::SourceName).c_str());
+            sourcePath.Erase(output);
+
             return true;
         }
 

Code/Sandbox/Editor/EditorVersion.rc

@@ -33,13 +33,13 @@ BEGIN
     BEGIN
         BLOCK "040904b0"
         BEGIN
-            VALUE "CompanyName", "Amazon.com, Inc."
-            VALUE "FileDescription", "Lumberyard Editor"
+            VALUE "CompanyName", "Open 3D Foundation"
+            VALUE "FileDescription", "O3DE Editor"
             VALUE "FileVersion", "0.1.0.1"
             VALUE "InternalName", "Editor"
             VALUE "LegalCopyright", "Portions of this file Copyright (c) Amazon.com, Inc. or its affiliates. All Rights Reserved. Original file Copyright (c) Crytek GMBH. Used under license by Amazon.com, Inc. and its affiliates."
             VALUE "OriginalFilename", "Editor.exe"
-            VALUE "ProductName", "Lumberyard Editor"
+            VALUE "ProductName", "O3DE Editor"
             VALUE "ProductVersion", "0.1.0.1"
         END
     END

Code/Sandbox/Editor/Util/ImageHDR.cpp

@@ -1,460 +0,0 @@
-/*
-* All or portions of this file Copyright (c) Amazon.com, Inc. or its affiliates or
-* its licensors.
-*
-* For complete copyright and license terms please see the LICENSE at the root of this
-* distribution (the "License"). All use of this software is governed by the License,
-* or, if provided, by the license below or the license accompanying this file. Do not
-* remove or modify any license notices. This file is distributed on an "AS IS" BASIS,
-* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-*
-*/
-// Original file Copyright Crytek GMBH or its affiliates, used under license.
-
-#include "EditorDefs.h"
-
-#include "ImageHDR.h"
-
-// Editor
-#include "Util/Image.h"
-
-// We need globals because of the callbacks (they don't allow us to pass state)
-static CryMutex globalFileMutex;
-static size_t globalFileBufferOffset = 0;
-static size_t globalFileBufferSize = 0;
-
-static char* fgets(char* _Buf, [[maybe_unused]] int _MaxCount, CCryFile* _File)
-{
-    while (globalFileBufferOffset < globalFileBufferSize)
-    {
-        char chr;
-
-        _File->ReadRaw(&chr, 1);
-        globalFileBufferOffset++;
-
-        *_Buf++ = chr;
-        if (chr == '\n')
-        {
-            break;
-        }
-    }
-
-    *_Buf = '\0';
-    return _Buf;
-}
-
-static size_t fread(void* _DstBuf, size_t _ElementSize, size_t _Count, CCryFile* _File)
-{
-    size_t cpy = min(_ElementSize * _Count, globalFileBufferSize - globalFileBufferOffset);
-
-    _File->ReadRaw(_DstBuf, cpy);
-    globalFileBufferOffset += cpy;
-
-    return cpy;
-}
-
-/* THIS CODE CARRIES NO GUARANTEE OF USABILITY OR FITNESS FOR ANY PURPOSE.
- * WHILE THE AUTHORS HAVE TRIED TO ENSURE THE PROGRAM WORKS CORRECTLY,
- * IT IS STRICTLY USE AT YOUR OWN RISK.  */
-
-/* utility for reading and writing Ward's rgbe image format.
-   See rgbe.txt file for more details.
-*/
-
-#include <stdio.h>
-
-typedef struct
-{
-    int valid;          /* indicate which fields are valid */
-    char programtype[16]; /* listed at beginning of file to identify it
-                         * after "#?".  defaults to "RGBE" */
-    float gamma;        /* image has already been gamma corrected with
-                         * given gamma.  defaults to 1.0 (no correction) */
-    float exposure;     /* a value of 1.0 in an image corresponds to
-             * <exposure> watts/steradian/m^2.
-             * defaults to 1.0 */
-    char instructions[512];
-} rgbe_header_info;
-
-/* flags indicating which fields in an rgbe_header_info are valid */
-#define RGBE_VALID_PROGRAMTYPE  0x01
-#define RGBE_VALID_GAMMA                0x02
-#define RGBE_VALID_EXPOSURE         0x04
-#define RGBE_VALID_INSTRUCTIONS 0x08
-
-/* return codes for rgbe routines */
-#define RGBE_RETURN_SUCCESS 0
-#define RGBE_RETURN_FAILURE -1
-
-/* read or write headers */
-/* you may set rgbe_header_info to null if you want to */
-int RGBE_ReadHeader(CCryFile* fp, uint32* width, uint32* height, rgbe_header_info* info);
-
-/* read or write pixels */
-/* can read or write pixels in chunks of any size including single pixels*/
-int RGBE_ReadPixels(CCryFile* fp, float* data, int numpixels);
-
-/* read or write run length encoded files */
-/* must be called to read or write whole scanlines */
-int RGBE_ReadPixels_RLE(CCryFile* fp, float* data, uint32 scanline_width,
-    uint32 num_scanlines);
-
-/* THIS CODE CARRIES NO GUARANTEE OF USABILITY OR FITNESS FOR ANY PURPOSE.
- * WHILE THE AUTHORS HAVE TRIED TO ENSURE THE PROGRAM WORKS CORRECTLY,
- * IT IS STRICTLY USE AT YOUR OWN RISK.  */
-
-#include <math.h>
-#include <string.h>
-#include <ctype.h>
-
-/* This file contains code to read and write four byte rgbe file format
- developed by Greg Ward.  It handles the conversions between rgbe and
- pixels consisting of floats.  The data is assumed to be an array of floats.
- By default there are three floats per pixel in the order red, green, blue.
- (RGBE_DATA_??? values control this.)  Only the mimimal header reading and
- writing is implemented.  Each routine does error checking and will return
- a status value as defined below.  This code is intended as a skeleton so
- feel free to modify it to suit your needs.
-
- (Place notice here if you modified the code.)
- posted to http://www.graphics.cornell.edu/~bjw/
- written by Bruce Walter  (bjw@graphics.cornell.edu)  5/26/95
- based on code written by Greg Ward
-*/
-
-#ifndef INLINE
-#ifdef _CPLUSPLUS
-/* define if your compiler understands inline commands */
-#define INLINE inline
-#else
-#define INLINE
-#endif
-#endif
-
-/* offsets to red, green, and blue components in a data (float) pixel */
-#define RGBE_DATA_RED    0
-#define RGBE_DATA_GREEN  1
-#define RGBE_DATA_BLUE   2
-#define RGBE_DATA_ALPHA  3
-/* number of floats per pixel */
-#define RGBE_DATA_SIZE   4
-
-enum rgbe_error_codes
-{
-    rgbe_read_error,
-    rgbe_write_error,
-    rgbe_format_error,
-    rgbe_memory_error,
-};
-
-/* default error routine.  change this to change error handling */
-static int rgbe_error(int rgbe_error_code, const char* msg)
-{
-    switch (rgbe_error_code)
-    {
-    case rgbe_read_error:
-        CLogFile::FormatLine("RGBE read error");
-        break;
-    case rgbe_write_error:
-        CLogFile::FormatLine("RGBE write error");
-        break;
-    case rgbe_format_error:
-        CLogFile::FormatLine("RGBE bad file format: %s\n", msg);
-        break;
-    default:
-    case rgbe_memory_error:
-        CLogFile::FormatLine("RGBE error: %s\n", msg);
-    }
-    return RGBE_RETURN_FAILURE;
-}
-
-/* standard conversion from rgbe to float pixels */
-/* note: Ward uses ldexp(col+0.5,exp-(128+8)).  However we wanted pixels */
-/*       in the range [0,1] to map back into the range [0,1].            */
-static INLINE void
-rgbe2type(char* red, char* green, char* blue, unsigned char rgbe[4])
-{
-    float f;
-
-    if (rgbe[3])     /*nonzero pixel*/
-    {
-        f = ldexp(1.0f, rgbe[3] - (int)(128 + 8)) * 255.0f;
-        *red   = (unsigned char) max(0.0f, min(rgbe[0] * f, 255.0f));
-        *green = (unsigned char) max(0.0f, min(rgbe[1] * f, 255.0f));
-        *blue  = (unsigned char) max(0.0f, min(rgbe[2] * f, 255.0f));
-    }
-    else
-    {
-        *red = *green = *blue = 0;
-    }
-}
-
-/* minimal header reading.  modify if you want to parse more information */
-int RGBE_ReadHeader(CCryFile* fp, uint32* width, uint32* height, rgbe_header_info* info)
-{
-    char buf[512];
-    int found_format;
-    float tempf;
-    int i;
-
-    found_format = 0;
-    if (info)
-    {
-        info->valid = 0;
-        info->programtype[0] = 0;
-        info->gamma = info->exposure = 1.0;
-    }
-    if (fgets(buf, sizeof(buf) / sizeof(buf[0]), fp) == NULL)
-    {
-        return rgbe_error(rgbe_read_error, NULL);
-    }
-    if ((buf[0] != '#') || (buf[1] != '?'))
-    {
-        /* if you want to require the magic token then uncomment the next line */
-        /*return rgbe_error(rgbe_format_error,"bad initial token"); */
-    }
-    else if (info)
-    {
-        info->valid |= RGBE_VALID_PROGRAMTYPE;
-        for (i = 0; i < sizeof(info->programtype) - 1; i++)
-        {
-            if ((buf[i + 2] == 0) || isspace(buf[i + 2]))
-            {
-                break;
-            }
-            info->programtype[i] = buf[i + 2];
-        }
-        info->programtype[i] = 0;
-        if (fgets(buf, sizeof(buf) / sizeof(buf[0]), fp) == 0)
-        {
-            return rgbe_error(rgbe_read_error, NULL);
-        }
-    }
-    for (;; )
-    {
-        if ((buf[0] == 0) || (buf[0] == '\n'))
-        {
-            return rgbe_error(rgbe_format_error, "no FORMAT specifier found");
-        }
-        else if (strcmp(buf, "FORMAT=32-bit_rle_rgbe\n") == 0)
-        {
-            break; /* format found so break out of loop */
-        }
-        else if (info && (azsscanf(buf, "GAMMA=%g", &tempf) == 1))
-        {
-            info->gamma = tempf;
-            info->valid |= RGBE_VALID_GAMMA;
-        }
-        else if (info && (azsscanf(buf, "EXPOSURE=%g", &tempf) == 1))
-        {
-            info->exposure = tempf;
-            info->valid |= RGBE_VALID_EXPOSURE;
-        }
-        else if (info && (!strncmp(buf, "INSTRUCTIONS=", 13)))
-        {
-            info->valid |= RGBE_VALID_INSTRUCTIONS;
-            for (i = 0; i < sizeof(info->instructions) - 1; i++)
-            {
-                if ((buf[i + 13] == 0) || isspace(buf[i + 13]))
-                {
-                    break;
-                }
-                info->instructions[i] = buf[i + 13];
-            }
-        }
-        if (fgets(buf, sizeof(buf) / sizeof(buf[0]), fp) == 0)
-        {
-            return rgbe_error(rgbe_read_error, NULL);
-        }
-    }
-    if (fgets(buf, sizeof(buf) / sizeof(buf[0]), fp) == 0)
-    {
-        return rgbe_error(rgbe_read_error, NULL);
-    }
-    if (strcmp(buf, "\n") != 0)
-    {
-        return rgbe_error(rgbe_format_error,
-            "missing blank line after FORMAT specifier");
-    }
-    if (fgets(buf, sizeof(buf) / sizeof(buf[0]), fp) == 0)
-    {
-        return rgbe_error(rgbe_read_error, NULL);
-    }
-    if (azsscanf(buf, "-Y %d +X %d", height, width) < 2)
-    {
-        return rgbe_error(rgbe_format_error, "missing image size specifier");
-    }
-    return RGBE_RETURN_SUCCESS;
-}
-
-/* simple read routine.  will not correctly handle run length encoding */
-int RGBE_ReadPixels(CCryFile* fp, char* data, int numpixels)
-{
-    unsigned char rgbe[4];
-
-    while (numpixels-- > 0)
-    {
-        if (fread(rgbe, sizeof(rgbe), 1, fp) < 1)
-        {
-            return rgbe_error(rgbe_read_error, NULL);
-        }
-        rgbe2type(&data[RGBE_DATA_RED], &data[RGBE_DATA_GREEN],
-            &data[RGBE_DATA_BLUE], rgbe);
-        data[RGBE_DATA_ALPHA] = 0.0f;
-        data += RGBE_DATA_SIZE;
-    }
-    return RGBE_RETURN_SUCCESS;
-}
-
-int RGBE_ReadPixels_RLE(CCryFile* fp, char* data, uint32 scanline_width,
-    uint32 num_scanlines)
-{
-    unsigned char rgbe[4], * scanline_buffer, * ptr, * ptr_end;
-    int i, count;
-    unsigned char buf[2];
-
-    if ((scanline_width < 8) || (scanline_width > 0x7fff))
-    {
-        /* run length encoding is not allowed so read flat*/
-        return RGBE_ReadPixels(fp, data, scanline_width * num_scanlines);
-    }
-    scanline_buffer = NULL;
-    /* read in each successive scanline */
-    while (num_scanlines > 0)
-    {
-        if (fread(rgbe, sizeof(rgbe), 1, fp) < 1)
-        {
-            free(scanline_buffer);
-            return rgbe_error(rgbe_read_error, NULL);
-        }
-        if ((rgbe[0] != 2) || (rgbe[1] != 2) || (rgbe[2] & 0x80))
-        {
-            /* this file is not run length encoded */
-            rgbe2type(&data[0], &data[1], &data[2], rgbe);
-            data += RGBE_DATA_SIZE;
-            free(scanline_buffer);
-            return RGBE_ReadPixels(fp, data, scanline_width * num_scanlines - 1);
-        }
-        if ((((int)rgbe[2]) << 8 | rgbe[3]) != scanline_width)
-        {
-            free(scanline_buffer);
-            return rgbe_error(rgbe_format_error, "wrong scanline width");
-        }
-        if (scanline_buffer == NULL)
-        {
-            scanline_buffer = (unsigned char*)
-                malloc(sizeof(unsigned char) * 4 * scanline_width);
-        }
-        if (scanline_buffer == NULL)
-        {
-            return rgbe_error(rgbe_memory_error, "unable to allocate buffer space");
-        }
-
-        ptr = &scanline_buffer[0];
-        /* read each of the four channels for the scanline into the buffer */
-        for (i = 0; i < 4; i++)
-        {
-            ptr_end = &scanline_buffer[(i + 1) * scanline_width];
-            while (ptr < ptr_end)
-            {
-                if (fread(buf, sizeof(buf[0]) * 2, 1, fp) < 1)
-                {
-                    free(scanline_buffer);
-                    return rgbe_error(rgbe_read_error, NULL);
-                }
-                if (buf[0] > 128)
-                {
-                    /* a run of the same value */
-                    count = buf[0] - 128;
-                    if ((count == 0) || (count > ptr_end - ptr))
-                    {
-                        free(scanline_buffer);
-                        return rgbe_error(rgbe_format_error, "bad scanline data");
-                    }
-                    while (count-- > 0)
-                    {
-                        *ptr++ = buf[1];
-                    }
-                }
-                else
-                {
-                    /* a non-run */
-                    count = buf[0];
-                    if ((count == 0) || (count > ptr_end - ptr))
-                    {
-                        free(scanline_buffer);
-                        return rgbe_error(rgbe_format_error, "bad scanline data");
-                    }
-                    *ptr++ = buf[1];
-                    if (--count > 0)
-                    {
-                        if (fread(ptr, sizeof(*ptr) * count, 1, fp) < 1)
-                        {
-                            free(scanline_buffer);
-                            return rgbe_error(rgbe_read_error, NULL);
-                        }
-                        ptr += count;
-                    }
-                }
-            }
-        }
-        /* now convert data from buffer into floats */
-        for (i = 0; i < scanline_width; i++)
-        {
-            rgbe[0] = scanline_buffer[i];
-            rgbe[1] = scanline_buffer[i + scanline_width];
-            rgbe[2] = scanline_buffer[i + 2 * scanline_width];
-            rgbe[3] = scanline_buffer[i + 3 * scanline_width];
-            rgbe2type(&data[RGBE_DATA_RED], &data[RGBE_DATA_GREEN],
-                &data[RGBE_DATA_BLUE], rgbe);
-            data[RGBE_DATA_ALPHA] = 0.0f;
-            data += RGBE_DATA_SIZE;
-        }
-        num_scanlines--;
-    }
-    free(scanline_buffer);
-    return RGBE_RETURN_SUCCESS;
-}
-
-///////////////////////////////////////////////////////////////////////////////////
-
-bool CImageHDR::Load(const QString& fileName, CImageEx& outImage)
-{
-    CCryFile file;
-    if (!file.Open(fileName.toUtf8().data(), "rb"))
-    {
-        CLogFile::FormatLine("File not found %s", fileName.toUtf8().data());
-        return false;
-    }
-
-    // We use some global variables in callbacks, so we must
-    // prevent multithread access to the data
-    CryAutoLock<CryMutex> tifAutoLock(globalFileMutex);
-
-    globalFileBufferSize = file.GetLength();
-    globalFileBufferOffset = 0;
-
-    bool bRet = false;
-    uint32 dwWidth, dwHeight;
-    rgbe_header_info info;
-
-    if (RGBE_RETURN_SUCCESS == RGBE_ReadHeader(&file, &dwWidth, &dwHeight, &info))
-    {
-        if (outImage.Allocate(dwWidth, dwHeight))
-        {
-            char* pDst = (char*)outImage.GetData();
-
-            if (RGBE_RETURN_SUCCESS == RGBE_ReadPixels_RLE(&file, (char*)pDst, dwWidth, dwHeight))
-            {
-                bRet = true;
-            }
-        }
-    }
-
-    if (!bRet)
-    {
-        outImage.Detach();
-    }
-
-    return bRet;
-}

Code/Sandbox/Editor/Util/ImageHDR.h

@@ -1,22 +0,0 @@
-/*
-* All or portions of this file Copyright (c) Amazon.com, Inc. or its affiliates or
-* its licensors.
-*
-* For complete copyright and license terms please see the LICENSE at the root of this
-* distribution (the "License"). All use of this software is governed by the License,
-* or, if provided, by the license below or the license accompanying this file. Do not
-* remove or modify any license notices. This file is distributed on an "AS IS" BASIS,
-* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-*
-*/
-// Original file Copyright Crytek GMBH or its affiliates, used under license.
-
-#pragma once
-
-class CImageEx;
-
-class CImageHDR
-{
-public:
-    bool Load(const QString& fileName, CImageEx& outImage);
-};

Code/Sandbox/Editor/Util/ImageUtil.cpp

@@ -21,7 +21,6 @@
 // Editor
 #include "Util/ImageGif.h"
 #include "Util/ImageTIF.h"
-#include "Util/ImageHDR.h"
 
 //////////////////////////////////////////////////////////////////////////
 bool CImageUtil::Save(const QString& strFileName, CImageEx& inImage)
@@ -275,10 +274,6 @@ bool CImageUtil::LoadImage(const QString& fileName, CImageEx& image, bool* pQual
     {
         return CImageUtil::Load(fileName, image);
     }
-    else if (azstricmp(ext, ".hdr") == 0)
-    {
-        return CImageHDR().Load(fileName, image);
-    }
     else
     {
         return CImageUtil::Load(fileName, image);

Code/Sandbox/Editor/editor_lib_files.cmake

@@ -737,8 +737,6 @@ set(FILES
     Util/GeometryUtil.cpp
     Util/GuidUtil.cpp
     Util/GuidUtil.h
-    Util/ImageHDR.cpp
-    Util/ImageHDR.h
     Util/IObservable.h
     Util/IndexedFiles.cpp
     Util/IndexedFiles.h

Gems/AWSCore/Code/CMakeLists.txt

@@ -144,6 +144,7 @@ if(PAL_TRAIT_BUILD_TESTS_SUPPORTED)
             FILES_CMAKE
                 awscore_editor_tests_files.cmake
                 ${pal_editor_include_dir}/platform_${PAL_PLATFORM_NAME_LOWERCASE}_files.cmake
+                Tests/Editor/Platform/${PAL_PLATFORM_NAME}/awscore_editor_tests_${PAL_PLATFORM_NAME_LOWERCASE}_files.cmake
             INCLUDE_DIRECTORIES
                 PRIVATE
                     Include/Private

Gems/AWSCore/Code/Source/Editor/Attribution/AWSCoreAttributionManager.cpp

@@ -263,15 +263,13 @@ namespace AWSCore
         SetApiEndpointAndRegion(config);
 
         ServiceAPI::AWSAttributionRequestJob* requestJob = ServiceAPI::AWSAttributionRequestJob::Create(
-            [this](ServiceAPI::AWSAttributionRequestJob* successJob)
+            [this]([[maybe_unused]] ServiceAPI::AWSAttributionRequestJob* successJob)
             {
-                AZ_UNUSED(successJob);
-                
                 UpdateLastSend();
                 AZ_Printf("AWSAttributionManager", "AWSAttribution metric submit success");
 
             },
-            [this](ServiceAPI::AWSAttributionRequestJob* failJob)
+            [this]([[maybe_unused]] ServiceAPI::AWSAttributionRequestJob* failJob)
             {
                 AZ_Error("AWSAttributionManager", false, "Metrics send error: %s", failJob->error.message.c_str());
             },

Gems/AWSCore/Code/Tests/Editor/Platform/Linux/awscore_editor_tests_linux_files.cmake

@@ -0,0 +1,13 @@
+#
+# All or portions of this file Copyright (c) Amazon.com, Inc. or its affiliates or
+# its licensors.
+#
+# For complete copyright and license terms please see the LICENSE at the root of this
+# distribution (the "License"). All use of this software is governed by the License,
+# or, if provided, by the license below or the license accompanying this file. Do not
+# remove or modify any license notices. This file is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+#
+
+set(FILES 
+)

Gems/AWSCore/Code/Tests/Editor/Platform/Mac/awscore_editor_tests_mac_files.cmake

@@ -0,0 +1,13 @@
+#
+# All or portions of this file Copyright (c) Amazon.com, Inc. or its affiliates or
+# its licensors.
+#
+# For complete copyright and license terms please see the LICENSE at the root of this
+# distribution (the "License"). All use of this software is governed by the License,
+# or, if provided, by the license below or the license accompanying this file. Do not
+# remove or modify any license notices. This file is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+#
+
+set(FILES 
+)

Gems/AWSCore/Code/Tests/Editor/Platform/Windows/awscore_editor_tests_windows_files.cmake

@@ -0,0 +1,22 @@
+#
+# All or portions of this file Copyright (c) Amazon.com, Inc. or its affiliates or
+# its licensors.
+#
+# For complete copyright and license terms please see the LICENSE at the root of this
+# distribution (the "License"). All use of this software is governed by the License,
+# or, if provided, by the license below or the license accompanying this file. Do not
+# remove or modify any license notices. This file is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+#
+
+set(FILES 
+    ../../AWSCoreEditorSystemComponentTest.cpp
+    ../../Attribution/AWSCoreAttributionManagerTest.cpp
+    ../../Attribution/AWSCoreAttributionMetricTest.cpp
+    ../../Attribution/AWSCoreAttributionSystemComponentTest.cpp
+    ../../Attribution/AWSAttributionServiceApiTest.cpp
+    ../../UI/AWSCoreEditorMenuTest.cpp
+    ../../UI/AWSCoreEditorUIFixture.h
+    ../../UI/AWSCoreResourceMappingToolActionTest.cpp
+    ../../AWSCoreEditorManagerTest.cpp
+)

Gems/AWSCore/Code/awscore_editor_tests_files.cmake

@@ -10,14 +10,5 @@
 #
 
 set(FILES 
-    Tests/AWSCoreEditorSystemComponentTest.cpp
-    Tests/Editor/Attribution/AWSCoreAttributionManagerTest.cpp
-    Tests/Editor/Attribution/AWSCoreAttributionMetricTest.cpp
-    Tests/Editor/Attribution/AWSCoreAttributionSystemComponentTest.cpp
-    Tests/Editor/Attribution/AWSAttributionServiceApiTest.cpp
-    Tests/Editor/UI/AWSCoreEditorMenuTest.cpp
-    Tests/Editor/UI/AWSCoreEditorUIFixture.h
-    Tests/Editor/UI/AWSCoreResourceMappingToolActionTest.cpp
-    Tests/Editor/AWSCoreEditorManagerTest.cpp
     Tests/Editor/AWSCoreEditorTest.cpp
 )

Gems/AWSCore/cdk/example/example_resources_stack.py

@@ -11,10 +11,10 @@ WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 import os
 
 from aws_cdk import (
-    aws_lambda as _lambda,
-    aws_s3 as _s3,
+    aws_lambda as lambda_,
+    aws_s3 as s3,
     aws_s3_deployment as s3_deployment,
-    aws_dynamodb as _dynamo,
+    aws_dynamodb as dynamo,
     core
 )
 
@@ -39,7 +39,7 @@ class ExampleResources(core.Stack):
         self._feature_name = feature_name
 
         self._policy = AuthPolicy(context=self).generate_admin_policy(stack=self)
-        self._s3 = self.__create_s3_bucket()
+        self._s3_bucket = self.__create_s3_bucket()
         self._lambda = self.__create_example_lambda()
         self._table = self.__create_dynamodb_table()
 
@@ -49,8 +49,8 @@ class ExampleResources(core.Stack):
         self.__grant_access(props=props_)
 
     def __grant_access(self, props: CoreStackProperties):
-        self._s3.grant_read(props.user_group)
-        self._s3.grant_read(props.admin_group)
+        self._s3_bucket.grant_read(props.user_group)
+        self._s3_bucket.grant_read(props.admin_group)
 
         self._lambda.grant_invoke(props.user_group)
         self._lambda.grant_invoke(props.admin_group)
@@ -58,42 +58,50 @@ class ExampleResources(core.Stack):
         self._table.grant_read_data(props.user_group)
         self._table.grant_read_data(props.admin_group)
 
-    def __create_s3_bucket(self) -> _s3.Bucket:
-        # create s3 bucket
-
-        # create s3 bucket
-        s3 = _s3.Bucket(self, f'{self._project_name}-{self._feature_name}-Example-S3bucket')
+    def __create_s3_bucket(self) -> s3.Bucket:
+        # Create a sample S3 bucket following S3 best practices
+        # # See https://docs.aws.amazon.com/AmazonS3/latest/dev/security-best-practices.html
+        # 1. Block all public access to the bucket
+        # 2. Use SSE-S3 encryption. Explore encryption at rest options via
+        #    https://docs.aws.amazon.com/AmazonS3/latest/userguide/serv-side-encryption.html
+        example_bucket = s3.Bucket(
+            self,
+            f'{self._project_name}-{self._feature_name}-Example-S3bucket',
+            block_public_access=s3.BlockPublicAccess.BLOCK_ALL,
+            encryption=s3.BucketEncryption.S3_MANAGED
+        )
 
         s3_deployment.BucketDeployment(
             self,
             f'{self._project_name}-{self._feature_name}-S3bucket-Deployment',
-            destination_bucket=s3,
+            destination_bucket=example_bucket,
             sources=[
                 s3_deployment.Source.asset('example/s3_content')
             ],
             retain_on_delete=False
         )
+        return example_bucket
 
-        return s3
-
-    def __create_example_lambda(self) -> _lambda.Function:
+    def __create_example_lambda(self) -> lambda_.Function:
         # create lambda function
-        function = _lambda.Function(self,
-                                    f'{self._project_name}-{self._feature_name}-Lambda-Function',
-                                    runtime=_lambda.Runtime.PYTHON_3_8,
-                                    handler="lambda-handler.main",
-                                    code=_lambda.Code.asset(os.path.join(os.path.dirname(__file__), 'lambda')))
+        function = lambda_.Function(
+            self,
+            f'{self._project_name}-{self._feature_name}-Lambda-Function',
+            runtime=lambda_.Runtime.PYTHON_3_8,
+            handler="lambda-handler.main",
+            code=lambda_.Code.asset(os.path.join(os.path.dirname(__file__), 'lambda'))
+        )
         return function
 
-    def __create_dynamodb_table(self) -> _dynamo.Table:
+    def __create_dynamodb_table(self) -> dynamo.Table:
         # create dynamo table
         # NB: CDK does not support seeding data, see simple table_seeder.py
-        demo_table = _dynamo.Table(
+        demo_table = dynamo.Table(
             self,
             f'{self._project_name}-{self._feature_name}-Table',
-            partition_key=_dynamo.Attribute(
+            partition_key=dynamo.Attribute(
                 name="id",
-                type=_dynamo.AttributeType.STRING
+                type=dynamo.AttributeType.STRING
             )
         )
         return demo_table
@@ -106,7 +114,7 @@ class ExampleResources(core.Stack):
             id=f'ExampleBucketOutput',
             description='An example S3 bucket to use with AWSCore ScriptBehaviors',
             export_name=f"ExampleS3Bucket",
-            value=self._s3.bucket_arn)
+            value=self._s3_bucket.bucket_arn)
 
         # Define exports
         # Export resource group

Gems/AWSMetrics/cdk/aws_metrics/real_time_data_processing.py

@@ -182,7 +182,7 @@ class RealTimeDataProcessing:
         """
         Generate the analytics processing lambda to send processed data to CloudWatch for visualization.
         """
-        analytics_processing_function_name = f'{self._stack.stack_name}-AnalyticsProcessingLambda'
+        analytics_processing_function_name = f'{self._stack.stack_name}-AnalyticsProcessingLambdaName'
         self._analytics_processing_lambda_role = self._create_analytics_processing_lambda_role(
             analytics_processing_function_name
         )

Gems/Atom/Feature/Common/Assets/ShaderResourceGroups/SkyBox/SceneSrg.azsli

@@ -40,6 +40,11 @@ partial ShaderResourceGroup SceneSrg
     ConstantBuffer<PhysicalSkyData> m_physicalSkyData;
     bool m_physicalSky;
     
+    float m_fogTopHeight;
+    float m_fogBottomHeight;
+    float4 m_fogColor;
+    bool m_fogEnable;
+
     TextureCube m_skyboxCubemap;
     float4x4 m_cubemapRotationMatrix;
     float m_cubemapExposure;

Gems/Atom/Feature/Common/Assets/Shaders/SkyBox/SkyBox.azsl

@@ -152,6 +152,18 @@ PSOutput MainPS(VSOutput input)
                 float3 srgbColor = Z * HosekWilkie(cosGamma, gamma, cosTheta) * SceneSrg::m_physicalSkyData.m_physicalSkyAndSunIntensity.x;
                 color = TransformColor(srgbColor, ColorSpaceId::LinearSRGB, ColorSpaceId::ACEScg);
             }
+        } 
+
+        if (SceneSrg::m_fogEnable)
+        {
+            if (input.m_cubemapCoord.z >= 0.0 && input.m_cubemapCoord.z <= SceneSrg::m_fogTopHeight)
+            {
+                color = lerp(SceneSrg::m_fogColor.rgb, color, input.m_cubemapCoord.z > 0.0 ? input.m_cubemapCoord.z/SceneSrg::m_fogTopHeight : 0.0);
+            }
+            else if (input.m_cubemapCoord.z < 0.0 && input.m_cubemapCoord.z >= -SceneSrg::m_fogBottomHeight)
+            {
+                color = SceneSrg::m_fogColor.rgb;
+            }
         }
     }
     else

Gems/Atom/Feature/Common/Code/Include/Atom/Feature/SkyBox/SkyBoxFeatureProcessorInterface.h

@@ -16,6 +16,7 @@
 #include <Atom/RPI.Reflect/Image/Image.h>
 #include <Atom/Feature/CoreLights/PhotometricValue.h>
 #include <AzCore/Math/Matrix4x4.h>
+#include <SkyBox/SkyBoxFogSettings.h>
 
 namespace AZ
 {
@@ -49,8 +50,9 @@ namespace AZ
             AZ_RTTI(AZ::Render::SkyBoxFeatureProcessorInterface, "{71061869-1190-4451-A337-E9CFF16441B4}"); 
 
             virtual void Enable(bool enable) = 0;
-            virtual bool IsEnable() = 0;
+            virtual bool IsEnabled() = 0;
             virtual void SetSkyboxMode(SkyBoxMode mode) = 0;
+            virtual void SetFogSettings(const SkyBoxFogSettings& fogSettings) = 0;
 
             // HDRiSkyBox
             virtual void SetCubemap(Data::Instance<RPI::Image> cubemap) = 0;
@@ -64,6 +66,13 @@ namespace AZ
             virtual void SetSkyIntensity(float intensity, PhotometricUnit type) = 0;
             virtual void SetSunIntensity(float intensity, PhotometricUnit type) = 0;
             virtual void SetSunRadiusFactor(float factor) = 0;
+
+            // Fog Settings
+            virtual void SetFogEnabled(bool enable) = 0;
+            virtual bool IsFogEnabled() = 0;
+            virtual void SetFogColor(const AZ::Color &color) = 0;
+            virtual void SetFogTopHeight(float topHeight) = 0;
+            virtual void SetFogBottomHeight(float bottomHeight) = 0;
         };
     }
 }

Gems/Atom/Feature/Common/Code/Include/Atom/Feature/SkyBox/SkyBoxFogBus.h

@@ -0,0 +1,46 @@
+/*
+ * All or portions of this file Copyright (c) Amazon.com, Inc. or its affiliates or
+ * its licensors.
+ *
+ * For complete copyright and license terms please see the LICENSE at the root of this
+ * distribution (the "License"). All use of this software is governed by the License,
+ * or, if provided, by the license below or the license accompanying this file. Do not
+ * remove or modify any license notices. This file is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ *
+ */
+
+#pragma once
+
+#include <AzCore/Component/ComponentBus.h>
+#include <AzCore/Math/Color.h>
+
+namespace AZ
+{
+    namespace Render
+    {
+        // EBus to get and set fog settings rendered with the sky
+        class SkyBoxFogRequests
+            : public ComponentBus
+        {
+        public:
+            AZ_RTTI(AZ::Render::SkyBoxFogRequests, "{4D477566-54B1-49EC-B8FE-4264EA228482}");
+
+            static const EBusHandlerPolicy HandlerPolicy = EBusHandlerPolicy::Single;
+            virtual ~SkyBoxFogRequests() {}
+
+            virtual void SetEnabled(bool enable) = 0;
+            virtual bool IsEnabled() const = 0;
+            virtual void SetColor(const AZ::Color& color) = 0;
+            virtual const AZ::Color& GetColor() const = 0;
+            // Set and Get the height upwards from the horizon
+            virtual void SetTopHeight(float topHeight) = 0;
+            virtual float GetTopHeight() const = 0;
+            // Set and Get the height downwards from the horizon
+            virtual void SetBottomHeight(float bottomHeight) = 0;
+            virtual float GetBottomHeight() const = 0;
+        };
+
+        typedef AZ::EBus<SkyBoxFogRequests> SkyBoxFogRequestBus;
+    }
+}

Gems/Atom/Feature/Common/Code/Source/CommonSystemComponent.cpp

@@ -70,7 +70,7 @@
 #include <PostProcessing/BloomCompositePass.h>
 #include <ScreenSpace/DeferredFogPass.h>
 #include <Shadows/ProjectedShadowFeatureProcessor.h>
-
+#include <SkyBox/SkyBoxFogSettings.h>
 #include <SkyBox/SkyBoxFeatureProcessor.h>
 
 #include <Atom/RPI.Public/Pass/PassSystemInterface.h>
@@ -117,6 +117,7 @@ namespace AZ
             TransformServiceFeatureProcessor::Reflect(context);
             ProjectedShadowFeatureProcessor::Reflect(context);
             SkyBoxFeatureProcessor::Reflect(context);
+            SkyBoxFogSettings::Reflect(context);
             UseTextureFunctor::Reflect(context);
             DrawListFunctor::Reflect(context);
             SubsurfaceTransmissionParameterFunctor::Reflect(context);

Gems/Atom/Feature/Common/Code/Source/ReflectionScreenSpace/ReflectionScreenSpaceBlurChildPass.cpp

@@ -47,6 +47,9 @@ namespace AZ
                 m_updateSrg = true;
             }
 
+            params.m_viewportState = RHI::Viewport(0, static_cast<float>(m_imageSize.m_width), 0, static_cast<float>(m_imageSize.m_height));
+            params.m_scissorState = RHI::Scissor(0, 0, m_imageSize.m_width, m_imageSize.m_height);
+
             FullscreenTrianglePass::FrameBeginInternal(params);
         }
 

Gems/Atom/Feature/Common/Code/Source/SkyBox/SkyBoxFeatureProcessor.cpp

@@ -89,6 +89,10 @@ namespace AZ
             m_cubemapIndex.Reset();
             m_cubemapRotationMatrixIndex.Reset();
             m_cubemapExposureIndex.Reset();
+            m_fogEnableIndex.Reset();
+            m_fogColorIndex.Reset();
+            m_fogTopHeightIndex.Reset();
+            m_fogBottomHeightIndex.Reset();
 
             if (m_buffer)
             {
@@ -160,6 +164,14 @@ namespace AZ
                              m_mapBuffer = false;
                          }
 
+                        m_sceneSrg->SetConstant(m_fogEnableIndex, m_fogSettings.m_enable);
+                        if (m_fogSettings.m_enable)
+                        {
+                            m_sceneSrg->SetConstant(m_fogTopHeightIndex, m_fogSettings.m_topHeight);
+                            m_sceneSrg->SetConstant(m_fogBottomHeightIndex, m_fogSettings.m_bottomHeight);
+                            m_sceneSrg->SetConstant(m_fogColorIndex, m_fogSettings.m_color);
+                        }
+
                         m_sceneSrg->SetConstant(m_physicalSkyIndex, true);
                         break;
                     }
@@ -213,7 +225,7 @@ namespace AZ
             m_enable = enable;
         }
 
-        bool SkyBoxFeatureProcessor::IsEnable()
+        bool SkyBoxFeatureProcessor::IsEnabled()
         {
             return m_enable;
         }
@@ -238,6 +250,36 @@ namespace AZ
             m_skyboxMode = mode;
         }
 
+        void SkyBoxFeatureProcessor::SetFogSettings(const SkyBoxFogSettings& fogSettings)
+        {
+            m_fogSettings = fogSettings;
+        }
+
+        void SkyBoxFeatureProcessor::SetFogEnabled(bool enable)
+        {
+            m_fogSettings.m_enable = enable;
+        }
+
+        bool SkyBoxFeatureProcessor::IsFogEnabled()
+        {
+            return m_fogSettings.m_enable;
+        }
+
+        void SkyBoxFeatureProcessor::SetFogColor(const AZ::Color& color)
+        {
+            m_fogSettings.m_color = color;
+        }
+
+        void SkyBoxFeatureProcessor::SetFogTopHeight(float topHeight)
+        {
+            m_fogSettings.m_topHeight = topHeight;
+        }
+
+        void SkyBoxFeatureProcessor::SetFogBottomHeight(float bottomHeight)
+        {
+            m_fogSettings.m_bottomHeight = bottomHeight;
+        }
+
         void SkyBoxFeatureProcessor::SetSunPosition(SunPosition sunPosition)
         {
             m_skyNeedUpdate = true;

Gems/Atom/Feature/Common/Code/Source/SkyBox/SkyBoxFeatureProcessor.h

@@ -67,8 +67,14 @@ namespace AZ
 
             // SkyBoxFeatureProcessorInterface overrides ...
             void Enable(bool enable) override;
-            bool IsEnable() override;
+            bool IsEnabled() override;
             void SetSkyboxMode(SkyBoxMode mode) override;
+            void SetFogSettings(const SkyBoxFogSettings& fogSettings) override;
+            void SetFogEnabled(bool enable) override;
+            bool IsFogEnabled() override;
+            void SetFogColor(const AZ::Color& color) override;
+            void SetFogTopHeight(float topHeight) override;
+            void SetFogBottomHeight(float bottomHeight) override;
 
             void SetCubemapRotationMatrix(AZ::Matrix4x4 matrix) override;
             void SetCubemap(Data::Instance<RPI::Image> cubemap) override;
@@ -145,6 +151,10 @@ namespace AZ
             RHI::ShaderInputNameIndex m_cubemapIndex = "m_skyboxCubemap";
             RHI::ShaderInputNameIndex m_cubemapRotationMatrixIndex = "m_cubemapRotationMatrix";
             RHI::ShaderInputNameIndex m_cubemapExposureIndex = "m_cubemapExposure";
+            RHI::ShaderInputNameIndex m_fogEnableIndex = "m_fogEnable";
+            RHI::ShaderInputNameIndex m_fogColorIndex = "m_fogColor";
+            RHI::ShaderInputNameIndex m_fogTopHeightIndex = "m_fogTopHeight";
+            RHI::ShaderInputNameIndex m_fogBottomHeightIndex = "m_fogBottomHeight";
 
             bool m_skyNeedUpdate = true;
             bool m_sunNeedUpdate = true;
@@ -152,6 +162,7 @@ namespace AZ
 
             bool m_enable = false;
             SkyBoxMode m_skyboxMode = SkyBoxMode::None;
+            SkyBoxFogSettings m_fogSettings;
             Data::Instance<RPI::ShaderResourceGroup> m_sceneSrg = nullptr;
 
             Data::Instance<RPI::Image> m_cubemapTexture = nullptr;

Gems/Atom/Feature/Common/Code/Source/SkyBox/SkyBoxFogSettings.cpp

@@ -0,0 +1,83 @@
+/*
+ * All or portions of this file Copyright (c) Amazon.com, Inc. or its affiliates or
+ * its licensors.
+ *
+ * For complete copyright and license terms please see the LICENSE at the root of this
+ * distribution (the "License"). All use of this software is governed by the License,
+ * or, if provided, by the license below or the license accompanying this file. Do not
+ * remove or modify any license notices. This file is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ *
+ */
+
+#include <SkyBox/SkyBoxFogSettings.h>
+#include <AzCore/Serialization/SerializeContext.h>
+#include <AzCore/Serialization/EditContext.h>
+#include <AzCore/RTTI/BehaviorContext.h>
+#include <Atom/Feature/SkyBox/SkyBoxFogBus.h>
+
+namespace AZ
+{
+    namespace Render
+    {
+        void SkyBoxFogSettings::Reflect(ReflectContext* context)
+        {
+            if (auto serializeContext = azrtti_cast<AZ::SerializeContext*>(context))
+            {
+                serializeContext->Class<SkyBoxFogSettings>()
+                    ->Version(1)
+                    ->Field("Enable", &SkyBoxFogSettings::m_enable)
+                    ->Field("Color", &SkyBoxFogSettings::m_color)
+                    ->Field("TopHeight", &SkyBoxFogSettings::m_topHeight)
+                    ->Field("BottomHeight", &SkyBoxFogSettings::m_bottomHeight)
+                    ;
+
+                if (auto editContext = serializeContext->GetEditContext())
+                {
+                    editContext->Class<SkyBoxFogSettings>("SkyBoxFogSettings", "")
+                        ->ClassElement(AZ::Edit::ClassElements::EditorData, "")
+                            ->DataElement(AZ::Edit::UIHandlers::Default, &SkyBoxFogSettings::m_enable, "Enable Fog", "Toggle fog on or off")
+                            ->DataElement(AZ::Edit::UIHandlers::Default, &SkyBoxFogSettings::m_color, "Fog Color", "Color of the fog")
+                                ->Attribute(AZ::Edit::Attributes::ReadOnly, &SkyBoxFogSettings::IsFogDisabled)
+                            ->DataElement(AZ::Edit::UIHandlers::Slider, &SkyBoxFogSettings::m_topHeight, "Fog Top Height", "Height of the fog upwards from the horizon")
+                                ->Attribute(AZ::Edit::Attributes::ReadOnly, &SkyBoxFogSettings::IsFogDisabled)
+                                ->Attribute(AZ::Edit::Attributes::Min, 0.0)
+                                ->Attribute(AZ::Edit::Attributes::Max, 0.5)
+                                ->Attribute(AZ::Edit::Attributes::Step, 0.01)
+                            ->DataElement(AZ::Edit::UIHandlers::Slider, &SkyBoxFogSettings::m_bottomHeight, "Fog Bottom Height", "Height of the fog downwards from the horizon")
+                                ->Attribute(AZ::Edit::Attributes::ReadOnly, &SkyBoxFogSettings::IsFogDisabled)
+                                ->Attribute(AZ::Edit::Attributes::Min, 0.0)
+                                ->Attribute(AZ::Edit::Attributes::Max, 0.3)
+                                ->Attribute(AZ::Edit::Attributes::Step, 0.01)
+                        ;
+                }
+            }
+
+            if (auto behaviorContext = azrtti_cast<AZ::BehaviorContext*>(context))
+            {
+                behaviorContext->EBus<SkyBoxFogRequestBus>("SkyBoxFogRequestBus")
+                    ->Attribute(AZ::Script::Attributes::Scope, AZ::Script::Attributes::ScopeFlags::Common)
+                    ->Attribute(AZ::Script::Attributes::Category, "render")
+                    ->Attribute(AZ::Script::Attributes::Module, "render")
+                    ->Event("SetEnabled", &SkyBoxFogRequestBus::Events::SetEnabled)
+                    ->Event("IsEnabled", &SkyBoxFogRequestBus::Events::IsEnabled)
+                    ->Event("SetColor", &SkyBoxFogRequestBus::Events::SetColor)
+                    ->Event("GetColor", &SkyBoxFogRequestBus::Events::GetColor)
+                    ->Event("SetTopHeight", &SkyBoxFogRequestBus::Events::SetTopHeight)
+                    ->Event("GetTopHeight", &SkyBoxFogRequestBus::Events::GetTopHeight)
+                    ->Event("SetBottomHeight", &SkyBoxFogRequestBus::Events::SetBottomHeight)
+                    ->Event("GetBottomHeight", &SkyBoxFogRequestBus::Events::GetBottomHeight)
+                    ->VirtualProperty("Enable", "IsEnabled", "SetEnabled")
+                    ->VirtualProperty("Color", "GetColor", "SetColor")
+                    ->VirtualProperty("TopHeight", "GetTopHeight", "SetTopHeight")
+                    ->VirtualProperty("BottomHeight", "GetTopHeight", "SetBottomHeight")
+                ;
+            }
+        }
+
+        bool SkyBoxFogSettings::IsFogDisabled() const
+        {
+            return !m_enable;
+        }
+    }
+}

Gems/Atom/Feature/Common/Code/Source/SkyBox/SkyBoxFogSettings.h

@@ -0,0 +1,38 @@
+/*
+ * All or portions of this file Copyright (c) Amazon.com, Inc. or its affiliates or
+ * its licensors.
+ *
+ * For complete copyright and license terms please see the LICENSE at the root of this
+ * distribution (the "License"). All use of this software is governed by the License,
+ * or, if provided, by the license below or the license accompanying this file. Do not
+ * remove or modify any license notices. This file is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ *
+ */
+
+#pragma once
+
+#include <AzCore/Math/Color.h>
+#include <AzCore/RTTI/RTTI.h>
+
+namespace AZ
+{
+    namespace Render
+    {
+        struct SkyBoxFogSettings final
+        {
+            AZ_RTTI(AZ::Render::SkyBoxFogSettings, "{DB13027C-BA92-4E46-B428-BB77C2A80C51}");
+            
+            static void Reflect(ReflectContext* context);
+
+            SkyBoxFogSettings() = default;
+
+            bool IsFogDisabled() const;
+
+            AZ::Color m_color = AZ::Color::CreateOne();
+            bool m_enable = false;
+            float m_topHeight = 0.01;
+            float m_bottomHeight = 0.0;
+        };
+    }
+}

Gems/Atom/Feature/Common/Code/atom_feature_common_files.cmake

@@ -32,6 +32,7 @@ set(FILES
     Include/Atom/Feature/PostProcessing/SMAAFeatureProcessorInterface.h
     Include/Atom/Feature/PostProcess/PostFxLayerCategoriesConstants.h
     Include/Atom/Feature/ReflectionProbe/ReflectionProbeFeatureProcessor.h
+    Include/Atom/Feature/SkyBox/SkyBoxFogBus.h
     Include/Atom/Feature/SkyBox/SkyboxConstants.h
     Include/Atom/Feature/SkyBox/SkyBoxLUT.h
     Include/Atom/Feature/SphericalHarmonics/SphericalHarmonicsUtility.h
@@ -297,6 +298,8 @@ set(FILES
     Source/SkinnedMesh/SkinnedMeshVertexStreamProperties.h
     Source/SkyBox/SkyBoxFeatureProcessor.cpp
     Source/SkyBox/SkyBoxFeatureProcessor.h
+    Source/SkyBox/SkyBoxFogSettings.h
+    Source/SkyBox/SkyBoxFogSettings.cpp
     Source/TransformService/TransformServiceFeatureProcessor.cpp
     Source/Utils/GpuBufferHandler.cpp
     Source/LuxCore/LuxCoreTexturePass.cpp

Gems/Atom/RHI/Vulkan/Code/Source/RHI/PhysicalDevice.cpp

@@ -62,12 +62,7 @@ namespace AZ
             {
                 RHI::Ptr<PhysicalDevice> physicalDevice = aznew PhysicalDevice;
                 physicalDevice->Init(device);
-                size_t gpuMemSize =  physicalDevice->GetDescriptor().m_heapSizePerLevel[static_cast<size_t>(RHI::HeapMemoryLevel::Device)];
-                AZ_Warning("Vulkan", gpuMemSize >= MinGPUMemSize, "Rejecting GPU %s as it's gpu mem size of %zu bytes is less than min required size of %zu bytes for Vulkan API", physicalDevice->GetDescriptor().m_description.c_str(), gpuMemSize, MinGPUMemSize);
-                if (gpuMemSize >= MinGPUMemSize)
-                {
-                    physicalDeviceList.emplace_back(physicalDevice);
-                }
+                physicalDeviceList.emplace_back(physicalDevice);
             }
 
             return physicalDeviceList;

Gems/Atom/RPI/Code/Source/RPI.Public/Pass/FullscreenTrianglePass.cpp

@@ -181,9 +181,19 @@ namespace AZ
 
             RHI::Size targetImageSize = outputAttachment->m_descriptor.m_image.m_size;
 
-            // Base viewport and scissor off of target attachment
-            m_viewportState = RHI::Viewport(0, static_cast<float>(targetImageSize.m_width), 0, static_cast<float>(targetImageSize.m_height));
-            m_scissorState = RHI::Scissor(0, 0, targetImageSize.m_width, targetImageSize.m_height);
+            m_viewportState = params.m_viewportState;
+            if (m_viewportState.IsNull())
+            {
+                // compute viewport from target attachment
+                m_viewportState = RHI::Viewport(0, static_cast<float>(targetImageSize.m_width), 0, static_cast<float>(targetImageSize.m_height));
+            }
+
+            m_scissorState = params.m_scissorState;
+            if (m_scissorState.IsNull())
+            {
+                // compute scissor from target attachment
+                m_scissorState = RHI::Scissor(0, 0, targetImageSize.m_width, targetImageSize.m_height);
+            }
 
             RenderPass::FrameBeginInternal(params);
         }

Gems/AtomContent/Sponza/.src/objects/Test_Sponza_Materials.mb

@@ -0,0 +1,3 @@
+version https://git-lfs.github.com/spec/v1
+oid sha256:5f4d124e84c8387f06b4b3a77bb3be1e7c3e0dcecb26a8934b89faa8203ed380
+size 84608

Gems/AtomContent/Sponza/Assets/TestData/Test_Sponza_Material_Conversion.fbx

@@ -0,0 +1,3 @@
+version https://git-lfs.github.com/spec/v1
+oid sha256:9b936b72b5b45b52c188bf9f930d6066af65f0d79ff8abf5dc08927d97ac465e
+size 55264

Gems/AtomContent/Sponza/Assets/TestData/Test_Sponza_Material_Conversion_black.material

@@ -0,0 +1,35 @@
+{
+    "description": "",
+    "materialType": "Materials/Types/StandardPBR.materialtype",
+    "parentMaterial": "",
+    "propertyLayoutVersion": 3,
+    "properties": {
+        "baseColor": {
+            "color": [
+                0.0,
+                0.0,
+                0.0,
+                1.0
+            ]
+        },
+        "emissive": {
+            "color": [
+                0.0,
+                0.0,
+                0.0,
+                1.0
+            ]
+        },
+        "irradiance": {
+            "color": [
+                0.0,
+                0.0,
+                0.0,
+                1.0
+            ]
+        },
+        "opacity": {
+            "factor": 1.0
+        }
+    }
+}

Gems/AtomContent/Sponza/Assets/TestData/Test_Sponza_Material_Conversion_green.material

@@ -0,0 +1,35 @@
+{
+    "description": "",
+    "materialType": "Materials/Types/StandardPBR.materialtype",
+    "parentMaterial": "",
+    "propertyLayoutVersion": 3,
+    "properties": {
+        "baseColor": {
+            "color": [
+                0.0,
+                1.0,
+                0.0,
+                1.0
+            ]
+        },
+        "emissive": {
+            "color": [
+                0.0,
+                0.0,
+                0.0,
+                1.0
+            ]
+        },
+        "irradiance": {
+            "color": [
+                0.0,
+                1.0,
+                0.0,
+                1.0
+            ]
+        },
+        "opacity": {
+            "factor": 1.0
+        }
+    }
+}

Gems/AtomContent/Sponza/Assets/TestData/Test_Sponza_Material_Conversion_mat_arch.material

@@ -0,0 +1,49 @@
+{
+    "description": "",
+    "materialType": "Materials/Types/StandardPBR.materialtype",
+    "parentMaterial": "",
+    "propertyLayoutVersion": 3,
+    "properties": {
+        "baseColor": {
+            "color": [
+                0.800000011920929,
+                0.800000011920929,
+                0.800000011920929,
+                1.0
+            ],
+            "textureMap": "Textures/arch_1k_basecolor.png"
+        },
+        "general": {
+            "applySpecularAA": true
+        },
+        "irradiance": {
+            "color": [
+                1.0,
+                0.885053813457489,
+                0.801281750202179,
+                1.0
+            ]
+        },
+        "metallic": {
+            "textureMap": "Textures/arch_1k_metallic.png"
+        },
+        "normal": {
+            "textureMap": "Textures/arch_1k_normal.jpg"
+        },
+        "occlusion": {
+            "diffuseTextureMap": "Textures/arch_1k_ao.png"
+        },
+        "opacity": {
+            "factor": 1.0
+        },
+        "parallax": {
+            "factor": 0.050999999046325687,
+            "pdo": true,
+            "quality": "High",
+            "useTexture": false
+        },
+        "roughness": {
+            "textureMap": "Textures/arch_1k_roughness.png"
+        }
+    }
+}

Gems/AtomContent/Sponza/Assets/TestData/Test_Sponza_Material_Conversion_mat_bricks.material

@@ -0,0 +1,54 @@
+{
+    "description": "",
+    "materialType": "Materials/Types/StandardPBR.materialtype",
+    "parentMaterial": "",
+    "propertyLayoutVersion": 3,
+    "properties": {
+        "baseColor": {
+            "color": [
+                0.800000011920929,
+                0.800000011920929,
+                0.800000011920929,
+                1.0
+            ],
+            "textureMap": "Textures/bricks_1k_basecolor.png"
+        },
+        "clearCoat": {
+            "factor": 0.5,
+            "normalMap": "Textures/bricks_1k_normal.jpg",
+            "roughness": 0.5
+        },
+        "general": {
+            "applySpecularAA": true
+        },
+        "irradiance": {
+            "color": [
+                1.0,
+                0.9703211784362793,
+                0.9703211784362793,
+                1.0
+            ]
+        },
+        "metallic": {
+            "textureMap": "Textures/bricks_1k_metallic.png"
+        },
+        "normal": {
+            "textureMap": "Textures/bricks_1k_normal.jpg"
+        },
+        "occlusion": {
+            "diffuseTextureMap": "Textures/bricks_1k_ao.png"
+        },
+        "opacity": {
+            "factor": 1.0
+        },
+        "parallax": {
+            "algorithm": "ContactRefinement",
+            "factor": 0.03500000014901161,
+            "quality": "Medium",
+            "useTexture": false
+        },
+        "roughness": {
+            "textureMap": "Textures/bricks_1k_roughness.png"
+        }
+    }
+}

Gems/AtomContent/Sponza/Assets/TestData/Test_Sponza_Material_Conversion_mat_floor.material

@@ -0,0 +1,51 @@
+{
+    "description": "",
+    "materialType": "Materials/Types/StandardPBR.materialtype",
+    "parentMaterial": "",
+    "propertyLayoutVersion": 3,
+    "properties": {
+        "baseColor": {
+            "color": [
+                0.800000011920929,
+                0.800000011920929,
+                0.800000011920929,
+                1.0
+            ],
+            "textureMap": "Textures/floor_1k_basecolor.png"
+        },
+        "clearCoat": {
+            "enable": true,
+            "influenceMap": "Textures/floor_1k_ao.png",
+            "normalMap": "Textures/floor_1k_normal.png",
+            "roughness": 0.25
+        },
+        "general": {
+            "applySpecularAA": true
+        },
+        "irradiance": {
+            "color": [
+                1.0,
+                0.9404135346412659,
+                0.8688944578170776,
+                1.0
+            ]
+        },
+        "normal": {
+            "textureMap": "Textures/floor_1k_normal.png"
+        },
+        "occlusion": {
+            "diffuseTextureMap": "Textures/floor_1k_ao.png"
+        },
+        "opacity": {
+            "factor": 1.0
+        },
+        "parallax": {
+            "factor": 0.012000000104308129,
+            "pdo": true,
+            "useTexture": false
+        },
+        "roughness": {
+            "textureMap": "Textures/floor_1k_roughness.png"
+        }
+    }
+}

Gems/AtomContent/Sponza/Assets/TestData/Test_Sponza_Material_Conversion_mat_roof.material

@@ -0,0 +1,44 @@
+{
+    "description": "",
+    "materialType": "Materials/Types/StandardPBR.materialtype",
+    "parentMaterial": "",
+    "propertyLayoutVersion": 3,
+    "properties": {
+        "baseColor": {
+            "color": [
+                0.800000011920929,
+                0.800000011920929,
+                0.800000011920929,
+                1.0
+            ],
+            "textureBlendMode": "Lerp",
+            "textureMap": "Textures/roof_1k_basecolor.png"
+        },
+        "general": {
+            "applySpecularAA": true
+        },
+        "metallic": {
+            "useTexture": false
+        },
+        "normal": {
+            "factor": 0.5,
+            "flipY": true,
+            "textureMap": "Textures/roof_1k_normal.jpg"
+        },
+        "occlusion": {
+            "diffuseTextureMap": "Textures/roof_1k_ao.png"
+        },
+        "opacity": {
+            "factor": 1.0
+        },
+        "parallax": {
+            "algorithm": "ContactRefinement",
+            "factor": 0.019999999552965165,
+            "quality": "Medium",
+            "useTexture": false
+        },
+        "roughness": {
+            "textureMap": "Textures/roof_1k_roughness.png"
+        }
+    }
+}

Gems/AtomContent/Sponza/Assets/TestData/Test_Sponza_Material_Conversion_phong5.material

@@ -0,0 +1,35 @@
+{
+    "description": "",
+    "materialType": "Materials/Types/StandardPBR.materialtype",
+    "parentMaterial": "",
+    "propertyLayoutVersion": 3,
+    "properties": {
+        "baseColor": {
+            "color": [
+                0.0,
+                0.0,
+                1.0,
+                1.0
+            ]
+        },
+        "emissive": {
+            "color": [
+                0.0,
+                0.0,
+                0.0,
+                1.0
+            ]
+        },
+        "irradiance": {
+            "color": [
+                0.0,
+                0.0,
+                1.0,
+                1.0
+            ]
+        },
+        "opacity": {
+            "factor": 1.0
+        }
+    }
+}

Gems/AtomContent/Sponza/Assets/TestData/Test_Sponza_Material_Conversion_red.material

@@ -0,0 +1,35 @@
+{
+    "description": "",
+    "materialType": "Materials/Types/StandardPBR.materialtype",
+    "parentMaterial": "",
+    "propertyLayoutVersion": 3,
+    "properties": {
+        "baseColor": {
+            "color": [
+                0.800000011920929,
+                0.0,
+                0.0,
+                1.0
+            ]
+        },
+        "emissive": {
+            "color": [
+                0.0,
+                0.0,
+                0.0,
+                1.0
+            ]
+        },
+        "irradiance": {
+            "color": [
+                1.0,
+                0.0,
+                0.0,
+                1.0
+            ]
+        },
+        "opacity": {
+            "factor": 1.0
+        }
+    }
+}

Gems/AtomContent/Sponza/Assets/TestData/Test_Sponza_Material_Conversion_white.material

@@ -0,0 +1,19 @@
+{
+    "description": "",
+    "materialType": "Materials/Types/StandardPBR.materialtype",
+    "parentMaterial": "",
+    "propertyLayoutVersion": 3,
+    "properties": {
+        "emissive": {
+            "color": [
+                0.0,
+                0.0,
+                0.0,
+                1.0
+            ]
+        },
+        "opacity": {
+            "factor": 1.0
+        }
+    }
+}

Gems/AtomContent/Sponza/Assets/TestData/white_latlong_iblskyboxcm.exr

@@ -0,0 +1,3 @@
+version https://git-lfs.github.com/spec/v1
+oid sha256:d9c14dc81887be7647d9afa9e5481634eded0b1d0285b59bb76bb2a91326ca8a
+size 3369

Gems/AtomLyIntegration/AtomFont/Code/Source/AtomFont.rc

@@ -81,10 +81,10 @@ BEGIN
     BEGIN
         BLOCK "000904b0"
         BEGIN
-            VALUE "CompanyName", "Amazon.com, Inc."
+            VALUE "CompanyName", "Open 3D Foundation"
             VALUE "FileVersion", "1, 0, 0, 1"
             VALUE "LegalCopyright", "Portions of this file Copyright (c) Amazon.com, Inc. or its affiliates. All Rights Reserved. Original file Copyright (c) Crytek GMBH. Used under license by Amazon.com, Inc. and its affiliates."
-            VALUE "ProductName", "Lumberyard"
+            VALUE "ProductName", "O3DE"
             VALUE "ProductVersion", "1, 0, 0, 1"
         END
     END

Gems/AtomLyIntegration/CommonFeatures/Code/Include/AtomLyIntegration/CommonFeatures/SkyBox/PhysicalSkyComponentConfig.h

@@ -15,6 +15,7 @@
 #include <AzCore/Component/Component.h>
 #include <Atom/Feature/CoreLights/PhotometricValue.h>
 #include <Atom/Feature/SkyBox/SkyboxConstants.h>
+#include <SkyBox/SkyBoxFogSettings.h>
 
 namespace AZ
 {
@@ -35,6 +36,8 @@ namespace AZ
             int m_turbidity = 1;
             float m_sunRadiusFactor = 1.0f;
 
+            SkyBoxFogSettings m_skyBoxFogSettings;
+
             //! Returns characters for a suffix for the light type including a space. " lm" for lumens for example.
             const char* GetIntensitySuffix() const;
 
@@ -45,6 +48,8 @@ namespace AZ
             //! Returns the maximum intensity value allowed depending on the m_intensityMode
             float GetSkyIntensityMax() const;
             float GetSunIntensityMax() const;
+
+            bool IsFogDisabled() const { return !m_skyBoxFogSettings.m_enable; }
         };
     }
 }

Gems/AtomLyIntegration/CommonFeatures/Code/Source/PostProcess/PostFxLayerComponentController.cpp

@@ -79,7 +79,11 @@ namespace AZ
             // Add the current view which can potentially be the editor view
             auto atomViewportRequests = AZ::Interface<AZ::RPI::ViewportContextRequestsInterface>::Get();
             const AZ::Name contextName = atomViewportRequests->GetDefaultViewportContextName();
-            allSceneViews.insert(atomViewportRequests->GetCurrentView(contextName).get());
+            auto currentView = atomViewportRequests->GetCurrentView(contextName);
+            if (IsEditorView(currentView))
+            {
+                allSceneViews.insert(currentView.get());
+            }
 
             // calculate blend weights for all cameras
             PostProcessSettingsInterface::ViewBlendWeightMap perViewBlendWeights;
@@ -146,6 +150,13 @@ namespace AZ
             {
                 m_cameraEntities.insert(cameraId);
             }
+
+            AZ::RPI::ViewPtr view = nullptr;
+            AZ::RPI::ViewProviderBus::EventResult(view, cameraId, &AZ::RPI::ViewProvider::GetView);
+            if (view != nullptr)
+            {
+                m_allCameraViews.insert(view.get());
+            }
         }
 
         void PostFxLayerComponentController::OnCameraRemoved(const AZ::EntityId& cameraId)
@@ -176,6 +187,11 @@ namespace AZ
             }
         }
 
+        bool PostFxLayerComponentController::IsEditorView(const AZ::RPI::ViewPtr view)
+        {
+            return m_allCameraViews.find(view.get()) == m_allCameraViews.end() ? true : false;
+        }
+
         bool PostFxLayerComponentController::HasTags(const AZ::EntityId& entityId, const AZStd::vector<AZStd::string>& tags) const
         {
             bool hasTag = false;

Gems/AtomLyIntegration/CommonFeatures/Code/Source/PostProcess/PostFxLayerComponentController.h

@@ -73,13 +73,15 @@ namespace AZ
             void BusConnectToTags();
 
             const AZStd::unordered_set<AZ::EntityId>& GetCameraEntityList() const;
-
+            bool IsEditorView(const AZ::RPI::ViewPtr view);
             bool HasTags(const AZ::EntityId& entityId, const AZStd::vector<AZStd::string>& tags) const;
 
             // list of entities containing tags set in this component's property.
             AZStd::unordered_set<AZ::EntityId> m_taggedCameraEntities;
             // a list of cameras tracked by this component. This is used if no camera tags are specified.
             AZStd::unordered_set<AZ::EntityId> m_cameraEntities;
+            // a list of camera views in the scene. This is used to test if a view is an editor view.
+            AZStd::unordered_set<AZ::RPI::View*> m_allCameraViews;
 
             PostProcessFeatureProcessorInterface* m_featureProcessorInterface = nullptr;
             PostProcessSettingsInterface* m_postProcessInterface = nullptr;

Gems/AtomLyIntegration/CommonFeatures/Code/Source/SkyBox/EditorPhysicalSkyComponent.cpp

@@ -68,13 +68,17 @@ namespace AZ
                                 ->Attribute(AZ::Edit::Attributes::Min, 1)
                                 ->Attribute(AZ::Edit::Attributes::Max, 10)
                                 ->Attribute(AZ::Edit::Attributes::Step, 1)
+                            ->DataElement(AZ::Edit::UIHandlers::Default, &PhysicalSkyComponentConfig::m_skyBoxFogSettings, "Fog", "Fog settings for rendering on top of physical sky")
+                                ->Attribute(AZ::Edit::Attributes::AutoExpand, true)
                         ;
                 }
             }
 
             if (auto behaviorContext = azrtti_cast<BehaviorContext*>(context))
             {
-                behaviorContext->Class<EditorPhysicalSkyComponent>()->RequestBus("PhysicalSkyRequestBus");
+                behaviorContext->Class<EditorPhysicalSkyComponent>()
+                    ->RequestBus("PhysicalSkyRequestBus")
+                    ->RequestBus("SkyBoxFogRequestBus");
 
                 behaviorContext->ConstantProperty("EditorPhysicalSkyComponentTypeId", BehaviorConstant(Uuid(EditorPhysicalSkyComponentTypeId)))
                     ->Attribute(AZ::Script::Attributes::Module, "render")

Gems/AtomLyIntegration/CommonFeatures/Code/Source/SkyBox/HDRiSkyboxComponentController.cpp

@@ -69,7 +69,7 @@ namespace AZ
             m_featureProcessorInterface = RPI::Scene::GetFeatureProcessorForEntity<SkyBoxFeatureProcessorInterface>(entityId);
 
             // only activate if there is no other skybox activate
-            if (!m_featureProcessorInterface->IsEnable())
+            if (!m_featureProcessorInterface->IsEnabled())
             {
                 m_featureProcessorInterface->SetSkyboxMode(SkyBoxMode::Cubemap);
                 m_featureProcessorInterface->Enable(true);

Gems/AtomLyIntegration/CommonFeatures/Code/Source/SkyBox/PhysicalSkyComponent.cpp

@@ -34,7 +34,9 @@ namespace AZ
 
             if (auto behaviorContext = azrtti_cast<BehaviorContext*>(context))
             {
-                behaviorContext->Class<PhysicalSkyComponent>()->RequestBus("PhysicalSkyRequestBus");
+                behaviorContext->Class<PhysicalSkyComponent>()
+                    ->RequestBus("PhysicalSkyRequestBus")
+                    ->RequestBus("SkyBoxFogRequestBus");
 
                 behaviorContext->ConstantProperty("PhysicalSkyComponentTypeId", BehaviorConstant(Uuid(PhysicalSkyComponentTypeId)))
                     ->Attribute(AZ::Script::Attributes::Module, "render")

Gems/AtomLyIntegration/CommonFeatures/Code/Source/SkyBox/PhysicalSkyComponentConfig.cpp

@@ -29,6 +29,7 @@ namespace AZ
                     ->Field("SunIntensity", &PhysicalSkyComponentConfig::m_sunIntensity)
                     ->Field("Turbidity", &PhysicalSkyComponentConfig::m_turbidity)
                     ->Field("SunRadiusFactor", &PhysicalSkyComponentConfig::m_sunRadiusFactor)
+                    ->Field("FogSettings", &PhysicalSkyComponentConfig::m_skyBoxFogSettings)
                     ;
             }
         }

Gems/AtomLyIntegration/CommonFeatures/Code/Source/SkyBox/PhysicalSkyComponentController.cpp

@@ -12,8 +12,6 @@
 
 #include <SkyBox/PhysicalSkyComponentController.h>
 #include <AzCore/RTTI/BehaviorContext.h>
-
-
 #include <Atom/RPI.Public/Scene.h>
 
 namespace AZ
@@ -34,6 +32,9 @@ namespace AZ
             if (AZ::BehaviorContext* behaviorContext = azrtti_cast<AZ::BehaviorContext*>(context))
             {
                 behaviorContext->EBus<PhysicalSkyRequestBus>("PhysicalSkyRequestBus")
+                    ->Attribute(AZ::Script::Attributes::Scope, AZ::Script::Attributes::ScopeFlags::Common)
+                    ->Attribute(AZ::Script::Attributes::Category, "render")
+                    ->Attribute(AZ::Script::Attributes::Module, "render")
                     ->Event("SetTurbidity", &PhysicalSkyRequestBus::Events::SetTurbidity)
                     ->Event("GetTurbidity", &PhysicalSkyRequestBus::Events::GetTurbidity)
                     ->Event("SetSunRadiusFactor", &PhysicalSkyRequestBus::Events::SetSunRadiusFactor)
@@ -76,7 +77,7 @@ namespace AZ
             m_featureProcessorInterface = RPI::Scene::GetFeatureProcessorForEntity<SkyBoxFeatureProcessorInterface>(entityId);
 
             // only activate if there is no other skybox activate
-            if (!m_featureProcessorInterface->IsEnable())
+            if (!m_featureProcessorInterface->IsEnabled())
             {
                 m_featureProcessorInterface->SetSkyboxMode(SkyBoxMode::PhysicalSky);
                 m_featureProcessorInterface->Enable(true);
@@ -95,8 +96,10 @@ namespace AZ
 
                 const AZ::Transform& transform = m_transformInterface ? m_transformInterface->GetWorldTM() : Transform::Identity();
                 m_featureProcessorInterface->SetSunPosition(GetSunTransform(transform));
+                m_featureProcessorInterface->SetFogSettings(m_configuration.m_skyBoxFogSettings);
 
                 PhysicalSkyRequestBus::Handler::BusConnect(m_entityId);
+                SkyBoxFogRequestBus::Handler::BusConnect(m_entityId);
                 TransformNotificationBus::Handler::BusConnect(m_entityId);
 
                 m_isActive = true;
@@ -113,8 +116,9 @@ namespace AZ
             // Run deactivate if this skybox is activate
             if (m_isActive)
             {
-                PhysicalSkyRequestBus::Handler::BusDisconnect(m_entityId);
-                TransformNotificationBus::Handler::BusDisconnect(m_entityId);
+                PhysicalSkyRequestBus::Handler::BusDisconnect();
+                SkyBoxFogRequestBus::Handler::BusDisconnect();
+                TransformNotificationBus::Handler::BusDisconnect();
 
                 m_featureProcessorInterface->Enable(false);
                 m_featureProcessorInterface = nullptr;
@@ -229,5 +233,49 @@ namespace AZ
 
             return SunPosition(atan2(sunPosition.GetZ(), sunPosition.GetX()), asin(sunPosition.GetY()));
         }
+
+        void PhysicalSkyComponentController::SetEnabled(bool enable)
+        {
+            m_configuration.m_skyBoxFogSettings.m_enable = enable;
+            m_featureProcessorInterface->SetFogEnabled(enable);
+        }
+
+        bool PhysicalSkyComponentController::IsEnabled() const
+        {
+            return m_configuration.m_skyBoxFogSettings.m_enable;
+        }
+
+        void PhysicalSkyComponentController::SetColor(const AZ::Color& color)
+        {
+            m_configuration.m_skyBoxFogSettings.m_color = color;
+            m_featureProcessorInterface->SetFogColor(color);
+        }
+
+        const AZ::Color& PhysicalSkyComponentController::GetColor() const
+        {
+            return m_configuration.m_skyBoxFogSettings.m_color;
+        }
+
+        void PhysicalSkyComponentController::SetTopHeight(float topHeight)
+        {
+            m_configuration.m_skyBoxFogSettings.m_topHeight = topHeight;
+            m_featureProcessorInterface->SetFogTopHeight(topHeight);
+        }
+
+        float PhysicalSkyComponentController::GetTopHeight() const
+        {
+            return m_configuration.m_skyBoxFogSettings.m_topHeight;
+        }
+
+        void PhysicalSkyComponentController::SetBottomHeight(float bottomHeight)
+        {
+            m_configuration.m_skyBoxFogSettings.m_bottomHeight = bottomHeight;
+            m_featureProcessorInterface->SetFogBottomHeight(bottomHeight);
+        }
+
+        float PhysicalSkyComponentController::GetBottomHeight() const
+        {
+            return m_configuration.m_skyBoxFogSettings.m_bottomHeight;
+        }
     }
 }

Gems/AtomLyIntegration/CommonFeatures/Code/Source/SkyBox/PhysicalSkyComponentController.h

@@ -16,6 +16,7 @@
 #include <AzCore/Component/TransformBus.h>
 #include <AtomLyIntegration/CommonFeatures/SkyBox/PhysicalSkyComponentConfig.h>
 #include <AtomLyIntegration/CommonFeatures/SkyBox/PhysicalSkyBus.h>
+#include <Atom/Feature/SkyBox/SkyBoxFogBus.h>
 #include <Atom/Feature/SkyBox/SkyBoxFeatureProcessorInterface.h>
 
 namespace AZ
@@ -25,6 +26,7 @@ namespace AZ
         class PhysicalSkyComponentController final
             : public TransformNotificationBus::Handler
             , public PhysicalSkyRequestBus::Handler
+            , public SkyBoxFogRequestBus::Handler
         {
         public:
             friend class EditorPhysicalSkyComponent;
@@ -63,7 +65,17 @@ namespace AZ
             float GetSunIntensity(PhotometricUnit unit) override;
             float GetSunIntensity() override;
 
-            //! Get Sun azimuth and altitude from entity transfom, without scale
+            // SkyBoxFogRequestBus::Handler overrides ...
+            void SetEnabled(bool enable) override;
+            bool IsEnabled() const override;
+            void SetColor(const AZ::Color& color) override;
+            const AZ::Color& GetColor() const override;
+            void SetTopHeight(float topHeight) override;
+            float GetTopHeight() const override;
+            void SetBottomHeight(float bottomHeight) override;
+            float GetBottomHeight() const override;
+
+            //! Get Sun azimuth and altitude from entity transform, without scale
             SunPosition GetSunTransform(const AZ::Transform& world);
 
             TransformInterface* m_transformInterface = nullptr;

Gems/EMotionFX/Code/EMotionFX/CommandSystem/Source/ColliderCommands.cpp

@@ -10,6 +10,7 @@
 *
 */
 
+#include <cinttypes>
 #include <AzCore/std/smart_ptr/make_shared.h>
 #include <MCore/Source/ReflectionSerializer.h>
 #include <EMotionFX/CommandSystem/Source/CommandManager.h>
@@ -717,7 +718,7 @@ namespace EMotionFX
         const size_t shapeCount = nodeConfig->m_shapes.size();
         if (m_colliderIndex >= shapeCount)
         {
-            outResult = AZStd::string::format("Cannot remove collider. The joint '%s' is only holding %zu colliders and the index %zu is out of range.", m_jointName.c_str(), shapeCount, m_colliderIndex);
+            outResult = AZStd::string::format("Cannot remove collider. The joint '%s' is only holding %zu colliders and the index %llu is out of range.", m_jointName.c_str(), shapeCount, m_colliderIndex);
             return false;
         }