Skip to content
This repository was archived by the owner on Mar 31, 2026. It is now read-only.

feat: add support for low-cost instances#313

Merged
larkee merged 4 commits intogoogleapis:masterfrom
zoercai:lci
Jun 23, 2021
Merged

feat: add support for low-cost instances#313
larkee merged 4 commits intogoogleapis:masterfrom
zoercai:lci

Conversation

@zoercai
Copy link
Copy Markdown
Contributor

@zoercai zoercai commented Apr 22, 2021

Allow users to create an instance using processing_units.
1 node = 1000 processing units. There is currently a default node count used if node count is not specified.
create and update now only update processing units rather than node count, as only one can be specified.

Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

  • Ensure the tests and linter pass
  • Code coverage does not decrease (if any source code was changed)
  • Appropriate docs were updated (if necessary)

@zoercai zoercai requested a review from a team April 22, 2021 00:53
@zoercai zoercai requested a review from a team as a code owner April 22, 2021 00:53
@zoercai zoercai requested review from leahecole and removed request for a team April 22, 2021 00:53
@product-auto-label product-auto-label bot added the api: spanner Issues related to the googleapis/python-spanner API. label Apr 22, 2021
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Apr 22, 2021
@zoercai zoercai requested a review from larkee April 22, 2021 00:54
@zoercai zoercai added the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Apr 22, 2021
@leahecole
Copy link
Copy Markdown
Contributor

I think it's adding me as a reviewer because of the synthtool changes in .github/header-checker-lint.yml and those LGTM :)

@zoercai zoercai requested a review from larkee April 26, 2021 07:02
@zoercai zoercai requested a review from larkee April 26, 2021 07:44
Copy link
Copy Markdown
Contributor

@larkee larkee left a comment

Choose a reason for hiding this comment

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

Just a couple comments to go with the proposed changes to resolve the update issue we discussed 👍

@zoercai zoercai requested a review from larkee May 7, 2021 06:27
@leahecole leahecole removed their request for review June 4, 2021 18:11
zoercai and others added 2 commits June 22, 2021 12:00
Co-authored-by: larkee <31196561+larkee@users.noreply.github.com>
@zoercai zoercai removed the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Jun 22, 2021
Co-authored-by: larkee <31196561+larkee@users.noreply.github.com>
@larkee larkee added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jun 23, 2021
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jun 23, 2021
@larkee larkee added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jun 23, 2021
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jun 23, 2021
@larkee larkee merged commit 44aa7cc into googleapis:master Jun 23, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

api: spanner Issues related to the googleapis/python-spanner API. 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