Skip to content

Commit

Permalink
fix(dbus): fix users issues detection (#1317)
Browse files Browse the repository at this point in the history
Recently, we implemented the Issues API to users management (which was
using the old validation API). However, the manager and the D-Bus
service where using separate `Users` instances.
  • Loading branch information
imobachgs authored Jun 12, 2024
2 parents 33b55d2 + b1961af commit e311083
Show file tree
Hide file tree
Showing 4 changed files with 2 additions and 38 deletions.
1 change: 0 additions & 1 deletion doc/dbus/bus/org.opensuse.Agama1.ServiceStatus.bus.xml

This file was deleted.

35 changes: 0 additions & 35 deletions doc/dbus/org.opensuse.Agama1.ServiceStatus.doc.xml

This file was deleted.

2 changes: 1 addition & 1 deletion service/lib/agama/dbus/manager_service.rb
Original file line number Diff line number Diff line change
Expand Up @@ -125,7 +125,7 @@ def manager_dbus
end

def users_dbus
@users_dbus ||= Agama::DBus::Users.new(users, logger)
@users_dbus ||= Agama::DBus::Users.new(manager.users, logger)
end
end
end
Expand Down
2 changes: 1 addition & 1 deletion service/lib/agama/users.rb
Original file line number Diff line number Diff line change
Expand Up @@ -149,7 +149,7 @@ def update_issues
unless root_password? || root_ssh_key? || first_user?
new_issues << Issue.new(
_("Defining a user, setting the root password or a SSH public key is required"),
source: Issue::Source::CONFIG,
source: Issue::Source::CONFIG,
severity: Issue::Severity::ERROR
)
end
Expand Down

0 comments on commit e311083

Please sign in to comment.