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

allow dbname and sql user username to be overridden via spec #882

Merged
merged 2 commits into from
Apr 6, 2020
Merged

allow dbname and sql user username to be overridden via spec #882

merged 2 commits into from
Apr 6, 2020

Conversation

frodopwns
Copy link
Contributor

closes #874

What this PR does / why we need it:
Allows user to override the default functionality of using the metatdata.name for username and db name in sqluser and sqldatabase operators.

@jananivMS jananivMS self-requested a review April 6, 2020 15:31
Copy link
Contributor

@jananivMS jananivMS left a comment

Choose a reason for hiding this comment

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

looks good from the testing. The sql user secret is still using instance.Name, added a comment to verify if that is intentional.

api/v1alpha1/azuresqluser_types.go Show resolved Hide resolved
pkg/resourcemanager/azuresql/azuresqluser/azuresqluser.go Outdated Show resolved Hide resolved
@jananivMS jananivMS self-requested a review April 6, 2020 17:16
@jananivMS jananivMS dismissed their stale review April 6, 2020 18:02

changes updated

Copy link
Contributor

@jananivMS jananivMS left a comment

Choose a reason for hiding this comment

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

Looks good

@frodopwns frodopwns merged commit e1372ea into Azure:master Apr 6, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants