CORS-2831: WIP: PoC: alternate AWS infrastructure provider#7573
CORS-2831: WIP: PoC: alternate AWS infrastructure provider#7573r4f4 wants to merge 31 commits intoopenshift:masterfrom
Conversation
Introduces an infrastructure package with a stages interface to allow various infrastructure providers. The stages interface is based on the existing pkg/terraform/stages interface, but allows each stage to have its own Provision (and Destroy) function which can call any infrastructure provider in addition to Terraform. Currently, the input to the provider is still expecting TFVars, which the underlying stage provision logic can translate.
Use the existing terraform providers with the infrastructure stages interface.
Adapts the terraform package to implement the infrastructure/stages interface. Encapsulates terraform-specific implementation details in pkg/terraform.
Replaces asset usage of terraform with infrastructure interface.
Implements a build tag strategy to allow producing installer binaries using an alternate (to terraform) infrastructure provider. Using build tags we can select different ProviderForPlatform implementations. Because the altinfra implementation of that function does not depend on any terraform dependencies, none are included in the binary.
Co-authored-by: Enxebre <alberto.garcial@hotmail.com>
openshift-ci
bot
added
the
do-not-merge/work-in-progress
|
Skipping CI for Draft Pull Request. |
|
/test ? |
|
@r4f4: The following commands are available to trigger required jobs:
The following commands are available to trigger optional jobs:
Use
DetailsIn 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-aws-ovn |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
r4f4
force-pushed
the
patrick-alt-infra-aws
branch
from
49dfbe5 to
b2dd8d7
Compare
|
/test altinfra-e2e-aws-ovn |
The altinfra tag does not use the Terraform binaries, so we can skip building them.
|
/test altinfra-e2e-aws-ovn |
|
/test altinfra-e2e-aws-ovn |
r4f4
force-pushed
the
patrick-alt-infra-aws
branch
from
6f14032 to
73b1bdc
Compare
|
/test altinfra-e2e-aws-ovn-shared-vpc |
|
/test altinfra-e2e-aws-ovn-shared-vpc |
r4f4
force-pushed
the
patrick-alt-infra-aws
branch
from
c45313e to
9d6e722
Compare
|
/test altinfra-e2e-aws-ovn-shared-vpc |
|
/test altinfra-e2e-aws-ovn |
|
@r4f4: all tests passed! Full PR test history. Your PR dashboard. DetailsInstructions 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. |
|
/retitle CORS-2831: WIP: PoC: alternate AWS infrastructure provider |
openshift-merge-robot
added
the
needs-rebase
|
PR needs rebase. DetailsInstructions 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
bot
changed the title
openshift-ci-robot
added
the
jira/valid-reference
|
@r4f4: This pull request references CORS-2831 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. DetailsIn 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. |
|
/close |
|
@r4f4: Closed this PR. DetailsIn 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. |
4 participants
PR created to test in CI. Not to merge.