| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-03-28 | |||
| 09:53:27 | rabel | first of all: what is a quota class? is that just a collection of quotas? so if i change the quotas for a project, that means there is a quota class assigned to that project in the background and i actually change the quotas in that class? | |
| 09:53:40 | rabel | is "default" also a quota class or something different? | |
| 09:54:23 | rabel | and what are default quotas for a project? since i thought the default quotas are those assigned to a new project. is this for creating new projects with --parent option? | |
| 09:59:47 | kaisers | mdbooth: Hi! I tested on centos 7.3 and in a xenial vm. The tests returned exit code 0 in all variants (except when i had a systemd-run parameter typo) and all tested sleep calls returned immediately | |
| 10:01:05 | kaisers | mdbooth: right now i'm looking into another option, mount.quobyte does send a systemd ready notification upon successful mount, possibly this is a simple solution. But currently i'm looking into retrieving possible error information in such a variant | |
| 10:01:29 | mdbooth | kashyap: On centos 7.3? | |
| 10:02:11 | mdbooth | kashyap: That was for kaisers :) | |
| 10:02:18 | mdbooth | kaisers: Confirmed here :( | |
| 10:02:28 | mdbooth | Behaviour is different on F25 | |
| 10:04:38 | openstackgerrit | Moshe Levi proposed openstack/os-vif master: Explain why we bring up the lb in hybrd mode https://review.openstack.org/450692 | |
| 10:05:48 | openstackgerrit | Moshe Levi proposed openstack/os-vif master: Explain why we bring up the lb in hybrd mode https://review.openstack.org/447269 | |
| 10:08:45 | kaisers | mdbooth: yep, just re-verified: http://paste.openstack.org/show/604445/ | |
| 10:09:12 | kaisers | mdbooth: ok, I'll get back when i've looked into using systemd notify... | |
| 10:10:17 | mdbooth | kaisers: You could solve it with the post-verify method, but I think it would be much better if you didn't have to | |
| 10:23:18 | openstackgerrit | Stephen Finucane proposed openstack/nova-specs master: Add 'move-nova-cmds-to-cliff' spec https://review.openstack.org/433603 | |
| 10:37:50 | mdbooth | kaisers: I suspect this is what we really want: https://www.reddit.com/r/linux/comments/4ypb2g/systemd_rolls_out_its_own_mount_tool/d6qicwi/ | |
| 10:37:53 | mdbooth | Bit new, though :( | |
| 10:38:01 | mdbooth | It's not even in Fedora | |
| 10:43:56 | openstackgerrit | Sergey Nikitin proposed openstack/nova master: [WIP] PIC NUMA policy PoC https://review.openstack.org/390520 | |
| 11:01:23 | openstackgerrit | Jianghua Wang proposed openstack/nova-specs master: XenAPI: support for VGPU https://review.openstack.org/450122 | |
| 11:29:30 | openstackgerrit | Sergey Nikitin proposed openstack/nova master: [WIP] PIC NUMA policy PoC https://review.openstack.org/390520 | |
| 11:52:12 | kaisers | mdbooth: yeah! This keeps on dangling fobidden ('cos too fresh) fruits in front of my face... :-D | |
| 12:04:08 | gabor_antal | bausas, sfinucan: would someone please readd WF+1, I had to rebase the patch: https://review.openstack.org/#/c/448655/ | |
| 12:04:19 | gabor_antal | oops, bauzas | |
| 12:04:57 | jaypipes | gabor_antal: done | |
| 12:05:17 | gabor_antal | thanks, Jay! that was quick :) | |
| 12:05:40 | jaypipes | gabor_antal: heh, well I was actually just reviewing that patch :) | |
| 12:10:50 | openstackgerrit | John Garbutt proposed openstack/nova-specs master: Add policy-remove-scope-checks spec https://review.openstack.org/433037 | |
| 12:35:53 | openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient master: Explicitly specify arguments of server_groups creation https://review.openstack.org/450763 | |
| 12:42:35 | openstackgerrit | John Garbutt proposed openstack/nova-specs master: Add spec to use cinder's new attachment API https://review.openstack.org/373203 | |
| 12:43:13 | johnthetubaguy | mdbooth: would be awesome if you get chance to double check I didn't break anything with my fixes for mriedman's comments | |
| 12:53:12 | openstackgerrit | John Garbutt proposed openstack/nova master: Use HostAddressOpt for opts that accept IP and hostnames https://review.openstack.org/445782 | |
| 12:54:11 | openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient master: Explicitly specify arguments of server_groups creation https://review.openstack.org/450763 | |
| 13:08:07 | kaisers | mdbooth: I'm changing this to rely on the sd_notify approach and i'm moving the systemd mount to a separate method as it becomes a completely separate mount/verify code path. This will take a bit, i'll ping you here once it's up for review, if that's ok... | |
| 13:08:57 | mdbooth | kaisers: Sounds good. Are you going to use the systemd-notify cmdline tool? | |
| 13:09:47 | kaisers | mdbooth: Nope, the mount.quobyte client is sending this, so the call will stay mostly the same | |
| 13:10:16 | mdbooth | kashyap: Does it send a notification on failure? | |
| 13:10:58 | mdbooth | kashyap: That was for kaisers (again) | |
| 13:11:14 | kashyap | Yeah, no worries, I know the tab-complete snafu all too well | |
| 13:11:16 | kaisers | mdbooth: no, only a success notification | |
| 13:11:33 | kaisers | mdbooth: but i'm adding a check for this | |
| 13:12:18 | mdbooth | Do you happen to know if there's anywhere we can run a functional test of this and inject a failure? | |
| 13:13:10 | kaisers | mdbooth: Should be doable by providing an invalid backend address | |
| 13:13:40 | kaisers | mdbooth: That'd provide a range of output and a timeout on the unit startup | |
| 13:13:41 | mdbooth | kaisers: I mean: do we have infra to run it? | |
| 13:13:51 | kaisers | mdbooth: our CI? | |
| 13:14:02 | mdbooth | Does your CI just run tempest? | |
| 13:14:11 | kaisers | mdbooth: currently yes | |
| 13:14:13 | mdbooth | Because this wouldn't be very tempesty | |
| 13:14:42 | mdbooth | This is a prime candidate for a functional test | |
| 13:14:45 | kaisers | mdbooth: hmm, i can add something like that. Is there doc about functional testing in CIs | |
| 13:14:46 | kaisers | ? | |
| 13:15:09 | mdbooth | kaisers: sdague might know ^^^ ? | |
| 13:16:32 | kaisers | <-- is adding a ticket for himself to add func testing to CI | |
| 13:25:37 | openstackgerrit | John Garbutt proposed openstack/nova-specs master: Add additional-default-policy-roles spec https://review.openstack.org/427872 | |
| 13:36:21 | openstackgerrit | Radoslav Gerganov proposed openstack/nova master: VMware: serial console log (completed) https://review.openstack.org/450636 | |
| 13:41:18 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: remove ugly local import https://review.openstack.org/450787 | |
| 13:43:19 | gibi | johnthetubaguy: hi! thanks for the review https://review.openstack.org/#/c/418488/ ( | |
| 13:43:43 | gibi | johnthetubaguy: I've just noticed your comment in that | |
| 13:43:59 | gibi | johnthetubaguy: here is the follow up fixing your comment https://review.openstack.org/#/c/450787 | |
| 13:46:10 | johnthetubaguy | gibi: cool, thanks | |
| 13:51:49 | openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-manage: Deprecate 'shell' commands https://review.openstack.org/450794 | |
| 13:51:50 | openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-manage: Deprecate 'agent' commands https://review.openstack.org/450795 | |
| 13:57:18 | openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Rename '[neutron] ovs_bridge' https://review.openstack.org/450266 | |
| 13:57:19 | openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Move 'floating_ips' opts into 'network' https://review.openstack.org/431039 | |
| 13:57:19 | openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Deprecate 'default_floating_pool' https://review.openstack.org/431038 | |
| 13:57:20 | openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Add neutron.default_floating_pool https://review.openstack.org/431037 | |
| 13:57:32 | openstackgerrit | Maciej Szankin proposed openstack/nova master: Add fix_opt_description to database conf https://review.openstack.org/450799 | |
| 13:57:45 | tdurakov | paul-carlton2 eliqiao shaohe_feng pkoniszewski yuntongjin mdbooth kashyap kmartin claudiub tobasco abhishekk mrhillsman raj_singh wznoinsk siva_krishnan, live migration subteam meeting in 3 minutes on #openstack-meeting-3 | |
| 13:58:26 | mdbooth | johnthetubaguy: Are you still changing the cinder spec? | |
| 13:59:43 | johnthetubaguy | mdbooth: I am happy with the latest rev being reviewed again | |
| 14:02:49 | openstackgerrit | Diana Clarke proposed openstack/python-novaclient master: Fix 'nova list --fields' error in no instances case https://review.openstack.org/396074 | |
| 14:06:09 | mdbooth | ildikov: Got a sec? | |
| 14:06:11 | openstackgerrit | Gábor Antal proposed openstack/nova master: Transform instance.volume_attach notification https://review.openstack.org/401992 | |
| 14:06:36 | mdbooth | ildikov: Wanted to talk about locking and shared connections. | |
| 14:07:50 | openstackgerrit | Maciej Szankin proposed openstack/nova master: config options: remove libvirt imagecache https://review.openstack.org/378497 | |
| 14:10:03 | mriedem | oh hello | |
| 14:10:19 | mdbooth | mriedem: Have you been induced? Inducted? | |
| 14:10:56 | ildikov | mdbooth: hi, I'm here | |
| 14:11:26 | mdbooth | ildikov: Yeah, just wanted to talk through locking around the shared connection thing. | |
| 14:11:49 | mdbooth | It's not super urgent, but potentially detailed :) | |
| 14:12:51 | mriedem | induced? the baby isn't due for another week. | |
| 14:12:53 | ildikov | mdbooth: I'm not sure I'm the expert on that one, but happy to dig into my memories about it from the PTG, etc. | |
| 14:13:10 | mdbooth | ildikov: So, I've already got a patch which solved this issue using the current apis here: https://review.openstack.org/#/c/383859/ | |
| 14:13:24 | ildikov | mdbooth: I think shared connections are just confusing in general at this point... | |
| 14:14:09 | mdbooth | If we could reduce the complexity of that with the new apis I'd be all for it, but I've thought it through, and I've got quite a lot of context on this, and I don't see how it's possible to do race-free outside the host driver | |
| 14:14:53 | mdbooth | Well, we could construct something, but I think it would be more complex and fragile than doing it on the host | |
| 14:15:02 | mriedem | alex_xu: per the question about the multinic implementation of the neutron backed in nova, you might want to update this blueprint for pike https://review.openstack.org/#/c/384261/ | |
| 14:15:47 | mdbooth | ildikov: Anyway, happy to have that detailed discussion whenever. | |
| 14:16:19 | mdbooth | ildikov: The high level point, though, is that any api call which says "you're the last user" is inherently racy | |
| 14:16:34 | ildikov | mdbooth: so the new API's supposed to make it better, but beyond talking about the locks I'm not sure we fully see it at this point | |
| 14:16:52 | mdbooth | You're always going to have to confirm that. IOW it doesn't really serve any purpose. | |
| 14:16:57 | ildikov | mdbooth: I also think it would be a good discussion to the meeting we have on Thursday, so other folks can chime in too | |
| 14:17:27 | ildikov | I think we want to lock on the host | |
| 14:17:33 | mdbooth | ildikov: Ok, I'll bring it up there. | |
| 14:18:05 | mdbooth | Probably the os-brick part doesn't need to directly concern itself with locking: that could be done by Nova. | |
| 14:18:19 | mdbooth | However, I'm convinced it does need to live on the host. | |
| 14:18:56 | ildikov | mdbooth: I think we are in an agreement on that | |
| 14:19:08 | ildikov | but I might miss something now if that's not the case :) | |
| 14:19:38 | pjm6 | hi there | |