Bug#911185: commit hash not found in translation-check header breaks the wml build
Hello
El 19/10/18 a las 12:19, Wouter Verhelst escribió:
> On Thu, Oct 18, 2018 at 12:27:35PM +0200, Wouter Verhelst wrote:
>> On Tue, Oct 16, 2018 at 11:52:55PM +0200, Laura Arjona Reina wrote:
>>> * I'm not sure if we want that, or we prefer that the build fails and
>>> make the CI not ignore these errors (then I guess the person committing
>>> the file with wrong translation hash would get a mail, and hopefully fix
>>> the file. If the page is built, maybe these kind of issues pass by
>>> unnoticed).
>>
>> Yes. I think that should be done, indeed, and I've been working on it.
>
> FWIW, that's !33. I've been adding a "weak error checks" mode, where the
> intention is that CI will fail, unless something is expected not to be
> in the repository, such as the "Mirrors.masterlist" file, which seems to
> be used all over the repository.
>
Mirrors.masterlist file I think it's the easier one to get, it's in the repo
https://salsa.debian.org/mirror-team/masterlist.git/ so we can get it cloning
the repo and putting the file in /english/mirror folder.
> One issue I'm running against now is /english/international/l10n/data;
> that directory seems almost empty, but it would appear that it is
> supposed to contain configuration data for the website itself (i.e., not
> data that I would expect to find in another repository).
>
> Why is that not just added to git and committed?
>
The files are generated by cron scripts run daily:
https://salsa.debian.org/webmaster-team/cron/blob/master/lessoften-parts/1l10n-data
I have done some tests to include some parts of what is done via cron scripts to be done in .gitlab-ci.yml, to minimise the areas of webwml failing in the CI, but later I thought that if those processes are indeed in separated cron jobs that only run 6 times a day (and some others, once a day), maybe it's not reasonable to make them run in salsa with each commit...
Kind regards
--
Laura Arjona Reina
https://wiki.debian.org/LauraArjona
--
Laura Arjona Reina
https://wiki.debian.org/LauraArjona
Sent with K-9 mail
Reply to: