Github user rafaelweingartner commented on the pull request:
https://github.com/apache/cloudstack/pull/1366#issuecomment-206425673
Do you mean that the need for 2LGTMs was voted after this PR was open? It
seems that it was a direct commit. Why did you not use the merge tool?
Is it just a guideline? I have seen much less complicated PRs being
reverted because they did not have those 2LGTMS.
Could you refer the outstanding PR here too, for tracking purpose?
---
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.
---
