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 template folder for vsphere IPI in vSphere cleanup #8620

Merged

Conversation

vavuthu
Copy link
Contributor

@vavuthu vavuthu commented Oct 3, 2023

Signed-off-by: vavuthu [email protected]

@vavuthu vavuthu requested a review from a team as a code owner October 3, 2023 11:48
@pull-request-size pull-request-size bot added the size/M PR that changes 30-99 lines label Oct 3, 2023
@vavuthu vavuthu added the Needs Testing Run tests and provide logs link label Oct 3, 2023
@openshift-ci
Copy link

openshift-ci bot commented Oct 3, 2023

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: 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

@vavuthu
Copy link
Contributor Author

vavuthu commented Oct 3, 2023

verified locally

(venv1) vijay ~/VJ/projects/ocs-ci (delete_folder_cleanup_script_ipi)$ vsphere-cleanup --cluster-name j-008vi1cs33-t4a --ocsci-conf vSphere7-DC-ECO_VC1 --cluster-path /clusterdirs/j-008vi1cs33-t4a


2023-10-03 18:47:47,134 - MainThread - ocs_ci.cleanup.vsphere.cleanup - INFO - Resource pool j-008vi1cs33-t4a does not exists
2023-10-03 18:47:47,134 - MainThread - ocs_ci.cleanup.vsphere.cleanup - INFO - Deleting vSphere IPI nodes
2023-10-03 18:54:13,487 - MainThread - ocs_ci.utility.vsphere - INFO - Destroying folder j-008vi1cs33-t4a-hr8hk in templates
2023-10-03 18:54:24,972 - MainThread - ocs_ci.cleanup.vsphere.cleanup - INFO - Removing IP for node j-008vi1cs33-t4a-0 from IPAM server

@vavuthu vavuthu merged commit d4fc50e into red-hat-storage:master Oct 3, 2023
4 checks passed
@petr-balogh petr-balogh added the team/ecosystem Ecosystem team related issues/PRs label Nov 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm Needs Testing Run tests and provide logs link size/M PR that changes 30-99 lines team/ecosystem Ecosystem team related issues/PRs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants