-
Notifications
You must be signed in to change notification settings - Fork 369
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
Referencer for SubnetGroup AWS ElasticCache #314
Referencer for SubnetGroup AWS ElasticCache #314
Conversation
6f2d975
to
5044f86
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome! Thank you @krishchow 🙌 A few comments then we should be able to get this merged!
5044f86
to
6ad977c
Compare
6ad977c
to
b84511c
Compare
@krishchow could you do a rebase here? |
Signed-off-by: Krish Chowdhary <krish@redhat.com>
b84511c
to
3f44344
Compare
@hasheddan done! :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
…roupSelector Referencer for SubnetGroup AWS ElasticCache
…roupSelector Referencer for SubnetGroup AWS ElasticCache
Description of your changes
Fixes #313
Checklist
I have:
make reviewable
to ensure this PR is ready for review.app.yaml
to include any new role permissions.