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

feat: add Microsoft SQL Server module #72

Merged
merged 6 commits into from
Nov 29, 2023
Merged

Conversation

kymmt90
Copy link
Contributor

@kymmt90 kymmt90 commented Nov 28, 2023

This PR adds a Microsoft SQL Server module based on the official Docker images.

https://hub.docker.com/_/microsoft-mssql-server

Thanks!

@kymmt90 kymmt90 changed the title feat: Add Microsoft SQL Server feat: add Microsoft SQL Server Nov 28, 2023
@DDtKey DDtKey changed the title feat: add Microsoft SQL Server feat: add Microsoft SQL Server image Nov 28, 2023
@DDtKey DDtKey changed the title feat: add Microsoft SQL Server image feat: add Microsoft SQL Server module Nov 28, 2023
Copy link
Contributor

@DDtKey DDtKey left a comment

Choose a reason for hiding this comment

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

Thanks for your contribution! 👍

src/mssql_server/mod.rs Show resolved Hide resolved
examples/mssql_server.rs Outdated Show resolved Hide resolved
@kymmt90 kymmt90 requested a review from DDtKey November 29, 2023 12:40
@kymmt90
Copy link
Contributor Author

kymmt90 commented Nov 29, 2023

I'll take a look at the CI failures...

@kymmt90
Copy link
Contributor Author

kymmt90 commented Nov 29, 2023

The failed tests sometimes tried to login before master db (a system db where SA user is in) was recovered and got auth errors.

 failures:
  
  ---- mssql_server::tests::one_plus_one stdout ----
  Error: Server(TokenError { code: 18456, state: 1, class: 14, message: "Login failed for user 'sa'.", server: "51a9d25accfc", procedure: "", line: 1 })

Fixed ready_conditions to be able to login as SA user sure. Now these tests are not flaky in my local environment.

@DDtKey Could you please run CI again for this changes?

src/mssql_server/mod.rs Outdated Show resolved Hide resolved
@DDtKey DDtKey merged commit 0c579ca into testcontainers:main Nov 29, 2023
6 checks passed
@github-actions github-actions bot mentioned this pull request Nov 29, 2023
DDtKey pushed a commit that referenced this pull request Nov 29, 2023
## 🤖 New release
* `testcontainers-modules`: 0.1.4 -> 0.2.0 (✓ API compatible changes)

<details><summary><i><b>Changelog</b></i></summary><p>

<blockquote>

## [0.2.0] - 2023-11-29

### Features

- Add `Microsoft SQL Server` module
([#72](#72))

### Miscellaneous Tasks

- [**breaking**] Update `elasticmq` default image version
([#73](#73))
- Bump the aws-dependencies group with 5 updates
([#71](#71))

<!-- generated by git-cliff -->
</blockquote>


</p></details>

---
This PR was generated with
[release-plz](https://github.com/MarcoIeni/release-plz/).

Signed-off-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@DDtKey
Copy link
Contributor

DDtKey commented Nov 29, 2023

Thanks for all efforts to get it merged!

New version has been released and available on crates.io
https://github.com/testcontainers/testcontainers-rs-modules-community/releases/tag/v0.2.0

@kymmt90
Copy link
Contributor Author

kymmt90 commented Nov 29, 2023

Thank you! 🥳

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants