Bug 634884 - can not manage channel with repositories through XMLRPC API
Summary: can not manage channel with repositories through XMLRPC API
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat Satellite 5
Classification: Red Hat
Component: Server
Version: 540
Hardware: All
OS: Linux
low
medium
Target Milestone: ---
Assignee: Lukas Zapletal
QA Contact: Šimon Lukašík
URL:
Whiteboard:
Depends On:
Blocks: sat540-blockers sat540-api, sat540-apis
TreeView+ depends on / blocked
 
Reported: 2010-09-17 09:01 UTC by Jan Hutař
Modified: 2010-10-28 14:51 UTC (History)
3 users (show)

Fixed In Version: spacewalk-java-1.2.39-7-sat
Doc Type: Bug Fix
Doc Text:
Clone Of:
: 641279 (view as bug list)
Environment:
Last Closed: 2010-10-28 14:51:22 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)

Description Jan Hutař 2010-09-17 09:01:35 UTC
Description of problem:
I have tried to create channel with assigned repository using XMLRPC API call and was not able to do so.


Version-Release number of selected component (if applicable):
Galactica 20100910.2


How reproducible:
1 of 1


Steps to Reproduce:
1. Use this XMLRPC call to API:
2. API: ret = client.channel.software.create(key, CHANNEL, \
                     'Name %s' % CHANNEL, \
                     'Summary %s' % CHANNEL, \
                     ARCH, PARENT, CHECKSUM, \
                     {'url': '', 'id': '', 'fingerprint': ''}, \
                     {'url': 'http://10.34.26.211/TESTING/repo6/', 'label': CHANNEL+'repo', 'sync': True})
3. Check if channel exist and have repository assigned


Actual results:
Channel is created, but repo not and it is also not assigned to it. This also do not allow me to create channel with multiple repos.


Expected results:
I should be able to create channel with multiple repos. I guess that as repos are standalone entities, proper fix here would be to remove these possibilities from client.channel.software.create() call (as they do not work anyway) and create client.repo.create()/.delete()/getDetails()/... and a call to assign (or remove) a repo to a channel.

IMO this should be fixed or removed from docs before GA.


Additional info:
I was not able to find any error in logs regarding this.

Also client.channel.software.getDetails(key, CHANNEL) shows *only one* assigned repository, although channel have two in WebUI.

Comment 1 Clifford Perry 2010-09-17 15:37:15 UTC
I agree with the "remove these possibilities from the client.channel.software.create() call (as they do not work anyway)" - if we can, just comment out/remove it from the API doc for the API call. 

We can look at correctly implementing these API's post 5.4 GA.

Cliff

Comment 2 Lukas Zapletal 2010-09-20 07:51:58 UTC
Functionality added with bug 562463. I will revert it for now and reopen the bug so we can add this functionality properly later on.

Comment 3 Lukas Zapletal 2010-09-20 08:48:49 UTC
Nonfunctional possibility to add repository along with new channel over XMLRPC API has been removed from code and documentation. We do need to create new API (client.repo.create()/.delete()/getDetails()/...) that will handle repositories.

3d3626e2bb9073f53b3fabc15fbc52d8bdf2aac5

Comment 4 Lukas Zapletal 2010-09-20 09:39:44 UTC
commit a20579181e4a52706acfed94ff78ca8951766460
Author: Lukas Zapletal <lzap+git>
Date:   Mon Sep 20 10:44:29 2010 +0200

    634884 - managing repositories within channels through XMLRPC API disabled
    (cherry picked from commit 3d3626e2bb9073f53b3fabc15fbc52d8bdf2aac5)

Comment 5 Šimon Lukašík 2010-10-08 09:24:27 UTC
Changing to VERIFIED:

Testing procedure:
Repository structure was removed from 
documentation as well as from handler.

Verified against:
Satellite-5.4.0-RHEL5-re20101001.1

Comment 7 Clifford Perry 2010-10-28 14:46:26 UTC
The 5.4.0 RHN Satellite and RHN Proxy release has occurred. This issue has been resolved with this release. 


RHEA-2010:0801 - RHN Satellite Server 5.4.0 Upgrade
https://rhn.redhat.com/rhn/errata/details/Details.do?eid=10332

RHEA-2010:0803 - RHN Tools enhancement update
https://rhn.redhat.com/rhn/errata/details/Details.do?eid=10333

RHEA-2010:0802 - RHN Proxy Server 5.4.0 bug fix update
https://rhn.redhat.com/rhn/errata/details/Details.do?eid=10334

RHEA-2010:0800 - RHN Satellite Server 5.4.0
https://rhn.redhat.com/rhn/errata/details/Details.do?eid=10335

Docs are available:

http://docs.redhat.com/docs/en-US/Red_Hat_Network_Satellite/index.html 

Regards,
Clifford


Note You need to log in before you can comment on or make changes to this bug.