Skip to content
GitLab
Projects Groups Topics Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Register
  • Sign in
  • S Schema
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributor statistics
    • Graph
    • Compare revisions
    • Locked files
  • Issues 25
    • Issues 25
    • List
    • Boards
    • Service Desk
    • Milestones
    • Iterations
    • Requirements
  • Merge requests 9
    • Merge requests 9
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
    • Test cases
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Package Registry
    • Container Registry
    • Terraform modules
  • 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
  • Schema
  • Merge requests
  • !163

Usage of Publisher Facade

  • Review changes

  • Download
  • Patches
  • Plain diff
Closed Nikhil Singh[MicroSoft] requested to merge users/nikhil/UsageOfPublisherFacade into master Sep 13, 2021
  • Overview 12
  • Commits 11
  • Pipelines 22
  • Changes 9

All Submissions:


  • [YES] 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?
  • [NA] I have updated the documentation accordingly.
  • [YES] 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.

What is the issue or story related to the change?


The MR contains changes to use publisher facade for publishing to message brokers. The schema service is a producer service for the notification service i.e it publishes notifications to Event grid that were getting pushed to notification service. Now the notification service has been migrated to Service bus and all the producer services are provided by a facade which they can use and start publishing to service bus.

Link to facade MR:

osdu/platform/system/lib/cloud/azure/os-core-lib-azure!143 (merged)

Link of related issue:

notification#25 (closed)

Does this introduce a breaking change?


  • [NO]

NA

Pending items


No

Edited Sep 14, 2021 by Nikhil Singh[MicroSoft]
Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: users/nikhil/UsageOfPublisherFacade