Skip to content

Commit

Permalink
enable retry based on labels
Browse files Browse the repository at this point in the history
Fixes #150
  • Loading branch information
isaaguilar committed Oct 25, 2023
1 parent ec174bd commit 721c66b
Show file tree
Hide file tree
Showing 3 changed files with 69 additions and 1 deletion.
15 changes: 14 additions & 1 deletion deploy/crds/tf.galleybytes.com_terraforms_crd.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,8 @@ apiVersion: apiextensions.k8s.io/v1
kind: CustomResourceDefinition
metadata:
annotations:
controller-gen.kubebuilder.io/version: v0.12.0
controller-gen.kubebuilder.io/version: v0.9.2
creationTimestamp: null
name: terraforms.tf.galleybytes.com
spec:
group: tf.galleybytes.com
Expand Down Expand Up @@ -2563,6 +2564,18 @@ spec:
it, the chance of recycling existing resources is reduced to virtually
nil.
type: string
retryEventReson:
description: "RetryEventReason writes a copy of the current resource
label for 'kubernetes.io/change-cause'. The value will be written
as here as the retry reason. \n When '.setup' is is the suffix,
the pipeline will be started from the setup task. \n Example of
starting from setup: \n ```yaml metadata: labels: kubernetes.io/change-cause:
triggered-by-isa_aguilar-20231025T011600.setup ``` \n A default
retry will start from the init task otherwise."
type: string
retryTimestamp:
format: date-time
type: string
stage:
description: Stage stores information about the current stage
properties:
Expand Down
17 changes: 17 additions & 0 deletions pkg/apis/tf/v1beta1/terraform_types.go
Original file line number Diff line number Diff line change
Expand Up @@ -675,6 +675,23 @@ type TerraformStatus struct {
// refreshed each generation.
// +optional
PluginsStarted []TaskName `json:"pluginsStarted,omitempty"`

// RetryEventReason writes a copy of the current resource label for 'kubernetes.io/change-cause'. The
// value will be written as here as the retry reason.
//
// When '.setup' is is the suffix, the pipeline will be started from the setup task.
//
// Example of starting from setup:
//
// ```yaml
// metadata:
// labels:
// kubernetes.io/change-cause: triggered-by-isa_aguilar-20231025T011600.setup
// ```
//
// A default retry will start from the init task otherwise.
RetryEventReason *string `json:"retryEventReson,omitempty"`
RetryTimestamp *metav1.Time `json:"retryTimestamp,omitempty"`
}

type Exported string
Expand Down
38 changes: 38 additions & 0 deletions pkg/controllers/terraform_controller.go
Original file line number Diff line number Diff line change
Expand Up @@ -588,6 +588,34 @@ func (r *ReconcileTerraform) Reconcile(ctx context.Context, request reconcile.Re
return reconcile.Result{}, err
}

if tf.Labels != nil {
if label, found := tf.Labels["kubernetes.io/change-cause"]; found {
retry := false
if tf.Status.RetryEventReason == nil {
retry = true
} else if *tf.Status.RetryEventReason != label {
retry = true
}

if retry {
now := metav1.Now()
tf.Status.RetryEventReason = &label
tf.Status.RetryTimestamp = &now
taskType := tfv1beta1.RunInit
if strings.HasSuffix(label, ".setup") {
taskType = tfv1beta1.RunSetup
}
stage := newStage(tf, taskType, label, isTaskInterruptable(taskType), tfv1beta1.StateInitializing)
tf.Status.Stage = *stage
err := r.updateStatusWithRetry(ctx, tf, &tf.Status, reqLogger)
if err != nil {
reqLogger.V(1).Info(err.Error())
}
return reconcile.Result{}, nil
}
}
}

// Final delete by removing finalizers
if tf.Status.Phase == tfv1beta1.PhaseDeleted {
reqLogger.Info("Remove finalizers")
Expand Down Expand Up @@ -765,6 +793,16 @@ func (r *ReconcileTerraform) Reconcile(ctx context.Context, request reconcile.Re
return reconcile.Result{}, nil
}

if tf.Status.RetryTimestamp != nil {
podSlice := []corev1.Pod{}
for _, pod := range pods.Items {
if pod.CreationTimestamp.IsZero() || !pod.CreationTimestamp.Before(tf.Status.RetryTimestamp) {
podSlice = append(podSlice, pod)
}
}
pods.Items = podSlice
}

if len(pods.Items) == 0 && tf.Status.Stage.State == tfv1beta1.StateInProgress {
// This condition is generally met when the user deletes the pod.
// Force the state to transition away from in-progress and then
Expand Down

0 comments on commit 721c66b

Please sign in to comment.