Return status code 1 from sync command on warnings/errors

b2/_internal/b2v3/registry.py

@@ -13,6 +13,7 @@
 from b2._internal._cli.b2api import _get_b2api_for_profile
 from b2._internal.arg_parser import enable_camel_case_arguments
 from .rm import Rm
+from .sync import Sync
 
 enable_camel_case_arguments()
 

b2/_internal/b2v3/sync.py

@@ -0,0 +1,16 @@
+######################################################################
+#
+# File: b2/_internal/b2v3/sync.py
+#
+# Copyright 2024 Backblaze Inc. All Rights Reserved.
+#
+# License https://www.backblaze.com/using_b2_code.html
+#
+######################################################################
+
+from b2._internal.b2v4.registry import Sync as SyncV4
+
+
+class Sync(SyncV4):
+    __doc__ = SyncV4.__doc__
+    FAIL_ON_REPORTER_ERRORS_OR_WARNINGS = False

b2/_internal/console_tool.py

@@ -2972,6 +2972,8 @@ class Sync(
     DEFAULT_DOWNLOAD_THREADS = 10
     DEFAULT_UPLOAD_THREADS = 10
 
+    FAIL_ON_REPORTER_ERRORS_OR_WARNINGS = True
+
     @classmethod
     def _setup_parser(cls, parser):
         add_normalized_argument(parser, '--no-progress', action='store_true')
@@ -3089,6 +3091,8 @@ def _run(self, args):
                 raise CommandError(f'{ex.path} is not a directory')
             except UnableToCreateDirectory as ex:
                 raise CommandError(f'unable to create directory {ex.path}')
+            if self.FAIL_ON_REPORTER_ERRORS_OR_WARNINGS and reporter.has_errors_or_warnings():
+                return 1
         return 0
 
     def get_policies_manager_from_args(self, args):

changelog.d/+sync-status-code.changed.md

@@ -0,0 +1 @@
+Changed `sync` command exit status code from 0 to 1 if any warnings or errors were encountered during the operation.

pyproject.toml

@@ -25,7 +25,7 @@ classifiers = [
 dependencies = [
     "argcomplete>=2,<4",
     "arrow>=1.0.2,<2.0.0",
-    "b2sdk>=2.1.0,<3",
+    "b2sdk>=2.2.0,<3",
     "docutils>=0.18.1",
     "idna~=3.4; platform_system == 'Java'",
     "importlib-metadata>=3.3; python_version < '3.8'",