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

Improve error reporting when configuration cache is enabled and task references the output of an artifact transform without declaring this as an input #13567

Open
adammurdoch opened this issue Jun 23, 2020 · 1 comment

Comments

@adammurdoch
Copy link
Member

The user can receive a cryptic internal exception when a task references the output of an artifact transform without declaring this as an input and the configuration cache is enabled.

This case is also broken for vintage builds, and fails in arbitrary ways based on the state of the file system, the tasks that are currently running and the transform implementation.

This is essentially a special case of the more general situation, where a task references the output of some other task or transform (or any work) without declaring that as an input.

Expected Behavior

Inform the user that this situation will not work. There are many options for implementing this, ranging from just improving the error message for this specific case (transform outputs + configuration cache) through to deprecating all cases where a task uses the output of some other task or work without declaring this as an input for both vintage and configuration cache builds.

Current Behavior

Writing to the configuration cache fails with a cryptic internal exception, when configuration cache is enabled and fails in different ways when not enabled.

@stale
Copy link

stale bot commented Apr 17, 2022

This issue has been automatically marked as stale because it has not had recent activity. Given the limited bandwidth of the team, it will be automatically closed if no further activity occurs. If you're interested in how we try to keep the backlog in a healthy state, please read our blog post on how we refine our backlog. If you feel this is something you could contribute, please have a look at our Contributor Guide. Thank you for your contribution.

@stale stale bot added the stale label Apr 17, 2022
@eskatos eskatos removed the stale label Apr 22, 2022
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