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

Restructure installation and add a pulp scenario #822

Draft
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

ekohl
Copy link
Member

@ekohl ekohl commented Dec 2, 2022

Heavy draft, but at a basic level it works. What doesn't:

  • Kafo can't parse pulpcore's types/logger.pp (https://projects.theforeman.org/issues/21398) - workaround is to join it all on one line
  • Kafo interprets functions literally: extlib::cache_data() and worker count calculation is put as the answer - workaround is pass it as installer parameters
  • There is no certificate setup: it assumes /etc/pki/tls/certs/localhost.crt and /etc/pki/tls/private/localhost.key exist - workaround is to run openssl req -x509 -nodes -days 365 -newkey rsa:2048 -keyout /etc/pki/tls/private/localhost.key -out /etc/pki/tls/certs/localhost.crt prior
  • Requires a packaging change

Additionally: various parameters should be marked as advanced but currently aren't. This makes --help less useful.

This introduces a standalone Pulp scenario. All plugins are exposed as
optional.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants