Skip to content

fix: correct indentation in utils.py and add docker compose validation#14168

Merged
xushiyan merged 1 commit intoapache:masterfrom
deepakpanda93:br_fix_docker_notebooks
Oct 28, 2025
Merged

fix: correct indentation in utils.py and add docker compose validation#14168
xushiyan merged 1 commit intoapache:masterfrom
deepakpanda93:br_fix_docker_notebooks

Conversation

@deepakpanda93
Copy link
Collaborator

Describe the issue this Pull Request addresses

This PR addresses two issues:

  • Python Indentation Error : Fixed inconsistent indentation (mixed tabs and spaces) in utils.py which caused an IndentationError at runtime.
  • Docker Compose Compatibility : Added a validation function in the shell script to dynamically detect and use the correct Docker Compose command (docker compose or docker-compose) based on availability.

Summary and Changelog

  • Fixed indentation in /opt/notebooks/utils.py to use 4-space indentation consistently.
  • Added a validation function in the shell script to dynamically detect and use the correct Docker Compose command (docker compose or docker-compose) based on availability.

Impact

None

Risk Level

none

Documentation Update

None required.

Contributor's checklist

  • Read through contributor's guide
  • Enough context is provided in the sections above
  • Adequate tests were added if applicable

@github-actions github-actions bot added the size:S PR with lines of changes in (10, 100] label Oct 28, 2025
@hudi-bot
Copy link
Collaborator

CI report:

Bot commands @hudi-bot supports the following commands:
  • @hudi-bot run azure re-run the last Azure build

Copy link
Collaborator

@rangareddy rangareddy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This PR is clean and ready to go.

@xushiyan xushiyan marked this pull request as ready for review October 28, 2025 15:11
@xushiyan xushiyan merged commit 7f72c2b into apache:master Oct 28, 2025
70 checks passed
@xushiyan xushiyan changed the title feat: correct indentation in utils.py and add docker compose validation fix: correct indentation in utils.py and add docker compose validation Oct 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:S PR with lines of changes in (10, 100]

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants