Skip to content

Commit 442177b

Browse files
alafanecherealafanechere
authored
airbyte-ci: temporarily disable python dependency upload on publish (#36961)
1 parent 09c59d0 commit 442177b

File tree

3 files changed

+11
-8
lines changed

3 files changed

+11
-8
lines changed

‎airbyte-ci/connectors/pipelines/README.md

+5-4
Original file line numberDiff line numberDiff line change
@@ -295,14 +295,14 @@ flowchart TD
295295
#### Options
296296

297297
| Option | Multiple | Default value | Description |
298-
| ------------------------------------------------------- | -------- | ------------- | -------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- | ------------------------------------------------------------------------------------------- |
298+
| ------------------------------------------------------- | -------- | ------------- | -------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- |
299299
| `--skip-step/-x` | True | | Skip steps by id e.g. `-x unit -x acceptance` |
300300
| `--only-step/-k` | True | | Only run specific steps by id e.g. `-k unit -k acceptance` |
301301
| `--fail-fast` | False | False | Abort after any tests fail, rather than continuing to run additional tests. Use this setting to confirm a known bug is fixed (or not), or when you only require a pass/fail result. |
302302
| `--code-tests-only` | True | False | Skip any tests not directly related to code updates. For instance, metadata checks, version bump checks, changelog verification, etc. Use this setting to help focus on code quality during development. |
303303
| `--concurrent-cat` | False | False | Make CAT tests run concurrently using pytest-xdist. Be careful about source or destination API rate limits. |
304304
| `--<step-id>.<extra-parameter>=<extra-parameter-value>` | True | | You can pass extra parameters for specific test steps. More details in the extra parameters section below |
305-
| `--ci-requirements` | False | | | Output the CI requirements as a JSON payload. It is used to determine the CI runner to use. |
305+
| `--ci-requirements` | False | | | Output the CI requirements as a JSON payload. It is used to determine the CI runner to use.
306306

307307
Note:
308308

@@ -649,9 +649,10 @@ E.G.: running Poe tasks on the modified internal packages of the current branch:
649649

650650
| Version | PR | Description |
651651
| ------- | ---------------------------------------------------------- | -------------------------------------------------------------------------------------------------------------------------- |
652-
| 4.7.0 | [#36892](https://github.com/airbytehq/airbyte/pull/36892) | Upload Python connectors dependencies list to GCS on publish. |
652+
| 4.7.1 | [#36961](https://github.com/airbytehq/airbyte/pull/36961) | Temporarily disable python connectors dependencies upload until we find a schema the data team can work with. |
653+
| 4.7.0 | [#36892](https://github.com/airbytehq/airbyte/pull/36892) | Upload Python connectors dependencies list to GCS on publish. |
653654
| 4.6.5 | [#36722](https://github.com/airbytehq/airbyte/pull/36527) | Fix incorrect pipeline names |
654-
| 4.6.4 | [#36480](https://github.com/airbytehq/airbyte/pull/36480) | Burst the Gradle Task cache if a new CDK version was released |
655+
| 4.6.4 | [#36480](https://github.com/airbytehq/airbyte/pull/36480) | Burst the Gradle Task cache if a new CDK version was released |
655656
| 4.6.3 | [#36527](https://github.com/airbytehq/airbyte/pull/36527) | Handle extras as well as groups in `airbyte ci test` [poetry packages] |
656657
| 4.6.2 | [#36220](https://github.com/airbytehq/airbyte/pull/36220) | Allow using `migrate-to-base-image` without PULL_REQUEST_NUMBER |
657658
| 4.6.1 | [#36319](https://github.com/airbytehq/airbyte/pull/36319) | Fix `ValueError` related to PR number in migrate-to-poetry |

‎airbyte-ci/connectors/pipelines/pipelines/airbyte_ci/connectors/publish/pipeline.py

+5-3
Original file line numberDiff line numberDiff line change
@@ -365,9 +365,11 @@ def create_connector_report(results: List[StepResult]) -> ConnectorReport:
365365
if build_connector_results.status is not StepStatus.SUCCESS:
366366
return create_connector_report(results)
367367

368-
if context.connector.language in [ConnectorLanguage.PYTHON, ConnectorLanguage.LOW_CODE]:
369-
upload_dependencies_step = await UploadDependenciesToMetadataService(context).run(build_connector_results.output)
370-
results.append(upload_dependencies_step)
368+
# Temporarily disabling it until we find the correct schema that the data team can ingest
369+
370+
# if context.connector.language in [ConnectorLanguage.PYTHON, ConnectorLanguage.LOW_CODE]:
371+
# upload_dependencies_step = await UploadDependenciesToMetadataService(context).run(build_connector_results.output)
372+
# results.append(upload_dependencies_step)
371373

372374
built_connector_platform_variants = list(build_connector_results.output.values())
373375
push_connector_image_results = await PushConnectorImageToRegistry(context).run(built_connector_platform_variants)

‎airbyte-ci/connectors/pipelines/pyproject.toml

+1-1
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ build-backend = "poetry.core.masonry.api"
44

55
[tool.poetry]
66
name = "pipelines"
7-
version = "4.7.0"
7+
version = "4.7.1"
88
description = "Packaged maintained by the connector operations team to perform CI for connectors' pipelines"
99
authors = ["Airbyte <[email protected]>"]
1010

0 commit comments

Comments
 (0)