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

Error: Could not parse application options: invalid option: --to_yaml #118

Closed
sathieu opened this issue Nov 24, 2017 · 6 comments · Fixed by #119
Closed

Error: Could not parse application options: invalid option: --to_yaml #118

sathieu opened this issue Nov 24, 2017 · 6 comments · Fixed by #119
Labels
needs-feedback Further information is requested

Comments

@sathieu
Copy link

sathieu commented Nov 24, 2017

Affected Puppet, Ruby, OS and module versions/distributions

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

Use older Puppet agent (we use Puppet master 4.8.2).

What are you seeing

[...]
Info: Loading facts
Error: Could not parse application options: invalid option: --to_yaml
Could not retrieve fact='prometheus_alert_manager_running', resolution='<anonymous>': undefined method `[]' for false:FalseClass
Info: Caching catalog for mynode
[...]

What behaviour did you expect instead

No warning

Output log

Any additional information you'd like to impart

@tuxmea tuxmea added the bug Something isn't working label Nov 24, 2017
@tuxmea
Copy link
Member

tuxmea commented Nov 24, 2017

Puppet: 3.8.7 is no longer supported. check metadata.json:
"requirements": [ { "name": "puppet", "version_requirements": "=> 4.7.1 < 6.0.0" } ],

@tuxmea tuxmea added needs-feedback Further information is requested and removed bug Something isn't working labels Nov 24, 2017
@tuxmea
Copy link
Member

tuxmea commented Nov 24, 2017

Can you please check that your agent is really running 3.8.7? That version is very outdated.
Additionally you mentioned puppet master 4.8.2.
The behavior you are describing comes from a fact. Facts are gathered on the agent and are unrelated to the master.
Can you update your agent to also run on 4.8.2?

@sathieu
Copy link
Author

sathieu commented Nov 24, 2017

I known that agent 3.x is not supported. but it was working before.

I can't upgrade the agent to 4.x.

@tuxmea
Copy link
Member

tuxmea commented Nov 24, 2017

I suppose that you have to stay with older version of this module which is working on puppet 3. You can not use master branch as this is puppet 4 and later only. @bastelfreak how about your opinion?

@sathieu
Copy link
Author

sathieu commented Nov 24, 2017

The node with agent 3.x is not the one with prometheus installed. Having this fact executed on all nodes is unecessary I think.

tuxmea pushed a commit to tuxmea/puppet-prometheus that referenced this issue Nov 24, 2017
@tuxmea
Copy link
Member

tuxmea commented Nov 24, 2017

@sathieu can you please check #119 and verify whether this version of the fact fixes your issue?

cegeka-jenkins pushed a commit to cegeka/puppet-prometheus that referenced this issue Aug 28, 2019
Rovanion pushed a commit to Rovanion/puppet-prometheus that referenced this issue May 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs-feedback Further information is requested
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants