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

Refs 6297 - use foreman-tasks instead of delayed jobs #35

Merged
merged 1 commit into from
Aug 20, 2014

Conversation

iNecas
Copy link
Member

@iNecas iNecas commented Aug 11, 2014

We now enable the foreman::plugin::tasks to handle the async tasks

We now enable the foreman::plugin::tasks to handle the async tasks
@@ -1,13 +1,4 @@
# Katello Services
class katello::service {
include pulp::service

service {'katello-jobs':
ensure => running,
Copy link
Member

Choose a reason for hiding this comment

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

For upgrade purposes ensure => stopped?

Copy link
Member Author

Choose a reason for hiding this comment

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

That's a good question. I don't think we are in the phase to support the upgrades just yet, there were still changes accepted in the other parts of Katello, that require fresh install, reset.

Copy link
Member

Choose a reason for hiding this comment

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

Fair enough.

Copy link
Contributor

Choose a reason for hiding this comment

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

we aren't going to support upgrades from 1.4 -> 2.0 unfortunately.

@jlsherrill
Copy link
Contributor

ACK

iNecas added a commit that referenced this pull request Aug 20, 2014
Refs 6297 - use foreman-tasks instead of delayed jobs
@iNecas iNecas merged commit c475783 into theforeman:master Aug 20, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants