Make neutron-client creation in Kuryr from admin credentials configurable

Bug #1528019 reported by Gal Sagie
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
kuryr-libnetwork
Fix Released
Low
Dongcan Ye

Bug Description

In order to pass gate testing, Kuryr creates neutron client from devstack admin credentials given that
a devstack installation exists.

This is not correct for production environments and we should disable this by default and let the user
configure the user/password

Gal Sagie (gal-sagie)
Changed in kuryr:
assignee: nobody → Gal Sagie (gal-sagie)
Changed in kuryr:
importance: Undecided → Low
status: New → Triaged
affects: kuryr → kuryr-libnetwork
Revision history for this message
Dongcan Ye (hellochosen) wrote :

@Gal, hope you dont mind me taking charge of this bug.
@Toni, I had confirmed it, we still using devstack admin credentials to run functional tests.
Maybe we can read it from ENV configured by openrc(DevStack) or admin-openrc configured in production environments.

Changed in kuryr-libnetwork:
assignee: Gal Sagie (gal-sagie) → Dongcan Ye (hellochosen)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to kuryr-libnetwork (master)

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

Changed in kuryr-libnetwork:
status: Triaged → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to kuryr-libnetwork (master)

Reviewed: https://review.openstack.org/399585
Committed: https://git.openstack.org/cgit/openstack/kuryr-libnetwork/commit/?id=465ab9b5d64cade7d684a99fab2b914d4731e5aa
Submitter: Jenkins
Branch: master

commit 465ab9b5d64cade7d684a99fab2b914d4731e5aa
Author: Dongcan Ye <email address hidden>
Date: Fri Nov 18 21:06:59 2016 +0800

    Fullstack: Using the credentials from openrc config file

    Add getting credentials from ENV, this requires us source
    openrc file in our DevStack or production environment.
    Also removing get_neutron_client_from_creds into fullstack
    code, keeping or removing it should be discussed.

    Change-Id: I10d62368573976725e793e80150ed45074008970
    Closes-Bug: #1528019

Changed in kuryr-libnetwork:
status: In Progress → 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.