Skip to content
New issue

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

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

Already on GitHub? # to your account

dbaas: support for users #401

Merged
merged 15 commits into from
Dec 24, 2024
Merged

Conversation

tgrondier
Copy link
Member

@tgrondier tgrondier commented Dec 10, 2024

Description

For services that allow user creation, we are adding new resources that correspond to a user.

Checklist

(For exoscale contributors)

  • Changelog updated (under Unreleased block)
  • Acceptance tests OK
  • For a new resource, datasource or new attributes: acceptance test added/updated

Testing

Local tests + acceptance tests

@tgrondier tgrondier force-pushed the tgrondier/sc-104245/tf-provider-dbaas-users branch from ec9e8ea to 907c87b Compare December 13, 2024 10:19
@kobajagi
Copy link
Contributor

Also fixes #326

@tgrondier tgrondier force-pushed the tgrondier/sc-104245/tf-provider-dbaas-users branch from 907c87b to 61fd030 Compare December 18, 2024 15:40
@sauterp
Copy link
Member

sauterp commented Dec 20, 2024

One test seems to be failing. Probably unrelated to this PR but it's DBaaS related. Just to be sure.

Copy link
Member

@sauterp sauterp left a comment

Choose a reason for hiding this comment

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

LGTM, it's cool you managed to avoid some boilerplate through generics.

pkg/resources/database/resource_kafka_test.go Outdated Show resolved Hide resolved
pkg/resources/database/resource_user.go Outdated Show resolved Hide resolved
@tgrondier
Copy link
Member Author

One test seems to be failing. Probably unrelated to this PR but it's DBaaS related. Just to be sure.

Yes I can't reproduce the issue, seems to have been transient.

@tgrondier
Copy link
Member Author

tgrondier commented Dec 23, 2024

One test seems to be failing. Probably unrelated to this PR but it's DBaaS related. Just to be sure.

I tested it a bit more on my side: it seems when a mysql dbaas goes from rebuilding to running, the user list returns only the default user avnadmin and not those created while the service was rebuilding.

Unsure how to handle that edge case in the tests

EDIT: Updated to wait for MySQL DBAAS to be running before allowing user updates

@tgrondier tgrondier force-pushed the tgrondier/sc-104245/tf-provider-dbaas-users branch 2 times, most recently from 13ef3a1 to 54cf5b0 Compare December 24, 2024 12:32
@tgrondier tgrondier force-pushed the tgrondier/sc-104245/tf-provider-dbaas-users branch from 54cf5b0 to 9ce0c14 Compare December 24, 2024 14:47
@tgrondier tgrondier merged commit 077c43f into master Dec 24, 2024
2 of 3 checks passed
@tgrondier tgrondier deleted the tgrondier/sc-104245/tf-provider-dbaas-users branch December 24, 2024 15:38
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants