Bug 690902 - cds sync does not relay the message to the client if the hostname provided is not a registered CDS.
Summary: cds sync does not relay the message to the client if the hostname provided is...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Pulp
Classification: Retired
Component: nodes
Version: 1.0.0
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
: ---
Assignee: Jay Dobies
QA Contact: Preethi Thomas
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2011-03-25 18:33 UTC by Preethi Thomas
Modified: 2012-02-24 20:19 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2012-02-24 20:14:33 UTC

Attachments (Terms of Use)
Add an attachment (proposed patch, testcase, etc.)

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.
Note You need to log in before you can comment on or make changes to this bug.