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

MH-13144, only set Job startDate if no set before #553

Merged
merged 1 commit into from Nov 12, 2018

Conversation

Projects
None yet
3 participants
@JamesUoM
Copy link
Contributor

JamesUoM commented Nov 6, 2018

Stops jobs that update while they are running (eg START_WORKFLOW) from having their startDate reset each time

@staubesv staubesv self-requested a review Nov 6, 2018

@staubesv staubesv self-assigned this Nov 6, 2018

@KatrinIhler KatrinIhler assigned KatrinIhler and unassigned staubesv Nov 6, 2018

@staubesv staubesv requested review from KatrinIhler and removed request for staubesv Nov 6, 2018

@KatrinIhler

This comment has been minimized.

Copy link
Contributor

KatrinIhler commented Nov 6, 2018

@JamesUoM Is there a way to see the startdate of the START_WORKFLOW job change? It's not visible in the admin ui, is it?

@JamesUoM

This comment has been minimized.

Copy link
Contributor

JamesUoM commented Nov 7, 2018

@JamesUoM Is there a way to see the startdate of the START_WORKFLOW job change? It's not visible in the admin ui, is it?

No, I think the "start" date the Admin UI displays is actually the created date or at least is never updated. I started OC with a MySQL database, ran a long workflow and monitored the job in my database viewer.

@KatrinIhler
Copy link
Contributor

KatrinIhler left a comment

I watched the job table in the database to confirm this bug, and the start date does indeed change, which doesn't make any sense. This fixes it, so let's merge this! :)

@KatrinIhler KatrinIhler merged commit 6826308 into opencast:r/5.x Nov 12, 2018

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment