Skip to content
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

feat: SKIP_WORKFLOW_DURATION_ESTIMATION #1

Open
wants to merge 1 commit into
base: feature/zg
Choose a base branch
from

Conversation

talebzeghmi
Copy link

@talebzeghmi talebzeghmi commented May 15, 2024

see argoproj#7271

And the many slow queries of form

Query: SELECT name,
       namespace,
       uid,
       phase,
       startedat,
       finishedat,
       Coalesce(workflow ->> '$.metadata.labels', '{}')      AS labels,
       Coalesce(workflow ->> '$.metadata.annotations', '{}') AS annotations,
       Coalesce(workflow ->> '$.status.progress', '')        AS progress
FROM   `argo_archived_workflows`
WHERE  ( ( `clustername` = ?
           AND `instanceid` = ? )
         AND `namespace` = ?
         AND EXISTS (SELECT 1
                     FROM   argo_archived_workflows_labels
                     WHERE  clustername = argo_archived_workflows.clustername
                            AND uid = argo_archived_workflows.uid
                            AND name = 'workflows.argoproj.io/phase'
                            AND value = 'Succeeded')
         AND EXISTS (SELECT 1
                     FROM   argo_archived_workflows_labels
                     WHERE  clustername = argo_archived_workflows.clustername
                            AND uid = argo_archived_workflows.uid
                            AND name = 'workflows.argoproj.io/workflow-template'
                            AND value = 'helloflow') )
ORDER  BY `startedat` DESC
LIMIT  1 

@talebzeghmi talebzeghmi requested a review from tmckay May 15, 2024 22:55
@talebzeghmi talebzeghmi changed the title AIP-8409 SKIP_WORKFLOW_DURATION_ESTIMATION feat: SKIP_WORKFLOW_DURATION_ESTIMATION May 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant