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

vdo: document versions that have read cache options #53291

Merged
merged 1 commit into from
Mar 6, 2019
Merged

vdo: document versions that have read cache options #53291

merged 1 commit into from
Mar 6, 2019

Conversation

bgurney-rh
Copy link
Contributor

The "readcache" and "readcachesize" options, which correspond to
the same options in the "vdo create" and "vdo modify" commands,
have been removed in VDO version 6.2. Document that the read
cache options are only in VDO 6.1 and older.

SUMMARY

The read cache feature in VDO has been removed in VDO version 6.2. No code needs to be changed; instead, add documentation lines to the "readcache" and "readcachesize" options, stating that they are available in VDO versions 6.1 and older, allowing those versions to continue to use playbooks with those options.

ISSUE TYPE
  • Docs Pull Request
COMPONENT NAME

vdo

ADDITIONAL INFORMATION

If playbooks for systems with VDO 6.2 and newer do not specify the "readcache" or "readcachesize" options, there should not be any errors. If either of these options are specified for VDO 6.2 and above, there will be an error from the "vdo" command stating "unrecognized arguments: --readCache..." and/or "unrecognized arguments: --readCache..."; these errors should be passed through by ansible, and visible to the user.


The "readcache" and "readcachesize" options, which correspond to
the same options in the "vdo create" and "vdo modify" commands,
have been removed in VDO version 6.2.

Signed-off-by: Bryan Gurney <bgurney@redhat.com>
@ansibot ansibot added affects_2.8 This issue/PR affects Ansible v2.8 community_review In order to be merged, this PR must follow the community review workflow. docs This issue/PR relates to or includes documentation. module This issue/PR relates to a module. needs_triage Needs a first human triage before being processed. owner_pr This PR is made by the module's maintainer. small_patch support:community This issue/PR relates to code supported by the Ansible community. system System category labels Mar 4, 2019
@acozine acozine removed the needs_triage Needs a first human triage before being processed. label Mar 6, 2019
@acozine
Copy link
Contributor

acozine commented Mar 6, 2019

Thanks for maintaining the Ansible documentation @bgurney-rh!

@acozine acozine merged commit 5da0de0 into ansible:devel Mar 6, 2019
@ansible ansible locked and limited conversation to collaborators Jul 25, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
affects_2.8 This issue/PR affects Ansible v2.8 community_review In order to be merged, this PR must follow the community review workflow. docs This issue/PR relates to or includes documentation. module This issue/PR relates to a module. owner_pr This PR is made by the module's maintainer. small_patch support:community This issue/PR relates to code supported by the Ansible community. system System category
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants