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

[apps::backup::veeam::vbem::restapi::plugin] --mode=jobs: option --unit is not working #5028

Open
joschi99 opened this issue May 16, 2024 · 5 comments
Labels
enhancement Enhance existing question

Comments

@joschi99
Copy link
Contributor

The option --unit=h is not working, the value defined with --timeframe is still calculated in seconds, so it's not possible to define thresholds in hours, days or weeks.

@lucie-dubrunfaut
Copy link
Contributor

Hello :)

I took the time to inspect the code of the mode concerned and I don't find where this issue could occures with the option --unit itself. Can you provide us an example output with the --verbose and --debug options to help me understanding the problem?
nlabels should contains "hours" if the option --unit=h is handled.
Looking at the mode code and I don't find there is any interaction between the --unit option and the --timeframe option. The ---timeframe must be expressed in seconds and is not converted according to the unit provided in the --unit option.

@joschi99
Copy link
Contributor Author

Hi @lucie-dubrunfaut,
I think this is my error. Probably the option --unit is related only to the threshold job-execution-last and not also to the --timeframe. In this case you can close the ticket

@lucie-dubrunfaut
Copy link
Contributor

Hello :)

Yes, from what I've seen, the timeframe option is only expressed in seconds and doesn't take into account the information provided in the unit option. However, this could be a possible enhancement suggestion if it is blocking you, or at least make this more explicit in the help section.

@joschi99
Copy link
Contributor Author

I think it makes sense to extend the plugin to use the unit also for the timeframe, it's normally much easier to handle hours/days/weeks instead of seconds.

@lucie-dubrunfaut
Copy link
Contributor

I'll notice this as an enhancement suggestion :)

@lucie-dubrunfaut lucie-dubrunfaut added enhancement Enhance existing and removed need-more-infos labels Aug 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Enhance existing question
Projects
None yet
Development

No branches or pull requests

2 participants