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
Fixes #11950: Add a rudder-server init script that will replace the rudder script on server #1439
Conversation
661a4c0
to
3c5ff72
Compare
Commit modified |
3c5ff72
to
b59e7ba
Compare
PR rebased |
true | ||
|
||
install: | ||
mkdir -p $(DESTDIR)/etc/init.d/ |
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.
should be inside the ifneq
RUDDER_DIR = /opt/rudder | ||
DESTDIR = $(CURDIR)/target | ||
INSTALL_DIR = $(DESTDIR)$(RUDDER_DIR) | ||
|
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.
You should probably add GET too
|
||
# Init files | ||
ifneq (true,$(NO_INIT)) | ||
$(INSTALL) -m 755 rudder-server.init $(DESTDIR)/etc/init.d/rudder-server |
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.
There is no reference to this file in the specfile/debianrules how does it suddenly appear here
;; | ||
|
||
upgrade) | ||
exit 0 |
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.
we should not abort here
@@ -1,2 +1,2 @@ | |||
opt/rudder/etc | |||
opt/rudder/bin | |||
opt/rudder/bin |
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.
missing newline at end of file
echo "Rudder is now installed but not configured." | ||
echo "Please run /opt/rudder/bin/rudder-init" | ||
echo "************************************************************" | ||
fi |
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.
missing eol
then | ||
if type chkconfig > /dev/null | ||
then | ||
chkconfig --add rudder-server |
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.
You should add an || true since the rudder-agent package may just have done the same thing.
(and add it to the agent package too)
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.
Not on rudder-server, which is managed completely in rudder-server-root
…udder script on server
b59e7ba
to
43d7ded
Compare
PR replaced by #1449 |
https://www.rudder-project.org/redmine/issues/11950