__eq__ of Resource return True even if the two objects have different value
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Glance Client |
Fix Released
|
Undecided
|
Rui Chen | ||
oslo-incubator |
Won't Fix
|
Undecided
|
Rui Chen | ||
python-ceilometerclient |
Fix Released
|
Undecided
|
Rui Chen | ||
python-cinderclient |
Fix Released
|
Undecided
|
Rui Chen | ||
python-heatclient |
Fix Released
|
Undecided
|
Rui Chen | ||
python-ironicclient |
Fix Released
|
Low
|
Rui Chen | ||
python-keystoneclient |
Fix Released
|
Undecided
|
Rui Chen | ||
python-muranoclient |
Fix Released
|
Undecided
|
Rui Chen | ||
python-novaclient |
In Progress
|
Undecided
|
Rui Chen |
Bug Description
the __eq__ of apiclient.
The behavior is weird and don't match the semantics of object equal. The objects that have different value should be different objects.
Think about the following case:
I have an application, it will query a nova instance in fixed interval by using novaclient, and check whether some changes occur on the instance, like: status, fixed-ip and so on. I can't use obj1 == obj2 to check, because even if the instance's status is changed from ACTIVE to PAUSED, the result still is True.
I found this issue when I'm working on a Congress BP.
Congress nova driver will query nova instances in fixed interval by using novaclient, and check whether some changes occur on the instance, like: status, fixed-ip and so on. I can't use obj1 == obj2 to check, because even if the instance's status is changed from ACTIVE to PAUSED, the result still is True.
Changed in oslo-incubator: | |
assignee: | nobody → Rui Chen (kiwik-chenrui) |
Changed in oslo-incubator: | |
status: | New → Won't Fix |
description: | updated |
Changed in python-novaclient: | |
assignee: | nobody → Rui Chen (kiwik-chenrui) |
Changed in python-cinderclient: | |
assignee: | nobody → Rui Chen (kiwik-chenrui) |
Changed in python-ceilometerclient: | |
assignee: | nobody → Rui Chen (kiwik-chenrui) |
Changed in python-heatclient: | |
assignee: | nobody → Rui Chen (kiwik-chenrui) |
Changed in python-glanceclient: | |
status: | New → In Progress |
Changed in python-ironicclient: | |
assignee: | nobody → Rui Chen (kiwik-chenrui) |
Changed in python-keystoneclient: | |
assignee: | nobody → Rui Chen (kiwik-chenrui) |
Changed in python-muranoclient: | |
assignee: | nobody → Rui Chen (kiwik-chenrui) |
Changed in python-ironicclient: | |
importance: | Undecided → Low |
Changed in python-muranoclient: | |
assignee: | Rui Chen (kiwik-chenrui) → Ekaterina Chernova (efedorova) |
Changed in python-muranoclient: | |
assignee: | Ekaterina Chernova (efedorova) → Rui Chen (kiwik-chenrui) |
Changed in python-muranoclient: | |
milestone: | none → 0.8.1 |
Changed in python-ceilometerclient: | |
status: | Fix Committed → Fix Released |
Changed in python-glanceclient: | |
status: | Fix Committed → Fix Released |
Add patch URL /review. openstack. org/#/c/ 228797/
https:/