Skip to content

adding ghostbuster setup documentation#665

Open
advorkin wants to merge 1 commit intovoxpupuli:masterfrom
advorkin:ghostbuster
Open

adding ghostbuster setup documentation#665
advorkin wants to merge 1 commit intovoxpupuli:masterfrom
advorkin:ghostbuster

Conversation

@advorkin
Copy link
Copy Markdown

No description provided.

@bastelfreak
Copy link
Copy Markdown
Member

@advorkin thanks for the PR! Could you please sign the commit with an ssh or gpg key?

@@ -0,0 +1,158 @@
# puppet-ghostbuster
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please have a look at our existing posts, they all have a specific header encapsulated in three dashes: https://github.com/voxpupuli/voxpupuli.github.io/blob/master/_posts/2026-04-16-openvox-releases-at-speed-of-light.md?plain=1

That's required for the blog software to render to post correctly.

## Installation

gem install puppet-lint
gem install puppet-ghostbuster
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's enough to install puppet-gostbuster. It has a dependency to puppet-lint, so you don't have to install it explicitly.

Copy link
Copy Markdown

@m0dular m0dular left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This script is clearly an AI generated plagiarism of puppet-ghostbuster. If there are issues with the module, we should fix them in the module itself instead of AI generating a new script.

Dir.glob('./manifests/**/*.pp')

files.each do |file|
next unless file.match(%r{^\./(:?[^/]+/){2}?manifests/.+$})
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All of these blocks are just copy/pastes of the code from puppet-ghostbuster. This .match is from here https://github.com/voxpupuli/puppet-ghostbuster/blob/19b8acf7a0877c576f062ffc722bbc735822d3e5/lib/puppet-lint/plugins/check_ghostbuster_classes.rb#L19

return if path.match(%r{^\./(:?[^/]+/){2}?manifests/.+$}).nil?

PuppetLint::Data.manifest_lines = content.split("\n", -1)
PuppetLint::Data.tokens = PuppetLint::Lexer.new.tokenise(content)

PuppetLint::Data.send(:class_indexes).each do |idx|
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

Successfully merging this pull request may close these issues.

3 participants