Repository navigation
render.sh: 生成物の確かめを manifest の書き方に左右されないようにする - #786
Merged
Merged
Conversation
talosctl v1.15.0-beta.0 から、inline の manifest が 1 行のエスケープ文字列ではなく ブロックで書き出されるようになり、エスケープした引用符と改行を探していた確かめが落ちた (#778 の boot)。エスケープを戻して字下げを落としたものを見るようにして、どちらの書き方でも通す。 Namespace の 3 行は grep では行をまたげないので awk で見る。 Co-Authored-By: Claude <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01T8TqVdo6LUrWKdVQgbScVU
|
Claude finished @5ym's task in 20s —— View job レビュー結果: 問題なし
軽い補足 (指摘ではありません)
テストについては、このスクリプトの確かめ自体が検証にあたります。alpha.2 と beta.0 の両方で |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
なぜ
#778 (Talos v1.15.0-beta.0) の
bootが、起動より前のbootstrap/render.shの確かめで落ちている。talosctl は inline の manifest を、v1.15.0-alpha.2 までは 1 行のエスケープ文字列で、beta.0 からはブロック (
|-) で書き出す (生成物は 1000 行 → 9591 行)。確かめはエスケープした引用符と改行を探していたので、中身は同じなのに見つからなくなった。何を変えたか
kind: Namespace/metadata:/name: …) は grep では行をまたげないので、awk で 3 行が続いていることを見る。確かめる中身は減らしていない。
確かめたこと
gen configを回し、前者がエスケープ文字列、後者がブロックになることを再現した。bootは alpha.2 (古い書き方) で回る。マージ後に chore(deps): update talos to v1.15.0-beta.0 #778 が beta.0 (新しい書き方) で回る。🤖 Generated with Claude Code
https://claude.ai/code/session_01T8TqVdo6LUrWKdVQgbScVU