Skip to content

Cleanup EdpmMultipathdImage container references#1928

Draft
katarimanojk wants to merge 1 commit into
openstack-k8s-operators:mainfrom
katarimanojk:multipathd_container_ref_cleanup
Draft

Cleanup EdpmMultipathdImage container references#1928
katarimanojk wants to merge 1 commit into
openstack-k8s-operators:mainfrom
katarimanojk:multipathd_container_ref_cleanup

Conversation

@katarimanojk
Copy link
Copy Markdown

multipathd service has been moved from container to host in edpm-ansible. Clean up the related EdpmMultipathdImage container references following the same pattern as EdpmIscsidImage cleanup:

  • Mark EdpmMultipathdImage field as deprecated in API types with appropriate comments
  • Remove edpm_multipathd_image from inventory variable assignments
  • Comment out RELATED_IMAGE_EDPM_MULTIPATHD_IMAGE_URL_DEFAULT in build configuration
  • Remove edpm_multipathd_image from documentation examples
  • Remove EdpmMultipathdImage test assertions
  • Remove edpmMultipathdImage field documentation

The API fields are preserved but marked as deprecated to maintain backward compatibility. All usage of the field has been removed from the codebase.

multipathd service has been moved from container to host in
edpm-ansible. Clean up the related EdpmMultipathdImage container
references following the same pattern as EdpmIscsidImage cleanup:

- Mark EdpmMultipathdImage field as deprecated in API types with
  appropriate comments
- Remove edpm_multipathd_image from inventory variable assignments
- Comment out RELATED_IMAGE_EDPM_MULTIPATHD_IMAGE_URL_DEFAULT in
  build configuration
- Remove edpm_multipathd_image from documentation examples
- Remove EdpmMultipathdImage test assertions
- Remove edpmMultipathdImage field documentation

The API fields are preserved but marked as deprecated to maintain
backward compatibility. All usage of the field has been removed
from the codebase.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
@openshift-ci openshift-ci Bot requested review from dprince and fultonj May 21, 2026 13:05
@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented May 21, 2026

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: katarimanojk
Once this PR has been reviewed and has the lgtm label, please assign slagle for approval. For more information see the Code Review Process.

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

Details 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

@github-actions
Copy link
Copy Markdown

OpenStackControlPlane CRD Size Report

Metric Value
CRD JSON size 333227 bytes (325KB)
Base branch size 333227 bytes
Change +0.00%
Status yellow — growing
Threshold reference
Color Range Meaning
🟢 green < 300KB Comfortable
🟡 yellow 300–400KB Growing
🟠 orange 400–750KB Concerning
🔴 red > 750KB Approaching 1.5MB etcd limit (cut in half to allow space for update)

@katarimanojk katarimanojk marked this pull request as draft May 21, 2026 13:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant