-
Notifications
You must be signed in to change notification settings - Fork 44
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
Conway HF update (cardano-node 8.9.0) #830
Conversation
f84e1da
to
5d5b5e5
Compare
cec53c7
to
c323d14
Compare
035b21c
to
ab8f22e
Compare
c87a911
to
ebbdccb
Compare
31357fa
to
ba9fdf9
Compare
9b29bc5
to
e941cfc
Compare
579affc
to
45be3e6
Compare
45be3e6
to
d6188fc
Compare
@@ -729,7 +729,7 @@ whenNonEmptySpec = parallel $ describe "Non-Empty When contracts" $ aroundAll se | |||
emptyMarloweTransactionMetadata | |||
[NormalInput $ IDeposit address address ada 1_000_000] | |||
SubmitFailed msg <- expectLeft "Expected a failure" =<< submit' wallet era txBody | |||
liftIO $ msg `shouldContain` "MissingRequiredSigners" | |||
liftIO $ msg `shouldContain` "MissingVKeyWitnessesUTXOW" | |||
it "should accept a Role2 deposit from wallet 2" $ runAsIntegration \(ContractCreated _ ContractCreatedInEra{..}) -> do |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
MissingRequiredSigners
doesn't exist anymore (because it is redundant with MissingVKeyWitnessesUTXOW
) and some tests were expecting this particular exception :
- https://github.com/IntersectMBO/cardano-ledger/blob/0e115dd5a38484da1ac2d72559aff8579b887970/eras/alonzo/impl/src/Cardano/Ledger/Alonzo/Rules/Utxow.hs#L118
- IntersectMBO/cardano-ledger#3972
@@ -345,7 +346,7 @@ protocolTestnet = | |||
, protocolParamStakeAddressDeposit = C.Lovelace 2000000 | |||
, protocolParamStakePoolDeposit = C.Lovelace 500000000 | |||
, protocolParamMinPoolCost = C.Lovelace 340000000 | |||
, protocolParamPoolRetireMaxEpoch = C.EpochNo 18 | |||
, protocolParamPoolRetireMaxEpoch = CI.EpochInterval 18 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All of these parameters should be upgraded to Conway sometime before the hard fork.
@@ -1772,13 +1773,12 @@ protocolTestnet = | |||
, protocolParamStakeAddressDeposit = Lovelace 2000000 | |||
, protocolParamStakePoolDeposit = Lovelace 500000000 | |||
, protocolParamMinPoolCost = Lovelace 340000000 | |||
, protocolParamPoolRetireMaxEpoch = EpochNo 18 | |||
, protocolParamPoolRetireMaxEpoch = EpochInterval 18 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Upgrade all of these to Conway values before the hard fork.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The changes look great. Excellent work!
This weekend I'll try deploying this locally and running all of the benchmarks.
No description provided.