moderate-words logs all events

kubernetes-sigs / slack-infra

Tooling for kubernetes.slack.com

Apache License 2.0

90 stars 36 forks source link

moderate-words logs all events #56

Closed BenTheElder closed 2 months ago

BenTheElder commented 2 years ago

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 commented 2 years ago

cc @cpanato

k8s-triage-robot commented 1 year ago

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 commented 1 year ago

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

BenTheElder commented 1 year ago

/lifecycle frozen this is still super wasteful

jmhbnz commented 1 year ago

/assign

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

BenTheElder commented 1 year ago

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

k8s-ci-robot commented 1 year ago

@BenTheElder: Reopened this issue.

In response to [this](https://github.com/kubernetes-sigs/slack-infra/issues/56#issuecomment-1447680997): >/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](https://git.k8s.io/community/contributors/guide/pull-requests.md). If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository.

BenTheElder commented 2 months ago

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 commented 2 months ago

xref: #66

BenTheElder commented 2 months ago

This is actually fixed now.

BenTheElder commented 2 months ago

/assign