Review how to handle change / delete config properties

Description

There is some concern about changing property type / validation (e.g. change max, min, pattern). We currently do not handle such logic. We need to agree on some behaviour. May be add an option to force override invalid config when it change schema. Same as delete property. It will be painful if we fail to start opennms and customer need fix it in database. 

 

Delete property

Config contain 3 property, we want to remove 1 of them.

Current:

If you delete property from schema. The changelog will fail due to config contain unknown property. (the one deleted in schema)

Expect:

Either remove or ignore the removed property.

 

Change property:

current config is 1000, we change schema max to 100. Or current config is 0 / 1, we change to boolean

Current:

In some case, changelog will be fail due to invalid existing value in config.

 

Expect:

Besides schema, config should properly handled. 

 

Acceptance / Success Criteria

None

Lucidchart Diagrams

Activity

Show:

Details

Assignee

Reporter

Priority

PagerDuty

Created December 15, 2021 at 3:51 PM
Updated December 15, 2021 at 4:39 PM

Flag notifications