Skip to content

Add executor Cilium network policy guard#870

Open
marcleblanc2 wants to merge 1 commit into
mainfrom
add-executor-cilium-network-policy
Open

Add executor Cilium network policy guard#870
marcleblanc2 wants to merge 1 commit into
mainfrom
add-executor-cilium-network-policy

Conversation

@marcleblanc2
Copy link
Copy Markdown
Contributor

Summary

  • add optional CiliumNetworkPolicy deny guards for executor controller and job pods
  • select executor-created job pods by sourcegraph/job-id and sourcegraph/run-id instead of sourcegraph/queue
  • deny other Sourcegraph pods, sourcegraph-frontend-internal, and frontend internal/debug ports while leaving existing frontend/code-host/DNS egress policies intact

Test plan

  • helm template sourcegraph-executor charts/sourcegraph-executor/k8s --namespace executor-controller --set executor.queueNames='{batches,codeintel}' --set executor.namespace=executor-jobs --set executor.frontendPassword=dummy --set executor.frontendUrl=http://sourcegraph-frontend.sourcegraph.svc.cluster.local:30080 --set executor.ciliumNetworkPolicy.enabled=true --set executor.ciliumNetworkPolicy.sourcegraphNamespace=sourcegraph
  • helm template sourcegraph-executor charts/sourcegraph-executor/k8s --namespace executor-controller --set executor.queueNames='{batches,codeintel}' --set executor.frontendPassword=dummy --set executor.frontendUrl=http://sourcegraph-frontend:30080 | rg -n "CiliumNetworkPolicy|cilium.io" || true
  • helm lint charts/sourcegraph-executor/k8s --set executor.queueNames='{batches,codeintel}' --set executor.frontendPassword=dummy --set executor.frontendUrl=http://sourcegraph-frontend.sourcegraph.svc.cluster.local:30080 --set executor.ciliumNetworkPolicy.enabled=true --set executor.ciliumNetworkPolicy.sourcegraphNamespace=sourcegraph
  • ./scripts/helm-docs.sh
  • ruby -e 'require "yaml"; YAML.load_stream(File.read("/tmp/sourcegraph-executor-cilium-render.yaml")); puts "render yaml ok"'

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant