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

Out of retries, exiting! #390

Closed
mcandre opened this issue Apr 14, 2015 · 27 comments
Closed

Out of retries, exiting! #390

mcandre opened this issue Apr 14, 2015 · 27 comments
Labels

Comments

@mcandre
Copy link

mcandre commented Apr 14, 2015

Summary

When I try to initialize and run flow, it quits with Out of retries, exiting!

Trace

$ flow init
$ cat .flowconfig
[ignore]

[include]

[libs]

[options]

$ flow
Flow server launched for /Users/apennebaker
Spawned flow server (child pid=67099)
Logs will go to /var/folders/_9/shv02r4n1nx9ksvx6w_nd4pw0000gp/T/flow_apennebaker/apennebaker-zSUserszSapennebaker.log
The flow server will be ready in a moment.
Error: flow server is busyzing. If it was just started this can take some time. Retrying... |
Error: flow server is busyzing. If it was just started this can take some time. Retrying... -
Error: flow server is busyzing. If it was just started this can take some time. Retrying... |
Out of retries, exiting!lizing. If it was just started this can take some time. Retrying... -

System

$ specs flow os
Specs:

specs 0.14
https://github.com/mcandre/specs#readme

flow --version
Flow, a static type checker for JavaScript, version 0.7.0

system_profiler SPSoftwareDataType | grep 'System Version'
      System Version: OS X 10.10.2 (14C1514)
@gabelevi
Copy link
Contributor

Yeah this has been happening to me too and is pretty annoying. Here's what's happening:

  1. You run flow (aka flow status)
  2. flow starts a flow client.
  3. The flow client sees that no flow server is running and starts one up
  4. The flow client keeps asking the flow server "are you done yet are you done yet are you done yet"
  5. Sometimes the flow server is too busy thinking and doesn't respond
  6. The flow client doesn't like being ignored, and counts each missed response as an error that consumes a retry

I think this should be fixable by just making the flow client a little bit more patient

@gabelevi gabelevi added the bug label Apr 14, 2015
@mcandre
Copy link
Author

mcandre commented Apr 14, 2015

Yeah, it just needs to be patient. Maybe:

  • Add a 2-3 second delay before attempting to contact the server
  • Increase the delay between retries
  • Multiply the maximum number of retries by a factor of 2 or 3
  • Increase the TCP timeout

?

@Kureev
Copy link

Kureev commented Apr 15, 2015

Any solution so? It's really annoying when you're debugging react-native applications. Every refresh ~ 5-7 secs because of this.

@clauderic
Copy link

I'm also running into this problem when working on react-native applications, any input on this would be helpful

@Kureev
Copy link

Kureev commented Apr 17, 2015

@clauderic I think I solved the problem for myself:

  • Get sure that you have .flowconfig in the directory you run flow
  • Make sure you excluded all unnecessary stuff (like node_modules etc)

After that it started working for me

@clark800
Copy link

Ignoring node_modules does not seem to work because it creates many errors like:
Failure("internal error: package /home/user/node_modules/lodash/package.json not found")

@clark800
Copy link

I think the solution is to either run flow check or use the --timeout command line option.

@samwgoldman
Copy link
Member

@gabelevi IIUC, the [facepalm] commit fixes this? If so, please tag as willfix next release and I'll close this out once that commit is released.

@armish
Copy link

armish commented Jul 2, 2015

also running into this problem and since this doesn't exit with status 0, it makes harder to rely on this within a Bash script. Would be great to see this fixed with the upcoming version ;)

@raould
Copy link

raould commented Sep 28, 2015

I guess I have something similar. I just cloned yesterday and built today.
$ flow --version
Flow, a static type checker for JavaScript, version 0.7.0

$ bin/flow
Flow server launched for /home/xyzpdq/Dev/flow
Spawned flow server (child pid=6458)
Logs will go to /tmp/flow/zShomezSxyzpdqzSDevzSflow.log
The Flow server will be ready in a moment.
Flow server launched for /home/xyzpdq/Dev/flow
Spawned flow server (child pid=6475)
Logs will go to /tmp/flow/zShomezSxyzpdqzSDevzSflow.log
The Flow server will be ready in a moment.
Flow server launched for /home/xyzpdq/Dev/flow
Spawned flow server (child pid=6492)
Logs will go to /tmp/flow/zShomezSxyzpdqzSDevzSflow.log
The Flow server will be ready in a moment.
Flow server launched for /home/xyzpdq/Dev/flow
Spawned flow server (child pid=6509)
Logs will go to /tmp/flow/zShomezSxyzpdqzSDevzSflow.log
Out of retries, exiting!

@nmn
Copy link
Contributor

nmn commented Mar 4, 2017

Pretty sure this is fixed now.

@nmn nmn closed this as completed Mar 4, 2017
@Infonautica
Copy link

Infonautica commented Sep 8, 2017

Issue isn't gone. I have a project, and from start of it's developing 1 of 3 flow runs is failing with error
Out of retries, exiting!

On second run it always success. Version 0.54.0. Tried it on create-react-app and custom boilerplate. Windows 10 Pro

@FezVrasta
Copy link
Contributor

Still happening with 0.66

@TrySound
Copy link
Contributor

TrySound commented Mar 5, 2018

@FezVrasta It happens when another instance with different version of flow is run. Just run killall flow && flow

@FezVrasta
Copy link
Contributor

FezVrasta commented Mar 5, 2018

It doesn't help, as soon I run flow again it will return the same error. I can run flow just once before I start getting errors.

Is there any verbose mode I can use to catch the real error?

@TrySound
Copy link
Contributor

TrySound commented Mar 5, 2018

@FezVrasta It may doesn't help when your ide tries to run flow before you.

@FezVrasta
Copy link
Contributor

So what could the solution be like?

@ardok
Copy link

ardok commented Mar 6, 2018

Happening to me too. 0.65.0 and 0.66.0

@mirek
Copy link

mirek commented Mar 14, 2018

Still happens on 0.67.1

@caiogondim
Copy link

Having the same issue with "flow-bin": "0.68.0".

@TrySound
Copy link
Contributor

@caiogondim If you have mac os and yarn run killall flow && yarn flow.

@caiogondim
Copy link

Just tried. Same issue 😢
I'm on macOS.

@FezVrasta
Copy link
Contributor

The problem seem to happen when you have a global flow-bin and a local one, your editor is most likely going to run the global one and your shell is going to call the local one, and this makes Flow go mad.

The solution is to never have flow-bin installed globally.

@yutingkuang
Copy link

It conflicts with the vscode's kit (Flow Language Support).
I stop the kit and resolved.

@dshinzie-zz
Copy link

@FezVrasta I can confirm that deleting flow-bin globally resolved this specific issue for me

@luiguild
Copy link

luiguild commented Aug 18, 2019

I not having flow-bin installed globally and I get this error ever that I open Atom and try edit some file... I not installed nothing related (directly) with Flow in Atom, just added eslint-plugin-flowtype-errors locally in my project... I'm trying using Flow with Vue Single File Components, someone already make it?

I´m trying follow this tutorial https://alligator.io/vuejs/components-flow/

@aghh1504
Copy link

rm -rf node_modules works for me

