Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix the loadtest dependency on server IPs vs URIs #22

Closed
jbonacci opened this issue Oct 15, 2014 · 4 comments
Closed

Fix the loadtest dependency on server IPs vs URIs #22

jbonacci opened this issue Oct 15, 2014 · 4 comments
Assignees
Labels
3 Estimate - m - This is a small change, but there's some uncertainty.

Comments

@jbonacci
Copy link

Because I prefer this:
make test SERVER_URL=https://sync-1-us-east-1...#BLAH
over this
make test SERVER_URL=#BLAH

What was the requirement here for moving to hard-coded IPs vs. the URI?

@jbonacci jbonacci added the 3 Estimate - m - This is a small change, but there's some uncertainty. label Oct 15, 2014
@jbonacci
Copy link
Author

Ah, so I am using the write-heavy load test (as was Karl).
I see this is "Active":
https://github.com/mozilla-services/server-syncstorage/tree/rfk/write-heavy-loadtest

@rfk
Copy link
Contributor

rfk commented Oct 16, 2014

I don't get it, we didn't do anything in the code that requires you to use server IPs. We disabled ssl verification so you can use the raw IP, but testing by hostname should still work fine. Perhaps this is more a question for :bobm.

@jbonacci
Copy link
Author

I will see if I can catch Bob today. I have no idea why it works only this way now...

@jbonacci
Copy link
Author

Actually I am closing this and opening an OPs ticket.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3 Estimate - m - This is a small change, but there's some uncertainty.
Projects
None yet
Development

No branches or pull requests

2 participants