landscape-config crashes if there is more than one tag, can't register computer

Bug #1228301 reported by Andreas Hasenack
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Landscape Client
Fix Released
High
Chris Glass

Bug Description

Trunk revision 725

# cat /etc/landscape/client.conf
[client]
ping_url = http://staging.landscape.canonical.com/ping
url = https://staging.landscape.canonical.com/message-system
# hi andreas
tags = check,linode,package-profile-test
computer_title = ls3-precise
registration_key = some-secret
include_manager_plugins = ScriptExecution
script_users = ALL
account_name = landscape-devel

# landscape-config
(...)
Script users [ALL]:

You may provide tags for this computer e.g. server,precise.

Tags [['check', 'linode', 'package-profile-test']]:
'list' object has no attribute 'split'
Aborting Landscape configuration
root@ls3-precise:~#

And you get this in the broker log if you tried landscape-config --silent:
2013-09-20 17:45:08,383 ERROR [MainThread] Error running event handler landscape.broker.registration.RegistrationHandler._handle_pre_exchange() for event type 'pre-exchange' with args () {}.
Traceback (most recent call last):
  File "/usr/lib/python2.7/dist-packages/landscape/reactor.py", line 92, in fire
    results.append(handler(*args, **kwargs))
  File "/usr/lib/python2.7/dist-packages/landscape/broker/registration.py", line 261, in _handle_pre_exchange
    if not is_valid_tag_list(tags):
  File "/usr/lib/python2.7/dist-packages/landscape/lib/tag.py", line 17, in is_valid_tag_list
    tags = [tag.strip() for tag in tag_list.split(",")]
AttributeError: 'list' object has no attribute 'split'

Tags: defect

Related branches

tags: removed: kanban
description: updated
description: updated
summary: - landscape-config crashes if there is more than one tag
+ landscape-config crashes if there is more than one tag, can't register
+ computer
Chris Glass (tribaal)
Changed in landscape-client:
status: New → In Progress
assignee: nobody → Chris Glass (tribaal)
Changed in landscape-client:
status: In Progress → Fix Committed
Changed in landscape-client:
status: Fix Committed → Fix Released
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.