-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1
On 04/22/2015 02:13 PM, Wilder Rodrigues wrote: > Hi Wido, > > Thanks for the reply and making a good point concerning Ubuntu > 14.04. > > Besides the difficulty in writing testes without increasing even > more out technical dept, another point on the Java 8 platform is > the EOL (end of this month) of Java 1.7. > Yes, I'm aware of that. So that makes this situation difficult. There is a bug open for backporting OpenJDK 8 to Ubuntu 14.04: https://bugs.launchpad.net/ubuntu/+source/openjdk-8/+bug/1341628 More votes there would really help. > For now I created a ticket on Apache Jira to keep track of it: > https://issues.apache.org/jira/browse/CLOUDSTACK-8397. Could you > please have a look and let me know if the content of the ticket is > appropriate? > > We will start a new sprint in 1 week and will take some time to > discuss what to do and when. Will keep the community updated on > that matter. > > Thanks a gain. > > Cheers, Wilder > > > On 21 Apr 2015, at 15:49, Wido den Hollander > <w...@widodh.nl<mailto:w...@widodh.nl>> wrote: > > > > On 04/21/2015 03:27 PM, Wilder Rodrigues wrote: Hi all, > > Yesterday I started working on the LibvirtComputingResource class > in order to apply the same patterns I used in the > CitrixResourceBase + add more unit tests to it After 10 hours of > work I got a bit stuck with the 1st test, which would cover the > refactored LibvirtStopCommandWrapper. Why did I get stuck? The > class used a few static methods that call native libraries, which > I would like to mock. However, when writing the tests I faced > problems with the current Mockito/PowerMock we are using: they are > simply not enough for the task. > > What did I do then? I added a dependency to EasyMock and > PowerMock-EasyMock API. It worked almost fine, but I had to add a > “-noverify” to both my Eclipse Runtime configuration and also to > the cloud-plugin-hypervisor-kvm/pom.xml file. I agree that’s not > nice, but was my first attempt of getting it to work. After trying > to first full build I faced more problems related to > ClassDefNotFoundExpcetion which were complaining about Mockito > classes. I then found out that adding the PowerMockRunner to all > the tests classes was going to be a heavy burden and would also > mess up future changes (e.g. the -noverify flag was removed from > Java 8, thus adding it now would be a problem soon). > > Now that the first 2 paragraphs explain a bit about the problem, > let’s get to the solution: Java 8 > > The VerifyError that I was getting was due to the use of the > latest EasyMock release (3.3.1). I tried to downgrade it to > 3.1/3.2 but it also did not work. My decision: do not refactor if > the proper tests cannot be added. This left me with one action: > migrate to Java 8. > > There were mentions about Java 8 in february[1] and now I will put > some energy in making it happen. > > What is your opinion on it? > > > I'm not against it technically, but practically I am. > > Ubuntu 14.04 does not ship a Java 8 JRE in the repositories. > > CentOS 7 has java-1.8.0-openjdk.x86_64 available, so it would work > there. But Ubuntu is also widely used with CloudStack, so those > users couldn't use CloudStack without any additional repositories. > > Since that isn't easy I would vote -1 on this if it came that far. > > Wido > > Thanks in advance. > > Cheers, Wilder > > http://mail-archives.apache.org/mod_mbox/cloudstack-dev/201502.mbox/%3 > > c54eef6be.5040...@shapeblue.com<mailto:c54eef6be.5040...@shapeblue.com>% 3E<http://mail-archives.apache.org/mod_m > box/cloudstack-dev/201502.mbox/<54eef6be.5040...@shapeblue.com<mailto: 54eef6be.5040...@shapeblue.com>>> > > > > -----BEGIN PGP SIGNATURE----- Version: GnuPG v1 iQIcBAEBAgAGBQJVN5dgAAoJEAGbWC3bPspCzX8P/0o8V8ZZPF+mOiXwfvZxOoqn Xtb084SajpLvB4KFT207FecJ6rKJyJiSXZSW6esj1F5OBzoDzF30vWHityvvatCA LsY5zCj2LF01itmjK8SVXuuwK8sSINycJPu2jJVotYr4ooPM1pHJjv/UnQfrUgp3 yv8vT3VKhrPLkGOIIcRR8zmIPH6qtgTf/ILBsc9hUrkvYgfmReH1dkeQY4gid3TZ sHGrnYHby2SgW+9KbuEfdvOrHvItYbJpRWz6W3R6l+DQeZxMt/pMbZRXEM4LXkuL /t450iwybpMZzwwnPYqu0TjTjSI0AFZp9gq+obygEnbDsbCXMuUKRHNymCdUZH3r hldfP1dmAwPEjJ0Z9PgybvmitaAqvUg80BeS7iS/6SPulGx6hFiafrdCgjdQ7bxM qN4nGcFIwxmzphhljnLARxRxl2/50KuZTYmC3XmyfbrUYX+BiDW8FoMbCd5qJJyC r48w2gPQW52HyVckM532PFpMWah7If8Q0Ee9w0JrTEF3RRkMQ7NySUvK0Y9er8ay D6fmw5szuWWNF7bSC6wNx5bY7EyNGp7rDa1Ki4f4G7chh6m16yhYSQ0fLBjro6fw sFbs7EKAa9iCsyHbPQBpz3IoqkSbCknnHbXAInQOyybLKendwZ2f9LVA8K0GFlHH uMsBvF/FH1bnVg+oX8N3 =oFqF -----END PGP SIGNATURE-----