Project

Profile

Help

Issue #1835

closed

export fails when units are not downloaded

Added by mhrivnak about 8 years ago. Updated over 4 years ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
High
Assignee:
Sprint/Milestone:
-
Start date:
Due date:
Estimated time:
Severity:
2. Medium
Version:
2.8.0
Platform Release:
2.8.3
OS:
Triaged:
Yes
Groomed:
No
Sprint Candidate:
No
Tags:
Pulp 2
Sprint:
Quarter:

Description

When a repository has been sync'd with an on_demand download policy, and one or more units have not been fully downloaded, the export distributor will fail, reporting numerous file copy operation failures. The distributor should instead fail gracefull.

A good option would be to check at the beginning of the task if all units in the repo are downloaded, using a simple database query. Fail gracefully if not. This would prevent it from attempting any copy operations.

Actions #2

Updated by mhrivnak about 8 years ago

  • Platform Release set to 2.8.3
Actions #3

Updated by mhrivnak about 8 years ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to mhrivnak
Actions #4

Updated by mhrivnak about 8 years ago

https://github.com/pulp/pulp/pull/2520

There will be a pulp_rpm PR also.

Actions #5

Updated by mhrivnak about 8 years ago

When trying to export a repo with undownloaded files, this should be the new behavior:

$ pulp-admin rpm repo export run --repo-id=rhel7
+----------------------------------------------------------------------+
                     Publishing Repository [rhel7]
+----------------------------------------------------------------------+

This command may be exited via ctrl+c without affecting the request.

Task Failed

The repository cannot be exported because some units have not yet been
downloaded.

When trying to export a repo group with undownloaded files, this should be the new behavior:

$ pulp-admin rpm repo group export run --group-id=g1
+----------------------------------------------------------------------+
                    Exporting Repository Group [g1]
+----------------------------------------------------------------------+

This command may be exited via ctrl+c without affecting the request.

Task Failed

The repository group cannot be exported because these repos have units that have
not yet been downloaded: rhel7, rhel7-copy

Added by mhrivnak about 8 years ago

Revision 9488db3a | View on GitHub

added distributor methods to ensure all units in a repo have been downloaded

https://pulp.plan.io/issues/1835

re #1835

Added by mhrivnak about 8 years ago

Revision 9488db3a | View on GitHub

added distributor methods to ensure all units in a repo have been downloaded

https://pulp.plan.io/issues/1835

re #1835

Added by mhrivnak about 8 years ago

Revision 2b8093fd | View on GitHub

Export distributors now fail gracefully when files have not been downloaded

https://pulp.plan.io/issues/1835

fixes #1835

Actions #9

Updated by mhrivnak about 8 years ago

  • Status changed from ASSIGNED to POST
Actions #10

Updated by mhrivnak about 8 years ago

  • Priority changed from Normal to High
  • Triaged changed from No to Yes
Actions #11

Updated by mhrivnak about 8 years ago

  • Status changed from POST to MODIFIED
  • % Done changed from 0 to 100
Actions #12

Updated by pthomas@redhat.com almost 8 years ago

Verified

[root@mgmt5 ~]# rpm -qa pulp-server
pulp-server-2.8.3-0.1.beta.git.68.46caf15.el7.noarch
[root@mgmt5 ~]# 

[root@mgmt5 ~]# pulp-admin rpm repo group export run --group-id grp1
+----------------------------------------------------------------------+
                   Exporting Repository Group [grp1]
+----------------------------------------------------------------------+

This command may be exited via ctrl+c without affecting the request.

Task Failed

The repository group cannot be exported because these repos have units that are
not downloaded: zoo

[root@mgmt5 ~]# pulp-admin rpm repo export run  --repo-id zoo
+----------------------------------------------------------------------+
                      Publishing Repository [zoo]
+----------------------------------------------------------------------+

This command may be exited via ctrl+c without affecting the request.

Task Failed

The repository cannot be exported because some units are not downloaded.

[root@mgmt5 ~]# 
Actions #13

Updated by semyers almost 8 years ago

  • Status changed from MODIFIED to 5
Actions #14

Updated by pthomas@redhat.com almost 8 years ago

  • Status changed from 5 to 6
Actions #15

Updated by pthomas@redhat.com almost 8 years ago

verified

Actions #16

Updated by semyers almost 8 years ago

  • Status changed from 6 to CLOSED - CURRENTRELEASE
Actions #18

Updated by bmbouter about 5 years ago

  • Tags Pulp 2 added

Also available in: Atom PDF