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

nginx: Mask nginx service after install #35

Merged
merged 1 commit into from
Mar 27, 2019
Merged

nginx: Mask nginx service after install #35

merged 1 commit into from
Mar 27, 2019

Conversation

hedss
Copy link
Contributor

@hedss hedss commented Mar 26, 2019

Connects-to: #34
Change-type: patch
Signed-off-by: Heds Simons [email protected]

@hedss
Copy link
Contributor Author

hedss commented Mar 26, 2019

Have tested, works fine, and now doesn't cause either UI or Admin to fail to restart!

Dockerfile Outdated
@@ -16,3 +16,6 @@ RUN echo 'deb http://deb.debian.org/debian jessie main' >> /etc/apt/sources.list
&& rm /etc/init.d/nginx \
&& rm -rf /etc/nginx/conf.d/* \
&& rm -rf /var/lib/apt/lists/*

# Stop nginx from being available, as we run it manually in downstream images
RUN systemctl mask nginx.service
Copy link
Contributor

Choose a reason for hiding this comment

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

Wanna whack this in as part of the last RUN?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yeah, that makes sense, shall edit!

Connects-to: #34
Change-type: patch
Signed-off-by: Heds Simons <[email protected]>
@hedss hedss merged commit 609e2db into master Mar 27, 2019
@hedss hedss deleted the 34-mask-nginx branch March 27, 2019 13:05
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