[v2,2/2] ref-manual: document overlayfs-etc image feature

Message ID 20211216112235.129737-2-uvv.mail@gmail.com
State New
Headers show
Series [v2,1/2] ref-manual: add overlayfs-etc class | expand

Commit Message

Vyacheslav Yurkov Dec. 16, 2021, 11:22 a.m. UTC
Signed-off-by: Vyacheslav Yurkov <uvv.mail@gmail.com>
---
 documentation/ref-manual/features.rst | 4 ++++
 1 file changed, 4 insertions(+)

Comments

Michael Opdenacker Dec. 16, 2021, 4:52 p.m. UTC | #1
Hi Slava,

Many thanks for the updates!

On 12/16/21 12:22 PM, Vyacheslav Yurkov wrote:
> Signed-off-by: Vyacheslav Yurkov <uvv.mail@gmail.com>
> ---
>  documentation/ref-manual/features.rst | 4 ++++
>  1 file changed, 4 insertions(+)
>
> diff --git a/documentation/ref-manual/features.rst b/documentation/ref-manual/features.rst
> index ded653221..57fe0cc3c 100644
> --- a/documentation/ref-manual/features.rst
> +++ b/documentation/ref-manual/features.rst
> @@ -219,6 +219,10 @@ Here are the image features available for all images:
>  -  *empty-root-password:* Sets the root password to an empty string,
>     which allows logins with a blank password.
>  
> +-  *overlayfs-etc:* Configures the ``/etc`` directory to be in ``overlayfs``.
> +   This allows to store device specific information elsewhere, especially
> +   if root filesystem is configured to be read-only.
> +
>  -  *package-management:* Installs package management tools and preserves
>     the package manager database.


For both patches: Reviewed-by: Michael Opdenacker
<michael.opdenacker@bootlin.com>
There were a few remaining punctuation and missing article issues (that
I probably failed to report), but I was happy to fix them!

Merged in "master-next".

Thanks again.
Michael.
Vyacheslav Yurkov Dec. 17, 2021, 6:07 a.m. UTC | #2
Hi Michael,
Thanks for the review and updates.

I assume it might be easier to send an update patch separately if 
needed. Or you can do fix-ups in master-next? :)

Cheers,
Slava

On 16.12.2021 17:52, Michael Opdenacker wrote:
> Hi Slava,
>
> Many thanks for the updates!
>
> On 12/16/21 12:22 PM, Vyacheslav Yurkov wrote:
>
> For both patches: Reviewed-by: Michael Opdenacker
> <michael.opdenacker@bootlin.com>
> There were a few remaining punctuation and missing article issues (that
> I probably failed to report), but I was happy to fix them!
>
> Merged in "master-next".
>
> Thanks again.
> Michael.
Michael Opdenacker Dec. 17, 2021, 7:07 a.m. UTC | #3
Hi Slava

On 12/17/21 7:07 AM, Vyacheslav Yurkov wrote:
> Hi Michael,
> Thanks for the review and updates.
>
> I assume it might be easier to send an update patch separately if
> needed. Or you can do fix-ups in master-next? :)


I can make fixups in master-next indeed. This way, we just have one commit.
Cheers
Michael.

Patch

diff --git a/documentation/ref-manual/features.rst b/documentation/ref-manual/features.rst
index ded653221..57fe0cc3c 100644
--- a/documentation/ref-manual/features.rst
+++ b/documentation/ref-manual/features.rst
@@ -219,6 +219,10 @@  Here are the image features available for all images:
 -  *empty-root-password:* Sets the root password to an empty string,
    which allows logins with a blank password.
 
+-  *overlayfs-etc:* Configures the ``/etc`` directory to be in ``overlayfs``.
+   This allows to store device specific information elsewhere, especially
+   if root filesystem is configured to be read-only.
+
 -  *package-management:* Installs package management tools and preserves
    the package manager database.