microsoft / service-fabric-cli

Service Fabric CLI Tools
Other
53 stars 52 forks source link

adding get-loaded-partition-info-list command #224

Closed mrdakj closed 3 years ago

mrdakj commented 4 years ago

Fixes # .

Changes include:

Verified:

jeffj6123 commented 4 years ago

@mrdakj the API version is 7.3. Are the changes here meant for release with 7.2?

mrdakj commented 4 years ago

@mrdakj the API version is 7.3. Are the changes here meant for release with 7.2?

No, this is meant for the next release.

jeffj6123 commented 4 years ago

@mrdakj can you try recreating the PR, im not sure why the CI isnt starting.

jeffj6123 commented 4 years ago

@mrdakj looking at the swagger spec released for 7.2, I dont see this API in there. I see it added for the next version 7.3/8.0.

mrdakj commented 4 years ago

@mrdakj looking at the swagger spec released for 7.2, I dont see this API in there. I see it added for the next version 7.3/8.0.

Yes. API is meant for 8.0 version.

mrdakj commented 4 years ago

@jeffj6123 , is API version fine?