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: added example backend chart for mojaloop #517

Merged
merged 14 commits into from
Nov 8, 2022

Conversation

vijayg10
Copy link
Contributor

@vijayg10 vijayg10 commented Nov 2, 2022

No description provided.

- updated mojaloop/values.yaml to include CONFIG section with YAML Anchor variables to make configurations easier
- cleaned up example-mojaloop-backend
    - replaced kowl with redpanda-console
    - added MongoDB for CEP
    - updated shell helper scripts to include example-mojaloop-backend charts
@mdebarros
Copy link
Member

mdebarros commented Nov 8, 2022

Test results for commit f0e6805, with default backends being installed:

--------------------FINAL REPORT--------------------

Test Suite:GP Tests
Environment:Development
┌───────────────────────────────────────────────────┐
│                      SUMMARY                      │
├───────────────────┬───────────────────────────────┤
│ Total assertions  │ 2634                          │
├───────────────────┼───────────────────────────────┤
│ Passed assertions │ 2634                          │
├───────────────────┼───────────────────────────────┤
│ Failed assertions │ 0                             │
├───────────────────┼───────────────────────────────┤
│ Total requests    │ 598                           │
├───────────────────┼───────────────────────────────┤
│ Total test cases  │ 130                           │
├───────────────────┼───────────────────────────────┤
│ Passed percentage │ 100.00%                       │
├───────────────────┼───────────────────────────────┤
│ Started time      │ Tue, 08 Nov 2022 16:11:39 GMT │
├───────────────────┼───────────────────────────────┤
│ Completed time    │ Tue, 08 Nov 2022 16:14:51 GMT │
├───────────────────┼───────────────────────────────┤
│ Runtime duration  │ 191625 ms                     │
└───────────────────┴───────────────────────────────┘
TTK-Assertion-Report-multi-2022-11-08T16:14:51.137Z.html was generated
--------------------FINAL REPORT--------------------

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      │ Tue, 08 Nov 2022 16:14:58 GMT │
├───────────────────┼───────────────────────────────┤
│ Completed time    │ Tue, 08 Nov 2022 16:15:23 GMT │
├───────────────────┼───────────────────────────────┤
│ Runtime duration  │ 24833 ms                      │
└───────────────────┴───────────────────────────────┘
TTK-Assertion-Report-multi-2022-11-08T16:15:23.238Z.html was generated

@mdebarros
Copy link
Member

Test results for commit 59a9736, with example-mojaloop-backend deployed:

--------------------FINAL REPORT--------------------

Test Suite:GP Tests
Environment:Development
┌───────────────────────────────────────────────────┐
│                      SUMMARY                      │
├───────────────────┬───────────────────────────────┤
│ Total assertions  │ 2634                          │
├───────────────────┼───────────────────────────────┤
│ Passed assertions │ 2634                          │
├───────────────────┼───────────────────────────────┤
│ Failed assertions │ 0                             │
├───────────────────┼───────────────────────────────┤
│ Total requests    │ 598                           │
├───────────────────┼───────────────────────────────┤
│ Total test cases  │ 130                           │
├───────────────────┼───────────────────────────────┤
│ Passed percentage │ 100.00%                       │
├───────────────────┼───────────────────────────────┤
│ Started time      │ Tue, 08 Nov 2022 17:24:48 GMT │
├───────────────────┼───────────────────────────────┤
│ Completed time    │ Tue, 08 Nov 2022 17:28:14 GMT │
├───────────────────┼───────────────────────────────┤
│ Runtime duration  │ 205838 ms                     │
└───────────────────┴───────────────────────────────┘
TTK-Assertion-Report-multi-2022-11-08T17:28:14.383Z.html was generated
--------------------FINAL REPORT--------------------

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      │ Tue, 08 Nov 2022 17:28:22 GMT │
├───────────────────┼───────────────────────────────┤
│ Completed time    │ Tue, 08 Nov 2022 17:28:47 GMT │
├───────────────────┼───────────────────────────────┤
│ Runtime duration  │ 25335 ms                      │
└───────────────────┴───────────────────────────────┘
TTK-Assertion-Report-multi-2022-11-08T17:28:47.797Z.html was generated

@mdebarros mdebarros marked this pull request as ready for review November 8, 2022 17:34
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.

+1

