landscape module not working

Bug #1277746 reported by Matt Fischer
14
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Landscape Client
Invalid
Undecided
Unassigned
cloud-init
Fix Released
Low
Unassigned

Bug Description

I've been trying to boot a saucy guest and have been unable to get landscape configured properly. Following the landscape example, I've run into a few issues.

My first config was pretty full (like the example), but after lots of failures, I pared it down to this simple case:

#cloud-config
landscape:
  client:
    computer_title = fred
    account_name = example

When I do that, it writes out the landscape client conf like this:
client = computer_title = fred account_name = example

This is obviously incorrect and landscape complains thusly.

Am I doing this wrong or are the docs perhaps incorrect?

Related branches

Revision history for this message
David Britton (dpb) wrote :

Hi --

I think your yaml is off under 'client', you need those to be properlly formatted key/value declarations with ':', not '='. See the following python interactive example to see the difference:

dpb@helo:cloudinit$ python
Python 2.7.5+ (default, Sep 19 2013, 13:48:49)
[GCC 4.8.1] on linux2
Type "help", "copyright", "credits" or "license" for more information.
>>> import yaml
>>> yaml.load("client:\n computer_title = fred\n account_name = example")
{'client': 'computer_title = fred account_name = example'}
>>> yaml.load("client:\n computer_title: fred\n account_name: example\n")
{'client': {'computer_title': 'fred', 'account_name': 'example'}}
>>>

So, try instead:

#cloud-config
landscape:
  client:
    computer_title: fred
    account_name: example

Also, let us know if documentation is off anywhere that shows this example.

Thanks!

Revision history for this message
Matt Fischer (mfisch) wrote :

This is where I found the incorrect info, I believe this same file is in the cloud-init source package:

https://github.com/number5/cloud-init/blob/master/doc/examples/cloud-config-landscape.txt

Revision history for this message
Matt Fischer (mfisch) wrote :

Confirmed it is also wrong in the cloud-init source package (as an example)

Revision history for this message
David Britton (dpb) wrote :

OK, Thanks --

I'll look into getting this fixed.

Revision history for this message
Scott Moser (smoser) wrote :

Matt,
  Thanks for the bug.

David,
  Thanks for the fix.

Changed in cloud-init:
status: New → Fix Committed
importance: Undecided → Low
Scott Moser (smoser)
Changed in landscape-client:
status: New → Invalid
Changed in cloud-init:
status: Fix Committed → Fix Released
Revision history for this message
James Falcon (falcojr) wrote :
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.