Skip to content

examples: Add XDS client example#6631

Merged
ejona86 merged 2 commits intogrpc:masterfrom
ejona86:hello-xds
Jan 31, 2020
Merged

examples: Add XDS client example#6631
ejona86 merged 2 commits intogrpc:masterfrom
ejona86:hello-xds

Conversation

@ejona86
Copy link
Copy Markdown
Member

@ejona86 ejona86 commented Jan 22, 2020

This depends on #6630, and includes a commit from it.

Comment thread examples/example-xds/README.md Outdated
Comment thread examples/example-xds/README.md
@ejona86
Copy link
Copy Markdown
Member Author

ejona86 commented Jan 29, 2020

@dapengzhang0, PTAL

1. The server does not use XDS, so recent releases work fine. Building using
recent releases is much easier, so check out the most recent release tag:
```
$ git checkout v1.27.0
Copy link
Copy Markdown
Contributor

@dapengzhang0 dapengzhang0 Jan 29, 2020

Choose a reason for hiding this comment

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

This number and below will be changed after every release. Add a line of instruction at https://github.com/grpc/grpc-java/pull/6631/files#diff-60cd2f42437b5cb3c2c09391a6f4c54aR125 ?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

If you look at the history of https://github.com/grpc/grpc-java/commits/master/documentation/android-channel-builder.md, we miss a commit "Update README etc to reference 1.27.0", but RELEASING.md doesn't document to add this commit in the master branch, while similar commits were added for the past releases. I'm confused.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

As far as I'm concerned, I'm okay if we don't update this. Two reasons: 1) it'll work fine, no big deal. 2) grpc-xds should be published the next release or the release after, so it will only be "wrong" for a maximum of one release. Once grpc-xds is published this will be rewritten to look like the other examples and just say "use the latest tag"

Copy link
Copy Markdown
Contributor

@dapengzhang0 dapengzhang0 left a comment

Choose a reason for hiding this comment

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

LGTM

@ejona86 ejona86 merged commit 2e12b53 into grpc:master Jan 31, 2020
@ejona86 ejona86 deleted the hello-xds branch January 31, 2020 19:13
@lock lock Bot locked as resolved and limited conversation to collaborators May 5, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants