Uploaded image for project: 'Hadoop YARN'
  1. Hadoop YARN
  2. YARN-8840

Add missing cleanupSSLConfig() call for TestTimelineClient test

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Resolved
    • Minor
    • Resolution: Fixed
    • None
    • 2.10.0, 3.2.0
    • test, timelineclient
    • None
    • Reviewed

    Description

      Tests that setup SSLConfigs can leave conf-files lingering unless they are cleaned up via KeyStoreTestUtil.cleanupSSLConfig call. TestTimelineClient test is missing this call.

      If the cleanup method is not called explicitly, a modified ssl-client.xml is left in test-classes, might affect subsequent test cases.

       

      There was a similar report in HDFS-11042, but looks that we need to fix TestTimelineClient test too.

       

      $ mvn test -Dtest=TestTimelineClient
      $ find .|grep ssl-client.xml$
      ./hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/target/test-classes/ssl-client.xml
      $ cat ./hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/target/test-classes/ssl-client.xml
      <?xml version="1.0" encoding="UTF-8" standalone="no"?><configuration>
      <property><name>ssl.client.truststore.reload.interval</name><value>1000</value><final>false</final><source>programmatically</source></property>
      <property><name>ssl.client.truststore.location</name><value>/Users/tanaka/work/hadoop/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/target/test-dir/trustKS.jks</value><final>false</final><source>programmatically</source></property>
      <property><name>ssl.client.keystore.keypassword</name><value>clientP</value><final>false</final><source>programmatically</source></property>
      <property><name>ssl.client.keystore.location</name><value>/Users/tanaka/work/hadoop/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/target/test-dir/clientKS.jks</value><final>false</final><source>programmatically</source></property>
      <property><name>ssl.client.truststore.password</name><value>trustP</value><final>false</final><source>programmatically</source></property>
      <property><name>ssl.client.keystore.password</name><value>clientP</value><final>false</final><source>programmatically</source></property>
      

       

      After applying this patch, the ssl-client.xml is not generated.

      Attachments

        1. YARN-8840.003.patch
          3 kB
          Aki Tanaka
        2. YARN-8840.002.patch
          2 kB
          Aki Tanaka
        3. YARN-8840.001.patch
          0.9 kB
          Aki Tanaka

        Activity

          People

            tanakahda Aki Tanaka
            tanakahda Aki Tanaka
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: