Skip to content

Restart services when enabling cockpit - #1228

Draft
ehelms wants to merge 1 commit into
theforeman:masterfrom
ehelms:fix-cockpit-restart
Draft

Restart services when enabling cockpit#1228
ehelms wants to merge 1 commit into
theforeman:masterfrom
ehelms:fix-cockpit-restart

Conversation

@ehelms

@ehelms ehelms commented Jul 11, 2025

Copy link
Copy Markdown
Member

No description provided.

@adamruzicka

Copy link
Copy Markdown
Contributor

For some reason I'm failing to reproduce the original problem, but I guess it won't hurt to be explicit. Passing in --enable-foreman-plugin-remote-execution-cockpit seems to work both without and with this patch.

On the other hand, passing in --no-enable-foreman-plugin-remote-execution-cockpit still requires a manual httpd restart both without and with this patch.

Signed-off-by: Eric D. Helms <ericdhelms@gmail.com>
@ehelms
ehelms force-pushed the fix-cockpit-restart branch from 3af1822 to 2fa8c5d Compare July 16, 2025 12:48
@ehelms

ehelms commented Jul 16, 2025

Copy link
Copy Markdown
Member Author

I ran into this last time I tested it. I'll try again with latest.

@ehelms
ehelms marked this pull request as draft November 14, 2025 16:11
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.

2 participants