Undocument --bes_best_effort

We've heared from several people that they tried passing the flag and it did
not have any efect (as expected from a no-op), so let's undocument the flag
so it's no longer discoverable.

On a related note, we should document `--bes_upload_mode` which seems to be
what people want to do.

Closes #14333.

PiperOrigin-RevId: 435317406

src/main/java/com/google/devtools/build/lib/bazel/rules/BazelRulesModule.java

@@ -505,6 +505,17 @@ public static final class AllCommandGraveyardOptions extends OptionsBase {
         metadataTags = {OptionMetadataTag.INCOMPATIBLE_CHANGE},
         help = "No-op.")
     public boolean incompatibleApplicableLicenses;
+
+    @Option(
+        name = "bes_best_effort",
+        defaultValue = "false",
+        deprecationWarning =
+            "BES best effort upload has been removed. The flag has no more "
+                + "functionality attached to it and will be removed in a future release.",
+        documentationCategory = OptionDocumentationCategory.UNDOCUMENTED,
+        effectTags = {OptionEffectTag.AFFECTS_OUTPUTS},
+        help = "No-op")
+    public boolean besBestEffort;
   }
 
   @Override

src/main/java/com/google/devtools/build/lib/buildeventservice/BuildEventServiceOptions.java

@@ -65,19 +65,6 @@ public class BuildEventServiceOptions extends OptionsBase {
       allowMultiple = true)
   public List<Map.Entry<String, String>> besHeaders;
 
-  @Option(
-      name = "bes_best_effort",
-      defaultValue = "false",
-      deprecationWarning =
-          "BES best effort upload has been removed. The flag has no more "
-              + "functionality attached to it and will be removed in a future release.",
-      documentationCategory = OptionDocumentationCategory.LOGGING,
-      effectTags = {OptionEffectTag.AFFECTS_OUTPUTS},
-      help =
-          "BES best effort upload has been removed. The flag has no more "
-              + "functionality attached to it and will be removed in a future release.")
-  public boolean besBestEffort;
-
   @Option(
     name = "bes_lifecycle_events",
     defaultValue = "true",