Skip to content

Conversation

@ekohl
Copy link
Member

@ekohl ekohl commented Dec 1, 2025

Every plugin is expected to expose a feature which means it works. This uses a shared example to test it in various places.

Every plugin is expected to expose a feature which means it works. This
uses a shared example to test it in various places.
Copy link
Member Author

@ekohl ekohl left a comment

Choose a reason for hiding this comment

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

Looks like the container gateway somehow doesn't start up and crashes, but we don't have the logs in acceptance tests.

Comment on lines +19 to +20
cert_dir = '/etc/foreman-proxy'
describe curl_command("https://#{host_inventory['fqdn']}:8443/features", cacert: "#{cert_dir}/certificate.pem", cert: "#{cert_dir}/certificate.pem", key: "#{cert_dir}/key.pem") do
Copy link
Member Author

Choose a reason for hiding this comment

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

Actually, we don't need client auth so can be simplified to:

Suggested change
cert_dir = '/etc/foreman-proxy'
describe curl_command("https://#{host_inventory['fqdn']}:8443/features", cacert: "#{cert_dir}/certificate.pem", cert: "#{cert_dir}/certificate.pem", key: "#{cert_dir}/key.pem") do
describe curl_command("https://#{host_inventory['fqdn']}:8443/features", cacert: '/etc/foreman-proxy/certificate.pem') do

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.

1 participant