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

Elastic deployment improvement #66

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
+12 −3
Open

Elastic deployment improvement #66

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
73cae8a
Fix NOTIFY_INBOX_ONLY value in workspace deployment
0xtejas Oct 14, 2024
6203ebb
Update elastic-deployment.yaml with namespace, security context, and …
0xtejas Oct 14, 2024
1b81199
Revert "Update elastic-deployment.yaml with namespace, security conte…
0xtejas Oct 14, 2024
ab9759f
Update elastic-deployment.yaml with minor improvements
0xtejas Oct 14, 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
13 changes: 11 additions & 2 deletions kube/elastic/elastic-deployment.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -16,11 +16,17 @@ spec:
labels:
app: elastic
spec:
securityContext:
runAsUser: 1000
runAsGroup: 1000
fsGroup: 1000
containers:
- args:
- /bin/sh
- -c
- |-
chown -R 1000:1000 /usr/share/elasticsearch/data;
apt-get update && apt-get install -y curl;
./bin/elasticsearch-plugin list | grep -q ingest-attachment || yes | ./bin/elasticsearch-plugin install --silent ingest-attachment;
/usr/local/bin/docker-entrypoint.sh eswrapper
env:
Expand All @@ -40,9 +46,12 @@ spec:
livenessProbe:
exec:
command:
- /bin/sh
- -c
- curl -s http://localhost:9200/_cluster/health | grep -vq '"status":"red"'
failureThreshold: 10
initialDelaySeconds: 60
periodSeconds: 20
failureThreshold: 10
name: elastic
ports:
- containerPort: 9200
Expand All @@ -55,4 +64,4 @@ spec:
volumes:
- name: elastic
persistentVolumeClaim:
claimName: elastic
claimName: elastic
2 changes: 1 addition & 1 deletion kube/workspace/workspace-deployment.yaml
View file
Open in desktop
Copy link
Contributor Author

@0xtejas 0xtejas Oct 23, 2024 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please discard these changes in the file kube/workspace/workspace-deployment.yaml when merging if possible. If that isn't possible, please let me know, as it has been removed in another PR.

Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,7 @@ spec:
- name: TRANSACTOR_URL
value: ws://transactor:3333;ws://localhost:3333
- name: NOTIFY_INBOX_ONLY
value: true
value: "true"
image: hardcoreeng/workspace:latest
name: workspace
resources:
Expand Down