You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cloudstack.apache.org by borisroman <gi...@git.apache.org> on 2015/11/23 00:58:08 UTC
[GitHub] cloudstack pull request: Debian/Ubuntu packaging issues
GitHub user borisroman opened a pull request:
https://github.com/apache/cloudstack/pull/1105
Debian/Ubuntu packaging issues
See individual issues.
Before:
```
root@acs46:~# ls -la /var/lib/cloudstack
total 16
drwxr-xr-x 4 root root 4096 Nov 23 00:20 .
drwxr-xr-x 46 root root 4096 Nov 23 00:21 ..
drwxrwx--- 2 root cloud 4096 Nov 17 01:15 management
drwxr-xr-x 2 root root 4096 Nov 17 01:15 mnt
```
```
root@acs46:~# ls -la /usr/share/cloudstack-management/webapps/client/WEB-INF/classes/scripts/storage/secondary/
total 152
drwxr-xr-x 2 root root 4096 Nov 23 00:20 .
drwxr-xr-x 4 root root 4096 Nov 23 00:20 ..
-rw-r--r-- 1 root root 10092 Nov 17 01:15 cloud-install-sys-tmplt
-rw-r--r-- 1 root root 10591 Nov 17 01:15 cloud-install-sys-tmplt.py
-rw-r--r-- 1 root root 11254 Nov 23 00:20 cloud-install-sys-tmplt.pyc
-rw-r--r-- 1 root root 2362 Nov 17 01:15 create_privatetemplate_from_snapshot_xen.sh
-rw-r--r-- 1 root root 5446 Nov 17 01:15 createtmplt.sh
-rw-r--r-- 1 root root 5388 Nov 17 01:15 createvolume.sh
-rw-r--r-- 1 root root 2532 Nov 17 01:15 installIso.sh
-rw-r--r-- 1 root root 1622 Nov 17 01:15 listvmtmplt.sh
-rw-r--r-- 1 root root 1621 Nov 17 01:15 listvolume.sh
-rw-r--r-- 1 root root 75712 Nov 17 01:15 swift
```
After:
```
root@acs46:~# ls -la /var/lib/cloudstack
total 16
drwxr-xr-x 4 cloud cloud 4096 Nov 23 00:51 .
drwxr-xr-x 46 root root 4096 Nov 23 00:53 ..
drwxrwx--- 2 cloud cloud 4096 Nov 23 00:33 management
drwxr-xr-x 2 cloud cloud 4096 Nov 23 00:33 mnt
```
```
root@acs46:~# ls -la /usr/share/cloudstack-management/webapps/client/WEB-INF/classes/scripts/storage/secondary/
total 152
drwxr-xr-x 2 root root 4096 Nov 23 00:53 .
drwxr-xr-x 4 root root 4096 Nov 23 00:53 ..
-rwxr-xr-x 1 root root 10092 Nov 23 00:33 cloud-install-sys-tmplt
-rwxr-xr-x 1 root root 10591 Nov 23 00:33 cloud-install-sys-tmplt.py
-rwxr-xr-x 1 root root 11254 Nov 23 00:53 cloud-install-sys-tmplt.pyc
-rwxr-xr-x 1 root root 2362 Nov 23 00:33 create_privatetemplate_from_snapshot_xen.sh
-rwxr-xr-x 1 root root 5446 Nov 23 00:33 createtmplt.sh
-rwxr-xr-x 1 root root 5388 Nov 23 00:33 createvolume.sh
-rwxr-xr-x 1 root root 2532 Nov 23 00:33 installIso.sh
-rwxr-xr-x 1 root root 1622 Nov 23 00:33 listvmtmplt.sh
-rwxr-xr-x 1 root root 1621 Nov 23 00:33 listvolume.sh
-rwxr-xr-x 1 root root 75712 Nov 23 00:33 swift
```
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/borisroman/cloudstack packagingIssues
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/cloudstack/pull/1105.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 #1105
----
commit 1e73f667c93fe7bc7a765f205e1eb62fb778f99c
Author: Boris Schrijver <bo...@pcextreme.nl>
Date: 2015-11-22T23:05:53Z
CLOUDSTACK-9076: Changed ownership of directory /var/lib/cloudstack to cloud.
commit e2fc270480916901b2bfd4f4bab376a6008f9f57
Author: Boris Schrijver <bo...@pcextreme.nl>
Date: 2015-11-22T23:07:29Z
CLOUDSTACK-9078: Gave scripts executable permissions.
----
---
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: Debian/Ubuntu packaging issues
Posted by bhaisaab <gi...@git.apache.org>.
Github user bhaisaab commented on the pull request:
https://github.com/apache/cloudstack/pull/1105#issuecomment-158873996
LGTM, should we fix the scripts in src paths?
---
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: Debian/Ubuntu packaging issues
Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:
https://github.com/apache/cloudstack/pull/1105
---
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: Debian/Ubuntu packaging issues
Posted by bhaisaab <gi...@git.apache.org>.
Github user bhaisaab commented on the pull request:
https://github.com/apache/cloudstack/pull/1105#issuecomment-158973478
@borisroman I mean, if it helps to do chmod +x to the scripts in scripts/storage in the git repo itself?
---
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: Debian/Ubuntu packaging issues
Posted by borisroman <gi...@git.apache.org>.
Github user borisroman commented on the pull request:
https://github.com/apache/cloudstack/pull/1105#issuecomment-158918940
Hi @bhaisaab, what do you mean with "fix the scripts in src paths" ? Regards
---
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: Debian/Ubuntu packaging issues
Posted by borisroman <gi...@git.apache.org>.
Github user borisroman commented on the pull request:
https://github.com/apache/cloudstack/pull/1105#issuecomment-158980731
@bhaisaab Ah, i've already looked them up, and in fact, in the git repo, they have the +x flag. So during the install, they have that flagged stripped...
---
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: Debian/Ubuntu packaging issues
Posted by remibergsma <gi...@git.apache.org>.
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/1105#issuecomment-160290985
LGTM, haven't verified Debian packages though.
---
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.
---