Skip to content

Commit 6309230

Browse files
authored
Merge pull request #8972 from upodroid/argocd-differ-patch
fix argocd-differ
2 parents d294cb7 + f3fd71f commit 6309230

File tree

3 files changed

+68
-3
lines changed

3 files changed

+68
-3
lines changed

.github/workflows/argocd-diff.yaml

Lines changed: 13 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -28,15 +28,26 @@ jobs:
2828

2929
- name: Generate Diff
3030
run: |
31+
# Kustomize and Helm don't work by default with argocd-diff, so we need to amend argocd a bit
32+
kind create cluster
33+
helm repo add argo https://argoproj.github.io/argo-helm
34+
helm upgrade --install argocd argo/argo-cd --version 9.3.4 --create-namespace --namespace argocd-diff-preview \
35+
--set config.params."kustomize\.buildOptions"="--load-restrictor LoadRestrictionsNone --enable-alpha-plugins"
36+
kubectl apply -n argocd-diff-preview -f pull-request/kubernetes/gke-utility/argocd/clusters.yaml || true # ignore errors from missing ES
37+
kubectl apply -n argocd-diff-preview -f pull-request/kubernetes/gke-utility/argocd/clusters-diff.yaml # ES based clusters
38+
3139
docker run \
3240
--network=host \
41+
-v ~/.kube:/root/.kube \
3342
-v /var/run/docker.sock:/var/run/docker.sock \
3443
-v $(pwd)/main:/base-branch \
3544
-v $(pwd)/pull-request:/target-branch \
3645
-v $(pwd)/output:/output \
3746
-e TARGET_BRANCH=refs/pull/${{ github.event.number }}/merge \
3847
-e REPO=${{ github.repository }} \
39-
dagandersen/argocd-diff-preview:v0.1.19
48+
dagandersen/argocd-diff-preview:v0.1.21 \
49+
--argocd-namespace=argocd-diff-preview \
50+
--create-cluster=false
4051
4152
- name: Upload artifact
4253
uses: actions/upload-artifact@b7c566a772e6b6bfb58ed0dc250532a479d7789f #v6.0.0
@@ -57,5 +68,4 @@ jobs:
5768
env:
5869
GH_TOKEN: ${{ secrets.K8S_INFRA_CI_BOT_PR_TOKEN }}
5970
run: |
60-
gh pr comment ${{ github.event.number }} --repo ${{ github.repository }} --body-file output/diff.md --edit-last || \
61-
gh pr comment ${{ github.event.number }} --repo ${{ github.repository }} --body-file output/diff.md
71+
gh pr comment ${{ github.event.number }} --repo ${{ github.repository }} --body-file output/diff.md --edit-last --create-if-none

kubernetes/eks-prow-build/datadog/values.yaml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@ datadog:
77
logs:
88
enabled: true
99
containerCollectAll: true
10+
autoMultiLineDetection: true
1011
prometheusScrape:
1112
enabled: true
1213
serviceEndpoints: true
@@ -24,6 +25,9 @@ datadog:
2425
uncompressedLayersSupport: true
2526
host:
2627
enabled: true
28+
apm:
29+
instrumentation:
30+
skipKPITelemetry: true # https://github.com/DataDog/helm-charts/issues/1395
2731
operator:
2832
enabled: false
2933
kubernetesResourcesLabelsAsTags:
Lines changed: 51 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,51 @@
1+
# This file is used by the argocd-diff github actions workflow, never add it kustomization.yaml
2+
# The IBM clusters are loaded via ExternalSecrets which the ephemeral argocd-diff-preview cluster
3+
# cannot access. Therefore we create the same clusters with server set to localhost:8080
4+
# and random username/password so it appears as a valid cluster and we can generate the appsets
5+
apiVersion: v1
6+
kind: Secret
7+
metadata:
8+
name: ibm-ppc64le
9+
labels:
10+
name: ibm-ppc64le
11+
argocd.argoproj.io/secret-type: cluster
12+
clusterType: prow
13+
environment: prod
14+
prowNamespace: test-pods
15+
cloud: ibm
16+
type: Opaque
17+
stringData:
18+
name: ibm-ppc64le
19+
server: http://localhost:8080
20+
config: |
21+
{
22+
"username": "foobar",
23+
"password": "string",
24+
"tlsClientConfig": {
25+
"insecure": true
26+
}
27+
}
28+
---
29+
apiVersion: v1
30+
kind: Secret
31+
metadata:
32+
name: ibm-s390x
33+
labels:
34+
name: ibm-s390x
35+
argocd.argoproj.io/secret-type: cluster
36+
clusterType: prow
37+
environment: prod
38+
prowNamespace: test-pods
39+
cloud: ibm
40+
type: Opaque
41+
stringData:
42+
name: ibm-s390x
43+
server: http://localhost:8080
44+
config: |
45+
{
46+
"username": "foobar",
47+
"password": "string",
48+
"tlsClientConfig": {
49+
"insecure": true
50+
}
51+
}

0 commit comments

Comments
 (0)