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

Epochs: Clean-up old key-assignment code #1664

Closed
Tracked by #1516
insumity opened this issue Feb 27, 2024 · 1 comment · Fixed by #1672
Closed
Tracked by #1516

Epochs: Clean-up old key-assignment code #1664

insumity opened this issue Feb 27, 2024 · 1 comment · Fixed by #1672
Assignees
@insumity
Labels
C:x/provider Assigned automatically by the PR labeler S: NewThings Work towards your business objectives with new products, features, or integrations

Comments

@insumity
Copy link
Contributor

insumity commented Feb 27, 2024 •
edited
Loading

Problem

After we move to epochs we won't need the key-assignment code any more. This issue is a reminder to go and clean up the code that has to do with key assignments.
Also, we should update the key-assignment ADR to capture those changes.
@insumity insumity added status: waiting-triage This issue/PR has not yet been triaged by the team. S: NewThings Work towards your business objectives with new products, features, or integrations and removed status: waiting-triage This issue/PR has not yet been triaged by the team. labels Feb 27, 2024
@insumity insumity mentioned this issue Feb 27, 2024
Closed
@insumity insumity added the C:x/provider Assigned automatically by the PR labeler label Feb 27, 2024
@mpoke
Copy link
Contributor

mpoke commented Feb 28, 2024

@insumity Keep in mind that the pruning mechanism is still needed.

@mpoke mpoke mentioned this issue Feb 28, 2024
Merged
21 tasks
@insumity insumity self-assigned this Feb 29, 2024
@insumity insumity mentioned this issue Mar 1, 2024
Merged
21 tasks
@insumity insumity linked a pull request Mar 7, 2024 that will close this issue
refactor!: remove key-assignment replacements #1672
Merged
21 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@insumity insumity

Labels
C:x/provider Assigned automatically by the PR labeler S: NewThings Work towards your business objectives with new products, features, or integrations
Projects
Cosmos Hub
Status: ✅ Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

refactor!: remove key-assignment replacements cosmos/interchain-security
2 participants
@insumity @mpoke