Make pm as optional#4636
Conversation
✅ Deploy Preview for chef-web-docs ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
swathi-s3
requested review from
VarshiSB,
sandhi18,
shanmugapriya-tr and
vipin230
| : The package manager. | ||
|
|
||
| Use this parameter only for Chef Infra Client Enterprise and Chef Infra Client Legacy Migration. | ||
| : The package manager. This parameter is optional. |
There was a problem hiding this comment.
The package manager is automatically set from the computer downloading the package, correct?
This should probably be clarified here.
There was a problem hiding this comment.
Added a better description here. @IanMadd
Signed-off-by: Swathi <swathis@progress.com>
Signed-off-by: Swathi <swathis@progress.com>
swathi-s3
requested review from
VarshiSB,
shanmugapriya-tr,
vipin230 and
vviveksharma
|
@IanMadd - Requesting you to review this again |
shanmugapriya-tr
force-pushed
the
optional_pm
branch
from
fcf981e to
9c36bc9
Compare
|
|
|
||
| Possible values: `debian`, `el` (for RHEL derivatives), `freebsd`, `mac_os_x`, `solaris2`, `sles`, `suse`, `ubuntu` or | ||
| `windows`. | ||
| Possible values: `debian`, `el` (for RHEL derivatives), `freebsd`, `mac_os_x`, `solaris2`, `sles`, `suse`, `ubuntu` or `windows`. |
There was a problem hiding this comment.
The other updates mention generic platforms, like linux. Are there others that need to be updated here? Is there a set of generic platforms that make pm required?
|
|
||
| Use this parameter only for Chef Infra Client Enterprise and Chef Infra Client Legacy Migration. | ||
| : This parameter is optional. | ||
| If not provided, the API automatically detects the platform and derives the package format from it. |
There was a problem hiding this comment.
It looks like this is optional only if the p is set with a specific platform, and required if it specifies a generic platform.
6 participants
Description
Updates the Commercial API documentation to reflect that the pm (package manager) query parameter is now optional for all products, not restricted to Chef Infra Client Enterprise and Chef Infra Client Legacy Migration.
Issues Resolved
https://progresssoftware.atlassian.net/browse/CHEF-30749
Check List