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

Unable to create fixed supply Marker at genesis #2223

Open
4 tasks
vwagner opened this issue Nov 21, 2024 · 1 comment
Open
4 tasks

Unable to create fixed supply Marker at genesis #2223

vwagner opened this issue Nov 21, 2024 · 1 comment

Comments

@vwagner
Copy link

vwagner commented Nov 21, 2024

Summary of Bug

When creating a Marker at genesis, the provenanced command line help says that one can use the flag --supplyFixed to create a Marker with a fixed supply. However, when I attempt to use that flag, I get a marker supply invariant panic.

panic: invariant broken: failed to assess invariant: marker: required-marker-supply invariant

	marker: required-marker-supply invariant
	invalid somecoin supply: required (100) current (0somecoin)

Version

1.20

Steps to Reproduce

provenanced -t --home /home/provenance genesis add-marker 100somecoin --supplyFixed --manager funding --access mint,burn,admin,withdraw,deposit --activate --keyring-backend test

For Admin Use

  • Not duplicate issue
  • Appropriate labels applied
  • Appropriate contributors tagged
  • Contributor assigned/self-assigned
@iramiller
Copy link
Member

The issue here is that the marker module balances are not getting properly set in the bank module genesis handling. So if you for example set the escrow amount it will fail because it doesn't count that correctly. If you set the --supplyFixed flag this will turn on invariant checking but unless accounts have been set with the required balances to match things will blow up.

While this command feels like it matches the transaction commands that we are familiar with it is actually different since it is only setting up the various genesis.json entries which will subsequently be loaded when the chain starts...

At a minimum the help documentation could use a refresh here to explain what needs to happen ... but the command itself could stand some updates to try and better work with the bank module genesis to ensure balance amount line up on start.

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

No branches or pull requests

2 participants