Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add MSPileup and refactor MongoDB in the WM Schematic #11789

Merged
merged 1 commit into from
Nov 8, 2023

Conversation

amaltaro
Copy link
Contributor

@amaltaro amaltaro commented Nov 8, 2023

Fixes #11788

Status

ready

Description

In this pull request we add MSPileup to the overall WM Schematic.
In addition, MongoDB has been separated in its own cluster (box).

Is it backward compatible (if not, which system it affects?)

YES

Related PRs

None

External dependencies / deployment changes

None

@cmsdmwmbot
Copy link

Jenkins results:

  • Python3 Unit tests: succeeded
    • 1 changes in unstable tests
  • Python3 Pylint check: succeeded
  • Pylint py3k check: succeeded
  • Pycodestyle check: succeeded

Details at https://cmssdt.cern.ch/dmwm-jenkins/view/All/job/DMWM-WMCore-PR-test/14612/artifact/artifacts/PullRequestReport.html

Copy link
Contributor

@todor-ivanov todor-ivanov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That looks great. Thanks @amaltaro
Just an idea here - While looking at this new picture I quite liked how you separated the MongoDB as a CMSWEB managed DB and this drove me to the idea, could we also split the central CMSWEB hosted Services in the middle of the diagram in two pieces - one which are just the services as a set of K8 deployed services and the other the CMSWeb managed CouchDB ....?

@amaltaro
Copy link
Contributor Author

amaltaro commented Nov 8, 2023

I would prefer to keep the "popular" names for WM services. But I agree that separating CouchDB in its own box would make it more readable. I won't have time to update it today, as I need this diagram for tomorrow. But will keep it in mind whenever we update it again. Thanks for the suggestion.

@amaltaro amaltaro merged commit 84cb987 into dmwm:master Nov 8, 2023
3 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update WM schematic diagram with the addition of MSPileup service
3 participants