Reduce ATS cache size #462
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
MasterPtato
force-pushed
the
02-06-Reduce_ATS_cache_size
branch
from
49e5188 to
c7b0a9a
Compare
Your org requires the Graphite merge queue for merging into mainYou must have a Graphite account and log in to Graphite in order to use the merge queue. Sign up using this link. You can enable merging using labels in your Graphite merge queue settings. |
MasterPtato
force-pushed
the
02-06-Reduce_ATS_cache_size
branch
from
c7b0a9a to
c28d68d
Compare
NathanFlurry
requested changes
Feb 16, 2024
|
Last nitpick on this -- let's crank it to leave 4 GB free just to leave a lot of extra breathing room. Did you manually validate that this configuration is applied & getting picked up by ATS? |
NathanFlurry
requested changes
Feb 19, 2024
|
@NathanFlurry I know the config is being written, not sure if its being picked up. Does ATS have to be restarted? |
MasterPtato
force-pushed
the
02-06-Reduce_ATS_cache_size
branch
from
c28d68d to
3f4bbb3
Compare
NathanFlurry
requested changes
Feb 22, 2024
|
@NathanFlurry looks like a restart is all thats required, that file is the only thing configuring storage size |
NathanFlurry
approved these changes
Feb 26, 2024
NathanFlurry
force-pushed
the
02-06-Reduce_ATS_cache_size
branch
2 times, most recently
from
7658ec8 to
91442d3
Compare
This was referenced Feb 27, 2024
Merged
NathanFlurry
changed the base branch from
02-27-chore_format
to
02-27-fix_updated_docs_error_url
NathanFlurry
force-pushed
the
02-06-Reduce_ATS_cache_size
branch
from
91442d3 to
79213dd
Compare
Merge activity
|
<!-- Please make sure there is an issue that this PR is correlated to. --> Fixes RVT-3633 ## Changes <!-- If there are frontend changes, please include screenshots. -->
NathanFlurry
force-pushed
the
02-27-fix_updated_docs_error_url
branch
from
1e5c087 to
7099658
Compare
NathanFlurry
force-pushed
the
02-06-Reduce_ATS_cache_size
branch
from
79213dd to
3bea61f
Compare
NathanFlurry
changed the base branch from
02-27-fix_updated_docs_error_url
to
main
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixes RVT-3633
Changes