[Yahoo-eng-team] [Bug 1659169] [NEW] Nested loop breaking out doesn't handle correctly

2017-01-24 Thread Zhenguo Niu
Public bug reported:

https://github.com/openstack/nova/blob/master/nova/network/neutronv2/api.py#L1674-L1681

for p in ports:
for ip in p['fixed_ips']:
if ip['ip_address'] == address:
port_id = p['id']
break

in the above nested loops, when we find the port_id, it just break one
for loop, the outside loop will continue, it's a bit waste, should
handle this gracefully.

** Affects: nova
 Importance: Undecided
 Assignee: Zhenguo Niu (niu-zglinux)
 Status: In Progress

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to OpenStack Compute (nova).
https://bugs.launchpad.net/bugs/1659169

Title:
  Nested loop breaking out doesn't handle correctly

Status in OpenStack Compute (nova):
  In Progress

Bug description:
  
https://github.com/openstack/nova/blob/master/nova/network/neutronv2/api.py#L1674-L1681

  for p in ports:
  for ip in p['fixed_ips']:
  if ip['ip_address'] == address:
  port_id = p['id']
  break

  in the above nested loops, when we find the port_id, it just break one
  for loop, the outside loop will continue, it's a bit waste, should
  handle this gracefully.

To manage notifications about this bug go to:
https://bugs.launchpad.net/nova/+bug/1659169/+subscriptions

-- 
Mailing list: https://launchpad.net/~yahoo-eng-team
Post to : yahoo-eng-team@lists.launchpad.net
Unsubscribe : https://launchpad.net/~yahoo-eng-team
More help   : https://help.launchpad.net/ListHelp


[Yahoo-eng-team] [Bug 1587708] Re: RebootInstance should be a soft reboot

2016-05-31 Thread Zhenguo Niu
we use 'hard reboot' and 'soft reboot' explicitly, so this bug is
invalid, sorry for the confusion.

** Changed in: horizon
   Status: New => Invalid

** Changed in: horizon
 Assignee: Zhenguo Niu (niu-zglinux) => (unassigned)

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to OpenStack Dashboard (Horizon).
https://bugs.launchpad.net/bugs/1587708

Title:
  RebootInstance should be a soft reboot

Status in OpenStack Dashboard (Horizon):
  Invalid

Bug description:
  We can soft and hard reboot a running instance, A soft reboot attempts
  a graceful shut down and restart of the instance. A hard reboot power
  cycles the instance, by default when you reboot an instance, is a soft
  reboot in nova CLI, and you need to explicitly pass a --hard parameter
  to perform a hard reboot, but on horizon, RebootInstance is a hard
  reboot.

To manage notifications about this bug go to:
https://bugs.launchpad.net/horizon/+bug/1587708/+subscriptions

-- 
Mailing list: https://launchpad.net/~yahoo-eng-team
Post to : yahoo-eng-team@lists.launchpad.net
Unsubscribe : https://launchpad.net/~yahoo-eng-team
More help   : https://help.launchpad.net/ListHelp


[Yahoo-eng-team] [Bug 1587708] [NEW] RebootInstance should be a soft reboot

2016-05-31 Thread Zhenguo Niu
Public bug reported:

We can soft and hard reboot a running instance, A soft reboot attempts a
graceful shut down and restart of the instance. A hard reboot power
cycles the instance, by default when you reboot an instance, is a soft
reboot in nova CLI, and you need to explicitly pass a --hard parameter
to perform a hard reboot, but on horizon, RebootInstance is a hard
reboot.

** Affects: horizon
 Importance: Undecided
 Assignee: Zhenguo Niu (niu-zglinux)
 Status: New

** Changed in: horizon
 Assignee: (unassigned) => Zhenguo Niu (niu-zglinux)

** Summary changed:

- RebootInstance should be soft reboot instead of hard reboot
+ RebootInstance should be a soft reboot

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to OpenStack Dashboard (Horizon).
https://bugs.launchpad.net/bugs/1587708

Title:
  RebootInstance should be a soft reboot

Status in OpenStack Dashboard (Horizon):
  New

Bug description:
  We can soft and hard reboot a running instance, A soft reboot attempts
  a graceful shut down and restart of the instance. A hard reboot power
  cycles the instance, by default when you reboot an instance, is a soft
  reboot in nova CLI, and you need to explicitly pass a --hard parameter
  to perform a hard reboot, but on horizon, RebootInstance is a hard
  reboot.

To manage notifications about this bug go to:
https://bugs.launchpad.net/horizon/+bug/1587708/+subscriptions

-- 
Mailing list: https://launchpad.net/~yahoo-eng-team
Post to : yahoo-eng-team@lists.launchpad.net
Unsubscribe : https://launchpad.net/~yahoo-eng-team
More help   : https://help.launchpad.net/ListHelp


[Yahoo-eng-team] [Bug 1583025] [NEW] Warning in extract_messages tox environment

2016-05-18 Thread Zhenguo Niu
Public bug reported:

When running the extract_messages tox environment the following warning
is issued:

WARNING:test command found but not installed in testenv
  cmd: /bin/rm
  env: /home/jenkins/workspace/gate-horizon-pep8/.tox/pep8
Maybe you forgot to specify a dependency? See also the whitelist_externals 
envconfig setting.

** Affects: horizon
 Importance: Undecided
 Assignee: Zhenguo Niu (niu-zglinux)
 Status: New

** Changed in: horizon
 Assignee: (unassigned) => Zhenguo Niu (niu-zglinux)

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to OpenStack Dashboard (Horizon).
https://bugs.launchpad.net/bugs/1583025

Title:
  Warning in extract_messages tox environment

Status in OpenStack Dashboard (Horizon):
  New

Bug description:
  When running the extract_messages tox environment the following
  warning is issued:

  WARNING:test command found but not installed in testenv
cmd: /bin/rm
env: /home/jenkins/workspace/gate-horizon-pep8/.tox/pep8
  Maybe you forgot to specify a dependency? See also the whitelist_externals 
envconfig setting.

To manage notifications about this bug go to:
https://bugs.launchpad.net/horizon/+bug/1583025/+subscriptions

