canonical / canonical/mlops-libs

Remove `pydantic` as dependency for the k8s_service_info library to reduce charm pack times?

Open
#7 2 comments 0 reactions 0 assignees View on GitHub
enhancement
Dominant language
Python
Stars
0
Forks
0
PR merge metrics
No merged PRs in 30d

Description

### Context

[this comment](https://github.com/canonical/kfp-operators/pull/436#discussion_r1569349087) documents how using the k8s_service_info library increase charm packing time by ~8 min. This isn't a complete blocker, but that's a significant increase and makes running integration tests (local and CI) a lot slower. Should we consider doing something about this?

Options:
* remove the dependency (it is only lightly used, and can be replaced by a dataclass)
* ? Maybe enable caching pip builds between our CI runs so we dont rebuild the pydantic whl every time?

### What needs to get done

See below

### Definition of Done

1. implement something that mitigates the increase in charm packing time caused by the k8s_service_info library

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.