Skip to content

PROD-1658 bug fix cli configure #100

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.

Sign up for GitHub

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

Merged
merged 2 commits into from
Feb 16, 2024
Merged

PROD-1658 bug fix cli configure #100

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
9a88e65
add configuration to init
brandon-kaplan Feb 16, 2024
bbcf517
version bump
brandon-kaplan Feb 16, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion sync/__init__.py
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
"""Library for leveraging the power of Sync"""
__version__ = "1.0.1"
__version__ = "1.0.2"

TIME_FORMAT = "%Y-%m-%dT%H:%M:%SZ"
16 changes: 10 additions & 6 deletions sync/cli/__init__.py
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
"""
Basic Sync CLI
"""

import logging

import click
Expand All @@ -9,7 +10,7 @@
from sync.cli import awsdatabricks, azuredatabricks, projects, workspaces
from sync.cli.util import OPTIONAL_DEFAULT
from sync.clients.sync import get_default_client
from sync.config import API_KEY, DB_CONFIG, APIKey, DatabricksConf, init
from sync.config import API_KEY, CONFIG, DB_CONFIG, APIKey, Configuration, DatabricksConf, init

LOG_FORMAT = "%(asctime)s %(levelname)s [%(name)s] %(message)s"

Expand Down Expand Up @@ -77,11 +78,14 @@ def configure(

init(
APIKey(api_key_id=api_key_id, api_key_secret=api_key_secret),
DatabricksConf(host=dbx_host, token=dbx_token, aws_region_name=dbx_region)
if dbx_host != OPTIONAL_DEFAULT
and dbx_token != OPTIONAL_DEFAULT
and dbx_region != OPTIONAL_DEFAULT
else None,
Configuration(api_url=CONFIG.api_url if CONFIG else "https://api.synccomputing.com"),
(
DatabricksConf(host=dbx_host, token=dbx_token, aws_region_name=dbx_region)
if dbx_host != OPTIONAL_DEFAULT
and dbx_token != OPTIONAL_DEFAULT
and dbx_region != OPTIONAL_DEFAULT
else None
),
)


Expand Down