Deprecate append_minionid_config_dirs — redundant with explicit path config

Đang mở
  • Có gì không ổn?
#69,975 0 bình luận 0 reaction 0 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 3 ngày

Chưa có ai nhận issue này.

Đánh giá

Độ khó
4/5
Thời gian dự kiến
3-5 ngày
Mức phù hợp với người mới
55/100
Loại issue
Tái cấu trúc
Độ rõ ràng
Khá rõ ràng
Mức độ hoạt động
Ít trao đổi
Công nghệ
python
Lĩnh vực
devops, infrastructure

Hướng nghiên cứu

Bắt đầu tại salt/config/init.py:4152-4169 để hiểu cách append_minionid_config_dirs được áp dụng cho các đường dẫn đã cấu hình, sau đó đọc phần tài liệu của nó trong tài liệu tham chiếu về cấu hình minion. Công việc được hoàn tất khi khóa đã cấu hình phát ra cảnh báo ngừng sử dụng được đề xuất, tài liệu hiển thị các đường dẫn rõ ràng cho từng minion và mốc thời gian loại bỏ được ghi lại mà không thay đổi hành vi của 3006.x/3007.x.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

append_minionid_config_dirs (docs: https://docs.saltproject.io/en/3006/ref/configuration/minion.html#append-minionid-config-dirs) is a 2016-era convenience (PR #35178) that auto-appends opts["id"] to pki_dir / cachedir / extension_modules / default_include / pidfile.

Redundant: each of those paths is directly configurable in /etc/salt/minion (or minion.d/*.conf), so operators running multiple minions/proxies on one host can just write the per-minion path they want:

pki_dir: /etc/salt/pki/minion/{{ opts["id"] }}
cachedir: /var/cache/salt/minion/{{ opts["id"] }}
pidfile: /run/salt/minion/{{ opts["id"] }}.pid

The convenience option adds a second, less-discoverable way to do the same thing and forces salt/config/__init__.py:4152-4169 to special-case each supported key (with an unrelated sock_dir truncation hack right after).

Proposal:

  • Emit DeprecationWarning on 3008.x/master when the key is set in loaded opts.
  • Remove in the release after next.
  • Docs: replace the section with an example of explicit per-minion path config.

No runtime behavior change for existing users on 3006.x/3007.x; only a docs cite + warning + removal timeline.

/cc @dwoz

Ngôn ngữ chính
Python
Star
15.8k
Fork
5.6k
Merge trung bình
5 ngày 9 giờ
Pull request đã merge (30 ngày)
99

Chuẩn bị môi trường

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. 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.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của saltstack/salt

Tất cả issue của saltstack/salt

Issue tương tự

Thêm issue về Python

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.