Re: [virttools-planet PATCH 2/4] Introduce use of GitLab CI for publishing to GitLab Pages

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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!

IIUC Planet itself is abandonware now, and has been for a fairly long
time. Can we investigate an alternative?

> +++ b/README.md
> @@ -34,14 +33,11 @@ Please send a patch email to `libvir-list@xxxxxxxxxx`:
>  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.

> @@ -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.


If you add the missing period and leave out the "edit this page"
link,

  Reviewed-by: Andrea Bolognani <abologna@xxxxxxxxxx>

-- 
Andrea Bolognani / Red Hat / Virtualization





[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux