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

Propose fix some typos #3

Closed
wants to merge 1 commit into from

Conversation

jeis2497052
Copy link

OK to remove some trailing white space and fix typos?

@reedy
Copy link
Member

reedy commented Apr 20, 2018

Typos definitely, whitespace probably...

Unfortunately, github is just a mirror, we develop and review over at https://gerrit.wikimedia.org

Are you able to submit a changeset to gerrit?

@jeis2497052
Copy link
Author

sure, I haven't used gerrit before? Do I fork or just clone directly? Should I just close this PR? I assume yes?

@reedy
Copy link
Member

reedy commented Apr 20, 2018

You can just clone the repo from gerrit (or change the remote if you already have it from github). You can then push a patch for review

wmfgerrit pushed a commit that referenced this pull request Jul 18, 2022
Third time's a charm

Change-Id: Ifd94d8787a4224df02e40bdfe29e755cb0914f84
wmfgerrit pushed a commit that referenced this pull request Oct 11, 2022
Partitioning the ATS cache effectively wipes it so we need to
proceed one server per cluster and DC at a time to avoid overwhelming
the applayer.

Affected servers:
* cp1085
* cp1086
* cp2037
* cp2038
* cp3060
* cp3061
* cp4026
* cp4030
* cp5006
* cp5012
* cp6005
* cp6013

ATS needs to be restarted to apply the new cache partition schema

Bug: T317748
Change-Id: I2e29c092d4b01850c0af60baebb824962fa2c6b4
wmfgerrit pushed a commit that referenced this pull request Jul 3, 2023
Bug: T329827
Change-Id: Idb5872e7471419b7a9ccf340c0ed4af40f78e550
wmfgerrit pushed a commit that referenced this pull request Jul 5, 2024
Why:
Step #3 of the evaluation. Let's see what happens when 1 out of 5
servers doesn't prefetch the image

Hosts: auto
Bug: T366778
Change-Id: Ia4dda6c884d3146c9560eefaad32fbe9593ed843
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
2 participants