-
Notifications
You must be signed in to change notification settings - Fork 233
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
Add support for cluster install on Alibaba Cloud #1704
Add support for cluster install on Alibaba Cloud #1704
Conversation
/assign @abutcher just includes APIs for now. Working on install code |
Codecov Report
@@ Coverage Diff @@
## master #1704 +/- ##
==========================================
- Coverage 41.95% 41.63% -0.33%
==========================================
Files 346 349 +3
Lines 32119 32374 +255
==========================================
+ Hits 13476 13479 +3
- Misses 17512 17760 +248
- Partials 1131 1135 +4
|
API changes look good to me 👍 |
4096d73
to
0a558f2
Compare
/retest |
0a558f2
to
2338548
Compare
@abutcher This is ready for a review. |
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.
Looks good, a few small things. Is there anything outstanding?
pkg/clusterresource/alibabacloud.go
Outdated
type AlibabaCloudBuilder struct { | ||
// AccessKeyID is the Alibaba Cloud access key ID. | ||
AccessKeyID string | ||
// AccessKeySecret is the AWS access key secret. |
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.
s/AWS/Alibaba Cloud
hack/update-codegen.sh
Outdated
@@ -19,7 +19,7 @@ GOFLAGS="" bash ${CODEGEN_PKG}/generate-groups.sh "all" \ | |||
GOFLAGS="" bash ${CODEGEN_PKG}/generate-groups.sh "deepcopy" \ | |||
github.com/openshift/hive/pkg/client \ | |||
github.com/openshift/hive/apis \ | |||
"hive:v1/agent hive:v1/aws hive:v1/azure hive:v1/baremetal hive:v1/gcp hive:v1/none hive:v1/openstack hive:v1/ovirt hive:v1/vsphere hive:v1/ibmcloud hivecontracts:v1alpha1" \ | |||
"hive:v1/agent hive:v1/alibabacloud hive:v1/aws hive:v1/azure hive:v1/baremetal hive:v1/gcp hive:v1/openstack hive:v1/ovirt hive:v1/vsphere hive:v1/ibmcloud hivecontracts:v1alpha1" \ |
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.
hive:v1/none
is removed and should stay, right?
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.
I do not know how that got removed but I will add it
2338548
to
018d192
Compare
@abutcher This is ready for another review |
/retest |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: abutcher, akhil-rane 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 |
@akhil-rane: all tests passed! 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. |
x-ref: https://issues.redhat.com/browse/HIVE-1609
x-ref: https://issues.redhat.com/browse/HIVE-1610
x-ref: https://issues.redhat.com/browse/HIVE-1611