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

Port the 'reconnect' and the time string support changes from top of tree #1404

Merged
merged 3 commits into from
Jul 2, 2024

Conversation

nichamon
Copy link
Collaborator

No description provided.

The 'interval' attribute is still supported, but it will be removed
ieventually. LDMSD reports a warning message if 'interval' is used.

ldmsd_controller also displays a warnning message if 'interval' is used.
The patch makes LDMSD support time-interval strings. A time-interval
string is a float followed by a unit string. A unit string is one of
the followings:
  us -- microseconds
  ms -- milliseconds
  s  -- seconds
  m  -- minutes
  h  -- hours
  d  -- days

For example, with this patch, '1s', '1.5s', and '100'ms' can be used.
@nichamon nichamon marked this pull request as draft June 17, 2024 19:06
ldmsd reports a warning message that 'interval' is being deprecated if
'interval' is used.
@nichamon nichamon force-pushed the interval2reconnect-time-string branch from 8f859ee to 04f6dc2 Compare June 17, 2024 19:10
@nichamon nichamon marked this pull request as ready for review June 17, 2024 19:14
@nichamon nichamon requested a review from tom95858 June 17, 2024 19:14
@tom95858 tom95858 merged commit dcc0067 into ovis-hpc:b4.4 Jul 2, 2024
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants