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

nginx package spectest failing #1190

Closed
JacobHenner opened this issue Mar 16, 2018 · 2 comments
Closed

nginx package spectest failing #1190

JacobHenner opened this issue Mar 16, 2018 · 2 comments
Labels
bug Something isn't working

Comments

@JacobHenner
Copy link
Contributor

Affected Puppet, Ruby, OS and module versions/distributions

  • Puppet: 4, 5
  • Ruby: 2.1.9, 2.4.3, 2.5.0
  • Distribution: Arch (through spectest)
  • Module version: 65b7c74

How to reproduce (e.g Puppet code you use)

Execute Travis build against master.

What are you seeing

rspec is failing, since it expects the package "nginx" to be installed on Arch Linux:

else
  it { is_expected.to contain_package('nginx') }
end

However, in params.pp, the package_name is masked by override value "nginx-mainline":

'ArchLinux': {
      $_module_os_overrides = {
        'pid'          => false,
        'daemon_user'  => 'http',
        'log_group'    => 'log',
        'package_name' => 'nginx-mainline',
      }
    }

Output log

Failures:
  1) nginx on archlinux-4-x86_64 nginx::package should contain Package[nginx]
     Failure/Error: it { is_expected.to contain_package('nginx') }
       expected that the catalogue would contain Package[nginx]
     # ./spec/classes/nginx_spec.rb:169:in `block (5 levels) in <top (required)>'
Finished in 9 minutes 11 seconds (files took 19.44 seconds to load)
3457 examples, 1 failure
Failed examples:
rspec ./spec/classes/nginx_spec.rb[1:1:2:1] # nginx on archlinux-4-x86_64 nginx::package should contain Package[nginx]
@JacobHenner
Copy link
Contributor Author

Unless I've missed something, it looks like rerunning the Travis build against the current master would result in this failure. Still looking into the cause, since Travis reports that tests of the latest commit succeeded when it was run a few days back.

@juniorsysadmin juniorsysadmin added the bug Something isn't working label Mar 16, 2018
bastelfreak added a commit that referenced this issue Mar 17, 2018
Fix #1190 Accommodate default package name nginx-mainline for Arch Linux
@bastelfreak
Copy link
Member

fixed by #1191

ceonizm pushed a commit to ceonizm/puppet-nginx that referenced this issue Jan 23, 2019
ceonizm pushed a commit to ceonizm/puppet-nginx that referenced this issue Jan 23, 2019
cegeka-jenkins pushed a commit to cegeka/puppet-nginx that referenced this issue Sep 13, 2019
Rubueno pushed a commit to Rubueno/puppet-nginx that referenced this issue Oct 19, 2020
Rubueno pushed a commit to Rubueno/puppet-nginx that referenced this issue Oct 19, 2020
Fix voxpupuli#1190 Accommodate default package name nginx-mainline for Arch Linux
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants