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 proxy client tests #1409

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

ehelms
Copy link
Member

@ehelms ehelms commented Sep 6, 2021

This re-uses the existing client tests but runs them separately against a proxy to ensure that any changes in the mechanisms on the proxy are warded against regressions. This requires the ordering to a bit different as we need to ensure the Test environment is synced on the proxy and that hammer is configured. The current setup of hammer on the proxy is admittedly a smidge ugly. The installer scenario for foreman-proxy-content and others does not have the cli classes available to it for configuration through the installer. I could implement a hammer CLI role to capture this configuration.

My goal was to get as quickly as I could a test online for clients against a proxy with content and some initial work.

By using rpm to install the consumer RPM dependencies are not resolved
and the tests can end up in a bad state of available dependencies such
as subscription-manager which is removed when older RPMs are cleaned up.
@ehelms ehelms force-pushed the add-proxy-client-tests branch from 75ed2ff to 7cab41f Compare September 6, 2021 15:25
@ehelms ehelms requested a review from evgeni September 8, 2021 19:33
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