Skip to content

Remove airflow.utils dependency from Docker provider#69900

Open
shaealh wants to merge 1 commit into
apache:mainfrom
shaealh:shaealh/airflow-62063-docker-utils
Open

Remove airflow.utils dependency from Docker provider#69900
shaealh wants to merge 1 commit into
apache:mainfrom
shaealh:shaealh/airflow-62063-docker-utils

Conversation

@shaealh

@shaealh shaealh commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

Why

Issue #62063 tracks removing provider dependencies on airflow.utils as part of the provider/client separation work. The Docker provider had one remaining dependency in DockerSwarmOperator for generating temporary service names.

What changed

  • Replaced the airflow.utils.strings.get_random_string import with a private provider-local helper.
  • Preserved the existing service-name format: the configured prefix followed by eight alphanumeric characters.
  • Added focused unit coverage for the generated service name.
  • Confirmed that providers/docker/src no longer imports airflow.utils.

This does not introduce any user-facing behavior changes.

Testing

  • Commit and pre-commit checks passed.
  • Docker Swarm operator tests: 25 passed.
  • Full Docker provider suite attempted: 143 passed and 2 skipped; 15 Docker-dependent tests could not run because a local Docker daemon was unavailable.

related: #62063


The Docker provider should not require a core utility import for generating ephemeral service names as provider/client separation progresses in apache#62063.
@shaealh shaealh marked this pull request as ready for review July 15, 2026 03:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants