Skip to content

Conversation

@wasaga
Copy link
Contributor

@wasaga wasaga commented Oct 25, 2024

Summary

Adds Kustomization manifests that may be used to install Pomerium Zero directly via kubectl like:

kubectl apply -k github.com/pomerium/install/zero/kustomize

Related issues

Checklist

  • reference any related issues
  • add appropriate tag (improvement / bug / etc)
  • ready for review

@wasaga wasaga requested a review from a team as a code owner October 25, 2024 22:37
@wasaga wasaga requested a review from kralicky October 25, 2024 22:37
- name: pomerium
env:
- name: TMPDIR
value: "/tmp/pomerium"
Copy link
Contributor

@desimone desimone Oct 25, 2024

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.

Copy link
Contributor Author

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
Copy link
Contributor

@desimone desimone Oct 25, 2024

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:
Copy link
Contributor

@desimone desimone Oct 25, 2024

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
Copy link
Contributor

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
Copy link
Contributor

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
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
- 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....

wasaga and others added 2 commits October 31, 2024 10:53
Co-authored-by: bobby <[email protected]>
Co-authored-by: bobby <[email protected]>
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.

3 participants