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

Puppet 8 compatibility / puppet-lint #779

Open
siebrand opened this issue May 2, 2023 · 3 comments
Open

Puppet 8 compatibility / puppet-lint #779

siebrand opened this issue May 2, 2023 · 3 comments

Comments

@siebrand
Copy link
Contributor

siebrand commented May 2, 2023

puppet-lint issues in the module -- not all are puppet 8 blockers:

./datadog_agent/manifests/init.pp - WARNING: legacy fact 'operatingsystem' on line 435 (check: legacy_facts)
./datadog_agent/manifests/init.pp - WARNING: legacy fact 'operatingsystem' on line 487 (check: legacy_facts)
./datadog_agent/manifests/init.pp - WARNING: legacy fact 'operatingsystem' on line 506 (check: legacy_facts)
./datadog_agent/manifests/init.pp - WARNING: legacy fact 'operatingsystem' on line 526 (check: legacy_facts)
./datadog_agent/manifests/init.pp - WARNING: legacy fact 'operatingsystem' on line 762 (check: legacy_facts)
./datadog_agent/manifests/init.pp - WARNING: top scope fact instead of facts hash on line 435 (check: top_scope_facts)
./datadog_agent/manifests/init.pp - WARNING: top scope fact instead of facts hash on line 487 (check: top_scope_facts)
./datadog_agent/manifests/init.pp - WARNING: top scope fact instead of facts hash on line 506 (check: top_scope_facts)
./datadog_agent/manifests/init.pp - WARNING: top scope fact instead of facts hash on line 526 (check: top_scope_facts)
./datadog_agent/manifests/init.pp - WARNING: top scope fact instead of facts hash on line 762 (check: top_scope_facts)
./datadog_agent/manifests/install_integration.pp - WARNING: defined type not documented on line 1 (check: documentation)
./datadog_agent/manifests/integration.pp - WARNING: defined type not documented on line 1 (check: documentation)
./datadog_agent/manifests/integrations/directory.pp - WARNING: class not documented on line 57 (check: documentation)
./datadog_agent/manifests/integrations/directory.pp - WARNING: line has more than 140 characters on line 14 (check: 140chars)
./datadog_agent/manifests/integrations/directory.pp - WARNING: line has more than 140 characters on line 17 (check: 140chars)
./datadog_agent/manifests/integrations/directory.pp - WARNING: line has more than 140 characters on line 20 (check: 140chars)
./datadog_agent/manifests/integrations/directory.pp - WARNING: line has more than 140 characters on line 23 (check: 140chars)
./datadog_agent/manifests/integrations/directory.pp - WARNING: line has more than 140 characters on line 29 (check: 140chars)
./datadog_agent/manifests/integrations/haproxy.pp - WARNING: legacy fact 'ipaddress' on line 21 (check: legacy_facts)
./datadog_agent/manifests/integrations/haproxy.pp - WARNING: top scope fact instead of facts hash on line 21 (check: top_scope_facts)
./datadog_agent/manifests/integrations/http_check.pp - WARNING: class not documented on line 170 (check: documentation)
./datadog_agent/manifests/integrations/linux_proc_extras.pp - WARNING: class not documented on line 16 (check: documentation)
./datadog_agent/manifests/integrations/logs.pp - WARNING: class not documented on line 39 (check: documentation)
./datadog_agent/manifests/integrations/nginx.pp - WARNING: class not documented on line 62 (check: documentation)
./datadog_agent/manifests/integrations/ntp.pp - WARNING: class not documented on line 26 (check: documentation)
./datadog_agent/manifests/integrations/oom_kill.pp - WARNING: class not documented on line 22 (check: documentation)
./datadog_agent/manifests/integrations/php_fpm.pp - WARNING: class not documented on line 29 (check: documentation)
./datadog_agent/manifests/integrations/process.pp - WARNING: line has more than 140 characters on line 51 (check: 140chars)
./datadog_agent/manifests/integrations/rabbitmq.pp - WARNING: class not documented on line 53 (check: documentation)
./datadog_agent/manifests/integrations/snmp.pp - WARNING: class not documented on line 54 (check: documentation)
./datadog_agent/manifests/integrations/ssh.pp - WARNING: class not documented on line 29 (check: documentation)
./datadog_agent/manifests/integrations/ssh.pp - WARNING: legacy fact 'fqdn' on line 30 (check: legacy_facts)
./datadog_agent/manifests/integrations/ssh.pp - WARNING: top scope fact instead of facts hash on line 30 (check: top_scope_facts)
./datadog_agent/manifests/integrations/supervisord.pp - WARNING: class not documented on line 44 (check: documentation)
./datadog_agent/manifests/integrations/tcp_check.pp - WARNING: class not documented on line 88 (check: documentation)
./datadog_agent/manifests/params.pp - WARNING: class not documented on line 6 (check: documentation)
./datadog_agent/manifests/params.pp - WARNING: line has more than 140 characters on line 56 (check: 140chars)
./datadog_agent/manifests/params.pp - WARNING: legacy fact 'operatingsystem' on line 26 (check: legacy_facts)
./datadog_agent/manifests/params.pp - WARNING: legacy fact 'operatingsystem' on line 68 (check: legacy_facts)
./datadog_agent/manifests/params.pp - WARNING: top scope fact instead of facts hash on line 26 (check: top_scope_facts)
./datadog_agent/manifests/params.pp - WARNING: top scope fact instead of facts hash on line 68 (check: top_scope_facts)
./datadog_agent/manifests/redhat.pp - WARNING: class not documented on line 6 (check: documentation)
./datadog_agent/manifests/redhat.pp - WARNING: legacy fact 'operatingsystem' on line 27 (check: legacy_facts)
./datadog_agent/manifests/redhat.pp - WARNING: legacy fact 'operatingsystemrelease' on line 30 (check: legacy_facts)
./datadog_agent/manifests/redhat.pp - WARNING: legacy fact 'architecture' on line 48 (check: legacy_facts)
./datadog_agent/manifests/redhat.pp - WARNING: legacy fact 'architecture' on line 52 (check: legacy_facts)
./datadog_agent/manifests/redhat.pp - WARNING: legacy fact 'architecture' on line 56 (check: legacy_facts)
./datadog_agent/manifests/redhat.pp - WARNING: top scope fact instead of facts hash on line 27 (check: top_scope_facts)
./datadog_agent/manifests/redhat.pp - WARNING: top scope fact instead of facts hash on line 30 (check: top_scope_facts)
./datadog_agent/manifests/redhat.pp - WARNING: top scope fact instead of facts hash on line 48 (check: top_scope_facts)
./datadog_agent/manifests/redhat.pp - WARNING: top scope fact instead of facts hash on line 52 (check: top_scope_facts)
./datadog_agent/manifests/redhat.pp - WARNING: top scope fact instead of facts hash on line 56 (check: top_scope_facts)
./datadog_agent/manifests/reports.pp - WARNING: legacy fact 'operatingsystem' on line 32 (check: legacy_facts)
./datadog_agent/manifests/reports.pp - WARNING: top scope fact instead of facts hash on line 32 (check: top_scope_facts)
./datadog_agent/manifests/security_agent.pp - WARNING: class not documented on line 1 (check: documentation)
./datadog_agent/manifests/security_agent.pp - WARNING: legacy fact 'operatingsystem' on line 18 (check: legacy_facts)
./datadog_agent/manifests/security_agent.pp - WARNING: top scope fact instead of facts hash on line 18 (check: top_scope_facts)
./datadog_agent/manifests/service.pp - WARNING: class not documented on line 6 (check: documentation)
./datadog_agent/manifests/service.pp - WARNING: legacy fact 'operatingsystem' on line 13 (check: legacy_facts)
./datadog_agent/manifests/service.pp - WARNING: top scope fact instead of facts hash on line 13 (check: top_scope_facts)
./datadog_agent/manifests/suse.pp - WARNING: class not documented on line 6 (check: documentation)
./datadog_agent/manifests/suse.pp - WARNING: legacy fact 'architecture' on line 42 (check: legacy_facts)
./datadog_agent/manifests/suse.pp - WARNING: legacy fact 'operatingsystemmajrelease' on line 80 (check: legacy_facts)
./datadog_agent/manifests/suse.pp - WARNING: top scope fact instead of facts hash on line 42 (check: top_scope_facts)
./datadog_agent/manifests/suse.pp - WARNING: top scope fact instead of facts hash on line 80 (check: top_scope_facts)
./datadog_agent/manifests/system_probe.pp - WARNING: class not documented on line 8 (check: documentation)
./datadog_agent/manifests/system_probe.pp - WARNING: legacy fact 'operatingsystem' on line 35 (check: legacy_facts)
./datadog_agent/manifests/system_probe.pp - WARNING: top scope fact instead of facts hash on line 35 (check: top_scope_facts)
./datadog_agent/manifests/ubuntu.pp - WARNING: class not documented on line 6 (check: documentation)
./datadog_agent/manifests/ubuntu.pp - WARNING: legacy fact 'operatingsystem' on line 64 (check: legacy_facts)
./datadog_agent/manifests/ubuntu.pp - WARNING: legacy fact 'operatingsystemrelease' on line 64 (check: legacy_facts)
./datadog_agent/manifests/ubuntu.pp - WARNING: legacy fact 'operatingsystem' on line 65 (check: legacy_facts)
./datadog_agent/manifests/ubuntu.pp - WARNING: legacy fact 'operatingsystemrelease' on line 65 (check: legacy_facts)
./datadog_agent/manifests/ubuntu.pp - WARNING: top scope fact instead of facts hash on line 64 (check: top_scope_facts)
./datadog_agent/manifests/ubuntu.pp - WARNING: top scope fact instead of facts hash on line 64 (check: top_scope_facts)
./datadog_agent/manifests/ubuntu.pp - WARNING: top scope fact instead of facts hash on line 65 (check: top_scope_facts)
./datadog_agent/manifests/ubuntu.pp - WARNING: top scope fact instead of facts hash on line 65 (check: top_scope_facts)
./datadog_agent/manifests/windows.pp - WARNING: class not documented on line 6 (check: documentation)
./datadog_agent/manifests/windows.pp - WARNING: line has more than 140 characters on line 48 (check: 140chars)
@flepoutre
Copy link

Hi,
Any news on this issue ?
Thanks

@mjprinvale
Copy link

mjprinvale commented Dec 29, 2023

Also looking for updates on this.

@sa-dbilling
Copy link

Any idea when Puppet 8 support will be added?

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

4 participants