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

Upgrade version of MySQL to 2023-12-30 #4549

Merged
merged 10 commits into from
Feb 18, 2025
Merged

Conversation

theunrepentantgeek
Copy link
Member

@theunrepentantgeek theunrepentantgeek commented Jan 30, 2025

What this PR does

Updates the supported version of dbformysql to v2023-12-30.

Closes #3839

How does this PR make you feel?

gif

Checklist

  • this PR contains tests
  • this PR contains YAML Samples

@theunrepentantgeek theunrepentantgeek added this to the v2.12.0 milestone Jan 30, 2025
@theunrepentantgeek theunrepentantgeek self-assigned this Jan 30, 2025
@theunrepentantgeek theunrepentantgeek changed the title Updage version of MySQL to 2023-12-30 Upgrade version of MySQL to 2023-12-30 Jan 31, 2025
)

func Test_DBForMySQL_FlexibleServer_20231230_CRUD(t *testing.T) {
t.Parallel()
Copy link
Member

Choose a reason for hiding this comment

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

If we're pretty sure that this is correct but can't possibly record tests for it for some reason, we can always mark this t.Skip() and file an item to come back to it once we get to the root cause of why they're rejecting us...

Not ideal though.

Copy link
Member Author

Choose a reason for hiding this comment

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

We got quota and I was able to record the tests.

@theunrepentantgeek theunrepentantgeek modified the milestones: v2.12.0, v2.13.0 Feb 11, 2025
@theunrepentantgeek theunrepentantgeek added this pull request to the merge queue Feb 17, 2025
github-merge-queue bot pushed a commit that referenced this pull request Feb 17, 2025
* Update azure-arm.yaml

* Update generated files

* Implement conversion override for SourceServerResourceId

* Add samples

* Add test for new version

* Update version released

* Add recordings

* Remove custom region from test
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Feb 17, 2025
@theunrepentantgeek theunrepentantgeek added this pull request to the merge queue Feb 17, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Feb 17, 2025
@theunrepentantgeek theunrepentantgeek added this pull request to the merge queue Feb 18, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Feb 18, 2025
@theunrepentantgeek theunrepentantgeek added this pull request to the merge queue Feb 18, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Feb 18, 2025
@theunrepentantgeek theunrepentantgeek added this pull request to the merge queue Feb 18, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Feb 18, 2025
@matthchr
Copy link
Member

Test failed in merge queue due to: test timed out after 15m0s. This timeout was raised to 20m in #4571 - can you rebase onto that change? I think it should help the tests pass more reliably...

Though now that I look, I actually missed fixing this fully... I will send another PR

@theunrepentantgeek theunrepentantgeek added this pull request to the merge queue Feb 18, 2025
Merged via the queue into main with commit 5c5253d Feb 18, 2025
8 checks passed
@theunrepentantgeek theunrepentantgeek deleted the feature/mysql-version branch February 18, 2025 23:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Ready for Release
Development

Successfully merging this pull request may close these issues.

Consider switching to 'rapid' from 'gopter'
2 participants