Skip to content

Execution time improvement #14

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.

Sign up for GitHub

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

Merged
gleb-urvanov merged 4 commits into from
Sep 11, 2020
Merged

Execution time improvement #14

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
8425ad8
new chainspec builr
gleb-urvanov Sep 11, 2020
51cddbb
altered playbook in workflow
gleb-urvanov Sep 11, 2020
9d0cc7f
fixed election parameters tests to work with altered block generation…
gleb-urvanov Sep 11, 2020
0423933
elction proposal parameters fixed to work with reduced block time, re…
gleb-urvanov Sep 11, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 4 additions & 0 deletions devops/ansible/build-and-run-tests-exported-chainspec-playbook.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,10 @@
include_role:
name: install_dependencies

- name: alter block creation time
include_role:
name: alter_block_creation_time

- name: build node
include_role:
name: build_docker_image
Expand Down
4 changes: 4 additions & 0 deletions devops/ansible/build-and-run-tests-single-node-playbook.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,10 @@
include_role:
name: install_dependencies

- name: alter block creation time
include_role:
name: alter_block_creation_time

- name: build node
include_role:
name: build_docker_image
Expand Down
7 changes: 6 additions & 1 deletion devops/ansible/build-and-run-tests-two-nodes-playbook.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,14 +4,19 @@
become_method: sudo

tasks:

- name: install dependencies
include_role:
name: install_dependencies

- name: alter block creation time
include_role:
name: alter_block_creation_time

- name: build node
include_role:
name: build_docker_image

- name: run tests
include_role:
name: run_tests_two_nodes
7 changes: 6 additions & 1 deletion devops/ansible/build-image-playbook.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,10 +4,15 @@
become_method: sudo

tasks:

- name: install dependencies
include_role:
name: install_dependencies

- name: alter block creation time
include_role:
name: alter_block_creation_time

- name: build node
include_role:
name: build_docker_image
name: build_docker_image
4 changes: 4 additions & 0 deletions devops/ansible/roles/alter_block_creation_time/tasks/main.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
- name: alter block creation time
shell: ./scripts/alter-block-creation-time.sh
args:
chdir: ../../
4 changes: 4 additions & 0 deletions scripts/alter-block-creation-time.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
#!/bin/bash

perl -i -pe's/pub const MILLISECS_PER_BLOCK: Moment = 6000;/pub const MILLISECS_PER_BLOCK: Moment = 2000;/' runtime/src/constants.rs
perl -i -pe's/pub const SLOT_DURATION: Moment = 6000;/pub const SLOT_DURATION: Moment = 2000;/' runtime/src/constants.rs
7 changes: 0 additions & 7 deletions scripts/build-joystream-testing-node-docker-image.sh
View file
Open in desktop

This file was deleted.

52 changes: 24 additions & 28 deletions testnets/nicaea-exported-state/chain_spec.json
View file
Open in desktop

Large diffs are not rendered by default.

9,535 changes: 4,768 additions & 4,767 deletions testnets/nicaea-exported-state/raw_chain_spec.json
View file
Open in desktop

Large diffs are not rendered by default.

8 changes: 4 additions & 4 deletions tests/network-tests/src/iznik/tests/fixtures/proposalsModule.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -561,12 +561,12 @@ export class ElectionParametersProposalFixture implements Fixture {
const description: string = 'Testing validator count proposal ' + uuid().substring(0, 8)
const runtimeVoteFee: BN = this.apiWrapper.estimateVoteForProposalFee()
await this.apiWrapper.transferBalanceToAccounts(this.treasury, this.councilKeyPairs, runtimeVoteFee)
const announcingPeriod: BN = await this.apiWrapper.getAnnouncingPeriod()
const votingPeriod: BN = await this.apiWrapper.getVotingPeriod()
const revealingPeriod: BN = await this.apiWrapper.getRevealingPeriod()
const announcingPeriod: BN = new BN(28800)
const votingPeriod: BN = new BN(14400)
const revealingPeriod: BN = new BN(14400)
const councilSize: BN = await this.apiWrapper.getCouncilSize()
const candidacyLimit: BN = await this.apiWrapper.getCandidacyLimit()
const newTermDuration: BN = await this.apiWrapper.getNewTermDuration()
const newTermDuration: BN = new BN(144000)
const minCouncilStake: BN = await this.apiWrapper.getMinCouncilStake()
const minVotingStake: BN = await this.apiWrapper.getMinVotingStake()

Expand Down