Metric notification breaks on secure bay

Bug #1503460 reported by hongbin
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Magnum
Fix Released
Undecided
hongbin

Bug Description

Magnum introduced tls support in swarm bay. We need to incorporate tls support to metric notification modules, in which docker client is used to pull metrics from bay.

hongbin (hongbin034)
Changed in magnum:
assignee: nobody → hongbin (hongbin034)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to magnum (master)

Fix proposed to branch: master
Review: https://review.openstack.org/232138

Changed in magnum:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Fix proposed to branch: master
Review: https://review.openstack.org/232139

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to magnum (master)

Reviewed: https://review.openstack.org/232138
Committed: https://git.openstack.org/cgit/openstack/magnum/commit/?id=49cf17c12dc18eef5d540388fac9a9fe825f5264
Submitter: Jenkins
Branch: master

commit 49cf17c12dc18eef5d540388fac9a9fe825f5264
Author: Hongbin Lu <email address hidden>
Date: Wed Oct 7 10:38:24 2015 -0400

    Move 'docker_for_container' to a common module

    This method is for obtaining a docker client that is used to
    communicate with a swarm bay. Pulling it out of docker conductor
    allows it to be reused later.

    Change-Id: I495835595ee049103f97ee8b5bfe9eb74fe9ad45
    Partial-Bug: #1503460

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Reviewed: https://review.openstack.org/232139
Committed: https://git.openstack.org/cgit/openstack/magnum/commit/?id=30eacb52c965e3107616431d7567ddb86d3dc080
Submitter: Jenkins
Branch: master

commit 30eacb52c965e3107616431d7567ddb86d3dc080
Author: Hongbin Lu <email address hidden>
Date: Wed Oct 7 14:09:40 2015 -0400

    Pull metrics by using TLS enabled client

    Magnum introduced TLS support in swarm bay, so swarm monitor needs
    to use TLS enabled client to pull data.

    In addition, this commit eliminates data pulling from the docker
    daemon of individual bay nodes, since they are not secured by TLS.
    Instead, we always pull from swarm. Due to that, additional text
    parsing is needed, since the output of docker.info from swarm is
    not well formed.

    Change-Id: I121bfa98622c240d5c777901794a35c0d8990345
    Closes-Bug: #1503460

Changed in magnum:
status: In Progress → Fix Committed
Adrian Otto (aotto)
Changed in magnum:
milestone: none → mitaka-1
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.