Implement escalation handler #6

jlevesy · 2022-08-09T16:06:58Z

This PR

Does some refactoring again (examples, have a common escalation package instead of split webhook and event handlers packages)
Fixes the controller init sequence which was broken in Day to day life improvements + use lister for webhook handler #4
an event handler now accepts context.
Adds a "cutallthepossiblecorners" implementation of the escalation event handler, as well as CRDs / RBAC fixes related to this. This should be redone, but we have a clearer idea of the challenges behind this controller :)

…ate it

jlevesy added 4 commits August 9, 2022 17:39

fix: declare status in CRD escalation schema, allow controller to upd…

b6bc2c4

…ate it

fix: grant necessary permissions to controller to create role bindings

c05cff6

chore: reorganize a bit the example directory

efad29a

doc: add some more notes regarding reclaiming

b84b45e

jlevesy force-pushed the jl/implement-escalation-handler branch from cd3e473 to 2b06ca6 Compare August 9, 2022 16:13

jlevesy added 6 commits August 9, 2022 18:18

refactor: move webhook/escalation into escalation

c962c34

refactor: move webhook/run into pkg/webhhooksupport

331f6e9

refactor: pass down context in event handler

b9c2425

fix: allow escalation status to carries multiple target refs

db97c86

feat: (wip) implemeent the escalation handler

411cc55

chore: wire up everything into the controller

5d929d9

jlevesy force-pushed the jl/implement-escalation-handler branch from 2b06ca6 to 5d929d9 Compare August 9, 2022 16:19

jlevesy merged commit e12ea7c into main Aug 9, 2022

jlevesy deleted the jl/implement-escalation-handler branch August 9, 2022 16:25

Provide feedback