Skip to content

Added UninstallChefClient option in Set-AzureVMChefExtension for ASM#1674

Merged
hovsepm merged 1 commit intoAzure:devfrom
MSys-IT:vj/add_uninstall_chef_client_option
Feb 23, 2016
Merged

Added UninstallChefClient option in Set-AzureVMChefExtension for ASM#1674
hovsepm merged 1 commit intoAzure:devfrom
MSys-IT:vj/add_uninstall_chef_client_option

Conversation

@Vasu1105
Copy link
Copy Markdown

@Vasu1105 Vasu1105 commented Jan 19, 2016

No description provided.

@azuresdkci
Copy link
Copy Markdown

azuresdkci commented Jan 19, 2016

Can one of the admins verify this patch?

@azurecla
Copy link
Copy Markdown

azurecla commented Jan 19, 2016

Hi @Vasu1105, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla.azure.com.

TTYL, AZPRBOT;

@hovsepm
Copy link
Copy Markdown
Contributor

hovsepm commented Jan 19, 2016

@Vasu1105 Please sign the CLA so we can accept your code changes.

@azurecla
Copy link
Copy Markdown

azurecla commented Jan 20, 2016

@Vasu1105, Thanks for signing the contribution license agreement so quickly! Actual humans will now validate the agreement and then evaluate the PR.

Thanks, AZPRBOT;

@markcowl
Copy link
Copy Markdown
Member

markcowl commented Jan 20, 2016

@azuresdkci add to whitelist

@Vasu1105
Copy link
Copy Markdown
Author

Vasu1105 commented Jan 22, 2016

@hovsepm We signed the CLA. Please review this PR .

@hovsepm
Copy link
Copy Markdown
Contributor

hovsepm commented Jan 26, 2016

@Vasu1105 I've asked the owner team to review your contribution.

@hovsepm hovsepm added the Compute label Feb 4, 2016
@hovsepm hovsepm assigned hyonholee and unassigned hyonholee Feb 4, 2016
@adamedx
Copy link
Copy Markdown

adamedx commented Feb 5, 2016

@hovsepm, any updates? Looks like this passes all the tests.

@andrewelizondo
Copy link
Copy Markdown

andrewelizondo commented Feb 19, 2016

LGTM 👍

hovsepm pushed a commit that referenced this pull request Feb 23, 2016
Merge pull request #1674 from ClogenyTechnologies/vj/add_uninstall_ch…
92d6174 
…ef_client_option

Added UninstallChefClient option  in Set-AzureVMChefExtension for ASM
@hovsepm hovsepm merged commit 92d6174 into Azure:dev Feb 23, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

Compute

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

8 participants

@Vasu1105 @azuresdkci @azurecla @hovsepm @markcowl @adamedx @andrewelizondo @hyonholee