Cleanup attribute names

[Kurt Griffiths]

Lots of __init__ methods assign values to attributes that don't start with an underscore, but should.

[chandan kumar (chkumar246)]

Hello,

@Kurt and @Flavio, i am not getting what i have to do from the above bug description.
Please explain what actually i have to do solve this bug?

Thanks,
Chandan Kumar

[Kurt Griffiths (kgriffs)]

Hi Chandan,

Basically, if an attribute is only used within the class, the name should be prefixed with an underscore to designate it as a "private" attribute.

You'll need to just go through all the classes looking for cases where:

  self.should_be_private

can be renamed to:

  self._should_be_private

[Wei Wang (damon-devops)]

Hi,

@Kurt, just like the line 67 in https://github.com/openstack/marconi/blob/master/marconi/queues/bootstrap.py ?

"self.conf" should be "self._conf"?

This maybe take some time.

[sai krishna (krishna1256)]

Hi,

I could see this bug is there since long. I can start working if no one is working as of now. Please respond with in a day, so that I could start.

Thanks,
--sai krishna.

[prashanth raghu (p-isprashanth)]

Edited Files:
-----------------

1. Folder: marconi/queues/transport/wsgi/v1_0
    Files: health.py homedoc.py stats.py queues.py claims.py stats.py metadata.py

2. Folder: marconi/queues/transport/wsgi/v1_0
    Files: health.py homedoc.py stats.py queues.py claims.py stats.py metadata.py

[sai krishna (krishna1256)]

Hi Prashanth,

For submitting a patch, you should follow Gerrit workflow.Please find additional details in the below link:
https://wiki.openstack.org/wiki/Gerrit_Workflow

and before working on a bug, you will need to assign it to yourself(only if it unassigned or its idle for long time.)

coming to this one, I am already working on this bug.

Thanks,
--sai krishna

[prashanth raghu (p-isprashanth)]

Hi Sai Krishna,

Sorry for the wrong submission. I did submit a patch following the workflow.

It is here: https://review.openstack.org/#/c/80767/

[sai krishna (krishna1256)]

Hi Prasanth,

Since you have already made code changes and sent for an initial round of review, please assign this bug to your name and continue working. I will update marconi team about the same.

Thanks,
--sai krishna.

[prashanth raghu (p-isprashanth)]

hi Sai Krishna,

Sure I will do that.

Thanks,
Prashanth

[Victoria Martinez de la Cruz (vkmc)]

prashanth_ I assumed you were not working anymore in this bug, so a newcomer is taking it.

[prashanth raghu (p-isprashanth)]

vkmc: Yes you are right.

[Kurt Griffiths (kgriffs)]

Use your best judgment, with these guidelines in mind:

1. If a member variable is only ever used within the class in which it is defined, it should start with an underscore
2. If a member variable is only ever accessed by one or two "friend" classes, it should still start with an underscore to discourage tight coupling with internal state outside of those friends.
3. If a member variable is part of the public interface, meaning it is envisioned to be used by lots of different classes (conceptually, 3rd parties), the name does not start with an underscore.

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/117574
Committed: https://git.openstack.org/cgit/openstack/zaqar/commit/?id=d1fd0c2b8920bd34c8800098ac601abec77aacb9
Submitter: Jenkins
Branch: master

commit d1fd0c2b8920bd34c8800098ac601abec77aacb9
Author: earnThis <email address hidden>
Date: Wed Aug 27 18:23:00 2014 -0400

    Add _ prefix to local variables

    A lot of __init__ methods were assigning
    values to attributes that were not prefixed with
    a single underscore, the underscore prefix has been
    added as needed.

    Closes-Bug: 1239834
    Change-Id: Idf7e29b413187f726fb9f8a85959f948db6d669c