dtcenter / METplus

Python scripting infrastructure for MET tools.
https://metplus.readthedocs.io
Apache License 2.0
92 stars 36 forks source link

Bugfix: Inconsistent RUN_ID values when using instances #2596

Open georgemccabe opened 1 month ago

georgemccabe commented 1 month ago

Describe the Problem

If a use case utilizes instance identifiers (see Instance Names in Process List), then different RUN_ID values are logged.

Expected Behavior

The same run ID should be used in all logs for a given METplus run.

Environment

Describe your runtime environment: Any

To Reproduce

Describe the steps to reproduce the behavior: 1. Copy METplus/parm/use_cases/met_tool_wrapper/Example/Example.conf to another directory 2. Modify the config to change the process list and add an instance header:

Change:

PROCESS_LIST = Example

to:

PROCESS_LIST = Example, Example(my_instance)

[my_instance]

[config]

3. Run use case and review ID values in the logs:

05/21 15:45:31.591 metplus.7d86825e (config_util.py:131) INFO: **** 05/21 15:45:31.592Z metplus.c3256cbe (runtime_freq_wrapper.py:351) INFO: Processing forecast lead 3 hours

Relevant Deadlines

List relevant project deadlines here or state NONE.

Funding Source

Define the source of funding and account keys here or state NONE.

Define the Metadata

Assignee

Labels

Milestone and Projects

Define Related Issue(s)

Consider the impact to the other METplus components.

Bugfix Checklist

See the METplus Workflow for details.

georgemccabe commented 3 weeks ago

This fix will be included with the changes for #2612. This issue can be closed when that issue is resolved.