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

Update values.yaml metrics #323

Closed
wants to merge 3 commits into from
Closed

Conversation

Austin37AF
Copy link

Metrics were indented one too many spaces. Fixed indentation so it is not under repos:. It should be same level as repos and not under

what

Changed indentation on metrics section under repos. Indentation was too many spaces causing metrics to be under repos instead of at the same level in the server side config.

why

This caused it to not send metrics properly as it was not formatted correctly in the yaml config.

tests

I have tested my changes by running a local environment and passing metrics along to Grafana to make sure they are properly working. Performed Atlantis tests to make sure all metrics were being sent via statsd.

references

Metrics were indented one too many spaces. Fixed indentation so it is not under repos:. It should be same level as repos and not under
@Austin37AF Austin37AF requested a review from a team as a code owner October 17, 2023 16:47
@jamengual
Copy link
Contributor

@Austin37AF can you bump the chart version please?, and Thanks for the contribution

@jamengual jamengual added the waiting-on-response Waiting for a response from the user label Nov 16, 2023
Copy link

This issue is stale because it has been open for 1 month with no activity. Remove stale label or comment or this will be closed in 1 month.

@github-actions github-actions bot added the Stale label Feb 29, 2024
@github-actions github-actions bot closed this Mar 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Stale waiting-on-response Waiting for a response from the user
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants