search

cloud-bulldozer / benchmark-operator

The Chuck Norris of cloud benchmarks
Apache License 2.0
282 stars 128 forks source link

fix firewall for centos stream8 #742

Closed ebattat closed 2 years ago

ebattat commented 2 years ago

Description

Firewall-cmd fails in cloud-init in centos stream8

Fixes

Use firewall-offline-cmd instead of Firewall-cmd

jeniferh commented 2 years ago

LGTM!

codecov-commenter commented 2 years ago

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@f296081). Click here to learn what that means. The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #742   +/-   ##
=========================================
  Coverage          ?   53.82%           
=========================================
  Files             ?        8           
  Lines             ?      327           
  Branches          ?        0           
=========================================
  Hits              ?      176           
  Misses            ?      151           
  Partials          ?        0
Flag Coverage Δ
gha 53.82% <0.00%> (?)
python-3.9 53.82% <0.00%> (?)
unit 53.82% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update f296081...8ebbe6f. Read the comment docs.