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.

Sign up for GitHub

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

Jump to bottom

SPLAT-1201: vSphere - poc removing terraform #7554

Closed
wants to merge 18 commits into from
Closed

SPLAT-1201: vSphere - poc removing terraform #7554

wants to merge 18 commits into from

Conversation

jcpowermac
Copy link
Contributor

No description provided.

@openshift-ci openshift-ci bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Oct 4, 2023
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Oct 4, 2023

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Oct 4, 2023

[APPROVALNOTIFIER] This PR is NOT APPROVED

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

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

@jcpowermac
Copy link
Contributor Author

/test e2e-vsphere-ovn

@jcpowermac
add logs for debugging
7736e5a
@jcpowermac
Check for path in topology folder
d7713be 
Somewhere, I need to find it, we set Topology.Folder to the infraId
using a non-path. This is really wrong, probably for terraform
since it can only take a name.
@jcpowermac
get tag id from terraform data
328ddb1
@jcpowermac
Copy link
Contributor Author

this should at least install now...
/test e2e-vsphere-ovn

@jcpowermac
Copy link
Contributor Author

ci cluster issues
/test e2e-vsphere-ovn

@jcpowermac
Copy link
Contributor Author

/test e2e-vsphere-ovn

1 similar comment
@jcpowermac
Copy link
Contributor Author

/test e2e-vsphere-ovn

@jcpowermac
Copy link
Contributor Author

/refresh

@jcpowermac
Copy link
Contributor Author

/test e2e-vsphere-ovn

@jcpowermac
Copy link
Contributor Author

/test e2e-vsphere-ovn

@jcpowermac jcpowermac changed the title vSphere: poc removing pre-bootstrap terraform stage vSphere: poc removing terraform Oct 6, 2023
@jcpowermac
Copy link
Contributor Author

/test e2e-vsphere-ovn
/test e2e-vsphere-zones

@jcpowermac
Copy link
Contributor Author

/test e2e-vsphere-ovn

@jcpowermac
Copy link
Contributor Author

/refresh

@jcpowermac
Copy link
Contributor Author

/test e2e-vsphere-ovn

1 similar comment
@jcpowermac
Copy link
Contributor Author

/test e2e-vsphere-ovn

@jcpowermac
Copy link
Contributor Author

/test e2e-vsphere-zones

@jcpowermac jcpowermac changed the title vSphere: poc removing terraform SPLAT-1201: vSphere - poc removing terraform Oct 9, 2023
@openshift-ci-robot
Copy link
Contributor

openshift-ci-robot commented Oct 9, 2023 •
edited by openshift-ci bot

@jcpowermac: This pull request references SPLAT-1201 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 spike to target the "4.15.0" version, but no target version was set.

In response to this:

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/test-infra repository.

@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 9, 2023
@jcpowermac
Copy link
Contributor Author

/test e2e-vsphere-ovn
/test e2e-vsphere-zones

@jcpowermac
Copy link
Contributor Author

/test e2e-vsphere-zones

@jcpowermac
Copy link
Contributor Author

/test e2e-vsphere-zones

@jcpowermac
Copy link
Contributor Author

/test e2e-vsphere-ovn

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Oct 11, 2023

@jcpowermac: 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/altinfra-images f025d0b link true /test altinfra-images
ci/prow/e2e-vsphere-zones 3629032 link false /test e2e-vsphere-zones
ci/prow/e2e-vsphere-ovn 3629032 link true /test e2e-vsphere-ovn

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/test-infra repository. I understand the commands that are listed here.

@jcpowermac
Copy link
Contributor Author

/close
no longer needed

@openshift-merge-robot openshift-merge-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Nov 1, 2023
@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/test-infra repository.

@openshift-ci openshift-ci bot closed this Nov 1, 2023
@openshift-ci OpenShift CI
Copy link
Contributor

openshift-ci bot commented Nov 1, 2023

@jcpowermac: Closed this PR.

In response to this:

/close
no longer needed

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/test-infra repository.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. 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
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@jcpowermac @openshift-ci-robot @openshift-merge-robot