Partition merge requestshttps://community.opengroup.org/osdu/platform/system/partition/-/merge_requests2021-09-03T10:31:36Zhttps://community.opengroup.org/osdu/platform/system/partition/-/merge_requests/88Updating OSDU dependencies2021-09-03T10:31:36ZDavid Diederichd.diederich@opengroup.orgUpdating OSDU dependenciesUpdating OSDU dependencies, to maintain use of the latest release among those that were previously doing soUpdating OSDU dependencies, to maintain use of the latest release among those that were previously doing soM8 - Release 0.11David Diederichd.diederich@opengroup.orgDavid Diederichd.diederich@opengroup.orghttps://community.opengroup.org/osdu/platform/system/partition/-/merge_requests/85FIX AWS CI2023-08-18T15:32:40ZMatt WiseFIX AWS CIM8 - Release 0.11Matt WiseMatt Wisehttps://community.opengroup.org/osdu/platform/system/partition/-/merge_requests/84Move to EKS2023-08-18T15:32:42ZMatt WiseMove to EKScommit 11ae9581
Author: Spencer Sutton <suttonsp@amazon.com>
Date: Mon Jul 19 2021 11:50:06 GMT-0500 (Central Daylight Time)
Fixing build
commit e9bd8727
Author: Spencer Sutton <suttonsp@amazon.com>
Date: Mon Jul 19 2021 10:4...commit 11ae9581
Author: Spencer Sutton <suttonsp@amazon.com>
Date: Mon Jul 19 2021 11:50:06 GMT-0500 (Central Daylight Time)
Fixing build
commit e9bd8727
Author: Spencer Sutton <suttonsp@amazon.com>
Date: Mon Jul 19 2021 10:43:12 GMT-0500 (Central Daylight Time)
Adding sonar,dependency-check to awsM8 - Release 0.11Matt WiseMatt Wisehttps://community.opengroup.org/osdu/platform/system/partition/-/merge_requests/83Using Istio auth in Partition Service2023-08-18T15:32:44ZVibhuti Sharma [Microsoft]Using Istio auth in Partition ServiceFor partition service to authenticate managed identity, we are enabling it to use istio rules like other services. However, partition service only authorizes token of type service principal, not user or guest. To make sure this behavior ...For partition service to authenticate managed identity, we are enabling it to use istio rules like other services. However, partition service only authorizes token of type service principal, not user or guest. To make sure this behavior is not affected, a filter is added which sets the authentication context. This context is retrieved by the AuthorizationService to check the user type.
A check is also added on the issuer, to make sure only issuer with v1 or v2 aad url is accepted.
Testing
* Tested locally by passing different claims values in "x-payload" header and seeing response as expected.M8 - Release 0.11Vibhuti Sharma [Microsoft]Vibhuti Sharma [Microsoft]https://community.opengroup.org/osdu/platform/system/partition/-/merge_requests/80Added Service Version Endpoint (GONRG-2886)2022-09-16T08:46:58ZAnastasiia GelmutAdded Service Version Endpoint (GONRG-2886)## Type of change
- [ ] Bug Fix
- [x] Feature
osdu/platform/system/lib/core/os-core-common#47
## Does this introduce a change in the core logic?
- [YES]
## Does this introduce a change in the cloud provider implementation, if so whic...## Type of change
- [ ] Bug Fix
- [x] Feature
osdu/platform/system/lib/core/os-core-common#47
## Does this introduce a change in the core logic?
- [YES]
## Does this introduce a change in the cloud provider implementation, if so which cloud?
- [x] AWS
- [x] Azure
- [x] GCP
- [x] IBM
## Does this introduce a breaking change?
- [YES]
## What is the current behavior?
Provides info about maven build and gitM8 - Release 0.11Rustam Lotsmanenko (EPAM)rustam_lotsmanenko@epam.comRostislav Dublin (EPAM)Rustam Lotsmanenko (EPAM)rustam_lotsmanenko@epam.comhttps://community.opengroup.org/osdu/platform/system/partition/-/merge_requests/79Update resources limits [GONRG-2974]2023-08-18T15:32:46ZMikhail Piatliou (EPAM)Update resources limits [GONRG-2974]M8 - Release 0.11Oleksandr Kosse (EPAM)Oleksandr Kosse (EPAM)https://community.opengroup.org/osdu/platform/system/partition/-/merge_requests/78Configurable partition admin accounts (GONRG-2687)2021-08-02T18:28:27ZRustam Lotsmanenko (EPAM)rustam_lotsmanenko@epam.comConfigurable partition admin accounts (GONRG-2687)# Description:
Partition admin account now can be provided as list of accounts
`PARTITION_ADMIN_ACCOUNTS=admin@domen.iam.gserviceaccount.com,osdu-gcp-sa,workload-identity`
# Changes include:
- [x] New feature (a non-breaking change tha...# Description:
Partition admin account now can be provided as list of accounts
`PARTITION_ADMIN_ACCOUNTS=admin@domen.iam.gserviceaccount.com,osdu-gcp-sa,workload-identity`
# Changes include:
- [x] New feature (a non-breaking change that adds functionality).
# Changes in:
- [x] GCP
# Dev Checklist:
- [x] Added Unit Tests, wherever applicable.
- [x] Updated the Readme, if applicable.
- [x] Existing Tests pass
- [x] Verified functionality locally
- [x] Self Reviewed my code for formatting and complex business logic.M8 - Release 0.11Rostislav Dublin (EPAM)Rostislav Dublin (EPAM)https://community.opengroup.org/osdu/platform/system/partition/-/merge_requests/77Added rollme annotation2023-08-18T15:32:47ZOleksii Tsyganov (EPAM)Added rollme annotationM8 - Release 0.11Oleksii Tsyganov (EPAM)Oleksii Tsyganov (EPAM)https://community.opengroup.org/osdu/platform/system/partition/-/merge_requests/76Revert "Merge branch 'Migrate-partition-to-GKE' into 'master'"2023-08-18T15:32:49ZOleksii Tsyganov (EPAM)Revert "Merge branch 'Migrate-partition-to-GKE' into 'master'"This reverts merge request !75This reverts merge request !75M8 - Release 0.11Oleksii Tsyganov (EPAM)Oleksii Tsyganov (EPAM)https://community.opengroup.org/osdu/platform/system/partition/-/merge_requests/75Added required helm variables2023-08-18T15:32:51ZOleksii Tsyganov (EPAM)Added required helm variablesM8 - Release 0.11Oleksii Tsyganov (EPAM)Oleksii Tsyganov (EPAM)https://community.opengroup.org/osdu/platform/system/partition/-/merge_requests/60Add node selector2023-08-18T15:33:10ZDzmitry_Paulouski (slb)Add node selector## All Submissions:
-------------------------------------
* [YES] Have you followed our code review [guidelines](https://github.com/microsoft/code-with-engineering-playbook/blob/master/pull-requests/code-reviews/readme.md)?
* [YES] Have ...## All Submissions:
-------------------------------------
* [YES] Have you followed our code review [guidelines](https://github.com/microsoft/code-with-engineering-playbook/blob/master/pull-requests/code-reviews/readme.md)?
* [YES] Have you added an explanation of what your changes do and why you'd like us to include them?
* [NO] I have updated the documentation accordingly.
* [NA] I have added tests to cover my changes.
* [YES] 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.
## What is the current behavior?
-------------------------------------
Currently, we have only one node pool for services and system workloads.
## What is the new behavior?
-------------------------------------
We are going to move workloads to a new node pool with enabled autoscaling. Here you can find the details:
[Issue link](https://community.opengroup.org/osdu/platform/deployment-and-operations/infra-azure-provisioning/-/issues/167)
## Does this introduce a breaking change?
-------------------------------------
- [NO]
This change has a strong dependency on the following MR and should be merged **only** after it:
https://community.opengroup.org/osdu/platform/deployment-and-operations/infra-azure-provisioning/-/merge_requests/324M8 - Release 0.11https://community.opengroup.org/osdu/platform/system/partition/-/merge_requests/108Upgrade OSDU dependencies2021-10-26T04:30:44ZDavid Diederichd.diederich@opengroup.orgUpgrade OSDU dependenciesM9 - Release 0.12David Diederichd.diederich@opengroup.orgDavid Diederichd.diederich@opengroup.orghttps://community.opengroup.org/osdu/platform/system/partition/-/merge_requests/105update core lib and common - Partition2023-08-18T15:32:24ZNikhil Singh[MicroSoft]update core lib and common - Partition**Introduction**
This MR upgrades the os core-common and core lib version for azure provider to verify the recent security fixes made in the below mentioned MR are working properly.
**Issue Link**
https://community.opengroup.org/osdu/...**Introduction**
This MR upgrades the os core-common and core lib version for azure provider to verify the recent security fixes made in the below mentioned MR are working properly.
**Issue Link**
https://community.opengroup.org/osdu/platform/system/lib/core/os-core-common/-/merge_requests/119M9 - Release 0.12Nikhil Singh[MicroSoft]Nikhil Singh[MicroSoft]https://community.opengroup.org/osdu/platform/system/partition/-/merge_requests/101Added logback into GCP IT module (GONRG-3445)2021-10-08T10:34:02ZRiabokon Stanislav(EPAM)[GCP]Added logback into GCP IT module (GONRG-3445)## Type of change
- [X] Bug Fix
- [ ] Feature
## Does this introduce a change in the core logic?
- [NO]
## Does this introduce a change in the cloud provider implementation, if so which cloud?
- [ ] AWS
- [ ] Azure
- [X] GCP
- [ ] IBM...## Type of change
- [X] Bug Fix
- [ ] Feature
## Does this introduce a change in the core logic?
- [NO]
## Does this introduce a change in the cloud provider implementation, if so which cloud?
- [ ] AWS
- [ ] Azure
- [X] GCP
- [ ] IBM
## Does this introduce a breaking change?
- [NO]
## What is the current behavior?
The id_token is exposed in int-tests logs
## What is the new/expected behavior?
The id_token logs are disabledM9 - Release 0.12Riabokon Stanislav(EPAM)[GCP]Riabokon Stanislav(EPAM)[GCP]https://community.opengroup.org/osdu/platform/system/partition/-/merge_requests/100Integration Test Implement Clean-up for Datastore (GONRG-3532)2021-10-07T13:49:05ZAnastasiia GelmutIntegration Test Implement Clean-up for Datastore (GONRG-3532)# Description:
Implemented clean-up for Datastore.
# How to test:
Partition service can be tested with the help of business cases.
# Changes include:
- [ ] Refactor (a non-breaking change that improves code maintainability).
- [x] Bugf...# Description:
Implemented clean-up for Datastore.
# How to test:
Partition service can be tested with the help of business cases.
# Changes include:
- [ ] Refactor (a non-breaking change that improves code maintainability).
- [x] Bugfix (a non-breaking change that solves an issue).
- [ ] New feature (a non-breaking change that adds functionality).
- [ ] Breaking change (a change that is not backward-compatible and/or changes current functionality).
# Changes in
- [ ] Common code
- [x] GCP
- [ ] Azure
- [ ] AWS
- [ ] IBM
# Dev Checklist:
* [ ] Added Unit Tests, wherever applicable.
* [ ] Updated the Readme, if applicable.
* [x] Existing Tests pass
* [x] Verified functionality locally
* [x] Self Reviewed my code for formatting and complex business logic.M9 - Release 0.12Riabokon Stanislav(EPAM)[GCP]Rostislav Dublin (EPAM)Riabokon Stanislav(EPAM)[GCP]https://community.opengroup.org/osdu/platform/system/partition/-/merge_requests/99Update pom.xml2023-08-18T15:32:25ZKrishna Nikhil VedurumudiUpdate pom.xmlPom file clean up to have it consistent with other services.Pom file clean up to have it consistent with other services.M9 - Release 0.12Krishna Nikhil VedurumudiKrishna Nikhil Vedurumudihttps://community.opengroup.org/osdu/platform/system/partition/-/merge_requests/98Setup hpa for partition service2022-12-22T15:10:19ZKelly ZhouSetup hpa for partition serviceFixes https://community.opengroup.org/osdu/platform/system/partition/-/issues/21Fixes https://community.opengroup.org/osdu/platform/system/partition/-/issues/21M9 - Release 0.12https://community.opengroup.org/osdu/platform/system/partition/-/merge_requests/97CloudTableClient has no timeouts2023-08-18T15:32:27ZDmitrii GerashchenkoCloudTableClient has no timeoutshttps://community.opengroup.org/osdu/platform/system/partition/-/issues/19
Configured timeouts for CloudTableClient.
---
MS TableStorage's client - CloudTableClient uses default timeout settings.
The client can try to connect to the MS...https://community.opengroup.org/osdu/platform/system/partition/-/issues/19
Configured timeouts for CloudTableClient.
---
MS TableStorage's client - CloudTableClient uses default timeout settings.
The client can try to connect to the MS server for up to 2 minutes: 3 retry attempts with 30 seconds delay between attempts.
The MaximumExecutionTime is null. I created a dummy server and tested the case when MS TableStorage responds with latency. There is no timeout for a response in the client so the client could be blocked infinitely. The client doesn't throw errors on long TableStorage's latencies what could be the cause of 504 errors for API consumers. Also, it means that we can't see any exceptions even if MS TableStorage responds with latencies.M9 - Release 0.12Dmitrii GerashchenkoDmitrii Gerashchenkohttps://community.opengroup.org/osdu/platform/system/partition/-/merge_requests/95Update helms for Partition [GONRG-3403]2023-08-18T15:32:30ZMikhail Piatliou (EPAM)Update helms for Partition [GONRG-3403]M9 - Release 0.12Oleksandr Kosse (EPAM)Oleksandr Kosse (EPAM)https://community.opengroup.org/osdu/platform/system/partition/-/merge_requests/94Add memory limits2023-08-18T15:32:32ZKelly ZhouAdd memory limitsFixes https://community.opengroup.org/osdu/platform/system/partition/-/issues/17Fixes https://community.opengroup.org/osdu/platform/system/partition/-/issues/17M9 - Release 0.12