Skip to content

Conversation

@bgklein
Copy link
Contributor

@bgklein bgklein commented Jun 2, 2020

Description
Batch has added a new API version (already released as Python SDK). These new features are part of JEDI deliverables.

History Notes
[Batch/Network] az network private-link-resource: Add support for the Microsoft.Batch batchAccount resource
[Batch/Network] az network private-endpoint-connection: Add support for the Microsoft.Batch batchAccount resource
[Batch] az batch account create: Added new parameter --public-network-access
[Batch] az batch account create: Added new parameter --identity-type
[Batch] az batch account set: Added new parameter --identity-type
[Batch] BREAKING CHANGE: az batch pool create: When creating a pool using a custom image, the --image property of can now only refer to a Shared Image Gallery image.
[Batch] BREAKING CHANGE: az batch pool create: When creating a pool with --json-file option and specifying a networkConfiguration, the publicIPs property has moved in to a new property publicIPAddressConfiguration. This new property also supports a new ipAddressProvisioningType property which specifies how the pool should allocate IP's and a publicIPs property which allows for configuration of a list of PublicIP resources to use in the case ipAddressProvisioningType is set to UserManaged


This checklist is used to make sure that common guidelines for a pull request are followed.

@bgklein
Copy link
Contributor Author

bgklein commented Jun 2, 2020

@myronfanqiu where is the CredScan omit file for the CLI? Seems to be catching temporary keys in the test yamls that it didn't before (cleaned up as part of test automation).

@yungezz yungezz added this to the S170 milestone Jun 2, 2020
@yonzhan yonzhan modified the milestones: S170, S171 Jun 2, 2020
@yonzhan
Copy link
Collaborator

yonzhan commented Jun 2, 2020

add to S171

@yonzhan yonzhan requested a review from jsntcy June 2, 2020 22:22
@bgklein bgklein force-pushed the brklein/mgmt-mar branch 2 times, most recently from 7df0456 to 07b1eb2 Compare June 3, 2020 16:43
@bgklein
Copy link
Contributor Author

bgklein commented Jun 3, 2020

@jsntcy please review as early as possible. I would like to address any concerns as soon as possible to not delay other work

@bgklein bgklein force-pushed the brklein/mgmt-mar branch from 07b1eb2 to ba37e96 Compare June 3, 2020 16:57
@haroldrandom
Copy link
Contributor

What does it mean [Batch/Network] in the PR body ?

@jsntcy
Copy link
Member

jsntcy commented Jun 4, 2020

Yes, Fan and I added some comments, please take a look.


In reply to: 638317849 [](ancestors = 638317849)

@bgklein
Copy link
Contributor Author

bgklein commented Jun 4, 2020

What does it mean [Batch/Network] in the PR body ?

Just that private endpoint/link is a networking feature that each RP has to add support for. We are adding Batch support to a Networking feature. It should be classified as Batch so customers know that it is now added, but could also be classified as networking additionally

@bgklein bgklein force-pushed the brklein/mgmt-mar branch from f029772 to ae400df Compare June 5, 2020 00:05
@bgklein bgklein force-pushed the brklein/mgmt-mar branch from ae400df to 3eeb4f6 Compare June 8, 2020 16:21
@mmyyrroonn
Copy link
Contributor

@jsntcy Please take a look at this PR.

@bgklein bgklein requested a review from jsntcy June 9, 2020 20:37
Copy link
Member

@jsntcy jsntcy left a comment

Choose a reason for hiding this comment

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

:shipit:

@Juliehzl Juliehzl merged commit e42b1a3 into Azure:dev Jun 11, 2020
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.

9 participants