Differential D5142

deposit.tests: Reproduce staging error
AbandonedPublicDraft
Actions

Authored by ardumont on Feb 25 2021, 12:03 PM.

Details

Reviewers

None

Group Reviewers

Reviewers

Maniphest Tasks

T3070: Fix longstanding deposit status verified which is usually a symptom of loading failure

Summary

Now, on to fixing it.

Related to T3070

Diff Detail

Repository

rDDEP Push deposit

Branch

master

Lint

No Linters Available

Unit

No Unit Test Coverage

Build Status

Buildable 19461
Build 30193: Phabricator diff pipeline on jenkins	Jenkins console · Jenkins
Build 30192: arc lint + arc unit

Unit TestsFailed
View All

	Time	Test
	79 ms	Jenkins > .tox.py3.lib.python3.7.site-packages.swh.deposit.tests.api.test_deposit_private_read_archive::test_aggregate_tarballs_with_strange_archive datadir = '/var/lib/jenkins/workspace/DDEP/tests-on-diff/.tox/py3/lib/python3.7/site-packages/swh/deposit/tests/api/../data' tmp_path = '/tmp/pytest-of-jenkins/pytest-0/test_aggregate_tarballs_with_s0'
	93 ms	Jenkins > .tox.py3.lib.python3.7.site-packages.swh.deposit.tests.api.test_checks::test_api_checks_check_metadata_ko[metadata_ko0-expected_summary0]
	93 ms	Jenkins > .tox.py3.lib.python3.7.site-packages.swh.deposit.tests.api.test_checks::test_api_checks_check_metadata_ko[metadata_ko1-expected_summary1]
	93 ms	Jenkins > .tox.py3.lib.python3.7.site-packages.swh.deposit.tests.api.test_checks::test_api_checks_check_metadata_ko[metadata_ko2-expected_summary2]
	92 ms	Jenkins > .tox.py3.lib.python3.7.site-packages.swh.deposit.tests.api.test_checks::test_api_checks_check_metadata_ko[metadata_ko3-expected_summary3]
		View Full Test Results (1 Failed · 242 Passed)

Event Timeline

ardumont created this revision.Feb 25 2021, 12:03 PM

Herald added a reviewer: Reviewers. · View Herald TranscriptFeb 25 2021, 12:03 PM

Build has FAILED

Patch application report for D5142 (id=18383)

Rebasing onto 6275caba93...

Current branch diff-target is up to date.

Changes applied before test

commit e660679a9577fb484cf5393d29b6a46789435687
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date:   Thu Feb 25 12:01:58 2021 +0100

    tests: Reproduce error
    
    Now, on to fixing it.
    
    Related to T3070

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

Harbormaster returned this revision to the author for changes because remote builds failed.Feb 25 2021, 12:05 PM

Harbormaster failed remote builds in B19461: Diff 18383!

Add the tarball which will fail the test

Build has FAILED

Patch application report for D5142 (id=18384)

Rebasing onto 6275caba93...

Current branch diff-target is up to date.

Changes applied before test

commit f96f85a03a2182e75d5b3ed70e588f52e6f852f8
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date:   Thu Feb 25 12:01:58 2021 +0100

    tests: Reproduce error
    
    Now, on to fixing it.
    
    Related to T3070

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

Harbormaster returned this revision to the author for changes because remote builds failed.Feb 25 2021, 12:09 PM

Harbormaster failed remote builds in B19462: Diff 18384!

ardumont mentioned this in D5143: Aggregate deposit archives into a temporary tarball instead of a zip.Feb 25 2021, 2:34 PM

Closing. Fixed by D5143.

(Kept aside for history purposes)