This reverts commit 576550d.

This reverts commit 46ce5b8.

PiperOrigin-RevId: 543539118

Source-Link: googleapis/googleapis@ae18706

Source-Link: https://github.com/googleapis/googleapis-gen/commit/5d05516f84e53aaba63a4b8767ff955ac5bb4a87
Copy-Tag: eyJwIjoiLmdpdGh1Yi8uT3dsQm90LnlhbWwiLCJoIjoiNWQwNTUxNmY4NGU1M2FhYmE2M2E0Yjg3NjdmZjk1NWFjNWJiNGE4NyJ9

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

…up from 30 days to 90 days

PiperOrigin-RevId: 544356969

Source-Link: googleapis/googleapis@c35889a

Source-Link: https://github.com/googleapis/googleapis-gen/commit/c00326ec78565b5d16f92c845ff0bb18f11ca05d
Copy-Tag: eyJwIjoiLmdpdGh1Yi8uT3dsQm90LnlhbWwiLCJoIjoiYzAwMzI2ZWM3ODU2NWI1ZDE2ZjkyYzg0NWZmMGJiMThmMTFjYTA1ZCJ9

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

PiperOrigin-RevId: 547553954

Source-Link: googleapis/googleapis@c4e6427

Source-Link: https://github.com/googleapis/googleapis-gen/commit/f552269609d4183546543bfe3a022f544d4f5bdb
Copy-Tag: eyJwIjoiLmdpdGh1Yi8uT3dsQm90LnlhbWwiLCJoIjoiZjU1MjI2OTYwOWQ0MTgzNTQ2NTQzYmZlM2EwMjJmNTQ0ZDRmNWJkYiJ9

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

PiperOrigin-RevId: 551191182

Source-Link: googleapis/googleapis@51e04ba

Source-Link: https://github.com/googleapis/googleapis-gen/commit/4b90e8ead4477eff96c31b9b0fdef36ed975b15f
Copy-Tag: eyJwIjoiLmdpdGh1Yi8uT3dsQm90LnlhbWwiLCJoIjoiNGI5MGU4ZWFkNDQ3N2VmZjk2YzMxYjliMGZkZWYzNmVkOTc1YjE1ZiJ9

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

docs: fixed links in the generated Markdown documentation

PiperOrigin-RevId: 551610576

Source-Link: googleapis/googleapis@73b1313

Source-Link: https://github.com/googleapis/googleapis-gen/commit/8bec066492a6da2855b1b8ce562664c0a6b30b01
Copy-Tag: eyJwIjoiLmdpdGh1Yi8uT3dsQm90LnlhbWwiLCJoIjoiOGJlYzA2NjQ5MmE2ZGEyODU1YjFiOGNlNTYyNjY0YzBhNmIzMGIwMSJ9

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

PiperOrigin-RevId: 554648220

Source-Link: googleapis/googleapis@53cd9ad

Source-Link: https://github.com/googleapis/googleapis-gen/commit/7e8867efbed7dbfe5ef6ec3c2c92a4bce4280f7a
Copy-Tag: eyJwIjoiLmdpdGh1Yi8uT3dsQm90LnlhbWwiLCJoIjoiN2U4ODY3ZWZiZWQ3ZGJmZTVlZjZlYzNjMmM5MmE0YmNlNDI4MGY3YSJ9

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

PiperOrigin-RevId: 557192020

Source-Link: googleapis/googleapis@b4c238f

Source-Link: https://github.com/googleapis/googleapis-gen/commit/feccb30e3177da8b7b7e68149ca4bb914f8faf2a
Copy-Tag: eyJwIjoiLmdpdGh1Yi8uT3dsQm90LnlhbWwiLCJoIjoiZmVjY2IzMGUzMTc3ZGE4YjdiN2U2ODE0OWNhNGJiOTE0ZjhmYWYyYSJ9

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

For the `fallback` parameter, all values considered as `true`
in Boolean context will enable HTTP/1.1 REST fallback,
since the other fallback transport, proto over HTTP, is
removed from `google-gax` v4.

PiperOrigin-RevId: 559812260

Source-Link: googleapis/googleapis@6a6fd29

Source-Link: https://github.com/googleapis/googleapis-gen/commit/56c16657e7a59122b1da94771a9ef40989c282c0
Copy-Tag: eyJwIjoiLmdpdGh1Yi8uT3dsQm90LnlhbWwiLCJoIjoiNTZjMTY2NTdlN2E1OTEyMmIxZGE5NDc3MWE5ZWY0MDk4OWMyODJjMCJ9

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

PiperOrigin-RevId: 565090488

Source-Link: googleapis/googleapis@e8a136f

