search

armadaproject / armada-operator

Apache License 2.0
13 stars 10 forks source link

Implement queue controller #323

Open Sovietaced opened 3 months ago

Sovietaced commented 3 months ago

The current queue controller does nothing. We tried to use it and realized it didn't work after looking at the source code. 

func (r *QueueReconciler) Reconcile(ctx context.Context, req ctrl.Request) (ctrl.Result, error) {
    _ = log.FromContext(ctx)

    // TODO(user): your logic here

    return ctrl.Result{}, nil
}

FWIW, we will be implementing this in our fork and I can upstream the work.

dave-gantenbein commented 3 months ago

Thanks @Sovietaced, we're looking forward to your contribution!

dejanzele commented 2 months ago

Hi @Sovietaced,

Thanks for raising this issue!

The Queue controller is currently just a placeholder.

If you implement it in your fork, feel free to submit a PR.