Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Delete leftover dns records when destroying ibmcloud clusters #11300

Merged

Conversation

clacroix12
Copy link
Contributor

@clacroix12 clacroix12 added Needs Deployment Testing Needs Deployment and Some Tier1 Testing team/ecosystem Ecosystem team related issues/PRs labels Feb 4, 2025
@clacroix12 clacroix12 requested a review from a team February 4, 2025 23:22
@clacroix12 clacroix12 requested a review from a team as a code owner February 4, 2025 23:22
@pull-request-size pull-request-size bot added the size/M PR that changes 30-99 lines label Feb 4, 2025
Copy link

@ocs-ci ocs-ci left a comment

Choose a reason for hiding this comment

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

PR validation

Cluster Name:
Cluster Configuration:
PR Test Suite: deployment
PR Test Path: tests/
Additional Test Params:
OCP VERSION: 4.18
OCS VERSION: 4.18
tested against branch: master

Job FAILED (installation failed, tests not executed).

vavuthu
vavuthu previously approved these changes Feb 5, 2025
Copy link
Contributor

@dahorak dahorak left a comment

Choose a reason for hiding this comment

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

Just one small suggestion, otherwise LGTM.

ocs_ci/ocs/constants.py Outdated Show resolved Hide resolved
ocs_ci/ocs/constants.py Outdated Show resolved Hide resolved
ocs_ci/utility/ibmcloud.py Show resolved Hide resolved
ocs_ci/utility/ibmcloud.py Show resolved Hide resolved
ocs_ci/utility/ibmcloud.py Outdated Show resolved Hide resolved
Copy link

@ocs-ci ocs-ci left a comment

Choose a reason for hiding this comment

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

PR validation

Cluster Name:
Cluster Configuration:
PR Test Suite: deployment
PR Test Path: tests/
Additional Test Params:
OCP VERSION: 4.18
OCS VERSION: 4.18
tested against branch: master

Job PASSED.

Signed-off-by: Coady LaCroix <[email protected]>
@clacroix12
Copy link
Contributor Author

Verified and confirmed the leftover record was deleted here: https://jenkins-csb-odf-qe-ocs4.dno.corp.redhat.com/job/qe-destroy-ocs-cluster/52685/consoleFull

The only issue was incorrectly referencing the wrong list in the log statement showing which IDs were found for deletion. I've pushed a fix addressing that.

@clacroix12 clacroix12 added Verified Mark when PR was verified and log provided and removed Needs Deployment Testing Needs Deployment and Some Tier1 Testing labels Feb 5, 2025
@clacroix12
Copy link
Contributor Author

I should also mention that this shouldn't be merged prior to the MR adding the config details to our credentials.

Copy link

openshift-ci bot commented Feb 7, 2025

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: clacroix12, dahorak, petr-balogh, vavuthu

The full list of commands accepted by this bot can be found 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

@clacroix12 clacroix12 merged commit 357b61a into red-hat-storage:master Feb 7, 2025
6 of 7 checks passed
@clacroix12 clacroix12 deleted the ibm-cloud-delete-dns-records branch February 7, 2025 18:11
@clacroix12
Copy link
Contributor Author

/cherry-pick 4.17

@openshift-cherrypick-robot
Copy link
Collaborator

@clacroix12: cannot checkout 4.17: error checking out "4.17": exit status 1 error: pathspec '4.17' did not match any file(s) known to git

In response to this:

/cherry-pick 4.17

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.

@clacroix12
Copy link
Contributor Author

/cherry-pick release-4.17

@openshift-cherrypick-robot
Copy link
Collaborator

@clacroix12: new pull request created: #11313

In response to this:

/cherry-pick release-4.17

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.

@clacroix12
Copy link
Contributor Author

/cherry-pick release-4.16

@openshift-cherrypick-robot
Copy link
Collaborator

@clacroix12: #11300 failed to apply on top of branch "release-4.16":

Applying: Delete leftover dns records when destroying ibmcloud clusters
Using index info to reconstruct a base tree...
M	ocs_ci/deployment/ibmcloud.py
M	ocs_ci/ocs/constants.py
M	ocs_ci/utility/ibmcloud.py
Falling back to patching base and 3-way merge...
Auto-merging ocs_ci/utility/ibmcloud.py
CONFLICT (content): Merge conflict in ocs_ci/utility/ibmcloud.py
Auto-merging ocs_ci/ocs/constants.py
Auto-merging ocs_ci/deployment/ibmcloud.py
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
hint: When you have resolved this problem, run "git am --continue".
hint: If you prefer to skip this patch, run "git am --skip" instead.
hint: To restore the original branch and stop patching, run "git am --abort".
hint: Disable this message with "git config advice.mergeConflict false"
Patch failed at 0001 Delete leftover dns records when destroying ibmcloud clusters

In response to this:

/cherry-pick release-4.16

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.

clacroix12 added a commit to clacroix12/ocs-ci that referenced this pull request Feb 7, 2025
…lete-dns-records

Delete leftover dns records when destroying ibmcloud clusters

Signed-off-by: Coady LaCroix <[email protected]>
clacroix12 added a commit to clacroix12/ocs-ci that referenced this pull request Feb 7, 2025
…lete-dns-records

Delete leftover dns records when destroying ibmcloud clusters

Signed-off-by: Coady LaCroix <[email protected]>
clacroix12 added a commit to clacroix12/ocs-ci that referenced this pull request Feb 7, 2025
…lete-dns-records

Delete leftover dns records when destroying ibmcloud clusters

Signed-off-by: Coady LaCroix <[email protected]>
@clacroix12
Copy link
Contributor Author

Manually cherry-picked for other release branches here:

#11314
#11315
#11316

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm size/M PR that changes 30-99 lines team/ecosystem Ecosystem team related issues/PRs Verified Mark when PR was verified and log provided
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants