Bugfix - bug fix for baremetal support (#127)

- add image pull secret for redis and copilot-chat
- all dockerfile of alertmanager
- fix the bug when remote_log_dir is null

Author: yukirora

.github/workflows/build-all.yaml

@@ -5,7 +5,9 @@ permissions:
 
 on:
   push:
-    branches: ['release/*']
+    branches: ["release/*"]
+  pull_request:
+    branches: ["release/*"]
   release:
     types: [published]
   workflow_dispatch:

src/alert-manager/build/alertmanager.common.dockerfile

@@ -0,0 +1,4 @@
+# Copyright (c) Microsoft Corporation.
+# Licensed under the MIT License.
+
+FROM prom/alertmanager:v0.29.0

src/alert-manager/deploy/alert-manager-deployment.yaml.template

@@ -42,6 +42,7 @@ spec:
       containers:
       - name: alertmanager
         image: {{ cluster_cfg['cluster']['docker-registry']['prefix'] }}alertmanager:{{ cluster_cfg['cluster']['docker-registry']['tag'] }}
+        imagePullPolicy: Always
         args:
           - '--config.file=/etc/alertmanager/config.yml'
           - '--storage.path=/alertmanager'

src/alert-manager/deploy/redis-deployment.yaml.template

@@ -72,11 +72,13 @@ spec:
           periodSeconds: 10
           successThreshold: 1
           failureThreshold: 3
+      imagePullSecrets:
+      - name: {{ cluster_cfg["cluster"]["docker-registry"]["secret-name"] }}
       volumes:
       - name: redis-data
         hostPath:
           path: /data/redis-data
           type: ""
       - name: redis-config
         configMap:
-          name: redis-config
+          name: redis-config

src/copilot-chat/deploy/copilot-chat-deployment.yaml.template

@@ -101,6 +101,8 @@ spec:
         securityContext:
           runAsUser: 0
           runAsGroup: 0
+      imagePullSecrets:
+      - name: {{ cluster_cfg["cluster"]["docker-registry"]["secret-name"] }}
       volumes:
       - name: copilot-chat-logs
         emptyDir: {}

src/log-manager/src/nginx/get_log_content.lua

@@ -70,12 +70,12 @@ end
 
 local log_path = log_dir..log_name
 local use_remote_log_dir = false
-if path.isdir(log_path) and util.is_directory_empty(log_path) then
+if path.isdir(log_path) and util.is_directory_empty(log_path) and remote_log_dir then
   ngx.log(ngx.INFO, "falling back to remote log directory")
   log_path = remote_log_dir..log_name
   log_dir = remote_log_dir
   use_remote_log_dir = true
-elseif not path.exists(log_path) then
+elseif not path.exists(log_path) and remote_log_dir then
   ngx.log(ngx.INFO, "path not exist, falling back to remote log directory")
   log_path = remote_log_dir..log_name
   log_dir = remote_log_dir
@@ -120,3 +120,4 @@ else
   ngx.req.set_uri("/~/current/"..string.sub(path.abspath(log_path), string.len(file_prefix) + 1), true)
 end
 
+