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

DCES-728 Grafana Fixes - Contribution Scheduler #30130

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

LAA-Dave-Edwards
Copy link
Contributor

Added changes for the contribution scheduler count.
To see how the panel changes will be surface in the proper environment.

…title )

Swapping to using Thanos as the data is sparce, and granularity not needed.
Forcing 30d range. Should only be 1 per day, so pointless to track at default values.
Forcing 1h interval, keeps it from displaying oddly. The return to 0 helps with understanding it's not a single value we're tracking through the days.
@LAA-Dave-Edwards LAA-Dave-Edwards requested a review from a team as a code owner March 4, 2025 12:31
@cloud-platform-actions
Copy link

Team name: hmpps-assessments

  • Valid: true

Repository: https://github.com/ministryofjustice/hmpps-strengths-based-needs-assessments-ui

  • Public: true

@cloud-platform-actions
Copy link

Team name: hmpps-assessments

  • Valid: true

Repository: https://github.com/ministryofjustice/hmpps-strengths-based-needs-assessments-ui

  • Public: true

@sablumiah
Copy link
Contributor

Terraform Plan Summary

Terraform Plan: 0 to be created, 0 to be destroyed, 1 to be updated, 0 to be replaced and 88 unchanged.

Resources to update:

! module.rds.aws_db_parameter_group.custom_parameters

@sablumiah
Copy link
Contributor

This PR CANNOT be auto approved and requires manual approval from the Cloud Platform team.
Reason:
🕵️‍♂️ Manual review required: OPA auto approve policy checks did not pass.
🕵️‍♂️ Detected changes to K8s YAML files. Manual review needed.
Please raise it in #ask-cloud-platform Slack channel.

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