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

DAT-19348 DevOps :: refactor databricks test-harness workflow to run terraform once #244

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

Conversation

jandroav
Copy link
Contributor

This pull request includes significant changes to the GitHub Actions workflow configuration in the .github/workflows/lth.yml file. The changes primarily involve restructuring the jobs for better organization and dependency management.

Key changes include:

  • Job Renaming and Reorganization:

    • Renamed the liquibase-test-harness job to start-test-harness-infra and updated its name to "Start Liquibase Test Harness infra."
    • Moved the liquibase-test-harness job to run after start-test-harness-infra and updated its configuration.
  • Environment Variables and Permissions:

    • Removed the environment variables WORKSPACE_ID, LIQUIBOT_TOKEN, and GITHUB_TOKEN from the start-test-harness-infra job.
    • Added the removed environment variables and permissions back to the newly defined liquibase-test-harness job.
  • Job Dependencies and Cleanup:

    • Added a new job stop-test-harness-infra to stop the test harness infrastructure after the liquibase-test-harness job completes.

…nd organization by renaming jobs and adding dependencies

♻️ (lth.yml): separate infrastructure setup and teardown into distinct jobs for better management and readability
💡 (lth.yml): add comments to clarify the purpose of each job and step in the workflow
…nection

♻️ (lth.yml): refactor Databricks config collection step for better clarity
🐛 (lth.yml): fix reference to DATABRICKS_URL in Liquibase test harness command
📝 (lth.yml): remove redundant stop test database logic for cleaner workflow
…ne workflow

✨ (lth.yml): update needs syntax for stop-test-harness-infra job for clarity and consistency
…the process of setting DATABRICKS_URL in GitHub Actions workflow
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.

1 participant