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

Bump cardano-node to 1.34.1 #3160

Merged
merged 6 commits into from
Mar 17, 2022
Merged

Bump cardano-node to 1.34.1 #3160

merged 6 commits into from
Mar 17, 2022

Conversation

Anviking
Copy link
Member

@Anviking Anviking commented Mar 3, 2022

  • Bump node to 1.34.1
    • Bump node
    • Tweak readme
    • Fix a few new compilation warnings/errors
  • Drop support for stack build
    • Tweak docs
    • Remove stack build step from buildkite CI

Comments

Blocked by stack + sublib problem

Guided by git diff 1.33.0 1.34.0 -- cabal.project in cardano-node

Issue Number

ADP-1433

@Anviking Anviking self-assigned this Mar 3, 2022
@Anviking Anviking force-pushed the anviking/ADP-1433/node-1.34 branch 5 times, most recently from d245326 to bf59df7 Compare March 6, 2022 20:43
Copy link
Member

@jonathanknowles jonathanknowles left a comment

Choose a reason for hiding this comment

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

Looks very reasonable to me. 👍🏻

I'm guessing that all of the nix/materialized/stack-nix/.stack-to-nix.cache.* files were updated automatically, via nix/regenerate.sh?

Should the title of this PR be "Bump cardano-node to 1.34.1", given the changes within the README.md?

@Anviking Anviking changed the title Bump cardano-node to 1.34.0 Bump cardano-node to 1.34.1 Mar 10, 2022
@Anviking Anviking force-pushed the anviking/ADP-1433/node-1.34 branch from c32c8d6 to ff002a1 Compare March 15, 2022 12:07
@Anviking
Copy link
Member Author

bors try

iohk-bors bot added a commit that referenced this pull request Mar 15, 2022
@iohk-bors
Copy link
Contributor

iohk-bors bot commented Mar 15, 2022

try

Timed out.

@Anviking Anviking force-pushed the anviking/ADP-1433/node-1.34 branch 5 times, most recently from cda63a7 to d6f3af1 Compare March 15, 2022 22:27
@Anviking
Copy link
Member Author

bors try

iohk-bors bot added a commit that referenced this pull request Mar 16, 2022
@iohk-bors
Copy link
Contributor

iohk-bors bot commented Mar 16, 2022

try

Build succeeded:

docs/developers/Building.md Outdated Show resolved Hide resolved
Copy link
Contributor

@sevanspowell sevanspowell left a comment

Choose a reason for hiding this comment

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

Looks good to me 👍

@Anviking
Copy link
Member Author

bors r+

iohk-bors bot added a commit that referenced this pull request Mar 17, 2022
3160: Bump cardano-node to 1.34.1 r=Anviking a=Anviking

- [x] Bump node to 1.34.1
    - [x] Bump node
    - [x] Tweak readme
    - [x] Fix a few new compilation warnings/errors
- [x] Drop support for `stack build`
    - Tweak docs
    - Remove `stack build` step from buildkite CI

### Comments

<s>Blocked by `stack` + sublib problem</s>

Guided by `git diff 1.33.0 1.34.0 -- cabal.project` in `cardano-node`

### Issue Number

ADP-1433


Co-authored-by: Johannes Lund <[email protected]>
Co-authored-by: IOHK <[email protected]>
Co-authored-by: Samuel Evans-Powell <[email protected]>
@iohk-bors
Copy link
Contributor

iohk-bors bot commented Mar 17, 2022

Build failed:

  src/Test/Integration/Scenario/API/Shelley/TransactionsNew.hs:2116:54: 
  1) API Specifications, NEW_SHELLEY_TRANSACTIONS, Plutus scenarios, withdrawal
       uncaught exception: ProcessHasExited
       ProcessHasExited "cardano-cli failed: Command failed: transaction submit  Error: Error while submitting tx: ShelleyTxValidationError ShelleyBasedEraAlonzo (ApplyTxError [DelegsFailure (DelplFailure (DelegFailure (StakeKeyAlreadyRegisteredDELEG (ScriptHashObj (ScriptHash \"1b8374d0b294a4a3399ff25f06a0707b16214f1e9b67cf89ba2454a6\")))))])\n" (ExitFailure 1)

  To rerun use: --match "/API Specifications/NEW_SHELLEY_TRANSACTIONS/Plutus scenarios/withdrawal/"

#3057

@Anviking
Copy link
Member Author

bors r+

