Project

Profile

Help

Issue #8357

closed

Sync validation error messages aren't helpful

Added by daviddavis almost 4 years ago. Updated over 3 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:
Easy Fix
Sprint:
Sprint 93
Quarter:

Description

Run this code:

#!/bin/bash

set -ev

repo_name="repo$RANDOM"
remote_name="remote$RANDOM"
url="https://fixtures.pulpproject.org/file-invalid/PULP_MANIFEST"

pulp file remote create --name $remote_name --url $url
pulp file repository create --name $repo_name --remote $remote_name
pulp file repository sync --name $repo_name

It'll error and you'll get a stack trace (pulpcore.exceptions.validation.DigestValidationError: A file failed validation due to checksum.). However, it doesn't tell you much information like what the problematic file is.

Actions #1

Updated by daviddavis almost 4 years ago

This was also reported by a user on our pulp mailing list.

Actions #2

Updated by daviddavis almost 4 years ago

  • Description updated (diff)
Actions #3

Updated by daviddavis almost 4 years ago

  • Tags Easy Fix added
Actions #4

Updated by ttereshc almost 4 years ago

  • Sprint/Milestone set to 3.12.0
  • Triaged changed from No to Yes
Actions #5

Updated by ipanova@redhat.com over 3 years ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to ipanova@redhat.com
Actions #6

Updated by ipanova@redhat.com over 3 years ago

  • Sprint set to Sprint 93
Actions #7

Updated by pulpbot over 3 years ago

  • Status changed from ASSIGNED to POST

Added by ipanova@redhat.com over 3 years ago

Revision 2045553e | View on GitHub

Made digest and size sync erros more helpful by logging url of the requested files.

closes #8357

Actions #8

Updated by ipanova@redhat.com over 3 years ago

  • Status changed from POST to MODIFIED
Actions #9

Updated by pulpbot over 3 years ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

Also available in: Atom PDF