cloud registration doesn't work if there's no user-data
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Landscape Client |
Fix Released
|
Medium
|
Christopher Armstrong | ||
Landscape Server |
Fix Released
|
Medium
|
Christopher Armstrong | ||
landscape-client (Ubuntu) |
Fix Released
|
Undecided
|
Unassigned |
Bug Description
The case where the client is put into cloud mode manually (and an account_name and registration_
Here's what happens:
ubuntu@
2009-03-17 11:28:02,029 INFO [MainThread] Message exchange completed in 1.42s.
2009-03-17 11:28:34,838 INFO [MainThread] Broker stopped on 'system' bus with config /etc/landscape/
2009-03-17 11:28:35,461 INFO [MainThread] Broker started on 'system' bus with config /etc/landscape/
2009-03-17 11:28:35,492 ERROR [MainThread] Got error while fetching meta-data: <HTTPCodeError http_code=404>
Traceback (most recent call last):
File "/usr/lib/
result = f(*args, **kwargs)
File "/usr/lib/
raise HTTPCodeError(
HTTPCodeError: Server returned HTTP code 404
2009-03-17 11:29:35,468 INFO [MainThread] Queueing message to register with account 'christopher.
2009-03-17 11:29:35,468 ERROR [MainThread] Error running event handler landscape.
Traceback (most recent call last):
File "/usr/lib/
results.
File "/usr/lib/
message.
AttributeError: 'RegistrationHa
2009-03-17 11:29:35,470 INFO [MainThread] Starting urgent message exchange with https:/
2009-03-17 11:29:36,990 INFO [Dummy-1 ] Sent 488 bytes and received 70 bytes in 1.52s.
2009-03-17 11:29:36,994 INFO [MainThread] Switching to normal exchange mode.
First it's getting that 404, which causes it to not create the _ec2_data attribute, and then later when actually trying to send the register-cloud-vm message it blows up because that attribute isn't there.
Changed in landscape: | |
milestone: | none → mountainview-pre-7 |
Changed in landscape-client: | |
assignee: | nobody → radix |
importance: | Undecided → Medium |
status: | New → In Progress |
Changed in landscape: | |
assignee: | nobody → radix |
importance: | Undecided → Medium |
status: | New → In Progress |
Changed in landscape: | |
milestone: | mountainview-pre-7 → mountainview-pre-8 |
Changed in landscape: | |
status: | Fix Committed → Fix Released |
Changed in landscape: | |
milestone: | mountainview-pre-8 → mountainview |
status: | Fix Released → Fix Committed |
Changed in landscape: | |
status: | Fix Committed → Fix Released |
Changed in landscape-client: | |
status: | Fix Committed → Fix Released |
Here we go!
[1]
reactor_ mock = self.mocker. patch(self. reactor)
reactor_ mock.fire( "registration- failed" )
self. mocker. replay( )
- # Mock registration-failed call
+ # Mock registration-failed call. why? I don't know.
You created a docstring, so I suppose you know now?
The rest is great, +1.