Project

Profile

Help

Task #89

Add automation to run unit tests for all PRs against pulp/<plugin> automatically

Added by bcourt over 5 years ago. Updated 3 months ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
Normal
Assignee:
Start date:
Due date:
% Done:

100%

Estimated time:
Platform Release:
Target Release - Packaging:
Groomed:
Yes
Sprint Candidate:
Yes
Tags:
Pulp 2
Sprint:
April 2015
Quarter:

Description

Us the jenkins github pull request builder plugin to build all PRs against the core Pulp project automatically. This includes creating comments on the PRs indicating the success/failure of the test run.

This will require dynamically figuring out which nightly repo should be used to install the base pulp and plugin.

General procedure for running the unit tests

  1. Determine the base repository to install from
  2. Install @pulp-server-qpid
  3. Install the plugin that is being tested from RPM (this will ensure all dependencies are installed)
  4. Uninstall the plugin that is being tested
  5. Install the plugin from source
  6. Run the tests

Related issues

Blocked by Pulp - Task #85: Add automation to run unit tests for all PRs against pulp/pulp automaticallyCLOSED - CURRENTRELEASE

<a title="Actions" class="icon-only icon-actions js-contextmenu" href="#">Actions</a>

History

#1 Updated by bmbouter over 5 years ago

Will this include an infrastructure user to post back as? I'm not sure which user will be posting the comment.

#2 Updated by bcourt over 5 years ago

The infrastructure user to post back the results to github will be under task 85.

#3 Updated by bcourt over 5 years ago

Once 85 has been completed the difficulty in this task is that we will need to the proper base version of Pulp to install before running the tests for the given plugin. Whether that is done using a custom build on merges to core Pulp or it's done via some other build mechanism will have to be worked out here.

#4 Updated by bcourt over 5 years ago

  • Priority changed from Normal to High

#5 Updated by bcourt over 5 years ago

  • Priority changed from High to Normal

#6 Updated by bcourt over 5 years ago

  • Priority changed from Normal to High

#7 Updated by bcourt over 5 years ago

This will require reading all the Requires: lines directly from the spec file to install dependencies. If that is added to the base pulp/pulp builder as well we could perhaps avoid the uninstall core pulp portion of that builder.

#8 Updated by bcourt over 5 years ago

  • Tags Sprint Candidate added

#9 Updated by bcourt over 5 years ago

  • Description updated (diff)

#10 Updated by mhrivnak over 5 years ago

  • Priority changed from High to Normal

#11 Updated by bmbouter over 5 years ago

  • Tags Groomed added

#12 Updated by bcourt over 5 years ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to bcourt

#13 Updated by bcourt over 5 years ago

  • Status changed from ASSIGNED to CLOSED - CURRENTRELEASE
  • Sprint/Milestone set to 15
  • % Done changed from 0 to 100

#14 Updated by bmbouter over 5 years ago

  • Groomed set to Yes
  • Tags deleted (Groomed)

#15 Updated by bmbouter over 5 years ago

  • Sprint Candidate set to Yes
  • Tags deleted (Sprint Candidate)

#16 Updated by bmbouter over 2 years ago

  • Sprint set to April 2015

#17 Updated by bmbouter over 2 years ago

  • Sprint/Milestone deleted (15)

#18 Updated by bmbouter over 1 year ago

  • Tags Pulp 2 added

Please register to edit this issue

Also available in: Atom PDF