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

Hiera: Resource Statement, Nginx::Resource::Upstream[radius]: has no parameter named 'upstream_context' #1316

Closed
linuxmail opened this issue Feb 25, 2019 · 1 comment
Labels
docs Improvements or additions to documentation skip-changelog

Comments

@linuxmail
Copy link

Affected Puppet, Ruby, OS and module versions/distributions

  • Puppet: 5.5.10-1stretch
  • Ruby: 2.3.3p222
  • Distribution: Debian Stretch (9.5)
  • Module version: 0.16.0

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

nginx::nginx_streamhosts:
  'radius':
    ensure: 'present'
    listen_port: 1812
    listen_options: 'udp'
    proxy: 'radius'
    proxy_read_timeout: '1'
    proxy_connect_timeout: '1'
    raw_append:
      - 'error_log off;'
nginx::nginx_upstreams:
  'radius':
   upstream_context: 'stream'
    members:
      '10.0.0.1:1812'
        server: '10.0.0.1'
        port: '1812'
      '10.0.0.2:1812'
        server: '10.0.0.2'
        port: '1812'

What are you seeing

Error: Could not retrieve catalog from remote server: Error 500 on SERVER: Server Error: Evaluation Error: Error while evaluating a Resource Statement, Nginx::Resource::Upstream[radius]: has no parameter named 'upstream_context' (file: /opt/puppet/environments/newmod/modules/nginx/manifests/init.pp, line: 202) on node fra-test-lbint-01.example.local

What behaviour did you expect instead

No errors, like in the documentation.

@alexjfisher
Copy link
Member

Looks like the parameter name was changed as part of #1233 but the README wasn't updated.

alexjfisher added a commit to alexjfisher/puppet-nginx that referenced this issue Feb 26, 2019
voxpupuli#1233 was a breaking
change that amongst other things, renamed this parameter.

Fixes voxpupuli#1316
@alexjfisher alexjfisher added docs Improvements or additions to documentation skip-changelog labels Feb 26, 2019
cegeka-jenkins pushed a commit to cegeka/puppet-nginx that referenced this issue Sep 13, 2019
voxpupuli#1233 was a breaking
change that amongst other things, renamed this parameter.

Fixes voxpupuli#1316
Rubueno pushed a commit to Rubueno/puppet-nginx that referenced this issue Oct 19, 2020
voxpupuli#1233 was a breaking
change that amongst other things, renamed this parameter.

Fixes voxpupuli#1316
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Improvements or additions to documentation skip-changelog
Projects
None yet
Development

No branches or pull requests

2 participants