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

Downloading dependencies: "checking for connectivity" hangs for certain URLs. #326

Closed
adrienbernede opened this issue Oct 13, 2020 · 1 comment · Fixed by #328
Closed
Assignees
Labels
bug Description of reproducible unexpected behavior. Confirmed Confirmed reproduction of a posted bug.
Milestone

Comments

@adrienbernede
Copy link
Member

adrienbernede commented Oct 13, 2020

Context:

I’m on MacOS Catalina, using virtualenv configured to use the homebrew installed python3.8.5.
I install maestrowf in this environment with pip.

Then I’m relying on lulesh example to create my own study:
https://github.com/LLNL/maestrowf/blob/develop/samples/lulesh/lulesh_sample1_macosx.yaml

Problem:

I’m defining a git dependency:

      dependencies:
          git:
              - name: MFEM
                path: $(OUTPUT_PATH)
                url: https://github.com/mfem/mfem.git
                tag: v4.1

but it hangs at "checking for connectivity":

 [2020-10-13 14:02:42: INFO] Acquiring -- MFEM
[2020-10-13 14:02:42: INFO] Checking for connectivity to 'https://github.com/mfem/mfem.git'

With the following stack trace when aborting:

Traceback (most recent call last):
  File "/Users/bernede1/Environments/Python/maestrowf/bin/maestro", line 8, in <module>
    sys.exit(main())
  File "/Users/bernede1/Environments/Python/maestrowf/lib/python3.8/site-packages/maestrowf/maestro.py", line 430, in main
    rc = args.func(args)
  File "/Users/bernede1/Environments/Python/maestrowf/lib/python3.8/site-packages/maestrowf/maestro.py", line 240, in run_study
    study.setup_environment()
  File "/Users/bernede1/Environments/Python/maestrowf/lib/python3.8/site-packages/maestrowf/datastructures/core/study.py", line 392, in setup_environment
    self.environment.acquire_environment()
  File "/Users/bernede1/Environments/Python/maestrowf/lib/python3.8/site-packages/maestrowf/datastructures/core/studyenvironment.py", line 194, in acquire_environment
    value.acquire(substitutions=self.substitutions.values())
  File "/Users/bernede1/Environments/Python/maestrowf/lib/python3.8/site-packages/maestrowf/datastructures/environment/gitdependency.py", line 173, in acquire
    retcode = clone.wait()
  File "/usr/local/Cellar/python@3.8/3.8.5/Frameworks/Python.framework/Versions/3.8/lib/python3.8/subprocess.py", line 1079, in wait
    return self._wait(timeout=timeout)
  File "/usr/local/Cellar/python@3.8/3.8.5/Frameworks/Python.framework/Versions/3.8/lib/python3.8/subprocess.py", line 1804, in _wait
    (pid, sts) = self._try_wait(0)
  File "/usr/local/Cellar/python@3.8/3.8.5/Frameworks/Python.framework/Versions/3.8/lib/python3.8/subprocess.py", line 1762, in _try_wait
    (pid, sts) = os.waitpid(self.pid, wait_flags)
KeyboardInterrupt

More:

It seems to depend on the repo:

OK:
https://github.com/LLNL/axom.git
https://github.com/LLNL/Umpire.git
https://github.com/LLNL/lulesh.git
https://github.com/prunkdump/arduino-variometer.git
https://github.com/mfem/web.git
https://github.com/geosx/geosx.git

KO:
https://github.com/spack/spack.git
https://github.com/mfem/mfem.git

@FrankD412 FrankD412 added bug Description of reproducible unexpected behavior. Confirmed Confirmed reproduction of a posted bug. labels Oct 13, 2020
@FrankD412 FrankD412 added this to the Release 1.1.9 milestone Oct 13, 2020
@FrankD412 FrankD412 self-assigned this Oct 13, 2020
@adrienbernede
Copy link
Member Author

adrienbernede commented Oct 13, 2020

https://docs.python.org/2/library/subprocess.html#subprocess.Popen.wait

The use of wait() for a supprocess with PIPE on stdout and/or stderr can lead to a deadlock if the output is too large.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Description of reproducible unexpected behavior. Confirmed Confirmed reproduction of a posted bug.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants