infrastructure/blender-devops
infrastructure/blender-devops
1
1
Fork 8
Code Pull Requests 3 Activity

User Manual: Remove old files for when deploying #7

Merged
Bart van der Braak merged 4 commits from Blendify/blender-devops:manual-doc-rsync-delete into main 2024-10-04 10:56:45 +02:00
Conversation 4 Commits 4 Files Changed 1 +2 -1
1 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 6984802315 - Show all commits

3
buildbot/worker/doc_manual.py
View File

@ -216,7 +216,8 @@ def deliver(builder: ManualBuilder) -> None:
# Sync html files
source_path = f"{builder.build_path}/{doc_format}/{locale}/"
dest_path = f"{connect_id}:{version_remote_path}/"
worker.utils.rsync(source_path, dest_path, exclude_paths=[".doctrees"])
# Exclude zip files so `--delete` does not delete existing archives
Blendify marked this conversation as resolved Outdated
Sergey Sharybin commented 2024-09-11 11:28:35 +02:00
Outdated
Review
Copy Link

The comment can become more descriptive. It is possibly to deduct that detete=True and --exclude=*.zip will preserve archives. But why is it needed? What the existing archives referred from the comment exactly are?

The comment can become more descriptive. It is possibly to deduct that `detete=True` and `--exclude=*.zip` will preserve archives. But why is it needed? What the existing archives referred from the comment exactly are?
Brecht Van Lommel commented 2024-09-12 01:05:35 +02:00
Outdated
Review
Copy Link

There are html and epub zip files in the same folder, for downloading the manual. The epub zip in particular is uploaded by a builder that runs in parallel with this one.

There are html and epub zip files in the same folder, for downloading the manual. The epub zip in particular is uploaded by a builder that runs in parallel with this one.
worker.utils.rsync(source_path, dest_path, exclude_paths=[".doctrees", ".*zip"], delete=True, delete_path_check=docs_remote_path)
Blendify marked this conversation as resolved Outdated
Brecht Van Lommel commented 2024-09-10 09:51:17 +02:00
Outdated
Review
Copy Link

I have not tested, but the pattern looks wrong and the folder could be more specific:
.*.zip -> *.zip
docs_remote_path -> version_remote_path

I have not tested, but the pattern looks wrong and the folder could be more specific: `.*.zip` -> `*.zip` `docs_remote_path` -> `version_remote_path`
# Create links
if builder.track_id == "vdev":