changes after code library build.

Author: meyertst-aws

python/example_code/ecr/ecr_getting_started.py

@@ -132,7 +132,6 @@ def run(self, role_arn: str) -> None:
         self.docker_image = self.docker_client.images.build(
             path="docker_files", tag=self.full_tag_name
         )[0]
-        self.docker_image.tag(repository=repository_uri, tag=self.tag)
         print(f"Docker image {self.full_tag_name} successfully built.")
         press_enter_to_continue()
         print_dashes()
@@ -179,7 +178,7 @@ def run(self, role_arn: str) -> None:
 * Retrieve an ECR authorization token.
 
 You need an authorization token to securely access and interact with the Amazon ECR registry.
-The `getAuthorizationToken` method of the `EcrAsyncClient` is responsible for securely accessing
+The `get_authorization_token` method of the `ecr` client is responsible for securely accessing
 and interacting with an Amazon ECR repository. This operation is responsible for obtaining a
 valid authorization token, which is required to authenticate your requests to the ECR service.
 
@@ -233,9 +232,8 @@ def run(self, role_arn: str) -> None:
             """
 * Push a docker image to the Amazon ECR Repository.
 
-The method uses the authorization token to create an `AuthConfig` object, which is used to authenticate
-the Docker client when pushing the image. Finally, the method tags the Docker image with the specified
-repository name and image image_tag, and then pushes the image to the ECR repository using the Docker client.
+The Docker client uses the authorization token is used to authenticate the when pushing the image to the 
+ECR repository.
         """
         )
         decoded_authorization = base64.b64decode(authorization_token).decode("utf-8")

python/example_code/ecr/ecr_wrapper.py

@@ -74,9 +74,9 @@ def delete_repository(self, repository_name: str):
             self.ecr_client.delete_repository(
                 repositoryName=repository_name, force=True
             )
-            logger.info("Deleted repository %s.", repository_name)
+            print(f"Deleted repository {repository_name}.")
         except ClientError as err:
-            logger.exception(
+            logger.error(
                 "Couldn't delete repository %s.. Here's why %s",
                 repository_name,
                 err.response["Error"]["Message"],
@@ -97,13 +97,13 @@ def set_repository_policy(self, repository_name: str, policy_text: str):
             self.ecr_client.set_repository_policy(
                 repositoryName=repository_name, policyText=policy_text
             )
-            logger.info("Set repository policy for repository %s.", repository_name)
+            print(f"Set repository policy for repository {repository_name}.")
         except ClientError as err:
             if err.response["Error"]["Code"] == "RepositoryPolicyNotFoundException":
-                logger.info("Repository does not exist. %s.", repository_name)
+                logger.error("Repository does not exist. %s.", repository_name)
                 raise
             else:
-                logger.exception(
+                logger.error(
                     "Couldn't set repository policy for repository %s. Here's why %s",
                     repository_name,
                     err.response["Error"]["Message"],
@@ -127,10 +127,10 @@ def get_repository_policy(self, repository_name: str) -> str:
             return response["policyText"]
         except ClientError as err:
             if err.response["Error"]["Code"] == "RepositoryPolicyNotFoundException":
-                logger.info("Repository does not exist. %s.", repository_name)
+                logger.error("Repository does not exist. %s.", repository_name)
                 raise
             else:
-                logger.exception(
+                logger.error(
                     "Couldn't get repository policy for repository %s. Here's why %s",
                     repository_name,
                     err.response["Error"]["Message"],
@@ -150,7 +150,7 @@ def get_authorization_token(self) -> str:
             response = self.ecr_client.get_authorization_token()
             return response["authorizationData"][0]["authorizationToken"]
         except ClientError as err:
-            logger.exception(
+            logger.error(
                 "Couldn't get authorization token. Here's why %s",
                 err.response["Error"]["Message"],
             )
@@ -172,7 +172,7 @@ def describe_repositories(self, repository_names: list[str]) -> list[dict]:
             )
             return response["repositories"]
         except ClientError as err:
-            logger.exception(
+            logger.error(
                 "Couldn't describe repositories. Here's why %s",
                 err.response["Error"]["Message"],
             )
@@ -193,9 +193,9 @@ def put_lifecycle_policy(self, repository_name: str, lifecycle_policy_text: str)
                 repositoryName=repository_name,
                 lifecyclePolicyText=lifecycle_policy_text,
             )
-            logger.info("Put lifecycle policy for repository %s.", repository_name)
+            print(f"Put lifecycle policy for repository {repository_name}.")
         except ClientError as err:
-            logger.exception(
+            logger.error(
                 "Couldn't put lifecycle policy for repository %s. Here's why %s",
                 repository_name,
                 err.response["Error"]["Message"],
@@ -228,7 +228,7 @@ def describe_images(
                 image_descriptions.extend(page["imageDetails"])
             return image_descriptions
         except ClientError as err:
-            logger.exception(
+            logger.error(
                 "Couldn't describe images. Here's why %s",
                 err.response["Error"]["Message"],
             )

python/example_code/ecr/test/conftest.py

@@ -30,10 +30,6 @@ def __init__(self):
         self.image_tag = (None,)
         self.path = None
 
-    def tag(self, repository: str, tag: str) -> None:
-        assert repository == self.repository, "MockDockerImage.tag"
-        assert tag == self.image_tag, "MockDockerImage.tag"
-
     def build(self, path: str, tag: str) -> tuple["MockDockerImage", None]:
         assert path == self.path, "MockDockerImage.build"
         assert tag == f"{self.repository}:{self.image_tag}", "MockDockerImage.build"