Skip to content

Commit

Permalink
[Refactor] Move #sync_queue back down by #sync
Browse files Browse the repository at this point in the history
This is probably where the issue was with the private method as
mentioned and related to the following PRs:

- ManageIQ#19020
- ManageIQ/manageiq-ui-classic#5849
- ManageIQ#19024

My guess is that at-Fryguy was testing this in this same branch, but
split it out since it was out of scope.  In the second commit of this
branch, the `private` is completely removed, so fixing this here to
provide context to the changes and a retrospective as to what went
wrong.
  • Loading branch information
NickLaMuro committed Jul 22, 2019
1 parent 3bf73b6 commit c0c2c45
Showing 1 changed file with 4 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -38,10 +38,6 @@ def raw_delete_in_provider
destroy!
end

def sync_queue(auth_user = nil)
queue("sync", [], "Synchronizing", auth_user)
end

def git_repository
super || begin
transaction do
Expand All @@ -67,6 +63,10 @@ def sync
true
end

def sync_queue(auth_user = nil)
queue("sync", [], "Synchronizing", auth_user)
end

def playbooks_in_git_repository
git_repository.update_repo
git_repository.entries(scm_branch, "").grep(/\.ya?ml$/)
Expand Down

0 comments on commit c0c2c45

Please sign in to comment.