[PATCH 3/8] docs/process: branching-checklist: Remove reference to root

Julien Grall posted 8 patches 3 years, 1 month ago
There is a newer version of this series
[PATCH 3/8] docs/process: branching-checklist: Remove reference to root
Posted by Julien Grall 3 years, 1 month ago
From: Julien Grall <jgrall@amazon.com>

The steps to generate the documentation for the new branch requires
to ssh as root and then su to run with the user xendocs.

The release technician may not (and should not) have access to root.
So update the step to directly ssh as xendocs.

Signed-off-by: Julien Grall <jgrall@amazon.com>
---
 docs/process/branching-checklist.txt | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/docs/process/branching-checklist.txt b/docs/process/branching-checklist.txt
index b8101dd0e359..a98f6559ab1a 100644
--- a/docs/process/branching-checklist.txt
+++ b/docs/process/branching-checklist.txt
@@ -25,8 +25,7 @@ ov=4.0
 * make 13:37 <ijc> https://xenbits.xen.org/docs/4.2-testing/ is now live true
 #14:17 <ijc> HOWTO: login to xenbits. become "xendocs" . cd cronjobs . edit
 #            xenbits-docs-all.sh in the obvious way. git commit
-    ssh root@xenbits.xen.org
-    su - xendocs
+    ssh xendocs@xenbits.xen.org
     cd cronjobs
     ed xenbits-docs-all.sh
     /for branch
-- 
2.38.1
Re: [PATCH 3/8] docs/process: branching-checklist: Remove reference to root
Posted by George Dunlap 3 years, 1 month ago
On Mon, Dec 12, 2022 at 9:34 AM Julien Grall <julien@xen.org> wrote:

> From: Julien Grall <jgrall@amazon.com>
>
> The steps to generate the documentation for the new branch requires
> to ssh as root and then su to run with the user xendocs.
>
> The release technician may not (and should not) have access to root.
> So update the step to directly ssh as xendocs.
>
> Signed-off-by: Julien Grall <jgrall@amazon.com>
>

Acked-by: George Dunlap <george.dunlap@cloud.com>