[GitHub] cloudstack pull request: IGNORE: Testing that travis-ci runs on pu...

2014-09-01 Thread imduffy15
GitHub user imduffy15 opened a pull request: https://github.com/apache/cloudstack/pull/13 IGNORE: Testing that travis-ci runs on pull requests and reports a status. Remove a failing test form .travis.yml. This commit should be ignored. I am doing this as a pull request to

[GitHub] cloudstack pull request: IGNORE: Testing that travis-ci runs on pu...

2014-09-02 Thread imduffy15
Github user imduffy15 closed the pull request at: https://github.com/apache/cloudstack/pull/13 --- 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

[GitHub] cloudstack pull request: IGNORE: Testing that travis-ci runs on pu...

2014-09-02 Thread imduffy15
Github user imduffy15 commented on the pull request: https://github.com/apache/cloudstack/pull/13#issuecomment-54129007 Status successfully shown. Test completed. Closing. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well

[GitHub] cloudstack pull request: Fix CID-1222222

2014-09-16 Thread imduffy15
GitHub user imduffy15 opened a pull request: https://github.com/apache/cloudstack/pull/21 Fix CID-122 Fixes 122 issue. `xen.check.hvm` param was not being used. You can merge this pull request into a Git repository by running: $ git pull https://github.com

[GitHub] cloudstack pull request: Move Devcloud4 into main repository

2015-04-10 Thread imduffy15
Github user imduffy15 commented on the pull request: https://github.com/apache/cloudstack/pull/154#issuecomment-91738656 Does this look OK to people? :) --- 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

[GitHub] cloudstack pull request: Move Devcloud4 into main repository

2015-04-09 Thread imduffy15
GitHub user imduffy15 opened a pull request: https://github.com/apache/cloudstack/pull/154 Move Devcloud4 into main repository Hi All, As per the discussion on the mailing list I want to bring devcloud4 into a community place. This PR exists to do that. I have

[GitHub] cloudstack pull request: Move Devcloud4 into main repository

2015-04-09 Thread imduffy15
Github user imduffy15 commented on the pull request: https://github.com/apache/cloudstack/pull/154#issuecomment-91314767 Will do once I get back on a machine On 9 Apr 2015 18:56, "runseb" wrote: > @imduffy15 <https://github.com/imduffy15> can you s

[GitHub] cloudstack pull request: Move Devcloud4 into main repository

2015-04-11 Thread imduffy15
Github user imduffy15 closed the pull request at: https://github.com/apache/cloudstack/pull/154 --- 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

[GitHub] cloudstack pull request: modified travis.yml added sudo: required

2015-04-17 Thread imduffy15
Github user imduffy15 commented on the pull request: https://github.com/apache/cloudstack/pull/177#issuecomment-93969318 Looks good to me @karuturi --- 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

[GitHub] cloudstack pull request: Use the new build env on Travis

2014-11-18 Thread imduffy15
Github user imduffy15 commented on the pull request: https://github.com/apache/cloudstack/pull/43#issuecomment-63457295 Hi @joshk this is going to cause issues as we use sudo to install packages within the before_install script. Is it possible to have sudo access and the

[GitHub] cloudstack pull request: Use the new build env on Travis

2014-12-01 Thread imduffy15
Github user imduffy15 commented on the pull request: https://github.com/apache/cloudstack/pull/43#issuecomment-65054605 No. It disables sudo usage so it breaks our setup. On 1 Dec 2014 11:48, "Rohit Yadav" wrote: > Ping, should we do anything on this?

[GitHub] cloudstack pull request: NGUI: Added more functionalities, final p...

2014-12-22 Thread imduffy15
Github user imduffy15 commented on the pull request: https://github.com/apache/cloudstack/pull/62#issuecomment-67850765 I'm out cramming for exams like crazy over the christmas. Would probably start this from scratch to be honest. I dislike the flask part of it, angu

[GitHub] cloudstack-cloudmonkey pull request: Build a CI environment in ord...

2015-01-10 Thread imduffy15
GitHub user imduffy15 opened a pull request: https://github.com/apache/cloudstack-cloudmonkey/pull/2 Build a CI environment in order to allow automated testing Hi All, I'm creating this pull request to get some assistance with finishing an idea. I thought it

[GitHub] cloudstack-cloudmonkey pull request: Build a CI environment in ord...

2015-01-10 Thread imduffy15
Github user imduffy15 commented on the pull request: https://github.com/apache/cloudstack-cloudmonkey/pull/2#issuecomment-69450354 For anybody interested in seeing the current travis run: https://travis-ci.org/imduffy15/cloudstack-cloudmonkey/builds/46537749

[GitHub] cloudstack-cloudmonkey pull request: Build a CI environment in ord...

2015-01-12 Thread imduffy15
Github user imduffy15 commented on the pull request: https://github.com/apache/cloudstack-cloudmonkey/pull/2#issuecomment-69542479 > To test CloudMonkey we can mock a fake mgmt server and use a standard set of queries to check cloudmonkey's output as it receives known jso

[GitHub] cloudstack-cloudmonkey pull request: Build a CI environment in ord...

2016-03-28 Thread imduffy15
Github user imduffy15 closed the pull request at: https://github.com/apache/cloudstack-cloudmonkey/pull/2 --- 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

[GitHub] cloudstack pull request: Set default networkDomain to empty instea...

2016-05-02 Thread imduffy15
Github user imduffy15 commented on the pull request: https://github.com/apache/cloudstack/pull/1485#issuecomment-216353475 I'm not familiar with that block of code, remi's changes and justification seem fine though. Some very oddly/badly worded variable names. -