Skip to content
Snippets Groups Projects

Dependency Management & Azure SDK Upgrade

Merged Abhishek Patil requested to merge abpatil/dpndncy_mngmnt_az_sdk_upgrd into master

All Submissions:


  • [YES] Have you followed our code review guidelines?
  • [YES] Have you added an explanation of what your changes do and why you'd like us to include them?
  • [YES] 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.
  • [NO] I ran lint checks locally prior to submission.

What is the current behavior?


  • Old version of Azure SDKs are used in service.
  • Versions of dependencies which are common across all OSDU services are not consistent/same in all OSDU services.

What is the new behavior?


  • All Azure SDKs are upgraded to latest stable version.
  • Common dependencies are inherited from core-lib-azure as managed dependencies. This will make sure that version info of those dependencies are inherited from core-lib-azure. All OSDU services using same version of core-lib-azure will automatically use same versions for all those dependencies.
  • Changes to support new cosmos-store & azure-cosmos sdk version.
  • Changes to make service work for single partition (opendes) and bypass partition service.

Does this introduce a breaking change?


  • [NO]

Any relevant logs, error output, etc?


(If it’s long, please paste to https://ghostbin.com/ and insert the link here.)

Other information


Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • harshit aggarwal
  • Abhishek Patil added 1 commit

    added 1 commit

    • 8b620e41 - Using actual parameter instead of any()

    Compare with previous version

  • harshit aggarwal approved this merge request

    approved this merge request

  • Abhishek Patil resolved all threads

    resolved all threads

  • Abhishek Patil added 1 commit

    added 1 commit

    • b8e5dfef - Disabling keyvault & using updated core-lib-azure

    Compare with previous version

  • Abhishek Patil added 16 commits

    added 16 commits

    • b8e5dfef...cfa3f011 - 10 commits from branch master
    • 8212ab64 - Using latest core-lib-azure
    • e5d6bcdb - Support for single partition & bypass partition service
    • bdc1b4bc - Removing temporary changes
    • 29cee1fb - Using properties for defining version
    • 9ff12a63 - Using actual parameter instead of any()
    • 99c3a5af - Disabling keyvault & using updated core-lib-azure

    Compare with previous version

  • Kishore Battula approved this merge request

    approved this merge request

  • Kishore Battula enabled an automatic merge when the pipeline for 99c3a5af succeeds

    enabled an automatic merge when the pipeline for 99c3a5af succeeds

  • Abhishek Patil aborted the automatic merge because source branch was updated

    aborted the automatic merge because source branch was updated

  • Abhishek Patil added 1 commit

    added 1 commit

    Compare with previous version

  • Kishore Battula approved this merge request

    approved this merge request

  • Kishore Battula enabled an automatic merge when the pipeline for 8f9e7de5 succeeds

    enabled an automatic merge when the pipeline for 8f9e7de5 succeeds

  • harshit aggarwal approved this merge request

    approved this merge request

  • harshit aggarwal aborted the automatic merge because source branch was updated

    aborted the automatic merge because source branch was updated

  • harshit aggarwal added 27 commits

    added 27 commits

    Compare with previous version

  • harshit aggarwal approved this merge request

    approved this merge request

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading