potiuk commented on code in PR #23395:
URL: https://github.com/apache/airflow/pull/23395#discussion_r862520564
##########
dev/breeze/src/airflow_breeze/shell/enter_shell.py:
##########
@@ -17,57 +17,21 @@
"""Command to enter container shell for Breeze."""
import subprocess
import sys
-from pathlib import Path
-from typing import Dict, Optional, Union
+from typing import Optional, Union
-from airflow_breeze import global_constants
-from airflow_breeze.build_image.ci.build_ci_image import build_ci_image
-from airflow_breeze.build_image.ci.build_ci_params import BuildCiParams
from airflow_breeze.shell.shell_params import ShellParams
-from airflow_breeze.utils.cache import (
- read_and_validate_value_from_cache,
- read_from_cache_file,
- write_to_cache_file,
-)
+from airflow_breeze.utils.cache import read_from_cache_file
from airflow_breeze.utils.console import get_console
from airflow_breeze.utils.docker_command_utils import (
- SOURCE_OF_DEFAULT_VALUES_FOR_VARIABLES,
- VARIABLES_IN_CACHE,
- check_docker_compose_version,
check_docker_is_running,
check_docker_resources,
- check_docker_version,
construct_env_variables_docker_compose_command,
)
-from airflow_breeze.utils.path_utils import BUILD_CACHE_DIR
+from airflow_breeze.utils.rebuild_image_if_needed import
rebuild_ci_image_if_needed
from airflow_breeze.utils.run_utils import filter_out_none, run_command
from airflow_breeze.utils.visuals import ASCIIART, ASCIIART_STYLE, CHEATSHEET,
CHEATSHEET_STYLE
-def synchronize_cached_params(parameters_passed_by_the_user: Dict[str, str])
-> Dict[str, str]:
Review Comment:
This is not needed any more at shell entry.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
