-
Notifications
You must be signed in to change notification settings - Fork 119
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
fix: query jobs before creating #79
Conversation
Signed-off-by: Thomas Schuetz <thomas.schuetz@dynatrace.com>
Signed-off-by: Thomas Schuetz <thomas.schuetz@dynatrace.com>
Since we do not use the job name We can remove this line and the consequent update |
acedbc6
to
08f14f0
Compare
Signed-off-by: Thomas Schuetz <thomas.schuetz@dynatrace.com>
No, as it's nice to have the jobname in the status |
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.
tested and verified on my cluster - now the task controller does not create duplicate jobs for the same task anymore
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.
LGTM
Signed-off-by: Thomas Schuetz thomas.schuetz@dynatrace.com