Skip to content

Remove Docker configuration warning#4229

Merged
tpdownes merged 1 commit into
GoogleCloudPlatform:developfrom
tpdownes:remove_docker_config_warning
Jun 3, 2025
Merged

Remove Docker configuration warning#4229
tpdownes merged 1 commit into
GoogleCloudPlatform:developfrom
tpdownes:remove_docker_config_warning

Conversation

@tpdownes

@tpdownes tpdownes commented Jun 2, 2025

Copy link
Copy Markdown
Contributor

With the EOL of Ubuntu 20.04 and #4167, we can now use built-in Ansible features to validate Docker configurations and fail explicitly. This removes a generic warning which the UI presents very similarly to an error.

Submission Checklist

NOTE: Community submissions can take up to 2 weeks to be reviewed.

Please take the following actions before submitting this pull request.

  • Fork your PR branch from the Toolkit "develop" branch (not main)
  • Test all changes with pre-commit in a local branch #
  • Confirm that "make tests" passes all tests
  • Add or modify unit tests to cover code changes
  • Ensure that unit test coverage remains above 80%
  • Update all applicable documentation
  • Follow Cluster Toolkit Contribution guidelines #

With the EOL of Ubuntu 20.04 and GoogleCloudPlatform#4167, we can now use built-in Ansible
features to validate Docker configurations and fail explicitly. This
removes a generic warning which the UI presents very similarly to an
error.
@tpdownes tpdownes requested a review from alyssa-sm June 2, 2025 14:55
@tpdownes tpdownes requested review from a team and samskillman as code owners June 2, 2025 14:55
@tpdownes tpdownes added the release-improvements Added to release notes under the "Improvements" heading. label Jun 2, 2025
@tpdownes tpdownes enabled auto-merge June 2, 2025 16:43
@tpdownes tpdownes merged commit 5183c47 into GoogleCloudPlatform:develop Jun 3, 2025
15 of 67 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release-improvements Added to release notes under the "Improvements" heading.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants