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

Add support to install on Ubuntu 24.04 Noble Numbat #2001

Closed
wants to merge 1 commit into from

Conversation

sblaisot
Copy link
Contributor

What does this PR do?

This PR add support for salt 3007 installation as onedir or stable package on Ubuntu 24.04 Noble Numbat using salt's Ubuntu 24.04 repositories.

It currently only support salt 3007 as other versions are not available in saltstack's Ubuntu 24.04 repos

Previous Behavior

Installation failure on Ubuntu 24.04

New Behavior

Installation works on Ubuntu 24.04

@sblaisot
Copy link
Contributor Author

@dmurphy18 any news on this?

@EHJ-52n
Copy link
Contributor

EHJ-52n commented Jun 17, 2024

I would recommend to change the default version to the latest supported, hence noble and not trusty.

        *)
            DISTRO_CODENAME="noble"
            ;;

@sblaisot
Copy link
Contributor Author

I would recommend to change the default version to the latest supported, hence nobel and not trusty.

        *)
            DISTRO_CODENAME="noble"
            ;;

This is unrelated to this PR and subject to various side-effects I can not anticipate so I would prefer this to be done in another PR.

@dmurphy18
Copy link
Contributor

Work is ongoing to add support for Ubuntu 24.04 in #1987.
Don't want to duplicate work and testing.

@Foorack
Copy link

Foorack commented Jun 20, 2024

Work is ongoing to add support for Ubuntu 24.04 in #1987. Don't want to duplicate work and testing.

Maybe, although the changes in that PR are quite substansive and bigger in scope. Considering Ubuntu 24.04 has been released for two months now, it would be nice to support it as quick as possible.

@dmurphy18
Copy link
Contributor

@Foorack Given the recent reduction in resources (personal) since the Broadcom acquisition, time has become limited, and I am close with the PR under development. Given the bootstrap script is a simple bash script, it would be easy to change the selection of 'trusty' to 'noble' in your copy as a work-around.

The Salt Core don't have unlimited time, hence concentration on PR's under development at the moment.

@Kenneth-Sills
Copy link

I'm inclined to agree with @Foorack, salt bootstrapping is broken for common Ubuntu and Debian LTS setups and has been for months. The refactor is going to be great and is much needed for sure, but these changes are a couple of lines while the refactor is thousands of lines of changes and AFAICT hasn't started a real review yet.

With limited resources, it seems like it would make sense to take care of the bite-size fix PRs since that larger refactor and review could take weeks or months yet to see release, and may need its own patch cycles to see stability.

I hope I don't come off as argumentative, we obviously appreciate the hard work you're putting in and understand the struggle you're navigating! And, naturally, I'm biased given #1995. 😆

@dmurphy18
Copy link
Contributor

@Kenneth-Sills @Foorack @sblaisot Re-working the PR to not remove kitchen, so that I can get this out quicker.
Note: even the develop branch has lint issues, so not so simple as just adding 'noble' for Ubuntu 24.04, The develop branch needs work too.

Working through test failures on #1987 at the moment, so soon.

@dmurphy18
Copy link
Contributor

@sblaisot Closing this PR since PR #1987 has been merged which supports Ubuntu 24.04, along with fixing a number of other issues with the bootstrap script on the develop branch. A new release has been cut and is in stable (2024-07-12) and waiting for the web site to sync up at the moment from the S3 bucket, see https://github.com/saltstack/salt-bootstrap/actions/runs/9910182797.

If there are any issues with closing this PR please let me know and it can be re-opened or a new issue generated to resolve the issue.

@dmurphy18 dmurphy18 closed this Jul 12, 2024
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.

5 participants