| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-05-30 | |||
| 03:43:36 | gmann | jwnx: 2gb i doubt, but try with minimum services | |
| 05:05:54 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add policy description for os-networks https://review.openstack.org/465452 | |
| 06:43:12 | gmann | sfinucan: updated this, check now if looks fine- https://review.openstack.org/#/c/465452/ | |
| 07:13:13 | bhagyashris | Hi all, Can anyone please guide me how should I stress the disk, memory and cpu on one node? | |
| 08:13:47 | sfinucan | gmann: Comments left. A further change needed | |
| 08:17:38 | openstackgerrit | Sergey Nikitin proposed openstack/nova master: Standardization of VM diagnostics info API. https://review.openstack.org/355540 | |
| 08:21:32 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add policy description for os-networks https://review.openstack.org/465452 | |
| 08:21:45 | gmann | sfinucan: thanks , done^^ | |
| 08:23:00 | sfinucan | gmann: That's going to complain about >79 char wrapping :) | |
| 08:23:13 | sfinucan | other than that, lgtm | |
| 08:23:56 | gmann | sfinucan: oh, :) ll update. thanks | |
| 08:24:51 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add policy description for os-networks https://review.openstack.org/465452 | |
| 09:14:46 | sfinucan | gmann: Fancy taking ownership of this patch -> https://review.openstack.org/#/c/445504/ ? | |
| 09:15:02 | sfinucan | There haven't been any changes since end of March and I can't review if I also author :) | |
| 09:20:14 | openstackgerrit | Stephen Finucane proposed openstack/nova master: remove mox from unit/virt/vmwareapi/test_driver_api.py https://review.openstack.org/452128 | |
| 10:01:42 | openstackgerrit | Sylvain Bauza proposed openstack/nova master: _schedule_instances() supporting a RequestSpec object https://review.openstack.org/469037 | |
| 10:02:52 | openstackgerrit | Gábor Antal proposed openstack/nova master: Transform compute.api_fault notification https://review.openstack.org/469038 | |
| 10:26:28 | openstackgerrit | Bob Ball proposed openstack/nova master: XenAPI: nova-compute cannot restart after manually delete VM https://review.openstack.org/467926 | |
| 10:48:47 | openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Update placement devref to modern features https://review.openstack.org/469047 | |
| 10:48:48 | openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Update the placement deployment instructions https://review.openstack.org/469048 | |
| 11:11:35 | openstackgerrit | Gábor Antal proposed openstack/nova master: Transform instance.resize_prep notification https://review.openstack.org/465081 | |
| 11:23:54 | egonzalez | Hi, someone have ever seen this error on nova-api, regarding metadata service? http://paste.openstack.org/show/610953/ | |
| 11:25:32 | egonzalez | seems is not able to retrieve info from memcached or db, both services are running fine, can be a race condition with some recent change? | |
| 11:57:16 | openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova master: Do not rely on dogpile internals for mocks https://review.openstack.org/468743 | |
| 11:58:00 | openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova master: Do not rely on dogpile internals for mocks https://review.openstack.org/468743 | |
| 12:02:44 | openstackgerrit | Gábor Antal proposed openstack/nova master: Transform compute.api_fault notification https://review.openstack.org/469038 | |
| 12:26:13 | bauzas | edleafe: morning | |
| 12:28:19 | edleafe | bauzas: afternoon! | |
| 12:28:46 | bauzas | edleafe: FWIW, I'm providing on a new PS for https://review.openstack.org/#/c/465171/ | |
| 12:28:52 | bauzas | given my comments | |
| 12:29:10 | bauzas | of course, if people dislike it, then we could provide a new PS | |
| 12:31:00 | edleafe | bauzas: sure, go for it. I think that superdan might have an opinion on that | |
| 12:32:17 | bauzas | edleafe: also, do you agree if you could use my change https://review.openstack.org/#/c/465175/ instead of https://review.openstack.org/#/c/465767/ ? | |
| 12:32:23 | bauzas | both sound duplicate | |
| 12:32:28 | edleafe | Another option would be to pass a single req spec, a list of instance_uuids, and drop the num_instances, since it would then be redundant | |
| 12:32:51 | bauzas | edleafe: I agree with deprecating num_instances | |
| 12:33:17 | bauzas | edleafe: I'm providing that revision for discussing about the interface | |
| 12:33:38 | bauzas | tbh, I'd prefer to provide a list of specs, since each of them is related to only one instance | |
| 12:33:40 | edleafe | bauzas: I disagree with using CONF.scheduler.max_attempts | |
| 12:34:05 | bauzas | edleafe: okay, so could you please then comment on my change and provide your own rev if you want so, so we could Co-Author ? | |
| 12:34:21 | bauzas | having duplicates could confuse people | |
| 12:34:57 | edleafe | operators may want more potential hosts returned than attempts, since on a retry there is a chance that another VM has used up a host in the interim | |
| 12:39:50 | openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Support tag instances when boot(2/4) https://review.openstack.org/459569 | |
| 12:48:15 | ildikov | sfinucan: hi :) | |
| 12:49:00 | sfinucan | ildikov: Hey - what's up? | |
| 12:49:16 | ildikov | sfinucan: if you happen to have some of the usually non-existent free time, I would like to ask you to take a look at these two tiny reviews plz: https://review.openstack.org/#/c/459111/ https://review.openstack.org/#/c/459113/ :) | |
| 13:03:02 | openstackgerrit | Jeremy Liu proposed openstack/python-novaclient master: Fix reservation_id not supported by Nova API https://review.openstack.org/468444 | |
| 13:09:32 | openstackgerrit | Béla Vancsics proposed openstack/nova master: Transform keypair.import notification https://review.openstack.org/467514 | |
| 13:09:33 | openstackgerrit | Béla Vancsics proposed openstack/nova master: Transform keypair.create notification https://review.openstack.org/408689 | |
| 13:20:24 | peter-hamilton | hi all, anyone willing to give these patches a review? https://review.openstack.org/#/c/357202/ and https://review.openstack.org/#/c/457678/ | |
| 13:21:21 | peter-hamilton | i'm hoping to rekindle interest after a couple of quiet weeks | |
| 13:21:48 | peter-hamilton | if you have any questions on the certificate validation process just let me know | |
| 13:26:46 | ildikov | sfinucan: tnx :) | |
| 13:27:07 | sfinucan | ildikov: No problemo. Mind taking a look at the comments in the first to see if they had any merit? | |
| 13:27:25 | sfinucan | (I didn't consider them important enough to hold up the patches, but would still like answers, heh) | |
| 13:28:33 | sfinucan | peter-hamilton: I had a look, but tbh I don't know nearly enough about the topic to review it properly | |
| 13:28:43 | sfinucan | maybe ask again tomorrow once all the US folks are back online? | |
| 13:29:18 | bauzas | AFAIK, Memorial Day was only yesterday | |
| 13:29:26 | bauzas | edleafe: do you confirm ? ^ | |
| 13:29:55 | bauzas | should we understand that US people are in holidays ? | |
| 13:30:56 | ildikov | sfinucan: I answered your question about the exception with attachment_create, so I think we're on the same with that | |
| 13:31:41 | ildikov | sfinucan: I can add a comment to the simple create, although we don't plan to touch that code maybe someone gets interested to rethink that one | |
| 13:31:52 | sfinucan | peter-hamilton: Actually, I take that back. The second one looks good to me, with the exception of the incorrect version. Fix that and I can +2/+W | |
| 13:32:11 | edleafe | bauzas: Yes, just Monday. But hangovers can last longer. :) | |
| 13:32:29 | bauzas | heh, like French folks :p | |
| 13:33:31 | sfinucan | ildikov: Perfect, thanks. The create case is a nit so we don't need to bother about. Once I know there's something there :) | |
| 13:34:56 | openstackgerrit | Béla Vancsics proposed openstack/nova master: Transform aggregate.update_metadata notification https://review.openstack.org/460625 | |
| 13:36:58 | peter-hamilton | sfinucan: thanks, i'll push out an update in a bit | |
| 13:38:51 | openstackgerrit | Merged openstack/nova master: Add `img_hide_hypervisor_id` image property https://review.openstack.org/459753 | |
| 13:39:49 | mriedem | bauzas: https://review.openstack.org/#/c/468549/ and the one after it are cells v2 regressions and need to go back to newton, so would appreciate a review on those | |
| 13:39:54 | mriedem | first is the recreate, 2nd is the fix | |
| 13:40:41 | bauzas | mriedem: will look very shortly, just amending a change | |
| 13:41:07 | openstackgerrit | Peter Hamilton proposed openstack/nova master: Add configuration options for certificate validation https://review.openstack.org/457678 | |
| 13:41:24 | danpawlik | mriedem: \o/ | |
| 13:41:28 | danpawlik | thanks | |
| 13:42:25 | mriedem | danpawlik: yw | |
| 13:43:20 | openstackgerrit | OpenStack Proposal Bot proposed openstack/python-novaclient master: Updated from global requirements https://review.openstack.org/468494 | |
| 13:43:31 | openstackgerrit | Sylvain Bauza proposed openstack/nova master: Pass a list of instance UUIDs to scheduler https://review.openstack.org/465171 | |
| 13:43:32 | openstackgerrit | Sylvain Bauza proposed openstack/nova master: _schedule_instances() supporting a RequestSpec object https://review.openstack.org/469037 | |
| 13:43:44 | bauzas | edleafe: there it is ^ | |
| 13:44:13 | bauzas | edleafe: I just did cut your change in twice because it was becoming a big big | |
| 13:44:16 | bauzas | bit big | |
| 13:44:25 | danpawlik | Does someone have problem with Windows server 2012/2016 instance when is suspended and after resume Windows is freezed ? | |
| 13:46:03 | bauzas | mriedem: I had a question for https://review.openstack.org/#/c/466088/2 | |
| 13:46:21 | bauzas | mriedem: do you think we should add a functional regression test for that, given it's only for a driver ? | |
| 13:46:30 | bauzas | my guts are still yes tho | |
| 13:49:04 | mriedem | bauzas: functional regression test for what specifically? | |
| 13:49:10 | mriedem | we use the fake virt driver in our in-tree functional tests | |
| 13:49:54 | bauzas | mriedem: for the fact that a returning evacuation doesn't work for libvirt because of permissions | |
| 13:50:27 | mriedem | bauzas: we don't have libvirt in our functional test tree | |
| 13:50:29 | bauzas | and you're right, I knew for the fake driver, but I do wonder if we would need to verify some other drivers | |
| 13:50:36 | bauzas | okay, then... | |
| 13:51:08 | mriedem | we've talked about having an 'integrated' set of tests with a live devstack or something in the backend for more low-level testing, or things tempest can't test, like evacuate | |
| 13:51:21 | mriedem | but until someone makes that happen it's just talk | |
| 13:51:30 | bauzas | mriedem: I was wondering if we were having something like a LibvirtFixture :p | |
| 13:51:50 | bauzas | okay, then nevermind, I thought it was already there | |
| 13:52:33 | mriedem | there is already the FakeLibvirt fixture yes | |
| 13:52:46 | mriedem | you could use that in some functional tests i guess | |
| 13:53:01 | andymccr | morning nova - since https://review.openstack.org/#/c/458635/ & https://review.openstack.org/#/c/458634/ - we're seeing gate failures in OSA due to host not being mapped to a cell - if you try the boot a short while later it works. I'm wondering if there is a step we are maybe missing, or something that might've changed? | |
| 13:54:14 | mriedem | andymccr: when are you running discover_hosts? | |