Skip to content
New issue

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

#

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

Already on GitHub? # to your account

Jump to bottom

In worker Operator #60

Draft
wants to merge 6 commits into
base: main
Choose a base branch
from
Draft

In worker Operator #60

wants to merge 6 commits into from

Conversation

luiztauffer
Copy link
Member

This PR takes the necessary steps towards Fix #49

  • new column piece_execution_mode for PieceRepository table
  • make migrations for database model changes
  • add rest endpoints for getting, creating, deleting and editing Airflow Connections
  • add DominoBackendRestClient.check_create_airflow_connection() function
  • DominoWorkerOperator
    • activate DominoWorkerOperator when execution_mode==worker
    • load Piece code from tar file (or similar method), the Piece should require no installation in the worker environment
    • get secrets with DominoBackendRestClient
    • get conn_id with DominoBackendRestClient

@luiztauffer luiztauffer added enhancement New feature or request REST REST API related issues domino-py Domino Python package related issues labels Jul 18, 2023
@luiztauffer luiztauffer mentioned this pull request Sep 26, 2023
Closed
@luiztauffer luiztauffer changed the base branch from main to dev November 13, 2023 22:22
Base automatically changed from dev to main November 27, 2023 11:35
@luiztauffer luiztauffer mentioned this pull request Apr 19, 2024
Open
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
domino-py Domino Python package related issues enhancement New feature or request REST REST API related issues
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[feature] Run Pieces in-worker
1 participant
@luiztauffer