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

Integration deploy, don't overwrite genesis block if upgrading testnet #1315

Merged
merged 5 commits into from
Oct 14, 2019

Conversation

tkporter
Copy link
Contributor

@tkporter tkporter commented Oct 11, 2019

Description

Redeployed integration. This also prevents the google cloud genesis file from being overwritten when running celotool deploy upgrade testnet without the --reset flag, in which case the chain data is preserved and the genesis should stay the same.

Tested

Redeployed integration, ran an app & verified

Other changes

n/a

Related issues

n/a

Backwards compatibility

Yes

@codecov
Copy link

codecov bot commented Oct 11, 2019

Codecov Report

Merging #1315 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1315   +/-   ##
=======================================
  Coverage   66.41%   66.41%           
=======================================
  Files         264      264           
  Lines        7745     7745           
  Branches      513      444   -69     
=======================================
  Hits         5144     5144           
- Misses       2500     2502    +2     
+ Partials      101       99    -2
Flag Coverage Δ
#mobile 66.41% <ø> (ø) ⬆️
Impacted Files Coverage Δ
packages/mobile/src/utils/formatting.ts 89.74% <0%> (ø) ⬆️
packages/mobile/src/identity/reducer.ts 41.66% <0%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e8f6f1d...bdcf619. Read the comment docs.

@tkporter tkporter added the automerge Have PR merge automatically when checks pass label Oct 14, 2019
@ashishb ashishb merged commit 82a9d1c into master Oct 14, 2019
@ashishb ashishb deleted the trevor/integration-deploy-10-11 branch October 14, 2019 20:15
aaronmgdr added a commit that referenced this pull request Oct 23, 2019
* master:
  Integration deploy, don't overwrite genesis block if upgrading testnet (#1315)
  Fix broken Discord link (#1317)
  [Wallet] Add check for $ENVFILE to pre-dev script (#1324)
  Add GRADLE_OPTS note to SETUP.md (#1311)
  Allow validators to specify an attestationKey with which they sign attestations (#1313)
  Add step to install typescript and other minor edits (#1256)
  [Wallet] Add script to build sdk for env before running yarn dev (#1312)
  [Wallet] Fix ListView deprecation warning (#1293)
  Set IN_MEMORY_DISCOVERY_TABLE=true for integration (#1296)
  Add ability to generate addresses directly from env file in celotool (#1294)
aaronmgdr added a commit that referenced this pull request Oct 23, 2019
* master:
  Integration deploy, don't overwrite genesis block if upgrading testnet (#1315)
  Fix broken Discord link (#1317)
  [Wallet] Add check for $ENVFILE to pre-dev script (#1324)
  Add GRADLE_OPTS note to SETUP.md (#1311)
  Allow validators to specify an attestationKey with which they sign attestations (#1313)
  Add step to install typescript and other minor edits (#1256)
  [Wallet] Add script to build sdk for env before running yarn dev (#1312)
  [Wallet] Fix ListView deprecation warning (#1293)
  Set IN_MEMORY_DISCOVERY_TABLE=true for integration (#1296)
  Add ability to generate addresses directly from env file in celotool (#1294)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge Have PR merge automatically when checks pass
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants