Skip to content

Added iaas support and tests based on swagger for Azure Batch #2061

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 51 commits into from
Apr 19, 2016

Conversation

brnleehng
Copy link
Contributor

Fixed all the scenario tests and unit tests that were using hyak code generator

@azurecla
Copy link

azurecla commented Apr 6, 2016

Hi @brnleehng, 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 (brhoan). 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;

@azuresdkci
Copy link

Can one of the admins verify this patch?

@hovsepm
Copy link
Contributor

hovsepm commented Apr 6, 2016

@azuresdkci add to whitelist

@brnleehng
Copy link
Contributor Author

The test that failed was the ProfileModuleTests.WarningOnIncompatibleVersions. I did a diff with my profile module and dev. It looks identical. The only changes I made were in the batch module. How should I go about fixing this? Thanks!

@hovsepm
Copy link
Contributor

hovsepm commented Apr 8, 2016

@hovsepm
Copy link
Contributor

hovsepm commented Apr 11, 2016

@hovsepm
Copy link
Contributor

hovsepm commented Apr 12, 2016

@hovsepm
Copy link
Contributor

hovsepm commented Apr 12, 2016

@hovsepm
Copy link
Contributor

hovsepm commented Apr 13, 2016

@hovsepm
Copy link
Contributor

hovsepm commented Apr 13, 2016

LGTM.

@hovsepm hovsepm merged commit c12bca2 into Azure:dev Apr 19, 2016
@brnleehng brnleehng deleted the feature/swagger branch May 16, 2016 23:30
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.

4 participants