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

Fails after 1st run (Win7, v2.0.2) #121

Closed
davidhund opened this issue May 24, 2017 · 11 comments
Closed

Fails after 1st run (Win7, v2.0.2) #121

davidhund opened this issue May 24, 2017 · 11 comments

Comments

@davidhund
Copy link

Running pwmetrics http://localhost:3000/ --runs=3 fails after the first run.

Steps to reproduce.

pwmetrics http://localhost:3000/ --runs=3

Stack trace

Launching Chrome
  ✓ Success: Run 1 of 3 finished successfully.
events.js:160
      throw er; // Unhandled 'error' event
      ^

Error: connect ECONNREFUSED 127.0.0.1:9222
    at Object.exports._errnoException (util.js:1022:11)
    at exports._exceptionWithHostPort (util.js:1045:20)
    at TCPConnectWrap.afterConnect [as oncomplete] (net.js:1090:14)

Environment

  1. pwmetrics version: 2.0.2
  2. Chrome version: 58.0.3029.110 (64-bit)
  3. OS version: Win 7

I got similar errors on non-localhost sites. Doing just 1 run is fine.

Also: I've noticed the Chrome Translation Dialog pops up, possibly interfering with Lighthouse?
Current Lighthouse version supports --disable-translate but adding that flag does not seem to work. I assume support will be added with #119 ?

@pedro93
Copy link
Collaborator

pedro93 commented May 24, 2017

Hello, thanks for the filling the issue.

Reproducing localhost wouldn't help without having the source code of what you serve nor the method by which you serve it. For instance if your http server is a node process, to run pwmetrics it has to be on a separate process.

Could you provide an example of a non-localhost site where I can reproduce this?

Support for lighthouse v2 flags will be added with #119 yes

@davidhund
Copy link
Author

Thanks for your quick reply @pedro93!

Just tested with pwmetrics https://google.com/ --runs=3:

Launching Chrome
  ✓ Success: Run 1 of 3 finished successfully.
events.js:160
      throw er; // Unhandled 'error' event
      ^

Error: connect ECONNREFUSED 127.0.0.1:9222
    at Object.exports._errnoException (util.js:1022:11)
    at exports._exceptionWithHostPort (util.js:1045:20)
    at TCPConnectWrap.afterConnect [as oncomplete] (net.js:1090:14)

As for localhost: I am serving it through browserSync which proxies a local Apache.
Running pwmetrics directly on the local Apache url fails too b.t.w.

@pedro93
Copy link
Collaborator

pedro93 commented May 24, 2017

That is very strange, I just ran pwmetrics on my machine (ubuntu) and it worked:

pwmetrics

The only variation I see is the operating system. I have a windows 8 machine at home, I will try there in a couple of hours.

In the meantime could you see in the lighthouse repo if there are windows-related issues opened? Also could try to run lighthouse v2 from the command line and see if you get current output? This could be a windows-specific 1.6.5 lighthouse bug, if thats the case #119 will solve the problem when landed..

@denar90
Copy link
Collaborator

denar90 commented May 24, 2017

Hey guys. I assume that this problem can be present on windows and it could be related to chrome launcher. But it should work after updating to LH v2. I'll test current version and with LH v2 on windows later today. Will see how it goes.

@denar90
Copy link
Collaborator

denar90 commented May 24, 2017

hmm, I'm facing kinda similar errors even with LH v2. I used virtualbox and windows10, so it was painful
image

@samccone
Copy link
Collaborator

it seems related to this eh...
GoogleChrome/lighthouse#2337

@pedro93
Copy link
Collaborator

pedro93 commented May 25, 2017

@samccone FYI pwmetrics does not use lighthouse v2 yet.

That is in a PR currently. Does the issue GoogleChrome/lighthouse#2337 also happens for lighthouse v1.6.5? If not this could be something else altogether.

We use a commit specific version: paulirish/lighthouse#c74ea959"

@davidhund
Copy link
Author

@samccone @pedro93 — thanks for looking into this!

I assume — as GoogleChrome/lighthouse#2359 fixes the related LH v2.0.0 issue — we can expect this to be coming in with #119?

@pedro93
Copy link
Collaborator

pedro93 commented May 25, 2017

Yes we intend to merge #119 as soon GoogleChrome/lighthouse#2251 is landed and published.

@samccone
Copy link
Collaborator

The root issues was pre lighthouse 2 :)

@denar90
Copy link
Collaborator

denar90 commented Jun 20, 2017

Should be fixed by #120
Feel free to reopen.

@denar90 denar90 closed this as completed Jun 20, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants