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

[ci] #4184: Bump to 2024-01-12 toolchain & Revert to custom Coveralls settings #4196

Conversation

BAStos525
Copy link
Contributor

@BAStos525 BAStos525 commented Jan 12, 2024

Description

  1. Update iroha2ci-image to the latest nightly toolchain (2024-01-12).
  2. Bump ci-contaier in workflows to hyperledger/iroha2-ci:nightly-2024-01-12
  3. Revert to the custom base branch comparison for Coveralls

Linked issue

#4184 #4131

Benefits

  1. Use the freshest rust nightly toolchain in CI
  2. Attempt to normalize the Coveralls work

IMPORTANT NOTE

Since we have updated the ci-image, WE MUST build and push it before the actual workflows will start to use it in CI. Previously we did it outside iroha repo (usually from forks) using the special HL DockerHub credentials that Hyperledger provided to us. Unfortunately, as Ry Jones told, they decided to remove these credentials since they can't continue to pay for that. Thus, we must to build and push the new ci-image asap when this PR will be merged.

UPD

Ry Jones decided to give us one more year for external HL DockerHub account. So, the new ci-image
has been already pushed from my fork.

Checklist

  • I've read CONTRIBUTING.md
  • I've used the standard signed-off commit format (or will squash just before merging)
  • All applicable CI checks pass (or I promised to make them pass later)
  • (optional) I've written unit tests for the code changes
  • I replied to all comments after code review, marking all implemented changes with thumbs up

@BAStos525 BAStos525 self-assigned this Jan 12, 2024
@github-actions github-actions bot added the iroha2-dev The re-implementation of a BFT hyperledger in RUST label Jan 12, 2024
@BAStos525 BAStos525 added CI and removed iroha2-dev The re-implementation of a BFT hyperledger in RUST labels Jan 12, 2024
@mversic
Copy link
Contributor

mversic commented Jan 12, 2024

please also update wasm_builder/src/lib.rs

mversic
mversic previously approved these changes Jan 12, 2024
@BAStos525 BAStos525 requested a review from mversic January 12, 2024 09:35
@BAStos525 BAStos525 added the iroha2-dev The re-implementation of a BFT hyperledger in RUST label Jan 12, 2024
@mversic mversic self-assigned this Jan 12, 2024
@Arjentix Arjentix self-assigned this Jan 12, 2024
Copy link
Contributor

@Arjentix Arjentix left a comment

Choose a reason for hiding this comment

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

Hope it's okay that CI is failing.

Don't forget to squash the commits before merging. At least that one with the merge

… custom Coveralls settings

Signed-off-by: BAStos525 <[email protected]>
@BAStos525 BAStos525 force-pushed the feature/DOPS-2943/i2-bump-toolchain-ver-revert-coveralls branch from f076e6d to 0758f7a Compare January 13, 2024 12:21
@BAStos525 BAStos525 merged commit 1320cb2 into hyperledger-iroha:iroha2-dev Jan 13, 2024
@BAStos525 BAStos525 deleted the feature/DOPS-2943/i2-bump-toolchain-ver-revert-coveralls branch January 13, 2024 12:43
Asem-Abdelhady pushed a commit to Asem-Abdelhady/iroha that referenced this pull request Jan 22, 2024
… custom Coveralls settings (hyperledger-iroha#4196)

[ci] hyperledger-iroha#4184: Bump to 2024-01-12 toolchain & revert to custom Coveralls settings

Signed-off-by: BAStos525 <[email protected]>
Signed-off-by: Asem-Abdelhady <[email protected]>
Asem-Abdelhady pushed a commit to Asem-Abdelhady/iroha that referenced this pull request Feb 9, 2024
… custom Coveralls settings (hyperledger-iroha#4196)

[ci] hyperledger-iroha#4184: Bump to 2024-01-12 toolchain & revert to custom Coveralls settings

Signed-off-by: BAStos525 <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI iroha2-dev The re-implementation of a BFT hyperledger in RUST
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants