diff options
| author | Sam Doran <sdoran@redhat.com> | 2021-08-11 23:33:23 -0400 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2021-08-11 20:33:23 -0700 |
| commit | 5af0420cab320af6ab8587c28f87434cdf1a4ea5 (patch) | |
| tree | 278784a5dac37e8aede1d1bb886e8a949777cf55 | |
| parent | 7450e87615b0d9b1503a0f966bd3327901decb75 (diff) | |
| download | ansible-5af0420cab320af6ab8587c28f87434cdf1a4ea5.tar.gz | |
hacking - account for job names that contain separators (#75454)
Some job names contain '/' which causes the download job to fail.
Example job name: `Post-job: Checkout ansible/ansible@refs/pull/75445/merge to ansible.log`
| -rwxr-xr-x | hacking/azp/download.py | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/hacking/azp/download.py b/hacking/azp/download.py index c573e0a7e1..117e9da5b7 100755 --- a/hacking/azp/download.py +++ b/hacking/azp/download.py @@ -214,6 +214,10 @@ def download_run(args): parent_id = parent_of.get(p['id'], None) path = " ".join(names) + + # Some job names have the separator in them. + path = path.replace(os.sep, '_') + log_path = os.path.join(output_dir, '%s.log' % path) if args.verbose: print(log_path) |