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

debian12.2-slim, latest rspamd and gucci dependancies #23

Merged
merged 5 commits into from
Dec 15, 2023

Conversation

diroots
Copy link

@diroots diroots commented Dec 12, 2023

fix #22

README.md Show resolved Hide resolved
README.md Show resolved Hide resolved
README.md Show resolved Hide resolved
@AndrewSav
Copy link
Collaborator

Looking good to me, let's get these versions sorted, then I'll run it for a week with the new image (we will also need a mailserver PR for that I'm guessing) and then we can merge.

@diroots
Copy link
Author

diroots commented Dec 13, 2023

i did update the dockerfile so all versions are aligned @AndrewSav

@diroots
Copy link
Author

diroots commented Dec 13, 2023

ok i think i know why the 3 skarnet dependancies were not updated in the dockerfile, but in the readme : i did run the 'update-dependencies.sh script with bash instead of zsh and it could not do the string replacements in the dockerfile, it did only in the readme file. fully fixed now

@AndrewSav
Copy link
Collaborator

@diroots thank you for this. Have you tested this? If so I think I'll just merge it without testing because we can test it when we do the PR for the mailserver repo, but I'd like to know if you ran it and it worked for you

@diroots
Copy link
Author

diroots commented Dec 14, 2023

i did not run this as is yet. i did update postfix and clamav from with container to the latest version from debian repo, and i did test the update to rspamd 3.7.4 with adding the repo in the container and install it from there.

all seemed to work when restarting container with these versions

@diroots
Copy link
Author

diroots commented Dec 14, 2023

@AndrewSav i would also say it 'safe' to merge, so it could biuld the underlying debian-mail-overlay image.

then we could test to spin up a mailserver container based on that, on a test env / branch

@AndrewSav
Copy link
Collaborator

@diroots we are building/pushing the image manually. I'll try to find time to work on this over the weekend

@diroots
Copy link
Author

diroots commented Dec 15, 2023

@AndrewSav

  • i did build a mailserver based on this new image.
  • i did deploy this mailserver on some of our staging env. upgrade from previous 1.1.15 version went seamlessly, and the stack is running as expected. i can see updated rspamd/postfix/clamav. all these service logs show it runs smoothly as it was before.

@AndrewSav AndrewSav merged commit 19e8d55 into mailserver2:master Dec 15, 2023
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.

rspamd 3.5 ratelimit issues, bundled in debian-mail-overlay image
2 participants