Github user bhaisaab commented on the pull request:
https://github.com/apache/cloudstack/pull/10#issuecomment-66041100
Hi Tim,
IMO, the fixes are in main methods of those classes, so do they even affect
CloudStack mgmt server, I'm not sure?
Both the PRs cannot be accepted the keyIvPair variable was deleted, the one
defined in the class is non-static so cannot be used in the main method. I
suggest that you close both of them and open a new one that fixes the classes
the both the locations in a single PR on master. Since the fix is small, put
both of them in a single patch so it's easier for me to cherry-pick them to
4.5, 4.4 and 4.3 branches.
Cheers.
---
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.
---
