Skip to content

[PSM Interop] Enable xDS affinity test for Node#34288

Merged
murgatroid99 merged 1 commit intogrpc:masterfrom
murgatroid99:xds_test_node_enable_affinity
Sep 15, 2023
Merged

[PSM Interop] Enable xDS affinity test for Node#34288
murgatroid99 merged 1 commit intogrpc:masterfrom
murgatroid99:xds_test_node_enable_affinity

Conversation

@murgatroid99
Copy link
Copy Markdown
Member

@murgatroid99 murgatroid99 commented Sep 8, 2023

Similar to #34146, this will only run on master for now. This will work after grpc/grpc-node#2568 is merged.

@murgatroid99 murgatroid99 added lang/node release notes: no Indicates if PR should not be in release notes area/psm interop labels Sep 8, 2023
@murgatroid99
Copy link
Copy Markdown
Member Author

murgatroid99 commented Sep 8, 2023

@sergiitk
Copy link
Copy Markdown
Member

sergiitk commented Sep 9, 2023 •
edited
Loading

framework.helpers.retryers.RetryError: Retry error calling framework.test_app.client_app.XdsTestClient.find_server_channel_with_state: timeout 0:05:00 (h:mm:ss) exceeded. Last exception: ChannelNotFound: [psm-grpc-client-6cf9cc7d9c-t9sdv] Client has no READY channel with server xds:///psm-grpc-server:59164

Reason: The client couldn't connect to the server.

@murgatroid99
Copy link
Copy Markdown
Member Author

murgatroid99 commented Sep 12, 2023

Attempt 2:

@murgatroid99
Copy link
Copy Markdown
Member Author

murgatroid99 commented Sep 12, 2023 •
edited
Loading

Attempt 3:

@murgatroid99
Copy link
Copy Markdown
Member Author

murgatroid99 commented Sep 13, 2023

The test passed, and I merged the implementation PR.

@murgatroid99 murgatroid99 merged commit 2f05ddc into grpc:master Sep 15, 2023
@copybara-service copybara-service bot added the imported Specifies if the PR has been imported to the internal repository label Sep 15, 2023
sergiitk pushed a commit to sergiitk/grpc that referenced this pull request Dec 8, 2023
@murgatroid99
[PSM Interop] Enable xDS affinity test for Node (grpc#34288)
283c530 
Similar to grpc#34146, this will only run on master for now. This will work
after grpc/grpc-node#2568 is merged.
@ti-chi-bot ti-chi-bot bot mentioned this pull request Jul 29, 2025
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@sergiitk sergiitk sergiitk approved these changes

Assignees

No one assigned

Labels

area/psm interop imported Specifies if the PR has been imported to the internal repository lang/node release notes: no Indicates if PR should not be in release notes

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@murgatroid99 @sergiitk