fix(cache): address in-memory cache invalidation using global tenant as key_prefix

[Chethan-rao]

Type of Change

• Bugfix
• New feature
• Enhancement
• Refactoring
• Dependency updates
• Documentation
• CI/CD

Description

This PR fixes an inconsistency in the redis key_prefix that was used when invalidating the cache entries which affected all the cache types (configs, accounts, routing, etc).
The delete_key redis call during cache invalidation was using global as the key_prefix (key like global:test_key was being deleted in redis) while each of the tenants were reading using their corresponding key_prefix (key like public:test_key was being set/read in redis).

Additional Changes

• This PR modifies the API contract
• This PR modifies the database schema
• This PR modifies application configuration/environment variables

Motivation and Context

How did you test it?

PUT/GET/UPDATE/DELETE on configs table as configs has cache usage

1. PUT

curl --location 'http://localhost:8080/configs/' \
--header 'Content-Type: application/json' \
--header 'Accept: application/json' \
--header 'x-tenant-id: public' \
--header 'api-key: test_admin' \
--data '{
    "key": "test_key_1",
    "value": "test_val_1"
}'

2. GET

curl --location 'http://localhost:8080/configs/test_key_1' \
--header 'Content-Type: application/json' \
--header 'Accept: application/json' \
--header 'x-tenant-id: public' \
--header 'api-key: test_admin' \
--data ''

3. UPDATE

curl --location 'http://localhost:8080/configs/test_key_1' \
--header 'Content-Type: application/json' \
--header 'Accept: application/json' \
--header 'x-tenant-id: public' \
--header 'api-key: test_admin' \
--data '{
    "value": "test_val_2"
}'

4. GET

curl --location 'http://localhost:8080/configs/test_key_1' \
--header 'Content-Type: application/json' \
--header 'Accept: application/json' \
--header 'x-tenant-id: public' \
--header 'api-key: test_admin' \
--data ''

5. DELETE

curl --location --request DELETE 'http://localhost:8080/configs/test_key_1' \
--header 'Content-Type: application/json' \
--header 'Accept: application/json' \
--header 'x-tenant-id: public' \
--header 'api-key: test_admin' \
--data ''

6. GET

curl --location 'http://localhost:8080/configs/test_key_1' \
--header 'Content-Type: application/json' \
--header 'Accept: application/json' \
--header 'x-tenant-id: public' \
--header 'api-key: test_admin' \
--data ''

Checklist

• I formatted the code cargo +nightly fmt --all
• I addressed lints thrown by cargo clippy
• I reviewed the submitted code
• I added unit tests for my changes where possible

[semanticdiff-com]

Review changes with  

Changed Files

File	Status
crates/redis_interface/src/commands.rs	55% smaller
crates/router/src/db/configs.rs	33% smaller
crates/storage_impl/src/redis/cache.rs	3% smaller
crates/router/src/core/admin.rs	0% smaller
crates/router/src/core/cache.rs	0% smaller
crates/router/src/core/routing.rs	0% smaller
crates/router/src/core/routing/helpers.rs	0% smaller
crates/router/src/db/merchant_account.rs	0% smaller
crates/storage_impl/src/redis/pub_sub.rs	0% smaller

[Chethan-rao]

Removed this function because its not being used anywhere. moreover its not meant to be used in public