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

Set add_mocked_facts #693

Merged
merged 1 commit into from
Aug 1, 2022
Merged

Set add_mocked_facts #693

merged 1 commit into from
Aug 1, 2022

Conversation

ekohl
Copy link
Member

@ekohl ekohl commented Dec 14, 2020

This uses the new add_mocked_facts from voxpupuli-test, which is currently only a PR. That's why this is a draft.

@ekohl ekohl marked this pull request as ready for review February 23, 2021 13:20
@ekohl
Copy link
Member Author

ekohl commented Feb 23, 2021

Question is: should this be on by default?

@smortex
Copy link
Member

smortex commented Jun 22, 2022

Question is: should this be on by default?

Just hit this woking with a module that want to use the postgresql module that wants to use the $service_provider fact form stdlib. Seems a good idea 😀

@ekohl
Copy link
Member Author

ekohl commented Jun 22, 2022

I just went through theforeman/foreman-installer-modulesync#148 (or really, PRs are still open) where I did the same thing. I completely forgot I already submitted this PR. I'll rebase it tomorrow.

And I'm sure @evgeni will appreciate this, or laugh at me.

This uses the new add_mocked_facts from voxpupuli-test, which is
currently only a PR.
@ekohl
Copy link
Member Author

ekohl commented Jul 18, 2022

Rebased to resolve conflicts. When we merge this and modulesync we should go over the modules to purge any service_provider fact that may be defined to reduce duplicated code.

@ekohl ekohl merged commit 0ded33a into voxpupuli:master Aug 1, 2022
@ekohl ekohl deleted the facts branch August 1, 2022 20:38
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.

3 participants