Skip to content

ENT-12129: Added full hosts delete and 2FA support to distributed cleanup script#3113

Merged
craigcomstock merged 2 commits intocfengine:masterfrom
aleksandrychev:ENT-12129
Apr 6, 2026
Merged

ENT-12129: Added full hosts delete and 2FA support to distributed cleanup script#3113
craigcomstock merged 2 commits intocfengine:masterfrom
aleksandrychev:ENT-12129

Conversation

@aleksandrychev
Copy link
Copy Markdown
Contributor

@aleksandrychev aleksandrychev commented Mar 25, 2026

Ticket: ENT-12129

Ticket: ENT-12129

ChangeLog: Changed distributed_cleanup.py to issue a real DELETE FROM __hosts instead of soft deletion via INSERT with a deleted timestamp
Signed-off-by: Ihor Aleksandrychiev <ihor.aleksandrychiev@northern.tech>
…anup setup

Ticket: ENT-12129

ChangeLog: Title
Signed-off-by: Ihor Aleksandrychiev <ihor.aleksandrychiev@northern.tech>
@aleksandrychev aleksandrychev marked this pull request as draft March 25, 2026 14:43
@aleksandrychev
Copy link
Copy Markdown
Contributor Author

@cf-bottom jenkins please

@cf-bottom
Copy link
Copy Markdown

@aleksandrychev
Copy link
Copy Markdown
Contributor Author

@cf-bottom jenkins please again

@cf-bottom
Copy link
Copy Markdown

@aleksandrychev aleksandrychev marked this pull request as ready for review March 27, 2026 11:28
@aleksandrychev
Copy link
Copy Markdown
Contributor Author

aleksandrychev commented Mar 27, 2026

As result I see hosts are permanently deleted. I used users that are not "admin" and with 2FA.

Ran post processing SQL. returncode was 0, stderr:

stdout:
SET
DELETE 14
SET
DELETE 14
DELETE 154
DELETE 0
DELETE 0
DELETE 14
DELETE 8708
DELETE 1470
DELETE 840
DELETE 0
DELETE 0
DELETE 2142
DELETE 0
DELETE 14
DELETE 0

cocodrillo: 14 host deletions processed

Copy link
Copy Markdown
Contributor

@craigcomstock craigcomstock left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

cool. thanks!

@craigcomstock craigcomstock merged commit 13f9471 into cfengine:master Apr 6, 2026
36 of 37 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants