Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-05-22
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 ^
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

Earlier   Later