Panduan untuk meluncurkan perubahan yang dapat menyebabkan gangguan

Laporkan masalah Lihat sumber Nightly · 7.4 . 7.3 · 7.2 · 7.1 · 7.0 · 6.5

Kami tidak dapat menghindari perubahan yang menyebabkan error pada Bazel. Kita harus mengubah desain dan memperbaiki hal-hal yang tidak berfungsi dengan baik. Namun, kita perlu memastikan bahwa komunitas dan ekosistem Bazel dapat mengikutinya. Untuk itu, project Bazel telah mengadopsi kebijakan kompatibilitas mundur. Dokumen ini menjelaskan proses bagi kontributor Bazel dalam melakukan perubahan yang dapat menyebabkan gangguan di Bazel agar mematuhi kebijakan ini.

  1. Ikuti kebijakan dokumen desain.

  2. Laporkan masalah GitHub.

  3. Terapkan perubahan.

  4. Perbarui label.

  5. Memperbarui repositori.

  6. Balik tanda yang tidak kompatibel.

Masalah GitHub

Laporkan masalah GitHub di repositori Bazel. Lihat contoh.

Sebaiknya:

  • Judulnya dimulai dengan nama flag (nama flag akan diawali dengan incompatible_).

  • Anda menambahkan label incompatible-change.

  • Deskripsi berisi deskripsi perubahan dan link ke dokumen desain yang relevan.

  • Deskripsi berisi resep migrasi, untuk menjelaskan kepada pengguna cara memperbarui kode mereka. Idealnya, jika perubahan bersifat mekanis, sertakan link ke alat migrasi.

  • Deskripsi menyertakan contoh pesan error yang akan diterima pengguna jika mereka tidak melakukan migrasi. Tindakan ini akan membuat masalah GitHub lebih mudah ditemukan dari mesin telusur. Pastikan pesan error bermanfaat dan dapat ditindaklanjuti. Jika memungkinkan, pesan error harus menyertakan nama flag yang tidak kompatibel.

Untuk alat migrasi, pertimbangkan untuk berkontribusi pada Buildifier. Alat ini dapat menerapkan perbaikan otomatis ke file BUILD, WORKSPACE, dan .bzl. Alat ini juga dapat melaporkan peringatan.

Penerapan

Buat flag baru di Bazel. Nilai defaultnya harus salah. Teks bantuan harus berisi URL masalah GitHub. Karena nama tanda dimulai dengan incompatible_, tanda ini memerlukan tag metadata:

      metadataTags = {
        OptionMetadataTag.INCOMPATIBLE_CHANGE,
      },

Dalam deskripsi commit, tambahkan ringkasan singkat flag tersebut. Tambahkan juga RELNOTES: dalam bentuk berikut: RELNOTES: --incompatible_name_of_flag has been added. See #xyz for details

Commit juga harus memperbarui dokumentasi yang relevan, sehingga tidak ada periode commit saat kode tidak konsisten dengan dokumen. Karena dokumentasi kami memiliki versi, perubahan pada dokumen tidak akan dirilis secara tidak sengaja sebelum waktunya.

Label

Setelah commit digabungkan dan perubahan yang tidak kompatibel siap digunakan, tambahkan label migration-ready ke masalah GitHub.

Jika masalah ditemukan pada tanda dan pengguna belum diharapkan untuk bermigrasi: hapus tanda migration-ready.

Jika Anda berencana untuk mengaktifkan tanda tersebut di rilis utama berikutnya, tambahkan label `breaking-change-X.0" ke masalah tersebut.

Mengupdate repositori

Bazel CI menguji daftar project penting di Bazel@HEAD + Downstream. Sebagian besar darinya sering kali merupakan dependensi dari project Bazel lainnya, sehingga penting untuk memigrasikannya guna menghentikan pemblokiran migrasi untuk komunitas yang lebih luas. Untuk memantau status migrasi project tersebut, Anda dapat menggunakan pipeline bazelisk-plus-incompatible-flags. Lihat cara kerja pipeline ini di sini.

Tim dukungan developer kami memantau label migration-ready. Setelah Anda menambahkan label ini ke masalah GitHub, mereka akan menangani hal berikut:

  1. Buat komentar di masalah GitHub untuk melacak daftar kegagalan dan project downstream yang perlu dimigrasikan (lihat contoh)

  2. Laporkan masalah Github untuk memberi tahu pemilik setiap project downstream yang rusak karena perubahan Anda yang tidak kompatibel (lihat contoh)

  3. Tindak lanjuti untuk memastikan semua masalah ditangani sebelum tanggal rilis target

Memigrasikan project di pipeline downstream BUKAN sepenuhnya tanggung jawab penulis perubahan yang tidak kompatibel, tetapi Anda dapat melakukan hal berikut untuk mempercepat migrasi dan mempermudah pengguna Bazel dan Tim Hijau Bazel.

  1. Mengirim PR untuk memperbaiki project downstream.

  2. Hubungi komunitas Bazel untuk mendapatkan bantuan terkait migrasi (misalnya, Bazel Rules Authors SIG).

Membalik tanda

Sebelum membalik nilai default tanda ke benar (true), pastikan bahwa:

  • Repositori inti di ekosistem dimigrasikan.

    Di pipeline bazelisk-plus-incompatible-flags, tanda akan muncul di bagian The following flags didn't break any passing Bazel team owned/co-owned projects.

  • Semua masalah dalam checklist ditandai sebagai telah diperbaiki/ditutup.

  • Masalah dan pertanyaan pengguna telah diselesaikan.

Jika flag siap dibalik di Bazel, tetapi diblokir pada migrasi internal di Google, sebaiknya tetapkan nilai flag ke salah (false) di file blazerc internal untuk membatalkan pemblokiran flag flip. Dengan melakukan hal ini, kita dapat memastikan pengguna Bazel bergantung pada perilaku baru secara default sedini mungkin.

Saat mengubah flag default menjadi benar, harap:

  • Gunakan RELNOTES[INC] dalam deskripsi commit, dengan format berikut: RELNOTES[INC]: --incompatible_name_of_flag is flipped to true. See #xyz for details Anda dapat menyertakan informasi tambahan dalam deskripsi commit lainnya.
  • Gunakan Fixes #xyz dalam deskripsi, sehingga masalah GitHub ditutup saat commit digabungkan.
  • Tinjau dan perbarui dokumentasi jika perlu.
  • Ajukan masalah baru #abc untuk melacak penghapusan tanda.

Menghapus tanda

Setelah tanda dibalik di HEAD, tanda tersebut pada akhirnya akan dihapus dari Bazel. Jika Anda berencana menghapus flag yang tidak kompatibel:

  • Pertimbangkan untuk memberikan lebih banyak waktu bagi pengguna untuk bermigrasi jika ini adalah perubahan besar yang tidak kompatibel. Idealnya, flag harus tersedia di setidaknya satu rilis utama.
  • Untuk commit yang menghapus flag, gunakan Fixes #abc dalam deskripsi sehingga masalah GitHub ditutup saat commit digabungkan.