SwissDataScienceCenter / renku-data-services / 14885165451

86%

Build Type

push

github

Committed by web-flow

Commit Message

feat: add multi-k8s client support (#815)

* Add `KubeClusterSettings` and a `cluster` field of that type to
  `ResourcePool`

* Add kr8s Api builder

* Add Cluster entity to API & DB

* Refactor k8s_client.py & K8sClient wrappers
  * Made sure we had a single sanitize function pointer, no need for
    multiple ones.
  * Added `K8sClusterClientsPool` which manages multiple Kubernetes API
    connections.
  * Prefixed private items (classes, variables & methods) with `_` to
    be more explicit.
  * Improve logging in k8s_client
    * Add log when kube config are loaded
    * Use sanic.log.logger everywhere
  * Rename server to session in method, variables & parameters names

* Added the cluster id as a parameter in the hashed string used for
  unique references of user sessions.

* Cleanups
   * Fix warnings
   * Fix missing call to `super().__init__()`
   * Remove unused fixtures
   * Remove dependency on `ClusterRequired`
   * Remove dead code
   * Remove unused return value of create_secret
   * Rename private fields
   * Order crc/apisepc.yaml
   * Fix formatting of crc/apisepc.yaml

 * Rename `namespace` to differentiate between the one for GitLab and the one
    for Kubernetes.

* Update apispec_base.py
  * Add ULID serialization / deserialization

---------

Co-authored-by: Ralf Grubenmann <ralf.grubenmann@sdsc.ethz.ch>

Run Details

496 of 673 new or added lines in 22 files covered. (73.7%)

14 existing lines in 6 files now uncovered.

21091 of 24415 relevant lines covered (86.39%)

1.52 hits per line