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

feat: Import code for fapolicyd system role #1

Merged
merged 21 commits into from
Nov 16, 2023
Merged

feat: Import code for fapolicyd system role #1

merged 21 commits into from
Nov 16, 2023

Conversation

radosroka
Copy link
Collaborator

No description provided.

@richm richm changed the title Import code for fapolicyd system role feat: Import code for fapolicyd system role Oct 24, 2023
.github/CODEOWNERS Outdated Show resolved Hide resolved
examples/simple.yml Outdated Show resolved Hide resolved
.ansible-lint Outdated Show resolved Hide resolved
examples/inventory Outdated Show resolved Hide resolved
examples/simple.yml Outdated Show resolved Hide resolved
handlers/main.yml Outdated Show resolved Hide resolved
handlers/main.yml Outdated Show resolved Hide resolved
tasks/enable.yml Outdated Show resolved Hide resolved
tasks/enable.yml Outdated Show resolved Hide resolved
tasks/enable.yml Outdated Show resolved Hide resolved
tasks/enable.yml Outdated Show resolved Hide resolved
tasks/enable.yml Outdated Show resolved Hide resolved
tasks/enable.yml Outdated Show resolved Hide resolved
tasks/enable.yml Outdated Show resolved Hide resolved
tasks/enable.yml Outdated Show resolved Hide resolved
tasks/enable.yml Outdated Show resolved Hide resolved
.ansible-lint Show resolved Hide resolved
Signed-off-by: Radovan Sroka <[email protected]>
tasks/enable.yml Outdated Show resolved Hide resolved
defaults/main.yml Outdated Show resolved Hide resolved
tasks/enable.yml Outdated Show resolved Hide resolved
tasks/enable.yml Outdated Show resolved Hide resolved
tasks/enable.yml Outdated Show resolved Hide resolved
tasks/enable.yml Outdated Show resolved Hide resolved
tasks/enable.yml Outdated Show resolved Hide resolved
tasks/enable.yml Outdated Show resolved Hide resolved
Signed-off-by: Radovan Sroka <[email protected]>
Copy link
Contributor

@richm richm left a comment

Choose a reason for hiding this comment

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

lgtm
next - we will need some tests
@radosroka do you want to add tests to this PR, or create additional PRs?

tasks/main.yml Outdated Show resolved Hide resolved
tasks/main.yml Show resolved Hide resolved
tasks/main.yml Show resolved Hide resolved
tasks/main.yml Show resolved Hide resolved
tasks/main.yml Show resolved Hide resolved
tasks/main.yml Show resolved Hide resolved
tasks/main.yml Show resolved Hide resolved
tasks/main.yml Show resolved Hide resolved
templates/fapolicyd.conf.j2 Show resolved Hide resolved
templates/fapolicyd.conf.j2 Show resolved Hide resolved
Signed-off-by: Radovan Sroka <[email protected]>
@radosroka
Copy link
Collaborator Author

lgtm next - we will need some tests @radosroka do you want to add tests to this PR, or create additional PRs?

Lets merge this PR. @Koncpa will create a new PR with tests.

@richm richm merged commit c5c2791 into main Nov 16, 2023
7 checks passed
@richm richm deleted the init-branch branch November 16, 2023 14:22
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.

3 participants