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

Inconsistent pod name with parallelization enabled #10237

Closed
2 of 3 tasks
agarwalankitnyu opened this issue Dec 15, 2022 · 3 comments · Fixed by #10457
Closed
2 of 3 tasks

Inconsistent pod name with parallelization enabled #10237

agarwalankitnyu opened this issue Dec 15, 2022 · 3 comments · Fixed by #10457
Assignees
Labels
P3 Low priority type/bug

Comments

@agarwalankitnyu
Copy link

agarwalankitnyu commented Dec 15, 2022

Pre-requisites

  • I have double-checked my configuration
  • I can confirm the issues exists when I tested with :latest
  • I'd like to contribute the fix myself (see contributing guide)

What happened/what you expected to happen?

I am noticing some inconsistent behavior for the pod names calculation when using parallelization with v1 POD_NAME spec. I have parallelization enabled to run 5 max workflows at the same time. I submit 20 workflows at the same time, the first 5 calculate pod names for all tasks without the template name(according to v1 spec), after that every workflow that was stuck in the Pending state appends template name to the pod name as well(v2 spec). Argo UI doesn't show this, but when you monitor the pods using kubectl, you can see the name of the pods being created after the initial 5 workflows will have template names appended to it(v2 spec)

Version

3.4.0

Paste a small workflow that reproduces the issue. We must be able to run the workflow; don't enter a workflows that uses private images.

apiVersion: argoproj.io/v1alpha1
kind: Workflow
metadata:
  generateName: steps-
spec:
  entrypoint: hello-hello-hello

  # This spec contains two templates: hello-hello-hello and whalesay
  templates:
  - name: hello-hello-hello
    # Instead of just running a container
    # This template has a sequence of steps
    steps:
    - - name: hello1            # hello1 is run before the following steps
        template: whalesay
        arguments:
          parameters:
          - name: message
            value: "hello1"
    - - name: hello2a           # double dash => run after previous step
        template: whalesay
        arguments:
          parameters:
          - name: message
            value: "hello2a"
      - name: hello2b           # single dash => run in parallel with previous step
        template: whalesay
        arguments:
          parameters:
          - name: message
            value: "hello2b"

  # This is the same template as from the previous example
  - name: whalesay
    inputs:
      parameters:
      - name: message
    container:
      image: docker/whalesay
      command: [cowsay]
      args: ["{{inputs.parameters.message}}"]

Logs from the workflow controller

kubectl logs -n argo deploy/workflow-controller | grep ${workflow}

Logs from in your workflow's wait container

kubectl logs -n argo -c wait -l workflows.argoproj.io/workflow=${workflow},workflow.argoproj.io/phase!=Succeeded
@stale
Copy link

stale bot commented Dec 31, 2022

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. If this is a mentoring request, please provide an update here. Thank you for your contributions.

@stale stale bot added the problem/stale This has not had a response in some time label Dec 31, 2022
@sarabala1979 sarabala1979 added P3 Low priority and removed problem/stale This has not had a response in some time labels Jan 5, 2023
@stale
Copy link

stale bot commented Jan 21, 2023

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. If this is a mentoring request, please provide an update here. Thank you for your contributions.

@stale stale bot added the problem/stale This has not had a response in some time label Jan 21, 2023
@rohankmr414
Copy link
Member

bump

@rohankmr414 rohankmr414 removed the problem/stale This has not had a response in some time label Jan 23, 2023
alexec added a commit that referenced this issue Feb 13, 2023
…0457)

Signed-off-by: Isitha Subasinghe <isitha@pipekit.io>
Co-authored-by: Alex Collins <alexec@users.noreply.github.com>
terrytangyuan pushed a commit that referenced this issue Mar 29, 2023
…0457)

Signed-off-by: Isitha Subasinghe <isitha@pipekit.io>
Co-authored-by: Alex Collins <alexec@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P3 Low priority type/bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants