[pve-devel] [PATCH docs 3/3] Fix #2018: CephFS client needs newer binaries

[Wolfgang Bumiller]

On Tue, Feb 12, 2019 at 10:44:51AM +0100, Alwin Antreich wrote:
> To run cephfs client the debian stock packages need to be updated to
> luminous to get it running.
> 
> This patch adds a section with our Ceph repositories to our package repo
> chapter to where the cephfs storage plugin chapter links to.
> 
> Signed-off-by: Alwin Antreich <a.antreich at proxmox.com>
> ---
>  pve-package-repos.adoc  | 28 ++++++++++++++++++++++++++++
>  pve-storage-cephfs.adoc |  9 +++++++++
>  2 files changed, 37 insertions(+)
> 
> diff --git a/pve-package-repos.adoc b/pve-package-repos.adoc
> index 77c1690..a5cad49 100644
> --- a/pve-package-repos.adoc
> +++ b/pve-package-repos.adoc
> @@ -95,6 +95,31 @@ deb http://download.proxmox.com/debian/pve stretch pvetest
>  WARNING: the `pvetest` repository should (as the name implies) only be used
>  for testing new features or bug fixes.
>  
> +[[sysadmin_package_repositories_ceph]]
> +{pve} Ceph Repository
> +~~~~~~~~~~~~~~~~~~~~~
> +
> +This is {pve}'s main Ceph repository and holds the Ceph versions, for

Remove the comma. (Maybe also s/versions/packages/).

> +production use. You can also use this repository to only update the Ceph

I'd swap `only` and `update`.
`only update X` is more like saying "do nothing else to X except
updating it`.
`update only X` is more like `update X without also upgrading any other
Y`, which I think is what you mean.

> +client.
> +
> +.File `/etc/apt/sources.list.d/ceph.list`
> +----
> +deb http://download.proxmox.com/debian/ceph-luminous stretch main
> +----
> +
> +
> +{pve} Ceph Testing Repository
> +~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> +
> +This Ceph repository contains the Ceph packages before the go into the main

the => they
maybe go => are moved?

> +repository and is used to test the new Ceph release on {pve}.

`test the new Ceph release` => `test new Ceph releases`

> +
> +.File `/etc/apt/sources.list.d/ceph.list`
> +----
> +deb http://download.proxmox.com/debian/ceph-luminous stretch test
> +----
> +
>  
>  SecureApt
>  ~~~~~~~~~
> @@ -215,3 +240,6 @@ upgrade to latest version as soon as possible.
>  
>  
>  endif::wiki[]
> +ifdef::manvolnum[]
> +include::pve-copyright.adoc[]
> +endif::manvolnum[]
> diff --git a/pve-storage-cephfs.adoc b/pve-storage-cephfs.adoc
> index 96f4991..5c11382 100644
> --- a/pve-storage-cephfs.adoc
> +++ b/pve-storage-cephfs.adoc
> @@ -18,6 +18,15 @@ configuring a CephFS storage easier. As recent hardware has plenty of CPU power
>  and RAM, running storage services and VMs on same node is possible without a
>  big performance impact.
>  
> +To use the CephFS storage you need update the debian stock Ceph client. Add our

I'd say `the CephFS storage _plugin_`, or just `CephFS` with neither `the`
nor `storage`.

> +Ceph repository xref:sysadmin_package_repositories_ceph[Ceph repository]. Once
> +added, run a `apt update` and `apt dist-upgrade` cycle to get the newest

-a (or `an`)

> +packages.
> +
> +You need to make sure that there is no other Ceph repository configured,
> +otherwise the installation will fail or there will be different package

(optionally different => wrong ? )

> +versions on the node, leading to unexpected behavior.
> +
>  [[storage_cephfs_config]]
>  Configuration
>  ~~~~~~~~~~~~~
> -- 
> 2.11.0