@mdebarros mdebarros merged commit c295f30 into master Nov 8, 2022
@mdebarros mdebarros deleted the feat/add-sample-mojaloop-backend-helm-chart branch November 8, 2022 18:06
vijayg10 added a commit that referenced this pull request Nov 9, 2022
* feat(mojaloop/#2352): update and standardise helm charts to use networking v1 (#508)

feat(mojaloop/#2352): update and standardise helm charts to use networking v1 - mojaloop/project#2352

# Notes

This PR is re-created from #506 which includes some fixes based on tests on k8s v1.24

# Changes:

## Update the Chart.yaml's and standardise all charts to helm v3 (API v2)
    - move any dependencies in the requirements.yaml to Charts.yaml
    - update the apiVersion for helm in all the Charts.yaml to 2.0
    - add the common dependency to each chart that already has an ingress
    - remove all requirements.lock 
    - update maintainers in chart.yaml to include tomd@crosslaketech.com
    - bump chart versions 

## Update the ingress and networking API to v1 and standardise where possible 
   -  if the chart has an ingress template the chart  is not on the exclusion list (see below) => the ingress template is updated to be a clone of the bitnami example (a version of what already exists in the charts repo)
   - updates the values files for the new ingress settings
   - ensure the updated values files have the correct hostname for the ingress
   - checks for paths and extra paths correctly allowed for (i.e. all charts with extraPaths or extraHosts are on e
   - ensure the updated values files have the correct port number for the ingress
    - update config/default.json files for values.ingress.api.host or similar to use .Values.ingress.hostname
    - _helper.tpl's updated to use correct ingress APIs and remove redundant ingress logic and templates

## General Maintenance
    - Fixed lint issue
    - Removed forensicloggingsidecar, centralenduserregistry, kube-system from lint scripts since they are no longer supported
    - Updated Maintainers list for each chart

## Exclusion list of charts
- i.e. these charts are not yet standardised to the new ingress modeled on the bitnami example :
- Any existing ingress in these charts has been updated "in place" rather than replaced with the newer standardised bitnami modeled chart.
- So the ingress.yaml, config/*json, _helper.tpls , values.yaml files etc have been updated to use the latest ingress but they remain non-standard compared to the rest of the code-base.
- "finance-portal-settlement-management",
- "finance-portal",
- ~"thirdparty"~,
- ~"thirdparty/chart-tp-api-svc"~,
- ~"thirdparty/chart-consent-oracle"~,
- ~"thirdparty/chart-auth-svc"~,
- ~"mojaloop-simulator"~,
- "keycloak",
- "monitoring",
- "monitoring/promfana",
- "monitoring/elk",
- "ml-testing-toolkit/chart-keycloak",
- "ml-testing-toolkit/chart-backend",
- "ml-testing-toolkit/chart-frontend",
- "ml-testing-toolkit/chart-connection-manager-backend",
- "ml-testing-toolkit/chart-connection-manager-frontend"

Co-authored-by: tdaly61 <tdaly61@gmail.com>
Co-authored-by: Sam <10507686+elnyry-sam-k@users.noreply.github.com>

* fix: als-admin api ingress service port (#511)

- fixed als-admin api ingress service port from 4002 to 4001
- fixed formatting issues in for protocol_versions configs

* fix(mojaloop/2995): update ttk-ui (#512)

* chore: bumped up ttk-ui

* chore: some change in appversion

* fix: als-admin api ingress service port (#513)

- fixed als api ingress service port fix from 4002/4001 to 80
- fixed formatting issues in for protocol_versions configs

* chore: updated TTK and fixed some values

* feat: added example backend chart for mojaloop (#517)

feat: added example backend chart for mojaloop (#517) - mojaloop/project#3000
- updated mojaloop/values.yaml to include CONFIG section with YAML Anchor variables to make configurations easier
- added example-mojaloop-backend for external backend dependencies to enable deployments on Kubernetes v1.22+
    - updated shell helper scripts to include example-mojaloop-backend charts
- updates CI config to add helm dependency repos and updated readme

Co-authored-by: Miguel de Barros <miguel@debarros.me>

* updates to chart appVersion description

* fix: changed the default jws file path in sdk helm chart

* fix: sdk helm values

Co-authored-by: Miguel de Barros <miguel@debarros.me>
Co-authored-by: tdaly61 <tdaly61@gmail.com>
Co-authored-by: Sam <10507686+elnyry-sam-k@users.noreply.github.com>
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.

2 participants