-
Notifications
You must be signed in to change notification settings - Fork 122
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
Implement a configurable timeout for OpenStack #843
Implement a configurable timeout for OpenStack #843
Conversation
Hi @cedi. Thanks for your PR. I'm waiting for a kubermatic member to verify that this patch is reasonable to test. If it is, they should reply with Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. 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. |
Thanks @cedi! |
/assign @moadqassem |
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.
@cedi Thanks for the PR. The current PR introduces new helper functions in order to treat int values which got from secrets. Where I find this as a good utility to have, however, for the fields which these functions are created, shouldn't be treated that way and should be added explicitly instead of deriving them from secrets. I am not quite sure that there is a case where we have(or will have) a value which holds a sensitive information and being of a type int. Thus I would suggest to remove these helper functions.
pkg/apis/cluster/v1alpha1/conversions/testdata/machinesv1alpha1machine/openstack.yaml
Outdated
Show resolved
Hide resolved
pkg/apis/cluster/v1alpha1/conversions/testdata/machinesv1alpha1machine/openstack.yaml
Outdated
Show resolved
Hide resolved
pkg/apis/cluster/v1alpha1/conversions/testdata/migrated_clusterv1alpha1machine/openstack.yaml
Outdated
Show resolved
Hide resolved
pkg/apis/cluster/v1alpha1/conversions/testdata/migrated_clusterv1alpha1machine/openstack.yaml
Outdated
Show resolved
Hide resolved
/ok-to-test |
Make instanceReadyCheckTimeout for OpenStack Provider configurable Signed-off-by: cedi <cedric.kienzler@gec.io>
@cedi Thank you very much for you PR. |
LGTM label has been added. Git tree hash: 6c7c57ae3ccc47fdef09319e5d91823bf70acee8
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: cedi, moadqassem 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 |
What this PR does / why we need it:
Make instanceReadyCheckTimeout for OpenStack Provider configurable
Which issue(s) this PR fixes
#819
Special notes for your reviewer:
This haven't been tested properly yet.
It is still work in progress.
Optional Release Note: