| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-04-21 | |||
| 09:07:36 | asettle | Provided we're not going to have substantially broken docs for another month, I'm happy to do what we need to do. | |
| 09:10:56 | bauzas | asettle: cdent: we have an internal bug triaging meeting today, will raise it up | |
| 09:11:05 | cdent | bauzas++ | |
| 09:11:22 | asettle | Thank you bauzas :) | |
| 09:11:32 | asettle | Could you keep us up to date on the launchpad bug if I'm not around? | |
| 09:11:39 | asettle | I mean, I'm sure I will be :( | |
| 09:11:39 | bauzas | we = Nova team in Red Hat, not RDO team | |
| 09:12:30 | bauzas | asettle: that's at 1400UTC | |
| 09:12:37 | asettle | Lovely. Just in time for tea. | |
| 09:12:59 | cdent | asettle: I think it is pretty clear that the communication for "how to do placement" went awry to many parties over the past... six months? So we can probably do some kind of retrospective or something to figure out what to do better in the future. From my standpoint I didn't have enough insight into the process and made too many assumptions about it being under control | |
| 09:13:42 | bauzas | cdent: well, mostly because that's deployment choices | |
| 09:14:09 | bauzas | cdent: so that's hard to find a common denominator for all | |
| 09:14:37 | bauzas | and we all say that Service Catalog serves the purpose to enable services the way you want | |
| 09:14:49 | cdent | bauzas: sure, but I think there's been a lot of confusion that we could have made less of a problem. The <Directory> thing, for example, seems like something that never should have happened because when that's wrong things simply don't work. | |
| 09:15:10 | bauzas | so, you, as a deployer, just needs to make consistent what you have in your service catalog entry and what you configure server-side | |
| 09:15:41 | asettle | cdent: thanks for your honesty. What can I do to help? | |
| 09:16:00 | bauzas | cdent: the port discussion clearly showed up that a lot people were having wrong expectations | |
| 09:16:33 | bauzas | the fact that the service catalog exists is for the exact reason that you can provide any endpoint you want | |
| 09:16:52 | bauzas | this is not aimed to be consistent across all openstack deployments | |
| 09:17:13 | cdent | bauzas: yes, I know, I'm not suggesting we change any of that. but the docs process has been more fraught with error than it should have been. we should figure out why and try to make it better | |
| 09:17:17 | cdent | asettle: will you be in boston? | |
| 09:17:29 | asettle | cdent: sure will be. You? | |
| 09:17:29 | bauzas | and application developers wanting to integrate with APIs need to use the Service Catalog as the discovery system | |
| 09:18:09 | cdent | yup, we should gather and chat | |
| 09:18:17 | bauzas | cdent: sure, I guess that's the crux of providing upstream documentation that sets kind of de-facto standard | |
| 09:19:15 | asettle | Okay, let's aim for that. I've got a shitty tight schedule Mon - Wed with updates and all that jazz. | |
| 09:19:29 | asettle | But I'm there until Friday :) | |
| 09:19:41 | asettle | bauzas: you gonna be there? | |
| 09:20:17 | bauzas | yup | |
| 09:20:33 | bauzas | we have a couple of sessions around placement | |
| 09:20:39 | asettle | Lucky meeee | |
| 09:20:48 | bauzas | both in the Forum and the conference itself | |
| 09:21:15 | bauzas | like, jay (pipes) has 2 sessions for talking $placement, one targeted for users and architects, the other more for devs | |
| 09:21:17 | asettle | Well, it should be interesting. | |
| 09:21:19 | bauzas | 2 talks | |
| 09:21:33 | asettle | Okay. Well. I'll try and make sure I get along to some of them. I'm a bit of a lone wolf this summit. | |
| 09:33:08 | openstackgerrit | fengzhr proposed openstack/nova master: Modify the description of flat_injected in nova.conf https://review.openstack.org/458789 | |
| 09:35:07 | frickler | it seems that the scheduler docs are outdated, can anyone confirm that it should mention e.g. filter_scheduler.enabled_filters instead of scheduler_default_filters here? https://docs.openstack.org/ocata/config-reference/compute/schedulers.html | |
| 09:35:33 | frickler | I'm also struggling to get scheduling to a specific az to work with Ocata, are there known issues? | |
| 09:39:44 | frickler | bauzas: cdent: regarding deployment of placement-api, are end users supposed to talk to it or would it rather be for admins/operators only? i.e. do I really need to deploy a public endpoint for it or would I rather only do an internal one? | |
| 09:40:52 | cdent | frickler: it is admin _for now_ but the expectation is that might change in the future, as having dashboard views over one's own usage is expected | |
| 09:42:32 | openstackgerrit | Lee Yarwood proposed openstack/nova master: compute: Detach volumes on _rebuild_default_impl failure https://review.openstack.org/442105 | |
| 09:42:33 | frickler | cdent: ah, that is a good point, so a public endpoint will make sense in the long run. | |
| 09:42:46 | cdent | frickler: another one of the things that might happen in the future warnings (like described above with asettle) is that the preference is for interface type to notmatter | |
| 09:42:55 | cdent | but that's a long way off | |
| 09:46:18 | asettle | Hey cdent since you're loving doc nova bugs, want another one? :D | |
| 09:46:37 | cdent | asettle: I can certainly look, but I'm making no promises | |
| 09:46:44 | asettle | Hooray! | |
| 09:46:46 | asettle | :p thanks | |
| 09:46:47 | openstack | Launchpad bug 1683104 in openstack-manuals "instance is not created rightly" [Undecided,New] | |
| 09:46:47 | asettle | https://bugs.launchpad.net/openstack-manuals/+bug/1683104 | |
| 09:47:00 | asettle | I'm not convinced of this one, let's put it that way | |
| 09:47:57 | asettle | But, my nova deep dive konwledge is bad :( | |
| 09:48:06 | cdent | mine too! | |
| 09:49:36 | cdent | asettle: heh, interesting. their solution works but is undesirable :) | |
| 09:49:40 | cdent | s | |
| 09:50:10 | asettle | cdent: I guess my real concern is whether or not what we have is going to work or not. | |
| 09:51:03 | cdent | asettle: while the apache configuration is incorrect things will not work and the bugs that cascade from that will be various | |
| 09:51:15 | asettle | -.- | |
| 09:51:33 | cdent | the original cause of this person's problem is that their apache config is wrong and they have worked around it by not using apache at all for the placement service | |
| 09:52:00 | asettle | Well, that's info I can work with. | |
| 09:52:11 | asettle | Okay. | |
| 09:52:18 | asettle | Thanks cdent. I can reply to the reporter with that :) | |
| 09:52:32 | asettle | Unless you've already said something? | |
| 09:52:43 | cdent | I've started something, yeah, but haven't finished it yet | |
| 09:52:50 | asettle | Okay :) tahnks cdent I appreciate this | |
| 09:56:47 | cdent | asettle: I left a comment but more might be needed | |
| 09:59:18 | asettle | cdent: lgtm. | |
| 09:59:45 | asettle | Thanks again :) | |
| 10:00:33 | cdent | asettle: you're very welcome | |
| 10:00:39 | asettle | I owe you and bauzas a drink | |
| 10:34:48 | openstackgerrit | jichenjc proposed openstack/nova master: Use plain routes list for limits endpoint instead of stevedore https://review.openstack.org/458805 | |
| 10:37:24 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add keypairs field to InstancePayload https://review.openstack.org/419730 | |
| 10:37:25 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Adding auto_disk_config field to InstancePayload https://review.openstack.org/419185 | |
| 10:37:27 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Adding tags field to InstancePayload https://review.openstack.org/407228 | |
| 10:59:52 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add keypairs field to InstancePayload https://review.openstack.org/419730 | |
| 10:59:53 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Adding auto_disk_config field to InstancePayload https://review.openstack.org/419185 | |
| 10:59:54 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: add tags field to instance.update notification https://review.openstack.org/407228 | |
| 11:11:47 | openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Always disconnect_volume after rebase failures https://review.openstack.org/458807 | |
| 11:26:37 | ralonsoh | sfinucan: hi | |
| 11:29:37 | openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Always disconnect_volume after rebase failures https://review.openstack.org/458807 | |
| 11:33:53 | lyarwood | kashyap, mdbooth ; ^ hopefully a straight forward bugfix for review if you have anytime this afternoon | |
| 11:35:12 | kashyap | lyarwood: Yeah, will look after lunch | |
| 11:57:54 | openstackgerrit | Andy McCrae proposed openstack/nova master: Allow CONTENT_LENGTH to be present but empty https://review.openstack.org/455710 | |
| 11:59:15 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: explain payload inheritance in notification devref https://review.openstack.org/453667 | |
| 12:10:27 | sfinucan | ralonsoh: Hey | |
| 12:10:32 | sfinucan | Sorry - just back from lunch | |
| 12:17:00 | ralonsoh | sfinucan, me too | |
| 12:17:14 | ralonsoh | sfinucan, it's about https://review.openstack.org/#/c/427145/ | |
| 12:18:27 | ralonsoh | sfinucan: don't you think optimizing this function (and adding a tool for future optimizations) worth the effort? | |
| 12:23:27 | sfinucan | ralonsoh: not really, no | |
| 12:23:40 | sfinucan | I mean, we're saving a fraction of a fraction of a second per instance boot | |
| 12:24:15 | sfinucan | and that's a not insignificant chunk of code to do so | |
| 12:24:39 | ralonsoh | sfinucan: I know, but the point is this function should have been done like this | |
| 12:24:51 | sfinucan | ralonsoh: Would there be any real world improvements by doing that? | |
| 12:25:20 | ralonsoh | sfinucan: and the biggest part of this code is a function useful for other possible optimizations | |
| 12:27:14 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add keypairs field to InstancePayload https://review.openstack.org/419730 | |
| 12:27:14 | sfinucan | and if they yield more significant results, I'd be happy to reassess that :) | |
| 12:28:05 | openstackgerrit | Matthew Booth proposed openstack/nova master: libvirt: Pass instance to connect_volume and disconnect_volume https://review.openstack.org/450383 | |
| 12:28:05 | openstackgerrit | Matthew Booth proposed openstack/nova master: libvirt: Fix races with nfs volume mount/umount https://review.openstack.org/383859 | |