Skip to content

Commit

Permalink
[PSM Interop] Fix bootstrap generator interop test (grpc#33893)
Browse files Browse the repository at this point in the history
This PR fixes the bootstrap generator interop test by making the node
metadata flag dependent on version, which was causing a breakage
previously as all bootstrap generator version's don't necessarily
support the deexpiermentalized flag.
  • Loading branch information
zasweq authored Jul 29, 2023
1 parent ff905cf commit 9a6fc5c
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -119,6 +119,7 @@ def run( # pylint: disable=arguments-differ,too-many-branches
secure_mode: bool = False,
replica_count: int = 1,
log_to_stdout: bool = False,
bootstrap_version: Optional[str] = None,
) -> List[XdsTestServer]:
if not maintenance_port:
maintenance_port = self._get_default_maintenance_port(secure_mode)
Expand Down Expand Up @@ -202,6 +203,7 @@ def run( # pylint: disable=arguments-differ,too-many-branches
test_port=test_port,
maintenance_port=maintenance_port,
secure_mode=secure_mode,
bootstrap_version=bootstrap_version,
)

pod_names = self._wait_deployment_pod_count(
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -65,7 +65,11 @@ spec:
% if xds_server_uri:
- "--xds-server-uri=${xds_server_uri}"
% endif
% if bootstrap_version=="v0.11.0" or bootstrap_version=="v0.12.0":
- "--node-metadata-experimental=app=${namespace_name}-${deployment_name}"
% else:
- "--node-metadata=app=${namespace_name}-${deployment_name}"
% endif
resources:
limits:
cpu: 100m
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -205,6 +205,7 @@ def test_baseline_in_server_with_bootstrap_version(self, version, image):
maintenance_port=self.server_maintenance_port,
xds_host=self.server_xds_host,
xds_port=self.server_xds_port,
bootstrap_version=version,
)

# Load backends.
Expand Down

0 comments on commit 9a6fc5c

Please sign in to comment.