refactor: remove duplicate code and fix indentation

* Set the right indentation across `package`, `deploy`,
  `package/deploy --resolve-s3` & `deploy --guided`
* Set the right indentation on `Skipping upload` to S3.

Author: sriram-mv

samcli/commands/deploy/command.py

@@ -36,7 +36,7 @@
 )
 from samcli.commands.deploy.core.command import DeployCommand
 from samcli.commands.deploy.utils import sanitize_parameter_overrides
-from samcli.lib.bootstrap.bootstrap import manage_stack
+from samcli.lib.bootstrap.bootstrap import manage_stack, print_managed_s3_bucket_info
 from samcli.lib.bootstrap.companion_stack.companion_stack_manager import sync_ecr_stack
 from samcli.lib.cli_validation.image_repository_validation import image_repository_validation
 from samcli.lib.telemetry.metric import track_command
@@ -307,9 +307,7 @@ def do_cli(
             if bool(s3_bucket):
                 raise DeployResolveS3AndS3SetError()
             s3_bucket = manage_stack(profile=profile, region=region)
-            click.echo(f"\n\t\tManaged S3 bucket: {s3_bucket}")
-            click.echo("\t\tA different default S3 bucket can be set in samconfig.toml")
-            click.echo("\t\tOr by specifying --s3-bucket explicitly.")
+            print_managed_s3_bucket_info(s3_bucket)
 
         # TODO Refactor resolve-s3 and resolve-image-repos into one place
         # after we figure out how to enable resolve-images-repos in package

samcli/commands/deploy/guided_context.py

@@ -23,7 +23,7 @@
 from samcli.commands.deploy.exceptions import GuidedDeployFailedError
 from samcli.commands.deploy.guided_config import GuidedConfig
 from samcli.commands.deploy.utils import sanitize_parameter_overrides
-from samcli.lib.bootstrap.bootstrap import manage_stack
+from samcli.lib.bootstrap.bootstrap import manage_stack, print_managed_s3_bucket_info
 from samcli.lib.bootstrap.companion_stack.companion_stack_manager import CompanionStackManager, sync_ecr_stack
 from samcli.lib.config.samconfig import DEFAULT_CONFIG_FILE_NAME, DEFAULT_ENV
 from samcli.lib.intrinsic_resolver.intrinsics_symbol_table import IntrinsicsSymbolTable
@@ -182,11 +182,8 @@ def guided_prompts(self, parameter_override_keys):
 
         click.echo("\n\tLooking for resources needed for deployment:")
         managed_s3_bucket = manage_stack(profile=self.profile, region=region)
-        click.secho(f"\n\tManaged S3 bucket: {managed_s3_bucket}", bold=True)
-        click.echo(
-            "\tA different default S3 bucket can be set in samconfig.toml"
-            " and auto resolution of buckets turned off by setting resolve_s3=False"
-        )
+        print_managed_s3_bucket_info(managed_s3_bucket, bold=True, show_s3_bucket_option=False)
+        click.echo("\tAuto resolution of buckets turned off by setting resolve_s3=False")
 
         image_repositories = (
             sync_ecr_stack(

samcli/commands/package/command.py

@@ -23,7 +23,7 @@
     use_json_option,
 )
 from samcli.commands.package.core.command import PackageCommand
-from samcli.lib.bootstrap.bootstrap import manage_stack
+from samcli.lib.bootstrap.bootstrap import manage_stack, print_managed_s3_bucket_info
 from samcli.lib.cli_validation.image_repository_validation import image_repository_validation
 from samcli.lib.telemetry.metric import track_command, track_template_warnings
 from samcli.lib.utils.resources import resources_generator
@@ -168,9 +168,7 @@ def do_cli(
 
     if resolve_s3:
         s3_bucket = manage_stack(profile=profile, region=region)
-        click.echo(f"\n\t\tManaged S3 bucket: {s3_bucket}")
-        click.echo("\t\tA different default S3 bucket can be set in samconfig.toml")
-        click.echo("\t\tOr by specifying --s3-bucket explicitly.")
+        print_managed_s3_bucket_info(s3_bucket)
 
     with PackageContext(
         template_file=template_file,

samcli/lib/bootstrap/bootstrap.py

@@ -7,6 +7,7 @@
 from typing import Optional
 
 import boto3
+import click
 from botocore.exceptions import ClientError
 
 from samcli import __version__
@@ -34,6 +35,25 @@ def manage_stack(profile, region):
     # This bucket name is what we would write to a config file
     return bucket_name
 
+def print_managed_s3_bucket_info(s3_bucket, bold=False, show_s3_bucket_option=True):
+    """
+    Print information about the managed S3 bucket.
+
+    Parameters
+    ----------
+    s3_bucket : str
+        The name of the managed S3 bucket
+    bold : bool, optional
+        Whether to print the message in bold, by default False
+    show_s3_bucket_option : bool, optional
+        Whether to show the s3-bucket option message, by default True
+    """
+    message = f"\n\tManaged S3 bucket: {s3_bucket}"
+    click.secho(message, bold=bold)
+    click.echo("\tA different default S3 bucket can be set in samconfig.toml")
+    if show_s3_bucket_option:
+        click.echo("\tOr by specifying --s3-bucket explicitly.")
+
 
 def get_current_account_id(profile: Optional[str] = None):
     """Returns account ID based on used AWS credentials."""

samcli/lib/package/s3_uploader.py

@@ -85,7 +85,7 @@ def upload(self, file_name: str, remote_path: str) -> str:
 
         # Check if a file with same data exists
         if not self.force_upload and self.file_exists(remote_path):
-            LOG.info("File with same data already exists at %s, skipping upload", remote_path)
+            LOG.info("\n\tFile with same data already exists at %s, skipping upload", remote_path)
             return self.make_url(remote_path)
 
         try: