Earlier  
Posted Nick Remark
#openstack-nova - 2017-12-29
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: libvirt: Allow multiple volume attachments https://review.openstack.org/267587
09:02:25 openstackgerrit Ildiko Vancsa proposed openstack/nova master: WIP: Allow multi-attach in compute api https://review.openstack.org/271047
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/ ?
12:21:59 bhagyashris ianw_pto: Hi, are you around?
12:53:39 ameeda can you please review my code here https://review.openstack.org/#/c/528069/ ?
12:54:29 openstackgerrit Takashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in delete (2) https://review.openstack.org/526263
12:54:47 openstackgerrit Takashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in delete (3) https://review.openstack.org/526557
12:55:04 openstackgerrit Takashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in post https://review.openstack.org/526823
12:55:25 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Parameter verification for servers.inc https://review.openstack.org/528201
12:55:39 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Example verification for servers.inc https://review.openstack.org/529520
12:56:22 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Fix parameter order in rebuild https://review.openstack.org/529718
12:57:05 openstackgerrit Takashi NATSUME proposed openstack/python-novaclient master: Add CLI to show instance usage audit logs https://review.openstack.org/409413
12:57:32 openstackgerrit Takashi NATSUME proposed openstack/nova master: List/show all server migration types (1/2) https://review.openstack.org/430608
12:58:01 openstackgerrit Takashi NATSUME proposed openstack/nova master: List/show all server migration types (2/2) https://review.openstack.org/459483
12:58:37 openstackgerrit Takashi NATSUME proposed openstack/nova master: [cellv2] Improve getting BDMs in multiple cells https://review.openstack.org/521400
12:58:57 openstackgerrit Takashi NATSUME proposed openstack/nova master: Fix 500 error while passing 4-byte unicode data https://review.openstack.org/407514
12:59:15 openstackgerrit Takashi NATSUME proposed openstack/python-novaclient master: Microversion 2.59 - List/Show all server migration types https://review.openstack.org/430839
12:59:44 openstackgerrit Takashi NATSUME proposed openstack/nova master: [placement] Add functional tests for resource class API https://review.openstack.org/524506
12:59:57 openstackgerrit Takashi NATSUME proposed openstack/nova master: [placement] Add functional tests for traits API https://review.openstack.org/524094
13:00:12 openstackgerrit Takashi NATSUME proposed openstack/nova master: Add a warning in 'nova-manage cell_v2 delete_cell' https://review.openstack.org/513771
13:00:32 openstackgerrit Takashi NATSUME proposed openstack/nova master: Fix the order of target host checks https://review.openstack.org/526225

Earlier   Later