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

feat!: upgrade ci, image, packages, audit #381

Merged
merged 16 commits into from
Jul 15, 2022
Merged

Conversation

kleyow
Copy link
Contributor

@kleyow kleyow commented Jul 11, 2022

  • Upgrade to node v16
  • Update packages
  • Update ci with dep/audit checks
  • Update integration tests to add some funds to payer due to central-ledger v14.0.0 breaking change.
  • Lint

Breaking change: Docker image folder structure no longer uses opt/central-settlement/ as the work directory and now uses /opt/app/. Update any mount file paths accordingly.

Major version bump since this is a big upgrade.

@kleyow
Copy link
Contributor Author

kleyow commented Jul 12, 2022

NAME: moja3
LAST DEPLOYED: Tue Jul 12 12:25:55 2022
NAMESPACE: moja3
STATUS: deployed
REVISION: 1
TEST SUITE:     moja3-test
Last Started:   Tue Jul 12 12:39:23 2022
Last Completed: Tue Jul 12 12:39:36 2022
Phase:          Succeeded
TEST SUITE:     moja3-test
Last Started:   Tue Jul 12 12:39:37 2022
Last Completed: Tue Jul 12 12:39:49 2022
Phase:          Succeeded
TEST SUITE:     moja3-ml-ttk-test-setup
Last Started:   Tue Jul 12 12:34:55 2022
Last Completed: Tue Jul 12 12:35:47 2022
Phase:          Succeeded
TEST SUITE:     moja3-ml-ttk-test-val-bulk
Last Started:   Tue Jul 12 12:38:54 2022
Last Completed: Tue Jul 12 12:39:23 2022
Phase:          Succeeded
TEST SUITE:     moja3-ml-ttk-test-val-gp
Last Started:   Tue Jul 12 12:35:47 2022
Last Completed: Tue Jul 12 12:38:53 2022
Phase:          Succeeded

@kleyow kleyow marked this pull request as ready for review July 12, 2022 17:51
Copy link
Member

@mdebarros mdebarros left a comment

Choose a reason for hiding this comment

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

See my comments

.ncurc.yml Show resolved Hide resolved
package.json Show resolved Hide resolved
.circleci/config.yml Show resolved Hide resolved
@kleyow
Copy link
Contributor Author

kleyow commented Jul 14, 2022

Test Suite:GP Tests
Environment:Development
┌───────────────────────────────────────────────────┐
│                      SUMMARY                      │
├───────────────────┬───────────────────────────────┤
│ Total assertions  │ 2573                          │
├───────────────────┼───────────────────────────────┤
│ Passed assertions │ 2573                          │
├───────────────────┼───────────────────────────────┤
│ Failed assertions │ 0                             │
├───────────────────┼───────────────────────────────┤
│ Total requests    │ 587                           │
├───────────────────┼───────────────────────────────┤
│ Total test cases  │ 129                           │
├───────────────────┼───────────────────────────────┤
│ Passed percentage │ 100.00%                       │
├───────────────────┼───────────────────────────────┤
│ Started time      │ Thu, 14 Jul 2022 22:05:11 GMT │
├───────────────────┼───────────────────────────────┤
│ Completed time    │ Thu, 14 Jul 2022 22:08:06 GMT │
├───────────────────┼───────────────────────────────┤
│ Runtime duration  │ 175246 ms                     │
└───────────────────┴───────────────────────────────┘
TTK-Assertion-Report-multi-2022-07-14T22:08:06.506Z.html was generated
Terminate with exit code 0
Done

Test Suite:Bulk Tests
Environment:Development
┌───────────────────────────────────────────────────┐
│                      SUMMARY                      │
├───────────────────┬───────────────────────────────┤
│ Total assertions  │ 183                           │
├───────────────────┼───────────────────────────────┤
│ Passed assertions │ 183                           │
├───────────────────┼───────────────────────────────┤
│ Failed assertions │ 0                             │
├───────────────────┼───────────────────────────────┤
│ Total requests    │ 17                            │
├───────────────────┼───────────────────────────────┤
│ Total test cases  │ 4                             │
├───────────────────┼───────────────────────────────┤
│ Passed percentage │ 100.00%                       │
├───────────────────┼───────────────────────────────┤
│ Started time      │ Thu, 14 Jul 2022 22:08:15 GMT │
├───────────────────┼───────────────────────────────┤
│ Completed time    │ Thu, 14 Jul 2022 22:08:38 GMT │
├───────────────────┼───────────────────────────────┤
│ Runtime duration  │ 23563 ms                      │
└───────────────────┴───────────────────────────────┘
TTK-Assertion-Report-multi-2022-07-14T22:08:38.861Z.html was generated
Terminate with exit code 0
Done

@kleyow kleyow merged commit 12ccec2 into mojaloop:master Jul 15, 2022
@kleyow kleyow deleted the chore/upgrade branch July 15, 2022 06:50
Copy link

@koekiebox koekiebox left a comment

Choose a reason for hiding this comment

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

👍

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.

4 participants