RFR(S): 8162727 - Testbug: additional requires properties can't be used for filtering server vm in jdk jtreg tests

Dmitrij Pochepko dmitrij.pochepko at oracle.com
Thu Aug 4 10:56:23 UTC 2016



Hi,

I'm aware of one test only so far.
Shoud I move this single test or a whole testgroup which this test 
belongs to?

Thanks,
Dmitrij
> Dima,
>
> I’d prefer to have tests moved into hotspot repo instead. none of jdk tests should depend on vm kind or any other details of vm. your change just makes things worse, we will not just have tests which depend on vm implementation details in jdk, we will also have an mechanism for other tests to express such dependency, which can lead to increased number of “hotspot” tests in jdk. we went similar path w/ testlibrary and we are still paying for that.
>
> so how many tests which need these properties do we have now?
>
> Thanks,
> — Igor
>
>> On Aug 3, 2016, at 4:25 PM, Dmitrij Pochepko <dmitrij.pochepko at oracle.com> wrote:
>>
>> Hi,
>>
>> can somebody with official reviewer role review this?
>>
>> Thanks,
>> Dmitrij
>>
>> On 29.07.2016 21:42, Dmitrij Pochepko wrote:
>>> Sure,
>>>
>>> now including core-libs-dev
>>>
>>> Thanks,
>>> Dmitrij
>>>
>>> On 29.07.2016 20:16, joe darcy wrote:
>>>> Hello,
>>>>
>>>> I also recommend sending this review request over to core-libs-dev since it affect the jdk repo.
>>>>
>>>> Thanks,
>>>>
>>>> -Joe
>>>>
>>>>
>>>> On 7/29/2016 12:35 AM, Dmitry Fazunenko wrote:
>>>>> Hi Dmitrij,
>>>>>
>>>>> The change itself looks good.
>>>>>
>>>>> One note: this change adds a little overhead (a very little) for running tests in jdk repository, but it's required only for VM specific tests stored in jdk. As alternative of this change I can suggest moving VM specific tests from the 'jdk' to 'hotspot' repository. Perhaps, such massive update is too late for JDK9 time frame and could be done only in JDK10.
>>>>> So, if the changes depending on 8162727 are not so critical and could be deferred I would prefer to postpone this fix.
>>>>>
>>>>> Thanks,
>>>>> Dima
>>>>>
>>>>>
>>>>> On 29.07.2016 4:29, Vladimir Kozlov wrote:
>>>>>> It affects all groups. Should be reviewed on hotspot-dev.
>>>>>>
>>>>>> Thanks,
>>>>>> Vladimir
>>>>>>
>>>>>> On 7/28/16 10:19 AM, Dmitrij Pochepko wrote:
>>>>>>> Hi,
>>>>>>>
>>>>>>> please review small patch for 8162727 - Testbug: additional requires properties can't be used for filtering server vm in jdk jtreg tests
>>>>>>>
>>>>>>> A problem is that tests under jdk repo can't use additional jtreg requires properties(like vm.flavor to filter out client or server vm). The same functionality was introduced for hotspot repo as part
>>>>>>> of JDK-8154956. So, in order to filter tests using these additional properties a small fix is created.
>>>>>>>
>>>>>>> webrev: http://cr.openjdk.java.net/~dpochepk/8162727/webrev.01/
>>>>>>> CR: https://bugs.openjdk.java.net/browse/JDK-8162727
>>>>>>>
>>>>>>> Thanks,
>>>>>>> Dmitrij



More information about the core-libs-dev mailing list