mik-laj commented on pull request #430:
URL: https://github.com/apache/airflow-site/pull/430#issuecomment-855321437
For some reason, we're still marked yellow.
https://infra-reports.apache.org/site-source/
--
This is an automated message from the Apache Git Service.
To respond to the
mik-laj commented on pull request #8009:
URL: https://github.com/apache/airflow/pull/8009#issuecomment-855320510
We already have ``airflow kubernetes airflow kubernetes generate-dag-yaml``
command. Closing...
--
This is an automated message from the Apache Git Service.
To respond to the
mik-laj closed pull request #8009:
URL: https://github.com/apache/airflow/pull/8009
--
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 specific comment.
For queries about this service,
mik-laj edited a comment on issue #14938:
URL: https://github.com/apache/airflow/issues/14938#issuecomment-855319387
Hi.
We added an improvement to `docker-compose.yaml` file that should improve
reliability and stability. Can you check it now?
mik-laj commented on issue #14938:
URL: https://github.com/apache/airflow/issues/14938#issuecomment-855319387
Hi.
We added an improvement to `docker-compose.yaml` that should improve
reliability and stability. Can you check it now?
https://github.com/apache/airflow/pull/16180
This is an automated email from the ASF dual-hosted git repository.
github-bot pushed a commit to branch gh-pages
in repository https://gitbox.apache.org/repos/asf/airflow-site.git
The following commit(s) were added to refs/heads/gh-pages by this push:
new e76fbac Deploying to gh-pages
mik-laj merged pull request #430:
URL: https://github.com/apache/airflow-site/pull/430
--
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 specific comment.
For queries about this service,
An error occurred while running github feature in .asf.yaml!:
[GitHub] Request error with message: "Branch not found". (status code: 404)
This is an automated email from the ASF dual-hosted git repository.
kamilbregula pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow-site.git
The following commit(s) were added to refs/heads/main by this push:
new 8cf8c85 Switch publishing process from
This is an automated email from the ASF dual-hosted git repository.
kamilbregula pushed a change to branch gh-pages
in repository https://gitbox.apache.org/repos/asf/airflow-site.git.
from fac8fc4 Deploying to asf-site from @
45db8c0f01f562008bafea2955f678b8e88cdf8a
add 58bc295
This is an automated email from the ASF dual-hosted git repository.
kamilbregula pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/main by this push:
new ef83f73 feat: Helm chart adding
boring-cyborg[bot] commented on pull request #16262:
URL: https://github.com/apache/airflow/pull/16262#issuecomment-855310443
Awesome work, congrats on your first merged pull request!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log
mik-laj merged pull request #16262:
URL: https://github.com/apache/airflow/pull/16262
--
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 specific comment.
For queries about this service,
mik-laj closed issue #16256:
URL: https://github.com/apache/airflow/issues/16256
--
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 specific comment.
For queries about this service, please
lewismc commented on pull request #16230:
URL: https://github.com/apache/airflow/pull/16230#issuecomment-855299943
Does anyone know what this means?
```
blackFailed
- hook id: black
- files were
potiuk edited a comment on issue #16148:
URL: https://github.com/apache/airflow/issues/16148#issuecomment-855291949
I am not sure _repr is a good idea.
If anything, then I guess we should use str() output rather than __str or
__repr. I think it is more important what str() does
potiuk commented on issue #16148:
URL: https://github.com/apache/airflow/issues/16148#issuecomment-855291949
I am not sure _repr is a good idea.
If anything, then I guess we should use str() output rather than __str or
__repr. I think it is more important what str() does (which is
mik-laj edited a comment on issue #16148:
URL: https://github.com/apache/airflow/issues/16148#issuecomment-855283667
Should we mask the elements? What do you think about using `repr` function
and mask text representation of object?
--
This is an automated message from the Apache Git
uranusjr commented on issue #16148:
URL: https://github.com/apache/airflow/issues/16148#issuecomment-855285884
Oh that's a good idea. An iterator's repr is extremely unlikely to consume
it (and I'd argue it's a bug if it does).
--
This is an automated message from the Apache Git
mik-laj edited a comment on issue #16148:
URL: https://github.com/apache/airflow/issues/16148#issuecomment-855283667
Should we mask the elements? What do you think about using repr method and
mask text representation of object?
--
This is an automated message from the Apache Git
mik-laj commented on issue #16148:
URL: https://github.com/apache/airflow/issues/16148#issuecomment-855283667
Should we mask the elements? What do you think about using repr method and
mask the result?
--
This is an automated message from the Apache Git Service.
To respond to the
mrbaguvix commented on a change in pull request #15852:
URL: https://github.com/apache/airflow/pull/15852#discussion_r632651990
##
File path: airflow/www/static/js/graph.js
##
@@ -155,8 +156,8 @@ function draw() {
if (nodeId in taskInstances) tryNumber =
uranusjr commented on issue #16148:
URL: https://github.com/apache/airflow/issues/16148#issuecomment-855281405
> we should remove the "iterable" handling and just limit it to cases where
those are explicit 'lists' of objects rather than Iterable objects ?
Alternatively we can only
potiuk commented on issue #12920:
URL: https://github.com/apache/airflow/issues/12920#issuecomment-855274019
I think if someone wants to contribute, it would be great. Maybe even
grafana provider could be developed by Grafana Labs? We have a number of
providers developed and controlled by
github-actions[bot] commented on pull request #16242:
URL: https://github.com/apache/airflow/pull/16242#issuecomment-855273553
The PR is likely OK to be merged with just subset of tests for default
Python and Database versions without running the full matrix of tests, because
it does not
This is an automated email from the ASF dual-hosted git repository.
potiuk pushed a change to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git.
from 363477f Don't run migration for adhoc command (#16255)
add 7744f05 Improve argument handling in
potiuk merged pull request #16258:
URL: https://github.com/apache/airflow/pull/16258
--
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 specific comment.
For queries about this service,
potiuk closed issue #16252:
URL: https://github.com/apache/airflow/issues/16252
--
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 specific comment.
For queries about this service, please
mik-laj commented on a change in pull request #16241:
URL: https://github.com/apache/airflow/pull/16241#discussion_r646015111
##
File path: tests/providers/amazon/aws/transfers/test_s3_to_redshift.py
##
@@ -65,6 +65,45 @@ def test_execute(self, mock_run, mock_session):
potiuk commented on issue #16148:
URL: https://github.com/apache/airflow/issues/16148#issuecomment-855266414
I think adding lists and primitive strings/sets/ tuples /lista dicts
recurrently should be quite safe
--
This is an automated message from the Apache Git Service.
To respond to
potiuk edited a comment on issue #16025:
URL: https://github.com/apache/airflow/issues/16025#issuecomment-855265599
Can you copy the SIGTERM message ?.
--
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
potiuk commented on issue #16025:
URL: https://github.com/apache/airflow/issues/16025#issuecomment-855265599
Can you copy the soften message ?.
--
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
sunki-hong commented on a change in pull request #16241:
URL: https://github.com/apache/airflow/pull/16241#discussion_r646005770
##
File path: tests/providers/amazon/aws/transfers/test_s3_to_redshift.py
##
@@ -65,6 +65,45 @@ def test_execute(self, mock_run, mock_session):
Danirill commented on issue #15799:
URL: https://github.com/apache/airflow/issues/15799#issuecomment-855257409
Hello, i don't have airflow, but got the same error. There is my django app
log:
```
[2021-06-05 15:30:08,313: DEBUG/MainProcess] Channel open
[2021-06-05 15:30:08,314:
maxcountryman commented on issue #16025:
URL: https://github.com/apache/airflow/issues/16025#issuecomment-855250292
@potiuk Yes, understood--unfortunately there's very little additional
context to share. There are three errors which all seem to be related to each
other. One mentions the
ashb commented on issue #16148:
URL: https://github.com/apache/airflow/issues/16148#issuecomment-855249587
Yeah, looking at Iterables was a mistake since many are non-rewindable or
have side effects such as this.
I was trying to protect against secrets in object representations and
potiuk commented on issue #16025:
URL: https://github.com/apache/airflow/issues/16025#issuecomment-855249332
Yeah. I understand that - but I would expect to have some logs generated by
the celery failures. They might be either in the logs of airflow - explaining
what happened, or in case
maxcountryman commented on issue #16025:
URL: https://github.com/apache/airflow/issues/16025#issuecomment-855247661
@potiuk An important correction: the containers are not failing. It's the
Celery commands that seem to be falling over for some unknown reason (some
messages claim memory
XD-DENG commented on a change in pull request #16264:
URL: https://github.com/apache/airflow/pull/16264#discussion_r645996867
##
File path: airflow/providers/amazon/aws/log/s3_task_handler.py
##
@@ -93,7 +94,10 @@ def close(self):
# read log and remove old logs to
maxcountryman commented on issue #16025:
URL: https://github.com/apache/airflow/issues/16025#issuecomment-855247456
@potiuk Container Insight metrics look well below any resource starvation,
looking back several weeks.
I'm not seeing anything too obvious in the logs, but will
potiuk commented on issue #16025:
URL: https://github.com/apache/airflow/issues/16025#issuecomment-855246004
> @potiuk Container insights are already enabled so happy to go grab these
and share.
>
> Is there any other info that would be helpful?
Not really - I think if you
potiuk closed issue #16254:
URL: https://github.com/apache/airflow/issues/16254
--
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 specific comment.
For queries about this service, please
potiuk commented on issue #16254:
URL: https://github.com/apache/airflow/issues/16254#issuecomment-855245620
Closing for now unless it does not solve your problem (please let us know if
it did/did not).
--
This is an automated message from the Apache Git Service.
To respond to the
potiuk commented on issue #16254:
URL: https://github.com/apache/airflow/issues/16254#issuecomment-855245562
When using the UI, you should not configure extras but you can now place the
values in the right fields - instead of manually adding "extras" via JSON dict:
You have those
maxcountryman commented on issue #16025:
URL: https://github.com/apache/airflow/issues/16025#issuecomment-855244383
@potiuk Container insights are already enabled so happy to go grab these and
share.
Is there any other info that would be helpful?
--
This is an automated message
potiuk commented on issue #16025:
URL: https://github.com/apache/airflow/issues/16025#issuecomment-855243894
@maxcountryman (and possibly @7yl4r ) - it's next to impossible to help
without more information.
I think what would be extremely helpful to investigate that one is to
github-actions[bot] commented on pull request #16257:
URL: https://github.com/apache/airflow/pull/16257#issuecomment-855243005
The PR is likely ready to be merged. No tests are needed as no important
environment files, nor python files were modified by it. However, committers
might decide
potiuk commented on pull request #16257:
URL: https://github.com/apache/airflow/pull/16257#issuecomment-855242979
Close/Reopen to rebuild
--
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
potiuk closed pull request #16257:
URL: https://github.com/apache/airflow/pull/16257
--
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 specific comment.
For queries about this service,
potiuk commented on a change in pull request #15599:
URL: https://github.com/apache/airflow/pull/15599#discussion_r645992078
##
File path: airflow/utils/log/secrets_masker.py
##
@@ -0,0 +1,221 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more
potiuk edited a comment on issue #16148:
URL: https://github.com/apache/airflow/issues/16148#issuecomment-855240855
Thanks for being so persistent @ConstantinoSchillebeeckx . Great
investigation! It's a very subtle problem it seems.
I believe @mik-laj is right. We seem to have
potiuk edited a comment on issue #16148:
URL: https://github.com/apache/airflow/issues/16148#issuecomment-855240855
Thanks for being so persistent @ConstantinoSchillebeeckx . Great
investigation! It's a very subtle problem it seems.
I believe @mik-laj is right. We seem to have
potiuk commented on issue #16148:
URL: https://github.com/apache/airflow/issues/16148#issuecomment-855240855
Thanks for being so persistent @ConstantinoSchillebeeckx . Great
investigation! It's a very subtle problem it seems.
I believe @mik-laj is right. We seem to have indeed a
zacharya19 commented on issue #14529:
URL: https://github.com/apache/airflow/issues/14529#issuecomment-855231044
> hey @zacharya19 have you made any progress on this? I'd be willing to help
too.
Sadly, no, I've been way too busy lately.
Feel free to take it off my hands, I would
potiuk edited a comment on issue #16271:
URL: https://github.com/apache/airflow/issues/16271#issuecomment-855229979
Nice analysis. @coolacid Would you be willing to submit PRs fixing the
"communication" broken (seems you have the right fix) ?
I am not sure if you already got the right
potiuk commented on issue #16271:
URL: https://github.com/apache/airflow/issues/16271#issuecomment-855229979
Nice analysis. Would you be willing to submit PRs fixing the "communication"
broken (seems you have the right fix) ?
I am not sure if you already got the right fix for the
potiuk commented on a change in pull request #16268:
URL: https://github.com/apache/airflow/pull/16268#discussion_r645982026
##
File path: tests/jobs/test_scheduler_job.py
##
@@ -3921,7 +3921,7 @@ def
test_do_schedule_max_active_runs_upstream_failed(self):
potiuk commented on a change in pull request #16268:
URL: https://github.com/apache/airflow/pull/16268#discussion_r645981961
##
File path: UPDATING.md
##
@@ -409,7 +409,7 @@ The following table shows changes in import paths.
potiuk commented on a change in pull request #16268:
URL: https://github.com/apache/airflow/pull/16268#discussion_r645981961
##
File path: UPDATING.md
##
@@ -409,7 +409,7 @@ The following table shows changes in import paths.
potiuk commented on pull request #13514:
URL: https://github.com/apache/airflow/pull/13514#issuecomment-855228055
About prior art - not that I am aware of it but also Apache beam started
using it after the discussion
https://github.com/apache/beam/blob/master/.gitmodules
--
This is an
ashb commented on pull request #13514:
URL: https://github.com/apache/airflow/pull/13514#issuecomment-855227987
@RA80533 this brought on a setting by Apache Software Foundation Infra and
security teams changed on our GitHub org - they only allow Actions from
official GitHub, or ASF repos.
potiuk edited a comment on pull request #13514:
URL: https://github.com/apache/airflow/pull/13514#issuecomment-855227731
> @ashb How did this idea come about? This is the only project I've come
across that partitions their third-party Actions in such a manner. Was there
any prior attempts
potiuk commented on pull request #13514:
URL: https://github.com/apache/airflow/pull/13514#issuecomment-855227731
> @ashb How did this idea come about? This is the only project I've come
across that partitions their third-party Actions in such a manner. Was there
any prior attempts at
This is an automated email from the ASF dual-hosted git repository.
potiuk pushed a change to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git.
from 0319c79 Run Kubernetes integration tests when Helm Chart is changed
(#16276)
add 363477f Don't run migration
potiuk merged pull request #16255:
URL: https://github.com/apache/airflow/pull/16255
--
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 specific comment.
For queries about this service,
potiuk closed pull request #16258:
URL: https://github.com/apache/airflow/pull/16258
--
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 specific comment.
For queries about this service,
potiuk commented on pull request #16258:
URL: https://github.com/apache/airflow/pull/16258#issuecomment-855226834
Close/reopen to rebuild
--
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
github-actions[bot] commented on pull request #16258:
URL: https://github.com/apache/airflow/pull/16258#issuecomment-855226825
The PR most likely needs to run full matrix of tests because it modifies
parts of the core of Airflow. However, committers might decide to merge it
quickly and
RA80533 edited a comment on pull request #13514:
URL: https://github.com/apache/airflow/pull/13514#issuecomment-855226302
@ashb How did this idea come about? This is the only project I've come
across that partitions their third-party Actions in such a manner. Was there
any prior attempts
RA80533 commented on pull request #13514:
URL: https://github.com/apache/airflow/pull/13514#issuecomment-855226302
@ashb What prompted this from being introduced? This is the only project
I've come across that partitions their third-party Actions in such a manner.
Was there any prior
XD-DENG commented on pull request #16276:
URL: https://github.com/apache/airflow/pull/16276#issuecomment-855218815
Cool. Thanks @potiuk
--
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
potiuk commented on pull request #16276:
URL: https://github.com/apache/airflow/pull/16276#issuecomment-855218720
@XD-DENG -> I updated the subject during merge :)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
This is an automated email from the ASF dual-hosted git repository.
potiuk pushed a change to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git.
from 6736290 Update target branch in `boring-cyborg` for `checkUpToDate`
(#16274)
add 0319c79 Run Kubernetes
potiuk merged pull request #16276:
URL: https://github.com/apache/airflow/pull/16276
--
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 specific comment.
For queries about this service,
github-actions[bot] commented on pull request #16276:
URL: https://github.com/apache/airflow/pull/16276#issuecomment-855218678
The PR most likely needs to run full matrix of tests because it modifies
parts of the core of Airflow. However, committers might decide to merge it
quickly and
ashb commented on a change in pull request #16030:
URL: https://github.com/apache/airflow/pull/16030#discussion_r645969935
##
File path: airflow/cli/commands/task_command.py
##
@@ -42,9 +45,26 @@
get_dags,
suppress_logs_and_warning,
)
+from airflow.utils.dates
jjlyr commented on issue #16249:
URL: https://github.com/apache/airflow/issues/16249#issuecomment-855213396
@bbovenzi I had though in something like this, if we want to keep DAG
Dependencies as it is, create a new view where we need to select the Dags we
want to monitor(up to a limit) via
This is an automated email from the ASF dual-hosted git repository.
kaxilnaik pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/main by this push:
new 6736290 Update target branch in
kaxil merged pull request #16274:
URL: https://github.com/apache/airflow/pull/16274
--
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 specific comment.
For queries about this service,
kaxil merged pull request #16275:
URL: https://github.com/apache/airflow/pull/16275
--
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 specific comment.
For queries about this service,
This is an automated email from the ASF dual-hosted git repository.
kaxilnaik pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/main by this push:
new 3bdcd1a Docs: Fix url for ``Elasticsearch``
XD-DENG commented on pull request #16272:
URL: https://github.com/apache/airflow/pull/16272#issuecomment-855206601
Welcome to the community :)
--
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.
xddeng pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/main by this push:
new c046c49 Add Clicksign to INTHEWILD.md (#16272)
github-actions[bot] commented on pull request #16272:
URL: https://github.com/apache/airflow/pull/16272#issuecomment-855206506
The PR is likely ready to be merged. No tests are needed as no important
environment files, nor python files were modified by it. However, committers
might decide
boring-cyborg[bot] commented on pull request #16272:
URL: https://github.com/apache/airflow/pull/16272#issuecomment-855206504
Awesome work, congrats on your first merged pull request!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log
XD-DENG merged pull request #16272:
URL: https://github.com/apache/airflow/pull/16272
--
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 specific comment.
For queries about this service,
github-actions[bot] commented on pull request #16275:
URL: https://github.com/apache/airflow/pull/16275#issuecomment-855203620
The PR is likely OK to be merged with just subset of tests for default
Python and Database versions without running the full matrix of tests, because
it does not
XD-DENG commented on pull request #16276:
URL: https://github.com/apache/airflow/pull/16276#issuecomment-855203428
nit: The PR subject itself leads to think it's relating to section
`tests-helm` instead, while it's actually `tests-kubernetes`.
--
This is an automated message from the
This is an automated email from the ASF dual-hosted git repository.
xddeng pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/main by this push:
new c4e9c01 Fix Helm git sync secrets typo (#16278)
XD-DENG merged pull request #16278:
URL: https://github.com/apache/airflow/pull/16278
--
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 specific comment.
For queries about this service,
90 matches
Mail list logo