Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-07-19
23:25:09 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (6) https://review.openstack.org/574113
23:43:11 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (7) https://review.openstack.org/574974
23:43:14 oomichi takashin: hi, can you check https://review.openstack.org/#/c/578040/ again ?
23:43:25 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (8) https://review.openstack.org/575311
23:44:07 takashin oomichi: okay. I will review it again.
23:44:22 oomichi thanks :)
23:44:59 openstackgerrit Matt Riedemann proposed openstack/nova master: Use consumer generation in _heal_allocations_for_instance https://review.openstack.org/577905
#openstack-nova - 2018-07-20
00:46:10 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (9) https://review.openstack.org/575581
00:46:21 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (10) https://review.openstack.org/576017
00:46:37 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (11) https://review.openstack.org/576018
01:05:49 oomichi takashin: Can you check my comment on https://review.openstack.org/#/c/571992 ?
01:06:07 takashin oomichi: okay
01:07:23 takashin oomichi: Thank you for your review. I will fix them.
01:11:52 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Add regression test for bug 1781880 https://review.openstack.org/583927
01:11:53 openstack bug 1781880 in OpenStack Compute (nova) "nova service-list for new compute service causes TypeError with servicegroup_driver mc " [Medium,In progress] https://launchpad.net/bugs/1781880 - Assigned to Zhenyu Zheng (zhengzhenyu)
01:12:30 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Fix nova service-list for new compute service causes TypeError https://review.openstack.org/583936
01:12:56 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Fix nova service-list for new compute service causes TypeError https://review.openstack.org/583936
01:21:04 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Remove stevedore extensions server_create method https://review.openstack.org/583882
01:27:56 gmann mriedem_away: we have now tempest-scenario-all job which will cover all scenario tests including slow. can i update this one to use that job - https://review.openstack.org/#/c/567697 ?
01:34:29 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Use Tempest scenario job to run all scenario tests including slow https://review.openstack.org/567697
01:51:08 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (12) https://review.openstack.org/576019
01:51:28 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (13) https://review.openstack.org/576020
01:51:46 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (14) https://review.openstack.org/576027
02:08:18 openstackgerrit melanie witt proposed openstack/nova master: DNM: Don't overwrite greenthread-local context in target_cell https://review.openstack.org/582332
02:09:16 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Merge extended availability zone controller https://review.openstack.org/584197
02:13:31 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Merge extended availability zone controller https://review.openstack.org/502859
02:16:54 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Merge extended availability zone response into server controller https://review.openstack.org/502859
02:19:00 openstackgerrit fupingxie proposed openstack/nova master: Delete allocations when it is re-allocated https://review.openstack.org/582899
02:42:00 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Report 0 root_gb in resource tracker if instance is bfv. https://review.openstack.org/584204
02:55:43 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (15) https://review.openstack.org/576031
02:56:04 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (16) https://review.openstack.org/576299
02:56:34 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (17) https://review.openstack.org/576344
03:35:57 openstackgerrit fupingxie proposed openstack/nova master: Trivial: Fix spelling error https://review.openstack.org/584212
04:14:48 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Consider forbidden traits in early exit of _get_by_one_request https://review.openstack.org/584218
04:55:44 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Merge config drive extension response into server controller https://review.openstack.org/584223
04:59:24 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (18) https://review.openstack.org/576673
04:59:34 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (19) https://review.openstack.org/576676
05:00:11 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (20) https://review.openstack.org/576689
05:10:58 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (21) https://review.openstack.org/576709
05:32:18 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Merge config drive extension response into server controller https://review.openstack.org/584223
05:48:51 openstackgerrit fupingxie proposed openstack/nova master: Delete allocations when it is re-allocated https://review.openstack.org/582899
06:44:29 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Update installation guide to be more clear about cellsv2 https://review.openstack.org/584244
07:21:15 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Consider forbidden traits in early exit of _get_by_one_request https://review.openstack.org/584218
08:19:21 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Report 0 root_gb in resource tracker if instance is bfv. https://review.openstack.org/584204
08:31:59 openstackgerrit zhufl proposed openstack/nova master: Fix missing print format error https://review.openstack.org/584277
08:39:17 openstackgerrit Lei Zhang proposed openstack/nova master: Add method to get cpu traits https://review.openstack.org/560317
08:49:40 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Add regression test for bug 1781880 https://review.openstack.org/583927
08:49:41 openstack bug 1781880 in OpenStack Compute (nova) "nova service-list for new compute service causes TypeError with servicegroup_driver mc " [Medium,In progress] https://launchpad.net/bugs/1781880 - Assigned to Zhenyu Zheng (zhengzhenyu)
09:20:20 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Update launch-from-volume doc to latest version. https://review.openstack.org/572241
09:28:00 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Add regression test for bug 1781880 https://review.openstack.org/583927
09:28:02 openstack bug 1781880 in OpenStack Compute (nova) "nova service-list for new compute service causes TypeError with servicegroup_driver mc " [Medium,In progress] https://launchpad.net/bugs/1781880 - Assigned to Zhenyu Zheng (zhengzhenyu)
09:35:59 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Add regression test for bug 1781880 https://review.openstack.org/583927
09:36:00 openstack bug 1781880 in OpenStack Compute (nova) "nova service-list for new compute service causes TypeError with servicegroup_driver mc " [Medium,In progress] https://launchpad.net/bugs/1781880 - Assigned to Zhenyu Zheng (zhengzhenyu)
09:52:49 openstackgerrit do3meli proposed openstack/nova master: docs: add nova host-evacuate command to evacuate documentation https://review.openstack.org/578040
10:40:04 Shilpa stephenfin: Hi
10:40:09 stephenfin Shilpa: o/
10:40:42 Shilpa stephenfin: on Nova side, for tox -e pep8, i am facing error 'ERROR: pep8: could not install deps [-r/opt/stack/nova/test-requirements.txt]; v = InvocationError('/opt/stack/nova/.tox/pep8/bin/pip install -chttps://git.openstack.org/cgit/openstack/requirements/plain/upper-constraints.txt -r/opt/stack/nova/test-requirements.txt (see /opt/stack/nova/.tox/pep8/log/pep8-1.log)', 1)
10:40:42 Shilpa '
10:41:10 Shilpa stephenfin: i have done 'sudo pip install tox', but no sucess
10:41:20 stephenfin Shilpa: Could you share the full output on paste.openstack.org?
10:42:25 Shilpa stephenfin:http://paste.openstack.org/show/726332/
10:43:09 stephenfin Shilpa: What distro are you using? You need to install python-dev(el)
10:43:33 stephenfin Shilpa: Sorry - python3-dev(el)
10:43:45 stephenfin Per this error message
10:43:45 stephenfin x86_64-linux-gnu-gcc -pthread -DNDEBUG -g -fwrapv -O2 -Wall -Wstrict-prototypes -g -fstack-protector-strong -Wformat -Werror=format-security -Wdate-time -D_FORTIFY_SOURCE=2 -fPIC -I/usr/include/python3.5m -I/opt/stack/nova/.tox/pep8/include/python3.5m -c ext/_yaml.c -o build/temp.linux-x86_64-3.5/ext/_yaml.o
10:43:45 stephenfin ext/_yaml.c:4:20: fatal error: Python.h: No such file or directory
10:44:49 Shilpa stephenfin:any solution to resolve this?
10:45:14 stephenfin Shilpa: Yeah, install that package. 'sudo apt install python-dev' or the likes
10:47:19 Shilpa stephenfin:thanks, will check and let you know in next 5-10 min
10:52:23 openstackgerrit Chris Dent proposed openstack/nova master: [placement] Add /reshaper handler for POST https://review.openstack.org/576927
11:09:18 Shilpa stephenfin: Thanks able to resolve the pepe8 issue, ' sudo apt-get install python3.5-dev'
11:34:07 openstackgerrit Stephen Finucane proposed openstack/nova master: libvirt: Start populating NUMACell.network_metadata field https://review.openstack.org/564441
11:34:08 openstackgerrit Stephen Finucane proposed openstack/nova master: Add functional tests for numa-aware-vswitches https://review.openstack.org/583288
11:34:09 openstackgerrit Stephen Finucane proposed openstack/nova master: scheduler: Start utilizing RequestSpec.network_metadata https://review.openstack.org/564452
11:34:10 openstackgerrit Stephen Finucane proposed openstack/nova master: Assorted cleanups from numa-aware-vswitches series https://review.openstack.org/582651
11:57:01 openstackgerrit Lei Zhang proposed openstack/nova master: Add method to get cpu traits https://review.openstack.org/560317
12:05:16 openstackgerrit Andrey Volkov proposed openstack/nova master: Docs: Add Placement to Nova system architecture https://review.openstack.org/584338
12:25:39 openstackgerrit Andrey Volkov proposed openstack/nova master: Docs: Add more colors to Nova system architecture https://review.openstack.org/584345
12:44:08 mriedem gibi: can you also check the updated regression test? https://review.openstack.org/#/c/583339/
12:59:25 kashyap mriedem: On that PowerISA thing, what Lei needs to provide is something like this to baselineCPU(): https://libvirt.org/git/?p=libvirt.git;a=blob_plain;f=tests/cputestdata/ppc64-baseline-same-model.xml
12:59:34 kashyap What he's providing right now is _guest_ CPU XML; hence the error.
12:59:51 kashyap I wrote a comment here with a precise example he can use: https://review.openstack.org/#/c/560317/27
13:00:46 mriedem gmann: question in https://review.openstack.org/#/c/583538/
13:00:57 mriedem gmann: and i'm fine if you want to take over the nova slow job stuff, i'm not working on it
13:01:41 mriedem alex_xu: maybe you can answer my question in https://review.openstack.org/#/c/583538/
13:02:05 mriedem but if we're dropping the extensions, i don't know why we also aren't moving into the view builder code to centralize the response body creation
13:07:42 openstackgerrit Corey Bryant proposed openstack/nova master: In Python3.7 async is a keyword [1] https://review.openstack.org/584365
13:17:27 mriedem kashyap: ok
13:20:30 mriedem gmann: alex_xu: i see several of these already merged https://review.openstack.org/#/q/topic:bp/api-extensions-merge-rocky+(status:open+OR+status:merged) but 2 things:
13:20:47 mriedem 1. rather than cram it all into that single create() method, it would be nice to instead continue the pattern we have in here https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/helpers.py#L55
13:21:07 mriedem 2. this series isn't moving the view builder code from the extensions into the actual server ViewBuilder, which is something we should do
13:21:43 mriedem maybe the goal for rocky is to just remove the server_create extension method and then in stein we refactor/cleanup the create() controller method to move things into https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/helpers.py#L55 and also move stuff into the view builder?
13:22:12 mriedem personally it's the view builder that bothers me the most, because i have to hunt all over the extensions to find what puts something into the server response body
13:22:45 mriedem and the view builder part is going to become very important for tssurya's change for handling a down cell
13:27:53 mriedem i see in https://review.openstack.org/#/c/583882/ it says "Remaining part in this BP is to merge the response extensions which is last part of this work." - is that the view builder part? if so, then i guess we can just merge what you have in rocky and do the view builder (and move stuff out of create() into helpers) in stein
13:37:07 openstackgerrit Andrey Volkov proposed openstack/nova master: Docs: Add more colors to Nova system architecture https://review.openstack.org/584345
14:03:59 dansmith finucannot: you see that the second patch in your series is failing on pep8 right?
14:04:16 finucannot superdan: I did not but I'll fix now
14:19:36 openstackgerrit Stephen Finucane proposed openstack/nova master: libvirt: Start populating NUMACell.network_metadata field https://review.openstack.org/564441
14:19:37 openstackgerrit Stephen Finucane proposed openstack/nova master: Add functional tests for numa-aware-vswitches https://review.openstack.org/583288

Earlier   Later