-- 
Mailing list: https://launchpad.net/~yahoo-eng-team
Post to : yahoo-eng-team@lists.launchpad.net
Unsubscribe : https://launchpad.net/~yahoo-eng-team
More help   : https://help.launchpad.net/ListHelp


[Yahoo-eng-team] [Bug 1582680] [NEW] Ironic: wrong check for ready to deploy

2016-05-17 Thread Zhenguo Niu
Public bug reported:

In ironic driver, before really do deploy, we will first validate
whether the node is ready to deploy by calling ironic node.validate
method.

# validate we are ready to do the deploy
validate_chk = self.ironicclient.call("node.validate", node_uuid)
if not validate_chk.deploy or not validate_chk.power:
# something is wrong. undo what we have done

Here, the condition validte_chk.deploy and validate_chk.power will
always be true, as they are dict like {deploy: {reason: "", result:
False}, power: {result: False}}

so we should change to check validate_chk.deploy.get('result') and
validate_chk.power.get('result').

** Affects: nova
 Importance: Undecided
     Assignee: Zhenguo Niu (niu-zglinux)
 Status: New

** Changed in: nova
 Assignee: (unassigned) => Zhenguo Niu (niu-zglinux)

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to OpenStack Compute (nova).
https://bugs.launchpad.net/bugs/1582680

Title:
  Ironic: wrong check for ready to deploy

Status in OpenStack Compute (nova):
  New

Bug description:
  In ironic driver, before really do deploy, we will first validate
  whether the node is ready to deploy by calling ironic node.validate
  method.

  # validate we are ready to do the deploy
  validate_chk = self.ironicclient.call("node.validate", node_uuid)
  if not validate_chk.deploy or not validate_chk.power:
  # something is wrong. undo what we have done

  Here, the condition validte_chk.deploy and validate_chk.power will
  always be true, as they are dict like {deploy: {reason: "",
  result: False}, power: {result: False}}

  so we should change to check validate_chk.deploy.get('result') and
  validate_chk.power.get('result').

To manage notifications about this bug go to:
https://bugs.launchpad.net/nova/+bug/1582680/+subscriptions

-- 
Mailing list: https://launchpad.net/~yahoo-eng-team
Post to : yahoo-eng-team@lists.launchpad.net
Unsubscribe : https://launchpad.net/~yahoo-eng-team
More help   : https://help.launchpad.net/ListHelp


[Yahoo-eng-team] [Bug 1549068] [NEW] Ironic vif_port_id mismatch with neutron port id if specified multi networks

2016-02-23 Thread Zhenguo Niu
Public bug reported:

when booting an ironic instance with multi networks, we create the
network resources with the macs get from ironic node ports, and then set
the neutron port id back to ironic port's extra/vif_port_id, but the
current logic will lead the vif-pif ids pair mismatch.

code:

for vif, pif in zip(network_info, ports):
  port_id = six.text_type(vif['id'])
  patch = [{'op': 'add',
   'path': '/extra/vif_port_id',
   'value':  port_id}]
  self.ironicclient.call("port.update", pif.uuid, patch)

we should check whether the mac addresses match between vif and pif
before setting the 'vif_port_id'.

** Affects: ironic
 Importance: Undecided
 Status: New

** Affects: nova
 Importance: Undecided
 Assignee: Zhenguo Niu (niu-zglinux)
 Status: New

** Changed in: nova
 Assignee: (unassigned) => Zhenguo Niu (niu-zglinux)

** Description changed:

  when booting an ironic instance with multi networks, we create the
  network resources with the macs get from ironic node ports, and then set
  the neutron port id back to ironic port's extra/vif_port_id, but the
  current logic will lead the vif-pif ids pair mismatch.
  
  code:
  
  for vif, pif in zip(network_info, ports):
- port_id = six.text_type(vif['id'])
- patch = [{'op': 'add',
-'path': '/extra/vif_port_id',
-'value':  port_id}]
- self.ironicclient.call("port.update", pif.uuid, patch)
+   port_id = six.text_type(vif['id'])
+   patch = [{'op': 'add',
+    'path': '/extra/vif_port_id',
+    'value':  port_id}]
+   self.ironicclient.call("port.update", pif.uuid, patch)
  
  we should check whether the mac addresses match between vif and pif
  before setting the 'vif_port_id'.

** Also affects: ironic
   Importance: Undecided
   Status: New

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to OpenStack Compute (nova).
https://bugs.launchpad.net/bugs/1549068

Title:
  Ironic vif_port_id mismatch with neutron port id if specified multi
  networks

Status in Ironic:
  New
Status in OpenStack Compute (nova):
  New

Bug description:
  when booting an ironic instance with multi networks, we create the
  network resources with the macs get from ironic node ports, and then
  set the neutron port id back to ironic port's extra/vif_port_id, but
  the current logic will lead the vif-pif ids pair mismatch.

  code:

  for vif, pif in zip(network_info, ports):
    port_id = six.text_type(vif['id'])
    patch = [{'op': 'add',
     'path': '/extra/vif_port_id',
     'value':  port_id}]
    self.ironicclient.call("port.update", pif.uuid, patch)

  we should check whether the mac addresses match between vif and pif
  before setting the 'vif_port_id'.

To manage notifications about this bug go to:
https://bugs.launchpad.net/ironic/+bug/1549068/+subscriptions

-- 
Mailing list: https://launchpad.net/~yahoo-eng-team
Post to : yahoo-eng-team@lists.launchpad.net
Unsubscribe : https://launchpad.net/~yahoo-eng-team
More help   : https://help.launchpad.net/ListHelp


[Yahoo-eng-team] [Bug 1495976] [NEW] Image location help text only reference http

2015-09-15 Thread Zhenguo Niu
Public bug reported:

Glance supports http/https url to upload, but Image location field help
text only reference http, should add https as well to keep consistent
with the backend.

** Affects: horizon
 Importance: Undecided
 Assignee: Zhenguo Niu (niu-zglinux)
 Status: In Progress

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to OpenStack Dashboard (Horizon).
https://bugs.launchpad.net/bugs/1495976

Title:
  Image location help text only reference http

Status in OpenStack Dashboard (Horizon):
  In Progress

Bug description:
  Glance supports http/https url to upload, but Image location field
  help text only reference http, should add https as well to keep
  consistent with the backend.

To manage notifications about this bug go to:
https://bugs.launchpad.net/horizon/+bug/1495976/+subscriptions

-- 
Mailing list: https://launchpad.net/~yahoo-eng-team
Post to : yahoo-eng-team@lists.launchpad.net
Unsubscribe : https://launchpad.net/~yahoo-eng-team
More help   : https://help.launchpad.net/ListHelp


[Yahoo-eng-team] [Bug 1493576] Re: Incorrect usage of python-novaclient

2015-09-09 Thread Zhenguo Niu
** Also affects: mistral
   Importance: Undecided
   Status: New

** Changed in: mistral
 Assignee: (unassigned) => Zhenguo Niu (niu-zglinux)

** Description changed:

  All projects should use only `novaclient.client` as entry point. It designed 
with some version checks and backward compatibility.
  Direct import of versioned client object(i.e. novaclient.v2.client) is a way 
to "shoot yourself in the foot".
  
  Python-novaclient's doc: http://docs.openstack.org/developer/python-
  novaclient/api.html
  
  Horizon: 
https://github.com/openstack/horizon/blob/69d6d50ef4a26e2629643ed35ebd661e82e10586/openstack_dashboard/api/nova.py#L31
  Manila: 
https://github.com/openstack/manila/blob/473b46f6edc511deaba88b48392b62bfbb979787/manila/compute/nova.py#L23
  Cinder: 
https://github.com/openstack/cinder/blob/de64f5ad716676b7180365798efc3ea69a4fef0e/cinder/compute/nova.py#L23
+ Mistral:
+ 
https://github.com/openstack/mistral/blob/master/mistral/actions/openstack/actions.py#L23

** Changed in: mistral
   Status: New => In Progress

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to OpenStack Dashboard (Horizon).
https://bugs.launchpad.net/bugs/1493576

Title:
  Incorrect usage of python-novaclient

Status in Cinder:
  New
Status in OpenStack Dashboard (Horizon):
  Fix Committed
Status in Manila:
  In Progress
Status in Mistral:
  In Progress

Bug description:
  All projects should use only `novaclient.client` as entry point. It designed 
with some version checks and backward compatibility.
  Direct import of versioned client object(i.e. novaclient.v2.client) is a way 
to "shoot yourself in the foot".

  Python-novaclient's doc: http://docs.openstack.org/developer/python-
  novaclient/api.html

  Horizon: 
https://github.com/openstack/horizon/blob/69d6d50ef4a26e2629643ed35ebd661e82e10586/openstack_dashboard/api/nova.py#L31
  Manila: 
https://github.com/openstack/manila/blob/473b46f6edc511deaba88b48392b62bfbb979787/manila/compute/nova.py#L23
  Cinder: 
https://github.com/openstack/cinder/blob/de64f5ad716676b7180365798efc3ea69a4fef0e/cinder/compute/nova.py#L23
  Mistral:
  
https://github.com/openstack/mistral/blob/master/mistral/actions/openstack/actions.py#L23

To manage notifications about this bug go to:
https://bugs.launchpad.net/cinder/+bug/1493576/+subscriptions

-- 
Mailing list: https://launchpad.net/~yahoo-eng-team
Post to : yahoo-eng-team@lists.launchpad.net
Unsubscribe : https://launchpad.net/~yahoo-eng-team
More help   : https://help.launchpad.net/ListHelp


[Yahoo-eng-team] [Bug 1480001] [NEW] Little improvement of errors show on volume Extend form

2015-07-30 Thread Zhenguo Niu
Public bug reported:

Now when extending a volume to a new size which is less then the current
one, we will pop up an error warning message on the top of the form instead
of beside the new size field. now change to use field specified error class
for more accurate.

And as Kirill Zaitsev suggested, it 
would be even better if we could incorporate 
https://docs.djangoproject.com/en/1.8/ref/forms/api/#django.forms.Form.add_error,
 which would not make us fiddle with form internals. but it was only added in 
django 1.7, and we support up to 1.5 but moving forward with django support 
seems like a nice idea.

** Affects: horizon
 Importance: Undecided
 Assignee: Zhenguo Niu (niu-zglinux)
 Status: In Progress

** Changed in: horizon
 Assignee: (unassigned) = Zhenguo Niu (niu-zglinux)

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to OpenStack Dashboard (Horizon).
https://bugs.launchpad.net/bugs/1480001

Title:
  Little improvement of errors show on volume Extend form

Status in OpenStack Dashboard (Horizon):
  In Progress

Bug description:
  Now when extending a volume to a new size which is less then the current
  one, we will pop up an error warning message on the top of the form instead
  of beside the new size field. now change to use field specified error class
  for more accurate.

  And as Kirill Zaitsev suggested, it 
  would be even better if we could incorporate 
https://docs.djangoproject.com/en/1.8/ref/forms/api/#django.forms.Form.add_error,
 which would not make us fiddle with form internals. but it was only added in 
django 1.7, and we support up to 1.5 but moving forward with django support 
seems like a nice idea.

To manage notifications about this bug go to:
https://bugs.launchpad.net/horizon/+bug/1480001/+subscriptions

-- 
Mailing list: https://launchpad.net/~yahoo-eng-team
Post to : yahoo-eng-team@lists.launchpad.net
Unsubscribe : https://launchpad.net/~yahoo-eng-team
More help   : https://help.launchpad.net/ListHelp


[Yahoo-eng-team] [Bug 1479727] [NEW] Exceptions when user email field is empty

2015-07-30 Thread Zhenguo Niu
Public bug reported:

Keystone doesn't return a user email in it's json response if the field
is empty, which will raise exceptions when loading users table:

The attribute email doesn't exist on User {u'username': u'nova', u'enabled': 
True, u'name': u'nova', u'id': u'1b57074e239b49c8b77175d0107dd2a1'}.
The attribute email doesn't exist on User {u'username': u'glance', u'enabled': 
True, u'name': u'glance', u'id': u'24aa02268a22413b9cb922283273387b'}.
The attribute email doesn't exist on User {u'username': u'cinder', u'enabled': 
True, u'name': u'cinder', u'id': u'532d95cdcd914e95a0d29178161dba65'}.
The attribute email doesn't exist on User {u'username': u'admin', u'enabled': 
True, u'name': u'admin', u'id': u'67bffb7f749a40fa898453ebf5ef83bb'}.
The attribute email doesn't exist on User {u'username': u'neutron', 
u'enabled': True, u'name': u'neutron', u'id': 
u'a5f96e136c124737be25e59ec66af08b'}.
The attribute email doesn't exist on User {u'username': u'ironic', u'enabled': 
True, u'name': u'ironic', u'id': u'c09d491925ed42709a7aaf9b7fe50928'}.

