-
Notifications
You must be signed in to change notification settings - Fork 361
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
[Backport release-1.30] chore(backport): make helm timeout backward compatible #5041
[Backport release-1.30] chore(backport): make helm timeout backward compatible #5041
Conversation
c92782f
to
214a277
Compare
in the past the helm chart install timeout was moved from an integer to a string. this commit brings back the capability of using integers to set the helm chart timeout. Signed-off-by: Ricardo Maraschini <[email protected]>
214a277
to
dcf997f
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.
Some nits in the comments. You can also use git cherry-pick -x
to automatically include the cherry-picked commit hash in the cherry-pick's commit message, as e.g. advertised in #5017 (comment).
Version: "0.0.1", | ||
Values: "values", | ||
TargetNS: "default", | ||
Timeout: k0sv1beta1.BackwardCompatibleDuration( |
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.
In the original PR, this is below ForceUpgrade
.
"github.com/k0sproject/k0s/pkg/apis/k0s/v1beta1" | ||
k0sv1beta1 "github.com/k0sproject/k0s/pkg/apis/k0s/v1beta1" |
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.
"github.com/k0sproject/k0s/pkg/apis/k0s/v1beta1" | |
k0sv1beta1 "github.com/k0sproject/k0s/pkg/apis/k0s/v1beta1" | |
k0sv1beta1 "github.com/k0sproject/k0s/pkg/apis/k0s/v1beta1" |
@@ -159,7 +160,9 @@ func addOpenEBSHelmExtension(helmSpec *k0sv1beta1.HelmExtensions, storageExtensi | |||
TargetNS: "openebs", | |||
Version: constant.OpenEBSVersion, | |||
Values: values, | |||
Timeout: metav1.Duration{Duration: time.Duration(time.Minute * 30)}, // it takes a while to install openebs | |||
Timeout: v1beta1.BackwardCompatibleDuration( |
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.
Timeout: v1beta1.BackwardCompatibleDuration( | |
Timeout: k0sv1beta1.BackwardCompatibleDuration( |
Super-seeded by #5048 |
Description
** this is a backport of #5017**
in the past the helm chart install timeout was moved from an integer to a string. this commit brings back the capability of using integers to set the helm chart timeout.
Type of change
How Has This Been Tested?
Checklist: