| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-05-17 | |||
| 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 | mriedem | artom: new tests should be written in mock | |
| 12:42:28 | artom | ;) | |
| 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 | |
| 13:03:22 | sdague | it does also mean though if you exceed 1000 instances in your filter set before ips, you'll get the wrong answer, right? | |
| 13:03:32 | sdague | that's actually the thing I'm more concerned about | |
| 13:03:54 | mriedem | true, if the ip you're looking for is the 1001 instance, you're not going to get anything back | |
| 13:04:15 | sdague | if nothing else, we should describe that issue | |
| 13:05:15 | mriedem | yeah, definitely, | |
| 13:05:19 | mriedem | could go into the api-ref | |
| 13:05:44 | mriedem | i feel like there was something else we said we'd doc in the api-ref but i'm forgetting - i think notes are in etherpads though, and i haven't digested those yet | |
| 13:06:25 | mriedem | ip (Optional) query string An IPv4 address to filter results by. | |
| 13:06:25 | sdague | well, in mysql we could - https://dev.mysql.com/doc/refman/5.7/en/json-search-functions.html#function_json-contains | |
| 13:06:32 | mriedem | ^ is the api-ref and doesn't mention anything about this | |
| 13:06:40 | sdague | mriedem: yep, I know | |
| 13:06:46 | sdague | I honestly didn't realize how bad it was | |
| 13:07:59 | mriedem | looks like postgresql does too https://www.postgresql.org/docs/9.3/static/functions-json.html | |
| 13:08:27 | mriedem | http://schinckel.net/2014/05/25/querying-json-in-postgres/ | |
| 13:09:52 | rabel | Hi there, I'm a little bit confused about quotas in nova. To start with: Can anyone tell me, what a quota class is? And how are those related to default quotas? How do default quotas work in detail? For example why does a project have default quotas? I thought default quotas are global quotas that take effect for projects without quotas? | |
| 13:12:06 | mriedem | rabel: se | |
| 13:12:07 | mriedem | *sec | |
| 13:12:37 | mriedem | rabel: some notes here https://docs.openstack.org/developer/nova/quotas.html | |
| 13:12:52 | mriedem | rabel: there is only one quota class and it's "default" | |
| 13:13:18 | mriedem | some more info here https://review.openstack.org/#/c/411035/ | |
| 13:14:47 | rabel | mriedem: thank you very much! do you know if other openstack components differ in this regard? | |
| 13:18:35 | mkoderer | mriedem: ping | |
| 13:18:40 | bhagyashris | Hi all, Can anyone please guide me How to check the memory consumption or memory during the process initialization suppose for n-cpu service? Is there any command to check this? | |
| 13:22:10 | rabel | mriedem: what is still confusing to me is the --default option of "openstack quota show", which shows default quotas for a given project or class. that does not seem to make a lot of sense. what do you think? | |
| 13:30:35 | mriedem | rabel: i think that's trying to match the nova quota-defaults CLI | |
| 13:31:38 | mriedem | mkoderer: pong | |
| 13:32:10 | mkoderer | mriedem: dgonzalez and I wanted to discuss a tricky bug with you: https://bugs.launchpad.net/nova/+bug/1691131 | |
| 13:32:11 | openstack | Launchpad bug 1691131 in OpenStack Compute (nova) "IO stuck causes nova compute agent outage" [Undecided,New] | |