** Affects: horizon
 Importance: Undecided
 Assignee: Zhenguo Niu (niu-zglinux)
 Status: New

** Changed in: horizon
 Assignee: (unassigned) = Zhenguo Niu (niu-zglinux)

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to OpenStack Dashboard (Horizon).
https://bugs.launchpad.net/bugs/1479727

Title:
  Exceptions when user email field is empty

Status in OpenStack Dashboard (Horizon):
  New

Bug description:
  Keystone doesn't return a user email in it's json response if the
  field is empty, which will raise exceptions when loading users table:

  The attribute email doesn't exist on User {u'username': u'nova', u'enabled': 
True, u'name': u'nova', u'id': u'1b57074e239b49c8b77175d0107dd2a1'}.
  The attribute email doesn't exist on User {u'username': u'glance', 
u'enabled': True, u'name': u'glance', u'id': 
u'24aa02268a22413b9cb922283273387b'}.
  The attribute email doesn't exist on User {u'username': u'cinder', 
u'enabled': True, u'name': u'cinder', u'id': 
u'532d95cdcd914e95a0d29178161dba65'}.
  The attribute email doesn't exist on User {u'username': u'admin', 
u'enabled': True, u'name': u'admin', u'id': 
u'67bffb7f749a40fa898453ebf5ef83bb'}.
  The attribute email doesn't exist on User {u'username': u'neutron', 
u'enabled': True, u'name': u'neutron', u'id': 
u'a5f96e136c124737be25e59ec66af08b'}.
  The attribute email doesn't exist on User {u'username': u'ironic', 
u'enabled': True, u'name': u'ironic', u'id': 
u'c09d491925ed42709a7aaf9b7fe50928'}.

To manage notifications about this bug go to:
https://bugs.launchpad.net/horizon/+bug/1479727/+subscriptions

-- 
Mailing list: https://launchpad.net/~yahoo-eng-team
Post to : yahoo-eng-team@lists.launchpad.net
Unsubscribe : https://launchpad.net/~yahoo-eng-team
More help   : https://help.launchpad.net/ListHelp


[Yahoo-eng-team] [Bug 1461856] [NEW] Volume retype list should not contain the current volume_type

2015-06-04 Thread Zhenguo Niu
Public bug reported:

Instead of adding an extra check and throw an error message when retype
volume_type is same with the current one, we should exclude it form the
choice list.

** Affects: horizon
 Importance: Undecided
 Assignee: Zhenguo Niu (niu-zglinux)
 Status: New

** Changed in: horizon
 Assignee: (unassigned) = Zhenguo Niu (niu-zglinux)

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to OpenStack Dashboard (Horizon).
https://bugs.launchpad.net/bugs/1461856

Title:
  Volume retype list should not contain the current volume_type

Status in OpenStack Dashboard (Horizon):
  New

Bug description:
  Instead of adding an extra check and throw an error message when
  retype volume_type is same with the current one, we should exclude it
  form the choice list.

To manage notifications about this bug go to:
https://bugs.launchpad.net/horizon/+bug/1461856/+subscriptions

-- 
Mailing list: https://launchpad.net/~yahoo-eng-team
Post to : yahoo-eng-team@lists.launchpad.net
Unsubscribe : https://launchpad.net/~yahoo-eng-team
More help   : https://help.launchpad.net/ListHelp


[Yahoo-eng-team] [Bug 1459904] [NEW] Split unit tests of volumes from admin/volumes/tests.py to it's own test file

2015-05-28 Thread Zhenguo Niu
Public bug reported:

volumes should have it's own test file like snapshots and volume_types.

** Affects: horizon
 Importance: Undecided
 Assignee: Zhenguo Niu (niu-zglinux)
 Status: New

** Changed in: horizon
 Assignee: (unassigned) = Zhenguo Niu (niu-zglinux)

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to OpenStack Dashboard (Horizon).
https://bugs.launchpad.net/bugs/1459904

Title:
  Split unit tests of volumes from admin/volumes/tests.py to it's own
  test file

Status in OpenStack Dashboard (Horizon):
  New

Bug description:
  volumes should have it's own test file like snapshots and
  volume_types.

To manage notifications about this bug go to:
https://bugs.launchpad.net/horizon/+bug/1459904/+subscriptions

-- 
Mailing list: https://launchpad.net/~yahoo-eng-team
Post to : yahoo-eng-team@lists.launchpad.net
Unsubscribe : https://launchpad.net/~yahoo-eng-team
More help   : https://help.launchpad.net/ListHelp


[Yahoo-eng-team] [Bug 1459625] [NEW] TemplateDoesNotExist when click manage/unmanage volume

2015-05-28 Thread Zhenguo Niu
Public bug reported:

manage/unmanage template includes wrong templates:

project/volumes/volumes/_unmanage_volume.html
project/volumes/volumes/_manage_volume.html

it should be admin/volumes/... instead of project/volumes/...

** Affects: horizon
 Importance: Undecided
 Assignee: Zhenguo Niu (niu-zglinux)
 Status: New

** Changed in: horizon
 Assignee: (unassigned) = Zhenguo Niu (niu-zglinux)

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to OpenStack Dashboard (Horizon).
https://bugs.launchpad.net/bugs/1459625

Title:
  TemplateDoesNotExist when click manage/unmanage volume

Status in OpenStack Dashboard (Horizon):
  New

Bug description:
  manage/unmanage template includes wrong templates:

  project/volumes/volumes/_unmanage_volume.html
  project/volumes/volumes/_manage_volume.html

  it should be admin/volumes/... instead of project/volumes/...

To manage notifications about this bug go to:
https://bugs.launchpad.net/horizon/+bug/1459625/+subscriptions

-- 
Mailing list: https://launchpad.net/~yahoo-eng-team
Post to : yahoo-eng-team@lists.launchpad.net
Unsubscribe : https://launchpad.net/~yahoo-eng-team
More help   : https://help.launchpad.net/ListHelp


[Yahoo-eng-team] [Bug 1435153] Re: ironic hypervisor is still available when the node is in maintenance status.

2015-04-24 Thread Zhenguo Niu
I'll upload a patch to address this issue on nova side.

** Also affects: nova
   Importance: Undecided
   Status: New

** Changed in: nova
 Assignee: (unassigned) = Zhenguo Niu (niu-zglinux)

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to OpenStack Compute (nova).
https://bugs.launchpad.net/bugs/1435153

Title:
  ironic hypervisor is still available when the node is in maintenance
  status.

Status in OpenStack Bare Metal Provisioning Service (Ironic):
  Triaged
Status in OpenStack Compute (Nova):
  New

Bug description:
  In my env, we have two ironic nodes, so in hypervisor-stats, show
  count=2, after I set one of them to maintenance state, the hypervisor-
  stats still show count=2, I understand for the node which is in
  maintenance status, it should not be counted into nova hypervisor
  stats.

  [root@rhel7-osee ~]# nova hypervisor-stats
  +--+---+
  | Property | Value |
  +--+---+
  | count| 2 |
  | current_workload | 0 |
  | disk_available_least | 0 |
  | free_disk_gb | 0 |
  | free_ram_mb  | 0 |
  | local_gb | 40|
  | local_gb_used| 40|
  | memory_mb| 2048  |
  | memory_mb_used   | 2048  |
  | running_vms  | 2 |
  | vcpus| 2 |
  | vcpus_used   | 2 |
  +--+---+
  [root@rhel7-osee ~]# ironic node-set-maintenance 
d4edf8c7-ae8d-40ed-b3a6-c5600ff09287 on
  [root@rhel7-osee ~]# nova hypervisor-stats
  +--+---+
  | Property | Value |
  +--+---+
  | count| 2 |
  | current_workload | 0 |
  | disk_available_least | 0 |
  | free_disk_gb | 0 |
  | free_ram_mb  | 0 |
  | local_gb | 40|
  | local_gb_used| 40|
  | memory_mb| 2048  |
  | memory_mb_used   | 2048  |
  | running_vms  | 2 |
  | vcpus| 2 |
  | vcpus_used   | 2 |
  +--+---+

To manage notifications about this bug go to:
https://bugs.launchpad.net/ironic/+bug/1435153/+subscriptions

-- 
Mailing list: https://launchpad.net/~yahoo-eng-team
Post to : yahoo-eng-team@lists.launchpad.net
Unsubscribe : https://launchpad.net/~yahoo-eng-team
More help   : https://help.launchpad.net/ListHelp


[Yahoo-eng-team] [Bug 1398725] [NEW] Missing icon on AddDHCPAgent button

2014-12-02 Thread Zhenguo Niu
Public bug reported:

Admin Networks DHCPAgent table Add button missing '+' icon.

** Affects: horizon
 Importance: Undecided
 Assignee: Zhenguo Niu (niu-zglinux)
 Status: In Progress

** Changed in: horizon
 Assignee: (unassigned) = Zhenguo Niu (niu-zglinux)

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to OpenStack Dashboard (Horizon).
https://bugs.launchpad.net/bugs/1398725

Title:
  Missing icon on AddDHCPAgent button

Status in OpenStack Dashboard (Horizon):
  In Progress

Bug description:
  Admin Networks DHCPAgent table Add button missing '+' icon.

To manage notifications about this bug go to:
https://bugs.launchpad.net/horizon/+bug/1398725/+subscriptions

-- 
Mailing list: https://launchpad.net/~yahoo-eng-team
Post to : yahoo-eng-team@lists.launchpad.net
Unsubscribe : https://launchpad.net/~yahoo-eng-team
More help   : https://help.launchpad.net/ListHelp


[Yahoo-eng-team] [Bug 1369883] Re: detach a volume tips translate error

2014-09-22 Thread Zhenguo Niu
** Changed in: horizon
   Status: New = Won't Fix

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to OpenStack Dashboard (Horizon).
https://bugs.launchpad.net/bugs/1369883

Title:
  detach a volume tips translate error

Status in OpenStack Dashboard (Horizon):
  Won't Fix

Bug description:
  when detach a volume,the tips showing to be 断开zhong云硬盘,if dashboard 
language choose chinese.It should be 断开中云硬盘.
   Maybe there is a translating error with file 
openstack\horizon\openstack_dashboard\locale\zh_CN\LC_MESSAGES\django.po.

To manage notifications about this bug go to:
https://bugs.launchpad.net/horizon/+bug/1369883/+subscriptions

-- 
Mailing list: https://launchpad.net/~yahoo-eng-team
Post to : yahoo-eng-team@lists.launchpad.net
Unsubscribe : https://launchpad.net/~yahoo-eng-team
More help   : https://help.launchpad.net/ListHelp


[Yahoo-eng-team] [Bug 1335688] Re: Inconsistent capitalization on Networks tab under Launch Instance

2014-06-30 Thread Zhenguo Niu
we have already fixed the inconsistent capitalization in j1. see:
https://bugs.launchpad.net/horizon/+bug/1325684

** Changed in: horizon
   Status: In Progress = Invalid

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to OpenStack Dashboard (Horizon).
https://bugs.launchpad.net/bugs/1335688

Title:
  Inconsistent capitalization on Networks tab under Launch Instance

Status in OpenStack Dashboard (Horizon):
  Invalid

Bug description:
  Under Launch Instance - Networks, the two options are Selected
  Networks and Available networks.  Capitalization of networks
  should probably be consistent.

  Obviously minor and doesn't affect usability.

To manage notifications about this bug go to:
https://bugs.launchpad.net/horizon/+bug/1335688/+subscriptions

-- 
Mailing list: https://launchpad.net/~yahoo-eng-team
Post to : yahoo-eng-team@lists.launchpad.net
Unsubscribe : https://launchpad.net/~yahoo-eng-team
More help   : https://help.launchpad.net/ListHelp


