GoogleCloudPlatform / GoogleCloudPlatform/gsutil

rsync regression 4.42 -> 4.44

Open
#918 1 comment 1 reaction 0 assignees View on GitHub
Dominant language
Python
Stars
918
Forks
335
PR merge metrics
No merged PRs in 30d

Description

Using the old gsutil, I can run the following commands without issue:

```
mkdir test
touch test/test.txt
gsutil rsync test gs://my-bucket/test-test
gsutil rsync test gs://my-bucket/test
```

The new version fails on the final command with `CommandException: arg (gs://my-bucket/test) does not name a directory, bucket, or bucket subdir.`. Changing the final command fixes it:

```
gsutil rsync test gs://my-bucket/test/
```

Was this change intentional?

Contributor guide

Open the contributing guide

Research direction

Start by reproducing the two gsutil rsync sequences against the affected versions and compare the final destination-argument validation. Trace the rsync command entry point and its directory or bucket-subdirectory checks; done means the intended trailing-slash behavior is documented by a regression test and the compatibility decision is clear.

Written by the indexing model from the issue text.

Assessment

Tech stack
google-cloud, python
Domain
cli, cloud
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.