| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-02-26 | |||
| 12:14:23 | openstackgerrit | Sean Dague proposed openstack/nova master: api/metadata/vendordata_dynamic: don't import Requests for its constants https://review.openstack.org/434339 | |
| 15:36:01 | miss_hypothermia | mriedem: I fixed the issues and added a small test too for the service version check | |
| 15:36:33 | miss_hypothermia | mriedem: smcginnis had a question about when to clean up the code, I hope that's the last issue we need to figure out there and otherwise the fix is good to go :) | |
| 15:39:46 | mriedem | ok i'll take a look | |
| 15:43:46 | mriedem | miss_hypothermia: you're right, it's pike-eol | |
| 15:44:53 | miss_hypothermia | mriedem: ok, cool, tnx | |
| 15:56:57 | openstackgerrit | Kevin Benton proposed openstack/os-vif master: [wip] https://review.openstack.org/438272 | |
| 16:13:12 | mriedem | miss_hypothermia: found some test issues | |
| 16:15:50 | miss_hypothermia | mriedem: I'll check the patch if you added comments about it there | |
| 16:16:27 | mriedem | yup still going | |
| 16:20:49 | miss_hypothermia | mriedem: did you find anything that's complicated enough to talk about it here? | |
| 16:21:08 | mriedem | it's not really complicated, just an oversight, but actually something i brought up long long ago i think | |
| 16:21:20 | mriedem | cinder.API.check_attach would raise InvalidVolume, | |
| 16:21:41 | mriedem | we're now calling cinder.API.reserve_volume, which raises InvalidInput if cinder gives a 400 | |
| 16:22:03 | mriedem | the tests were mocked to raise InvalidVolume from reserve_volume, but that's not actually what would happen in reality | |
| 16:22:23 | mriedem | so _validate_bdm needs to handle both InvalidVolume and InvalidInput | |
| 16:23:38 | miss_hypothermia | mriedem: oh, ok, so one extra error handling and fix mock in the test | |
| 16:23:55 | miss_hypothermia | mriedem: I will fix it in the afternoon, when I get to the airport | |
| 16:24:13 | miss_hypothermia | thanks for pointing it out | |
| 16:25:58 | smcginnis | mriedem, miss_hypothermia: Commented on there, but if we only care about Ocata talking to newer versions, I'm having a brain block as to why we need to wait for pike-eol. | |
| 16:27:15 | mriedem | smcginnis: you know master is pike now right? | |
| 16:27:23 | smcginnis | mriedem: Yes. | |
| 16:27:25 | mriedem | smcginnis: this is a change to the nova api in pike | |
| 16:27:37 | mriedem | so ocata computes won't have the change introduced here | |
| 16:27:45 | smcginnis | mriedem: And we need to support pre-pike that does not have this change. | |
| 16:27:46 | mriedem | but pike computes will | |
| 16:28:03 | smcginnis | mriedem: So once pre-pike is gone, it's all on the new api. | |
| 16:28:24 | mriedem | which would be ocata-eol, i get what you're saying now | |
| 16:28:39 | smcginnis | mriedem: OK, so I'm not completely insane? | |
| 16:28:44 | mriedem | no | |
| 16:28:47 | mriedem | just partialy | |
| 16:28:50 | mriedem | *partially | |
| 16:28:53 | smcginnis | I can live with that. | |
| 16:29:40 | mriedem | i acked in the review | |
| 16:29:43 | mriedem | miss_hypothermia: comments are in | |
| 16:32:49 | mriedem | i'm going to go numb my brain in front of the tv for awhile now | |
| 16:33:48 | smcginnis | I'm going to go numb my brain and work on my taxes. | |
| 17:02:37 | miss_hypothermia | mriedem: smcginnis: Tnx | |
| 17:03:20 | miss_hypothermia | mriedem: smcginnis: I'll numb my brain at airports and airplanes :) | |
| 17:17:22 | smcginnis | miss_hypothermia: Safe travels! | |
| 17:28:00 | miss_hypothermia | smcginnis: thanks! | |
| 18:45:11 | openstackgerrit | Ildiko Vancsa proposed openstack/nova master: Implement new attach/detach Cinder flow https://review.openstack.org/330285 | |
| 18:45:12 | openstackgerrit | Ildiko Vancsa proposed openstack/nova master: Remove check_attach https://review.openstack.org/335358 | |
| 23:31:15 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Handle VolumeBDMIsMultiAttach in os-assisted-volume-snapshots https://review.openstack.org/429413 | |
| 23:31:15 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Handle conflicts for os-assisted-volume-snapshots https://review.openstack.org/429476 | |
| #openstack-nova - 2017-02-27 | |||
| 01:15:54 | openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/438309 | |
| 01:40:55 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Implement new attach/detach Cinder flow https://review.openstack.org/330285 | |
| 01:40:55 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Remove check_attach https://review.openstack.org/335358 | |
| 01:41:34 | mriedem | johnthetubaguy: i'm +2 on https://review.openstack.org/#/c/335358/ remove check_attach now | |
| 01:41:35 | mriedem | let's do it | |
| 02:26:03 | openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova master: Remove usage of config option verbose https://review.openstack.org/432271 | |
| 02:35:16 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Refactor a test method including 7 test cases https://review.openstack.org/395436 | |
| 03:19:11 | openstackgerrit | Jianghua Wang proposed openstack/nova master: XenAPI: device tagging https://review.openstack.org/333781 | |
| 03:41:31 | openstackgerrit | Nguyen Hung Phuong proposed openstack/nova master: Remove mox from nova/tests/unit/test_configdrive2.py https://review.openstack.org/322754 | |
| 05:18:47 | openstackgerrit | foundjem proposed openstack/nova master: adding more stuffs in the first-file https://review.openstack.org/438331 | |
| 08:40:47 | openstackgerrit | Kevin Benton proposed openstack/os-vif master: [wip] https://review.openstack.org/438272 | |
| 10:26:30 | openstackgerrit | Sivasathurappan Radhakrishnan proposed openstack/nova master: Remove mox from nova/tests/unit/api/openstack/compute/test_create_backup.py https://review.openstack.org/435610 | |
| 10:26:30 | openstackgerrit | Sivasathurappan Radhakrishnan proposed openstack/nova master: [WIP] Remove mox from unit/api/openstack/compute/admin_only_action_common.py https://review.openstack.org/438416 | |
| 10:54:30 | ildikov | johnthetubaguy: o/ | |
| 10:58:12 | kashyap | johnthetubaguy: Was the PTG productive? | |
| 11:00:27 | openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/438309 | |
| 11:03:50 | johnthetubaguy | kashyap: I think it was | |
| 11:04:07 | johnthetubaguy | kashyap: easier to tell how successful after the specs and code starts to land | |
| 11:04:18 | johnthetubaguy | ildikov: warmer now I guess? | |
| 11:04:27 | johnthetubaguy | ildikov: I have your check_attach patch open | |
| 11:04:56 | ildikov | johnthetubaguy: I made it back to Europe so outside it's colder, but inside warmer :) | |
| 11:05:08 | johnthetubaguy | ildikov: same here, lol | |
| 11:05:27 | ildikov | johnthetubaguy: haha, that patch went through a few iterations over the weekend :) | |
| 11:05:51 | ildikov | johnthetubaguy: I'm optimistic ;) | |
| 12:17:27 | openstackgerrit | John Garbutt proposed openstack/nova-specs master: Add live-migration-force-after-timeout spec https://review.openstack.org/438467 | |
| 12:20:59 | danpawlik | Can someone take a look on https://review.openstack.org/#/c/436953/ ? | |
| 12:37:08 | johnthetubaguy | ildikov: when we detach a volume, that always does unreserve right? | |
| 12:38:29 | ildikov | johnthetubaguy: yes , it does | |
| 12:38:43 | johnthetubaguy | ildikov: thats in the Cinder API I assume? | |
| 12:40:24 | zioproto | hello there, there is a much of commits between the latest tag and stable/mitaka ... is there going to be a new mitaka release for nova ? | |
| 12:40:33 | zioproto | there is a bug that blocks the mitaka to newton upgrade for me | |
| 12:40:48 | zioproto | the fix is merged in stable/mitaka but never released in packaging | |
| 12:41:19 | johnthetubaguy | zioproto: lyarwood might now what the plans are there, I don't remember where we are with that now | |
| 12:43:09 | zioproto | lyarwood, let me know :) This is merged but out of any distro: https://review.openstack.org/#/c/382639/ and this is blocking for mitaka to newton upgrade for me. | |
| 13:08:52 | satyar | Hi Jeffrey4l | |
| 13:09:52 | satyar | Hi diana_clarke | |
| 13:10:33 | diana_clarke | satyar: hello? | |
| 13:11:01 | satyar | How are you diana_clarke | |
| 13:11:32 | lyarwood | zioproto / johnthetubaguy ; I would say soon given mitaka is about to hit EOL upstream, I'll talk to matt about it later today | |
| 13:11:42 | diana_clarke | satyar: Okay. Just getting the kids out the door to school. You? | |
| 13:11:56 | satyar | i am doing ok | |
| 13:12:00 | lyarwood | zioproto: that said if this is for RH OSP you can file a support case and we can look at getting this in a build for you ASAP | |
| 13:12:10 | satyar | just want to check have you checked the recent nova code | |
| 13:12:45 | diana_clarke | satyar: Do you have a specific nova question? There's lots of nova code ;) | |
| 13:13:13 | satyar | yes | |
| 13:13:37 | satyar | i created 20 vms(boot from volume) my quota for volume is 10 | |
| 13:13:48 | satyar | so 10 VMs gone to error state | |
| 13:14:34 | satyar | after i delete all the VMs the quota for CPU/RAM/Instances are showing used 10 | |
| 13:15:04 | diana_clarke | satyar: It sounds like your question might be a better fit for the #openstack channel (see channel description above). | |
| 13:15:09 | satyar | ideally it(used quota) should be 0 as all VMs are deleted | |
| 13:15:39 | satyar | i think its a issue with the quota code | |
| 13:15:45 | satyar | from nova | |
| 13:16:46 | satyar | if you can point me the the file which runs the code of releasing quota after VM deletion will help me to find the root cause and fix it | |
| 13:24:17 | openstack | Launchpad bug 1667667 in OpenStack Compute (nova) "Rebuilding instance ignores image's property hw_disk_bus" [Low,Confirmed] | |
| 13:24:17 | akijak | hi, I've found a bug in nova concerning rebuilding and 'hw_disk_bus' image property (https://bugs.launchpad.net/nova/+bug/1667667), I'm looking for someone who could give me some directions how to fix that :) can you help? | |