[Yahoo-eng-team] [Bug 1331329] [NEW] success_url is unneeded in PasswordView

2014-06-18 Thread Zhenguo Niu
Public bug reported:

Since PasswordForm handle success return
http.HttpResponseRedirect(settings.LOGOUT_URL), it's no need to set
success_url in PasswordView.

see:
https://github.com/openstack/horizon/blob/master/openstack_dashboard/dashboards/settings/password/forms.py#L60

** Affects: horizon
 Importance: Undecided
 Assignee: Zhenguo Niu (niu-zglinux)
 Status: New

** Changed in: horizon
 Assignee: (unassigned) = Zhenguo Niu (niu-zglinux)

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to OpenStack Dashboard (Horizon).
https://bugs.launchpad.net/bugs/1331329

Title:
  success_url is unneeded in PasswordView

Status in OpenStack Dashboard (Horizon):
  New

Bug description:
  Since PasswordForm handle success return
  http.HttpResponseRedirect(settings.LOGOUT_URL), it's no need to set
  success_url in PasswordView.

  see:
  
https://github.com/openstack/horizon/blob/master/openstack_dashboard/dashboards/settings/password/forms.py#L60

To manage notifications about this bug go to:
https://bugs.launchpad.net/horizon/+bug/1331329/+subscriptions

-- 
Mailing list: https://launchpad.net/~yahoo-eng-team
Post to : yahoo-eng-team@lists.launchpad.net
Unsubscribe : https://launchpad.net/~yahoo-eng-team
More help   : https://help.launchpad.net/ListHelp


[Yahoo-eng-team] [Bug 1250029] Re: Default port for The MS SQL Security Group is 1433 instead of 1443

2014-06-05 Thread Zhenguo Niu
** Also affects: puppet-horizon
   Importance: Undecided
   Status: New

** Changed in: puppet-horizon
 Assignee: (unassigned) = Zhenguo Niu (niu-zglinux)

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to OpenStack Dashboard (Horizon).
https://bugs.launchpad.net/bugs/1250029

Title:
  Default port for The MS SQL Security Group is 1433 instead of 1443

Status in OpenStack Dashboard (Horizon):
  Fix Released
Status in OpenStack Dashboard (Horizon) havana series:
  Fix Released
Status in Puppet module for Horizon:
  In Progress

Bug description:
  The default port for the MS SQL Security group is 1433 instead of
  1443.

To manage notifications about this bug go to:
https://bugs.launchpad.net/horizon/+bug/1250029/+subscriptions

-- 
Mailing list: https://launchpad.net/~yahoo-eng-team
Post to : yahoo-eng-team@lists.launchpad.net
Unsubscribe : https://launchpad.net/~yahoo-eng-team
More help   : https://help.launchpad.net/ListHelp


[Yahoo-eng-team] [Bug 884451] Re: End User Has No Forgot Password Option

2014-04-21 Thread Zhenguo Niu
** Also affects: django-openstack-auth
   Importance: Undecided
   Status: New

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to Keystone.
https://bugs.launchpad.net/bugs/884451

Title:
  End User Has No Forgot Password Option

Status in Django OpenStack Auth:
  New
Status in OpenStack Dashboard (Horizon):
  Confirmed
Status in OpenStack Identity (Keystone):
  Incomplete

Bug description:
  Related to blueprint: improve-user-experience.

To manage notifications about this bug go to:
https://bugs.launchpad.net/django-openstack-auth/+bug/884451/+subscriptions

-- 
Mailing list: https://launchpad.net/~yahoo-eng-team
Post to : yahoo-eng-team@lists.launchpad.net
Unsubscribe : https://launchpad.net/~yahoo-eng-team
More help   : https://help.launchpad.net/ListHelp


[Yahoo-eng-team] [Bug 1302316] Re: Ability to toggle Admin Pass fields

2014-04-03 Thread Zhenguo Niu
*** This bug is a duplicate of bug 1250713 ***
https://bugs.launchpad.net/bugs/1250713

** This bug has been marked a duplicate of bug 1250713
   Add option to disable instance admin password fields

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to OpenStack Dashboard (Horizon).
https://bugs.launchpad.net/bugs/1302316

Title:
  Ability to toggle Admin Pass fields

Status in OpenStack Dashboard (Horizon):
  In Progress

Bug description:
  We should have the ability to disable the Admin Pass fields in the
  Launch Instance page. For the admin password to properly be set, it
  requires the guest OS be properly configured to support it.

  The images we support work properly, however, we support the uploading
  of custom images, by tenants which are usually not configured
  properly.

  Rather than confusing the end user with some images that work and
  others that do not, we would rather disable this functionality all
  together.

To manage notifications about this bug go to:
https://bugs.launchpad.net/horizon/+bug/1302316/+subscriptions

-- 
Mailing list: https://launchpad.net/~yahoo-eng-team
Post to : yahoo-eng-team@lists.launchpad.net
Unsubscribe : https://launchpad.net/~yahoo-eng-team
More help   : https://help.launchpad.net/ListHelp


[Yahoo-eng-team] [Bug 1284506] [NEW] Tabbed volumes and volume types tables on Admin panel

2014-02-24 Thread Zhenguo Niu
Public bug reported:

It would be better to put volumes and volume types tables in separate
tabs rather than one page.

** Affects: horizon
 Importance: Undecided
 Assignee: Zhenguo Niu (niu-zglinux)
 Status: New

** Changed in: horizon
 Assignee: (unassigned) = Zhenguo Niu (niu-zglinux)

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to OpenStack Dashboard (Horizon).
https://bugs.launchpad.net/bugs/1284506

Title:
  Tabbed volumes and volume types tables on Admin panel

Status in OpenStack Dashboard (Horizon):
  New

Bug description:
  It would be better to put volumes and volume types tables in separate
  tabs rather than one page.

To manage notifications about this bug go to:
https://bugs.launchpad.net/horizon/+bug/1284506/+subscriptions

-- 
Mailing list: https://launchpad.net/~yahoo-eng-team
Post to : yahoo-eng-team@lists.launchpad.net
Unsubscribe : https://launchpad.net/~yahoo-eng-team
More help   : https://help.launchpad.net/ListHelp


