Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

User check mode #208

Merged
merged 8 commits into from
Aug 27, 2024
Merged

User check mode #208

merged 8 commits into from
Aug 27, 2024

Conversation

rmocanu-ionos
Copy link
Contributor

@rmocanu-ionos rmocanu-ionos commented Aug 22, 2024

What does this fix or implement?

Fixes #202

Checklist

  • PR name added as appropriate (e.g. feat/fix/doc/test/refactor/etc)
  • Tests added or updated
  • Documentation updated
  • Changelog updated and version incremented (label: upcoming release)
  • Github Issue linked if any

@rmocanu-ionos rmocanu-ionos changed the base branch from master to new-release August 27, 2024 07:43
plugins/module_utils/common_ionos_module.py Outdated Show resolved Hide resolved
Copy link

sonarcloud bot commented Aug 27, 2024

@rmocanu-ionos rmocanu-ionos merged commit 9009ac8 into new-release Aug 27, 2024
19 of 20 checks passed
rmocanu-ionos added a commit that referenced this pull request Aug 27, 2024
* fix: mark RegEx pattern as a raw string (#200)

* feat: use filters to speed up users_get (#204)

* fix: stop trying to use pagination for groups_users_get, add query params to get_users

* fix: check if filters is None

* fix: import get_users

* fix: use none as default value

* fix: make two calls to get users for all given emails

* test: fix s3key test

* test: added some user tests

* feat: use getById when id is available

* feat: use filters when we retrieve users in other modules when possible

* feat: added ignored_properties to user (#205)

* feat: added ignored_properties to user

* test: fix s3key test

* feat: user check mode (#208)

* feat: working on adding checkmode and diff support for user

* feat: added method to calculate diff for objects

* chore: reverted small changes

* fix: use existing values for diff if none are set to avoid changes in output, add tests

* test: fix s3 key test

* test: fix user test

* chore: user return for check_mode for clarity

* doc: added docs for check_mode and diff, fix typo, added to summary

* chore: increment version, added check_mode doc file

* [no ci] doc: regenerated docs

* doc: updated changelog, changed version
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.

Support check mode in user module
2 participants