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

Improper "unquoted node name found" #904

Closed
cyberious opened this issue Nov 7, 2019 · 2 comments
Closed

Improper "unquoted node name found" #904

cyberious opened this issue Nov 7, 2019 · 2 comments

Comments

@cyberious
Copy link

cyberious commented Nov 7, 2019

I have a function that is used for ExecutionResult sets and checks them... and I am getting unquoted node name found b/c the function is written in puppet.

Exepectation is to either ignore these in a function or fix the issue with regards to this in a function.

functions/process_result.pp:57:38: unquoted node name found

// offending declaration is reported as the `.ok` 
if ($abort_on_errors and !$results.ok) { 
    alert(@(EOT))
      Aborting plan due to task failure
    | EOT
    pe::exit(1)
  }```
@usev6
Copy link
Contributor

usev6 commented Nov 15, 2019

I'm not able to reproduce this problem with the code you pasted (neither on HEAD nor with one of the recent releases).

$ cat gh/manifests/issue904.pp
## gh issue 904
class gh::issue904 {
  if ($abort_on_errors and !$results.ok) {
    alert(@(EOT))
      Aborting plan due to task failure
    | EOT
    pe::exit(1)
  }
}
$ puppet-lint gh/manifests/issue904.pp 
WARNING: top-scope variable being used without an explicit namespace on line 3
WARNING: top-scope variable being used without an explicit namespace on line 3

@cyberious could you maybe provide a more complete example that shows the failure mode? Also, what version of puppet-lint do you use?

@rodjek
Copy link
Owner

rodjek commented Feb 8, 2020

Closing for now, please feel free to reopen again with more information.

@rodjek rodjek closed this as completed Feb 8, 2020
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

3 participants