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

Salt upgrade #129

Open
wants to merge 17 commits into
base: integration
Choose a base branch
from
Open

Salt upgrade #129

wants to merge 17 commits into from

Conversation

dxenes1
Copy link
Contributor

@dxenes1 dxenes1 commented Nov 21, 2024

This PR is ready for review.

Changes:

  • Updates salt bootstrap to support latest Salt LTS
  • Directly specifies system python for all pip installs. WARNING: This is probably the last python version we can do this with, as system installs are not recommended and usually are blocked from python 3.12 onwards.
  • Removes editable install for heaviside
  • Upgrades base AMI to ubuntu 22.04 (python 3.10)
  • Installs uwsgi via apt-get instead of pip due to issues with setuptools

@sandyhider sandyhider changed the base branch from integration to py311 November 25, 2024 17:36
@sandyhider sandyhider changed the base branch from py311 to integration November 25, 2024 17:37
Copy link
Member

@sandyhider sandyhider Dec 4, 2024

Choose a reason for hiding this comment

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

I am wondering if we should be using 3006. This is what ChatGPT says about versions:

SaltStack adheres to a [CalVer](https://calver.org/) versioning scheme, where:

Versions are formatted as YYYY.MM (e.g., 3006 corresponds to a release in 2023, June).
Odd-numbered versions (e.g., 3007) are feature releases, introducing new functionality.
Even-numbered versions (e.g., 3006) are LTS (Long-Term Support) releases, focusing on stability and bug fixes.
Key Differences
Purpose of Releases:

Odd Versions (Feature Releases):
Introduce new features and enhancements.
Serve as a preview for the next stable LTS release.
May include experimental features or breaking changes.
Typically have shorter support lifespans.
Even Versions (LTS Releases):
Focus on stability, reliability, and performance.
Incorporate features introduced in the previous odd release, but fully tested and refined.
Receive long-term updates for security and critical bug fixes.
Stability:

LTS versions (even) are better suited for production environments where stability is critical.
Feature releases (odd) are more appropriate for development, testing, or environments where early adoption of new features is desired.
Support Lifecycle:

Odd-numbered versions have a shorter support window, typically ending shortly after the release of the next LTS version.
Even-numbered (LTS) versions are supported for an extended period, ensuring long-term stability for enterprise deployments.```

What do you think?

Copy link
Member

@sandyhider sandyhider left a comment

Choose a reason for hiding this comment

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

What takes it Python 3.11 I wasn't sure from the review?

@dxenes1
Copy link
Contributor Author

dxenes1 commented Dec 4, 2024

It's the ubuntu version,
20.04 comes with python 3.8
22.04 comes with python 3.10
24.04 comes with python 3.12

22.04 works best because python 3.12 doesn't allow system pip installs

@dxenes1 dxenes1 requested a review from movestill December 5, 2024 15:33
@@ -6,7 +6,7 @@ include:
pyminifier:
pip.installed:
- name: pyminifier==2.1
- bin_env: /usr/local/bin/pip3
- pip_bin: /usr/bin/pip3
- require:
- sls: python.python35
Copy link
Member

Choose a reason for hiding this comment

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

These v3.5 requirements aren't causing problems?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ah I missed that require, looks like the pyminifier salt file is only used in the "workstation" packer AMI though so it might not be executed when creating the core AMIs for the stack...Not sure what "workstation" is for? A developer environment I'm guessing?

    'workstation*':
        - python.python35
        - git
        - vault.client
        - aws.boto3
        - python.pyminifier

@movestill
Copy link
Member

movestill commented Dec 10, 2024 via email

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.

3 participants