Skip to content

Commit 9a2fbe2

Browse files
agluszaksmukherj1
andauthored
Rename exec_properties to internal_exec_properties in _rbe_config (#912)
* Rename exec_properties to _exec_properties to make rbe_config compatible with --experimental_repo_remote_exec * Rename exec_properties to internal_exec_properties in _rbe_autoconfig Co-authored-by: Suvanjan Mukherjee <[email protected]>
1 parent 923ee94 commit 9a2fbe2

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

rules/rbe_repo.bzl

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -587,7 +587,7 @@ def _rbe_autoconfig_impl(ctx):
587587
ctx.report_progress("creating export platform")
588588
create_export_platform(
589589
ctx,
590-
exec_properties = ctx.attr.exec_properties,
590+
exec_properties = ctx.attr.internal_exec_properties,
591591
exec_compatible_with = exec_compatible_with,
592592
target_compatible_with = target_compatible_with,
593593
image_name = resolve_rbe_original_image_name(ctx, image_name),
@@ -617,7 +617,7 @@ def _rbe_autoconfig_impl(ctx):
617617
ctx.report_progress("creating external repo platform")
618618
create_external_repo_platform(
619619
ctx,
620-
exec_properties = ctx.attr.exec_properties,
620+
exec_properties = ctx.attr.internal_exec_properties,
621621
exec_compatible_with = exec_compatible_with,
622622
target_compatible_with = target_compatible_with,
623623
image_name = resolve_rbe_original_image_name(ctx, image_name),
@@ -632,7 +632,7 @@ def _rbe_autoconfig_impl(ctx):
632632
create_config_aliases(ctx, toolchain_config_spec_name)
633633
create_alias_platform(
634634
ctx,
635-
exec_properties = ctx.attr.exec_properties,
635+
exec_properties = ctx.attr.internal_exec_properties,
636636
exec_compatible_with = exec_compatible_with,
637637
target_compatible_with = target_compatible_with,
638638
image_name = resolve_rbe_original_image_name(ctx, image_name),
@@ -782,7 +782,7 @@ _rbe_autoconfig = repository_rule(
782782
"the platform in its constraint_values attr). For " +
783783
"example, [\"@bazel_tools//platforms:linux\"]."),
784784
),
785-
"exec_properties": attr.string_dict(
785+
"internal_exec_properties": attr.string_dict(
786786
doc = (
787787
"Optional. The execution properties to be used when creating the " +
788788
"underlying platform. When providing this attribute, " +
@@ -1192,7 +1192,7 @@ def rbe_autoconfig(
11921192
digest = digest,
11931193
env = env,
11941194
exec_compatible_with = exec_compatible_with,
1195-
exec_properties = exec_properties,
1195+
internal_exec_properties = exec_properties,
11961196
export_configs = export_configs,
11971197
java_home = java_home,
11981198
toolchain_config_suite_spec = toolchain_config_suite_spec_stripped,

0 commit comments

Comments
 (0)