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

gitlab-housekeeping add app_sre_bot label to qontract_reconcile_merged_merge_requests #4010

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

maorfr
Copy link
Contributor

@maorfr maorfr commented Jan 23, 2024

with this label we can differentiate between MRs submitted by the bot or not.

this will allow us to measure increase in AppSRE productivity by seeing an increase in bot MRs and a decrease in team members MRs.

@rporres
Copy link
Contributor

rporres commented Jan 23, 2024

How about replacing the app_sre label with something like author_category that we make sure it's bounded. It could have the three values that we are interested in:

  • app_sre
  • app_sre_bot
  • other

The cardinality of the metric wouldn't increase in that way

@maorfr
Copy link
Contributor Author

maorfr commented Jan 24, 2024

do you really mind that much? i'll implement what you are proposing but it someone will say "why are we hardcoding app_sre and not something else".

this PR is straight forward.

@rporres
Copy link
Contributor

rporres commented Jan 24, 2024

It's way nicer to say what we want with just a label and certainly more future-proof.

Cardinality wise I was wrong, we would have the same increase: three time series instead of one in each case.

Do I care that much so as to block this unless changed? At this moment with the cases that we need to solve, no. But who knows...

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