Skip to content

fix: aws credentials cache uses the right key on cedentials removal#7717

Open
JorTurFer wants to merge 1 commit intokedacore:mainfrom
JorTurFer:fix-aws-cache
Open

fix: aws credentials cache uses the right key on cedentials removal#7717
JorTurFer wants to merge 1 commit intokedacore:mainfrom
JorTurFer:fix-aws-cache

Conversation

@JorTurFer
Copy link
Copy Markdown
Member

@JorTurFer JorTurFer commented May 3, 2026

As @rickbrouwer found, there is a bug with the removal of the items from AWS cache.

This PR fixes the bug and includes a test to validate that it works

Checklist

  • Commits are signed with Developer Certificate of Origin (DCO - learn more)

Relates to #7665 (comment)

Signed-off-by: Jorge Turrado <jorge_turrado@hotmail.es>
@JorTurFer JorTurFer requested a review from a team as a code owner May 3, 2026 16:32
@snyk-io
Copy link
Copy Markdown

snyk-io Bot commented May 3, 2026

Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 3, 2026

Thank you for your contribution! 🙏

Please understand that we will do our best to review your PR and give you feedback as soon as possible, but please bear with us if it takes a little longer as expected.

While you are waiting, make sure to:

  • Add an entry in our changelog in alphabetical order and link related issue
  • Update the documentation, if needed
  • Add unit & e2e tests for your changes
  • GitHub checks are passing
  • Is the DCO check failing? Here is how you can fix DCO issues

Once the initial tests are successful, a KEDA member will ensure that the e2e tests are run. Once the e2e tests have been successfully completed, the PR may be merged at a later date. Please be patient.

Learn more about our contribution guide.

@keda-automation keda-automation requested a review from a team May 3, 2026 16:32
@JorTurFer
Copy link
Copy Markdown
Member Author

JorTurFer commented May 3, 2026

/run-e2e aws
Update: You can check the progress here

@rickbrouwer rickbrouwer added the Awaiting/2nd-approval This PR needs one more approval review label May 3, 2026
@rickbrouwer rickbrouwer enabled auto-merge (squash) May 3, 2026 19:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Awaiting/2nd-approval This PR needs one more approval review required:keda-v2.20

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants