Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ubuntu 24.04 Support for BOLT #3376

Draft
wants to merge 7 commits into
base: main
Choose a base branch
from
Draft

Ubuntu 24.04 Support for BOLT #3376

wants to merge 7 commits into from

Conversation

alex501212
Copy link
Contributor

This pull request primarily focuses on ubuntu24 support for Bolt

Issue

The issue with ubuntu24 is that when running bolt apply the version of puppet-agent that is being pulled is puppet-release-noble.deb and not puppet8-release-noble.deb although other OSes like ubuntu22 work fine, In the case of Ubuntu24 the puppet-release version causes bolt apply to fail.

Solution

My solution is to when running an apply task to first check if the user has specified a collection as an argument in the cli using collection=<COLLECTION> and if not use Puppet.version to check what version of puppet bolt is using so specifically the puppet-agent install task doesn't default to 'puppet' (pulling the latest ubuntu release) see: install_shell.sh and if all fails default to the latest 'puppet' release version.

Testing

I ran acceptance tests with ubuntu2404-64 as the controller and ubuntu2404-64, redhat9-64, fedora40-64 and windows10ent-64 as the nodes and all tests are passing:

Screenshot 2025-02-25 at 12 34 35 pm

Additional notes

I have also updated all GitHub workflows to use ubuntu-24.04 instead of ubuntu-22.04.

@donoghuc
Copy link
Contributor

IMO this is the wrong approach. This target-specific logic does not belong in apply_prep.

* **Added Ubuntu24 Support**

  Added support for Ubuntu-24.04-x64
@alex501212 alex501212 requested a review from a team February 25, 2025 16:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants