Skip to content

Rename env keys#70

Merged
tadeas-hejnic merged 10 commits into
developfrom
6-rename-env-keys
Apr 24, 2026
Merged

Rename env keys#70
tadeas-hejnic merged 10 commits into
developfrom
6-rename-env-keys

Conversation

@tadeas-hejnic
Copy link
Copy Markdown
Contributor

@tadeas-hejnic tadeas-hejnic commented Apr 13, 2026

Changelog Description

Use more clear env variable names in ayon-usd addon.

Additional review information

AYONLOGGERLOGLVL -> AYON_USD_RESOLVER_LOG_LVL
AYONLOGGERFILELOGGING -> AYON_USD_RESOLVER_LOG_FILE_ENABLED
AYONLOGGERFILEPOS -> AYON_USD_RESOLVER_LOG_FILE

ENABLE_STATIC_GLOBAL_CACHE -> AYON_USD_RESOLVER_ENABLE_PINNING
PINNING_FILE_PATH -> AYON_USD_RESOLVER_PINNING_FILE
PROJECT_ROOTS -> AYON_USD_RESOLVER_PINNING_ROOTS

Testing notes:

The builds are available here: lakefs://ayon-usd/new-env-keys
This PR is tested along ynput/ayon-usd#119

  1. Test basic workflow with resolver in DCCs
  2. Test resolver in farm environment with pinning

@tadeas-hejnic tadeas-hejnic self-assigned this Apr 13, 2026
@tadeas-hejnic tadeas-hejnic added the type: enhancement Enhancements to existing functionality label Apr 13, 2026
Comment thread src/AyonUsdResolver/helpers/conf_info.md Outdated
@BigRoy
Copy link
Copy Markdown
Member

BigRoy commented Apr 13, 2026

New names do look better. We may need to update AYON USD addon along? And perhaps let some clients know? @MustafaJafar thoughts?

@tadeas-hejnic tadeas-hejnic marked this pull request as ready for review April 20, 2026 12:20
@MustafaJafar
Copy link
Copy Markdown
Member

MustafaJafar commented Apr 20, 2026

New names do look better. We may need to update AYON USD addon along? And perhaps let some clients know? @MustafaJafar thoughts?

This PR requires a counter part in ayon-usd addon. let me spin a PR to use the new keys.
About the documentation, I may need to update screenshots but most importantly, update any mentions for the new keys in https://help.ayon.app/en/help/articles/4862376-ayon-openusd-resolver

@tadeas-hejnic
Copy link
Copy Markdown
Contributor Author

tadeas-hejnic commented Apr 20, 2026

The PR in ayon-usd is already created.

@MustafaJafar
Copy link
Copy Markdown
Member

I can see the logger file works as expected.

file logging is ON
Input log path: "H:\\logFile.json"
[AyonLogger] Initializing async file logger at: H:\logFile.json
[AyonLogger] Async file logger initialized successfully.
file logging is ON
Input log path: "E:\\Ynput\\tmp\\resolver_log\\logFile.json"
[AyonLogger] Initializing async file logger at: E:\Ynput\tmp\resolver_log\logFile.json
[AyonLogger] Async file logger initialized successfully.
before fileLoggerFilePath - E:\Ynput\tmp\resolver_log
file logging is OFF
before fileLoggerFilePath - 
file logging is OFF

@tadeas-hejnic tadeas-hejnic merged commit b4b449f into develop Apr 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: enhancement Enhancements to existing functionality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants