Skip to content

A3 Mega Slurm: fix package URL after Ansible upgrade in #4167#4176

Merged
tpdownes merged 1 commit into
GoogleCloudPlatform:developfrom
tpdownes:fix_package
May 22, 2025
Merged

A3 Mega Slurm: fix package URL after Ansible upgrade in #4167#4176
tpdownes merged 1 commit into
GoogleCloudPlatform:developfrom
tpdownes:fix_package

Conversation

@tpdownes

Copy link
Copy Markdown
Contributor

The formatting of the fact for ansible_distribution_version changed from 12 to 12.11 for Debian 12.11. This change addresses that, but should not be replicated in Ubuntu solutions where ansible_distribution_version retains the correct formatting for the URL being targeted.

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 #

@tpdownes tpdownes merged commit 24645ff into GoogleCloudPlatform:develop May 22, 2025
13 of 65 checks passed
@tpdownes tpdownes deleted the fix_package branch May 22, 2025 18:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release-chore To not include into release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants