I don't think the name needs to be in the state_data. When the job is passed to the StateChangeWatcher, it can call `get_name()` to use as part of the key. The name is actually part of the config , not the state, so we don't want to confuse that.

I believe this will require uncommenting `get_name()` above as well.