Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-12-20
20:41:20 mriedem my plan is to further split the large patch into compute / conductor / api parts, and then i'll start thinking about how to write new stuff for the compute layer since the conductor/api stuff might remain mostly unchanged
20:41:45 mriedem melwitt: are you running the nova meeting today or am i?
21:22:28 mriedem jackding: should really have kashyap look at this https://review.openstack.org/#/c/620959/
21:22:43 mriedem but he's probably on PTO
21:40:47 mriedem jackding: cfriesen: question inline about the performance impact of the cpu_models spec https://review.openstack.org/#/c/620959/
22:20:50 cfriesen mriedem: responded in the review. thanks for the +W
22:22:48 openstackgerrit Merged openstack/nova-specs master: Select cpu model from a list of cpu models https://review.openstack.org/620959
22:48:27 mriedem jaypipes: i've got a head scratcher in a functional test i wrote https://review.openstack.org/#/c/619123/ - i have a single host which is reporting 2 VCPU to inventory. i create a server on it with 1 VCPU and then resize to a flavor with 2 VCPU to trigger no valid host due to insufficient capacity,
22:48:51 mriedem however, i'm sort of wondering why the cpu_allocation_ratio (16.0) isn't being taken into account, to really say the VCPU inventory is 32
22:49:06 jaypipes mriedem: hmm.
22:49:07 mriedem i don't see anything setting global allocation ratios in our fake virt driver or test fixtures
22:49:19 mriedem the test fails as i wanted, but now i'm sort of wondering why
22:49:22 jaypipes mriedem: what gets written to compute_nodes table?
22:49:35 mriedem i'll add some debug
22:49:40 jaypipes compute_nodes.cpu_allocation_ratio that is.
22:50:02 mriedem debug shows this as well
22:50:03 mriedem Updating ProviderTree inventory for provider c79d1a42-b934-43c3-9e5d-fed394e98d46 from _refresh_and_get_inventory using data: {'VCPU': {'max_unit': 2, 'min_unit': 1, 'step_size': 1, 'reserved': 0, 'total': 2, 'allocation_ratio': 16.0}, 'DISK_GB': {'max_unit': 1028, 'min_unit': 1, 'step_size': 1, 'reserved': 0, 'total': 1028, 'allocation_ratio': 1.0}, 'MEMORY_MB': {'max_unit': 8192, 'min_unit': 1, 'step_size': 1, 'reserved': 5
22:50:03 mriedem 'total': 8192, 'allocation_ratio': 1.5}}"
22:53:37 jaypipes mriedem: max_unit is 2.
22:53:41 jaypipes mriedem: that's the reason.
22:54:04 mriedem ah, yeah
22:54:18 mriedem fake virt driver implements upt and it reports max_unit = total
22:54:56 mriedem hmm, but i'm requesting 2 when it fails
22:55:21 mriedem and the resource provider inventory capacity in placement is: int((self.total - self.reserved) * self.allocation_ratio)
22:57:44 mriedem http://paste.openstack.org/show/737810/ is the placement GET /a_c debug logs
22:57:54 mriedem ha it's disk that is the problem
22:57:55 mriedem found 0 providers with available 1028 DISK_GB'
22:58:03 mriedem b/c disk_allocation_ratio=1
22:58:10 mriedem saved by the disk!
22:58:47 jaypipes tada
23:09:38 openstackgerrit Matt Riedemann proposed openstack/nova master: Add functional regression recreate test for bug 1790204 https://review.openstack.org/619123
23:09:38 openstack bug 1790204 in OpenStack Compute (nova) "Allocations are "doubled up" on same host resize even though there is only 1 server on the host" [Medium,Triaged] https://launchpad.net/bugs/1790204
#openstack-nova - 2018-12-21
00:15:28 openstackgerrit Hongbin Lu proposed openstack/nova-specs master: [WIP] Support scheduling VM's NICs to different PFs https://review.openstack.org/626055
01:05:36 openstackgerrit Takashi NATSUME proposed openstack/nova stable/rocky: Add API ref guideline for examples https://review.openstack.org/626776
01:10:46 openstackgerrit Takashi NATSUME proposed openstack/nova stable/rocky: Add API ref guideline for body text https://review.openstack.org/626778
01:11:29 openstackgerrit Takashi NATSUME proposed openstack/nova stable/rocky: Add a link to the doc contrib guide https://review.openstack.org/626779
01:33:39 openstackgerrit Merged openstack/os-traits master: Update author-email in setup.cfg https://review.openstack.org/621973
02:01:48 openstackgerrit Merged openstack/nova master: Fix a broken link https://review.openstack.org/625481
02:25:12 yonglihe mriedem: POC already released, of course i gonna explain how the info is retrieved in spec too. POC code: https://review.openstack.org/#/c/621476/3
02:29:35 openstackgerrit Takashi NATSUME proposed openstack/nova stable/rocky: Fix a broken link https://review.openstack.org/626798
02:51:42 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Add support matrix for Delete (Abort) on-going live migration https://review.openstack.org/625781
03:21:13 openstackgerrit Yongli He proposed openstack/nova-specs master: add 'show-server-group' spec https://review.openstack.org/612255
05:08:03 openstackgerrit Yongli He proposed openstack/nova-specs master: add spec "show-server-numa-topology" https://review.openstack.org/612256
05:30:38 openstackgerrit Takashi NATSUME proposed openstack/nova master: Add links to summit videos in user/cells.rst https://review.openstack.org/626809
07:19:53 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Allow run metadata api per cell https://review.openstack.org/624612
08:12:00 openstackgerrit Yongli He proposed openstack/nova-specs master: add spec "show-server-numa-topology" https://review.openstack.org/612256
08:18:29 openstackgerrit Maciej Jozefczyk proposed openstack/nova master: Force refresh instance info_cache during heal https://review.openstack.org/591607
08:28:11 openstackgerrit Maciej Jozefczyk proposed openstack/nova master: Force refresh instance info_cache during heal https://review.openstack.org/591607
08:28:11 openstackgerrit Maciej Jozefczyk proposed openstack/nova master: Add fill_virtual_interface_list online_data_migration script https://review.openstack.org/614167
08:35:17 openstackgerrit int32bit proposed openstack/nova master: Convert port to str when validate console port https://review.openstack.org/626853
08:40:47 openstackgerrit int32bit proposed openstack/nova master: Convert port to str when validate console port https://review.openstack.org/626853
08:51:23 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Don't call begin_detaching when detaching volume from sheved vm https://review.openstack.org/624959
09:16:26 maciejjozefczyk hey!
09:17:15 maciejjozefczyk I'm thinking what kind of test I need here: https://review.openstack.org/#/c/614167/ for online_data_migration script. Any clue?
09:21:14 maciejjozefczyk functional test similar to this will be ok? https://review.openstack.org/#/c/302828/12/nova/tests/functional/db/test_request_spec.py
10:41:47 openstackgerrit Stephen Finucane proposed openstack/os-traits master: Change openstack-dev to openstack-discuss https://review.openstack.org/623910
10:52:19 openstackgerrit Stephen Finucane proposed openstack/nova stable/rocky: Fix destination_type attribute in the bdm_v2 documentation https://review.openstack.org/626874
10:52:54 openstackgerrit Stephen Finucane proposed openstack/nova stable/queens: Fix destination_type attribute in the bdm_v2 documentation https://review.openstack.org/626875
10:53:05 openstackgerrit Stephen Finucane proposed openstack/nova stable/pike: Fix destination_type attribute in the bdm_v2 documentation https://review.openstack.org/626876
11:52:07 openstackgerrit Merged openstack/nova master: Add irrelevant-files for grenade-py3 jobs https://review.openstack.org/624997
11:52:13 openstackgerrit Merged openstack/nova master: nit: Add space to feature support docs https://review.openstack.org/626523
11:52:25 openstackgerrit Merged openstack/nova master: Fix destination_type attribute in the bdm_v2 documentation https://review.openstack.org/625020
12:03:39 ShilpaSD stephenfin: stack is failing with error '[ERROR] /home/<NAME>/devstack/lib/placement:179 placement-api did not start', any solution?
12:33:54 mdbooth bauzas: Hey, I just spotted something weird about weighers (in Queens), and I'm wondering if it's known and/or deliberate: minval and maxval for a weigher seem to be global for all requests and hosts weighed by that weigher
12:35:32 mdbooth bauzas: If I haven't misread that, it would mean, for eg, that if any host ever suffered a long succession of build failures, then the BuildFailureWeigher of all subsequent scheduling requests would be normalized against that very negative value.
12:36:43 mdbooth So if 1 host suffers 100 build failures one time, then forever more a single build failure is normalized to 0.99 rather than 0
12:37:34 mdbooth That seems weird. Not sure what the side-effects would be.
12:37:51 mdbooth Also: it would be really nice if weight values were logged :/
13:07:13 mdbooth I'm convinced there's something weird going on with our request-id generation
13:07:20 mdbooth I think we're re-using them
13:48:19 openstackgerrit Corey Bryant proposed openstack/nova master: Ensure rbd auth fallback uses matching credentials https://review.openstack.org/626897
13:51:16 stephenfin ShilpaSD: Are you restacking on a machine where you previously had stacked?
13:51:38 ShilpaSD stephenfin: yes
13:52:57 stephenfin ShilpaSD: Remove anything related to OpenStack in /etc/apache2/sites-(available|enabled)/ and try again
13:53:21 stephenfin ShilpaSD: They changed how placement is deployed so that's all you really need to remove, but I think I saw issues with other services
13:53:47 stephenfin ShilpaSD: If that doesn't work, the easiest thing to do is reinstall the OS. DevStack doesn't properly clean up after itself so upgrades aren't really a thing
13:54:07 stephenfin which it is, if you're restacking with newer code
13:54:49 ShilpaSD stephenfin: ok, the last solution i started, once this done, will check cleanup of /etc/apache2/sites and get back to you
13:55:06 ShilpaSD stephenfin: thanks for solution
14:28:18 ShilpaSD stephenfin: great, stack is successfully created
14:28:24 ShilpaSD thank you
15:21:19 openstackgerrit Stephen Finucane proposed openstack/nova master: doc: Switch header styles in support doc https://review.openstack.org/626930
15:21:19 openstackgerrit Stephen Finucane proposed openstack/nova master: doc: Add solution to live migration ssh issues https://review.openstack.org/626931
15:21:20 openstackgerrit Stephen Finucane proposed openstack/nova master: conf: Remove 'disable_libvirt_livesnapshot' option https://review.openstack.org/626932
15:54:39 mriedem melwitt: huzzah https://review.openstack.org/#/c/509042/
16:00:15 mriedem comments on the counting quotas from placement spec - mostly just looking to flesh out some details, and optimize the data migration
16:13:58 stephenfin Whoops
16:21:16 coreycb mriedem: does this look ok to you? really small fix. https://review.openstack.org/#/c/626897
16:32:24 mriedem coreycb: comments inline
16:34:07 mriedem looks like my 'remove all this shit in pike' didn't happen
16:34:38 mriedem heh or queens
16:49:04 finucannot cdent isn't around but I've now answered ShilpaD's "why did placement-api not start when stacking question" for another person. That's three (including me). DevStack should probably just delete that old Apache file
16:49:08 finucannot maybe after Xmas
17:04:09 coreycb mriedem: thanks. i responded but wasn't fully understanding the one comment.
17:04:18 coreycb mriedem: haha yeah
17:14:32 mriedem coreycb: ok i'll check after lunch
17:14:52 coreycb thanks mriedem_afk
17:16:34 openstackgerrit Lee Yarwood proposed openstack/nova master: DNM WIP Workaround known eventlet monkey_patch issue https://review.openstack.org/626952
17:59:21 openstackgerrit Jack Ding proposed openstack/nova master: Preserve UEFI NVRAM variable store https://review.openstack.org/621646
19:03:28 openstackgerrit Merged openstack/nova master: Fix best_match() deprecation warning https://review.openstack.org/611204

Earlier   Later