StackStorm/st2

Predictive error message when config hasn't been loaded

Open

#3,435 创建于 2017年5月25日

在 GitHub 查看
 (1 评论) (0 反应) (0 负责人)Python (740 fork)batch import
documentationfeaturegood first issue

仓库指标

Star
 (5,794 star)
PR 合并指标
 (30 天内没有已合并 PR)

描述

I've done support a few times now and one of the most common problems arises from the fact that someone has changed a pack config file but haven't reloaded it.

The error message that usually pops up is a KeyError, since the action or sensor will likely be looking up some key in the config that may not exist in the config stored in the database. However, for some packs, where the default configuration is provided with some dummy values, this may not always be the case

It may be worth putting some thought into how we can detect whether or not the configuration has been reloaded recently, and provide something useful to the user that indicates this is what they should do. At the very least, we should catch "common" exceptions like KeyError and provide this additional context before re-raising.

贡献者指南