Modify

Opened 4 years ago

Closed 4 years ago

Last modified 4 years ago

#17305 closed defect (not_a_bug)

procd: procd_add_reload_trigger doesn't trigger reload_service

Reported by: larsg Owned by: developers
Priority: normal Milestone:
Component: base system Version: Trunk
Keywords: Cc:

Description

This has never quite worked for me on BB trunk, at least in the past 3 months. The whole init script: https://github.com/SeattleMeshnet/meshbox/blob/mgmt/cjdns/files/etc/init.d/cjdns

reload_service()
{
	date >> /tmp/reload_service
	ubus send cjdns.reload '{}'
}

service_triggers()
{
	date >> /tmp/service_triggers
	procd_add_reload_trigger cjdns
}

service_triggers gets called correctly on procd startup, and produces the expected output in /tmp. In order to trigger the reload, I do this: uci set cjdns.cjdns.foo=bar && uci commit, but nothing seems to happen. Calling /etc/init.d/cjdns reload by hand works as expected.

Attachments (0)

Change History (2)

comment:1 Changed 4 years ago by nbd

  • Resolution set to not_a_bug
  • Status changed from new to closed

uci commit does not imply reloading the config. call /sbin/reload_config if you want to apply your changes.

comment:2 Changed 4 years ago by larsg

I wasn't notified about your comment somehow, sorry about the poking on IRC ;) Anyway, thanks for the hint, misunderstanding on my part.

Add Comment

Modify Ticket

Action
as closed .
The resolution will be deleted. Next status will be 'reopened'.
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.