Skip to content

HDInsight: Powershell changes for creating/deleting IaaS clusters #251

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 7 commits into from
Mar 19, 2015
Merged

HDInsight: Powershell changes for creating/deleting IaaS clusters #251

merged 7 commits into from
Mar 19, 2015

Conversation

vhamine
Copy link
Contributor

@vhamine vhamine commented Mar 10, 2015

No description provided.

@azuresdkci
Copy link

Can one of the admins verify this patch?

@azurecla
Copy link

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


It looks like you're working at Microsoft (vhamine). If you're full-time, we DON'T require a contribution license agreement.



If you are a vendor, or work for Microsoft Open Technologies, DO please sign the electronic contribution license agreement. It will take 2 minutes and there's no faxing! https://cla.azure.com.

TTYL, AZPRBOT;

@hovsepm
Copy link
Contributor

hovsepm commented Mar 11, 2015

@azuresdkci add to whitelist

@AzureRT
Copy link
Contributor

AzureRT commented Mar 11, 2015

Can one of the admins verify this patch?

@vhamine
Copy link
Contributor Author

vhamine commented Mar 16, 2015

The test failure does not appear to be related to my changes (which are isolated to HDInsight cmdlets). Is there any action item on my side before we proceed with the merge.

@vhamine
Copy link
Contributor Author

vhamine commented Mar 18, 2015

Can someone with write access please merge these changes?

hovsepm
hovsepm reviewed Mar 18, 2015
View reviewed changes
src/ServiceManagement/HDInsight/Commands.HDInsight/Model/DataObjects/AzureHDInsightCluster.cs
@@ -188,5 +188,13 @@ public string VersionStatus
{
get { return this.cluster.VersionStatus.ToString(); }
}

/// <summary>
/// Gets the type of Operating System installed on cluster nodes.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please fix the spacing in the comments

ogail added a commit that referenced this pull request Mar 19, 2015
@ogail
Merge pull request #251 from vhamine/dev
159d126 
HDInsight: Powershell changes for creating/deleting IaaS clusters
@ogail ogail merged commit 159d126 into Azure:dev Mar 19, 2015
huangpf added a commit to AzureRT/azure-powershell that referenced this pull request Dec 15, 2015
@huangpf
Merge pull request Azure#251 from huangpf/dev
a1b08f2 
Dev
markcowl pushed a commit that referenced this pull request Apr 28, 2016
@pikumarmsft16
Merge pull request #251 from MabOneSdk/pikumar-dev1
ad81bb1 
Adding Exceptions to Resx file
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
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@vhamine @azuresdkci @azurecla @hovsepm @AzureRT @ogail