pdns-recursor silent on erroneous rec_control reload-lua-script #278

Closed
Habbie opened this Issue Apr 26, 2013 · 2 comments

Projects

None yet

1 participant

@Habbie
Member
Habbie commented Apr 26, 2013

After starting up pdns-recursor with a lua-dns-script script, then removing the stript and doing a rec_control reload-lua-script, no error/warning message appears in the logs

root@ds9:/etc/powerdns# cat recursor.conf | grep "^lua"
lua-dns-script=/etc/powerdns/test.lua
root@ds9:/etc/powerdns# ls -la test.lua
-r--r--r-- 1 root root 745 2010-03-08 17:36 test.lua
root@ds9:/etc/powerdns# /etc/init.d/pdns-recursor start
 * Starting PowerDNS recursor pdns-recursor
Mar 08 17:40:33 PowerDNS recursor 3.2 (C) 2001-2010 PowerDNS.COM BV (Mar  7 2010, 22:05:14, gcc 4.2.4 (Ubuntu 4.2.4-1ubuntu4)) starting up
[...]
root@ds9:/etc/powerdns# mv test.lua /tmp
root@ds9:/etc/powerdns# rec_control reload-lua-script
ok, reload/unload queued
root@ds9:/etc/powerdns# tail -3 /var/log/messages
Mar  8 17:41:16 ds9 pdns_recursor[32052]: 0 (Re)loaded lua script from '/etc/powerdns/test.lua'
Mar  8 17:41:16 ds9 pdns_recursor[32052]: 1 (Re)loaded lua script from '/etc/powerdns/test.lua'
Mar  8 17:41:20 ds9 pdns[31258]: gmysql Connection succesful
root@ds9:/etc/powerdns# 

The old lua script still works. There should be some error/warning message that the reload failed.

@Habbie Habbie was assigned Apr 26, 2013
@Habbie Habbie closed this Apr 26, 2013
@Habbie Habbie added a commit that referenced this issue Apr 26, 2013
@Habbie Habbie report error on rec_control reload-lua-script, fixing #278
git-svn-id: svn://svn.powerdns.com/pdns/trunk/pdns@2627 d19b8d6e-7fed-0310-83ef-9ca221ded41b
0f39c1a
@Habbie
Member
Habbie commented Apr 26, 2013

Author: peter
current SVN logs

Jun 15 11:59:23 0 Retaining current script, error from './powerdns-example-script.lua': Error loading LUA file './powerdns-example-script.lua': cannot open ./powerdns-example-script.lua: No such file or directory
Jun 15 11:59:23 0 (Re)loaded lua script from './powerdns-example-script.lua'

Reporting the error to rec_control is harder; keeping ticket open for that.

@Habbie
Member
Habbie commented Apr 26, 2013

Author: peter
Fixed in r2627

@mind04 mind04 pushed a commit to mind04/pdns that referenced this issue Apr 26, 2013
peter report error on rec_control reload-lua-script, fixing #278
git-svn-id: svn://svn.powerdns.com/pdns/trunk/pdns@2627 d19b8d6e-7fed-0310-83ef-9ca221ded41b
4904399
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment