Skip to content

Feature/dags reserialize core api 1 #1

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

Conversation

BharatPratap
Copy link

What this PR will resolve?
The GitHub issue [#47844](apache#47844) proposes adding a REST API endpoint to Apache Airflow to programmatically trigger the DAG reserialization process, which is currently only accessible via the CLI command airflow dags reserialize.

Summary of the Issue

Steps to Resolve

  1. Design the API Endpoint: Define the API route, HTTP method (likely POST), and request/response schema for triggering DAG reserialization.

  2. Implement the Endpoint: Develop the backend logic to handle the API request and invoke the DAG reserialization process.

  3. Security Considerations: Ensure that only authorized users can access this endpoint, possibly by integrating with Airflow's existing authentication and authorization mechanisms.

  4. Testing: Write unit and integration tests to validate the functionality and security of the new API endpoint.

  5. Documentation: Update the Airflow API documentation to include the new endpoint, detailing its usage and any necessary permissions.

Status

As of the latest update, a pull request [#48174](apache#48174) has been submitted to address this feature request. Reviewing and merging this PR would implement the proposed API endpoint, resolving the issue.


^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in airflow-core/newsfragments.

@BharatPratap
Copy link
Author

@coderabbitai review

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants