-
Notifications
You must be signed in to change notification settings - Fork 42
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
pull job action away from track_statuses #711
Conversation
@soxofaan does this design match with what we discussed yesterday? |
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.
some quick notes
@soxofaan failing unit tests seem to be unrelated? |
yes failing tests are already failed on master (#715 ), so I would merge master in your branch |
@soxofaan there we go |
From my end, it's ready to merge. Uncertain if it will make a difference for LCFM. There we would need to pre-create a queue of created jobs |
… jobs (this is a squashed commit based on #711)
squashed and merged in 6dbf2f0 |
No description provided.