Skip to content

Conversation

EvanKrall
Copy link
Member

No description provided.

…ed to python 3.10 a while ago. This fixes 'make quick-test'.
…ml. This allows overriding sensu parameters for a specific check.
…ervice's monitoring section overrides the alert's defaults, not the other way around. PAASTA-18103
@EvanKrall EvanKrall requested a review from a team as a code owner August 26, 2025 21:42
Copy link
Member

@nemacysts nemacysts left a comment

Choose a reason for hiding this comment

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

heh, looks like we missed a couple places for the py3.10 upgrade :p

# monitoring.yaml
team: myteam # For most alerts, myteam will be notified.
check_overrides:
check_autoscaler_max_instances:
Copy link
Member

Choose a reason for hiding this comment

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

might be nice to have a central location with all the possible checks somewhere

Comment on lines +93 to +95
max_instances_suggestion = ""
alert_threshold_suggestion = ""
disable_paging_suggestion = ""
Copy link
Member

Choose a reason for hiding this comment

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

it'd be nice to refactor things to not need to ""-init these, but it's not a big deal imo

return team_data


def get_check_specific_overrides(overrides, check_name):
Copy link
Member

Choose a reason for hiding this comment

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

nit: types?

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