facebook-github-bot referenced this issue Oct 24, 2019
Summary:
Bumps [rubyzip](https://github.com/rubyzip/rubyzip) from 1.2.2 to 1.3.0.
<details>
<summary>Release notes</summary>

*Sourced from [rubyzip's releases](https://github.com/rubyzip/rubyzip/releases).*

> ## v1.3.0
> Security
>
> - Add `validate_entry_sizes` option so that callers can trust an entry's reported size when using `extract` [https://github.com/facebook/flow/issues/403](https://github-redirect.dependabot.com/rubyzip/rubyzip/pull/403)
>    - This option defaults to `false` for backward compatibility in this release, but you are strongly encouraged to set it to `true`. It will default to `true` in rubyzip 2.0.
>
> New Feature
>
> - Add `add_stored` method to simplify adding entries without compression [https://github.com/facebook/flow/issues/366](https://github-redirect.dependabot.com/rubyzip/rubyzip/pull/366)
>
> Tooling / Documentation
>
> - Add more gem metadata links [https://github.com/facebook/flow/issues/402](https://github-redirect.dependabot.com/rubyzip/rubyzip/pull/402)
>
> ## v1.2.4
> - Do not rewrite zip files opened with `open_buffer` that have not changed [https://github.com/facebook/flow/issues/360](https://github-redirect.dependabot.com/rubyzip/rubyzip/pull/360)
>
> Tooling / Documentation
>
> - Update `example_recursive.rb` in README [https://github.com/facebook/flow/issues/397](https://github-redirect.dependabot.com/rubyzip/rubyzip/pull/397)
> - Hold CI at `trusty` for now, automatically pick the latest ruby patch version, use rbx-4 and hold jruby at 9.1 [https://github.com/facebook/flow/issues/399](https://github-redirect.dependabot.com/rubyzip/rubyzip/pull/399)
>
> ## v1.2.3
> * Allow tilde in zip entry names [https://github.com/facebook/flow/issues/391](https://github-redirect.dependabot.com/rubyzip/rubyzip/pull/391) (fixes regression in 1.2.2 from [https://github.com/facebook/flow/issues/376](https://github-redirect.dependabot.com/rubyzip/rubyzip/pull/376))
> * Support frozen string literals in more files [https://github.com/facebook/flow/issues/390](https://github-redirect.dependabot.com/rubyzip/rubyzip/pull/390)
> * Require `pathname` explicitly [https://github.com/facebook/flow/issues/388](https://github-redirect.dependabot.com/rubyzip/rubyzip/pull/388) (fixes regression in 1.2.2 from [https://github.com/facebook/flow/issues/376](https://github-redirect.dependabot.com/rubyzip/rubyzip/pull/376))
>
> Tooling / Documentation:
>
> * CI updates [https://github.com/facebook/flow/issues/392](https://github-redirect.dependabot.com/rubyzip/rubyzip/pull/392), [https://github.com/facebook/flow/issues/394](https://github-redirect.dependabot.com/rubyzip/rubyzip/pull/394)
>    * Bump supported ruby versions and add 2.6
>    * JRuby failures are no longer ignored (reverts [https://github.com/facebook/flow/issues/375](https://github-redirect.dependabot.com/rubyzip/rubyzip/pull/375) / part of [https://github.com/facebook/flow/issues/371](https://github-redirect.dependabot.com/rubyzip/rubyzip/pull/371))
> * Add changelog entry that was missing for last release [https://github.com/facebook/flow/issues/387](https://github-redirect.dependabot.com/rubyzip/rubyzip/pull/387)
> * Comment cleanup [https://github.com/facebook/flow/issues/385](https://github-redirect.dependabot.com/rubyzip/rubyzip/pull/385)
>
> Since the GitHub release information for 1.2.2 is missing, I will also include it here:
>
> ### 1.2.2
>
> NB: This release drops support for extracting symlinks, because there was no clear way to support this securely. See https://github-redirect.dependabot.com/rubyzip/rubyzip/pull/376#issue-210954555 for details.
>
> * Fix CVE-2018-1000544 [https://github.com/facebook/flow/issues/376](https://github-redirect.dependabot.com/rubyzip/rubyzip/pull/376) / [https://github.com/facebook/flow/issues/371](https://github-redirect.dependabot.com/rubyzip/rubyzip/pull/371)
> * Fix NoMethodError: undefined method `glob' [https://github.com/facebook/flow/issues/363](https://github.com/rubyzip/rubyzip/pull/363)
> * Fix handling of stored files (i.e. files not using compression) with general purpose bit 3 set [https://github.com/facebook/flow/issues/358](https://github.com/rubyzip/rubyzip/pull/358)
> * Fix `close` on StringIO-backed zip file [https://github.com/facebook/flow/issues/353](https://github.com/rubyzip/rubyzip/pull/353)
> * Add `Zip.force_entry_names_encoding` option [https://github.com/facebook/flow/issues/340](https://github-redirect.dependabot.com/rubyzip/rubyzip/pull/340)
> * Update rubocop, apply auto-fixes, and fix regressions caused by said auto-fixes [https://github.com/facebook/flow/issues/332](https://github-redirect.dependabot.com/rubyzip/rubyzip/pull/332), [https://github.com/facebook/flow/issues/355](https://github-redirect.dependabot.com/rubyzip/rubyzip/pull/355)
> * Save temporary files to temporary directory (rather than current directory) [https://github.com/facebook/flow/issues/325](https://github-redirect.dependabot.com/rubyzip/rubyzip/pull/325)
>
> Tooling / Documentation:
></tr></table> ... (truncated)
</details>
<details>
<summary>Changelog</summary>

*Sourced from [rubyzip's changelog](https://github.com/rubyzip/rubyzip/blob/master/Changelog.md).*

> # 1.3.0 (2019-09-25)
>
> Security
>
> - Add `validate_entry_sizes` option so that callers can trust an entry's reported size when using `extract` [https://github.com/facebook/flow/issues/403](https://github-redirect.dependabot.com/rubyzip/rubyzip/pull/403)
>    - This option defaults to `false` for backward compatibility in this release, but you are strongly encouraged to set it to `true`. It will default to `true` in rubyzip 2.0.
>
> New Feature
>
> - Add `add_stored` method to simplify adding entries without compression [https://github.com/facebook/flow/issues/366](https://github-redirect.dependabot.com/rubyzip/rubyzip/pull/366)
>
> Tooling / Documentation
>
> - Add more gem metadata links [https://github.com/facebook/flow/issues/402](https://github-redirect.dependabot.com/rubyzip/rubyzip/pull/402)
>
> # 1.2.4 (2019-09-06)
>
> - Do not rewrite zip files opened with `open_buffer` that have not changed [https://github.com/facebook/flow/issues/360](https://github-redirect.dependabot.com/rubyzip/rubyzip/pull/360)
>
> Tooling / Documentation
>
> - Update `example_recursive.rb` in README [https://github.com/facebook/flow/issues/397](https://github-redirect.dependabot.com/rubyzip/rubyzip/pull/397)
> - Hold CI at `trusty` for now, automatically pick the latest ruby patch version, use rbx-4 and hold jruby at 9.1 [https://github.com/facebook/flow/issues/399](https://github-redirect.dependabot.com/rubyzip/rubyzip/pull/399)
>
> # 1.2.3
>
> - Allow tilde in zip entry names [https://github.com/facebook/flow/issues/391](https://github-redirect.dependabot.com/rubyzip/rubyzip/pull/391) (fixes regression in 1.2.2 from [https://github.com/facebook/flow/issues/376](https://github-redirect.dependabot.com/rubyzip/rubyzip/pull/376))
> - Support frozen string literals in more files [https://github.com/facebook/flow/issues/390](https://github-redirect.dependabot.com/rubyzip/rubyzip/pull/390)
> - Require `pathname` explicitly [https://github.com/facebook/flow/issues/388](https://github-redirect.dependabot.com/rubyzip/rubyzip/pull/388) (fixes regression in 1.2.2 from [https://github.com/facebook/flow/issues/376](https://github-redirect.dependabot.com/rubyzip/rubyzip/pull/376))
>
> Tooling / Documentation:
>
> - CI updates [https://github.com/facebook/flow/issues/392](https://github-redirect.dependabot.com/rubyzip/rubyzip/pull/392), [https://github.com/facebook/flow/issues/394](https://github-redirect.dependabot.com/rubyzip/rubyzip/pull/394)
>   - Bump supported ruby versions and add 2.6
>   - JRuby failures are no longer ignored (reverts [https://github.com/facebook/flow/issues/375](https://github-redirect.dependabot.com/rubyzip/rubyzip/pull/375) / part of [https://github.com/facebook/flow/issues/371](https://github-redirect.dependabot.com/rubyzip/rubyzip/pull/371))
> - Add changelog entry that was missing for last release [https://github.com/facebook/flow/issues/387](https://github-redirect.dependabot.com/rubyzip/rubyzip/pull/387)
> - Comment cleanup [https://github.com/facebook/flow/issues/385](https://github-redirect.dependabot.com/rubyzip/rubyzip/pull/385)
</details>
<details>
<summary>Commits</summary>

- [`e79d9ea`](rubyzip/rubyzip@e79d9ea) Merge pull request [https://github.com/facebook/flow/issues/407](https://github-redirect.dependabot.com/rubyzip/rubyzip/issues/407) from rubyzip/v1-3-0
- [`7c65e1e`](rubyzip/rubyzip@7c65e1e) Bump version to 1.3.0
- [`d65fe7b`](rubyzip/rubyzip@d65fe7b) Merge pull request [https://github.com/facebook/flow/issues/403](https://github-redirect.dependabot.com/rubyzip/rubyzip/issues/403) from rubyzip/check-size
- [`97cb6ae`](rubyzip/rubyzip@97cb6ae) Warn when an entry size is invalid
- [`7849f73`](rubyzip/rubyzip@7849f73) Default validate_entry_sizes to false for 1.3 release
- [`4167f0c`](rubyzip/rubyzip@4167f0c) Validate entry sizes when extracting
- [`94b7fa2`](rubyzip/rubyzip@94b7fa2) [ci skip] Update changelog
- [`93505ca`](rubyzip/rubyzip@93505ca) Check expected entry size in add_stored test
- [`6619bf3`](rubyzip/rubyzip@6619bf3) Merge pull request [https://github.com/facebook/flow/issues/366](https://github-redirect.dependabot.com/rubyzip/rubyzip/issues/366) from hainesr/add-stored
- [`ecb2776`](rubyzip/rubyzip@ecb2776) Zip::File.add_stored() to add uncompressed files.
- Additional commits viewable in [compare view](rubyzip/rubyzip@v1.2.2...v1.3.0)
</details>
<br />

[![Dependabot compatibility score](https://dependabot-badges.githubapp.com/badges/compatibility_score?dependency-name=rubyzip&package-manager=bundler&previous-version=1.2.2&new-version=1.3.0)](https://help.github.com/articles/configuring-automated-security-fixes)

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting `dependabot rebase`.

[//]: # (dependabot-automerge-start)
[//]: # (dependabot-automerge-end)

 ---

<details>
<summary>Dependabot commands and options</summary>
<br />

You can trigger Dependabot actions by commenting on this PR:
- `dependabot rebase` will rebase this PR
- `dependabot recreate` will recreate this PR, overwriting any edits that have been made to it
- `dependabot merge` will merge this PR after your CI passes on it
- `dependabot squash and merge` will squash and merge this PR after your CI passes on it
- `dependabot cancel merge` will cancel a previously requested merge and block automerging
- `dependabot reopen` will reopen this PR if it is closed
- `dependabot ignore this [patch|minor|major] version` will close this PR and stop Dependabot creating any more for this minor/major version (unless you reopen the PR or upgrade to it yourself)
- `dependabot ignore this dependency` will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)
- `dependabot use these labels` will set the current labels as the default for future PRs for this repo and language
- `dependabot use these reviewers` will set the current reviewers as the default for future PRs for this repo and language
- `dependabot use these assignees` will set the current assignees as the default for future PRs for this repo and language
- `dependabot use this milestone` will set the current milestone as the default for future PRs for this repo and language

You can disable automated security fix PRs for this repo from the [Security Alerts page](https://github.com/facebook/flow/network/alerts).

</details>
Pull Request resolved: #8147

Differential Revision: D18120615

Pulled By: nmote

fbshipit-source-id: 8bb7feb00ca2f10db52fbf16dc1939dad3557856
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests