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

Fixing hyperV container deployment for Azure App Service Deploy task #16367

Merged
merged 9 commits into from
Jul 20, 2022

Conversation

SatishRanjan
Copy link
Contributor

Task name: Azure App Service Deploy

Description: The changes are to enable deployment of windows container apps on the azure app services

Documentation changes required: (Y/N) N

Added unit tests: (Y/N) NA

Attached related issue: (Y/N) ADO task https://msazure.visualstudio.com/Antares/_workitems/edit/9172309

Checklist:

  • Task version was bumped - please check instruction how to do it
  • Checked that applied changes work as expected

@vijayma vijayma requested a review from PhilipsonJoseph June 1, 2022 19:20
@mmrazik
Copy link
Collaborator

mmrazik commented Jun 1, 2022

Please bump the task version as per the Checklist.

@SatishRanjan
Copy link
Contributor Author

Hi @PhilipsonJoseph, can you please take a look at this PR.

@SatishRanjan
Copy link
Contributor Author

Hi @mmrazik - since there's no response on this from @PhilipsonJoseph, can you please proceed with approving the PR?

@SatishRanjan
Copy link
Contributor Author

Hi @@mmrazik - a gentle reminder to merge on this PR

@geekzter
Copy link
Member

@SatishRanjan looks good from my end but you need to update your branch

@SatishRanjan
Copy link
Contributor Author

@geekzter - Hi Eric, updated this branch from master latest.

@alexander-smolyakov
Copy link
Contributor

/azp run

@mmrazik
Copy link
Collaborator

mmrazik commented Jul 19, 2022

@SatishRanjan sorry for missing this :(
I am approving now but can you please bump the version once more? It should be 208 now. Thanks!

@azure-pipelines
Copy link

Azure Pipelines successfully started running 4 pipeline(s).

@alexander-smolyakov
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 4 pipeline(s).

@SatishRanjan
Copy link
Contributor Author

Hi @mmrazik and /or @alexander-smolyakov - with your approvals, request you to please merge this PR as I don't have write access to perform the merge operation.

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