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

Add ENC / reports capability #79

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

Dimadozen
Copy link

Support setting up ENC and sending reports to external systems. Added supporting files for Foreman (default Foreman files).

@Dimadozen Dimadozen requested a review from a team as a code owner April 12, 2024 08:58
README.md Show resolved Hide resolved
@@ -0,0 +1,470 @@
#!/usr/bin/env ruby
Copy link
Member

Choose a reason for hiding this comment

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

I see a few problems with this:

  • the source is GPL-3.0+, we are Apache-2. I don't know if we're allowed to vendor this
  • the two files have no indication that they are coming from the foreman project
  • How do we keep track of upstream changes? Would it make more sense to pull the files in when we built the container?

Copy link
Author

@Dimadozen Dimadozen Apr 12, 2024

Choose a reason for hiding this comment

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

I see

  • I have no idea :)
  • Would it help with a disclaimer at the top of the files? (don't know if that modification of files breaks any licenses)
  • Those two files haven't changed in years, so in practice it probably won't be an issue. But I understand the concern.

Copy link
Member

Choose a reason for hiding this comment

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

maybe adding a script, that if $SOMETHING is set, it curls in the upstream files from their sources?

Copy link
Author

Choose a reason for hiding this comment

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

I run this in an air gapped environment, so that wouldn't work for me, unfortunately

Copy link
Member

Choose a reason for hiding this comment

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

can you please provide the upstrem urls from where the files come from? so that we know how/where to update, look for updates?

Copy link
Author

Choose a reason for hiding this comment

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

I picked it out from The Foreman's own git repo for their contaiers here.

I can't seem to find other official sources. It is, however, the one that's installed when you install Foreman and Puppet using the installer.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

3 participants