Skip to content
This repository has been archived by the owner on Sep 20, 2024. It is now read-only.

Farm publishing: check if published items do exist #1573

Merged
merged 10 commits into from
Jun 15, 2021
Merged

Farm publishing: check if published items do exist #1573

merged 10 commits into from
Jun 15, 2021

Conversation

create-issue-branch[bot]
Copy link

closes #170

@kalisp kalisp self-assigned this Jun 2, 2021
@kalisp
DL expected files validation - added injection of dependency ids for …
2a65671 
…possible validation
@kalisp
DL expected files validation - added 'targets' to 'publish' command f…
5ca1eb7 
…or more precise plugin selections

Validator for expected files should run only during publishing on farm, targets=deadline handles that
@kalisp
DL expected files validation - extracted post and get method as they …
1c5eda8 
…both should be used on more places
@kalisp
DL expected files validation - save render job from metadata json to …
692f470 
…instance for future validation
@kalisp
DL expected files validation - added expected file validator
5ed11bc 
Checks if expected files (stored in metadata json and collected) are actually rendered to limit later failure in integrate_new
Allows artist override to change rendered frame range
@kalisp
DL expected files validation - added 'targets' to args to push to pub…
3fd5d3c 
…lish job
@kalisp
Merge branch 'develop' into bugfix/170-farm-publishing-check-if-publi…
d106f9f 
…shed-items-do-exist
@kalisp kalisp requested a review from a team June 3, 2021 16:33
@kalisp kalisp marked this pull request as ready for review June 3, 2021 16:33
@mkolar mkolar added the type: bug Something isn't working label Jun 4, 2021
@mkolar mkolar requested a review from antirotor June 9, 2021 08:22
@antirotor
Copy link
Member

Having issue with Python 3 syntax leaking into Maya 2020 Python 2 host:

...
 File "E:\projects\pype\pype-3\repos\avalon-core\avalon\tools\lib.py", line 11, in <module>
    from openpype.modules.sync_server import sync_server_module
  File "E:\projects\pype\pype-3\openpype\modules\__init__.py", line 17, in <module>
    from .webserver import (
  File "E:\projects\pype\pype-3\openpype\modules\webserver\__init__.py", line 1, in <module>
    from .webserver_module import (
  File "E:\projects\pype\pype-3\openpype\modules\webserver\webserver_module.py", line 10, in <module>
    from openpype.modules.webserver.host_console_listener import HostListener
  File "E:\projects\pype\pype-3\openpype\modules\webserver\host_console_listener.py", line 88
    async def websocket_handler(self, request):

@jakubjezek001 jakubjezek001 removed the request for review from a team June 14, 2021 10:40
antirotor
antirotor approved these changes Jun 14, 2021
View reviewed changes
Copy link
Member

@antirotor antirotor left a comment

Choose a reason for hiding this comment

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

Tested with Maya renders and it works

@kalisp kalisp merged commit 5ffbec9 into develop Jun 15, 2021
@kalisp kalisp deleted the bugfix/170-farm-publishing-check-if-published-items-do-exist branch June 15, 2021 10:28
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@antirotor antirotor antirotor approved these changes

Assignees

@kalisp kalisp

Labels
type: bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Farm publishing: check if published items do exist
3 participants
@antirotor @mkolar @kalisp