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

Crystalin 2nd ci fix #412

Merged
merged 2 commits into from May 14, 2021
Merged

Crystalin 2nd ci fix #412

merged 2 commits into from May 14, 2021

Conversation

crystalin
Copy link
Collaborator

Forces ts-tests to fail when the node is not starting correctly

@crystalin crystalin merged commit b42f392 into joshy-total-nimbus May 14, 2021
@crystalin crystalin deleted the crystalin-2nd-ci-fix branch May 14, 2021 15:22
crystalin added a commit that referenced this pull request May 14, 2021
* Attempt to point to new dependencies. Weird compile errors. Will revisit tomorrow.

* geting closer

* More updates to Cargo.lock. Back to weird compile errors.

* patch author inherent (even though it will be pruned soon)

* EVM to right version

* Temporarily disable author inherent, to work around some errors.

* change scale codec dep made a difference

* Moar progress (reenable author inherent)

* tiny comment improvement

* Fix pallet mocks

* Fix long-standing sudo omission - no noticeable differente though. weird

* It was ParachainSystem that needed Event<T> Thanks @thiolliere

* runtime compiles

* everything compiles (still some cleanup)

* Author inherent from nimbus

* bump cumulonimbus

* 🎵 bumpity-bump-bump ⛄

* remove rand hack

* Author slot filter from nimbus

* Oops, update client-side too

* prune dead code and dealias

* Bump some versions and update types so we don't get confused.

* Implement runtime api (but don't call it yet)

* s/author_inherent/pallet_author_inherent/

* Updates frontier

* Reflect updated fc-rpc version in test

* Restore missing parens

* Update Cargo.lock

* Change fork test behavior. Test now retrieves re-inserted tx at new c… (#398)

* Change fork test behavior. Test now retrieves re-inserted tx at new canonical chain

* Fix format

* Change let for const

* Bump

* Specify hex as return type for net_peerCount @ NetApi::new

* Point frontier back to 3aa29c2

* Skip test to make CI green for now

* No parallel tests to make CI pass

* rustfmt

* Update specs (add palletAuthorSlotFilter)

* bump nimbus

* another bump

* checkpoint bringing in client-side. Need AuthorId != AccountId

* make parachain staking compatible with author id

* ake node compile (but decided I want to re-work the author mapping)

* write out mapping pallet

* restore parachain staking to former glory

* checkpoint writing the mapped wrappers.

* runtime compiles again

* re-enable rewards

* A single dev node can produce blocks!

* Add `max_past_logs` to cli args

* Remove `sc-transaction-graph`

* Update node/Cargo.toml

* Apply suggestions from code review

* Fix debugging snafu

* cargo.lock

* prettier

* Fix tools readme

* one more for the readme

* Don't specify fp-rpc version

* line length

* fix runtime integration test

* Fix typescript integration test

* bump nimbus

* bump nimbus

* Shorten runtime line

* update alphanet spec template

* update stagenet spec template

* ⚠️ DISABLE INTEGRATION TESTS ⚠️

They pass locally, but don't run in CI. I'm disabling them for now so we get a docker image that ops can test.

* Restore ts tests

* Updates script to use known user keys

* Improve CI cargo test consistency

* Change CI to use github runner for non heavy tasks

* Crystalin 2nd ci fix (#412)

* Reduced spawn time check on ts tests

* Moving rust flag as env variable

Co-authored-by: Stephen Shelton <steve@brewcraft.org>
Co-authored-by: Crystalin <alan@purestake.com>
Co-authored-by: girazoki <gorka.irazoki@gmail.com>
Co-authored-by: tgmichel <telmo@purestake.com>
Co-authored-by: Alan Sapede <alan.sapede@gmail.com>
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.

None yet

1 participant