Return-Path: 
 <dev-return-89377-apmail-cloudstack-dev-archive=cloudstack.apache.org@cloudstack.apache.org>
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 1084718263
	for <apmail-cloudstack-dev-archive@www.apache.org>;
 Fri,  5 Feb 2016 16:01:22 +0000 (UTC)
Received: (qmail 11271 invoked by uid 500); 5 Feb 2016 16:01:21 -0000
Delivered-To: apmail-cloudstack-dev-archive@cloudstack.apache.org
Received: (qmail 11210 invoked by uid 500); 5 Feb 2016 16:01:21 -0000
Mailing-List: contact dev-help@cloudstack.apache.org; run by ezmlm
Precedence: bulk
List-Help: <mailto:dev-help@cloudstack.apache.org>
List-Unsubscribe: <mailto:dev-unsubscribe@cloudstack.apache.org>
List-Post: <mailto:dev@cloudstack.apache.org>
List-Id: <dev.cloudstack.apache.org>
Reply-To: dev@cloudstack.apache.org
Delivered-To: mailing list dev@cloudstack.apache.org
Received: (qmail 11199 invoked by uid 99); 5 Feb 2016 16:01:21 -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; Fri, 05 Feb 2016 16:01:21 +0000
Received: by git1-us-west.apache.org (ASF Mail Server at
 git1-us-west.apache.org, from userid 33)
	id 45661DFF96; Fri,  5 Feb 2016 16:01:21 +0000 (UTC)
From: remibergsma <git@git.apache.org>
To: dev@cloudstack.apache.org
Reply-To: dev@cloudstack.apache.org
Message-ID: <git-pr-1402-cloudstack@git.apache.org>
Subject: [GitHub] cloudstack pull request: Check the existence of 'forceencap'
 param...
Content-Type: text/plain
Date: Fri,  5 Feb 2016 16:01:21 +0000 (UTC)

GitHub user remibergsma opened a pull request:

    https://github.com/apache/cloudstack/pull/1402

    Check the existence of 'forceencap' parameter before use

    Check the existence of 'forceencap' parameter before use.
    
    Error seen:
    
    ```
    Traceback (most recent call last):
      File "/opt/cloud/bin/update_config.py", line 140, in <module>
        process_file()
      File "/opt/cloud/bin/update_config.py", line 54, in process_file
        finish_config()
      File "/opt/cloud/bin/update_config.py", line 44, in finish_config
        returncode = configure.main(sys.argv)
      File "/opt/cloud/bin/configure.py", line 1003, in main
        vpns.process()
      File "/opt/cloud/bin/configure.py", line 488, in process
        self.configure_ipsec(self.dbag[vpn])
      File "/opt/cloud/bin/configure.py", line 544, in configure_ipsec
        file.addeq(" forceencaps=%s" % CsHelper.bool_to_yn(obj['encap']))
    KeyError: 'encap'
    ```


You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/remibergsma/cloudstack forceencap_check

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/cloudstack/pull/1402.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 #1402
    
----
commit 4ad05d1ade9d98e3a1bd268bcc1d7a0f0767d26d
Author: Remi Bergsma <github@remi.nl>
Date:   2016-02-05T15:58:01Z

    Check the existence of 'forceencap' parameter before use

----


---
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.
---