cleanup, disable skip in local_toolchains integration test

Author: rickeylev

tests/integration/local_toolchains/BUILD.bazel

@@ -23,11 +23,11 @@ py_test(
     # NOTE: This isn't strictly correct because is_toolchain_py39plus
     # ends up using the config state coming into the target, not from within
     # the target. There's no transitions going on, though, so it works OK.
-    target_compatible_with = select({
-        # Our RBE's system python is 3.6, which is too old.
-        ":is_toolchain_py39plus": [],
-        "//conditions:default": ["@platforms//:incompatible"],
-    }),
+    ##target_compatible_with = select({
+    ##    # Our RBE's system python is 3.6, which is too old.
+    ##    ":is_toolchain_py39plus": [],
+    ##    "//conditions:default": ["@platforms//:incompatible"],
+    ##}),
 )
 
 config_setting(

tests/runtime_env_toolchain/BUILD.bazel

@@ -30,19 +30,8 @@ py_reconfig_test(
         CC_TOOLCHAIN,
     ],
     main = "toolchain_runs_test.py",
-    # Doesn't work -- doesn't have ConstraintValueInfo provider?
-    #target_compatible_with = [
-    #    "@platforms//host",
-    #],
-    # Doesn't work -- not a valid select condition?
-    #target_compatible_with = select({
-    #    "@platforms//host": [],
-    #    "//conditions:default": ["@platforms//:incompatible"],
-    #}),
-    ##target_compatible_with = select({
-    ##    ":is_host": [],
-    ##    "//conditions:default": ["@platforms//:incompatible"],
-    ##}),
+    # Our RBE has Python 3.6, which is too old for the language features
+    # we use now.
     tags = ["no-remote-exec"],
     deps = ["//python/runfiles"],
 )