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.

Sign up for GitHub

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

Jump to bottom

Deprecate sftp async provider #1419

Closed
1 task
Lee-W opened this issue Jan 2, 2024 · 1 comment · Fixed by #1470
Closed
1 task

Deprecate sftp async provider #1419

Lee-W opened this issue Jan 2, 2024 · 1 comment · Fixed by #1470
Assignees
@pankajkoti @utkarsharma2
Labels
feature New feature or request
Milestone
1.19.0

Comments

@Lee-W
Copy link
Contributor

Lee-W commented Jan 2, 2024 •
edited by pankajkoti
Loading

Is your feature request related to a problem? Please describe.

as most of the functionality has been contributed back to OSS airflow, we're going to leverage the code there and deprecate this project

Describe the solution you'd like

  1. Contribute the delta to the OSS provider
  2. Point the astronomer provider to the OSS provider
  3. Do the release and wait for a month or so before we remove the astronomer provider in a major release

Describe alternatives you've considered

Additional context

Current Progress

  • Number of operator changes on OSS airflow needed: 1
  • Number of operators to deprecate: 1
  1. SFTPSensorAsync
@Lee-W Lee-W added the feature New feature or request label Jan 2, 2024
@Lee-W Lee-W self-assigned this Jan 2, 2024
@Lee-W Lee-W changed the title Drepcreate sftp provider Deprecate sftp async provider Jan 2, 2024
@vatsrahul1001 vatsrahul1001 assigned vatsrahul1001 and unassigned Lee-W Jan 18, 2024
@Lee-W Lee-W added this to the 1.19.0 milestone Jan 19, 2024
@pankajkoti pankajkoti mentioned this issue Feb 2, 2024
Merged
@pankajkoti pankajkoti self-assigned this Feb 2, 2024
@pankajkoti
Copy link
Collaborator

Waiting for OSS provider release

pankajkoti added a commit that referenced this issue Feb 16, 2024
@pankajkoti
Deprecate async SFTP sensor (#1470)
8b691f1 
Deprecate SFTPSensorAsync and proxy it to its Airflow
OSS provider's counterpart

closes: #1419
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@pankajkoti pankajkoti

@utkarsharma2 utkarsharma2

Labels
feature New feature or request
Projects
None yet
Milestone
1.19.0
Development

Successfully merging a pull request may close this issue.

Deprecate async SFTP sensor astronomer/astronomer-providers
4 participants
@Lee-W @pankajkoti @utkarsharma2 @vatsrahul1001