Skip to content

Support server_listener_resource_name_template#14

Merged
easwars merged 1 commit intoGoogleCloudPlatform:masterfrom
easwars:server_listener_resource_name_template
Mar 2, 2021
Merged

Support server_listener_resource_name_template#14
easwars merged 1 commit intoGoogleCloudPlatform:masterfrom
easwars:server_listener_resource_name_template

Conversation

@easwars
Copy link
Copy Markdown
Collaborator

@easwars easwars commented Feb 24, 2021

No description provided.

@easwars
Copy link
Copy Markdown
Collaborator Author

easwars commented Feb 24, 2021

I went with the assumption that since this bootstrap generator is meant to be used with TD as the control plane, we can hardcode the listener_name template and not give the option of overriding with a flag. Please let me know if you think otherwise.

@easwars easwars requested a review from ejona86 February 24, 2021 18:07
@easwars easwars added the enhancement New feature or request label Feb 24, 2021
Comment thread main.go
GRPCServerResourceNameID string `json:"grpc_server_resource_name_id,omitempty"`
XdsServers []server `json:"xds_servers,omitempty"`
Node *node `json:"node,omitempty"`
CertificateProviders map[string]certificateProviderConfig `json:"certificate_providers,omitempty"`
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The above 3 lines don't contain an actual change - just tab/whitespace changes?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's gofmts work.

@easwars
Copy link
Copy Markdown
Collaborator Author

easwars commented Mar 1, 2021

Gentle ping ...

Copy link
Copy Markdown
Collaborator

@ejona86 ejona86 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Since this changes all-at-once, we'll probably need to upgrade this in the test environment at the same time that gRPC java is upgraded.

@easwars
Copy link
Copy Markdown
Collaborator Author

easwars commented Mar 2, 2021

Since this changes all-at-once, we'll probably need to upgrade this in the test environment at the same time that gRPC java is upgraded.

@sanjaypujare FYI

@easwars easwars merged commit 8af4336 into GoogleCloudPlatform:master Mar 2, 2021
@easwars easwars deleted the server_listener_resource_name_template branch March 2, 2021 22:50
sergiitk added a commit to grpc/psm-interop that referenced this pull request Nov 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants