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

feat: add support for spanner copy backup feature by asthamohta · Pull Request #600 · googleapis/python-spanner · GitHub

This repository was archived by the owner on Jun 8, 2026. It is now read-only.
/ python-spanner Public archive

feat: add support for spanner copy backup feature - #600

Merged
asthamohta merged 13 commits into
googleapis:mainfrom
asthamohta:spanner-copy-backup
Mar 25, 2022
Merged

asthamohta merged 13 commits into
googleapis:mainfrom
asthamohta:spanner-copy-backup

Conversation

asthamohta commented Sep 29, 2021
edited
Loading

Copy link
Copy Markdown
Contributor

Adding support for Cloud Spanner Cross-Region Backups. This feature allows customers to copy the backup to instances in different regions after creating a backup of the database. The following PR adds the following functionalities:

  1. Support for copy_backup in python CL
  2. Tests for copy_backup

asthamohta requested a review from a team September 29, 2021 22:36
product-auto-label Bot added the api: spanner Issues related to the googleapis/python-spanner API. label Sep 29, 2021
google-cla Bot added the cla: yes This human has signed the Contributor License Agreement. label Sep 29, 2021
asthamohta added the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Sep 29, 2021
Comment thread google/cloud/spanner_v1/backup.py Outdated

larkee commented Nov 17, 2021

Copy link
Copy Markdown
Contributor

Also, PR titles should follow the Commit Message Guidelines.

The rest LGTM 👍

asthamohta requested a review from a team November 18, 2021 14:39
tseaver changed the title feat: Adding support for spanner copy backup feature feat: add support for spanner copy backup feature Nov 18, 2021
Comment thread tests/system/test_backup_api.py Outdated
op.result() # blocks indefinitely


def test_copy_backup_create_w_invalid_expire_time(

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

nit: prefer to include this case in test_copy_backup_workflow in order to reuse the created source backup and minimize the number of created backups.

asthamohta force-pushed the spanner-copy-backup branch 2 times, most recently from 6fd58af to 8c1c404 Compare December 24, 2021 13:35
asthamohta requested a review from a team as a code owner March 10, 2022 04:50

snippet-bot Bot commented Mar 10, 2022
edited
Loading

Copy link
Copy Markdown

Here is the summary of changes.

You are about to add 1 region tag.

This comment is generated by snippet-bot.
If you find problems with this result, please file an issue at:
https://github.com/googleapis/repo-automation-bots/issues.
To update this comment, add snippet-bot:force-run label or use the checkbox below:

  • Refresh this comment

asthamohta force-pushed the spanner-copy-backup branch from 64ece0b to caf4b0b Compare March 23, 2022 12:01
asthamohta force-pushed the spanner-copy-backup branch from 3568e9d to 22bed64 Compare March 24, 2022 05:17
vi3k6i5 added do not merge Indicates a pull request not ready for merge, due to either quality or timing. and removed do not merge Indicates a pull request not ready for merge, due to either quality or timing. labels Mar 24, 2022
asthamohta removed the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Mar 24, 2022
asthamohta force-pushed the spanner-copy-backup branch 6 times, most recently from d8aa15d to eacd97c Compare March 25, 2022 05:25
asthamohta force-pushed the spanner-copy-backup branch from eacd97c to ac4e13b Compare March 25, 2022 07:26
asthamohta merged commit 97faf6c into googleapis:main Mar 25, 2022
vi3k6i5 pushed a commit to vi3k6i5/python-spanner that referenced this pull request Mar 29, 2022
* changes for copy backup feature

* changes to test case

* changes to documenttation

* feat: changes as per review, adding shared_backup

* changes for cross region backup

* samples: changes to list backup operations

* chore(deps): update all dependencies (googleapis#689)

* chore(deps): update dependency pytest to v7.1.1 (googleapis#690)

* feat: add support for Cross region backup proto changes (googleapis#691)

* Synchronize new proto/yaml changes.

PiperOrigin-RevId: 436114471

Source-Link: googleapis/googleapis@6379d5f

Source-Link: googleapis/googleapis-gen@a59984b
Copy-Tag: eyJwIjoiLmdpdGh1Yi8uT3dsQm90LnlhbWwiLCJoIjoiYTU5OTg0YjRjYjcxMWVlYjE4NmJjYTRmNWIzNWFkYmZlNjA4MjVkZiJ9

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>

* feat: adding samples

* linting

Co-authored-by: WhiteSource Renovate <bot@renovateapp.com>
Co-authored-by: gcf-owl-bot[bot] <78513119+gcf-owl-bot[bot]@users.noreply.github.com>
Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
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 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


Back | FazBrowse Home | New Git URL