Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-12-28
16:00:41 openstackgerrit Guoqiang Ding proposed openstack/python-novaclient master: Update new documentation PTI jobs https://review.openstack.org/530372
16:19:43 edleafe dansmith: tired of playing with your new toys already?
16:22:43 openstackgerrit Merged openstack/python-novaclient master: boot: show warning if more than one match when setting --image-with https://review.openstack.org/530187
23:11:47 openstackgerrit Jianghua Wang proposed openstack/nova master: XenAPI: Unit tests must mock os_xenapi calls https://review.openstack.org/499573
#openstack-nova - 2017-12-29
02:18:53 openstackgerrit Chen Hanxiao proposed openstack/python-novaclient master: boot: error out if no images match the property from --image-with https://review.openstack.org/530426
03:18:50 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Add index(instance_uuid, updated_at) on instance_actions table https://review.openstack.org/530429
06:28:46 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Add index(instance_uuid, updated_at) on instance_actions table https://review.openstack.org/530429
07:22:57 openstackgerrit OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.openstack.org/524795
07:27:46 openstackgerrit Minho Ban proposed openstack/nova master: remove meaningless if statement https://review.openstack.org/530445
08:36:08 openstackgerrit Balazs Gibizer proposed openstack/nova master: Add regression test for bug 1735407 https://review.openstack.org/526095
08:36:09 openstackgerrit Balazs Gibizer proposed openstack/nova master: Add late server group policy check to rebuild https://review.openstack.org/525242
08:36:09 openstack bug 1735407 in OpenStack Compute (nova) "[Nova] Evacuation doesn't respect anti-affinity rules" [Medium,In progress] https://launchpad.net/bugs/1735407 - Assigned to Balazs Gibizer (balazs-gibizer)
09:04:00 openstackgerrit Balazs Gibizer proposed openstack/nova master: Add sample test for instance audit https://review.openstack.org/480955
09:11:38 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: [libvirt] Add _get_vcpu_realtime_scheduler() https://review.openstack.org/527630
09:11:39 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: [libvirt] Add _get_XXXpin_cpuset() https://review.openstack.org/527631
09:11:39 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: [libvirt] Add _get_numa_memnode() https://review.openstack.org/529906
09:11:40 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: Add NumaTopology support for libvirt/qemu driver https://review.openstack.org/530451
10:07:07 gibi_away stephenfin: I left couple questions in https://review.openstack.org/#/c/527472
10:18:51 openstackgerrit Balazs Gibizer proposed openstack/nova master: Add late server group policy check to rebuild https://review.openstack.org/525242
10:26:14 catmando hey all
10:26:18 catmando anyone awake?
11:05:09 openstackgerrit Merged openstack/nova master: XenAPI: Unit tests must mock os_xenapi calls https://review.openstack.org/499573
12:09:34 openstackgerrit Merged openstack/nova master: trivial: Rename 'policy_check' -> 'policy' https://review.openstack.org/515148
12:09:48 openstackgerrit Stephen Finucane proposed openstack/nova master: Address nits in I46d483f9de6776db1b025f925890624e5e682ada https://review.openstack.org/530462
12:10:04 stephenfin gibi_away: Thanks for reviewing that :) I've addressed the comments in the follow-up patch above. I can squash it in if you'd prefer, but I'd rather not lose jaypipes' +2
12:10:29 stephenfin gibi_away: Let me know what you'd like me to do
14:17:58 openstackgerrit Lee Yarwood proposed openstack/nova master: libvirt: Introduce disk encryption config classes https://review.openstack.org/464008
14:40:18 openstackgerrit Brianna Poulos proposed openstack/nova master: Add trusted_certs object https://review.openstack.org/489408
14:40:19 openstackgerrit Brianna Poulos proposed openstack/nova master: Add trusted_image_certificates to REST API https://review.openstack.org/486204
15:07:55 guoguo pike qemu 2.10 Can not attach a volume to a instance
15:43:09 jaypipes stephenfin: I'm here if you need me to re-approve something.
17:34:14 openstackgerrit Stephen Finucane proposed openstack/nova master: Add an online migration for PciDevice.uuid https://review.openstack.org/530487
17:36:13 stephenfin jaypipes: Fancy taking a look at that ^ ?
17:36:29 stephenfin Based on https://review.openstack.org/#/c/525599/
17:37:26 stephenfin jaypipes: As for your question earlier, should I squash this in? https://review.openstack.org/#/c/530462/1
17:39:20 jaypipes stephenfin: yes, in about half hour
17:39:38 stephenfin No rush. It's nice-to-have yoke
17:39:42 stephenfin *thing
17:42:23 openstackgerrit Stephen Finucane proposed openstack/nova master: Add an online migration for PciDevice.uuid https://review.openstack.org/530487
20:27:58 openstackgerrit Merged openstack/python-novaclient master: Update new documentation PTI jobs https://review.openstack.org/530372
20:57:04 mnaser https://github.com/openstack/nova/blob/stable/pike/nova/policies/volumes_attachments.py
20:57:15 mnaser anyone knows why update is RULE_ADMIN_API but everything else is RULE_ADMIN_OR_OWNER
20:57:55 mnaser https://github.com/openstack/nova/blob/mitaka-eol/etc/nova/policy.json#L193
20:58:04 mnaser oddly enough, it was admin_or_owner before
20:59:50 mnaser https://github.com/openstack/nova/commit/f738483e843fc27379b85c5401859ccc854adc5e
21:00:11 mnaser but retyping a volume can result in a volume migration in the backend
21:00:17 mnaser so im not sure if thats a cinder or nova bug
21:06:04 mnaser well - https://bugs.launchpad.net/cinder/+bug/1740544 -- if anyone's interested
21:06:05 openstack Launchpad bug 1740544 in Cinder "Volume retype fails when migration occurs" [Undecided,New]
22:00:55 lbragstad mnaser: interesting, it looks service token related
22:01:26 mnaser lbragstad: some conversation about this happened in #openstack-cinder if you want to read over
22:01:49 smcginnis lbragstad: Yeah, not token related.
22:02:07 lbragstad ack
#openstack-nova - 2017-12-30
04:43:04 guoguo pike qemu 2.10 Can not attach a volume to a instance
07:23:32 openstackgerrit OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.openstack.org/524795
10:16:21 gibi_away stephenfin: thanks for the follow up patch. I +W on the original
11:40:16 openstackgerrit Merged openstack/nova master: Add PCI NUMA policies https://review.openstack.org/527472
11:47:17 openstackgerrit Merged openstack/nova master: trivial: Modify signature of _filter_non_requested_pfs https://review.openstack.org/527473
12:07:46 openstackgerrit Chen Hanxiao proposed openstack/python-novaclient master: boot: error out if no images match the property from --image-with https://review.openstack.org/530426
19:53:19 openstackgerrit Maciej Kucia proposed openstack/nova-specs master: Non-unique network names in Servers IPs API response https://review.openstack.org/521392
20:27:45 openstackgerrit Maciej Kucia proposed openstack/nova master: SR-IOV passthrough: Check PF only if VF is enabled https://review.openstack.org/476642
#openstack-nova - 2017-12-31
07:22:52 openstackgerrit OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.openstack.org/524795
#openstack-nova - 2018-01-01
22:35:33 openstackgerrit Andreas Karis proposed openstack/nova master: Add debug output for selected page size https://review.openstack.org/530662
22:53:29 openstackgerrit Merged openstack/nova master: Remove 'nova-manage logs' command https://review.openstack.org/522133
#openstack-nova - 2018-01-02
01:34:24 openstackgerrit Eli Qiao proposed openstack/nova master: Api-guide: Add Block Device Mapping https://review.openstack.org/522084
06:54:31 openstackgerrit Ameed Ashour proposed openstack/nova master: change instance_system_metadata column type https://review.openstack.org/526900
07:00:31 gmann__ alex_xu: ping
07:01:20 gmann__ alex_xu: regarding this - https://review.openstack.org/#/c/526258/3/nova/policies/security_groups.py
07:01:55 gmann__ alex_xu: in server response we show only security_groups name not details like you mentioned the case of keypair.
07:03:07 gmann__ alex_xu i got your point of leaking information of sec group in server show if sec group things are hidden via policy on os-security-group api
07:03:53 gmann__ alex_xu: i am considering the same case for keypair also.
07:05:19 gmann__ but with open question like - is name (sec group, key pair) something we should hide in server response if sec group or keypair specific APIs are not permit in policy ?
07:05:57 gmann__ or we consider that is not harmful to show only name in server response without policy control .
07:06:38 openstackgerrit Jianle He proposed openstack/nova-specs master: Back up volume-backed-instance https://review.openstack.org/530214
07:10:56 alex_xu gmann__: ah, right, we only have the sec_group name in the server API
07:11:18 gmann__ yea
07:12:18 alex_xu gmann__: If the admin doesn't allow the user to access the keypairs API, I guess the admin doesn't want to know the detail about the keypairs
07:12:23 alex_xu same as sec_group
07:12:34 gmann__ yea
07:13:11 alex_xu gmann__: so I think we are good
07:14:06 gmann__ alex_xu: ok, thanks. please check the deprecation msg if looks good or we need more/short info.
07:14:41 alex_xu gmann__: I think they are good
07:14:51 gmann__ k
07:15:58 openstackgerrit OpenStack Proposal Bot proposed openstack/python-novaclient master: Updated from global requirements https://review.openstack.org/530697
07:17:12 gmann__ alex_xu: thanks for review :)
07:18:46 openstackgerrit OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.openstack.org/524795
07:20:09 alex_xu gmann__: np
07:24:40 openstackgerrit Ameed Ashour proposed openstack/nova master: change instance_system_metadata column type https://review.openstack.org/526900
07:24:41 openstackgerrit Chen Hanxiao proposed openstack/nova master: manager: more detailed info of unsupported compute driver https://review.openstack.org/530701
07:45:56 openstackgerrit Ameed Ashour proposed openstack/nova master: change instance_system_metadata column type https://review.openstack.org/526900
09:02:25 openstackgerrit Ildiko Vancsa proposed openstack/nova master: WIP: Allow multi-attach in compute api https://review.openstack.org/271047
09:02:25 openstackgerrit Ildiko Vancsa proposed openstack/nova master: WIP: libvirt: Allow multiple volume attachments https://review.openstack.org/267587
09:48:48 openstackgerrit Ildiko Vancsa proposed openstack/nova master: [libvirt] Allow multiple volume attachments https://review.openstack.org/267587
09:48:49 openstackgerrit Ildiko Vancsa proposed openstack/nova master: [api] Allow multi-attach in compute api https://review.openstack.org/271047
10:09:37 openstackgerrit Lee Yarwood proposed openstack/nova master: libvirt: Refactor encryptor attach and detach calls https://review.openstack.org/460243
10:09:38 openstackgerrit Lee Yarwood proposed openstack/nova master: libvirt: Introduce disk encryption config classes https://review.openstack.org/464008
10:35:39 openstackgerrit Rajesh Tailor proposed openstack/nova master: Host addition host-aggregate should be case-sensitive https://review.openstack.org/498334
12:01:31 openstackgerrit Stephen Finucane proposed openstack/nova master: Address nits in I46d483f9de6776db1b025f925890624e5e682ada https://review.openstack.org/530462
12:19:48 ameeda Hi
12:19:57 ameeda can you please review my code here https://review.openstack.org/#/c/528069/ ?

Earlier   Later