Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Expose custom_attribute methods to ext_management_system service model. #12602

Merged
merged 1 commit into from Nov 15, 2016

Conversation

gmcculloug
Copy link
Member

Create service_model mixin for exposing custom_attribute methods.

Links

http://talk.manageiq.org/t/acess-drb-remote-object/1869

Create service_model mixin for exposing custom_attribute methods.
@mkanoor mkanoor merged commit bd01f6e into ManageIQ:master Nov 15, 2016
@chessbyte chessbyte added this to the Sprint 50 Ending Dec 5, 2016 milestone Nov 16, 2016
@simaishi
Copy link
Contributor

simaishi commented Jan 5, 2017

@gmcculloug is there a BZ for this?

@gmcculloug
Copy link
Member Author

No, the work was done based on the talk topic linked in the description.

@simaishi
Copy link
Contributor

simaishi commented Jan 6, 2017

simaishi pushed a commit that referenced this pull request Jan 6, 2017
…odel

Expose custom_attribute methods to ext_management_system service model.
(cherry picked from commit bd01f6e)

https://bugzilla.redhat.com/show_bug.cgi?id=1410851
@simaishi
Copy link
Contributor

simaishi commented Jan 6, 2017

Euwe backport details:

$ git log -1
commit f52d2b32b8d6d8bee0d4b18d89af2d7890619cec
Author: Madhu Kanoor <mkanoor@redhat.com>
Date:   Tue Nov 15 16:31:40 2016 -0500

    Merge pull request #12602 from gmcculloug/custom_attributes_service_model
    
    Expose custom_attribute methods to ext_management_system service model.
    (cherry picked from commit bd01f6ec2070cde083032b01f002521c76e0655f)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1410851

@gmcculloug gmcculloug deleted the custom_attributes_service_model branch January 23, 2017 22:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants