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

Support CentOS 8 and a supervisor alternative #16

Closed
navbruce opened this issue Oct 16, 2019 · 16 comments
Closed

Support CentOS 8 and a supervisor alternative #16

navbruce opened this issue Oct 16, 2019 · 16 comments

Comments

@navbruce
Copy link

The build is currently failing. Apparently supervisor is not available:

$ docker build --tag docker-postfix .
Sending build context to Docker daemon  122.4kB
Step 1/13 : FROM centos:latest

[...]

Installed:
  libxkbcommon-0.8.2-1.el8.x86_64       diffutils-3.6-5.el8.x86_64
  kbd-2.0.4-8.el8.x86_64                xkeyboard-config-2.24-3.el8.noarch
  kbd-legacy-2.0.4-8.el8.noarch         kbd-misc-2.0.4-8.el8.noarch

Complete!
Last metadata expiration check: 0:00:47 ago on Wed Oct 16 23:02:49 2019.
No match for argument: supervisor
Error: Unable to find a match
The command '/bin/sh -c yum install -y epel-release && yum update -y &&     yum install -y cyrus-sasl cyrus-sasl-plain cyrus-sasl-md5 mailx     perl supervisor postfix rsyslog     && rm -rf /var/cache/yum/*     && yum clean all' returned a non-zero code: 1

If I find a fix I'll report back, but I thought I should note the problem first.

@juanluisbaptiste
Copy link
Owner

Sure ? I just successfully built it locally with latest CentOS image.

@navbruce
Copy link
Author

Hmm, odd. I just cloned your repo and built it and had this problem, and I only tried it with a clean local copy because it failed to build (well, my somewhat modified version) in my deployment pipeline.

@juanluisbaptiste
Copy link
Owner

You are right, I forgot to use --no-cache last time, I just checked and I get the build failure.

@juanluisbaptiste
Copy link
Owner

The issue is with the CentOS version, now centos:latest points to CentOS 8 and epel repo for that version does not include supervisor.

@juanluisbaptiste
Copy link
Owner

Fixed, new build available.

Thanks for the heads up 👍

@blackholegalaxy
Copy link

Do you plan any update to get this working on centos 8? Any supervisor alternative?

@juanluisbaptiste
Copy link
Owner

Do you plan any update to get this working on centos 8? Any supervisor alternative?

Yes, but not in the near future, too much going on currently and this issue needs its time to be researched and properly implemented.

PR's accepted though.

@juanluisbaptiste juanluisbaptiste changed the title Build is failing, can't find "supervisor" Support CentOS 8 and a supervisor alternative Oct 22, 2019
@juanluisbaptiste
Copy link
Owner

@blackholegalaxy

I just pushed a new version running on Alpine + supervisor with tag alpine. Please test it downloading juanluisbaptiste/postfix:alpine and let me know how it is working.

Later I also plan to migrate from supervisor to s6-init, but I prefer going step by step.

@blackholegalaxy
Copy link

Oh that's pretty nice! I will test it out tomorrow and let you know

@juanluisbaptiste
Copy link
Owner

Excellent, @navbruce would you like to test it out too ?

@navbruce
Copy link
Author

I'd like to, but mail is pretty far from what I'm working on right now. Looks like a good move, though.

@juanluisbaptiste
Copy link
Owner

@blackholegalaxy , did you have time to test ? for me the new alpine image is working fine, but there's issue #21 where is not working for someone else. Could you do a quick test and report back ?

@blackholegalaxy
Copy link

blackholegalaxy commented Apr 9, 2020

My usage of docker-postfix is to redirect to another provider as it meant to be. And it works pretty fine on the alpine image :)

@juanluisbaptiste
Copy link
Owner

My usage of docker-postfix is to redirect to another provider as it meant to be. And it works pretty fine on the alpine image :)

Excellent, many thanks for the test.

@juanluisbaptiste
Copy link
Owner

Hi,

After being running the alpine image for a long time on production systems with no issues, this week I will make this image the default one, merging the migrate_to_alpine branch into master and moving the latest tag to this one.

@juanluisbaptiste
Copy link
Owner

Merged migrate_to_alpine branch into master, closing this issue.

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

No branches or pull requests

3 participants