Skip to content

Add docstring to indicate only one agent_pool_profile can be in the list - #3256

Closed
WebSpider wants to merge 1 commit into
Azure:masterfrom
WebSpider:managed_cluster_docstring
Closed

Add docstring to indicate only one agent_pool_profile can be in the list#3256
WebSpider wants to merge 1 commit into
Azure:masterfrom
WebSpider:managed_cluster_docstring

Conversation

@WebSpider

Copy link
Copy Markdown

Enhance documentation so limit of one agent_pool_profile per managed_cluster is more clear

@WebSpider
WebSpider requested a review from lmazuel as a code owner September 2, 2018 06:03
@codecov-io

codecov-io commented Sep 2, 2018

Copy link
Copy Markdown

Codecov Report

Merging #3256 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #3256   +/-   ##
=======================================
  Coverage    55.2%    55.2%           
=======================================
  Files        9198     9198           
  Lines      184891   184891           
=======================================
  Hits       102070   102070           
  Misses      82821    82821
Impacted Files Coverage Δ
...re/mgmt/containerservice/models/managed_cluster.py 27.77% <ø> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4a76ec8...cbac577. Read the comment docs.

@azuresdkci

Copy link
Copy Markdown
Contributor

Can one of the admins verify this patch?

@lmazuel

lmazuel commented Sep 4, 2018

Copy link
Copy Markdown
Member

@mboersma ?

@mboersma

mboersma commented Sep 5, 2018

Copy link
Copy Markdown
Member

Yes, this is a good clarification--thanks! Only one AKS node (agent) pool is supported currently.

But shouldn't we make this change here, in the Swagger spec so it propagates to all the SDKs?

@WebSpider

WebSpider commented Sep 5, 2018 via email

Copy link
Copy Markdown
Author

@lmazuel

lmazuel commented Sep 5, 2018

Copy link
Copy Markdown
Member

Yes @mboersma , it has to be done in the Swagger. But at first I wasn't even sure it was accurate :). Could you fix the Swagger and link this issue? Thanks!

@mboersma

mboersma commented Sep 7, 2018

Copy link
Copy Markdown
Member

Swagger PR opened at Azure/azure-rest-api-specs#3835.

@lmazuel

lmazuel commented Sep 7, 2018

Copy link
Copy Markdown
Member

Done in #3298

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants