Skip to content

docs(tickets): reflect python-inference distroless trial merged state - #528

Merged
ayutaz merged 2 commits into
devfrom
docs/t-012-merged-status-sync
May 20, 2026
Merged

docs(tickets): reflect python-inference distroless trial merged state#528
ayutaz merged 2 commits into
devfrom
docs/t-012-merged-status-sync

Conversation

@ayutaz

@ayutaz ayutaz commented May 20, 2026

Copy link
Copy Markdown
Owner

Summary

python-inference distroless trial が merged された後、 対象 ticket の Status / PR フィールド / §Trial PR scope note のヘッダが「TBD / 起票予定」 表現のまま残っており、 trial 投入済の事実と HF Space promotion が未完である点が ticket 単体で読み取れない状態だった。 PR #525 で実施した webui / cpp-inference 版と同 pattern の post-merge ticket sync。 docs only の保守作業。

Affected Components

  • Documentation

Type

  • Documentation

Risk Level

  • patch (内部 docs 更新のみ、 production paths / build pipeline / contract gate 影響ゼロ)

Contract Impact

  • None — docs/spec/*.toml 不変更、 audit fixture (tests/fixtures/doc_examples_audit/audit.json) block 数 446 不変

変更内容

機能名 動作 これがないと起こること
python-inference ticket Status 明確化 「着手中 (trial PR scope: ...)」→「着手中 (trial 投入済 — PR #523 merged 2026-05-20、 base cgr.dev/chainguard/python:latest、 multi-stage python:3.13-slim-trixie builder)。 canonical 置換 + HF Space staging 検証は別 promotion PR」 へ書き換え trial 投入済の事実が ticket から読み取れず、 後続着手者が再着手する / promotion PR で HF Space staging cold start 検証必須を見逃す
PR フィールドを実 PR 番号に更新 feat/python-inference-distroless-trial (/create-pr skill で起票予定)」 → 「#523 (merged 2026-05-20、 base / builder 構成併記)」 PR 番号が ticket からトレースできず、 merge 履歴と ticket の対応が取れない
§Trial PR scope note ヘッダの PR 番号確定 「PR #TBD」→「PR #523、 merged 2026-05-20」 trial PR 中の議論経緯と実 PR が照合できない
変更履歴に 2026-05-20 行追加 投入ファイル / 新規 CI workflow / smoke test 実証範囲 + promotion で必要な検証 (CVE 数値計測 / HF Space staging cold start / canonical 置換) を明記 promotion PR 時に「trial で何が検証済 / 何が未検証か」 が ticket から判らず、 同じ範囲を二重検証する / 必須項目を抜かす

設計判断

  • Status を「着手中」のまま維持 — trial 投入は完了したが canonical 置換 + HF Space staging 検証が残るため。 Status を「完了」にすると promotion 観測期間中に未完作業が見えなくなるリスクが大きく、 PR docs(tickets): mark distroless webui/cpp-inference trial as merged #525 (webui / cpp-inference) と同じ「着手中 + Note 明記」 方式を踏襲
  • HF Space staging 検証の必要性を変更履歴に明示 — python-inference は M3 milestone で HF Space deploy 対象として AC-1.2 (user 手動 deploy 検証) が必須化されている唯一の image。 promotion PR でこの検証を抜かすと user-facing cold start 失敗 (R-4) に直結するため、 ticket の変更履歴に書き残す
  • canonical Dockerfile.cpu / docker-compose / deploy-huggingface.yml に一切触れない — production paths は trial 投入時点で不変更が前提。 ticket Status 整合のためだけにそれらの安定経路を触る必要はない
  • PR docs(tickets): mark distroless webui/cpp-inference trial as merged #525 と同 pattern を踏襲 — webui / cpp-inference の post-merge sync で確立した「Status 行リライト + PR 行更新 + 変更履歴追加」 の 3 点セットをそのまま適用、 cross-ticket での表記揺れを抑止

Test Plan

  • uv run python scripts/check_doc_examples.py audit --check-snapshot tests/fixtures/doc_examples_audit/audit.json で audit fixture 不変 (block 数 446、 exit 0) を確認
  • T-012 ticket を render し、 Status 行 / PR 行 / §Trial PR scope note ヘッダ / 変更履歴 の文言が canonical 不変更と promotion 別 PR (HF Space staging 検証必須) を明示していることを目視確認
  • git diff dev...HEAD -- docker/ .github/ src/ が空 (production paths への副作用なし) を確認
  • markdownlint / pre-commit が pass すること

Checklist

  • Tests pass locally (該当 contract gate / audit がすべて green)
  • No GPL-LGPL deps (該当なし、 docs only)
  • Documentation updated (本 PR 自体が documentation 更新)

Related Issues

なし (PR #523 の merge 後追い保守、 issue 紐付けなし)

PR #523 merged 2026-05-20 で python-inference の Dockerfile.cpu.distroless が投入されたが、 ticket Status / PR フィールド / Note の PR 番号が「TBD / 着手前」表現のまま残り、 trial 投入済の事実と HF Space promotion が未完である点が ticket から読み取れなかった。

- Status 行: 「trial 投入済 (PR #523 merged 2026-05-20、 base `cgr.dev/chainguard/python:latest`、 multi-stage `python:3.13-slim-trixie` builder)」 + canonical 置換は promotion PR (HF Space staging 検証後) であることを明記
- PR 行: 起票予定ブランチ名 → 実 PR 番号 #523 + merged 日付 + base image
- §Trial PR scope note のヘッダを「PR #TBD」→「PR #523、 merged 2026-05-20」
- 変更履歴に 2026-05-20 行追加 (投入ファイル / 新規 CI workflow / smoke test 実証範囲 + promotion で必要な検証 = CVE 数値計測 / HF Space staging cold start)

canonical Dockerfile.cpu / docker-compose / deploy-huggingface.yml は不変更、 ticket 整合性のみの保守的な更新。
Copilot AI review requested due to automatic review settings May 20, 2026 13:11
@github-actions

github-actions Bot commented May 20, 2026

Copy link
Copy Markdown
Contributor

Required status-check gate (deferred)

Head SHA f13fc37 is no longer the branch tip (latest: e2fee8f). Waiting for the new commit's spoke runs to complete before re-evaluating.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the internal ticket T-012 to reflect the post-merge state of the python-inference distroless trial work (docs-only maintenance), ensuring the ticket’s header fields and change log are synchronized with the merged implementation.

Changes:

  • Updated Status and PR fields to point to the merged trial PR and describe follow-up promotion work.
  • Updated the “Trial PR scope note” header to reference the concrete PR number and merge date.
  • Added a 2026-05-20 change-log entry documenting the trial merge and what remains for promotion.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread docs/tickets/tickets/T-012-distroless-python-inference.md Outdated
Comment thread docs/tickets/tickets/T-012-distroless-python-inference.md Outdated
… distroless trial

Copilot review (PR #528) 指摘の factual error 修正。 ticket Status 行と変更履歴に記載した base image / builder が、 実際の merged Dockerfile (docker/python-inference/Dockerfile.cpu.distroless) と乖離していた:

- base `cgr.dev/chainguard/python:latest` → `gcr.io/distroless/python3-debian12` (Debian-glibc 整合のため、 webui trial と同構成)
- builder `python:3.13-slim-trixie` → `python:3.11-slim-trixie` (final stage の Python 3.11 と minor 一致)

PR #523 の body draft の古い表現を元に書いてしまい、 実 file との照合を怠った結果。 promotion PR で base/builder 構成を参照するときに誤誘導するため修正必須。 §2.2 移行後 base image 候補比較表 (line 41 / 87) は ticket 設計時の論述として正しいため不変更。
@ayutaz
ayutaz merged commit b488cad into dev May 20, 2026
34 checks passed
@ayutaz
ayutaz deleted the docs/t-012-merged-status-sync branch May 20, 2026 14:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants