Skip to content
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

Added support for custom certificates #6467

Merged
merged 4 commits into from
Mar 10, 2018
Merged

Added support for custom certificates #6467

merged 4 commits into from
Mar 10, 2018

Conversation

ross-bohr
Copy link
Contributor

@ross-bohr ross-bohr commented Feb 19, 2018

This implements the issue #6418 which aims to add support for adding custom certificates to the Azure Cloud Service when the VSTS task AzureCloudPowerShellDeployment creates a new ACS.

Known issues: Translation of resource strings to all languages besides en-US is not yet provided, thus the necessary resource strings are imported to all localized resource files "as is" (i.e. in English language). Should you direct me to a translation service, I will add the translations as well.

@Ajay-MS
Copy link

Ajay-MS commented Feb 26, 2018

@ross-bohr

I took a look at PR today. Overall logic seems fine to me. I will test the task and will approve changes.

@ross-bohr
Copy link
Contributor Author

@Ajay-MS

Did you have the chance to test the task so far?

@Ajay-MS
Copy link

Ajay-MS commented Mar 9, 2018

@ross-bohr

I apologize for being a delay in this. I have done testing with few of scenario's but not completed.

I will do remaining testing and merge this PR by this weekend.

@Ajay-MS Ajay-MS merged commit 5bbb7f0 into microsoft:master Mar 10, 2018
@ross-bohr ross-bohr deleted the add-support-for-custom-certificates branch March 12, 2018 15:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants