Skip to content
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

Failed release of master ticket blocks dependent encodings #180

Open
zuntrax opened this issue Jun 25, 2018 · 2 comments
Open

Failed release of master ticket blocks dependent encodings #180

zuntrax opened this issue Jun 25, 2018 · 2 comments

Comments

@zuntrax
Copy link

zuntrax commented Jun 25, 2018

When the state of a master ticket is set to "releasing failed", its dependent tickets are no longer offered to workers, for encoding, postencoding and postprocessing.

I think we should ignore errors in states after the minimum required state to start dependent tickets.

@a-tze
Copy link
Collaborator

a-tze commented Jun 26, 2018

Sounds right, but on the other hand sometimes we set tickets on "released failed" or "releasing failed" manually, to show the fact that something is wrong, but the content is published in some way. One aspect of doing this is exactly to stop dependent tickets from wasting ressources.

I would dislike to add more complexity, users are already confused enough about the internals. "Failed is failed, if something shall be done then fix it." seems more memorable to me.

@jjeising
Copy link
Member

jjeising commented Jun 26, 2018

I like the current behaviour, too. As it was introduced as a new behaviour in #169 we should probably add a message in _status.html.php as documentation.

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

No branches or pull requests

3 participants