Project

Profile

Help

Issue #1659

closed

ISO Sync is not performed correctly if download policy was changed.

Added by ipanova@redhat.com over 8 years ago. Updated over 5 years ago.

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

Description

1. create iso repo with on_demand policy
2 sync&publish
3 delete repo
4 re-create same repo with same name and feed but change the policy to immediate
5 sync and publish
6 observe that ISOs were not downloaded.

$ pulp-admin iso repo create --repo-id iso --feed https://repos.fedorapeople.org/repos/pulp/pulp/demo_repos/test_file_repo/ --download-policy on_demand

Successfully created repository [iso]

[ipanova@ina pulp_rpm]$ pulp-admin iso repo sync run --repo-id iso

+----------------------------------------------------------------------+
                     Synchronizing Repository [iso]
+----------------------------------------------------------------------+

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

[-]
Waiting to begin...
Downloading the Pulp Manifest...
The Pulp Manifest was downloaded successfully.

There are no ISOs that need to be downloaded.

Task Succeeded

The repository was successfully published.

Task Succeeded

[ipanova@ina pulp_rpm]$ pulp-admin iso repo delete --repo-id iso

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

[\]
Running...

Repository [iso] successfully deleted

[ipanova@ina pulp_rpm]$ pulp-admin iso repo create --repo-id iso --feed https://repos.fedorapeople.org/repos/pulp/pulp/demo_repos/test_file_repo/

Successfully created repository [iso]

[ipanova@ina pulp_rpm]$ pulp-admin iso repo sync run --repo-id iso

+----------------------------------------------------------------------+
                     Synchronizing Repository [iso]
+----------------------------------------------------------------------+

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

Downloading the Pulp Manifest...
The Pulp Manifest was downloaded successfully.

There are no ISOs that need to be downloaded.

Task Succeeded

The repository was successfully published.

Task Succeeded

]# ll /var/lib/pulp/published/https/isos/iso
total 16
-rw-r--r--. 1 apache apache 251 Feb 11 12:25 PULP_MANIFEST
lrwxrwxrwx. 1 apache apache 107 Feb 11 12:25 test2.iso -> /var/lib/pulp/content/units/iso/ae/03e75b8d77b5f6cb1a6f95c91c883b7dc0321bf952978d424cb1fb573826ad/test2.iso
lrwxrwxrwx. 1 apache apache 107 Feb 11 12:25 test3.iso -> /var/lib/pulp/content/units/iso/7d/4ac67f9e933d8c11d202d759d6be598ddc287fb0e55b2e77035b5b03797f72/test3.iso
lrwxrwxrwx. 1 apache apache 106 Feb 11 12:25 test.iso -> /var/lib/pulp/content/units/iso/06/87471ad1bb04c3757fb52e3bf466644f49cfceafdcfcb88de7f771ca3f6484/test.iso

# ll /var/lib/pulp/content/units/iso/ae/03e75b8d77b5f6cb1a6f95c91c883b7dc0321bf952978d424cb1fb573826ad/test2.iso
ls: cannot access /var/lib/pulp/content/units/iso/ae/03e75b8d77b5f6cb1a6f95c91c883b7dc0321bf952978d424cb1fb573826ad/test2.iso: No such file or directory
Actions #1

Updated by ipanova@redhat.com over 8 years ago

  • Project changed from Pulp to RPM Support
  • Platform Release set to 2.8.0
Actions #2

Updated by ipanova@redhat.com over 8 years ago

Issue is caused by this piece of code https://github.com/pulp/pulp_rpm/blob/master/plugins/pulp_rpm/plugins/importers/iso/sync.py#L344 logic should be re-worked.

Actions #3

Updated by ipanova@redhat.com over 8 years ago

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

Updated by ipanova@redhat.com over 8 years ago

  • Status changed from ASSIGNED to POST
Actions #5

Updated by bmbouter over 8 years ago

  • Triaged changed from No to Yes
Actions #6

Updated by amacdona@redhat.com over 8 years ago

  • Status changed from POST to MODIFIED
Actions #7

Updated by dkliban@redhat.com over 8 years ago

  • Status changed from MODIFIED to 5
Actions #8

Updated by dkliban@redhat.com over 8 years ago

  • Status changed from 5 to CLOSED - CURRENTRELEASE
Actions #9

Updated by bmbouter over 5 years ago

  • Tags Pulp 2 added

Also available in: Atom PDF