[Yahoo-eng-team] [Bug 1282845] [NEW] Creating volume form snapshot should not display Use a volume as source option

2014-02-20 Thread Zhenguo Niu
Public bug reported:

When user clicks on 'Create Volume' on snapshot table. 'Use a volume as
source' option should not be displayed.

** Affects: horizon
 Importance: Undecided
 Status: New

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to OpenStack Dashboard (Horizon).
https://bugs.launchpad.net/bugs/1282845

Title:
  Creating volume form snapshot should not display Use a volume as
  source option

Status in OpenStack Dashboard (Horizon):
  New

Bug description:
  When user clicks on 'Create Volume' on snapshot table. 'Use a volume
  as source' option should not be displayed.

To manage notifications about this bug go to:
https://bugs.launchpad.net/horizon/+bug/1282845/+subscriptions

-- 
Mailing list: https://launchpad.net/~yahoo-eng-team
Post to : yahoo-eng-team@lists.launchpad.net
Unsubscribe : https://launchpad.net/~yahoo-eng-team
More help   : https://help.launchpad.net/ListHelp


[Yahoo-eng-team] [Bug 1267733] [NEW] Live Migration can set new host to the current host.

2014-01-10 Thread Zhenguo Niu
Public bug reported:

When migrating a server, current host should not contain in the new host
list.

def populate_host_choices(self, request, initial):
hosts = initial.get('hosts')
current_host = initial.get('current_host')
host_list = [(host.hypervisor_hostname,
  host.hypervisor_hostname)
 for host in hosts
 if host.hypervisor_hostname != current_host]    we 
should not use hypervisor_hostname compare with current_host, they can be 
different on the same host.

** Affects: horizon
 Importance: Undecided
 Assignee: Zhenguo Niu (niu-zglinux)
 Status: New

** Changed in: horizon
 Assignee: (unassigned) = Zhenguo Niu (niu-zglinux)

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to OpenStack Dashboard (Horizon).
https://bugs.launchpad.net/bugs/1267733

Title:
  Live Migration can set new host to the current host.

Status in OpenStack Dashboard (Horizon):
  New

Bug description:
  When migrating a server, current host should not contain in the new
  host list.

  def populate_host_choices(self, request, initial):
  hosts = initial.get('hosts')
  current_host = initial.get('current_host')
  host_list = [(host.hypervisor_hostname,
host.hypervisor_hostname)
   for host in hosts
   if host.hypervisor_hostname != current_host]    we 
should not use hypervisor_hostname compare with current_host, they can be 
different on the same host.

To manage notifications about this bug go to:
https://bugs.launchpad.net/horizon/+bug/1267733/+subscriptions

-- 
Mailing list: https://launchpad.net/~yahoo-eng-team
Post to : yahoo-eng-team@lists.launchpad.net
Unsubscribe : https://launchpad.net/~yahoo-eng-team
More help   : https://help.launchpad.net/ListHelp


[Yahoo-eng-team] [Bug 1265140] [NEW] Volume Snapshot name and description can't be edited

2013-12-30 Thread Zhenguo Niu
Public bug reported:

It should be possible to edit the name and description of a volume
snapshot.

** Affects: horizon
 Importance: Undecided
 Assignee: Zhenguo Niu (niu-zglinux)
 Status: New

** Changed in: horizon
 Assignee: (unassigned) = Zhenguo Niu (niu-zglinux)

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to OpenStack Dashboard (Horizon).
https://bugs.launchpad.net/bugs/1265140

Title:
  Volume Snapshot name and description can't be edited

Status in OpenStack Dashboard (Horizon):
  New

Bug description:
  It should be possible to edit the name and description of a volume
  snapshot.

To manage notifications about this bug go to:
https://bugs.launchpad.net/horizon/+bug/1265140/+subscriptions

-- 
Mailing list: https://launchpad.net/~yahoo-eng-team
Post to : yahoo-eng-team@lists.launchpad.net
Unsubscribe : https://launchpad.net/~yahoo-eng-team
More help   : https://help.launchpad.net/ListHelp


[Yahoo-eng-team] [Bug 1264411] [NEW] Volume Name and description can't be edited

2013-12-26 Thread Zhenguo Niu
Public bug reported:

It should be possible to edit the name and description of a volume.

** Affects: horizon
 Importance: Undecided
 Assignee: Zhenguo Niu (niu-zglinux)
 Status: New

** Changed in: horizon
 Assignee: (unassigned) = Zhenguo Niu (niu-zglinux)

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to OpenStack Dashboard (Horizon).
https://bugs.launchpad.net/bugs/1264411

Title:
  Volume Name and description can't be edited

Status in OpenStack Dashboard (Horizon):
  New

Bug description:
  It should be possible to edit the name and description of a volume.

To manage notifications about this bug go to:
https://bugs.launchpad.net/horizon/+bug/1264411/+subscriptions

-- 
Mailing list: https://launchpad.net/~yahoo-eng-team
Post to : yahoo-eng-team@lists.launchpad.net
Unsubscribe : https://launchpad.net/~yahoo-eng-team
More help   : https://help.launchpad.net/ListHelp


[Yahoo-eng-team] [Bug 1260617] Re: Provide the ability to attach volumes in the read-only mode

2013-12-16 Thread Zhenguo Niu
** Also affects: python-novaclient
   Importance: Undecided
   Status: New

** Changed in: python-novaclient
 Assignee: (unassigned) = Zhenguo Niu (niu-zglinux)

** Also affects: nova
   Importance: Undecided
   Status: New

** Changed in: nova
 Assignee: (unassigned) = Zhenguo Niu (niu-zglinux)

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to OpenStack Dashboard (Horizon).
https://bugs.launchpad.net/bugs/1260617

Title:
  Provide the ability to attach volumes in the read-only mode

Status in OpenStack Dashboard (Horizon):
  Confirmed
Status in OpenStack Compute (Nova):
  New
Status in Python client library for Nova:
  New

Bug description:
  Cinder now support the ability to attach volumes in the read-only
  mode, this should be exposed through horizon. Read-only mode could be
  ensured by hypervisor configuration during the attachment. Libvirt,
  Xen, VMware and Hyper-V support R/O volumes.

