| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-08-01 | |||
| 06:10:24 | openstackgerrit | Michael Still proposed openstack/nova master: Move execs of tee to privsep. https://review.openstack.org/489438 | |
| 06:10:25 | openstackgerrit | Michael Still proposed openstack/nova master: Read from console ptys using privsep. https://review.openstack.org/489486 | |
| 06:10:25 | openstackgerrit | Michael Still proposed openstack/nova master: rootwrap.d cleanup mislabeled files https://review.openstack.org/486831 | |
| 06:14:35 | openstackgerrit | Zhaokun Fu proposed openstack/nova master: Smaller disk flavor check https://review.openstack.org/489484 | |
| 06:17:28 | openstackgerrit | jichenjc proposed openstack/nova master: Move common definition into common.py https://review.openstack.org/489491 | |
| 06:17:29 | openstackgerrit | jichenjc proposed openstack/nova master: WIP:check query param for service's index function https://review.openstack.org/489492 | |
| 06:53:35 | openstackgerrit | jichenjc proposed openstack/nova master: WIP:check query param for service's index function https://review.openstack.org/489492 | |
| 07:13:43 | openstackgerrit | jichenjc proposed openstack/nova master: WIP: allow null to be input for key_name https://review.openstack.org/489505 | |
| 07:22:48 | openstack | Launchpad bug 1707256 in OpenStack Compute (nova) "Scheduler report client does not account for shared resource providers" [High,Confirmed] - Assigned to Jay Pipes (jaypipes) | |
| 07:22:48 | bhagyashris | jaypipes: Hi, Found one bug reported on launchpad https://bugs.launchpad.net/nova/+bug/1707256 so is this will solve the issue that we have discussed yesterday. | |
| 07:53:57 | openstackgerrit | jichenjc proposed openstack/nova master: enhance error message for input field https://review.openstack.org/489519 | |
| 08:03:47 | openstackgerrit | jichenjc proposed openstack/nova master: WIP:enhance error message for input field https://review.openstack.org/489519 | |
| 08:18:42 | openstackgerrit | jichenjc proposed openstack/nova master: add description about key_name https://review.openstack.org/489525 | |
| 08:26:57 | openstackgerrit | jichenjc proposed openstack/nova master: Move common definition into common.py https://review.openstack.org/489491 | |
| 08:26:58 | openstackgerrit | jichenjc proposed openstack/nova master: WIP:check query param for service's index function https://review.openstack.org/489492 | |
| 08:32:23 | mamandle | bauzas: sfinucan: mriedem https://review.openstack.org/#/c/483911/ is awaiting another +2, can you please take a look, Thanks. | |
| 08:39:34 | bhagyashris | Hi all I am facing one issue to boot instance in multinode setup. Error ""Host 'host-name' is not mapped to any cell". because of this instance is going to error state Can any one have any idea about this to resolve. | |
| 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 | |