Skip to content
GitLab
  • Menu
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Sign in / Register
  • O OS Core Lib Azure
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Locked Files
  • Issues 10
    • Issues 10
    • List
    • Boards
    • Service Desk
    • Milestones
    • Iterations
    • Requirements
  • Merge requests 4
    • Merge requests 4
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
    • Test Cases
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages & Registries
    • Packages & 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 Software
  • Platform
  • System
  • Lib
  • cloud
  • azure
  • OS Core Lib Azure
  • Merge requests
  • !139

Draft: Msrivastava/resiliency extension legal

  • Review changes

  • Download
  • Email patches
  • Plain diff
Closed Muskan Srivastava requested to merge msrivastava/resiliency-extension-legal into master Aug 19, 2021
  • Overview 0
  • Commits 8
  • Pipelines 15
  • Changes 2

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

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.

High level design: This MR adds 2 classes in total.

  • 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

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 followsScreenshot__29_

  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_

Edited Aug 25, 2021 by Muskan Srivastava
Assignee
Assign to
Reviewer
Request review from
Time tracking
Source branch: msrivastava/resiliency-extension-legal