[GitHub] cloudstack pull request: CLOUDSTACK-9010: adjust packaging for cen...

2015-11-30 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/1008#issuecomment-160588176 @davidamorimfaria @remibergsma should we port this to 4.5? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] cloudstack pull request: CLOUDSTACK-9010: adjust packaging for cen...

2015-11-30 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1008#issuecomment-160589376 @bhaisaab I think it is fine to do so, but centos7 support was still experimental. Not sure if we spring hopes there that are unjust. --- If your project is

[GitHub] cloudstack pull request: CLOUDSTACK-9010: adjust packaging for cen...

2015-11-30 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/1008#issuecomment-160590038 @DaanHoogland I see, leaving it then. CentOS7 users should use 4.6/4.7 then. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] cloudstack pull request: CLOUDSTACK-9010: adjust packaging for cen...

2015-11-30 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/1008#issuecomment-160599550 @davidamorimfaria thanks, I personally won't be able to fix/test this soonish so dropping this unless of course you or someone wants this in 4.5 and sends a PR.

[GitHub] cloudstack pull request: CLOUDSTACK-9010: adjust packaging for cen...

2015-11-30 Thread davidamorimfaria
Github user davidamorimfaria commented on the pull request: https://github.com/apache/cloudstack/pull/1008#issuecomment-160594088 @bhaisaab I see no harm in trying to package 4.5 with this spec. It should just work (read, services start) if all the files expected by the spec file are

[GitHub] cloudstack pull request: CLOUDSTACK-9010: adjust packaging for cen...

2015-10-31 Thread davidamorimfaria
Github user davidamorimfaria commented on the pull request: https://github.com/apache/cloudstack/pull/1008#issuecomment-152712182 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

[GitHub] cloudstack pull request: CLOUDSTACK-9010: adjust packaging for cen...

2015-10-31 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1008 --- 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

[GitHub] cloudstack pull request: CLOUDSTACK-9010: adjust packaging for cen...

2015-10-30 Thread davidamorimfaria
Github user davidamorimfaria commented on the pull request: https://github.com/apache/cloudstack/pull/1008#issuecomment-152536728 I haven't installed a cloud with the packages made with this change, but so far the management servers start and talk to each other: Oct 30

[GitHub] cloudstack pull request: CLOUDSTACK-9010: adjust packaging for cen...

2015-10-30 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1008#issuecomment-152578560 @davidamorimfaria Final request, please squash your commits. We need them to be atomic. Please ping me when done, thanks! --- If your project is set up for

[GitHub] cloudstack pull request: CLOUDSTACK-9010: adjust packaging for cen...

2015-10-30 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1008#issuecomment-152568600 based on the review of the changes and looking at the deployment of the management servers i think this LGTM --- If your project is set up for it, you can

[GitHub] cloudstack pull request: CLOUDSTACK-9010: adjust packaging for cen...

2015-10-30 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1008#issuecomment-152573997 This PR fixes CentOS 7 packaging and obsoletes #888 and CLOUDSTACK-8812. I will merge this and close the other. --- If your project is set up for it, you can