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.

Sign up for GitHub

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

Jump to bottom

docs: answers to the woke police #7942

Merged
merged 1 commit into from
Dec 10, 2024
Merged

docs: answers to the woke police #7942

merged 1 commit into from
Dec 10, 2024

Conversation

fstagni
Copy link
Contributor

@fstagni fstagni commented Dec 9, 2024

No description provided.

@DIRACGridBot DIRACGridBot added the alsoTargeting:integration Cherry pick this PR to integration after merge label Dec 9, 2024
@marianne013
Copy link
Contributor

😄

@chaen
Copy link
Contributor

chaen commented Dec 9, 2024

strongly disapprove ! 👎

@fstagni fstagni force-pushed the 80_woke branch 4 times, most recently from 6c6523f to 6cc989c Compare December 9, 2024 17:18
andresailer
andresailer reviewed Dec 10, 2024
View reviewed changes
docs/source/AdministratorGuide/ServerInstallations/scalingAndLimitations.rst
+ +---------------------------------------------------------------------------------------------------+-------------+---------------------------------------------------------------------------+-----------+
| | :mod:`ReportGenerator <DIRAC.AccountingSystem.Service.ReportGeneratorHandler>` | | | +
+--------------------+---------------------------------------------------------------------------------------------------+-------------+---------------------------------------------------------------------------+-----------+
| Configuration | :mod:`Configuration <DIRAC.ConfigurationSystem.Service.ConfigurationHandler>` | PARTIAL | One master (rw) and slaves (ro). It's advised to have several CS slaves | YES +
| Configuration | :mod:`Configuration <DIRAC.ConfigurationSystem.Service.ConfigurationHandler>` | PARTIAL | One controller (rw) and workers (ro). Should have several CS workers | YES +
Copy link
Contributor

Choose a reason for hiding this comment

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

These tables are very whitespace sensitive

@fstagni fstagni merged commit fdbff56 into DIRACGrid:rel-v8r0 Dec 10, 2024
26 checks passed
@DIRACGridBot DIRACGridBot added sweep:done All sweeping actions have been done for this PR sweep:failed Sweeping failed and needs manual intervention labels Dec 10, 2024
@DIRACGridBot DIRACGridBot mentioned this pull request Dec 10, 2024
Closed
@DIRACGridBot
Copy link

Sweep summary

Sweep ran in https://github.com/DIRACGrid/DIRAC/actions/runs/12255304806

Failed:

  • integration
    cherry-pick fdbff56 into integration failed
    check merge conflicts on a local copy of this repository 
    git fetch upstream
git checkout upstream/integration -b cherry-pick-2-fdbff5614-integration
git cherry-pick -x -m 1 fdbff5614
# Fix the conflicts
git cherry-pick --continue
git commit --amend -m 'sweep: #7942 docs: answers to the woke police' --author='Federico Stagni <[email protected]>'
git push -u origin cherry-pick-2-fdbff5614-integration

# If you have the GitHub CLI installed the PR can be made with
gh pr create \
     --label 'sweep:from rel-v8r0' \
     --base integration \
     --repo DIRACGrid/DIRAC \
     --title '[sweep:integration] docs: answers to the woke police' \
     --body 'Sweep #7942 `docs: answers to the woke police` to `integration`.

Adding original author @fstagni as watcher.


Closes #7944'

fstagni added a commit to fstagni/DIRAC that referenced this pull request Dec 10, 2024
@fstagni
sweep: DIRACGrid#7942 docs: answers to the woke police
3ac4c23
@fstagni fstagni mentioned this pull request Dec 10, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@aldbr aldbr aldbr left review comments

@andresailer andresailer andresailer left review comments

Assignees
No one assigned
Labels
alsoTargeting:integration Cherry pick this PR to integration after merge sweep:done All sweeping actions have been done for this PR sweep:failed Sweeping failed and needs manual intervention
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@fstagni @marianne013 @chaen @DIRACGridBot @andresailer @aldbr