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

install.sh: instance size menu does not pre-select the default size

Đang mở Phù hợp với người mới
  • Có gì không ổn?
#134 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ó
2/5
Thời gian dự kiến
1-3 giờ
Mức phù hợp với người mới
84/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, shell
Lĩnh vực
cli, cloud

Hướng nghiên cứu

Đọc logic chọn kích thước trong install.sh, dòng 2556–2583, đặc biệt là phép tính default_type và lời gọi gum choose --selected. Ánh xạ mỗi loại phiên bản được hỗ trợ với nhãn tùy chọn đầy đủ tương ứng, sau đó kiểm tra xem lựa chọn mặc định có khớp với kích thước tối thiểu mà pack yêu cầu hay không. Hoàn tất khi kích thước cần thiết được chọn sẵn và nhấn Enter không triển khai một phiên bản quá nhỏ.

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

Mô tả

bug good first issue

install.sh L2579-2583:

_gum_or_die size_choice $GUM choose --header "Instance size" \
  --selected "${default_type}  — recommended" \
  "t4g.medium  — 2 vCPU,  4GB  ~\$25/mo   light use" \
  "t4g.large   — 2 vCPU,  8GB  ~\$50/mo   regular use" \
  "t4g.xlarge  — 4 vCPU, 16GB  ~\$100/mo  recommended" || size_choice=""

Problem: gum choose --selected must get the full option text. The text "t4g.xlarge — recommended" is not equal to any option, so no option is pre-selected and the cursor stays on t4g.medium. If the user pushes Enter, the installer deploys t4g.medium, also when the pack needs a larger minimum size (L2556-2572).

Fix: Map default_type to its full option label and pass that label to --selected. For example:

local sel
case "$default_type" in
  t4g.medium) sel="t4g.medium  — 2 vCPU,  4GB  ~\$25/mo   light use" ;;
  t4g.large)  sel="t4g.large   — 2 vCPU,  8GB  ~\$50/mo   regular use" ;;
  *)          sel="t4g.xlarge  — 4 vCPU, 16GB  ~\$100/mo  recommended" ;;
esac

Found by a code read at commit 1e50295. The code was read, but not run on a real instance.

Ngôn ngữ chính
Shell
Star
73
Fork
11
Merge trung bình
6 giờ 46 phút
Pull request đã merge (30 ngày)
5

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 inceptionstack/lowkey

Tất cả issue của inceptionstack/lowkey

Issue tương tự

Thêm issue về Shell/Bash

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.