Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-05-17
01:42:30 openstackgerrit Artom Lifshitz proposed openstack/nova master: Delete disk metadata when detaching volume https://review.openstack.org/435082
01:42:48 openstackgerrit Artom Lifshitz proposed openstack/nova master: Plumbing for tagged volume attachment https://review.openstack.org/391941
01:42:49 openstackgerrit Artom Lifshitz proposed openstack/nova master: API support for tagged device attachment https://review.openstack.org/376021
01:42:52 openstackgerrit Artom Lifshitz proposed openstack/nova master: Libvirt support for tagged nic attachment https://review.openstack.org/415520
02:30:44 openstackgerrit Huan Xie proposed openstack/nova master: WIP: XenAPI use os-xenapi v2 in nova https://review.openstack.org/453493
03:53:54 openstackgerrit OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/465283
03:56:15 openstackgerrit OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/451049
03:58:18 openstackgerrit OpenStack Proposal Bot proposed openstack/python-novaclient master: Updated from global requirements https://review.openstack.org/465371
05:16:21 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add policy description for 'os-hide-server-addresses' https://review.openstack.org/461728
05:21:06 openstackgerrit jichenjc proposed openstack/nova-specs master: WIP: add z/VM Driver https://review.openstack.org/464915
05:32:07 openstackgerrit Luo Gangyi proposed openstack/nova master: convert unicode to string before we connect to rados https://review.openstack.org/465404
05:49:25 openstackgerrit Songming Yan proposed openstack/nova-specs master: Sriov-bond https://review.openstack.org/463526
05:49:29 openstackgerrit Songming Yan proposed openstack/nova-specs master: Sriov-bond https://review.openstack.org/463526
06:36:59 openstackgerrit Wang Shilong proposed openstack/nova master: Lustre support https://review.openstack.org/446288
06:57:32 alex_xu johnthetubaguy: gmann , I'm at bug smash, and there is event at evening, not sure I can back to the hotel in time. If i'm late on the api meeting, would you mind to run the api meeting first?
07:00:53 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Example verification for servers-actions.inc https://review.openstack.org/454565
07:03:16 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Catch exception.OverQuota when create image for volume backed instance https://review.openstack.org/464548
07:03:53 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add policy description for 'limits' https://review.openstack.org/465426
07:19:59 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add policy description for multinic https://review.openstack.org/465433
07:26:17 openstackgerrit fengzhr proposed openstack/nova master: Modify the description of flat_injected in nova.conf https://review.openstack.org/458789
07:42:07 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add policy description for server_usage https://review.openstack.org/465437
07:59:37 openstackgerrit Luo Gangyi proposed openstack/nova master: convert unicode to string before we connect to rados https://review.openstack.org/465404
08:00:15 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add policy description for networks_associate https://review.openstack.org/465443
08:03:32 openstackgerrit Jay Pipes proposed openstack/nova master: placement: test for agg association not sharing https://review.openstack.org/461901
08:14:52 johnthetubaguy alex_xu: no worries, I will try be around for the meeting
08:15:03 alex_xu johnthetubaguy: thanks!
08:17:32 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add policy description for os-networks https://review.openstack.org/465452
08:31:10 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add policy description for os-host https://review.openstack.org/465458
08:31:24 gmann alex_xu: sure, i will also be there
08:31:37 alex_xu gmann: thanks a lot
08:32:27 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add policy description for os-fixed-ips https://review.openstack.org/461724
08:34:02 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add policy description for 'os-hide-server-addresses' https://review.openstack.org/461728
08:35:04 gmann alex_xu: johnthetubaguy ^^ with this policy doc are finished : https://review.openstack.org/#/q/topic:bp/policy-docs+status:open+project:+openstack/nova
08:35:30 johnthetubaguy gmann: superstar, I should take a look at those soon
08:36:09 gmann johnthetubaguy: thanks
08:38:55 alex_xu gmann: yea, superstar :)
08:39:38 openstackgerrit Artom Lifshitz proposed openstack/nova master: Use VIR_DOMAIN_BLOCK_REBASE_COPY_DEV when rebasing https://review.openstack.org/465205
08:40:25 artom kashyap, mind taking a look at ^^ when you have a moment? You offered help with the downstream bug )
08:40:27 artom ;)
09:03:01 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Support tag instances when boot(3/4) https://review.openstack.org/459593
09:09:11 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Catch exception.OverQuota when create image for volume backed instance https://review.openstack.org/464548
09:23:35 openstackgerrit Tao Li proposed openstack/nova master: Fix the display of updated_at time when using memcache driver. https://review.openstack.org/414463
09:40:07 openstackgerrit Béla Vancsics proposed openstack/nova master: [WIP] Transform keypair.delete notification https://review.openstack.org/463946
09:41:52 openstackgerrit Ferenc Horváth proposed openstack/nova master: Add AggregateMultiTenancyIsolation filter tests https://review.openstack.org/464002
09:47:40 openstackgerrit Alex Szarka proposed openstack/nova master: Transform instance.resize.error notifications https://review.openstack.org/417882
10:01:17 openstackgerrit jichenjc proposed openstack/nova-specs master: WIP: add z/VM Driver https://review.openstack.org/464915
10:52:08 openstackgerrit Sandeep Pawar proposed openstack/nova master: Libvirt volume driver for Veritas HyperScale https://review.openstack.org/443951
11:01:25 openstackgerrit Stephen Finucane proposed openstack/python-novaclient master: Remove the `--volume-service-name` option https://review.openstack.org/465502
11:07:01 openstackgerrit jichenjc proposed openstack/nova master: WIP: remove flavor policy https://review.openstack.org/465505
11:19:31 openstackgerrit Stephen Finucane proposed openstack/python-novaclient master: Remove --cell_name, --tenant options https://review.openstack.org/465509
11:19:58 openstackgerrit jichenjc proposed openstack/nova master: WIP: Remove os_compute_api:os-server-groups policy https://review.openstack.org/465510
11:23:34 openstackgerrit jichenjc proposed openstack/nova master: WIP: remove flavor policy https://review.openstack.org/465505
11:40:11 openstackgerrit Alex Szarka proposed openstack/nova master: [WIP] Transform instance.exists notification https://review.openstack.org/403660
11:48:13 openstackgerrit Andrey Volkov proposed openstack/nova master: PoC: Select PCI devices with distinct tag values https://review.openstack.org/448008
11:53:17 openstackgerrit Alex Szarka proposed openstack/nova master: Add method for verify multiple notifications https://review.openstack.org/465526
11:57:06 openstackgerrit Alex Szarka proposed openstack/nova master: Add method for verify multiple notifications https://review.openstack.org/465526
12:07:19 openstackgerrit jichenjc proposed openstack/nova master: WIP: remove flavor policy https://review.openstack.org/465505
12:21:52 openstackgerrit Alex Szarka proposed openstack/nova master: [WIP] Transform instance.exists notification https://review.openstack.org/403660
12:22:33 openstackgerrit Alex Szarka proposed openstack/nova master: [WIP] Transform instance.exists notification https://review.openstack.org/403660
12:35:45 Miouge Is there an extra spec (flavor or image) to configure <cpu mode='host-passthrough'> with libvirt? I know of cpu_mode in nova.conf but that’s for a whole compute node
12:37:59 openstackgerrit Alex Szarka proposed openstack/nova master: Add method for verify multiple notifications https://review.openstack.org/465526
12:38:13 openstackgerrit Alex Szarka proposed openstack/nova master: [WIP] Transform instance.exists notification https://review.openstack.org/403660
12:39:52 sfinucan Miouge: not that I'm aware of, and it's not something we'd be eager to enable
12:40:38 artom mriedem, I thought it was OK to continue using mox when it's already in place?
12:40:51 artom I essentially copy-pasted an existing test and tweaked it a bit
12:42:22 Miouge sfinucan: curious to why not something eager to enable, recommended to use cpu_nova.conf and a separate host aggregate instead?
12:42:23 mriedem artom: not for new tests
12:42:24 artom mriedem, anyways, I think we had this discussion previously, and it was only dansmith who persuaded you to let it go, so I'll just fix my test, shall I
12:42:28 artom ;)
12:42:28 mriedem artom: new tests should be written in mock
12:42:53 mriedem artom: using mox to tweak existing tests that are using mox is acceptable
12:43:53 sfinucan Miouge: The comments in this spec should explain the position :) https://review.openstack.org/#/c/387295/
12:44:12 sfinucan ...and host aggregates would be preferred, yes
12:44:54 artom mriedem, fair enough. If I ever find the discussion I'm thinking of I'll wave it in your face, but yeah, I won't be lazy and do the new test with mock ;)
12:45:16 mriedem artom: i remember it, it was some nova-network live migration thing
12:45:29 mriedem and i believe dansmith went back afterward and re-wrote the tests in mock
12:46:08 mriedem artom: see #4 here https://docs.openstack.org/infra/manual/developers.html#peer-review
12:51:30 dane-fichter hey dansmith, could you take a look at my trusted certs patch when you get a chance?
12:51:37 dane-fichter https://review.openstack.org/#/c/457711/
12:52:26 dane-fichter It's passing unit tests, not sure if there's anything else I need to implement here
12:53:36 sdague dansmith / mriedem: thinking about the thing that came up in the cells forum session around searching by ip ....
12:53:51 sdague it's really pathologically bad in our code
12:54:18 mriedem i looked it up after the session,
12:54:23 mriedem and it wasn't mysql or pg specific
12:54:30 sdague nope
12:54:31 mriedem it was originally just about a security bug
12:54:39 sdague mriedem: what was the security bug?
12:55:00 mriedem api would call down to nova-network which would pull all instances from the db and iterate them filtering on ip, something like that
12:56:39 sdague ok, so removing nova-network out of the flow, we're now just loading every instance, loading every network info cache, and grepping through each json?
12:56:46 sdague especially in the all_tenants case
12:57:49 mriedem https://github.com/openstack/nova/commit/ae781ee97947c33d6d43e4c21df4f338c875bf1c was the original change
12:58:43 mriedem sdague: not sure how you'd do the filtering in the db when the instance_info_cache.network_data is serialized json
12:59:14 mriedem *network_info
12:59:30 sdague right, in order to make it efficient we'd need to extract the search keys that we wanted to hit into a column
12:59:48 sdague which you could do, just denormalize it a bit
13:00:55 sdague given that the ip search is regex, you could even just make it a comma separated list, and still do the second stage filter, but it would drop a lot of db bloat
13:01:37 sdague anyway, just pondering that, given it seemed like it was a query many wanted to do, and it's one of our most pathologically bad ones
13:01:51 mriedem and it's also been this way forever..
13:02:26 mriedem if we had numbers on how long it takes to do the filter on an all_tenants call with 1000 instances or something, then it might be more interesting
13:02:53 sdague mriedem: yeh data would be good

Earlier   Later