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

vanadium-browser-test-web is stuck on Initializing Vanadium #84

Closed
afandria opened this issue May 24, 2015 · 10 comments
Closed

vanadium-browser-test-web is stuck on Initializing Vanadium #84

afandria opened this issue May 24, 2015 · 10 comments
Assignees

Comments

@afandria
Copy link
Contributor

This test fails about 25% of the time on Jenkins. According to the screenshots, we get stuck on initializing Vanadium, so this may be a timing issue. It's possible that my test's timeout is too short.

For now, I have disabled the test. I plan to take a look on Tuesday.

@arup42 @aghassemi @wmleler

@afandria afandria self-assigned this May 24, 2015
afandria added a commit that referenced this issue May 26, 2015
The test has been somewhat flaky on Jenkins, and it usually times out at
the caveat tab point.

I think that increasing the timeout will help. If it doesn't solve the
flakiness, then we'll at least rule out 1 thing.

Related: #84
Change-Id: I5d50e7ba36d9fd010da61c4a52b8befa4e5edb91
@afandria
Copy link
Contributor Author

Re-enabled with a longer timeout. Has not failed for the past few runs, but will wait for a day to see if the issue has really been fixed.

@arup42
Copy link

arup42 commented May 27, 2015

We have a timeout in this test again. Is this different from the earlier timeout?

org.openqa.selenium.TimeoutException: Timed out after 20 seconds waiting for io.v.webdriver.nsbrowser.MainPage$1@3d894c74

@afandria
Copy link
Contributor Author

Same problem. The caveats tab is not appearing after the page load. (Note: This also affects the other web tests.)

20s (and even 10s, the previous timeout) are really a long time to wait for the tab to show up, and it doesn't fix the problem. I will have to try something else.

@arup42
Copy link

arup42 commented May 28, 2015

Any ETA on this one? (Can we do something to make the test green in the meantime? The last 6 runs have failed.)

@afandria
Copy link
Contributor Author

No ETA yet, I still have not identified the problem. Something in the extension is hanging, and while it may be something between wspr and js, it may also be that the identity server isn't responding (and the test timeouts before the identity server gets back to us.)

Even more oddly, the tutorial tests do approximately the same thing, and they have a much lower rate of failure.

For now, I've disabled the test.

@afandria
Copy link
Contributor Author

The current plan is to see if the identity server is at fault.

  • add log lines to prod identityd @asimshankar
  • add a console log of the public key in the JS code. (Is it possible to obtain this during vanadium.init?)
  • obtain the ChromeDriver console output in the webdriver test and ensure that it is available on the Jenkins console output.

Analysis of the ingoing/outgoing log lines and timestamps may implicate/absolve identityd.

@arup42
Copy link

arup42 commented May 29, 2015

@jakline

Note that Alex's theory is that disabling this test caused the problem to move to the vanadium-www-tutorials-js-web test. There's a separate top-level bug open for that one:

vanadium/issues#520

@afandria
Copy link
Contributor Author

A more short-term solution (which doesn't actually fix the underlying bug) would be for the test to try again if it encounters this problem. I will likely end up implementing this so that the test can be re-enabled. Since the test would try multiple times, we can still succeed while collecting debug information.

asimshankar added a commit to vanadium-archive/go.ref that referenced this issue May 29, 2015
afandria added a commit to vanadium-archive/js that referenced this issue May 29, 2015
The caveat page doesn't always show up.
vanadium/issues#516

This has affected other tests
vanadium/issues#520
vanadium-archive/browser#84

In order to improve test stability, the test will retry after a 60s
delay. Refreshing the page will very likely allow the test to
continue and check what it really ought to be checking.

The actual bug with wspr/identityd will have to be looked at
separately. Log lines were added to assist in debugging this problem.
- screenshots are taken of the first caveat error
- IP addresses are printed at the start of any web test
- if the test fails, all console output is printed out
  (did not print this upon first caveat error, might want to do that)

MultiPart: 1/3
Change-Id: I3b8cbaa6867a67bde6174e6f51ad164dff8123b9
@afandria
Copy link
Contributor Author

The retry-logic has been added as well as more screenshot and console log grabbing. Once we get another failure, I'll start inspecting logs. (Note: The tests may show up as green because of retry logic. I will have to look at the console output and screenshots to determine if the problem occurred.)

@afandria
Copy link
Contributor Author

This test is fine now with the retry logic. The theory that this test "warms up" the identity server seems to be correct, though the underlying cause for its sql server's behavior is still unknown. The bug with identityd is being tracked separately.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants