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

Fix missing NETDATA_STOP_CMD in netdata-installer.sh #8897

Merged
merged 3 commits into from
May 6, 2020
Merged

Fix missing NETDATA_STOP_CMD in netdata-installer.sh #8897

merged 3 commits into from
May 6, 2020

Conversation

prologic
Copy link
Contributor

@prologic prologic commented May 4, 2020

Summary

ssia

Component Name
  • area/packaging
Test Plan
  • Test on a SystemD system
  • Test on macOS
  • Test on FreeBSD
  • Test on a SysV system
  • Test on something non-standard
Additional Information

Looks like this was always missing in our packaging tooling :/

@squash-labs
Copy link

squash-labs bot commented May 4, 2020

Manage this branch in Squash

Test this branch here: https://prologicfix-missing-stop-cmd-uh88j.squash.io

@github-actions github-actions bot added the area/packaging Packaging and operating systems support label May 4, 2020
@prologic
Copy link
Contributor Author

prologic commented May 5, 2020

Tested on non-Standard systems (_plain 'ol install with no supervisor/init managing Netdata_): $ ./netdata-installer.sh --dont-wait --dont-start-it --disable-go --install /tmp/netdata . . . To stop netdata run: /tmp/netdata/netdata/usr/sbin/netdatacli shutdown-agent To start netdata run: /tmp/netdata/netdata/usr/sbin/netdata . . .

@thiagoftsm
Copy link
Contributor

@prologic shellcheck continues giving error:

          NETDATA_STOP_CMD="rc-service netdata stop"
          ^--------------^ SC2034: NETDATA_STOP_CMD appears unused. Verify use (or export if used externally).

@thiagoftsm
Copy link
Contributor

Everything worked as expected on Slackware current, when you finish the work I will retest and approve it.

@prologic
Copy link
Contributor Author

prologic commented May 5, 2020

Tested on a SystemD system (_Debian Buster_):
$ sudo ./netdata-installer.sh --dont-wait --disable-go
.
.
.
To stop netdata run:

  systemctl stop netdata

To start netdata run:

  systemctl start netdata
.
.
.

@prologic
Copy link
Contributor Author

prologic commented May 5, 2020

Tested on FreeBSD:
root@freebsd:~/netdata # ./netdata-installer.sh --dont-wait --disable-go
.
.
.
To stop netdata run:

  service netdata stop

To start netdata run:

  service netdata start
.
.
.

@prologic prologic marked this pull request as ready for review May 5, 2020 04:33
@prologic prologic self-assigned this May 5, 2020
thiagoftsm
thiagoftsm previously approved these changes May 5, 2020
Copy link
Contributor

@thiagoftsm thiagoftsm left a comment

Choose a reason for hiding this comment

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

I tested again on Slackware, and everything worked as expected!

@prologic
Copy link
Contributor Author

prologic commented May 6, 2020

Rebasing...

@prologic prologic merged commit 935eb39 into netdata:master May 6, 2020
@prologic prologic deleted the fix_missing_stop_cmd branch May 6, 2020 17:34
Saruspete pushed a commit to Saruspete/netdata that referenced this pull request May 21, 2020
* Fix missing NETDATA_STOP_CMD in netdata-installer.sh

* Fix shellcheck warnings

* Fix more shellcheck warnings
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/packaging Packaging and operating systems support
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants