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

control-service: Add username to executions #159

Merged
merged 1 commit into from
Sep 1, 2021

Conversation

doks5
Copy link
Contributor

@doks5 doks5 commented Aug 31, 2021

As part of improving the user experience when
doing manual job executions, and troubleshooting
failed executions, we need to be able to tell by
whom was an execution started.

This change adds the username of the user, who has
initiated an execution, to the startedBy annotation.

Testing Done: Updated existing unit test to reflect the
change.

Signed-off-by: Andon Andonov [email protected]

@doks5 doks5 force-pushed the person/andonova/improve-execute-ux3 branch from 7bbe7cb to 13a6fc9 Compare August 31, 2021 07:54
@doks5 doks5 force-pushed the person/andonova/improve-execute-ux3 branch from 13a6fc9 to 8d9cafc Compare August 31, 2021 14:34
@doks5 doks5 force-pushed the person/andonova/improve-execute-ux3 branch from 8d9cafc to 5d78022 Compare September 1, 2021 15:28
As part of improving the user experience when
doing manual job executions, and troubleshooting
failed executions, we need to be able to tell by
whom was an execution started.

This change adds the username of the user, who has
initiated an execution, to the startedBy annotation.

Testing Done: Updated existing unit test to reflect the
change.

Signed-off-by: Andon Andonov <[email protected]>
@doks5 doks5 force-pushed the person/andonova/improve-execute-ux3 branch from 5d78022 to 16046a8 Compare September 1, 2021 16:01
@doks5 doks5 merged commit 00f3787 into main Sep 1, 2021
@doks5 doks5 deleted the person/andonova/improve-execute-ux3 branch September 1, 2021 16:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants