Skip to content

Notify Lifecycle Manager if Service/Sec Accounts are going to expire. - #79

Closed
pmedinar01 wants to merge 1 commit into
masterfrom
nlmanager-sec-serv-accounts
Closed

Notify Lifecycle Manager if Service/Sec Accounts are going to expire.#79
pmedinar01 wants to merge 1 commit into
masterfrom
nlmanager-sec-serv-accounts

Conversation

@pmedinar01

@pmedinar01 pmedinar01 commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

At the moment we only notify the Lifecycle Manager if a primary account is going to expire. We want to extend this to service/secondary accounts as well.

Secondary/Service Accounts always report active as false, hence we send all of them to GracePeriod if we treated them equally to primary accounts. As a consequence, i would rather use the blocked flag.

{
  "type": "Secondary",
  "total": 660,
  "blocked": 4,
  "disabled": 0,
  "activeUser_false": 660,
  "blocked_o_disabled": 4
}
{
  "type": "Service",
  "total": 841,
  "blocked": 0,
  "disabled": 0,
  "activeUser_false": 841,
  "blocked_o_disabled": 0
}

@pmedinar01
pmedinar01 force-pushed the nlmanager-sec-serv-accounts branch from 8d9a76f to fa130d0 Compare September 3, 2026 08:26
@pmedinar01
pmedinar01 force-pushed the nlmanager-sec-serv-accounts branch from fa130d0 to 506cbce Compare September 3, 2026 08:34
@glpatcern

Copy link
Copy Markdown
Member

As discussed, we cannot do this. A blocked account is something else and its transitions are already handled outside of Reva. Therefore I'm closing the PR.

@glpatcern glpatcern closed this Sep 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants