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

[Governance] Enable x/gov account to remove onchain services #1049

Open
7 tasks
Olshansk opened this issue Jan 30, 2025 · 0 comments
Open
7 tasks

[Governance] Enable x/gov account to remove onchain services #1049

Olshansk opened this issue Jan 30, 2025 · 0 comments
Assignees
Labels
on-chain On-chain business logic service Anything related to general-purpose RPC service support tooling Tooling - CLI, scripts, helpers, off-chain, etc...

Comments

@Olshansk
Copy link
Member

Olshansk commented Jan 30, 2025

Objective

Remove unmaintained, outdated or other legacy onchain services

Origin Document

Lots of users (including Grove employees) usually hit issues where a service has ZERO high-qaulity suppliers.

If these services did not exist, RelayMiners would fail to start.

https://discord.com/channels/553741558869131266/1234943674903953529/1334302035444764733

Image

Goals

  • Give x/gov the ability to delete onchain services
  • Prevent onchain bloat (garbage collection) as more stale / legacy services are added
  • Make RelayMiners fail on non-existenst services by having the foundation / DAO manually clean up those that are unmaintained

Deliverables

  • A new autocli command under poktrolld tx service called delete-service
  • Validation that only the service owner or x/gov module owner can delete the service
  • Sufficient documentation such that this behaviour is searchable in docusaurus

Non-goals / Non-deliverables

  • Any smart / complex business logic to automatically clean up unused services based on onchain claims / proofs

General deliverables

  • Comments: Add/update TODOs and comments alongside the source code so it is easier to follow.
  • Testing: Add new tests (unit and/or E2E) to the test suite.
  • Makefile: Add new targets to the Makefile to make the new functionality easier to use.
  • Documentation: Update architectural or development READMEs; use mermaid diagrams where appropriate.

Creator: @Olshansk
Co-Owners: @bryanchriswhite

@Olshansk Olshansk added on-chain On-chain business logic service Anything related to general-purpose RPC service support tooling Tooling - CLI, scripts, helpers, off-chain, etc... labels Jan 30, 2025
@Olshansk Olshansk added this to the Beta TestNet Iteration milestone Jan 30, 2025
@Olshansk Olshansk added this to Shannon Jan 30, 2025
@github-project-automation github-project-automation bot moved this to 📋 Backlog in Shannon Jan 30, 2025
@Olshansk Olshansk moved this from 📋 Backlog to 🔖 Ready in Shannon Jan 30, 2025
@Olshansk Olshansk changed the title [Governance] Enable x/gov account to remove onchain services [Governance] Enable x/gov account to remove onchain services Jan 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
on-chain On-chain business logic service Anything related to general-purpose RPC service support tooling Tooling - CLI, scripts, helpers, off-chain, etc...
Projects
Status: 🔖 Ready
Development

No branches or pull requests

2 participants