Bug 562881

Summary:	Need to add more cobbler related apis
Product:	Red Hat Satellite 5	Reporter:	Justin Sherrill <jsherril>
Component:	API	Assignee:	Tomas Lestach <tlestach>
Status:	CLOSED ERRATA	QA Contact:	Garik Khachikyan <gkhachik>
Severity:	medium	Docs Contact:
Priority:	low
Version:	530	CC:	gkhachik, mkoci
Target Milestone:	---
Target Release:	---
Hardware:	All
OS:	Linux
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2010-03-03 14:18:16 UTC	Type:	---
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:
Bug Depends On:
Bug Blocks:	518263

Description Justin Sherrill 2010-02-08 16:16:46 UTC

We need to add some more cobbler related apis.  Here's a list:

kickstart.snippet.addCustom
kickstart.snippet.deleteCustom
kickstart.snippet.modifyCustom
kickstart.snippet.listDefault
kickstart.snippet.listCustom
kickstart.snippet.listAll'

kickstart.profile.getVariables
kickstart.profile.setVariables


system.createSystemRecord
system.getVariables
system.setVariables

Comment 1 Justin Sherrill 2010-02-08 16:23:44 UTC

https://fedorahosted.org/spacewalk/wiki/ApiAdditions

Comment 2 Justin Sherrill 2010-02-08 23:04:02 UTC

Added 

kickstart.snippet.listDefault
kickstart.snippet.listCustom
kickstart.snippet.listAll

in commit to master:

8d5c5b3f577c9c885e94861d33a27345844f575a

Comment 3 Justin Sherrill 2010-02-09 21:52:16 UTC

Added

kickstart.snippet.createOrUpdate
kickstart.snippet.delete



25db32510ffc84f6acd9d40f0c02d54c829ffba9

Comment 4 Tomas Lestach 2010-02-11 17:29:47 UTC

Followin api call added:

kickstart.profile.getVariables
kickstart.profile.setVariables
system.createSystemRecord
system.getVariables
system.setVariables   

spacewalk.git: 5e19c997d4208175942b2bfb43586a5965bc8b1d

Comment 6 Tomas Lestach 2010-02-17 13:54:50 UTC

satellite.git:
 fddd6ff12c47ec43a0ec25470746b97ed602b2c7
 b3fc934fccda0923adeea352f4f5b8bb9e3444e0
 66a3ffa91b4fd0a31a5bedc08217d817319cd499

Comment 10 Tomas Lestach 2010-02-23 20:54:13 UTC

additional commits:

* fixing docs for list snippet api calls
  - spacewalk.git: 21ee1bf40342e0e9f5e0e194e6be1906ebf7d470
  - satellite.git: 256bfae7438d4ae04f19baf2ad6ad25db31101e8

* cobbler system record check
  - spacewalk.git: cdd1c75f89ab346e5eff998dc235096bf62ff23c
  - satellite.git: 26574dd733a1eff8e8673efecadbd9f296a7b426

* apidoc fixes
  - spacewalk.git: eb0ec53e6e9af526d64061a675604d2add332712
  - satellite.git: ca9cec2dc0bf5b8acc03851a59f7523da502137c

* save cobbler object after setting kickstart variables
  - spacewalk.git: 268907eb29b62d6ffd9c86af48c3cd2c9db4d79e
  - satellite.git: a6e2e99a2d3bba0aec89a894753e76c0e5e803a9

Comment 11 Garik Khachikyan 2010-02-24 11:25:01 UTC

# COMMENT for myself

the API call system.getVariables(key,systemid) returns:
---
[variables - {'redhat_management_server': 'rlx-0-24.rhndev.redhat.com', 'media_path': '/ty/5kqXpBBn'}]
[netboot - True]
---
for a system that is/was provisioning and has ks profile assigned to it. For the ones not having the ks profile system returns with:
---
xmlrpclib.Fault: <Fault -214: 'redstone.xmlrpc.XmlRpcFault: No such cobbler system record'>
---

Comment 12 Garik Khachikyan 2010-02-24 12:42:10 UTC

# VERIFIED (gkhachik)

All the calls specified in the bug comments 2,3,4 are functioning ok. API documentation part is also reflects to the real call behavior.

Checked against Satellite installed on RHEL 4 AS and RHEL 5 Server.

Package(s) fixing the issue are: spacewalk-java-0.5.44-66.6.el[4,5]sat

Comment 14 errata-xmlrpc 2010-03-03 14:18:16 UTC

An advisory has been issued which should help the problem
described in this bug report. This report is therefore being
closed with a resolution of ERRATA. For more information
on therefore solution and/or where to find the updated files,
please follow the link below. You may reopen this bug report
if the solution does not work for you.

http://rhn.redhat.com/errata/RHBA-2010-0128.html