Return-Path: X-Original-To: archive-asf-public-internal@cust-asf2.ponee.io Delivered-To: archive-asf-public-internal@cust-asf2.ponee.io Received: from cust-asf.ponee.io (cust-asf.ponee.io [163.172.22.183]) by cust-asf2.ponee.io (Postfix) with ESMTP id 109D4200CC5 for ; Tue, 11 Jul 2017 20:11:07 +0200 (CEST) Received: by cust-asf.ponee.io (Postfix) id 0F6B2166CEF; Tue, 11 Jul 2017 18:11:07 +0000 (UTC) Delivered-To: archive-asf-public@cust-asf.ponee.io Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by cust-asf.ponee.io (Postfix) with SMTP id 5D930166CEA for ; Tue, 11 Jul 2017 20:11:06 +0200 (CEST) Received: (qmail 98795 invoked by uid 500); 11 Jul 2017 18:11:05 -0000 Mailing-List: contact commits-help@airflow.incubator.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@airflow.incubator.apache.org Delivered-To: mailing list commits@airflow.incubator.apache.org Received: (qmail 98786 invoked by uid 99); 11 Jul 2017 18:11:05 -0000 Received: from pnap-us-west-generic-nat.apache.org (HELO spamd3-us-west.apache.org) (209.188.14.142) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 11 Jul 2017 18:11:05 +0000 Received: from localhost (localhost [127.0.0.1]) by spamd3-us-west.apache.org (ASF Mail Server at spamd3-us-west.apache.org) with ESMTP id CB0611810D7 for ; Tue, 11 Jul 2017 18:11:04 +0000 (UTC) X-Virus-Scanned: Debian amavisd-new at spamd3-us-west.apache.org X-Spam-Flag: NO X-Spam-Score: -100.001 X-Spam-Level: X-Spam-Status: No, score=-100.001 tagged_above=-999 required=6.31 tests=[RP_MATCHES_RCVD=-0.001, SPF_PASS=-0.001, URIBL_BLOCKED=0.001, USER_IN_WHITELIST=-100] autolearn=disabled Received: from mx1-lw-eu.apache.org ([10.40.0.8]) by localhost (spamd3-us-west.apache.org [10.40.0.10]) (amavisd-new, port 10024) with ESMTP id lpkC0fQBWS2q for ; Tue, 11 Jul 2017 18:11:03 +0000 (UTC) Received: from mailrelay1-us-west.apache.org (mailrelay1-us-west.apache.org [209.188.14.139]) by mx1-lw-eu.apache.org (ASF Mail Server at mx1-lw-eu.apache.org) with ESMTP id D758F6245D for ; Tue, 11 Jul 2017 18:11:02 +0000 (UTC) Received: from jira-lw-us.apache.org (unknown [207.244.88.139]) by mailrelay1-us-west.apache.org (ASF Mail Server at mailrelay1-us-west.apache.org) with ESMTP id 8D15FE0E29 for ; Tue, 11 Jul 2017 18:11:01 +0000 (UTC) Received: from jira-lw-us.apache.org (localhost [127.0.0.1]) by jira-lw-us.apache.org (ASF Mail Server at jira-lw-us.apache.org) with ESMTP id 91295246F2 for ; Tue, 11 Jul 2017 18:11:00 +0000 (UTC) Date: Tue, 11 Jul 2017 18:11:00 +0000 (UTC) From: "Peter Dolan (JIRA)" To: commits@airflow.incubator.apache.org Message-ID: In-Reply-To: References: Subject: [jira] [Updated] (AIRFLOW-1401) Standardize GCP project, region, and zone argument names MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-JIRA-FingerPrint: 30527f35849b9dde25b450d4833f0394 archived-at: Tue, 11 Jul 2017 18:11:07 -0000 [ https://issues.apache.org/jira/browse/AIRFLOW-1401?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Peter Dolan updated AIRFLOW-1401: --------------------------------- Affects Version/s: (was: 1.9.0) > Standardize GCP project, region, and zone argument names > -------------------------------------------------------- > > Key: AIRFLOW-1401 > URL: https://issues.apache.org/jira/browse/AIRFLOW-1401 > Project: Apache Airflow > Issue Type: Improvement > Components: contrib > Affects Versions: 1.8.1 > Reporter: Peter Dolan > Assignee: Peter Dolan > > At the moment, there isn't standard usage of operator arguments for Google Cloud Platform across the contributions, primarily in the usage of the parameter meaning the GCP project name/id. This makes it difficult to specify default_arguments that work across all GCP-centric operators in a graph. > Using the command `grep -r project airflow/contrib/*`, we can see these uses: > project_id: > * gcp_dataproc_hook > * datastore_hook > * gcp_api_base_hook > * bigquery_hook > * dataproc_operator > * bigquery_sensor > project: > * gcp_pubsub_hook (here 'project' means project id or project name, which does not fully understand the distinction within GCP between project id and project name as elements of the REST api) > * dataflow_operator (see note below) > * pubsub_operator > project_name: > * gcp_cloudml_hook > * cloudml_operator > Notably, the Dataflow Operator diverges from the pattern of using top-level operator parameters by specifying an options dict, which can be populated by the dataflow_default_options dict. This can contain 'project', and 'zone.' > This improvement proposes to standardize the above operators (at least) on > * project_id (meaning '' in this example request: GET https://www.googleapis.com/compute/v1/projects//zones//instances/) > * region > * zone > This can be done by changing the names of parameters of operators and hooks that were not included in the 1.8.2 release (cloud ml and pubsub), and by adding parameters to operators and hooks that were included in 1.8.2 (and internally copying the old parameter name to the new one, and deprecating the old one). -- This message was sent by Atlassian JIRA (v6.4.14#64029)