Project

Profile

Help

Story #2691

closed

As a user, I can verify if content exists on a disk during publish

Added by jluza about 7 years ago. Updated almost 5 years ago.

Status:
CLOSED - WONTFIX
Priority:
Normal
Assignee:
-
Category:
-
Sprint/Milestone:
-
Start date:
Due date:
% Done:

0%

Estimated time:
Platform Release:
Groomed:
No
Sprint Candidate:
No
Tags:
Pulp 2
Sprint:
Quarter:

Description

In the case when something bad happened and files in /var/lib/pulp/content got lost, pulp publishes
repo with links pointing to non-existing files.

source server/pulp/plugins/util/misc.py
function create_symlink

if symlink exists and it's parent directory exists, function returns here:

https://github.com/pulp/pulp/blob/master/server/pulp/plugins/util/misc.py#L107

with no validation of source.

Actions #1

Updated by ttereshc about 7 years ago

  • Tracker changed from Issue to Story
  • Subject changed from Pulp should verify if source for link exists to As a user, I can verify if content exists on a disk during publish
  • % Done set to 0
Actions #2

Updated by ttereshc about 7 years ago

Several points from the IRC discussion:
1. There are valid cases when symlink can point to non-existing file, like on_demand case.
2. Several people pointed out that publish may not be a good time to validate content and it is better to do at sync time.
3. I personally think that with operational sync, content on a filesystem will be back.
When there is an optimization not to download already existing content we check if file exists on a filesystem. If there is no such optimization, content will be downloaded and moved to its final destination in any case. So one can fix /v/l/p/content with force syncs.

Actions #3

Updated by bmbouter almost 7 years ago

  • Tags RCM added
Actions #4

Updated by bmbouter about 5 years ago

  • Status changed from NEW to CLOSED - WONTFIX
Actions #5

Updated by bmbouter about 5 years ago

Pulp 2 is approaching maintenance mode, and this Pulp 2 ticket is not being actively worked on. As such, it is being closed as WONTFIX. Pulp 2 is still accepting contributions though, so if you want to contribute a fix for this ticket, please reopen or comment on it. If you don't have permissions to reopen this ticket, or you want to discuss an issue, please reach out via the developer mailing list.

Actions #6

Updated by bmbouter about 5 years ago

  • Tags Pulp 2 added
Actions #7

Updated by bmbouter almost 5 years ago

  • Tags deleted (RCM)

Also available in: Atom PDF