-
Notifications
You must be signed in to change notification settings - Fork 90
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
ec_deployment: Add support for deployment aliases #298
Conversation
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 for the .md files
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.
🏅
I'll take a look. |
Adds support for deployment aliases on both the `ec_deployment` resource and `ec_deployment` datasource. Signed-off-by: Marc Lopez <marc5.12@outlook.com>
Signed-off-by: Marc Lopez <marc5.12@outlook.com>
45ed7f4
to
b5520f4
Compare
Signed-off-by: Marc Lopez <marc5.12@outlook.com>
Signed-off-by: Marc Lopez <marc5.12@outlook.com>
Tests look alright, I think we can merge this. @karencfv |
Description
Adds support for deployment aliases on both the
ec_deployment
resourceand
ec_deployment
datasource.Related Issues
Closes #287
How Has This Been Tested?
Manual, unit and acceptance tested.
Types of Changes