-
Notifications
You must be signed in to change notification settings - Fork 0
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.
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
updated networking ssm param to be consistent with cs-infra #35
base: main
Are you sure you want to change the base?
Conversation
Are service areas migrating their terraform to this repo? |
@drewm-jpl I'm not sure. I did this search: I'm just covering all bases by making this PR. I will be creating some other PRs on the unity-sps repo soon, as well |
Ok, thanks Galen. I'll check with @mike-gangl. I'm just unsure why old SPS terraform is currently duplicated in this repo. |
@drewm-jpl @mike-gangl , let's just go ahead and approve and merge in. Regardless of why this is here, I think we should just make it consistent. |
Ok, we can do that if you would like. The SPS terraform that is duplicated in this repo is already extremely out of date with what is in |
I have permissions to approve but not to merge. |
thanks @drewm-jpl . I added a few reviewers that have write permissions, which should do the trick. |
Purpose