Skip to content
This repository has been archived by the owner on Jun 7, 2023. It is now read-only.

SERVICES-1164 add prod tag to phalanx address #9706

Closed
wants to merge 1 commit into from

Conversation

sqreek
Copy link
Contributor

@sqreek sqreek commented Feb 12, 2016

https://wikia-inc.atlassian.net/browse/SERVICES-1164

please merge this after email about configuration change
ping @Wikia/services-team

@sqreek
Copy link
Contributor Author

sqreek commented Feb 12, 2016

tests are not passing cause config change is not merged.

@ArturKlajnerok
Copy link
Contributor

I don't think it's good idea to have this hardcoded in app, we should use the hostname & port from : LocalSettings see my comment on https://github.com/Wikia/config/pull/1615

@sqreek
Copy link
Contributor Author

sqreek commented Feb 12, 2016

this code is for production. So default is always prod. Actually we don't need this change if we assume that every phalanx on production is production phalanx.

@harnash
Copy link
Contributor

harnash commented Feb 15, 2016

One more thing:

harnas@dev-harnas:~$ host phalanx.service.consul
phalanx.service.consul has address 10.14.30.159
phalanx.service.consul has address 10.14.30.175

harnas@dev-harnas:~$ host prod.phalanx.service.consul
Host prod.phalanx.service.consul not found: 3(NXDOMAIN)

harnas@dev-harnas:~$ host prod.phalanx.service.sjc.consul
prod.phalanx.service.sjc.consul has address 10.8.68.82
prod.phalanx.service.sjc.consul has address 10.8.62.73
prod.phalanx.service.sjc.consul has address 10.8.62.77

@harnash
Copy link
Contributor

harnash commented Feb 15, 2016

So above comment is obviously wrong. It tries to find services with tag prod on my dev-box which is in dev data center.

@ArturKlajnerok
Copy link
Contributor

So can we just remove wgPhalanxServiceUrl here as we will get the default from https://github.com/Wikia/config ? Sorry, for late response.

@sqreek
Copy link
Contributor Author

sqreek commented Feb 18, 2016

ok , we can go to pure config setup.

@sqreek sqreek closed this Feb 18, 2016
@sqreek sqreek deleted the SERVICES-1164_add_tag_to_phalanx_address branch February 23, 2016 18:50
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants