-
Notifications
You must be signed in to change notification settings - Fork 278
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
Add ACR service connection resource #119
Conversation
6484252
to
d7a6a60
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.
Looks great! Thank you so much for contributing. I've only got a few nitpicky comments :)
azuredevops/internal/service/serviceendpoint/resource_serviceendpoint_azurecr.go
Outdated
Show resolved
Hide resolved
# azure container registry service connection | ||
resource "azuredevops_serviceendpoint_azurecr" "azurecr" { | ||
project_id = azuredevops_project.project.id | ||
service_endpoint_name = "Sample AzureCR" |
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.
Could you also format this terraform?
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 don't think this is formatted still
These have different indentation:
service_endpoint_name = "Sample AzureCR"
resource_group = "sample-rg"
349462a
to
be2f196
Compare
So that an ACR service connection resource can be defined in Terraform.
be2f196
to
92d8899
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.
LGTM
So that an ACR service connection resource can be defined in
Terraform.
All Submissions:
What about the current behavior has changed?
Added
azuredevops_serviceendpoint_azurecr
, which allows to create an ACR service connection.Issue Number: #33
Does this introduce a change to
go.mod
,go.sum
orvendor/
?Does this introduce a breaking change?
Any relevant logs, error output, etc?
(If it’s long, please paste to https://ghostbin.com/ and insert the link here.)
Other information