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

Fix service account login #376

Merged
merged 5 commits into from
Oct 27, 2022
Merged

Fix service account login #376

merged 5 commits into from
Oct 27, 2022

Conversation

BuJo
Copy link
Collaborator

@BuJo BuJo commented Oct 27, 2022

Fixes problems unearthed by changes in #364

Changes:

  • Use ForceNew on target user changes. This just simplifies the code.
  • Fix the test if the service account can log in. This check cannot happen using the GUI, as service accounts can not log into it that way. This rewrites the check to simply list all buckets and checks success. It works by using the admin minio user as target user, which also removes the oddity of using a non-existent target user in the tests.
  • Revise logic around failures deleting the service account, which could lead to hiding the real error.

BuJo added 5 commits October 27, 2022 10:18
* Simplifies code, moves recreation code into the terraform infrastructure
* The access key was not filled correctly, the target user was used
  to check the service account.
* Also simplifies changes for secret and status
* The target user must exist to create a correct test case,
  use the `minio` user as target, as it has enough rights.
* Service accounts cannot log in to the UI, it thus needs a different
  way to check the correct password.
@felladrin felladrin merged commit b04a073 into master Oct 27, 2022
@felladrin felladrin deleted the fix-service-account-login branch October 27, 2022 08:56
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