there is no public method to convert `log.Severity` to `minsev.Severity`

[okhowang]

Problem Statement

minsev define same name severity with log, but has different number.

minsev.Severity.Severity return log.Severity.

but there is no public method to convert log.Severity to minsev.Severity

Proposed Solution

provide public method to convert log.Severity to minsev.Severity

Alternatives

make minsev.Severity defines same number with log.Severity

Prior Art

Additional Context

[dmathieu]

cc @MrAlias as the codeowner of the minsev processor.

[MrAlias]

Can you explain your use case? When are you needing this conversion?

[okhowang]

I want expose public method to set/get log level with log.Severity.
when setting log.Severity, it call minsev.SeverityVar.Set with minsev.Severity

[dmathieu]

This is still a bit unclear. Could you provide a short code sample showing what you'd like to do?

[okhowang]

// LogLevel is the log level for the logger. It is set to log.SeverityUndefined by default.
// Can be inited by OTEL_LOG_LEVEL
func LogLevel() log.Severity {
	return logLevel.Severity()
}

// SetLogLevel sets the log level for the logger.
func SetLogLevel(level log.Severity) {
	// TODO no method convert log.Severity to minsev.Severity
	logLevel.Set(minsev.Severity(level))
}

var logLevel minsev.SeverityVar

like this.
exposing minsev.Severity to SetLogLevel make no sense

[dmathieu]

To be fair, setting the log level to be the min severity makes little sense.
Your log level should be based on the importance of a log message. Hence it should be static for a type of log.
I suppose you want to ensure a log is always emitted. But that, doesn't that means you need to set a level you know will always be set?

[okhowang]

Actually, I want adjust log level to report at runtime, it means that keep process running.
I expose a http api POST /_log_level to set log level to report realtimely

[MrAlias]

I expose a http api POST /_log_level to set log level to report realtimely

When you parse the user provided value to this endpoint why wouldn't you parse it directly into a minsev.Severity?

[MrAlias]

Can you link to the HTTP server handler code?

[okhowang]

I expose a http api POST /_log_level to set log level to report realtimely

When you parse the user provided value to this endpoint why wouldn't you parse it directly into a minsev.Severity?

minsev.Severity is just a const in middleware. maybe using const in otel look likes more standard?