On Sun, 24 Sep 2017 17:51:50 +0300
Eyal Edri <eedri(a)redhat.com> wrote:
On Sun, Sep 24, 2017 at 4:52 PM, Eyal Edri <eedri(a)redhat.com>
wrote:
>
> On Sun, Sep 24, 2017 at 4:48 PM, Eyal Edri <eedri(a)redhat.com> wrote:
>
>> Hi,
>>
>> So I've found linkchecker[1], which is quite a powerful and useful tool
>> for that, and run it on
oVirt.org, the results are not good, unfortunately.
>> We have over 15K(!) of broken links to fix before we can even consider
>> adding a constant CI job to check new commits.
How about checking that no *new* broken links are introduced by pull
requests? Would that be easy enough to do?
>>
>> Snippet example from the huge report attached to this email [2]. ( btw
>> there are other formats available to generate this report if we want )
>>
>
> resending with zipped report ( the file was 16MB, too big for the list ).
>
I've rerun the tool excluding .png images, since there are many of them and
the broken links got down to 2373, to much more reasonable number
to start working on.
2k is "reasonable"? Really? :)
Attached new report w/o images.
>
>
>>
>> After we'll do an initial cleanup of the broken links ( similar to what
>> we did on findbugs errors ), we can consider adding CI for it, 2 ideas I
>> had in mind:
>>
>>
>> 1. Running the tool on GitHub PR using Travis CI ( which ovirt-site is
>> already connected to it ), the tool seems to be using it as well for
>> testing itself
>> 2. Adding ovirt-site as a std-ci project and using it to also properly
>> deploy to the site in the end ( more complex and long term goal, which
>> might require more work ).
>>
>>
>> TL;DR, let's schedule a doc-fix day first to address the sheer amount of
>> broken links and then worry about adding CI.
>>
>>
>>
>> [1]
https://github.com/wummel/linkchecker
>> [2]
>>
>> URL `develop/release-management/features/sla/hosted-engine-agent
>> -offloading/#documentation'
>> Name `Documentation/External references'
>> Parent URL
https://ovirt.org/develop/release-management/features/sla/
>> hosted-engine-agent-offloading/, line 1870, col 131 (HTML
>>
<
http://validator.w3.org/check?ss=1&uri=https://ovirt.org/develop/rele...>)
>> (CSS
>>
<
http://jigsaw.w3.org/css-validator/validator?uri=https://ovirt.org/develo...
>> )
>> Real URL
https://ovirt.org/develop/release-management/features/sla/
>> hosted-engine-agent-offloading/develop/release-management/
>> features/sla/hosted-engine-agent-offloading/#documentation
>> Size 14.63KB
>> Check time 3.496 seconds
>> Result Error: 404 Not Found
>>
>>
>> URL `/images/apple-touch-icon-precomposed.png'
>> Parent URL
https://www.ovirt.org, line 22, col 1 (HTML
>> <
http://validator.w3.org/check?ss=1&uri=https://www.ovirt.org>) (CSS
>>
<
http://jigsaw.w3.org/css-validator/validator?uri=https://www.ovirt.org&am...
>> )
>> Real URL
https://www.ovirt.org/images/apple-touch-icon-precomposed.png
>> Size 14.63KB
>> Check time 1.851 seconds
>> Result Error: 404 Not Found
>>
>>
>
> --
>
> Eyal edri
>
>
> ASSOCIATE MANAGER
>
> RHV DevOps
>
> EMEA VIRTUALIZATION R&D
>
>
> Red Hat EMEA <
https://www.redhat.com/>
> <
https://red.ht/sig> TRIED. TESTED. TRUSTED.
<
https://redhat.com/trusted>
> phone: +972-9-7692018 <+972%209-769-2018>
> irc: eedri (on #tlv #rhev-dev #rhev-integ)
>
--
Eyal edri
ASSOCIATE MANAGER
RHV DevOps
EMEA VIRTUALIZATION R&D
Red Hat EMEA <
https://www.redhat.com/>
<
https://red.ht/sig> TRIED. TESTED. TRUSTED. <
https://redhat.com/trusted>
phone: +972-9-7692018
irc: eedri (on #tlv #rhev-dev #rhev-integ)
--
Tomáš Golembiovský <tgolembi(a)redhat.com>