-
Notifications
You must be signed in to change notification settings - Fork 25
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
MDBF-804 - BB NGINX configuration in GH CI #590
Merged
Merged
Commits on Oct 10, 2024
-
MDBF-804 - BB NGINX configuration in GH CI
About This Patch: - Using templates (present since NGINX 1.19) to populate conf.d. - Templates allow for the use of environment variables defined in the .env files, enabling us to distinguish between PROD and DEV environments, particularly for the server name and certificate paths. - A proxy_params file is required according to the PROD configuration. - Mounting /etc/letsencrypt/live for SSL certificates. The base path is the same in both environments. - NGINX_ARTIFACTS_SSL_PATH variable is necessary because, in DEV, the same certificate is used for both CI and BB. - Attaching net_back to the NGINX container to facilitate communication with master-web via DNS. - Removing net_front from master-web; communication will be handled through NGINX. - NGINX access/error logs are written to the Docker-Compose relative path logs/nginx, which is needed for Zabbix collection. TODO Before Migration to PROD: - Address all FIXME comments. - Cross-reference proxy pass - helper_files directory name on hz-bbm2. - location /cloud-init ? TODO Before Deployment in DEV: - Disable the HAProxy service.
Configuration menu - View commit details
-
Copy full SHA for 6da2f1b - Browse repository at this point
Copy the full SHA 6da2f1bView commit details -
This will require some changes in Zabbix configuration for collecting these NGINX metrics. (replacing HAProxy metrics)
Configuration menu - View commit details
-
Copy full SHA for d2f8f13 - Browse repository at this point
Copy the full SHA d2f8f13View commit details -
Configure hosts file for CrossReference host
We don't have CrossReference in DEV, yet. For PROD I've configured the WG IP, for DEV it's just the loopback interface and will raise a BAD GATEWAY error.
Configuration menu - View commit details
-
Copy full SHA for 9f8f1bc - Browse repository at this point
Copy the full SHA 9f8f1bcView commit details
Commits on Oct 14, 2024
-
Use certbot container for letsencrypt certificates
- add certbot container to docker-compose - bind mount www path for certbot/nginx to use it in webroot mode - bind mount certboot cnf path to allow nginx find the certificates Fixes: - remove location /cloud-init as per review - remove NGINX_ARTIFACTS_SSL_PATH variable . Will use multiple domain names in PROD also. - adapt bb and ci NGINX conf files to handle the acme challenge on port 80 - fix generate-config to avoid exposing nginx env variables to other containers other than nginx
Configuration menu - View commit details
-
Copy full SHA for 4834fe4 - Browse repository at this point
Copy the full SHA 4834fe4View commit details
Commits on Oct 15, 2024
-
Configuration menu - View commit details
-
Copy full SHA for dfc86de - Browse repository at this point
Copy the full SHA dfc86deView commit details -
Configuration menu - View commit details
-
Copy full SHA for 673fe22 - Browse repository at this point
Copy the full SHA 673fe22View commit details -
Configuration menu - View commit details
-
Copy full SHA for 941b6cb - Browse repository at this point
Copy the full SHA 941b6cbView commit details -
Configuration menu - View commit details
-
Copy full SHA for 2283b32 - Browse repository at this point
Copy the full SHA 2283b32View commit details
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.