landscape-client should have a native Upstart job

Bug #808488 reported by Mark Russell
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Landscape Client
Invalid
Low
Unassigned

Bug Description

I've attached a proposed script. Due to the way Upstart works, I don't think there's a need to manage the PID file. However, I'm a little less sure about whether the just-in-case cleaning of /var/lib/landscape/client/sockets is necessary. Might still be the case.

Also: again, due to Upstart, it makes more sense to run landscape-client in the foreground rather than daemonize. It's simpler that way. The only downside is that it will emit "started landscape-client" as soon as it is executed. In some cases this is a problem. The started event might fire before the daemon has fully initialized or started listening. Since landscape-client does not listen, and as far as I know, has no reverse dependencies, this should be no problem.

Revision history for this message
Mark Russell (marrusl) wrote :
Changed in landscape-client:
milestone: none → backlog
importance: Undecided → Low
Revision history for this message
Mark Russell (marrusl) wrote :

Looks like landscape-config calls the old init script directly to restart:

"sh: /etc/init.d/landscape-client: not found
Couldn't restart the Landscape client.
This machine will be registered with the provided details when the client runs."

Revision history for this message
Andreas Hasenack (ahasenack) wrote :

It's lacking a symlink /etc/init.d/landscape-client -> /lib/init/upstart-job

Revision history for this message
🤖 Landscape Builder (landscape-builder) wrote :

This bug has not seen any activity in the last 6 months, so it is being automatically closed.

If you are still experiencing this issue, please feel free to re-open.

Landscape Team

Changed in landscape-client:
status: New → Invalid
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.