Bug#911185: commit hash not found in translation-check header breaks the wml build

Laura Arjona Reina <[email protected]>
Newsgroups gmane.linux.debian.devel.www
Message-ID <7DDF0226-8E2B-4147-879F-94D868642631__32781.1987758198$1539955534$gmane$org@debian.org>
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
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.