Skip to content

refactor: remove consumer to require gov authority on some operations#53

Merged
julienrbrt merged 6 commits into
mainfrom
julien/launch
Jun 9, 2026
Merged

refactor: remove consumer to require gov authority on some operations#53
julienrbrt merged 6 commits into
mainfrom
julien/launch

Conversation

@julienrbrt

@julienrbrt julienrbrt commented Jun 4, 2026

Copy link
Copy Markdown
Member

Closes: #48

  • RemoveConsumer now is authority-only (via gov)
  • Restrict owner post-launch to metadata-only
  • Add consumer chains events to allow building tooling for chain updates
  • Add downtime grace period (after launch)

Launching the chain is fine, as you need 66% of the validators to be up to be marked as launched.

@julienrbrt julienrbrt changed the title refactor remove consumer to require gov authority on some operations refactor: remove consumer to require gov authority on some operations Jun 4, 2026
@julienrbrt julienrbrt marked this pull request as ready for review June 4, 2026 13:22
@giunatale

Copy link
Copy Markdown
Contributor

I don't know we need to discuss what model are we exactly after here.

In a permissionless setting, the owner should be allowed to shutdown the consumer I guess. The way it's set up by this PR the only way to do that is to let the consumer run out of funds, or ask provider governance (which may have an incentive to let the consumer continue spending the funds left in the fee pool). And with #46 the owner can't sweep funds if the chain is launched, so they have to wait for funds to be depleted and can't recover them.

Not saying this is wrong necessarily, but I see some contradictory pulls between wanting a permissionless vs permissioned model.

@giunatale giunatale left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, pretty straightforward

Provided we clarified our divergences outside of this PR, so my earlier comment doesn't apply.

@julienrbrt julienrbrt merged commit 18cc2e5 into main Jun 9, 2026
4 checks passed
Comment thread x/vaas/provider/keeper/consumer_equivocation.go
Comment thread x/vaas/provider/keeper/consumer_equivocation.go
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Launch coordination and owners

3 participants