Skip to content

some repositories need $repos to be empty value but limit doesn't allow for that #1225

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

Open
XANi opened this issue Mar 4, 2025 · 3 comments

Comments

@XANi
Copy link

XANi commented Mar 4, 2025

Describe the Bug

    qownnotes:
        comment: QOwnNotes
        location: http://download.opensuse.org/repositories/home:/pbek:/QOwnNotes/Debian_11/
        release: /
        repos: ""

stopped working. This particular package repository doesn't need/use release/repos at all:

http://download.opensuse.org/repositories/home:/pbek:/QOwnNotes/Debian_11/
https://www.qownnotes.org/installation/debian.html

and similar problem for

https://pkg.origin.jenkins.io/debian/

APT has no problems with it

deb [arch=${ARCHITECTURE} signed-by=${SIGNED_BY}] http://download.opensuse.org/repositories/home:/pbek:/QOwnNotes/Debian_12/ /

and it worked in previous versions, but in 10.0.0/10.0/1 just gives me

Error: Evaluation Error: Error while evaluating a Resource Statement, Apt::Source[qownnotes]: parameter 'repos' expects a value of type String[1] or Array[String[1]], got String (file: /var/lib/dpp/repos/public/modules/core/manifests/init.pp, line: 34) 

passing string containing one space works as workaround

@kenyon
Copy link

kenyon commented Mar 5, 2025

Same as #1132?

@XANi
Copy link
Author

XANi commented Mar 5, 2025

Kinda. I think it should still be required but allow empty value, because not having repos is pretty rare, so there is a bigger chance of "someone forgot to add repos" than "they did that on purpose because repos isn't needed"

@lroehrs
Copy link

lroehrs commented Apr 24, 2025

Also got this bug for:

https://github.com/voxpupuli/puppet-proxysql/blob/master/manifests/repo.pp#L20

Error: Could not retrieve catalog from remote server: Error 500 on SERVER: Server Error: Evaluation Error: Error while evaluating a Resource Statement, Apt::Source[proxysql_repo]: parameter 'repos' expects a value of type String[1] or Array[String[1]], got String (file: /opt/puppetlabs/puppet/modules/proxysql/manifests/repo.pp, line: 24) on node foo

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

No branches or pull requests

3 participants