ld-monitor: Remove session.set_status()
calls and define protocol in compose example
#812
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
This removes the old
session.set_status()
calls that are automated in modern ocs. I also found another change to the configs we need, specifying UDP in the port mapping.Motivation and Context
Ran into the status trying to go backwards when running at site:
Also noticed the protocol defined as
tcp
when the container was online, though knew we wanted UDP.How Has This Been Tested?
Tested the change at the site.
Types of changes
Checklist: