Fix tests to detect issue #1626 - Githubissues

nv-morpheus / Morpheus

Morpheus SDK

Apache License 2.0

333 stars 124 forks source link

Fix tests to detect issue #1626 #1629

Closed dagardner-nv closed 4 months ago

dagardner-nv commented 4 months ago

Description

PR #659 inadvertently excluded the monitor stage from several of the end-to-end pipeline tests.
Adds an environment variable MORPHEUS_MONITOR_ALWAYS_ENABLED which when set, will force the monitor stage to always be enabled.
Adds an auto-use fixture monitor_stage_always_enabled which ensures the environment variable is set & present.

Requires nv-morpheus/MRC#473 to be merged first

By Submitting this PR I confirm:

I am familiar with the Contributing Guidelines.
When the PR is ready for review, new or existing tests cover these changes.
When the PR is ready for review, the documentation is up to date with these changes.

mdemoret-nv commented 4 months ago

/merge