cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From terbolous <...@git.apache.org>
Subject [GitHub] cloudstack pull request: Embedded Tomcat & Jetty
Date Tue, 09 Jun 2015 11:10:59 GMT
Github user terbolous commented on the pull request:

    https://github.com/apache/cloudstack/pull/372#issuecomment-110320351
  
    Thanks @rsafonseca, your efforts are well appreciated, don't think otherwise.
    
    Small changes equals, atleast usually, a smaller set of testing. 
    The CloudStack community consists of a lot of different people (and their companies),
who all have different needs, agendas, and time available. 
    
    So, while some might have the time and willingness to test packaging, they might not have
time, the experience or willingness to test the other changes.
    For instance, I have no idea how to test the embedded jetty/tomcat changes.
    
    The bigger a change is, the harder it is to track why it was done, and it's implications,
especially when it's not related to a Jira issue/FS or similar.
    
    This is not to say that your PR cannot be merged, but waiting for someone to either have
tested all the changes, or for enough people to have tested each subset could take time.


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

Mime
View raw message