Project

Profile

Help

Task #7411

Story #7416: [Epic] As a plugin user, I can use the next verison of pulpcore without upgrading my plugin

Add a new "test existing plugin against next pulpcore" entry to the CI matrix in plugin_template

Added by bmbouter about 2 months ago. Updated 8 days ago.

Status:
POST
Priority:
High
Category:
-
Start date:
Due date:
% Done:

0%

Estimated time:
Platform Release:
Groomed:
No
Sprint Candidate:
No
Tags:
CI/CD
Sprint:
Sprint 84
Quarter:

Description

Background

https://www.redhat.com/archives/pulp-dev/2020-August/msg00065.html

To Do

The plugin_template needs an additional matrix test that tests the current GA of a plugin against pulpcore master. This captures the "one ahead" situation where a user is using a newer version of pulpcore and the "one back" version of that plugin.

History

#1 Updated by bmbouter about 2 months ago

  • Parent task set to #7416

#2 Updated by daviddavis about 2 months ago

  • Sprint set to Sprint 80

#3 Updated by rchan about 2 months ago

  • Sprint changed from Sprint 80 to Sprint 81

#4 Updated by bmbouter about 1 month ago

  • Sprint/Milestone set to 3.7.0

#5 Updated by bmbouter about 1 month ago

  • Priority changed from Normal to High

#6 Updated by mdellweg about 1 month ago

I am a bit unsure, what the consequence of this test failing should be. As the breaking change here will most likely be introduced by pulpcore, does it mean the corresponding change (in pulpcore) must be reverted and delayed to the next minor release? Who is seeing the failure and reporting that pulpcore didn't follow the deprecation policy?

Is this test supposed to run with all pipelines, or only nightly?

#7 Updated by bmbouter about 1 month ago

mdellweg wrote:

I am a bit unsure, what the consequence of this test failing should be. As the breaking change here will most likely be introduced by pulpcore, does it mean the corresponding change (in pulpcore) must be reverted and delayed to the next minor release? Who is seeing the failure and reporting that pulpcore didn't follow the deprecation policy?

Is this test supposed to run with all pipelines, or only nightly?

It's up for debate, but here's my take. This check would be in the nightly of each plugin. It can only catch errors where pulp/pulpcore:master has already merged the problem. Then when it fails, the plugin writer needs to let the core team saying, "hey you broke me please revert on master".

It's unfortunate that this can only occur post-pulpcore-merge, but the other option is running every plugin's code against every pulpcore change which I believe is infeasible. Since the CI check in the plugin isn't checking changes in the plugin's code itself, I don't see running it pre-merge (versus cron) as productive. This leads me to the conclusion that this should be a nightly cron CI run in plugins.

#8 Updated by dkliban@redhat.com about 1 month ago

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

#9 Updated by bmbouter about 1 month ago

  • Sprint/Milestone deleted (3.7.0)

#10 Updated by rchan about 1 month ago

  • Sprint changed from Sprint 81 to Sprint 82

#11 Updated by pulpbot about 1 month ago

  • Status changed from ASSIGNED to POST

#13 Updated by rchan 22 days ago

  • Sprint changed from Sprint 82 to Sprint 83

#14 Updated by rchan 8 days ago

  • Sprint changed from Sprint 83 to Sprint 84

Please register to edit this issue

Also available in: Atom PDF