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.

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

(redis) Add to blocky for #785 #1403

Merged
merged 1 commit into from
Aug 19, 2022
Merged

(redis) Add to blocky for #785 #1403

merged 1 commit into from
Aug 19, 2022

Conversation

h3mmy
Copy link
Owner

@h3mmy h3mmy commented Aug 19, 2022

Description of the change

Adds Redis for blocky

Benefits

Less network load due to unnecessary duplication

Possible drawbacks

Might need to prevent blocky from running on the ARM nodes.

Applicable issues

Additional information

@h3mmy h3mmy self-assigned this Aug 19, 2022
@bloopy-boi bloopy-boi bot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. area/cluster Changes made in the cluster directory labels Aug 19, 2022
@h3mmy h3mmy added enhancement New feature or request size/L Denotes a PR that changes 100-499 lines, ignoring generated files. area/cluster Changes made in the cluster directory and removed size/L Denotes a PR that changes 100-499 lines, ignoring generated files. area/cluster Changes made in the cluster directory labels Aug 19, 2022
@bloopy-boi
Copy link
Contributor

bloopy-boi bot commented Aug 19, 2022

Path: cluster/apps/networking/blocky/helm-release.yaml

@@ -87,6 +87,11 @@
 # - 192.168.178.29
 caching:
 prefetching: true
+ redis:
+ address: blocky-redis-master.networking:6379
+ password: ${BLOCKY_REDIS_PASSWORD}
+ database: 2
+ required: true
 prometheus:
 enable: true
 path: /metrics

@bloopy-boi
Copy link
Contributor

bloopy-boi bot commented Aug 19, 2022

MegaLinter status: ✅ SUCCESS

Descriptor Linter Files Fixed Errors Elapsed time
✅ COPYPASTE jscpd yes no 0.97s
✅ KUBERNETES kubeval 5 0 0.94s
✅ YAML prettier 5 2 0 0.67s
✅ YAML yamllint 5 0 0.14s

See errors details in artifact MegaLinter reports on CI Job page
Set VALIDATE_ALL_CODEBASE: true in mega-linter.yml to validate all sources, not only the diff

@h3mmy h3mmy merged commit 88acba1 into main Aug 19, 2022
@h3mmy h3mmy deleted the redis-chart branch August 19, 2022 03:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/cluster Changes made in the cluster directory enhancement New feature or request size/L Denotes a PR that changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Redis Operator
1 participant