-
Notifications
You must be signed in to change notification settings - Fork 0
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
zero: add kustomize #4
base: main
Are you sure you want to change the base?
Conversation
- name: pomerium | ||
env: | ||
- name: TMPDIR | ||
value: "/tmp/pomerium" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you add a comment explaining the purpose of each tmp
folder? Having multiple makes the structure unclear.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
various parts of stdlib and other libraries we use make use of TMPDIR and XDG_CACHE
runAsGroup: 1000 | ||
runAsUser: 1000 | ||
sysctls: | ||
- name: net.ipv4.ip_unprivileged_port_start |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would it make sense to include port 443 here as well? Consider adding a comment explaining why certain ports are prioritized.
spec: | ||
containers: | ||
- name: pomerium | ||
resources: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you provide additional context on these resources? A comment explaining their purpose and how they’re used here and why they are the defaults would be helpful.
containers: | ||
- name: pomerium | ||
securityContext: | ||
readOnlyRootFilesystem: true |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Consider adding a comment explaining why this is important for security
containerPort: 80 | ||
protocol: TCP | ||
- name: metrics | ||
containerPort: 9090 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
consider explaining what each port is for (especially 80 and 9090 being for redirect and metrics)
automountServiceAccountToken: true | ||
serviceAccountName: pomerium-zero | ||
containers: | ||
- name: pomerium |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- name: pomerium | |
- name: pomerium | |
readinessProbe: | |
httpGet: | |
path: /healthz | |
port: 80 | |
initialDelaySeconds: 10 | |
periodSeconds: 5 | |
livenessProbe: | |
httpGet: | |
path: /healthz | |
port: 80 | |
initialDelaySeconds: 15 | |
periodSeconds: 10 |
Not sure this maps to true readyness /live however....
Co-authored-by: bobby <[email protected]>
Co-authored-by: bobby <[email protected]>
Summary
Adds Kustomization manifests that may be used to install Pomerium Zero directly via
kubectl
like:Related issues
Checklist
improvement
/bug
/ etc)