Config update results in `FeatureFlags` section being added and removed

Description

Here's what he said:

first, a `Configurations` record is added with the change and marked as active in the database. then, and the following process happens as many times as there are nodes in the cluster:
1. a record is added and marked active with the change and also this additional content added to the config data:
```
"FeatureFlags": {
"TestFeature": "off",
"TestBoolFeature": false,
"CloudDelinquentEmailJobsEnabled": false
}
```
2. a record is added and marked active with the change and without the `FeatureFlags` data

Issue created from a message in Mattermost.

QA Test Steps

  • Make sure the reported issue doesn't reproduce anymore.

  • A smoke test around general functioning of the application, especially aimed at config changes.

Activity

Show:
Paul Rothrock
January 30, 2021, 3:44 AM

To replicate, set up a Mattermost cluster with config in database and run the attached script. If you time it right you should see something like the attached screenshot.

 

Lindy Isherwood
February 3, 2021, 7:56 PM

I tested the app functionality and config changes in the sys console on the PR. Saturn will test the reported issue no longer reproduces.

Lindy Isherwood
February 5, 2021, 5:55 AM

I tested the app functionality and config changes in the sys console on prev.test set to v5.31.1-rc1 and did not find any issues.

Saturnino Abril
February 5, 2021, 9:44 PM

Tested and passed against v5.31.1-rc1 and master. Configurations table only changes as expected and no unexpected additional row both in on-prem and cloud. Will take a look again once v5.32.0-rc2 becomes available.

Saturnino Abril
February 11, 2021, 8:31 PM

Tested and passed on 5.32.0-rc2

Done

Mana

None

Assignee

Claudio Costa

QA Assignee

Lindy Isherwood

Reporter

Agniva De Sarker

Epic Link

None

Mattermost Team

Server

Sprint

None

QA Testing Areas

Other (write in QA test steps)

GitHub Issue

None

Components

None

Severity

None