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

Upgrade to latest Salt 3006.x for all provisioning #1845

Merged
merged 1 commit into from
Apr 30, 2024

Conversation

ScriptAutomate
Copy link
Contributor

@ScriptAutomate ScriptAutomate commented Mar 25, 2024

  • Upgrade to latest Salt v3006.x for all Salt provisioning
    • Included upgrading the salt provisioning scripts to point to the official repo download location of onedir archives
    • Went with Salt v3006.8 due to Windows-related bug(s) being fixed
  • Upgrades base Python 3.8.x to Python 3.10.x on Windows operating systems
    • Required an upgrade to pyyaml to avoid cython attribute errors
  • Fix Photon OS 4 errors caused by GPG key expirations

@ScriptAutomate
Copy link
Contributor Author

Looks like an issue that may be a bug in newer versions of Salt running on Windows:

[ERROR   ][salt.loaded.int.module.win_pkg:1714] Unable to cache file salt://win/repo-ng/salt-winrepo-ng/cwrsync/install.cmd from saltenv: base
[ERROR   ][salt.state:325 ] {'cwrsync': {'unable to cache': 'salt://win/repo-ng/salt-winrepo-ng/cwrsync/install.cmd'}}

This problem is specific only the cwrsync package, and the state fails on every version of supported Windows. Using an older version of Salt, this issue does not exist:

@twangboy
Copy link
Contributor

May be related to this:
saltstack/salt#66184
I'll take a look

@ScriptAutomate ScriptAutomate force-pushed the upgrade-salt branch 2 times, most recently from 5815fc0 to 5748fba Compare March 27, 2024 16:54
@twangboy
Copy link
Contributor

Actually, it looks like this is the one that broke it.
saltstack/salt#65017
It changed the path that gets created when you run winrepo.update_git_repos.

@ScriptAutomate ScriptAutomate changed the title Upgrade to Salt 3007.0 for all provisioning Upgrade to latest Salt 3006.x for all provisioning Apr 25, 2024
@ScriptAutomate ScriptAutomate force-pushed the upgrade-salt branch 2 times, most recently from 8c032a2 to e983112 Compare April 30, 2024 21:39
@ScriptAutomate ScriptAutomate enabled auto-merge (squash) April 30, 2024 22:37
@ScriptAutomate ScriptAutomate disabled auto-merge April 30, 2024 23:16
@ScriptAutomate ScriptAutomate merged commit 3e54431 into saltstack:main Apr 30, 2024
50 checks passed
@ScriptAutomate ScriptAutomate deleted the upgrade-salt branch April 30, 2024 23:17
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