-
Notifications
You must be signed in to change notification settings - Fork 53
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
add ssh secrets #19
add ssh secrets #19
Conversation
This change adds the capability of generating ssh secrets to the secret operator. While implementing this, I also moved the password generation to a different file, which allows for further extension of the operator with additional secret generators. I also moved the test cases to generator specific files. Furthermore, I noticed, that golangs string formatter on the time type is quite verbose. The generated-at annotation now contains a RFC3339 compliant time string.
@martin-helmich I've rebased this on the current master. This change is finally ready for review :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a nitpick and a typo, the rest looks good!
Dismissing, as all requested changes are outdated.
rebased on #18 / master
This change adds the capability of generating ssh secrets to the secret
operator.
While implementing this, I also moved the password generation to a
different file, which allows for further extension of the operator with
additional secret generators.
I also moved the test cases to generator specific files.
Furthermore, I noticed, that golangs string formatter on the time type
is quite verbose. The generated-at annotation now contains a
RFC3339 compliant time string.
Since the string generator now also respects the length annotation:
fixes #16