seismic-dms-service merge requestshttps://community.opengroup.org/osdu/platform/domain-data-mgmt-services/seismic/seismic-dms-suite/seismic-store-service/-/merge_requests2021-09-16T07:48:22Zhttps://community.opengroup.org/osdu/platform/domain-data-mgmt-services/seismic/seismic-dms-suite/seismic-store-service/-/merge_requests/198ci: adding proper resource limits2021-09-16T07:48:22ZDaniel Perezci: adding proper resource limitsEnabling resource limits for seismic store serviceEnabling resource limits for seismic store serviceM9 - Release 0.12Daniel PerezDaniel Perezhttps://community.opengroup.org/osdu/platform/domain-data-mgmt-services/seismic/seismic-dms-suite/seismic-store-service/-/merge_requests/194fix: added required variable2021-09-16T07:48:49ZOleksii Tsyganov (EPAM)fix: added required variableM9 - Release 0.12Oleksii Tsyganov (EPAM)Oleksii Tsyganov (EPAM)https://community.opengroup.org/osdu/platform/domain-data-mgmt-services/seismic/seismic-dms-suite/seismic-store-service/-/merge_requests/205ci: added default variables2021-09-16T09:25:54ZOleksii Tsyganov (EPAM)ci: added default variablesM9 - Release 0.12Oleksii Tsyganov (EPAM)Oleksii Tsyganov (EPAM)https://community.opengroup.org/osdu/platform/domain-data-mgmt-services/seismic/seismic-dms-suite/seismic-store-service/-/merge_requests/207fix: execution order for node extra arguments2021-09-16T10:07:23ZDiego Moltenifix: execution order for node extra argumentsfixed execution order for node extra arguments (before where used as script args, not as node args)fixed execution order for node extra arguments (before where used as script args, not as node args)M9 - Release 0.12Varunkumar ManoharDaniel PerezSacha BrantsVarunkumar Manoharhttps://community.opengroup.org/osdu/platform/domain-data-mgmt-services/seismic/seismic-dms-suite/seismic-store-service/-/merge_requests/208ci: changed Helm targetPort from 8080 to 50002021-09-16T11:01:31ZOleksii Tsyganov (EPAM)ci: changed Helm targetPort from 8080 to 5000M9 - Release 0.12Oleksii Tsyganov (EPAM)Oleksii Tsyganov (EPAM)https://community.opengroup.org/osdu/platform/domain-data-mgmt-services/seismic/seismic-dms-suite/seismic-store-service/-/merge_requests/209ci: configured helm publishing2021-09-16T13:36:29ZOleksii Tsyganov (EPAM)ci: configured helm publishingM9 - Release 0.12Oleksii Tsyganov (EPAM)Oleksii Tsyganov (EPAM)https://community.opengroup.org/osdu/platform/domain-data-mgmt-services/seismic/seismic-dms-suite/seismic-store-service/-/merge_requests/210fix: auth policy check on subproject delete endpoint2021-09-17T07:04:37ZDiego Moltenifix: auth policy check on subproject delete endpointThe tenant admin role was wrongly authorized to delete a subproject while the correct role is the subproject admin (as per documentation)The tenant admin role was wrongly authorized to delete a subproject while the correct role is the subproject admin (as per documentation)M9 - Release 0.12Varunkumar ManoharSacha BrantsVarunkumar Manoharhttps://community.opengroup.org/osdu/platform/domain-data-mgmt-services/seismic/seismic-dms-suite/seismic-store-service/-/merge_requests/211feat: enforce conventional commits for every update and fix husky hook execu...2021-09-20T17:21:28ZVarunkumar Manoharfeat: enforce conventional commits for every update and fix husky hook executionThis MR achieves the following
1. Fixes the execution of husky hooks
- Fixes the issue with secret scanner execution during the precommit phase of a check-in
2. Ensures husky hooks work with a monorepo structure
- Husky has a requ...This MR achieves the following
1. Fixes the execution of husky hooks
- Fixes the issue with secret scanner execution during the precommit phase of a check-in
2. Ensures husky hooks work with a monorepo structure
- Husky has a requirement that the .git and pakcage.json needs to be at the root of the repo.
- Using `npm prepare` scripts the requirement is achieved
3. Enforce conventional commits on every check-in to the source base
- `@commitlint/cli`, `commitlint/config-conventional` are added as dev dependencies
4. Fixes commitizen configuration to align with the monrepo structure
If the commit messages do not follow the conventional commits syntax, the following error is displayed to the user
![image](/uploads/a1313316a5913ce0126fefd7d0aacdd3/image.png)M9 - Release 0.12https://community.opengroup.org/osdu/platform/domain-data-mgmt-services/seismic/seismic-dms-suite/seismic-store-service/-/merge_requests/213ci: moving gcp files to proper place and updating pipeline2021-09-22T14:02:12ZDaniel Perezci: moving gcp files to proper place and updating pipelineMoving GCP files inside of app/sdms since are related to seismic store service
Updating GCP CI/CD pipeline to match new paths and as well to work with multi service in single repoMoving GCP files inside of app/sdms since are related to seismic store service
Updating GCP CI/CD pipeline to match new paths and as well to work with multi service in single repoM9 - Release 0.12Daniel PerezDaniel Perezhttps://community.opengroup.org/osdu/platform/domain-data-mgmt-services/seismic/seismic-dms-suite/seismic-store-service/-/merge_requests/218feat: added cache layer for exchanged tokens & multi audience in the imperson...2021-09-24T14:43:14ZDiego Moltenifeat: added cache layer for exchanged tokens & multi audience in the impersonation token- added cache layer for exchanged tokens to reduce the number of calls to the auth provider
- added multi audience support (multi scope) to the impersonation token- added cache layer for exchanged tokens to reduce the number of calls to the auth provider
- added multi audience support (multi scope) to the impersonation tokenM9 - Release 0.12https://community.opengroup.org/osdu/platform/domain-data-mgmt-services/seismic/seismic-dms-suite/seismic-store-service/-/merge_requests/214feat: enable subid to email conversion2021-09-24T22:22:51ZVarunkumar Manoharfeat: enable subid to email conversionThis MR enables the conversion of `subid` to `email` during dataset GET and subproject GET operations.This MR enables the conversion of `subid` to `email` during dataset GET and subproject GET operations.M9 - Release 0.12https://community.opengroup.org/osdu/platform/domain-data-mgmt-services/seismic/seismic-dms-suite/seismic-store-service/-/merge_requests/224ci: fix filemetadata pipeline2021-09-29T06:57:13ZDaniel Perezci: fix filemetadata pipelineM9 - Release 0.12https://community.opengroup.org/osdu/platform/domain-data-mgmt-services/seismic/seismic-dms-suite/seismic-store-service/-/merge_requests/215feat: merge filemetadata service2021-09-29T12:23:40ZVladimir Moiseevfeat: merge filemetadata servicemerge seismic file meta data service into seismic dmsmerge seismic file meta data service into seismic dmsM9 - Release 0.12Diego MolteniVladimir MoiseevDiego Moltenihttps://community.opengroup.org/osdu/platform/domain-data-mgmt-services/seismic/seismic-dms-suite/seismic-store-service/-/merge_requests/219fix: subproject delete auth check restored2021-09-29T12:23:47ZDiego Moltenifix: subproject delete auth check restoredThe subproject delete endpoint requires tenant.admin (data partition admin) role to be executed.
The endpoint and the documentation have been accordingly updated.The subproject delete endpoint requires tenant.admin (data partition admin) role to be executed.
The endpoint and the documentation have been accordingly updated.M9 - Release 0.12Sacha BrantsSacha Brantshttps://community.opengroup.org/osdu/platform/domain-data-mgmt-services/seismic/seismic-dms-suite/seismic-store-service/-/merge_requests/216fix: user addition to groups and enforces idempotency2021-10-01T05:54:59ZVarunkumar Manoharfix: user addition to groups and enforces idempotencyThis MR fixes an issue with user addition
1. Previously during user addition, the `userEmail` was explicitly removed first and then re-added. This operation can be performed any number of times using a JWT issued for any authorized entit...This MR fixes an issue with user addition
1. Previously during user addition, the `userEmail` was explicitly removed first and then re-added. This operation can be performed any number of times using a JWT issued for any authorized entity other the `userEmail`.
However when the same `userEmail` is added the second time using a JWT authorized for the `userEmail`, the user addition logic removes
the user first. Since this removal, the JWT with azp set to `userEmail` loses complete access to the subproject/dataset.
2. There is a need to maintain idempotency for the user entitlements to fix 1).
This MR catches `409 conflicts` and if the expectedRole for the input `userEmail` does not match the existing entitlements role then an error is thrown. If the roles match no error is returned.
This ensures the useremail can be added any number of times if the entitlements role and the user input role matches.
3. If an error is returned with `409 Conflicts`, then the end-user will need to first delete the user completely and re-add.
3. Refactors the code for improving readabilityM9 - Release 0.12https://community.opengroup.org/osdu/platform/domain-data-mgmt-services/seismic/seismic-dms-suite/seismic-store-service/-/merge_requests/185feat: added DB transactional model on dataset register2021-10-01T05:55:57ZDiego Moltenifeat: added DB transactional model on dataset registerAdded DB transactional model on dataset registerAdded DB transactional model on dataset registerM9 - Release 0.12https://community.opengroup.org/osdu/platform/domain-data-mgmt-services/seismic/seismic-dms-suite/seismic-store-service/-/merge_requests/227fix: fixed vulnerabilities and removed custom metrix2021-10-01T15:33:00ZDiego Moltenifix: fixed vulnerabilities and removed custom metrixfixed vulnerabilities and removed custom metrixfixed vulnerabilities and removed custom metrixM9 - Release 0.12https://community.opengroup.org/osdu/platform/domain-data-mgmt-services/seismic/seismic-dms-suite/seismic-store-service/-/merge_requests/199fix: quick code review on CSP implementation and fixed typos around2021-10-05T07:10:04ZDiego Moltenifix: quick code review on CSP implementation and fixed typos around- quick code review on CSP implementation and fixed typos around in all code
- missed implementations marked with [TODO]
- obsoleted implementation to remove with interface change V3->V4 marked as [OBSOLETE]- quick code review on CSP implementation and fixed typos around in all code
- missed implementations marked with [TODO]
- obsoleted implementation to remove with interface change V3->V4 marked as [OBSOLETE]M9 - Release 0.12Anuj GuptaVarunkumar ManoharYunhua KoglinSacha BrantsAnuj Guptahttps://community.opengroup.org/osdu/platform/domain-data-mgmt-services/seismic/seismic-dms-suite/seismic-store-service/-/merge_requests/228fix: replaced transactional rollback with internal check on thrown errors2021-10-05T11:37:39ZDiego Moltenifix: replaced transactional rollback with internal check on thrown errorsThe recently introduced transactional model flow, on the dataset registration endpoint, showed issues on making query with no ancestor and during concurrency jobs execution. To avoid the extra overhead introduced, the transactional model...The recently introduced transactional model flow, on the dataset registration endpoint, showed issues on making query with no ancestor and during concurrency jobs execution. To avoid the extra overhead introduced, the transactional model flow has been replaced with a coherency check in the dataset registration step.M9 - Release 0.12https://community.opengroup.org/osdu/platform/domain-data-mgmt-services/seismic/seismic-dms-suite/seismic-store-service/-/merge_requests/231fix: making the user add endpoint idempotent2021-10-06T13:35:23ZDiego Moltenifix: making the user add endpoint idempotentmaking the user add endpoint idempotent by returning 200 even in the case the user was already added (the call in entitlement is not idempotent and return 409 -> conflict)making the user add endpoint idempotent by returning 200 even in the case the user was already added (the call in entitlement is not idempotent and return 409 -> conflict)M9 - Release 0.12