Uploaded image for project: 'Yetus'
  1. Yetus
  2. YETUS-693

Add warning about requirements for XML test-patch test

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Resolved
    • Minor
    • Resolution: Fixed
    • 0.6.0, 0.7.0, 0.8.0
    • None
    • Precommit

    Description

      When running Test-Patch with some JVM's (currently seen in Azul JDK7 over in HBase) the "xml" test always votes -1

      ...

15:51:42 ============================================================================
15:51:42 ============================================================================
15:51:42                           XML verification: patch
15:51:42 ============================================================================
15:51:42 ============================================================================
15:51:42 
15:51:42 
15:51:42 
15:51:42 
15:51:42 ============================================================================
...
17:24:45 |  -1  |           xml  |   0m  0s   | The patch has 1 ill-formed XML file(s). 
...
17:24:45               Reason | Tests
17:24:45                 XML  |  Parsing Error(s): 
17:24:45                      |  pom.xml 
...

      The actual output indicates the problem is the JDK we have doesn't support javascript scripting:

      pom.xml:

script engine for language js can not be found

      Maybe we could do a quick known-pass and known-fail check in setup for the plugin?

      Attachments

        Issue Links

          Is contained by

          Improvement - An improvement or enhancement to an existing feature or task. YETUS-1012 Major overhaul of precommit documentation

          • Major - Major loss of function.
          • Resolved
          relates to

          Bug - A problem which impairs or prevents the functions of the product. HBASE-22732 XML verification fails in precommit

          • Minor - Minor loss of function, or other problem where easy workaround is present.
          • Open

          Improvement - An improvement or enhancement to an existing feature or task. YETUS-670 xml test should try xmllint --valid

          • Minor - Minor loss of function, or other problem where easy workaround is present.
          • Resolved

          Activity

            People

              Unassigned Unassigned
              busbey Sean Busbey
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: