Hướng dẫn triển khai các thay đổi có thể gây lỗi

Báo cáo vấn đề Xem nguồn Hằng đêm · 7.3 · 7.2 · 7.1 · 7 · 6,5

Chúng tôi sẽ thực hiện một số thay đổi có thể gây lỗi đối với Bazel. Chúng tôi sẽ phải thay đổi thiết kế của chúng tôi và khắc phục những thứ không hoạt động tốt. Tuy nhiên, chúng tôi cần để đảm bảo cộng đồng và hệ sinh thái Bazel có thể tuân thủ. Để đạt được mục tiêu đó, Dự án Bazel đã áp dụng một chính sách về khả năng tương thích ngược. Tài liệu này mô tả quy trình giúp cộng tác viên Bazel tạo bản sửa lỗi thay đổi trong Bazel để tuân thủ chính sách này.

  1. Tuân thủ chính sách tài liệu thiết kế.

  2. Báo cáo vấn đề trên GitHub.

  3. Triển khai thay đổi này.

  4. Cập nhật nhãn.

  5. Cập nhật kho lưu trữ.

  6. Chuyển cờ không tương thích.

Vấn đề trên GitHub

Báo cáo vấn đề trên GitHub trong kho lưu trữ Bazel. Xem ví dụ.

Bạn nên:

  • Tiêu đề bắt đầu bằng tên cờ (tên cờ sẽ bắt đầu bằng incompatible_).

  • Bạn thêm nhãn incompatible-change.

  • Phần mô tả chứa nội dung mô tả về thay đổi và một đường liên kết đến trang tài liệu thiết kế.

  • Nội dung mô tả chứa công thức di chuyển để giải thích cho người dùng cách họ cần hãy cập nhật mã của mình. Tốt nhất là khi thay đổi mang tính cơ học, hãy bao gồm một đường liên kết đến công cụ di chuyển.

  • Nội dung mô tả có một ví dụ về thông báo lỗi mà người dùng sẽ nhận được nếu chúng không di chuyển. Điều này sẽ giúp vấn đề GitHub dễ tìm thấy hơn từ công cụ tìm kiếm. Đảm bảo rằng thông báo lỗi là hữu ích và có thể xử lý được. Khi có thể, thông báo lỗi phải bao gồm tên của sự kiện không tương thích cờ.

Đối với công cụ di chuyển, hãy cân nhắc đóng góp cho Trình tạo bản dựng. Công cụ này có thể áp dụng bản sửa lỗi tự động cho các tệp BUILD, WORKSPACE.bzl. Ngoài ra, tính năng này cũng có thể báo cáo cảnh báo.

Triển khai

Tạo một cờ mới trong Bazel. Giá trị mặc định phải là false. Văn bản trợ giúp phải chứa URL của vấn đề GitHub. Khi tên cờ bắt đầu bằng incompatible_ thì cần có các thẻ siêu dữ liệu:

      metadataTags = {
        OptionMetadataTag.INCOMPATIBLE_CHANGE,
      },

Trong phần mô tả cam kết, hãy thêm nội dung tóm tắt ngắn gọn về cờ. Ngoài ra, hãy thêm RELNOTES: vào biểu mẫu sau: RELNOTES: --incompatible_name_of_flag has been added. See #xyz for details

Cam kết cũng phải cập nhật tài liệu liên quan để không cửa sổ thay đổi trong đó mã không nhất quán với tài liệu. Vì tài liệu được tạo phiên bản, các thay đổi đối với tài liệu sẽ không vô tình phát hành sớm.

Nhãn

Sau khi hợp nhất cam kết và áp dụng thay đổi không tương thích, hãy thêm nhãn migration-ready vấn đề trên GitHub.

Nếu phát hiện có vấn đề với cờ và người dùng chưa được di chuyển: xoá cờ migration-ready.

Nếu bạn có kế hoạch lật ngược bản phát hành chính tiếp theo, hãy thêm nhãn "breaking-change-X.0" vào vấn đề.

Đang cập nhật kho lưu trữ

