FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

chore: update CHANGELOG and abi dumps in preparation for release by scotthart · Pull Request #6667 · googleapis/google-cloud-cpp · GitHub

chore: update CHANGELOG and abi dumps in preparation for release - #6667

Merged
scotthart merged 4 commits into
googleapis:mainfrom
scotthart:release_2021_06
Jun 2, 2021
Merged

chore: update CHANGELOG and abi dumps in preparation for release#6667
scotthart merged 4 commits into
googleapis:mainfrom
scotthart:release_2021_06

Conversation

scotthart commented Jun 1, 2021
edited by coryan
Loading

Copy link
Copy Markdown
Member

This change is 

scotthart requested a review from a team June 1, 2021 21:58
google-cla Bot added the cla: yes This human has signed the Contributor License Agreement. label Jun 1, 2021

Copy link
Copy Markdown
Contributor

Google Cloud Build Logs
For commit: bf69667e079211225ceb245a1c1be27b0dde9056

ℹ️ NOTE: Kokoro logs are linked from "Details" below.

codecov Bot commented Jun 1, 2021
edited
Loading

Copy link
Copy Markdown

Codecov Report

Merging #6667 (b39c0c1) into main (2c32710) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #6667   +/-   ##
=======================================
  Coverage   95.77%   95.77%           
=======================================
  Files        1212     1212           
  Lines      107079   107079           
=======================================
  Hits       102558   102558           
  Misses       4521     4521           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2c32710...b39c0c1. Read the comment docs.

Copy link
Copy Markdown
Contributor

Google Cloud Build Logs
For commit: 55dbf35896ace459f8142291796515fa7415593a

ℹ️ NOTE: Kokoro logs are linked from "Details" below.

coryan left a comment

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.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Some ideas to simplify the notes even further.

Comment thread CHANGELOG.md Outdated
Comment on lines +121 to +123
* feat(storage): support method overrides in emulator (#6474)
* fix(storage): support gzip request body in emulator (#6472)
* fix(storage): update emulator to use MultipartDecoder (#6453)

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.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

These 3 are related to the emulator, probably can skip them.

Comment thread CHANGELOG.md Outdated
Comment on lines +111 to +112
* feat(storage): add Retry Test API to emulator (#6611)
* fix(storage): address behavior bugs (#6574)

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.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

These two are related to the emulator, probably can skip them.

Comment thread CHANGELOG.md Outdated
* feat(storage): support method overrides in emulator (#6474)
* fix(storage): support gzip request body in emulator (#6472)
* fix(storage): update emulator to use MultipartDecoder (#6453)
* feat(storage): impersonate service account credentials (#6441)

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.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

I think #6441 #6488 #6518 #6531 #6617 can be summarized as:

 * Support unified credentials (`google::cloud::Credentials`) to initialize both the REST and gRPC plugins with the same classes.

Comment thread CHANGELOG.md Outdated

### Common Libraries

* feat: custom trust store with unified credentials (#6617)

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.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

I think you can summarize #6617 #6614 #6531 #6518 as:

 * Add support for "unified" credentials, which allow applications to
   * Configure the REST and gRPC-based libraries using the same credential classes
   * Support complex credential types, such as service account impersonation
   * With this release, only usable in the `storage` library supports these classes.

Comment thread CHANGELOG.md Outdated
* feat: example using identity tokens (#6569)
* feat: implement unified service account credentials (#6531)
* feat: implement unified insecure credentials (#6518)
* feat(common): streaming RPCs that always fail (#6473)

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.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

I think this and the next are internal-only, we can skip them.

dbolduc left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Reviewable status: 0 of 7 files reviewed, 7 unresolved discussions (waiting on @scotthart)


CHANGELOG.md, line 86 at r2 (raw file):

* feat(bigtable): add async sample rows (#6561)
  • Add asynchronous method for sampling row keys (#6561). Users that extend bigtable::DataClient and wish to use the new method will need to override PrepareAsyncSampleRowKeys().

CHANGELOG.md, line 87 at r2 (raw file):

* fix(bigtable): async bulk apply retry, backoff policy fix  (#6510)
  • bigtable::Table::AsyncBulkApply() now respects the Retry and Backoff policies (#6510)

scotthart left a comment

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.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Reviewable status: 0 of 7 files reviewed, 7 unresolved discussions (waiting on @coryan, @dbolduc, and @scotthart)


CHANGELOG.md, line 86 at r2 (raw file):

Previously, dbolduc (Darren Bolduc) wrote…
* feat(bigtable): add async sample rows (#6561)
  • Add asynchronous method for sampling row keys (#6561). Users that extend bigtable::DataClient and wish to use the new method will need to override PrepareAsyncSampleRowKeys().

Done.


CHANGELOG.md, line 112 at r2 (raw file):

Previously, coryan (Carlos O'Ryan) wrote…

These two are related to the emulator, probably can skip them.

Done.


CHANGELOG.md, line 123 at r2 (raw file):

Previously, coryan (Carlos O'Ryan) wrote…

These 3 are related to the emulator, probably can skip them.

Done.


CHANGELOG.md, line 124 at r2 (raw file):

Previously, coryan (Carlos O'Ryan) wrote…

I think #6441 #6488 #6518 #6531 #6617 can be summarized as:

 * Support unified credentials (`google::cloud::Credentials`) to initialize both the REST and gRPC plugins with the same classes.

Done.


CHANGELOG.md, line 128 at r2 (raw file):

Previously, coryan (Carlos O'Ryan) wrote…

I think you can summarize #6617 #6614 #6531 #6518 as:

 * Add support for "unified" credentials, which allow applications to
   * Configure the REST and gRPC-based libraries using the same credential classes
   * Support complex credential types, such as service account impersonation
   * With this release, only usable in the `storage` library supports these classes.

Done.


CHANGELOG.md, line 134 at r2 (raw file):

Previously, coryan (Carlos O'Ryan) wrote…

I think this and the next are internal-only, we can skip them.

Done.

Copy link
Copy Markdown
Contributor

Google Cloud Build Logs
For commit: 80870bef0709a49e9b596dc2e9ca2f2c75094322

ℹ️ NOTE: Kokoro logs are linked from "Details" below.

Comment thread CHANGELOG.md Outdated
* feat: Add support for "unified" credentials, which allow applications to
Configure the REST and gRPC-based libraries using the same credential classes
Support complex credential types, such as service account impersonation
With this release, only usable in the `storage` library supports these classes.

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.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Ugh, sorry, my grammar was awful here .. maybe "With this release only the storage library supports unified credentials"?

scotthart left a comment

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.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Reviewable status: 0 of 7 files reviewed, 8 unresolved discussions (waiting on @coryan, @dbolduc, and @scotthart)


CHANGELOG.md, line 127 at r3 (raw file):

Previously, coryan (Carlos O'Ryan) wrote…

Ugh, sorry, my grammar was awful here .. maybe "With this release only the storage library supports unified credentials"?

Done.

Copy link
Copy Markdown
Contributor

Google Cloud Build Logs
For commit: ef7622dd04debdb8f4aecf8ec4854713d009d1a3

ℹ️ NOTE: Kokoro logs are linked from "Details" below.

Copy link
Copy Markdown
Contributor

Google Cloud Build Logs
For commit: b39c0c12059640c4e88f0031202fbb13ee72379d

ℹ️ NOTE: Kokoro logs are linked from "Details" below.

scotthart merged commit 855ff01 into googleapis:main Jun 2, 2021
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants


Back | FazBrowse Home | New Git URL