[GitHub] airavata pull request: Fixed gfac instance list init issue when ca...

2014-09-05 Thread shamrath
GitHub user shamrath opened a pull request: https://github.com/apache/airavata/pull/4 Fixed gfac instance list init issue when call Orchestrator with initial Gfac instants list. You can merge this pull request into a Git repository by running: $ git pull https://github.com

[GitHub] airavata-php-gateway pull request: Sync new php client model class...

2015-07-20 Thread shamrath
GitHub user shamrath opened a pull request: https://github.com/apache/airavata-php-gateway/pull/7 Sync new php client model classes and changed ExperiementUtilities.php according to that Sync new php client mode classes and changed ExperimentUtilities.php You can merge this pull

[GitHub] airavata-php-gateway pull request: Fixed local experiment create a...

2015-07-20 Thread shamrath
GitHub user shamrath opened a pull request: https://github.com/apache/airavata-php-gateway/pull/8 Fixed local experiment create and launch issues with Airavata master With this fixes, pga can be used to save and launch experiment. with Airavata master code, Airavata master is under

[GitHub] airavata-php-gateway pull request: Fixed local experiment create a...

2015-07-20 Thread shamrath
GitHub user shamrath opened a pull request: https://github.com/apache/airavata-php-gateway/pull/9 Fixed local experiment create and launch issues with Airavata master With this fixes, pga can be used to save and launch experiment. with Airavata master code, Airavata master is under

[GitHub] airavata issue #41: Made design changes

2016-08-12 Thread shamrath
Github user shamrath commented on the issue: https://github.com/apache/airavata/pull/41 Which build tool are you using to build your code? --- 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

[GitHub] airavata issue #41: Made design changes

2016-08-12 Thread shamrath
Github user shamrath commented on the issue: https://github.com/apache/airavata/pull/41 could you explain the use of modules/cloud/AuroraMarathonIntegration/src/README.md ? and also 0.16 is released and you should change your cloud module version to 0.17-SNAPSHOT --- If your

[GitHub] airavata issue #41: Made design changes

2016-08-12 Thread shamrath
Github user shamrath commented on the issue: https://github.com/apache/airavata/pull/41 Change airavata-cloud parent module version too in your pom.xml, I am just here try to make travis-ci works fine with your code. --- If your project is set up for it, you can reply to this

[GitHub] airavata issue #41: Made design changes

2016-08-12 Thread shamrath
Github user shamrath commented on the issue: https://github.com/apache/airavata/pull/41 let me change cloud module version for you. --- 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

[GitHub] airavata issue #41: Made design changes

2016-08-12 Thread shamrath
Github user shamrath commented on the issue: https://github.com/apache/airavata/pull/41 get a git pull now and it will have 0.17-SNAPSHOT. --- 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

[GitHub] airavata issue #41: Made design changes

2016-08-12 Thread shamrath
Github user shamrath commented on the issue: https://github.com/apache/airavata/pull/41 Your local changes preserve with git pull, sometimes you will get conflicts with remote code changes, which you need to fix and commit again. Anyway as a best practice and avoid a lot of merge

[GitHub] airavata issue #41: Made design changes

2016-08-12 Thread shamrath
Github user shamrath commented on the issue: https://github.com/apache/airavata/pull/41 could you also run git merge airavata:develop and make sure there are no any conflicts? BTW hope your git branch code is sync with airavata git develop branch. --- If your project is set up for

[GitHub] airavata issue #54: Adding SSH Credential summary data model

2016-10-10 Thread shamrath
Github user shamrath commented on the issue: https://github.com/apache/airavata/pull/54 Hi Anuj Followings are few suggestions to improve your pull request. 1. We can do this in a better way by adding generic API method(to AiravataAPIServer) like

[GitHub] airavata pull request #52: Local provider Fix

2016-10-12 Thread shamrath
Github user shamrath commented on a diff in the pull request: https://github.com/apache/airavata/pull/52#discussion_r83044484 --- Diff: modules/orchestrator/orchestrator-core/src/main/java/org/apache/airavata/orchestrator/cpi/impl/SimpleOrchestratorImpl.java --- @@ -406,11

[GitHub] airavata pull request #52: Local provider Fix

2016-10-12 Thread shamrath
Github user shamrath commented on a diff in the pull request: https://github.com/apache/airavata/pull/52#discussion_r83044405 --- Diff: modules/orchestrator/orchestrator-core/src/main/java/org/apache/airavata/orchestrator/cpi/impl/SimpleOrchestratorImpl.java --- @@ -406,11

[GitHub] airavata issue #54: Adding SSH Credential summary data model

2016-10-12 Thread shamrath
Github user shamrath commented on the issue: https://github.com/apache/airavata/pull/54 @anujbhan , Ok let's make this an exception. Please create follow up JIRA to track the pending improvements. I will review and merge this. --- If your project is set up for it, you can