Hi Nikolay, can we add option enables AES 128 for test mode? This should work well without update env.
Sincerely, ср, 27 июн. 2018 г. в 18:43, Petr Ivanov <mr.wei...@gmail.com>: > > > > On 27 Jun 2018, at 18:06, Nikolay Izhikov <nizhi...@apache.org> wrote: > > > > Hello, Eduard. > > > >> We should make suites which are restricted by agents to make as small > as> possible. > > > > Agree. That means we should enable java cryptography on all agents. > Isn't it? > > > >> So, I strictly against adding such test (therefore restriction) to > these> suites.> > > > > What is wrong with tests? > > Tests in current configuration require specific environment. > I guess we should update environment to meet test requirements prior > putting changes to master. > > > > Do you looked at it or something? > > > > > > В Ср, 27/06/2018 в 14:24 +0300, Eduard Shangareev пишет: > >> Nikolay, Petr, > >> > >> We should make suites which are restricted by agents to make as small as > >> possible. > >> > >> So, I strictly against adding such test (therefore restriction) to these > >> suites. > >> > >> On Wed, Jun 27, 2018 at 11:39 AM, Petr Ivanov <mr.wei...@gmail.com> > wrote: > >> > >>> IgniteSpiTestSuite is called from ’SPI’ build type and I’ve added agent > >>> filter there. > >>> IgniteKernalSelfTestSuite is not found in build types. > >>> > >>> > >>> > >>>> On 27 Jun 2018, at 11:26, Nikolay Izhikov <nizhi...@apache.org> > wrote: > >>>> > >>>> Well, it's a good question :) > >>>> > >>>> I've added encryption tests in `IgniteKernalSelfTestSuite` and in > >>> > >>> `IgniteSpiTestSuite`. > >>>> Not sure, what builds on Team City runs these suites. > >>>> > >>>> В Ср, 27/06/2018 в 11:18 +0300, Petr Ivanov пишет: > >>>>> Nikolay, > >>>>> > >>>>> > >>>>> What is “core” module? > >>>>> > >>>>> > >>>>>> On 27 Jun 2018, at 00:42, Nikolay Izhikov <nizhi...@apache.org> > wrote: > >>>>>> > >>>>>> Hello, Petr. > >>>>>> > >>>>>> Thanks a lot! > >>>>>> I see success spring encryption tests now [1] > >>>>>> > >>>>>> I've added encryption tests to core module also. > >>>>>> Can we set same rule for a core module? > >>>>>> > >>>>>> [1] https://ci.ignite.apache.org/project.html?projectId= > >>> > >>> IgniteTests24Java8&testNameId=-2290193573767653560&tab=testDetails > >>>>>> > >>>>>> В Вт, 26/06/2018 в 23:38 +0300, Petr Ivanov пишет: > >>>>>>> I’ve added JCE as it is described in documentation, but for some > >>> > >>> unknown reason it won’t work on 1.8.0_u151. > >>>>>>> So I’ve fix agent filter rule to run Spring build type only on > >>> > >>> 1.8.0_u171 (49 agents) and run it on all enabled compatible agents. > >>>>>>> Looks like it fixed the issue. > >>>>>>> > >>>>>>> > >>>>>>> > >>>>>>>> On 26 Jun 2018, at 19:27, Nikolay Izhikov <nizhi...@apache.org> > >>> > >>> wrote: > >>>>>>>> > >>>>>>>> Petr, can you make suggested jdk upgrade? > >>>>>>>> > >>>>>>>> В Пн, 25/06/2018 в 14:28 +0300, Nikolay Izhikov пишет: > >>>>>>>>> Petr. > >>>>>>>>> > >>>>>>>>> One more thing: > >>>>>>>>> > >>>>>>>>> We need to install "Java Cryptography Extension (JCE) Unlimited > >>> > >>> Strength Jurisdiction Policy Files" [1] to enable usage of JDK provided > >>> crypto algorithmes. > >>>>>>>>> Otherwise it fails with "java.security.InvalidKeyException: > >>> > >>> Illegal key size at > javax.crypto.Cipher.checkCryptoPerm(Cipher.java:1039)" > >>> [2] > >>>>>>>>> > >>>>>>>>> Can you do it for Team City agents? > >>>>>>>>> > >>>>>>>>> [1] http://www.oracle.com/technetwork/java/javase/ > >>> > >>> downloads/jce8-download-2133166.html > >>>>>>>>> [2] https://ci.ignite.apache.org/viewLog.html?tab=buildLog&; > >>> > >>> logTab=tree&filter=debug&expand=all&buildId=1421337&_focus=30899 > >>>>>>>>> > >>>>>>>>> В Пн, 25/06/2018 в 13:24 +0300, Petr Ivanov пишет: > >>>>>>>>>> I’ve modified filter requirements for JDK for Spring build > >>> > >>> configuration to run only on u151/u171 of JDK-1.8.0 (it will be 55 > agent). > >>>>>>>>>> Hope this will be enough. > >>>>>>>>>> > >>>>>>>>>> > >>>>>>>>>> > >>>>>>>>>>> On 25 Jun 2018, at 12:54, Nikolay Izhikov <nizhi...@apache.org > > > >>> > >>> wrote: > >>>>>>>>>>> > >>>>>>>>>>> Hello, Petr. > >>>>>>>>>>> > >>>>>>>>>>>> I can temporarily assign special rule for agent JDK and fix > >>> > >>> theirs Java later. > >>>>>>>>>>> > >>>>>>>>>>> I'm not Team City guru, so I'm don't understand your proposal > :) > >>>>>>>>>>> > >>>>>>>>>>> All I need is make sure Core and Spring suites for my branch > [1] > >>> > >>> are executed with latest jdk. > >>>>>>>>>>> > >>>>>>>>>>> Can you make it for me? > >>>>>>>>>>> > >>>>>>>>>>> [1] https://github.com/apache/ignite/pull/4167 > >>>>>>>>>>> > >>>>>>>>>>> > >>>>>>>>>>> В Пн, 25/06/2018 в 11:21 +0300, Petr Ivanov пишет: > >>>>>>>>>>>> Nokolay, > >>>>>>>>>>>> > >>>>>>>>>>>> > >>>>>>>>>>>> Agents prior publicagent10 has u77 inded, while > publicagent10-17 > >>> > >>> has already u171. > >>>>>>>>>>>> I can temporarily assign special rule for agent JDK and fix > >>> > >>> theirs Java later. > >>>>>>>>>>>> > >>>>>>>>>>>> > >>>>>>>>>>>> > >>>>>>>>>>>>> On 25 Jun 2018, at 10:33, Nikolay Izhikov < > nizhi...@apache.org> > >>> > >>> wrote: > >>>>>>>>>>>>> > >>>>>>>>>>>>> Hello, Igniters. > >>>>>>>>>>>>> > >>>>>>>>>>>>> I'm trying to run tests for TDE branch. > >>>>>>>>>>>>> > >>>>>>>>>>>>> It seems I'm facing with known JDK bug [1] > >>>>>>>>>>>>> Here is log from Team City [2]. > >>>>>>>>>>>>> > >>>>>>>>>>>>> As I can see current Team City jdk is 1.8.0_77-b03 > >>>>>>>>>>>>> Issue JDK-8149411 fixed in 8u102. > >>>>>>>>>>>>> > >>>>>>>>>>>>> Can we upgrade JDK on Team City? > >>>>>>>>>>>>> > >>>>>>>>>>>>> [1] https://bugs.openjdk.java.net/browse/JDK-8149411 > >>>>>>>>>>>>> [2] https://ci.ignite.apache.org/viewLog.html?tab=buildLog&; > >>> > >>> logTab=tree&filter=debug&expand=all&buildId=1420504&_focus=28867 > >>>>> > >>> > >