Remove unused app service variable values 'repository' and 'dockerfile'

All Submissions:

[x] Have you followed the guidelines in our Contributing document?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[N/A] I have updated the documentation accordingly.
[N/A] I have added tests to cover my changes.
[x] All new and existing tests passed.
[x] My code follows the code style of this project.
[x] I ran lint checks locally prior to submission.
[x] Have you checked to ensure there aren't other open Pull Requests for the same update/change?

What is the current behavior?

Currently, the as-isolated-service-single-region template assigns and expects the fields repository and dockerfile; however, these are not used by the app services that are created. The result is a confusing addition of required but unused variables.

Issue Number: Closes #209

What is the new behavior?

Behavior of the deployment does not change; however removal of the unused variables helps clarify usage.

[x] Removed https://github.com/microsoft/cobalt/blob/master/infra/templates/az-isolated-service-single-region/variables.tf#L45-L46 so that unused and unnecessary assignments would not be required
[x] Updated integration and unit tests to remove assignments
[x] Updated template .tfvars to remove assignments

microsoft / cobalt

Remove unused app service variable values 'repository' and 'dockerfile' #312

All Submissions:

What is the current behavior?

What is the new behavior?

Does this introduce a breaking change?

Any relevant logs, error output, etc?

Other information