refactor: ensure that using previous replaced does not remove files and reload unless necessary#310
Conversation
![sourcery-ai[bot]](https://avatars.githubusercontent.com/in/48477?s=60&v=4){kind=small}
There was a problem hiding this comment.
Sorry @richm, your pull request is larger than the review limit of 150000 diff characters
richm
force-pushed
the
previous-replaced-idempotent
branch
from
1248c50 to
1c35ffb
Compare
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #310 +/- ##
===========================================
- Coverage 61.09% 42.53% -18.57%
===========================================
Files 2 3 +1
Lines 910 2278 +1368
===========================================
+ Hits 556 969 +413
- Misses 354 1309 +955
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
richm
force-pushed
the
previous-replaced-idempotent
branch
from
1c35ffb to
1f2d624
Compare
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
| if HAS_FIREWALLD: | ||
| firewall.config.FIREWALLD_POLICIES | ||
|
|
||
| HAS_POLICIES = True |
Check notice
Code scanning / CodeQL
Unused global variable Note
|
|
||
| HAS_POLICIES = True | ||
| except AttributeError: | ||
| HAS_POLICIES = False |
Check notice
Code scanning / CodeQL
Unused global variable Note
| try: | ||
| from firewall.functions import check_mac | ||
|
|
||
| HAS_CHECK_MAC = True |
Check notice
Code scanning / CodeQL
Unused global variable Note
|
|
||
| HAS_CHECK_MAC = True | ||
| except ImportError: | ||
| HAS_CHECK_MAC = False |
Check notice
Code scanning / CodeQL
Unused global variable Note
richm
force-pushed
the
previous-replaced-idempotent
branch
3 times, most recently
from
7d5e884 to
81292e4
Compare
|
[citest] |
richm
force-pushed
the
previous-replaced-idempotent
branch
from
81292e4 to
bb240fa
Compare
|
[citest] |
richm
force-pushed
the
previous-replaced-idempotent
branch
from
bb240fa to
a697edc
Compare
richm
force-pushed
the
previous-replaced-idempotent
branch
8 times, most recently
from
d4a3ad6 to
6ff5467
Compare
|
[citest] |
|
[citest_bad] |
richm
force-pushed
the
previous-replaced-idempotent
branch
from
6ff5467 to
3345ff4
Compare
richm
force-pushed
the
previous-replaced-idempotent
branch
3 times, most recently
from
ce87778 to
b5a04c1
Compare
|
[citest] |
richm
force-pushed
the
previous-replaced-idempotent
branch
from
b5a04c1 to
b4ce3bb
Compare
…nd reload unless necessary The old implemenation of `previous: replaced` would remove all files and reload the firewall every time, even if not necessary - it wasn't technically idempotent, although it would check the new files to see if they matched the old files, and report `changed: false` if nothing actually changed state. The new implementation uses an in-memory backend to apply the changes, then checks if anything changed, and then removes the files and reloads firewall only if something actually changed. Signed-off-by: Rich Megginson <rmeggins@redhat.com>
richm
force-pushed
the
previous-replaced-idempotent
branch
from
b4ce3bb to
cc1608a
Compare
|
[citest] |
1 participant
The old implemenation of
previous: replacedwould remove all files and reload thefirewall every time, even if not necessary - it wasn't technically idempotent, although
it would check the new files to see if they matched the old files, and report
changed: falseif nothing actually changed state.The new implementation uses an in-memory backend to apply the changes, then checks if
anything changed, and then removes the files and reloads firewall only if something
actually changed.
Signed-off-by: Rich Megginson rmeggins@redhat.com