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

Multiple managers for Kubernetes #39

Closed
4 of 9 tasks
moolitayer opened this issue Jun 13, 2017 · 5 comments
Closed
4 of 9 tasks

Multiple managers for Kubernetes #39

moolitayer opened this issue Jun 13, 2017 · 5 comments

Comments

@moolitayer
Copy link

moolitayer commented Jun 13, 2017

WAT ?

  • Monitoring = Metrics & Alerts
  • Container providers should have a main manager ContainerManager and a secondary manager MonitoringManager linked by parant_manager_id.
  • All monitoring workers should move under the MonitoringManager
  • There should be two possible implementations for the Monitoring Manager living side by side. Hawkular and Prometheus. The implementation should be different files.

WHY ?

Move closer to other providers where separate entities in the provider are separate managers. Better worker management. Examples:

  • If the hawkular authentication status is bad, we should stop the related workers: metrics(currently).
  • We are adding a new event listener from hawkular alerts on top of the kubernetes event listener. That would belong under the Monitoring manager.

HOW ?

(In merge order)

prometheus related work

ManageIQ/manageiq#15473
ManageIQ/manageiq-ui-classic#1640

@cben
Copy link
Contributor

cben commented Jun 13, 2017 via email

@moolitayer
Copy link
Author

moolitayer commented Jun 14, 2017

  1. Does this involve upgrading k8s and openshift from being managers (EMS)
    to Providers?
    Or will there be some sibling manager -> manager link?

Turns out both these approaches exist in ManageIQ[1][2][3]. After consultation with @blomquisg and @Ladas the manager + parent_manager route seems more productive for us.

The foreman example uses a provider, the other two a link between managers;
[1] https://github.com/ManageIQ/manageiq-providers-amazon/blob/087f2bb574304ce915945425f8e91d0252fc492a/app/models/manageiq/providers/amazon/cloud_manager.rb#L69
[2] https://github.com/ManageIQ/manageiq-providers-azure/blob/7bfc70f2a76090c80de55d714eb0c90f82bd2384/app/models/manageiq/providers/azure/cloud_manager.rb#L34
[3] https://github.com/ManageIQ/manageiq-providers-foreman/blob/1a9257535c6b60183a07141dc08996411f263e7a/app/models/manageiq/providers/foreman/provider.rb#L23

  1. Will the metrics Endpoints eventually move into the monitoring manager?
    IOW how will API look? (not opposed to changing, just curious)

Don't know yet. I will probably start by taking a look at what other providers are doing.

@moolitayer moolitayer changed the title Multiple managers for Kubernetes + Prometheus support Multiple managers for Kubernetes + Prometheus alerts support Jul 4, 2017
@blomquisg
Copy link
Member

It's worth pointing out that we're discussing the Provider/Manager relationship: http://talk.manageiq.org/t/rethinking-providers-and-managers/2494

I'm not sure exactly all the overlaps with this yet. But, I'm sure there's going to be some overlaps.

Also, the notion of controlling workers via Managers will hopefully be a thing of the past as we dive more and more in the ReArch ("H" Release).

/cc @Fryguy @gtanzillo, @agrare, @durandom

@moolitayer moolitayer changed the title Multiple managers for Kubernetes + Prometheus alerts support Multiple managers for Kubernetes Sep 3, 2017
@miq-bot
Copy link
Member

miq-bot commented Apr 9, 2018

This issue has been automatically marked as stale because it has not been updated for at least 6 months.

If you can still reproduce this issue on the current release or on master, please reply with all of the information you have about it in order to keep the issue open.

Thank you for all your contributions!

@agrare
Copy link
Member

agrare commented Dec 12, 2019

Kubernetes has a monitoring manager now so I'm going to say this is done. Please reopen and update the description if you want to do more here.

@agrare agrare closed this as completed Dec 12, 2019
@agrare agrare removed the stale label Dec 12, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants