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

New workergroups configuration UI (dashboard) #172

Open
wants to merge 19 commits into
base: 0.9.x
Choose a base branch
from

Conversation

DasFranck
Copy link

@DasFranck DasFranck commented Jul 5, 2017

Plus:

  • Adding job moving feature
  • Job retry count shown in the dashboard

@@ -223,6 +223,18 @@ define(["jquery", "underscore", "views/generic/datatablepage", "models"],functio
});
self.refreshCallStack(job_id);

} else if (action == "move") {

var queue = prompt("Test");
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

test

self.collection.update({
"_id": {"$in": jobs_by_queue[queue]}
}, {"$set": updates}, multi=True)

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

utiliser le code de "requeue" plutôt que tout dupliquer

mrq/job.py Outdated
self._attach_original_exception(exc)
self.requeue(queue, retry_count)
raise exc

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove

@@ -105,6 +105,10 @@ define(["jquery", "underscore", "views/generic/datatablepage", "models"],functio
var action = $(evt.target).data("action");
var data = _.clone(this.filters);

if (action == "move") {
data["destination_queue"] = prompt("Enter destination queue");
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

faire comme en dessous, ne rien faire si data["destination_queue"] est vide

@DasFranck DasFranck force-pushed the 0.9.x branch 2 times, most recently from e5164e1 to d3db499 Compare July 6, 2017 09:31
@DasFranck DasFranck changed the title Adding job moving feature New workergroups UI configuration (dashboard) Jul 11, 2017
@DasFranck DasFranck changed the title New workergroups UI configuration (dashboard) New workergroups configuration UI (dashboard) Jul 11, 2017
@sylvinus sylvinus added this to the 1.0.0 milestone Mar 2, 2018
@menzow
Copy link

menzow commented Jun 14, 2018

What happened with this PR? Looked promising.

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.

None yet

4 participants