Skip to content

Add new options to Wait-AzureRmHDInsightJob #2464

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

Merged
merged 5 commits into from
Jun 28, 2016
Merged

Conversation

pattipaka
Copy link
Contributor

No description provided.

@azurecla
Copy link

Hi @pattipaka, 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 (supattip). If you're full-time, we DON'T require a contribution license agreement.



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

TTYL, AZPRBOT;

@markcowl
Copy link
Member

@pattipaka Your tests are not running as part of check-in, please add your test assembly here: https://github.com/Azure/azure-powershell/blob/dev/AzurePowershell.Test.targets#L91 as part of this PR.

@markcowl
Copy link
Member

@markcowl
Copy link
Member

@pattipaka There are merge conflicts, please update

@markcowl
Copy link
Member

markcowl commented Jun 27, 2016

@pattipaka
Copy link
Contributor Author

The issue seems to be with original branch itself. http://azuresdkci.cloudapp.net/view/1-AzurePowerShell/job/powershell-demand/924/consoleFull

Looks like kore poull requests may be effected due to this?
http://azuresdkci.cloudapp.net/view/1-AzurePowerShell/job/powershell-demand/921/consoleFull

@pattipaka
Copy link
Contributor Author

FYI, the changes I made are limited to HDInsight solution and it doesn't effect any other module.

@markcowl markcowl merged commit 2e32322 into Azure:dev Jun 28, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants