Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-11-08
16:03:14 jaypipes efried: good :)
16:03:36 efried jaypipes At the moment, I'm at the top of your stack: https://review.openstack.org/#/c/517119/6
16:03:55 efried I put together a test case with 14 different aggregates
16:04:17 efried jaypipes Did a request with no traits, and I'm getting candidates that cross aggregates. Which is bad.
16:05:51 efried jaypipes Does one write a bug against a patch in progress? Or better for me to put this test up with its actual expected assertions, which will fail and block the patch until they're fixed?
16:06:00 jaypipes the latter
16:06:43 efried ack. Though I suspect some of this behavior may be applicable to current master. I can check that out.
16:09:08 efried jaypipes If you could look at the test refactor patches and we can get those in, I can start rebasing this series accordingly while I'm doing this.
16:09:19 efried please and thank you :)
16:09:43 efried (gibi did a first pass)
16:13:44 jaypipes efried: l
16:13:48 jaypipes efried: k
16:13:54 jaypipes ugh, typing sucks today
16:14:06 efried jaypipes Still getting used to that Mac keyboard?
16:14:14 jaypipes efried: lol, no.
16:14:24 jaypipes efried: I don't use the Mac keyboard at all unless I'm traveling.
16:15:01 jaypipes efried: I use Synergy to share my regular Deck Francium keyboard across all my puters while at home.
16:15:39 jaypipes efried: just wrapped up a new article: https://github.com/jaypipes/articles/blob/master/openstack/walkthrough-launch-instance-request.md
16:17:23 efried jaypipes looking...
16:19:56 efried jaypipes How does a guy mark up one of these things?
16:20:07 jaypipes efried: pull request :)
16:20:37 efried jaypipes I don't know how to do that. I'll just take notes on the side. If it gets extensive, I'll look into this "pull request" thing.
16:21:38 jaypipes efried: ok dokey :)
16:30:37 openstackgerrit Marco Chiappero proposed openstack/nova master: Add a communication channel for power management support https://review.openstack.org/512815
16:57:28 openstackgerrit Marco Chiappero proposed openstack/nova-specs master: Enable VM communication channel for power management https://review.openstack.org/511453
18:33:35 openstackgerrit Mike Lowe proposed openstack/nova master: live-mig: keep disk device address same https://review.openstack.org/518022
20:48:51 openstackgerrit Mike Lowe proposed openstack/nova master: live-mig: keep disk device address same https://review.openstack.org/518022
23:59:17 openstackgerrit Eric Fried proposed openstack/nova master: Test allocation candidates: multiple aggregates https://review.openstack.org/518633
23:59:19 efried jaypipes Check this puppy out ^ (See https://bugs.launchpad.net/nova/+bug/1731072)
23:59:20 openstack Launchpad bug 1731072 in OpenStack Compute (nova) "AllocationCandidates.get_by_filters returns garbage with multiple aggregates" [Undecided,New]
#openstack-nova - 2017-11-09
00:18:32 openstackgerrit OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/511035
01:31:37 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Remove incorrect comment about instance.locked https://review.openstack.org/515306
01:37:30 openstackgerrit Zhenyu Zheng proposed openstack/nova master: block_device_mapping_v2.bus_type is missing from api-ref https://review.openstack.org/499488
01:39:31 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Add tags fields for server list detail/show api-ref https://review.openstack.org/465842
01:43:40 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Clean up exception caught in _validate_and_build_base_options https://review.openstack.org/463253
02:00:51 openstackgerrit Zhenyu Zheng proposed openstack/nova master: nova-manage db archive_deleted_rows is not multi-cell aware https://review.openstack.org/507486
03:29:56 openstackgerrit Yikun Jiang proposed openstack/nova master: Implement query param schema for migration index https://review.openstack.org/518644
07:10:21 openstackgerrit Yikun Jiang proposed openstack/nova master: Implement query param schema for migration index https://review.openstack.org/518644
07:27:56 openstackgerrit Lajos Katona proposed openstack/nova master: Change live_migrate tests to use fakedriver https://review.openstack.org/505202
08:02:56 Tengu hello there!
08:03:43 openstackgerrit Marco Chiappero proposed openstack/nova master: Add a communication channel for power management support https://review.openstack.org/512815
08:04:50 Tengu small question: what can prevent a nova instance to be resized with a "no valid host found" while we have plenty of resources, and another instance CAN actually be resized (same original flavor, same destination flavor)?
08:06:30 Tengu it looks like some weird flag on the instance that prevents the resize. We had some issues with the infra yesterday that prevented the resize (i.e. it crashed due to 1730949) that might have trigger some safe-guard?
08:09:46 openstackgerrit Yikun Jiang proposed openstack/nova master: Implement query param schema for migration index https://review.openstack.org/518644
09:54:18 openstackgerrit Merged openstack/os-vif master: Updated from global requirements https://review.openstack.org/511035
10:14:40 openstackgerrit Forest Romain proposed openstack/nova master: Fix list servers by hypervisor https://review.openstack.org/518699
10:20:19 openstackgerrit Yikun Jiang proposed openstack/nova master: Implement query param schema for migration index https://review.openstack.org/518644
11:48:02 openstackgerrit Merged openstack/nova master: Don't overwrite binding-profile https://review.openstack.org/505613
12:24:29 openstackgerrit Yikun Jiang proposed openstack/nova master: Add pagination and Changes-since filter support for os-migrations. https://review.openstack.org/330406
13:23:00 efried jaypipes Is creating a pull request as simple as punching the little pencil icon in github?
13:24:28 openstackgerrit Balazs Gibizer proposed openstack/nova stable/ocata: WIP: Functional test for regression bug #1713783 https://review.openstack.org/505160
13:24:29 openstack bug 1713783 in OpenStack Compute (nova) ocata "After failed evacuation the recovered source compute tries to delete the instance" [High,Triaged] https://launchpad.net/bugs/1713783
13:26:56 openstackgerrit Balazs Gibizer proposed openstack/nova stable/ocata: Functional test for regression bug #1713783 https://review.openstack.org/505160
13:26:57 openstack bug 1713783 in OpenStack Compute (nova) ocata "After failed evacuation the recovered source compute tries to delete the instance" [High,Triaged] https://launchpad.net/bugs/1713783
13:42:40 openstackgerrit Balazs Gibizer proposed openstack/nova stable/ocata: Set error state after failed evacuation https://review.openstack.org/518733
13:43:43 jianghuaw jaypipes, are you around?
14:00:46 jianghuaw jaypipes, do you have time to have a quick look at the comment at here:https://review.openstack.org/#/c/518294/1/nova/compute/resource_tracker.py@795
14:01:33 jianghuaw basically I think we can remove _resource_change() which is to check if resources changed in compute_node.
14:02:28 openstackgerrit Marco Chiappero proposed openstack/nova master: Add a communication channel for power management support https://review.openstack.org/512815
14:06:50 jianghuaw jaypipes, I guess we defer the checking to self._provider_tree.has_inventory_changed: https://github.com/openstack/nova/blob/master/nova/scheduler/client/report.py#L602
14:07:24 jianghuaw If my understand is correct, I think we can totally resource_tracker.py@remove _resource_change()
14:08:06 jianghuaw totally resource_tracker.py@remove _resource_change()
14:09:12 jianghuaw sigh! => totally remove resource_tracker.py@_resource_change()
14:33:49 openstackgerrit Balazs Gibizer proposed openstack/nova master: Functional test: cold migrate to compute down https://review.openstack.org/496280
14:34:26 openstackgerrit Balazs Gibizer proposed openstack/nova master: Functional test: evacuate with no compute https://review.openstack.org/498482
14:34:35 efried jaypipes https://github.com/jaypipes/articles/pull/17
14:36:28 jaypipes jianghuaw: I'm here now.
14:38:18 jianghuaw jaypipes, do you think we can remove resource_tracker.py@_resource_change()?
14:38:22 jianghuaw and defer the checking to self._provider_tree.has_inventory_changed: https://github.com/openstack/nova/blob/master/nova/scheduler/client/report.py#L602
14:38:51 jaypipes jianghuaw: we should be able to do that, yes.
14:39:38 jianghuaw jaypipes, cool. I will rework this patch https://review.openstack.org/#/c/518294 to cover this part.
14:40:09 jaypipes jianghuaw: sounds good.
14:42:37 jianghuaw jaypipes, is nested resource provider supported by allocation_candidate?
14:42:41 jianghuaw I guess not.
14:43:14 jaypipes jianghuaw: no, not yet. I am trying my best to get this series merged before having n-r-p do allocation candidates: https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:refactor-alloc-cands-get-by-filters
14:43:17 jianghuaw But I'm not if we already have in-progress patches for that.
14:43:29 jaypipes jianghuaw: I would hold off a week
14:43:49 jaypipes jianghuaw: besides reviews from alex_xu, haven't been able to get many reviews on the series this week due to the summit.
14:44:01 jaypipes alex_xu: ty, btw for your reviews and your patience with me :)
14:44:26 jianghuaw jaypipes, understood. I will have a look the series of patches also. thanks.
14:44:34 jaypipes jianghuaw: ty sir!
14:45:49 jianghuaw :-)
14:49:52 openstackgerrit David Wahlstrom proposed openstack/nova stable/pike: Fix changes-since search filter https://review.openstack.org/518740
15:00:22 efried jaypipes Would like to discuss a couple of design points with you when you have a sec.
15:00:31 jaypipes efried: shoot.
15:00:56 jaypipes efried: hangout?
15:01:11 efried jaypipes Could do.
15:01:36 efried jaypipes You'll have to set it up, though.
15:01:42 jaypipes efried: np, one sec.
15:02:34 jaypipes efried: https://hangouts.google.com/call/NveOHfR-Vd1PJXV8V2SEAAEE
15:02:48 efried jaypipes http://paste.openstack.org/show/625946/
15:09:58 efried jaypipes Here's what that guy is *currently* returning: http://paste.openstack.org/show/625947/
15:26:02 efried jaypipes https://review.openstack.org/#/c/518123/ <= bottom of test refactor & adds
15:46:52 snasir Hi is there a way to delete a nova hypervisor manually ? I deleted a node but something went wrong and the hypervisor did not get removed... Can anybody assist . Thanks
17:07:26 efried jaypipes About to push that test case into https://review.openstack.org/#/c/517119/6 -- you got local revs on that series?
17:07:54 jaypipes efried: nope, go for it.
17:08:10 openstackgerrit Eric Fried proposed openstack/nova master: handle traits with sharing providers https://review.openstack.org/517119
17:08:13 efried jaypipes hecho
17:08:55 jaypipes efried: -1 on the bottom patch of your test series...
17:11:21 efried jaypipes Responded - I didn't change it.

Earlier   Later