Skip to content
GitLab
Projects Groups Topics Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Register
  • Sign in
  • O OS Core Lib Azure
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributor statistics
    • Graph
    • Compare revisions
    • Locked files
  • Issues 8
    • Issues 8
    • List
    • Boards
    • Service Desk
    • Milestones
    • Iterations
    • Requirements
  • Merge requests 8
    • Merge requests 8
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
    • Test cases
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Package Registry
    • Container Registry
    • Infrastructure Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Code review
    • Insights
    • Issue
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • Open Subsurface Data Universe SoftwareOpen Subsurface Data Universe Software
  • Platform
  • System
  • Lib
  • cloud
  • azure
  • OS Core Lib Azure
  • Merge requests
  • !220

fix issue of the sp multiple calls for each partitions

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Yauheni Lesnikau requested to merge fix-sp-kv-calls into master Jul 19, 2022
  • Overview 7
  • Commits 1
  • Pipelines 11
  • Changes 3

We noticed that the services try to fetch app-dev-sp-username from keyvault for the first call for each partition. Especially it has an effect when there are a lot of partitions on the environment. Because of this we updated the TenantFactoryImpl not to retrieve the app-dev-sp-username on initPartition() call but use the initially fetched value from AzureOSDUConfig

Similar issue was found for the redis-hostname and redis-password. We used to do unnecessary KeyVault requests during checking whether the values presented into the KeyVault. Now it fixed as well.

Edited Aug 04, 2022 by Yauheni Lesnikau
Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: fix-sp-kv-calls