Remove edpm_iscsid_image container references#1923
Conversation
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: katarimanojk The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
OpenStackControlPlane CRD Size Report
Threshold reference
|
The iscsid service has been moved from container to host-based deployment in edpm-ansible (PR openstack-k8s-operators#984). This change removes or deprecates all references to EdpmIscsidImage throughout the codebase: - Removed from service containerImageFields (nova, run-os) - Removed from inventory variable assignment - Deprecated API fields with comments (preserves backward compatibility) - Updated sample configurations with deprecation notes - Removed from documentation - Commented out build-time image variables - Updated tests to remove EdpmIscsidImage assertions The EdpmIscsidImage field remains in the API types but is marked as deprecated to avoid breaking changes for existing deployments. Related: openstack-k8s-operators/edpm-ansible#984 Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
63799fb to
6c7fc49
Compare
|
@katarimanojk: The following tests failed, say
Full PR test history. Your PR dashboard. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here. |
The iscsid service has been moved from container to host-based deployment in edpm-ansible (PR #984). This change removes or deprecates all references to EdpmIscsidImage throughout the codebase:
The EdpmIscsidImage field remains in the API types but is marked as deprecated to avoid breaking changes for existing deployments.
Related: openstack-k8s-operators/edpm-ansible#984
Closes: https://redhat.atlassian.net/browse/OSPRH-17429