| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-12-19 | |||
| 14:47:14 | mriedem | bauzas: NonCallableMock just does the thing i already want | |
| 14:47:17 | bauzas | I see | |
| 14:47:38 | bauzas | anway, I don't want to discuss about the pattern | |
| 14:47:58 | bauzas | my point is just that when reviewing the change, we need to understand that noncallablemock already supports that | |
| 14:48:12 | bauzas | without needing to verify the call count | |
| 14:48:30 | bauzas | less explicit, but interesting tho | |
| 14:48:33 | mriedem | we = you? | |
| 14:48:39 | mriedem | now you know :) | |
| 14:48:55 | mriedem | i expect to see it in all of your new tests now | |
| 14:49:25 | bauzas | heh | |
| 14:50:49 | openstackgerrit | Jackie Truong proposed openstack/python-novaclient master: Microversion 2.59 - Add trusted_image_certificates https://review.openstack.org/500396 | |
| 15:00:54 | openstackgerrit | Merged openstack/nova stable/newton: Make request_spec.spec MediumText https://review.openstack.org/528338 | |
| 15:02:20 | mriedem | huh https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:stable/newton | |
| 15:02:25 | mriedem | to eol or not to eol | |
| 15:02:26 | lyarwood | \o/ | |
| 15:02:53 | mriedem | i worry about not having https://review.openstack.org/#/c/528835/ in newton | |
| 15:03:24 | mriedem | but i also don't know how many people running newton are going to have conductor split out yet and not have the cell conductor configured to hit the api db | |
| 15:03:35 | mriedem | as bauzas noted, probably not a real worry | |
| 15:06:17 | mriedem | unrelated, i also started thinking about https://review.openstack.org/#/q/topic:fix-bfv-boot-resources+(status:open+OR+status:merged) again... | |
| 15:06:42 | mriedem | and whether or not we should just take on the debt since shared provider modeling is who knows how far off yet | |
| 15:13:25 | mriedem | no shame intended | |
| 15:13:57 | mriedem | it's that we put that off for a few releases because we were saying placement would fix the problem, and we haven't yet, and people (ops) ask for it at least once per cycle | |
| 15:14:02 | jaypipes | cdent shaming is indeed the best kind of shaming. second only to pug shaming. | |
| 15:14:56 | openstackgerrit | Merged openstack/nova master: [placement] Add x-openstack-request-id in API ref https://review.openstack.org/523007 | |
| 15:15:09 | cdent | mriedem: don't worry, I'll feel shame, even for things entirely outside my control and/or the result of perfectly reasonable decision making processes | |
| 15:15:19 | cdent | I may be part pug | |
| 15:29:30 | maciejjozefczyk | jaypipes: Hey :) I responded to your comment https://review.openstack.org/#/c/520024/ Could you please check it? Is it possible to discuss it when you'll check it? Maybe on Thursdays meeting? Thanks :) | |
| 15:30:09 | openstackgerrit | Jackie Truong proposed openstack/python-novaclient master: Microversion 2.59 - Add trusted_image_certificates https://review.openstack.org/500396 | |
| 15:38:03 | jaypipes | maciejjozefczyk: I should be able to get to that patch today, yes. | |
| 15:39:24 | maciejjozefczyk | jaypipes: thanks a lot :) | |
| 16:07:04 | mriedem | lyarwood: artom: bauzas: did we or did we not say that we needed a minor version bump on stable for the release with the schema migration? | |
| 16:07:54 | lyarwood | mriedem: we don't \need\ it for anything but I think we agreed it would be nice to have a minor version bump for this, yes. | |
| 16:11:14 | mriedem | ok here is ocata https://review.openstack.org/529100 | |
| 16:14:04 | mriedem | and newton: https://review.openstack.org/529102 | |
| 16:27:12 | openstackgerrit | Merged openstack/nova master: Updated from global requirements https://review.openstack.org/528881 | |
| 16:32:11 | mriedem | jaypipes: on maciejjozefczyk's patch, i'm assuming the shutdown instances thing is a problem because of _update_usage_from_instance which is called between the initial compute node update and the final one, | |
| 16:32:19 | mriedem | and _update_usage_from_instance calls self.stats.update_stats_for_instance(instance, is_removed_instance) | |
| 16:32:24 | mriedem | which looks at things like vm_sate | |
| 16:32:25 | mriedem | *state | |
| 16:32:57 | jaypipes | yeah | |
| 16:33:04 | mriedem | and calls _update_usage | |
| 16:33:33 | mriedem | i'm not sure wth cn.current_workload = self.stats.calculate_workload() is for | |
| 16:34:06 | mriedem | no filters use that, it's just for reporting out of the API i guess | |
| 16:34:12 | jaypipes | mriedem: switched my vote on it. | |
| 16:39:04 | openstackgerrit | Merged openstack/python-novaclient master: Updated from global requirements https://review.openstack.org/528911 | |
| 16:39:14 | mriedem | jaypipes: i think he still has changes to make | |
| 16:39:17 | mriedem | per my earlier review | |
| 16:39:44 | mriedem | in _check_for_nodes_rebalance | |
| 16:46:45 | jaypipes | mriedem: sure, though that's only going to be valid for baremetal nodes... | |
| 16:47:04 | jaypipes | mriedem: not sure there's much of a race interval for that... but maybe | |
| 17:01:34 | openstackgerrit | Stephen Finucane proposed openstack/nova master: console: introduce framework for RFB authentication https://review.openstack.org/345397 | |
| 17:01:35 | openstackgerrit | Stephen Finucane proposed openstack/nova master: console: Provide an RFB security proxy implementation https://review.openstack.org/345399 | |
| 17:01:35 | openstackgerrit | Stephen Finucane proposed openstack/nova master: console: introduce the VeNCrypt RFB authentication scheme https://review.openstack.org/345398 | |
| 17:01:36 | openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Document TLS security setup for noVNC proxy https://review.openstack.org/500544 | |
| 17:02:43 | cdent | jaypipes: speaking of cdent shaming, I was hoping you were going to shame me for my infinite resource classes crack in the latest placement update | |
| 17:04:34 | jaypipes | cdent: haven't gotten that far yet. | |
| 17:05:24 | jaypipes | cdent: still trying to wrestle with friggin server groups. | |
| 17:13:37 | ludovic_ | Hello everyone | |
| 17:13:54 | stephenfin | ludovic_: o/ | |
| 17:14:33 | openstackgerrit | Merged openstack/os-vif master: Check if interface belongs to a Linux Bridge before removing https://review.openstack.org/526079 | |
| 17:14:36 | mnaser | Spec-ing out new compute and in the interest of deployers and doing things in open we’re planning to publish the document ... I wanted to gather some feedback at what sort of cpu overcommit you’ve ran/seen people run? | |
| 17:14:45 | ludovic_ | Maybe someone can help me to understand the Filter Scheduler ? I have found a strange scheduler behaviour while host-evacuate | |
| 17:18:32 | ludovic_ | I have two compute Nodes. The first compute Node with 31 instances ( total RAM allocated = 177152 M) , the second one with 2 instances ( Total RAM allocated = 24576 ) | |
| 17:19:23 | ludovic_ | RAM of computes Nodes is 196483 (memory_mb) | |
| 17:20:50 | ludovic_ | Host-evacuate work but 5 VMs was on ERROR with insufficient memory (nova-compute log) and on was with NO STATE (No Host found by RAM Filter) | |
| 17:21:36 | ludovic_ | I expected the same ERROR on these 6 instances | |
| 17:22:44 | ludovic_ | I used tripleO to deploy my OpenStack environment | |
| 17:22:57 | ludovic_ | with Ocata repository | |
| 17:24:03 | ludovic_ | nova.conf : enabled_filters=RetryFilter,AggregateInstanceExtraSpecsFilter,AvailabilityZoneFilter,RamFilter,DiskFilter,ComputeFilter,ComputeCapabilitiesFilter,ImagePropertiesFilter | |
| 17:25:53 | ludovic_ | Maybe someone can explain this situation ? Thanks a lot | |
| 17:26:03 | ludovic_ | nova specialist ? | |
| 17:29:55 | ludovic_ | is someone have already wrote a scheduler filter to prioritize rebuild of instances ? | |
| 17:44:21 | jaypipes | mriedem: http://paste.openstack.org/show/629349/ .. is there some other place other nova/api/openstack/api_version_request.py that I need to bump a max microversion? | |
| 17:50:53 | cfriesen | question about the DB interfaces...why does some code go through nova.db.api and other code directly uses nova.db.sqlalchemy.api ? | |
| 17:57:48 | mriedem | jaypipes: yeah the version samples | |
| 17:58:11 | mriedem | jaypipes: https://github.com/openstack/nova/tree/master/doc/api_samples/versions | |
| 17:58:46 | jaypipes | mriedem: I looked there but all I see is an interpolation marker for max_api_version | |
| 17:59:09 | jaypipes | mriedem: gah, never mind. | |
| 17:59:15 | jaypipes | mriedem: sigh... | |
| 17:59:26 | jaypipes | mriedem: was looking in nova/tests/functional/api_samples/ | |
| 17:59:44 | jaypipes | mriedem: have I mentioned I hate these? :) | |
| 18:04:35 | cfriesen | is all quota information now going into the API DB? or will we still put some in the main DB? | |
| 18:05:32 | openstackgerrit | Merged openstack/nova master: Fix 4 doc typos https://review.openstack.org/529084 | |
| 18:06:18 | jaypipes | melwitt: see cfriesen's ? above... | |
| 18:40:25 | openstackgerrit | Merged openstack/python-novaclient master: CommandError is raised for invalid server fields https://review.openstack.org/525110 | |
| 18:51:24 | mriedem | cfriesen: api db | |
| 18:51:54 | mriedem | cfriesen: starting in pike we don't use the usages or reservations tables anymore | |
| 19:01:48 | openstackgerrit | Jay Pipes proposed openstack/nova-specs master: Support aggregate affinity scheduler filters https://review.openstack.org/529135 | |
| 19:31:02 | openstackgerrit | Jay Pipes proposed openstack/nova-specs master: Support aggregate affinity scheduler filters https://review.openstack.org/529135 | |
| 20:11:11 | rybridges | So I am seeing errors every time i make a snapshot | |
| 20:11:43 | rybridges | regarldess of whether i use disable_libvirt_livesnapshot = true or disable_libvirt_livesnapshot = false | |
| 20:11:49 | rybridges | the error is the same every time | |
| 20:11:53 | rybridges | it happens on the hypervisor | |
| 20:12:04 | rybridges | this is nova-compute.log https://pastebin.com/WiqYtZzF | |
| 20:13:34 | rybridges | this is the error for libvirt log: https://pastebin.com/4NeSuNvX | |
| 20:14:17 | rybridges | its impossible to take a snapshot on ocata from the horizon ui | |
| 20:14:20 | rybridges | as far as we can tell | |
| 20:18:42 | mriedem | https://www.jrssite.com/wordpress/?p=302 | |
| 20:18:49 | mriedem | https://bugs.launchpad.net/nova/+bug/1381153 | |
| 20:18:50 | openstack | Launchpad bug 1381153 in OpenStack Compute (nova) "Cannot create instance live snapshots in Centos7 (icehouse)" [Undecided,Invalid] | |