Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-05-07
15:42:02 kashyap (Not in France)
15:42:30 kashyap mriedem: BTW, thanks for the careful review here: https://review.openstack.org/#/c/565242/4/nova/virt/libvirt/driver.py@a4798
15:43:00 kashyap mriedem: Even I wrote "After second thoughts", I think hard-failing at Compute start up sounds cleaner
15:44:54 openstackgerrit Stephen Finucane proposed openstack/nova master: objects: Add NUMANetworkTopology object https://review.openstack.org/564439
15:44:55 openstackgerrit Stephen Finucane proposed openstack/nova master: conf: Add '[neutron] physnets' and related options https://review.openstack.org/564440
15:44:56 openstackgerrit Stephen Finucane proposed openstack/nova master: libvirt: Start populating NUMACell.networks field https://review.openstack.org/564441
15:44:57 openstackgerrit Stephen Finucane proposed openstack/nova master: objects: Add NetworkRequest.physnet, tunnelled https://review.openstack.org/564442
15:44:58 openstackgerrit Stephen Finucane proposed openstack/nova master: network: Unchain _get_phynet_info from _get_port_vnic_info https://review.openstack.org/564443
15:44:59 openstackgerrit Stephen Finucane proposed openstack/nova master: network: Add 'populate_networks' to network API https://review.openstack.org/564444
15:45:00 openstackgerrit Stephen Finucane proposed openstack/nova master: network: Add '_get_network_tunnel_status' https://review.openstack.org/564445
15:45:01 openstackgerrit Stephen Finucane proposed openstack/nova master: db: Add network_requests to instance_extra https://review.openstack.org/564446
15:45:02 openstackgerrit Stephen Finucane proposed openstack/nova master: objects: Add Instance.network_requests https://review.openstack.org/564447
15:45:03 openstackgerrit Stephen Finucane proposed openstack/nova master: hardware: Start accounting for network requests https://review.openstack.org/564448
15:45:04 openstackgerrit Stephen Finucane proposed openstack/nova master: rt: Start utilizing Instance.network_requests https://review.openstack.org/564449
15:45:05 openstackgerrit Stephen Finucane proposed openstack/nova master: objects: Add RequestSpec.requested_networks https://review.openstack.org/564450
15:45:06 openstackgerrit Stephen Finucane proposed openstack/nova master: api: Remove unnecessary default parameter https://review.openstack.org/564451
15:45:07 openstackgerrit Stephen Finucane proposed openstack/nova master: scheduler: Start utilizing network_requests https://review.openstack.org/564452
15:45:35 mriedem kashyap: time to get dansmith to arbitrate
15:46:05 kashyap mriedem: Hehe, thought so. FWIW, I checked with a "real Operator" (runs OpenStack in Fedora Infra)
15:46:20 kashyap mriedem: And he agrees with your suggestion
15:53:43 dansmith mriedem: kashyap: commented
15:53:47 kashyap Completely unrelated thing about English: mriedem once corrected on a commit message: "we told it was" --> "we said [...]". I can't explain _why_ but his correction of s/told/said indeed "sounds" correct.
15:53:59 kashyap dansmith: Thanks; /me wears prickle-safe suit and heads to Gerrit
15:55:15 kashyap Oh, nothing prickly there this time.
15:56:28 kashyap dansmith: Okido; we're all on the same page on where it should fail.
15:56:50 kashyap dansmith: But I'm not even sure if a deprecation cycle is required, when all along the feature was a misfeature
15:57:23 kashyap But I see what you mean, though: "something valid until that point suddenly disappearing is not nice"
15:57:41 dansmith kashyap: I don't think the deprecation people care about that, they care about not breaking configs without a warning
15:58:05 kashyap Yeah, reasonable point.
15:58:32 kashyap But the "deprecation people" might agree in this case?
16:04:07 dansmith kashyap: mriedem is all the "deprecation people" we need on this
16:04:35 openstackgerrit Stephen Finucane proposed openstack/nova master: Remove get_config_vhostuser https://review.openstack.org/565471
16:04:36 openstackgerrit Stephen Finucane proposed openstack/nova master: libvirt: MIN_LIBVIRT_VHOSTUSER_MQ https://review.openstack.org/565472
16:04:37 openstackgerrit Stephen Finucane proposed openstack/nova master: libvirt: Drop MIN_LIBVIRT_PF_WITH_NO_VFS_CAP_VERSION https://review.openstack.org/565517
16:08:58 mriedem kashyap: if we aren't going to do a deprecation cycle with a warning if people are using those deprecated options, then we can't just make the service fail to start,
16:09:08 mriedem so i was wrong about the hard-fail on startup
16:09:29 mriedem consider you're rolling from queens to rocky and nova-compute doesn't startup because of something you were using in config which never had a warning in queens
16:09:59 mriedem we could say, "release notes bra!" but that doesn't make people less pissed off
16:10:39 mriedem so you can (a) (1) deprecate in rocky and (2) hard-fail in stein or (b) just do what you have now - don't hard-fail on start but don't honor the options either
16:11:08 dansmith kashyap: exhibit a ^
16:18:15 melwitt fwiw, I agree with mriedem on that
16:20:48 finucannot whoops
16:21:14 efried thanks for all the reviews stephenfin
16:21:36 stephenfin efried: No problem. I've been lagging there the last few weeks
16:35:14 openstackgerrit Merged openstack/nova master: Fix typos in Host aggregates documentation https://review.openstack.org/566591
18:17:30 openstackgerrit Dan Smith proposed openstack/nova master: WIP: Use oslo.messaging per-call heartbeating https://review.openstack.org/566696
18:53:51 openstackgerrit Jay Pipes proposed openstack/nova master: add consumer.generation/create missing consumers https://review.openstack.org/557958
18:53:52 openstackgerrit Jay Pipes proposed openstack/nova master: placement: add Project, User and Consumer objects https://review.openstack.org/565403
18:53:53 openstackgerrit Jay Pipes proposed openstack/nova master: Add create() methods to Project, User and Consumer https://review.openstack.org/565404
18:53:54 openstackgerrit Jay Pipes proposed openstack/nova master: move consumer ensure to API layer https://review.openstack.org/565405
18:53:55 openstackgerrit Jay Pipes proposed openstack/nova master: remove Allocation.project_id & Allocation.user_id https://review.openstack.org/565406
18:53:56 openstackgerrit Jay Pipes proposed openstack/nova master: rework allocation handler _allocations_dict() https://review.openstack.org/565407
18:53:57 openstackgerrit Jay Pipes proposed openstack/nova master: Add a microversion for consumer generation support https://review.openstack.org/565604
19:21:57 openstackgerrit Brianna Poulos proposed openstack/nova master: Add notification support for trusted_certs https://review.openstack.org/563269
19:30:16 openstackgerrit Brianna Poulos proposed openstack/nova master: Add notification support for trusted_certs https://review.openstack.org/563269
19:33:16 mriedem melwitt: we should probably cut a stable release for pike and queens for these fixes https://review.openstack.org/#/q/If9e1581db9c1ae14340b787d03c815d243d5a50c - i know we just did one, but apparently this regression bit a few people based on the duplicate bugs we've seen in triage; do you want to wait for a stable review push to flush the queue or just do the releases now?
19:34:12 mriedem actually i know of 3 other regression fixes that should probably also get in
19:34:47 mriedem 2 of those are queens and 1 goes back to ocata
19:55:56 openstackgerrit MultipleCrashes proposed openstack/nova master: NCP-2367-deallocate-network-retry https://review.openstack.org/563418
19:59:08 openstackgerrit Merged openstack/nova master: Replace Chinese punctuation with English punctuation https://review.openstack.org/565655
19:59:15 openstackgerrit Merged openstack/nova master: doc: add note about xenapi aggregate upcall being resolved https://review.openstack.org/563724
20:04:29 openstackgerrit Artom Lifshitz proposed openstack/nova master: Add InstanceNUMATopology to LibvirtLiveMigrateData https://review.openstack.org/566398
20:04:30 openstackgerrit Artom Lifshitz proposed openstack/nova master: WIP: Service version check for NUMA live migration https://review.openstack.org/566723
20:09:46 openstackgerrit Merged openstack/nova master: [doc]remove nova-cert leftover in doc https://review.openstack.org/566013
20:18:56 mriedem efried: that NCP retry patch seems totally random
20:18:58 mriedem and misplaced
20:20:19 openstackgerrit Matt Riedemann proposed openstack/nova master: Add nova-manage placement heal_allocations CLI https://review.openstack.org/565886
20:20:20 mriedem jaypipes: ^ is now ready for review / building on when you want, i think i've got all the test coverage and stuff i can think of
20:24:14 jaypipes mriedem: cool, will review later this evening.
20:29:31 openstackgerrit Brianna Poulos proposed openstack/nova master: Add notification support for trusted_certs https://review.openstack.org/563269
21:13:39 melwitt mriedem: I guess we could just focus on the regressions for the stable releases -- don't have a strong opinion about it. what do you think?
21:15:48 openstackgerrit Jay Pipes proposed openstack/nova master: remove Allocation.project_id & Allocation.user_id https://review.openstack.org/565406
21:15:49 openstackgerrit Jay Pipes proposed openstack/nova master: rework allocation handler _allocations_dict() https://review.openstack.org/565407
21:15:50 openstackgerrit Jay Pipes proposed openstack/nova master: Add a microversion for consumer generation support https://review.openstack.org/565604
21:49:39 openstackgerrit Jay Pipes proposed openstack/nova master: add consumer.generation/create missing consumers https://review.openstack.org/557958
21:49:40 openstackgerrit Jay Pipes proposed openstack/nova master: placement: add Project, User and Consumer objects https://review.openstack.org/565403
21:49:41 openstackgerrit Jay Pipes proposed openstack/nova master: Add create() methods to Project, User and Consumer https://review.openstack.org/565404
21:49:42 openstackgerrit Jay Pipes proposed openstack/nova master: move consumer ensure to API layer https://review.openstack.org/565405
21:49:43 openstackgerrit Jay Pipes proposed openstack/nova master: remove Allocation.project_id & Allocation.user_id https://review.openstack.org/565406
21:49:44 openstackgerrit Jay Pipes proposed openstack/nova master: rework allocation handler _allocations_dict() https://review.openstack.org/565407
21:49:45 openstackgerrit Jay Pipes proposed openstack/nova master: Add a microversion for consumer generation support https://review.openstack.org/565604
21:55:30 openstackgerrit Jay Pipes proposed openstack/nova master: move consumer ensure to API layer https://review.openstack.org/565405
21:55:31 openstackgerrit Jay Pipes proposed openstack/nova master: rework allocation handler _allocations_dict() https://review.openstack.org/565407
21:55:32 openstackgerrit Jay Pipes proposed openstack/nova master: Add a microversion for consumer generation support https://review.openstack.org/565604
21:56:56 openstackgerrit Jay Pipes proposed openstack/nova master: add consumer.generation/create missing consumers https://review.openstack.org/557958
21:56:57 openstackgerrit Jay Pipes proposed openstack/nova master: placement: add Project, User and Consumer objects https://review.openstack.org/565403
21:56:58 openstackgerrit Jay Pipes proposed openstack/nova master: Add create() methods to Project, User and Consumer https://review.openstack.org/565404
21:56:59 openstackgerrit Jay Pipes proposed openstack/nova master: move consumer ensure to API layer https://review.openstack.org/565405
21:57:00 openstackgerrit Jay Pipes proposed openstack/nova master: rework allocation handler _allocations_dict() https://review.openstack.org/565407
21:57:01 openstackgerrit Jay Pipes proposed openstack/nova master: Add a microversion for consumer generation support https://review.openstack.org/565604
22:23:37 openstackgerrit Merged openstack/nova stable/pike: Handle @safe_connect returns None side effect in _ensure_resource_provider https://review.openstack.org/566164
22:40:09 openstackgerrit melanie witt proposed openstack/nova master: Migrate tempest-dsvm-multinode-live-migration job in-tree https://review.openstack.org/555945
23:35:40 arvindn05 quick testing question, how can we mock 2 methods on the same instance object with mock.patch.object?
23:36:05 arvindn05 return_value=fake_rp_tree)
23:36:05 arvindn05 'get_provider_tree_and_ensure_root',
23:36:05 arvindn05 mock.patch.object(self.conductor_manager.report_client,
23:36:05 arvindn05 return_value=fake_instance_alloc),
23:36:05 arvindn05 'get_allocations_for_consumer',
23:36:17 arvindn05 should the above work?
23:36:36 melwitt that should work

Earlier   Later