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.

Sign up for GitHub

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

Jump to bottom

Introduce pg dump metrics #69

Merged
merged 6 commits into from
Feb 8, 2022
Merged

Introduce pg dump metrics #69

merged 6 commits into from
Feb 8, 2022

Conversation

pacospace
Copy link
Contributor

Related Issues and Dependencies

2022-02-08 11:03:16,863 182620 INFO     thoth.common:366: Logging to rsyslog endpoint is turned off
2022-02-08 11:03:16,863 182620 INFO     thoth.graph_metrics_exporter:303: graph-metrics-exporter  v0.5.8+common.0.34.0.storages.0.68.1 starting...
2022-02-08 11:03:16,930 182620 INFO     thoth.graph_metrics_exporter:276: graph_database_dumps_check task starting...
2022-02-08 11:03:17,615 182620 INFO     thoth.graph_metrics_exporter:239: Number of database dumps available on Ceph is: 21
2022-02-08 11:03:17,616 182620 INFO     thoth.graph_metrics_exporter:242: Number of database dumps expected: 21
2022-02-08 11:03:17,617 182620 INFO     thoth.graph_metrics_exporter:252: Last database dump was stored on: 2022-02-06
2022-02-08 11:03:17,617 182620 INFO     thoth.graph_metrics_exporter:257: Last expected database dump date is: 2022-02-01

Related-To: thoth-station/metrics-exporter#821

Francesco Murdaca added 4 commits February 8, 2022 12:00
Add metrics for pg dumps
7f4b071 
Signed-off-by: Francesco Murdaca <[email protected]>
update README
a8b65a4 
Signed-off-by: Francesco Murdaca <[email protected]>
adjust pre-commit
51f2056 
Signed-off-by: Francesco Murdaca <[email protected]>
remove comments
37a020f 
Signed-off-by: Francesco Murdaca <[email protected]>
@sesheta sesheta added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Feb 8, 2022
@pacospace pacospace requested a review from fridex February 8, 2022 11:05
fridex
fridex reviewed Feb 8, 2022
View reviewed changes
Copy link

@fridex fridex left a comment

Choose a reason for hiding this comment

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

Nice, lgtm! 💯 👍🏻

app.py Outdated Show resolved Hide resolved
app.py Outdated Show resolved Hide resolved
Francesco Murdaca and others added 2 commits February 8, 2022 13:46
@fridex
Update app.py
98f53bc 
Co-authored-by: Fridolín Pokorný <[email protected]>
@fridex
Update app.py
e6e0833 
Co-authored-by: Fridolín Pokorný <[email protected]>
@pacospace pacospace requested a review from fridex February 8, 2022 12:46
fridex
fridex approved these changes Feb 8, 2022
View reviewed changes
Copy link

@fridex fridex left a comment

Choose a reason for hiding this comment

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

lgtm, thanks! 💯 👍🏻

@sesheta
Copy link
Member

sesheta commented Feb 8, 2022

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: fridex

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@sesheta sesheta added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Feb 8, 2022
@sesheta sesheta merged commit 051b70b into thoth-station:master Feb 8, 2022
@pacospace pacospace mentioned this pull request Feb 9, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@fridex fridex fridex approved these changes

@harshad16 harshad16 Awaiting requested review from harshad16

@KPostOffice KPostOffice Awaiting requested review from KPostOffice

Assignees
No one assigned
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@pacospace @sesheta @fridex