BYO Algorithm training should have support for vpc (#370)

* vpc config should be initialized for Estimator

* add tags also to signature

* Add change log and docstring

Author: whiletruelearn

CHANGELOG.rst

@@ -6,6 +6,7 @@ CHANGELOG
 ========
 
 * bug-fix: Local Mode: Create output/data directory expected by SageMaker Container.
+* bug-fix: Estimator accepts the vpc configs made capable by 1.9.1
 
 1.9.2
 =====

src/sagemaker/estimator.py

@@ -429,7 +429,7 @@ class Estimator(EstimatorBase):
     def __init__(self, image_name, role, train_instance_count, train_instance_type,
                  train_volume_size=30, train_max_run=24 * 60 * 60, input_mode='File',
                  output_path=None, output_kms_key=None, base_job_name=None, sagemaker_session=None,
-                 hyperparameters=None):
+                 hyperparameters=None, tags=None, subnets=None, security_group_ids=None):
         """Initialize an ``Estimator`` instance.
 
         Args:
@@ -467,7 +467,8 @@ def __init__(self, image_name, role, train_instance_count, train_instance_type,
         self.hyperparam_dict = hyperparameters.copy() if hyperparameters else {}
         super(Estimator, self).__init__(role, train_instance_count, train_instance_type,
                                         train_volume_size, train_max_run, input_mode,
-                                        output_path, output_kms_key, base_job_name, sagemaker_session)
+                                        output_path, output_kms_key, base_job_name, sagemaker_session,
+                                        tags, subnets, security_group_ids)
 
     def train_image(self):
         """