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

[Databricks E2E] Dynamic ddo transform wheel Build #1083

Open
wants to merge 1 commit into
base: e2e/databricks/parking-sensors-V1
Choose a base branch
from

Conversation

raafatzarka
Copy link
Contributor

@raafatzarka raafatzarka commented Feb 5, 2025

Type of PR

  • Code changes

Purpose

Alter the deployment code to build the whl package at deploy time and remove it locally

Does this introduce a breaking change? If yes, details on what can break

Author pre-publish checklist

  • Added test to prove my fix is effective or new feature works
  • No PII in logs
  • Made corresponding changes to the documentation

Validation steps

  • Run the deployment. Check that WHL file has been copied successfully to Azure Databricks under DDO Cluster -> libraries (dbfs:/ddo_transform-localdev-py2.py3-none-any.whl)
  • Make sure the deployment runs successfully.

Issues Closed or Referenced

@raafatzarka raafatzarka self-assigned this Feb 5, 2025
@raafatzarka raafatzarka changed the base branch from e2e/databricks/parkingsensor_v1_liliam_1042 to e2e/databricks/parking-sensors-V1 February 5, 2025 18:14
@raafatzarka raafatzarka requested a review from hemaverma February 5, 2025 18:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant