| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-02-02 | |||
| 16:59:23 | jaypipes | ah. thx mriedem. | |
| 17:01:16 | alex_xu | mriedem: let me try to take care of https://review.openstack.org/#/c/421760 | |
| 17:02:02 | openstackgerrit | John Garbutt proposed openstack/nova master: [WIP] Placement api: Add json_error_formatter to defaults https://review.openstack.org/395194 | |
| 17:02:31 | ildikov | johnthetubaguy: mriedem: can you come to the Cinder-NOva meeting? | |
| 17:04:12 | johnthetubaguy | ildikov: ah, thats moved to today then I guess | |
| 17:04:50 | johnthetubaguy | ildikov: I have updated my calendar, but honestly I have to chase a few of these RC related patches | |
| 17:05:04 | mriedem | ildikov: busy with rc1 things | |
| 17:05:37 | ildikov | johnthetubaguy: mriedem: yep, it got moved, but will ping you directly if something pops up then | |
| 17:06:55 | mriedem | alex_xu: thanks, i think at this point i'd rather just make the reno a separate patch | |
| 17:07:08 | mriedem | the api-ref change is too large and needs a much more detailed review | |
| 17:07:32 | alex_xu | mriedem: yea, agree with that, just working on separating the patch | |
| 17:29:15 | openstackgerrit | Diana Clarke proposed openstack/nova master: Add offset & limit docs & tests https://review.openstack.org/411441 | |
| 17:33:02 | openstackgerrit | John Garbutt proposed openstack/nova master: Report reserved_host_disk_mb in GB not KB https://review.openstack.org/428120 | |
| 17:39:53 | jaypipes | johnthetubaguy: lol between 2.7 and 3.5 float handling, eh? | |
| 17:43:06 | melwitt | jaypipes: mriedem1 and I were discussing yesterday, is everything already merged that handles the local disk for volume backed instances to treat it as 0? does it also request 0 disk during a rebuild, etc? | |
| 17:46:00 | mriedem | mdbooth: i'm going through your nfs mount patch now | |
| 17:46:05 | mriedem | mdbooth: so don't drop off yet | |
| 17:46:09 | openstackgerrit | Alex Xu proposed openstack/nova master: Add release note for filter/sort whitelist https://review.openstack.org/428285 | |
| 17:46:14 | alex_xu | mriedem: ^ done | |
| 17:46:18 | mriedem | thanks | |
| 17:46:21 | openstackgerrit | Maciej Szankin proposed openstack/nova master: conf: mark glance.api_servers opt as required https://review.openstack.org/428241 | |
| 17:46:24 | alex_xu | np | |
| 17:46:38 | dansmith | melwitt: yeah, I think we should be good there | |
| 17:47:24 | melwitt | sweet | |
| 17:53:46 | johnthetubaguy | jaypipes: the joys! | |
| 17:54:06 | johnthetubaguy | jaypipes: I have a concern on this ironic patch still: https://review.openstack.org/#/c/404472 | |
| 17:54:25 | johnthetubaguy | jaypipes: but something tells me I am miss understanding the resource tracker sequence | |
| 17:55:17 | diana_clarke | sfinucan: thanks a bunch for reviewing that! | |
| 18:02:24 | sfinucan | diana_clarke: np :) | |
| 18:02:25 | mriedem | mdbooth: lyarwood: before i go further, some comments/questions inline https://review.openstack.org/#/c/383859/13/nova/virt/libvirt/volume/mount.py | |
| 18:09:43 | BobBall | johnthetubaguy / mriedem FYI I believe https://review.openstack.org/#/c/428180/ will fix the Citrix XenServer CI | |
| 18:10:26 | mriedem | cool | |
| 18:10:28 | BobBall | johnthetubaguy / mriedem: A dodgy Cinder commit meant that "older" versions of pvs would all fail (as they don't have the newly required option) - the check for the version was broken. | |
| 18:11:11 | BobBall | We will monitor the CI after this cinder commit is merged and let you know the results and when we'd like to re-start voting. | |
| 18:11:16 | mriedem | sure | |
| 18:11:31 | openstackgerrit | Maciej Szankin proposed openstack/nova master: conf: resolved TODOs in compute configuration https://review.openstack.org/426793 | |
| 18:11:43 | mriedem | i guess if LVM.supports_pvs_ignoreskippedcluster was just set to False | |
| 18:11:56 | mriedem | because of the old pvs | |
| 18:12:05 | mriedem | *lvm | |
| 18:13:03 | BobBall | Actually it was always returning true... LVM.supports_pvs_ignoreskippedcluster was *meant* to be reading a property with a method behind it (using self.supportsXXX) but LVM.supportsXXX is the method itself so that was not None. | |
| 18:13:11 | openstackgerrit | melanie witt proposed openstack/nova master: Add step-by-step examples for Cells V2 setup https://review.openstack.org/427981 | |
| 18:13:13 | BobBall | oh sorr I mis-read. | |
| 18:13:33 | BobBall | Couldn't "set" it because it's just a method on the class | |
| 18:20:29 | mriedem | mdbooth: lyarwood: done with that review, only real questions are in that mount module | |
| 18:21:20 | jaypipes | johnthetubaguy: +W from me on you disk_gb patch. | |
| 18:28:53 | mriedem | is anyone working a fix for this? https://bugs.launchpad.net/nova/+bug/1659266 | |
| 18:28:53 | openstack | Launchpad bug 1659266 in OpenStack Compute (nova) "Disk allocation for instance is not good with swap" [High,Confirmed] - Assigned to Tanvir Talukder (tanvirt16) | |
| 18:29:18 | mriedem | bauzas: ^ you reported it | |
| 18:29:29 | EmilienM | 2 questions : | |
| 18:29:53 | EmilienM | 1. Where [cells] section needs to be configured (on which services, eg: nova-api? etc) | |
| 18:29:53 | mriedem | (1) who is your daddy and (2) what does he do? | |
| 18:30:04 | mriedem | EmilienM: cells config is only for cells v1 | |
| 18:30:06 | mriedem | you don't care about it | |
| 18:30:14 | EmilienM | 2. Do we *need* cells/enable= true in Ocata, | |
| 18:30:17 | mriedem | no | |
| 18:30:19 | EmilienM | amoralej: ^ | |
| 18:30:27 | mriedem | [cells] is all about cells v1 | |
| 18:30:44 | EmilienM | mriedem: ok | |
| 18:30:49 | amoralej | thanks mriedem | |
| 18:30:55 | EmilienM | yeah thx | |
| 18:31:01 | mriedem | i'll update the config option help to make that more clear | |
| 18:31:09 | EmilienM | mriedem: we're unable to spawn VMs anymore | |
| 18:31:28 | EmilienM | maybe you'll know why | |
| 18:31:47 | EmilienM | http://logs.openstack.org/15/424915/8/check/gate-puppet-openstack-integration-4-scenario001-tempest-centos-7/b29f35b/logs/nova/nova-compute.txt.gz#_2017-02-02_12_57_00_754 | |
| 18:32:16 | amoralej | mriedem, we getg "Instance 6a26cec0-3769-44b6-851c-b77359897e07 could not be found." kind of errors in certain tempest tests | |
| 18:32:38 | EmilienM | mriedem: we're deploying trunk | |
| 18:32:41 | EmilienM | with UC | |
| 18:33:17 | johnthetubaguy | jaypipes: thanks! | |
| 18:35:56 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Clarify the [cells] config option help https://review.openstack.org/428344 | |
| 18:35:57 | mriedem | melwitt: dansmith: ^ | |
| 18:37:06 | mriedem | EmilienM: that trace says the bdm is invalid | |
| 18:37:37 | EmilienM | bdm? | |
| 18:37:40 | mriedem | EmilienM: because the volume isn't found http://logs.openstack.org/15/424915/8/check/gate-puppet-openstack-integration-4-scenario001-tempest-centos-7/b29f35b/logs/nova/nova-compute.txt.gz#_2017-02-02_12_56_57_274 | |
| 18:37:45 | mriedem | bdm == block_device_mapping | |
| 18:37:48 | EmilienM | Block Device Mapping | |
| 18:37:49 | EmilienM | ok | |
| 18:38:01 | EmilienM | well, it worked yesterday | |
| 18:38:13 | mriedem | EmilienM: did it work with devstack? :P | |
| 18:38:19 | johnthetubaguy | mriedem: I can put something up for that bug 1659266 | |
| 18:38:19 | openstack | bug 1659266 in OpenStack Compute (nova) "Disk allocation for instance is not good with swap" [High,Confirmed] https://launchpad.net/bugs/1659266 - Assigned to Tanvir Talukder (tanvirt16) | |
| 18:38:26 | mriedem | johnthetubaguy: thanks | |
| 18:38:30 | EmilienM | mriedem: I was about to say it but well... | |
| 18:38:38 | johnthetubaguy | I have to run, soon, but lets see what can be done | |
| 18:38:50 | mriedem | johnthetubaguy: you're not allowed to leave | |
| 18:38:55 | EmilienM | mriedem: FYI we're delaying TripleO release because upgrades are not working currently from Newton to Ocata | |
| 18:38:55 | mriedem | sally will understand | |
| 18:39:07 | EmilienM | mriedem: we hope to solve blockers by next week :/ | |
| 18:39:14 | johnthetubaguy | mriedem: you haven't met the conductor, she is scary | |
| 18:41:15 | mriedem | EmilienM: my wild guess would be something related to https://review.openstack.org/#/c/427782/ | |
| 18:41:22 | mriedem | which merged last night | |
| 18:41:33 | mriedem | EmilienM: but would also mean you don't have things setup for cells v2 | |
| 18:42:07 | EmilienM | mriedem: do you run experimental job sometimes? | |
| 18:42:12 | EmilienM | it seems like you're not | |
| 18:42:44 | EmilienM | mriedem: I think we are pretty well covered for cells v2 but I'll need to confirm | |
| 18:43:00 | diana_clarke | EmilienM: I've run it, but I've yet to see it pass ;( | |
| 18:43:28 | EmilienM | diana_clarke: where? | |
| 18:43:48 | diana_clarke | EmilienM: for example: https://review.openstack.org/#/c/420186/ | |
| 18:44:10 | diana_clarke | (it's not recent, but people are running it) | |
| 18:44:52 | EmilienM | diana_clarke: cool | |
| 18:46:49 | dansmith | I run it too on occasion, but it takes a long time, fails often, and the logs are harder to sift through | |