Bug 1678476

New commits detected on ManageIQ/manageiq/hammer:

https://github.com/ManageIQ/manageiq/commit/2aa3228428546cfd16f4d67319fb5aac01b7d25e
commit 2aa3228428546cfd16f4d67319fb5aac01b7d25e
Author:     Greg McCullough <gmccullo>
AuthorDate: Mon Feb 11 12:04:57 2019 -0500
Commit:     Greg McCullough <gmccullo>
CommitDate: Mon Feb 11 12:04:57 2019 -0500

    Merge pull request #18443 from d-m-u/remove_lbs_from_retirement_check

    Don't retire load balancers

    (cherry picked from commit d8a6ba00be00d215a44af0f71e27f7ad364967ba)

    https://bugzilla.redhat.com/show_bug.cgi?id=1678476

 app/models/load_balancer.rb | 1 -
 app/models/retirement_manager.rb | 2 +-
 spec/models/retirement_manager_spec.rb | 4 +-
 3 files changed, 2 insertions(+), 5 deletions(-)


https://github.com/ManageIQ/manageiq/commit/65aa7d015bb1d6782b1ca8c5908d21980c33bc92
commit 65aa7d015bb1d6782b1ca8c5908d21980c33bc92
Author:     Brandon Dunne <brandondunne>
AuthorDate: Mon Feb 18 15:38:18 2019 -0500
Commit:     Brandon Dunne <brandondunne>
CommitDate: Mon Feb 18 15:38:18 2019 -0500

    Merge pull request #18437 from d-m-u/adding_user_existence_check

    Add check for existence of user for system_context

    (cherry picked from commit 265774dd3ab68ae26f117e2cca073003cb5066d3)

    Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1678476

 app/models/mixins/retirement_mixin.rb | 10 +-
 spec/models/orchestration_stack/retirement_management_spec.rb | 43 +-
 spec/models/service/retirement_management_spec.rb | 63 +-
 spec/models/vm/retirement_management_spec.rb | 46 +-
 4 files changed, 107 insertions(+), 55 deletions(-)

*** Bug 1678474 has been marked as a duplicate of this bug. ***

Reverted the backport of https://github.com/ManageIQ/manageiq/pull/18443 and https://github.com/ManageIQ/manageiq/pull/18437

https://github.com/ManageIQ/manageiq/pull/18483

Hey you wonderful people, I can't close this bug because the PR in comment 4 hasn't been merged. It'd be super great to get the target release set and get this puppy merged cause it's a couple months old now and I'd really like to close this ticket.

Hey Gellert, I get that this ticket is associated with the one about the valid retirement date (https://bugzilla.redhat.com/show_bug.cgi?id=1697524) because they both involve the same error, but I think that comment 6 should be on the other ticket since this isn't about a retirement date at all. I think the other bug probably needs the PR that fixes this and it will then need to be retested to make sure there's nothing else wrong with it.

Sure. 
create a service with a user that isn't admin
provision the service
delete the user from the db
schedule the service to retire more than fifteen minutes from now
check to see who the requester for the retirement shows up as

Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHBA-2019:1247