Skip to content

[bugfix] Do not load environments twice #441

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.

Sign up for GitHub

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

Merged
merged 6 commits into from
Sep 5, 2018
Merged

[bugfix] Do not load environments twice #441

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
58e037d
WIP: revisit the tests
ajocksch Sep 4, 2018
417a68a
no double module loading any more
ajocksch Sep 5, 2018
96384f9
added comment for temporary module loading
ajocksch Sep 5, 2018
d773842
Merge branch 'master' of https://github.com/ajocksch/reframe into bug…
ajocksch Sep 5, 2018
8c19643
Remove stale files
Sep 5, 2018
4ddf527
moved comment
ajocksch Sep 5, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 5 additions & 1 deletion reframe/core/pipeline.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@
import reframe.utility.os_ext as os_ext
from reframe.core.buildsystems import BuildSystem, BuildSystemField
from reframe.core.deferrable import deferrable, _DeferredExpression, evaluate
from reframe.core.environments import Environment
from reframe.core.environments import Environment, EnvironmentSnapshot
from reframe.core.exceptions import (BuildError, PipelineError, SanityError,
user_deprecation_warning)
from reframe.core.launchers.registry import getlauncher
Expand Down Expand Up @@ -775,12 +775,16 @@ def _setup_environ(self, environ):
for k, v in self.variables.items():
self._current_environ.set_variable(k, v)

# Temporarily load the test's environment to record the actual module
# load/unload sequence
environ_save = EnvironmentSnapshot()
# First load the local environment of the partition
self.logger.debug('loading environment for the current partition')
self._current_partition.local_env.load()

self.logger.debug("loading test's environment")
self._current_environ.load()
environ_save.load()

def _setup_paths(self):
"""Setup the check's dynamic paths."""
Expand Down
6 changes: 2 additions & 4 deletions unittests/test_pipeline.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -69,11 +69,9 @@ def test_environ_setup(self):
test.local = True

test.setup(self.partition, self.progenv)
for m in test.modules:
self.assertTrue(rt.runtime().modules_system.is_module_loaded(m))

for k, v in test.variables.items():
self.assertEqual(os.environ[k], v)
for k in test.variables.keys():
self.assertNotIn(k, os.environ)

# Manually unload the environment
self.progenv.unload()
Expand Down