Return-Path: X-Original-To: apmail-cloudstack-dev-archive@www.apache.org Delivered-To: apmail-cloudstack-dev-archive@www.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 83BF21986C for ; Mon, 25 Apr 2016 12:40:30 +0000 (UTC) Received: (qmail 97613 invoked by uid 500); 25 Apr 2016 12:40:30 -0000 Delivered-To: apmail-cloudstack-dev-archive@cloudstack.apache.org Received: (qmail 97558 invoked by uid 500); 25 Apr 2016 12:40:30 -0000 Mailing-List: contact dev-help@cloudstack.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@cloudstack.apache.org Delivered-To: mailing list dev@cloudstack.apache.org Received: (qmail 97547 invoked by uid 99); 25 Apr 2016 12:40:29 -0000 Received: from git1-us-west.apache.org (HELO git1-us-west.apache.org) (140.211.11.23) by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 25 Apr 2016 12:40:29 +0000 Received: by git1-us-west.apache.org (ASF Mail Server at git1-us-west.apache.org, from userid 33) id 7CC09DFE16; Mon, 25 Apr 2016 12:40:29 +0000 (UTC) From: nvazquez To: dev@cloudstack.apache.org Reply-To: dev@cloudstack.apache.org Message-ID: Subject: [GitHub] cloudstack pull request: CLOUDSTACK-9368: Fix for Support configur... Content-Type: text/plain Date: Mon, 25 Apr 2016 12:40:29 +0000 (UTC) GitHub user nvazquez opened a pull request: https://github.com/apache/cloudstack/pull/1518 CLOUDSTACK-9368: Fix for Support configurable NFS version for Secondary Storage mounts ## Description JIRA TICKET: https://issues.apache.org/jira/browse/CLOUDSTACK-9368 This pull request address a problem introduced in #1361 in which NFS version couldn't be changed after hosts resources were configured on startup (for hosts using `VmwareResource`), and as host parameters didn't include `nfs.version` key, it was set `null`. ## Proposed solution In this proposed solution `nfsVersion` would be passed in `NfsTO` through `CopyCommand` to `VmwareResource`, who will check if NFS version is still configured or not. If not, it will use the one sent in the command and will set it to its storage processor and storage handler. After those setups, it will proceed executing command. You can merge this pull request into a Git repository by running: $ git pull https://github.com/nvazquez/cloudstack testnfs Alternatively you can review and apply these changes as the patch at: https://github.com/apache/cloudstack/pull/1518.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 #1518 ---- commit d0fa3245484b4e74a9f7be364deb6bb47da12de6 Author: nvazquez Date: 2016-04-22T19:47:05Z CLOUDSTACK-9368: Fix NFS version set on VmwareResource ---- --- 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. ---