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

Create a gitlab status plugin #649

Merged
merged 1 commit into from Apr 28, 2019

Conversation

globin
Copy link
Member

@globin globin commented Apr 11, 2019

This plugin expects as inputs to a jobset the following:

  • gitlab_status_repo => Name of the repository input pointing to that
    status updates should be POST'ed, i.e. the jobset has a git input
    "nixexprs": "https://gitlab.example.com/project/nixexprs", in which
    case "gitlab_status_repo" would be "nixexprs".
  • gitlab_project_id => ID of the project in Gitlab, i.e. in the above
    case the ID in gitlab of "nixexprs"

cc @Ericson2314 @grahamc

See https://git.mayflower.de/eso/test-libs/pipelines/4794 and https://hydra-eso-test.yflower.de/jobset/test-libs/pkgs-mr-3 for an example.

This plugin expects as inputs to a jobset the following:
 - gitlab_status_repo => Name of the repository input pointing to that
   status updates should be POST'ed, i.e. the jobset has a git input
   "nixexprs": "https://gitlab.example.com/project/nixexprs", in which
   case "gitlab_status_repo" would be "nixexprs".
 - gitlab_project_id => ID of the project in Gitlab, i.e. in the above
   case the ID in gitlab of "nixexprs"
@globin globin mentioned this pull request Apr 15, 2019
@grahamc grahamc merged commit c887451 into NixOS:master Apr 28, 2019
@globin globin deleted the upstream-gitlab-status branch April 28, 2019 20:57
@edolstra
Copy link
Member

This plugin should probably be explicitly enabled by a hydra.conf option, because currently it's doing some non-trivial amount of work just to decide whether it needs to do anything (e.g. query $build->jobsetevals, look for a gitlab_status_repo input, and this for all dependents). That's quite a bit of overhead when we're not using this plugin.

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.

None yet

3 participants