iohk-bors bot added a commit that referenced this pull request Mar 17, 2022
3160: Bump cardano-node to 1.34.1 r=Anviking a=Anviking

- [x] Bump node to 1.34.1
    - [x] Bump node
    - [x] Tweak readme
    - [x] Fix a few new compilation warnings/errors
- [x] Drop support for `stack build`
    - Tweak docs
    - Remove `stack build` step from buildkite CI

### Comments

<s>Blocked by `stack` + sublib problem</s>

Guided by `git diff 1.33.0 1.34.0 -- cabal.project` in `cardano-node`

### Issue Number

ADP-1433


Co-authored-by: Johannes Lund <[email protected]>
Co-authored-by: IOHK <[email protected]>
Co-authored-by: Samuel Evans-Powell <[email protected]>
@Anviking Anviking force-pushed the anviking/ADP-1433/node-1.34 branch from 14535d6 to 2de4ca9 Compare March 17, 2022 16:00
@Anviking
Copy link
Member Author

bors r+

iohk-bors bot added a commit that referenced this pull request Mar 17, 2022
3160: Bump cardano-node to 1.34.1 r=Anviking a=Anviking

- [x] Bump node to 1.34.1
    - [x] Bump node
    - [x] Tweak readme
    - [x] Fix a few new compilation warnings/errors
- [x] Drop support for `stack build`
    - Tweak docs
    - Remove `stack build` step from buildkite CI

### Comments

<s>Blocked by `stack` + sublib problem</s>

Guided by `git diff 1.33.0 1.34.0 -- cabal.project` in `cardano-node`

### Issue Number

ADP-1433


Co-authored-by: Johannes Lund <[email protected]>
Co-authored-by: IOHK <[email protected]>
Co-authored-by: Samuel Evans-Powell <[email protected]>
@iohk-bors
Copy link
Contributor

iohk-bors bot commented Mar 17, 2022

Build failed:

Failures:

  test/unit/Cardano/Wallet/Shelley/Launch/BlockfrostSpec.hs:37:27: 
  1) Cardano.Wallet.Shelley.Launch.Blockfrost, Blockfrost CLI options, modeOption --node-socket
       ParserFailure(option --node-socket: Invalid pipe name. Windows named pipes are of the form \\.\pipe\cardano-node
       
       Usage: <program> (--node-socket PIPENAME | --light --blockfrost-token-file FILE),ExitFailure 1,80)

  To rerun use: --match "/Cardano.Wallet.Shelley.Launch.Blockfrost/Blockfrost CLI options/modeOption --node-socket/"

  test/unit/Cardano/Wallet/Shelley/Launch/BlockfrostSpec.hs:61:17: 
  2) Cardano.Wallet.Shelley.Launch.Blockfrost, Blockfrost CLI options, modeOption requires --light flag
       expected: "Missing: --light\n\nUsage:  (--node-socket FILE | --light --blockfrost-token-file FILE)"
        but got: "Missing: --light\n\nUsage:  (--node-socket PIPENAME | --light --blockfrost-token-file FILE)"

  To rerun use: --match "/Cardano.Wallet.Shelley.Launch.Blockfrost/Blockfrost CLI options/modeOption requires --light flag/"

Randomized with seed 1502943060

@Anviking
Copy link
Member Author

bors r+

@iohk-bors
Copy link
Contributor

iohk-bors bot commented Mar 17, 2022

Build succeeded:

@iohk-bors iohk-bors bot merged commit 0491924 into master Mar 17, 2022
@iohk-bors iohk-bors bot deleted the anviking/ADP-1433/node-1.34 branch March 17, 2022 21:57
WilliamKingNoel-Bot pushed a commit that referenced this pull request Mar 17, 2022
iohk-bors bot added a commit that referenced this pull request Apr 14, 2022
3229: Ensure 1.34.1 node bump is reflected in cabal.project r=Anviking a=Anviking

- [x] Update cabal.project revisions which should have been updated in #3160 
    - This affects `cabal build` **outside** of `nix`. 
    - [x] Check that CI check passes 

### Comments

- Prompted by and includes #3225
- Note: cabal build outside of nix is now tested in normal Buildkite since #3200

<!-- Additional comments, links, or screenshots to attach, if any. -->

### Issue Number

None.

<!-- Reference the Jira/GitHub issue that this PR relates to, and which requirements it tackles.
  Note: Jira issues of the form ADP- will be auto-linked. -->


Co-authored-by: Óscar Nájera <[email protected]>
Co-authored-by: Johannes Lund <[email protected]>
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.

3 participants