OS Core Lib Azure merge requestshttps://community.opengroup.org/osdu/platform/system/lib/cloud/azure/os-core-lib-azure/-/merge_requests2022-07-12T15:42:56Zhttps://community.opengroup.org/osdu/platform/system/lib/cloud/azure/os-core-lib-azure/-/merge_requests/177Draft: Cosmos Query : Handle variable page size from cosmosDb2022-07-12T15:42:56ZVibhuti Sharma [Microsoft]Draft: Cosmos Query : Handle variable page size from cosmosDbVibhuti Sharma [Microsoft]Vibhuti Sharma [Microsoft]https://community.opengroup.org/osdu/platform/system/lib/cloud/azure/os-core-lib-azure/-/merge_requests/172Draft: retry test code changes in core lib azure2022-07-12T15:42:55ZSHEFFALI JAINDraft: retry test code changes in core lib azureSHEFFALI JAINSHEFFALI JAINhttps://community.opengroup.org/osdu/platform/system/lib/cloud/azure/os-core-lib-azure/-/merge_requests/167Fixed log4j CVE-2021-442282021-12-14T17:12:07ZKhilesh SahuFixed log4j CVE-2021-44228Upgraded log4j-core and log4j-api to 2.15.0 as direct dependency
Default spring boot was using 2.13.0, we are upgrading to 2.15.0.
Issue link : https://community.opengroup.org/osdu/platform/system/lib/cloud/azure/os-core-lib-azure/-/is...Upgraded log4j-core and log4j-api to 2.15.0 as direct dependency
Default spring boot was using 2.13.0, we are upgrading to 2.15.0.
Issue link : https://community.opengroup.org/osdu/platform/system/lib/cloud/azure/os-core-lib-azure/-/issues/13https://community.opengroup.org/osdu/platform/system/lib/cloud/azure/os-core-lib-azure/-/merge_requests/164Draft: Upgrade vulnerable dependencies2021-11-16T14:41:13ZDmitrii GerashchenkoDraft: Upgrade vulnerable dependencieshttps://community.opengroup.org/osdu/platform/security-and-compliance/entitlements/-/issues/88
**Summary**
WhiteSource's vulnerabilities list contain alerts about:
- spring-web-5.3.6.jar
- netty-codec-4.1.63.Final.jar
- netty-codec-4.1...https://community.opengroup.org/osdu/platform/security-and-compliance/entitlements/-/issues/88
**Summary**
WhiteSource's vulnerabilities list contain alerts about:
- spring-web-5.3.6.jar
- netty-codec-4.1.63.Final.jar
- netty-codec-4.1.63.Final.jar
- spring-security-oauth2-client-5.4.6.jar
- netty-all-4.1.63.Final.jar
- netty-handler-4.1.63.Final.jar
Updated:
- spring-boot.version from 2.4.[-5-] to 2.4.[+12+]
- netty-bom.version form 4.1.[-63-].Final to 4.1.[+70+].Final
**Related MRs:**
- Entitlements
- MR: https://community.opengroup.org/osdu/platform/security-and-compliance/entitlements/-/merge_requests/143
- Trusted branch pipeline is passed: https://community.opengroup.org/osdu/platform/security-and-compliance/entitlements/-/pipelines/76742
- os-core-common
- MR: https://community.opengroup.org/osdu/platform/system/lib/core/os-core-common/-/merge_requests/124
- Trusted branch pipeline is passed: https://community.opengroup.org/osdu/platform/system/lib/core/os-core-common/-/pipelines/76515
- os-core-lib-azure
- MR: https://community.opengroup.org/osdu/platform/system/lib/cloud/azure/os-core-lib-azure/-/merge_requests/164
- Trusted branch pipeline is passed: https://community.opengroup.org/osdu/platform/system/lib/cloud/azure/os-core-lib-azure/-/pipelines/76697Dmitrii GerashchenkoDmitrii Gerashchenkohttps://community.opengroup.org/osdu/platform/system/lib/cloud/azure/os-core-lib-azure/-/merge_requests/163Draft: Override vulnerable netty version <4.1.682021-11-09T16:40:57ZDmitrii GerashchenkoDraft: Override vulnerable netty version <4.1.68Dmitrii GerashchenkoDmitrii Gerashchenkohttps://community.opengroup.org/osdu/platform/system/lib/cloud/azure/os-core-lib-azure/-/merge_requests/151Draft: expose CosmosQueryRequestOptions for pagination query2022-03-16T17:10:28ZNeelesh ThakurDraft: expose CosmosQueryRequestOptions for pagination queryhttps://community.opengroup.org/osdu/platform/system/lib/cloud/azure/os-core-lib-azure/-/merge_requests/146Removing Autowire for builder2022-03-16T17:10:28ZKomal MakkarRemoving Autowire for builderEG and SB are trying to autowire a builder which fails when the flags are set.EG and SB are trying to autowire a builder which fails when the flags are set.https://community.opengroup.org/osdu/platform/system/lib/cloud/azure/os-core-lib-azure/-/merge_requests/144Remove conditional beans2021-09-17T07:15:45ZNikhil Singh[MicroSoft]Remove conditional beansThis MR removes the conditional bean property from message broker beans because the MessagePublisher Bean uses both service bus and event grid beans.This MR removes the conditional bean property from message broker beans because the MessagePublisher Bean uses both service bus and event grid beans.Nikhil Singh[MicroSoft]Nikhil Singh[MicroSoft]https://community.opengroup.org/osdu/platform/system/lib/cloud/azure/os-core-lib-azure/-/merge_requests/139Draft: Msrivastava/resiliency extension legal2022-02-03T15:11:13ZMuskan SrivastavaDraft: Msrivastava/resiliency extension legal## All Submissions:
-------------------------------------
* I have added an explanation of what changes in this merge do and why we should include it? - YES
* Does the MR contain pipeline/ helm chart related changes?- NO
* I have updated...## All Submissions:
-------------------------------------
* I have added an explanation of what changes in this merge do and why we should include it? - YES
* Does the MR contain pipeline/ helm chart related changes?- NO
* I have updated the documentation accordingly. - NO
* I have added tests to cover my changes. -NA
* All new and existing tests passed. -YES
* My code follows the code style of this project. -YES
* I ran lint checks locally prior to submission. -YES
## What is the issue or story related to the change?
-------------------------------------
This change is a step forward to enable the services to configure following variables for service-to-service communication:
- Circuit Breaker
- Retry count for service unavailable strategy
- Connect timeout
- Connection Request Timeout
- Socket Timeout
<br>
All the above features are handled by HttpClientAzure class.
This MR essentially enables use of HttpClientAzure for legal service.
These configurations can be used for legal service now.
<br>
High level design: This MR adds 2 classes in total. <br>
- LegalAPIConfigAzure : creates a bean of type LegalAPIConfig
- LegalFactoryAzure : uses newly created HttpClientAzure internally so that configurable retires, CB and timeouts can be used for Legal Service.
## Does this introduce a breaking change?
-------------------------------------
- NO
<!-- If this introduces a breaking change, please describe the impact and migration path for existing applications below. -->
## Tests and Results
To test the CB and timeouts, I used the scenario where storage service calls legal service. Simulations were triggered by Firefly Flight
1. Circuit Breaker: To test this legal service was killed. Result is as follows![Screenshot__29_](/uploads/b3398d97c9b06a574209eeddf28d3231/Screenshot__29_.png)
2. Timeouts: To test this Socket Timeouts was configured to be 40 sec for any calls made to legal service. Latency was induced to legal service between the range of 45 sec and 50 sec.
![Screenshot__30_](/uploads/66d4191433187731e39be420c98fea5a/Screenshot__30_.png)Muskan SrivastavaMuskan Srivastavahttps://community.opengroup.org/osdu/platform/system/lib/cloud/azure/os-core-lib-azure/-/merge_requests/136Draft: Rosakhuj/cb22022-02-03T15:11:14ZRonak SakhujaDraft: Rosakhuj/cb2https://community.opengroup.org/osdu/platform/system/lib/cloud/azure/os-core-lib-azure/-/merge_requests/135Draft: Making changes to test cb2021-08-11T19:50:24ZRonak SakhujaDraft: Making changes to test cbhttps://community.opengroup.org/osdu/platform/system/lib/cloud/azure/os-core-lib-azure/-/merge_requests/130Update common.properties to enable health check bean2022-01-11T13:24:30ZSmitha ManjunathUpdate common.properties to enable health check bean## All Submissions:
-------------------------------------
* [YES/NO] I have added an explanation of what changes in this merge do and why we should include it?
* [YES/NO] I have updated the documentation accordingly.
* [YES/NO/NA] I have...## All Submissions:
-------------------------------------
* [YES/NO] I have added an explanation of what changes in this merge do and why we should include it?
* [YES/NO] I have updated the documentation accordingly.
* [YES/NO/NA] I have added tests to cover my changes.
* [YES/NO/NA] All new and existing tests passed.
* [YES/NO/NA] My code follows the code style of this project.
* [YES/NO/NA] I ran lint checks locally prior to submission.
## What is the issue or story related to the change?
-------------------------------------
<!-- Please describe the current behavior that you are modifying, 'or' link to a relevant issue.
Feel free to add references to any design documents you might have shared with the team or any
related MR that you are building on top of. -->
High level design:
Issue: <!-- Link any __GitLab__ workitem(s) to this pull request. -->
<!-- Please add implementation details of current set of changes and how the code changes are
doing what they are expected to do. Are there any complex loops or designated code blocks that
should be elaborated? Is there some contextual knowledge that the reviewer should be aware of? -->
Change details:
## Test coverage:
------------------
<!-- Mention unit test coverage of changes. -->
## Does this introduce a breaking change?
-------------------------------------
- [YES/NO]
<!-- If this introduces a breaking change, please describe the impact and migration path for existing applications below. -->
## Pending items
----------------
<!-- Are there changes that you'll introduce in upcoming MRs and hence did not add in this one? Next steps of your
feature can also be mentioned here. -->
## Reviewer request
-------------------
- Please provide an ETA when you plan to review this MR. Write a comment to decline or provide an ETA.
- Block the MR if you feel there is less testing or no details in the MR
- Please cover the following aspects in the MR
-- Coding design: _\<Reviewer1>_
-- Backward Compatibility: _\<Reviewer2>_
-- Feature Logic: _\<Logic design\>_
-- _\<Any other context mention here>_
OR
-- _\<Component 1>_: _\<Reviewer1>_
-- _\<CosmosDB>_: _\<Reviewer2>_
-- _\<ServiceBus>_ _\<Reviewer3>_
-- _\<Mention any other component and owner>_
## Other information
-------------------------------------
<!-- Any other information that is important to this MR such as screenshots of how the component looks before and after the change. -->Smitha ManjunathSmitha Manjunathhttps://community.opengroup.org/osdu/platform/system/lib/cloud/azure/os-core-lib-azure/-/merge_requests/127Added docs - updated notice file2022-01-11T13:24:34ZSmitha ManjunathAdded docs - updated notice file## All Submissions:
-------------------------------------
* [YES/NO] I have added an explanation of what changes in this merge do and why we should include it?
* [YES/NO] I have updated the documentation accordingly.
* [YES/NO/NA] I have...## All Submissions:
-------------------------------------
* [YES/NO] I have added an explanation of what changes in this merge do and why we should include it?
* [YES/NO] I have updated the documentation accordingly.
* [YES/NO/NA] I have added tests to cover my changes.
* [YES/NO/NA] All new and existing tests passed.
* [YES/NO/NA] My code follows the code style of this project.
* [YES/NO/NA] I ran lint checks locally prior to submission.
## What is the issue or story related to the change?
-------------------------------------
<!-- Please describe the current behavior that you are modifying, 'or' link to a relevant issue.
Feel free to add references to any design documents you might have shared with the team or any
related MR that you are building on top of. -->
High level design:
Issue: <!-- Link any __GitLab__ workitem(s) to this pull request. -->
<!-- Please add implementation details of current set of changes and how the code changes are
doing what they are expected to do. Are there any complex loops or designated code blocks that
should be elaborated? Is there some contextual knowledge that the reviewer should be aware of? -->
Change details:
## Test coverage:
------------------
<!-- Mention unit test coverage of changes. -->
## Does this introduce a breaking change?
-------------------------------------
- [YES/NO]
<!-- If this introduces a breaking change, please describe the impact and migration path for existing applications below. -->
## Pending items
----------------
<!-- Are there changes that you'll introduce in upcoming MRs and hence did not add in this one? Next steps of your
feature can also be mentioned here. -->
## Reviewer request
-------------------
- Please provide an ETA when you plan to review this MR. Write a comment to decline or provide an ETA.
- Block the MR if you feel there is less testing or no details in the MR
- Please cover the following aspects in the MR
-- Coding design: _\<Reviewer1>_
-- Backward Compatibility: _\<Reviewer2>_
-- Feature Logic: _\<Logic design\>_
-- _\<Any other context mention here>_
OR
-- _\<Component 1>_: _\<Reviewer1>_
-- _\<CosmosDB>_: _\<Reviewer2>_
-- _\<ServiceBus>_ _\<Reviewer3>_
-- _\<Mention any other component and owner>_
## Other information
-------------------------------------
<!-- Any other information that is important to this MR such as screenshots of how the component looks before and after the change. -->Smitha ManjunathSmitha Manjunathhttps://community.opengroup.org/osdu/platform/system/lib/cloud/azure/os-core-lib-azure/-/merge_requests/123Draft: Made changes for adding cache staleness for the new version of core-li...2022-01-11T13:24:34ZAbhiram BondadaDraft: Made changes for adding cache staleness for the new version of core-lib-azurehttps://community.opengroup.org/osdu/platform/system/lib/cloud/azure/os-core-lib-azure/-/merge_requests/122Draft: Added Cache parameter2021-06-26T10:54:12ZAbhiram BondadaDraft: Added Cache parametermade changes to enable gateway mode, with changes in PartitionInfoAzure using additional getters for gateway endpointmade changes to enable gateway mode, with changes in PartitionInfoAzure using additional getters for gateway endpointhttps://community.opengroup.org/osdu/platform/system/lib/cloud/azure/os-core-lib-azure/-/merge_requests/120[WIP] Added support for File Share in core lib azure2021-06-24T06:17:49ZAalekh Jain[WIP] Added support for File Share in core lib azureThis MR targets to add the implementation for File Share in core lib azure.
cc: @vineethguna , @kibattulThis MR targets to add the implementation for File Share in core lib azure.
cc: @vineethguna , @kibattulhttps://community.opengroup.org/osdu/platform/system/lib/cloud/azure/os-core-lib-azure/-/merge_requests/117Draft: Added Initial configuration for circuitbreakers2021-06-29T05:25:26ZRonak SakhujaDraft: Added Initial configuration for circuitbreakershttps://community.opengroup.org/osdu/platform/system/lib/cloud/azure/os-core-lib-azure/-/merge_requests/116Extending cosmosStore class to support CRUD on system cosmos db2021-06-21T12:28:14ZAman VermaExtending cosmosStore class to support CRUD on system cosmos dbProblem:
====
The `cosmosStore` class today is the facade to perform CRUD on data partition related cosmos dbs. All the methods in this class are tightly coupled with data-partition-id. Since the system data resources won't be coupled wi...Problem:
====
The `cosmosStore` class today is the facade to perform CRUD on data partition related cosmos dbs. All the methods in this class are tightly coupled with data-partition-id. Since the system data resources won't be coupled with a data-partition-id, the existing facade can't support CRUD on system resources.
Changes in MR:
===
1. Enhanced the current CosmosClientFactory to return system level cosmos clients too.
2. Overloaded existing methods. The new methods don't accept data-partition-id parameter
3. Minor code refactoring
Abandoning this MR. Find new MR here: https://community.opengroup.org/osdu/platform/system/lib/cloud/azure/os-core-lib-azure/-/merge_requests/118Aman VermaAman Vermahttps://community.opengroup.org/osdu/platform/system/lib/cloud/azure/os-core-lib-azure/-/merge_requests/115Draft: Gateway mode enabled2021-06-26T08:27:52ZAbhiram BondadaDraft: Gateway mode enabledGateway mode enabled in "createCosmosClient" method of CosmosClientFactoryImplGateway mode enabled in "createCosmosClient" method of CosmosClientFactoryImplAbhiram BondadaAbhiram Bondadahttps://community.opengroup.org/osdu/platform/system/lib/cloud/azure/os-core-lib-azure/-/merge_requests/114Draft: Enabling Integrated gateway cache2021-06-14T14:37:46ZAbhiram BondadaDraft: Enabling Integrated gateway cacheAdding gateway mode to "createCosmosClient" in "CosmosClientFactoryImpl.java"Adding gateway mode to "createCosmosClient" in "CosmosClientFactoryImpl.java"Abhiram BondadaAbhiram Bondada