Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
14 changes: 5 additions & 9 deletions service/lib/agama/autoyast/services_manager_reader.rb
Original file line number Diff line number Diff line change
Expand Up @@ -50,8 +50,7 @@ def read

attr_reader :profile

SYSTEMD_MULTIUSER_TARGET = "systemctl set-default multi-user.target"
SYSTEMD_GRAPHICAL_TARGET = "systemctl set-default graphical.target"
SYSTEMD_VALID_TARGETS = ["graphical", "multi-user"].freeze

def services_manager_section
@services_manager_section ||= profile.fetch("services-manager", {})
Expand Down Expand Up @@ -84,13 +83,10 @@ def script(body)
end

def target_to_cmd
# May we raise an exception if nothing matches?
case services_manager_section["default_target"]
when "muti_user"
SYSTEMD_MULTIUSER_TARGET
when "graphical"
SYSTEMD_GRAPHICAL_TARGET
end
target = services_manager_section["default_target"].to_s
return unless SYSTEMD_VALID_TARGETS.include?(target)

"systemctl set-default #{target}.target"
end

def disabled_to_cmd
Expand Down
6 changes: 6 additions & 0 deletions service/package/rubygem-agama-yast.changes
Original file line number Diff line number Diff line change
@@ -1,3 +1,9 @@
-------------------------------------------------------------------
Fri Dec 12 06:21:47 UTC 2025 - Imobach Gonzalez Sosa <igonzalezsosa@suse.com>

- Fix systemd target import from AutoYaST profiles
(gh#agama-project/agama#2826).

-------------------------------------------------------------------
Thu Nov 13 13:30:58 UTC 2025 - Ladislav Slezák <lslezak@suse.com>

Expand Down
4 changes: 3 additions & 1 deletion service/test/agama/autoyast/services_manager_reader_test.rb
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,8 @@
let(:profile) do
{
"services-manager" => {
"services" => {
"default_target" => "multi-user",
"services" => {
"enable" => ["service"]
}
}
Expand All @@ -57,6 +58,7 @@

expect(result).not_to be_empty
expect(result.first["content"]).to match(/service/)
expect(result.first["content"]).to match(/set-default multi-user/)
end
end
end
Expand Down