Skip to content

[openstack] new cloud admin check structure#2303

Closed
talwai wants to merge 8 commits intomasterfrom
talwai/openstack_admin
Closed

[openstack] new cloud admin check structure#2303
talwai wants to merge 8 commits intomasterfrom
talwai/openstack_admin

Conversation

@talwai
Copy link
Contributor

@talwai talwai commented Feb 29, 2016

The existing openstack.py check is prohibitively hard to configure for cloud administrators wishing to monitor several projects spanning several domains. These changes introduce a new structure for OpenStack monitoring as a cloud admin, using an easier to configure domain-based auth scheme. The new scheme, and its prerequisites, are described in detail here. This doc should eventually be absorbed into the main OpenStack documentation

This new monitoring scheme involves configuring 2 checks:

  • openstack_admin : collects metrics on project limits / quotas for all projects accessible by the configured admin domain. Can run on just a single host which has access to the keystone server
  • openstack_hyp: collects hypervisor-level metrics, and metrics about the servers managed by a single hypervisor. Should be deployed on each compute host.

Currently there is substantial code duplication between the two checks, which should probably be pulled out into a util module of some sort. But when used together, they output the exam same metrics and tags as the original openstack.py check with the advantage of being far easier to configure.

@irabinovitch
Copy link
Contributor

CC @mtougeron as he likely has interest in this one.

@mtougeron
Copy link
Contributor

Love it! This will make things a lot easier for us. :)

@olivielpeau olivielpeau added this to the 5.8.0 milestone Feb 29, 2016
@olivielpeau olivielpeau modified the milestones: Triage, 5.8.0 Apr 29, 2016
@olivielpeau
Copy link
Member

Moving back to Triage. If need be the checks can be used as custom checks before they're shipped in a future release.

@remh remh modified the milestones: 5.9.0, Triage May 2, 2016
@truthbk truthbk modified the milestones: 5.10.0, 5.9.0 Aug 31, 2016
@olivielpeau olivielpeau modified the milestones: 5.11.0, 5.10.0 Oct 24, 2016
@masci masci modified the milestones: 5.11.0, 5.12.0 Jan 24, 2017
@truthbk
Copy link
Member

truthbk commented Jan 30, 2017

Closing, reconsider on the SDK side.

@truthbk truthbk closed this Jan 30, 2017
@truthbk
Copy link
Member

truthbk commented Jan 30, 2017

Hi @talwai ,

Thanks again for your contribution, always very much appreciated. We're currently transitioning our checks to a new SDK-based model for the agent. An unfortunate side-effect is that we have to impose a freeze on the dd-agent repo. The SDK consists of two repositories https://github.com/DataDog/integrations-core and https://github.com/DataDog/integrations-extras, where all of our checks will reside. This PR would belong in the core repository. Would you be so kind as to re-open the PR there? We have facilitated some tools to help you with the SDK development that should help you get up and running in no time:

We are very sorry for the inconvenience caused.

Thank you for your patience as we make this transition!

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.

7 participants