Project

General

Profile

Actions
Copy link

Bug #36696

closed

upload-salt-reports missing module salt

Added by Viet Pham 9 months ago. Updated 8 months ago.

Status:
Resolved
Priority:
Normal
Assignee:
-
Category:
-
Target version:
-
Difficulty:
Triaged:
No
Bugzilla link:
Pull request:
Fixed in Releases:
Found in Releases:
Red Hat JIRA:

Description

I went from Foreman 3.5 with Katello 4.7 to Foreman 3.7 with Katello 4.9 running on Rocky Linux 8.8.
I am running Salt 3004 using rpm install.

I started to get problem with the upload-salt-reports where it would output this error.
It was working on Foreman 3.5.
I am using this version rubygem-smart_proxy_salt-5.0.1-1.fm3_6.el8.noarch, this was on the foreman-plugin 3.7, and I don't see a fm3_7 version of this.

Traceback (most recent call last):
  File "/usr/sbin/upload-salt-reports", line 22, in <module>
    import salt.config
ModuleNotFoundError: No module named 'salt'

A "pip install salt" resolves this, but I am not sure if this would cause other problems with my rpm install.

Actions
Copy link
 #1

Updated by Viet Pham 9 months ago

It seems like upload-salt-reports was using python3.9, and switching it to 3.6 works.

For Salt 3006, how would this work, since they are switching to onedir, and they ship with their own python3.10 version that is separate from the OS python.

Actions
Copy link
 #2

Updated by Bastian Schmidt 8 months ago

Thanks for opening up the issue!

I think it is more difficult to use the script especially with the onedir installation. Therefore, I would actually promote removing the script and switching to the Salt Reactor-based report upload as shown here in the docs:

https://docs.theforeman.org/nightly/Managing_Configurations_Salt/index-foreman-el.html#Enabling_Salt_Report_Uploads_managing-configurations-salt

Actions
Copy link
 #3

Updated by Viet Pham 8 months ago

  • Status changed from New to Resolved

Bastian Schmidt wrote in #note-2:

Thanks for opening up the issue!

I think it is more difficult to use the script especially with the onedir installation. Therefore, I would actually promote removing the script and switching to the Salt Reactor-based report upload as shown here in the docs:

https://docs.theforeman.org/nightly/Managing_Configurations_Salt/index-foreman-el.html#Enabling_Salt_Report_Uploads_managing-configurations-salt

Bastian Schmidt wrote in #note-2:

Thanks for opening up the issue!

I think it is more difficult to use the script especially with the onedir installation. Therefore, I would actually promote removing the script and switching to the Salt Reactor-based report upload as shown here in the docs:

https://docs.theforeman.org/nightly/Managing_Configurations_Salt/index-foreman-el.html#Enabling_Salt_Report_Uploads_managing-configurations-salt

Thank you, I did not see that guide on 3.7.
The salt reactor works for me.
I'll test it with the new onedir and report back

Actions
Copy link

Also available in: Atom PDF