Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Convert remaining providers to the new structure #46045

Open
20 of 86 tasks
potiuk opened this issue Jan 25, 2025 · 14 comments
Open
20 of 86 tasks

Convert remaining providers to the new structure #46045

potiuk opened this issue Jan 25, 2025 · 14 comments
Labels
area:providers kind:meta High-level information important to the community

Comments

@potiuk
Copy link
Member

potiuk commented Jan 25, 2025

We have now a number of providers converted to the new structure. We have almost 90 of them to be converted. And we have automated scripts that do bulk of the work. This was part of #44511

Process of the conversion of each of the provider is easy - and described in https://github.com/apache/airflow/blob/main/dev/moving_providers/README.md

The devlist announcement: https://lists.apache.org/thread/dzbj5yx5kwpbwyr5yscp4wnlsp6p9v8l

The list below keeps track of the conversion. The list is generated via:

find providers/src -name 'provider.yaml' | sed 's/providers\/src\/airflow\/providers/ - [ ] /' | sed s'/\/provider.yaml//'  | sor
  • /amazon
  • /apache/beam
  • /apache/cassandra
  • /apache/drill
  • /apache/druid
  • /apache/flink
  • /apache/hdfs
  • /apache/hive
  • /apache/impala
  • /apache/kafka
  • /apache/kylin
  • /apache/livy
  • /apache/pig
  • /apache/pinot
  • /apache/spark
  • /apprise
  • /arangodb
  • /asana
  • /atlassian/jira
  • /cloudant
  • /cncf/kubernetes
  • /cohere
  • /common/compat
  • /common/io
  • /databricks
  • /datadog
  • /dbt/cloud
  • /dingding
  • /discord
  • /docker
  • /elasticsearch
  • /exasol
  • /fab
  • /facebook
  • /ftp
  • /github
  • /google
  • /grpc
  • /hashicorp
  • /http
  • /imap
  • /influxdb
  • /jdbc
  • /jenkins
  • /microsoft/azure
  • /microsoft/mssql
  • /microsoft/psrp
  • /microsoft/winrm
  • /mongo
  • /mysql
  • /neo4j
  • /odbc
  • /openai
  • /openfaas
  • /openlineage
  • /opensearch
  • /opsgenie
  • /oracle
  • /pagerduty
  • /papermill
  • /pgvector
  • /pinecone
  • /postgres
  • /presto
  • /qdrant
  • /redis
  • /salesforce
  • /samba
  • /segment
  • /sendgrid
  • /sftp
  • /singularity
  • /slack
  • /smtp
  • /snowflake
  • /sqlite
  • /ssh
  • /tableau
  • /telegram
  • /teradata
  • /trino
  • /vertica
  • /weaviate
  • /yandex
  • /ydb
  • /zendesk
@potiuk potiuk converted this from a draft issue Jan 25, 2025
@dosubot dosubot bot added area:providers kind:meta High-level information important to the community labels Jan 25, 2025
@rawwar
Copy link
Collaborator

rawwar commented Jan 26, 2025

I'll work on Zendesk, ydb

@Prab-27
Copy link
Contributor

Prab-27 commented Jan 26, 2025

I'm working on Trino, Apprise

@eladkal
Copy link
Contributor

eladkal commented Jan 26, 2025

Lets please make sure we merge open PRs for the providers (if we can) before the conversion.

@vatsrahul1001
Copy link
Collaborator

PR for Weaviate.

@potiuk
Copy link
Member Author

potiuk commented Jan 26, 2025

Lets please make sure we merge open PRs for the providers (if we can) before the conversion.

Yeah before attempting to convert a particular provider It would be great to check if there are outsanding PRs. I am going through the PRs from the last few weeks to see if they apply - but if anyone will want to take on a provider - just checking if there are no related PRs before will do the job :)

@josix
Copy link
Contributor

josix commented Jan 26, 2025

I'm working on openai and mongodb

@potiuk
Copy link
Member Author

potiuk commented Jan 26, 2025

Unfortunately a log of those result in conflicts if they are not done in sequence - so you will have to rebase those that have conflicts :(

@potiuk
Copy link
Member Author

potiuk commented Jan 26, 2025

We really need to batch the providers now - seems that many of them can be fully automated - so I suggest to select a range of those providers (you can provide multiple providers as input to the move script) and submit them together. That will make resolving of the conflict a bit easier.

@rawwar
Copy link
Collaborator

rawwar commented Jan 27, 2025

Working on sqlite, redis, vertica - #46101 - All tests passed

@vatsrahul1001
Copy link
Collaborator

We really need to batch the providers now - seems that many of them can be fully automated - so I suggest to select a range of those providers (you can provide multiple providers as input to the move script) and submit them together. That will make resolving of the conflict a bit easier.

Yes I have merged mysql odbc jenkins pagerduty providers in PR

@amoghrajesh
Copy link
Contributor

There should not really be an issue related to the migration but I'd prefer not batching them. In case there is an issue, reverting the batch would be uglier than individually isolating and debugging

@jason810496
Copy link
Contributor

jason810496 commented Jan 27, 2025

Working on

@potiuk
Copy link
Member Author

potiuk commented Jan 27, 2025

There should not really be an issue related to the migration but I'd prefer not batching them. In case there is an issue, reverting the batch would be uglier than individually isolating and debugging

Yeah. Resolving the conflicts is not that bad and actually the more providers we migrate, the less likely conflicts will be - becase all the conflicting stuff is only when you have adjacent changes in the file - i.e. two providers migrated separatelyt that are "neighbours".

I think our aim should be to migrate providers semi-randomly :)

@Prab-27
Copy link
Contributor

Prab-27 commented Jan 27, 2025

Working on
/exasol
/influxdb
/apache/impala
/dingding
/apache/cassandra
/yandex

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:providers kind:meta High-level information important to the community
Projects
Status: In progress
Development

No branches or pull requests

8 participants