Project

General

Profile

Actions

Bug #17720

closed

If a second rollback starts when a first is processing, system group/technique may be lost

Added by François ARMAND over 4 years ago. Updated almost 4 years ago.

Status:
Released
Priority:
N/A
Category:
Web - Config management
Target version:
Severity:
Major - prevents use of part of Rudder | no simple workaround
UX impact:
User visibility:
Infrequent - complex configurations | third party integrations
Effort required:
Priority:
38
Name check:
To do
Fix check:
Checked
Regression:

Description

We had a case where starting two rollback at a couple second appart led to the erasing of system group and technique - actually not lost, but hidden for one who doesn't know where to look at.

So, the problem is that we use LDAP (which doesn't have transaction) and that a rollback proceeds like that:

1/ move current group and technique root into ou=Archives, ou=Rudder, cn=rudder-configuration
2/ restore Git archive corresponding to rollback point, which recreate group/technique, but without system elements
3/ copy from archie system elements
4/ delete archive

Here, a second rollback was started during 2 and before 3. So what was copied in archive that time was missing system elements, as seen here:

This also likely broke first rollback, which wasn't able to complete. Second rollback continues, but then, it can't restore system elements, since they are missing, and it fails, too. And we end up without system techniques in the awaited place (but nothing lost).

A first not too hard correction would be to put rollback queries in a queue so that they don't interfer from one other to each other. This is stil rather barok and won't prevent cases were other ldap op breaks expectation during the process, but it will avoid that horror case.

A real solution would be to just commit back entries corresponding to whatever action is rollback, and only that, in a new apeend-only modification, so that nothing is lost. It could be done with a bit of hand waving and a git checkout commitId -- only-modified-files.xml && git commit && restore archive from last git.
It could even be better to read entries that were modified, get back their value from git at that point, materialize them in code and save them back.
That second change is massive in semantic evolution of rollback, and need a major version of rudder.


Files

clipboard-202006112144-oknae.png (34.8 KB) clipboard-202006112144-oknae.png François ARMAND, 2020-06-11 21:44

Related issues 2 (0 open2 closed)

Related to Rudder - Bug #18500: clicking twice on rollbacking event in 6.1 breaks rudderResolvedActions
Related to Rudder - Bug #18711: rollback error when trying to revert allowed networks: "event log xxx don't have a matching commitId"RejectedFrançois ARMANDActions
Actions #1

Updated by Nicolas CHARLES about 4 years ago

  • Related to Bug #18500: clicking twice on rollbacking event in 6.1 breaks rudder added
Actions #2

Updated by François ARMAND about 4 years ago

  • Target version set to 6.1.7
  • Priority changed from 41 to 38
Actions #3

Updated by François ARMAND about 4 years ago

  • Status changed from New to In progress
  • Assignee set to François ARMAND
Actions #4

Updated by François ARMAND about 4 years ago

  • Status changed from In progress to Pending technical review
  • Assignee changed from François ARMAND to Nicolas CHARLES
  • Pull Request set to https://github.com/Normation/rudder/pull/3336
Actions #5

Updated by François ARMAND about 4 years ago

  • Status changed from Pending technical review to Pending release
Actions #6

Updated by François ARMAND almost 4 years ago

  • Related to Bug #18711: rollback error when trying to revert allowed networks: "event log xxx don't have a matching commitId" added
Actions #7

Updated by François ARMAND almost 4 years ago

  • Fix check changed from To do to Checked
Actions #8

Updated by Vincent MEMBRÉ almost 4 years ago

  • Status changed from Pending release to Released

This bug has been fixed in Rudder 6.1.7 which was released today.

Actions

Also available in: Atom PDF