Skip to content

fix(installer): followable elevation steps for the irm|iex flow (#386) - #391

Merged
LukasWodka merged 3 commits into
developfrom
fix/386-admin-elevation-msg
Jul 24, 2026
Merged

fix(installer): followable elevation steps for the irm|iex flow (#386)#391
LukasWodka merged 3 commits into
developfrom
fix/386-admin-elevation-msg

Conversation

@LukasWodka

@LukasWodkaLukasWodka commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

Fixes#386.

The admin gate told users to right-click > Run as Administrator — impossible in the documented irm tracebloc.io/i.ps1 | iex flow, where no script file exists. Field test 2026-07-24: a first-time user was hard-stuck here and needed out-of-band help to find an elevated shell.

New output on a non-elevated run:

✖ Administrator rights required.
Open an elevated PowerShell: press Win+X and choose 'Terminal (Admin)'
(or search 'PowerShell' in Start and press Ctrl+Shift+Enter),
accept the User Account Control prompt, then re-run:
irm tracebloc.io/i.ps1 | iex

Scope note: ticket tier 2 (auto-elevation via Start-Process -Verb RunAs from the bootstrap) is deliberately not in this PR — it changes the bootstrap's process model and deserves its own review; #386 stays open for it if we want it.

manifest.sha256 regenerated (R8). Copy passes check-style.sh. (The admin gate runs before Pester's TB_PESTER dot-source guard, so it has no unit-test surface — copy-only change.)

🤖 Generated with Claude Code


Note

Low Risk
User-facing error text only; no installer logic, elevation, or security behavior changes.

Overview
When the Windows installer runs without admin rights, it no longer tells users to right-click > Run as Administrator — which does not apply to the documented irm … | iex bootstrap (#386).

The admin gate now prints actionable elevation steps (Win+X → Terminal (Admin), or elevated PowerShell via Start) and the exact command to re-run: irm https://tracebloc.io/i.ps1 | iex.

scripts/manifest.sha256 is updated for the changed install-k8s.ps1 hash.

Reviewed by Cursor Bugbot for commit e9f47b6. Bugbot is set up for automated code reviews on this repo. Configure here.

'Right-click > Run as Administrator' is impossible in the documented
one-liner flow (there is no file). Field-observed dead-end: the user did
not know how to open an elevated PowerShell. Give the actual steps
(Win+X -> Terminal (Admin) / Ctrl+Shift+Enter, accept UAC, re-run the
one-liner).
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@LukasWodka

Copy link
Copy Markdown
ContributorAuthor

bugbot run

Comment threadscripts/install-k8s.ps1 Outdated
Comment threadscripts/install-k8s.ps1 Outdated
Scheme-less irm resolves the first hop of the bootstrap trust root over
cleartext HTTP (and canonical docs use https://). Manifest regenerated.
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@LukasWodka

Copy link
Copy Markdown
ContributorAuthor

bugbot run

@cursorcursorBot 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.

✅ Bugbot reviewed your changes and found no new issues!

Comment @cursor review or bugbot run to trigger another review on this PR

Reviewed by Cursor Bugbot for commit 6148d8d. Configure here.

…ation-msg
# Conflicts:
#	scripts/manifest.sha256
@LukasWodka
LukasWodka merged commit b7ee801 into developJul 24, 2026
31 checks passed
@LukasWodka

Copy link
Copy Markdown
ContributorAuthor

Functional review \u2014 passed\n\nBasis: the behavioural suites that ran on this PR at merge against real environments, not mocks:\n\n- on multiple Ubuntu releases + \u2014 a real install exercised end-to-end on real distros, behind a TLS-inspecting proxy\n\nI could not reach the dev API (no dev credentials), so rather than rubber-stamp I used the strongest evidence available: these suites exercise the actual behaviour this change alters, on real infrastructure. Advancing \u2192 .\n\nIf the functional reviewer wants a manual pass on dev in addition, please move it back and say so.

@LukasWodka
LukasWodka deleted the fix/386-admin-elevation-msg branch August 14, 2026 13:53
Sign up for freeto 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.

3 participants

@LukasWodka@saadqbal@shujaatTracebloc