[GitHub] feng-tao opened a new pull request #1: Create README.md

2019-01-09 Thread GitBox
feng-tao opened a new pull request #1: Create README.md URL: https://github.com/apache/airflow-site/pull/1 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the

[GitHub] mik-laj commented on issue #1: Create README.md

2019-01-09 Thread GitBox
mik-laj commented on issue #1: Create README.md URL: https://github.com/apache/airflow-site/pull/1#issuecomment-452944297 I suggest you add an address where you can find this website on the internet. This is an automated

[GitHub] mik-laj opened a new issue #2: Website address in repo description

2019-01-09 Thread GitBox
mik-laj opened a new issue #2: Website address in repo description URL: https://github.com/apache/airflow-site/issues/2 Hello, I propose to add in the description of the repository the address where this website is located on the internet to make it easier to reach it.

[GitHub] [airflow-on-k8s-operator] kaxil merged pull request #5: Move to kubebuilder v2 generated project

2020-01-22 Thread GitBox
kaxil merged pull request #5: Move to kubebuilder v2 generated project URL: https://github.com/apache/airflow-on-k8s-operator/pull/5 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [airflow-on-k8s-operator] aijamalnk opened a new pull request #1: First commit. Performing code donation.

2020-01-15 Thread GitBox
aijamalnk opened a new pull request #1: First commit. Performing code donation. URL: https://github.com/apache/airflow-on-k8s-operator/pull/1 This is an automated message from the Apache Git Service. To respond to the

[GitHub] [airflow-on-k8s-operator] kaxil merged pull request #1: First commit. Performing code donation.

2020-01-15 Thread GitBox
kaxil merged pull request #1: First commit. Performing code donation. URL: https://github.com/apache/airflow-on-k8s-operator/pull/1 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [airflow-on-k8s-operator] kaxil commented on issue #1: First commit. Performing code donation.

2020-01-15 Thread GitBox
kaxil commented on issue #1: First commit. Performing code donation. URL: https://github.com/apache/airflow-on-k8s-operator/pull/1#issuecomment-574845244  - Merged to keep the first version of the code unchanged This is an

[GitHub] [airflow-on-k8s-operator] kaxil merged pull request #2: Fix repo - populate vendor, rename paths etc.

2020-01-21 Thread GitBox
kaxil merged pull request #2: Fix repo - populate vendor, rename paths etc. URL: https://github.com/apache/airflow-on-k8s-operator/pull/2 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [airflow-on-k8s-operator] barney-s opened a new pull request #2: Fix repo

2020-01-21 Thread GitBox
barney-s opened a new pull request #2: Fix repo URL: https://github.com/apache/airflow-on-k8s-operator/pull/2 Regenrating vendor files Updating to newer libraries renaming paths This is an automated message from the

[GitHub] [airflow-on-k8s-operator] barney-s commented on issue #3: Move to go modules and update libs

2020-01-22 Thread GitBox
barney-s commented on issue #3: Move to go modules and update libs URL: https://github.com/apache/airflow-on-k8s-operator/pull/3#issuecomment-577058473 Expect a couple more pull requests as i try to get this to a working shape. Will take this oppurtunity to modernize the code base

[GitHub] [airflow-on-k8s-operator] barney-s opened a new pull request #3: Move to go modules and update libs

2020-01-22 Thread GitBox
barney-s opened a new pull request #3: Move to go modules and update libs URL: https://github.com/apache/airflow-on-k8s-operator/pull/3 Moving from gopkg to gomod dependency management. - This removes vendor folder. - Manually vendor in controller-reconciler. Update

[GitHub] [airflow-on-k8s-operator] kaxil commented on issue #4: Add repo name to readme

2020-01-22 Thread GitBox
kaxil commented on issue #4: Add repo name to readme URL: https://github.com/apache/airflow-on-k8s-operator/pull/4#issuecomment-577295517 Good catch @feng-tao This is an automated message from the Apache Git Service. To

