| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-07-13 | |||
| 13:48:52 | mriedem | we should have 2 schemas, | |
| 13:49:03 | mriedem | 1 for the instanceAction response, which has the 'events' key which is optional and is an array | |
| 13:49:27 | stephenfin | sean-k-mooney, ralonsoh: https://review.openstack.org/483398 | |
| 13:49:29 | mriedem | and 1 for the instanceAction.events, which is an object type, which has fields like start_time, finish_time, traceback, etc - those are required | |
| 13:49:34 | stephenfin | Ye silly gooses :) | |
| 13:50:33 | stephenfin | gmann_: Can you take a second look at my replies here when you have a chance? https://review.openstack.org/#/c/478484/ | |
| 13:51:41 | ralonsoh | stephenfin: thanks! I'll review the patch now | |
| 13:51:55 | gmann_ | mriedem: current schema will check 'events' as optional as it is policy based. as on some cloud it can be present and on other it cannot be | |
| 13:53:02 | gmann_ | mriedem: and my point is to have 'events' array schema also same way which validate start_time etc if those are present in response otherwise skip | |
| 13:53:31 | gmann_ | mriedem: but if we say those as mandatory in api-ref i am ok to validate those strictly by not looking at the code. | |
| 13:53:56 | gmann_ | which i should not when i work from tempest perspective :) | |
| 13:54:47 | gmann_ | stephenfin: sure, ll check tomorrow morning. may be i still argue on those doc in reference but let me rethink on those | |
| 13:55:41 | stephenfin | gmann_: Sounds good to me | |
| 13:59:34 | mriedem | nova meeting in 1 minute | |
| 14:02:27 | sean-k-mooney | stephenfin: hehe thanks for taking a look at openstack/devstack-plugin-libvirt-qemu most of the crft can be removed but we shoudl keep the docs/release note stuff | |
| 14:03:24 | stephenfin | sean-k-mooney: Yeah, you can rework it to add that | |
| 14:03:28 | stephenfin | fine by me :) | |
| 14:05:21 | sean-k-mooney | stephenfin: yep ill be happy too. i didnt actully expect a patch. i was just hopeing for some pointers to make sure we were uptodate with the new ways to handel pbr and the other project config stuff | |
| 14:05:48 | stephenfin | Ah, it's all good. Easy fix | |
| 14:05:54 | openstackgerrit | Hironori Shiina proposed openstack/nova master: Ironic: Get IP address for volume connector https://review.openstack.org/468353 | |
| 14:11:22 | openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Populate the 'configuration' section https://review.openstack.org/478473 | |
| 14:11:23 | openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Populate the 'cli' section https://review.openstack.org/478472 | |
| 14:11:23 | openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Populate the 'user' section https://review.openstack.org/478475 | |
| 14:11:24 | openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Switch to openstackdocstheme https://review.openstack.org/477751 | |
| 14:11:24 | openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Populate the 'contributor' section https://review.openstack.org/478474 | |
| 14:11:25 | openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Start using oslo_config.sphinxext https://review.openstack.org/482961 | |
| 14:11:25 | openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Rework README to reflect new doc URLs https://review.openstack.org/480074 | |
| 14:11:26 | openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Start using oslo_policy.sphinxext https://review.openstack.org/479358 | |
| 14:11:26 | openstackgerrit | Stephen Finucane proposed openstack/nova master: policies: Fix Sphinx issues https://review.openstack.org/480516 | |
| 14:11:27 | openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Populate the 'reference' section https://review.openstack.org/478484 | |
| 14:11:27 | openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Rework index page per new sections https://review.openstack.org/478485 | |
| 14:11:28 | openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Remove dead files https://review.openstack.org/478470 | |
| 14:19:30 | jangutter | sean-k-mooney: so how would I write an external plugin _without_ creating a new VIF type? | |
| 14:31:13 | bauzas | jaypipes: I saw your rant yesterday about the scheduler, could you please clarify your concerns ? | |
| 14:31:36 | bauzas | jaypipes: the fact that we're amending the ReqSpec by adding the found host ? | |
| 14:31:55 | jaypipes | bauzas: hmm? no, nothing to do with amending reqspec | |
| 14:32:04 | bauzas | it's just because of the affinity filter, but not sure I understand your problem honestly | |
| 14:32:21 | bauzas | jaypipes: you were telling dan you had a problem with that yesterday night :) | |
| 14:32:44 | jaypipes | bauzas: oh, yeah, sorry, that was a different rant :) | |
| 14:33:04 | bauzas | anway, if I can help you... | |
| 14:33:17 | bauzas | at least by sharing thoughts | |
| 14:33:19 | jaypipes | bauzas: my general annoyance with the scheduler is its tight coupling of state with decision-making. | |
| 14:34:00 | jaypipes | bauzas: And I'd much prefer to deal with things like num_instances > 0 outside of the filter scheduler. | |
| 14:34:46 | bauzas | jaypipes: the real problem is that since we really only supported FilterScheduler (ChanceScheduler being only for tests or crazy operators), we really didn't had a clear interface | |
| 14:35:09 | bauzas | so we built a pile of dependencies between the driver, the host manager and the filters | |
| 14:35:29 | jaypipes | bauzas: that and the unit tests are a pile of spaghetti that test way too much surface area and therefore if you change any little thing, a million tests crash and burn. | |
| 14:35:47 | bauzas | the code itself isn't very large, that's just the fact that we're poorly defining our internal interfaces | |
| 14:35:56 | jaypipes | that, and there's no reason for the WeighedHost class to begin with. | |
| 14:36:25 | bauzas | I just feel we need to be pragmatic for Pike and do the nice stuff later | |
| 14:36:30 | jaypipes | but whatever, I'm back off the cliff edge and making slow progress. | |
| 14:36:47 | jaypipes | bauzas: not sure what you mean by "nice stuff"./ | |
| 14:36:55 | bauzas | honestly, our internal interfaces are... internal, so I don't care with us changing those a couple of times | |
| 14:37:26 | bauzas | jaypipes: I'm talking of getting results first, and then clean up our model later | |
| 14:37:50 | jaypipes | bauzas: yeah... that's what has caused all of the issues in the first place :) shoot first, ask questions later. | |
| 14:38:02 | bauzas | I know | |
| 14:38:26 | bauzas | do you remember I started changing those interfaces in 2014 ? | |
| 14:38:38 | bauzas | that's just a funny story | |
| 14:46:01 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(1/2) https://review.openstack.org/408955 | |
| 14:47:02 | jaypipes | bauzas: I know you know. And I know you know that I know you know. | |
| 14:48:00 | bauzas | jaypipes: wake me before you know-know | |
| 14:48:05 | bauzas | aaaaaah | |
| 14:48:18 | bauzas | wake me *up before you know-know | |
| 14:48:45 | cdent | good effort though | |
| 14:49:00 | openstackgerrit | Stephen Finucane proposed openstack/nova master: trivial: Remove unnecessary function https://review.openstack.org/483433 | |
| 14:51:52 | jaypipes | bauzas: hehe | |
| 14:52:53 | mriedem | so we're making wham! jokes now? | |
| 14:53:28 | dansmith | oh jeez, you'll get mriedem all worked up | |
| 14:53:36 | mriedem | well look who it is | |
| 14:53:46 | mriedem | someone has a 'wham!' keyword listener | |
| 14:53:54 | dansmith | heh | |
| 14:54:07 | bauzas | :) | |
| 14:55:04 | efried | mordred mriedem Now a good time to talk about ksa/sta/os-service-types releases in support of use-service-catalog-for-endpoints ? | |
| 14:56:13 | mriedem | i need some time | |
| 14:56:58 | edleafe | jaypipes: I've bookmarked that quote | |
| 14:58:24 | jaypipes | edleafe: ha! it was paraphrasing White Goodman from "Dodgeball: A True Underdog Story" (http://www.imdb.com/title/tt0364725/quotes) | |
| 14:58:38 | jaypipes | "I know you. You know you. And I know you know that I know you." | |
| 14:58:51 | edleafe | oh, not that quote. http://p.anticdent.org/38I | |
| 14:59:19 | jaypipes | edleafe: ooohhhhh, that one. | |
| 14:59:31 | edleafe | jaypipes: :) | |
| 15:02:52 | mriedem | stvnoyes: i've gone through the compute manager changes on https://review.openstack.org/#/c/463987/ - there is a definite bug in the way that _detach_volume is called | |
| 15:03:32 | mriedem | stvnoyes: beyond that, i'm concerned that since we don't always call rollback_live_migration_at_destination, we might not cleanup properly | |
| 15:04:52 | mordred | efried_bbiab, mriedem: ping me when you're ready | |
| 15:25:01 | edleafe | jaypipes: let me check my understanding of compute-land with you | |
| 15:25:13 | edleafe | jaypipes: There is a compute service. It has a single resource tracker. That RT can have 1 or more compute nodes | |
| 15:25:30 | edleafe | jaypipes: Normally it's one (VMs), but for Ironic it can be (and usually is) >1 | |
| 15:25:36 | edleafe | jaypipes: is that correct? | |
| 15:25:40 | jaypipes | yup | |
| 15:25:56 | edleafe | jaypipes: ok thx | |
| 15:25:58 | dansmith | correct in the technical sense | |
| 15:26:04 | jaypipes | edleafe: always a single resource tracker. that resource tracker tracks >1 compute node objects in case of ironic. | |
| 15:26:05 | dansmith | not correct in the moral should-this-be-done sense | |
| 15:26:11 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: List/show all server migration types (1/2) https://review.openstack.org/430608 | |
| 15:26:28 | edleafe | dansmith: my soul is already unclean | |
| 15:26:52 | bauzas | edleafe: because jaypipes fixed that :) | |
| 15:27:10 | bauzas | edleafe: we previously had 1 RT per compute "node" | |
| 15:27:21 | bauzas | <3 jaypipes | |
| 15:28:19 | edleafe | bauzas: get a room, you two | |
| 15:28:45 | jaypipes | lol | |
| 15:29:05 | isantosp | Hi, I have an issue on nova newton, im trying to set ipv6 taking it from the port creation in neutron newton, apparently neutron is doing his job perfectly, the problem is when nova try to call port_crete_minimal, I have the following: ``` | |
| 15:29:09 | bauzas | edleafe: you mean #openstack-loves-jaypipes IRC room ? :p | |