slack-infra icon indicating copy to clipboard operation
slack-infra copied to clipboard

moderate-words logs all events

Open BenTheElder opened this issue 2 years ago • 9 comments

This is super wasteful 😱

Right now it's down due to https://github.com/kubernetes/community/issues/6841 so it's not logging at the moment.

We should only log relevant debug info, not every single event processed in full. None of the other tools in this repo do this.

BenTheElder avatar Aug 30 '22 23:08 BenTheElder

cc @cpanato

BenTheElder avatar Aug 30 '22 23:08 BenTheElder

The Kubernetes project currently lacks enough contributors to adequately respond to all issues and PRs.

This bot triages issues and PRs according to the following rules:

  • After 90d of inactivity, lifecycle/stale is applied
  • After 30d of inactivity since lifecycle/stale was applied, lifecycle/rotten is applied
  • After 30d of inactivity since lifecycle/rotten was applied, the issue is closed

You can:

  • Mark this issue or PR as fresh with /remove-lifecycle stale
  • Mark this issue or PR as rotten with /lifecycle rotten
  • Close this issue or PR with /close
  • Offer to help out with Issue Triage

Please send feedback to sig-contributor-experience at kubernetes/community.

/lifecycle stale

k8s-triage-robot avatar Nov 28 '22 23:11 k8s-triage-robot

The Kubernetes project currently lacks enough active contributors to adequately respond to all issues and PRs.

This bot triages issues and PRs according to the following rules:

  • After 90d of inactivity, lifecycle/stale is applied
  • After 30d of inactivity since lifecycle/stale was applied, lifecycle/rotten is applied
  • After 30d of inactivity since lifecycle/rotten was applied, the issue is closed

You can:

  • Mark this issue or PR as fresh with /remove-lifecycle rotten
  • Close this issue or PR with /close
  • Offer to help out with Issue Triage

Please send feedback to sig-contributor-experience at kubernetes/community.

/lifecycle rotten

k8s-triage-robot avatar Dec 29 '22 00:12 k8s-triage-robot

/lifecycle frozen this is still super wasteful

BenTheElder avatar Dec 31 '22 19:12 BenTheElder

/assign

Thanks for raising this @BenTheElder, I've had a go at a fix via #62.

jmhbnz avatar Feb 25 '23 06:02 jmhbnz

/reopen @jeefy we still need to get this change deployed into k8s infra cc @ameukam less logging costs for k8s infra :+)

BenTheElder avatar Feb 28 '23 07:02 BenTheElder

@BenTheElder: Reopened this issue.

In response to this:

/reopen @jeefy we still need to get this change deployed into k8s infra cc @ameukam less logging costs :+)

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.

k8s-ci-robot avatar Feb 28 '23 07:02 k8s-ci-robot

I don't think we've deployed a new build for 3+ years now https://github.com/kubernetes/k8s.io/blob/main/apps/slack-infra/resources/slack-moderator-words/deployment.yaml

BenTheElder avatar Jul 10 '24 23:07 BenTheElder

xref: #66

BenTheElder avatar Jul 10 '24 23:07 BenTheElder

This is actually fixed now.

BenTheElder avatar Jul 11 '24 16:07 BenTheElder

/assign

BenTheElder avatar Jul 11 '24 16:07 BenTheElder