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

Update CSV Parser tool #2631

Merged
merged 4 commits into from
Mar 15, 2022
Merged

Update CSV Parser tool #2631

merged 4 commits into from
Mar 15, 2022

Conversation

yanazaeva
Copy link
Contributor

Related issue
#2468

Description

This closes #2468. In this pull, we add a new method in order to sum the parent process values and the children process values. Up to now, the values of the children's processes were not taken into account. Besides, we modified the thresholds accordingly to the new changes implemented in 4.3.

@yanazaeva yanazaeva self-assigned this Mar 3, 2022
@yanazaeva yanazaeva changed the title Feature/2468 update csv parser tool Update CSV Parser tool Mar 4, 2022
@yanazaeva yanazaeva force-pushed the feature/2468-update-csv-parser-tool branch from 5ad3ec8 to 4d2788a Compare March 8, 2022 10:05
@yanazaeva yanazaeva force-pushed the feature/2468-update-csv-parser-tool branch from 4d2788a to 2e1e362 Compare March 8, 2022 14:53
@yanazaeva yanazaeva requested a review from Selutario March 8, 2022 14:54
Copy link
Member

@Rebits Rebits left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Rebits Rebits merged commit b8247ca into master Mar 15, 2022
@Rebits Rebits deleted the feature/2468-update-csv-parser-tool branch March 15, 2022 10:19
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.

Update cluster CSV parser tool and test thresholds
5 participants