Hacktoberfest 2026: những issue maintainer đã đánh dấu cho tháng Mười, đang mở và phù hợp người mới. Xem issue Hacktoberfest

Feedback request: ran a config-diff tool against PR #1663 — is this useful on chart values?

Đang mở
  • Có gì không ổn?
#1,671 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 1 ngày

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
20/100
Loại issue
Tính năng
Độ rõ ràng
Cần làm rõ
Mức độ hoạt động
Sôi nổi
Công nghệ
yaml
Lĩnh vực
devops

Hướng nghiên cứu

Start by reviewing PR #1663 and the cited charts/*/values.yaml changes, then compare the Flecto output with the raw diff and the repository's current review or CI workflow. This issue is done when maintainers decide whether changed-path comments provide useful signal at this repository's volume; no implementation scope is specified.

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

Mô tả

Hi — apologies for arriving via the issue tracker. This is a feedback request rather than a bug report, and "not interested" is a completely fine answer; I'll close it myself if so.

I maintain Flecto, a small open-source tool that reports what a config change actually did, in plain English, as a pull request comment. I've been running it against real repositories rather than my own fixtures, because fixtures tell you nothing about whether the output is useful.

I ran it read-only against your PR #1663 ("remove placeholder values from monitoring chart"):

✅ Check passing — 2 changes in 1 file — 2 changed, 0 added, 0 removed.
Policy: no findings.

| changed | prometheus.server.alertmanagers[0].relabel_configs[0].regex | "<namespace>"    | "monitoring" |
| changed | prometheus.server.alertmanagers[0].relabel_configs[1].regex | "<release-name>" | "monitoring" |

The reason I picked this PR: those two values sit five levels down inside
relabel_configs, and in a raw YAML diff they are two lines that look like every other two lines. Naming the full path and the before/after is most of what I am trying to test — whether that framing actually helps a reviewer, or whether it is just the same information rearranged.

Two questions, neither of which needs you to install anything:

  1. When a PR touches charts/*/values.yaml in this repo, what do you look at today? Rendered output, the raw diff, or mostly trust plus CI?
  2. With 1600+ PRs, would a comment naming changed paths be signal or noise here?

If the honest answer to 2 is "noise at our volume", that is genuinely the more useful answer for me, and I would rather hear it.

Happy to close this right away if issue-tracker outreach isn't welcome — just say so.

Ngôn ngữ chính
Rust
Star
7
Fork
6
Merge trung bình
1 ngày 31 phút
Pull request đã merge (30 ngày)
83

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

Dự án này không cung cấp dev container, Dockerfile hay hướng dẫn đóng góp, nên bạn cần tự thiết lập môi trường: hãy bắt đầu từ README và xem hướng dẫn đóng góp lần đầu của chúng tôi để biết các bước chung.

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 DiamondLightSource/workflows

Tất cả issue của DiamondLightSource/workflows

Issue tương tự

Thêm issue về Rust

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.