fix(ci): update PyAirbyte version and add -preview suffix to prod release detection (#70972)
Co-authored-by: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
This commit is contained in:
committed by
GitHub
parent
20fdfc31bf
commit
128e8f7b98
@@ -533,7 +533,9 @@ def generate_and_persist_registry_entry(
|
|||||||
|
|
||||||
# For latest versions that are disabled, delete any existing registry entry to remove it from the registry
|
# For latest versions that are disabled, delete any existing registry entry to remove it from the registry
|
||||||
if (
|
if (
|
||||||
"-rc" not in metadata_dict["data"]["dockerImageTag"] and "-dev" not in metadata_dict["data"]["dockerImageTag"]
|
"-rc" not in metadata_dict["data"]["dockerImageTag"]
|
||||||
|
and "-dev" not in metadata_dict["data"]["dockerImageTag"]
|
||||||
|
and "-preview" not in metadata_dict["data"]["dockerImageTag"]
|
||||||
) and not metadata_dict["data"]["registryOverrides"][registry_type]["enabled"]:
|
) and not metadata_dict["data"]["registryOverrides"][registry_type]["enabled"]:
|
||||||
logger.info(
|
logger.info(
|
||||||
f"{registry_type} is not enabled: deleting existing {registry_type} registry entry for {metadata_dict['data']['dockerRepository']} at latest path."
|
f"{registry_type} is not enabled: deleting existing {registry_type} registry entry for {metadata_dict['data']['dockerRepository']} at latest path."
|
||||||
|
|||||||
@@ -25,7 +25,7 @@ from pipelines.helpers.utils import raise_if_not_user
|
|||||||
from pipelines.models.steps import STEP_PARAMS, Step, StepResult
|
from pipelines.models.steps import STEP_PARAMS, Step, StepResult
|
||||||
|
|
||||||
# Pin the PyAirbyte version to avoid updates from breaking CI
|
# Pin the PyAirbyte version to avoid updates from breaking CI
|
||||||
PYAIRBYTE_VERSION = "0.20.2"
|
PYAIRBYTE_VERSION = "0.35.1"
|
||||||
|
|
||||||
|
|
||||||
class PytestStep(Step, ABC):
|
class PytestStep(Step, ABC):
|
||||||
|
|||||||
@@ -111,7 +111,6 @@ class TestUnitTests:
|
|||||||
]
|
]
|
||||||
|
|
||||||
|
|
||||||
@pytest.mark.skip(reason="PyAirbyte validation tests failing due to CDK module import issues in test environment")
|
|
||||||
class TestPyAirbyteValidationTests:
|
class TestPyAirbyteValidationTests:
|
||||||
@pytest.fixture
|
@pytest.fixture
|
||||||
def compatible_connector(self):
|
def compatible_connector(self):
|
||||||
@@ -157,7 +156,8 @@ class TestPyAirbyteValidationTests:
|
|||||||
result = await PyAirbyteValidation(context_for_valid_connector)._run(mocker.MagicMock())
|
result = await PyAirbyteValidation(context_for_valid_connector)._run(mocker.MagicMock())
|
||||||
assert isinstance(result, StepResult)
|
assert isinstance(result, StepResult)
|
||||||
assert result.status == StepStatus.SUCCESS
|
assert result.status == StepStatus.SUCCESS
|
||||||
assert "Getting `spec` output from connector..." in result.stdout
|
# Verify the connector name appears in output (stable across PyAirbyte versions)
|
||||||
|
assert context_for_valid_connector.connector.technical_name in (result.stdout + result.stderr)
|
||||||
|
|
||||||
async def test__run_validation_skip_unpublished_connector(
|
async def test__run_validation_skip_unpublished_connector(
|
||||||
self,
|
self,
|
||||||
|
|||||||
Reference in New Issue
Block a user