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

feat(bakery): Clean up cached data created by Rosco. #912

Merged
merged 6 commits into from
Nov 17, 2022

Conversation

armory-abedonik
Copy link
Contributor

@armory-abedonik armory-abedonik commented Nov 6, 2022

Description

Rosco service uses Redis for caching purposes. The records created in Redis can be up to 50MB size and will never be removed. As a result, Redis can become quite expensive. This pull request changes solve this issue.

@dogonthehorizon dogonthehorizon added the ready to merge Approved and ready for merge label Nov 17, 2022
@mergify mergify bot added the auto merged label Nov 17, 2022
@mergify mergify bot merged commit 96cd6d2 into spinnaker:master Nov 17, 2022
@armory-abedonik armory-abedonik deleted the SPLAT-405 branch November 17, 2022 16:24
@dogonthehorizon
Copy link
Contributor

@mergify backport release-1.27.x release-1.28.x release-1.29.x

mergify bot pushed a commit that referenced this pull request Nov 23, 2022
* feat(bakery): Clean up cached data created by Rosco.

* fix: typo.

* feat: Add unit test.

Co-authored-by: Fernando Freire <dogonthehorizon@gmail.com>
(cherry picked from commit 96cd6d2)
mergify bot pushed a commit that referenced this pull request Nov 23, 2022
* feat(bakery): Clean up cached data created by Rosco.

* fix: typo.

* feat: Add unit test.

Co-authored-by: Fernando Freire <dogonthehorizon@gmail.com>
(cherry picked from commit 96cd6d2)
@mergify
Copy link
Contributor

mergify bot commented Nov 23, 2022

backport release-1.27.x release-1.28.x release-1.29.x

✅ Backports have been created

mergify bot pushed a commit that referenced this pull request Nov 23, 2022
* feat(bakery): Clean up cached data created by Rosco.

* fix: typo.

* feat: Add unit test.

Co-authored-by: Fernando Freire <dogonthehorizon@gmail.com>
(cherry picked from commit 96cd6d2)
mergify bot added a commit that referenced this pull request Nov 23, 2022
* feat(bakery): Clean up cached data created by Rosco.

* fix: typo.

* feat: Add unit test.

Co-authored-by: Fernando Freire <dogonthehorizon@gmail.com>
(cherry picked from commit 96cd6d2)

Co-authored-by: armory-abedonik <106548537+armory-abedonik@users.noreply.github.com>
mergify bot added a commit that referenced this pull request Nov 23, 2022
* feat(bakery): Clean up cached data created by Rosco.

* fix: typo.

* feat: Add unit test.

Co-authored-by: Fernando Freire <dogonthehorizon@gmail.com>
(cherry picked from commit 96cd6d2)

Co-authored-by: armory-abedonik <106548537+armory-abedonik@users.noreply.github.com>
mergify bot added a commit that referenced this pull request Nov 23, 2022
* feat(bakery): Clean up cached data created by Rosco.

* fix: typo.

* feat: Add unit test.

Co-authored-by: Fernando Freire <dogonthehorizon@gmail.com>
(cherry picked from commit 96cd6d2)

Co-authored-by: armory-abedonik <106548537+armory-abedonik@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
5 participants