EinavDanielDX commented on issue #35610:
URL: https://github.com/apache/airflow/issues/35610#issuecomment-1879997134
I have update on the matter.
More background regarding the deployment:
We are using airflow with Celery Executor on Kubernetes, in addition, as we
don't need the workers
eladkal closed issue #16638: Search by Owner in task instances view
URL: https://github.com/apache/airflow/issues/16638
--
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.
To unsub
eladkal closed issue #24954: Multiple pgbouncer instances using the helm chart
URL: https://github.com/apache/airflow/issues/24954
--
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
eladkal commented on issue #24954:
URL: https://github.com/apache/airflow/issues/24954#issuecomment-1879998734
https://github.com/apache/airflow/pull/27439 made it configurable
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub
eladkal commented on issue #25222:
URL: https://github.com/apache/airflow/issues/25222#issuecomment-187012
Should be solved by https://github.com/apache/airflow/pull/35639
@mikebell90 can you help with review?
--
This is an automated message from the Apache Git Service.
To respond t
potiuk commented on PR #36638:
URL: https://github.com/apache/airflow/pull/36638#issuecomment-188485
Update @jscheffl.
I spent a little time and implemented the final version that I foresaw
initiallly for that. That one is quite a bit more complex, because it
implements all the s
jscheffl commented on code in PR #36638:
URL: https://github.com/apache/airflow/pull/36638#discussion_r1443966349
##
.github/workflows/ci.yml:
##
@@ -550,6 +552,52 @@ jobs:
DEFAULT_BRANCH: ${{ needs.build-info.outputs.default-branch }}
RUFF_FORMAT: "github"
jscheffl commented on PR #36638:
URL: https://github.com/apache/airflow/pull/36638#issuecomment-1880007727
Convinced. I'd do a final review and approve once the CI turns green :-D
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHu
potiuk commented on code in PR #36638:
URL: https://github.com/apache/airflow/pull/36638#discussion_r1443972876
##
.github/workflows/ci.yml:
##
@@ -550,6 +552,52 @@ jobs:
DEFAULT_BRANCH: ${{ needs.build-info.outputs.default-branch }}
RUFF_FORMAT: "github"
pankajastro commented on code in PR #36622:
URL: https://github.com/apache/airflow/pull/36622#discussion_r1443972162
##
airflow/providers/microsoft/azure/hooks/synapse.py:
##
@@ -56,8 +56,64 @@ class AzureSynapseSparkBatchRunStatus:
TERMINAL_STATUSES = {SUCCESS, DEAD, KILLE
potiuk commented on PR #36638:
URL: https://github.com/apache/airflow/pull/36638#issuecomment-1880012160
Yeah. I had to add and expand a bit the tests and the code because I forgot
about one more case (but the seletive check unit tests reminded me about it).
We ALSO want to run `airfl
potiuk commented on PR #36638:
URL: https://github.com/apache/airflow/pull/36638#issuecomment-1880013497
BTW. Nice side effect of this one will be that our CI jobs will run even
faster.
Static checks was quite a long job (and mypy was a significant part of it).
By separating out the
elaye-canopy commented on code in PR #36545:
URL: https://github.com/apache/airflow/pull/36545#discussion_r1443975480
##
airflow/providers/google/cloud/transfers/sftp_to_gcs.py:
##
@@ -168,6 +204,68 @@ def _copy_single_object(
self.log.info("Executing delete of %s",
jscheffl commented on code in PR #36638:
URL: https://github.com/apache/airflow/pull/36638#discussion_r1443975448
##
.github/workflows/ci.yml:
##
@@ -550,6 +551,53 @@ jobs:
DEFAULT_BRANCH: ${{ needs.build-info.outputs.default-branch }}
RUFF_FORMAT: "github"
jscheffl commented on code in PR #36638:
URL: https://github.com/apache/airflow/pull/36638#discussion_r1443975448
##
.github/workflows/ci.yml:
##
@@ -550,6 +551,53 @@ jobs:
DEFAULT_BRANCH: ${{ needs.build-info.outputs.default-branch }}
RUFF_FORMAT: "github"
jscheffl commented on PR #36638:
URL: https://github.com/apache/airflow/pull/36638#issuecomment-1880015507
The "mypy (airflow)" check took now very long (9min) - is there some
`--exclude` missing such that it does not check provider subfolder redundant to
the parallel provider test?
--
T
elaye-canopy commented on issue #34995:
URL: https://github.com/apache/airflow/issues/34995#issuecomment-1880016235
@he-gong-sardine
Great to see this is worked on, we independently solved it on our side and I
put a comment in the code about that.
Your solution does seem to work b
elaye-canopy commented on issue #34995:
URL: https://github.com/apache/airflow/issues/34995#issuecomment-1880016337
Also, again, not directly related but logically related - perhaps similar
logic should be made for the converse operator as well (GCS to SFTP)
--
This is an automated messag
potiuk commented on PR #36638:
URL: https://github.com/apache/airflow/pull/36638#issuecomment-1880016862
> The "mypy (airflow)" check took now very long (9min) - is there some
`--exclude` missing such that it does not check provider subfolder redundant to
the parallel provider test?
potiuk commented on code in PR #36638:
URL: https://github.com/apache/airflow/pull/36638#discussion_r1443977169
##
.github/workflows/ci.yml:
##
@@ -550,6 +551,53 @@ jobs:
DEFAULT_BRANCH: ${{ needs.build-info.outputs.default-branch }}
RUFF_FORMAT: "github"
potiuk commented on PR #36638:
URL: https://github.com/apache/airflow/pull/36638#issuecomment-1880018607
BTW. We also might want to add some extra instructions on how to run the
tests if `Mypy` test fails - because it's not obvious that you should run
```
MYPY_PACKAGES="airflow" p
eladkal merged PR #36640:
URL: https://github.com/apache/airflow/pull/36640
--
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.
To unsubscribe, e-mail: commits-unsubscr...@airflow.
This is an automated email from the ASF dual-hosted git repository.
eladkal 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 19ebcac239 Prepare docs 1st wave of Providers Janu
potiuk commented on code in PR #36638:
URL: https://github.com/apache/airflow/pull/36638#discussion_r1443982164
##
.github/workflows/ci.yml:
##
@@ -550,6 +551,53 @@ jobs:
DEFAULT_BRANCH: ${{ needs.build-info.outputs.default-branch }}
RUFF_FORMAT: "github"
potiuk commented on PR #36638:
URL: https://github.com/apache/airflow/pull/36638#issuecomment-1880024443
Added the instructions. And also added one more thing that was wrong - when
we update constraints we need to also make a dependency on the `mypy` job -
otherwise we will update constrain
Author: eladkal
Date: Sun Jan 7 11:02:06 2024
New Revision: 66455
Log:
Add artifacts for Airflow Providers 2024-01-07
Added:
dev/airflow/providers/apache_airflow_providers_amazon-8.16.0-py3-none-any.whl
(with props)
dev/airflow/providers/apache_airflow_providers_amazon-8.16.0-py3-no
potiuk commented on PR #36638:
URL: https://github.com/apache/airflow/pull/36638#issuecomment-1880028121
And added `STATIC_CHECKS` documentaiton explaining all that.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
blcksrx commented on PR #35244:
URL: https://github.com/apache/airflow/pull/35244#issuecomment-1880031917
Yeah, I do! Actually i asked a question but didn’t got any answers On
Sun, Jan 7, 2024 at 07:47, Elad Kalif ***@***.***> wrote:
@blcksrx are you planning completing this PR?
jscheffl commented on code in PR #36638:
URL: https://github.com/apache/airflow/pull/36638#discussion_r1443987182
##
scripts/ci/pre_commit/pre_commit_mypy.py:
##
@@ -51,19 +55,26 @@
"MOUNT_SOURCES": "selected",
},
)
+ci_environment = os.environ.get("CI")
if res.r
potiuk commented on PR #36638:
URL: https://github.com/apache/airflow/pull/36638#issuecomment-1880032729
Final touches: Added `--color always` (in CI this is a pseudo-terminal and
pre-commit does not detect color terminal support automatically) and
`--verbose` - we will see a little more ou
potiuk commented on code in PR #36638:
URL: https://github.com/apache/airflow/pull/36638#discussion_r1443988423
##
scripts/ci/pre_commit/pre_commit_mypy.py:
##
@@ -51,19 +55,26 @@
"MOUNT_SOURCES": "selected",
},
)
+ci_environment = os.environ.get("CI")
if res.ret
potiuk commented on code in PR #36638:
URL: https://github.com/apache/airflow/pull/36638#discussion_r1443988533
##
scripts/ci/pre_commit/pre_commit_mypy.py:
##
@@ -51,19 +55,26 @@
"MOUNT_SOURCES": "selected",
},
)
+ci_environment = os.environ.get("CI")
if res.ret
eladkal commented on PR #35244:
URL: https://github.com/apache/airflow/pull/35244#issuecomment-1880033476
> Actually i asked a question but didn’t got any answers
Which question?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
blcksrx commented on PR #35244:
URL: https://github.com/apache/airflow/pull/35244#issuecomment-1880034326
About warning or decision about deprecation on the Major release On Sun,
Jan 7, 2024 at 12:39, Elad Kalif ***@***.***> wrote:
Actually i asked a question but didn’t got any
potiuk commented on PR #36638:
URL: https://github.com/apache/airflow/pull/36638#issuecomment-1880034630
A little too much verbosity :)
--
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 specifi
eladkal commented on PR #35244:
URL: https://github.com/apache/airflow/pull/35244#issuecomment-1880035274
> About warning or decision about deprecation on the Major release
I answered about it in
https://github.com/apache/airflow/pull/35244#discussion_r1404152273
Is there a followu
potiuk commented on PR #36638:
URL: https://github.com/apache/airflow/pull/36638#issuecomment-1880037772
Right...
That's good enough:
https://github.com/apache/airflow/assets/595491/cc3cdeaa-ccad-45dc-8d97-413eaa66889b";>
--
This is an automated message from the Apache Gi
This is an automated email from the ASF dual-hosted git repository.
eladkal pushed a change to branch add-documentation-2024-01-07
in repository https://gitbox.apache.org/repos/asf/airflow-site.git
at 52580d73d0 Add documentation for packages - 2024-01-07
This branch includes the followin
eladkal opened a new pull request, #931:
URL: https://github.com/apache/airflow-site/pull/931
(no comment)
--
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.
To unsubscribe, e
HarryWu-CHN commented on PR #36641:
URL: https://github.com/apache/airflow/pull/36641#issuecomment-1880040419
> Very nice! Indeed that nicely solves the most pressing need of partial
retrieval of the DAG / DAGRun.
>
> Can you please add some tests covering this ?
@potiuk Hi! Si
eladkal opened a new issue, #36644:
URL: https://github.com/apache/airflow/issues/36644
### Body
I have a kind request for all the contributors to the latest provider
packages release.
Could you please help us to test the RC versions of the providers?
The guidelines on how t
kaxil opened a new pull request, #36645:
URL: https://github.com/apache/airflow/pull/36645
Similar to Apache Superset, adding this repo to README so that Airflow repo
shows up nicely with thumbnails on various Topics pages like
https://github.com/topics/airflow
potiuk opened a new pull request, #36646:
URL: https://github.com/apache/airflow/pull/36646
The CI runners did not have latest version of `pip` and `pipx`. This change
updates the installation scripts to fix `pip` to the same version as in the CI
image and down-binds pipx to 1.4.1 which is
potiuk commented on code in PR #36646:
URL: https://github.com/apache/airflow/pull/36646#discussion_r1443996825
##
dev/breeze/pyproject.toml:
##
@@ -54,6 +54,7 @@ dependencies = [
"jinja2>=3.1.0",
"jsonschema>=4.19.1",
"packaging>=23.2",
+"pipx>=1.4.1",
Revie
potiuk commented on code in PR #36537:
URL: https://github.com/apache/airflow/pull/36537#discussion_r144436
##
INSTALL:
##
@@ -1,118 +1,324 @@
# INSTALL / BUILD instructions for Apache Airflow
-This is a generic installation method that requires a number of dependencies
potiuk merged PR #36646:
URL: https://github.com/apache/airflow/pull/36646
--
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.
To unsubscribe, e-mail: commits-unsubscr...@airflow.a
This is an automated email from the ASF dual-hosted git repository.
potiuk 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 75bc05ce1f Upgrade to latest versions of `pip` and
potiuk merged PR #36638:
URL: https://github.com/apache/airflow/pull/36638
--
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.
To unsubscribe, e-mail: commits-unsubscr...@airflow.a
This is an automated email from the ASF dual-hosted git repository.
potiuk 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 f7b663d9af Run mypy checks for full packages in CI
This is an automated email from the ASF dual-hosted git repository.
husseinawala 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 9a20aa8f0f Add Airflow logo to the README (#3
hussein-awala merged PR #36645:
URL: https://github.com/apache/airflow/pull/36645
--
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.
To unsubscribe, e-mail: commits-unsubscr...@ai
hussein-awala commented on PR #36645:
URL: https://github.com/apache/airflow/pull/36645#issuecomment-1880058362
It didn't work; I will check the README files in the other repositories
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to G
potiuk opened a new pull request, #36647:
URL: https://github.com/apache/airflow/pull/36647
The `graphviz` dependency has been problematic as Airflow required
dependency - especially for ARM-based installations. Graphviz packages require
binary graphviz libraries - which is already a limita
elaye-canopy commented on code in PR #36545:
URL: https://github.com/apache/airflow/pull/36545#discussion_r1443975333
##
airflow/providers/google/cloud/transfers/sftp_to_gcs.py:
##
@@ -168,6 +204,68 @@ def _copy_single_object(
self.log.info("Executing delete of %s",
potiuk commented on code in PR #36537:
URL: https://github.com/apache/airflow/pull/36537#discussion_r1444006201
##
airflow/utils/dot_renderer.py:
##
@@ -19,9 +19,14 @@
"""Renderer DAG (tasks and dependencies) to the graphviz object."""
from __future__ import annotations
+imp
This is an automated email from the ASF dual-hosted git repository.
potiuk pushed a change to branch switch-to-hatch
in repository https://gitbox.apache.org/repos/asf/airflow.git
discard 5cba0a1607 Standardize airflow build process and switch to Hatchling
build backend
add 4469baad4e Add
elaye-canopy commented on code in PR #36545:
URL: https://github.com/apache/airflow/pull/36545#discussion_r1443975333
##
airflow/providers/google/cloud/transfers/sftp_to_gcs.py:
##
@@ -168,6 +204,68 @@ def _copy_single_object(
self.log.info("Executing delete of %s",
jscheffl commented on PR #36645:
URL: https://github.com/apache/airflow/pull/36645#issuecomment-1880062299
The logo is not served from README :-(
You need to be repo admin, go to repo settings and there add a "Social
preview" image.
--
This is an automated message from the Apache Git S
jscheffl commented on PR #36645:
URL: https://github.com/apache/airflow/pull/36645#issuecomment-1880062687
I assume we _could_ set it manually but a follow-up weekend activity was to
check if there is an option in `.asf.yaml` or maybe a feature needs to be added
there to define it programat
RNHTTR commented on issue #36629:
URL: https://github.com/apache/airflow/issues/36629#issuecomment-1880070155
For what it's worth, [the `SFTPSensor` does support templating
`newer_than`](https://github.com/apache/airflow/blob/providers-sftp/4.4.0/airflow/providers/sftp/sensors/sftp.py#L47),
kaxil commented on PR #36645:
URL: https://github.com/apache/airflow/pull/36645#issuecomment-1880071661
> The logo is not served from README :-( You need to be repo admin, go to
repo settings and there add a "Social preview" image.
Are you sure about that? Most of the repos I am seein
This is an automated email from the ASF dual-hosted git repository.
potiuk pushed a change to branch switch-to-hatch
in repository https://gitbox.apache.org/repos/asf/airflow.git
omit e18efed381 Standardize airflow build process and switch to Hatchling
build backend
add 7c8df822a0 Stan
potiuk commented on code in PR #36537:
URL: https://github.com/apache/airflow/pull/36537#discussion_r1444014436
##
hatch_build.py:
##
Review Comment:
Moved `hatch_build.py` to `dev` folder.
--
This is an automated message from the Apache Git Service.
To respond to the m
This is an automated email from the ASF dual-hosted git repository.
github-bot pushed a commit to branch constraints-main
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/constraints-main by this push:
new f55337839c Updating con
kaxil commented on PR #36645:
URL: https://github.com/apache/airflow/pull/36645#issuecomment-1880073879
You are probably right, I experimented with another repo
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL a
Nimo139 commented on PR #35704:
URL: https://github.com/apache/airflow/pull/35704#issuecomment-1880074271
Sorry for the late reply, I didn't have time to set up the test environment
during the holidays. I have now adjusted the tests for the parameter so that
they pass. I have added a very b
kaxil commented on PR #36645:
URL: https://github.com/apache/airflow/pull/36645#issuecomment-1880074340
Yup found the Superset thing -
[here](https://issues.apache.org/jira/browse/INFRA-20803?jql=text%20~%20%22social%20preview%22),
we could request the same.
And we should make it sel
kaxil commented on PR #36645:
URL: https://github.com/apache/airflow/pull/36645#issuecomment-1880075252
Created this ticket - https://issues.apache.org/jira/browse/INFRA-25351 for
now
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
potiuk commented on PR #36641:
URL: https://github.com/apache/airflow/pull/36641#issuecomment-1880078290
> > Very nice! Indeed that nicely solves the most pressing need of partial
retrieval of the DAG / DAGRun.
> > Can you please add some tests covering this ?
>
> @potiuk Hi! Since
potiuk commented on PR #36645:
URL: https://github.com/apache/airflow/pull/36645#issuecomment-1880079809
Yeah.. I'v already added a feature there - it's Python based so it's not a
rocket science https://github.com/apache/infrastructure-p6/pull/1740. - we just
need to check if there is an AP
jscheffl commented on PR #36645:
URL: https://github.com/apache/airflow/pull/36645#issuecomment-1880082614
> You are probably right, I experimented with another repo
@kaxil I was not fast enough with response... Yes I also tried with a
temporary repo which was empty yesterday... from
jscheffl commented on PR #36645:
URL: https://github.com/apache/airflow/pull/36645#issuecomment-1880083915
> Created this ticket - https://issues.apache.org/jira/browse/INFRA-25351
for now
Oh, they already added it - that was FAST. Problem solved for now :-D
--
This is an automated
potiuk opened a new pull request, #36648:
URL: https://github.com/apache/airflow/pull/36648
After #36591 has been implemented, in cas of `--spellcheck-only` checks for
parallel packages, it might be that some build errors remain in build errors
output (depends on some race conditions while
RNHTTR commented on issue #23020:
URL: https://github.com/apache/airflow/issues/23020#issuecomment-1880089536
I'm interested in taking this on (or at least moving it forward). Would it
be sufficient to start by:
1. Adding something like `named_mapped_index` to BaseOperator and as a
c
Andrew-Wichmann commented on issue #11334:
URL: https://github.com/apache/airflow/issues/11334#issuecomment-1880091543
I hear ya'. I'm not demanding anything. I really like the software. Thank
you for your time and effort
--
This is an automated message from the Apache Git Service.
To res
potiuk opened a new pull request, #36649:
URL: https://github.com/apache/airflow/pull/36649
This PR upgrades to latest ruff, and removes the ISC001 warning that warns
us against potential conflict between ruff and ruff-formatter when two strings
in one line get concatenated.
This war
potiuk merged PR #36647:
URL: https://github.com/apache/airflow/pull/36647
--
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.
To unsubscribe, e-mail: commits-unsubscr...@airflow.a
This is an automated email from the ASF dual-hosted git repository.
potiuk 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 89f1737afb Make `graphviz` dependency optional (#36
potiuk merged PR #36648:
URL: https://github.com/apache/airflow/pull/36648
--
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.
To unsubscribe, e-mail: commits-unsubscr...@airflow.a
This is an automated email from the ASF dual-hosted git repository.
potiuk 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 579c4b5c59 Remove build errors output from spellche
potiuk commented on PR #36649:
URL: https://github.com/apache/airflow/pull/36649#issuecomment-1880099482
These warnings get removed:
https://github.com/apache/airflow/assets/595491/08a30262-8612-46ba-9615-000cb9ea778f";>
--
This is an automated message from the Apache Git Servic
potiuk commented on code in PR #36649:
URL: https://github.com/apache/airflow/pull/36649#discussion_r1444031321
##
pyproject.toml:
##
@@ -65,7 +65,7 @@ extend-select = [
"ISC", # Checks for implicit literal string concatenation (auto-fixable)
"B006", # Checks for uses
kaxil commented on issue #23020:
URL: https://github.com/apache/airflow/issues/23020#issuecomment-1880105381
> I'm interested in taking this on (or at least moving it forward). Would it
be sufficient to start by:
>
> 1. Adding something like `named_mapped_index` to BaseOperator and as
ighoshsubho opened a new pull request, #36650:
URL: https://github.com/apache/airflow/pull/36650
## Functionality added -
- SnowflakeSqlApiOperator support for OAuth
closes: #36621
---
**^ Add meaningful description above**
Read the **[Pu
boring-cyborg[bot] commented on PR #36650:
URL: https://github.com/apache/airflow/pull/36650#issuecomment-1880109101
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 Contribution
blcksrx commented on PR #35244:
URL: https://github.com/apache/airflow/pull/35244#issuecomment-1880109251
I see, Im gonna fix it this week. On Sun, Jan 7, 2024 at 12:47, Elad
Kalif ***@***.***> wrote:
About warning or decision about deprecation on the Major release
I answ
jbampton opened a new pull request, #36651:
URL: https://github.com/apache/airflow/pull/36651
https://github.com/apache/airflow/blob/579c4b5c59e3855dd1285cbfe482f41c57b2b82e/.pre-commit-config.yaml#L281C6-L281C6
Convert the `spelling_wordlist.txt` to all lowercase. Thi
potiuk commented on PR #36645:
URL: https://github.com/apache/airflow/pull/36645#issuecomment-1880118253
The image though is probably not perfect (refreshed preview in LinkedIn -
Facebook is similar).
https://github.com/apache/airflow/assets/595491/6d121782-2a67-48be-8803-4e457e58e911
This is an automated email from the ASF dual-hosted git repository.
potiuk pushed a change to branch switch-to-hatch
in repository https://gitbox.apache.org/repos/asf/airflow.git
discard 7c8df822a0 Standardize airflow build process and switch to Hatchling
build backend
add 9a20aa8f0f Add
potiuk closed pull request #36650: [FEATURE] Added oauth support with snowflake
sql api operator
URL: https://github.com/apache/airflow/pull/36650
--
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 PR #36650:
URL: https://github.com/apache/airflow/pull/36650#issuecomment-1880124442
You need to add tests, it makes little sense to run CI if there are no tests
added for it.
--
This is an automated message from the Apache Git Service.
To respond to the message, pleas
potiuk commented on PR #36650:
URL: https://github.com/apache/airflow/pull/36650#issuecomment-1880124501
Closed by mistake.
--
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.
potiuk commented on PR #35704:
URL: https://github.com/apache/airflow/pull/35704#issuecomment-1880125333
Should be good enough - now the only thing left is to rebase to latest
`main` and fix failing static checks
--
This is an automated message from the Apache Git Service.
To respond to t
potiuk commented on code in PR #36537:
URL: https://github.com/apache/airflow/pull/36537#discussion_r1444047774
##
dev/old_setup/MANIFEST.in:
##
Review Comment:
OK. Removed it for now. I have a copy in git and it is enough :)
--
This is an automated message from the Ap
ighoshsubho commented on PR #36650:
URL: https://github.com/apache/airflow/pull/36650#issuecomment-1880129786
> You need to add tests, it makes little sense to run CI if there are no
tests added for it.
sure will do it
--
This is an automated message from the Apache Git Service.
To
eladkal commented on code in PR #35704:
URL: https://github.com/apache/airflow/pull/35704#discussion_r1444052299
##
tests/providers/google/cloud/transfers/test_gcs_to_sftp.py:
##
@@ -116,7 +116,55 @@ def test_execute_move_single_file(
)
sftp_hook_mock.return_
EinavDanielDX opened a new issue, #35610:
URL: https://github.com/apache/airflow/issues/35610
### Apache Airflow version
2.7.3
### What happened
I've executed a task, on flower and worker log I see this error:
```
airflow.exceptions.AirflowException: Celery command
potiuk commented on issue #35610:
URL: https://github.com/apache/airflow/issues/35610#issuecomment-1880141693
That's an interesting lead. I will mark it for 2.8.1 and maybe we could
guesss what could be the readon. Do you see any logs from the workers
indicating that there might be an error
potiuk merged PR #36649:
URL: https://github.com/apache/airflow/pull/36649
--
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.
To unsubscribe, e-mail: commits-unsubscr...@airflow.a
This is an automated email from the ASF dual-hosted git repository.
potiuk 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 11c46fd2ec Upgrade to latest ruff and remove ISC001
1 - 100 of 146 matches
Mail list logo