Skip to content

Commit 53b7bbf

Browse files
committed
fix(deployment): Fix naming and some conditions
1 parent 3db0513 commit 53b7bbf

File tree

2 files changed

+10
-14
lines changed

2 files changed

+10
-14
lines changed

.github/workflows/ci.yml

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,7 @@ jobs:
5959
helm-validate:
6060
runs-on: ubuntu-latest
6161
name: Validate Helm Chart
62-
if: github.event_name == 'push'
62+
if: github.event_name == 'pull_request'
6363

6464
steps:
6565
- uses: actions/checkout@v4
@@ -96,7 +96,7 @@ jobs:
9696
helm-unittest:
9797
runs-on: ubuntu-latest
9898
name: Run Helm Unit Tests
99-
if: github.event_name == 'push'
99+
if: github.event_name == 'pull_request'
100100

101101
steps:
102102
- uses: actions/checkout@v4
@@ -108,7 +108,7 @@ jobs:
108108

109109
- name: Install Helm Unittest Plugin
110110
run: |
111-
helm plugin install https://github.com/helm-unittest/helm-unittest.git
111+
helm plugin install https://github.com/helm-unittest/helm-unittest.git --verify=false
112112
113113
- name: Run Helm unit tests
114114
run: |
@@ -118,7 +118,6 @@ jobs:
118118
docker-build:
119119
runs-on: ubuntu-latest
120120
name: Build Docker Image
121-
if: github.event_name == 'pull_request'
122121

123122
steps:
124123
- name: Checkout repository

.github/workflows/release.yml

Lines changed: 7 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ on:
99
workflow_dispatch:
1010

1111
env:
12-
IMAGE_NAME: secops-mcp-server
12+
IMAGE_NAME: mcp-server
1313
REGISTRY: ghcr.io
1414

1515
jobs:
@@ -108,11 +108,8 @@ jobs:
108108
publish-helm-chart:
109109
runs-on: ubuntu-latest
110110
name: Publish Helm Chart
111-
needs: [detect-changes, build-and-push-docker]
112-
if: |
113-
always() &&
114-
(github.event_name == 'push' && (github.ref_type == 'tag' || github.ref == 'refs/heads/main')) &&
115-
(needs.detect-changes.outputs.chart-changed == 'true' || github.ref_type == 'tag')
111+
needs: [ detect-changes, build-and-push-docker ]
112+
if: github.event_name == 'push' && (github.ref_type == 'tag' || github.ref == 'refs/heads/main')
116113
permissions:
117114
contents: read
118115
packages: write
@@ -168,14 +165,14 @@ jobs:
168165
CHART_VERSION=$(grep "^version:" ./helm/mcp-server/Chart.yaml | cut -d' ' -f2 | tr -d '"')
169166
echo "## ✅ Helm Chart Released" >> $GITHUB_STEP_SUMMARY
170167
echo "" >> $GITHUB_STEP_SUMMARY
171-
echo "Chart: gitguardian-secops-mcp-server" >> $GITHUB_STEP_SUMMARY
168+
echo "Chart: mcp-server" >> $GITHUB_STEP_SUMMARY
172169
echo "Version: ${CHART_VERSION}" >> $GITHUB_STEP_SUMMARY
173170
echo "" >> $GITHUB_STEP_SUMMARY
174171
echo "Published to: \`oci://${{ env.REGISTRY }}/${{ github.repository_owner }}/helm-charts\`" >> $GITHUB_STEP_SUMMARY
175172
echo "" >> $GITHUB_STEP_SUMMARY
176173
echo "Install with:" >> $GITHUB_STEP_SUMMARY
177174
echo '```bash' >> $GITHUB_STEP_SUMMARY
178-
echo "helm pull oci://${{ env.REGISTRY }}/${{ github.repository_owner }}/helm-charts/gitguardian-secops-mcp-server --version ${CHART_VERSION}" >> $GITHUB_STEP_SUMMARY
175+
echo "helm pull oci://${{ env.REGISTRY }}/${{ github.repository_owner }}/helm-charts/mcp-server --version ${CHART_VERSION}" >> $GITHUB_STEP_SUMMARY
179176
echo '```' >> $GITHUB_STEP_SUMMARY
180177
181178
publish-to-pypi:
@@ -235,7 +232,7 @@ jobs:
235232
publish-to-mcp-registry:
236233
runs-on: ubuntu-latest
237234
name: Publish to MCP Registry
238-
needs: [publish-to-pypi]
235+
needs: [ publish-to-pypi ]
239236
if: |
240237
false &&
241238
always() &&
@@ -302,7 +299,7 @@ jobs:
302299
create-github-release:
303300
runs-on: ubuntu-latest
304301
name: Create GitHub Release
305-
needs: [publish-helm-chart, publish-to-pypi]
302+
needs: [ publish-helm-chart, publish-to-pypi ]
306303
if: |
307304
always() &&
308305
github.event_name == 'push' &&

0 commit comments

Comments
 (0)