[GitHub] cloudstack pull request: CLOUDSTACK-8304: disable snapshots from n...

2015-04-18 Thread K0zka
Github user K0zka closed the pull request at: https://github.com/apache/cloudstack/pull/172 --- 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 e

[GitHub] cloudstack pull request: CLOUDSTACK-8304: disable snapshots from n...

2015-04-15 Thread K0zka
Github user K0zka commented on the pull request: https://github.com/apache/cloudstack/pull/172#issuecomment-93639076 Backport was requested by @wilderrodrigues It is not really meant as build *performance* improvement, but rather a build stability patch. --- If your project is s

[GitHub] cloudstack pull request: CLOUDSTACK-8304: disable snapshots from n...

2015-04-15 Thread karuturi
Github user karuturi commented on the pull request: https://github.com/apache/cloudstack/pull/172#issuecomment-93635210 do we need this in 4.4? its a build time performance improvement. --- If your project is set up for it, you can reply to this email and have your reply appear on Gi

[GitHub] cloudstack pull request: CLOUDSTACK-8304: disable snapshots from n...

2015-04-15 Thread K0zka
GitHub user K0zka opened a pull request: https://github.com/apache/cloudstack/pull/172 CLOUDSTACK-8304: disable snapshots from non-snapshot repositories Backport from master to 4.4 Signed-off-by: Laszlo Hornyak You can merge this pull request into a Git repository by runni

[GitHub] cloudstack pull request: CLOUDSTACK-8304: disable snapshots from n...

2015-03-10 Thread K0zka
Github user K0zka commented on the pull request: https://github.com/apache/cloudstack/pull/100#issuecomment-78041502 @karuturi Correct, if you do not have the libvirt jar and libvirt.org goes down, the build will still fail. It would be nice if we could use only the central repository

[GitHub] cloudstack pull request: CLOUDSTACK-8304: disable snapshots from n...

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

[GitHub] cloudstack pull request: CLOUDSTACK-8304: disable snapshots from n...

2015-03-10 Thread karuturi
Github user karuturi commented on the pull request: https://github.com/apache/cloudstack/pull/100#issuecomment-78038271 understood. This will definitely save build time. (will do the merge in sometime) But, the build will fail irrespective of whether snapshot is enabled right? if

[GitHub] cloudstack pull request: CLOUDSTACK-8304: disable snapshots from n...

2015-03-10 Thread K0zka
Github user K0zka commented on the pull request: https://github.com/apache/cloudstack/pull/100#issuecomment-78022061 @karuturi So the problem that it adresses is this: We have SNAPSHOT dependencies, since snapshots are not disabled on those repositories, they will be checked. Now the

[GitHub] cloudstack pull request: CLOUDSTACK-8304: disable snapshots from n...

2015-03-09 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/100#issuecomment-77995743 Not a maven expert, leaving for someone to review/merge who is. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHu

[GitHub] cloudstack pull request: CLOUDSTACK-8304: disable snapshots from n...

2015-03-09 Thread karuturi
Github user karuturi commented on the pull request: https://github.com/apache/cloudstack/pull/100#issuecomment-77817353 I dont think this fixes the issue. we are specifying a version number. maven will not check for snapshots. --- If your project is set up for it, you can reply to th

[GitHub] cloudstack pull request: CLOUDSTACK-8304: disable snapshots from n...

2015-03-09 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/100#issuecomment-77811516 LGTM. @K0zka please merge it on both master and 4.5! --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well.

[GitHub] cloudstack pull request: CLOUDSTACK-8304: disable snapshots from n...

2015-03-06 Thread K0zka
GitHub user K0zka opened a pull request: https://github.com/apache/cloudstack/pull/100 CLOUDSTACK-8304: disable snapshots from non-snapshot repositories Signed-off-by: Laszlo Hornyak You can merge this pull request into a Git repository by running: $ git pull https://github.co