[GitHub] [airflow-on-k8s-operator] feng-tao commented on issue #4: Add repo name to readme

2020-01-22 Thread GitBox
feng-tao commented on issue #4: Add repo name to readme URL: https://github.com/apache/airflow-on-k8s-operator/pull/4#issuecomment-577295074 great job to get the repo into Apache! This is an automated message from the Apache

[GitHub] [airflow-on-k8s-operator] kaxil merged pull request #4: Add repo name to readme

2020-01-22 Thread GitBox
kaxil merged pull request #4: Add repo name to readme URL: https://github.com/apache/airflow-on-k8s-operator/pull/4 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[GitHub] [airflow-on-k8s-operator] feng-tao opened a new pull request #4: Add repo name to readme

2020-01-22 Thread GitBox
feng-tao opened a new pull request #4: Add repo name to readme URL: https://github.com/apache/airflow-on-k8s-operator/pull/4 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [airflow-on-k8s-operator] barney-s opened a new pull request #5: Move to kubebuilder v2 generated project

2020-01-22 Thread GitBox
barney-s opened a new pull request #5: Move to kubebuilder v2 generated project URL: https://github.com/apache/airflow-on-k8s-operator/pull/5 Re-generated scaffolding and merged existing controller files into it kubebuilder init --domain apache.org kubebuilder create api --group

[GitHub] [airflow-client-go] houqp opened a new pull request #1: Add generated go client

2020-06-30 Thread GitBox
houqp opened a new pull request #1: URL: https://github.com/apache/airflow-client-go/pull/1 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above

[GitHub] [airflow-client-go] mik-laj commented on a change in pull request #1: Add generated go client

2020-06-30 Thread GitBox
mik-laj commented on a change in pull request #1: URL: https://github.com/apache/airflow-client-go/pull/1#discussion_r447492769 ## File path: airflow/.travis.yml ## @@ -0,0 +1,8 @@ +language: go Review comment: This is automatic generated configuration. As far as we

[GitHub] [airflow-client-go] turbaszek commented on a change in pull request #1: Add generated go client

2020-06-30 Thread GitBox
turbaszek commented on a change in pull request #1: URL: https://github.com/apache/airflow-client-go/pull/1#discussion_r447491415 ## File path: airflow/.travis.yml ## @@ -0,0 +1,8 @@ +language: go Review comment: I would suggest to use Github Actions as we do in

[GitHub] [airflow-client-go] mik-laj commented on pull request #1: Add generated go client

2020-06-30 Thread GitBox
mik-laj commented on pull request #1: URL: https://github.com/apache/airflow-client-go/pull/1#issuecomment-651638355 @gerardo We're already working to fix it. This is an automated message from the Apache Git Service. To

[GitHub] [airflow-client-go] turbaszek commented on pull request #1: Add generated go client

2020-06-30 Thread GitBox
turbaszek commented on pull request #1: URL: https://github.com/apache/airflow-client-go/pull/1#issuecomment-651624991 Also most of the files miss ASF license so I think it would be nice to bring some Airflow pre-commits to this repo? We can also add compilation step to be sure the

[GitHub] [airflow-client-go] potiuk commented on pull request #1: Add generated go client

2020-06-30 Thread GitBox
potiuk commented on pull request #1: URL: https://github.com/apache/airflow-client-go/pull/1#issuecomment-651624109 > Do we plan to have E2E tests? I think the plan is just to make PR out of the generated code here but the tests will be executed still in the main airflow repo. That

[GitHub] [airflow-client-go] mik-laj commented on pull request #1: Add generated go client

2020-06-30 Thread GitBox
mik-laj commented on pull request #1: URL: https://github.com/apache/airflow-client-go/pull/1#issuecomment-651634016 > I think the plan is just to make PR out of the generated code here but the tests will be executed still in the main airflow repo. That seems like most sensible thing to

[GitHub] [airflow-client-go] mik-laj commented on a change in pull request #1: Add generated go client

