| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-08-01 | |||
| 08:40:19 | bhagyashris | On controller + compute node the n-cond-cell1 is configure | |
| 08:53:13 | saphi | bhagyashris Did you discover new compute node? Run `nova-manage cell_v2 discover_hosts` on controller node | |
| 09:03:11 | openstackgerrit | Jacek Tomasiak proposed openstack/nova master: ironic: Use internal API endpoint https://review.openstack.org/489537 | |
| 09:09:57 | trinaths | hi, when I installed compute node using devstack ocata, I see that nova-compute is running with nova-cpu.conf. but actually it must run with nova.conf and nova-compute.conf. is there any localrc changes I need to make for this? | |
| 09:13:46 | openstackgerrit | Takashi NATSUME proposed openstack/nova-specs master: Abort Cold Migration https://review.openstack.org/334732 | |
| 09:20:31 | bhagyashris | saphi: Hi, I have done this too, but still getting same error. | |
| 09:23:58 | trinaths | I get this error when I run nova-compute, IncompatibleObjectVersion: Version 2.4 of InstanceList is not supported | |
| 09:25:01 | saphi | Can you post your nova-compute.log? | |
| 09:25:12 | saphi | bhagyashris | |
| 09:28:41 | openstackgerrit | Zhaokun Fu proposed openstack/nova master: Smaller disk flavor check https://review.openstack.org/489484 | |
| 09:28:57 | openstackgerrit | Alexandru Muresan proposed openstack/nova master: Pass config object to oslo_reports https://review.openstack.org/485575 | |
| 09:30:30 | trinaths | Cannot update service status on host "compute" due to an unexpected exception | |
| 09:36:41 | bauzas | johnthetubaguy: not sure you're around, but I'm wondering if other drivers but libvirt and vmware can support shrinking root disks when resizing | |
| 09:36:55 | bauzas | I tend to think so for ironic at least | |
| 09:37:10 | openstack | Launchpad bug 1707830 in OpenStack Compute (nova) "resize to smaller flavor failed, but has not error message on dashboard" [Medium,In progress] - Assigned to Zhaokun Fu (fuzhaokun) | |
| 09:37:10 | bauzas | the context being https://bugs.launchpad.net/nova/+bug/1707830 | |
| 09:37:19 | bauzas | I don't want to make it something verified by the API | |
| 09:42:05 | trinaths | oslo_messaging.rpc.server IncompatibleObjectVersion: Version 2.4 of InstanceList is not supported | |
| 09:49:56 | openstackgerrit | Sylvain Bauza proposed openstack/nova master: Accept any scheduler driver entrypoint https://review.openstack.org/484828 | |
| 09:54:13 | cdent | sdague++ on the mass abandonment | |
| 09:54:32 | openstackgerrit | Sylvain Bauza proposed openstack/nova master: Destroy the ReqSpec object when deleting the instance https://review.openstack.org/391060 | |
| 09:55:06 | sdague | cdent: yeh, the in progress list is a lot less accurate than we think | |
| 09:55:27 | sdague | I trimmed back the criteria, this is just stuff that's been in merge conflict for 4 weeks | |
| 09:56:03 | sdague | cdent: what comms channel did you notice this on? | |
| 09:56:19 | cdent | i’m getting a lot of email for reviews that I’m watching | |
| 09:56:25 | cdent | or otherwise participating in | |
| 09:56:36 | cdent | so: from gerrit | |
| 09:57:55 | sdague | gotcha | |
| 10:02:00 | openstackgerrit | Gábor Antal proposed openstack/nova master: Transform instance.resize_prep notification https://review.openstack.org/465081 | |
| 10:18:35 | openstackgerrit | Chris Dent proposed openstack/nova master: Test resize with placement api https://review.openstack.org/487958 | |
| 10:22:20 | gibi | cdent: I will push an update to ^^ soon with some refactoring | |
| 10:22:42 | gibi | cdent: removing duplicated code and better naming variables | |
| 10:23:21 | cdent | gibi: are you up to date on the plan there? make it pass on current master, with expectations commented out, and then adjust it based on all the code jay’s been working. | |
| 10:23:31 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Test resize with placement api https://review.openstack.org/487958 | |
| 10:23:39 | gibi | cdent: yes, I read back | |
| 10:23:45 | cdent | go you | |
| 10:24:01 | gibi | cdent: what I did is that I update ps 14 on master with some refactoring | |
| 10:24:23 | gibi | cdent: also locally I moved top of the bugfix to see that confirm works with the uncommented asserts as well | |
| 10:24:38 | cdent | there was a hangout between dan and jay last night that went on for quite some time, I was only able to particpate for a short while. it was complicated. | |
| 10:24:41 | bauzas | cdent: I still need to understand the consensus now | |
| 10:24:58 | cdent | bauzas: as far as I can tell there isn’t yet a consensus on a fix | |
| 10:25:06 | bauzas | okay | |
| 10:25:10 | bauzas | let's wait for the others | |
| 10:25:19 | cdent | we are aware of the problem and want to make it as visible as possible, thus the functional test going in first | |
| 10:25:26 | gibi | exactly this is why I only did refactoring that does not impact whatwhever will be the fix for the bug | |
| 10:27:24 | cdent | bauzas: in the conversations last night, each time a potential solution was discussed another problem was revealed | |
| 10:27:32 | bauzas | heh | |
| 10:28:09 | cdent | one idea that was well liked was that a migration/resize would have a second allocation with a different consumer id, instead of the doubling | |
| 10:28:36 | cdent | but that discussion exposed problems with migrations happening between ocata computes and pike computes | |
| 10:28:41 | bauzas | shouldn't we buy https://www.amazon.com/Hasbro-40509-Whac-A-Mole-Game/dp/B0001GDP00 ? | |
| 10:28:48 | cdent | probably yes | |
| 10:29:05 | bauzas | I see | |
| 10:29:12 | bauzas | about the 2nd consumer discussion | |
| 10:29:24 | cdent | sdague: and now the launchpad emails are starting | |
| 10:29:31 | bauzas | honestly, looks like a doubling allocation seems difficult | |
| 10:29:54 | bauzas | when I provided my comment, I was just thinking of getting the original allocation and just passing it back in case of an exception | |
| 10:30:12 | bauzas | I didn't thought about all the other problems we could have :( | |
| 10:31:05 | sdague | cdent: yeh, I'm flushing the in progress bugs, it moves at least 25 out of in progress | |
| 10:31:30 | bauzas | sdague: I did that too | |
| 10:32:24 | cdent | bauzas: It’s good that we are talking this stuff out, because we don’t really have sufficient testing to find the bugs that we are creating (we need more gibi ) so applying brains is necessary. Each question, though annoying, is revealing something, and that’s better in the long run. | |
| 10:33:18 | bauzas | sdague: do we have jobs for resizing to a 2nd host ? I know that for live-migration, but I do wonder if we only verify resizes for the same host | |
| 10:39:04 | johnthetubaguy | bauzas: I think XenAPI already shrinks the disks when resizing down | |
| 10:40:10 | bauzas | johnthetubaguy: ack | |
| 10:40:30 | bauzas | johnthetubaguy: anyway, stopping to accept at the API level a resize down would probably need a microversion | |
| 10:40:37 | johnthetubaguy | cdent: when we chatted about this before, I remember we kinda liked the doubling up, are folks finding thats bad? | |
| 10:40:58 | johnthetubaguy | bauzas: I guess it should, although that's kinda removing a feature some folks use | |
| 10:41:15 | cdent | johnthetubaguy: we need the doubling up, the problem is that it is hard to manage the cleaning up afterwards | |
| 10:41:16 | bauzas | johnthetubaguy: exactly my point I wrote in the review | |
| 10:41:34 | johnthetubaguy | cdent: oh, I see what you mean now | |
| 10:41:56 | johnthetubaguy | cdent: would the migration uuid holding an allocation work, or mess things up totally for sync logic? | |
| 10:41:58 | bauzas | johnthetubaguy: either we say nova will stop supporting that for all drivers and then it requires a microversion, or we say it's per-driver and then we don't want to have the API verifying it | |
| 10:42:29 | johnthetubaguy | bauzas: I would rather the API new if the compute host could do it | |
| 10:42:40 | johnthetubaguy | knew | |
| 10:43:02 | bauzas | johnthetubaguy: yeah, agreed, a capability | |
| 10:43:05 | cdent | johnthetubaguy: if there were a migration uuid, that would help, but there’s not, and getting access to it in the confirm_resize is weird (not sure I have all the details of this right, was very tired while listening to dan and jay last night) | |
| 10:43:29 | bauzas | cdent: we have a migration object, couldn't that help ? (well, for all migrations except live-mig :D) | |
| 10:43:32 | johnthetubaguy | cdent: oh, damm, did we never add that | |
| 10:43:45 | cdent | johnthetubaguy: that’s what dan and jay said :) | |
| 10:43:51 | cdent | (the “oh damn”) | |
| 10:43:58 | johnthetubaguy | cdent: oh, thats correct, migration is completed before the confirm/revert phase I think | |
| 10:44:02 | bauzas | cdent: and FWIW we expose those migration objects to the API | |
| 10:44:14 | bauzas | s/FWIW/AFAIK | |
| 10:45:07 | johnthetubaguy | bauzas: in my head that was getting added as part of the cancel resize/migrate work | |
| 10:45:59 | openstackgerrit | Sean Dague proposed openstack/nova master: Show quota detail when inject file quota exceeds https://review.openstack.org/453040 | |
| 10:46:18 | bauzas | johnthetubaguy: you mean os-migrations ? | |
| 10:46:30 | johnthetubaguy | bauzas: I think so | |
| 10:46:56 | bauzas | but whatever, we haven't yet a migration object for live migrations :) | |
| 10:47:21 | johnthetubaguy | bauzas: can't remember if the additions merged now, I thought it was shared for both now | |
| 10:47:31 | bauzas | maybe we could just split nikola's patch in two and just at least add the migration object at first, before trying to claim | |
| 10:47:34 | johnthetubaguy | anyways, not sure any of that helps | |
| 10:47:51 | sdague | bauzas: we run the resize tests in multihost I think | |
| 10:48:01 | bauzas | sdague: cool then | |
| 10:48:06 | sdague | bauzas: but I haven't looked that hard to verify | |
| 10:48:16 | bauzas | sdague: I can check | |
| 10:51:41 | cdent | sdague, bauzas: we talked about that last night, and dan confirmed there are some, but the issue with them is that they don’t run in a constrained environment nor concurrent placements and don’t validate the allocations, so the fact that there isn’t yet doubling of allocations isn’t an issue because there’s spare capacity | |
| 10:52:02 | sdague | cdent: yeh, that makes sense | |
| 10:52:31 | sdague | honestly, that's one of those things where doing the in tree functional testing with a couple of fake computes is probably the best way to flush it out | |
| 10:52:43 | cdent | sdague: that’s what gibi’s new test does | |
| 10:52:46 | cdent | so is very good to have | |
| 10:52:57 | sdague | cdent: does that still need review? | |