Bug 690902

Summary: cds sync does not relay the message to the client if the hostname provided is not a registered CDS.
Product: [Retired] Pulp Reporter: Preethi Thomas <pthomas>
Component: nodesAssignee: Jay Dobies <jason.dobies>
Status: CLOSED CURRENTRELEASE QA Contact: Preethi Thomas <pthomas>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 1.0.0CC: jconnor, skarmark
Target Milestone: ---Keywords: Triaged
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2012-02-24 20:14:33 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:

Description Preethi Thomas 2011-03-25 18:33:47 UTC
Description of problem:


[root@preethi ~]# pulp-admin cds sync --hostname=pulp-cds.usersys.redhat
Sync for CDS [pulp-cds.usersys.redhat] started
Use "cds status" to check on the progress



from the pulp.log


2011-03-25 14:38:36,603 [INFO][Thread-60] cds_sync() @ cds.py:302 - Synchronizing CDS [preethi.usersys.redhat.com]
2011-03-25 14:38:36,605 [ERROR][Thread-60] failed() @ task.py:207 - Task id:1857e9b8-570f-11e0-9615-002564a85a58, method_name:cds_sync:
Traceback (most recent call last):
  File "/usr/lib/python2.7/site-packages/pulp/server/tasking/task.py", line 131, in run
    result = self.callable(*self.args, **self.kwargs)
  File "/usr/lib/python2.7/site-packages/pulp/server/api/cds.py", line 307, in cds_sync
    raise PulpException('CDS with hostname [%s] could not be found' % cds_hostname)
PulpException: u'CDS with hostname [preethi.usersys.redhat.com] could not be found'

Comment 1 Jay Dobies 2011-10-26 14:33:59 UTC
commit 750ad0789797686701fdcaf6ce3d79ba70a3d974
Author: Jay Dobies <jason.dobies>
Date:   Wed Oct 26 10:33:47 2011 -0400

    690902 - Since the sync runs in an async task, we need to add a check
    for hostname validity before triggering the task so we can inform the
    caller that the invocation was invalid

src/pulp/server/webservices/controllers/cds.py

Comment 2 Jeff Ortel 2011-10-27 03:22:26 UTC
build 0.242-2

Comment 3 Preethi Thomas 2011-11-15 14:23:40 UTC
verified
[root@preethi ~]# rpm -q pulp
pulp-0.0.248-1.fc15.noarch
[root@preethi ~]# 

[root@preethi ~]# pulp-admin cds sync --hostname=pulp-cds.usersys.redhat
error:  error: operation failed: Could not find CDS with hostname [pulp-cds.usersys.redhat]
[root@preethi ~]#

Comment 4 Preethi Thomas 2012-02-24 20:14:33 UTC
Pulp v1.0 is released
Closed Current Release.

Comment 5 Preethi Thomas 2012-02-24 20:19:24 UTC
Pulp v1.0 is released.