2020-06-30 Thread GitBox
mik-laj commented on a change in pull request #1: URL: https://github.com/apache/airflow-client-go/pull/1#discussion_r447490387 ## File path: airflow/.travis.yml ## @@ -0,0 +1,8 @@ +language: go Review comment: Should this file be in the root directory?

[GitHub] [airflow-client-go] mik-laj commented on pull request #1: Add generated go client

2020-06-30 Thread GitBox
mik-laj commented on pull request #1: URL: https://github.com/apache/airflow-client-go/pull/1#issuecomment-651636094 > I couldn't find an option to include license header in all generated files in openapi-generator. You need to generate a client and then run another tool that will

[GitHub] [airflow-client-go] gerardo commented on pull request #1: Add generated go client

2020-06-30 Thread GitBox
gerardo commented on pull request #1: URL: https://github.com/apache/airflow-client-go/pull/1#issuecomment-651636800 This PR started polluting the dev@airflow.apache.org list. Was this intended? This is an automated message

[GitHub] [airflow-client-go] houqp commented on pull request #1: Add generated go client

2020-06-30 Thread GitBox
houqp commented on pull request #1: URL: https://github.com/apache/airflow-client-go/pull/1#issuecomment-651632819 I couldn't find an option to include license header in all generated files in openapi-generator. This is an

[GitHub] [airflow-client-go] houqp commented on pull request #1: Add generated go client

2020-06-30 Thread GitBox
houqp commented on pull request #1: URL: https://github.com/apache/airflow-client-go/pull/1#issuecomment-651622366 @turbaszek i think that would be something useful to add, but would be better added as a separate PR to reduce scope of this current since it's already a pretty big change :P

[GitHub] [airflow-client-go] houqp edited a comment on pull request #1: Add generated go client

2020-06-30 Thread GitBox
houqp edited a comment on pull request #1: URL: https://github.com/apache/airflow-client-go/pull/1#issuecomment-651622366 @turbaszek i think that would be something useful to add, but would be better added as a follow up PR to reduce scope of this current since it's already a pretty big

[GitHub] [airflow-client-go] turbaszek commented on pull request #1: Add generated go client

2020-06-30 Thread GitBox
turbaszek commented on pull request #1: URL: https://github.com/apache/airflow-client-go/pull/1#issuecomment-651616314 Do we plan to have E2E tests? This is an automated message from the Apache Git Service. To respond to

[GitHub] [airflow-client-go] mik-laj commented on pull request #1: Add generated go client

2020-06-30 Thread GitBox
mik-laj commented on pull request #1: URL: https://github.com/apache/airflow-client-go/pull/1#issuecomment-651631792 I am wondering about the lack of license headers, but I think we can do them later. The main thing is to do it before the release of the first version.

[GitHub] [airflow-client-go] houqp commented on a change in pull request #1: Add generated go client

2020-06-30 Thread GitBox
houqp commented on a change in pull request #1: URL: https://github.com/apache/airflow-client-go/pull/1#discussion_r447496649 ## File path: airflow/.travis.yml ## @@ -0,0 +1,8 @@ +language: go Review comment: this is generated by openapi-spec, which is not useful for

[GitHub] [airflow-client-go] mik-laj edited a comment on pull request #1: Add generated go client

2020-06-30 Thread GitBox
mik-laj edited a comment on pull request #1: URL: https://github.com/apache/airflow-client-go/pull/1#issuecomment-651636094 > I couldn't find an option to include license header in all generated files in openapi-generator. You need to generate a client and then run another tool that

[GitHub] [airflow-client-go] mik-laj commented on a change in pull request #1: Add generated go client

2020-06-30 Thread GitBox
mik-laj commented on a change in pull request #1: URL: https://github.com/apache/airflow-client-go/pull/1#discussion_r447504501 ## File path: airflow/.travis.yml ## @@ -0,0 +1,8 @@ +language: go Review comment: You will still be able to do this if you use -