Implement --dry-run flag for snapshot load#272
Merged
anisaoshafi merged 2 commits intoJul 24, 2026
Conversation
anisaoshafi
force-pushed
the
devx-869-implement-dry-run-flag-for-snapshot-load
branch
4 times, most recently
from
May 28, 2026 17:56
6eba568 to
6cd63f7
Compare
anisaoshafi
marked this pull request as ready for review
May 29, 2026 11:12
anisaoshafi
requested review from
carole-lavillonniere and
silv-io
as code owners
May 29, 2026 11:12
anisaoshafi
marked this pull request as draft
May 29, 2026 16:46
anisaoshafi
force-pushed
the
devx-869-implement-dry-run-flag-for-snapshot-load
branch
from
July 22, 2026 16:36
6cd63f7 to
984508f
Compare
anisaoshafi
added
the
docs: skip
anisaoshafi
marked this pull request as ready for review
July 23, 2026 14:47
skyrpex
approved these changes
Jul 24, 2026
skyrpex
left a comment
skyrpex
left a comment
Contributor
There was a problem hiding this comment.
Looks good to me but I wonder: would it make sense to have a lstk snapshot diff command instead? 🤔
Collaborator
Author
Collaborator
Author
|
Merging for now, open to later feedback. |
anisaoshafi
deleted the
devx-869-implement-dry-run-flag-for-snapshot-load
branch
July 24, 2026 16:28
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
2 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.
Added
--dry-runto lstk snapshot load.When passed with a pod ref, it calls
GET /_localstack/pods/{name}/diffinstead of loading, and shows a per-service preview of what would be added or modified, without touching running state.Local snapshot refs return an error since the emulator has no diff endpoint for them.
The
--mergeflag is also accepted to contextualize the output.ℹ️ Deletions are not tracked, because it's not implemented in cloud pods' backend (in emulator).
ℹ️ During
lstk snapshot load --dry-run(pod refs only),--mergeis ignored becauseGET /_localstack/pods/{name}/diffendpoint which is computed by the emulator doesn't support a strategy param, so it has zero effect on the diff computation itself.