infra-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Sebb (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (INFRA-16286) [mail-private] bugs in refresh-index.pl
Date Mon, 08 Apr 2019 09:17:00 GMT

    [ https://issues.apache.org/jira/browse/INFRA-16286?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16812254#comment-16812254
] 

Sebb commented on INFRA-16286:
------------------------------

I think I have addressed most of the above in the scripts I attached to INFRA-18017

> [mail-private] bugs in refresh-index.pl
> ---------------------------------------
>
>                 Key: INFRA-16286
>                 URL: https://issues.apache.org/jira/browse/INFRA-16286
>             Project: Infrastructure
>          Issue Type: Planned Work
>          Components: Mail Archives
>            Reporter: Sebb
>            Priority: Major
>
> The script refresh-index.pl has various bugs.
> 1) It uses system("umask ...").
> AFAICT that is useless, as the setting only applies to the sub-process.
> 2) It runs 'chmod -R' on the whole mbox tree each time.
> That seems wasteful.
> 3) There is no protection against running the script concurrently.
> It relies on the script completing before the next invocation by cron.
> (Currently hourly)
> 4) If an instance of mbox-util takes too long (currently 3600 seconds = 1hr) it writes
a message to stdout, which is thrown away by the cronjob.
> It does not try to stop the process.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Mime
View raw message