Skip to content

fix: SSM prefix #2704

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 1 commit into from
Dec 1, 2022
Merged

fix: SSM prefix #2704

merged 1 commit into from
Dec 1, 2022

Conversation

npalm
Copy link
Member

@npalm npalm commented Nov 30, 2022

SSM prefix not applied for path.

  • Use prefix for all SSM paramaters
  • Moved prefix to path instead of name for mult-runner
  • Add optionals for ssm path variable

@npalm npalm requested a review from ScottGuymer November 30, 2022 16:32
GuptaNavdeep1983
GuptaNavdeep1983 reviewed Nov 30, 2022
View reviewed changes
@npalm npalm merged commit 87a1944 into next Dec 1, 2022
@npalm npalm deleted the fix/ssm-prefix branch December 1, 2022 06:45
npalm added a commit that referenced this pull request Dec 13, 2022
@npalm
fix: SSM prefix (#2704)
6b25c31
npalm added a commit that referenced this pull request Dec 13, 2022
@npalm
fix: SSM prefix (#2704)
8afffad
npalm added a commit that referenced this pull request Dec 28, 2022
@npalm
fix: SSM prefix (#2704)
cfd8bd8
npalm added a commit that referenced this pull request Dec 28, 2022
@npalm
fix: SSM prefix (#2704)
d1c1497
npalm added a commit that referenced this pull request Dec 28, 2022
@npalm
chore: SSM prefix (#2704)
12a403d
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@GuptaNavdeep1983 GuptaNavdeep1983 GuptaNavdeep1983 approved these changes

@ScottGuymer ScottGuymer Awaiting requested review from ScottGuymer

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@npalm @GuptaNavdeep1983