commit 9e7b29e1352dcdb45bf135a29ac5b16eb018a6a7
author Alex Klein <saklein@chromium.org> Tue Apr 11 16:10:31 2023 -0600
committer Chromeos LUCI <chromeos-scoped@luci-project-accounts.iam.gserviceaccount.com> Wed Apr 12 03:45:03 2023 +0000
tree 75a2ac12ad0cf7adf1bdf18dd1b75eac5b1f2a5b
parent 9c4be3d78658a4d7b05f1b7acfe80bceeb717934

scripts: Lint fixes.

line-too-long and docstring-section-indent fixes. Misc spelling
and grammar fixes.

BUG=None
TEST=CQ

Change-Id: I430f8c770fa60be0136220b7658b2542d68908f2
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/chromite/+/4416214
Tested-by: Alex Klein <saklein@chromium.org>
Commit-Queue: Jack Rosenthal <jrosenth@chromium.org>
Auto-Submit: Alex Klein <saklein@chromium.org>
Reviewed-by: Jack Rosenthal <jrosenth@chromium.org>

scripts/cbuildbot.py

diff --git a/scripts/cbuildbot.py b/scripts/cbuildbot.py
index ddd7f52..2a2e3b6 100644
--- a/scripts/cbuildbot.py
+++ b/scripts/cbuildbot.py
@@ -873,9 +873,9 @@
 
     patches = bool(options.gerrit_patches)
 
-    # When running in release mode, make sure we are running with checked-in code.
-    # We want checked-in cbuildbot/scripts to prevent errors, and we want to build
-    # a release image with checked-in code for CrOS packages.
+    # When running in release mode, make sure we are running with checked-in
+    # code. We want checked-in cbuildbot/scripts to prevent errors, and we want
+    # to build a release image with checked-in code for CrOS packages.
     if options.buildbot and patches and not options.debug:
         cros_build_lib.Die(
             "Cannot provide patches when running with --buildbot!"
@@ -1051,8 +1051,8 @@
     # 1) Based on options and build_config, decide whether we are a production
     # run, debug run, or standalone run.
     # 2) Set up cidb instance accordingly.
-    # 3) Update topology info from cidb, so that any other service set up can use
-    # topology.
+    # 3) Update topology info from cidb, so that any other service set up can
+    # use topology.
     # 4) Set up any other services.
     run_type = _GetRunEnvironment(options, build_config)
 
@@ -1203,10 +1203,11 @@
         # Preserve chromite; we might be running from there!
         options.preserve_paths = set(["chromite"])
         if log_file is not None:
-            # We don't want the critical section to try to clean up the tee process,
-            # so we run Tee (forked off) outside of it. This prevents a deadlock
-            # because the Tee process only exits when its pipe is closed, and the
-            # critical section accidentally holds on to that file handle.
+            # We don't want the critical section to try to clean up the tee
+            # process, so we run Tee (forked off) outside of it. This prevents a
+            # deadlock because the Tee process only exits when its pipe is
+            # closed, and the critical section accidentally holds on to that
+            # file handle.
             stack.Add(tee.Tee, log_file)
             options.preserve_paths.add(_DEFAULT_LOG_DIR)
 
@@ -1261,5 +1262,6 @@
         try:
             _RunBuildStagesWrapper(options, site_config, build_config)
         except failures_lib.ExitEarlyException as ex:
-            # This build finished successfully. Do not re-raise ExitEarlyException.
+            # This build finished successfully. Do not re-raise
+            # ExitEarlyException.
             logging.info("One stage exited early: %s", ex)