Skip to content

docs(skills): record statuses:write startup_failure pattern - #1071

Merged
castrojo merged 1 commit into
testingfrom
docs/promote-statuses-startup-failure
Jun 30, 2026
Merged

docs(skills): record statuses:write startup_failure pattern#1071
castrojo merged 1 commit into
testingfrom
docs/promote-statuses-startup-failure

Conversation

@castrojo

@castrojo castrojo commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

Records the root cause and fix for promote startup_failures discovered during the 2026-06-23 promote pipeline recovery session.

When projectbluefin/actions@v1 adds a new permission to a reusable promote job, the dakota caller must mirror it in its top-level permissions: block. Missing permissions produce startup_failure with no log output.

No code changes — docs only.

Summary by CodeRabbit

  • Documentation
    • Added guidance for a GitHub Actions startup failure during workflow dispatches when required permissions are missing.
    • Clarified how to identify and fix permission-related regressions in reusable workflows.

@castrojo
castrojo requested a review from a team as a code owner June 23, 2026 23:26
@coderabbitai

coderabbitai Bot commented Jun 23, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: b43494c0-8a71-4911-bf5e-cef8aed973aa

📥 Commits

Reviewing files that changed from the base of the PR and between d10b2b1 and 8339925.

📒 Files selected for processing (1)
  • docs/skills/ci-reference.md

📝 Walkthrough

Walkthrough

Adds a new troubleshooting entry to docs/skills/ci-reference.md documenting a startup_failure error on promote-testing-to-main.yml dispatches caused by missing statuses: write in the caller's top-level permissions:, including the fix and detection guidance.

CI Reference Documentation

Layer / File(s) Summary
startup_failure / statuses:write troubleshooting entry
docs/skills/ci-reference.md
New subsection describing the symptom, root cause (caller permissions cap reusable workflow jobs), required statuses: write fix, and detection guideline for future projectbluefin/actions@v1 permission changes.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

  • projectbluefin/dakota#700: Documents a related startup_failure root cause for reusable-workflow dispatches involving missing/incorrect permissions: in the caller.
  • projectbluefin/dakota#802: Adds documentation on the broader startup_failure/missing top-level permissions pattern that this PR extends with the statuses: write case.
  • projectbluefin/dakota#817: Updates caller workflow top-level permissions for promote-testing-to-main.yml to fix startup_failure, directly related to the scenario documented here.

Suggested labels

area/ci, kind/improvement

Suggested reviewers

  • p5
  • tulilirockz

Poem

🐇 A hop through the workflow, a failure in sight,
statuses: write was missing — oh what a plight!
The caller must grant what the callee may need,
Or startup_failure will make your pipeline bleed.
Now documented clearly, the bunny hops free! 🌟

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch docs/promote-statuses-startup-failure

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions

Copy link
Copy Markdown
Contributor

Thanks for the PR! A maintainer will review it.

While you wait, make sure these pass locally:

just validate        # element graph check
just build default   # build the image
just boot-test       # confirm the desktop boots (exits 0 = pass)
just lint            # bootc container lint

If this PR fixes a bug, add verify steps to the linked issue so users can confirm the fix on their hardware after the next nightly ships:

```verify
ujust <something>   # what users should run to confirm the fix
```

@castrojo
castrojo changed the base branch from main to testing June 24, 2026 01:44
hanthor
hanthor previously approved these changes Jun 24, 2026

@hanthor hanthor left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM. Reviewed 2026-06-24.

When projectbluefin/actions@v1 adds a new job-level permission to the
reusable promote workflow, the caller must grant it at the top level.
Missing permissions cause startup_failure with no log output.

Symptom confirmed: 3 consecutive startup_failures on promote dispatch
after actions@v1 was updated to post validate commit status.
Fix: add statuses:write to promote-testing-to-main.yml permissions.

