Project

Profile

Help

Issue #6481

closed

test_delete_middle_version is sporadically failing

Added by mdellweg almost 4 years ago. Updated almost 4 years ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
Normal
Category:
-
Sprint/Milestone:
Start date:
Due date:
Estimated time:
Severity:
2. Medium
Version:
Platform Release:
OS:
Triaged:
Yes
Groomed:
No
Sprint Candidate:
No
Tags:
Sprint:
Sprint 70
Quarter:

Description

This happens roughly one in six times (16 out of a sample of 100).

INCIDENTS=0; for i in $(seq 100); do pulpcore-manager test pulpcore.tests.functional.api.using_plugin.test_repo_versions.AddRemoveRepoVersionTestCase.test_delete_middle_version || INCIDENTS=$(($INCIDENTS + 1)); done ; echo $INCIDENTS

Delete a middle version.
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/vagrant/devel/pulpcore/pulpcore/tests/functional/api/using_plugin/test_repo_versions.py", line 363, in test_delete_middle_version
    self.assertIn(self.content[index]['artifact'], artifact_paths)
AssertionError: '/pulp/api/v3/artifacts/e4d865d1-3335-4ddb-aeaa-81ac3802722a/' not found in {'/pulp/api/v3/artifacts/09dd8e86-11c3-4ca5-8769-c8023c338328/', '/pulp/api/v3/artifacts/3431332e-2ea9-4076-aa72-c4529eed135b/', '/pulp/api/v3/artifacts/f49a9851-8455-4b3a-b700-2f3a47c8ad22/', '/pulp/api/v3/artifacts/fbf7ccf0-fdc3-4118-886b-a8738587fb9e/', '/pulp/api/v3/artifacts/22081b36-cbad-4ad5-a9c2-266f06472d40/'}```
Actions #1

Updated by dkliban@redhat.com almost 4 years ago

This failure only occurs after you run the test in pulpcore.tests.functional.api.using_plugin.test_repair

Actions #2

Updated by dkliban@redhat.com almost 4 years ago

The repair functional test leaves behind 3 content units called 1.iso, 2.iso, 3.iso. The repository version delete test populates pulp with 10 content units called 1.iso, 2.iso, 3.iso ... 10.iso.

The test then creates 9 repository versions by grabbing 9 content units in pulp and adding one at a time to the repository. If there are two units with the same relative path, the adding of the second one removes the first from the repository. As a result the first one is not found in the repository after this.

I am going to call orphan cleanup at the beginning of this test.

Actions #3

Updated by dkliban@redhat.com almost 4 years ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to dkliban@redhat.com
Actions #4

Updated by pulpbot almost 4 years ago

  • Status changed from ASSIGNED to POST

Added by dkliban@redhat.com almost 4 years ago

Revision b810c8ec | View on GitHub

Adds orphan cleanup to begining of repository version tests

fixes: #6481 https://pulp.plan.io/issues/6481

Actions #5

Updated by fao89 almost 4 years ago

  • Triaged changed from No to Yes
  • Sprint set to Sprint 70
Actions #6

Updated by dkliban@redhat.com almost 4 years ago

  • Status changed from POST to MODIFIED
Actions #7

Updated by ttereshc almost 4 years ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE
  • Sprint/Milestone set to 3.3.0

Also available in: Atom PDF