BIND 10 #2036: Integrate valgrind into our test suite

[BIND 10 Development]

#2036: Integrate valgrind into our test suite
-------------------------------------+-------------------------------------
                   Reporter:  muks   |                 Owner:  muks
                       Type:         |                Status:  reviewing
  enhancement                        |             Milestone:
                   Priority:         |  Sprint-20120619
  medium                             |            Resolution:
                  Component:  build  |             Sensitive:  0
  system                             |           Sub-Project:  DNS
                   Keywords:         |  Estimated Difficulty:  0
            Defect Severity:  N/A    |           Total Hours:  0
Feature Depending on Ticket:         |
        Add Hours to Ticket:  0      |
                  Internal?:  0      |
-------------------------------------+-------------------------------------
Changes (by jelte):

 * owner:  jelte => muks


Comment:

 I still see lots and lots of output with --enable-valgrind-suppressions
 btw. If you want i have a diff that blindly adds them to the .revisit file
 (more than 5000 lines of additional suppressions...).

 there's also the 'problem' we discussed on jabber; The FDTest.read and
 FDTest.write tests fail, and if they are fixed with a memset, they still
 fail if run together with the FDShare test). I guess that fixing those
 isn't in scope for this ticket, but it certainly is not ready to be run
 automatically yet :) (at least on systems similar to mine). (testing now:
 suppressing the errors also works, but it is the test itself that is
 failing).

 I did get confused about what the two configure options meant, reading
 interpretation error on my part though. One thing; make check did not stop
 the moment it found a suppression with --enable-valgrind-suppressions
 (which actually did help in blindly adding them all to .revisit). It
 error'd at the end, not at the first non-suppressed message.

 But in itself I think this is a good step towards getting these things
 fixed, so I think we can go ahead and merge this.

-- 
Ticket URL: <http://bind10.isc.org/ticket/2036#comment:8>
BIND 10 Development <http://bind10.isc.org>
BIND 10 Development