Assisted-by: Claude Sonnet 4.6 via GitHub Copilot

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@castrojo
castrojo force-pushed the docs/promote-statuses-startup-failure branch from 272038a to 8339925 Compare June 30, 2026 02:12
@castrojo
castrojo merged commit 557355b into testing Jun 30, 2026
4 of 8 checks passed
@castrojo
castrojo deleted the docs/promote-statuses-startup-failure branch June 30, 2026 02:13
github-actions Bot added a commit that referenced this pull request Jul 23, 2026
* docs: clean house — delete stale docs, align all docs with OCI-native daily factory (#1090)

Delete banned changelog/planning files:
- docs/factory/IMPROVEMENTS.md: banned changelog file (per skill-improvement.md)
- docs/epics.md: alpha-era planning doc with issues 200-548; GitHub board is source of truth

Fix factual errors in CI/promotion docs:
- docs/ci.md: rewrite stable promotion section; remove deleted promote-testing-to-main.yml
  refs and squash-PR flow; fix schedule note (daily 13:00 UTC); fix testing branch attribution
- AGENTS.md: fix stale promotion PR reference -> execute-release.yml description; fix
  weekly squash PR -> daily automated; fix :testing/:stable cadence description
- docs/skills/overview.md: fix image stream table (testing on testing branch, not main;
  daily not weekly promotion)
- docs/skills/quickstart.md: branch from upstream/testing; remove outdated human-approval
  step 5 (weekly-testing-promotion.yml + 2 approvals)
- docs/skills/aarch64.md: rewrite triggers (workflow_run + push + dispatch); remove
  deleted cache-warm.yml section; fix hard rules to remove deleted workflow refs
- docs/skills/workflow-map.md: move boot-check to publish.yml (where it actually is);
  remove it from execute-release.yml; add push trigger to build-aarch64.yml entry
- docs/skills/release-promotion.md: fix boot-check placement in promotion map and core process
- docs/skills/ci-tooling.md: replace sections 12+13 with DELETED pattern notes; remove
  stale red flag about sync-main-to-testing
- docs/skills/ci.md: add Lessons Learned banner noting pre-2026-06-23 entries may reference
  deleted workflows; fix execute-release quick reference (no boot-check)

Fix other doc errors:
- README.md: fix image stream table (testing branch, daily not weekly); remove duplicate
  Help section; fix agent-ready queue label (status/queued not queue/agent-ready)
- CONTRIBUTING.md: fix directory table (patches/+files/ instead of nonexistent junctions/
  +system_files/); fix bst install method (just bst, not pip); fix just check -> validate+lint
- docs/build.md: fix force-push note for release bookmark model

Add ci.md to routing tables:
- docs/skills/README.md: add ci.md as load-first for CI failures
- docs/SKILL.md: add ci.md entry to Task -> Skill table

Closes 0 issues. This is a documentation clean-house aligned with the OCI-native
automated factory that shipped 2026-06-23.

Assisted-by: Claude Sonnet 4.6 via GitHub Copilot

* docs(skills): record statuses:write startup_failure pattern (2026-06-23) (#1071)

When projectbluefin/actions@v1 adds a new job-level permission to the
reusable promote workflow, the caller must grant it at the top level.
Missing permissions cause startup_failure with no log output.

Symptom confirmed: 3 consecutive startup_failures on promote dispatch
after actions@v1 was updated to post validate commit status.
Fix: add statuses:write to promote-testing-to-main.yml permissions.

Assisted-by: Claude Sonnet 4.6 via GitHub Copilot

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* feat(bluefin): add Incus native library elements (1/2) (#1125)

* feat(bluefin): add Incus native library elements

Add the from-source BuildStream elements for Incus's native dependency
stack. None of these exist in the freedesktop-sdk / gnome-build-meta
junctions, so they are authored here:

- libuv (cmake)
- raft (autotools) — cowsql's Raft consensus
- cowsql (autotools) — embedded replicated SQLite (dqlite fork)
- liblxc (meson) — system-container runtime
- lxcfs (meson) — cgroup-aware /proc for containers

Consumed by bluefin/incus.bst in the follow-up PR. Validated by a full
oci/bluefin.bst image build.

Part of #1124

Assisted-by: Claude Opus 4.8 (1M context)

* fix(incus): remove obsolete cowsql raft configure flag

Assisted-by: Claude Sonnet 4.6 via GitHub Copilot

---------

Co-authored-by: Jorge O. Castro <jorge.castro@gmail.com>

* feat(bluefin): add Incus daemon, systemd integration, and ujust recipes (#1126)

Wire Incus into the image on top of the native library elements:

- bluefin/incus.bst: build incusd + incus + helpers from the vendored
  release tarball (CGO, -tags libsqlite3, static agent/migrate). Pulls
  QEMU/OVMF/virtiofsd/swtpm/dnsmasq/nftables/storage tooling from the
  junctions as runtime deps.
- deps.bst: add bluefin/incus.bst to the image manifest.
- files/incus/: systemd units (incus.socket/service, incus-startup,
  incus-user), preset (enables sockets + lxcfs), sysusers.d
  (incus-admin/incus groups), tmpfiles.d (/run/incus + state dirs),
  modules-load.d (vhost_vsock/vhost_net/nbd/br_netfilter),
  /etc/default/incus (INCUS_EDK2_PATH), and /etc/subuid+subgid.
- system.just: `ujust incus-init` and `ujust incus-status`.

Includes overlap-whitelist for /etc/subuid and /etc/subgid.
Builds green end-to-end (oci/bluefin.bst). Runtime boot/launch
validation tracked in the issue.

Closes #1124

Assisted-by: Claude Opus 4.8 via GitHub Copilot
Assisted-by: Claude Sonnet 4.6 via GitHub Copilot

Co-authored-by: Jorge O. Castro <jorge.castro@gmail.com>

* chore(deps): update taiki-e/install-action digest to 16b0581 (#1127)

Co-authored-by: mergeraptor[bot] <267480593+mergeraptor[bot]@users.noreply.github.com>

* chore: regenerate chunkah filemap and fakecap manifest

---------

Co-authored-by: Jorge O. Castro <jorge.castro@gmail.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Brian Ketelsen <bketelsen@gmail.com>
Co-authored-by: mergeraptor[bot] <267480593+mergeraptor[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@castrojo castrojo added 4-review A pull request is awaiting review. and removed area/ci labels Jul 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4-review A pull request is awaiting review.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants