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.

Sign up for GitHub

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

Jump to bottom

Remove operator nonce from cluster definition #891

Closed
corverroos opened this issue Aug 2, 2022 · 0 comments
Closed

Remove operator nonce from cluster definition #891

corverroos opened this issue Aug 2, 2022 · 0 comments
Assignees
@corverroos
Labels
enhancement New feature or request

Comments

@corverroos
Copy link
Contributor

Problem to be solved

The current cluster definition spec includes a operator nonce field. It is not used and current defaults to 0.

The idea was to use this for key rotation, but we will most probably not be using the field for that purpose, since the cluster definition and lock files are immutable and we will most likely append mutations to the file making that nonce fields redundant.

See mutation design doc.

Proposed solution

Remove the nonce from operator struct as part of v1.2.0. Note the new code should remain backwards compatible with all prevous versions.

Out of Scope

Release v1.2.0 is not included as part of this issue.
@corverroos corverroos added the enhancement New feature or request label Aug 2, 2022
@corverroos corverroos self-assigned this Aug 23, 2022
@corverroos corverroos mentioned this issue Aug 23, 2022
Merged
obol-bulldozer bot pushed a commit that referenced this issue Aug 23, 2022
@corverroos
cluster: remove definition operator nonce and release v1.2 (#1018)
d66662b 
Removes operator nonce from cluster definition v1.2 and releases it.

category: feature
ticket: #891
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@corverroos corverroos

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@corverroos