| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-05-23 | |||
| 02:00:07 | mriedem | dansmith: looks like https://review.openstack.org/#/c/464059/ is happy | |
| 02:26:28 | mriedem | efried: it's starred but i'm shot, ping me about it tomorrow | |
| 02:27:08 | mriedem | efried: and review this https://review.openstack.org/#/c/459753/ since you asked me to review something | |
| 02:28:49 | mriedem | xenserver ci seems to have gone nuts | |
| 02:54:25 | jichen | mriedem: hi, sorry, could you please help in guide some 3rd CI related questions? | |
| 02:59:19 | mriedem | jichen: if you're not proposing a driver in-tree then running after patches merge is fine, it's really up to you as it's all external to nova | |
| 02:59:31 | mriedem | jichen: i don't see value in reporting CI results to nova changes for an out of tree driver | |
| 03:00:35 | jichen | mriedem: we want to make it in tree just like powervm did in last cycle, we knew CI is the mandatory work that we need to show | |
| 03:01:07 | mriedem | jichen: powervm worked on that for about 2 years i think | |
| 03:01:23 | mriedem | btw, which z driver is this for? because there are at least 3 z-related drivers | |
| 03:01:24 | jichen | mriedem: so some of my colleagues thought the only request is to report 'gate' result while my understanding is we need report in 'check' work | |
| 03:01:49 | mriedem | jichen: there is no guide really | |
| 03:01:59 | jichen | mriedem: https://github.com/openstack/nova-zvm-virt-driver is the one we are working on | |
| 03:02:18 | mriedem | honestly no one upstream is even thinking about adding zvm in tree because it hasn't come up | |
| 03:03:10 | jichen | mriedem: actually we have been work on open source for a few years but our CI is not ready and need more resource, we knew CI is mandatory so we didn't bring it up until now | |
| 03:03:27 | mriedem | i think you might have asked me about this in barcelona? or someone did in atlanta? | |
| 03:03:34 | mriedem | the zHPM (?) thing also came up | |
| 03:03:39 | mriedem | which is a different team | |
| 03:03:52 | mriedem | and zKVM still never started any CI, which they talked about doing since Paris | |
| 03:04:12 | jichen | yes, they are from India team and you can think they zHPM are barematel on z | |
| 03:04:52 | mriedem | jichen: well, nothing is going to happen in pike, | |
| 03:04:59 | jichen | there are 3 different hypervisors z/VM is old one and last for some big customers like ADP etc, zKVM is a new one and libvirt based , DPM (zHPM) is more like a barematel thing | |
| 03:05:17 | mriedem | i think we've previously told z driver people and huawei fusionsphere people to follow what the powervm team did, | |
| 03:05:25 | jichen | mriedem: yes, I know, so we want to bring this out at next PTG | |
| 03:05:56 | jichen | mriedem: ok, for the CI , we need to be as good as or better to PowerVM, need 'check' for every patch, right? | |
| 03:06:36 | mriedem | i'd need to ask thorst or efried what they were doing, but i thought their ci was pretty solid on their out of tree driver first, including on check queue changes | |
| 03:07:01 | mriedem | but their driver is also quite different from the zvm driver i believe, but honestly i don't really know anything about the zvm driver architecture, | |
| 03:07:03 | mriedem | so i can't really say | |
| 03:07:26 | mriedem | this probably needs some advertising on the dev ML first | |
| 03:07:46 | mriedem | if you guys have summit sessions / slides / whatever that might be a good start | |
| 03:08:38 | jichen | ok, we do have some thing there, I already drafted a WIP spec there, I will try to collect whatever info needed and send in the ML | |
| 03:08:48 | jichen | thanks a lot for your info and guide | |
| 03:15:28 | mriedem | jichen: sure, sorry i can't be of more help. i'd talk to thorst and the powervm team about their experience. | |
| 03:16:51 | jichen | mriedem: thanks a lot for your help and we will try to make PTG/summit and show more info there | |
| 03:21:59 | openstackgerrit | Ngo Quoc Cuong proposed openstack/nova master: Remove translation of log messages https://review.openstack.org/466995 | |
| 03:31:12 | openstackgerrit | John Haan proposed openstack/nova-specs master: Support volume_type with BDM paramter https://review.openstack.org/466595 | |
| 03:33:18 | mriedem | ha ^ | |
| 03:36:24 | mriedem | cburgess: ^ | |
| 03:36:51 | mriedem | cburgess: i think if you can get enough different cloud providers to open separate specs to pass through the volume type, at some point we just have to give in | |
| 03:37:15 | mriedem | it's in page 45 of the nova core dungeon master's guide | |
| 03:50:53 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: List/show all server migration types (1/2) https://review.openstack.org/430608 | |
| 04:04:28 | openstackgerrit | falseuser proposed openstack/nova master: [BugFix] Change the condition to delete a resizing instance https://review.openstack.org/463466 | |
| 07:06:19 | openstackgerrit | Ngo Quoc Cuong proposed openstack/nova master: Remove translation of log messages https://review.openstack.org/466995 | |
| 07:34:25 | openstackgerrit | jichenjc proposed openstack/nova master: Deprecate unused policy from policy doc https://review.openstack.org/465505 | |
| 07:45:58 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: List/show all server migration types (2/2) https://review.openstack.org/459483 | |
| 08:03:59 | openstackgerrit | Takashi NATSUME proposed openstack/nova-specs master: Amend the "list-show-all-server-migration-types" spec https://review.openstack.org/460847 | |
| 08:26:25 | openstackgerrit | sahid proposed openstack/nova master: network: update pci request spec to handle trusted tags https://review.openstack.org/458820 | |
| 08:26:25 | openstackgerrit | sahid proposed openstack/nova master: libvirt: configure trust mode for vfs https://review.openstack.org/458514 | |
| 08:26:26 | openstackgerrit | sahid proposed openstack/nova master: network: add command to configure trusted mode for VFs https://review.openstack.org/458513 | |
| 08:32:29 | sfinucan | johnthetubaguy, alex_xu: Could you take a look at [1] as it's blocking 4 other patches [1] https://review.openstack.org/#/c/461630/ | |
| 08:33:49 | sfinucan | oomichi too ^ | |
| 08:35:29 | openstackgerrit | Huan Xie proposed openstack/nova master: XenAPI use os-xenapi v2 in nova https://review.openstack.org/453493 | |
| 09:32:08 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Fix parameter order in os-services.inc https://review.openstack.org/467099 | |
| 09:47:19 | openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Send out notifications when instance tags changed https://review.openstack.org/459923 | |
| 09:54:14 | openstackgerrit | Felipe Alfaro Solana proposed openstack/nova-specs master: Implements: blueprint nova-lxd-profiles https://review.openstack.org/467104 | |
| 09:57:14 | openstackgerrit | jichenjc proposed openstack/nova master: Use plain routes list for os-migrations endpoint instead of stevedore https://review.openstack.org/461628 | |
| 09:59:50 | openstackgerrit | jichenjc proposed openstack/nova master: Use plain routes list for os-migrations endpoint instead of stevedore https://review.openstack.org/461628 | |
| 10:02:09 | openstackgerrit | jichenjc proposed openstack/nova master: Use plain routes list for os-services endpoint instead of stevedore https://review.openstack.org/460818 | |
| 10:09:48 | openstackgerrit | Gábor Antal proposed openstack/nova master: Transform instance.resize_prep notification https://review.openstack.org/465081 | |
| 10:18:30 | openstackgerrit | Vu Cong Tuan proposed openstack/nova master: Repair links in Nova documentation https://review.openstack.org/467114 | |
| 11:15:36 | artom | o_O | |
| 11:16:05 | artom | https://review.openstack.org/#/c/400763/3 merged without unit tests | |
| 11:16:30 | artom | Is that legit? We're allowed to do that? I'd have though making sure the exception is caught would be a good test | |
| 11:32:54 | sfinucan | artom: Hmm, yeah. It should have tests, yes | |
| 11:47:48 | sfinucan | sdague: If I want to check out a particular commit of nova and run it with Devstack, I simply check it out and reboot all nova services with 'systemctl restart devstack@n-*', right? | |
| 11:48:10 | sfinucan | (First time tinkering with it since the systemd changes went in) | |
| 11:48:21 | sdague | yep | |
| 11:50:08 | sfinucan | cool | |
| 12:16:46 | openstackgerrit | Feodor Tersin proposed openstack/nova master: Implement ScaleIO image backend https://review.openstack.org/407440 | |
| 12:17:06 | ftersin | mdbooth: hi. did you have a chance to look at ^^^^ ? | |
| 12:18:25 | mdbooth | ftersin: Only very briefly, I'm afraid. I'm stuck on downstream work right now. It's top of my review list, though. | |
| 12:22:46 | ftersin | mdbooth: ok, i understand. Probably you might review it by part. E.g. driver.py at the first, or whatever you assume is the most 'suspicious' and requires attention | |
| 12:23:14 | mdbooth | ftersin: That is indeed where I started :) | |
| 12:23:54 | ftersin | mdbooth: to let us an opportunity to discuss and to rework complex things not being in rush | |
| 12:24:06 | ftersin | cool | |
| 12:24:07 | mdbooth | Incidentally, if the driver could be confined to imagebackend.py I'd be in favour pretty much on your word that it works. | |
| 12:24:21 | mdbooth | IIRC it's a bit differenty in a couple of places, though. | |
| 12:25:22 | openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/467170 | |
| 12:29:18 | ftersin | mdbooth: i think there are 3 kings of changes in driver.py. the 1st is trivial, i.e. the adding 'sio' into some places where 'rbd', 'lvm', etc already exist | |
| 12:30:07 | ftersin | the 2nd is intended to map/unmap feature. it cannot be removed but might be reworked | |
| 12:30:58 | ftersin | the 3rd one are some side changes, which might be separated into different patches or patch series | |
| 12:31:21 | openstackgerrit | Felipe Alfaro Solana proposed openstack/nova-specs master: Implements: blueprint nova-lxd-profiles https://review.openstack.org/467104 | |
| 12:31:26 | mdbooth | ftersin: Are the side changes generally applicable, or specific to this driver? | |
| 12:33:43 | ftersin | both. e.g. the fix of swap device is generally applicable, but the filtering of disk_info items (to prevent use them as files) is required for SIO driver | |
| 12:34:07 | ftersin | however it is required for Cinder driver as well | |
| 12:34:20 | ftersin | it or some other solution of the problem | |
| 12:35:18 | mdbooth | ftersin: If you have time, I'd appreciate splitting them out, I think. I'll review without, though. | |
| 12:35:43 | openstackgerrit | Eric Berglund proposed openstack/nova master: Add strict option to discover_hosts https://review.openstack.org/467218 | |
| 12:38:21 | ftersin | mdbooth: i proposed a series, which resolve that problem separately: https://review.openstack.org/#/q/topic:disk_info_processing | |
| 12:50:47 | openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Exclude deleted service records when calling hyervisor statistics https://review.openstack.org/467220 | |
| 12:56:39 | openstackgerrit | Eric Berglund proposed openstack/nova master: Add strict option to discover_hosts https://review.openstack.org/467218 | |
| 13:00:26 | openstackgerrit | sahid proposed openstack/nova master: libvirt: configure trust mode for vfs https://review.openstack.org/458514 | |
| 13:11:44 | mriedem | ahoy matey | |
| 13:11:49 | mriedem | (s) | |
| 13:11:57 | cdent | is it talk like a pirate day again? | |
| 13:12:12 | asettle | Yar | |
| 13:12:15 | asettle | it is | |
| 13:12:45 | cdent | since I'm in cornwall, I claim authenticity | |
| 13:13:08 | asettle | Please, please tell me you're in Penzance | |
| 13:15:57 | mriedem | dansmith: when you're decent i have some questions about your bottom cells change for target_cell | |