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

add ability to set custom attributes on services via api #85

Merged
merged 5 commits into from
Oct 5, 2017

Conversation

ailisp
Copy link
Member

@ailisp ailisp commented Sep 27, 2017

Issue: missing ability to set custom attributes on services via api, which is now fixed

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

\cc @abellotti @gtanzillo
@miq-bot add-label wip, bug

@abellotti
Copy link
Member

api.yml declaration (identifiers for the subcollection actions, see vms/instances/providers for example), also, the specs would be needed, also look at instances for reference. Thanks!!

@ailisp
Copy link
Member Author

ailisp commented Sep 27, 2017

@abellotti Hi, should we do similar spec instances_spec.rb and services_spec.rb for custom attributes. Or, since they're almost the same, test for custom_attributes mixin. Thanks!

@ailisp ailisp force-pushed the services-api-custom-attr branch 2 times, most recently from 865f08b to ef1cb3e Compare September 30, 2017 00:42
@ailisp
Copy link
Member Author

ailisp commented Sep 30, 2017

@miq-bot remove-label wip

@miq-bot miq-bot changed the title [WIP] add ability to set custom attributes on services via api add ability to set custom attributes on services via api Sep 30, 2017
@miq-bot miq-bot removed the wip label Sep 30, 2017
@miq-bot
Copy link
Member

miq-bot commented Oct 3, 2017

This pull request is not mergeable. Please rebase and repush.

@miq-bot
Copy link
Member

miq-bot commented Oct 4, 2017

Checked commits ailisp/manageiq-api@c4f74c0~...07049fd with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0
2 files checked, 0 offenses detected
Everything looks fine. 🏆

@abellotti abellotti added enhancement and removed bug labels Oct 5, 2017
@abellotti abellotti self-assigned this Oct 5, 2017
@abellotti
Copy link
Member

Thanks @ailisp for the API Enhancement !! 👍

The Hakiri warning above is unrelated and was taken care in a separate PR so I'll go ahead and merge.

@abellotti abellotti merged commit a7084be into ManageIQ:master Oct 5, 2017
@abellotti abellotti added this to the Sprint 71 Ending Oct 16, 2017 milestone Oct 5, 2017
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.

3 participants