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

calculate-min-fee no longer working for Shelley-era transactions #700

Open
mkoura opened this issue Apr 5, 2024 · 11 comments
Open

calculate-min-fee no longer working for Shelley-era transactions #700

mkoura opened this issue Apr 5, 2024 · 11 comments
Labels

Comments

@mkoura
Copy link
Contributor

mkoura commented Apr 5, 2024

Description

Building basic Shelley-era transactions using build-raw is still supported (e.g. for bootstrapping testnets). However calculate-min-fee command stopped working for Shelley-era tx after recent fee calculation refactoring. The command fails with

Command failed: transaction calculate-min-fee Error: Error while converting protocol parameters: Missing parameter: decentralization

Steps to Reproduce

  1. Try to calculate fee for Shelley-era tx

cardano-cli shelley transaction calculate-min-fee --testnet-magic 42 --protocol-params-file pparams.json --tx-body-file estimate_tx.body --reference-script-size 0 --witness-count 1

  1. See the error

Additional Context

Tested in Babbage era with cardano-cli 8.22.0.0.
Tx body and pparams files: issue_calc_min_fee.tar.gz

@Jimbo4350
Copy link
Contributor

Jimbo4350 commented Apr 5, 2024

This is not a conway-era-integration-bug. We need to deprecate these old commands as nobody is likely using them.

@gitmachtl
Copy link
Contributor

Building basic Shelley-era transactions using build-raw is still supported
@mkoura

We need to deprecate these old commands as nobody is likely using them.
@Jimbo4350

What exactly do you mean with "we need to deprecate these old commands?"
Using build-raw and calculate-min-fee all the time, the SPO scripts for SPOs are built on that!
Do you mean we cannot use it for testnets in the bootstrap phase until we get to a later era?

This comment was marked as outdated.

@github-actions github-actions bot added the Stale label May 9, 2024

This comment was marked as outdated.

@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale Sep 6, 2024
@gitmachtl
Copy link
Contributor

keep alive

@carbolymer carbolymer reopened this Sep 6, 2024
@carbolymer
Copy link
Contributor

carbolymer commented Sep 6, 2024

@Jimbo4350 I think this needs your input here

@carbolymer
Copy link
Contributor

@gitmachtl what's your use case for running CLI commands in testnets in older eras?
Would it be feasible just hardfork to the target era, at the very beginning, like it's done in cardano-testnet?

@gitmachtl
Copy link
Contributor

i would say yes. as long as we don't have any testnets anymore in the future that are staying a longer period in shelley era, it can be removed for shelley era.

@github-actions github-actions bot removed the Stale label Sep 7, 2024
Copy link

github-actions bot commented Oct 7, 2024

This issue is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 120 days.

@github-actions github-actions bot added the Stale label Oct 7, 2024
@carbolymer carbolymer removed the Stale label Oct 7, 2024
Copy link

github-actions bot commented Nov 7, 2024

This issue is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 120 days.

@github-actions github-actions bot added the Stale label Nov 7, 2024
@carbolymer carbolymer removed the Stale label Nov 12, 2024
Copy link

This issue is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 120 days.

@github-actions github-actions bot added the Stale label Dec 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants