Project

Profile

Help

Task #3530

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

Added by dkliban@redhat.com over 1 year ago. Updated 6 months ago.

Status:
MODIFIED
Priority:
Normal
Category:
-
Sprint/Milestone:
Start date:
Due date:
% Done:

100%

Platform Release:
Blocks Release:
Backwards Incompatible:
No
Groomed:
Yes
Sprint Candidate:
Yes
Tags:
QA Contact:
Complexity:
Smash Test:
Verified:
No
Verification Required:
No
Sprint:
Sprint 35

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.


Checklist

Associated revisions

Revision 5c60ba1d View on GitHub
Added by dkliban@redhat.com over 1 year ago

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

Revision 5c60ba1d View on GitHub
Added by dkliban@redhat.com over 1 year ago

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

Revision 5c60ba1d View on GitHub
Added by dkliban@redhat.com over 1 year ago

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

History

#1 Updated by bmbouter over 1 year 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.

#2 Updated by dkliban@redhat.com over 1 year ago

  • Tracker changed from Issue to Task
  • % Done set to 0

#3 Updated by daviddavis over 1 year 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.

#4 Updated by dkliban@redhat.com over 1 year ago

  • Description updated (diff)

#5 Updated by daviddavis over 1 year ago

  • Sprint set to Sprint 35

#6 Updated by dkliban@redhat.com over 1 year ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to dkliban@redhat.com

#7 Updated by dkliban@redhat.com over 1 year ago

  • Status changed from ASSIGNED to POST

#8 Updated by dkliban@redhat.com over 1 year ago

  • Status changed from POST to MODIFIED
  • % Done changed from 0 to 100

#9 Updated by dkliban@redhat.com over 1 year ago

  • Sprint/Milestone set to 3.0

#10 Updated by bmbouter 6 months ago

  • Tags deleted (Pulp 3, Pulp 3 MVP)

Please register to edit this issue

Also available in: Atom PDF