Bug 1032132
Summary: | puppet repo sync cancel failing | ||
---|---|---|---|
Product: | [Retired] Pulp | Reporter: | Preethi Thomas <pthomas> |
Component: | puppet-support | Assignee: | Jason Connor <jconnor> |
Status: | CLOSED CURRENTRELEASE | QA Contact: | Preethi Thomas <pthomas> |
Severity: | unspecified | Docs Contact: | |
Priority: | unspecified | ||
Version: | 2.3 | CC: | jortel, mhrivnak, mmccune |
Target Milestone: | --- | Keywords: | Triaged |
Target Release: | 2.3.0 | ||
Hardware: | Unspecified | ||
OS: | Unspecified | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | Bug Fix | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2013-12-09 14:31:44 UTC | Type: | Bug |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: |
Description
Preethi Thomas
2013-11-19 15:32:06 UTC
The bug is here: https://github.com/pulp/pulp_puppet/blob/9be8c67/pulp_puppet_plugins/pulp_puppet/plugins/importers/sync.py#L94 This call to cancel() requires an argument that isn't being passed. It's worth noting that the cancel method that requires an argument does not actually use that argument. That should be resolved by either removing it or adding the missing logic to use it. It's confusing that there are different kinds of downloaders with slightly different APIs. The nectar downloaders have a "cancel()" method that takes no arguments, but now the puppet importer apparently has its own objects called "downloaders". They have a cancel method that does take an argument. It might be less confusing if the "downloaders" in the puppet import had a different name, like DownloadCoordinator. They don't actually do any downloading, but rather are a layer between the importer and the nectar downloaders. Awaiting review: https://github.com/pulp/pulp_puppet/pull/88 build: pulp-2.3.0-0.38.beta. puppet-sync gets cancelled now. It does not give a message. But that was the same behavior in 2.2 as well. So I will write up another bug on it. [root@pulp-v2-server ~]# rpm -qa pulp-server pulp-server-2.3.0-0.38.beta.el6.noarch [root@pulp-v2-server ~]# [root@pulp-v2-server ~]# pulp-admin puppet repo sync run --repo-id puppet-repo +----------------------------------------------------------------------+ Synchronizing Repository [puppet-repo] +----------------------------------------------------------------------+ This command may be exited by pressing ctrl+c without affecting the actual operation on the server. Downloading metadata... [==================================================] 100% Metadata Query: 1/1 items ... completed Downloading new modules... [= ] 2% Module: 45/1813 items Pulp 2.3 released. |