tree bdd35f818c7aca6d2f0bac93501d7a56fcfc2911
parent afd3cf8bb5c99e9a8db23a5f435f7c1e31c96f91
author Serge Bazanski <serge@monogon.tech> 1681982340 +0200
committer Serge Bazanski <serge@monogon.tech> 1682423810 +0000

cloud/bmaas: split ShepherdAccess into Shepherd{AgentStart,Recovery}

This effectively undoes our previous attempted consolidation of all
Shepherd accesses under one tag. We now use two separate tags for the
two main Shepherd work processes, and mutually exclude them in SQL.

We do this so that we can see more clearly in work history (and in
general when processing machines) what the Shepherd is actually trying
to do to a machine.

The downside of this implementation is that we now extend the BMDB/ETP
model to being able to mutually exclude different processes. This is
easy enough to express in SQL, but might make future generic modelling
more difficult.

An alternative would be to add an extra field to work/work history that
acts as an informative field for operators to know the details of a work
item. We might still want to do that in the future. However, that field
being freeform, we could not really rely on it for machine parsing.

Change-Id: I9578ac000f6112514fe587e9fddf7e85671c6437
Reviewed-on: https://review.monogon.dev/c/monogon/+/1584
Reviewed-by: Leopold Schabel <leo@monogon.tech>
Tested-by: Jenkins CI