[astute] Check cobbler system getks before deployment

Bug #1264979 reported by Matthew Mosesohn
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Won't Fix
Low
Unassigned

Bug Description

We could prevent bugs #1264515 from happening if we checked the exit code from cobbler system getks --name=node-XX for each host before rebooting nodes. If there is a server error, it will return 1. The output is quite useless in all the cases I can produce an error, so we should simply fail deployment with "Check Cobbler logs for details".

Tags: astute
Changed in fuel:
assignee: Fuel Python Team (fuel-python) → Vladimir Sharshov (vsharshov)
Revision history for this message
Vladimir Sharshov (vsharshov) wrote :

Hi, Matt.

Can you help me and find XML-RPC call which do same as 'cobbler system getks --name=node-XX'. Is it http://fossies.org/dox/cobbler-cobbler-2.4.0-1/classcobbler_1_1remote_1_1CobblerXMLRPCInterface.html#a7d8b60e18aa4c481c479668e02e54af5 ?

Revision history for this message
Matthew Mosesohn (raytrac3r) wrote :

Vladimir,
You don't need XMLRPC to download KS. It's available via HTTP. Log into Cobbler Web, and hit view kickstart for the host you are interested in. That URL will work and you can substitute the host name for each host you need. The page will return 500 internal server error if you have problems rendering the file.

Changed in fuel:
importance: Undecided → Medium
Dmitry Pyzhov (dpyzhov)
Changed in fuel:
status: New → Confirmed
assignee: Vladimir Sharshov (vsharshov) → nobody
tags: added: astute
removed: nailgun
Dmitry Pyzhov (dpyzhov)
Changed in fuel:
milestone: 4.1 → 5.0
Revision history for this message
Matthew Mosesohn (raytrac3r) wrote :

We still need this because Ubuntu preseed rendering failures result in cryptic errors.

Changed in fuel:
assignee: nobody → Fuel Python Team (fuel-python)
Revision history for this message
Matthew Mosesohn (raytrac3r) wrote :

Also, bugs should still always have an owner (even if it's a team)

Revision history for this message
Dmitry Pyzhov (dpyzhov) wrote :

We are going to switch to ironic

Changed in fuel:
importance: Medium → Low
milestone: 5.0 → 5.1
Revision history for this message
Matthew Mosesohn (raytrac3r) wrote :

We should consider this fix for 5.1 (we won't see ironic till Fuel 6)

Dmitry Ilyin (idv1985)
summary: - Check cobbler system getks before deployment
+ [neutron] Check cobbler system getks before deployment
Revision history for this message
Dima Shulyak (dshulyak) wrote : Re: [neutron] Check cobbler system getks before deployment

How this is related to neutron?

summary: - [neutron] Check cobbler system getks before deployment
+ [astute] Check cobbler system getks before deployment
Dima Shulyak (dshulyak)
Changed in fuel:
assignee: Fuel Python Team (fuel-python) → Fuel Astute Team (fuel-astute)
Revision history for this message
Dmitry Pyzhov (dpyzhov) wrote :
Changed in fuel:
status: Confirmed → Won't Fix
Curtis Hovey (sinzui)
Changed in fuel:
assignee: Registry Administrators (registry) → nobody
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.