Skip to content

Add partial update support of users#1414

Merged
Motouom merged 16 commits into
mainfrom
feat/910-add-partial-update-support
Mar 9, 2026
Merged

Add partial update support of users#1414
Motouom merged 16 commits into
mainfrom
feat/910-add-partial-update-support

Conversation

@Nkwenti-Severian-Ndongtsop

@Nkwenti-Severian-Ndongtsop Nkwenti-Severian-Ndongtsop commented Feb 25, 2026

Copy link
Copy Markdown
Collaborator

What this PR does / why we need it:

  • This PR adds a configurable mechanism to ignore selected user properties during user updates via a new CLI property option: --import.behaviors.user-update-ignored-properties
    Example: --import.behaviors.user-update-ignored-properties=attributes,email

  • It keeps the existing behavior working and makes the “ignored properties” list configurable for LDAP-friendly imports.

Which issue this PR fixes fixes #910

Special notes for your reviewer:

PR Readiness Checklist:

Complete these before marking the PR as ready to review:

  • the CHANGELOG.md release notes have been updated to reflect any significant (and particularly user-facing) changes introduced by this PR

@Motouom Motouom left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

It is fine for me

@mbiti2 mbiti2 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

@Nkwenti-Severian-Ndongtsop I tested the PR, it works for --import.behaviors.user-update-ignored-properties
But now, after excluding the fields, the user might want to include it in the future, which the --import.behaviors.user-update-ignored-properties-remove was supposed to help with that
But it is not working. Please can you have a look at it?

@Nkwenti-Severian-Ndongtsop

Nkwenti-Severian-Ndongtsop commented Mar 2, 2026

Copy link
Copy Markdown
Collaborator Author

@Nkwenti-Severian-Ndongtsop I tested the PR, it works for --import.behaviors.user-update-ignored-properties But now, after excluding the fields, the user might want to include it in the future, which the --import.behaviors.user-update-ignored-properties-remove was supposed to help with that But it is not working. Please can you have a look at it?

Thanks @mbiti2, I will have to check the issue with the flag logic

@Motouom Motouom self-requested a review March 4, 2026 11:38

@Motouom Motouom left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Please @Nkwenti-Severian-Ndongtsop , @mbiti2 left some comments that must be attended to

@sonarqubecloud

sonarqubecloud Bot commented Mar 4, 2026

Copy link
Copy Markdown

@Motouom Motouom moved this from Review to Review Requested in os-competence-center-board Mar 9, 2026
@Motouom Motouom merged commit bf43240 into main Mar 9, 2026
14 checks passed
@Motouom Motouom deleted the feat/910-add-partial-update-support branch March 9, 2026 11:53
@github-project-automation github-project-automation Bot moved this from Changes Requested to Done in os-competence-center-board Mar 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

Add support for partial update of users

3 participants