seismic-dms-service merge requestshttps://community.opengroup.org/osdu/platform/domain-data-mgmt-services/seismic/seismic-dms-suite/seismic-store-service/-/merge_requests2023-11-08T15:03:13Zhttps://community.opengroup.org/osdu/platform/domain-data-mgmt-services/seismic/seismic-dms-suite/seismic-store-service/-/merge_requests/958feat: added quality code scripts and fixed spelling errors.2023-11-08T15:03:13ZDiego Moltenifeat: added quality code scripts and fixed spelling errors.## Type of change
- [ ] Bug Fix
- [x] Feature
- [ ] Pipeline
- [ ] Test
- [ ] Documentation
## Does this introduce a change in the core logic?
- [x] No
- [ ] Yes
## Does this introduce a change in the cloud provider implementation, if ...## Type of change
- [ ] Bug Fix
- [x] Feature
- [ ] Pipeline
- [ ] Test
- [ ] Documentation
## Does this introduce a change in the core logic?
- [x] No
- [ ] Yes
## Does this introduce a change in the cloud provider implementation, if so which cloud?
- [ ] AWS
- [ ] Anthos
- [ ] Azure
- [ ] GCP
- [ ] IBM
## Does this follow [conventional commits spec](https://www.conventionalcommits.org/en/v1.0.0/#summary)?
- [ ] No
- [x] Yes
## Have you set the target Milestone?
- [ ] No
- [x] Yes
## Have you set the no-detached-pipeline label?
- [ ] No
- [x] Yes
## Updates description?
> centralized git pre-commit hooks for all software of the monorepoM22 - Release 0.25Diego MolteniDiego Moltenihttps://community.opengroup.org/osdu/platform/domain-data-mgmt-services/seismic/seismic-dms-suite/seismic-store-service/-/merge_requests/988fix: fixed subproject acls2023-11-07T09:04:15ZDiego Moltenifix: fixed subproject acls## Type of change
- [x] Bug Fix
- [ ] Feature
- [ ] Pipeline
- [ ] Test
- [ ] Documentation
## Does this introduce a change in the core logic?
- [x] No
- [ ] Yes
## Does this introduce a change in the cloud provider implementation, if ...## Type of change
- [x] Bug Fix
- [ ] Feature
- [ ] Pipeline
- [ ] Test
- [ ] Documentation
## Does this introduce a change in the core logic?
- [x] No
- [ ] Yes
## Does this introduce a change in the cloud provider implementation, if so which cloud?
- [ ] AWS
- [ ] Anthos
- [ ] Azure
- [ ] GCP
- [ ] IBM
## Does this follow [conventional commits spec](https://www.conventionalcommits.org/en/v1.0.0/#summary)?
- [ ] No
- [x] Yes
## Have you set the target Milestone?
- [ ] No
- [x] Yes
## Have you set the no-detached-pipeline label?
- [ ] No
- [x] Yes
## Updates description?
> Fixed default subproject ACLs checkM22 - Release 0.25Diego MolteniDiego Moltenihttps://community.opengroup.org/osdu/platform/domain-data-mgmt-services/seismic/seismic-dms-suite/seismic-store-service/-/merge_requests/911feat: add insights logging2023-11-07T06:56:29ZRashaad Grayfeat: add insights logging## Type of change
- [ ] Bug Fix
- [X] Feature
- [ ] Pipeline
- [ ] Test
- [ ] Documentation
## Does this introduce a change in the core logic?
- [X] No
- [ ] Yes
## Does this introduce a change in the cloud provider implementation, if ...## Type of change
- [ ] Bug Fix
- [X] Feature
- [ ] Pipeline
- [ ] Test
- [ ] Documentation
## Does this introduce a change in the core logic?
- [X] No
- [ ] Yes
## Does this introduce a change in the cloud provider implementation, if so which cloud?
- [ ] AWS
- [ ] Anthos
- [x] Azure
- [ ] GCP
- [ ] IBM
## Does this follow [conventional commits spec](https://www.conventionalcommits.org/en/v1.0.0/#summary)?
- [ ] No
- [X] Yes
## Have you set the target Milestone?
- [ ] No
- [X] Yes
## Have you set the no-detached-pipeline label?
- [ ] No
- [X] Yes
## Updates description?
> Register all the requests into application insight. Record the error codes and response.M21 - Release 0.24Rashaad GrayRashaad Grayhttps://community.opengroup.org/osdu/platform/domain-data-mgmt-services/seismic/seismic-dms-suite/seismic-store-service/-/merge_requests/975[For MSFT Team Review] Compute Size background service2023-11-06T15:37:00ZIzabela Kulakowska[For MSFT Team Review] Compute Size background service## Type of change
- [ ] Bug Fix
- [x] Feature
- [ ] Pipeline
- [ ] Test
- [ ] Documentation
## Does this introduce a change in the core logic?
- [x] No
- [ ] Yes
## Does this introduce a change in the cloud provider implementation, if ...## Type of change
- [ ] Bug Fix
- [x] Feature
- [ ] Pipeline
- [ ] Test
- [ ] Documentation
## Does this introduce a change in the core logic?
- [x] No
- [ ] Yes
## Does this introduce a change in the cloud provider implementation, if so which cloud?
- [ ] AWS
- [ ] Anthos
- [x] Azure
- [ ] GCP
- [ ] IBM
## Does this follow [conventional commits spec](https://www.conventionalcommits.org/en/v1.0.0/#summary)?
- [ ] No
- [ ] Yes
## Have you set the target Milestone?
- [ ] No
- [ ] Yes
## Have you set the no-detached-pipeline label?
- [ ] No
- [ ] Yes
## Updates description?
This MR introduces a background service for computing dataset sizes.
It fetches the messages from the Redis queue and processes them:
- retrieves blob sizes and calculates the dataset size
- if the size changed executes a partial update CosmosDB metadata store for the dataset
Plese not it's an MR for internal review for MSFT team.
Locking the dataset for write will be added in the next MR.Izabela KulakowskaIzabela Kulakowskahttps://community.opengroup.org/osdu/platform/domain-data-mgmt-services/seismic/seismic-dms-suite/seismic-store-service/-/merge_requests/974fix: fixed bug on ls subprojects2023-11-06T15:11:37ZDiego Moltenifix: fixed bug on ls subprojects## Type of change
- [x] Bug Fix
- [ ] Feature
- [ ] Pipeline
- [ ] Test
- [ ] Documentation
## Does this introduce a change in the core logic?
- [x] No
- [ ] Yes
## Does this introduce a change in the cloud provider implementation, if ...## Type of change
- [x] Bug Fix
- [ ] Feature
- [ ] Pipeline
- [ ] Test
- [ ] Documentation
## Does this introduce a change in the core logic?
- [x] No
- [ ] Yes
## Does this introduce a change in the cloud provider implementation, if so which cloud?
- [ ] AWS
- [ ] Anthos
- [ ] Azure
- [ ] GCP
- [ ] IBM
## Does this follow [conventional commits spec](https://www.conventionalcommits.org/en/v1.0.0/#summary)?
- [ ] No
- [x] Yes
## Have you set the target Milestone?
- [ ] No
- [x] Yes
## Have you set the no-detached-pipeline label?
- [ ] No
- [x] Yes
## Updates description?
> Fixed a bug on the subproject ls endpointM22 - Release 0.25Diego MolteniDiego Moltenihttps://community.opengroup.org/osdu/platform/domain-data-mgmt-services/seismic/seismic-dms-suite/seismic-store-service/-/merge_requests/985test: updated dataset register random test2023-11-06T14:45:31ZPratiksha Shedgetest: updated dataset register random test## Type of change
* [ ] Bug Fix
* [ ] Feature
* [ ] Pipeline
* [x] Test
* [ ] Documentation
## Does this introduce a change in the core logic?
* [x] No
* [ ] Yes
## Does this introduce a change in the cloud provider implementation, i...## Type of change
* [ ] Bug Fix
* [ ] Feature
* [ ] Pipeline
* [x] Test
* [ ] Documentation
## Does this introduce a change in the core logic?
* [x] No
* [ ] Yes
## Does this introduce a change in the cloud provider implementation, if so which cloud?
* [x] AWS
* [x] Anthos
* [x] Azure
* [x] GCP
* [x] IBM
## Does this follow [conventional commits spec](https://www.conventionalcommits.org/en/v1.0.0/#summary)?
* [ ] No
* [x] Yes
## Have you set the target Milestone?
* [ ] No
* [x] Yes
## Have you set the no-detached-pipeline label?
* [ ] No
* [x] Yes
## Updates description?
> Issue description-In [this MR](https://community.opengroup.org/osdu/platform/domain-data-mgmt-services/seismic/seismic-dms-suite/seismic-store-service/-/merge_requests/926) I see that the variables _seismic_default_admins_ and _seismic_default_viewers_ are set three times with different values, but they are used only once and tests don't fail when the value changes. The logic of tests needs to be reviewed.
>
> Solution- The request body used for registering the dataset 'random' with variables 'seismic_default_admins' and 'seismic_default_viewer' was found to be unacceptable for that endpoint. I have since modified the endpoint's request body, and I can confirm that it is now working fine.M22 - Release 0.25https://community.opengroup.org/osdu/platform/domain-data-mgmt-services/seismic/seismic-dms-suite/seismic-store-service/-/merge_requests/926test: subproject delete in e2e gitlab2023-11-06T09:27:21ZPratiksha Shedgetest: subproject delete in e2e gitlab## Type of change
* [ ] Bug Fix
* [ ] Feature
* [ ] Pipeline
* [x] Test
* [ ] Documentation
## Does this introduce a change in the core logic?
* [x] No
* [ ] Yes
## Does this introduce a change in the cloud provider implementation, i...## Type of change
* [ ] Bug Fix
* [ ] Feature
* [ ] Pipeline
* [x] Test
* [ ] Documentation
## Does this introduce a change in the core logic?
* [x] No
* [ ] Yes
## Does this introduce a change in the cloud provider implementation, if so which cloud?
* [x] AWS
* [x] Anthos
* [x] Azure
* [x] GCP
* [x] IBM
## Does this follow [conventional commits spec](https://www.conventionalcommits.org/en/v1.0.0/#summary)?
* [ ] No
* [x] Yes
## Have you set the target Milestone?
* [ ] No
* [x] Yes
## Have you set the no-detached-pipeline label?
* [ ] No
* [x] Yes
## Updates description?
> Improvement of the subproject deletion e2e test in gitlabM21 - Release 0.24Pratiksha ShedgePratiksha Shedgehttps://community.opengroup.org/osdu/platform/domain-data-mgmt-services/seismic/seismic-dms-suite/seismic-store-service/-/merge_requests/892test: added e2e search and select2023-11-06T09:20:24ZPratiksha Shedgetest: added e2e search and select## Type of change
* [ ] Bug Fix
* [ ] Feature
* [ ] Pipeline
* [x] Test
* [ ] Documentation
## Does this introduce a change in the core logic?
* [x] No
* [ ] Yes
## Does this introduce a change in the cloud provider implementation, i...## Type of change
* [ ] Bug Fix
* [ ] Feature
* [ ] Pipeline
* [x] Test
* [ ] Documentation
## Does this introduce a change in the core logic?
* [x] No
* [ ] Yes
## Does this introduce a change in the cloud provider implementation, if so which cloud?
* [x] AWS
* [x] Anthos
* [x] Azure
* [x] GCP
* [x] IBM
## Does this follow [conventional commits spec](https://www.conventionalcommits.org/en/v1.0.0/#summary)?
* [ ] No
* [x] Yes
## Have you set the target Milestone?
* [ ] No
* [x] Yes
## Have you set the no-detached-pipeline label?
* [ ] No
* [x] Yes
## Updates description?
> Added End-to-End testing for search and select query for dataset list endpointM22 - Release 0.25https://community.opengroup.org/osdu/platform/domain-data-mgmt-services/seismic/seismic-dms-suite/seismic-store-service/-/merge_requests/978[For MSFT Team Review]: Trigger compute size operation on dataset close2023-11-03T10:36:46ZMaggie Salak[For MSFT Team Review]: Trigger compute size operation on dataset close## Type of change
- [ ] Bug Fix
- [x] Feature
- [ ] Pipeline
- [ ] Test
- [ ] Documentation
## Does this introduce a change in the core logic?
- [x] No
- [ ] Yes
## Does this introduce a change in the cloud provider implementation, if ...## Type of change
- [ ] Bug Fix
- [x] Feature
- [ ] Pipeline
- [ ] Test
- [ ] Documentation
## Does this introduce a change in the core logic?
- [x] No
- [ ] Yes
## Does this introduce a change in the cloud provider implementation, if so which cloud?
- [ ] AWS
- [ ] Anthos
- [x] Azure
- [ ] GCP
- [ ] IBM
## Does this follow [conventional commits spec](https://www.conventionalcommits.org/en/v1.0.0/#summary)?
- [ ] No
- [x] Yes
## Have you set the target Milestone?
- [x] No
- [ ] Yes
## Have you set the no-detached-pipeline label?
- [x] No
- [ ] Yes
## Updates description?
This MR adds a trigger to re-compute the size of a dataset after it was closed.
As per the [ADR](https://dev.azure.com/ms-slb-cobuild/Zenith/_git/Smart%20Data%20Management/pullrequest/425?_a=files&path=/adrs/2023-09-29-compute-dataset-size.md), the trigger was added to the dataset `PATCH` endpoint, in case dataset is being closed with or without an update to its metadata.
Computing the size of the dataset is an operation executed asynchronously in a separate component which will be added as part of a separate MR. The trigger creates a queue message and pushes into onto a Redis queue, where it can be picked up by the separate service.
How to test this MR?
- Run SDMS locally and register a new dataset.
- Close the dataset, by calling the `PATCH` endpoint and pass the `sbit` value of the dataset (returned when dataset is registered) in the `close` parameter. The body can be left empty.
![image](/uploads/981efbb1641378176d5a4d0f2ec308d7/image.png)
- In order to see the message, connect to the configured `queueOperations` Redis instance in a client of your choice.Maggie SalakMaggie Salakhttps://community.opengroup.org/osdu/platform/domain-data-mgmt-services/seismic/seismic-dms-suite/seismic-store-service/-/merge_requests/979test: Changes made to add Seismic file metadata unit tests to Service IT pipe...2023-11-01T22:46:56ZVaibhavi Kamanitest: Changes made to add Seismic file metadata unit tests to Service IT pipeline## Type of change
- [ ] Bug Fix
- [ ] Feature
- [ ] Pipeline
- [x] Test
- [ ] Documentation
## Does this introduce a change in the core logic?
- [x] No
- [ ] Yes
## Does this introduce a change in the cloud provider implementation, if ...## Type of change
- [ ] Bug Fix
- [ ] Feature
- [ ] Pipeline
- [x] Test
- [ ] Documentation
## Does this introduce a change in the core logic?
- [x] No
- [ ] Yes
## Does this introduce a change in the cloud provider implementation, if so which cloud?
- [ ] AWS
- [ ] Anthos
- [x] Azure
- [ ] GCP
- [ ] IBM
## Does this follow [conventional commits spec](https://www.conventionalcommits.org/en/v1.0.0/#summary)?
- [ ] No
- [x] Yes
## Have you set the target Milestone?
- [ ] No
- [ ] Yes
## Have you set the no-detached-pipeline label?
- [x] No
- [ ] Yes
## Updates description?
> Changes made to add Seismic file metadata unit tests to Service IT pipeline and make it compatible with ADO.Vaibhavi KamaniVaibhavi Kamanihttps://community.opengroup.org/osdu/platform/domain-data-mgmt-services/seismic/seismic-dms-suite/seismic-store-service/-/merge_requests/971feat: add e2e tests for bulk-delete operation2023-10-31T09:53:43ZLaura Damianfeat: add e2e tests for bulk-delete operation## Type of change
- [ ] Bug Fix
- [ ] Feature
- [ ] Pipeline
- [x] Test
- [ ] Documentation
## Does this introduce a change in the core logic?
- [x] No
- [ ] Yes
## Does this introduce a change in the cloud provider implementation, if ...## Type of change
- [ ] Bug Fix
- [ ] Feature
- [ ] Pipeline
- [x] Test
- [ ] Documentation
## Does this introduce a change in the core logic?
- [x] No
- [ ] Yes
## Does this introduce a change in the cloud provider implementation, if so which cloud?
- [ ] AWS
- [ ] Anthos
- [ ] Azure
- [ ] GCP
- [ ] IBM
## Does this follow [conventional commits spec](https://www.conventionalcommits.org/en/v1.0.0/#summary)?
- [ ] No
- [x] Yes
## Have you set the target Milestone?
- [ ] No
- [ ] Yes
## Have you set the no-detached-pipeline label?
- [ ] No
- [x] Yes
## Updates description?
This PR brings in the end to end tests for the [newly introduced hierarchical bulk deletion](https://community.opengroup.org/osdu/platform/domain-data-mgmt-services/seismic/seismic-dms-suite/seismic-store-service/-/merge_requests/891).
This supersedes [MR 902](https://community.opengroup.org/osdu/platform/domain-data-mgmt-services/seismic/seismic-dms-suite/seismic-store-service/-/merge_requests/902)M22 - Release 0.25Laura DamianLaura Damianhttps://community.opengroup.org/osdu/platform/domain-data-mgmt-services/seismic/seismic-dms-suite/seismic-store-service/-/merge_requests/951Delta changes from M16 branch to M182023-10-30T23:52:58ZVaibhavi KamaniDelta changes from M16 branch to M18## Type of change
- [ ] Bug Fix
- [ ] Feature
- [ ] Pipeline
- [ ] Test
- [ ] Documentation
- [X] Merge changes to branch
## Does this introduce a change in the core logic?
- [X] No
- [ ] Yes
## Does this introduce a change in the clou...## Type of change
- [ ] Bug Fix
- [ ] Feature
- [ ] Pipeline
- [ ] Test
- [ ] Documentation
- [X] Merge changes to branch
## Does this introduce a change in the core logic?
- [X] No
- [ ] Yes
## Does this introduce a change in the cloud provider implementation, if so which cloud?
- [ ] AWS
- [ ] Anthos
- [X] Azure
- [ ] GCP
- [ ] IBM
## Does this follow [conventional commits spec](https://www.conventionalcommits.org/en/v1.0.0/#summary)?
- [ ] No
- [X] Yes
## Have you set the target Milestone?
- [X] No
- [ ] Yes
## Have you set the no-detached-pipeline label?
- [X] No
- [ ] Yes
## Updates description?
> Merge delta changes from M16 file master branch to M18 file master branch.Vaibhavi KamaniVaibhavi Kamanihttps://community.opengroup.org/osdu/platform/domain-data-mgmt-services/seismic/seismic-dms-suite/seismic-store-service/-/merge_requests/929fix: convert SEGY to SEGY, not OpenZGY2023-10-23T10:49:19ZSacha Brantsfix: convert SEGY to SEGY, not OpenZGY## Type of change
- [x] Bug Fix
- [ ] Feature
- [ ] Pipeline
- [ ] Test
- [ ] Documentation
## Does this introduce a change in the core logic?
- [ ] No
- [x] Yes
## Does this introduce a change in the cloud provider implementation, if ...## Type of change
- [x] Bug Fix
- [ ] Feature
- [ ] Pipeline
- [ ] Test
- [ ] Documentation
## Does this introduce a change in the core logic?
- [ ] No
- [x] Yes
## Does this introduce a change in the cloud provider implementation, if so which cloud?
- [ ] AWS
- [ ] Anthos
- [ ] Azure
- [ ] GCP
- [ ] IBM
## Does this follow [conventional commits spec](https://www.conventionalcommits.org/en/v1.0.0/#summary)?
- [ ] No
- [x] Yes
## Have you set the target Milestone?
- [ ] No
- [ ] Yes
## Have you set the no-detached-pipeline label?
- [ ] No
- [ ] Yes
## Updates description?
> Update this part with a description of your changesM21 - Release 0.24https://community.opengroup.org/osdu/platform/domain-data-mgmt-services/seismic/seismic-dms-suite/seismic-store-service/-/merge_requests/932feat: [For MSFT Team Review]: advanced query filters2023-10-23T09:31:08ZAlexandre Gattikerfeat: [For MSFT Team Review]: advanced query filters[Internal PR, target branch is not `master`]
## Updates description?
First part of the implementation of advanced filters for #117.
This PR includes:
- Support for match (with operators like `=`, `LIKE`, `STARTSWITH`) and `and` opera...[Internal PR, target branch is not `master`]
## Updates description?
First part of the implementation of advanced filters for #117.
This PR includes:
- Support for match (with operators like `=`, `LIKE`, `STARTSWITH`) and `and` operator (service layer only)
Still missing, will be provided in follow-up PRs:
- Support for `or` and `not` operators
- API layer accepting filter payload
- Advanced filters in Bulk Delete operationhttps://community.opengroup.org/osdu/platform/domain-data-mgmt-services/seismic/seismic-dms-suite/seismic-store-service/-/merge_requests/955fix: aws deployment2023-10-20T13:35:01ZYunhua Koglinfix: aws deployment## Type of change
- [x] Bug Fix
- [ ] Feature
- [ ] Pipeline
- [ ] Test
- [ ] Documentation
## Does this introduce a change in the core logic?
- [x] No
- [ ] Yes
## Does this introduce a change in the cloud provider implementation, if ...## Type of change
- [x] Bug Fix
- [ ] Feature
- [ ] Pipeline
- [ ] Test
- [ ] Documentation
## Does this introduce a change in the core logic?
- [x] No
- [ ] Yes
## Does this introduce a change in the cloud provider implementation, if so which cloud?
- [x] AWS
- [ ] Anthos
- [ ] Azure
- [ ] GCP
- [ ] IBM
## Does this follow [conventional commits spec](https://www.conventionalcommits.org/en/v1.0.0/#summary)?
- [ ] No
- [x] Yes
## Have you set the target Milestone?
- [ ] No
- [x] Yes
## Have you set the no-detached-pipeline label?
- [ ] No
- [x] Yes
## Updates description?
> Update this part with a description of your changesM22 - Release 0.25Yunhua KoglinYunhua Koglinhttps://community.opengroup.org/osdu/platform/domain-data-mgmt-services/seismic/seismic-dms-suite/seismic-store-service/-/merge_requests/944fix: list content utility2023-10-19T14:52:41ZDaniel Perezfix: list content utility## Type of change
- [X] Bug Fix
- [ ] Feature
- [ ] Pipeline
- [ ] Test
- [ ] Documentation
## Does this introduce a change in the core logic?
- [X] No
- [ ] Yes
## Does this introduce a change in the cloud provider implementation, if ...## Type of change
- [X] Bug Fix
- [ ] Feature
- [ ] Pipeline
- [ ] Test
- [ ] Documentation
## Does this introduce a change in the core logic?
- [X] No
- [ ] Yes
## Does this introduce a change in the cloud provider implementation, if so which cloud?
- [ ] AWS
- [ ] Anthos
- [ ] Azure
- [ ] GCP
- [ ] IBM
## Does this follow [conventional commits spec](https://www.conventionalcommits.org/en/v1.0.0/#summary)?
- [ ] No
- [X] Yes
## Have you set the target Milestone?
- [ ] No
- [X] Yes
## Have you set the no-detached-pipeline label?
- [ ] No
- [X] Yes
## Updates description?
> Update logic for list utility so it will list only datasets when in root path, other way is going to list folderM22 - Release 0.25Daniel PerezDaniel Perezhttps://community.opengroup.org/osdu/platform/domain-data-mgmt-services/seismic/seismic-dms-suite/seismic-store-service/-/merge_requests/931chore: [For MSFT Team Review]: Refactor getDatasetsQuery to remove code dupli...2023-10-19T12:17:52ZAlexandre Gattikerchore: [For MSFT Team Review]: Refactor getDatasetsQuery to remove code duplication[Internal PR, target branch is not master]
## Updates description?
The code of the getDatasetsQuery was duplicated between the abstract Journal class and its Cosmos DB implementation, which added some additional behaviour.
This duplic...[Internal PR, target branch is not master]
## Updates description?
The code of the getDatasetsQuery was duplicated between the abstract Journal class and its Cosmos DB implementation, which added some additional behaviour.
This duplication creates technical debt and complicates the implementation of additional behaviour such as #117.
Refactored to remove duplication, adding an empty method `customizeDatasetsQuery` in the abstract class which is overridden in the Cosmos DB implementation.
Other storage providers have the option to implement the `STARTSWITH` operator to support `recursive=true`.https://community.opengroup.org/osdu/platform/domain-data-mgmt-services/seismic/seismic-dms-suite/seismic-store-service/-/merge_requests/949fix: modify code to compliant with audit finding2023-10-19T11:26:05ZJason Wangfix: modify code to compliant with audit finding## Type of change
- [X] Bug Fix
- [ ] Feature
- [ ] Pipeline
- [ ] Test
- [ ] Documentation
## Does this introduce a change in the core logic?
- [X] No
- [ ] Yes
## Does this introduce a change in the cloud provider implementation, if ...## Type of change
- [X] Bug Fix
- [ ] Feature
- [ ] Pipeline
- [ ] Test
- [ ] Documentation
## Does this introduce a change in the core logic?
- [X] No
- [ ] Yes
## Does this introduce a change in the cloud provider implementation, if so which cloud?
- [ ] AWS
- [ ] Anthos
- [ ] Azure
- [ ] GCP
- [ ] IBM
## Does this follow [conventional commits spec](https://www.conventionalcommits.org/en/v1.0.0/#summary)?
- [ ] No
- [x] Yes
## Have you set the target Milestone?
- [ ] No
- [x] Yes
## Have you set the no-detached-pipeline label?
- [ ] No
- [X] Yes
## Updates description?
> Update this part with a description of your changesM22 - Release 0.25Jason WangJason Wanghttps://community.opengroup.org/osdu/platform/domain-data-mgmt-services/seismic/seismic-dms-suite/seismic-store-service/-/merge_requests/956fix: copy dockerfile contents2023-10-18T20:57:17ZNathan Strelserfix: copy dockerfile contents## Type of change
- [ ] Bug Fix
- [ ] Feature
- [ ] Pipeline
- [ ] Test
- [ ] Documentation
## Does this introduce a change in the core logic?
- [ ] No
- [ ] Yes
## Does this introduce a change in the cloud provider implementation, if ...## Type of change
- [ ] Bug Fix
- [ ] Feature
- [ ] Pipeline
- [ ] Test
- [ ] Documentation
## Does this introduce a change in the core logic?
- [ ] No
- [ ] Yes
## Does this introduce a change in the cloud provider implementation, if so which cloud?
- [ ] AWS
- [ ] Anthos
- [ ] Azure
- [ ] GCP
- [ ] IBM
## Does this follow [conventional commits spec](https://www.conventionalcommits.org/en/v1.0.0/#summary)?
- [ ] No
- [ ] Yes
## Have you set the target Milestone?
- [ ] No
- [ ] Yes
## Have you set the no-detached-pipeline label?
- [ ] No
- [ ] Yes
## Updates description?
> Update this part with a description of your changesNathan StrelserNathan Strelserhttps://community.opengroup.org/osdu/platform/domain-data-mgmt-services/seismic/seismic-dms-suite/seismic-store-service/-/merge_requests/950fix: improve sdms v3 codequality2023-10-18T19:22:40ZKajal Sawalefix: improve sdms v3 codequality## Type of change
* [ ] Bug Fix
* [ ] Feature
* [ ] Pipeline
* [ ] Test
* [ ] Documentation
* [x] Enhancement
## Does this introduce a change in the core logic?
* [x] No
* [ ] Yes
## Does this introduce a change in the cloud provider...## Type of change
* [ ] Bug Fix
* [ ] Feature
* [ ] Pipeline
* [ ] Test
* [ ] Documentation
* [x] Enhancement
## Does this introduce a change in the core logic?
* [x] No
* [ ] Yes
## Does this introduce a change in the cloud provider implementation, if so which cloud?
* [ ] AWS
* [ ] Anthos
* [ ] Azure
* [ ] GCP
* [ ] IBM
## Does this follow [conventional commits spec](https://www.conventionalcommits.org/en/v1.0.0/#summary)?
* [ ] No
* [x] Yes
## Have you set the target Milestone?
* [ ] No
* [x] Yes
## Have you set the no-detached-pipeline label?
* [ ] No
* [x] Yes
## Updates description?
> Updated code to remove complexities and code smellsM22 - Release 0.25Kajal SawaleKajal Sawale