[jira] [Work logged] (BEAM-2880) Artifact server proxies
[ https://issues.apache.org/jira/browse/BEAM-2880?focusedWorklogId=146787&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-146787 ] ASF GitHub Bot logged work on BEAM-2880: Author: ASF GitHub Bot Created on: 22/Sep/18 15:43 Start Date: 22/Sep/18 15:43 Worklog Time Spent: 10m Work Description: herohde closed pull request #6466: [BEAM-2880] Remove gcemd and gcsproxy container images URL: https://github.com/apache/beam/pull/6466 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull request (from a fork), the diff is supplied below (as it won't show otherwise due to GitHub magic): diff --git a/build.gradle b/build.gradle index 2d06c37745c..ba30ee92c9d 100644 --- a/build.gradle +++ b/build.gradle @@ -209,8 +209,6 @@ task goPreCommit() { dependsOn ":beam-sdks-java-container:build" dependsOn ":beam-sdks-python-container:build" dependsOn ":beam-sdks-go-container:build" - dependsOn ":beam-runners-gcp-gcemd:build" - dependsOn ":beam-runners-gcp-gcsproxy:build" } task goPostCommit() { diff --git a/runners/gcp/OWNERS b/runners/gcp/OWNERS deleted file mode 100644 index 6270c9cfc97..000 --- a/runners/gcp/OWNERS +++ /dev/null @@ -1,7 +0,0 @@ -# See the OWNERS docs at https://s.apache.org/beam-owners - -reviewers: - - pabloem - - herohde - - lukecwik - - aaltay diff --git a/runners/gcp/gcemd/Dockerfile b/runners/gcp/gcemd/Dockerfile deleted file mode 100644 index b8fa8aaf3e2..000 --- a/runners/gcp/gcemd/Dockerfile +++ /dev/null @@ -1,30 +0,0 @@ -### -# Licensed to the Apache Software Foundation (ASF) under one -# or more contributor license agreements. See the NOTICE file -# distributed with this work for additional information -# regarding copyright ownership. The ASF licenses this file -# to you under the Apache License, Version 2.0 (the -# "License"); you may not use this file except in compliance -# with the License. You may obtain a copy of the License at -# -# http://www.apache.org/licenses/LICENSE-2.0 -# -# Unless required by applicable law or agreed to in writing, software -# distributed under the License is distributed on an "AS IS" BASIS, -# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. -# See the License for the specific language governing permissions and -# limitations under the License. -### - -FROM debian:stretch -MAINTAINER "Apache Beam " - -RUN apt-get update && \ -DEBIAN_FRONTEND=noninteractive apt-get install -y \ -ca-certificates \ -&& \ -rm -rf /var/lib/apt/lists/* - -ADD target/linux_amd64/gcemd /opt/apache/beam/ - -ENTRYPOINT ["/opt/apache/beam/gcemd"] diff --git a/runners/gcp/gcemd/build.gradle b/runners/gcp/gcemd/build.gradle deleted file mode 100644 index 1b0d3f8e07e..000 --- a/runners/gcp/gcemd/build.gradle +++ /dev/null @@ -1,52 +0,0 @@ -/* - * Licensed to the Apache Software Foundation (ASF) under one - * or more contributor license agreements. See the NOTICE file - * distributed with this work for additional information - * regarding copyright ownership. The ASF licenses this file - * to you under the Apache License, Version 2.0 (the - * License); you may not use this file except in compliance - * with the License. You may obtain a copy of the License at - * - * http://www.apache.org/licenses/LICENSE-2.0 - * - * Unless required by applicable law or agreed to in writing, software - * distributed under the License is distributed on an AS IS BASIS, - * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. - * See the License for the specific language governing permissions and - * limitations under the License. - */ - -apply plugin: org.apache.beam.gradle.BeamModulePlugin -applyGoNature() -applyDockerNature() - -description = "Apache Beam :: Runners :: Google Cloud Platform :: GCE metadata provisioning" - -// Figure out why the golang plugin does not add a build dependency between projects. -// Without the line below, we get spurious errors about not being able to resolve -// "./github.com/apache/beam/sdks/go" -resolveBuildDependencies.dependsOn ":beam-sdks-go:build" - -dependencies { - golang { -// TODO(herohde): use "./" prefix to prevent gogradle use base github path, for now. -// TODO(herohde): get the pkg subdirectory only, if possible. We spend mins pulling cmd/beamctl deps. -build name: './github.com/apache/beam/sdks/go', dir: project(':beam-sdks-go').projectDir -test name: './github.com/apache/beam/sdks/go', dir: project(':beam-sdks-go').projectDir - } -} - -golang { - packagePath = 'github.com/apache/beam/runners/gcp/gcemd' -
[jira] [Work logged] (BEAM-2880) Artifact server proxies
[ https://issues.apache.org/jira/browse/BEAM-2880?focusedWorklogId=146589&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-146589 ] ASF GitHub Bot logged work on BEAM-2880: Author: ASF GitHub Bot Created on: 21/Sep/18 20:54 Start Date: 21/Sep/18 20:54 Worklog Time Spent: 10m Work Description: herohde commented on issue #6466: [BEAM-2880] Remove gcemd and gcsproxy container images URL: https://github.com/apache/beam/pull/6466#issuecomment-423667600 Run Python PreCommit This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org Issue Time Tracking --- Worklog Id: (was: 146589) Time Spent: 1h 10m (was: 1h) > Artifact server proxies > --- > > Key: BEAM-2880 > URL: https://issues.apache.org/jira/browse/BEAM-2880 > Project: Beam > Issue Type: Improvement > Components: beam-model >Reporter: Henning Rohde >Assignee: Henning Rohde >Priority: Minor > Labels: portability > Time Spent: 1h 10m > Remaining Estimate: 0h > > As per https://s.apache.org/beam-fn-api-container-contract, we should add > runner-agnostic (dockerized) artifact proxies for various runner > environments. These proxies implement the server side of the artifact API. > We'll likely need proxies for GCS, S3, Azure Storage, and local. If the > proxies are implemented in a statically-linked language like Go, say, then > they do not have a dependency footprint and can easily be used as native > binaries or in dockerized form -- at the discretion of the runner. -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Work logged] (BEAM-2880) Artifact server proxies
[ https://issues.apache.org/jira/browse/BEAM-2880?focusedWorklogId=146586&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-146586 ] ASF GitHub Bot logged work on BEAM-2880: Author: ASF GitHub Bot Created on: 21/Sep/18 20:53 Start Date: 21/Sep/18 20:53 Worklog Time Spent: 10m Work Description: herohde commented on issue #6466: [BEAM-2880] Remove gcemd and gcsproxy container images URL: https://github.com/apache/beam/pull/6466#issuecomment-423667372 Run Java PreCommit This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org Issue Time Tracking --- Worklog Id: (was: 146586) Time Spent: 40m (was: 0.5h) > Artifact server proxies > --- > > Key: BEAM-2880 > URL: https://issues.apache.org/jira/browse/BEAM-2880 > Project: Beam > Issue Type: Improvement > Components: beam-model >Reporter: Henning Rohde >Assignee: Henning Rohde >Priority: Minor > Labels: portability > Time Spent: 40m > Remaining Estimate: 0h > > As per https://s.apache.org/beam-fn-api-container-contract, we should add > runner-agnostic (dockerized) artifact proxies for various runner > environments. These proxies implement the server side of the artifact API. > We'll likely need proxies for GCS, S3, Azure Storage, and local. If the > proxies are implemented in a statically-linked language like Go, say, then > they do not have a dependency footprint and can easily be used as native > binaries or in dockerized form -- at the discretion of the runner. -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Work logged] (BEAM-2880) Artifact server proxies
[ https://issues.apache.org/jira/browse/BEAM-2880?focusedWorklogId=146588&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-146588 ] ASF GitHub Bot logged work on BEAM-2880: Author: ASF GitHub Bot Created on: 21/Sep/18 20:53 Start Date: 21/Sep/18 20:53 Worklog Time Spent: 10m Work Description: herohde commented on issue #6466: [BEAM-2880] Remove gcemd and gcsproxy container images URL: https://github.com/apache/beam/pull/6466#issuecomment-423667565 Thanks Ankur. I doubt the precommit failures are related to this change. This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org Issue Time Tracking --- Worklog Id: (was: 146588) Time Spent: 1h (was: 50m) > Artifact server proxies > --- > > Key: BEAM-2880 > URL: https://issues.apache.org/jira/browse/BEAM-2880 > Project: Beam > Issue Type: Improvement > Components: beam-model >Reporter: Henning Rohde >Assignee: Henning Rohde >Priority: Minor > Labels: portability > Time Spent: 1h > Remaining Estimate: 0h > > As per https://s.apache.org/beam-fn-api-container-contract, we should add > runner-agnostic (dockerized) artifact proxies for various runner > environments. These proxies implement the server side of the artifact API. > We'll likely need proxies for GCS, S3, Azure Storage, and local. If the > proxies are implemented in a statically-linked language like Go, say, then > they do not have a dependency footprint and can easily be used as native > binaries or in dockerized form -- at the discretion of the runner. -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Work logged] (BEAM-2880) Artifact server proxies
[ https://issues.apache.org/jira/browse/BEAM-2880?focusedWorklogId=146587&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-146587 ] ASF GitHub Bot logged work on BEAM-2880: Author: ASF GitHub Bot Created on: 21/Sep/18 20:53 Start Date: 21/Sep/18 20:53 Worklog Time Spent: 10m Work Description: herohde commented on issue #6466: [BEAM-2880] Remove gcemd and gcsproxy container images URL: https://github.com/apache/beam/pull/6466#issuecomment-423667423 Run Python PreCommit This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org Issue Time Tracking --- Worklog Id: (was: 146587) Time Spent: 50m (was: 40m) > Artifact server proxies > --- > > Key: BEAM-2880 > URL: https://issues.apache.org/jira/browse/BEAM-2880 > Project: Beam > Issue Type: Improvement > Components: beam-model >Reporter: Henning Rohde >Assignee: Henning Rohde >Priority: Minor > Labels: portability > Time Spent: 50m > Remaining Estimate: 0h > > As per https://s.apache.org/beam-fn-api-container-contract, we should add > runner-agnostic (dockerized) artifact proxies for various runner > environments. These proxies implement the server side of the artifact API. > We'll likely need proxies for GCS, S3, Azure Storage, and local. If the > proxies are implemented in a statically-linked language like Go, say, then > they do not have a dependency footprint and can easily be used as native > binaries or in dockerized form -- at the discretion of the runner. -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Work logged] (BEAM-2880) Artifact server proxies
[ https://issues.apache.org/jira/browse/BEAM-2880?focusedWorklogId=146571&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-146571 ] ASF GitHub Bot logged work on BEAM-2880: Author: ASF GitHub Bot Created on: 21/Sep/18 19:43 Start Date: 21/Sep/18 19:43 Worklog Time Spent: 10m Work Description: angoenka commented on issue #6466: [BEAM-2880] Remove gcemd and gcsproxy container images URL: https://github.com/apache/beam/pull/6466#issuecomment-423650553 LGTM. Please make sure the test passes This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org Issue Time Tracking --- Worklog Id: (was: 146571) Time Spent: 0.5h (was: 20m) > Artifact server proxies > --- > > Key: BEAM-2880 > URL: https://issues.apache.org/jira/browse/BEAM-2880 > Project: Beam > Issue Type: Improvement > Components: beam-model >Reporter: Henning Rohde >Assignee: Henning Rohde >Priority: Minor > Labels: portability > Time Spent: 0.5h > Remaining Estimate: 0h > > As per https://s.apache.org/beam-fn-api-container-contract, we should add > runner-agnostic (dockerized) artifact proxies for various runner > environments. These proxies implement the server side of the artifact API. > We'll likely need proxies for GCS, S3, Azure Storage, and local. If the > proxies are implemented in a statically-linked language like Go, say, then > they do not have a dependency footprint and can easily be used as native > binaries or in dockerized form -- at the discretion of the runner. -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Work logged] (BEAM-2880) Artifact server proxies
[ https://issues.apache.org/jira/browse/BEAM-2880?focusedWorklogId=146528&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-146528 ] ASF GitHub Bot logged work on BEAM-2880: Author: ASF GitHub Bot Created on: 21/Sep/18 18:15 Start Date: 21/Sep/18 18:15 Worklog Time Spent: 10m Work Description: herohde commented on issue #6466: [BEAM-2880] Remove gcemd and gcsproxy container images URL: https://github.com/apache/beam/pull/6466#issuecomment-423627089 R: @angoenka This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org Issue Time Tracking --- Worklog Id: (was: 146528) Time Spent: 20m (was: 10m) > Artifact server proxies > --- > > Key: BEAM-2880 > URL: https://issues.apache.org/jira/browse/BEAM-2880 > Project: Beam > Issue Type: Improvement > Components: beam-model >Reporter: Henning Rohde >Assignee: Henning Rohde >Priority: Minor > Labels: portability > Time Spent: 20m > Remaining Estimate: 0h > > As per https://s.apache.org/beam-fn-api-container-contract, we should add > runner-agnostic (dockerized) artifact proxies for various runner > environments. These proxies implement the server side of the artifact API. > We'll likely need proxies for GCS, S3, Azure Storage, and local. If the > proxies are implemented in a statically-linked language like Go, say, then > they do not have a dependency footprint and can easily be used as native > binaries or in dockerized form -- at the discretion of the runner. -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Work logged] (BEAM-2880) Artifact server proxies
[ https://issues.apache.org/jira/browse/BEAM-2880?focusedWorklogId=146498&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-146498 ] ASF GitHub Bot logged work on BEAM-2880: Author: ASF GitHub Bot Created on: 21/Sep/18 17:57 Start Date: 21/Sep/18 17:57 Worklog Time Spent: 10m Work Description: herohde opened a new pull request #6466: [BEAM-2880] Remove gcemd and gcsproxy containers images URL: https://github.com/apache/beam/pull/6466 Follow this checklist to help us incorporate your contribution quickly and easily: - [ ] Format the pull request title like `[BEAM-XXX] Fixes bug in ApproximateQuantiles`, where you replace `BEAM-XXX` with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue. - [ ] If this contribution is large, please file an Apache [Individual Contributor License Agreement](https://www.apache.org/licenses/icla.pdf). It will help us expedite review of your Pull Request if you tag someone (e.g. `@username`) to look at it. Post-Commit Tests Status (on master branch) Lang | SDK | Apex | Dataflow | Flink | Gearpump | Samza | Spark --- | --- | --- | --- | --- | --- | --- | --- Go | [![Build Status](https://builds.apache.org/job/beam_PostCommit_Go_GradleBuild/lastCompletedBuild/badge/icon)](https://builds.apache.org/job/beam_PostCommit_Go_GradleBuild/lastCompletedBuild/) | --- | --- | --- | --- | --- | --- Java | [![Build Status](https://builds.apache.org/job/beam_PostCommit_Java_GradleBuild/lastCompletedBuild/badge/icon)](https://builds.apache.org/job/beam_PostCommit_Java_GradleBuild/lastCompletedBuild/) | [![Build Status](https://builds.apache.org/job/beam_PostCommit_Java_ValidatesRunner_Apex_Gradle/lastCompletedBuild/badge/icon)](https://builds.apache.org/job/beam_PostCommit_Java_ValidatesRunner_Apex_Gradle/lastCompletedBuild/) | [![Build Status](https://builds.apache.org/job/beam_PostCommit_Java_ValidatesRunner_Dataflow_Gradle/lastCompletedBuild/badge/icon)](https://builds.apache.org/job/beam_PostCommit_Java_ValidatesRunner_Dataflow_Gradle/lastCompletedBuild/) | [![Build Status](https://builds.apache.org/job/beam_PostCommit_Java_ValidatesRunner_Flink_Gradle/lastCompletedBuild/badge/icon)](https://builds.apache.org/job/beam_PostCommit_Java_ValidatesRunner_Flink_Gradle/lastCompletedBuild/) | [![Build Status](https://builds.apache.org/job/beam_PostCommit_Java_ValidatesRunner_Gearpump_Gradle/lastCompletedBuild/badge/icon)](https://builds.apache.org/job/beam_PostCommit_Java_ValidatesRunner_Gearpump_Gradle/lastCompletedBuild/) | [![Build Status](https://builds.apache.org/job/beam_PostCommit_Java_ValidatesRunner_Samza_Gradle/lastCompletedBuild/badge/icon)](https://builds.apache.org/job/beam_PostCommit_Java_ValidatesRunner_Samza_Gradle/lastCompletedBuild/) | [![Build Status](https://builds.apache.org/job/beam_PostCommit_Java_ValidatesRunner_Spark_Gradle/lastCompletedBuild/badge/icon)](https://builds.apache.org/job/beam_PostCommit_Java_ValidatesRunner_Spark_Gradle/lastCompletedBuild/) Python | [![Build Status](https://builds.apache.org/job/beam_PostCommit_Python_Verify/lastCompletedBuild/badge/icon)](https://builds.apache.org/job/beam_PostCommit_Python_Verify/lastCompletedBuild/) | --- | [![Build Status](https://builds.apache.org/job/beam_PostCommit_Py_VR_Dataflow/lastCompletedBuild/badge/icon)](https://builds.apache.org/job/beam_PostCommit_Py_VR_Dataflow/lastCompletedBuild/) [![Build Status](https://builds.apache.org/job/beam_PostCommit_Py_ValCont/lastCompletedBuild/badge/icon)](https://builds.apache.org/job/beam_PostCommit_Py_ValCont/lastCompletedBuild/) | --- | --- | --- | --- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org Issue Time Tracking --- Worklog Id: (was: 146498) Time Spent: 10m Remaining Estimate: 0h > Artifact server proxies > --- > > Key: BEAM-2880 > URL: https://issues.apache.org/jira/browse/BEAM-2880 > Project: Beam > Issue Type: Improvement > Components: beam-model >Reporter: Henning Rohde >Assignee: Henning Rohde >Priority: Minor > Labels: portability > Time Spent: 10m > Remaining Estimate: 0h > > As per https://s.apache.org/beam-fn-api-container-contract, we should add > runner-agnostic (dockerized) artifact proxies for