Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-03-31
19:59:22 imacdonn wondering if it has/had any other purpose
20:00:02 fried_bunny https://review.openstack.org/#/c/557086/ -- I thought that was touching that test case, but no.
20:00:33 imacdonn yeah, I was also surprised that https://review.openstack.org/#/c/554759/ didn't at least touch it
20:01:44 imacdonn so Im going to nuke it, on the assumption that its only purpose was to ensure the validity of the following test
20:04:54 fried_bunny probably just overlooked tbh
20:05:05 imacdonn yeah
20:05:36 fried_bunny on that note, you can look around and remove all the places we set placement.[os_]region_name now.
20:05:43 fried_bunny ...in the test suites.
20:07:55 imacdonn not actually seeing any (others)
20:08:21 imacdonn well, there's this... nova/tests/unit/conf_fixture.py: self.conf.set_default('region_name', 'RegionOne', group='placement')
20:13:09 fried_bunny yup, don't need that anymore.
20:14:41 imacdonn agreed ... removed it already, and running all tests again
20:15:15 imacdonn d'you think we need comments at all the places where we made the placement fixtures, explaining why ?
20:15:52 fried_bunny It sure wouldn't hurt.
20:19:19 imacdonn right ... all tests are passing now ... I think I'll grab lunch, then work up some comments and a commit message
20:19:39 fried_bunny nice
20:20:00 imacdonn thanks very much for your help and patience!
20:22:23 fried_bunny imacdonn: Thank you very much for taking this on.
20:22:30 imacdonn np!
20:23:03 fried_bunny ooo, your first nova patch?
20:27:48 imacdonn probably, yeah
20:28:12 imacdonn have done a few cinder/os-brick ones
20:38:27 openstackgerrit Eric Berglund proposed openstack/nova master: PowerVM Driver: Snapshot https://review.openstack.org/543023
21:02:14 openstackgerrit Eric Berglund proposed openstack/nova master: PowerVM Driver: DiskAdapter parent class https://review.openstack.org/549053
21:03:19 openstackgerrit Eric Fried proposed openstack/nova master: Fix unit tests to work with new oslo.config https://review.openstack.org/558084
21:03:38 fried_bunny dhellmann ^
21:08:57 openstackgerrit Eric Berglund proposed openstack/nova master: PowerVM Driver: DiskAdapter parent class https://review.openstack.org/549053
21:09:25 openstackgerrit Eric Berglund proposed openstack/nova master: PowerVM Driver: Localdisk https://review.openstack.org/549300
21:20:22 openstackgerrit Eric Fried proposed openstack/nova master: Test alloc_cands with indirectly sharing RPs https://review.openstack.org/519601
21:20:22 openstackgerrit Eric Fried proposed openstack/nova master: Add unrequested resources to provider_summaries https://review.openstack.org/558045
21:20:23 openstackgerrit Eric Fried proposed openstack/nova master: Return anchor providers in a_r and p_s https://review.openstack.org/558014
21:20:23 openstackgerrit Eric Fried proposed openstack/nova master: Support relay RP for allocation candidates https://review.openstack.org/533437
21:28:04 fried_bunny imacdonn: Aha, I knew that sounded familiar: https://review.openstack.org/#/c/554759/1/nova/tests/unit/compute/test_compute_mgr.py@3708
21:33:36 openstackgerrit Eric Fried proposed openstack/nova master: Remove deprecated [placement] opts https://review.openstack.org/557086
21:42:06 imacdonn frickler: ahh, so I guess you decided to leave it alone that time ;)
21:42:21 imacdonn oops that was meant for fried_bunny ... grr, bad tab-completion
21:44:07 imacdonn zoiks ... nova could use some pep8 love ;)
21:59:34 fried_bunny imacdonn: Yeah, well, I was taking over someone else's patch, and kind of in a hurry, didn't want to get too involved.
21:59:38 fried_bunny What about pep8?
22:00:06 imacdonn I thought I'd have to pass pep8 with my change, but when I run it, I barfs on a ton of other stuff
22:00:38 fried_bunny How are you running it?
22:00:47 imacdonn just pep8 on the files I touched
22:00:55 imacdonn Trying to do it under tox instead now
22:02:36 openstackgerrit Eric Berglund proposed openstack/nova master: PowerVM Driver: Localdisk https://review.openstack.org/549300
22:04:16 fried_bunny {
22:04:16 fried_bunny pep ()
22:04:16 fried_bunny pep is a function
22:04:16 fried_bunny efried@efried-ThinkPad-W520:~/Neo/nova$ type pep
22:04:16 fried_bunny imacdonn: FYI, I use this:
22:04:17 fried_bunny num=${1:-1};
22:04:17 fried_bunny ( . .tox/pep8/bin/activate || return;
22:04:18 fried_bunny git diff --name-only HEAD~$num | grep '\.py$' | xargs flake8 )
22:04:18 fried_bunny echo "Running flake8 on files changed in the last $num commit(s)";
22:04:19 fried_bunny }
22:04:46 imacdonn ah, neat .. I'll steal that - thanks ;)
22:06:51 imacdonn ./nova/tests/unit/compute/test_compute_mgr.py:21:1: H306 imports not in alphabetical order (keystoneauth1.exceptions, cinderclient.exceptions)
22:06:54 imacdonn wow .. pickyyyyy
22:18:11 openstackgerrit Eric Berglund proposed openstack/nova master: WIP: Resize https://review.openstack.org/553583
22:34:01 openstackgerrit iain MacDonnell proposed openstack/nova master: Update check to ensure compute is using placement https://review.openstack.org/558089
22:52:55 openstackgerrit Eric Fried proposed openstack/nova master: update_provider_tree devref and docstring updates https://review.openstack.org/553476
23:02:52 openstackgerrit Eric Fried proposed openstack/nova master: Remove deprecated [placement] opts https://review.openstack.org/557086
23:12:31 fried_bunny Well, that's quite enough for today.
#openstack-nova - 2018-04-01
02:21:50 openstackgerrit Eric Berglund proposed openstack/nova master: WIP: PowerVM: Cold Migrate & Resize https://review.openstack.org/553583
06:28:26 srf_ hello i want to ask when i install devstack get some error, and i tired to find the error http://paste.openstack.org/show/718100/
07:04:21 openstackgerrit OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.openstack.org/548772
10:57:31 srf_ hello i want to ask when i install devstack get some error, and i tired to find the error http://paste.openstack.org/show/718100/
14:00:05 openstackgerrit Matt Riedemann proposed openstack/nova master: Fix cancel_all_events event name parsing https://review.openstack.org/558059
14:10:53 openstackgerrit Chris Dent proposed openstack/nova master: [placement] Filter resource providers by forbidden traits in db https://review.openstack.org/556472
14:10:54 openstackgerrit Chris Dent proposed openstack/nova master: [placement] Filter allocation candidates by forbidden traits in db https://review.openstack.org/556660
14:10:55 openstackgerrit Chris Dent proposed openstack/nova master: [placement] Support forbidden traits in API https://review.openstack.org/556820
14:10:55 openstackgerrit Chris Dent proposed openstack/nova master: [placement] Parse forbidden traits in query strings https://review.openstack.org/556819
14:28:57 srf_ hello i want to ask when i install devstack get some error, and i tired to find the error http://paste.openstack.org/show/718100/
14:50:36 openstackgerrit Merged openstack/nova-specs master: Mention (no) granular support for image traits https://review.openstack.org/554305
21:29:40 openstackgerrit Maciej Kucia proposed openstack/nova-specs master: Fix Non-unique network names in Servers IPs API response https://review.openstack.org/558125
22:08:36 openstackgerrit Maciej Kucia proposed openstack/nova-specs master: Commonalize and simplify IPs API response https://review.openstack.org/558129
#openstack-nova - 2018-04-02
00:40:21 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: [placement] Add test for provider summaries https://review.openstack.org/558044
00:40:22 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: Test alloc_cands with indirectly sharing RPs https://review.openstack.org/519601
00:40:22 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: Add unrequested resources to provider_summaries https://review.openstack.org/558045
00:40:23 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: Return anchor providers in a_r and p_s https://review.openstack.org/558014
00:40:23 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: Support relay RP for allocation candidates https://review.openstack.org/533437
00:40:24 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: Fix nits in return-anchor-providers series https://review.openstack.org/558133
00:43:02 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: Complement tests in allocation candidates https://review.openstack.org/558135
00:50:10 tetsuro good morning!
01:06:39 fried_bunny tetsuro: Hi there.
01:06:55 fried_bunny I won't get a chance to look at the series again until my morning (~12h from now).
01:07:12 fried_bunny Any major disruptions I should know about?
01:09:54 fried_bunny Oh, I see, you did the microversion paperwork, cool. I'll look in the morning. TTFN!
01:43:09 openstackgerrit Naichuan Sun proposed openstack/nova master: xenapi(N-R-P):Get vgpu info from `allocations` https://review.openstack.org/521717
01:52:41 openstackgerrit Zhenyu Zheng proposed openstack/nova master: [WIP] Send out notification when instance metadata updated https://review.openstack.org/558140
02:19:01 openstackgerrit Takashi NATSUME proposed openstack/nova master: [placement] api-ref: Fix parameters https://review.openstack.org/558142
02:42:48 openstackgerrit Naichuan Sun proposed openstack/nova master: xenapi(N-R-P):Get vgpu info from `allocations` https://review.openstack.org/521717
02:47:38 tetsuro @fired_bunny: Oh, I realized your message just now. Thanks in advance! I’m going to look into your patch on the top of the branch after lunch. Anyway, Good night!
03:47:07 openstackgerrit Naichuan Sun proposed openstack/nova master: xenapi: Use XAPI pool instead of aggregate pool for shared SR migration https://review.openstack.org/554154
05:47:03 openstackgerrit Maciej Kucia proposed openstack/nova-specs master: Fix Non-unique network names in Servers IPs API response https://review.openstack.org/558125
05:50:15 openstackgerrit Takashi NATSUME proposed openstack/nova master: Fix the request context in ServiceFixture https://review.openstack.org/558160
06:15:00 openstackgerrit Nguyen Hai proposed openstack/nova-specs master: Enhance nova-specs webpage and clean up repo https://review.openstack.org/551802
06:18:26 openstackgerrit Naichuan Sun proposed openstack/nova master: xenapi: Support live migration in pooled multi-nodes environment https://review.openstack.org/489451
06:20:05 openstackgerrit Naichuan Sun proposed openstack/nova master: xenapi: Use XAPI pool instead of aggregate pool for shared SR migration https://review.openstack.org/554154
06:35:55 yuqian #openstack-ceilometer
07:19:32 openstackgerrit Chen Hanxiao proposed openstack/nova master: libvirt: guest: introduce blockStats instead of domain.blockStats https://review.openstack.org/526833

Earlier   Later