| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-01-25 | |||
| 02:02:54 | openstackgerrit | Eric Fried proposed openstack/nova master: Move refresh time from report client to prov tree https://review.openstack.org/535517 | |
| 02:02:54 | openstackgerrit | Eric Fried proposed openstack/nova master: Fix nits in update_provider_tree series https://review.openstack.org/531260 | |
| 02:03:23 | gmann | mriedem: let me rebase them, not all are required. i will do it today and let you know | |
| 02:05:10 | oomichi | takashin: I just don't want to see similar patches in the future even after dropping the patch | |
| 02:05:21 | efried | d'oh | |
| 02:05:30 | openstackgerrit | Eric Fried proposed openstack/nova master: ProviderTree.get_provider_uuids: Top-down ordering https://review.openstack.org/536624 | |
| 02:05:31 | openstackgerrit | Eric Fried proposed openstack/nova master: New-style _set_inventory_for_provider https://review.openstack.org/537648 | |
| 02:05:31 | openstackgerrit | Eric Fried proposed openstack/nova master: set_{aggregates|traits}_for_provider: tolerate set https://review.openstack.org/536625 | |
| 02:05:32 | openstackgerrit | Eric Fried proposed openstack/nova master: WIP: Use update_provider_tree from resource tracker https://review.openstack.org/520246 | |
| 02:05:32 | openstackgerrit | Eric Fried proposed openstack/nova master: SchedulerReportClient.update_from_provider_tree https://review.openstack.org/533821 | |
| 02:05:33 | openstackgerrit | Eric Fried proposed openstack/nova master: Move refresh time from report client to prov tree https://review.openstack.org/535517 | |
| 02:05:33 | openstackgerrit | Eric Fried proposed openstack/nova master: Fix nits in update_provider_tree series https://review.openstack.org/531260 | |
| 02:06:17 | takashin | oomichi: okay. I agree with the change. | |
| 02:10:49 | takashin | oomichi: I voted +1 for the patch. | |
| 02:11:13 | oomichi | takashin: thanks :-) | |
| 02:25:16 | openstackgerrit | Takashi NATSUME proposed openstack/python-novaclient master: Add CLI to show instance usage audit logs https://review.openstack.org/409413 | |
| 03:09:20 | alex_xu | mriedem: thanks, will cleanup in a follow up | |
| 03:12:33 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Merge server create schema for availability zone extension https://review.openstack.org/451331 | |
| 03:13:27 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Merge server create for availability zone extension https://review.openstack.org/502574 | |
| 03:14:08 | openstackgerrit | Jackie Truong proposed openstack/nova master: Reduce complexity of _from_db_object https://review.openstack.org/537728 | |
| 03:16:20 | alex_xu | mriedem: Kevin_Zheng, it would be great we add some testcase ensure the substring format can pass the regex format in the followup patch, since our API schema value the ip filter input with regex format. https://review.openstack.org/525505 | |
| 03:17:37 | alex_xu | mriedem: Kevin_Zheng and, I'm curious how neutron side handle the different of 'like' in the different db | |
| 03:20:47 | Kevin_Zheng | alex_xu, sure | |
| 03:32:52 | Yikun | alex_xu, https://review.openstack.org/#/c/521683/21/neutron/plugins/ml2/plugin.py@1862, here is 'like' | |
| 03:34:31 | openstackgerrit | Jackie Truong proposed openstack/nova master: Reduce complexity of _from_db_object https://review.openstack.org/537728 | |
| 03:37:50 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Fix missing marker functions https://review.openstack.org/514579 | |
| 04:04:42 | Spazmotic | Morning | |
| 04:30:22 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Parameter verification for servers.inc https://review.openstack.org/528201 | |
| 04:45:09 | openstackgerrit | Jackie Truong proposed openstack/nova master: Add trusted_certs object https://review.openstack.org/489408 | |
| 04:46:46 | openstackgerrit | Jackie Truong proposed openstack/nova master: Implement certificate_utils https://review.openstack.org/479949 | |
| 04:51:04 | openstackgerrit | Jackie Truong proposed openstack/nova master: Add trusted_image_certificates to REST API https://review.openstack.org/486204 | |
| 04:57:51 | openstackgerrit | Brooks Kaminski proposed openstack/nova master: XenAPI/Stops the migration of volume backed VHDS https://review.openstack.org/533168 | |
| 05:18:10 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Fix missing 'if_notifications_enabled' decorator https://review.openstack.org/537745 | |
| 05:27:01 | openstackgerrit | Brooks Kaminski proposed openstack/nova master: XenAPI/Stops the migration of volume backed VHDS https://review.openstack.org/533168 | |
| 05:27:31 | Spazmotic | My first rebase.. kind of scary | |
| 05:27:43 | Spazmotic | things are puckered | |
| 05:56:21 | alex_xu | Yikun: ah, thanks | |
| 06:48:59 | openstackgerrit | Merged openstack/nova master: add whereto for testing redirect rules https://review.openstack.org/516385 | |
| 07:46:40 | Spazmotic | jianghuaw: addressed the nits by Bob and and rebased if you get a chance on Zuul confirms it :) | |
| 07:46:47 | Spazmotic | once* zuul | |
| 07:49:02 | jianghuaw | Spazmotic, thanks for the update. It looks good to me:-) | |
| 07:49:07 | Spazmotic | Thanks so much sir! | |
| 07:49:25 | Spazmotic | hunting around for my next project | |
| 07:54:28 | jianghuaw | Spazmotic, you may have interesting on this patch: https://review.openstack.org/#/c/532118/ | |
| 07:54:59 | jianghuaw | Huan created it for getting advices. | |
| 07:55:09 | jianghuaw | it's to fix a migration issue with xenapi. | |
| 07:55:31 | Spazmotic | sure will take a look | |
| 07:55:44 | jianghuaw | Spazmotic, thanks. | |
| 07:57:49 | Spazmotic | Interesting. We do volume based LM quite a bit but we get around this with --block-migrate for iSCSI | |
| 07:59:23 | jianghuaw | Spazmotic, iirc the issue happens only in the XS7.0 and forwarding releases. | |
| 08:00:01 | Spazmotic | That's definately going to be a problem i'll need to test out | |
| 08:00:47 | jianghuaw | or maybe 7.1 => . I put 7.1 in the log. | |
| 08:01:08 | jianghuaw | But anyway it will be issue for latest XS releases. | |
| 08:01:26 | jianghuaw | feel free to put comments in it. | |
| 08:01:54 | jianghuaw | If you interest, feel free to take it over:-) | |
| 08:02:06 | Spazmotic | I'll certainly look at it heavily, appeals to me greatly | |
| 08:02:55 | jianghuaw | Thanks. | |
| 08:08:20 | Spazmotic | Do you know if Huan has refactored it at all recently? Mentions they wanted to tweak the vmops method around, but i'm sure time was tight for you guys | |
| 08:14:57 | jianghuaw | Spazmotic, not yet I'm sure. | |
| 08:15:25 | Spazmotic | Alrighty cool.. I've got some 7.1s in an env, i'll run some tests over the next few days and get familiar with these maps, havn't had to use them before | |
| 08:15:34 | Spazmotic | Thanks for bringing to my attention is actually more important than you may know | |
| 08:16:03 | jianghuaw | cool. | |
| 08:29:29 | openstackgerrit | Hironori Shiina proposed openstack/nova master: virt/ironic: Implement rescue and unrescue https://review.openstack.org/416487 | |
| 08:35:32 | openstackgerrit | Maciej Jozefczyk proposed openstack/nova master: Do not normalize allocation_ratios https://review.openstack.org/532924 | |
| 08:38:29 | maciejjozefczyk | Hello | |
| 08:49:32 | gmann | mriedem: updated those. 2 are ready to go and 1 i need more work on test which i will tomorrow - https://review.openstack.org/#/q/topic:bp/api-extensions-merge-queens+status:open | |
| 08:50:30 | gmann | mriedem: there are other work also needed on this BP which we will continue on continued Rocky BP. if you want to move current patches to new one, it should be ok | |
| 08:52:47 | gmann | mriedem: or we defer the queens one to Rocky ? | |
| 09:05:03 | kashyap | It's middle of the night for Matt, so will take a few more hours for him to come online. | |
| 09:24:44 | Spazmotic | Aargh Zuul why you hate me | |
| 09:27:24 | Roamer` | Spazmotic, if it's any help, I doubt it's personal - it's in Zuul's nature to be very, very paranoid and suspicious of everyone, you just have to catch one of its rare good moods... :) | |
| 09:30:32 | openstackgerrit | Zengzhi Wang proposed openstack/nova stable/pike: Make eventlet hub use a monotonic clock https://review.openstack.org/537811 | |
| 09:57:45 | hrw | morning | |
| 09:58:40 | hrw | I am looking at nova's code. it adds graphics if it is needed. but I did not see where it adds usb host for usbtablet | |
| 09:58:55 | hrw | does it assume that usb host is already present in VM configuration? | |
| 09:59:54 | Roamer` | OK, so the StorPool volume attachment driver finally passed the Zuul check pipeline (after the Cinder backup bug was fixed)... I'm pretty sure mriedem intends to +W it once he wakes up and sees it's passed the tests, but the gate being what it is, I'd be really obliged if somebody else could +W it earlier :) Of course, "let's just wait for him" is a very reasonable option. | |
| 10:00:12 | Roamer` | oof, and that would be https://review.openstack.org/#/c/140733/19 of course | |
| 10:03:54 | kevinz | alex_xu: Could you review https://review.openstack.org/#/c/494156/ if you have time? Stephen already give +2 :-) | |
| 10:05:37 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Transform instance.exists notification https://review.openstack.org/403660 | |
| 10:08:12 | alex_xu | kevinz: got it | |
| 10:11:12 | hrw | tracking how nova gets guest components suxx | |
| 10:13:20 | stephenfin | mriedem, melwitt: Yeah, sorry about https://review.openstack.org/#/c/523958/ I was the approver and that's my bad. Won't happen again | |
| 10:18:35 | hrw | ok. nova assumes that usb host is present | |
| 10:19:51 | kashyap | stephenfin: mriedem: As I've said in the past, I don't quite agree with that line of thinking. Happy to also elaborate why in Dublin. | |
| 10:19:55 | mdbooth | mriedem: melwitt stephenfin I wonder if that policy of old might be up for review, especially in a case like this which is uncontroversial. IIRC the understanding came about after RH was accused of railroading a particular controversial patch. That's an entirely valid concern, but I don't think anybody is claiming it happened here. | |
| 10:20:15 | kashyap | (It somehow seem to imply / assume "malice") | |
| 10:20:49 | kashyap | (But it _should_ be a concern, if it's a _pattern_ of two people from the same company ACKing changes in rapid succession without giving time for others to chime in.) | |
| 10:20:57 | kashyap | (Or for some goddamned deadline's sake.) | |
| 10:21:18 | stephenfin | mdbooth: Indeed, but the discussion should have happened _before_ /me did that :) EAFP probably doesn't apply where matters of trust are involved | |
| 10:21:33 | stephenfin | A good PTG topic though, for sure | |
| 10:22:00 | kashyap | Sure. But I do agree with Matt that, we shouldn't be too eager to push something for a deadline | |
| 10:22:36 | kashyap | Because, let's be honest --- most deadlines milestones don't matter. It's all artificial lines in the sand. | |
| 10:23:56 | mdbooth | stephenfin: There was a spec, many rounds of review, at least 3 ML posts that I recall over a period of a few weeks, and additional thorough review from 2 core reviewers, including yourself. I resent the implication, tbh. | |
| 10:25:03 | kashyap | Yeah, I recall the history / and even postmortem done by markmc. | |
| 10:25:10 | kashyap | http://lists.openstack.org/pipermail/openstack-dev/2014-August/044550.html -- [openstack-dev] [nova] libvirt version_cap, a postmortem | |
| 10:25:59 | kashyap | mdbooth: Ah, I thought you were talking about a previous case which caused this no two +2s from the same company thing.) | |
| 10:27:00 | mdbooth | The previous case was also a misunderstanding, tbh, although there was more substance to it. | |
| 10:27:14 | kashyap | Yeah, absolutely it was a case: "too eager to misunderstand" | |
| 10:27:17 | hrw | https://bugs.launchpad.net/nova/+bug/1745340 - what should I add to get it more complete? | |
| 10:27:18 | openstack | Launchpad bug 1745340 in OpenStack Compute (nova) "Nova assumes that USB Host is present" [Undecided,New] | |