[virttools-planet PATCH 2/4] Introduce use of GitLab CI for publishing to GitLab Pages
Daniel P. Berrangé
berrange at redhat.com
Tue Apr 7 17:18:09 UTC 2020
On Tue, Apr 07, 2020 at 04:04:16PM +0200, Andrea Bolognani wrote:
> On Tue, 2020-03-31 at 11:08 +0100, Daniel P. Berrangé wrote:
> > diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
> > new file mode 100644
> > index 0000000..cfc6bf6
> > --- /dev/null
> > +++ b/.gitlab-ci.yml
> > @@ -0,0 +1,14 @@
> > +
> > +image: centos:7
> CentOS 7, ewww.
> > +pages:
> > + stage: build
> > + cache:
> > + paths:
> > + - cache/
> > + key: "$CI_JOB_NAME"
> > + script:
> > + - python2 planet.py virt-tools/config.ini
> Python 2, EWWWWW!
Yep, this is why I picked centos:7 image, as it contains py2 by
> IIUC Planet itself is abandonware now, and has been for a fairly long
> time. Can we investigate an alternative?
We could, but it is a matter of time & priorities.
This site just looks after itself without issues on the software
side really, so while python 2 is yuk, it isn't really a real
world problem for us that needs addressing any time soon.
> > +++ b/README.md
> > @@ -34,14 +33,11 @@ Please send a patch email to `libvir-list at redhat.com`:
> > How to run the site
> > -------------------
> > -The site is setup to run under OpenShift
> > +The site is intended to be published with GitLab Pages
> Missing period here.
> > +++ b/virt-tools/basic/index.html.tmpl
> > @@ -23,7 +23,7 @@
> > <TMPL_LOOP Channels>
> > <li><a href="<TMPL_VAR link ESCAPE="HTML">" title="<TMPL_VAR title ESCAPE="HTML">"><TMPL_VAR name></a> <a href="<TMPL_VAR url ESCAPE="HTML">">(feed)</a></li>
> > </TMPL_LOOP>
> > - <li><a href="https://libvirt.org/git/?p=virttools-planet.git;a=blob;f=README">Add your blog!</a></li>
> > + <li><a href="https://gitlab.com/libvirt/virttools-planet/-/blob/master/README.md">Add your blog!</a></li>
> I think we can just point people to the repository.
Linking to the README was intentional as it means the instructions are
immediately visible to the user, instead of hidden off the bottom of
the browser screen after the repo file listing.
> > @@ -73,9 +73,8 @@
> > <div id="planetfooter">
> > <a href="http://www.planetplanet.org/">Powered by Planet!</a><br>
> > <em>Last updated: <TMPL_VAR date></em>
> > - <a href="https://www.openshift.com/" title="Powered by OpenShift Online"><br>
> > - <img alt="Powered by OpenShift Online" src="https://www.openshift.com/images/logos/powered_by_openshift_reverse.png">
> > - </a>
> > + &emdash;
> > + <a href="https://gitlab.com/libvirt/virttools-planet/-/blob/master/virt-tools/basic/index.html.tmpl" title="Edit this page">Edit this page</a>
> This seems unnecessary. If anything, we might consider replacing the
> original OpenShift Online "powered by" with GitLab Pages.
The goal is that all the webpages we publish have an "edit page" link
that goes to the most appropriate source content for the page. It is
intended to encourage contribution by showing people where/what to try
editting instead of leaving them to try to find it themselves.
> If you add the missing period and leave out the "edit this page"
> Reviewed-by: Andrea Bolognani <abologna at redhat.com>
|: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org -o- https://fstop138.berrange.com :|
|: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|
More information about the libvir-list