| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-05-22 | |||
| 20:33:05 | mriedem | huh? | |
| 20:33:09 | cburgess | dansmith mriedem Something about live migrations and extra specs (no regular users can't see the extra specs). | |
| 20:33:17 | mriedem | anyway, time for cutoffs and sleeveless t for the neighbors | |
| 20:33:17 | dansmith | mriedem: you don't if you only have one cell | |
| 20:33:35 | cburgess | Oh looks like an old message from a few days ago. Weird. Sorry | |
| 20:33:52 | mriedem | cburgess: regular users can see extra specs, | |
| 20:33:57 | mriedem | because we do'nt scope the check to the flavor | |
| 20:34:14 | cburgess | mriedem Oh ok.. well I meant, I don't change anything with regard to extra spec visibility | |
| 20:34:26 | cburgess | So whatever upstream does we do | |
| 20:34:28 | mriedem | https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/flavors_extraspecs.py#L53 | |
| 20:34:55 | mriedem | cburgess: we all assumed that extra specs are hidden by default b/c they are admin or user in policy and the owner of a flavor/extra spec is the admin, | |
| 20:34:57 | mriedem | however, | |
| 20:35:13 | mriedem | we don't scope the policy check to the flavor, we scope it to the context, and the user context is the admin or owner - they are the owner of the request | |
| 20:35:25 | mriedem | so unless you change the policy to admin only, non-admins can see extra specs | |
| 20:35:30 | cburgess | Oh thats acute... | |
| 20:35:36 | cburgess | Cute even... | |
| 20:35:42 | mriedem | right, and johnthetubaguy has a spec to fix that | |
| 20:35:45 | cburgess | Like I said, I don't think we change that one. | |
| 20:35:53 | cburgess | Hmmm I wonder if that would be an issue.. | |
| 20:35:59 | cburgess | Thats a pretty big change at this point. | |
| 20:36:01 | mriedem | https://review.openstack.org/#/c/433037/ | |
| 20:36:13 | mriedem | it would break a bunch of shit i'm sure | |
| 20:36:22 | mriedem | b/c you'd start getting 401 or 403 or whatever | |
| 20:36:52 | cburgess | Yeah we have it default policy. | |
| 20:37:42 | cburgess | mriedem_afk What else is this spec going to break I wonder. | |
| 20:51:02 | amrith | mriedem_afk, when you get a chance would you please take a look at the email I sent you. | |
| 20:51:20 | amrith | wondering how best to make sure I get all of the gotcha's in one go. | |
| 21:18:58 | openstackgerrit | Dan Smith proposed openstack/nova master: De-complicate some of the instance delete path https://review.openstack.org/458636 | |
| 21:18:59 | openstackgerrit | Dan Smith proposed openstack/nova master: Move to proper target_cell calling convention https://review.openstack.org/458634 | |
| 21:19:00 | openstackgerrit | Dan Smith proposed openstack/nova master: Make target_cell() yield a new context https://review.openstack.org/458635 | |
| 21:19:01 | openstackgerrit | Dan Smith proposed openstack/nova master: Fix live migration devstack hook for multicell environment https://review.openstack.org/458537 | |
| 21:19:01 | openstackgerrit | Dan Smith proposed openstack/nova master: Target cell on local delete https://review.openstack.org/464059 | |
| 21:19:11 | dansmith | mriedem_afk: this is only the object context change to see if it breaks anything ^ | |
| 21:19:17 | dansmith | then I'll do the other things | |
| 21:49:59 | efried | mriedem_afk Can you please push https://review.openstack.org/#/c/391288/ ? Wanna get this pile moving again. Got some more stuff I want to add on top, features from previous review comments. | |
| #openstack-nova - 2017-05-23 | |||
| 00:19:24 | jaypipes | artom: hi! hey, what parts of http://specs.openstack.org/openstack/nova-specs/specs/ocata/approved/virt-device-tagged-attach-detach.html were fully implemented and what's still left to do? | |
| 00:24:47 | artom | ja | |
| 00:25:16 | artom | jaypipes, hey. We have device tagging at boot time - ie, nova boot --nic net-id=foo,tag=foo test | |
| 00:25:34 | jaypipes | artom: but not dynamic, right? | |
| 00:25:45 | jaypipes | artom: ala our ML convo from around a month ago, yeah? | |
| 00:25:59 | artom | jaypipes, yep :) | |
| 00:26:07 | jaypipes | artom: got it. thank you sir!] | |
| 00:26:18 | jaypipes | artom: internal product manager was wondering about it.. | |
| 00:26:42 | artom | jaypipes, whoa! | |
| 00:27:35 | jaypipes | artom: :) | |
| 00:37:13 | openstackgerrit | Zhaokun Fu proposed openstack/nova master: fix typo https://review.openstack.org/466949 | |
| 01:04:38 | openstackgerrit | Hironori Shiina proposed openstack/nova master: Ironic: Support boot from Cinder volume https://review.openstack.org/215385 | |
| 01:08:13 | openstackgerrit | Ngo Quoc Cuong proposed openstack/nova master: Remove translation of log messages https://review.openstack.org/466957 | |
| 01:46:46 | openstackgerrit | John Haan proposed openstack/nova-specs master: Support volume_type with BDM paramter https://review.openstack.org/466595 | |
| 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: libvirt: configure trust mode for vfs https://review.openstack.org/458514 | |
| 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: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 ^ | |