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 highstate generation doesn’t work on Fedora templates #6580

Closed
DemiMarie opened this issue May 4, 2021 · 24 comments · Fixed by QubesOS/qubes-mgmt-salt#28
Closed

Salt highstate generation doesn’t work on Fedora templates #6580

DemiMarie opened this issue May 4, 2021 · 24 comments · Fixed by QubesOS/qubes-mgmt-salt#28
Labels
C: Fedora C: mgmt diagnosed Technical diagnosis has been performed (see issue comments). P: default Priority: default. Default priority for new issues, to be replaced given sufficient information. r4.0-bullseye-stable r4.0-buster-stable r4.0-centos7-stable r4.0-centos8-stable r4.0-dom0-stable r4.0-fc32-stable r4.0-fc33-stable r4.0-stretch-stable r4.1-bullseye-stable r4.1-buster-stable r4.1-centos8-stable r4.1-dom0-stable r4.1-fc31-stable r4.1-fc32-stable r4.1-fc33-stable T: bug Type: bug report. A problem or defect resulting in unintended behavior in something that exists.

Comments

@DemiMarie
Copy link

Qubes OS version

Qubes release 4.0 (R4.0)

Affected component(s) or functionality

mgmt-salt

Brief summary

All attempts to invoke state.highstate on Fedora qubes fail with an error message about rendering top files.

How Reproducible

100%

To Reproduce

Steps to reproduce the behavior:

  1. Upgrade Fedora qube to the latest version, enabling updates-testing for security updates (only).
  2. Run sudo qubesctl --target=fedora-33 --skip-dom0 -- state.highstate
  3. Check the results

Expected behavior

Salt states are run

Actual behavior

An error

Screenshots

Additional context

Solutions you've tried

I have tried modifying my Salt states, with no luck.

Relevant documentation you've consulted

Related, non-duplicate issues

@DemiMarie DemiMarie added T: bug Type: bug report. A problem or defect resulting in unintended behavior in something that exists. P: default Priority: default. Default priority for new issues, to be replaced given sufficient information. labels May 4, 2021
@andrewdavidwong andrewdavidwong added C: mgmt needs diagnosis Requires technical diagnosis from developer. Replace with "diagnosed" or remove if otherwise closed. labels May 4, 2021
@andrewdavidwong andrewdavidwong added this to the Release 4.0 updates milestone May 4, 2021
@unman
Copy link
Member

unman commented May 4, 2021 via email

@DemiMarie
Copy link
Author

Salt does work, as evidenced by fact that GUI updater works for fedora-33. Also, you can state.apply states without issue. It looks as if this error is specific to generating highstate in fedora-33. Is enabling updates-testing important? I think not.

It is, indeed, specific to highstate generation. I have over a thousand lines of Salt last I checked, so this is rather annoying.

@andrewdavidwong andrewdavidwong changed the title Salt doesn’t work on Fedora templates Salt highstate generation doesn’t work on Fedora templates May 5, 2021
@unman
Copy link
Member

unman commented May 6, 2021 via email

@marmarek
Copy link
Member

marmarek commented May 7, 2021

It seems to be this issue: saltstack/salt#60003

I've posted a workaround patch there (to be applied in default-mgmt-dvm's template).

@andrewdavidwong andrewdavidwong added diagnosed Technical diagnosis has been performed (see issue comments). waiting for upstream This issue is waiting for something from an upstream project to arrive in Qubes. Remove when closed. and removed needs diagnosis Requires technical diagnosis from developer. Replace with "diagnosed" or remove if otherwise closed. labels May 7, 2021
@marmarek
Copy link
Member

marmarek commented May 9, 2021

This is specifically regression between salt-3002.6-1.fc33 and salt-3003-1.fc33. A failing test: https://openqa.qubes-os.org/tests/17154#step/TC_10_VMSalt_fedora-33/2

@qubesos-bot
Copy link

Automated announcement from builder-github

