You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cloudstack.apache.org by wido <gi...@git.apache.org> on 2015/09/14 13:31:02 UTC
[GitHub] cloudstack pull request: CLOUDSTACK-8840: Systemd service for the ...
GitHub user wido opened a pull request:
https://github.com/apache/cloudstack/pull/820
CLOUDSTACK-8840: Systemd service for the Usage Server
There already was a uncompleted systemd service file for the Usage
Server.
This new one replaces sysvinit and the old systemd service file.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/wido/cloudstack CLOUDSTACK-8840
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/cloudstack/pull/820.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 #820
----
commit f44a5c79b3e22b52b1b297e07e571cbddb90aba4
Author: Wido den Hollander <wi...@widodh.nl>
Date: 2015-09-14T11:30:32Z
CLOUDSTACK-8840: Systemd service for the Usage Server
There already was a uncompleted systemd service file for the Usage
Server.
This new one replaces sysvinit and the old systemd service file.
----
---
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] cloudstack pull request: CLOUDSTACK-8840: Systemd service for the ...
Posted by bhaisaab <gi...@git.apache.org>.
Github user bhaisaab commented on the pull request:
https://github.com/apache/cloudstack/pull/820#issuecomment-140675342
@wido looks similar so looking at the changes LGTM, but I lack infra and time to test it out right now
---
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] cloudstack pull request: CLOUDSTACK-8840: Systemd service for the ...
Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:
https://github.com/apache/cloudstack/pull/820
---
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] cloudstack pull request: CLOUDSTACK-8840: Systemd service for the ...
Posted by remibergsma <gi...@git.apache.org>.
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/820#issuecomment-140185056
@wido I get this error when I run `./package.sh -d centos7`
```
install: cannot stat ‘packaging/cloudstack-usage.service’: No such file or directory
error: Bad exit status from /var/tmp/rpm-tmp.biFx59 (%install)
```
---
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] cloudstack pull request: CLOUDSTACK-8840: Systemd service for the ...
Posted by wido <gi...@git.apache.org>.
Github user wido commented on the pull request:
https://github.com/apache/cloudstack/pull/820#issuecomment-140675073
@remibergsma @bhaisaab Could you take a look? It's almost similar to the Agent systemd profile :)
---
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] cloudstack pull request: CLOUDSTACK-8840: Systemd service for the ...
Posted by remibergsma <gi...@git.apache.org>.
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/820#issuecomment-140719214
@wido No luck yet:
```
error: File not found: /data/git/cs2/cloudstack/dist/rpmbuild/BUILDROOT/cloudstack-4.6.0-SNAPSHOT.el7.centos.x86_64/usr/sbin/cloudstack-usage-sysd
RPM build errors:
Binaries arch (1) not matching the package arch (2).
File not found: /data/git/cs2/cloudstack/dist/rpmbuild/BUILDROOT/cloudstack-4.6.0-SNAPSHOT.el7.centos.x86_64/usr/sbin/cloudstack-usage-sysd
RPM Build Failed
```
---
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] cloudstack pull request: CLOUDSTACK-8840: Systemd service for the ...
Posted by wido <gi...@git.apache.org>.
Github user wido commented on the pull request:
https://github.com/apache/cloudstack/pull/820#issuecomment-140305692
@remibergsma Yes, stupid. Fixed that. Pushed a new commit
---
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] cloudstack pull request: CLOUDSTACK-8840: Systemd service for the ...
Posted by wido <gi...@git.apache.org>.
Github user wido commented on the pull request:
https://github.com/apache/cloudstack/pull/820#issuecomment-140746033
@remibergsma Thank you for testing. Stupid mistake indeed. That should be fixed now.
I checked on a CentOS 7 machine:
```
+ exit 0
RPM Build Done
```
---
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] cloudstack pull request: CLOUDSTACK-8840: Systemd service for the ...
Posted by remibergsma <gi...@git.apache.org>.
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/820#issuecomment-140842317
@wido Yep, works fine now!
```
+ exit 0
RPM Build Done
```
Here they are:
```
[root@cs4 packaging]# ls -la ../dist/rpmbuild/RPMS/x86_64/
total 288928
drwxr-xr-x. 2 root root 4096 Sep 16 18:44 .
drwxr-xr-x. 3 root root 4096 Sep 16 18:43 ..
-rw-r--r--. 1 root root 50700428 Sep 16 18:44 cloudstack-agent-4.6.0-SNAPSHOT.el7.centos.x86_64.rpm
-rw-r--r--. 1 root root 5364 Sep 16 18:44 cloudstack-baremetal-agent-4.6.0-SNAPSHOT.el7.centos.x86_64.rpm
-rw-r--r--. 1 root root 38276 Sep 16 18:44 cloudstack-cli-4.6.0-SNAPSHOT.el7.centos.x86_64.rpm
-rw-r--r--. 1 root root 111008452 Sep 16 18:44 cloudstack-common-4.6.0-SNAPSHOT.el7.centos.x86_64.rpm
-rw-r--r--. 1 root root 88693724 Sep 16 18:43 cloudstack-management-4.6.0-SNAPSHOT.el7.centos.x86_64.rpm
-rw-r--r--. 1 root root 45395296 Sep 16 18:44 cloudstack-usage-4.6.0-SNAPSHOT.el7.centos.x86_64.rpm
```
LGTM
---
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] cloudstack pull request: CLOUDSTACK-8840: Systemd service for the ...
Posted by remibergsma <gi...@git.apache.org>.
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/820#issuecomment-140678150
@wido Sure, will do my tests again today!
---
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] cloudstack pull request: CLOUDSTACK-8840: Systemd service for the ...
Posted by wido <gi...@git.apache.org>.
Github user wido commented on the pull request:
https://github.com/apache/cloudstack/pull/820#issuecomment-140676140
@bhaisaab Tnx! I tested it on a CentOS 7 system:
```
[root@vm-d818122e-51ca-479f-9618-051b4eed7c10 system]# systemctl status cloudstack-usage
cloudstack-usage.service - CloudStack Usage Server
Loaded: loaded (/etc/systemd/system/cloudstack-usage.service; disabled)
Active: active (running) since wo 2015-09-16 10:38:48 CEST; 3s ago
Docs: http://www.cloudstack.org/
Main PID: 2492 (sh)
CGroup: /system.slice/cloudstack-usage.service
```
@remibergsma Can you verify if the packages now build properly? I would love to get this into 4.6 so that we are systemdified :)
---
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.
---