Hi Will, Could you check on Palo Alto's duplicate api commands? They suppose to be merged as one command I think.
BTW, how can this works? Did it broke SRX? --Sheng On Thu, Nov 14, 2013 at 2:55 AM, Santhosh Edukulla < santhosh.eduku...@citrix.com> wrote: > Team, > > While running code coverage analysis with sonar for integration tests, > based upon the errors thrown, i could see the below issues\notes with CS > project. > > Issue1: > > The coverage tool is complaining about duplicate sources for below files. > These are available with same name under folders > ./cloudstack/plugins/network-elements/juniper-srx and as well under > ..../palo-alto. > > > ./cloudstack/plugins/network-elements/palo-alto/src/com/cloud/api/commands/AddExternalFirewallCmd.java > > ./cloudstack/plugins/network-elements/palo-alto/src/com/cloud/api/commands/DeleteExternalFirewallCmd.java > > ./cloudstack/plugins/network-elements/palo-alto/src/com/cloud/api/commands/ListExternalFirewallCmd.java > > I renamed one while running analysis to a different name. It proceeded > further with its analysis once renamed. > > Is it intentional to have same name or can be renamed? > > Issue2: > > The source directory does not correspond to package declaration for code > files under > > /root/softwares/cscode/cloudstack/services/console-proxy-rdp/rdpconsole/src/main/java/rdpclient/ > > This error when compared to other files in the similar path has a > different package convention and usage. > > /root/softwares/cscode/cloudstack/services/secondary-storage/src/org/apache/cloudstack/storage/ > > Changing tool configuration properties worked to over come this, but Is it > intentional to have a different package structure for rdpclient code base > against others? > > > > Thanks! > Santhosh >
