Page MenuHomePhabricator

Archive the search/xgboot repo in gerrit
Closed, ResolvedPublic

Description

Reason

The search/xgboost repo in Gerrit isn't used anymore. Archiving it and disabling CI for this project would reduce confusion and ensure that we will not waste time maintaining it (in particular, aligning it with our current best practices in terms of CI).

To-do list

  • This task
    • Add maintainers of that extension/skin as subscribers to this task
  • Configuration/tests/integrations/etc.
  • Repositories
    • Empty repository, leaving only the README indicating the repo is archived, and referring to this task by URL.
    • Mark the repository read-only in Gerrit (also edit description to prepend "[ARCHIVED]" and a reference to this task): (provide Gerrit Repository URL).
    • Archive the Diffusion repository for the extension/skin: does not exist
    • Delete the GitHub mirror: (provide GitHub repository URL).

Event Timeline

Change 684276 had a related patch set uploaded (by Hashar; author: Hashar):

[integration/config@master] Archive search/xgboost

https://gerrit.wikimedia.org/r/684276

@EBernhardson the search/mjolnir CI configuration has a comment about xgboost ("build_timeout: 10 # minutes, xgboost takes a while to compile T184754"). Do we have a dependency there? Or can we really archive search/xgboost?

The JJB config for the search/mjolnir has a comment referring to search/xgboost so maybe it is still used. Though most probably it is an artifact from the past and the code has since been updated to no more rely on xgboost.

Though most probably it is an artifact from the past and the code has since been updated to no more rely on xgboost.

In the past we used the JVM wrapper for xgboost, for which we ended up needing to maintain a fork. We moved from that JVM wrapper to the python wrapper for xgboost. With the python wrapper we are using upstream exactly as-is, with no local fork. The dependency is pulled in from pypi via requirements.txt and compiled if pip decides that's the best installation method.

So xgboost is still involved in the CI process, but the local fork is no longer referenced.

Great, thank you @EBernhardson . Will do the archival with @Gehel :]

Change 684276 merged by jenkins-bot:

[integration/config@master] Archive search/xgboost

https://gerrit.wikimedia.org/r/684276

hashar claimed this task.
hashar updated the task description. (Show Details)

Should be good. It is gone from CI and read only in Gerrit.