Issue #6113
closed
Fix up pulp-ansible role metadata, including license string
Description
At a glance, there are the following problems with the metadata files (each role's meta/main.yml):
- They are inconsistent with eachother in terms of some fields like the name of the author, the minimum required version of ansible (they should all specify the same version), and the supported platforms (should all be the same, and represent what we test against.)
- Some refer to the license as GPLv2, others as GPLv3.
- They want us to use this list of license strings: https://spdx.org/licenses/
ansible-pulp's LICENSE would correspond to the string: GPL-2.0-or-later
This may or may not cause issues with galaxy parsing our roles when put them in a collectoin on galaxy, but it's better to correct the metadata now.
Updated by rchan almost 5 years ago
Here is another ticket capturing our fixing of license in other parts of the project: https://pulp.plan.io/issues/4592 which has links to the faq and licensing discussion. If you are seeing v3 we may need to check if that is intentional or a place we may have missed.
Updated by mdepaulo@redhat.com almost 5 years ago
These are all the license metadatas:
./pulp-content/meta/main.yml: license: GPL-3.0
./pulp-database/meta/main.yml: license: GPLv2
./pulp-devel/meta/main.yml: license: GPLv2
./pulp-redis/meta/main.yml: license: GPL-3.0
./pulp-resource-manager/meta/main.yml: license: GPL-3.0
./pulp-webserver/meta/main.yml: license: GPL-3.0
./pulp-workers/meta/main.yml: license: GPL-3.0
./pulp/meta/main.yml: license: GPLv2
"pulp" was the very 1st role, and has always been "GPLv2".
After running "git blame" some, my theory is that at one point someone accidentally created a new role specified as GPL-3.0, and it was copied into other new roles.
And the GPLv2 was never converted to GPLv2+ ("GPL-2.0-or-later").
Updated by fao89 almost 5 years ago
- Triaged changed from No to Yes
- Sprint set to Sprint 66
Added by Mike DePaulo almost 5 years ago
Added by Mike DePaulo almost 5 years ago
Revision a45411b8 | View on GitHub
Problem: role metadata is incorrect or has omissions
Solution: Update it, including the license strings: GPL-2.0 for some roles, GPL-3.0 for others -> GPL-2.0-or-later
Also remove a duplicate of copy of LICENSE (GPLv2) from the "pulp" role, which is without a COPYRIGHT file (which says GPLv2+ actually) or a COMMITMENT file.
Not addressed:
- The dependencies so that each service could run on different hosts. (Addressing them increases dynamic include runtime, and acheiving that entire end includes more work/testing.)
fixes: #6113 https://pulp.plan.io/issues/6113 Fix up pulp-ansible role metadata, including license string
Updated by Anonymous almost 5 years ago
- Status changed from NEW to MODIFIED
Applied in changeset ansible-pulp|a45411b87761452e77bebd9664a85ef82e77fa92.
Updated by Anonymous almost 5 years ago
Applied in changeset ansible-pulp3|a45411b87761452e77bebd9664a85ef82e77fa92.
Updated by mdepaulo@redhat.com over 4 years ago
- Status changed from MODIFIED to CLOSED - CURRENTRELEASE
Updated by bmbouter over 4 years ago
- Category set to Installer - Moved to GitHub issues
- Tags deleted (
Pulp 3 installer)
.
Problem: role metadata is incorrect or has omissions
Solution: Update it, including the license strings: GPL-2.0 for some roles, GPL-3.0 for others -> GPL-2.0-or-later
Also remove a duplicate of copy of LICENSE (GPLv2) from the "pulp" role, which is without a COPYRIGHT file (which says GPLv2+ actually) or a COMMITMENT file.
Not addressed:
fixes: #6113 https://pulp.plan.io/issues/6113 Fix up pulp-ansible role metadata, including license string