Skip to content

Conversation

@mdbooth
Copy link
Contributor

@mdbooth mdbooth commented Jun 19, 2025

This requires removing references to DeleteOnTermination, which was deprecated and renamed in openshift/api#2341.

@openshift-ci openshift-ci bot requested review from RadekManak and damdo June 19, 2025 15:22
@damdo
Copy link
Member

damdo commented Jun 19, 2025

Matt go mod tidy && go mod vendor shows issues

Copy link
Member

@damdo damdo left a comment

Choose a reason for hiding this comment

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

Thanks

/approve

left a comment about the go mod issue.

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jun 19, 2025

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: damdo

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

@openshift-ci openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jun 19, 2025
mdbooth added 3 commits June 19, 2025 17:52
This field has never been implemented in MAPI, and is now deprecated.
@damdo
Copy link
Member

damdo commented Jun 20, 2025

It looks like the fuzzy test is not happy, it still is expecting json matching for:

      "blockDevices": [
        {
          "ebs": {
            "deleteOnTermination": false,

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jul 10, 2025

@mdbooth: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/unit 1fd7572 link true /test unit
ci/prow/regression-clusterinfra-cucushift-rehearse-capi-aws-ipi 1fd7572 link false /test regression-clusterinfra-cucushift-rehearse-capi-aws-ipi
ci/prow/e2e-azure-ovn-techpreview 1fd7572 link false /test e2e-azure-ovn-techpreview
ci/prow/e2e-aws-ovn 1fd7572 link true /test e2e-aws-ovn
ci/prow/e2e-aws-ovn-techpreview-upgrade 1fd7572 link true /test e2e-aws-ovn-techpreview-upgrade

Full PR test history. Your PR dashboard.

Instructions 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.

@damdo
Copy link
Member

damdo commented Aug 11, 2025

@mdbooth this needs rebasing when you have some time now that #341 merged

@openshift-merge-robot openshift-merge-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Aug 11, 2025
@openshift-merge-robot
Copy link
Contributor

PR needs rebase.

Instructions 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.

@damdo
Copy link
Member

damdo commented Aug 28, 2025

The openshift/api has been bumped in the meantime already by #341
But we can still clean up the DeleteOnTermination references, now (DeprecatedDeleteOnTermination)
So if you can rebase and fix up that'd be great! TY

/retitle Removing references to DeleteOnTermination

@openshift-ci openshift-ci bot changed the title Bump openshift/api Removing references to DeleteOnTermination Aug 28, 2025
@shellyyang1989
Copy link

/retitle OCPCLOUD-2717: Removing references to DeleteOnTermination

@openshift-ci openshift-ci bot changed the title Removing references to DeleteOnTermination [OCPCLOUD-2717](https://issues.redhat.com/browse/OCPCLOUD-2717): Removing references to DeleteOnTermination Oct 15, 2025
@shellyyang1989
Copy link

shellyyang1989 commented Oct 15, 2025

/retitle OCPCLOUD-2717: Removing references to DeleteOnTermination

@openshift-ci openshift-ci bot changed the title [OCPCLOUD-2717](https://issues.redhat.com/browse/OCPCLOUD-2717): Removing references to DeleteOnTermination OCPCLOUD-2717: Removing references to DeleteOnTermination Oct 15, 2025
@openshift-ci-robot openshift-ci-robot added the jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. label Oct 15, 2025
@openshift-ci-robot
Copy link

openshift-ci-robot commented Oct 15, 2025

@mdbooth: This pull request references OCPCLOUD-2717 which is a valid jira issue.

Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the story to target the "4.21.0" version, but no target version was set.

In response to this:

This requires removing references to DeleteOnTermination, which was deprecated and renamed in openshift/api#2341.

Instructions 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 openshift-eng/jira-lifecycle-plugin repository.

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. jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants