Skip to content

Fix send_aggregated_alerts email alerts#975

Merged
pragya811 merged 6 commits into
mainfrom
alert-fix
Mar 9, 2026
Merged

Fix send_aggregated_alerts email alerts#975
pragya811 merged 6 commits into
mainfrom
alert-fix

Conversation

@pragya811

Copy link
Copy Markdown
Member

Type of change

Note: Fill x in []

  • bug
  • enhancement
  • documentation
  • dependencies

Description

The PR includes the fix for the traceback found while running policy send_aggregated_alerts in daily Jenkins policies run.

The DataFrame is built from Elasticsearch data that includes numeric fields (e.g. counts, days). Pandas infers columns like those as float64. In newer pandas (and with Python 3.13), fillna(value='') is no longer allowed on numeric dtypes because:

  1. Putting the string '' into a float64 column would require changing the column type.
  2. Pandas treats that as a lossy/invalid operation and raises:
    pandas.errors.LossySetitemError, then
    TypeError: Invalid value '' for dtype 'float64'.

For security reasons, all pull requests need to be approved first before running any automated CI

@ebattat

ebattat commented Mar 9, 2026

Copy link
Copy Markdown
Member

@pragya811,
can u pls check why ci is failing ?

@pragya811 pragya811 requested review from ebattat and inntran March 9, 2026 12:42

@ebattat ebattat left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

/LGTM

@pragya811 pragya811 merged commit ea338cf into main Mar 9, 2026
18 checks passed
@github-project-automation github-project-automation Bot moved this from In progress to Done in Cloud-Governance project Mar 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

2 participants