To manage notifications about this bug go to:
https://bugs.launchpad.net/horizon/+bug/1260617/+subscriptions

-- 
Mailing list: https://launchpad.net/~yahoo-eng-team
Post to : yahoo-eng-team@lists.launchpad.net
Unsubscribe : https://launchpad.net/~yahoo-eng-team
More help   : https://help.launchpad.net/ListHelp


[Yahoo-eng-team] [Bug 1260228] [NEW] Pre-select the network when there's only one.

2013-12-12 Thread Zhenguo Niu
*** This bug is a duplicate of bug 1260227 ***
https://bugs.launchpad.net/bugs/1260227

Public bug reported:

When launching instance with neutron, network field is required. If
there is only one, it should be pre-selected.

** Affects: horizon
 Importance: Undecided
 Status: New

** This bug has been marked a duplicate of bug 1260227
   Pre-select the network when there's only one.

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to OpenStack Dashboard (Horizon).
https://bugs.launchpad.net/bugs/1260228

Title:
  Pre-select the network when there's only one.

Status in OpenStack Dashboard (Horizon):
  New

Bug description:
  When launching instance with neutron, network field is required. If
  there is only one, it should be pre-selected.

To manage notifications about this bug go to:
https://bugs.launchpad.net/horizon/+bug/1260228/+subscriptions

-- 
Mailing list: https://launchpad.net/~yahoo-eng-team
Post to : yahoo-eng-team@lists.launchpad.net
Unsubscribe : https://launchpad.net/~yahoo-eng-team
More help   : https://help.launchpad.net/ListHelp


[Yahoo-eng-team] [Bug 1260617] [NEW] Provide the ability to attach volumes in the read-only mode

2013-12-12 Thread Zhenguo Niu
Public bug reported:

Cinder now support the ability to attach volumes in the read-only mode,
this should be exposed through horizon. Read-only mode could be ensured
by hypervisor configuration during the attachment. Libvirt, Xen, VMware
and Hyper-V support R/O volumes.

** Affects: horizon
 Importance: Undecided
 Status: New

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to OpenStack Dashboard (Horizon).
https://bugs.launchpad.net/bugs/1260617

Title:
  Provide the ability to attach volumes in the read-only mode

Status in OpenStack Dashboard (Horizon):
  New

Bug description:
  Cinder now support the ability to attach volumes in the read-only
  mode, this should be exposed through horizon. Read-only mode could be
  ensured by hypervisor configuration during the attachment. Libvirt,
  Xen, VMware and Hyper-V support R/O volumes.

To manage notifications about this bug go to:
https://bugs.launchpad.net/horizon/+bug/1260617/+subscriptions

-- 
Mailing list: https://launchpad.net/~yahoo-eng-team
Post to : yahoo-eng-team@lists.launchpad.net
Unsubscribe : https://launchpad.net/~yahoo-eng-team
More help   : https://help.launchpad.net/ListHelp


[Yahoo-eng-team] [Bug 1253903] Re: Inappropriate comment for flavor create api in python-novaclient

2013-11-21 Thread Zhenguo Niu
** Also affects: python-novaclient
   Importance: Undecided
   Status: New

** Changed in: python-novaclient
 Assignee: (unassigned) = Zhenguo Niu (niu-zglinux)

** Changed in: horizon
   Status: New = Invalid

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to OpenStack Dashboard (Horizon).
https://bugs.launchpad.net/bugs/1253903

Title:
  Inappropriate comment for flavor create api in python-novaclient

Status in OpenStack Dashboard (Horizon):
  Invalid
Status in Python client library for Nova:
  New

Bug description:
   def create(self, name, ram, vcpus, disk, flavorid=auto,
 ephemeral=0, swap=0, rxtx_factor=1.0, is_public=True):
  
  Create (allocate) a  floating ip for a tenant

  :param name: Descriptive name of the flavor
  :param ram: Memory in MB for the flavor
  :param vcpu: Number of VCPUs for the flavor
  :param disk: Size of local disk in GB
  :param flavorid: ID for the flavor (optional). You can use the 
reserved
   value ``auto`` to have Nova generate a UUID for the
   flavor in cases where you cannot simply pass 
``None``.
  :param swap: Swap space in MB
  :param rxtx_factor: RX/TX factor
  :rtype: :class:`Flavor`
  
  which is not the function for allocating floating ip.This is the flavor 
create function.

To manage notifications about this bug go to:
https://bugs.launchpad.net/horizon/+bug/1253903/+subscriptions

-- 
Mailing list: https://launchpad.net/~yahoo-eng-team
Post to : yahoo-eng-team@lists.launchpad.net
Unsubscribe : https://launchpad.net/~yahoo-eng-team
More help   : https://help.launchpad.net/ListHelp


[Yahoo-eng-team] [Bug 1251310] Re: admin instances details link does not work reliably

2013-11-14 Thread Zhenguo Niu
Admin page only list instances in current project  here

** Changed in: horizon
   Status: New = Invalid

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to OpenStack Dashboard (Horizon).
https://bugs.launchpad.net/bugs/1251310

Title:
  admin instances details link does not work reliably

Status in OpenStack Dashboard (Horizon):
  Invalid

Bug description:
  In the admin instances page, instances may get listed that are in
  several projects, and the api calls contain an admin=true flag to be
  able to list instances everywhere.

  However, the details link is produced like this:
  
https://github.com/openstack/horizon/blob/master/openstack_dashboard/dashboards/admin/instances/tables.py#L91

  When you examine the view this points to, it is a project view:
  
https://github.com/openstack/horizon/blob/master/openstack_dashboard/dashboards/admin/instances/urls.py#L36

  The project views make api calls where admin=true is not specified such as 
this:
  
https://github.com/openstack/horizon/blob/master/openstack_dashboard/dashboards/project/instances/views.py#L220

  This means if you click on a server instance in an outside project the
  details page may not work correctly.

To manage notifications about this bug go to:
https://bugs.launchpad.net/horizon/+bug/1251310/+subscriptions

-- 
Mailing list: https://launchpad.net/~yahoo-eng-team
Post to : yahoo-eng-team@lists.launchpad.net
Unsubscribe : https://launchpad.net/~yahoo-eng-team
More help   : https://help.launchpad.net/ListHelp