Skip to content

Use new environment variable names for resolver#119

Open
tadeas-hejnic wants to merge 2 commits into
developfrom
118-rename-env-variables-for-resolver
Open

Use new environment variable names for resolver#119
tadeas-hejnic wants to merge 2 commits into
developfrom
118-rename-env-variables-for-resolver

Conversation

@tadeas-hejnic
Copy link
Copy Markdown
Contributor

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

Changelog Description

This PR is applying the changes made and described in this resolver PR.

Additional review information

This should be merged together with the new release of the resolver builds. To ensure compatibility.

Testing notes:

The builds are available here: lakefs://ayon-usd/v1-2-0-prep

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

@tadeas-hejnic tadeas-hejnic self-assigned this Apr 20, 2026
@tadeas-hejnic tadeas-hejnic added the type: enhancement Improvement of existing functionality or minor addition label Apr 20, 2026
@tadeas-hejnic tadeas-hejnic linked an issue Apr 20, 2026 that may be closed by this pull request
@MustafaJafar MustafaJafar self-requested a review April 20, 2026 18:37
Comment thread client/ayon_usd/plugins/publish/extract_skeleton_pinning_json.py Outdated
Copy link
Copy Markdown
Member

@MustafaJafar MustafaJafar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

btw, it works once the tiny accident is fixed.

Image

Co-authored-by: Mustafa Zaky Jafar <mustafataherzaky@outlook.com>
@MustafaJafar
Copy link
Copy Markdown
Member

For reference, I've update the docs. I'll publish them when this PR is merged.

Tagging @tadeas-hejnic @BigRoy

@MustafaJafar
Copy link
Copy Markdown
Member

Log levels seem to work fine.

@tadeas-hejnic
Copy link
Copy Markdown
Contributor Author

This should be merged after the change in the default settings is made. Builds of resolver needs to be in the release on lakefs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: enhancement Improvement of existing functionality or minor addition

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Rename env variables for resolver

3 participants