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.

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

Jump to bottom

Benchmarking performance for ALS #3400

Closed
11 tasks done
PaulGregoryBaker opened this issue Jul 3, 2023 · 1 comment
Closed
11 tasks done

Benchmarking performance for ALS #3400

PaulGregoryBaker opened this issue Jul 3, 2023 · 1 comment
Assignees
@mdebarros @vijayg10
Labels
core-dev-squad pi-22 story

Comments

@PaulGregoryBaker
Copy link

PaulGregoryBaker commented Jul 3, 2023 •
edited by mdebarros
Loading

Goal:

As a Mojaloop Hub adopter
I want to know the isolated performance characterisation of the ALS
so that I can design an environment that depends on it's output.

Acceptance Criteria:

  • Verify that the transactions per second and the parallelisation is specified / identified for this test.
  • Verify that we run the test on two different machine size
  • Verify that any issues are identified and logged as stories.
  • Verify that the result are captured and documented

Complexity: <High|Medium|Low> > A short comment to remind the reason for the rating

Uncertainty: <High|Medium|Low> > A short comment to remind the reason for the rating

Tasks:

  • Do it

Done

  • Acceptance Criteria pass
  • Designs are up-to date
  • Unit Tests pass
  • Integration Tests pass
  • Code Style & Coverage meets standards
  • Changes made to config (default.json) are broadcast to team and follow-up tasks added to update helm charts and other deployment config.

Pull Requests:

Follow-up:

  • N/A

Dependencies:

  • N/A

Accountability:

  • Owner: TBC
  • QA/Review: TBC
@PaulGregoryBaker
Copy link
Author

Hey team! Please add your planning poker estimate with Zenhub @kleyow @mdebarros @vijayg10

@PaulGregoryBaker PaulGregoryBaker changed the title Execute performance Maintenance characterisation for the ALS Benchmarking performance for ALS Jul 17, 2023
@PaulGregoryBaker PaulGregoryBaker removed the to-be-refined This story is ready to be groomed label Jul 17, 2023
@mdebarros mdebarros mentioned this issue Jul 26, 2023
Merged
mdebarros added a commit to mojaloop/account-lookup-service that referenced this issue Jul 27, 2023
@mdebarros
fix: json stringify on every response (#460)
2525b6c 
fix: json stringify on every response
- performance fix for issues identified by @vijayg10 as part of profiling effort as part of mojaloop/project#3400
- updated dependencies

* chore(snapshot): 14.2.3-snapshot.0
mdebarros added a commit to mojaloop/ml-perf-characterization that referenced this issue Aug 1, 2023
@mdebarros
feat(mojaloop/#3424): analyse als perf results
6ccdcd6 
feat(mojaloop/#3424): analyse als perf results - mojaloop/project#3424
- Added analysis for scenarios 1-14

feat(mojaloop/#3400): benchmarking performance for als - mojaloop/project#3400
- Updated comments based on PR review.
@JaneS321 JaneS321 closed this as completed Aug 2, 2023
mdebarros added a commit to mojaloop/ml-perf-characterization that referenced this issue Aug 4, 2023
@mdebarros
Merge pull request #1 from mojaloop/feat/performance-maintenance-char…
2637465 
…acterisation-als

feat(mojaloop/#3400): benchmarking performance for ALS - mojaloop/project#3400
- created documentation, and scenarios for characterizing ALS
- documented performance data from ALS characterization runs

feat(mojaloop/#3424): analyse als perf results - mojaloop/project#3424
- Added analysis for scenarios 1-16
- re-factored structure into domains (i.e. fspiop-discovery, etc)
- added status/summary to the top of domain readmes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mdebarros mdebarros

@vijayg10 vijayg10

Labels
core-dev-squad pi-22 story
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@mdebarros @vijayg10 @PaulGregoryBaker @JaneS321