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

Passing through HOME environment variable in tox environments #23

Merged
merged 1 commit into from
Sep 12, 2024

Conversation

matt-graham
Copy link
Collaborator

We seem to be getting intermittent failures in the tox test and docs environments due to the lack of a HOME variable being set with the tox environment which is causing an error on importing dxh (and thus importing some dolfinx modules):

[fv-az881-524:03933] Error: Unable to get the user home directory
[fv-az881-524:03933] Error: Unable to get the user home directory
--------------------------------------------------------------------------
It looks like opal_init failed for some reason; your parallel process is
likely to abort.  There are many reasons that a parallel process can
fail during opal_init; some of which are due to configuration or
environment problems.  This failure appears to be an internal failure;
here's some additional information (which may only be relevant to an
Open MPI developer):

  mca_base_var_init failed
  --> Returned value -1 instead of OPAL_SUCCESS
--------------------------------------------------------------------------
--------------------------------------------------------------------------
It looks like MPI_INIT failed for some reason; your parallel process is
likely to abort.  There are many reasons that a parallel process can
fail during MPI_INIT; some of which are due to configuration or environment
problems.  This failure appears to be an internal failure; here's some
additional information (which may only be relevant to an Open MPI
developer):

  ompi_mpi_instance_init: opal_init_util failed
  --> Returned "Error" (-1) instead of "Success" (0)
--------------------------------------------------------------------------
--------------------------------------------------------------------------
It looks like MPI_INIT failed for some reason; your parallel process is
likely to abort.  There are many reasons that a parallel process can
fail during MPI_INIT; some of which are due to configuration or environment
problems.  This failure appears to be an internal failure; here's some
additional information (which may only be relevant to an Open MPI
developer):

  ompi_mpi_init: ompi_mpi_instance_init failed
  --> Returned "Error" (-1) instead of "Success" (0)
--------------------------------------------------------------------------
*** An error occurred in MPI_Init_thread
*** on a NULL communicator
*** MPI_ERRORS_ARE_FATAL (processes in this communicator will now abort,
***    and MPI will try to terminate your MPI job as well)
[fv-az881-524:03933] Local abort before MPI_INIT completed completed successfully, but am not able to aggregate error messages, and not able to guarantee that all other processes were killed!

From testing locally, setting passenv = HOME in the tox configuration to pass through the HOME environment variable seems to fix the issue.

Copy link

codecov bot commented Sep 12, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 97.1%. Comparing base (63d17ab) to head (172a552).
Report is 1 commits behind head on main.

Additional details and impacted files
@@          Coverage Diff          @@
##            main     #23   +/-   ##
=====================================
  Coverage   97.1%   97.1%           
=====================================
  Files          1       1           
  Lines        201     201           
  Branches      40      40           
=====================================
  Hits         195     195           
  Misses         4       4           
  Partials       2       2           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Member

@samcunliffe samcunliffe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🫠

@samcunliffe samcunliffe added bug Something isn't working upstream Something relating to one of our dependencies (DOLFINx or FEniCSx) labels Sep 12, 2024
@samcunliffe samcunliffe merged commit 5fb3f47 into main Sep 12, 2024
11 checks passed
@samcunliffe samcunliffe deleted the mmg/pass-through-env-var-tox branch September 12, 2024 16:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working upstream Something relating to one of our dependencies (DOLFINx or FEniCSx)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants