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

Katello-jobs for dynflow integration needs to be running before seed #6

Merged
merged 1 commit into from
Jan 30, 2014

Conversation

iNecas
Copy link
Member

@iNecas iNecas commented Jan 29, 2014

The executor needs to be ready before processing Dynflow execution plans, that
might occur as part of the seed process.

@iNecas
Copy link
Member Author

iNecas commented Jan 29, 2014

Installer changes for #6

@ehelms
Copy link
Member

ehelms commented Jan 29, 2014

ACK

@iNecas
Copy link
Member Author

iNecas commented Jan 29, 2014

Thanks to this change https://github.com/iNecas/foreman-tasks/pull/19 we don't need to have the jobs running before the rake tasks (as there was chicken and egg problem with rake db:migrate. therefore, the rake tasks use their own executor. So all we need to do is making sure to running before httpd

The executor needs to be ready before starting web server workers.
@iNecas
Copy link
Member Author

iNecas commented Jan 29, 2014

Well, I was not completely honest there: the jobs need to run before httpd, but seed needs to run after httpd is ready (for pulp), therefore we need to start httpd before seed, and jobs before httpd, which leads us to jobs needed db:migrate… anyway, this should be the version :)

@ehelms
Copy link
Member

ehelms commented Jan 29, 2014

ACK

@iNecas
Copy link
Member Author

iNecas commented Jan 30, 2014

Since this change is not dependent on the dynflow branch, I will merge it right away.

iNecas added a commit that referenced this pull request Jan 30, 2014
Katello-jobs for dynflow integration needs to be running before seed
@iNecas iNecas merged commit 2a9487c into theforeman:master Jan 30, 2014
ehelms added a commit to ehelms/puppet-katello that referenced this pull request Jun 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.

None yet

2 participants