| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-08-22 | |||
| 20:33:08 | mriedem | so yeah that ram check probably shouldn't even be in conductor | |
| 20:34:20 | cfriesen_ | arguably it makes sense in the current flow since it's a critical resource...hard to overcommit ram the way you can with cpu | |
| 20:58:45 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Restrict live migration to same cell https://review.openstack.org/496419 | |
| 20:58:55 | mriedem | cfriesen_: dansmith: there is the live migration restricted to same cell fix ^ | |
| 22:05:30 | cfriesen_ | mriedem: you're a machine with all these fixes. :) | |
| 22:07:04 | mriedem | all these 2 fixes | |
| 22:08:36 | cfriesen_ | is there a way to query the hosts in a cell? | |
| 22:08:54 | cfriesen_ | don't see anything in the api-ref | |
| 22:10:51 | cfriesen_ | where I'm going with this is...how does the admin know which hosts are part of the same cell? | |
| 22:20:53 | mriedem | cfriesen_: https://docs.openstack.org/nova/latest/user/cells.html#faqs | |
| 22:21:43 | mriedem | we probably need another cli in queens for that | |
| 22:21:54 | mriedem | or build on list_cells | |
| 22:33:04 | cfriesen_ | mriedem: okay, makes sense | |
| 22:36:34 | cfriesen_ | mriedem: would we need a spec for nova-manage? (does it count as a public API?) | |
| 22:57:04 | mriedem | nope | |
| 23:37:11 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Remove source node allocation after live migration completes https://review.openstack.org/496032 | |
| 23:37:12 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Restrict live migration to same cell https://review.openstack.org/496419 | |
| 23:37:13 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Allocate resources on forced dest host during live migration https://review.openstack.org/496031 | |
| #openstack-nova - 2017-08-23 | |||
| 00:04:06 | cali_boxer | hello all, | |
| 00:04:19 | cali_boxer | i am having an issue with nova-scheduler | |
| 00:04:38 | cali_boxer | it's reporting system has more free space than dataabase expected | |
| 00:04:49 | cali_boxer | this is on os liberty redhat 7.2 | |
| 00:05:19 | cali_boxer | i cannot create stacks as the server creation fails | |
| 00:05:49 | cali_boxer | i can create independent os images without an issue | |
| 01:22:05 | openstackgerrit | Naichuan Sun proposed openstack/nova master: xenapi: cached images should be cleaned up by time https://review.openstack.org/465954 | |
| 01:53:19 | openstackgerrit | Alex Xu proposed openstack/nova master: placement: ensure RP maps to those RPs that share with it https://review.openstack.org/480379 | |
| 02:06:01 | clarkb | dansmith: mriedem for tomorrow in reviewing https://review.openstack.org/#/c/491955/1 I noticed that the cell1 cond logs are slightly off whihc confuses os-loganalyze and will result in no lines being indexed for that service if we try indexing them as is | |
| 02:20:32 | alex_xu | gmann: the no-more-extension in queens https://etherpad.openstack.org/p/api-no-more-extensions-pike | |
| 02:21:08 | gmann | alex_xu, hi | |
| 02:21:27 | alex_xu | gmann: hi, i'm thinking of whether we have sample file for json-schema | |
| 02:21:53 | gmann | alex_xu, yea | |
| 02:22:20 | gmann | alex_xu, i will push 1 or 2 patch for each item as sample | |
| 02:22:33 | gmann | and based on that we can discuss those in PTG | |
| 02:22:36 | alex_xu | gmann: cool, thanks | |
| 02:23:01 | gmann | alex_xu, do you think we should add BP also or after PTG feedback? | |
| 02:23:40 | alex_xu | gmann: we can add one now | |
| 02:23:46 | gmann | alex_xu, ok | |
| 02:23:47 | alex_xu | gmann: do you want to create one? | |
| 02:24:15 | gmann | alex_xu, ok, i will push remaining things there with adding sample patches | |
| 02:24:33 | alex_xu | gmann: thanks! | |
| 02:25:15 | alex_xu | gmann: are you interesting on taking look at how to add sample file for json-schema? | |
| 02:25:50 | alex_xu | gmann: that can be used to avoid the bug like this https://bugs.launchpad.net/nova/+bug/1658571 | |
| 02:25:52 | openstack | Launchpad bug 1658571 in OpenStack Compute (nova) "Microversion 2.37 break 2.32 usage" [High,Fix released] - Assigned to Artom Lifshitz (notartom) | |
| 02:26:49 | gmann | alex_xu, yea that will be nice. | |
| 02:27:07 | artom | A long time ago I started https://review.openstack.org/#/c/430352/ in relation to that bug | |
| 02:27:32 | artom | It needs a massive amount of manual sample fixing to eventually work though :( | |
| 02:28:24 | gmann | artom, ohk, even i was thinking to run a gate job with 'latest' but not all tempest test will pass and it will need lot of refactoring | |
| 02:28:59 | artom | gmann, mine is just for api sample tests, to start with at least | |
| 02:29:08 | gmann | yea | |
| 02:29:59 | artom | I just got discouraged by the, like, 150 failing tests that 2.latest created, and the need to manually go and either skip or fix each one | |
| 02:30:47 | gmann | most of them need capping of microversion | |
| 02:31:55 | gmann | anyways we can think more on this. may be simple set of mandatory element of schema and test them with latest | |
| 02:32:03 | gmann | alex_xu, you have any other idea on this | |
| 02:33:36 | alex_xu | gmann: artom i'm just thinking of the API sample tests also generate all the json-schema to a sample file, then we validate the jsons-schema whether is changed unexpected | |
| 02:34:51 | artom | alex_xu, I'm not sure I follow (it may have something to do with it being 22:30 here) | |
| 02:35:39 | gmann | alex_xu, i see. | |
| 02:35:52 | alex_xu | artom: so the bug is about we change the old schema when change schema in the new version | |
| 02:36:48 | artom | alex_xu, yeah, we inadvertently removed something | |
| 02:36:50 | alex_xu | artom: then we generate all the json-schema sample files through api-sample tests, and compare all the json-schema sample file whether changed | |
| 02:37:29 | gmann | alex_xu, i am just remembering why current sample tests did not catch that bug? | |
| 02:37:48 | artom | gmann, because tests are pinned to a specific microversion | |
| 02:38:02 | artom | gmann, so 2.32 was fine, and what 2.37 added was also fine | |
| 02:38:17 | artom | gmann, we just never tested that what 2.32 added still worked in 2.33, 34, ... 37, etc | |
| 02:39:12 | artom | alex_xu, but some schema changes are intentional, so (if I'm understanding you correctly) we can't just fail if the schema changes | |
| 02:40:10 | alex_xu | gmann: oh...wait. that bugs break the schema in the microversion, not the schema in the old microversion | |
| 02:40:24 | gmann | yea | |
| 02:40:51 | gmann | if anyone change old schema then it should already catch by current tests | |
| 02:41:27 | alex_xu | yea, I see now why artom want to run all the api-samples test on the latest version now | |
| 02:43:19 | gmann | yea | |
| 02:45:55 | artom | alex_xu, gmann, if you are are up for reviewing that, I can try to conjure the motivation to beat into shape and get it to pass Jenkins ;) | |
| 02:46:39 | alex_xu | artom: thanks, I need to clear my mind first | |
| 02:47:14 | artom | alex_xu, of course :) Like I said, it's 22:30, so nothing's happening tonight, that's for sure | |
| 02:47:27 | artom | And it'll take me a few days, I suspect | |
| 02:47:28 | alex_xu | artom: yea :) | |
| 02:47:57 | gmann | artom, alex_xu yea, i also need to think more on this how can cover with minimal effort | |
| 05:59:06 | openstackgerrit | Alex Xu proposed openstack/nova master: Remove the section about extensions from the API concept doc https://review.openstack.org/496500 | |
| 06:08:49 | openstackgerrit | Merged openstack/nova master: update comment for dropping support https://review.openstack.org/494061 | |
| 06:20:57 | tobasco | cfriesen_: sorry for late reply had to leave work yesterday, thanks for testing it out, we create our instances using the api from our controlpanel perhaps there is some integration issue there, but we are sending with device_name but iirc libvirt does not honor user requested device names as it says in the log | |
| 06:21:14 | tobasco | i wonder what the difference is, we use a dell backend for our cinder volumes but i doubt it's really related to cinder though | |
| 06:21:40 | tobasco | cfriesen_: gonna see if i can figure something out, you dont do anything special with properties on your glance image i assume | |
| 06:50:37 | openstackgerrit | Wonil Choi proposed openstack/nova master: libvirt: Fix getting a wrong guest object https://review.openstack.org/496515 | |
| 06:50:42 | tobasco | cfriesen_: thanks for testing it out earlier, didn't have the same problem when booting from horizon either so must be that we are sending some param when we provision using the api | |
| 06:50:55 | tobasco | causing that behaviour, perhaps device_name, gonna see if i can narrow it down | |
| 07:18:03 | openstackgerrit | jichenjc proposed openstack/nova master: Do not attempt volume swap when guest is stopped/suspended https://review.openstack.org/389798 | |
| 07:41:45 | openstackgerrit | Ilya Popov proposed openstack/nova master: Tests: Add cleanup of 'instances' directory https://review.openstack.org/491589 | |
| 08:52:22 | openstackgerrit | Stephen Finucane proposed openstack/nova master: placement: Make API history doc more consistent https://review.openstack.org/477478 | |
| 08:53:04 | openstackgerrit | Stephen Finucane proposed openstack/nova master: pci: Validate behavior of empty devname https://review.openstack.org/396130 | |
| 08:54:32 | openstackgerrit | Merged openstack/nova master: Add language for compute node configuration https://review.openstack.org/489643 | |
| 08:54:47 | openstackgerrit | Stephen Finucane proposed openstack/nova master: objects: remove cpuset_reserved from __init__ of InstanceNUMATopology https://review.openstack.org/466030 | |
| 08:54:48 | openstackgerrit | Stephen Finucane proposed openstack/nova master: objects: remove related pinning from __init__ of InstanceNUMATopology https://review.openstack.org/485554 | |
| 08:54:48 | openstackgerrit | Stephen Finucane proposed openstack/nova master: objects: remove pagesize from __init__ of InstanceNUMATopology https://review.openstack.org/485553 | |
| 08:54:52 | openstackgerrit | jichenjc proposed openstack/nova master: WIP: use conf for keystone session creation https://review.openstack.org/485121 | |
| 09:26:12 | openstackgerrit | Gábor Antal proposed openstack/nova master: Transform aggregate.update_prop notification https://review.openstack.org/462576 | |
| 09:27:36 | openstackgerrit | Gábor Antal proposed openstack/nova master: Transform instance.resize_prep notification https://review.openstack.org/465081 | |
| 09:29:03 | openstackgerrit | Gábor Antal proposed openstack/nova master: Transform instance.rebuild_scheduled notification https://review.openstack.org/473929 | |
| 09:29:37 | openstackgerrit | Gábor Antal proposed openstack/nova master: Transform instance.live_migration_rollback_dest notification https://review.openstack.org/480214 | |
| 09:30:18 | openstackgerrit | Gábor Antal proposed openstack/nova master: Transform instance.live_migration_force_complete notification https://review.openstack.org/480561 | |
| 09:31:20 | openstackgerrit | Gábor Antal proposed openstack/nova master: Transform HostAPI-set_enabled notifications https://review.openstack.org/482622 | |
| 09:32:09 | openstackgerrit | Gábor Antal proposed openstack/nova master: Transform libvirt.error notification https://review.openstack.org/484851 | |
| 09:34:38 | openstackgerrit | Elod Illes proposed openstack/nova master: Functional test: cold migrate to compute down https://review.openstack.org/496280 | |