airavata-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Gagan Deep Juneja (JIRA)" <j...@apache.org>
Subject [jira] [Updated] (AIRAVATA-1283) local host is hard coded in all the thrift clients
Date Sat, 07 Jun 2014 09:55:01 GMT

     [ https://issues.apache.org/jira/browse/AIRAVATA-1283?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

Gagan Deep Juneja updated AIRAVATA-1283:
----------------------------------------

    Attachment: 0001-AIRAVATA-1283-Added-support-for-hostname-in-all-serv.patch

Hi Team,
I have made code changes to fix this issue. I also moved some code here and there for refactoring
purpose. I am new and not sure what process is being followed in Airavata Project for patch
review. So I am attaching the patch here for review.
I did not test this because I dont have ready setup with me. Request you to test for sanity
scenarios once although tests are passing in build.

Please do let me know your feedback.

Regards,
Gagan

> local host is hard coded in all the thrift clients 
> ---------------------------------------------------
>
>                 Key: AIRAVATA-1283
>                 URL: https://issues.apache.org/jira/browse/AIRAVATA-1283
>             Project: Airavata
>          Issue Type: Bug
>          Components: Airavata Client, Airavata Orchestrator
>    Affects Versions: 0.12
>         Environment: mac
>            Reporter: Lahiru Gunathilake
>             Fix For: 0.12
>
>         Attachments: 0001-AIRAVATA-1283-Added-support-for-hostname-in-all-serv.patch
>
>
> In all the places each thrift client is hard coded with host and some places with port
too. In this case even though these are different thirft services these cannot be hosted in
two different machines.
> I think this is a bad practice and unnecessary limitation. Ideally we should read the
ip from ip property and ports should be configurable.
> any thrift service can be removed from airavata by removing server list in airavata-server.properties.



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Mime
View raw message