AWS installer (beta): two uninstall report lines claim more than they show

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

@amarkanday đang làm issue này rồi.

Từ ngày 6/11/2026.

  • #1794 của @amarkanday — đang mở

Đánh giá

Độ khó
3/5
Thời gian dự kiến
1-2 ngày
Mức phù hợp với người mới
72/100
Loại issue
Lỗi
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
aws, python
Lĩnh vực
cli, cloud, testing

Hướng nghiên cứu

Start with the controlvar-aws uninstall reporting code and the installer fakes used by its tests. Locate the FAILED block and the logic that counts tag-index entries, including how resources intentionally retained by uninstall are distinguished from resources still being cleared. Update both report behaviors and add focused fake-based tests that fail against the current wording and exclude retained resources from the clearing count.

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

Mô tả

bug post-launch

Two lines in controlvar-aws uninstall's report say something that is not true in the case they print in. Both were
seen on a real AWS run of the beta.

  1. "Nothing else the install made is in your account" is printed in the FAILED block even when a line under "Not
    explained" is something the install made (for example the cache's automatic backup, while AWS is still deleting
    it). The sentence should not claim more than the report shows: either drop it when anything is unexplained, or
    say that the unexplained lines may include the install's own leftovers.

  2. " tag-index entries AWS is still clearing (minutes to hours)", after uninstalling an install that stopped
    before its stacks, counted the tag-index entries of the CDK's setup stack, which the uninstall keeps on purpose and
    which therefore never clear. Tag-index entries of resources the uninstall keeps should not be counted as clearing.

Each fix needs a test on the installer's fakes that fails on today's wording. Both lines are in the same part of the
uninstall's report as the fix for the cache's automatic backup, so this follows that change.

Ngôn ngữ chính
Python
Star
0
Fork
4
Merge trung bình
6 giờ 5 phút
Pull request đã merge (30 ngày)
453

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 Controlvar/controlvar

Tất cả issue của Controlvar/controlvar

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.