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

Stats aggregation script #35

Merged
merged 3 commits into from
Apr 19, 2024
Merged

Conversation

schuylermartin45
Copy link
Collaborator

@schuylermartin45 schuylermartin45 commented Apr 19, 2024

Adds a quick and dirty script for parsing-out JSON results from the CI integration test scripts

To use, download all the log files from the CI run of interest and point the script at that directory. It will parse-out all the JSON objects and aggregate the statistics into a final report, including a summary of the entire run.

Here are the current results:

{
  "summary": {
    "test_counts": {
      "anaconda_recipes_01": 2,
      "bioconda_recipes_01": 2,
      "bioconda_recipes_02": 2,
      "bioconda_recipes_03": 2,
      "bioconda_recipes_04": 2,
      "conda_forge_recipes_01": 2
    },
    "stages": {
      "recipe_conversion": {
        "total_recipe_files": 11336,
        "total_recipes_processed": 11336,
        "total_errors": 2,
        "total_warnings": 17806,
        "num_recipe_exceptions": 2993,
        "num_recipe_errors": 2,
        "num_recipe_warnings": 8342,
        "num_recipe_success": 8341,
        "num_theoretical_recipe_success": 8341,
        "percent_recipe_exceptions": 0.22,
        "percent_recipe_errors": 0.0,
        "percent_recipe_warnings": 0.78,
        "percent_recipe_success": 0.78,
        "percent_recipe_theoretical_success": 0.78
      },
      "rattler_dry_run": {
        "total_recipe_files": 8343,
        "total_recipes_processed": 8343,
        "total_errors": 5573,
        "percent_errors": 0.57,
        "percent_success": 0.43
      }
    }
  }
}

- Adds a new script to parse-out the JSON blobs returned when the CI scripts
  run
- This will allow us to more easily gather statistics on all tests run when a
  change is made
- Adds statistics aggregation mechanism, which is pretty generic. This should
  work if we add new statistics to the other scripts
- Adds a verbose mode for the final report output
- Cleans up some logic
@schuylermartin45 schuylermartin45 marked this pull request as ready for review April 19, 2024 18:12
@schuylermartin45 schuylermartin45 requested a review from a team as a code owner April 19, 2024 18:12
@schuylermartin45 schuylermartin45 merged commit 240e989 into main Apr 19, 2024
10 checks passed
@schuylermartin45 schuylermartin45 deleted the smartin_data_collection_script branch April 19, 2024 19:20
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