Skip to content

Conversation

imeliran
Copy link
Contributor

No description provided.

@imeliran imeliran requested a review from moshiko-s May 25, 2025 12:37
| `javaAgents[].containerSelector` | Selector for containers within the deployment to inject the Lightrun Java Agent. | Required |
| `javaAgents[].deploymentName` | Name of the Kubernetes deployment to attach the Lightrun Java Agent. | Required |
| `javaAgents[].initContainer.image` | Image for the Lightrun Java Agent init container. | Required |
| `javaAgents[].initContainer.imagePullPolicy` | Image pull policy for the init container. Can be one of: Always, IfNotPresent, or Never. | Optional (if not provided, defaults to `"IfNotPresent"`) |
Copy link
Contributor

Choose a reason for hiding this comment

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

the default depends on the image tag, no?
I would provide a link to the official docs

spec:
initContainer:
image: {{ .initContainer.image }}
imagePullPolicy: {{ .initContainer.imagePullPolicy | default "IfNotPresent" }}
Copy link
Contributor

Choose a reason for hiding this comment

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

can't we omit it if not provided? to use k8s defaults

imeliran and others added 2 commits June 3, 2025 12:50
@imeliran imeliran requested a review from moshiko-s June 3, 2025 09:55
moshiko-s
moshiko-s previously approved these changes Jun 11, 2025
…-operator-add-ability-to-set-image-pull-policy-for-init-container
@imeliran imeliran merged commit c174820 into main Jun 11, 2025
2 checks passed
@imeliran imeliran deleted the DEVOPS-2693-lightrun-k-8-s-operator-add-ability-to-set-image-pull-policy-for-init-container branch June 11, 2025 15:08
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.

2 participants