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

Add metrics about workflow latency (per manager) #993

Open
Tracked by #322
pacospace opened this issue Jan 31, 2022 · 10 comments
Open
Tracked by #322

Add metrics about workflow latency (per manager) #993

pacospace opened this issue Jan 31, 2022 · 10 comments
Assignees
@KPostOffice
Labels
kind/feature Categorizes issue or PR as related to a new feature. priority/backlog Higher priority than priority/awaiting-more-evidence. sig/user-experience Issues or PRs related to the User Experience of our Services, Tools, and Libraries. triage/accepted Indicates an issue or PR is ready to be actively worked on.

Comments

@pacospace
Copy link
Contributor

pacospace commented Jan 31, 2022 •
edited
Loading

As Thoth Owner,

I want to have a way to monitor latency for each of the Kebechet managers for the SLO
@pacospace pacospace mentioned this issue Jan 31, 2022
Open
24 tasks
@sesheta
Copy link
Member

sesheta commented Jan 31, 2022

@pacospace: This issue is currently awaiting triage.
If a refinement session determines this is a relevant issue, it will accept the issue by applying the
triage/accepted label and provide further guidance.

The triage/accepted label can be added by org members by writing /triage accepted in a comment.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@sesheta sesheta added the needs-triage Indicates an issue or PR lacks a `triage/...` label and requires one. label Jan 31, 2022
@pacospace pacospace changed the title workflow latency (per manager) Add metrics about workflow latency (per manager) Jan 31, 2022
@pacospace pacospace added the triage/needs-information Indicates an issue needs more information in order to work on it. label Jan 31, 2022
@pacospace pacospace transferred this issue from thoth-station/core Feb 7, 2022
@KPostOffice
Copy link
Member

Hey @pacospace can you explain a bit more what you are looking for? Are you asking for metrics on how long the managers take to run?

@pacospace
Copy link
Contributor Author

Hey @pacospace can you explain a bit more what you are looking for? Are you asking for metrics on how long the managers take to run?

We currently can see in metrics the time taken by the kebechet workflow, but we don't know how much time is used from each manager activated. It would be useful to also monitor how much time each manager takes as we run all managers in the same workflow.

@pacospace
Copy link
Contributor Author

  • time for cloning repo metrics
  • time for each manager

@goern goern removed the triage/needs-information Indicates an issue needs more information in order to work on it. label Feb 15, 2022
@codificat
Copy link
Member

/triage accepted
/assign @KPostOffice
/priority important-soon

@sesheta sesheta added triage/accepted Indicates an issue or PR is ready to be actively worked on. priority/important-soon Must be staffed and worked on either currently, or very soon, ideally in time for the next release. and removed needs-triage Indicates an issue or PR lacks a `triage/...` label and requires one. labels Feb 15, 2022
@goern goern added the kind/feature Categorizes issue or PR as related to a new feature. label Feb 16, 2022
@sesheta
Copy link
Member

sesheta commented May 17, 2022

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.

If this issue is safe to close now please do so with /close.

/lifecycle stale

@sesheta sesheta added the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label May 17, 2022
@goern
Copy link
Member

goern commented May 24, 2022

/remove-lifecycle stale
/priority backlog

@sesheta
Copy link
Member

sesheta commented Jun 23, 2022

Stale issues rot after 30d of inactivity.
Mark the issue as fresh with /remove-lifecycle rotten.
Rotten issues close after an additional 30d of inactivity.

If this issue is safe to close now please do so with /close.

/lifecycle rotten

@sesheta sesheta added lifecycle/rotten Denotes an issue or PR that has aged beyond stale and will be auto-closed. and removed lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. labels Jun 23, 2022
@codificat
Copy link
Member

hmmm it seems the bots were taking a nap during the previous attempt.

/remove-lifecycle rotten
/priority backlog
/remove-priority important-soon

@sesheta sesheta added priority/backlog Higher priority than priority/awaiting-more-evidence. and removed lifecycle/rotten Denotes an issue or PR that has aged beyond stale and will be auto-closed. priority/important-soon Must be staffed and worked on either currently, or very soon, ideally in time for the next release. labels Jun 23, 2022
@codificat
Copy link
Member

/sig user-experience

@sesheta sesheta added the sig/user-experience Issues or PRs related to the User Experience of our Services, Tools, and Libraries. label Aug 29, 2022
@codificat codificat moved this to 📋 Backlog in Planning Board Sep 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@KPostOffice KPostOffice

Labels
kind/feature Categorizes issue or PR as related to a new feature. priority/backlog Higher priority than priority/awaiting-more-evidence. sig/user-experience Issues or PRs related to the User Experience of our Services, Tools, and Libraries. triage/accepted Indicates an issue or PR is ready to be actively worked on.
Projects
Planning Board
Status: 📋 Backlog
Milestone
No milestone
Development

No branches or pull requests
5 participants
@goern @codificat @KPostOffice @pacospace @sesheta