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

Remove provenance --reset option #310

Closed
alexdunnjpl opened this issue Apr 10, 2023 · 0 comments Β· Fixed by #311
Closed

Remove provenance --reset option #310

alexdunnjpl opened this issue Apr 10, 2023 · 0 comments Β· Fixed by #311
Assignees

Comments

@alexdunnjpl
Copy link
Contributor

Checked for duplicates

No - I haven't checked

πŸ§‘β€πŸ”¬ User Persona(s)

Users, developers

πŸ’ͺ Motivation

(Ability to not use) --reset has been determined to be an invitation to generate invalid provenance history under certain circumstances, without significant performance benefit.

As such, probably best to remove it entirely

πŸ“– Additional Details

No response

Acceptance Criteria

provenance works identically to current behaviour with --reset option
provenance throws error if --reset flag is provided

βš™οΈ Engineering Details

Breaking change - will require removal of --reset from deployment crons.

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