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
CORS-3251: Create capz machine manifests #7969
CORS-3251: Create capz machine manifests #7969
Conversation
6100e36
to
e7648f8
Compare
2112314
to
11e863c
Compare
oh cool, with the provider in place we might be able to test this /test e2e-azure-capi-ovn |
@patrickdillon: The specified target(s) for
The following commands are available to trigger optional jobs:
Use
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. |
/test altinfra-e2e-azure-capi-ovn |
70e680d
to
6c37fb9
Compare
8f8fd98
to
9a7a006
Compare
9a7a006
to
fd7f2a4
Compare
11d7ed8
to
7f13ed5
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks like it's in good shape but just needs several house keeping items:
- remove wip from title (replace with jira)
- would be ideal to restructure the commits
- linter wants you to use a switch
- empty provider needs to be refactored a bit
note that the linter wants
7f13ed5
to
e606528
Compare
/approve /test altinfra-e2e-azure-capi-ovn This is looking really good. |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: patrickdillon 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 |
c43fcdd
to
8ef4d3e
Compare
/test altinfra-e2e-azure-capi-ovn |
/retezt |
/retest |
Creating machine manifests for azure CAPI implementation.
Fixing the resource group name set in the Azure ClusterSpec for CAPI.
Added the basic Azure CAPI provider.
Adding the latest version of the cluster api provider azure.
8ef4d3e
to
dcb741f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
/hold Revision dcb741f was retested 3 times: holding |
@rna-afk: The following tests failed, say
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. |
/hold cancel |
/skip |
/override ci/prow/e2e-aws-ovn ci/prow/e2e-azure-ovn |
These tests are permafailing. Installs look fine |
@patrickdillon: Overrode contexts on behalf of patrickdillon: ci/prow/e2e-aws-ovn, ci/prow/e2e-azure-ovn 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. |
Creating machine manifests for azure CAPI implementation.