dev@glassfish.java.net

Re: QL failures

From: Lloyd Chambers <Lloyd.Chambers_at_Sun.COM>
Date: Fri, 04 Sep 2009 14:31:57 -0700

I have changed the failing test name to:

testAMXComplianceMonitorFailureCount___KNOWN_FAILURE_20090904_ISSUE_9355_IGNORE_FOR_NOW
()

Seems clear enuf to me.

      [echo] =============Starting TestNG test at ..//classes/test
from testng.xml ============
     [mkdir] Created dir: /Volumes/Scratch/work/v3/code/tests/
quicklook/test-output
    [testng] [Parser] Running:
    [testng] /Volumes/Scratch/work/v3/code/tests/quicklook/amx/
testng.xml
    [testng]
    [testng] CHANGE COUNT: 10
    [testng] PASSED: bootAMX
    [testng] Core AMX tests
    [testng] PASSED: iterateAllSanityCheck
    [testng] PASSED: testAmxPref
    [testng] PASSED: connectorConnectionPoolTest
    [testng] PASSED: createChildTest
    [testng] PASSED: testConfigTools
    [testng] PASSED: createProfilerTest
    [testng] PASSED: testAMXConfigAttributeResolver
    [testng] PASSED: testAMXConfigDefaultValues
    [testng] PASSED: testForBogusConfigAnnotations
    [testng] PASSED: testCreateProperties
    [testng] PASSED: testCreateResource
    [testng] PASSED: testCreateProperty
    [testng] PASSED:
testAMXComplianceMonitorFailureCount___KNOWN_FAILURE_20090904_ISSUE_9355_IGNORE_FOR_NOW
    [testng] PASSED: testConnectorRuntimeAPIProvider
    [testng] PASSED: testMonitoringRoot
    [testng] PASSED: testDomainConfig
    [testng] PASSED: testBulkAccess
    [testng] PASSED: testExt
    [testng] PASSED: testPropertyParent
    [testng] PASSED: testTools
    [testng] PASSED: testPathnames
    [testng] PASSED: testSystemStatus
    [testng] PASSED: testApplications
    [testng] PASSED: testResources
    [testng] PASSED: testChildGetterVariants
    [testng] PASSED: testAllGenerically
    [testng] PASSED: testRuntimeRoot
    [testng] PASSED: testQuery
    [testng] PASSED: testJ2EEDomain
    [testng] PASSED: testSingletonOrNot
    [testng] PASSED: testDomainRoot
    [testng] PASSED: testSystemApplications
    [testng] PASSED: testSystemInfo
    [testng] PASSED: testSystemPropertyParent
    [testng] PASSED: testConfigs
    [testng] PASSED: testServerRuntime
    [testng] PASSED: testLogging
    [testng] PASSED: testServers
    [testng] PASSED: testForBogusAnnotations
    [testng] PASSED: testAutoConvert
    [testng]
    [testng] ===============================================
    [testng] amx_tests
    [testng] Tests run: 41, Failures: 0, Skips: 0
    [testng] ===============================================
    [testng]
    [testng]
    [testng] ===============================================
    [testng] QuickLookTests
    [testng] Total tests run: 41, Failures: 0, Skips: 0
    [testng] ===============================================
    [testng]



Lloyd

On Sep 4, 2009, at 2:25 PM, Lloyd Chambers wrote:

> The issue is that other changes could be made that cause other
> failures, which would then be masked by disabling the test.
>
> Since the code in question was just integrated and more changes are
> happening in that area, that puts us at further risk.
>
> It also puts me at risk if I make a change to fix other bugs.
>
> Others can continue with their chores.
>
> How about changing the test *name* to
> testAMXComplianceMonitorFailureCount_____THIS_IS_A_KNOWN_BUG_IGNORE_FOR_NOW
> ()
>
> ?
>
> Lloyd
>
> On Sep 4, 2009, at 2:30 PM, Kedar Mhaswade wrote:
>
>> I was suggesting a temporary disablement so others can continue with
>> their chores. It's lame too to expect a known failure to show up.
>> So, I vote for (temporarily) disabling it.
>>
>> Lloyd Chambers wrote:
>>> Abhijit is trending strongly towards disabling the test for this
>>> reason (confusion).
>>> They indicate real problem with the ORB MBeans, but not problems
>>> that affect the overall server operation.
>>> So what it comes down to is what our policy in removing tests that
>>> detect real problems? I think it's kinda lame to have tests, then
>>> turn them off when real problems crop up.
>>> My other suggestion was to have the test itself emit a message
>>> indicating that it is a known issue.
>>
>> But still, I won't know if the failure is because of the fact that
>> there is a known cause for it or because my changes are (also)
>> causing
>> it.
>>
>>> I have NOT seen amxtest.AMXCoreTests.iterateAllSanityCheck() fail,
>>> so that one needs looking into.
>>> Lloyd
>>> On Sep 4, 2009, at 1:54 PM, Kedar Mhaswade wrote:
>>>> I get the failure with:
>>>> amxtest.AMXCoreTests:testAMXComplianceMonitorFailureCount
>>>>
>>>> It's not clear if that's the same as the one reported in 9355.
>>>>
>>>> BTW, can we temporarily disable the test so that we get a clean
>>>> run every time? When the bug is fixed, we can re-enable it.
>>>>
>>>> Thanks.
>>>>
>>>> Jane Young wrote:
>>>>> I know #2 is a known issue. See IT 9355
>>>>> and
>>>>> http://gf-hudson.sfbay.sun.com/hudson/job/gf-trunk-build-continuous/2205/artifact/bundles/QL-GP-report.html
>>>>> I'm not sure about #1. Are you getting total of 2 failures or 1?
>>>>
>>>>> Richard S. Hall wrote:
>>>>>> Hello,
>>>>>>
>>>>>> I am trying to make sure that Felix 2.0 is ready for release.
>>>>>> My latest run comes up with the following QL failures:
>>>>>>
>>>>>> 1. iterateAllSanityCheck
>>>>>> java.lang.NullPointerException
>>>>>> at amxtest.AMXCoreTests.iterateAllSanityCheck
>>>>>> (AMXCoreTests.java:94)
>>>>>> 2. testAMXComplianceMonitorFailureCount
>>>>>> java.lang.AssertionError: Server indicates that there are
>>>>>> non-compliant AMX MBean validator failures, failure count = 20,
>>>>>> examine the server log for failures
>>>>>> at
>>>>>> amxtest.AMXCoreTests.testAMXComplianceMonitorFailureCount
>>>>>> (AMXCoreTests.java:112)
>>>>>>
>>>>>> Are these normal failures in GF trunk?
>>>>>>
>>>>>> -> richard
>>>>
>>>>
>>>> ---------------------------------------------------------------------
>>>> To unsubscribe, e-mail: dev-unsubscribe_at_glassfish.dev.java.net
>>>> For additional commands, e-mail: dev-help_at_glassfish.dev.java.net
>>>>
>>> Lloyd Chambers
>>> lloyd.chambers_at_sun.com
>>> GlassFish Team
>>> ---------------------------------------------------------------------
>>> To unsubscribe, e-mail: dev-unsubscribe_at_glassfish.dev.java.net
>>> For additional commands, e-mail: dev-help_at_glassfish.dev.java.net
>>
>>
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail: dev-unsubscribe_at_glassfish.dev.java.net
>> For additional commands, e-mail: dev-help_at_glassfish.dev.java.net
>>
>
> Lloyd Chambers
> lloyd.chambers_at_sun.com
> GlassFish Team
>
>
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe_at_glassfish.dev.java.net
> For additional commands, e-mail: dev-help_at_glassfish.dev.java.net
>

Lloyd Chambers
lloyd.chambers_at_sun.com
GlassFish Team