On Wed, 2020-04-08 at 17:41 +0100, Daniel P. Berrangé wrote:
+GitLab Configuration
+====================
+
+The `GitLab organization <
https://gitlab.com/libvirt>`_ hosts the master copy
+of all the libvirt Git repositories.
+
+When creating a new repository the following changes to the defaults are
+required under the ``Settings`` page
I think *Settings* would be more appropriate than ``Settings`` in
this context.
Missing ":" after "page".
+GitHub configuration
+====================
+
+The `GitHub organization <
https://github.com/libvirt>`_ hosts read-only
+mirrors of all the libvirt Git repositories.
+
+When creating a new repository the following changes to the defaults are
+required under the ``Settings`` page
Same comments as the GitLab equivalent.
+ * **Manage access**
+
+ * Add @committers with role "Write"
Probably ``@committers``.
I'd also clarify that this is supposed to be a team, and that it
needs to include ``libvirtmirror``, though assuming that teams are
per-org rather than per-repo then the latter might not be strictly
necessary.
+libvirt.org
+===========
+
+The `Libvirt project server <
https://libvirt.org>`_ hosts read-only mirrors of
+all the libvirt Git repositories in the directory ``/data/git``.
s/Libvirt/libvirt/
+When creating a new repository the following steps are required
Missing ":" after "required".
+* Create repo with
+ ::
+
+ $ sudo su -
+ # cd /data/git
+ # mkdir repo.git
+ # cd repo.git
In the other sections you've used ``$repo``, and it would be nice to
be consistent.
+ # cd ..
+ # chown -R gitmirror.gitmirror repo.git
+ # chmod -R g+w repo.git
+ # find -type d repo.git | xargs chmod g+s
The ``cd ..`` is not necessary if you just call all follow-up
commands on ``.``, but if you think it makes the example clearer then
I have no objection to leaving it in.
+* Set the ``description`` and ``config`` files following other repos
example
s/repos/repos'/
--
Andrea Bolognani / Red Hat / Virtualization