Skip to content

Conversation

jeffbearer
Copy link

Change imagePullPolicy from IfNotPresent to Always in both CSI driver deployment manifests to ensure automatic pickup of new patch releases without requiring manual intervention. Internal workitem https://msazure.visualstudio.com/One/_workitems/edit/33532215

Changes:

  • deploy/csi-azurelustre-controller.yaml: azurelustre container
  • deploy/csi-azurelustre-node.yaml: azurelustre container

What type of PR is this?
/kind cleanup

What this PR does / why we need it:

This PR changes the imagePullPolicy from IfNotPresent to Always for the azurelustre CSI driver containers in both the controller and node deployments. This ensures that Kubernetes will automatically pull the latest container images when pods are created or restarted, allowing for automatic pickup of new patch releases without requiring manual intervention or pod restarts.

This change improves the operational experience by ensuring that patch updates (like security fixes or bug fixes) are automatically applied when available, reducing the need for manual image updates in production environments.

Which issue(s) this PR fixes:

Fixes #

Requirements:

  • uses conventional commit messages
  • includes documentation (no documentation changes needed - change is in deployment manifests only)
  • adds unit tests (no unit tests needed - YAML configuration change)
  • tested upgrade from previous version (YAML validation performed)

Special notes for your reviewer:

  • This change only affects the azurelustre container in both deployment manifests
  • YAML syntax and structure validation has been performed to ensure no breaking changes
  • The change is backwards compatible and will take effect on the next pod restart/deployment
  • No other container configurations or image references were modified

Release note:

@k8s-ci-robot k8s-ci-robot added do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. kind/cleanup Categorizes issue or PR as related to cleaning up code, process, or technical debt. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. labels Sep 11, 2025
@k8s-ci-robot k8s-ci-robot added needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Sep 11, 2025
@jeffbearer jeffbearer force-pushed the feature/alwayspull-33532215 branch from 0e9bff6 to 62a1d57 Compare September 12, 2025 17:14
@k8s-ci-robot k8s-ci-robot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Sep 12, 2025
Change imagePullPolicy from IfNotPresent to Always in both CSI driver
deployment manifests to ensure automatic pickup of new patch releases
without requiring manual intervention.

Changes:
- deploy/csi-azurelustre-controller.yaml: azurelustre container
- deploy/csi-azurelustre-node.yaml: azurelustre container
@jeffbearer jeffbearer force-pushed the feature/alwayspull-33532215 branch from 62a1d57 to 41ecd7b Compare September 15, 2025 15:45
@jeffbearer jeffbearer marked this pull request as ready for review September 15, 2025 15:51
@k8s-ci-robot k8s-ci-robot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Sep 15, 2025
@coveralls
Copy link

Pull Request Test Coverage Report for Build 17738811371

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 86.623%

Totals Coverage Status
Change from base Build 17655781429: 0.0%
Covered Lines: 1839
Relevant Lines: 2123

💛 - Coveralls

Copy link
Collaborator

@dabradley dabradley left a comment

Choose a reason for hiding this comment

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

/lgtm

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Sep 16, 2025
@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: dabradley, jeffbearer

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@k8s-ci-robot k8s-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Sep 16, 2025
@k8s-ci-robot k8s-ci-robot merged commit 43465bd into kubernetes-sigs:development Sep 16, 2025
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. kind/cleanup Categorizes issue or PR as related to cleaning up code, process, or technical debt. lgtm "Looks good to me", indicates that a PR is ready to be merged. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants