You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@brooklyn.apache.org by alrick87 <gi...@git.apache.org> on 2016/04/21 21:56:00 UTC
[GitHub] brooklyn-dist pull request: Add upstart and systemd support
GitHub user alrick87 opened a pull request:
https://github.com/apache/brooklyn-dist/pull/35
Add upstart and systemd support
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/alrick87/brooklyn-dist add_upstart_and_systemd_support
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/brooklyn-dist/pull/35.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #35
----
commit 6ee1bf48f0c4df19705b620bdce8f10429eebd1f
Author: alrick87 <so...@gmail.com>
Date: 2016-04-21T14:58:27Z
Add upstart and systemd support
----
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] brooklyn-dist pull request: Add upstart and systemd support
Posted by alrick87 <gi...@git.apache.org>.
Github user alrick87 commented on the pull request:
https://github.com/apache/brooklyn-dist/pull/35#issuecomment-216530068
@neykov Removing sysvinit script fixes the issue:
```
ubuntu@wily-deb-test:~$ sudo systemctl status brooklyn
sudo: unable to resolve host wily-deb-test
\u25cf brooklyn.service - Apache Brooklyn Service
Loaded: loaded (/etc/systemd/system/multi-user.target.wants/brooklyn.service)
Active: active (running) since Tue 2016-05-03 13:36:57 UTC; 335ms ago
Docs: https://brooklyn.apache.org/documentation/index.html
Main PID: 13079 (java)
Memory: 36.9M
CPU: 432ms
CGroup: /system.slice/brooklyn.service
\u2514\u250013079 /usr/bin/java -Dbrooklyn.location.localhost.address=127.0.0.1 -Dlogback.configurationFile=/etc/brooklyn/logback.xml -Xms256m -Xmx1g -X...
May 03 13:36:57 wily-deb-test systemd[1]: Started Apache Brooklyn Service.
```
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] brooklyn-dist pull request: Add upstart and systemd support
Posted by bostko <gi...@git.apache.org>.
Github user bostko commented on a diff in the pull request:
https://github.com/apache/brooklyn-dist/pull/35#discussion_r61488066
--- Diff: src/service/upstart/deb/brooklyn.conf ---
@@ -0,0 +1,45 @@
+# Licensed to the Apache Software Foundation (ASF) under one
--- End diff --
@alrick87 use `git mv deb-packaging/src/conf/brooklyn.conf src/service/upstart/deb/brooklyn.conf` to preserve history.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] brooklyn-dist pull request: Add upstart and systemd support
Posted by neykov <gi...@git.apache.org>.
Github user neykov commented on the pull request:
https://github.com/apache/brooklyn-dist/pull/35#issuecomment-216494570
Looks good. Would be nice to have live tests against all supported OSes for deb & rpm packages.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] brooklyn-dist pull request: Add upstart and systemd support
Posted by neykov <gi...@git.apache.org>.
Github user neykov commented on the pull request:
https://github.com/apache/brooklyn-dist/pull/35#issuecomment-216508389
Testing with systemd I get:
```
vagrant@vagrant-ubuntu-wily-64:~$ sudo systemctl status brooklyn
\u25cf brooklyn.service - LSB: Apache Brooklyn
Loaded: loaded (/etc/init.d/brooklyn)
Active: active (running) since Tue 2016-05-03 12:00:07 UTC; 2min 19s ago
Docs: man:systemd-sysv-generator(8)
Process: 683 ExecStart=/etc/init.d/brooklyn start (code=exited, status=0/SUCCESS)
Memory: 284.9M
CPU: 12.405s
CGroup: /system.slice/brooklyn.service
\u2514\u2500684 /usr/bin/java -Dbrooklyn.location.localhost.address=127.0.0.1 -Dlogback.configurationFile=/etc/brooklyn/logback.xml -Xms256m -Xmx1g -XX:MaxPermSize=256m -cp /opt/brooklyn/conf:/opt/b...
```
Looks like systemd is managing the process through the `init.d` script. Is this the expected behaviour?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] brooklyn-dist pull request: Add upstart and systemd support
Posted by alrick87 <gi...@git.apache.org>.
Github user alrick87 commented on the pull request:
https://github.com/apache/brooklyn-dist/pull/35#issuecomment-216524150
@neykov Thanks for pointing this out, I totally missed that. The package copies both init.d and systemd scripts, but for some reason systemd on Ubuntu 15.10 has a priority set for init.d scripts. I guess, I will just remove init.d completely as it isn't used anywhere.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] brooklyn-dist pull request: Add upstart and systemd support
Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:
https://github.com/apache/brooklyn-dist/pull/35
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---