scripts: make --service a required argument

[matanl-starkware]

Drop the implicit Service.Core default for --service in update_config_and_restart_nodes
and require it explicitly, so the targeted service is always a conscious choice
rather than silently defaulting to Core.

Co-Authored-By: Claude Opus 4.8 (1M context) noreply@anthropic.com

[cursor]

PR Summary

Low Risk
Operational CLI change only; scripts or runbooks that omitted -j will fail until updated, with no runtime auth or data-path impact.

Overview
The prod script update_config_and_restart_nodes.py no longer assumes Core when -j / --service is omitted: that flag is now required, and the help text no longer advertises a default.

Callers must pass the service explicitly (e.g. -j Core, -j Gateway), which avoids accidentally updating Core configmaps/pods when another service was intended.

^{Reviewed by Cursor Bugbot for commit 64e946b. Bugbot is set up for automated code reviews on this repo. Configure here.}

[matanl-starkware]

• scripts: make --service a required argument #14306 👈 (View in Graphite)
• scripts: confirm before restarting all Core pods at once #14305
• scripts: add Committer to Service options #14304
• scripts: clean up restart-strategy and service help output #14303
• scripts: restart and wait on nodes in parallel #14302
• scripts: make metrics gate() thread-safe #14301
• scripts: add run_in_parallel helper for prod ops #14300
• ci: fix hybrid system test for prod-script changes and k3d install #14317
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

[reviewable-StarkWare]

This change is 

[ron-starkware]

@ron-starkware reviewed 1 file and all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on matanl-starkware).

[ron-starkware]

@ron-starkware reviewed 1 file and all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on matanl-starkware).

[matanl-starkware]

@ron-starkware — last one in the stack 🙏 #14305 merged, so this rebased onto main and its Reviewable check reset. Re-approve this revision and we're done! (Rebase only — no code changes.)

[graphite-app]

Merge activity

• Jun 3, 9:49 PM UTC: Graphite rebased this pull request, because this pull request is set to merge when ready.