cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From rsafonseca <>
Subject [GitHub] cloudstack pull request: Build and create tomcat, fix several othe...
Date Fri, 29 May 2015 08:18:10 GMT
Github user rsafonseca commented on the pull request:
    Hi @remibergsma 
    First of all. thank you for your comment :)
    I've been very busy these past couple of days, so i haven't gotten around to do what you
suggested, but i will take care of that over the weekend (squash commits, clean description
of fixes), I will also try to get a couple more stuff in as separate pull requests and clean
up a couple more things ;)
    As for testing this, there is no other way to test this other than manually since marvin
doesn't even use tomcat for testing, altough the travis setup scripts download an old version
of tomcat into the hard disk the tests do nothing with it :) That should also be cleaned up.
    Another handy thing that's really missing in the tests is a test to packaging! In the
past weeks a few things got into master that completely broke packaging.. although Rohit fixed
most of that already, centos6 packaging is still broken.
    As a side note, master also still doesn't work in Centos 7.1 and Fedora 21.. that's all
in here hehe ;)
    If you guys would like, i can also alter the test scripts to use the same tomcat that
will be used in the actual product instead of jetty, which will make for a test scenario closer
to reality. This can be done in a way which will easily allow switching between tomcat and
jetty in the future :)
    What do you think of that?

If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at or file a JIRA ticket
with INFRA.

View raw message