Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Add support to suspend and resume database#519

Merged
yec-akamai merged 2 commits intolinode:devlinode/linode_api4-python:devfrom
yec-akamai:TPT-3271/support-dbaas-suspend-resumeyec-akamai/linode_api4-python:TPT-3271/support-dbaas-suspend-resumeCopy head branch name to clipboard
Mar 20, 2025
Merged

Add support to suspend and resume database#519
yec-akamai merged 2 commits intolinode:devlinode/linode_api4-python:devfrom
yec-akamai:TPT-3271/support-dbaas-suspend-resumeyec-akamai/linode_api4-python:TPT-3271/support-dbaas-suspend-resumeCopy head branch name to clipboard

Conversation

@yec-akamai
Copy link
Contributor

📝 Description

Implement suspending and resuming MySQL and PostgreSQL database.

✔️ How to Test

Unit test:

make test-unit

Integration test:

May take longer time to run the DB test.

make test-int TEST_CASE=test_postgres_suspend_resume RUN_DB_TESTS=true      
make test-int TEST_CASE=test_mysql_suspend_resume RUN_DB_TESTS=true      

@yec-akamai yec-akamai added the new-feature for new features in the changelog. label Mar 5, 2025
@yec-akamai yec-akamai requested a review from a team as a code owner March 5, 2025 21:12
@yec-akamai yec-akamai requested review from ykim-akamai and zliang-akamai and removed request for a team March 5, 2025 21:12
@zliang-akamai zliang-akamai requested review from a team, ezilber-akamai and lgarber-akamai and removed request for a team March 6, 2025 07:56
Copy link
Contributor

@ezilber-akamai ezilber-akamai left a comment

Choose a reason for hiding this comment

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

LGTM!

linode_api4/objects/database.py Outdated Show resolved Hide resolved
Comment on lines +177 to +184
wait_for_condition(
10,
300,
get_sql_db_status,
test_linode_client,
test_create_sql_db.id,
"suspended",
)
Copy link
Contributor

Choose a reason for hiding this comment

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

Just a heads up for the future, we'll probably want to replace this with event polling once the update event issue is resolved in the API. I don't have an ETA for that so this should be fine for now 👍

Copy link
Contributor

@ykim-akamai ykim-akamai left a comment

Choose a reason for hiding this comment

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

LGTM, nice work!

@yec-akamai yec-akamai merged commit 3692712 into linode:dev Mar 20, 2025
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

new-feature for new features in the changelog.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.