Remove the user from group while update user's password

Bug #1522247 reported by huangtianhua
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Heat
Fix Released
Medium
huangtianhua

Bug Description

1. create a stack with a keystone user, the template is:
***************************************
heat_template_version: 2016-04-08
parameters:
  pw:
    type: string
resources:
  t_p:
    type: OS::Keystone::User
    properties:
      name: 'y'
      domain: test
      default_project: demo
      password: {get_param: pw}
      groups:
        - nonadmins
***************************************
2. the user create successful, and we can find the user in group 'nonadmins'

3. update the user with a new password, the stack update complete, but we can find the user was removed from the group 'nonadmins', the code handle_update() for user resource is incorrect, let' fix it.

4. there are also several other errors of user updation, will fix them together.

Changed in heat:
assignee: nobody → huangtianhua (huangtianhua)
Changed in heat:
importance: Undecided → Medium
milestone: none → mitaka-2
description: updated
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to heat (master)

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

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

Reviewed: https://review.openstack.org/252829
Committed: https://git.openstack.org/cgit/openstack/heat/commit/?id=a23e6a6b6361a7529fa18f274250b16caebc4fc4
Submitter: Jenkins
Branch: master

commit a23e6a6b6361a7529fa18f274250b16caebc4fc4
Author: huangtianhua <email address hidden>
Date: Thu Dec 3 17:00:01 2015 +0800

    Don't remove user from groups if groups don't change

    If there is no change of 'groups' property when updating
    a keystone user resource, do not remove the user from
    the groups.

    Change-Id: Ib8e920e6ef95d654d0641e8635bd586047c89047
    Closes-Bug: #1522247

Changed in heat:
status: In Progress → Fix Released
tags: added: liberty-backport-potential
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to heat (stable/liberty)

Fix proposed to branch: stable/liberty
Review: https://review.openstack.org/255038

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to heat (stable/liberty)

Reviewed: https://review.openstack.org/255038
Committed: https://git.openstack.org/cgit/openstack/heat/commit/?id=3dd5200c63b0d3b1c5b90fbb6a1e867e23394554
Submitter: Jenkins
Branch: stable/liberty

commit 3dd5200c63b0d3b1c5b90fbb6a1e867e23394554
Author: huangtianhua <email address hidden>
Date: Thu Dec 3 17:00:01 2015 +0800

    Don't remove user from groups if groups don't change

    If there is no change of 'groups' property when updating
    a keystone user resource, do not remove the user from
    the groups.

    Closes-Bug: #1522247
    (cherry picked from commit a23e6a6b6361a7529fa18f274250b16caebc4fc4)

    Conflicts:
     heat/engine/resources/openstack/keystone/user.py
     heat/tests/keystone/test_user.py

    Change-Id: Ib8e920e6ef95d654d0641e8635bd586047c89047

tags: added: in-stable-liberty
Revision history for this message
Thierry Carrez (ttx) wrote : Fix included in openstack/heat 6.0.0.0b2

This issue was fixed in the openstack/heat 6.0.0.0b2 development milestone.

Revision history for this message
Doug Hellmann (doug-hellmann) wrote :

This issue was fixed in the openstack/heat 6.0.0.0b2 development milestone.

Revision history for this message
Doug Hellmann (doug-hellmann) wrote : Fix included in openstack/heat 5.0.1

This issue was fixed in the openstack/heat 5.0.1 release.

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.