DOC-425: Remove stale &&/||/redirection limitation from SSM docs - #932
Draft
quetzalliwrites wants to merge 1 commit into
Draft
DOC-425: Remove stale &&/||/redirection limitation from SSM docs#932quetzalliwrites wants to merge 1 commit into
quetzalliwrites wants to merge 1 commit into
Conversation
Verified against localstack/localstack-pro:latest (2026-09-01): SSM Run Command invokes a real shell (sh -c) under the hood, so &&, ||, and > all work correctly today. This claim was left over from an earlier, more limited implementation. The adjacent claim about stdout/stderr not being captured on a non-zero exit code is still accurate and is kept (tracked separately as a feature request). Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
quetzalliwrites
requested review from
HarshCasper,
giograno and
viren-nadkarni
as code owners
September 9, 2026 19:28
Deploying localstack-docs with
|
| Latest commit: |
8802540
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://efb80f57.localstack-docs.pages.dev |
| Branch Preview URL: | https://doc-425-ssm-run-command-docs.localstack-docs.pages.dev |
Member
|
This is not actively tested. It may have worked for the customer for their particular script, I want to make sure this is covered by our test suite. Please hold this change until AWS-1817 is merged. |
Collaborator
Author
|
FYI @virtkot (edit), ty for logging this ticket. Viren has confirmed he needs to first check if this is covered by their test suite. Until we hear back from ENG, I'm moving this to draft mode. |
quetzalliwrites
marked this pull request as draft
September 10, 2026 13:46
Contributor
|
Collaborator
Author
|
ha! sorry about the spam tagging you @kostas-localstack 😸 you're quite right, I meant the other Kosta 😅 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
A customer (flagged via NISC evaluation trial) quoted our own docs claiming SSM Run Command doesn't support
&&,||, or redirection (>). This was verified againstlocalstack/localstack-pro:lateston 2026-09-01: all three work correctly, since SSM Run Command invokes a real shell (sh -c) under the hood rather than a restricted command parser. The claim was stale, left over from an earlier, more limited implementation.Removed the
&&/||/redirection bullet from the SSM service doc's "Current Limitations" section. The adjacent bullet about stdout/stderr not being captured on a non-zero exit code is still accurate and was left in place (tracked separately as a feature request, not a docs issue).Linear ticket
https://linear.app/localstack/issue/DOC-425/ssm-run-command-docs-incorrectly-list-andand-and-redirection-as
Test plan
npx astro buildcompletes successfully (417 pages built)