aboutsummaryrefslogtreecommitdiff
path: root/releasenotes/notes/config-handlers-fba9b642c84fc2d7.yaml
blob: 1dca84b22085839179b7f111d6e8defe87e899e3 (plain)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
---
features:
  - |
    Performance: Don't notify handlers during config

    This patch builds upon genconfig optimisation and it takes it
    further by not having genconfig ever touch the handlers!
    Calling the handlers and skipping them created an unnecessary slow down
    if only config was ran. It also depends on the config checking fix.

    This gets us closer to the single responsibility principle -
    config only generates the config, container checks only validate
    whether container restart is needed.

    And this also means that we will have single place were containers
    are restarted, were we can fix the ansible quirk of it restarting
    the whole group even when one container changed in the following patches.

    The only exception is the loadbalance role. As the loadbalancer services
    have their config altered by other roles registering their services
    using loadbalancer-config. This is in contrast to typical roles,
    which do config in one step and can then run check-containers in
    the next step.

fixes:
  - |
    Fixes some handlers that were missing the necessary guard,
    making genconfig actually able to restart some containers.