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

'Crystal snapshots' changes not propagated from UI #1479

Open
elmjag opened this issue Oct 29, 2024 · 3 comments
Open

'Crystal snapshots' changes not propagated from UI #1479

elmjag opened this issue Oct 29, 2024 · 3 comments

Comments

@elmjag
Copy link
Contributor

elmjag commented Oct 29, 2024

There is a Crystal snapshots drop-down in the queue settings:

image

Changing this value does not seem to have any effect. Regardless of what value you select in that drop-down, the collection hardware object is always invoked with the original value.

The take_snapshots key in the "data collect parameters" never changes, it always remain at the original value.

@marcus-oscarsson
Copy link
Member

Crystal snapshots does not change take_snapshots it changes number_of_snapshots on the collect hardware object.

@marcus-oscarsson
Copy link
Member

marcus-oscarsson commented Oct 29, 2024

This behavior was changed fairly recently and we might need to discuss if number of snapshots is a data collection parameter that is per data collection. In the past take_snapshots was used a bit ambiguously as a boolean, or sometimes as an int with the number of snapshots. The number of snapshots was passed via a somewhat unconventional patch that was replaced by simply setting collect.number_of_snapshots.

@elmjag
Copy link
Contributor Author

elmjag commented Oct 29, 2024

Crystal snapshots does not change take_snapshots it changes number_of_snapshots on the collect hardware object.

Aha! The fact that take_snapshots is set to numerical values, like 0, 2, 4.. etc confused me. The collect.number_of_snapshots works correctly.

I agree, it would be nice to modify this to make it less confusing.

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

No branches or pull requests

2 participants