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

Substrate version upgrade into iznik branch (Alexandria) #1163

Merged

Conversation

mnaamani
Copy link
Member

@mnaamani mnaamani commented Aug 17, 2020

Merge substrate_version-upgrade branch into iznik, fixing merge conflicts.

Additional Fixes:

  • substrate-test-runtime custom build script failing in VScode/IDE environment (in similar way to runtime build.rs?)
    With RLS extension enabled in IDE this causes constant high CPU as it keeps trying to rebuild.
    Source sc-service-test package used in tests: node/src/chain_spec.rs and node/src/service.rs

https://github.com/paritytech/substrate/blob/master/test-utils/runtime/build.rs

build failed
error: failed to run custom build command for `substrate-test-runtime v2.0.0-rc4 (https://github.com/paritytech/substrate.git?rev=00768a1f21a579c478fe5d4f51e1fa71f7db9fd4#00768a1f)`

Caused by:
  process didn't exit successfully: `/Users/mokhtar/joystream/joystream/target/rls/debug/build/substrate-test-runtime-963dba27e2a1de0d/build-script-build` (exit code: 1)
--- stdout
Unknown argument 'run'. Supported arguments:

    --version or -V to print the version and commit info
    --help or -h for this message
    --cli starts the RLS in command line mode
    No input starts the RLS as a language server
    
  • Travis CI Build not completing, while building with command: BUILD_DUMMY_WASM_BINARY=1 cargo test --release --verbose --all seems to stall while building Running rustc --crate-name joystream_node...
No output has been received in the last 10m0s, this potentially indicates a stalled build or something wrong with the build itself.

Check the details on how to adjust your build configuration on: https://docs.travis-ci.com/user/common-build-problems/#build-times-out-because-no-output-was-received

The build has been terminated

The issues haven't been fixed, only work around for now.
#1163 (comment) for IDE/RLS issue
unit test code was commented out for now.

# Conflicts:
#	Cargo.lock
#	runtime-modules/hiring/src/mock.rs
#	runtime-modules/recurring-reward/src/mock/mod.rs
#	runtime-modules/token-minting/src/mock.rs
@mnaamani mnaamani changed the title substrate version upgrade Substrate version upgrade into iznik branch (Alexandria) Aug 17, 2020
@mnaamani
Copy link
Member Author

Workaround for substrate-test-runtime custom build failing, set BUILD_DUMMY_WASM_BINARY=1 env variable when opening your IDE:

BUILD_DUMMY_WASM_BINARY=1 code ./joystream

or

BUILD_DUMMY_WASM_BINARY=1 atom ./joystream

@mnaamani
Copy link
Member Author

I changed cargo test --all to cargo test thinking maybe the integration tests which are tagged as ignore are somehow running in the background, but the build still stalls at:

Compiling sc-service-test v2.0.0-rc4 (https://github.com/paritytech/substrate.git?rev=00768a1f21a579c478fe5d4f51e1fa71f7db9fd4#00768a1f)

So should we disable these tests altogether?

@mnaamani
Copy link
Member Author

Trying build without node tests in #1175

@mnaamani mnaamani marked this pull request as draft August 19, 2020 08:21
@mnaamani mnaamani marked this pull request as ready for review August 24, 2020 08:24
Copy link
Contributor

@shamil-gadelshin shamil-gadelshin left a comment

Choose a reason for hiding this comment

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

LGTM.

Travis CI fails with an error during the frame-support compilation.
Here is the discussion: paritytech/substrate#6167

We could pin the exact nightly toolchain to the 1.45.0 version.

UPD: We'd already encountered that issue - (#495) and we even have a proper rust version.

@mnaamani
Copy link
Member Author

LGTM.

Travis CI fails with an error during the frame-support compilation.
Here is the discussion: paritytech/substrate#6167

We could pin the exact nightly toolchain to the 1.45.0 version.

UPD: We'd already encountered that issue - (#495) and we even have a proper rust version.

As discussed I reverted back to working nightly version nightly-2020-05-23

@shamil-gadelshin shamil-gadelshin merged commit 813ccbd into Joystream:iznik Aug 26, 2020
@mnaamani mnaamani deleted the iznik-substrate-version-upgrade branch September 1, 2020 14:09
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.

2 participants