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

Firewalld resource to include enable #294

Closed
jrmetzger opened this issue Dec 30, 2023 · 1 comment
Closed

Firewalld resource to include enable #294

jrmetzger opened this issue Dec 30, 2023 · 1 comment
Labels
Feature Request Enhancement to existing functionality or new functionality

Comments

@jrmetzger
Copy link

jrmetzger commented Dec 30, 2023

🗣️ Foreword

Thank for taking the time to fill this feature request fully. Without it we may not be able to , and the issue may be closed without resolution.

🙍 Problem Statement

Firewalld action to enable service

❔ Possible Solution

A clear and concise description of what you want to happen.

action :enable do
  service 'firewalld' do
    action :enable
  end
end

(assuming :restart is also :start)

⤴️ Describe alternatives you've considered

pull out service into own cookbook, but would be better to add :enable to list of action

➕ Additional context

@jrmetzger jrmetzger added the Feature Request Enhancement to existing functionality or new functionality label Dec 30, 2023
@decoyjoe
Copy link
Contributor

decoyjoe commented Jan 3, 2025

This was fixed via #304 with:

service 'firewalld' do
action [:enable, :start]
end

cc: @bmhughes

@decoyjoe decoyjoe mentioned this issue Jan 3, 2025
3 tasks
@bmhughes bmhughes closed this as completed Jan 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature Request Enhancement to existing functionality or new functionality
Projects
None yet
Development

No branches or pull requests

3 participants