old_mixer_repo icon indicating copy to clipboard operation
old_mixer_repo copied to clipboard

Integrate with Redis for rate limiting / quota

Open douglas-reid opened this issue 8 years ago • 3 comments

Tracking issue (from spreadsheet).

douglas-reid avatar Mar 15 '17 16:03 douglas-reid

Is this spreadsheet open for others? Any additional details?

What is the rationale for having yet another key value datastore in the control plane ? Why not use TPRs?

We now have three different configuration stores broadly speaking. TPR, config maps (k8s specific) and then redis (platform independent). Has any consideration been given to the proliferation of data stores in the config plane itself?

rshriram avatar Mar 22 '17 21:03 rshriram

Please ignore all mentions of spreadsheet here. It was a temporary holding place for things that were getting transferred to github issues. It isn't worth spending any time thinking about.

Here, redis is being used for quota enforcement. It does indeed reflect another data store, but I expect that once the config API is ready and supported, the storage of the underlying config storage will itself migrate towards redis (and away from TPR and configmaps and anything platform-specific).

@mandarjog can speak more to the plans here, but indeed, we want to reduce the number of config vectors.

douglas-reid avatar Mar 22 '17 21:03 douglas-reid

#435

chowchow316 avatar Mar 27 '17 20:03 chowchow316