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 CICD content verifying and fixing #666

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

cchenshu
Copy link
Contributor

@cchenshu cchenshu commented Feb 5, 2024

Type of PR

  • Code changes
  • Test changes
  • CI-CD changes

Purpose

  • The test failed when using ./devops/notebook-pipelines.yml as build pipeline. Failed at "Execute Nutter".
    Reason: jobs run more than 120s (default timeout)
    Fix: add timeout parameter and set it to 600s

  • For Spark Python Approach, the test failed when using ./devops/spark-python-pipelines.yml as build pipeline. Failed at "Launch integration on test", and the reason is " HTTPError: 400 Client Error: Bad Request for url: Response from server: { 'error_code': 'INVALID_PARAMETER_VALUE','message': 'Invalid python file reference: ''spark_python_jobs/tests/integration/main_test.py.Please visit ''the Databricks user guide for supported python references'}"
    Reason: Databricks versions has updated
    Fix:
    - update deployment json file
    - update project.json file
    - update spark and dbx version
    - update dbx deploy command
    - update dbx launch command

  • Fix typos for DATABRICKS_JON_NAME

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

  • ...

Issues Closed or Referenced

  • Closes #issue_number
  • References #issue_number

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.

None yet

2 participants