-
Notifications
You must be signed in to change notification settings - Fork 34
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Deadline: webservice password #200
Conversation
This provides Site Settings fields for Deadline user name and password.
Collects credentials if Project Settings have deadline authentication required.
Changed all calls to module's method which handle SSL
client/ayon_core/modules/deadline/plugins/publish/collect_user_credentials.py
Outdated
Show resolved
Hide resolved
client/ayon_core/modules/deadline/plugins/publish/help/validate_deadline_connection.xml
Outdated
Show resolved
Hide resolved
…e_deadline_connection.xml Co-authored-by: Roy Nieterau <roy_nieterau@hotmail.com>
…_credentials.py Co-authored-by: Roy Nieterau <roy_nieterau@hotmail.com>
…ture/AY-745_Deadline-webservice-password
…ce-password' into feature/AY-745_Deadline-webservice-password
Credentials for DL servers should be set only once, not for each project separately
Please make sure that you have deadline addon set in |
When I set a authentication in DL, Deadline web service validator fails:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Im getting error when trying to submit Render
to DL from the Blender
host app
INFO: Using C:\projects\Ayon_Tests\blender\b3d_shots\sh020\publish\workfile\workfileLighting\v003\ATS_sh020_workfileLighting_v003.blend for render/export.
Traceback (most recent call last):
File "C:\Users\lbate\AppData\Local\Ynput\AYON\dependency_packages\ayon_2402141620_windows.zip\dependencies\pyblish\plugin.py", line 527, in __explicit_process
runner(*args)
File "C:\Work\REPO\ayon-core\client\ayon_core\modules\deadline\abstract_submit_deadline.py", line 468, in process
auth = instance.data["deadline"]["auth"]
~~~~~~~~~~~~~~~~~~~~~~~~~^^^^^^^^
KeyError: 'auth'
Speaking of |
Sucessfully tested also in |
@MustafaJafar could you test in |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I found some bugs with cache publishing.
These two lines:
ayon-core/client/ayon_core/modules/deadline/plugins/publish/submit_publish_cache_job.py
Lines 358 to 359 in 25c9481
for inst in instances: | |
inst["deadlineUrl"] = self.deadline_url |
should be
# Inject deadline url to instances to query DL for job id for overrides
for inst in instances:
if "deadline" not in inst:
inst["deadline"] = {}
inst["deadline"] = instance.data["deadline"]
client/ayon_core/modules/deadline/plugins/publish/submit_publish_cache_job.py
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In AE it fails during DL web service validation:
When Nuke job is already on farm, there is DL error:
|
Must be collected before DL metadata
Changelog Description
This PR provides support for Deadline Webservice authentication.
Additional info
Settings are rudimentary and will need to be changed as we currently don't have
password
type ofSettingsField
which would mask and hash values. AFAIK I cannot useSecrets
as they are not per artist.Currently each artist needs to provide credentials via Site Settings, I didn't provide some default values as I assume if client expects authentication they will not use shared account.
This PR actually affects all existing DL integrations as they might need to have authentication injected.
Testing notes:
Tools>Configure Repository Options>Web Service Settings>Require Authentication
ayon+settings://deadline
Webservice password
)