The package qubes-mgmt-salt-4.1.11-1.fc32 has been pushed to the r4.1 testing repository for dom0.
To test this update, please install it with the following command:

sudo qubes-dom0-update --enablerepo=qubes-dom0-current-testing

Changes included in this update

@qubesos-bot
Copy link

Automated announcement from builder-github

The package mgmt-salt has been pushed to the r4.1 testing repository for the CentOS centos8 template.
To test this update, please install it with the following command:

sudo yum update --enablerepo=qubes-vm-r4.1-current-testing

Changes included in this update

@qubesos-bot
Copy link

Automated announcement from builder-github

The package qubes-mgmt-salt_4.1.11-1 has been pushed to the r4.1 testing repository for the Debian template.
To test this update, first enable the testing repository in /etc/apt/sources.list.d/qubes-*.list by uncommenting the line containing buster-testing (or appropriate equivalent for your template version), then use the standard update command:

sudo apt-get update && sudo apt-get dist-upgrade

Changes included in this update

@qubesos-bot
Copy link

Automated announcement from builder-github

The component mgmt-salt (including package qubes-mgmt-salt-4.1.11-1.fc32) has been pushed to the r4.1 testing repository for the Fedora template.
To test this update, please install it with the following command:

sudo yum update --enablerepo=qubes-vm-r4.1-current-testing

Changes included in this update

@qubesos-bot
Copy link

Automated announcement from builder-github

The package qubes-mgmt-salt-4.0.25-1.fc25 has been pushed to the r4.0 stable repository for dom0.
To install this update, please use the standard update command:

sudo qubes-dom0-update

Or update dom0 via Qubes Manager.

Changes included in this update

@qubesos-bot
Copy link

Automated announcement from builder-github

The package qubes-mgmt-salt-4.1.11-1.fc32 has been pushed to the r4.1 stable repository for dom0.
To install this update, please use the standard update command:

sudo qubes-dom0-update

Or update dom0 via Qubes Manager.

Changes included in this update

@qubesos-bot
Copy link

Automated announcement from builder-github

The component mgmt-salt (including package qubes-mgmt-salt-4.0.25-1.fc32) has been pushed to the r4.0 stable repository for the Fedora template.
To install this update, please use the standard update command:

sudo yum update

Changes included in this update

@qubesos-bot
Copy link

Automated announcement from builder-github

The package mgmt-salt has been pushed to the r4.0 stable repository for the CentOS centos8 template.
To install this update, please use the standard update command:

sudo yum update

Changes included in this update

@qubesos-bot
Copy link

Automated announcement from builder-github

The package qubes-mgmt-salt_4.0.25-1+deb10u1 has been pushed to the r4.0 stable repository for the Debian template.
To install this update, please use the standard update command:

sudo apt-get update && sudo apt-get dist-upgrade

Changes included in this update

@eloquence
Copy link

@marmarek We're now longer seeing version 4.0.25 in https://yum.qubes-os.org/r4.0/current/vm/fc33/rpm/ , which is a blocker for a clean transition of our users to fedora-33. We also noticed downtime of the repo earlier this week - possibly related?

@marmarek
Copy link
Member

Yes, related. Let me re-upload the package.

@marmarek
Copy link
Member

marmarek commented May 27, 2021

Done.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C: Fedora C: mgmt diagnosed Technical diagnosis has been performed (see issue comments). P: default Priority: default. Default priority for new issues, to be replaced given sufficient information. r4.0-bullseye-stable r4.0-buster-stable r4.0-centos7-stable r4.0-centos8-stable r4.0-dom0-stable r4.0-fc32-stable r4.0-fc33-stable r4.0-stretch-stable r4.1-bullseye-stable r4.1-buster-stable r4.1-centos8-stable r4.1-dom0-stable r4.1-fc31-stable r4.1-fc32-stable r4.1-fc33-stable T: bug Type: bug report. A problem or defect resulting in unintended behavior in something that exists.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants