| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-03-14 | |||
| 12:30:26 | sdague | frickler: that's fine, we were doing a soft transition. | |
| 12:30:53 | sdague | we should document this is the prefered mechanism going forward, and honestly probably backport that installguide change | |
| 12:42:13 | vdrok | jaypipes: cdent good morning. when you're around, wanted to discuss https://review.openstack.org/443628. these are the integration tests for resource classes in ironic. for now I just copy-pasted some stuff in the services/placement there to make the calls to placement api. and from what I heard from jaypipes there is no plan to have such client in tempest? | |
| 12:42:13 | vdrok | should we just put some stripped out version that will be doing get requests to some subset of endpoints in our tempest plugin? | |
| 12:42:43 | cdent | vdrok: looking | |
| 12:47:24 | openstackgerrit | Gábor Antal proposed openstack/nova master: Transform instance.volume_attach notification https://review.openstack.org/401992 | |
| 12:50:46 | openstackgerrit | Gábor Antal proposed openstack/nova master: Transform instance.volume_detach notification https://review.openstack.org/408676 | |
| 12:51:15 | openstackgerrit | Gábor Antal proposed openstack/nova master: Transform instance.volume_detach notification https://review.openstack.org/408676 | |
| 12:53:32 | cdent | vdrok: there are no immediate plans for creating a tempest client, since as you've discovered it is relatively easy to create one that "just does requests". | |
| 12:54:24 | vdrok | cdent: yup, for me personally, there is no problem with that. just thought that at least neutron may want to reuse it | |
| 12:56:04 | vdrok | jroll: so, are you ok with keeping the minimal client in our tree? | |
| 12:56:11 | cdent | vdrok: they've got one that they started too (internal to neutron, not part of tempest). I suspect at some point we'll want to extract something to remove duplication. | |
| 12:56:27 | cdent | vdrok: there's also a branch that was started to create a proper python-placementclient: | |
| 12:56:47 | cdent | https://github.com/miguellavalle/python-placementclient/tree/adding-grp-support | |
| 12:56:56 | cdent | but it ended up feeling like overkill | |
| 12:57:20 | cdent | at some point it will be necessary though as there are plans for managing things like shared disk from the command line | |
| 12:58:09 | vdrok | mhm, I see, thanks cdent ! | |
| 12:59:23 | openstackgerrit | Gábor Antal proposed openstack/nova master: Transform instance.reboot notifications https://review.openstack.org/382959 | |
| 13:10:42 | jroll | vdrok: if we agree those tests are useful, yes, I'm fine with that client in our tree for now | |
| 13:11:31 | raghav | while doing devstack of ocata release | |
| 13:11:40 | raghav | i am getting below error from nova | |
| 13:11:47 | vdrok | jroll: do you think it would be enough just having the functional ones in nova? | |
| 13:12:01 | raghav | NO hosts found to map to cell | |
| 13:12:12 | raghav | i have not enabled n-cell then why | |
| 13:12:20 | raghav | cell related error is coming | |
| 13:12:35 | jroll | vdrok: yeah, probably, but more tests can't hurt | |
| 13:14:43 | vdrok | jroll: I think for now, yes, they are not super-useful, as we're basically testing that ironic is returning what we expect in the virt driver. but, not mocking out ironic is a good thing :) | |
| 13:15:18 | jroll | vdrok: yeah, I think we'll expand, and later if ironic pushes data directly to placement they'll be very useful | |
| 13:15:23 | jroll | vdrok: I'm fine with landing them | |
| 13:16:03 | vdrok | ok cool :) I'll polish it a bit more, remove some unnecessary stuff today and they'll be ready for review | |
| 13:17:15 | cdent | vdrok: if you could add me to the review when it is ready, I'd appreciate it | |
| 13:17:30 | vdrok | cdent: sure, will do :) | |
| 13:19:17 | jroll | vdrok: awesome, thanks! | |
| 13:22:29 | raghav | hi all i am getting below error..while stacking | |
| 13:22:39 | raghav | NO hosts found to map to cell | |
| 13:22:49 | raghav | ocata release | |
| 13:23:07 | raghav | i have not enabled n-cell servcie then also i am getting this error | |
| 13:23:14 | raghav | could anyone please help me | |
| 13:24:38 | sdague | raghav: this is a completely fresh install? | |
| 13:26:45 | johnthetubaguy | raghav: the install guide is in the process of being updated for ocata, so if you followed that there are some missing steps right now | |
| 13:27:06 | johnthetubaguy | sfinucan: sorry, I mean update the link | |
| 13:27:14 | raghav | i have done clean.sh on my previous devstack installation and then did stack | |
| 13:27:33 | johnthetubaguy | for context I was thinking about https://review.openstack.org/#/c/438328/ | |
| 13:27:36 | johnthetubaguy | oh | |
| 13:27:40 | raghav | john: i am using devstack | |
| 13:28:36 | sdague | raghav: honestly, given some of the other issues, please start with a completely fresh OS install | |
| 13:28:51 | sdague | once changes have been made to config, clean.sh may not delete some of those things | |
| 13:28:55 | sfinucan | johnthetubaguy: OK, think I've done that now | |
| 13:30:24 | johnthetubaguy | hmm gerrit appears to hate me | |
| 13:30:36 | raghav | sdague: can there be any other soln than re installation | |
| 13:32:07 | sdague | raghav: given the set of issues you were showing yesterday, I think that debugging the current state of your machine isn't going to be a productive use of anyone's time, including yours | |
| 13:32:10 | sdague | I would start fresh | |
| 13:33:27 | raghav | sdague: ok thank you | |
| 13:43:50 | raghav | sdague: Hi in fresh installation aslo i got same error | |
| 13:44:31 | sdague | raghav: please pastebin all the logs and your local.conf, we know that it's working in the general case, it's tested all the time | |
| 13:49:22 | raghav | sdague: http://paste.openstack.org/show/602688/ please find local.conf | |
| 13:49:53 | tdurakov | Hi folks, caught cold, so couldn't run live migration meeting this week, sorry for inconvenience | |
| 13:50:19 | cdent | tdurakov: we share an internet virus | |
| 13:50:55 | tdurakov | Get well cdent | |
| 13:51:08 | cdent | you too | |
| 13:52:46 | sdague | raghav: why did you set *_BRANCH ? | |
| 13:52:57 | mriedem | lyarwood: this is kind of a monster backport but is something we need to get released sooner than later https://review.openstack.org/#/c/445236/ | |
| 13:53:56 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Transform instance.reboot.error notification https://review.openstack.org/411791 | |
| 13:55:43 | openstackgerrit | Danfly proposed openstack/nova master: Completely remove mox from unit/network/test_linux_net.py https://review.openstack.org/445504 | |
| 13:56:47 | lyarwood | mriedem: ack, I'll queue that up now, can you unblock https://review.openstack.org/#/c/431540/ when you have time, should be good to go now. | |
| 13:57:01 | mriedem | lyarwood: yeah will look now | |
| 13:57:15 | lyarwood | thanks | |
| 14:00:53 | mriedem | lyarwood: +2 | |
| 14:01:00 | mriedem | tonyb: mtreinish: can one of you hit this? https://review.openstack.org/#/c/431540/ | |
| 14:01:46 | mtreinish | mriedem: looking | |
| 14:02:07 | openstackgerrit | Dan Smith proposed openstack/nova master: Get instance availability_zone without hitting the api db https://review.openstack.org/439754 | |
| 14:02:08 | openstackgerrit | Dan Smith proposed openstack/nova master: Make os-availability-zones know about cells https://review.openstack.org/442163 | |
| 14:02:12 | mriedem | bauzas: cdent: jaypipes: so do we want to start looking at creating a placement client? i know mlavalle had already started an osc plugin. | |
| 14:02:54 | cdent | mriedem: just had a chat earlier in the day about that with vdrok | |
| 14:02:55 | dansmith | I thought the point of the simple design was to make it so we don't need a client | |
| 14:03:00 | mriedem | https://github.com/miguellavalle/python-placementclient/tree/adding-grp-support | |
| 14:03:05 | cdent | and we said pretty much what dansmith just said | |
| 14:03:23 | mtreinish | mriedem: +A | |
| 14:03:40 | mriedem | the original spec called out osc CLIs for things like associating resource providers via aggregates, | |
| 14:03:53 | mriedem | or simply listing resource providers as a sanity check | |
| 14:04:02 | mriedem | we expect everyone to just be using curl? | |
| 14:04:11 | mriedem | bra | |
| 14:04:13 | mriedem | *brb | |
| 14:04:40 | cdent | yes, the spec did say that, but I'm not sure we've actually hit those use cases yet, so might be just as easy to wait? | |
| 14:04:49 | cdent | sick bra, have a great time | |
| 14:05:45 | bauzas | mriedem: yeah, I began to work on providing a separate module for that, but I'd prefer to have a separate client | |
| 14:06:06 | dansmith | mriedem: a client for use in cron may make sense, but not a separate client for server code, IMHO | |
| 14:06:27 | dansmith | mriedem: like a simple thing you can run from the cli to report a particular thing, that's not overly complex, but I would expect cinder to just use requests | |
| 14:06:42 | bauzas | dansmith: well, a python client could be cool for OSC | |
| 14:06:58 | bauzas | and for other projects | |
| 14:07:04 | johnthetubaguy | dansmith: melwitt: macsz: seeing bug patterns emerge around this kind of thing: https://bugs.launchpad.net/nova/+bug/1672624 | |
| 14:07:04 | openstack | Launchpad bug 1672624 in OpenStack Compute (nova) "Ceph volumes attached to local deleted instance could not be correctly handled" [Undecided,New] | |
| 14:07:11 | dansmith | bauzas: for human usage, yes, which I think exists today | |
| 14:07:56 | openstackgerrit | Danfly proposed openstack/nova master: Completely remove mox from unit/network/test_linux_net.py https://review.openstack.org/445504 | |
| 14:08:54 | sdague | bauzas: so, honestly, I really think that clients are an anti-pattern | |
| 14:09:28 | bauzas | sdague: not sure I understand your concerns | |
| 14:09:53 | bauzas | IMHO, we could provide a python placementclient package so OSC could use it for providing CLI | |
| 14:10:08 | sdague | here is the issue with that | |
| 14:10:14 | sdague | and why I think it's an anti pattern | |
| 14:10:16 | bauzas | exactly like the same difference between nova CLI and novaclient python module | |