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

Extract the main APIcast nginx configuration into own file #964

Merged
merged 5 commits into from
Feb 4, 2019

Conversation

mikz
Copy link
Contributor

@mikz mikz commented Nov 30, 2018

Having all APIcast specific configuration in one file included by liquid allows us to replace that file with different server blocks etc. For example that is used by the standalone mode: #926.

Depends on #963.

@mikz mikz requested a review from a team as a code owner November 30, 2018 14:24
gateway/src/apicast/cli/filesystem.lua Outdated Show resolved Hide resolved
gateway/src/apicast/cli/filesystem.lua Outdated Show resolved Hide resolved
gateway/src/apicast/cli/filesystem.lua Outdated Show resolved Hide resolved
@mikz mikz force-pushed the template-main-nginx-config branch 2 times, most recently from ae2996a to 65db3f5 Compare December 3, 2018 10:51
.luacheckrc Show resolved Hide resolved
@mikz mikz force-pushed the template-main-nginx-config branch from 65db3f5 to 957d1c0 Compare December 3, 2018 15:01
@mikz
Copy link
Contributor Author

mikz commented Dec 3, 2018

I'll also add unit tests for the template renderer and environment file to verify the context feature.

@mikz mikz force-pushed the template-main-nginx-config branch 2 times, most recently from 69eae39 to 627a911 Compare December 4, 2018 13:37
@mikz mikz requested a review from davidor December 4, 2018 14:03
@mikz mikz force-pushed the template-main-nginx-config branch from 627a911 to 7d77d96 Compare February 1, 2019 14:00
@mikz mikz force-pushed the template-main-nginx-config branch from 7457a43 to 0d20656 Compare February 4, 2019 12:20
@mikz mikz requested a review from davidor February 4, 2019 13:02
@davidor davidor merged commit cbbff5f into master Feb 4, 2019
@davidor davidor deleted the template-main-nginx-config branch February 4, 2019 13:30
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