Differential D6826

misc/coverage: Do not put coverage page in cache
ClosedPublic
Actions

Authored by anlambert on Dec 13 2021, 1:37 PM.

Details

Reviewers

ardumont

Group Reviewers

Reviewers

Commits

rDWAPPSc9b40186e88a: misc/coverage: Do not put coverage page in cache

Summary

This should ensure proper update of origin counters, plus there is already
some Python LRU caches in use in view implementation to avoid requesting
the scheduler database too often.

Diff Detail

Repository

rDWAPPS Web applications

Branch

master

Lint

No Linters Available

Unit

No Unit Test Coverage

Build Status

Buildable 25601
Build 40034: Phabricator diff pipeline on jenkins	Jenkins console · Jenkins
Build 40033: arc lint + arc unit

Event Timeline

anlambert created this revision.Dec 13 2021, 1:37 PM

Herald added a reviewer: Reviewers. · View Herald TranscriptDec 13 2021, 1:37 PM

Build is green

Patch application report for D6826 (id=24750)

Rebasing onto 61647a4bfd...

Current branch diff-target is up to date.

Changes applied before test

commit c9b40186e88ada46f83d8a3c68d7c4efa1943b75
Author: Antoine Lambert <anlambert@softwareheritage.org>
Date:   Mon Dec 13 13:36:26 2021 +0100

    misc/coverage: Do not put coverage page in cache
    
    This should ensure proper update of origin counters, plus there is already
    some Python LRU caches in use in view implementation to avoid requesting
    the scheduler database too often.

See https://jenkins.softwareheritage.org/job/DWAPPS/job/tests-on-diff/1267/ for more details.

Harbormaster completed remote builds in B25601: Diff 24750.Dec 13 2021, 1:52 PM

anlambert requested review of this revision.Dec 13 2021, 1:52 PM

ardumont accepted this revision.Dec 13 2021, 2:09 PM

This revision is now accepted and ready to land.Dec 13 2021, 2:09 PM

Thanks

Closed by commit rDWAPPSc9b40186e88a: misc/coverage: Do not put coverage page in cache (authored by anlambert). · Explain WhyDec 13 2021, 2:19 PM

This revision was automatically updated to reflect the committed changes.

anlambert added a commit: rDWAPPSc9b40186e88a: misc/coverage: Do not put coverage page in cache.