Skip to content
GitLab
  • Menu
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Sign in / Register
  • S Storage
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Locked Files
  • Issues 54
    • Issues 54
    • List
    • Boards
    • Service Desk
    • Milestones
    • Iterations
    • Requirements
  • Merge requests 14
    • Merge requests 14
  • 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
  • Storage
  • Merge requests
  • !33

Azure - (3/4) - Adding missing context path for Storage

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Duvelis Carao requested to merge fix-context-path-storage into master Jul 07, 2020
  • Overview 8
  • Commits 4
  • Pipelines 9
  • Changes 3

This impacts only Azure Implementation

• What is the change? Adding context path to Azure Storage: /api/storage/v2/

• What does it impact? New URL for storage endpoint

• How do I reproduce or test the change?

This MR is part of group of MRs with dependencies between them

  1. Update infrastructure (Azure / osdu-infrastructure) to have reference to the new URLs (https://github.com/Azure/osdu-infrastructure/pull/23)
  2. Adding missing context path for entitlements.
  3. Adding missing context path for legal.
  4. Adding missing context path for storage.
  5. Adding missing context path for indexer

After merging this MR, the new Storage URL will be: https://{host}/api/storage/v2/

Before merging this MR:

  • Please make sure the dependent MR is done: osdu/platform/security-and-compliance/legal!32 (merged)

  • Update environment variable (AZURE - Common) with the new storage URL: STORAGE_URL: https://$ (AZURE_STORAGE_SERVICE_NAME).azurewebsites.net/api/storage/v2/


This PR is related to 6 other requests for changing a URL structure and adding in context paths for services.

  1. osdu-infrastructure 1/6 PR https://github.com/Azure/osdu-infrastructure/pull/23
  2. CI-CD Pipelines 2/6 This PR
  3. Entitlements Azure 3/6 osdu/platform/security-and-compliance/entitlements-azure!11 (merged)
  4. Legal 4/6 osdu/platform/security-and-compliance/legal!32 (merged)
  5. Storage 5/6 !33 (merged)
  6. Indexer 6/6 https://community.opengroup.org/osdu/platform/system/indexer-service

They are all required and any one but not the other will break builds and tests across the entire ecosystem for Azure.

Edited Jul 16, 2020 by Daniel Scholl
Assignee
Assign to
Reviewer
Request review from
Time tracking
Source branch: fix-context-path-storage