Skip to content

feat: support custom driver version for local testing of olake changes#321

Merged
vishalm0509 merged 6 commits intostagingfrom
chore/local_test_env
Mar 15, 2026
Merged

feat: support custom driver version for local testing of olake changes#321
vishalm0509 merged 6 commits intostagingfrom
chore/local_test_env

Conversation

@vishalm0509
Copy link
Copy Markdown
Collaborator

@vishalm0509 vishalm0509 commented Feb 13, 2026

Description

The goal of this PR is to bridge the gap between development on the olake core (CLI) and the olake-ui. It introduces a CUSTOM_DRIVER_VERSION configuration that allows developers to build local driver images and use them immediately in the UI without semantic versioning restrictions.

  • Local Image Injection: The value of CUSTOM_DRIVER_VERSION is automatically appended to the versions list for all connectors, making it selectable from the UI.
  • Semver Bypass: Compatibility gates are bypassed when a custom version is active.
  • Workflow Refactoring: Centralized the check logic to ensure developers don't have to manually update version across the codebase when testing new CLI changes or spec/ui-schema changes.

Fixes # (issue)

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

How Has This Been Tested?

  • Scenario A
  • Scenario B

Screenshots or Recordings

Related PR's (If Any):

datazip-inc/olake-docs#351

@vishalm0509 vishalm0509 changed the base branch from master to staging February 13, 2026 08:04
Copy link
Copy Markdown
Collaborator

@deepanshupal09-datazip deepanshupal09-datazip left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I was thinking about how we can simplify the configuration a bit more. Since we already use runmode = localdev for testing local UI changes, what if we use that existing runmode check instead of introducing APP_ENV?

We could also just specify CUSTOM_DRIVER_VERSION directly in app.conf. This way, developers wouldn't have to manage two separate environment modes, and all the local configuration stays in one central place.

What do you think about this approach?

Comment thread server/utils/docker_utils.go Outdated
Comment thread server/utils/docker_utils.go Outdated
Comment thread server/utils/docker_utils.go Outdated
Copy link
Copy Markdown
Collaborator

@deepanshupal09-datazip deepanshupal09-datazip left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@vishalm0509 vishalm0509 merged commit a0e4176 into staging Mar 15, 2026
10 checks passed
@vishalm0509 vishalm0509 deleted the chore/local_test_env branch March 15, 2026 15:04
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