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 Openstack Panko Events support #57

Merged
merged 1 commit into from
Mar 24, 2017

Conversation

aufi
Copy link
Member

@aufi aufi commented Feb 9, 2017

Openstack Panko service is about to take over Events login from Ceilometer.
Adding support for Panko service with fallback to Ceilometer for Events capture.
Panko API is identical to Events-related Ceilometer part of API.

It should be possible select Events service also manually in settings.

Depends on fog-openstack PR [1] merge and new version published and [2].

Links

@miq-bot miq-bot added the wip label Feb 9, 2017
@aufi aufi force-pushed the add_openstack_panko_events branch from 9cb9259 to 3af196b Compare February 9, 2017 15:15
@aufi
Copy link
Member Author

aufi commented Feb 17, 2017

fog/fog-openstack#269 was merged, waiting for its release to update gemspec with new fog-openstack version.

@miq-bot
Copy link
Member

miq-bot commented Feb 23, 2017

Checked commit aufi@e79508b with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0
4 files checked, 1 offense detected

lib/gems/pending/openstack/openstack_handle/handle.rb

@aufi aufi force-pushed the add_openstack_panko_events branch 2 times, most recently from e475ca7 to 4b7a3bf Compare March 16, 2017 10:49
@aufi
Copy link
Member Author

aufi commented Mar 16, 2017

Required PRs were merged, updating this PR and removing WIP label.

@miq-bot rm_label wip

@miq-bot miq-bot changed the title [WIP] Add Openstack Panko Events support Add Openstack Panko Events support Mar 16, 2017
@miq-bot miq-bot removed the wip label Mar 16, 2017
@aufi aufi force-pushed the add_openstack_panko_events branch 2 times, most recently from 847922d to 9e9fc2e Compare March 17, 2017 12:25
@aufi
Copy link
Member Author

aufi commented Mar 21, 2017

@blomquisg @Ladas Hi, may I ask for review?

@aufi
Copy link
Member Author

aufi commented Mar 21, 2017

@miq-bot add_label enhancement

@Ladas
Copy link
Contributor

Ladas commented Mar 21, 2017

looks great 👍

Openstack Panko service is about to take over Events login from Ceilometer.
Adding support for Panko service with fallback to Ceilometer for Events capture.
@aufi aufi force-pushed the add_openstack_panko_events branch from 9e9fc2e to 8f04148 Compare March 22, 2017 14:29
@blomquisg blomquisg added this to the Sprint 57 Ending Mar 27, 2017 milestone Mar 24, 2017
@blomquisg blomquisg merged commit 8dacc41 into ManageIQ:master Mar 24, 2017
@blomquisg blomquisg self-assigned this Mar 24, 2017
@tzumainn
Copy link
Contributor

@miq-bot add_label euwe/yes

@aufi
Copy link
Member Author

aufi commented Mar 28, 2017

Required by https://bugzilla.redhat.com/show_bug.cgi?id=1436340
@miq-bot add_label blocker

@simaishi
Copy link
Contributor

@aufi There are a couple of conflicts backporting this to Euwe. Can you create an Euwe PR for this?

@aufi
Copy link
Member Author

aufi commented Mar 29, 2017

@simaishi Sure, euwe backport: ManageIQ/manageiq#14557

@simaishi
Copy link
Contributor

Backported to Euwe via ManageIQ/manageiq#14557

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.

6 participants