boring-cyborg[bot] commented on issue #10502:
URL: https://github.com/apache/airflow/issues/10502#issuecomment-678908570
Thanks for opening your first issue here! Be sure to follow the issue
template!
This is an
andypp opened a new issue #10502:
URL: https://github.com/apache/airflow/issues/10502
**Apache Airflow version**: 2.0.0-dev (master)
mik-laj commented on issue #10306:
URL: https://github.com/apache/airflow/issues/10306#issuecomment-678878973
Small update:
I have a working PR, but it still needs some care to make everything easily
accessible with the breeze command.
One problem has already been found thanks to
xu-xiang commented on issue #10479:
URL: https://github.com/apache/airflow/issues/10479#issuecomment-678863748
Thank you. I'll test it again.
This is an automated message from the Apache Git Service.
To respond to the
This is an automated email from the ASF dual-hosted git repository.
github-bot pushed a commit to branch constraints-master
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/constraints-master by this
push:
new 6ee8f62 Updating
This is an automated email from the ASF dual-hosted git repository.
github-bot pushed a commit to branch constraints-master
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/constraints-master by this
push:
new 6ee8f62 Updating
potiuk commented on a change in pull request #10498:
URL: https://github.com/apache/airflow/pull/10498#discussion_r475295182
##
File path: airflow/www/views.py
##
@@ -2329,28 +2411,29 @@ def process_form(self, form, is_created):
form.extra.data = json.dumps(extra)
potiuk commented on a change in pull request #10498:
URL: https://github.com/apache/airflow/pull/10498#discussion_r475294374
##
File path: airflow/www/views.py
##
@@ -1565,36 +1629,37 @@ def graph(self, session=None):
nodes = []
edges = []
-for task
potiuk commented on a change in pull request #10498:
URL: https://github.com/apache/airflow/pull/10498#discussion_r475294290
##
File path: airflow/www/views.py
##
@@ -1437,27 +1500,27 @@ def tree(self):
node_count = 0
node_limit = 5000 / max(1,
potiuk commented on a change in pull request #10498:
URL: https://github.com/apache/airflow/pull/10498#discussion_r475293930
##
File path: airflow/www/views.py
##
@@ -827,23 +873,25 @@ def task(self):
return redirect(url_for('Airflow.index'))
task =
potiuk commented on a change in pull request #10498:
URL: https://github.com/apache/airflow/pull/10498#discussion_r475293510
##
File path: airflow/www/views.py
##
@@ -94,54 +95,53 @@ def get_safe_url(url):
return url_for('Airflow.index')
-def
potiuk commented on issue #10363:
URL: https://github.com/apache/airflow/issues/10363#issuecomment-678829750
@sanjana1604 The context is described mostly in
http://www.apache.org/legal/release-policy.html - Apache releases software as
sources, so the binaries are released mostly as
potiuk commented on pull request #10475:
URL: https://github.com/apache/airflow/pull/10475#issuecomment-678829355
Quarantined builds fail from time to time.
This is an automated message from the Apache Git Service.
To
potiuk commented on issue #10486:
URL: https://github.com/apache/airflow/issues/10486#issuecomment-678829237
I'd leave it up to @dimberman
This is an automated message from the Apache Git Service.
To respond to the message,
potiuk commented on a change in pull request #10497:
URL: https://github.com/apache/airflow/pull/10497#discussion_r475271195
##
File path: airflow/www/utils.py
##
@@ -181,7 +181,7 @@ def is_current(current, page):
def epoch(dttm):
"""Returns an epoch-type date"""
-
potiuk commented on pull request #10461:
URL: https://github.com/apache/airflow/pull/10461#issuecomment-678828823
The checks are being cancelled because we have not yet ported the recent CI
changes to the 1.10 branch. We are going to do that after the 1.10.12 release
gets actually
stale[bot] closed pull request #9364:
URL: https://github.com/apache/airflow/pull/9364
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
syudb commented on pull request #10461:
URL: https://github.com/apache/airflow/pull/10461#issuecomment-678819988
Apologies for the noob question but not sure why the checks are getting
cancelled, please let me know if there's something I need to fix!
potiuk opened a new pull request #10501:
URL: https://github.com/apache/airflow/pull/10501
---
**^ Add meaningful description above**
Read the **[Pull Request
Guidelines](https://github.com/apache/airflow/blob/master/CONTRIBUTING.rst#pull-request-guidelines)**
for more
turbaszek commented on a change in pull request #10499:
URL: https://github.com/apache/airflow/pull/10499#discussion_r475239161
##
File path: airflow/models/taskinstance.py
##
@@ -1135,7 +1135,11 @@ def _run_raw_task(
session.merge(self)
session.commit()
turbaszek commented on a change in pull request #10500:
URL: https://github.com/apache/airflow/pull/10500#discussion_r475239029
##
File path: airflow/models/crypto.py
##
@@ -85,7 +87,7 @@ def get_fernet():
for fernet_part in fernet_key.split(',')
turbaszek commented on a change in pull request #10497:
URL: https://github.com/apache/airflow/pull/10497#discussion_r475238894
##
File path: airflow/www/utils.py
##
@@ -253,6 +253,7 @@ def nobr(attr):
return nobr
+# pylint: disable=invalid-name
def
turbaszek commented on a change in pull request #10497:
URL: https://github.com/apache/airflow/pull/10497#discussion_r475238810
##
File path: airflow/www/utils.py
##
@@ -181,7 +181,7 @@ def is_current(current, page):
def epoch(dttm):
"""Returns an epoch-type date"""
-
turbaszek commented on a change in pull request #10498:
URL: https://github.com/apache/airflow/pull/10498#discussion_r475238330
##
File path: airflow/www/views.py
##
@@ -2329,28 +2411,29 @@ def process_form(self, form, is_created):
form.extra.data =
turbaszek commented on a change in pull request #10498:
URL: https://github.com/apache/airflow/pull/10498#discussion_r475237984
##
File path: airflow/www/views.py
##
@@ -1565,36 +1629,37 @@ def graph(self, session=None):
nodes = []
edges = []
-for
turbaszek commented on a change in pull request #10498:
URL: https://github.com/apache/airflow/pull/10498#discussion_r475237934
##
File path: airflow/www/views.py
##
@@ -1437,27 +1500,27 @@ def tree(self):
node_count = 0
node_limit = 5000 / max(1,
turbaszek commented on a change in pull request #10498:
URL: https://github.com/apache/airflow/pull/10498#discussion_r475237791
##
File path: airflow/www/views.py
##
@@ -1049,21 +1102,22 @@ def trigger(self, session=None):
dr = DagRun.find(dag_id=dag_id,
turbaszek commented on a change in pull request #10498:
URL: https://github.com/apache/airflow/pull/10498#discussion_r475237654
##
File path: airflow/www/views.py
##
@@ -827,23 +873,25 @@ def task(self):
return redirect(url_for('Airflow.index'))
task =
potiuk opened a new pull request #10500:
URL: https://github.com/apache/airflow/pull/10500
---
**^ Add meaningful description above**
Read the **[Pull Request
Guidelines](https://github.com/apache/airflow/blob/master/CONTRIBUTING.rst#pull-request-guidelines)**
for more
potiuk opened a new pull request #10499:
URL: https://github.com/apache/airflow/pull/10499
---
**^ Add meaningful description above**
Read the **[Pull Request
Guidelines](https://github.com/apache/airflow/blob/master/CONTRIBUTING.rst#pull-request-guidelines)**
for more
turbaszek commented on a change in pull request #10498:
URL: https://github.com/apache/airflow/pull/10498#discussion_r475236925
##
File path: airflow/www/views.py
##
@@ -94,54 +95,53 @@ def get_safe_url(url):
return url_for('Airflow.index')
-def
potiuk commented on pull request #10497:
URL: https://github.com/apache/airflow/pull/10497#issuecomment-678792585
All green !
This is an automated message from the Apache Git Service.
To respond to the message, please log on
potiuk commented on pull request #10498:
URL: https://github.com/apache/airflow/pull/10498#issuecomment-678792638
Also looks green :)
This is an automated message from the Apache Git Service.
To respond to the message,
potiuk opened a new pull request #10498:
URL: https://github.com/apache/airflow/pull/10498
---
**^ Add meaningful description above**
Read the **[Pull Request
Guidelines](https://github.com/apache/airflow/blob/master/CONTRIBUTING.rst#pull-request-guidelines)**
for more
This is an automated email from the ASF dual-hosted git repository.
potiuk pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/airflow.git.
from 5c6e4e2 Make Stats pylint compatible (#10496)
add 3a53039 GitHub Registry is now lowercase (#10489)
No new
This is an automated email from the ASF dual-hosted git repository.
potiuk pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/airflow.git.
from 5c6e4e2 Make Stats pylint compatible (#10496)
add 3a53039 GitHub Registry is now lowercase (#10489)
No new
This is an automated email from the ASF dual-hosted git repository.
potiuk pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/airflow.git.
from 5c6e4e2 Make Stats pylint compatible (#10496)
add 3a53039 GitHub Registry is now lowercase (#10489)
No new
This is an automated email from the ASF dual-hosted git repository.
potiuk pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/airflow.git.
from 6eb64ce Make macros.hive pylint compatible (#10495)
add 5c6e4e2 Make Stats pylint compatible (#10496)
No
This is an automated email from the ASF dual-hosted git repository.
potiuk pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/airflow.git.
from 6eb64ce Make macros.hive pylint compatible (#10495)
add 5c6e4e2 Make Stats pylint compatible (#10496)
No
potiuk merged pull request #10489:
URL: https://github.com/apache/airflow/pull/10489
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
potiuk merged pull request #10496:
URL: https://github.com/apache/airflow/pull/10496
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
LvffY closed issue #10191:
URL: https://github.com/apache/airflow/issues/10191
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
LvffY commented on issue #10191:
URL: https://github.com/apache/airflow/issues/10191#issuecomment-678782972
I finally found my problem. It cames from a bad installation of azure
dependencies.
If you fall into the same problem as me, you should see in your scheduler
logs an error
This is an automated email from the ASF dual-hosted git repository.
potiuk pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/airflow.git.
from 82369fa Removed the prerequisite for perf-kit path augmentation
(#10492)
add 6eb64ce Make macros.hive
This is an automated email from the ASF dual-hosted git repository.
potiuk pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/airflow.git.
from 82369fa Removed the prerequisite for perf-kit path augmentation
(#10492)
add 6eb64ce Make macros.hive
This is an automated email from the ASF dual-hosted git repository.
potiuk pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/airflow.git.
from c8c3f8b Remove old configuration from BoringCyborg (#10490)
add 82369fa Removed the prerequisite for
potiuk merged pull request #10495:
URL: https://github.com/apache/airflow/pull/10495
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
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 82369fa Removed the prerequisite for perf-kit
potiuk merged pull request #10492:
URL: https://github.com/apache/airflow/pull/10492
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
akuzni2 edited a comment on issue #10486:
URL: https://github.com/apache/airflow/issues/10486#issuecomment-678776406
@mik-laj - thanks!
@potiuk - not sure if this is invalid. The documentation on the chart page
https://github.com/apache/airflow/tree/master/chart shows to install the
akuzni2 edited a comment on issue #10486:
URL: https://github.com/apache/airflow/issues/10486#issuecomment-678776406
@mik-laj - thanks!
@potiuk - not sure if this is invalid. If you check the documentation on
the main chart page https://github.com/apache/airflow/tree/master/chart, it
mik-laj commented on a change in pull request #10497:
URL: https://github.com/apache/airflow/pull/10497#discussion_r475221829
##
File path: scripts/ci/pylint_todo.txt
##
@@ -5,5 +5,4 @@
./airflow/models/dagrun.py
./airflow/models/taskinstance.py
./airflow/stats.py
potiuk opened a new pull request #10497:
URL: https://github.com/apache/airflow/pull/10497
---
**^ Add meaningful description above**
Read the **[Pull Request
Guidelines](https://github.com/apache/airflow/blob/master/CONTRIBUTING.rst#pull-request-guidelines)**
for more
mik-laj commented on a change in pull request #10494:
URL: https://github.com/apache/airflow/pull/10494#discussion_r475221592
##
File path: airflow/configuration.py
##
@@ -878,7 +929,8 @@ def as_dict(*args, **kwargs): # noqa: D103
return conf.as_dict(*args, **kwargs)
akuzni2 commented on issue #10486:
URL: https://github.com/apache/airflow/issues/10486#issuecomment-678776406
@mik-laj - thanks!
@potiuk - not sure if I'd close this as invalid. If you check the
documentation on the main chart page
https://github.com/apache/airflow/tree/master/chart,
potiuk opened a new pull request #10496:
URL: https://github.com/apache/airflow/pull/10496
---
**^ Add meaningful description above**
Read the **[Pull Request
Guidelines](https://github.com/apache/airflow/blob/master/CONTRIBUTING.rst#pull-request-guidelines)**
for more
potiuk opened a new pull request #10495:
URL: https://github.com/apache/airflow/pull/10495
---
**^ Add meaningful description above**
Read the **[Pull Request
Guidelines](https://github.com/apache/airflow/blob/master/CONTRIBUTING.rst#pull-request-guidelines)**
for more
derrickqin commented on pull request #10475:
URL: https://github.com/apache/airflow/pull/10475#issuecomment-678770799
@turbaszek could you please have another look? CI build failed on
Quarantined tests, don't think it is related to my change...
Thanks!
potiuk opened a new pull request #10494:
URL: https://github.com/apache/airflow/pull/10494
---
**^ Add meaningful description above**
Read the **[Pull Request
Guidelines](https://github.com/apache/airflow/blob/master/CONTRIBUTING.rst#pull-request-guidelines)**
for more
potiuk commented on pull request #10487:
URL: https://github.com/apache/airflow/pull/10487#issuecomment-678767948
Besides this _all_libs is purely an "include" one - I think it's much worse
to make one function not included in "all_include" file. I am not sure what we
want to achieve
sanjana1604 commented on issue #10363:
URL: https://github.com/apache/airflow/issues/10363#issuecomment-678766658
@potiuk need more clarification on first point (context).
This is an automated message from the Apache Git
potiuk edited a comment on issue #10486:
URL: https://github.com/apache/airflow/issues/10486#issuecomment-678763852
@dimberman -> is this something we should close as invalid or maybe we
should already have some project (similar to the Docker image project
potiuk commented on issue #10486:
URL: https://github.com/apache/airflow/issues/10486#issuecomment-678763852
@dimberman -> is this something we shoudl close as invalid or maybe we
should already have some project (similar to the Docker image project
potiuk commented on issue #10429:
URL: https://github.com/apache/airflow/issues/10429#issuecomment-678763430
Would you be so kind to make a PR with that? It shoudl be rather easy?
BTW. When you open a new "security" issue from template, you should get
information that the right way
potiuk commented on pull request #10470:
URL: https://github.com/apache/airflow/pull/10470#issuecomment-678762991
I do not like the idea of failing imports, I have no idea whether there are
no side effects and I think those tests woudl fail if the import fails, because
there would be no
potiuk opened a new pull request #10493:
URL: https://github.com/apache/airflow/pull/10493
---
**^ Add meaningful description above**
Read the **[Pull Request
Guidelines](https://github.com/apache/airflow/blob/master/CONTRIBUTING.rst#pull-request-guidelines)**
for more
This is an automated email from the ASF dual-hosted git repository.
github-bot pushed a commit to branch constraints-master
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/constraints-master by this
push:
new a296ecf Updating
This is an automated email from the ASF dual-hosted git repository.
turbaszek pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/airflow.git.
from ef8df17 Fix typo in Facebook Ads Provider (#10484)
add c8c3f8b Remove old configuration from BoringCyborg
This is an automated email from the ASF dual-hosted git repository.
turbaszek pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/airflow.git.
from ef8df17 Fix typo in Facebook Ads Provider (#10484)
add c8c3f8b Remove old configuration from BoringCyborg
turbaszek merged pull request #10490:
URL: https://github.com/apache/airflow/pull/10490
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 opened a new pull request #10492:
URL: https://github.com/apache/airflow/pull/10492
---
**^ Add meaningful description above**
Read the **[Pull Request
Guidelines](https://github.com/apache/airflow/blob/master/CONTRIBUTING.rst#pull-request-guidelines)**
for more
potiuk commented on a change in pull request #10470:
URL: https://github.com/apache/airflow/pull/10470#discussion_r475199304
##
File path: tests/utils/perf/perf_kit/__init__.py
##
@@ -32,33 +32,34 @@
===
Review comment:
Ah yeah. Fixing it now.
potiuk opened a new pull request #10491:
URL: https://github.com/apache/airflow/pull/10491
You can now define secret in your own fork:
GITHUB_REGISTRY_WAIT_FOR_IMAGE
If you set it to "false", it skips building images in separate
workflow_run - images will be built in the
turbaszek opened a new pull request #10490:
URL: https://github.com/apache/airflow/pull/10490
This was redundant as PR template has been changed
---
**^ Add meaningful description above**
Read the **[Pull Request
mik-laj commented on a change in pull request #10447:
URL: https://github.com/apache/airflow/pull/10447#discussion_r475190210
##
File path: airflow/providers/cncf/kubernetes/hooks/kubernetes.py
##
@@ -46,11 +46,14 @@ def __init__(
):
super().__init__()
HaloKo4 commented on issue #7988:
URL: https://github.com/apache/airflow/issues/7988#issuecomment-678743588
I think this feature is already given.
if using CeleryExecutor you can just set a queue for backfill and queue for
regular jobs
Then the two don't share resources and you can
turbaszek commented on pull request #10475:
URL: https://github.com/apache/airflow/pull/10475#issuecomment-678743478
@derrickqin would you mind rebasing onto current master? There seems to be
an issue with CI :<
This is an
potiuk opened a new pull request #10489:
URL: https://github.com/apache/airflow/pull/10489
GitHub Registry must be lowercase and we are now using it to
store images/packages. We lowercase it now and we also
made sure that GITHUB_REGISTRY variables are only used
when GITHUB_REGISTRY
turbaszek commented on a change in pull request #10447:
URL: https://github.com/apache/airflow/pull/10447#discussion_r475186657
##
File path: airflow/providers/cncf/kubernetes/hooks/kubernetes.py
##
@@ -46,11 +46,14 @@ def __init__(
):
super().__init__()
stale[bot] closed pull request #8941:
URL: https://github.com/apache/airflow/pull/8941
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
mik-laj commented on a change in pull request #10417:
URL: https://github.com/apache/airflow/pull/10417#discussion_r475181656
##
File path: airflow/api_connexion/openapi/v1.yaml
##
@@ -1076,7 +,7 @@ paths:
/version:
get:
- summary: Get version information
+
mik-laj commented on a change in pull request #10417:
URL: https://github.com/apache/airflow/pull/10417#discussion_r475181157
##
File path: airflow/api_connexion/openapi/v1.yaml
##
@@ -1052,17 +1064,40 @@ paths:
application/json:
schema:
houqp commented on a change in pull request #10417:
URL: https://github.com/apache/airflow/pull/10417#discussion_r475180202
##
File path: airflow/api_connexion/openapi/v1.yaml
##
@@ -1076,7 +,7 @@ paths:
/version:
get:
- summary: Get version information
+
mik-laj opened a new pull request #10488:
URL: https://github.com/apache/airflow/pull/10488
https://github.com/apache/airflow/issues/10306
---
**^ Add meaningful description above**
Read the **[Pull Request
84 matches
Mail list logo