Skip to content

Change to allow pulp_file/certguard functional tests to work again. #177

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

ggainey
Copy link
Contributor

@ggainey ggainey commented Feb 25, 2025

No description provided.

@ggainey ggainey marked this pull request as draft February 25, 2025 18:58
@ggainey
Copy link
Contributor Author

ggainey commented Feb 25, 2025

Looks like my change works locally w/ podman - but there may be an issue w/ Docker. Draft-ing until I have time to investigate further

@gerrod3
Copy link
Contributor

gerrod3 commented Mar 28, 2025

Also, can you rebase to incorporate the CI changes we recently merged?

@ggainey ggainey force-pushed the allow_pulp_file_tests branch from fc9855d to 69be8f2 Compare May 7, 2025 20:11
@ggainey ggainey marked this pull request as ready for review May 7, 2025 20:18
@ggainey ggainey requested a review from gerrod3 May 7, 2025 20:18
@ggainey ggainey marked this pull request as draft May 7, 2025 23:38
@ggainey
Copy link
Contributor Author

ggainey commented May 7, 2025

redrafting - certguard only worked because I had old-certguard in a place run_functional could find it.

@ggainey ggainey force-pushed the allow_pulp_file_tests branch from 69be8f2 to bb86327 Compare May 8, 2025 14:24
@ggainey ggainey marked this pull request as ready for review May 8, 2025 14:25
@ggainey
Copy link
Contributor Author

ggainey commented May 8, 2025

Once more unto the breach

@ggainey ggainey changed the title Change to allow pulp_file functional tests to work again. Change to allow pulp_file/certguard functional tests to work again. May 8, 2025
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.

2 participants