Source-Link: https://github.com/googleapis/googleapis-gen/commit/9a8dcca0fb2117628a1a6a6c3625a6aa32fc2f75
Copy-Tag: eyJwIjoiLmdpdGh1Yi8uT3dsQm90LnlhbWwiLCJoIjoiOWE4ZGNjYTBmYjIxMTc2MjhhMWE2YTZjMzYyNWE2YWEzMmZjMmY3NSJ9

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

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

…(ESM)

PiperOrigin-RevId: 568643156

Source-Link: googleapis/googleapis@f95afc0

Source-Link: https://github.com/googleapis/googleapis-gen/commit/bbd2c49d2e423a8ce5cc85627402d512aeefc58b
Copy-Tag: eyJwIjoiLmdpdGh1Yi8uT3dsQm90LnlhbWwiLCJoIjoiYmJkMmM0OWQyZTQyM2E4Y2U1Y2M4NTYyNzQwMmQ1MTJhZWVmYzU4YiJ9

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

PiperOrigin-RevId: 571158646

Source-Link: googleapis/googleapis@bc3c83b

Source-Link: https://github.com/googleapis/googleapis-gen/commit/93366e84e4e6861e2e580eb000721d99bf54a0a4
Copy-Tag: eyJwIjoiLmdpdGh1Yi8uT3dsQm90LnlhbWwiLCJoIjoiOTMzNjZlODRlNGU2ODYxZTJlNTgwZWIwMDA3MjFkOTliZjU0YTBhNCJ9

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

PiperOrigin-RevId: 582493526

Source-Link: googleapis/googleapis@7c4e4b5

Source-Link: https://github.com/googleapis/googleapis-gen/commit/368cfb651016d6a93ca6e488cbc34e2d1d9d212c
Copy-Tag: eyJwIjoiLmdpdGh1Yi8uT3dsQm90LnlhbWwiLCJoIjoiMzY4Y2ZiNjUxMDE2ZDZhOTNjYTZlNDg4Y2JjMzRlMmQxZDlkMjEyYyJ9

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

…igtable into copy-backup-2-relevant-changes-only

Expiry time of the destination should be used for the call. Not for the source expiry time.

Use the config and not the backup object for copying backups. The expiry time is needed so a backup object can’t be used because the metadata cannot be introspected.

Rename it to DestinationBackupConfig.

Fix the signature of copyBackup to include the fact that the config now has the gax options.

…com/danieljbruce/nodejs-bigtable into copy-backup-2-relevant-changes-only

If the callback is not required then all calls match first signature overload and async calls return void which isn’t correct.

The system tests were giving an error about the source expiry time and now this is fixed.

Don’t use the name DestinationBackupConfig anymore.

Create a generic callback data structure in order to avoid callback and promise data structures not matching up. Also move the configs to a different file.

Eliminate the generic data structure as it can’t be used to capture the pattern we need exactly. Also modify the way that errors are passed into the callback function to be more consistent.

Copy the backup into the same cluster test is working. Still need to work on copy backup for test that looks at a different cluster to get that test working.

Fix the issue preventing the second test from working by allowing the test function to specify the instance where the new backups should be created.

The test should address a backup copied to a different cluster, but the same instance.

Test case for copying to another project uses environment variable to point to new project for CI pipeline.

Take one of the tests and structure it into a describe block instead. Each test in this block will now run the old test with a different expiry time format.

Rename backupId to id. id is a simpler name and it is the name that is used everywhere. Rename a function to testWithExpiryTimes. Modify source code for copy function to return a backup with the new id. Add test stub for restore backup.

The restore test should test to see if a table can be restored. It should also return a backup object that matches the new backup created.

Second argument is a backup that corresponds to a new backup. This needs to be explained in a comment.

The TODO is done

Modify the copy backup unit test to compare against copying the a destination with customized cluster, backup, instance and project.

The test is not necessarily about copying to a different project. It is more a test about copying to a specific project.

Make sure that the gax options get passed down to the request layer.

The promise and callbacks follow the same pattern that is applied everywhere.

It is actually more of a check than an action that ensures something.

The id is required. It can’t be generated automatically by the server. It needs to be provided.

Add description for the parameters and explain what the function does.

The tests were originally placed in a block at the very beginning so that they were easier to work through. This change moves them to the proper place.

cluster should be used instead of parent to specify the destination cluster.

cluster name should be used instead of parent to match the design doc.

Id is required so no question mark needed. Also inline the function that replaces the project name.

This test was accidentally changed by a refactor and should not change further.

Variables were created so that time can be used in a block of tests. That block of tests were moved so these. variables are no longer needed.

Use parent instead of cluster because that is what it was before.

CreateBackupConfig is not used anywhere.

General cleanup for readability

Reduce number of lines required in testWithExpiryTimes function. This makes the function easier to read.

Don’t use the variables explicitly. Inline them instead.

An extra variable is not needed for this.