cri-resource-manager icon indicating copy to clipboard operation
cri-resource-manager copied to clipboard

misc. improvements

Open klihub opened this issue 5 years ago • 2 comments

  • CRI Resource Manager

    • [ ] generalize memtier/topology-aware resource pools, split out to a common library
      • [ ] rework allocator to operate on these generalized pools
      • [ ] rework cache/container state storage for generalized pools
    • [ ] consider switching configuration to using CRDs instead of ConfigMaps
    • [ ] rework configuration handling code with more of reflect
      • [ ] global configuration could simply be a dynamically created struct with a pointer member of/for each registered configurable entity
    • [ ] dynamic policy switch
      • [ ] make policy Backend's Stop()able
      • [ ] allow switching policies via config update
    • [x] make both topology-aware and memtier to activate
      • [x] fix bugs
      • [ ] clean up
      • [x] replace topology-aware with memtier
  • Agent

    • [ ] combine cri-rm->agent and agent->cri-rm interfaces to a single entity, remove config polling (GetConfig)
    • [ ] try to handle ConfigMap + Adjustments with a single client connection

klihub avatar Aug 24 '20 12:08 klihub

@klihub I think many of these are actually done. Should we close this issue and create new issues for the open tasks that we think are still worth implementing (if any)?

marquiz avatar Dec 23 '22 09:12 marquiz

Hmm, I think only the TA/memtier merge is really done. We have a PoC/draft of dynamic policy switch+reflect-based config handling (a few months old already so probably a bit outdated wrt. more recent other unrelated commit/merged PRs). But we can close these and reopen individual items for those that we think still should be done and intend to do...

klihub avatar Jan 11 '23 14:01 klihub