PatrykKlimowicz commented on issue #25177:
URL: https://github.com/apache/airflow/issues/25177#issuecomment-1192248250
@potiuk I'll try to test 😄 Will be back with some feedback soon
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to Gi
Bowrna commented on PR #623:
URL: https://github.com/apache/airflow-site/pull/623#issuecomment-1192201127
> @Bowrna see: https://stackoverflow.com/a/12498485 This generates a
relative path for use in a container.
Could you tell me why we need to convert the absolute path into relative
uranusjr commented on PR #25215:
URL: https://github.com/apache/airflow/pull/25215#issuecomment-1192176606
Could you also do the same for `ExternalTaskMarker`? It’s in the same file.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to Gi
uranusjr commented on issue #25165:
URL: https://github.com/apache/airflow/issues/25165#issuecomment-1192167588
> I think the "root" fix is to prepend the group_id to
MappedOperator.task_id, like what BaseOperator does
This logic already exists; a `@task`-decorated task calls
`get_un
uranusjr commented on PR #25218:
URL: https://github.com/apache/airflow/pull/25218#issuecomment-1192147059
pip doesn’t do semver because there is not any reasonable point it could
ever bump major, and even when we need to make incompatible changes, we need
drag all the users to the new vers
eladkal commented on PR #23654:
URL: https://github.com/apache/airflow/pull/23654#issuecomment-1192117375
@kalyani-subbiah are you still working on 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
amoGLingle commented on issue #24909:
URL: https://github.com/apache/airflow/issues/24909#issuecomment-1192115740
As an experiment, we're turning off one of the Schedulers - no HA - to see
if we still get a deadlock.
--
This is an automated message from the Apache Git Service.
To respond
boring-cyborg[bot] commented on issue #25226:
URL: https://github.com/apache/airflow/issues/25226#issuecomment-1192077021
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 mess
pewpewp3w opened a new issue, #25226:
URL: https://github.com/apache/airflow/issues/25226
### Official Helm Chart version
1.6.0 (latest released)
### Apache Airflow version
2.3.3 (latest released)
### Kubernetes Version
does not matter
### Helm Chart
github-actions[bot] commented on issue #18628:
URL: https://github.com/apache/airflow/issues/18628#issuecomment-1192050435
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
github-actions[bot] closed issue #18628: The cleared dagrun and tasks are set
failed by scheduler
URL: https://github.com/apache/airflow/issues/18628
--
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
github-actions[bot] closed pull request #22806: Helmchart: Allow digest version
to be used
URL: https://github.com/apache/airflow/pull/22806
--
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 speci
github-actions[bot] closed issue #24400: Airflow db upgrade fails to insert
rows in log_template table
URL: https://github.com/apache/airflow/issues/24400
--
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 g
github-actions[bot] commented on issue #24400:
URL: https://github.com/apache/airflow/issues/24400#issuecomment-1192050381
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
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 1cb16d5588 Upgrade to Pip 22.2 (#25218)
1cb16d5588
potiuk merged PR #25218:
URL: https://github.com/apache/airflow/pull/25218
--
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
potiuk commented on PR #25218:
URL: https://github.com/apache/airflow/pull/25218#issuecomment-1192024482
> Interesting they don't include `.0`.
Yep. `pypi` does not use SemVer. And while you can use .0 in the first
version, th form of X.Y is generally ok for all the projects - not on
potiuk commented on PR #25224:
URL: https://github.com/apache/airflow/pull/25224#issuecomment-1192010510
> Huh. Where did those come from 👀
>
> Looks like the same tests are failing in main, so it's something that was
changed in the last couple of days. I'm heading out for a mini
erdos2n commented on issue #25165:
URL: https://github.com/apache/airflow/issues/25165#issuecomment-1191975901
I don't know if I can assign myself...
--
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
bbovenzi commented on issue #25165:
URL: https://github.com/apache/airflow/issues/25165#issuecomment-1191967441
@erdos2n go for it!
--
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 co
ferruzzi commented on PR #25224:
URL: https://github.com/apache/airflow/pull/25224#issuecomment-1191963364
Huh. Where did those come from :eyes:
--
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
erdos2n commented on issue #25165:
URL: https://github.com/apache/airflow/issues/25165#issuecomment-1191963111
Can I take this @bbovenzi ?
--
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 spec
potiuk commented on code in PR #25185:
URL: https://github.com/apache/airflow/pull/25185#discussion_r927104026
##
airflow/providers/microsoft/mssql/hooks/mssql.py:
##
@@ -51,6 +51,13 @@ def get_conn(
)
return conn
+def get_uri(self) -> str:
+from
potiuk commented on code in PR #25185:
URL: https://github.com/apache/airflow/pull/25185#discussion_r927103463
##
airflow/providers/microsoft/mssql/hooks/mssql.py:
##
@@ -51,6 +51,13 @@ def get_conn(
)
return conn
+def get_uri(self) -> str:
+from
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 17ec6dbcfe Stop SLA callbacks gazumping other callb
potiuk merged PR #25147:
URL: https://github.com/apache/airflow/pull/25147
--
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
potiuk commented on PR #25147:
URL: https://github.com/apache/airflow/pull/25147#issuecomment-1191932877
Merging now - unless someone will have objections - it's simple enough
change to be reverted/improved on in case.
--
This is an automated message from the Apache Git Service.
To respon
potiuk commented on PR #25215:
URL: https://github.com/apache/airflow/pull/25215#issuecomment-1191932073
@uranusjr ? does it make sense?
--
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 specif
potiuk closed issue #25211: Airflow scheduler spawn zombie process
URL: https://github.com/apache/airflow/issues/25211
--
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 unsubs
potiuk commented on PR #25183:
URL: https://github.com/apache/airflow/pull/25183#issuecomment-1191927935
You should add unit tests covering it.
--
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 issue #25207:
URL: https://github.com/apache/airflow/issues/25207#issuecomment-1191926593
Also to add - the very important part of this is security. We are working on
improving the security of Airflow and we are gradually fixing all the potential
places where - especiall
ferruzzi opened a new pull request, #25224:
URL: https://github.com/apache/airflow/pull/25224
Follows the template set forward in
https://github.com/apache/airflow/pull/24643
Also cleans up and makes some required changes within the
SystemTestContextBuilder to account for occasionall
potiuk closed issue #25209: Airflow database migration version scripts are
unsafe, have failed, and are guaranteed to fail in the future
URL: https://github.com/apache/airflow/issues/25209
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
potiuk commented on issue #25209:
URL: https://github.com/apache/airflow/issues/25209#issuecomment-1191915218
@vitaly-krugl - if you would like to propose some guidelines - feel free to
make PR. Like everything else in Airflow.,
Airflow is created by > 2100 contributors and if you wou
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 f6bda38e20 Convert RDS Export Sample DAG to System
potiuk merged PR #25205:
URL: https://github.com/apache/airflow/pull/25205
--
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
bbovenzi merged PR #25208:
URL: https://github.com/apache/airflow/pull/25208
--
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.
bbovenzi pushed a change to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
from 93b7a57cbc Include missing mention of `external_executor_id` in
`sql_engine_collation_for_ids` docs (#25197)
ad
potiuk commented on PR #25135:
URL: https://github.com/apache/airflow/pull/25135#issuecomment-1191905182
@chethanuk-plutoflume - I am not sure what the problem is - but the
instructions are pretty self-explanatory. Am I missing something?
--
This is an automated message from the Apache G
potiuk commented on PR #25135:
URL: https://github.com/apache/airflow/pull/25135#issuecomment-1191904426
Just copying the exact instructions:
```
Make sure to add those lines to
docs/apache-airflow-providers-airbyte/index.rst:
.. toctree::
:hidden:
:caption
potiuk merged PR #25197:
URL: https://github.com/apache/airflow/pull/25197
--
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 93b7a57cbc Include missing mention of `external_exe
boring-cyborg[bot] commented on PR #25197:
URL: https://github.com/apache/airflow/pull/25197#issuecomment-1191903543
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 on to GitHub
potiuk commented on PR #25218:
URL: https://github.com/apache/airflow/pull/25218#issuecomment-1191897907
:)
--
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,
potiuk commented on issue #25186:
URL: https://github.com/apache/airflow/issues/25186#issuecomment-1191885294
100% on board.
--
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.
dstandish commented on issue #25186:
URL: https://github.com/apache/airflow/issues/25186#issuecomment-1191854525
I am actually thinking now that we should abandon URI as the identifier for
datasets. I think part of the motivation for choosing URI was [this document
in
openlineage](https:/
boring-cyborg[bot] commented on issue #25222:
URL: https://github.com/apache/airflow/issues/25222#issuecomment-1191837770
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 mess
mikebell90 opened a new issue, #25222:
URL: https://github.com/apache/airflow/issues/25222
### Description
Our engineers want to run jobs in a separate namespace from the installed
release via helm chart.
It looks to me like right now the rbac configuration has two choices
This is an automated email from the ASF dual-hosted git repository.
potiuk pushed a commit to branch v2-3-test
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/v2-3-test by this push:
new f431503dc3 Temporary remove common.sql an
This is an automated email from the ASF dual-hosted git repository.
jedcunningham pushed a change to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
from 47b72056c4 SQSPublishOperator should allow sending messages to a FIFO
Queue (#25171)
add 03a81b66de Show d
jedcunningham closed issue #25033: next run should show deps fulfillment e.g. 0
of 3
URL: https://github.com/apache/airflow/issues/25033
--
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
jedcunningham merged PR #25141:
URL: https://github.com/apache/airflow/pull/25141
--
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
erdos2n commented on issue #23246:
URL: https://github.com/apache/airflow/issues/23246#issuecomment-1191795657
unnassigning myself, as I am unable to get the tests to pass
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and us
gitstart-airflow opened a new pull request, #25221:
URL: https://github.com/apache/airflow/pull/25221
This PR fixes the error raised on DateTimeTrigger when TimeSensorAsync
target_time is timezone-aware.
closes: #24736
related: #24736
---
**^ Add meaningful description ab
denimalpaca commented on code in PR #25164:
URL: https://github.com/apache/airflow/pull/25164#discussion_r926969512
##
airflow/providers/common/sql/operators/sql.py:
##
@@ -73,6 +79,7 @@ class SQLColumnCheckOperator(BaseSQLOperator):
}
}
+:param batch
eladkal commented on PR #25196:
URL: https://github.com/apache/airflow/pull/25196#issuecomment-1191778707
I don't think this is right (or at least not valid to all backends)
I was able to create a DAG with tag longer than 100 chars with Sqlite
(didn't test the others)
.
--
This is
potiuk commented on PR #25219:
URL: https://github.com/apache/airflow/pull/25219#issuecomment-1191641848
> Eh, I would prefer to just implement Prometheus/OpenMetrics native
metrics. ;-)
>
> I'm not really a fan of OpenTelemetry.
Why ? Srsly - we have not yet implemented it but
1 - 100 of 214 matches
Mail list logo
