	Time	Test
	28 ms	Jenkins > .tox.py3.lib.python3.7.site-packages.swh.storage.tests.algos.test_diff::test__get_rev request = <SubRequest 'swh_storage_postgresql' for <Function test__get_rev>> @pytest.fixture
	29 ms	Jenkins > .tox.py3.lib.python3.7.site-packages.swh.storage.tests.algos.test_origin::test_iter_origin_visit_status request = <SubRequest 'swh_storage_postgresql' for <Function test_iter_origin_visit_status>> @pytest.fixture
	28 ms	Jenkins > .tox.py3.lib.python3.7.site-packages.swh.storage.tests.algos.test_origin::test_iter_origin_visits request = <SubRequest 'swh_storage_postgresql' for <Function test_iter_origin_visits>> @pytest.fixture
	234 ms	Jenkins > .tox.py3.lib.python3.7.site-packages.swh.storage.tests.algos.test_origin::test_iter_origins swh_storage_backend_config = {'check_config': {'check_write': True}, 'cls': 'local', 'db': "dbname=storage user=postgres host=127.0.0.1 port=14622 options=''", 'journal_writer': {'cls': 'memory'}, ...} @pytest.fixture
	27 ms	Jenkins > .tox.py3.lib.python3.7.site-packages.swh.storage.tests.algos.test_origin::test_origin_get_latest_visit_status_filter_snapshot request = <SubRequest 'swh_storage_postgresql' for <Function test_origin_get_latest_visit_status_filter_snapshot>> @pytest.fixture
		View Full Test Results (168 Failed · 574 Passed · 23 Skipped)

Event Timeline

ardumont created this revision.Jan 20 2021, 10:47 AM

Herald added a reviewer: Reviewers. · View Herald TranscriptJan 20 2021, 10:47 AM

Build is green

Patch application report for D4892 (id=17376)

Rebasing onto d04165f5b4...

Current branch diff-target is up to date.

Changes applied before test

commit 14a08fa9e97f3cc4b5d8411775eeeca8c71aabdf
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date:   Wed Jan 20 10:38:34 2021 +0100

    db: Allow new status value 'not_found' to OriginVisitStatus
    
    Related to T2961

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

Harbormaster completed remote builds in B18511: Diff 17376.Jan 20 2021, 10:52 AM

ardumont requested review of this revision.Jan 20 2021, 10:52 AM

(As long as this matches what ends up being done in D4891).

This reminds me that I noticed this morning that the bump to schema version 165 was forgotten in c24d35f86a06 (in db.py and 30-schema.sql).

This revision is now accepted and ready to land.Jan 20 2021, 11:28 AM

This reminds me that I noticed this morning that the bump to schema version 165 was forgotten in c24d35f86a06 (in db.py and 30-schema.sql).

Indeed.

Adapt diff according to the model update (adding the failed state as well)

ardumont retitled this revision from db: Allow new status value 'not_found' to OriginVisitStatus to db: Allow new status values not_found and failed to OriginVisitStatus.Jan 20 2021, 2:05 PM

Build has FAILED

Patch application report for D4892 (id=17389)

Rebasing onto d04165f5b4...

Current branch diff-target is up to date.

Changes applied before test

commit 9ceeab6da5b19097b7c5df4d3c2460d2f8b6bdcc
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date:   Wed Jan 20 10:38:34 2021 +0100

    db: Allow new status values not_found, failed to OriginVisitStatus
    
    Related to T2961

Link to build: https://jenkins.softwareheritage.org/job/DSTO/job/tests-on-diff/1107/
See console output for more information: https://jenkins.softwareheritage.org/job/DSTO/job/tests-on-diff/1107/console

Harbormaster failed remote builds in B18525: Diff 17389!Jan 20 2021, 2:10 PM

Fix trailing comma which sql is not happy about ¯\_(ツ)_/¯

Build is green

Patch application report for D4892 (id=17398)

Rebasing onto d04165f5b4...

Current branch diff-target is up to date.

Changes applied before test

commit e433255dabff0976efa673866ccd62f80bc94f99
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date:   Wed Jan 20 10:38:34 2021 +0100

    db: Allow new status values not_found, failed to OriginVisitStatus
    
    Related to T2961

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

Harbormaster completed remote builds in B18534: Diff 17398.Jan 20 2021, 3:41 PM

Closed by commit rDSTOe433255dabff: db: Allow new status values not_found, failed to OriginVisitStatus (authored by ardumont). · Explain WhyJan 20 2021, 3:42 PM

This revision was automatically updated to reflect the committed changes.

ardumont added a commit: rDSTOe433255dabff: db: Allow new status values not_found, failed to OriginVisitStatus.