Automatic dynamic resolution of pip environments
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 5/5
- Thời gian dự kiến
- Hơn một tuần
- Mức phù hợp với người mới
- 35/100
- Loại issue
- Tính năng
- Độ rõ ràng
- Cần làm rõ
- Mức độ hoạt động
- Ít trao đổi
- Công nghệ
- python
- Lĩnh vực
- build-system
Hướng nghiên cứu
Bắt đầu với gist được liên kết, sau đó theo dõi cách các dependency của pip_A và pip_B được chọn cho py_library, py_test và py_binary. So sánh behavior được đề xuất của pip_dep với approach sử dụng string_flag, config_setting và transition được mô tả trong issue. Công việc được xem là hoàn tất khi một design hoặc implementation được document có thể resolve đúng environment mà không cần command-line flag cho từng target.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Description
Consider a monorepo with many separate pip environments
- Let py_library
Adepend onnumpyand have an associated pip environment (i.e. requirements.txt.lock file) - Let py_library
Bdepend onAand a specific versionnumpy==Xand also have its own pip environment.
For B, whether we end up using the version of numpy from A or from B's pip environment depends entirely on the order of deps in the definition of B.
py_library(
name = "A",
deps = [pip_A("numpy")],
)
py_library(
name = "B",
deps = [
# If we accidentally swap the order, we are going to use `pip_A("numpy")`
pip_B("numpy"),
"//path/to:A",
],
)
When we have a big monorepo with multiple bazel subpackages and multiple pip environments this very quickly gets out of hand and can a lot of problems.
Describe the solution you'd like
It would be extremely nice if the environment can be automatically resolved. For example, instead of the above, we instead write
py_library(
name = "A",
deps = [pip_dep("numpy")],
)
py_library(
name = "B",
deps = [
pip_dep("numpy"),
"//path/to:A",
],
)
Here pip_dep automatically resolves to the correct environment at build time. For A it resolves to pip_A, for B it resolves to pip_B (this can also be propagated from py_test and py_binary). For this to function correctly the assumption is that pip_B contains all pip_A packages that A requires, but not necessarily the same versions. This can very easily be achieved with proper dependency management across pip environments - either via -r A_requirements.in or by recursive inclusion in pyproject.toml.
The above behavior is theoretically possible today by adding a special macro pip_dep which resolves based on a string_flag and config_setting. However, this is quite far from convenient from an usage perspective, for example:
- The user needs to select and pass the correct flag for any binary or test that he runs
- Simultaneously running the entire suite of tests for the monorepo is not possible since bazel needs to be invoked with a different flag for every subpackage (this also can make things a lot slower)
Ideally the correct environment can be automatically resolved for a given target based on its path in the repo (if we assume all targets under B use pip_B and all targets under A use pip_A, which is IMO a sensible assumption). Then one can enforce the correct environment via transition. On the command line targets can be invoked without any extra configuration arguments (or configuration arguments can overwrite the automatic resolution).
Example implementation https://gist.github.com/nikonikolov/3204c93621c86b6f6d10723f65e7c1b7. Likely very suboptimal and can be improved. I am in no way a bazel expert, I coded it up with the help of Gemini
Describe alternatives you've considered
- Using a single lock file for the entire repo -> most of the time impossible due to conflicting requirements in different subpackages
- Make sure that if B depends on A, it always uses the same versions of the pip packages from A. This means user need to use
-c A_requirements.txt.lockto compileB_requirements.txt.lock. With multiple subpackages and conflicting environments, one would be constantly running into version conflicts which will have to be manually resolved (if at all possible), which is obviously far from desirable (hence the need for tools such asuv pip compileorpip-compile)
- Ngôn ngữ chính
- Starlark
- Star
- 690
- Fork
- 722
- Merge trung bình
- 1 ngày 55 phút
- Pull request đã merge (30 ngày)
- 38
Hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của bazel-contrib/rules_python
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
bazel-contrib/rules_python#4164 · 1 bình luận ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
bazel-contrib/rules_python#3821 ·
-
bazel-contrib/rules_python#4181 ·
-
bazel-contrib/rules_python#4179 ·
-
Release 2.4.0 Đang mởtype: release
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 25/100
bazel-contrib/rules_python#4175 · 3 bình luận ·
Tất cả issue của bazel-contrib/rules_python
Issue tương tự
-
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 85/100
-
CS0162 "Unreachable code detected" warning from a MSBuildTemp .tmp file in every game project Đang mởbug
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 90/100
-
area:engineering priority:p2 type:bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
jejjohnson/pyrox#234 ·
-
good first issue
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 90/100
-
good first issue
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 85/100
ros2/ros2_tracing#266 · 1 bình luận ·