Add hacking check to ensure not use xrange()
Bug #1538118 reported by
caoyue
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Magnum |
Fix Released
|
Undecided
|
Deeksha Sachdeva | ||
| OpenStack Shared File Systems Service (Manila) |
Fix Released
|
Low
|
Tin Lam | ||
| octavia |
Fix Released
|
Medium
|
Tin Lam | ||
Bug Description
xrange() does not exist in python3. Add hacking check rule so that codes with xragne() never passby test.
| Changed in octavia: | |
| assignee: | nobody → caoyue (yue-cao) |
| Changed in manila: | |
| assignee: | nobody → caoyue (yue-cao) |
| Changed in magnum: | |
| assignee: | nobody → caoyue (yue-cao) |
| Changed in octavia: | |
| importance: | Undecided → Medium |
| Changed in manila: | |
| importance: | Undecided → Low |
| milestone: | none → newton-1 |
| Changed in octavia: | |
| assignee: | caoyue (yue-cao) → Tin Lam (tl3438) |
| status: | New → In Progress |
Revision history for this message
| OpenStack Infra (hudson-openstack) wrote Fix merged to octavia (master) | #1 |
| Changed in octavia: | |
| status: | In Progress → Fix Released |
| Changed in magnum: | |
| assignee: | caoyue (yue-cao) → Sonali Goyal (sonaligoyal654321) |
| assignee: | Sonali Goyal (sonaligoyal654321) → nobody |
| Changed in magnum: | |
| assignee: | nobody → Deeksha Sachdeva (sachdevadeeksha5) |
| status: | New → Confirmed |
Revision history for this message
| OpenStack Infra (hudson-openstack) wrote Fix proposed to magnum (master) | #2 |
| Changed in magnum: | |
| status: | Confirmed → In Progress |
| Changed in manila: | |
| assignee: | caoyue (yue-cao) → Tin Lam (tl3438) |
| status: | New → In Progress |
Revision history for this message
| OpenStack Infra (hudson-openstack) wrote Fix merged to manila (master) | #3 |
| Changed in manila: | |
| status: | In Progress → Fix Released |
Revision history for this message
| OpenStack Infra (hudson-openstack) wrote Fix merged to magnum (master) | #4 |
| Changed in magnum: | |
| status: | In Progress → Fix Released |
Revision history for this message
| Thierry Carrez (ttx) wrote Fix included in openstack/manila 3.0.0.0b1 | #5 |
Revision history for this message
| OpenStack Infra (hudson-openstack) wrote Fix included in openstack/magnum 3.0.0 | #6 |
Revision history for this message
| OpenStack Infra (hudson-openstack) wrote Fix proposed to ceilometer (master) | #7 |
| Changed in ceilometer: | |
| assignee: | nobody → Tuan (tuanla) |
| status: | New → In Progress |
Revision history for this message
| OpenStack Infra (hudson-openstack) wrote Change abandoned on ceilometer (master) | #8 |
| no longer affects: | ceilometer |
To post a comment you must log in.
Reviewed: https:/
Committed: https:/
Submitter: Jenkins
Branch: master
commit 8d0b16292d0bb00
Author: Tin Lam <email address hidden>
Date: Thu Mar 10 19:10:46 2016 -0600
Add hacking check to ensure not use xrange()
Added hacking check to ensure not to use xrange. Also,
fixed issues with EoF missing newline [W391].
Change-Id: Iba8d240c042e46
Closes-Bug: #1538118