Github user prashanthvarma commented on the issue:

    https://github.com/apache/cloudstack/pull/1578
  
    @jburwell @rhtyd Ok, we will rebase the PR and squash our changes/commits, 
and let you know here. Moreover, we can re-run the added Marvin tests and share 
the results after the re-base.
    
    Generally, we have two commits in a PR, one for dev and one for test 
(Marvin). Is it fine to have this two commits per PR, and squash the other 
commits ?
    
    Our open PRs #1579 and #1580 have dependencies on this PR. Thus, we 
included this PR commits in those PRs.


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

Reply via email to