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

Remove existing ATS / "labels"-related code #2672

Open
Swatinem opened this issue Oct 14, 2024 · 1 comment
Open

Remove existing ATS / "labels"-related code #2672

Swatinem opened this issue Oct 14, 2024 · 1 comment
Assignees

Comments

@Swatinem
Copy link

A draft to remove this code from worker exists here: codecov/worker#782
This does clean up quite a bit, and in particular removes the datapoints which are saved in the report, as well as tasks related to labels.

However, in particular the label-analysis task is still being invoked from the API.

As removing this would make report processing simpler, and the internal report format more lightweight, I would love to go ahead with this deprecation / removal.

The remaining question here, given that a low number of requests/tasks does happen, is how to go about this.

We need to figure out which customers are using this feature, and figure out if they get any value from it at all. We would then need to inform those customers that this feature is going away for good.

@Swatinem Swatinem self-assigned this Oct 14, 2024
@Swatinem
Copy link
Author

Here is a metabase query listing all the existing/remaining users of this feature: https://metabase.codecov.dev/question/273-labelanalysis-users

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

No branches or pull requests

1 participant