Change NifiUser sync logic to reduce api calls on UserGroup endpoint

[juldrixx]

Q	A
Bug fix?	no
New feature?	no
API breaks?	no
Deprecations?	no
Related tickets	-
License	Apache 2.0

What's in this PR?

Update of the NifiUser sync logic to reduce the number of API call on the UserGroup endpoint.

Why?

Because if we have a lot of user or node, we will spam NiFi to retrieve the group information.

Additional context

Checklist

• Implementation tested
• Error handling code meets the guideline
• Logging code meets the guideline
• User guide and development docs updated (if needed)
• Append changelog with changes

[mh013370]

I have also noticed these endpoints in particular getting hammered while inspecting NiFi logs. This is a much needed change.

[mh013370]

This looks fine on the surface. I just want to carefully walk through the logic to make sure there aren't unintended consequences. You've tested this, yeah?

[juldrixx]

This looks fine on the surface. I just want to carefully walk through the logic to make sure there aren't unintended consequences. You've tested this, yeah?

I tested it and it seems to good. But if you want to retry to be sure, you can 😁.

[mh013370]

After review, i don't see anything glaringly incorrect about this.