-
Notifications
You must be signed in to change notification settings - Fork 66
chore: bump pinned localstack tag to 4.5 #363
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
base: main
Are you sure you want to change the base?
Conversation
@DDtKey Maybe you could take a look, is this acceptable? |
Hi @kairsas and thanks for the contribution! We usually don’t bump image version if it doesn’t require any changes except the version/tag itself See #138 (comment) for example I don’t mind updating it with the next major release though, so let’s keep PR active if you don’t mind |
Thanks, @DDtKey, for the detailed explanation. I see your point, but it doesn’t feel natural to me. Maybe if people want their build or test suite to remain stable, they could stick to a fixed testcontainers (or image) version? But of course, that's not a big problem, as it can be workarounded by overriding the tag. |
IMO, the current approach appears to be quite natural. While I agree users should stick to specific versions on their end, it is crucial to avoid breaking things in minor patches. I'm generally open to updating images, but it should be done with appropriate versioning (this PR can be a part of 0.13.0) |
Btw, could you address clippy warnings? |
Clippy should be fine now. |
@DDtKey do you have a clue what are those failing tests? |
Sorry for the late response, we have several flaky tests unfortunately and it's not related you your changes It's gonna be addressed separately |
Default
localstack
tag 3.0 was quite outdated,causing S3
put_object
operations to fail with recent versions of the AWS CLI.Updating to the latest stable tag, 4.5, resolves the issue.