Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Centos 7 - Bad yumrepo file names #224

Open
oniGino opened this issue Oct 16, 2021 · 3 comments
Open

Centos 7 - Bad yumrepo file names #224

oniGino opened this issue Oct 16, 2021 · 3 comments

Comments

@oniGino
Copy link

oniGino commented Oct 16, 2021

When working with Centos 7 and the latest stable of this module it installs for example base-debuginfo.repo however this breaks yum as it is a duplicate of a different builtin file in Centos which is named CentOS-Debuginfo.repo

Here are the correct repo files names from upstream centos

ViaWest/gino.mccarty@fordmariadb1-3:/etc/yum.repos.d$ rpm -ql centos-release
/etc/centos-release
/etc/centos-release-upstream
/etc/issue
/etc/issue.net
/etc/os-release
/etc/pki/rpm-gpg
/etc/pki/rpm-gpg/RPM-GPG-KEY-CentOS-7
/etc/pki/rpm-gpg/RPM-GPG-KEY-CentOS-Debug-7
/etc/pki/rpm-gpg/RPM-GPG-KEY-CentOS-Testing-7
/etc/redhat-release
/etc/rpm/macros.dist
/etc/system-release
/etc/system-release-cpe
/etc/yum.repos.d/CentOS-Base.repo
/etc/yum.repos.d/CentOS-CR.repo
/etc/yum.repos.d/CentOS-Debuginfo.repo
/etc/yum.repos.d/CentOS-Media.repo
/etc/yum.repos.d/CentOS-Sources.repo
/etc/yum.repos.d/CentOS-Vault.repo
/etc/yum.repos.d/CentOS-fasttrack.repo
/etc/yum.repos.d/CentOS-x86_64-kernel.repo
/etc/yum/vars/contentdir
/etc/yum/vars/infra
/usr/lib/os-release
/usr/lib/systemd/system-preset/85-display-manager.preset
/usr/lib/systemd/system-preset/90-default.preset
/usr/share/centos-release/EULA
/usr/share/doc/centos-release/Contributors
/usr/share/doc/centos-release/GPL
/usr/share/doc/redhat-release
/usr/share/redhat-release

Please adjust your module to install the same file names so it doesn't break yum with duplicate repos.

This was referenced Oct 26, 2021
@bastelfreak
Copy link
Member

I think that's solved with #225. please let me know if it's still an issue.

@oniGino
Copy link
Author

oniGino commented Oct 26, 2021

Confirmed, thank you for your quick response!

Yes this repos work is done, sadly upstream is still broken and ignores the target value

see

  newparam(:target) do
    desc 'The target parameter will be enabled in a future release and should not be used.'

    defaultto :absent
  end

In https://github.com/puppetlabs/puppetlabs-yumrepo_core/blob/main/lib/puppet/type/yumrepo.rb

So until target is honored the bug persists as the incorrect file is created for each repository

@oniGino
Copy link
Author

oniGino commented Oct 26, 2021

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants