raulk89 edited a comment on issue #10569:
URL: https://github.com/apache/airflow/issues/10569#issuecomment-680536818
http://localhost:8080
I am using rbac also (airflow.cfg file rbac = True)
Haven't noticed that any other link is broken.
Raul
Sent from my iPhone
raulk89 edited a comment on issue #10569:
URL: https://github.com/apache/airflow/issues/10569#issuecomment-680536818
http://localhost:8080
I am using rbac also (airflow.cfg file rbac = True)
Haven't noticed that any other link is broken except this "Log Url".
Raul
[
https://issues.apache.org/jira/browse/AIRFLOW-3311?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17184943#comment-17184943
]
ASF GitHub Bot commented on AIRFLOW-3311:
-
bhavaniravi commented on pull request #4160:
URL:
bhavaniravi commented on pull request #4160:
URL: https://github.com/apache/airflow/pull/4160#issuecomment-680655869
Any idea why this feature is closed? I would love to have this feature
This is an automated message from
raulk89 commented on issue #10569:
URL: https://github.com/apache/airflow/issues/10569#issuecomment-680536818
http://localhost:8080
Raul
Sent from my iPhone
> On 26. Aug 2020, at 02:29, Kamil Breguła wrote:
>
>
> What is your value for base_url option in
759803573 opened a new pull request #10573:
URL: https://github.com/apache/airflow/pull/10573
* complete class name from 1.10.12 CHANGELOG
The implementation location cannot be quickly searched in GitHub through
`ClusterPolicyViolation`
boring-cyborg[bot] commented on pull request #10573:
URL: https://github.com/apache/airflow/pull/10573#issuecomment-680464259
Congratulations on your first Pull Request and welcome to the Apache Airflow
community! If you have any issues or are unsure about any anything please check
our
mik-laj commented on a change in pull request #10488:
URL: https://github.com/apache/airflow/pull/10488#discussion_r476951974
##
File path: scripts/ci/docker-compose/integration-kerberos.yml
##
@@ -17,16 +17,55 @@
---
version: "2.2"
services:
- kerberos:
-image:
mik-laj commented on a change in pull request #10488:
URL: https://github.com/apache/airflow/pull/10488#discussion_r476946689
##
File path: airflow/providers/presto/hooks/presto.py
##
@@ -44,9 +59,28 @@ class PrestoHook(DbApiHook):
def get_conn(self):
"""Returns
mik-laj commented on pull request #10488:
URL: https://github.com/apache/airflow/pull/10488#issuecomment-680404365
@jaketf @potiuk It is ready to be reviewed if you could find the time.
This is an automated message from the
mik-laj commented on a change in pull request #10572:
URL: https://github.com/apache/airflow/pull/10572#discussion_r476940848
##
File path: airflow/api_connexion/openapi/v1.yaml
##
@@ -160,6 +160,14 @@ info:
For details on configuring the authentication, see
[API
This is an automated email from the ASF dual-hosted git repository.
kamilbregula pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/airflow.git.
from 7c1855f Add License to INTHEWILD.md (#10570)
add 855d495 Move more operators/hooks/sensors to
mik-laj commented on a change in pull request #10572:
URL: https://github.com/apache/airflow/pull/10572#discussion_r476931411
##
File path: airflow/api_connexion/openapi/v1.yaml
##
@@ -160,6 +160,14 @@ info:
For details on configuring the authentication, see
[API
This is an automated email from the ASF dual-hosted git repository.
kamilbregula pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/airflow.git.
from 7c1855f Add License to INTHEWILD.md (#10570)
add 855d495 Move more operators/hooks/sensors to
mik-laj merged pull request #10567:
URL: https://github.com/apache/airflow/pull/10567
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 to go
ephraimbuddy opened a new pull request #10572:
URL: https://github.com/apache/airflow/pull/10572
This PR adds information about update_mask to the REST API reference
documentation
---
**^ Add meaningful description above**
Read the **[Pull Request
mik-laj commented on issue #10469:
URL: https://github.com/apache/airflow/issues/10469#issuecomment-680351929
@jhtimmins Can I help with this ticket? If you have a blocker, let me know
and we'll try to find a solution together.
mik-laj edited a comment on issue #10560:
URL: https://github.com/apache/airflow/issues/10560#issuecomment-680325218
This here is a more complicated matter as some packages are deprecated and
some will not.
**airflow.api.auth**
it's not deprecated. We use it in Stable API also.
mik-laj commented on issue #10560:
URL: https://github.com/apache/airflow/issues/10560#issuecomment-680325218
This here is a more complicated matter as some packages are deprecated and
some will not.
**airflow.api.auth**
it's not deprecated. We use it in Stable API also.
ryw commented on issue #10560:
URL: https://github.com/apache/airflow/issues/10560#issuecomment-680320321
i was referencing source code structure - sorry that wasn't clear
This is an automated message from the Apache Git
mik-laj commented on issue #10560:
URL: https://github.com/apache/airflow/issues/10560#issuecomment-680319524
Have you been talking about package structure or URLs? This is not clear
from your ticket.
This is an automated
mik-laj commented on issue #10569:
URL: https://github.com/apache/airflow/issues/10569#issuecomment-680318723
What is your value for `base_url` option in `webserver` section ?
This is an automated message from the Apache Git
ryw commented on issue #10560:
URL: https://github.com/apache/airflow/issues/10560#issuecomment-680318201
Ok makes sense - didn't know how routes get rendered, but if it's reading
from file structure then I'm ok to keep it as it is for now. Closing issue.
ryw closed issue #10560:
URL: https://github.com/apache/airflow/issues/10560
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 to go to the
mik-laj commented on issue #10560:
URL: https://github.com/apache/airflow/issues/10560#issuecomment-680317639
@ryw Moving the API to a different path is a change that breaks all existing
integrations. Most of them have a fixed path, and it's not easy to change.
Some users won't even be
ephraimbuddy commented on pull request #10535:
URL: https://github.com/apache/airflow/pull/10535#issuecomment-680314223
That's awesome. Currently resolving a lot of conflicts in the other PR about
unifying API errors but I have pushed the update mask change, haven't created
the PR yet.
mik-laj commented on pull request #10535:
URL: https://github.com/apache/airflow/pull/10535#issuecomment-680311249
@ephraimbuddy I added introduction to API, so your change will be much
easier now.
This is an automated
paavanb opened a new issue #10571:
URL: https://github.com/apache/airflow/issues/10571
**Description**
`TaskInstance.xcom_pull` uses `XCom.get_many` in order to return multiple
XComs for the given parameters. However, it proceeds to only return the first
result if `task_ids` is not
boring-cyborg[bot] commented on issue #10571:
URL: https://github.com/apache/airflow/issues/10571#issuecomment-680308153
Thanks for opening your first issue here! Be sure to follow the issue
template!
This is an
mik-laj commented on pull request #10566:
URL: https://github.com/apache/airflow/pull/10566#issuecomment-680307447
We may think about it later. This change now needs an entry in UPDATING.md
anyway.
This is an automated
OmairK commented on pull request #9651:
URL: https://github.com/apache/airflow/pull/9651#issuecomment-680302053
> @OmairK Can you do a rebase and leave a comment so we get notifications?
Without it, we don't know that you've found the time to respond to our request.
@mik-laj I did
potiuk commented on pull request #10559:
URL: https://github.com/apache/airflow/pull/10559#issuecomment-680301183
I think it shoudl be ratrher nice now :)
This is an automated message from the Apache Git Service.
To respond
mik-laj commented on a change in pull request #10482:
URL: https://github.com/apache/airflow/pull/10482#discussion_r476797383
##
File path: docs/howto/connection/salesforce.rst
##
@@ -0,0 +1,73 @@
+ .. Licensed to the Apache Software Foundation (ASF) under one
+or more
potiuk commented on pull request #10566:
URL: https://github.com/apache/airflow/pull/10566#issuecomment-680299004
But If you want to remove it - feel free :)
This is an automated message from the Apache Git Service.
To
potiuk commented on pull request #10566:
URL: https://github.com/apache/airflow/pull/10566#issuecomment-680298754
> I wonder if it would be better to delete it. This was helpful when it was
easy available, but that leads to a lot of problems. The basic thing is that we
have to support
mik-laj commented on pull request #9651:
URL: https://github.com/apache/airflow/pull/9651#issuecomment-680297740
@OmairK Can you do a rebase and leave a comment so we get notifications?
Without it, we don't know that you've found the time to respond to our request.
This is an automated email from the ASF dual-hosted git repository.
potiuk pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/master by this push:
new 7c1855f Add License to INTHEWILD.md (#10570)
potiuk merged pull request #10570:
URL: https://github.com/apache/airflow/pull/10570
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 to go
kaxil opened a new pull request #10570:
URL: https://github.com/apache/airflow/pull/10570
Master is failing because the Markdown doesn't contain license
---
**^ Add meaningful description above**
Read the **[Pull Request
raulk89 commented on issue #10569:
URL: https://github.com/apache/airflow/issues/10569#issuecomment-680287641
Well this :443 is omitted from the URL ofcourse, since it is https default
port.
So URL is:
`https://hostname.domain.com/...`
raulk89 opened a new issue #10569:
URL: https://github.com/apache/airflow/issues/10569
**Apache Airflow version**: 1.10.11, 1.10.12
**Environment**: Virtual machine
- **OS** (e.g. from /etc/os-release): CentOS Linux release 7.8.2003 (Core)
- **Kernel** (e.g. `uname -a`):
mik-laj edited a comment on pull request #10333:
URL: https://github.com/apache/airflow/pull/10333#issuecomment-680286814
@houqp Can I ask for a review? If I remember correctly, you had a question
how do we want to implement it. Does it meet your expectations?
mik-laj commented on pull request #10333:
URL: https://github.com/apache/airflow/pull/10333#issuecomment-680286814
@houqp Can I aak for a review? If I remember correctly, you had a question
how do we want to implement it. Does it meet your expectations?
boring-cyborg[bot] commented on issue #10569:
URL: https://github.com/apache/airflow/issues/10569#issuecomment-680286955
Thanks for opening your first issue here! Be sure to follow the issue
template!
This is an
ephraimbuddy commented on pull request #10333:
URL: https://github.com/apache/airflow/pull/10333#issuecomment-680285503
Alright
This is an automated message from the Apache Git Service.
To respond to the message, please log
pingzh opened a new pull request #10568:
URL: https://github.com/apache/airflow/pull/10568
The value of annotations can store the raw dag_id, task_id and
execution_date so that k8s executor can easily map pod event back
to the task instance
---
**^ Add meaningful
mik-laj commented on pull request #10566:
URL: https://github.com/apache/airflow/pull/10566#issuecomment-680284178
I wonder if it would be better to delete it. This was helpful when it was
easy available, but that leads to a lot of problems. The basic thing is that
we have to support
eladkal commented on pull request #2460:
URL: https://github.com/apache/airflow/pull/2460#issuecomment-680283902
I wonder if this can be conflicted / redundant with the idea of
https://github.com/apache/airflow/pull/5787
If user choose `start` then execution_date = "start date" thus
[
https://issues.apache.org/jira/browse/AIRFLOW-1424?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17184748#comment-17184748
]
ASF GitHub Bot commented on AIRFLOW-1424:
-
eladkal commented on pull request #2460:
URL:
potiuk commented on a change in pull request #10559:
URL: https://github.com/apache/airflow/pull/10559#discussion_r476761961
##
File path: airflow/hooks/base_hook.py
##
@@ -20,7 +20,8 @@
import random
from typing import Any, List
-from airflow import secrets
+import
potiuk commented on a change in pull request #10559:
URL: https://github.com/apache/airflow/pull/10559#discussion_r476761637
##
File path: airflow/hooks/base_hook.py
##
@@ -20,7 +20,8 @@
import random
from typing import Any, List
-from airflow import secrets
+import
mik-laj opened a new pull request #10567:
URL: https://github.com/apache/airflow/pull/10567
These operators are at the core of Airflow, so we should consider them
fundamental.
---
**^ Add meaningful description above**
Read the **[Pull Request
kaxil commented on pull request #10333:
URL: https://github.com/apache/airflow/pull/10333#issuecomment-680279942
Can you rebase on latest master please @ephraimbuddy
This is an automated message from the Apache Git Service.
potiuk opened a new pull request #10566:
URL: https://github.com/apache/airflow/pull/10566
---
**^ Add meaningful description above**
Read the **[Pull Request
Guidelines](https://github.com/apache/airflow/blob/master/CONTRIBUTING.rst#pull-request-guidelines)**
for more
kaxil commented on a change in pull request #10559:
URL: https://github.com/apache/airflow/pull/10559#discussion_r476752027
##
File path: airflow/hooks/base_hook.py
##
@@ -20,7 +20,8 @@
import random
from typing import Any, List
-from airflow import secrets
+import
kaxil commented on a change in pull request #10559:
URL: https://github.com/apache/airflow/pull/10559#discussion_r476752232
##
File path: airflow/hooks/base_hook.py
##
@@ -20,7 +20,8 @@
import random
from typing import Any, List
-from airflow import secrets
+import
ryw commented on pull request #10513:
URL: https://github.com/apache/airflow/pull/10513#issuecomment-680278717
ok - backing down on this, closing this PR
hopefully we get Github discussions, though i do like the features of
[Discourse](https://www.discourse.org/) which we're using
ryw closed pull request #10513:
URL: https://github.com/apache/airflow/pull/10513
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 to go to
kaxil commented on a change in pull request #10559:
URL: https://github.com/apache/airflow/pull/10559#discussion_r476724420
##
File path: airflow/hooks/base_hook.py
##
@@ -20,7 +20,8 @@
import random
from typing import Any, List
-from airflow import secrets
+import
This is an automated email from the ASF dual-hosted git repository.
ryw pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/airflow.git.
from 51676b7 Migrate companies list to INTHEWILD.md (#10563)
add d84ec03 Add content to file (forgot to save before
This is an automated email from the ASF dual-hosted git repository.
ryw pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/airflow.git.
from 51676b7 Migrate companies list to INTHEWILD.md (#10563)
add d84ec03 Add content to file (forgot to save before
sztamas commented on pull request #10557:
URL: https://github.com/apache/airflow/pull/10557#issuecomment-680277876
Thanks for fixing this so quickly!
Tamas
This is an automated message from the Apache Git Service.
ryw commented on issue #10560:
URL: https://github.com/apache/airflow/issues/10560#issuecomment-680277895
I'm not suggesting we remove the old api, just give it a different code path
- and give /api directory to the new API -- took me a minute today to figure
out which one was the new API
This is an automated email from the ASF dual-hosted git repository.
ryw pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/airflow.git.
from 51676b7 Migrate companies list to INTHEWILD.md (#10563)
add d84ec03 Add content to file (forgot to save before
ryw merged pull request #10565:
URL: https://github.com/apache/airflow/pull/10565
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 to go to
ryw opened a new pull request #10565:
URL: https://github.com/apache/airflow/pull/10565
Forgot to save INTHEWILD.md before commit, got merged too quickly :)
This is an automated message from the Apache Git Service.
To
kaxil commented on pull request #10562:
URL: https://github.com/apache/airflow/pull/10562#issuecomment-680275469
Please also change the PR title to be more concise
This is an automated message from the Apache Git Service.
To
dimberman opened a new pull request #10564:
URL: https://github.com/apache/airflow/pull/10564
Reverts apache/airflow#10563
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
dimberman closed pull request #10564:
URL: https://github.com/apache/airflow/pull/10564
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 to
This is an automated email from the ASF dual-hosted git repository.
dimberman pushed a change to branch revert-10563-inthewild
in repository https://gitbox.apache.org/repos/asf/airflow.git.
at 8ecca4c Revert "Migrate companies list to INTHEWILD.md (#10563)"
This branch includes the
This is an automated email from the ASF dual-hosted git repository.
dimberman pushed a commit to branch revert-10563-inthewild
in repository https://gitbox.apache.org/repos/asf/airflow.git
commit 8ecca4c79b842b851cf511ef945a78719fb41b99
Author: Daniel Imberman
AuthorDate: Tue Aug 25 14:14:53
dimberman merged pull request #10563:
URL: https://github.com/apache/airflow/pull/10563
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 to
This is an automated email from the ASF dual-hosted git repository.
dimberman pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/airflow.git.
from 417f8de Update configs added in 1.10.12 (#10561)
add 51676b7 Migrate companies list to INTHEWILD.md
ryw opened a new pull request #10563:
URL: https://github.com/apache/airflow/pull/10563
Moves company list from README.md to INTHEWILD.md, docs only commit
This is an automated message from the Apache Git Service.
To respond
This is an automated email from the ASF dual-hosted git repository.
kaxilnaik pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/airflow.git.
from a0e7c40 Port isort config from pre-commit to setup.cfg (#10557)
add 417f8de Update configs added in
potiuk commented on issue #10560:
URL: https://github.com/apache/airflow/issues/10560#issuecomment-680271712
I actually proposed to deprecate the old API in 1.10.12 but the majority
vote was that we should not deprecate something unless we provide a
replacement - which makes perfect
sanjana1604 opened a new pull request #10562:
URL: https://github.com/apache/airflow/pull/10562
Issue: #10363
Updated the Official Docker Images in README.md file. The updated content is
mostly done for the users rather than developers of images.
I've added all the points
This is an automated email from the ASF dual-hosted git repository.
kaxilnaik pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/airflow.git.
from a0e7c40 Port isort config from pre-commit to setup.cfg (#10557)
add 417f8de Update configs added in
boring-cyborg[bot] commented on pull request #10562:
URL: https://github.com/apache/airflow/pull/10562#issuecomment-680271805
Congratulations on your first Pull Request and welcome to the Apache Airflow
community! If you have any issues or are unsure about any anything please check
our
This is an automated email from the ASF dual-hosted git repository.
kaxilnaik pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/airflow.git.
from cbf3603 Add Airflow 1.10.12 Changelog & Updating guide (#10558)
add a0e7c40 Port isort config from
kaxil merged pull request #10561:
URL: https://github.com/apache/airflow/pull/10561
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 to go to
This is an automated email from the ASF dual-hosted git repository.
kaxilnaik pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/airflow.git.
from cbf3603 Add Airflow 1.10.12 Changelog & Updating guide (#10558)
add a0e7c40 Port isort config from
kaxil commented on pull request #10557:
URL: https://github.com/apache/airflow/pull/10557#issuecomment-680270474
> That's fine as well. Though I'd keep the one in pre-commit as it then will
not even trigger if you change any of those files.
I agree on that one, it was mostly for
kaxil merged pull request #10557:
URL: https://github.com/apache/airflow/pull/10557
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 to go to
potiuk commented on issue #10560:
URL: https://github.com/apache/airflow/issues/10560#issuecomment-680270653
It's about our deprecation rules. If it's not absolutely necessary the rule
is that we have at least one release when we deprecate something before we
remove.
potiuk commented on pull request #10559:
URL: https://github.com/apache/airflow/pull/10559#issuecomment-680268355
I think this way it is even nicer.
This is an automated message from the Apache Git Service.
To respond to the
potiuk commented on a change in pull request #10559:
URL: https://github.com/apache/airflow/pull/10559#discussion_r476734337
##
File path: airflow/models/connection.py
##
@@ -380,3 +381,17 @@ def extra_dejson(self) -> Dict:
self.log.error("Failed parsing the
pingzh commented on pull request #10481:
URL: https://github.com/apache/airflow/pull/10481#issuecomment-680266639
> Hi ping, going forward we're actually going to be deleting all of the
k8sexecutor airflow.cfg configs involved in pod creation. This is because as of
1.10.12 we allow users
potiuk commented on a change in pull request #10559:
URL: https://github.com/apache/airflow/pull/10559#discussion_r476732162
##
File path: airflow/models/connection.py
##
@@ -380,3 +381,17 @@ def extra_dejson(self) -> Dict:
self.log.error("Failed parsing the
kaxil opened a new pull request #10561:
URL: https://github.com/apache/airflow/pull/10561
This PR adds version identified for configs added in 1.10.12
---
**^ Add meaningful description above**
Read the **[Pull Request
potiuk commented on a change in pull request #10559:
URL: https://github.com/apache/airflow/pull/10559#discussion_r476730492
##
File path: airflow/models/connection.py
##
@@ -380,3 +381,17 @@ def extra_dejson(self) -> Dict:
self.log.error("Failed parsing the
This is an automated email from the ASF dual-hosted git repository.
kaxilnaik pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/airflow.git.
from cd1f794 Bring back some inclusions before we solve cyclic deps
problems (#10551)
add cbf3603 Add Airflow
This is an automated email from the ASF dual-hosted git repository.
kaxilnaik pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/airflow.git.
from cd1f794 Bring back some inclusions before we solve cyclic deps
problems (#10551)
add cbf3603 Add Airflow
kaxil merged pull request #10558:
URL: https://github.com/apache/airflow/pull/10558
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 to go to
kaxil merged pull request #288:
URL: https://github.com/apache/airflow-site/pull/288
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 to go
sglickman commented on issue #7935:
URL: https://github.com/apache/airflow/issues/7935#issuecomment-680257021
This is happening to us also. No errors appear in the logs but the scheduler
will not create new pods, pipelines stall with tasks in 'queued' state, and the
scheduler pod must be
This is an automated email from the ASF dual-hosted git repository.
kaxilnaik pushed a change to branch docs-for-1.10.12-patch-1
in repository https://gitbox.apache.org/repos/asf/airflow-site.git.
discard 8abb5b0 Docs for 1.10.12 patch 1
omit 110c8e6 Docs for 1.10.12
add c8cb826
mik-laj commented on a change in pull request #10559:
URL: https://github.com/apache/airflow/pull/10559#discussion_r476720004
##
File path: airflow/models/connection.py
##
@@ -380,3 +381,17 @@ def extra_dejson(self) -> Dict:
self.log.error("Failed parsing the
mik-laj commented on a change in pull request #10559:
URL: https://github.com/apache/airflow/pull/10559#discussion_r476720110
##
File path: airflow/models/variable.py
##
@@ -181,3 +181,17 @@ def rotate_fernet_key(self):
fernet = get_fernet()
if self._val and
kaxil opened a new pull request #288:
URL: https://github.com/apache/airflow-site/pull/288
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
1 - 100 of 272 matches
Mail list logo