github-actions[bot] commented on pull request #16100:
URL: https://github.com/apache/airflow/pull/16100#issuecomment-856424036
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
msumit commented on pull request #16233:
URL: https://github.com/apache/airflow/pull/16233#issuecomment-856404610
Thanks @kaxil
--
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
jhtimmins opened a new pull request #16317:
URL: https://github.com/apache/airflow/pull/16317
Modifies `_get_all_non_dag_permissionviews` to
`_get_all_non_dag_permissions` so the updated permission names are used.
--
This is an automated message from the Apache Git Service.
To respond
mik-laj commented on pull request #14535:
URL: https://github.com/apache/airflow/pull/14535#issuecomment-856398136
It looks like a bug. I reopened this PR.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL
hbrls opened a new pull request #14535:
URL: https://github.com/apache/airflow/pull/14535
This may not be a ready PR. The code diff best expresses my ideas.
I'm initializing a mysql 5.7 db from empty using `hub.docker.com,
apache/airflow:2.0.1-python3.8`. The migration scripts throw
kevinbsilva commented on pull request #15947:
URL: https://github.com/apache/airflow/pull/15947#issuecomment-856397058
@ashb i've tested in our development cluster and it works fine. But it seems
a check from GitHub actions did not succeed. It raised a 404 when trying to
install the
hbrls commented on pull request #14535:
URL: https://github.com/apache/airflow/pull/14535#issuecomment-856386855
Is it closed? I've rebased the fixes.
--
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
uranusjr commented on issue #15994:
URL: https://github.com/apache/airflow/issues/15994#issuecomment-856366575
In my AIP-39 implementation I called a time table that never schedules
(`schedule_interval=None`) a `NullTimeTable` so that's another possibility.
It's probably a good idea to
github-actions[bot] closed pull request #14535:
URL: https://github.com/apache/airflow/pull/14535
--
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
github-actions[bot] commented on issue #14412:
URL: https://github.com/apache/airflow/issues/14412#issuecomment-856345568
This issue has been automatically marked as stale because it has been open
for 30 days with no response from the author. It will be closed in next 7 days
if no further
github-actions[bot] closed issue #15503:
URL: https://github.com/apache/airflow/issues/15503
--
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
github-actions[bot] closed pull request #13405:
URL: https://github.com/apache/airflow/pull/13405
--
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
github-actions[bot] commented on issue #15503:
URL: https://github.com/apache/airflow/issues/15503#issuecomment-856345543
This issue has been closed because it has not received response from the
issue author.
--
This is an automated message from the Apache Git Service.
To respond to the
ephraimbuddy commented on a change in pull request #16182:
URL: https://github.com/apache/airflow/pull/16182#discussion_r647021778
##
File path: airflow/jobs/scheduler_job.py
##
@@ -1417,6 +1420,37 @@ def _clean_tis_without_dagrun(self, session):
raise
boring-cyborg[bot] commented on issue #16316:
URL: https://github.com/apache/airflow/issues/16316#issuecomment-856341122
Thanks for opening your first issue here! Be sure to follow the issue
template!
--
This is an automated message from the Apache Git Service.
To respond to the
timgriffiths opened a new issue #16316:
URL: https://github.com/apache/airflow/issues/16316
**Description**
Some times we have more jobs than is able to be run in our k8s cluster, it
would be super nice if airflow had a feature that could catch the below error
and just re-queue the
mik-laj commented on pull request #433:
URL: https://github.com/apache/airflow-site/pull/433#issuecomment-856314618
This didn't help as our CI is pushing changes to the `gh-pages` branch. I
changed it last week, because it is required by asf infra.
> The ghp_branch setting can ONLY
This is an automated email from the ASF dual-hosted git repository.
kamilbregula pushed a change to branch asf-site
in repository https://gitbox.apache.org/repos/asf/airflow-site.git.
from 51287f4 Deploying to asf-site from @
2bce1a1f98c599e0c6abd4f0975020cd9561da18
add e76fbac
jedcunningham commented on issue #16298:
URL: https://github.com/apache/airflow/issues/16298#issuecomment-856312678
Looks like the scheduler is getting a sigterm signal. Any hints in the
events for the pod (you'll want a recent pod)?
```
kubectl get event --field-selector
jedcunningham commented on issue #16313:
URL: https://github.com/apache/airflow/issues/16313#issuecomment-856310140
If you exec into the webserver pod and run `echo
$AIRFLOW__CORE__FERNET_KEY`, does it match the "echo Fernet Key" command?
--
This is an automated message from the Apache
jedcunningham commented on issue #16313:
URL: https://github.com/apache/airflow/issues/16313#issuecomment-856309532
I'm not able to reproduce this. Do you get a base64 string when you run the
"echo Fernet Key" command `helm install` prints? e.g:
```
echo Fernet Key: $(kubectl get
jedcunningham opened a new pull request #16315:
URL: https://github.com/apache/airflow/pull/16315
Set default podAntiAffinity for the components that can scale, as in
most cases you'd want them spread across available nodes.
--
This is an automated message from the Apache Git Service.
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 3d082e0 Deploying to gh-pages
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 14ec0b9 Publish .asf.yaml on asf-site
mik-laj merged pull request #433:
URL: https://github.com/apache/airflow-site/pull/433
--
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)
cvsekhar commented on issue #16254:
URL: https://github.com/apache/airflow/issues/16254#issuecomment-856286918
Sure will test the RC
--
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
mik-laj opened a new pull request #433:
URL: https://github.com/apache/airflow-site/pull/433
Hello
ASF Publishing tools can't detect his file. According to [asf infra
report](https://infra-reports.apache.org/site-source/), we must have this file
on the `asf-site` branch.
>
SamWheating commented on a change in pull request #12822:
URL: https://github.com/apache/airflow/pull/12822#discussion_r646947662
##
File path: airflow/jobs/base_job.py
##
@@ -246,7 +246,6 @@ def run(self):
finally:
self.end_date =
malthe opened a new pull request #16314:
URL: https://github.com/apache/airflow/pull/16314
This adds support for specifying an SSH client host key along with its type,
e.g.
```
ssh-ed25519
C3NzaC1lZDI1NTE5IGTA+qK7VvFL3oZTgbVwFYcp5ZrPiRkPQd8YhVZDH946
```
potiuk commented on issue #16254:
URL: https://github.com/apache/airflow/issues/16254#issuecomment-856245357
OK. I see the problem @cvsekhar . It seems that the AzureFileServerHook was
wrongly defined and the way extras were used there was wrong. Passing extras
this way is not how airlfow
cvsekhar edited a comment on issue #16254:
URL: https://github.com/apache/airflow/issues/16254#issuecomment-856220634
I see from the commit list, this specific class has not be addressed and its
giving an error.
cvsekhar edited a comment on issue #16254:
URL: https://github.com/apache/airflow/issues/16254#issuecomment-856220634
I see from the commit list, this specific class has not be addressed and its
giving an error.
--
This is an automated message from the Apache Git Service.
To respond to
cvsekhar commented on issue #16254:
URL: https://github.com/apache/airflow/issues/16254#issuecomment-856220634
Can you help me as the wasb connection with those fields is not working for
fileshare.
--
This is an automated message from the Apache Git Service.
To respond to the message,
cvsekhar commented on issue #16254:
URL: https://github.com/apache/airflow/issues/16254#issuecomment-856219873
This has not addressed the issue, I was using those fields and getting an
error when I try to use this hook.
john-jac opened a new issue #14592:
URL: https://github.com/apache/airflow/issues/14592
**Apache Airflow version**:
1.10.12
**Kubernetes version (if you are using kubernetes)** (use `kubectl version`):
n/a
**Environment**:
- **Cloud provider or hardware
potiuk edited a comment on issue #14592:
URL: https://github.com/apache/airflow/issues/14592#issuecomment-856216348
Agree we have consistency issue here - Interestingly, the AWS secret manager
crashed originally for @subashcanapathy and @john-jac but did not crash for you
@fhoda. Not sure
potiuk commented on issue #14592:
URL: https://github.com/apache/airflow/issues/14592#issuecomment-856216348
Agree we have consistency issue here - Interestingly, the AWS secret manager
crashed originally for @subashcanapathy and @john-jac but did not crash for you
@fhoda. Not sure what
fhoda edited a comment on issue #14592:
URL: https://github.com/apache/airflow/issues/14592#issuecomment-856125981
As of now it seems the expected behavior is not what is happening and is
inconsistent across different secret backends.
I have tried to reproduce this issue with
jedcunningham commented on a change in pull request #16291:
URL: https://github.com/apache/airflow/pull/16291#discussion_r646891685
##
File path: chart/templates/jobs/migrate-database-job.yaml
##
@@ -38,6 +38,9 @@ metadata:
"helm.sh/hook": post-install,post-upgrade
Stormhand commented on issue #9722:
URL: https://github.com/apache/airflow/issues/9722#issuecomment-856184838
I have the same issue on 2.1.0. If i add the libs in PYTHONPATH i can import
them but the UI shows an error. Im using the community helm charts with
git-sync enabled.
--
This
boring-cyborg[bot] commented on issue #16313:
URL: https://github.com/apache/airflow/issues/16313#issuecomment-856182930
Thanks for opening your first issue here! Be sure to follow the issue
template!
--
This is an automated message from the Apache Git Service.
To respond to the
lewismc opened a new issue #16313:
URL: https://github.com/apache/airflow/issues/16313
**Apache Airflow version**: 2.1.0 (Helm chart) from main branch
5c7d758e24595c485553b0449583ff238114d47d
**Kubernetes version (if you are using kubernetes)** (use `kubectl version`):
SamWheating commented on pull request #14306:
URL: https://github.com/apache/airflow/pull/14306#issuecomment-856179712
Alright I've rebased on main and ran the tests locally - Happy to help
update this if we change the wording or some field names later on.
Could I get your reviews
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 5c7d758 Docs: Fix creating a connection docs (#16312)
add 29b7f79 fix wasb remote logging when blob already
potiuk closed issue #15907:
URL: https://github.com/apache/airflow/issues/15907
--
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 merged pull request #16280:
URL: https://github.com/apache/airflow/pull/16280
--
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,
XD-DENG commented on pull request #14306:
URL: https://github.com/apache/airflow/pull/14306#issuecomment-856148027
> Code looks fine, but I wonder if we want to rename this now to match the
proposed new external names as mentioned in #14459
Hi @ashb , this issue
ebdavison commented on issue #13853:
URL: https://github.com/apache/airflow/issues/13853#issuecomment-856147738
I am on airflow 2.1.0 and this is definitely an issue on that version.
Can someone comment on the status of this? I have had to update 400+ DAGs
to remove the
fhoda commented on issue #14592:
URL: https://github.com/apache/airflow/issues/14592#issuecomment-856125981
As of now it seems the expected behavior is not what is happening and is
inconsistent across different secret backends.
I have tried to reproduce this issue with Airflow 2.0
leahecole commented on pull request #16034:
URL: https://github.com/apache/airflow/pull/16034#issuecomment-856123635
> It looks like a breaking change. Can we avoid it and keep backward
compatibility with warning? This change also requires an entry in changelog to
facilitate migration.
kaxil commented on pull request #16233:
URL: https://github.com/apache/airflow/pull/16233#issuecomment-856116981
rebased your PR on latest Main, let's see if it solves some Build issues --
I know there was some bug few days back
--
This is an automated message from the Apache Git
This is an automated email from the ASF dual-hosted git repository.
kaxilnaik pushed a change to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git.
from 9c94b72 Updated documentation for June 2021 provider release (#16294)
add 5c7d758 Docs: Fix creating a
kaxil merged pull request #16312:
URL: https://github.com/apache/airflow/pull/16312
--
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,
lewismc commented on pull request #16230:
URL: https://github.com/apache/airflow/pull/16230#issuecomment-856110439
Thanks for explaining @kaxil I'll have another look at the black checks.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log
dnskr commented on a change in pull request #16291:
URL: https://github.com/apache/airflow/pull/16291#discussion_r646783130
##
File path: chart/templates/jobs/migrate-database-job.yaml
##
@@ -38,6 +38,9 @@ metadata:
"helm.sh/hook": post-install,post-upgrade
sunki-hong commented on a change in pull request #16241:
URL: https://github.com/apache/airflow/pull/16241#discussion_r646774062
##
File path: tests/providers/amazon/aws/transfers/test_s3_to_redshift.py
##
@@ -59,6 +59,61 @@ def test_execute(self, mock_run, mock_session):
SamWheating commented on issue #16023:
URL: https://github.com/apache/airflow/issues/16023#issuecomment-856087959
We were experiencing a similar issue, but it turned out to be due to the
SchedulerJob being marked as failed due to a slow heartbeat (due to adding a
few thousand DAGs).
Aakcht opened a new pull request #16312:
URL: https://github.com/apache/airflow/pull/16312
Minor fix for `Creating a Connection from the CLI` documentation
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL
msumit commented on pull request #16233:
URL: https://github.com/apache/airflow/pull/16233#issuecomment-856080760
@ashb @uranusjr @kaxil @potiuk can you guys plz review this.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub
fraciril closed issue #16307:
URL: https://github.com/apache/airflow/issues/16307
--
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
uranusjr commented on issue #16307:
URL: https://github.com/apache/airflow/issues/16307#issuecomment-856075261
Already fixed in #16018.
--
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
bbovenzi commented on issue #16249:
URL: https://github.com/apache/airflow/issues/16249#issuecomment-856070445
Got it.
I think this makes most sense as adding dependency info to the Graph view to
get all that detailed info per run. Dag Dependencies remains as an overview of
all the
kaxil commented on pull request #16230:
URL: https://github.com/apache/airflow/pull/16230#issuecomment-856059512
> Does anyone know what this means?
>
> ```
>
blackFailed
> - hook id: black
uranusjr commented on a change in pull request #16311:
URL: https://github.com/apache/airflow/pull/16311#discussion_r646723245
##
File path: airflow/utils/db.py
##
@@ -561,10 +571,21 @@ def create_default_connections(session=None):
)
-def initdb():
+@provide_session
uranusjr commented on issue #16310:
URL: https://github.com/apache/airflow/issues/16310#issuecomment-856055387
I’m curious though, why would you want to run two `init`s side by side?
There’s only one database, is it not?
--
This is an automated message from the Apache Git Service.
To
ashb commented on pull request #15947:
URL: https://github.com/apache/airflow/pull/15947#issuecomment-856053821
@kevinbsilva Does it work now? I _think_ this should, but could you confirm?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log
eladkal commented on issue #15914:
URL: https://github.com/apache/airflow/issues/15914#issuecomment-856037993
sorry, it was for @akshay654
--
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
anitakar opened a new pull request #16311:
URL: https://github.com/apache/airflow/pull/16311
Make sure that multiple db initialization processes on the same db are
executed one after another.
Tested manually by running airflow db init in 2 separate bash sessions.
Made sure that
dnskr commented on a change in pull request #16291:
URL: https://github.com/apache/airflow/pull/16291#discussion_r646682397
##
File path: chart/templates/jobs/migrate-database-job.yaml
##
@@ -38,6 +38,9 @@ metadata:
"helm.sh/hook": post-install,post-upgrade
anitakar commented on issue #16310:
URL: https://github.com/apache/airflow/issues/16310#issuecomment-856016574
About to submit PR for that bug.
--
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
anitakar opened a new issue #16310:
URL: https://github.com/apache/airflow/issues/16310
**Apache Airflow version**:
2.0.1
**Kubernetes version (if you are using kubernetes)** (use `kubectl version`):
Not applicable
**Environment**:
- **Cloud provider or hardware
sshepel commented on issue #15914:
URL: https://github.com/apache/airflow/issues/15914#issuecomment-856014709
@eladkal if the above statement was for me then I'm using `2.1.0` already...
and as I mentioned before the issue mentioned in ticket is just a warning right
now.
--
This is an
griseau commented on issue #15803:
URL: https://github.com/apache/airflow/issues/15803#issuecomment-856011406
@FabianF92 crystal clear !
I'll test that everything is working fine with 2.1.x and think about
migrating.
Thanks for your feedback
--
This is an automated message
kaxil commented on a change in pull request #16291:
URL: https://github.com/apache/airflow/pull/16291#discussion_r646665237
##
File path: chart/templates/jobs/migrate-database-job.yaml
##
@@ -38,6 +38,9 @@ metadata:
"helm.sh/hook": post-install,post-upgrade
jedcunningham commented on a change in pull request #16291:
URL: https://github.com/apache/airflow/pull/16291#discussion_r646660158
##
File path: chart/templates/jobs/migrate-database-job.yaml
##
@@ -38,6 +38,9 @@ metadata:
"helm.sh/hook": post-install,post-upgrade
jedcunningham commented on a change in pull request #16291:
URL: https://github.com/apache/airflow/pull/16291#discussion_r646655643
##
File path: chart/templates/jobs/migrate-database-job.yaml
##
@@ -38,6 +38,9 @@ metadata:
"helm.sh/hook": post-install,post-upgrade
github-actions[bot] commented on pull request #16305:
URL: https://github.com/apache/airflow/pull/16305#issuecomment-855997474
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
fraciril opened a new issue #16307:
URL: https://github.com/apache/airflow/issues/16307
**Apache Airflow version**:
v2.1.0
**Environment**:
- **Cloud provider or hardware configuration**: GCP
- **OS** (e.g. from /etc/os-release): Ubuntu 20.04
- **Server**: nginx
boring-cyborg[bot] commented on issue #16307:
URL: https://github.com/apache/airflow/issues/16307#issuecomment-855991275
Thanks for opening your first issue here! Be sure to follow the issue
template!
--
This is an automated message from the Apache Git Service.
To respond to the
mik-laj commented on a change in pull request #16241:
URL: https://github.com/apache/airflow/pull/16241#discussion_r646650620
##
File path: tests/providers/amazon/aws/transfers/test_s3_to_redshift.py
##
@@ -59,6 +59,61 @@ def test_execute(self, mock_run, mock_session):
sunki-hong commented on a change in pull request #16241:
URL: https://github.com/apache/airflow/pull/16241#discussion_r646130343
##
File path: tests/providers/amazon/aws/transfers/test_s3_to_redshift.py
##
@@ -59,6 +59,61 @@ def test_execute(self, mock_run, mock_session):
dnskr commented on a change in pull request #16291:
URL: https://github.com/apache/airflow/pull/16291#discussion_r646645539
##
File path: chart/templates/jobs/migrate-database-job.yaml
##
@@ -38,6 +38,9 @@ metadata:
"helm.sh/hook": post-install,post-upgrade
FabianF92 commented on issue #15803:
URL: https://github.com/apache/airflow/issues/15803#issuecomment-855976402
@griseau I checked again:
This is fixed in PR https://github.com/apache/airflow/pull/14024 for json
attribute.
It is included in versions >= 2.1.0 as I can see, I testet
jjlyr commented on issue #16306:
URL: https://github.com/apache/airflow/issues/16306#issuecomment-855957679
@ashb FYI
--
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
jjlyr opened a new issue #16306:
URL: https://github.com/apache/airflow/issues/16306
**Description**
To add a new flag to be able to disable SSL in Tableau connection(
{"verify": "False"}? ) as it is not present in the current version,
apache-airflow-providers-tableau 1.0.0
coolacid commented on issue #16271:
URL: https://github.com/apache/airflow/issues/16271#issuecomment-855956154
I'm not confident I could correctly get the PR for the connection check -
it's what I use in other projects, but pulling out the correct connection
details, and knowing the
jedcunningham opened a new pull request #16305:
URL: https://github.com/apache/airflow/pull/16305
Found a couple places where we have incorrect examples of `extraSecrets`.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and
kaxil opened a new issue #16304:
URL: https://github.com/apache/airflow/issues/16304
--
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,
jtommi commented on issue #14672:
URL: https://github.com/apache/airflow/issues/14672#issuecomment-855945233
If it can help, for me it also solved the issue.
I'm using a simple local executor with 2 docker containers (scheduler and
webserver).
The root cause is probably a rather slow
kaxil opened a new issue #16303:
URL: https://github.com/apache/airflow/issues/16303
--
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,
ephraimbuddy commented on a change in pull request #16030:
URL: https://github.com/apache/airflow/pull/16030#discussion_r646595504
##
File path: airflow/cli/commands/task_command.py
##
@@ -42,9 +45,27 @@
get_dags,
suppress_logs_and_warning,
)
+from
jedcunningham commented on a change in pull request #16291:
URL: https://github.com/apache/airflow/pull/16291#discussion_r646597798
##
File path: chart/templates/jobs/migrate-database-job.yaml
##
@@ -38,6 +38,9 @@ metadata:
"helm.sh/hook": post-install,post-upgrade
kaxil opened a new issue #16302:
URL: https://github.com/apache/airflow/issues/16302
--
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 #16233:
URL: https://github.com/apache/airflow/pull/16233#issuecomment-855933229
Yeah. I believe this is something of the AWS infrastructure for runners
which we are going to replace (I hope this week) with GCP one - simplified and
improved.
--
This is
kevinbsilva commented on pull request #15947:
URL: https://github.com/apache/airflow/pull/15947#issuecomment-855932744
Thank you for the tip! So, instead of using os.chmod we would use os.chown,
right? This would avoid every user having access to the file. I will change
that. Many thanks!
ephraimbuddy opened a new pull request #16301:
URL: https://github.com/apache/airflow/pull/16301
Currently tasks are not retried when they receive sigkill even if the task
has retries.
This change fixes it and added test for both sigterm and sigkill so we don't
experience regression
potiuk merged pull request #16294:
URL: https://github.com/apache/airflow/pull/16294
--
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 commented on pull request #16284:
URL: https://github.com/apache/airflow/pull/16284#issuecomment-855920447
> > We will need to find a better way for #13414 in that case :(
>
> Right, the issue was even then mentioned explicitly [#13414
kaxil commented on pull request #16257:
URL: https://github.com/apache/airflow/pull/16257#issuecomment-855913705
Just rebased this PR 爛
--
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
1 - 100 of 163 matches
Mail list logo