Bug 1375291 - :cold_sweat: Don't include AvailabilityMixin into Object, that's really bad
Summary: :cold_sweat: Don't include AvailabilityMixin into Object, that's really bad
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat CloudForms Management Engine
Classification: Red Hat
Component: Providers
Version: 5.6.0
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: GA
: 5.7.0
Assignee: Brandon Dunne
QA Contact: Jan Krocil
URL:
Whiteboard:
Depends On:
Blocks: 1376132
TreeView+ depends on / blocked
 
Reported: 2016-09-12 15:46 UTC by John Prause
Modified: 2019-08-06 20:07 UTC (History)
5 users (show)

Fixed In Version: 5.7.0.0
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
: 1376132 (view as bug list)
Environment:
Last Closed: 2017-01-11 20:24:07 UTC
Category: ---
Cloudforms Team: ---
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)

Description John Prause 2016-09-12 15:46:52 UTC
Before:

irb(main):001:0> Object.respond_to?(:is_available?)
=> false

irb(main):002:0> ManageIQ::Providers::ContainerManager
PostgreSQLAdapter#log_after_checkout, connection_pool: size: 5, connections: 1, in use: 1, waiting_in_queue: 0
=> ManageIQ::Providers::ContainerManager(id: integer, name: string, etc...)

irb(main):003:0> Object.respond_to?(:is_available?)
=> true

After:

irb(main):001:0> Object.respond_to?(:is_available?)
=> false

irb(main):002:0> ManageIQ::Providers::ContainerManager
PostgreSQLAdapter#log_after_checkout, connection_pool: size: 5, connections: 1, in use: 1, waiting_in_queue: 0
=> ManageIQ::Providers::ContainerManager(id: integer, name: string, etc...)

irb(main):003:0> Object.respond_to?(:is_available?)
=> false

Comment 2 John Prause 2016-09-12 15:47:48 UTC
https://github.com/ManageIQ/manageiq/pull/10285


Note You need to log in before you can comment on or make changes to this bug.