Bazel CI thử nghiệm một danh sách các dự án quan trọng tại Bazel@HEAD + Hạ nguồn. Hầu hết trong số họ thường các phần phụ thuộc của các dự án Bazel khác, do đó, bạn cần di chuyển các dự án đó để bỏ chặn quá trình di chuyển cho cộng đồng nói chung. Để theo dõi trạng thái di chuyển của các dự án đó, bạn có thể sử dụng quy trình bazelisk-plus-incompatible-flags. Xem cách hoạt động của quy trình này tại đây.

Nhóm hỗ trợ nhà phát triển của chúng tôi sẽ giám sát nhãn migration-ready. Sau khi bạn thêm nhãn này vào vấn đề trên GitHub, chúng sẽ xử lý các thao tác sau:

  1. Tạo nhận xét trong vấn đề GitHub để theo dõi danh sách lỗi và các dự án hạ nguồn cần di chuyển (xem ví dụ)

  2. Gửi vấn đề về GitHub để thông báo cho chủ sở hữu của mọi dự án hạ nguồn bị hỏng do thay đổi không tương thích của bạn (xem ví dụ)

  3. Tiếp tục liên hệ để đảm bảo bạn đã giải quyết mọi vấn đề trước ngày phát hành dự kiến

Việc di chuyển các dự án trong quy trình hạ nguồn KHÔNG hoàn toàn thuộc trách nhiệm của tác giả thay đổi không tương thích, nhưng bạn có thể làm như sau để đẩy nhanh quá trình di chuyển và mang lại trải nghiệm dễ dàng hơn cho cả người dùng Bazel và Nhóm Bazel Green.

  1. Gửi PR để khắc phục các dự án hạ nguồn.

  2. Liên hệ với cộng đồng Bazel để được trợ giúp về việc di chuyển (ví dụ: Bazel Rules Authors SIG).

Tung cờ

Trước khi chuyển giá trị mặc định của cờ thành true, vui lòng đảm bảo rằng:

  • Các kho lưu trữ chính trong hệ sinh thái sẽ được di chuyển.

    Trên quy trình bazelisk-plus-incompatible-flags, cờ này sẽ xuất hiện bên dưới The following flags didn't break any passing Bazel team owned/co-owned projects.

  • Tất cả vấn đề trong danh sách kiểm tra này đều được đánh dấu là đã khắc phục/đã đóng.

  • Chúng tôi đã giải quyết các thắc mắc và mối lo ngại của người dùng.

Khi cờ đã sẵn sàng tung ra trong Bazel nhưng bị chặn trong quá trình di chuyển nội bộ tại Google, vui lòng cân nhắc đặt giá trị cờ thành false trong tệp blazerc nội bộ để bỏ chặn việc tung cờ. Bằng cách này, chúng tôi có thể đảm bảo rằng người dùng Bazel phụ thuộc vào hành vi mới theo mặc định sớm nhất có thể.

Khi thay đổi giá trị mặc định của cờ thành true, vui lòng:

  • Sử dụng RELNOTES[INC] trong phần mô tả cam kết, với định dạng sau: RELNOTES[INC]: --incompatible_name_of_flag is flipped to true. See #xyz for details Bạn có thể đưa thêm thông tin vào phần còn lại của nội dung mô tả cam kết.
  • Sử dụng Fixes #xyz trong phần mô tả để xử lý vấn đề GitHub khi lệnh xác nhận được hợp nhất.
  • Xem lại và cập nhật tài liệu nếu cần.
  • Hãy báo cáo một vấn đề mới #abc để theo dõi việc xoá cờ.

Xoá cờ

Sau khi lật cờ ở HEAD, bạn phải xoá cờ này khỏi Bazel. Khi bạn định xoá cờ không tương thích:

  • Hãy cân nhắc để người dùng có thêm thời gian di chuyển nếu đó là sự thay đổi không tương thích lớn. Tốt nhất là cờ này nên có trong ít nhất một bản phát hành chính.
  • Đối với cam kết xoá cờ này, hãy sử dụng Fixes #abc trong phần mô tả để vấn đề GitHub bị đóng khi xác nhận được hợp nhất.