Skip to content
This repository has been archived by the owner on Nov 24, 2021. It is now read-only.

Fixes #14675: Start tomcat on upgrade before Foreman migrations #324

Merged
merged 1 commit into from Apr 19, 2016

Conversation

ehelms
Copy link
Member

@ehelms ehelms commented Apr 18, 2016

No description provided.

@@ -19,6 +19,10 @@ def migrate_candlepin
Kafo::Helpers.execute("/usr/share/candlepin/cpdb --update --password #{Kafo::Helpers.read_cache_data('candlepin_db_password')}")
end

def start_tomcat
Kafo::Helpers.execute('katello-service start --only tomcat')
Copy link
Contributor

Choose a reason for hiding this comment

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

On el6, the service is called tomcat6 :-(

Copy link
Contributor

Choose a reason for hiding this comment

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

But the script ignores non-existent services, so you can say --only tomcat,tomcat6

@ehelms
Copy link
Member Author

ehelms commented Apr 19, 2016

@stbenjam Updated based on your suggestion

@stbenjam
Copy link
Contributor

ACK

@ehelms ehelms merged commit e2af21a into master Apr 19, 2016
ehelms added a commit to ehelms/katello-installer that referenced this pull request Apr 19, 2016
@ekohl ekohl deleted the fixes-14675 branch April 20, 2016 09:22
zjhuntin pushed a commit to zjhuntin/katello-installer that referenced this pull request Oct 30, 2018
Refs #11701 - hide pulp_auth ping output if not present
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
3 participants