-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Fix: Marshal Input Vertices in a consistent order #4706
Conversation
…averse input vertices in a consistent order Signed-off-by: adamperlin <adamp@nanosoft.com>
Signed-off-by: adamperlin <adamp@nanosoft.com>
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 the issue you have pointed out a different place. I think it makes more sense to guarantee that Inputs()
always returns consistent results, rather than try to patch this in the specific caller later.
…FileOp).Inputs() and (*ExecOp).Inputs() Signed-off-by: adamperlin <adamp@nanosoft.com>
Signed-off-by: adamperlin <adamp@nanosoft.com>
@tonistiigi should be ready for another look. I've revised my fix to address the ordering directly in |
Signed-off-by: adamperlin <adamp@nanosoft.com>
Not sure what the CI failure is here? Unsure if it's related? |
Signed-off-by: adamperlin <adamp@nanosoft.com>
@tonistiigi should be ready for another look |
@tonistiigi can we run CI again? |
@tonistiigi CI is passing now, looks like this can be merged? |
This is a possible fix for #4695. I am opening to run against CI, as I've had trouble running full tests locally