Skip to content

Conversation

@cshiels-ie
Copy link
Contributor

@cshiels-ie cshiels-ie commented Nov 24, 2025

Add error handling and logging to data gathering in compute_anonymize…d_rollup (#281)

  • Implemented try-except blocks for data gathering functions to handle potential exceptions.
  • Added logging for errors encountered during data collection for execution environments, unified jobs, job host summary, and main job events.
  • Updated init.py to include compute_anonymized_rollup in the module exports.

Note

Adds try/except with logging around data gathering in compute_anonymized_rollup and exports it from the package.

  • Anonymized rollups:
    • compute_anonymized_rollup: Wraps all collectors (execution_environments, unified_jobs, job_host_summary_service, main_jobevent_service) in try/except, logs failures, and falls back to empty lists.
    • __init__.py: Exports compute_anonymized_rollup.

Written by Cursor Bugbot for commit 761f36b. This will update automatically on new commits. Configure here.

cshiels-ie and others added 2 commits November 24, 2025 14:08
…d_rollup (ansible#281)

- Implemented try-except blocks for data gathering functions to handle potential exceptions.
- Added logging for errors encountered during data collection for execution environments, unified jobs, job host summary, and main job events.
- Updated __init__.py to include compute_anonymized_rollup in the module exports.
@sonarqubecloud
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
60.0% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube Cloud

Copy link
Contributor

@himdel himdel left a comment

Choose a reason for hiding this comment

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

Sounds good, assuming we want partial collection to succeed.

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