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.

Sign up for GitHub

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

Remove node6 for CRIB chart values #13003

Merged
merged 8 commits into from
May 2, 2024
Merged

Remove node6 for CRIB chart values #13003

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
fb6bb1e
Remove node6 for CRIB chart values
chainchad Apr 26, 2024
e96f5c8
Use future helm chart and correct path to values-profiles override
chainchad Apr 26, 2024
c14be6e
Add back .gitignore
chainchad Apr 26, 2024
7f618a1
Update unnecessary trigger on ci-scripts owrkflow
chainchad Apr 29, 2024
1677836
Revert "Update unnecessary trigger on ci-scripts owrkflow"
chainchad May 2, 2024
00d530b
Merge branch 'develop' into re-2541/remove-node
chainchad May 2, 2024
6760284
Merge branch 'develop' into re-2541/remove-node
gheorghestrimtu May 2, 2024
499507d
Merge branch 'develop' into re-2541/remove-node
tateexon May 2, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 0 additions & 1 deletion .github/scripts/crib/pr-comment-crib-env.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,6 @@ function generateSubdomains(subdomainPrefix, prNumber) {
"node3",
"node4",
"node5",
"node6",
"geth-1337-http",
"geth-1337-ws",
"geth-2337-http",
Expand Down
3 changes: 3 additions & 0 deletions crib/.gitignore
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,3 @@
values-profiles/*
!values-profiles/values-dev.yaml.example
!values-profiles/README.md
17 changes: 3 additions & 14 deletions crib/devspace.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -74,7 +74,7 @@ images:
docker push $image
hooks:
- command: ./scripts/check_env_vars.sh
events: [ "before:deploy:app" ]
events: ["before:deploy:app"]
- wait:
running: true
terminatedWithCode: 0
Expand All @@ -100,7 +100,7 @@ deployments:
releaseName: "app"
chart:
name: ${CHAINLINK_CLUSTER_HELM_CHART_URI}
version: 0.5.0
version: 0.6.0
# for simplicity, we define all the values here
# they can be defined the same way in values.yml
# devspace merges these "values" with the "values.yaml" before deploy
Expand Down Expand Up @@ -217,8 +217,6 @@ deployments:
image: ${runtime.images.app}
- name: node-5
image: ${runtime.images.app}
- name: node-6
image: ${runtime.images.app}

# each CL node have a dedicated PostgreSQL 11.15
# use StatefulSet by setting:
Expand Down Expand Up @@ -348,15 +346,6 @@ deployments:
name: app-node-5
port:
number: 6688
- host: ${DEVSPACE_NAMESPACE}-node6.${DEVSPACE_INGRESS_BASE_DOMAIN}
http:
paths:
- path: /
backend:
service:
name: app-node-6
port:
number: 6688
- host: ${DEVSPACE_NAMESPACE}-geth-1337-http.${DEVSPACE_INGRESS_BASE_DOMAIN}
http:
paths:
Expand Down Expand Up @@ -432,4 +421,4 @@ profiles:
path: deployments.app.helm.values.chainlink.global.overridesToml
- op: add
path: deployments.app.helm.valuesFiles
value: ["../charts/chainlink-cluster/values-profiles/values-dev.yaml"]
value: ["./values-profiles/values-dev.yaml"]
Loading