configurable Storage dir

[Dariusz1989]

Hello

Description of changes

Added support for storage_dir venv.

Technical details

Just a config variable exposed so we can configure storage via venv for backend.

[Copilot]

Pull request overview

Adds a configurable storage root directory (via AYON_STORAGE_DIR) so backend installer artifacts (desktop installers and dependency packages) can be stored outside the hardcoded /storage/desktop path.

Changes:

• Introduces storage_dir in server configuration with default /storage.
• Updates installer “desktop” directory resolution to use ayonconfig.storage_dir/desktop instead of a hardcoded path.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File	Description
ayon_server/installer/common.py	Uses ayonconfig.storage_dir as the base path for desktop installers/dependency packages.
ayon_server/config/ayonconfig.py	Adds storage_dir config field to be populated from AYON_STORAGE_DIR.

Comments suppressed due to low confidence (1)

ayon_server/installer/common.py:22

• os.makedirs(directory) can still raise if the directory gets created between the isdir check and the call (race) and the current raise AyonException(...) drops the original traceback. Consider using exist_ok=True and exception chaining (... from e) so concurrent workers / retries don’t fail spuriously and debugging retains the underlying error context.

    directory = os.path.join(root, *args)
    if not os.path.isdir(directory):
        if for_writing:
            try:
                os.makedirs(directory)

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.