-
Notifications
You must be signed in to change notification settings - Fork 62
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(mojaloop/#2975): mojaloop helm release v14.1.0 #509
Conversation
* docs: added initial sdk-scheme-adapter folder with main readme * feat: ceate an sdk-scheme-adapter helm chart - added initial chart-service for sdk-scheme-adapter-svc - added parent sdk-scheme-adapter wrapper chart * deployment fixes * updates - added wait-for-redis - fixed env vars * fixes for ingress * fixes for ingress * cleanup of main values config for sdk-scheme-adapter-svc * fixes for sdk-scheme-adapter * clean up to sdk-scheme-adapter values * clean up to sdk-scheme-adapter values * updated readme with documentation * fix: for global vs local vars for kafka and redis - fixed global vs local assignment issues - updated documentation * fixes for configs * cleaned up configs * added 'sdk-scheme-adapter to package/update scripts * added sdk-scheme-adapter to lint script * cleaned up codeowners * bump chart version to v2 * updates for redis database support and kafka topic prefixes - added database to redis config and URI mapping - added chart name as a prefix to all sdk-scheme-adapter topics * bumped sdk-scheme-adapter to latest snapshot * Updated API_SERVER_* env configs to include DOMAIN_EVENT/COMMAND_EVENT prefix depending on context - Updated API_SERVER_* env configs to include DOMAIN_EVENT/COMMAND_EVENT prefix depending on context - updated serviceName on metric configs * typo on port
* chore: bump and validate sdk-scheme-adapter api-svc * chore: fix name in readme * bump to mojaloop-sim charts - bumped chart and dependencies - updated descriptions * updates to thirdparty charts Co-authored-by: Miguel de Barros <[email protected]>
…#503) feat(mojaloop/#2964): add three new simulator dfsps using sdk and ttk - mojaloop/project#2964 - added mojaloop-ttk-simulators parent wrapper chart that provides 3 simulators that are configurable for deployment - fixed configs with sdk-scheme-adapter to make them more configurable for the mojaloop-ttk-sim parent wrapper fix: "data: Too long: must have at most 1048576 bytes" error - `helm.go:84: [debug] Secret "sh.helm.release.v1.bulk.v1" is invalid: data: Too long: must have at most 1048576 bytes` <-- this error occurs because parent charts are including the "raw" child-* folders when they are packaged (i.e. copying unnecessary files), and are already being included in the "charts" folder as tgz packages. Fixed this by adding `chart-*` to all `.helmignore` files, and adding `.helmignore` files where missing. Co-authored-by: Vijay Kumar <[email protected]>
…y to mojaloop (#507) feat([mojaloop/2976](mojaloop/project#2976)): added mojaloop-ttk-simulators chart as dependency to mojaloop (#507) * feat: added mojaloop-ttk-simulators chart as dependency to mojaloop * feat: added helm test command * Updated README.md
- updated central-ledger from v15.1.0 to v16.3.0 - updated v14.1.0 release notes
- updated central-ledger release notes link to v14.1.2.1 which was missed
- updated bulk-api-adapter from v15.1.0 to v16.3.0 - updated v14.1.0 release notes
…esting-toolkit-test-cases-release-v14.1.0
All tests passing with commits:
|
- removed accidently paste line
# Conflicts: # mojaloop/Chart.yaml
* chore: bump sdk * chore: bump to release * chore: bump * chore: bump sdk
* feat: added jws support to sdk-scheme-adapter * chore: added checksums for jws keys in sdk chart
* 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 [email protected] - 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 <[email protected]> Co-authored-by: Sam <[email protected]> * 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 <[email protected]> * 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 <[email protected]> Co-authored-by: tdaly61 <[email protected]> Co-authored-by: Sam <[email protected]>
…se Switch as fspiop-source (#528) - upgraded bulk-api-adapter from v14.1.1 to v14.2.0 for mojaloop/project#2867 - upgraded central-ledger from v16.3.0 to v16.3.1 for mojaloop/project#3024 - added JWS config to ENDPOINT_SECURITY default.json for bulk-api-adapters helm charts - added endpointSecurity config to bulk-api-adapter notification handler config - added secret-jws template to bulk-api-adapter notification handler - added volume mounts to bulk-api-adapter notification handler deployment template
All tests pass for commit: add29f6
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks Miguel..
Made a few minor comments, please review
Co-authored-by: Sam <[email protected]>
Co-authored-by: Sam <[email protected]>
Co-authored-by: Sam <[email protected]>
Co-authored-by: Sam <[email protected]>
Co-authored-by: Sam <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
feat(mojaloop/#2975): mojaloop helm release v14.1.0 - mojaloop/project#2975 - updates to support [Helm v14.1.0 WIP release](mojaloop/helm#509) fix(mojaloop/#3024): [core bulk] POST /bulkTransfers from Switch to PayeeFSP to use Switch as fspiop-source - mojaloop/project#3024 - updated other/bulk_transfers to correctly reflect "DESTINATION" headers to switch Co-authored-by: vijayg10 <[email protected]>
- updated link to use TTK Test Collection release. - added TODO to update link once the response rules have been migrated to the SDK repo - updated release notes with correct SDK version
All tests pass as of commit: 3c63ea9
|
feat(mojaloop/#2975): mojaloop helm release v14.1.0 - mojaloop/project#2975
|
feat(mojaloop/#2975): mojaloop helm release v14.1.0 - mojaloop/project#2975