Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-11-08
14:31:49 efried Looks like this: AllocationRequestResource(amount=1,resource_class='VCPU',resource_provider=ResourceProvider(25d6392d-73e3-40a8-a415-f25a4e25ef00))
14:32:04 efried If we were hitting that method, I would have expected ResourceProvider(uuid=....)
14:32:34 jaypipes efried: how are you printing things?
14:32:49 efried for ar in p_alts.allocation_requests:
14:32:50 efried rp_obj.LOG.error("AR: %s", ar)
14:32:50 efried for rr in ar.resource_requests:
14:32:50 efried rp_obj.LOG.error("RR: %s", rr)
14:33:32 jaypipes efried: try "%r"
14:33:37 efried ...
14:34:18 efried jaypipes Same
14:34:25 jaypipes hmm
14:35:02 jaypipes efried: gimme a sec
14:36:29 jaypipes efried: and if you print out the resource provider object itself, what shows up?
14:37:48 efried jaypipes That comes out good: ResourceProvider(generation=2,id=23,name='twelve',uuid=1c6fc5a8-4d2c-4b50-ac9b-5c003a0b4c35)
14:38:30 jaypipes efried: weird...
14:38:35 efried agreed
14:39:21 openstackgerrit Tyler Blakeslee proposed openstack/nova master: TMB TEST https://review.openstack.org/518547
14:42:14 jaypipes efried: https://github.com/openstack/oslo.versionedobjects/blob/master/oslo_versionedobjects/fields.py#L814-L824
14:42:32 jaypipes efried: apparently, this is by design.
14:43:12 efried jaypipes Cool, that's fine, I can just stuff it in there for testing purposes. Thanks.
14:43:23 jaypipes np
15:04:08 openstackgerrit Surya Seetharaman proposed openstack/nova master: nova-manage update_cell allows more than once cell to have the same db/transport urls https://review.openstack.org/518556
15:05:53 efried jaypipes I couldn't even get that to work. Whenever I try to get the 'name' it complains about not being able to load it on the base object. Something about it being funneled through an ObjectField makes all those other fields blank out.
15:06:26 efried jaypipes I wound up writing a cache of uuid:name when I create the RPs. It's not a permanent thing - just to debug the test case.
15:06:35 jaypipes ok
15:06:43 efried jaypipes Though debugging in the future is going to be a real bear without *something* like this.
15:07:08 jaypipes efried: ack. note that the ProviderTree stores that name -> uuid and reverse mapping.
15:07:25 jaypipes efried: but of course, that's used on the compute host, not the placement service side.
15:07:32 efried Mm.
15:28:30 openstackgerrit OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/511035
16:03:05 efried jaypipes I'm uncovering all sorts of interesting bugs.
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

Earlier   Later