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

fix: update the address book and perform helm upgrade to fix mirror node importer #52

Merged
merged 14 commits into from
Feb 23, 2024

Conversation

jeromy-cannon
Copy link
Contributor

@jeromy-cannon jeromy-cannon commented Feb 21, 2024

Description

This pull request changes the following:

  • update the address book secret for mirror node importer
  • enhance solo node start to trigger a helm upgrade supplying the address book as a value
  • some changes from full-stack-testing repo that got missed during the migration
  • update workflow to upload solo.log to GitHub

Related Issues

@jeromy-cannon jeromy-cannon self-assigned this Feb 21, 2024
Copy link
Contributor

github-actions bot commented Feb 21, 2024

Unit Test Results

 1 files  ±0  13 suites  ±0   21s ⏱️ +2s
83 tests ±0  83 ✅ ±0  0 💤 ±0  0 ❌ ±0 
90 runs  ±0  90 ✅ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit 9ce1bfd. ± Comparison against base commit b7e60c1.

♻️ This comment has been updated with latest results.

Copy link
Contributor

github-actions bot commented Feb 21, 2024

E2E Test Results

732 tests  ±0   732 ✅ ±0   12m 2s ⏱️ +5s
  5 suites ±0     0 💤 ±0 
  1 files   ±0     0 ❌ ±0 

Results for commit 9ce1bfd. ± Comparison against base commit b7e60c1.

♻️ This comment has been updated with latest results.

Copy link

codecov bot commented Feb 21, 2024

Codecov Report

Attention: Patch coverage is 71.21212% with 19 lines in your changes are missing coverage. Please review.

Project coverage is 53.48%. Comparing base (b7e60c1) to head (9ce1bfd).

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main      #52      +/-   ##
==========================================
+ Coverage   51.85%   53.48%   +1.62%     
==========================================
  Files          27       27              
  Lines        2509     2556      +47     
  Branches      414      372      -42     
==========================================
+ Hits         1301     1367      +66     
+ Misses       1208     1189      -19     
Files Coverage Δ
src/commands/account.mjs 70.32% <100.00%> (-0.23%) ⬇️
src/core/chart_manager.mjs 17.18% <100.00%> (+9.37%) ⬆️
src/commands/network.mjs 0.00% <0.00%> (ø)
src/commands/node.mjs 54.31% <95.83%> (+2.97%) ⬆️
src/core/account_manager.mjs 85.12% <95.23%> (+1.79%) ⬆️
src/core/k8.mjs 78.33% <0.00%> (-3.71%) ⬇️

... and 3 files with indirect coverage changes

Impacted file tree graph

@jeromy-cannon jeromy-cannon marked this pull request as ready for review February 22, 2024 19:28
@jeromy-cannon jeromy-cannon changed the title fix: update the address book on mirror node importer and restart it fix: update the address book and perform helm upgrade to fix mirror node importer Feb 22, 2024
leninmehedy
leninmehedy previously approved these changes Feb 22, 2024
jest.config.mjs Outdated Show resolved Hide resolved
src/core/account_manager.mjs Outdated Show resolved Hide resolved
src/core/account_manager.mjs Show resolved Hide resolved
src/commands/node.mjs Outdated Show resolved Hide resolved
… included some changes that weren't synced previously

Signed-off-by: Jeromy Cannon <jeromy@swirldslabs.com>
Signed-off-by: Jeromy Cannon <jeromy@swirldslabs.com>
Signed-off-by: Jeromy Cannon <jeromy@swirldslabs.com>
Signed-off-by: Jeromy Cannon <jeromy@swirldslabs.com>
Signed-off-by: Jeromy Cannon <jeromy@swirldslabs.com>
Signed-off-by: Jeromy Cannon <jeromy@swirldslabs.com>
Signed-off-by: Jeromy Cannon <jeromy@swirldslabs.com>
Signed-off-by: Jeromy Cannon <jeromy@swirldslabs.com>
Signed-off-by: Jeromy Cannon <jeromy@swirldslabs.com>
Signed-off-by: Jeromy Cannon <jeromy@swirldslabs.com>
Signed-off-by: Jeromy Cannon <jeromy@swirldslabs.com>
Signed-off-by: Jeromy Cannon <jeromy@swirldslabs.com>
Signed-off-by: Jeromy Cannon <jeromy@swirldslabs.com>
Signed-off-by: Jeromy Cannon <jeromy@swirldslabs.com>
@jeromy-cannon jeromy-cannon merged commit 26e2e8f into main Feb 23, 2024
15 checks passed
@jeromy-cannon jeromy-cannon deleted the 00761-mirror-node-bug branch February 23, 2024 18:46
swirlds-automation added a commit that referenced this pull request Feb 23, 2024
## [0.21.1](v0.21.0...v0.21.1) (2024-02-23)

### Bug Fixes

* support SOLO_HOME environment variable for home directory ([#53](#53)) ([8745706](8745706))
* update the address book and perform helm upgrade to fix mirror node importer ([#52](#52)) ([26e2e8f](26e2e8f))
@swirlds-automation
Copy link
Contributor

🎉 This PR is included in version 0.21.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

jeromy-cannon added a commit that referenced this pull request Feb 23, 2024
…ode importer (#52)

Signed-off-by: Jeromy Cannon <jeromy@swirldslabs.com>
jeromy-cannon pushed a commit that referenced this pull request Feb 23, 2024
## [0.21.1](v0.21.0...v0.21.1) (2024-02-23)

### Bug Fixes

* support SOLO_HOME environment variable for home directory ([#53](#53)) ([8745706](8745706))
* update the address book and perform helm upgrade to fix mirror node importer ([#52](#52)) ([26e2e8f](26e2e8f))

Signed-off-by: Jeromy Cannon <jeromy@swirldslabs.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants