| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-03-24 | |||
| 11:25:30 | kashyap | mdbooth: This is ready for merge, no? -- https://review.openstack.org/#/c/383859/ ("libvirt: Fix races with nfs volume mount/umount ") | |
| 11:26:12 | kashyap | Needs another core reviewer to take a look at it | |
| 11:26:33 | mdbooth | kashyap: No, there was a bug in the parent commit | |
| 11:27:38 | mdbooth | kashyap: It was reverted :) | |
| 11:28:03 | kashyap | mdbooth: Mind throwing the URL? Gerrit's interface trips me up always | |
| 11:28:32 | mdbooth | Revert was change If1ee6d6e4004750f80424ec54d7dabc564ec9e06 | |
| 11:29:42 | kashyap | Ah, thanks. | |
| 11:48:18 | openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Start using oslo.sphinxext https://review.openstack.org/449584 | |
| 11:48:37 | sfinucan | johnthetubaguy, macsz: I think you'll like that one ^ | |
| 11:52:09 | cdent | sfinucan: I assume this meaning is something others can only grasp if they too go on the same journey and thus you cannot pass it on? | |
| 11:52:50 | nkorabli | Could you please comment my question - then I will create a bug | |
| 11:53:01 | nkorabli | if you create an instance as a demo user, shelve it as admin and then try to unshelve it as demo user again - you won’t see any errors or warnings, but instance will end up with an ERROR state. | |
| 11:53:01 | nkorabli | So, what should be the right way for this scenario? What do you think? | |
| 11:53:43 | cdent | nkorabli: I'd comment except for the fact that I don't know. | |
| 11:54:17 | sfinucan | cdent: I'm afraid so | |
| 11:54:19 | sfinucan | I intend to blog about some of it but if reading dhellman's musings [1] alone couldn't help me attain enlightenment, what hope has a mere mortal like myself of helping others? [1] https://doughellmann.com/blog/2010/05/09/defining-custom-roles-in-sphinx/ | |
| 11:57:30 | openstackgerrit | Lucas Alvares Gomes proposed openstack/nova master: [Ironic] Do not hard fail when Ironic is not available https://review.openstack.org/449587 | |
| 11:57:34 | nkorabli | cdent: thanks, fair enough :) | |
| 11:59:54 | sfinucan | nkorabli: I'd say just open a bug with your findings and let others come to their own conclusions about how to resolve it | |
| 12:01:09 | sdague | johnthetubaguy: you about? | |
| 12:01:18 | johnthetubaguy | I am | |
| 12:01:26 | sdague | I was just going through the policy spec, and I'm with mriedem, it's super confusing | |
| 12:01:36 | sdague | even though I know where you are trying to go with it, I think | |
| 12:01:58 | sdague | I left some thoughts on how to rewrite bits for clarity | |
| 12:03:17 | johnthetubaguy | sdague: thats cool, it is confusing | |
| 12:03:31 | johnthetubaguy | I tried a few different rewords, I might be stuck in the middle of a few of them right now | |
| 12:03:37 | mdbooth | Bless it's cotton socks, pylint does try hard in the face of overwhelming adversity | |
| 12:03:38 | openstackgerrit | John Garbutt proposed openstack/nova-specs master: Alternative for warning users about maintenance https://review.openstack.org/449593 | |
| 12:04:07 | johnthetubaguy | tojuvone: (and bauwser) I typed up some ideas I had in there ^ | |
| 12:04:11 | johnthetubaguy | sdague: is that both of them, btw? | |
| 12:04:15 | jroll | lucasagomes: nice find, thanks for that | |
| 12:04:33 | sdague | johnthetubaguy: I just went through the bottom one | |
| 12:05:25 | mdbooth | sdague: Actually, this sounds like something you might have tried. I'm looking for a test which checks function arguments and before writing something custom I thought I'd checkout pylint. Do you know if anybody's ever looked at a pylint job for Nova? | |
| 12:05:44 | johnthetubaguy | sdague: OK, thats the one I hoped was better to read, is it worth a quick hangout? | |
| 12:07:00 | johnthetubaguy | sdague: actually, I think I get your comment, let me try fix that | |
| 12:09:36 | sdague | johnthetubaguy: I can do a quick hangout in a few minutes if you like | |
| 12:10:05 | johnthetubaguy | sdague: maybe I try a quick rev of that first section, and see how we look later on? | |
| 12:10:17 | sdague | sure | |
| 12:12:32 | lucasagomes | jroll, o/ np thanks for the review | |
| 12:14:11 | nkorabli | sfinucan: thanks, I just thought if I would get a response, I could fix it now.. | |
| 12:18:43 | openstackgerrit | fengzhr proposed openstack/nova master: Make network:attach_external_network policy check outside nova-api https://review.openstack.org/449598 | |
| 12:22:12 | openstackgerrit | Andrey Volkov proposed openstack/nova master: Structure for simply managing placement-api-ref https://review.openstack.org/409340 | |
| 12:22:13 | openstackgerrit | Andrey Volkov proposed openstack/nova master: [placement] add api-ref for GET /resource_providers https://review.openstack.org/418503 | |
| 12:28:20 | sdague | sfinucan: you reading the email on the ipv6 issue infra hit | |
| 12:29:01 | sfinucan | sdague: I am not. Link? | |
| 12:29:25 | sfinucan | Is this 'Config drive claims ipv6_dhcp, neutron api says slaac' | |
| 12:33:18 | sdague | yeh | |
| 12:37:44 | sfinucan | Hmm, I've never even heard of slaac. It would appear I've not so much broken something as exposed existing broken behaviour. This is going to be fun to debug | |
| 12:39:01 | leakypipes | alex_xu: well, I've been trying to get those traits in to os-traits here: https://review.openstack.org/#/c/448283/ | |
| 12:39:32 | leakypipes | alex_xu: but cdent and sean-k-mooney don't seem to care for the patch before it that splits out traits into different modules. | |
| 12:40:12 | leakypipes | alex_xu: I originally had the NIC traits added as a single patch but sean-k-mooney requested some refactoring before expanding the single const.py further. | |
| 12:40:46 | cdent | leakypipes: that's a bit strong, it's more like it's not what it says on the tin. what's in the tin is good enough if the label is changed. However the label does sound pretty nice. | |
| 12:40:52 | sean-k-mooney | leakypipes: im always causing trouble :) | |
| 12:40:59 | alex_xu | leakypipes: yea, i see, just thought we can begin to use those NIC traits in https://review.openstack.org/435954 | |
| 12:41:38 | leakypipes | alex_xu: you mean replace "tso" with "HW_NIC_OFFLOAD_TSO" in the spec? | |
| 12:41:50 | alex_xu | leakypipes: yea | |
| 12:42:16 | leakypipes | alex_xu: well, that's fine of course :) but I won't hold up the spec for that. we can always enforce that in the code. | |
| 12:43:03 | sean-k-mooney | alex_xu: we could definetly do that and assuming the network traits land before pike release we chan change the impentation to use them once they are merged | |
| 12:43:50 | alex_xu | leakypipes: yea, I also did ask the doc work to replace tso to 'HW_NIC_OFFLOAD_TSO', just thought when we put those NIC traits in the code, those should be standard trait. otherwise, we will let user transfer to the standard trait in the future | |
| 12:44:28 | sean-k-mooney | leakypipes: i dont thnik we nesicalliy have all the feature flag we would need in your initall patch but we can extend in follow up patches | |
| 12:44:34 | alex_xu | s/did ask/didn't ask/ | |
| 12:44:36 | alex_xu | oops | |
| 12:45:15 | alex_xu | sean-k-mooney: yea, in the internal implement that is totally ok. | |
| 12:47:16 | leakypipes | sean-k-mooney: sure, agreed. | |
| 12:48:17 | alex_xu | sean-k-mooney: +1 | |
| 12:48:27 | openstackgerrit | Silvan Kaiser proposed openstack/nova master: Switch Nova Quobyte volume driver to mount via systemd-run https://review.openstack.org/432344 | |
| 12:49:32 | sean-k-mooney | alex_xu: so do you want us to update the spec to reflect or just enforce it via the code review of the implentation of the feature? | |
| 12:50:26 | alex_xu | sean-k-mooney: i think we only update this line https://review.openstack.org/#/c/435954/11/specs/pike/approved/enable-sriov-nic-features.rst@222 | |
| 12:59:03 | ralonsoh | alex_xu: hi | |
| 12:59:09 | ralonsoh | alex_xu: just for confirmation | |
| 12:59:36 | ralonsoh | alex_xu: when string values are read form libvirt, I need to normalize to os-traits values | |
| 12:59:47 | ralonsoh | alex_xu: these values will be stored in the DB | |
| 12:59:53 | alex_xu | ralonsoh: yea | |
| 13:00:03 | ralonsoh | alex_xu: and Neutron port will need to make the definition | |
| 13:00:20 | ralonsoh | alex_xu: https://review.openstack.org/#/c/435954/11/specs/pike/approved/enable-sriov-nic-features.rst@120 | |
| 13:00:32 | ralonsoh | alex_xu: using this normalized values | |
| 13:00:41 | ralonsoh | alex_xu: is that correct? | |
| 13:01:28 | alex_xu | ralonsoh: what does mean about neutron port will need to make the definition | |
| 13:02:15 | ralonsoh | alex_xu: I meant in https://review.openstack.org/#/c/435954/11/specs/pike/approved/enable-sriov-nic-features.rst@121, instead of writing low case value | |
| 13:02:32 | alex_xu | ralonsoh: yea | |
| 13:02:39 | ralonsoh | alex_xu: we need to write the normalized strings (defined in os-traits) | |
| 13:02:41 | ralonsoh | perfect! | |
| 13:02:52 | ralonsoh | alex_xu: I'll update the spec today | |
| 13:02:53 | alex_xu | ralonsoh: that is what i mean | |
| 13:02:58 | ralonsoh | and thanks! | |
| 13:02:58 | alex_xu | ralonsoh: thanks | |
| 13:09:52 | mriedem | bauwser: you might want to take a look at this https://bugs.launchpad.net/nova/+bug/1675607 | |
| 13:09:52 | openstack | Launchpad bug 1675607 in OpenStack Compute (nova) "test_list_migrations_in_flavor_resize_situation fails with NoValidHost - AvailabilityZoneFilter returned 0 hosts" [Undecided,Confirmed] | |
| 13:10:02 | mriedem | ES is backed up so i'm not sure if it's a recent regression or not | |
| 13:10:04 | bauwser | mriedem: yup, saw you adding me | |
| 13:10:19 | bauwser | it's in my pipe | |
| 13:10:39 | mriedem | well smoke'em if you got'em | |
| 13:11:22 | bauwser | :p | |
| 13:18:30 | markus_z | trivial fix: https://review.openstack.org/#/c/449543/1 | |
| 13:19:33 | openstackgerrit | John Garbutt proposed openstack/nova-specs master: WIP: Add policy-remove-scope-checks spec https://review.openstack.org/433037 | |
| 13:20:09 | johnthetubaguy | sdague: attempt to apply the first few of your comments, and I am heading for lunch now, but I am curious if that looks better or worse ^ | |
| 13:21:19 | sdague | ok, I'll check post breakfast | |
| 13:23:21 | mriedem | johnthetubaguy: i think sdague summed up my issues nicely :) | |
| 13:49:11 | dansmith | mriedem: so that az bug | |
| 13:49:17 | mriedem | yes superdan | |