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

775 use tiered cache in place of caffeine cache #783

Merged
merged 37 commits into from
Feb 17, 2025
Merged

775 use tiered cache in place of caffeine cache #783

merged 37 commits into from
Feb 17, 2025

Conversation

munishchouhan
Copy link
Member

@munishchouhan munishchouhan commented Dec 26, 2024 •
edited
Loading

Add the following:

  1. AwsEcrCache
  2. RegistryLookupCache
  3. RegistryAuthCache
  4. TieredCacheKey

@munishchouhan
Added AwsEcrCache
1e13c9f 
Signed-off-by: munishchouhan <hrma017@gmail.com>
@munishchouhan
Merge remote-tracking branch 'origin/master' into 775-use-tiered-cach…
df8a09e 
…e-in-place-of-caffeine-cache
@munishchouhan
fixed tests
00b4e5f 
Signed-off-by: munishchouhan <hrma017@gmail.com>
@munishchouhan
fixed tests
3d21816 
Signed-off-by: munishchouhan <hrma017@gmail.com>
@munishchouhan
fixed license
12f6234 
Signed-off-by: munishchouhan <hrma017@gmail.com>
@munishchouhan
fixed tests
d52e410 
Signed-off-by: munishchouhan <hrma017@gmail.com>
@munishchouhan
Added RegistryLookupCache
8660e8a 
Signed-off-by: munishchouhan <hrma017@gmail.com>
@munishchouhan
Added RegistryAuthCache
c7e8536 
Signed-off-by: munishchouhan <hrma017@gmail.com>
@munishchouhan
Added invalidate(key)
0185fc8 
Signed-off-by: munishchouhan <hrma017@gmail.com>
@munishchouhan munishchouhan linked an issue Dec 26, 2024 that may be closed by this pull request
Use tiered cache in place of Caffeine cache #775
Closed
3 tasks
@munishchouhan munishchouhan marked this pull request as draft December 26, 2024 21:41
@munishchouhan munishchouhan self-assigned this Dec 26, 2024
@munishchouhan munishchouhan added the WIP Work In Progress label Dec 26, 2024
munishchouhan and others added 3 commits December 30, 2024 20:56
@munishchouhan
fixed tests
fa2a2b4 
Signed-off-by: munishchouhan <hrma017@gmail.com>
@munishchouhan
Merge branch 'master' into 775-use-tiered-cache-in-place-of-caffeine-…
9bbe2a2 
…cache
@munishchouhan
fixed error
5a7c9e2 
Signed-off-by: munishchouhan <hrma017@gmail.com>
@munishchouhan
Copy link
Member Author

Tested locally with Redis:

  1. 10 parallel builds with tower token
  2. 10 parallel builds with tower token and freeze enabled

@munishchouhan
[release] bump 1.16.6-A0
33049bf 
Signed-off-by: munishchouhan <hrma017@gmail.com>
@munishchouhan
Revert "[release] bump 1.16.6-A0"
682b845 
This reverts commit 33049bf.
@munishchouhan
Copy link
Member Author

Tested Dev:

  1. 10 parallel builds with tower token
  2. 10 parallel builds with tower token and freeze-enabled

@munishchouhan
refactored
8fe8bff 
Signed-off-by: munishchouhan <hrma017@gmail.com>
@munishchouhan
refactored
53c2d58 
Signed-off-by: munishchouhan <hrma017@gmail.com>
@munishchouhan
fixed error
3de2824 
Signed-off-by: munishchouhan <hrma017@gmail.com>
@munishchouhan munishchouhan marked this pull request as ready for review January 2, 2025 14:52
@munishchouhan munishchouhan removed the WIP Work In Progress label Jan 2, 2025
pditommaso and others added 13 commits January 27, 2025 22:49
@pditommaso
Simplified tiered cache key impl
ea3dccd 
Signed-off-by: Paolo Di Tommaso <paolo.ditommaso@gmail.com>
@munishchouhan
Revert "Added RegistryAuthCache"
0c62712 
This reverts commit c7e8536.
@munishchouhan
removed invalidate(K key)
807ef92 
Signed-off-by: munishchouhan <hrma017@gmail.com>
@munishchouhan
fixed errors
dba3cd4 
Signed-off-by: munishchouhan <hrma017@gmail.com>
@munishchouhan
fixed errors
32855c5 
Signed-off-by: munishchouhan <hrma017@gmail.com>
@munishchouhan
fixed tests
c352959 
Signed-off-by: munishchouhan <hrma017@gmail.com>
@munishchouhan
fixed tests
068b28d 
Signed-off-by: munishchouhan <hrma017@gmail.com>
@munishchouhan
Merge branch 'master' into 775-use-tiered-cache-in-place-of-caffeine-…
2e39cc9 
…cache
@munishchouhan
Merge branch 'master' into 775-use-tiered-cache-in-place-of-caffeine-…
8411b20 
…cache
@pditommaso
Merge branch 'master' into 775-use-tiered-cache-in-place-of-caffeine-…
a81d848 
…cache
@pditommaso
Remove double cache + refactor
1289cde 
Signed-off-by: Paolo Di Tommaso <paolo.ditommaso@gmail.com>
@pditommaso
Refactor
e567e76 
Signed-off-by: Paolo Di Tommaso <paolo.ditommaso@gmail.com>
@pditommaso
[release] 1.17.3-B0
251bce3 
Signed-off-by: Paolo Di Tommaso <paolo.ditommaso@gmail.com>
pditommaso
pditommaso reviewed Feb 17, 2025
View reviewed changes
@pditommaso
Update VERSION [ci skip]
7491568
@pditommaso
Merge branch 'master' into 775-use-tiered-cache-in-place-of-caffeine-…
6b706f7 
…cache
@pditommaso
Remove unused try catch
aa8d4c3 
Signed-off-by: Paolo Di Tommaso <paolo.ditommaso@gmail.com>
@pditommaso
Remove unused attribute
a9ef143 
Signed-off-by: Paolo Di Tommaso <paolo.ditommaso@gmail.com>
@pditommaso pditommaso merged commit b1a76d1 into master Feb 17, 2025
4 checks passed
@pditommaso pditommaso deleted the 775-use-tiered-cache-in-place-of-caffeine-cache branch February 17, 2025 16:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pditommaso pditommaso pditommaso left review comments

Assignees

@munishchouhan munishchouhan

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Use tiered cache in place of Caffeine cache
2 participants
@munishchouhan @pditommaso