Fields should not be named the built-in type keyword

[Taylor Peoples]

There's a number of places where a field of an object is named "type", which isn't the best thing to do since type is a python built-in. For example, see Audit and Event.

[Jean-Emile DARTOIS (jed56)]

Hello Talyor,
Are you still working on this bug ?

[OpenStack Infra (hudson-openstack)]

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

[Digambar (digambarpatil15)]

I think this is not only about changing the type keyword to audit_type but it requires changes in db models & if we want to add or update field name in models it requires sqlalchemy migration.

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/338391
Committed: https://git.openstack.org/cgit/openstack/python-watcherclient/commit/?id=75dd8dc561ed38ab85b020518258ccb0d758d360
Submitter: Jenkins
Branch: master

commit 75dd8dc561ed38ab85b020518258ccb0d758d360
Author: digambar <email address hidden>
Date: Wed Jul 6 21:55:10 2016 +0530

    Fix field type to audit_type

    There's a number of places where a field of an object is named "type",
    which isn't good thing. Actually type is keyword in python, so wherever
    I found type used in models/api/object/test, I have fixed it to
    audit_type in the audit files

    Change-Id: Ic09458b455820787f5483e48aed940a2196b297e
    Closes-Bug: #1533392

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/336390
Committed: https://git.openstack.org/cgit/openstack/watcher/commit/?id=47ff9bb105b78d1ed1fb0f4ea31c7e3a76eeacd1
Submitter: Jenkins
Branch: master

commit 47ff9bb105b78d1ed1fb0f4ea31c7e3a76eeacd1
Author: digambar <email address hidden>
Date: Mon Jul 4 22:50:53 2016 +0530

    Fix field type to audit_type

    There's a number of places where a field of an object is named "type",
    which isn't good thing. Actually type is keyword in python, so wherever
    I found type used in models/api/object/test, I have fixed it to
    audit_type in the audit files

    Change-Id: Iea6bd3acb0b2af2a833b3916701aad88f6064bba
    Closes-Bug: #1533392

[Doug Hellmann (doug-hellmann)]

This issue was fixed in the openstack/python-watcherclient 0.24.0 release.

[Doug Hellmann (doug-hellmann)]

This issue was fixed in the openstack/watcher 0.28.0 release.

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/372583
Committed: https://git.openstack.org/cgit/openstack/watcher-dashboard/commit/?id=ea6c773facb3c4738b3372d737c8e13a618e8d85
Submitter: Jenkins
Branch: master

commit ea6c773facb3c4738b3372d737c8e13a618e8d85
Author: David TARDIVEL <email address hidden>
Date: Mon Sep 19 17:31:57 2016 +0200

    Clean the code by renaming 'type' parameters

    Closes-bug: #1533392
    Change-Id: I02b6aae718a3a7d043c3316dbdccec5e2ade5d2a

[OpenStack Infra (hudson-openstack)]

This issue was fixed in the openstack/watcher-dashboard 0.5.0 release.