Project

Profile

Help

Task #5810

closed

plugin-template and pulpcore should use additional_plugins var to specify the pulp_file branch

Added by mdepaulo@redhat.com over 4 years ago. Updated over 4 years ago.

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

100%

Estimated time:
Platform Release:
Groomed:
No
Sprint Candidate:
No
Tags:
CI/CD, Plugin Template
Sprint:
Quarter:

Description

When https://pulp.plan.io/issues/5782 was merged to fix CI quickly for non-master branches, we made an expedient design decision:

pulpcore (which has lots of special logic in plugin-template) used "pulpcore_branch" to specify the branch of pulp_file it wants.

We should use the new additional_plugins variable instead for the branch of pulp_file.


Related issues

Follows Pulp - Issue #5782: plugin-template needs to support both the pulpcore 3.0 and master branch for pluginsCLOSED - CURRENTRELEASEmdepaulo@redhat.comActions
Actions #1

Updated by mdepaulo@redhat.com over 4 years ago

  • Follows Issue #5782: plugin-template needs to support both the pulpcore 3.0 and master branch for plugins added

Added by Mike DePaulo over 4 years ago

Revision d9406951 | View on GitHub

Problem: plugin-template & pulpcore should use additional_plugins

to specify pulp_file's branch instead of pulpcore_branch.

Solution: Have pulpcore use additional_plugins for pulp_file, (except where hardcoding pulp_file leads to much simpler code) and do corresponding code cleanup and refactoring so that pulp_file does not get installed twice.

Also did some whitespace cleanup.

re: https://github.com/pulp/pulpcore/pull/409

Fixes: #5810 plugin-template and pulpcore should use additional_plugins var to specify the pulp_file branch https://pulp.plan.io/issues/5810

Added by Mike DePaulo over 4 years ago

Revision f4b76f24 | View on GitHub

Problem: Travis config is out of date

And I need a plugin to test the latest changes in it for regressions.

Solution: re-generate with latest plugin-template.

re: #5810 plugin-template and pulpcore should use additional_plugins var to specify the pulp_file branch https://pulp.plan.io/issues/5810 [noissue]

Added by Mike DePaulo over 4 years ago

Revision f4b76f24 | View on GitHub

Problem: Travis config is out of date

And I need a plugin to test the latest changes in it for regressions.

Solution: re-generate with latest plugin-template.

re: #5810 plugin-template and pulpcore should use additional_plugins var to specify the pulp_file branch https://pulp.plan.io/issues/5810 [noissue]

Actions #2

Updated by fao89 over 4 years ago

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

Updated by Anonymous over 4 years ago

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

Added by Mike DePaulo over 4 years ago

Revision 2a5fb16f | View on GitHub

Problem: plugin-template & pulpcore should use additional_plugins

to specify pulp_file's branch instead of pulpcore_branch.

Solution: re-generate with latest template, and update pre_before_install.sh to read additional_plugins.

re: https://github.com/pulp/pulpcore/pull/409

re: #5810 plugin-template and pulpcore should use additional_plugins var to specify the pulp_file branch https://pulp.plan.io/issues/5810

[noissue]

Added by Mike DePaulo over 4 years ago

Revision f0a8f884 | View on GitHub

Problem: plugin-template & pulpcore should use additional_plugins

to specify pulp_file's branch instead of pulpcore_branch.

Solution: re-generate with latest template, and update pre_before_install.sh to read additional_plugins.

re: https://github.com/pulp/pulpcore/pull/409

re: #5810 plugin-template and pulpcore should use additional_plugins var to specify the pulp_file branch https://pulp.plan.io/issues/5810

[noissue]

Added by Mike DePaulo over 4 years ago

Revision a023f4fe | View on GitHub

Problem: Travis config is out of date

And I need a plugin to test the latest changes in it for regressions.

Solution: re-generate with latest plugin-template.

re: #5810 plugin-template and pulpcore should use additional_plugins var to specify the pulp_file branch https://pulp.plan.io/issues/5810 [noissue]

(cherry picked from commit f4b76f2405f4b89b59ee4a35a318c0e295dd1c52)

Added by Mike DePaulo over 4 years ago

Revision a023f4fe | View on GitHub

Problem: Travis config is out of date

And I need a plugin to test the latest changes in it for regressions.

Solution: re-generate with latest plugin-template.

re: #5810 plugin-template and pulpcore should use additional_plugins var to specify the pulp_file branch https://pulp.plan.io/issues/5810 [noissue]

(cherry picked from commit f4b76f2405f4b89b59ee4a35a318c0e295dd1c52)

Actions #4

Updated by fao89 over 4 years ago

  • Tags CI/CD added
Actions #5

Updated by bmbouter over 4 years ago

  • Sprint/Milestone set to 3.0.0
Actions #6

Updated by bmbouter over 4 years ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

Also available in: Atom PDF