Project

Profile

Help

Actions
Send by e-mail Copy link

Task #3530

closed

Enable Travis to test pulp 3.0-dev PRs against pulp_file PRs and pulp-smash PRs

Added by dkliban@redhat.com over 6 years ago. Updated almost 5 years ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
Normal
Assignee:
dkliban@redhat.com
Category:
-
Sprint/Milestone:
3.0.0
Start date:
Due date:
% Done:

100%

Estimated time:
Platform Release:
Groomed:
Yes
Sprint Candidate:
Yes
Tags:
Sprint:
Sprint 35
Quarter:

Description

Each PR against 3.0-dev branch of pulp is tested using Travis. Travis is configured to run 2 blocking checks and 1 non-blocking check.

Blocking checks: pulpcore unit tests, pulp-smash tests for pulpcore
Non-blocking checks: pulp-smash tests for pulp_file

Once this task is completed the non-blocking check would become blocking. However, authors would have the ability to specify a URL for corresponding PRs in pulp-smash and pulp_file.

Any PRs that break pulp-smash tests and/or modify the plugin API would require the author to work with other devs/qe to update pulp_file and pulp-smash before their changes could be merged. Once the needed PRs are in place the author will add the following message for each PR to the original commit message:

Requires PR: <link to PR>

This change ensures that we always keep pulp-smash, pulpcore, and pulp_file compatible with each other.

Actions
Copy link
 #1

Updated by bmbouter over 6 years ago

I think we need this because we always want the merged code of pulp and pulp_file to pass when run with the latest pulp-smash. When change in pulp and pulp_file happen together we need a way to handle that.

+1 I think this is good to be groomed.

Actions
Copy link
 #2

Updated by dkliban@redhat.com over 6 years ago

  • Tracker changed from Issue to Task
  • % Done set to 0
Actions
Copy link
 #3

Updated by daviddavis over 6 years ago

  • Groomed changed from No to Yes

Setting to groomed. I'd like to discuss the commit message format though so maybe ask for a few people's opinions on this.

Actions
Copy link
 #4

Updated by dkliban@redhat.com over 6 years ago

  • Description updated (diff)
Actions
Copy link
 #5

Updated by daviddavis over 6 years ago

  • Sprint set to Sprint 35
Actions
Copy link
 #6

Updated by dkliban@redhat.com over 6 years ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to dkliban@redhat.com
Actions
Copy link
 #7

Updated by dkliban@redhat.com over 6 years ago

  • Status changed from ASSIGNED to POST

Added by dkliban@redhat.com over 6 years ago

Revision 5c60ba1d | View on GitHub

Require PRs from pulp_file and pulp-smash

The PRs listed in this commit message are just for demonstrating that Travis will checkout those PRs when testing the PR that contains this commit.

closes #3530 https://pulp.plan.io/issues/3530

Required PR: https://github.com/pulp/pulp_file/pull/65 Required PR: https://github.com/PulpQE/pulp-smash/pull/770

Added by dkliban@redhat.com over 6 years ago

Revision 5c60ba1d | View on GitHub

Require PRs from pulp_file and pulp-smash

The PRs listed in this commit message are just for demonstrating that Travis will checkout those PRs when testing the PR that contains this commit.

closes #3530 https://pulp.plan.io/issues/3530

Required PR: https://github.com/pulp/pulp_file/pull/65 Required PR: https://github.com/PulpQE/pulp-smash/pull/770

Actions
Copy link
 #8

Updated by dkliban@redhat.com over 6 years ago

  • Status changed from POST to MODIFIED
  • % Done changed from 0 to 100
Actions
Copy link
 #9

Updated by dkliban@redhat.com over 6 years ago

  • Sprint/Milestone set to 3.0.0
Actions
Copy link
 #10

Updated by bmbouter over 5 years ago

  • Tags deleted (Pulp 3, Pulp 3 MVP)
Actions
Copy link
 #11

Updated by bmbouter almost 5 years ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE
Actions
Send by e-mail Copy link

Also available in: Atom PDF