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

add limes_project_commitment_min_expires_at metric #659

Merged
merged 1 commit into from
Feb 11, 2025

Conversation

wagnerd3
Copy link
Contributor

Checklist:

  • If this PR is about a plugin, I tested the plugin against an OpenStack cluster.
  • I updated the documentation to describe the semantical or interface changes I introduced.

Local check:
image

| Gauge | `limes_project_usage` | `service`, `resource`, `domain`, `domain_id`, `project`, `project_id` |
| Type | Metric | Labels |
|-------|-----------------------------------------------------|----------------------------------------------------------------------------------------------------|
| Gauge | `limes_autogrow_growth_multiplier` | `service`, `resource` |
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I completed the list with all missing metrics from collector/metrics.go. Did not know if the other metrics were missing on purpose or whether this was just forgotten.

@wagnerd3 wagnerd3 force-pushed the min_expiresAt_metric branch from 1ccaa51 to 2502f34 Compare February 11, 2025 12:50
@wagnerd3 wagnerd3 force-pushed the min_expiresAt_metric branch from 2502f34 to 444dff4 Compare February 11, 2025 12:51
@@ -9,10 +9,10 @@ The API service exposes the standard HTTP metrics generated by the standard Go c

Copy link
Contributor Author

Choose a reason for hiding this comment

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

One question regarding markdown: My IDE automatically formatted the tables with whitespaces so that the | borders align. I disabled this now, as I saw this only sometimes being done - e.g. not in this file. Would it be desirable to somehow auto-style the markdown?

Copy link
Contributor

Choose a reason for hiding this comment

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

I prefer this style on the basis of my editor not having automatic alignment like this, so I would just break it when I edit the files again the next time. But not a strong preference from my side.

Copy link
Contributor

@majewsky majewsky left a comment

Choose a reason for hiding this comment

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

Very nice, thanks!

@@ -9,10 +9,10 @@ The API service exposes the standard HTTP metrics generated by the standard Go c

Copy link
Contributor

Choose a reason for hiding this comment

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

I prefer this style on the basis of my editor not having automatic alignment like this, so I would just break it when I edit the files again the next time. But not a strong preference from my side.

@majewsky majewsky merged commit a31f973 into master Feb 11, 2025
6 checks passed
@majewsky majewsky deleted the min_expiresAt_metric branch February 11, 2025 14:12
@coveralls
Copy link

coveralls commented Feb 13, 2025

Coverage Status

coverage: 79.556% (+0.05%) from 79.51%
when pulling 1ccaa51 on min_expiresAt_metric
into 644f2aa on master.

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.

3 participants