Skip to content

ci: tighten Linux smoke endpoint posture (TIN-1540) - #394

Merged
Jesssullivan merged 1 commit into
mainfrom
jess/linux-smoke-storage-posture-tin-1540
May 19, 2026
Merged

ci: tighten Linux smoke endpoint posture (TIN-1540)#394
Jesssullivan merged 1 commit into
mainfrom
jess/linux-smoke-storage-posture-tin-1540

Conversation

@Jesssullivan

Copy link
Copy Markdown
Owner

Summary

  • factor TCFS smoke endpoint posture checks into a reusable Python preflight helper
  • make GitHub-hosted Linux package smoke require public HTTPS storage endpoints
  • keep self-hosted/private Linux runner labels eligible for private or plaintext smoke backends
  • add a local regression test for hosted/public vs private-runner endpoint classification

Why

The latest Linux postinstall smoke failed before package install because the hosted Ubuntu runner could not resolve the private SeaweedFS endpoint. This makes that contract explicit: hosted runners need public HTTPS storage; private endpoints require a self-hosted runner.

Validation

  • python3 scripts/test-tcfs-smoke-endpoint-preflight.py
  • python3 -m py_compile scripts/tcfs-smoke-endpoint-preflight.py scripts/test-tcfs-smoke-endpoint-preflight.py
  • ruby -ryaml -e 'YAML.load_file(ARGV.fetch(0))' .github/workflows/linux-postinstall-smoke.yml
  • ruby -ryaml -e 'YAML.load_file(ARGV.fetch(0))' Taskfile.yaml
  • git diff --check origin/main...HEAD

Refs: TIN-1540, TIN-1422, TIN-1546

@Jesssullivan

Copy link
Copy Markdown
Owner Author

Ready-to-diagnose note:

This PR intentionally does not claim Linux package smoke is green. It tightens the preflight so the next run clearly separates:

  • hosted Ubuntu + public HTTPS endpoint: allowed
  • hosted Ubuntu + tailnet/bare/private/plaintext endpoint: fail fast
  • self-hosted/private Linux runner + private/plaintext endpoint: allowed

Focused validation already run locally:

  • python3 scripts/test-tcfs-smoke-endpoint-preflight.py
  • python3 -m py_compile scripts/tcfs-smoke-endpoint-preflight.py scripts/test-tcfs-smoke-endpoint-preflight.py
  • workflow YAML parse
  • Taskfile YAML parse
  • git diff --check origin/main...HEAD

@Jesssullivan
Jesssullivan marked this pull request as ready for review May 19, 2026 07:54
@Jesssullivan
Jesssullivan merged commit 156f95c into main May 19, 2026
12 checks passed
@Jesssullivan
Jesssullivan deleted the jess/linux-smoke-storage-posture-tin-1540 branch May 19, 2026 07:54
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.

1 participant